Gulp support

Gulp is proving increasingly popular as a task runner, but right now docular is (seemingly) tied to grunt.  It would be great to be able to use docular with gulp, typically gulp doesn't use plugins, prefering to use library APIs directly.  I'm not sure if docular is already in a good shape for this sort of usage and it's API (as opposed to grunt-docular) just needs documented, or if work would need to be done to decouple docular from grunt?

we should make a gulp-docular plugin
in the midtime, I tried a quick and dirty hack similar to the following in my gulpfile:

var docular = require('docular');
var path = require('path');
var glob = require("glob");
var _ = require('lodash');

gulp.task('docs', function() {

  var opt = {
      useHtml5Mode: false, //Use angular's html5 mode? true/false.
      docular_webapp_target: './docs', //The place where the docs will be generated
      showAngularDocs: false,
      showDocularDocs: false,
      examples: {}, //instructions for how to run the sandboxed examples
      groups: [
                 {
                   groupTitle: 'myproject',
                   groupId: 'myproject',
                   groupIcon: 'icon-book',
                   files: ["gettingstarted.doc"]
     	           },
                 {
                     groupTitle: 'Website',
                     groupId: 'web',
                     groupIcon: 'book',
                     sections: [
                         {
                             id: "website",
                             title:"IHM",
                             scripts: './view/*.js'
                         }
                     ]
                 },
                 {
                     groupTitle: 'Server API',
                     groupId: 'serverapi',
                     groupIcon: 'beer',
                     sections: [
                         {
                             id: "serverapi",
                             title: "API",
                             scripts: ['./server/actions/*.js', './server/initializers/*.js', './server/tasks/*.js']
                         }
                     ]
                 }
             ],
  }

 //grunt.file.expand replacer
  for (var g in opt.groups) {
    for (var s in opt.groups[g].sections) {
      if (typeof opt.groups[g].sections[s].scripts === 'string') {
        opt.groups[g].sections[s].scripts = glob.sync(opt.groups[g].sections[s].scripts, {})
      } else {
        var scriptArr = [];
        for (var ss in opt.groups[g].sections[s].scripts) {
          scriptArr.push(glob.sync(opt.groups[g].sections[s].scripts[ss], {}))
        }
        opt.groups[g].sections[s].scripts = _.flatten(scriptArr)
      }
    }
  }

  docular.genDocs(opt);
	return true;
    });

when I do gulp docs, it outputs:
[10:23:20] Starting 'docs'...
[10:23:20] Finished 'docs' after 24 ms
Loading individual files for gid: Base
Loading individual files for gid: myproject
Loading individual files for gid: website
Loading individual files for gid: serverapi
Parsing files for gid:  Base
Parsing files for gid:  myproject
Parsing files for gid:  website
Parsing files for gid:  serverapi
Backfilling file data for gid:  Base
Backfilling file data for gid:  myproject
Backfilling file data for gid:  website
Backfilling file data for gid:  serverapi
Creating docs from for gid:  Base
Creating docs from for gid:  tennis-paris
Creating docs from for gid:  website
Creating docs from for gid:  serverapi
Dependencies saved

I then go to my output dir and start a python simple server:
cd docs
python -m SimpleHTTPServer 8001

the website has the groups myproject, Website and Server API. but they are empty :/
I didn't add correct comments with right syntax for the JS files so it may be the cause but my gettingstarted.doc should appear...I guess!?

I think there's a problem with latest version of docular. For the project I work on I finally decided to go with another documentation generator. I'll be happy to switch back to docular for another project

@Zougi which document generator did you use? So far I'm using esdoc, but it has limitation in that it does not support multiple source folders.

At the time I went with doxx and gulp-doxx. It's not top quality compared to docular but it does the job.
For my latest project I needed more than just generated documentation. So I used instead mkdocs + documentationjs. I have a folder wiki with a bunch of markdown files in it and a gulp task that copies readme.md into that folder, generates jsdoc to markdown format with documentationjs and launch a server with mkdocs serve.

Is this the new name for doxx? https://github.com/mr-doc/mr-doc

Yes you're right doxx is now mr-doc

Since this isn't directly related to docular, closing.


M> Translate README.md to Greek

Measure of Completion: 100%

[x] Create the file README_GR.md
[x] Translate the content: 100%

[x] @ICA3DaR5: 40%

[x] @zero-andreou: 30%

[x] @thiefinshadows: 30%

[x] Review each other's work
[x] Check the spelling
[x] Have it reviewed by others with knowledge of English and Greek

Suggested Budget: $100

I think this has been completed.


Converted font won't compile as-is

We use GitHub issues for development related discussions.
Please use the forum to ask questions.
Describe the issue
I used the LittlevGL online font converter tool to convert a font.
I downloaded the resulting C file and added it to my project incorporating the lv_port_esp32_ili9341.
The font source isn't fully compatible with this version of LittlevGL and had errors I had to fix.
It is not actually clear to me whether this is an issue in the font converter being too new, or the code in this port being different, not sure.  The errors are minor, here is the diff
rbarris@oryx-i7:~/sandbox/ray/esp1$ diff ~/Downloads/build.c ~/sandbox/ray/esp1/main/build.c
1377c1377
<     .class_pair_values   = kern_class_values,

.class_pair_values   = (const uint8_t *)kern_class_values,

1412c1412
<     .subpx = LV_FONT_SUBPX_NONE,

//    .subpx = LV_FONT_SUBPX_NONE,

Code to reproduce the issue
font edits.zip
Expected Results
A converted font should integrate smoothly with LVGL
Actual Results
minor compile errors
ESP32 Chip version
m5stack gray
ESP-IDF version
ESP-IDF v4.1-dev-1086-g93a8603c5-dirty
Development machine OS
Ubuntu 17.10

This is because the font converter has been updated to support LittlevGL 6.1, but there are a few minor incompatibilities with 6.0.x now. There are two possible solutions:

Fix the minor incompatibility the way you did, and continue to use LittlevGL 6.0.
Upgrade to LittlevGL 6.1; it's a minor release so you shouldn't need to do much work to upgrade.

As the question is answered I close this issue.


Generate package.json in dist based on package.json in the root

To make this compatible with tools like semantic-release, The package.json under dist folder needs to be generated from package.json as part of the build process.
This way plugins like  can be used seemlessly with this seed.
If the dist/package.json is generated from the root, we can write a new script publish like this
{
  "scripts": {
    "clean": "npm prune && rimraf .tmp && rimraf dist",
    "transpile": "ngc",
    "package": "rollup -c",
    "minify": "./node_modules/uglify-js/bin/uglifyjs dist/bundles/ng2-loading-bar.umd.js --screw-ie8 --compress --mangle --comments --output dist/bundles/ng2-loading-bar.umd.min.js",
    "copy": "cpx './package-dist.json' dist && renamer --find 'package-dist.json' --replace 'package.json' ./dist/*",
    "build": "npm run clean && npm run transpile && npm run package && npm run minify && npm run copy",
    "publish": "semantic-release pre && npm build && npm publish dist && semantic-release post"
  }
}

The publish script will

Generates the version based on commits & updates root package.json in the bulid machine(travis)
Generate umd package
Publish the package to npm
Pushes the above generated package.json to github
Generate releases & tags with messages combined from commit history
Push these tags to github

That's a nice suggestion, thanks. I didn't know about the sematic-release plugin!
If I understand you correctly, the "copy" script and the package-dist.json would be unnecessary because semantic-release would take care of that, correct?
If that's the case, when testing the module locally using npm link, the whole code should be built but not published. Wouldn't that break the npm link since the dist folder wouldn't have its own package.json?
Thanks again for your contribution!

I wrote a script that does this..
I'll submit a pull request for it later today

People are using some hack to test their local changes using npm link. I'm yet explore that.

Please have a look at https://github.com/1tontech/ngx-http-interceptor/tree/angular2 for a project in which I'm using semantic release & this seed

I've a plugin that I've modified to use this & semantic-release plugin. Its publishing the versions properly with travis integration.

I've taken a look at it and I understand you're basically using a Node.js app to save yourself from changing the package.json manually... Not bad, but I don't see it that better as to implement it straight away. There might be a better way...

In the demo project that tests my angular plugin, I use yarn add file://dist to link the demo to the folder containing the development version

Don't think Yarn is that mainstream yet, so I'm not keen on the idea of "forcing" anyone to use it.

I'll submit a pull request for it later today

Feel free, I'll be happy to give it a look.

I've taken a look at it and I understand you're basically using a Node.js app to save yourself from changing the package.json manually... Not bad, but I don't see it that better as to implement it straight away. There might be a better way...

Would you mind sharing your thoughts on that

Don't think Yarn is that mainstream yet, so I'm not keen on the idea of "forcing" anyone to use it.

I'm not a JS developer myself. I work on server side mostly in the past. Since I'm new to yarn, I started using it for the goodies it offers vs npm.
Otherwise, your regular flow with npm should not be affected irrespective of how package.json under dist is generated

Feel free, I'll be happy to give it a look.

Looks like its better for me to submit one if we conclude on the approach.
Please let me know your thoughts

@thekalinga I've done a bit of research and I think I'll keep the whole thing manual by now. Digging a bit deeper, I feel than the npm update command is good enough for updating the distributed package.json. Thanks anyway for your valuable suggestions!


OAuth 2.0  ReferenceError: getExternalCollectionSecretsForActiveEnvironment is not defined

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
When I try to generate an access token using client credentials from the collection auth tab, I get the following error:
Uncaught (in promise) ReferenceError: getExternalCollectionSecretsForActiveEnvironment is not defined
    at _app-5ddd16132c496b1d.js:1:854329
    at new Promise (<anonymous>)
    at _app-5ddd16132c496b1d.js:1:854200
    at _app-5ddd16132c496b1d.js:1:10400
    at 626-4d2102350b3d9e2d.js:1:210544
    at s (main-ad5649da2d8f61d3.js:1:87145)
    at Generator._invoke (main-ad5649da2d8f61d3.js:1:86933)
    at P.forEach.e.<computed> [as next] (main-ad5649da2d8f61d3.js:1:87568)
    at n (_app-5ddd16132c496b1d.js:1:1536412)
    at s (_app-5ddd16132c496b1d.js:1:1536615)

It has been broken since at least v1.20.4.
.bru file to reproduce the bug
No response
Screenshots/Live demo link

Hey @jfpedroza, are you using the latest version of Bruno? The issue you reported has already been fixed and released in version v1.20.0. Please try the latest version v1.22.0.

I've noticed this as well, cloned the repo and I get this error as well

Hey @jfpedroza, we weren't able to reproduce the issue. Can you please confirm that you are on version v1.22.0 of Bruno once again?

@sanjai0py this is specific to FOSS build. The referenced methods are nowhere to be found in this repository.

@sanjai0py Yes, it's 1.22.0.

The issue has been fixed. I am Closing this.


Post your custom clients here

If anybody is interested in posting some custom clients, post them here. I'm curious to see them.
Here's my client: agarian.github.io.
Source code can be found here.

http://agarahg.tk
ok shameful advertisement

It appears that I can't connect to servers with that client.

@AlexHGaming Do you happen to know of any custom skin scripts? Or clients that have premium skins (besides yours)?

@Agarian http://josefredohd.byethost7.com
that is my friend client

I thought I'd mention that the custom skins script no longer works.

What about github links? What clients exclude CigarProject/Cigar do you know?

Dude honestly, I've been looking to implement premium skins into my own client, and I need one that already has them implemented to work off of.

http://atomical.byethost6.com/?i=1
https://trapkillo.github.io/trapclientv2/out.html#ip=<IP_ADDRESS>:443
http://agariox.net/

http://agarix.xyz/

Http://Agar.tn

Here's mine

@Fan7asy Wow, great job)

Thank you @cyclopentan 🙂

cmgar.000webhostapp.com (i have a free hosting) :D

http://www.nagar.eu
gone quite downhill and unmaintained recently though.

Ah, everyone clients remind me of my game, when it first released on the 13th of October I installed a new Cigar server back in the days when the client and server were combined, Brings back such good memories!
I want to rebuild my game! :(

http://asat-agar.online

@xN3BULA that is athex.io game haha

@Wair1o17 xAz lets me use it

@Fan7asy yours is my favorite so far

https://github.com/Fan7asy/Client/blob/master/js/main.js#L713 @Agarian

Cool, now I can check that off my to-do list lol

http://gkclan.me

@Luka967 Aye

@Luka967 What Luka said.

Yeah I was about to say somebody should add a label lol

lol

I need good client with macros for download please can send me, skype : yt.rememberme.br

I may have one

Shit

Agarian send me please,skype or discord please or public here

@AmorimGames look at the very first comment in the issue

@Agarian i need download to change to my language(portuguese)

Forget it,i see you profile and download tnx,can send me free host?

Can send me free host?
Not too sure what you mean by that too be honest, but if you mean send you a host that offer's free service, I can't help you on that.

how you host you game?

Oh you caught my server, didn't you? Well I use Cloud9, but it isn't free.

hm ok tnx

Used to have a really good client that I had optimized alot for laggy computers and its was for my website but lack of traffic and no one playing led to me closing it.

Codeanywhere is a suggestion, but it is terrible in terms of ping. (192ms minimum, 240ms maximum)

I personally use Cloud9, but I don't host my server. As for CodeAnywhere, I haven't heard too many good things about available RAM.

http://ka3boura.tn/

http://q96208pi.beget.tech/

Anybody have minimap code that I can use? I'm thinking about putting one in my client.

I'm using Cigar client and so far I have not had any success with getting a minimap to work. By that I mean I want a minimap to show all players not just my position. I want one like agarplus or alis.

I at least want one that shows my position.

https://agar.yt/
Used to be quite active, but I do not maintain it anymore. I am just keeping it online.
Thank you Megabyte918 and all contributors for MultiOgar-Edited!

@Agarian see this Cigar client https://github.com/dinhduynhat/Agar-Ogar-Cigar-web-client
has minimap that shows your position.
I have a few more clients that people have sent me but I don't use them because they're ripped. I hate when people do that so I refuse to use them. I'm willing to give them to you if you want.

@cowpits Not only is that code obfuscated, but the mini map its self is broken.

what about Jara's client? it has a minimap https://github.com/JaraLowell/OgarClient

@Droppers I love your client! Do tell me, how would one implement a mini map such as yours?

@Droppers would you be kind enough to send me the source code?

www.nagar.eu/client2/

https://classidie.xyz/web/

@ArixProject Your website isn't working through HTTPS. But don't fall for the paid certificate scam, set it up for free.

http://agar.rs

@Niall7459 there is no way I can do that, I can send you a specific part of the code.

i have the best client ever..this is it : https://www.facebook.com/SAV.404/videos/1849118388660969/
-just a video cz i don't want to share it yet.

@SAV-404 i dont see ur video

@mafiakb log in facebook first.

Hello , how do I make my client never close, should I have my computer always on?

are you hosting on your computer? windows IIS? then yes keep it running.

Buy a raspberry pi like £35 I run mine 24/7 With Client and MultiOgar.
It actually handles 20 players easily. at 40 it started to struggle.
I've hd 57 players on there and you only notice that the gameplay is a bit slower.
But Once I've fixed my main server I will probably use that because of its perfomance.

My custom client coded from scratch
https://senpa.io/web

bruh


gemini issue

Thank you for the fix. The translated srt file from gemini has correct content in it now.
However, the first line in the translated srt file is always empty. Please check it. thank you!
Here is the example:
Original srt file:
1
00:00:05,650 --> 00:00:09,450
小時候請人算過命的人
2
00:00:09,450 --> 00:00:11,650
你在長大的過程之中
3
00:00:11,650 --> 00:00:13,370
你是不是也在印證著
Transltaed srt file:
1
00:00:05,650 --> 00:00:09,450
2
00:00:09,450 --> 00:00:11,650
During the process of growing up
3
00:00:11,650 --> 00:00:13,370
Are you also verifying that

sorry what is your command can you share it ?

python make_book.py --book_name D:\SRT\test.srt --gemini_key
AIzaSyAaGWhDJo02sbEUcbRnkO9cXysfadse --model gemini --language en --prompt
prompt_template_sample.txt --single_translate
However, the gemini key in this command has been changed by me, so you
can't use it directly. Please use your own gemini key.
On Sat, Jan 27, 2024 at 10:55 PM yihong @.***> wrote:

sorry what is your command can you share it ?
—
Reply to this email directly, view it on GitHub
https://github.com/yihong0618/bilingual_book_maker/issues/377#issuecomment-1913477805,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASEQGPGNQVL2UG2BL2HM6XLYQXY5PAVCNFSM6AAAAABCN3SN2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGQ3TOOBQGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

so you use single_translate..I will check that, can you use it without --single_translate?

Here is without --single_tanslate, the English translated part in line 1 is
still missing.
1
00:00:05,650 --> 00:00:09,450
小時候請人算過命的人
2
00:00:09,450 --> 00:00:11,650
你在長大的過程之中
During your growing up process
3
00:00:11,650 --> 00:00:13,370
你是不是也在印證著
Are you also verifying
On Sun, Jan 28, 2024 at 1:03 AM yihong @.***> wrote:

so you use single_translate..I will check that, can you use it without
--single_translate?
—
Reply to this email directly, view it on GitHub
https://github.com/yihong0618/bilingual_book_maker/issues/377#issuecomment-1913522963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASEQGPAXXXYBZ27O7UU36C3YQYH6RAVCNFSM6AAAAABCN3SN2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGUZDEOJWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
I live in the USA, and it is too late for me. I have to sleep now.
On Sun, Jan 28, 2024 at 1:18 AM LIKANG KAO @.***> wrote:

Here is without --single_tanslate, the English translated part in line 1
is still missing.
1
00:00:05,650 --> 00:00:09,450
小時候請人算過命的人
2
00:00:09,450 --> 00:00:11,650
你在長大的過程之中
During your growing up process
3
00:00:11,650 --> 00:00:13,370
你是不是也在印證著
Are you also verifying
On Sun, Jan 28, 2024 at 1:03 AM yihong @.***> wrote:

so you use single_translate..I will check that, can you use it without
--single_translate?
—
Reply to this email directly, view it on GitHub
https://github.com/yihong0618/bilingual_book_maker/issues/377#issuecomment-1913522963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASEQGPAXXXYBZ27O7UU36C3YQYH6RAVCNFSM6AAAAABCN3SN2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGUZDEOJWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Here is my testing srt file. You can use it to test.
On Sun, Jan 28, 2024 at 1:35 AM LIKANG KAO @.***> wrote:

Hi,
I live in the USA, and it is too late for me. I have to sleep now.
On Sun, Jan 28, 2024 at 1:18 AM LIKANG KAO @.***> wrote:

Here is without --single_tanslate, the English translated part in line 1
is still missing.
1
00:00:05,650 --> 00:00:09,450
小時候請人算過命的人
2
00:00:09,450 --> 00:00:11,650
你在長大的過程之中
During your growing up process
3
00:00:11,650 --> 00:00:13,370
你是不是也在印證著
Are you also verifying
On Sun, Jan 28, 2024 at 1:03 AM yihong @.***> wrote:

so you use single_translate..I will check that, can you use it without
--single_translate?
—
Reply to this email directly, view it on GitHub
https://github.com/yihong0618/bilingual_book_maker/issues/377#issuecomment-1913522963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASEQGPAXXXYBZ27O7UU36C3YQYH6RAVCNFSM6AAAAABCN3SN2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGUZDEOJWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

ok will take a look thanks


sheet_to_json: date value is printed the day before

Hi Team,
Recently I am working about reading data which in excel sheet has the value of Date also Value(decimal value).
I am using xlsx -v 0.14.1. My purpose is to get the raw data from the excel file (exact date and value which are filled excel )
Sample Data:
Date                |        Value
12/31/2016     |      -1234567890.12345
12/31/2017     |<PHONE_NUMBER>.12345
After I uploaded the excel file, I got:
2016-12-30    -1,234,567,890.12345
2017-12-30     1,234,567,890.12345
I find out that the date value is printed the day before. I have checked the value before converting to Json, the date value is Fri Dec 30 2016 23:59:56 GMT+0700 (Indochina Time) then once it is converted to Json the value is 2016-12-30
Is there any solution referring to this case?

This bug might be linked to #1212.
Can you try the fix suggested by @cdmahoney to see if your problem has the same root cause ?
I've created a pull request to fix the problem

This should have been fixed in v0.16.0, please update and report back if there are issues.

It is not fixed yet, using 0.16.9 version

it still have the issue at version : 0.18.7

Is this still not fixed? I am facing the same error.


am I doing something wrong?

What exactly am I doing wrong? I just want to download this song " https://www.youtube.com/watch?v=OWjudyW7RGE&t"
And what I did to try to run the script is, to create a notepad file, name it yt_chapters.py and run that through Python.
How the hell can I do it right? :(

what I did to try to run the script is, to create a notepad file, name it yt_chapters.py and run that through Python

Hopefully you copied the code into the file?
If you are not working from a git-clone, you should be copying the raw code and not the section with github's nice reading layout.
I trust you have installed the requirements necessary for the code to run as per the Readme
Do you have py files associated to run with Python? If not, like any application you will need to pass the full path, including the interpreter

what I did to try to run the script is, to create a notepad file, name it yt_chapters.py and run that through Python

Hopefully you copied the code into the file? If you are not working from a git-clone, you should be copying the raw code and not the section with github's nice reading layout.
I trust you have installed the requirements necessary for the code to run as per the Readme
Do you have py files associated to run with Python? If not, like any application you will need to pass the full path, including the interpreter

I copied the code into a notepad, named it yt_chapters.py and double clicked, then I did the command for download and an error came across...Can u explain me how I do it correctly?

When you save a file in notepad, it appends ".txt" onto the filename. The filename you saved it as isn't "yt_chapters.py", it's "yt_chapters.py.txt" and that is why the system cannot find a file named "yt_chapters.py".

n you save a file in notepad, it appends ".txt" onto the filename. The filename you saved it as isn't "yt_chapters.py", it's "yt_chapters.py.txt" and that is why the system cannot find a file named "yt_chapters.py".

Huh? How then???? How do I save it as .py and not txt.py.

In Windows Explorer, under View menu, make sure the "File name extensions" checkbox is ticked.
Then you can rename "yt_chapters.py.txt" to "yt_chapters.py".


Ability to swap model during session

Thanks so much for creating this tool.
I sometimes need to switch the model I'm using, usually when the context window gets too large.  For example, someone might switch to the more expensive 16k or 32k versions of GPT 3 or 4 after hitting a token limit.
A /model <modelname> command might be a good way to do this.

Thanks for trying aider!
Which models are you wanting to switch from and to?
In some cases, it would be very hard to change the model mid-conversation. Changing from gpt-4 to gpt-3.5 or vice-versa would also require changing the "edit format" and rewriting all of the conversation history. Changing between different versions of 3.5 or between different versions of 4 might be possible.
But in practice I'm not sure how useful this feature would be? gpt-3.5-16k is pretty inexpensive, so why not just use it always? And for gpt-4, very few people have access to gpt-4-32k, so almost no one would have the ability to increase the context window size mid-conversation.

Maybe I'm one of the lucky ones.  I normally work with gpt-4 but sometimes want to switch to gpt-4-32k.
I had a look at the code the other day and came to the same conclusion - doesn't seem too bad to swap different versions of the same model, but swapping between models is tricky.

I have been mulling on a more versatile /set feature which would let you switch out a subset of the params you can configure on startup and reboot the relevant parts of aider while keeping your current chat history and file context.
model, edit-format, map-tokens, pretty, no-pretty, stream, no-stream, show-diffs, auto-commits, verbose were the ones I had in mind.

(copying this comment from discord)
It seems like a nice to have feature.
The real work is looking at each cmd line switch and deciding which bin it falls into:

Trivial to dynamically change at runtime. Something like --verbose or --map-tokens.

Possible to dynamically change at runtime, with special extra work when it changes. Something like --model could be made dynamic, as long as you only switch between identical models with different context windows. Or if you always discard the conversation history when you switch model (but is this really dynamic then). Other switches could have similar special limitations or requirements.

Impossible to sensibly make dynamic, like --openai-api-base.

After we do that binning, is it still worth adding a /set command? Or are there too few useful switches that we're willing to do the work to support? Or are we making a big complex machine to support the more complex switches (like --model) which will slow down and complicate future innovative work?

This looks like a duplicate of #402, so I'm going to close it so discussion can happen there. Please let me know if you think it's actually a distinct issue.


fix: add variable IMG_VERSION to build time vars

What type of PR is this?
This PR adds IMG_VERSION variable which is missing from some build time variables in certain SKUs. This will make pinning the marketplace versions for official branches easier.

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.077%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2450

Relevant Lines:
2985

💛  - Coveralls


Z algorithm implementation

Implemented the Z algorithm, as requested in issue 54

Codecov Report

Merging #77 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #77   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          89     90    +1
  Lines        1929   1956   +27
  Branches      342    347    +5
=====================================
+ Hits         1929   1956   +27

Impacted Files
Coverage Δ

src/algorithms/string/z-algorithm/zAlgorithm.js
100% <100%> (ø)

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 d74234d...c70f380. Read the comment docs.

Nice! Thank you @hariv!


Webhook publisher secret validation passes only for the latest revision

Description:

In webhook API, the secret validation passes only for the secret in the latest revision regardless of which revision is deployed.
Steps to reproduce:

Create webhook API and deploy it.
Get the secret from the publisher Runtime, calculate the hmac value and invoke the Callback URL to send a notification.
Signature validation will pass.
Generate a new secret in the publisher and deploy the new revision.
Same as step 2, calculate the hmac value and invoke the Callback URL to send a notification.
Signature validation will again pass.
Now from the deployments tab, revert back to the initial deployment.
If you check the secret again, it will now show the initial secret which you got in Step 2.
But if you send the same request which you sent in Step 2, the signature validation will fail.
If you send the request which you used in Step 5, the signature validation will pass.
There is now a mismatch between the secret that is shown in the publisher and the secret that is used in the Runtime.

Affected Product Version:

APIM 4.0.0-alpha

My understanding is wrong on how revisioning and deployment work.
Both are two separate things.

Creating a new revision does not mean the new revision will get deployed.
Restoring a new revision does not mean the restored revision will get deployed.
Current deployed revision will be shown under API Gateways.
If you want to change the Deployment in the Gateway, then you need to undeploy the revision from the gateway and select the revision that you want to deploy.
You can check the configurations in each revision in read only mode by selecting the Current API dropdown on the upper right corner.
Restoring the revision makes that Revision equal to the Current API so that you make changes on top of that and create a new revision from it.


AzureCore ResponseDownloader

Work necessary to comply with https://azure.github.io/azure-sdk/general_azurecore.html.
There is currently a class "ContentDecodePolicy". However, it doesn't work correctly because of difficulties allowing the client SDK to pass in the necessary model class in a way that the pipeline will accept it for deserialization. As such, deserialization is occurring in the client SDK.
However, this may be something different, in which case I didn't see anything in the Python SDK that matched.

[ ] Implement ResponseDownloader policy according to guidelines
[ ] Add test coverage

This has been done for a while. Closing.


backend: Add /sitemap endpoint

This is now ready. Once it's merged, I will alias /sitemap.txt to the new endpoint, and add the following robots.txt directly to HTTP server config:
User-agent: *
Allow: /
Host: https://beta.flathub.org
Sitemap: https://beta.flathub.org/sitemap.txt

But we don't want beta to be indexed?

This also should handle other languages

You could parse https://github.com/flathub/website/blob/main/frontend/next-i18next.config.js I guess? Or just copy that (and keep it up ta date)

It already is, though… I will add robots.txt to disallow crawlers. We're going to need this endpoint eventually anyway.

Might be old?

There are some results at https://www.google.com/search?hl=en&q=site%3Abeta.flathub.org so I don't think it fully works.

Seems like we could remove these with the google webmaster tools https://www.voog.com/blog/how-to-remove-your-website-or-web-page-from-google

https://github.com/flathub/website/pull/387 is an extended version of this


Add support for Telegram Bot test environment

Bot API 6.0 brought us official test environment.
The test environment is completely separate from the main environment, so you will need to create a new user account and a new bot with @BotFather.
After receiving your bot token, you can send requests to the Bot API in this format:
https://api.telegram.org/bot<token>/test/METHOD_NAME

I think it might be the time to introduce TelegramBotClientOptions class and put bool Test property there.

Done in #1102


update service endpoint definitions to include recent updates to dwn spec

Related to https://github.com/decentralized-identity/decentralized-web-node/pull/228
messageAttestationKeys?: string[]
messageAuthorizationKeys?: string[]
nodes: string[]
recordEncryptionKeys?: string[]

https://identity.foundation/decentralized-web-node/spec/#service-endpoints


Issue registering Files as default file manager

After registering Files as the defualt file manager, I have encountered an issue a few times where the computer will think that the Windows Key is being held down. Any usage of the keys on the keyboard will register as Windows Key + E which will just result in opening multiple tabs of the Files application. Unregistering the application as the default file manager will resolve the issue.

Hello, this repo is for Files' website. Please open this issue again on the app's repo.


Remove hiqdev/composer-config-plugin

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues

Merged. Thanks!


Mark filter stale for Besu nodes

Not all Ethereum nodes return exactly the same error message when a filter does not exist. This caused ETHConnect to not be able to recreate a filter automatically in some cases, for example if a Besu node was restarted. This would result in log messages like:
[2023-10-19T06:50:07.739Z] ERROR es-1fb7a291-487f-404b-44ab-0a36a0da4dae: subscription error: Filter not found

This PR fixes this issue to make ETHConnect a bit more flexible to be able to recreate the filter automatically in this case.

Codecov Report

Merging #238 (828d35a) into main (111eb89) will decrease coverage by 0.04%.
Report is 4 commits behind head on main.
The diff coverage is n/a.

:exclamation: Current head 828d35a differs from pull request most recent head 64c391b. Consider uploading reports for the commit 64c391b to get more accurate results

@@            Coverage Diff             @@
##             main     #238      +/-   ##
==========================================
- Coverage   97.26%   97.22%   -0.04%
==========================================
  Files          59       59
  Lines        7521     7524       +3
==========================================
  Hits         7315     7315
- Misses        163      165       +2
- Partials       43       44       +1

see 2 files with indirect coverage changes


Unable to compile FreeRDP on Raspbery Pi 2 Model B 1GB on raspbian

I am running the following command and getting these errors while compiling FreeRDP on raspberry pi 2 model B 1GB running raspbain:
cmake -DTARGET_ARCH=ARM -DWITH_NEON=OFF .
What am I doing wrong here? I am fairly new to Linux.
-- The C compiler identification is GNU 4.6.3
-- The CXX compiler identification is GNU 4.6.3
-- Check for working C compiler: /usr/bin/gcc
-- Check for working C compiler: /usr/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.26")
-- FreeRDP GIT-dirty
-- Git Revision 5d3fa29
-- Performing Test Wno-unused-result
-- Performing Test Wno-unused-result - Success
-- Performing Test Wno-unused-but-set-variable
-- Performing Test Wno-unused-but-set-variable - Success
-- Performing Test Wno-deprecated-declarations
-- Performing Test Wno-deprecated-declarations - Success
-- Performing Test Wno-deprecated-declarationsCXX
-- Performing Test Wno-deprecated-declarationsCXX - Success
-- Performing Test Wimplicit-function-declaration
-- Performing Test Wimplicit-function-declaration - Success
-- Performing Test Wredundant-decls
-- Performing Test Wredundant-decls - Success
-- Performing Test Wno-builtin-macro-redefined
-- Performing Test Wno-builtin-macro-redefined - Success
-- Performing Test Wno-builtin-macro-redefinedCXX
-- Performing Test Wno-builtin-macro-redefinedCXX - Success
-- Looking for include file /usr/include/fcntl.h
-- Looking for include file /usr/include/fcntl.h - found
-- Looking for include file unistd.h
-- Looking for include file unistd.h - found
-- Looking for include file execinfo.h
-- Looking for include file execinfo.h - found
-- Looking for include file stdint.h
-- Looking for include file stdint.h - found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - found
-- Looking for include file sys/modem.h
-- Looking for include file sys/modem.h - not found.
-- Looking for include file sys/filio.h
-- Looking for include file sys/filio.h - not found.
-- Looking for include file sys/strtio.h
-- Looking for include file sys/strtio.h - not found.
-- Looking for include file sys/select.h
-- Looking for include file sys/select.h - found
-- Performing Test HAVE_TM_GMTOFF
-- Performing Test HAVE_TM_GMTOFF - Success
-- Looking for include file pthread.h
-- Looking for include file pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found.
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Looking for pthread_tryjoin_np in pthread
-- Looking for pthread_tryjoin_np in pthread - found
-- Looking for include file sys/eventfd.h
-- Looking for include file sys/eventfd.h - found
-- Looking for include file sys/eventfd.h
-- Looking for include file sys/eventfd.h - found
-- Looking for include file sys/timerfd.h
-- Looking for include file sys/timerfd.h - found
-- Looking for include file poll.h
-- Looking for include file poll.h - found
-- Finding recommended feature X11 for X11 (X11 client and server)
--     Disable feature X11 using "-DWITH_X11=OFF"
-- Found X11: /usr/lib/arm-linux-gnueabihf/libX11.so
-- Finding recommended feature Wayland for Wayland (Wayland client)
--     Disable feature Wayland using "-DWITH_WAYLAND=OFF"
-- checking for module 'wayland-client'
--   package 'wayland-client' not found
-- Could NOT find WAYLAND (missing:  WAYLAND_LIBRARY WAYLAND_INCLUDE_DIR)
CMake Warning at cmake/FindWayland.cmake:57 (message):
Couldn't detect wayland version - no version check is done
Call Stack (most recent call first):
cmake/FindFeature.cmake:25 (find_package)
CMakeLists.txt:560 (find_feature)
CMake Warning at cmake/FindFeature.cmake:46 (message):
feature Wayland was requested but could not be found! ON / FALSE
Call Stack (most recent call first):
CMakeLists.txt:560 (find_feature)
-- Skipping optional feature DirectFB for DirectFB (DirectFB client)
--     Enable feature DirectFB using "-DWITH_DIRECTFB=ON"
-- Finding required feature ZLIB for compression (data compression)
-- Found ZLIB: /usr/lib/arm-linux-gnueabihf/libz.so (found version "1.2.7")
-- Finding required feature OpenSSL for cryptography (encryption, certificate validation, hashing functions)
-- Found OpenSSL: /usr/lib/arm-linux-gnueabihf/libssl.so;/usr/lib/arm-linux-gnueabihf/libcrypto.so (found version "1.0.1e")
-- Skipping optional feature OpenSLES for multimedia (OpenSLES audio / video)
--     Enable feature OpenSLES using "-DWITH_OPENSLES=ON"
-- Finding recommended feature ALSA for sound (audio input, audio output and multimedia redirection)
--     Disable feature ALSA using "-DWITH_ALSA=OFF"
-- Could NOT find ALSA (missing:  ALSA_LIBRARY ALSA_INCLUDE_DIR)
CMake Warning at cmake/FindFeature.cmake:46 (message):
feature ALSA was requested but could not be found! ON / FALSE
Call Stack (most recent call first):
CMakeLists.txt:570 (find_feature)
-- Skipping optional feature Pulse for sound (audio input, audio output and multimedia redirection)
--     Enable feature Pulse using "-DWITH_PULSE=ON"
-- Skipping optional feature Cups for printing (printer device redirection)
--     Enable feature Cups using "-DWITH_CUPS=ON"
-- Skipping optional feature PCSC for smart card (smart card device redirection)
--     Enable feature PCSC using "-DWITH_PCSC=ON"
-- Finding recommended feature FFmpeg for multimedia (multimedia redirection, audio and video playback)
--     Disable feature FFmpeg using "-DWITH_FFMPEG=OFF"
-- checking for module 'libavcodec'
--   package 'libavcodec' not found
-- checking for module 'libavutil'
--   package 'libavutil' not found
-- Could NOT find FFmpeg (missing:  AVUTIL_FOUND AVCODEC_FOUND)
Note: To build libavcodec version >= 0.8 (API >= 53.25.0) is required
CMake Warning at cmake/FindFeature.cmake:46 (message):
feature FFmpeg was requested but could not be found! ON / FALSE
Call Stack (most recent call first):
CMakeLists.txt:576 (find_feature)
-- Skipping optional feature GStreamer_0_10 for multimedia (multimedia redirection, audio and video playback, gstreamer 0.10 version)
--     Enable feature GStreamer_0_10 using "-DWITH_GSTREAMER_0_10=ON"
-- Finding recommended feature GStreamer_1_0 for multimedia (multimedia redirection, audio and video playback)
--     Disable feature GStreamer_1_0 using "-DWITH_GSTREAMER_1_0=OFF"
-- checking for module 'glib-2.0'
--   package 'glib-2.0' not found
Glib_INCLUDE_DIR=Glib_INCLUDE_DIR-NOTFOUND
GlibConfig_INCLUDE_DIR=GlibConfig_INCLUDE_DIR-NOTFOUND
Glib_LIBRARY=Glib_LIBRARY-NOTFOUND
Gobject_LIBRARY=Gobject_LIBRARY-NOTFOUND
CMake Error at cmake/LibFindMacros.cmake:74 (message):
Required library Glib NOT FOUND.
Install the library (dev version) and try again.  If the library is already
installed, use ccmake to set the missing variables manually.
Call Stack (most recent call first):
cmake/FindGlib.cmake:41 (libfind_process)
cmake/FindGStreamer_1_0.cmake:80 (find_package)
cmake/FindFeature.cmake:25 (find_package)
CMakeLists.txt:579 (find_feature)
-- Configuring incomplete, errors occurred!

It seams you don't have some packages installed. try to build after installing these packages glib-2.0,libavutil, wayland-client and alsa

Same exact errors after installing glib-2.0 and alsa.
sudo apt-get install was not able to find packages libavutil and wayland-client

@jasjitchopra that doesn't look like a FreeRDP bug but more like something you may do on your installation. Have you tried setting values as suggested by the cmake output ?

you should install "development" package for each software you are missing. if you are on ubuntu, you should try: sudo apt-get install libglib2.0-dev libasound2-dev libpulse-dev. NOTE: what i am typing may not be exact the software name, use TAB key in apt-get to get the correct package name.
PS: better read the freerdp wiki before you build your own version of freerdp :)


getting empty dataset.x_dict

Hi,
I am using the below snippet to create dataset, however, when I print dataset.x_dict, it;s coming out to be and empty dictionary. This is unlike the notebook that I followed, notebook , where dataset.x_dict seems to be automatically created.
`item_index = df[df['choice'] == 1].sort_values(by='case')['alt'].reset_index(drop=True)
print(item_index)
item_names = ['BRAND_1', 'BRAND_2', 'BRAND_3', 'BRAND_4']
num_items = 4
encoder = dict(zip(item_names, range(num_items)))
print(f"{encoder=:}")
item_index = item_index.map(lambda x: encoder[x])
item_index = torch.LongTensor(item_index)
print(f"{item_index=:}")
cost = utils.pivot3d(df, dim0='case', dim1='alt', values='cost')
msrp = utils.pivot3d(df, dim0='case', dim1='alt', values='msrp')
dataset = ChoiceDataset(item_index=item_index,
cost_data=cost,
msrp_data=msrp,
).to(device)
`
this is leading to the below error when I run the code

KeyError                                  Traceback (most recent call last)
 in
1 start_time = time()
----> 2 run(model, dataset, num_epochs=500, dataset_test=None, batch_size=-1, learning_rate=0.01, model_optimizer="Adam")
3 print('Time taken:', time() - start_time)
30 frames
/usr/local/lib/python3.8/dist-packages/torch_choice/model/conditional_logit_model.py in forward(self, batch, manual_coef_value_dict)
267             corresponding_observable = var_type.split("[")[0]
268             total_utility += coef(
--> 269                 x_dict[corresponding_observable],
270                 batch.user_index,
271                 manual_coef_value=None if manual_coef_value_dict is None else manual_coef_value_dict[var_type])
KeyError: 'cost_data'

PS: Along with the above mentioned notebook, I was also following your official documentation step by step and I am not able to find what am I missing.
Can you please help me with this issue.
Thanks!

Hey, thank you for spotting this.
The issue was caused by incorrectly naming the keyword variable while you constructed the dataset. Since I don't have the MSRP variable in my example dataset, I will use the cost variable to illustrate this issue. Here, the cost variable is an (item, session)-specific variable; when you build the ChoiceDataset data structure, you would need to name the cost variable with something starting with itemsession_ or sessionitem_ to register the variable as a covariate/feature in the dataset and tell the dataset about the level of variation (e.g., item-specific, user-specific, item-session-specific, etc) of the cost variable.
# this is what you did.
dataset = ChoiceDataset(item_index=item_index,
cost_data=cost,  <--- change to `itemsession_cost_data=cost`.
msrp_data=msrp,
).to(device)

For a complete running example:
import pandas as pd
import torch
from torch_choice.data import ChoiceDataset, utils

if __name__ == "__main__":
    device = "cpu"
    df = pd.read_csv('./tutorials/public_datasets/ModeCanada.csv')  # <------- You may need to change the path depending on where you execute your code,
    df = df.query('noalt == 4').reset_index(drop=True)
    df.sort_values(by='case', inplace=True)
    df.head()

    item_index = df[df['choice'] == 1].sort_values(by='case')['alt'].reset_index(drop=True)
    print(item_index)
    item_names = ['train', 'car', 'bus', 'air']  # <-- I changed this to fit the data I am using.
    num_items = 4
    encoder = dict(zip(item_names, range(num_items)))
    print(f"{encoder=:}")
    item_index = item_index.map(lambda x: encoder[x])
    item_index = torch.LongTensor(item_index)
    print(f"{item_index=:}")

    cost = utils.pivot3d(df, dim0='case', dim1='alt', values='cost')
    # msrp = utils.pivot3d(df, dim0='case', dim1='alt', values='msrp')  # <-- I removed this since there is not MSRP in this example dataset.

    dataset = ChoiceDataset(
        item_index=item_index,
        itemsession_cost_data=cost,
        # msrp_data=msrp,
    ).to(device)
    print(dataset.x_dict.keys())
    print(dataset.x_dict['itemsession_cost_data'].shape)  # should print torch.Size([4324, 4, 1]), where df['case'].nunique() = 4324.

For technical details, if you look into the ChoiceDataset.x_dict() method, the out dictionary returned only include elements from self.__dict__.items() if the name of this element (i.e., the key) passes the test self._is_attribute(key). The _is_attribute method checks whether the key correspond to a variable that will enter the regression model (e.g., the cost and MSRP in your example). The _is_attribute method simply checks whether the key name starts with certain pattern, for example, if key = item_SOMETHING, the dataset will recognize it as a item-sepcific variable, if key = itemsession_SOMETHING or sessionitem_SOMETHING, the dataset recognizes it as an item-session-sepcific variable. The dataset includes them in the out dictionary returned.
In your code, the key was cost_data and msrp_data and the dataset unfortunately failed to recognize it as an attribute/variable for the model. You would need to change them to things like itemsession_cost_data and itemsession_msrp_data to tell the model that these are variables you use in the regression model.


Changing renewable subscription in iOS In-app purchase triggers “Cannot connect to iTunes Store” error

I have multiple renewable subscriptions set up for my App in App Store Connect.

There are a handful of renewable subscriptions which are all part of the same subscription group, plus one which is a non-renewable subscription.
I am using a sandbox user account as created under "Users and Access" > "Sandbox" > "Testers" logged in as a sandbox user on the iPhone. I am logged in as that user on the iPhone under "iTunes & App Stores" > "SANDBOX ACCOUNT".
I can successfully fetch the subscriptions using 'RNIap.getSubscriptions()' and display them in the UI. I can then successfully trigger a purchase of a subscription using RNIap.requestSubscription(productId, false) after which my registered RNIap.purchaseUpdatedListener handler correctly fires, and I finish the transaction using RNIap.finishTransactionIOS(purchase.transactionId). So far, so good.
The problem arises when I try to make subsequent changes to the renewable subscription, by following the same process with one of the other subscriptions. I get a pop-up asking whether I want to confirm the modification to the new subscription, as expected, followed by "You're all set, Your purchase was successful". But I when this happens it actually calls the RNIap.purchaseErrorListener event handler, rather than the RNIap.purchaseUpdatedListener as expected.
The error message I get from the handler is:
{
	responseCode: "0",
	code: "E_UNKNOWN",
	debugMessage: "Cannot connect to iTunes Store",
	message: "Cannot connect to iTunes Store"
}

Which originates from the StoreKit api itself when transaction.transactionState is equal to SKPaymentTransactionStateFailed.
If I then ettempt to buy thnon-renewable subscription (1 Week in the above example) the purchase is successful, triggering the RNIap.purchaseUpdatedListener as expected. But whenever I try to change the subscription to another renewable subscription again the RNIap.purchaseErrorListener handler is triggered.

I'm running into the same issues. I think it might be related to #525?

I'm running into the same issues. I think it might be related to #525?

Yeah that looks like it's the same issue. Thanks for that. Gives me something more to go on.

Can you verify your subscription group looks something like this? With the numbers next to it? As that shows the correct order to properly allow upgrading/downgrading. You need to re-arrange (drag and drop) this yourself

Can you verify your subscription group looks something like this? With the numbers next to it? As that shows the correct order to properly allow upgrading/downgrading. You need to re-arrange (drag and drop) this yourself. So, for example, an upgrade is from 3 to 2. A downgrade is from 2 to 3.

Ah okay. I will have a look at that and see. Thanks

Can you verify your subscription group looks something like this? With the numbers next to it? As that shows the correct order to properly allow upgrading/downgrading. You need to re-arrange (drag and drop) this yourself. So, for example, an upgrade is from 3 to 2. A downgrade is from 2 to 3.

I didn't even know that page existed to be honest. I read this article which the page links to https://help.apple.com/app-store-connect/#/dev75708c031. It says that if they are all the same level you should be able to "crossgrade" between each one. Mine are all level 1, which matches the example they give under the section Identical services at different durations in that article. This is what it looks like now. I will try it out again when I get hold of a iOS test device.

I have the same error. Subscriptions with same level different duration

I haven't solved the issue yet. I will probably come back to it in the next month or so.

Still active. I will get round to this soon(ish)


Unable to set attributes on SVG Elements

I'm having trouble with SVGs. This creates the nodes and the polygon even has styles on it, but the attributes don't show up (nor do they when I use props):
<svg
      attrs={{
        height: "250",
        width: "450",
      }}
    >
      <polygon
        attrs={{
          points: "225,10 100,210 350,210",
        }}
        style={{
          fill: "white",
          stroke: "black",
        }}
      />
</svg>

Lemme take a look. The example we have uses SVG, but lemme see if I can repro it. Unless you already have an open repo with repro code.

I just figured it out. options allow text but not svgs lol. works fine, my bad. one thing I've found is that attributes cant be camelcased which isnt true elsewhere

What do you mean by “options allow text not svgs?”
Attributes not being camel cases is a html dom limitation that is imposed by browser. Snabbdom is simply making setAttribute calls.

html option elements dont allow arbitrary html inside, and were rendering unicode geometric characters but not svgs when I tried to switch over. my bad
I'm pretty sure I've noticed the props and style help transform camel case into kebab, but maybe I'm misremembering

Yeah snabbdom jsx lite doesn’t do any special string transformations.
String kungfu is expensive. That’s how we’re able to do a million vnodes in
100ms at ~10ns / jsx call.
The style module sets on elem.style which is camel cased. Same as props.
They are camelCased as well.


UIWindow: Full titlebar width when enable_close_button=0

Hi,
Is it possible to enlarge the titlebar of a windows to the full width if the enable_close_button is set to 0 ?  Otherwise it is not nice.
I have to write :
self.title_bar_button_width=0
self.title_bar.set_dimensions((self._window_root_container.relative_rect.width -
self.title_bar_button_width,
self.title_bar_height))

Thanks for reminding me about this bug. I'll make a note to fix this in the next version.

This should be fixed 0.5.5.


Meetings ?

Hi,
can I ask what the time /date is of the working group meetings ?

@DT-DawidWroblewski  : Would you please insert the meeting data in the Readme.md of the Sub Project?

Meetings information is now part of readme


chore(apps): bump Jupyterlab to 4.x

To make it easier to work with recent versions of it in chorus.


⚠ SSH 🇮🇩 Indonesia IDN 1 has degraded performance

In 97d31f5, SSH 🇮🇩 Indonesia IDN 1 (http://ids-3.opensvr.net:8080/) experienced degraded performance:

HTTP code: 400
Response time: 27078 ms

Resolved: SSH 🇮🇩 Indonesia IDN 1 performance has improved in 579748f.


Support Edge

make the extension compatible with MS Edge

Seems that this might be something that is worth reavaluating bearing in mind that they are looking at releasing Chromium Edge in the newyear. Should be fairly straightforward as pretty much all Chrome addons work on Chromium Edge.
It has a little more work than Chrome but should still be quite easy.
https://docs.microsoft.com/en-us/microsoft-edge/extensions-chromium/developer-guide/port-chrome-extension
https://docs.microsoft.com/en-us/microsoft-edge/extensions-chromium/publish/publish-extension

Edge is now based on Chromium and some people already installed our extension on it. So this is supported (unoficially).


Add notifications to home page

Is your feature request related to a problem? Please describe.
Vinay wanted us to have a single place on our app where our notifications would reside outside of push notifications. Sort of like the Facebook notifications where they are chronological and appear in one page. Can also have inbox-like features such as swipe to delete.
Describe the solution you'd like
Have the notifications appear as a list on the home page. Swiping a notification should delete it. Clicking a notification should take you to the relevant page in the app (usually go to book info activity).
We can have a separate table in our database for inapp notifications that is written to when we create/accept requests etc so that we can persist notifications.
Describe alternatives you've considered
N/A
Additional context
N/A

Do we still want to add notifications to the home page??

@MarcusBoay nah, that's fine. I kind of like the notifications being a separate activity better. You can close this. We can just leave the Home fragment blank for now.


Y-axis direction in box plots

The numeration on Y-axis does not match the scenario indices in the legend.

Ark yes 👍

Fixed with https://github.com/Simulation-Decomposition/simdec-python/commit/4e87e4416342bf8576f47abe43ffcc3dde7811c2


PLW1641: false positive for classes inheriting from builtins

class Foo(dict):
    def __eq__(self, other):
        return # some logic

reports PLW1641 eq-without-hash but __hash__ is inherited from dict.

Isn't dict.__hash__ None?

Also:

A class that overrides eq() and does not define hash() will have its hash() implicitly set to None.

Source: https://docs.python.org/3/reference/datamodel.html#object.__hash__

oh yes, sorry.


TimeSeek node in AnimationTreePlayer now propagates filters

There was a bug with the TimeSeek node not propagating filters down the tree. Fixed.

@m4nu3lf Please rebase the merge commit out of the PR.

@bojidar-bg Sorry about that, Can you instruct me how to do that on github?

you do that locally, not on github.
On Thu, Jun 9, 2016 at 11:48 PM, m4nu3lf<EMAIL_ADDRESS>wrote:

@bojidar-bg https://github.com/bojidar-bg Sorry about that, Can you
instruct me how to do that on github?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/5128#issuecomment-225021792,
or mute the thread
https://github.com/notifications/unsubscribe/AAAX0ysK-kE9m5IOealuoXwcZ336uL4Hks5qKHwygaJpZM4IyW5Z
.

@slapin I've done that locally, but now how do I reflect that in the pull request?

just force push it using
git push origin +branch1:branch2
where origin is remote name
branch1 - local branch name you're pushing
branch2 - remote branch name
You also can do
git push origin +branchname
if your remote and local branches have the same name.
if you don't know your remote name you can use
git remote
to list them.
If you did not add your repository to remotes, then you can use direct
repository specification, like
git push git@github.com:username/repo.git +branchname
(just replace origin with repo spec)
The branch used is the one you specify when made PR, this will update your
PR with new data.
On Fri, Jun 10, 2016 at 12:03 AM, m4nu3lf<EMAIL_ADDRESS>wrote:

@slapin https://github.com/slapin I've done that locally, but now how
do I reflect that in the pull request?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/5128#issuecomment-225025758,
or mute the thread
https://github.com/notifications/unsubscribe/AAAX0zVGJjHw-vk1xnbGbOnuMF7iv5UZks5qKH-GgaJpZM4IyW5Z
.

I screwed thigs up, please delete this pull request, I will reopen a new one ( after I learn how properly rebase)

Git can be hard to learn but you didn't have to close the PR, much less to delete your fork. I hope you can send another PR soon.


Warning on create account - recover account

Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.
in CreateAccount (created by Context.Consumer)
in Connect(CreateAccount) (created by Context.Consumer)

How should we classify/label this? Is it considered a bug?


Problem about the test.js handling sentry file

When I try to run the test/test.js, I found a little flaw about the sentry file storing. As the code below:
steamUser.on('updateMachineAuth', function(sentry, callback) {
    fs.writeFileSync('sentry', sentry.bytes)
    util.log("sentryfile saved");
    callback({
        sha_file: crypto.createHash('sha1').update(sentry.bytes).digest()
    });
});

The code save the sentry bytes directly which is ok for me. But when using the sentry to login, the code read the sentry file without hashing it.
...
try {
    var sentry = fs.readFileSync('sentry');
    if (sentry.length) logOnDetails.sha_sentryfile = sentry;
} catch (beef) {
    util.log("Cannae load the sentry. " + beef);
}

This may leading the situation that you will receive the 63 error continually when you login with the sentry.
My solution is store the hashed bytes or hash the sentry after read it.
 fs.writeFileSync('sentry', crypto.createHash('sha1').update(sentry.bytes).digest())

or hash the sentry after read it
if (sentry.length) logOnDetails.sha_sentryfile = crypto.createHash('sha1').update(sentry).digest();

Cheers!

Feel free to checkout the dev branch and make a pull request ☺
On 11 Jan 2016 12:07 pm, Roger<EMAIL_ADDRESS>wrote:
When I try to run the test/testjs, I found a little flaw about the sentry file storing As the code below:
steamUseron('updateMachineAuth', function(sentry, callback) { fswriteFileSync('sentry', sentrybytes) utillog("sentryfile saved"); callback({ sha_file: cryptocreateHash('sha1')update(sentrybytes)digest() }); });
The code save the sentry bytes directly witch is ok for me But when using the sentry to login, the code read the sentry file without hashing it
try { var sentry = fsreadFileSync('sentry'); if (sentrylength) logOnDetailssha_sentryfile = sentry; } catch (beef) { utillog("Cannae load the sentry " + beef); }
This may leading the situation that you will receive the 63 error continually when you login with the sentry
To solve it is simple as you can see Store the hashed bytes or hash the sentry after read it
fswriteFileSync('sentry', cryptocreateHash('sha1')update(sentrybytes)digest())
or
if (sentrylength) logOnDetailssha_sentryfile = cryptocreateHash('sha1')update(sentry)digest();
Cheers!
—
Reply to this email directly or view it on GitHub.


Add support for Jest 27

🐛 Bug Report
Upgrading to Jest 27 leads to the following error:
TypeError: Jest: a transform must export something.
Also the peer dependency scope should be updated alongside this fix:
warning " ><EMAIL_ADDRESS>has incorrect peer dependency "jest@>=26 <27".
To Reproduce
Upgrade from Jest 26 to Jest 27.
Expected behavior
Running the jest command to successfully run tests.
Debug log:
# content of ts-jest.log :
{"context":{"allowJs":false,"logLevel":20,"namespace":"jest-preset","package":"ts-jest","version":"26.5.6"},"message":"creating jest presets not handling JavaScript files","sequence":1,"time":"2021-05-25T10:57:05.252Z"}
{"context":{"allowJs":false,"logLevel":20,"namespace":"jest-preset","package":"ts-jest","version":"26.5.6"},"message":"creating jest presets not handling JavaScript files","sequence":2,"time":"2021-05-25T10:57:05.265Z"}
{"context":{"logLevel":20,"namespace":"Importer","package":"ts-jest","version":"26.5.6"},"message":"creating Importer singleton","sequence":3,"time":"2021-05-25T10:57:05.980Z"}

envinfo
System:
    OS: Windows 10
    Node version: 15.12.0

Npm packages:
    "jest": "^27.0.1",
    "ts-jest": "^26.5.6",
    "typescript": "^4.2.4"

Thanks for notification, we already prepare for it at next tag so we will prepare for latest tag soon.

@ahnpnl Can you give some indication of when soon is? Also, it would be helpful to keep this issue open while the problem exists, I had assumed it was resolved given you'd closed it.

it is resolved in next tag. If you are in a hurry, you can use ts-jest@next. next tag is basically ready to release.

Ideal, thank you.

@ahnpnl, looks like npm still points next to 27.0.0-next.12, which was published awhile back. It currently supports jest until 27.0.0-next9. However, latest jest release is 27.0.1

next tag is fully compatible despite the warning message. You can temporarily use next tag if you are in a hurry :) ts-jest 27 will be released within this week.

27.0.0 was out

Hello, I am having this issue in 27.0.1

    "ts-jest": "27.0.1"

It is related with this?

Please clear Jest cache.

Thanks @ahnpnl for your (I must say) incredibly fast response.
I have:

Cleared jest cache doing jest --showConfig | grep "cache" and removed all cache directories listed by the command
removed node_modules
confirmed yarn.lock is 27.0.1

You are using Jest 26 with ts-jest 27, which is not a valid combination, see https://github.com/sauldeleon/monorepo-starter/blob/623bc73b10f1a33f3f73695d96cf8c473c7aaedc/yarn.lock#L1744. ts-jest major version always follows Jest major version. For example: Jest 26 with ts-jest 26, Jest 27 with ts-jest 27

thanks a lot @ahnpnl . As you said, this was the issue. Seems that CRA installs jest 26! Thanks a lot

I'm having the same problem as @suvasishm

Hey everyone!
Following up on this thread, one basic question: How can I upgrade the versions of jest and @types/jest? (NPM)
I immediately assumed a npm i -D jest.. would do so automatically, but both versions remain at 26 and I need all of them to be at 27. I'm trying to solve this error -

I'd  appreciate your help!

Hey everyone!
Following up on this thread, one basic question: How can I upgrade the versions of jest and @types/jest? (NPM)
I immediately assumed a npm i -D jest.. would do so automatically, but both versions remain at 26 and I need all of them to be at 27. I'm trying to solve this error -

I'd appreciate your help!

I managed to solve it. Just run the npm install ts-jest@27 (npm install {package}@{version}) command. Applicable to any package.


Add Node option in the actual node list

I know you can manually add your own wallet node to the XT wallet, but for noobs its hard to find that setting...    I suggest (especially with the potential for SIP-35) that we add an option at the top or bottom of the below list to "Manually Add" a node..    all this option would do is take the user directly to the /settings/networks page so they can manually add the node.

Hi. I would like to add - related to the Topic:

if i want to add full URL: **http://**<IP_ADDRESS>:6876 - Wallet says - not valid URL. So, address only without HTTP looks is OK. But some of Signum nodes might use HTTPS for API, hence, HTTP (or HTTPS) is needed and a right format for URL.

I was trying adding my other testnet node located in my LAN - no success, so far.. Neither node in my LAN (private) IP, nor the same node via External (public) IP - Cannot reach network node.


Mac dlib run error

OS X version: 10.14.2
python version:  3.6.2
dlib version: 19.17.0
boost version: boost: stable 1.68.0 (bottled), HEAD
boost-python3: stable 1.68.0 (bottled), HEAD
GCC 4.2.1 Compatible Apple LLVM 9.1.0 (clang-9<IP_ADDRESS>)
I install dlib by:

brew cask install xquartz
brew install gtk+3 boost
brew install boost-python3

and i make virtualenv.
pip install dlib.
intsall successfully.
But when i run
import dlib
get this:
  File "<stdin>", line 1, in <module>
ImportError: dlopen(/usr/local/lib/python3.6/site-packages/dlib.cpython-36m-darwin.so, 2): Symbol not found: _png_set_longjmp_fn
  Referenced from: /usr/local/lib/python3.6/site-packages/dlib.cpython-36m-darwin.so
  Expected in: flat namespace
 in /usr/local/lib/python3.6/site-packages/dlib.cpython-36m-darwin.so

can you tell me how to deal this problem?
By the way, my brew gcc is /usr/local/Cellar/gcc/8.3.0.
I don't know whether  this affect .

Uninstall libpng. Brew probably installed a broken copy of it. Then reinstall dlib.

i tried,  i run brew reinstall libpng for several times.
And my libpng version is libpng: stable 1.6.36 (bottled), HEAD.
And i run ln -s /usr/local/Cellar/libpng/1.6.36/lib/libpng16.16.dylib /usr/local/lib/libpng16.16.dylib
But it still doesn't work..

Don’t reinstall. Delete

I tried, i run:

 rm -r /usr/local/Cellar/libpng/
rm /Users/fjl2401/Library/Caches/Homebrew/downloads/b401f5b07a537975da7de448c4bbdaf0c2112d5dc7d8e0955e0d4fb938909010--libpng-1.6.36.mojave.bottle.tar.gz
 brew install libpng
but it still don't work.

No. Do not install libpng from brew. Entirety remove it. Don’t reinstall. Make your computer not have libpng on it.

yeah...I rm libpng, but when i use brew to reinstall dlib. It will install libpng as dependency automatically.
I rm libpng, and pip reinstall dlib.  It shows another error:
>>> import dlib
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: dlopen(/usr/local/lib/python3.6/site-packages/dlib.cpython-36m-darwin.so, 2): Symbol not found: _png_set_longjmp_fn
  Referenced from: /usr/local/lib/python3.6/site-packages/dlib.cpython-36m-darwin.so
  Expected in: flat namespace
 in /usr/local/lib/python3.6/site-packages/dlib.cpython-36m-darwin.so

Don’t brew install anything, especially dlib. Brew is broken. Pip install dlib after removing libpng.

Thank you very much ! i rm all lib about libpng on my computer. And i run import dlib in python.
And it shows  can't find libpng.lib from php5...
So... i think i find the reason. I rm all libpng in /usr/local/lib and   rm -rf /usr/local/php5-5.6.31-20170817-164511 and pip install dlib --no-cache .After setup.py  build a long time. It works well ! Thank you very much.

I also just pushed a change to github that will cause cmake to build a little test program to see if the system copy of libpng actually works.  If it doesn't it will automatically do the right thing now.

Also just added checks like this for libjpeg.  Hopefully this fixes this kind of issue forever.  The previous checks were less robust.  Can you try installing the latest dlib from github? It should work, even if you brew install libpng first.

i installed use pip:  pip install git+https://github.com/davisking/dlib.git and Successfully installed dlib-19.17.99. And i run my script, it works well.

Sweet, thanks for testing it.

thanks for your sharing @Colaplusice


[blocked] Warn when removing a default from an introspected foreign key

Marking this PR as blocked because there is a test failure on scalar list tests, but we're in the process of completely changing this, so I'll wait until the other PRs are merged.


Stop using python 3.9-only features

In review-dependabot-prs.py, we were using a new parameter that was not available in Python 3.6 (https://docs.python.org/3.6/library/subprocess.html). Revert to the old way of capturing output (that still works in 3.9).
I can confirm that the script now runs on Python 3.6

Hurrah - I'll add a note to the Trello ticket to suggest the use of this script


fs/vfs: reuse file_dup2 directly in file_dup to fix segfault issue

Summary
Close the fd2 return by dup() would segment fault since filep->f_priv null access, while file_dup2 handle this rightly.
Impact
Testing
Test and verify under sim

LGTM

LGTM, this bug is found and fixed in our LTP testing:
https://github.com/linux-test-project/ltp


Plugin cannot be used on a new grafana installation due to signature type

What steps will reproduce the bug?

Download helm chart of grafana helm pull --untar grafana/grafana
Add the plugin to the helm installation to the values.yaml

...
plugins:
    - https://github.com/DevoInc/grafana-devo-datasource/releases/download/v3.1.0/devoinc-datasource-3.1.0.zip;devoinc-datasource
...

Install grafana helm install localgrafana grafana
In the grafana pod logs you will see the following message, and the plugin wont be available in grafana:

...
logger=plugins.signature t=2024-10-28T09:19:51.481349136Z level=warn msg="Could not find root URL that matches running application URL" plugin=devoinc-datasource appUrl=http://localhost:3000/ rootUrls=[https://devoinc.grafana.net/ │
│ logger=plugins.signature.validation t=2024-10-28T09:19:51.481386261Z level=warn msg="Skipping loading plugin due to problem with signature" pluginId=devoinc-datasource status=invalid                                                 │
│ logger=plugins.validation t=2024-10-28T09:19:51.481392677Z level=error msg="Plugin validation failed" pluginId=devoinc-datasource error="plugin 'devoinc-datasource' has an invalid signature"...

How often does it reproduce? Is there a required condition?
Always
What is the expected behavior? Why is that the expected behavior?
The expected behaviour is that the devo plugin is available, however, it isnt.
What do you see instead?
I can see that the plugin has been downloaded, but it doesnt appear in grafana

localgrafana-6db9864987-6fn58:/usr/share/grafana$ ls /var/lib/grafana/plugins
devoinc-datasource
localgrafana-6db9864987-6fn58:/usr/share/grafana$ ls /var/lib/grafana/plugins/devoinc-datasource/
CHANGELOG.md   LICENSE        MANIFEST.txt   README.md      img            module.js      module.js.map  plugin.json

Additional information
I think it could be related with the signature type of the plugin
$ cat /var/lib/grafana/plugins/devoinc-datasource/MANIFEST.txt
...
"signatureType": "private",
  "signedByOrg": "devoinc",
  "signedByOrgName": "devoinc",
  "rootUrls": [
    "https://devoinc.grafana.net/"
  ],
  "plugin": "devoinc-datasource"
...

Given it is private, it only can be used if the url of grafana is https://devoinc.grafana.net/

Found a workaround meanwhile, are you planning to change the licence type of the plugin?
grafana.ini:
  server:
    root_url: "https://devoinc.grafana.net"


What's different between idb and wda?

https://github.com/facebook/idb
idb is a flexible command line interface for automating iOS simulators and devices
wda project has been archived, so will you use idb?

哈哈哈
| |
xsxooxsx
|
|
<EMAIL_ADDRESS>|
Signature is customized by Netease Mail Master
On 07/18/2020 15:29, Carlos Hwang wrote:
https://github.com/facebook/idb
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

idb is a flexible command line interface for automating iOS simulators and devices
wda is A WebDriver server for iOS that runs inside the Simulator.

idb is a nice tools, it just not working on my mac.


What's problem with MudBlazor component v5.0.9?

I dont know how to fix it, please help!

Clear the Chrome cache ineffective but using firefox is perfect! Closed!


rmfakecloudctl: Don't error if there are no metadata files

Tested and works on my new rM2 :sunglasses:


Summarize learnings for customers

Workshop with Stian and product team.
Workshop revealed the following candidates:

[x] Probes

https://github.com/keycloak/keycloak/pull/22112

[x] JGroups and Keycloak thread pool configuration advice

waiting for final conclusion after retest in https://github.com/keycloak/keycloak-benchmark/issues/520
document JGroups https://github.com/keycloak/keycloak-benchmark/issues/553

[x] FlightRecorderOptions + Cryostat / backport to KC22, possibly add CryoStat docs

https://github.com/keycloak/keycloak-benchmark/issues/519

[ ] Load Shedding

https://github.com/keycloak/keycloak-benchmark/issues/547

[ ] DB Pool size -> pending
[ ] OpenTelemetry Support -> pending
[ ] Keycloak sizing guide (memory/CPU) for pods -> will end up in docs, currently available at https://www.keycloak.org/keycloak-benchmark/benchmark-guide/latest/report/rosa-benchmark-key-results
[x] Disable sticky sessions - outline in which scenario it would make sense -> pending
[ ] Metrics - Most important SLI to determine that Keycloak is healthy -> pending
[ ] Creating a Keycloak Grafana Dashboard

@kami619 - agreed, done.

Closing this as all items have been completed, and the Grafana dashboard is tracked independently.


Fixes #19 and #21: add more configuration options to ProviderVerifier

Introduces an interface for publishing of verification results (#21). Includes some fixes for validating by annotated method (#19) and making the ScalaTest trait more flexible by extending the AnyFlatSpecLike trait instead of the AnyFlatSpec class.

@jbwheatley there seems to be some issue with the annotated method scanning when running the entire test suite at once (sbt test) -- individually, all the tests pass, but collectively it seems like there might be some thread locking going on:
[39.251s][warning][gc,alloc] ClassGraph-worker-34: Retried waiting for GCLocker too often allocating 256 words
[39.251s][warning][gc,alloc] ClassGraph-worker-4: Retried waiting for GCLocker too often allocating 256 words
[39.251s][warning][gc,alloc] ClassGraph-worker-19: Retried waiting for GCLocker too often allocating 256 words
[39.251s][warning][gc,alloc] ClassGraph-worker-32: Retried waiting for GCLocker too often allocating 10299 words
[39.251s][warning][gc,alloc] ClassGraph-worker-48: Retried waiting for GCLocker too often allocating 11794 words
[39.341s][warning][gc,alloc] ClassGraph-worker-52: Retried waiting for GCLocker too often allocating 2050 words
[39.341s][warning][gc,alloc] ClassGraph-worker-28: Retried waiting for GCLocker too often allocating 5384 words
[39.342s][warning][gc,alloc] ClassGraph-worker-31: Retried waiting for GCLocker too often allocating 2050 words

I will have to spend more time debugging this to see what the issue is exactly. Possibly related:

https://bugs.openjdk.java.net/browse/JDK-8192647
https://github.com/classgraph/classgraph/issues/338

Apparently this fixes it:
-J-Xmx4096M
-J-XX:+UseG1GC

But having to use 4GB of data seems a bit insane. We will need to figure out how to do some kind of caching for the classpath scanning. Or maybe this is a good time to figure out how to not have to use it.

@jbwheatley other than the GC issues, anything else you'd like me to change in this PR before merge?

@jbwheatley ProviderInfo has a property on it called packagesToScan -- turns out if we isolate our messages into a package and just scan that package I can reduce the memory footprint to under 256M when running all the tests at the same time (previously it required 4GB to do that). So at least this will resolve that issue for now.

Cool, looks great, and seems to Just Work™ 😄


Fix GCP Project Var

This fixes a GCP project var for consistency. Identified by @arvindamirtaa in #55.

Appreciate that @robmorgan
I started on a fix but got busy with other things. Thanks

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.


Implemented string_agg_sql for  STRING_AGG/LISTAGG generator to handl…

Fixes #39
…e DISTINCT within the  arguments.

Updated string_agg_sql method to properly handle the separator for GROUP_CONCAT expressions when the separator is getting embedded within a DISTINCT node.

Core reason of the problem:
For this part of a query(snowflake) with pattern like
listagg(distinct "helpdesk_ticket_131195_FRESHSERVICE_ticket_picklist_values"."value", ',')

The AST that is generated before E6 query getting build up for this part is expected to be
this=GroupConcat(
        this=Distinct(
          expressions=[
            Column(
              this=Identifier(this=value, quoted=True),
              table=Identifier(this=helpdesk_ticket_131195_FRESHSERVICE_ticket_picklist_values, quoted=True))]),
        separator=Literal(this=,, is_string=True))

But unexpectedly turning out to be
this=GroupConcat(
        this=Distinct(
          expressions=[
            Column(
              this=Identifier(this=value, quoted=True),
              table=Identifier(this=helpdesk_ticket_131195_FRESHSERVICE_ticket_picklist_values, quoted=True)),
            Literal(this=,, is_string=True)])),

where the separator string ',' is been embedded into DISTINCT statement in first argument rather than being a second argument(separator) in GroupConcat Node due to which it is assuming there is no separator provided and adding the default separator i.e; ',' to the end resulting  to the below wrong query portion in E6
STRING_AGG(distinct "helpdesk_ticket_131195_FRESHSERVICE_ticket_picklist_values"."value", ',', ',')


Nits for GroupBuilder

Howdy there,
I had a couple nit picking comments for GroupBuilder()

Add additional constructor that accepts a built IGroup. This would offer a straight forward method to modify an existing Group.
GroupBuilder.Create() seems it should be a static factory.
Renaming WithComponent() to AddComponent() helps self document its behavior.
Renaming WithPredicate() to SetPredicate() helps self document it's behavior.

I didn't notice anything in the readme covering the contribution process (or how you feel about breaking changes :p ), but I would be happy to make a PR.
Cheers,
Dxslly

hey hey,
Am always open to PRs, although if it is a breaking change it would ideally go into a branch, but as long as the version released is a major increment doesnt matter too much.
As far as your points go:

Fine with this, makes sense
Completely makes sense, originally I think I was going to make an interface for further customization, but ended up adding blueprints and didnt go further, so this would make use cases far more succinct.
3 & 4. I generally prefer the notion of "with" when using builders, as you are not directly adding a group, just telling the builder that the group should be built "with" a component, but will ask in the gitter and get some opinions as I am happy to change it if a few others agree with the Add/Set vs With.


flexibee: 2<IP_ADDRESS> -> 2<IP_ADDRESS>

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/flexibee/versions.
meta.description for flexibee is: "Client for an accouting economic system"
meta.homepage for flexibee is: "https://www.flexibee.eu/"
meta.changelog for flexibee is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2<IP_ADDRESS> with grep in /nix/store/26i9al4iknlbrxkh99hmpzicagvalfx5-flexibee-2<IP_ADDRESS>

directory tree listing: https://gist.github.com/9241f72f44fc090738669132a7d23615

du listing: https://gist.github.com/c56b0c508ee4c9e20cac204be672f264

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

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
flexibee

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/26i9al4iknlbrxkh99hmpzicagvalfx5-flexibee-2<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/26i9al4iknlbrxkh99hmpzicagvalfx5-flexibee-2<IP_ADDRESS>
ls -la /nix/store/26i9al4iknlbrxkh99hmpzicagvalfx5-flexibee-2<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- flexibee

Maintainer pings
cc @mmahut for testing.

Still broken upstream.


helm_wrapper does a verbose rm for cleanup interfering with templating

Due to the verbose flag in the cleanup code for helm_wrapper, decrypted filenames are output:
    # cleanup on-the-fly decrypted files
    [[ ${#decfiles[@]} -gt 0 ]] && rm -v "${decfiles[@]}"

This means you can't run helm secrets template -f values.yaml -f secrets.yaml ./ > template.yaml without having to edit the file.
Removing the -v would fix this, or adding a new flag to run a silent cleanup. I suspect the goal is to avoid new flags which may interfere with helm compatibility?

+1
really block from using tools like Tilt (tilt.dev). Where we generate k8 specs via helm-template.


error when i am trying to declare filters with python wrapper

Required Info

Camera Model
D400

Platform
PC

SDK Version
2.11.0

Hey, I get the following error when i am trying to declare the following filter:
dec_filter = rs.decimation_filter()
"AttributeError: module 'pyrealsense2' has no attribute 'decimation_filter'
i tried to upgrade pyrealsense2 but it didn't solve the problem

problem solved, i reinstalled the sdk


follow compressDetStepMCs.simParticleTag change to a sequence

:sunny: The tests passed at 15f134784361f272988ab97bc8879883b105a90a.

Test
Result
Details

merge
:white_check_mark:
Merged 15f134784361f272988ab97bc8879883b105a90a at 5c901b0148a4a78244d591cd6d612475cdea327b

build (prof)
:white_check_mark:
Log file. Build time: 14 min 16 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 15f134784361f272988ab97bc8879883b105a90a after being merged into the base branch at 5c901b0148a4a78244d591cd6d612475cdea327b.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


merging in GSOC2021-AravindMurali

Result of git merge --squash
Squashed commit of the following:
commit c4b273c32bf750e5590e761369ef110f355f6cb5
Merge: e6014c04 d7716ba0
Author: ReimarBauer<EMAIL_ADDRESS>Date:   Mon Aug 23 15:10:26 2021 +0200
Merge pull request #1169 from aravindm711/gsoc-develop-merge-4

Gsoc develop merge 4

commit d7716ba0db286944b8f8a9a12d5954bc73dbc331
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Mon Aug 23 17:11:00 2021 +0530
refactored tests for help dialog

commit 111f64cfd858b9d051ed4278185bea03c2a15e07
Merge: e6014c04 da77f881
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Mon Aug 23 16:58:35 2021 +0530
Merge branch 'develop' into gsoc-develop-merge-4

# Conflicts:
#	mslib/mscolab/_tests/test_chat_manager.py
#	mslib/mscolab/_tests/test_file_manager.py
#	mslib/mscolab/_tests/test_files.py
#	mslib/mscolab/_tests/test_files_api.py
#	mslib/mscolab/_tests/test_seed.py
#	mslib/mscolab/_tests/test_server.py
#	mslib/mscolab/_tests/test_user.py
#	mslib/mscolab/_tests/test_utils.py
#	mslib/msui/_tests/test_mscolab.py

commit e6014c0431730e4749af5d856c53aff32340bbc9
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Sun Aug 22 11:49:12 2021 +0530
GSoC update - Updated MSColab help dialog (#1161)

* partially updated mscolab help dialog

* updated mscolab help dialog

* updated collaborator description

commit b6517e2c53cd57ef64d2c125cae9f2e80ad16258
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Sun Aug 22 02:46:33 2021 +0530
GSoC config editor update 3 - refactor of config_loader (#1159)

* refactor of config_loader

* moved config class to config submodule

* moved config related tests to its own submodule; extended test scope

* fixed restart bug

commit 7a104646b42d5971eb21196ea37bcbe3c97a3f7b
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Aug 18 01:04:20 2021 +0530
GSoC config editor update 2 - validations (#1154)

* added validations for config options

* removed comments; fixed bugs

* changed shortcut for views to meta for macOS

* changed dialogs from question to warning

commit 3cf5476d907c6f6f818b26d2634e02968de5ee66
Merge: 242f1f99 478af6b7
Author: ReimarBauer<EMAIL_ADDRESS>Date:   Wed Aug 11 15:14:21 2021 +0200
Merge pull request #1143 from aravindm711/GSOC2021-AravindMurali_gets_updated

GSoC develop merge 3

commit 478af6b7556990c9a9d4528a331d17c6e87f8f60
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Aug 11 17:37:40 2021 +0530
added assert statements from recent changes in develop

commit bec4268c6cc4debddd5a923230174e70f56753ea
Merge: 242f1f99 36f4467b
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Aug 11 17:22:26 2021 +0530
Merge branch 'develop' into GSOC2021-AravindMurali_gets_updated

# Conflicts:
#	mslib/msui/_tests/test_mscolab_version_history.py

commit 242f1f990461064ae58474d4e24df7c3c0b89ce3
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Aug 11 13:09:20 2021 +0530
GSOC Config editor update (#1103)

* initial implementation of profile window; extended test scope

* completed profile window implementation and related tests

* fixed flake8 test failure

* fixed http auth bugs; extended test scope

* added qt-json-view to support dir; added info in NOTICE

* modified qt-json-view; mentioned modifications in readme

* tested qt-json-view for proper usability

* Provide same test signature

* fixed failing test

* handled socket conn exception; cleaned up unwanted comments

* started implementation of config editor window

* partially implemented merging user settings with default; fixed tests

* implemented merging of default and user settings

* implemented remove option handler

* fixed bugs in 'remove' functionality

* implemented restore defaults functionality

* fixed restore default bugs and added logging statements

* implemented import, export, save_restart

* added comments

* moved import/save/export options to toolbar

Co-authored-by: May<EMAIL_ADDRESS>
commit ff38534156c4d540dff4480f8c09cc5400381f05
Merge: f57bbb8c ae254f06
Author: J. Ungermann<EMAIL_ADDRESS>Date:   Wed Aug 4 10:46:43 2021 +0200
Merge pull request #1119 from aravindm711/gsoc-develop-merge-2

Gsoc develop merge 2

commit ae254f0636b7f25c44cc44273b1246de3f03acb0
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Aug 3 14:04:24 2021 +0530
changed connect_window in tests to non modal to prevent time-out

commit 9dad7624e8b3ecc25a182b1f9a219a9fc0c6fe8b
Merge: f57bbb8c 7d4d924a
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Aug 3 10:12:47 2021 +0530
Merge remote-tracking branch 'upstream/develop' into gsoc-develop-merge-2

commit f57bbb8cff1536a0098558abab3c1c6ebe825899
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Fri Jul 23 16:22:12 2021 +0530
New Profile window (#1086)

* initial implementation of profile window; extended test scope

* completed profile window implementation and related tests

* fixed flake8 test failure

* fixed http auth bugs; extended test scope

* Provide same test signature

* fixed failing test

Co-authored-by: May<EMAIL_ADDRESS>
commit 853e2d581f73a000989659b1053507fdaa5b49db
Merge: 69672e4e 7050cd9d
Author: ReimarBauer<EMAIL_ADDRESS>Date:   Sun Jul 18 07:29:12 2021 +0200
Merge pull request #1082 from aravindm711/gsoc-develop-merge

Gsoc develop merge

commit 7050cd9d263e2141b963e6899dc716d4459035af
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Sat Jul 17 16:08:16 2021 +0530
fixed tests; merged open/import

commit 8c0f78846ba7849eddd85c6b4a7d73a56ae82657
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Thu Jul 15 14:24:31 2021 +0530
fixed mainwindow tests

commit 69672e4e0699e410aec634ed59c055ec732fc297
Merge: 018babe0 45b23432
Author: ReimarBauer<EMAIL_ADDRESS>Date:   Thu Jul 15 09:45:19 2021 +0200
Merge pull request #1079 from aravindm711/GSOC2021-AravindMurali

GSoC Progress Update 2 - Integrated plugins, refactored tests

commit 19faae626fd3ff96a3b75ed47e45119edee26e89
Merge: 45b23432 e2607fb8
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Jul 13 20:11:23 2021 +0530
Merge remote-tracking branch 'upstream/develop' into gsoc-develop-merge

# Conflicts:
#	mslib/msui/_tests/test_mscolab.py
#	mslib/msui/_tests/test_mss_pyui.py
#	mslib/msui/mscolab.py

commit 45b23432f87a52c85e69217911daf7fefc15ccc4
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Jul 13 19:51:05 2021 +0530
fixed requested changes

commit d8d79871f8645ecfedacef3a484e9c4bfebd1966
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Jul 13 10:55:53 2021 +0530
fixed failing tests and flake8 errors

commit 38107a772647abd8e1bc46fdaa8006eba7613199
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Sun Jul 11 15:47:38 2021 +0530
added plugin filter to add project browse; refactored tests

commit 018babe00170ffa737c730fd0d61f37978115166
Merge: 7895db82 4eecd757
Author: J. Ungermann<EMAIL_ADDRESS>Date:   Thu Jul 8 09:05:37 2021 +0200
Merge pull request #1059 from aravindm711/GSOC2021-AravindMurali

GSoC progress update

commit 4eecd7575e1882486a821680409b84b271058cb1
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Jul 7 15:03:13 2021 +0530
integrated plugins with save flighttrack

commit a4fdf64a8d17e5faaaf9182c01b2b6a9e5241ac3
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Mon Jul 5 21:23:04 2021 +0530
removed activate flighttrack submenu

commit 1f723613e39d41495b4838351443d407a54c8466
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Fri Jul 2 16:27:33 2021 +0530
made some improvements to ui

commit 943216bb1441f737dc449d5de33ffbe903a1773d
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Fri Jul 2 11:50:58 2021 +0530
fixed bug while opening views in local

commit 2ea1282b83948050841bd6e925c7638dde7fc5ca
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Thu Jul 1 21:44:14 2021 +0530
moved from tabs to parallel layout of lists

commit fb9d359cd160316e2860efedef255d82b4851f1d
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Thu Jul 1 14:41:21 2021 +0530
fixed view behaviour

commit 690b49ce8d15a1c6503ed8a3e21d869d3274d412
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Jun 30 23:24:02 2021 +0530
initial implementation of change in view behaviour

commit 21727f94f59f3a51d269f85905045e935d851a45
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Jun 30 19:53:43 2021 +0530
partially fixed mscolab tests

commit 2354ae997b2a9bc0147a4089dd312d37fd134185
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Wed Jun 30 00:21:46 2021 +0530
fixed view appearance; started test refactoring

commit 7895db82474edbf487ed4b6d5a27585d21e91fe4
Merge: e516945c f55501df
Author: J. Ungermann<EMAIL_ADDRESS>Date:   Fri Jun 25 13:41:58 2021 +0200
Merge pull request #1030 from aravindm711/GSOC2021-AravindMurali

Implementation of Connect Window

commit f55501df73739af04bf869e1f928a1765413d9e8
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Fri Jun 25 14:49:09 2021 +0530
fixed view submenus, fatal errors

commit 2649a4507a8a0927ebeb91159b02e1fb760b7f53
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Jun 22 21:12:30 2021 +0530
improvements to ui

commit 5b76093a0f0304201fb6b1e5b0f0feeb1169c87f
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Tue Jun 22 01:01:02 2021 +0530
completed merging of both window functionalities

commit 83fe70b2b5617d4dc3686d30cfd4b7bffbf5dfa6
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Sat Jun 19 00:27:12 2021 +0530
retained mscolab functionalities in separate file

commit 8660f5b27ccbde8a7b0b217b8c203b7911c30d65
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Mon Jun 14 13:59:02 2021 +0530
partial implementation of after_login in main window

commit 91c6b35809fecfc34330f59ca4cf7b63dc21cda7
Author: Aravind Murali<EMAIL_ADDRESS>Date:   Fri Jun 11 17:04:34 2021 +0530
implemented new connect window

updated.


Adding HAS_ROOT guard to source/srs/analysis/DataWriter.hpp

Hey Yanzhao,
for the build, having ROOT is optional (stated in README), however it doesn't compile.
There are HAS_ROOT guards missing in the header file:
source/srs/analysis/DataWriter.hpp
, akin to source/srs/analysis/DataWriter.cpp
Could you add the guard around
class RootFileSink; (line 11)
'std::unique_ptr root_file_;' (line 46)
Cheers,
-- Martin

Hi, Martin.
Thanks for this information. Yeah, I haven't test out compiling and running with ROOT, I will fix this and update it later today.

Hi, Martin
I have pushed the update to the dev branch and tested the program locally without ROOT dep.
I also updated the readme, where you can see an option to forcedly disable ROOT dep with a CMake option.
If you need something else, please let me know.


HelmDeploy: helm get manifest does not use configured namespace

Question, Bug, or Feature?
Type: Bug
Enter Task Name: HelmDeploy
This task runs helm get manifest [release name] before finishing. This fails when the release is installed to a namespace, as the command does not seem to include the configured namespace.
Environment
Azure Pipelines - team: aehrc, project: ontoserver, build: 20200710.12
Issue Description
Task name: Install Helm chart
Task logs
logs_39.zip
Error logs
Starting: Install Helm chart
==============================================================================
Task         : Package and deploy Helm charts
Description  : Deploy, configure, update a Kubernetes cluster in Azure Container Service by running helm commands
Version      : 0.171.3
Author       : Microsoft Corporation
Help         : https://aka.ms/azpipes-helm-tsg
==============================================================================
/usr/local/bin/helm upgrade --namespace atomio --install --set hostname=atomio.australiaeast.cloudapp.azure.com,image=ontoserver.azurecr.io/atomio:0.0.1-SNAPSHOT,storage=50Gi --wait --install atomio atomio
Release "atomio" has been upgraded. Happy Helming!
NAME: atomio
LAST DEPLOYED: Fri Jul 10 02:43:55 2020
NAMESPACE: atomio
STATUS: deployed
REVISION: 5
TEST SUITE: None
/usr/local/bin/helm get manifest atomio
Error: release: not found
Finishing: Install Helm chart

This issue has been addressed in this PR: https://github.com/microsoft/azure-pipelines-tasks/pull/13219
The rollout has started this week. cc: @shigupt202  to verify

Closing this duplicate issue.
Ref: https://github.com/microsoft/azure-pipelines-tasks/issues/13197