Theme selector on the client

This adds one of the 2 features mentioned in https://github.com/thelounge/lounge/issues/5#issuecomment-183358614, yup.
Beyond the obvious benefits from the user's perspective, this will help contributors/team members:

Test PRs with different themes: we have a history of breaking our own themes because we didn't test them right. No need to restart the server now, which may be a pain when testing content based on history.

Move the aesthetic choices (colors, sizes, margins/paddings, etc., everything non-system specific) outside of style.css: at the moment, users who want to set custom themes have to add the @import 'themes/<theme>.css'; hack into their custom CSS. Currently themes spend a lot of effort canceling out design choices set in style.css so that's fine, but as we make themes "independent", this hack will not reliably work anymore.

Results

Example
Crypto
Morning
Zenburn

Nice! Would it be possible to save the theme selection in the user config file?

I've had a quick look through the code, and I'll test it later, but just a quick point, can we not remove the default theme from the settings, because I have a feeling you'll get a flash on page load if you use a non default theme until Java script loads and chooses the theme. Also, I don't want to have to change theme on every machine. But I'll test it later.

can we not remove the default theme from the settings, because I have a feeling you'll get a flash on page load if you use a non default theme until Java script loads and chooses the theme

I haven't noticed such flash but I'll do some more research/testing. Note that the default config is not the right place to solve this if there is one, and there will be one whether or not I leave this in the defaults.

@williamboman: Nice! Would it be possible to save the theme selection in the user's config file?
@YaManicKill: Also, I don't want to have to change theme on every machine.

Actually, this is on purpose, for several reasons:

It's not unrealistic to use different themes whether you are on desktop or mobile. Actually, if there is a theme that's specifically targeted for Android and does it well but doesn't care about laptops, I'll take it! :-)
Custom CSS is not (yet) stored on the server, so it would be very bizarre to store one and not the other. Same with every other client settings, except for password change (because obviously) and custom highlights (because unread mechanisms happen on the server).
This will be more appropriate as part of #275 whenever we get to it.

So I'd strongly argue it should stay that way for consistency with the other settings and for usability purposes. Plus it really should not be an administrator decision to pick what background color you have to look at on your IRC client :D
The cost is to pick your theme on all devices you use, which is a very small cost overall.

Actually, this is on purpose, for several reasons:

Sorry, that's not what I mean.
I'm not suggesting that we have it sync to the server, that's a different issue (which as you say, should happen in #275)
However, if we have no "default" theme set on the user settings, which we use as the default for the browser, we aren't leaving that up to the user, we are enforcing the default to be the lounge default. Sure, I can change that on every machine that I use, and it will stay that theme once I change it on that machine, but we shouldn't take away the option of defining a default one in the first place.
All I'm asking is that we keep the setting for a user's default theme, set that as the default on the page. After that, everything else stays the same.

However, if we have no "default" theme set on the user settings, which we use as the default for the browser, we aren't leaving that up to the user, we are enforcing the default to be the lounge default. Sure, I can change that on every machine that I use, and it will stay that theme once I change it on that machine, but we shouldn't take away the option of defining a default one in the first place.

I actually have the opposite point of view on this and don't think we should do this.
First of all, with the addition of this PR, there is not much point in doing so. Actually, the only benefit would be not changing anything for the existing users and considering we only ship 4 themes and a major version, I really don't have an issue with that. If themes had been client-side from the beginning, making a default theme on the server-side would have never been an enhancement request we would have accepted :D
Now, on the longer term, picture us in a world where you can easily add themes to The Lounge. We'll want to ship it with at least one theme that you cannot remove. In terms of enhancements, it makes sense to have an official theme that we know works flawlessly with every new version, and other themes are separately maintained (maybe under the same organization, but still).
Anyway, I'm losing myself here. My point is: it's pointless to have the administrator-side make decision on what theme users will use, and it's even more pointless now that users can super easily change them.
On top of everything, that would be confusing and no other client settings have defaults set on the server.

it's pointless to have the administrator-side make decision on what theme users will use

It isn't pointless until we get #275 sorted, and settings can be synced. It's especially not pointless if you thin kthat the majority of people running the lounge are not running it on a hosted version run by someone else, but are running their own version. Then it's not "an administrator", but rather it is themselves.
I agree that it will be pointless once we have settings syncing, but until then we are removing a feature, which I highly disagree with.

In terms of enhancements, it makes sense to have an official theme that we know works flawlessly with every new version

Yeah, but we already have that. If a user doesn't set the default theme for themselves on the server, it is set as a default to be the default that we decide. But if we don't have an option on the server, the user will still have a default, but instead of set by them (or their administrator) it is set by us.

By the way, here is a copy of the flash I was talking about. Notice how it is white to start with, because it doesn't use the selected theme until Javascript is fully loaded and we have the user settings. If you have a server-side theme, then this doesn't happen as the theme is loaded from as soon as we get any html, so it always uses the right css. If you choose a different theme on 1 computer, then obviously you'll still get that flash, but there's no way around that, tbh.

I totally agree with @YaManicKill here. We should still keep the default theme config, and simply replace the loaded stylesheet when the user logs in, even if it wasn't for the flash. There are many reasons the theme should apply before the user logs in. This is also needed to set the default theme for public instances.

the majority of people running the lounge are not running it on a hosted version run by someone else, but are running their own version. Then it's not "an administrator", but rather it is themselves.

Well, that's a different story. This is not what The Lounge is, and making this assumption to build features means you are likely to misrepresent users that are not their own administrators.

I agree that it will be pointless once we have settings syncing, but until then we are removing a feature, which I highly disagree with.

Absolutely not. We are changing a feature, improving it even. And making it more consistent with all other user preferences.
And even when settings can sync, this is a different feature you are talking about, as someone's server-synced theme will only be applied after logging in, while you are defining the theme sent by the server before all loading.

If you have a server-side theme, then this doesn't happen as the theme is loaded from as soon as we get any html, so it always uses the right css.

This shows again that you are seeing The Lounge as a one user/administrator system.
So great, you set the default theme to the theme you want to use yourself so that you don't see a flash. Then all your users see a black to white flash if they use the example theme. Or the opposite.
As a matter of fact, users already see flashes on mobile: from the starting page to the loading page and from the loading page to the "themed" page.
Also, with theme being applied after authorizing (there is a bug unrelated to this PR at the moment, will fix), users see your default theme until they login. Very sloppy.
Also, that same flash is likely to happen if people set custom CSS, for example setting one that... changes the theme!
In a nutshell: I'm all for finding ways to reduce the flash issue, but having a default theme on the server configuration will not solve it. At best, it will hide for yourself in your very own configuration.
It seems to me that all your arguments are more personal preferences based on your usage of the app and not very objective.
Seriously, theme on the admin-side of things is not the right way to go. I don't see you complain about admin-side defaults for joins/quits, for custom CSS, for colored nicknames. All these have app defaults, pattern which this PR is applying for theme as well and makes a user decision consistent. Yes it's a change, but not one to be against really. In practice, setting it to your N devices will take you a couple minutes at worst, and then you're done. Forever maybe.
Honestly, I don't understand there is that much debate around a detail like that. I don't see it objectively evolve as your opinion is set, so I won't debate any further on this very point and focus on all the other things there is to do. If you're very much against this PR, please close it and do something better.
Open to feedback otherwise.

We should still keep the default theme config

... Because... ?

There are many reasons the theme should apply before the user logs in.

... Such as... ?

This is also needed to set the default theme for public instances.

Needed? No.
That the administrator wants to enforce a specific style, maybe, but that's a different story.

I re-added the default theme partly because @maxpoulin64 finally gave the first compelling argument on the subject (but also because I'm done fighting): if you own a bunch of services and you want to provide to your users The Lounge with a similar set of colors to others.
Very niche if you ask me but whatever, why not.
Note that I haven't tested changes around custom default theme, so feel free to test your use cases.

Does The Lounge need to be restarted for new themes to be picked up for the theme selector?

@vectr0n, with the theme selector on the client added by this PR, it doesn't need server restart (it does need restart on master currently). It doesn't even need reload, it changes when selecting it :-)

This shows again that you are seeing The Lounge as a one user/administrator system.
So great, you set the default theme to the theme you want to use yourself so that you don't see a flash. Then all your users see a black to white flash if they use the example theme. Or the opposite.

Huh, I had never noticed this was a global theme, rather than per-user. I would be happy with it moving to the user config.

I don't see you complain about admin-side defaults for joins/quits, for custom CSS, for colored nicknames.

No, because they are fundamentally different. When I set those settings on my client, they never change back. But with the theme, the default theme is set for milliseconds every single boot. The other ones having a different default only affects me when I go onto a new machine.
I am in favour of "inconsistency" because the theme works differently to everything else.
It drives me insane seeing that flash every single time I load the page.

Also, with theme being applied after authorizing (there is a bug unrelated to this PR at the moment, will fix), users see your default theme until they login. Very sloppy.

This shows again that you are seeing The Lounge as a one user/administrator system.
So great, you set the default theme to the theme you want to use yourself so that you don't see a flash. Then all your users see a black to white flash if they use the example theme. Or the opposite.

Huh, I had never noticed this was a global default, rather than per-user. I would be happy with it moving to the user config.

I don't see you complain about admin-side defaults for joins/quits, for custom CSS, for colored nicknames.

No, because they are fundamentally different. When I set those settings on my client, they never change back. But with the theme, the default theme is set for milliseconds every single boot. The other ones having a different default only affects me when I go onto a new machine.
I am in favour of "inconsistency" because the theme works differently to everything else.
It drives me insane seeing that flash every single time I load the page.

Also, with theme being applied after authorizing (there is a bug unrelated to this PR at the moment, will fix), users see your default theme until they login. Very sloppy.

Except that if we don't have this, then they will see ... the default theme until they login. The other solution is no different in this respect.

As a matter of fact, users already see flashes on mobile: from the starting page to the loading page and from the loading page to the "themed" page.

There is 1 flash from pressing the button to having the page actually load something, I accept that. However, it doesn't happen over refreshes (the default theme thing does) and there's (as far as I know) nothing we can do about that. Not ideal, but not as bad as the issue here.
I didn't want to continue arguing after you've fixed it, but I think you are being incredibly unfair with how you are putting my points across.
And the thing is, you are under the false presumption that us not having a default theme in the config means there is no default theme. This is fundamentally false. It just means that instead of the administrator deciding the default theme, we decide it, and they can't override it. The default is still there.

Looks good to me! The only think I notice is that the  box grey, which makes it looks like it's disabled. Maybe theme it the same as the other controls?

@maxpoulin64, very good point, I updated my PR accordingly. Also updated PR description to add screenshots for all themes, and rebased with master.
(Wow, AppVeyor finishing before Travis CI, in what world are we living...)

:+1:

Yeah, looks good. Great to get this feature finally in. :+1: and merging.
Oops, another feature in 2.0.0 without releasing :-P


Fix -Werror=format-security errors

On modern compiler, the build fails:
src/drawer.c: In function ‘draw_to_drawer’:
src/drawer.c:331:69: error: format not a string literal and no format arguments [-Werror=format-security]
331 |     mvwprintw(drawer->window, drawer->current_y, drawer->current_x, escape_buf);
|                                                                     ^~~~~~~~~~
Use a separate format string to fix the error and to protect us in case yle tries to send us % in texttv data.

Thank you for the pull request. Looks good to me :+1:


SlowOnDamage for all mobs now not just humans

About the PR
See title. Now carps won't be able to robust the entire crew as easily.
Applied it to basemob so this affects most animals and hostile space critters. Shouldn't affect species or silicone when we have that.
Screenshots

Changelog

:cl:

tweak: All mobs will now slow down when damaged enough. No more carps terrorising the crew as easy.

Doesn't scale to mob's health

Doesn't scale to mob's health

This was copied over from the human species so I'll have to take a closer look.

Just tested locally and works fine.


Missing info where to find the dependent "com.intellij.*" jars

Hi Matt, Please give some advises how to get the missing IntelliJ jars. Regards, Tom

Essentially, all IntelliJ jars, are supposed to be replaced with Eclipse imports. There's a lot of errors in the project, and I currently don't have the expertise to replace all of them.

So this isn't finished or working yet?

It is very, very, far from working or finished,

Could you please put that info in the README then, so people don't waste their time thinking this is a working plugin?

This project seems to be dead, but there is another plugin which works just fine:

Unofficial GitHub Copilot integration with Eclipse


I/O does not seem to work properly

Somehow, the output buffer does not seem to be properly flushed to the screen:
D:\github\grosu\k\k-distribution\tutorial\1_k\4_imp++\lesson_4>kompile imp.k
D:\github\grosu\k\k-distribution\tutorial\1_k\4_imp++\lesson_4>krun ..\lesson_1\tests\io.imp
10
Input two numbers: 20
Their sum is: 30
<T> <k> . </k> <env> x |-> 1 y |-> 2 </env> <store> 1 |-> 10 2 |-> 20 </store> <
in> ListItem ( #buffer ( "" ) ) ListItem ( "on" ) ListItem ( #istream ( 0 ) ) </
in> <out> ListItem ( #ostream ( 1 ) ) ListItem ( "on" ) ListItem ( #buffer ( ""
) ) </out> </T>
D:\github\grosu\k\k-distribution\tutorial\1_k\4_imp++\lesson_4>krun ..\lesson_1\
tests\sum-io.imp

10
Add numbers up to (<= 0 to quit)? 100
Sum = 55
Add numbers up to (<= 0 to quit)? 1000
Sum = 5050
Add numbers up to (<= 0 to quit)? 0
Sum = 500500
Add numbers up to (<= 0 to quit)? <T> <k> halt ; ~> while ( true ) { print ( "Ad
d numbers up to (<= 0 to quit)? " , .Ids ) ; n = read ( ) ; if ( n <= 0 ) { halt
 ; } else { s = 0 ; while ( ! ( n <= 0 ) ) { s = s + n ; n = n + -1 ; } print (
"Sum = " , ( s , ( "\n" , .Ids ) ) ) ; } } </k> <env> s |-> 2 n |-> 1 </env> <st
ore> 1 |-> 0 2 |-> 500500 </store> <in> ListItem ( #buffer ( "" ) ) ListItem ( "
on" ) ListItem ( #istream ( 0 ) ) </in> <out> ListItem ( #ostream ( 1 ) ) ListIt
em ( "on" ) ListItem ( #buffer ( "" ) ) </out> </T>
D:\github\grosu\k\k-distribution\tutorial\1_k\4_imp++\lesson_4>

I'm using Windows 10.

It works for my machine, Mac OS X 10.9. I suspect you didn't wait enough until it prints the outputs. I had to wait a few (1 or 2) seconds to see the prompt: Input two numbers: . It may be due to the recent slowdown of the java backend. Let me know if you still have the problem.
fix:lesson_4 daejunpark$ krun ../lesson_1/tests/io.imp
Input two numbers: 10 20
Their sum is: 30
<T> <k> . </k> <env> x |-> 1 y |-> 2 </env> <store> 1 |-> 10 2 |-> 20 </store> <in> ListItem ( #buffer ( "" ) ) ListItem ( "on" ) ListItem ( #istream ( 0 ) ) </in> <out> ListItem ( #ostream ( 1 ) ) ListItem ( "on" ) ListItem ( #buffer ( "" ) ) </out> </T>

fix:lesson_4 daejunpark$ krun ../lesson_1/tests/sum-io.imp
Add numbers up to (<= 0 to quit)? 10
Sum = 55
Add numbers up to (<= 0 to quit)? 100
Sum = 5050
Add numbers up to (<= 0 to quit)? 0
<T> <k> halt ; ~> while ( true ) { print ( "Add numbers up to (<= 0 to quit)? " , .Ids ) ; n = read ( ) ; if ( n <= 0 ) { halt ; } else { s = 0 ; while ( ! ( n <= 0 ) ) { s = s + n ; n = n + -1 ; } print ( "Sum = " , ( s , ( "\n" , .Ids ) ) ) ; } } </k> <env> s |-> 2 n |-> 1 </env> <store> 1 |-> 0 2 |-> 5050 </store> <in> ListItem ( #buffer ( "" ) ) ListItem ( "on" ) ListItem ( #istream ( 0 ) ) </in> <out> ListItem ( #ostream ( 1 ) ) ListItem ( "on" ) ListItem ( #buffer ( "" ) ) </out> </T>

That's not the problem.  I waited 1 min for io.imp.  Then I pushed a few ENTER.  Then I typed in a 10 followed by ENTER.  Then the text which was supposed to be printed at the beginning was printed immediately as soon as I hit ENTER.  Then I typed the second number.  Then the result was printed immediately:
D:\github\grosu\k\k-distribution\tutorial\1_k\4_imp++\lesson_4>krun ..\lesson_1\
tests\io.imp

10
Input two numbers: 20
Their sum is: 30
<T> <k> . </k> <env> x |-> 1 y |-> 2 </env> <store> 1 |-> 10 2 |-> 20 </store> <
in> ListItem ( #buffer ( "" ) ) ListItem ( "on" ) ListItem ( #istream ( 0 ) ) </
in> <out> ListItem ( #ostream ( 1 ) ) ListItem ( "on" ) ListItem ( #buffer ( ""
) ) </out> </T>
D:\github\grosu\k\k-distribution\tutorial\1_k\4_imp++\lesson_4>

In the VM which were assigned to us for CS427, this bug is not reproducible.
In the Ubuntu Linux, this is a bug and the fix is to replace the line 87 in imp.k "context print(HOLE:AExp, _);" with "context print(HOLE:AExp, AEs:AExps);
In Mac OSX, this is also a bug but the fix above does not work for OSX. The fix is adding a "[read]" at the end of the line 55.
We suspect this bug may relate to the underlying operating system.

@yeahyy, could you explain why it was failing before and why your changed fixed the issues?

Yes, this bug seems so deep with some problem with the underlying system. So what we have done is to format the imp.k file so that the bug won't appear in OSX system and Ubuntu system


Parse PubID with relationship (Reaffirmation)

A "Reaffirmation" PubID indicates that this standard was published but re-confirmed to be valid at some point. This means the document has not been updated since its last edition, but provides the latest date when it was re-confirmed.
ANSI C50.32-1976 and IEEE Std 117-1974 (Reaffirmed 1984) (Revision of IEEE Std 117-1956)
ANSI N42.13-2004 (Reaffirmation of ANSI N42.13-1986)
IEEE Std 218-1956 (Reaffirmed 1980, 56 IRE 28.S2)
IEEE Std 202-1954 (Reaffirmed 1972)
IEEE Std 792-1995 (Reaffirmation of IEEE Std 792-1988)

IEEE Std 218-1956 (Reaffirmed 1980, 56 IRE 28.S2)

What is "56 IRE 28.S2" here? Is it another PubID, not related to reaffirmation, right?

IRE is the institute of radio engineers, one of the predecessors of IEEE.
We had IRE PubIDs parsed already, right?

IRE is the institute of radio engineers, one of the predecessors of IEEE.
We had IRE PubIDs parsed already, right?

My question: why another PubID together with reaffirmation. If we follow that logic then we can build something like:
IEEE Std 792-1995 (Reaffirmed 1988, 56 IRE 28.S2, Supersedes ASTM E380-1974, Amendment to IEEE Std 802.11-2012, etc...)

What is "56 IRE 28.S2" here? Is it another PubID, not related to reaffirmation, right?

I misunderstood what you meant. Yes, I think you are right, i.e:

IEEE Std 218-1956 is a dual PubID (the new PubID) for 56 IRE 28.S2.
IEEE Std 218-1956 was reaffirmed in 1980.


Improvements for pointcloud_compute

Added '--use_cache', '--merge_points' and '--overwrite' options.
Added 'read_obj()' functions (see --use_cache option).
Made output more verbose.

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

LGTM, thanks!


mathjax 3 latex command underbrace gives js error (svg) or faulty output (chtml)

Issue Summary
When using tex->svg in Mathjax 3 the use of the latex command \underbrace results in a javascript error:
Unhandled Promise Rejection: TypeError: t.setAttribute is not a function.
   (In 't.setAttribute(Q,e)', 't.setAttribute' is undefined)

When using tex->chtml the equation is rendered but gives an ugly result:

This happens in Firefox 78.0.1, Safari 13.1.1 and Chrome 83.0.4103.116 tested on macOS 10.15.5
Steps to Reproduce:

write HTML as follows:

<!DOCtype html>
<html>
    <head>
        <title>Test underbrace</title>
    </head>
    <body>
        $$\underbrace{blablabla}_{bloe}$$
        <script type="text/javascript" id="MathJax-script" async
<EMAIL_ADDRESS>        </script>
    </body>
</html>

Open HTML-file in browser
Observe error in javascript-console
replace in HTML-file tex-mml-svg.js with tex-mml-chtml.js
Re-open HTML-file in browser
Observe output as included above

Technical details:

MathJax Version: 3
Client OS: Mac OS X 10.15.5
Browser: Firefox 78.0.1, Safari 13.1.1 and Chrome 83.0.4103.116

Supporting information:

no mathjax-configuration is used

The svg issue is a duplicate of #2408, which is in turn a duplicate of #2402.  There is a work-around in the latter that you may be able to use.
The CommonHTML issue is a known issue (though I don't think there is an issue tracker for it).  I hope to have a fix for it this summer, as we work on font issues.

Thank you very much for the speedy reply and fix!

The CommonHTML issue is a known issue (though I don't think there is an issue tracker for it). I hope to have a fix for it this summer, as we work on font issues.

Is it related to #661?

@eric-wieser, no, #661 is for v2, and the mechanism used for stretchy characters v3 is substantially different.  Also, #661 only affected the Asana-Math font, which is not yet available in v3.

Ah, ok. I'm still seeing the same problem, so I guess this hasn't been fixed yet:

Do you know of any other discussions about this elsewhere?

The cause seems to be the
mjx-stretchy-h > mjx-ext > mjx-c::before {
    transform: scalex(500);
}

If I change this to 200 the problem goes away.

We are preparing an alpha release that includes support for alternative fonts, and that includes a new font set for the default font.  Your example works for me with the current v3 in Safari 15.5 on Mac OS 12.4, as well as Firefox 104.0 and Chrome 104.0.5112.101.

and also with the new fonts.
Can you tell me the OS and browser versions that you are using?

I will try to update with exact versions tomorrow, but this was with Chrome on Windows 10.

Chrome is at Version 104.0.5112.102 (Official Build) (64-bit), and Windows 10 Pro is at version 1909.


Fix intersperse in Gen

The current implementation basically returns other in a very complicated way.
I presume that's not the intended behavior.

I think the intersperse method is correct. My test:
Gen<Integer> gen = Gen.of(0).intersperse(Gen.of(1));
Random rnd = new Random();
for (int i = 0; i < 10; i++) {
    System.out.print(gen.apply(rnd));
}

Output:
0101010101

as expected.

You're right, will update the description.
genSwitch = !genSwitch confused me...

yes, one of the rare embedded assignment usages - for me ok here.

@paplorinc the tests I added for that were rather minimal I confess. I wasn't quite sure the best way to cover them.  There is a test that checks the ordering bounces back and forth correctly tho.

Fixed in https://github.com/javaslang/javaslang/pull/1282


Add delta time to fx.update

Revolt FX is looknig nice, we're loving it and going to try it out. 👍
As a developer, it would be nice if we could have delta time as a parameter in the update function.
This is to take into consideration the PIXI.ticker.shared.speed variable.
We did this quick test to see that it works out.
  FX.prototype.update = function (delta) {         if (!this.active)             return;         var t = Date.now();         var dt = delta;         if (delta === undefined) {           dt = (t - this._timeElapsed);         }         dt *= 0.001;         var list = this._effects;         var node = list.first;         var next;         while (node) {             next = node.next;             node.update(dt);             node = next;         }         this._timeElapsed = t;     };

Good point! I've added support for delta time
(f8d8e7fd16452e38c6b6a2aa8e6d480546f82fdf)


[Feature] Add BetterDiscordApp installer to Homebrew cask repository

Describe the feature you'd like

I dislike how BetterDiscord is only available for download from the website so I propose that BD installer is added to Homebrew Cask repo in order to allow installation from the terminal
Additional context

I would be happy to do all the work and put BDI into Homebrew casks

I dislike how BetterDiscord is only available for download from the website

It is also possible to clone this repo and use the inject script

The BD Installer is a GUI so it won't allow installation from the terminal even if you add it to Homebrew.
As mentioned, you can clone the repo and run npm inject to install.
Or, if you want an actual manager, you can fork betterdiscordctl and adapt it for MacOS.

The BD Installer is a GUI so it won't allow installation from the terminal even if you add it to Homebrew.
As mentioned, you can clone the repo and run npm inject to install.
Or, if you want an actual manager, you can fork betterdiscordctl and adapt it for MacOS.

You can actually install GUI apps from Homebrew via brew cask, it unpacks the dmg and allows for it to be run from the Application folder or whatever

You can install and launch the installer from the terminal but you can't use it from the terminal.
If you're OK with that, no one's stopping you from creating and maintaining the cask yourself.


Error index_not_found_exception

Hi,
When I run a search with a controller I have this error displayed in the browser:
Missing404Exception in Connection.php line 572: {"error":{"root_cause":[{"type":"index_not_found_exception","reason":"no such index","resource.type":"index_or_alias","resource.id":"articles","index_uuid":"na","index":"articles"}],"type":"index_not_found_exception","reason":"no such index","resource.type":"index_or_alias","resource.id":"articles","index_uuid":"na","index":"articles"},"status":404}
Could you help me, please. Thanks in advance.

any ideas ?

Check the index articles exist or not, if not, create it and it should work.

I have same error
please guide me how to fix it


How to merge async function config @expo/metro-config with @ui-kitten/metro-config?

I am trying to merge react-native-svg-transformer in an expo bare workflow environment. The docs state that I should merge the following config with whatever I have:
const { getDefaultConfig } = require("@expo/metro-config");

module.exports = (async () => {
  const {
    resolver: { sourceExts, assetExts }
  } = await getDefaultConfig(__dirname);
  return {
    transformer: {
      babelTransformerPath: require.resolve("react-native-svg-transformer")
    },
    resolver: {
      assetExts: assetExts.filter(ext => ext !== "svg"),
      sourceExts: [...sourceExts, "svg"]
    }
  };
})();

@ui-kitten/metro-config will not accept an async function as a parameter (as far as I can see, it uses lodash.merge, which is a copy of Object.assign()). Also, UI Kitten requires @ui-kitten/metro-config to run in order to apply any cusom mapping. Not running it will simply fall back to not optmizing the theme config at bundle time.
I am very unsure how to merge that config using create from @ui-kitten/metro-config.
For now I am using the following:
const UIKittenMetroConfig = require('@ui-kitten/metro-config');
const { getDefaultConfig } = require("@expo/metro-config");

module.exports = (async () => {
	const {
		resolver: { sourceExts, assetExts }
	} = await getDefaultConfig(__dirname);

	const uiKittenConfig = UIKittenMetroConfig.create({
		evaPackage: "@eva-design/eva",
		customMappingPath: "./theme/mapping.json",
	});

	return {
		transformer: {
			assetPlugins: ['expo-asset/tools/hashAssetFiles'],
			babelTransformerPath: require.resolve("react-native-svg-transformer")
		},
		resolver: {
			assetExts: assetExts.filter(ext => ext !== "svg"),
			sourceExts: [...sourceExts, "svg"]
		},
		...uiKittenConfig
	};
})();

I can see the message success Successfully bootstrapped @eva-design/eva, but I am not sure whether this is the proper way of doing it. I can see that the .create() call returns an object with a reporter field, a general field in the metro config options.
@artyorsh Can you please point me the right way ?

👋 there.
I've never tried working with async module export to be honest.
But you're doing everything right while you see this Successfully bootstrapped @eva-design/eva message 😄
Since you're awaiting for the default config, you're still able to merge it according to the docs, right?
module.exports = (async () => {
	const { resolver: { sourceExts, assetExts } } = await getDefaultConfig(__dirname);

	return UIKittenMetroConfig.create({
		evaPackage: "@eva-design/eva",
		customMappingPath: "./theme/mapping.json",
	}, {
		transformer: {
			assetPlugins: ['expo-asset/tools/hashAssetFiles'],
			babelTransformerPath: require.resolve("react-native-svg-transformer")
		},
		resolver: {
			assetExts: assetExts.filter(ext => ext !== "svg"),
			sourceExts: [...sourceExts, "svg"]
		},
	});
})();

Since you're awaiting for the default config, you're still able to merge it according to the docs, right?
I thought I would, but apparently not. When I tried to run the config as you posted, the message Successfully bootstrapped @eva-design/eva did not appear.

And I say "did" because today I don't get that message anymore, no matter what config I use or if I clean the react native cache. Since yesterday I have installed some libs and did changes back and forth, I guess I've broken something... I will try this on a new project from scratch and let you know if it works. In the meantime, thank you :)

Nevermind, might have been just an NPM/yarn cache thingy, deleting node_modules and installing packages again seems to do the trick.
Apparently, @artyorsh, your suggestion, as well as my original code, works!
I guess the way you suggested is the more appropriate way though, so I'm going for it.
Thank you again :)


add bash completion to bonuses

adds bash completion, gets automatically installed at end with lnchannels & wallet before restart.

makes total sense :) thx


404 page missing

As for now, there will be no special layout for the 404. It can be created like any single page, which is up to the content provider.


[dv/lc_ctrl] lc_ctrl post M2.5 TODOs

Description

[x] This PR adds a new register: https://github.com/lowRISC/opentitan/pull/18302 ext_clk_switched.
Check this in block level.
Milestone: M3 because chip level already checked it.

[ ] Volatile unlock enhancement 1). check otp program output won't change if it is volatile_raw_unlock transition.

[ ] Volatile unlock enhancement 2). load otp states to rand states rather than just raw state.

[ ] Volatile unlock enhancement 3). Support enable scb for volatile_raw_unlock.

The following DV updates are needed:

Split the DV environment at the block-level so that we test both variants of the LC_CTRL (one with volatile unlock enabled, the other one with volatile unlock disabled)
Make the TLT parametric so that we can test both the enabled and disabled case.

This is a V3 issue, so probably doesn't belong in the M5 milestone. Marking as triage priority accordingly.

Moving to M7, as it's a V3 issue.

Moving to backlog as all remaining tasks in this issue are related to volatile unlock, which is not a production feature for Earlgrey. Please feel free to move to the next release of Earlgrey if it makes sense to cover these opens as part of V3.


Print error when command fails

Small one, I just realised that this doesn't actually give you any info on the failure.
I had a YAML error and it was masked from me.

LGTM


Selection issue with custom titlebar

Issue Type: Bug

Set window title style to custom.
Open any file.
Start selecting text from middle and extend selection to any menu on custom title bar and leave mouse button.
Now without clicking anywhere move mouse on the text area.

Now your selection changes only with mouse move without any click.
VS Code version: Code 1.27.2 (f46c4c469d6e6d8c46f268d1553c5dc4b475840f, 2018-09-12T16:17:45.060Z)
OS version: Windows_NT x64 10.0.17134
Extensions (9)

Extension
Author (truncated)
Version

vue-format
feb
0.1.2

auto-rename-tag
for
0.0.15

beautify
Hoo
1.4.3

sublime-babel-vscode
jos
0.2.10

vscode-icon-theme
jtl
1.6.2

debugger-for-chrome
msj
4.10.1

vetur
oct
0.12.6

material-icon-theme
PKi
3.6.0

code-spell-checker
str
1.6.10

(1 theme extensions excluded)

duplicate #58446


fixed #31

Results are neatly organized in table format
Description
Results are neatly organized in table format.
Motivation
Would look good
Fixes # (issue)
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] Package installs: julia> ]dev .
[ ] All tests pass: julia> ]test ChaoticEncryption
[ ] The documentation builds: $ cd docs and then $ julia make.jl

Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Acknowledgements
This Pull Request template was taken from the excellent PyBaMM repository.

@all-contributors please add @Muthukumar-Muthu and @Saransh-cpp for docs

@all-contributors please add @Muthukumar-Muthu for docs

@all-contributors please add @Saransh-cpp for docs


need data export

We will need a data structure to hold data as game progresses, which can eventually be used to deploy at expfactory.org

step 1: local export of data from data structure upon completion of game
step 2: integration with expfactory.org (meaning the expfactory.js object that uses backbone.js)

will close https://github.com/expfactory/expfactory-python/issues/91

so I believe this is ready to be hooked up with expfactory. There are two steps here:

On each trial, data is sent to the  save function in Run.js. We should make this function a standard for all games. Save then calls the input/sendData functions in..

saver.js which creates a dict from the data and then gives it to expfactory.js. This file is necessary because of the way Phaser handles states. We can't send data back to our template.html (which would be preferable) because it initializes the game state and communication is one-directional unless we utilize local memory, but this wouldn't update on a trial-by-trial basis. So saver exists outside of the state changes and can pass our data to expfactory.

Let me know if this setup won't work; it's pretty similar to what we were doing before but with the expfactory object added in. The main thing is just that it will require saver.js to be an additional dependency for every game.

This looks great! The save function would be detailed in docs to make new games, and saver looks like it can be a standard django template. Is the idea that we just need saver.js embedded in the document on ready function before the game is defined?
I finished most of thesis stuffs  yesterday - I need to read it through once more before sending out to committee, but I'll definitely be able to work on this soon. I'm pretty excited about it. :)

Yep! Just need it defined before Run.js.
And congrats! You're so close :smile:

@boomsbloom I'm almost finished getting games integrated - the last thing I need is some kind of variable in the trial (or somewhere) to indicate that the experiment is finished, that way I can notify the server to respond with a redirect to the next experiment (instead of just sending data back).

should be able to work on this in like an hour. Ended up having to spend a
bunch of time yesterday/this morning on a different project but almost
finished there
On Fri, Apr 1, 2016 at 12:32 PM, Vanessa Sochat<EMAIL_ADDRESS>wrote:

@boomsbloom https://github.com/boomsbloom I'm almost finished getting
games integrated - the last thing I need is some kind of variable in the
trial (or somewhere) to indicate that the experiment is finished, that way
I can notify the server to respond with a redirect to the next experiment
(instead of just sending data back).
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/expfactory/expfactory-games/issues/1#issuecomment-204536903

ok, awesome! Yeah I think we are going to have these up and running on mturk really soon :)  And no rush, it's Friday! :D

This is finished, will need testing, but closing issue!


Use single quotes in raw query.

Fixes migration under postgres.

Context: migration 0007 works fine with sqlite, but the double quotes throw off postgres:
django.db.utils.ProgrammingError: column ":" does not exist
LINE 1: UPDATE regcore_layer SET reference=version||":"||label

This version works under both.

Ooof; sorry about that. I can never remember which quote style to use in SQL.


Getting Double Users

I get double users and double prefixes on the users.

Update LuckPerms. It has been fixed: https://luckperms.net/


Add return to example in porting.md

Since we have the closing bracket there, adding the return statement that should be before this bracket.
I can also remove the closing bracket if you prefer.

Thanks for the fix!


Dataframe with datetime64[ns, UTC] is not ingestible

I have a dataframe that I've coerced to have a datetime that osc_ingest_trino understands:
sankey_data = [
['direct emitters', 'parent_attribution', 1],
['suppliers', 'parent_attribution', 2],
['parent_attribution', 'CO2e', 3],
['parent_attribution', 'LEI', 4],
['sec sub', '10-k', 5],
['sec sub', 'LEI', 6],
['sec sub', 'SIC', 7],
['sec num', '10-k', 8],
['10-k', 'revenue', 9],
['revenue', 'intensity', 0],
['CO2e', 'intensity', 1],
['intensity', 'company intensity', 2],
['intensity', 'sector intensity', 3],
['LEI', 'company intensity', 4],
['SIC', 'sector intensity', 5],
]
df = pd.DataFrame(data=sankey_data, columns=('source', 'target', 'qty'))
df['year'] = pd.to_datetime("2020-01-01")
df = df.convert_dtypes()
df.year = df.year.astype('datetime64[ns, UTC]')
I get this error message:
TrinoUserError: TrinoUserError(type=USER_ERROR, name=NOT_SUPPORTED, message="Incorrect timestamp precision for timestamp(6); the configured precision is MILLISECONDS", query_id=20211208_224842_00491_kx99a)
I get a different error message if don't do the manual conversion:
ValueError: unexpected pandas column type 'datetime64[ns]'
I'd like to be able to create a dataframe that contains date values.

Out of curiosity, have you tried giving create_table_schema_pairs an override to use timestamp(3) ?

Have not.  I have in the past manually copied in that code and made a change, but haven't tried override syntax.  Will have a look at that.
Michael

From: Erik Erlandson @.***>
Sent: Thursday, December 9, 2021 9:30:13 AM
To: os-climate/osc-ingest-tools
Cc: Michael Tiemann; Author
Subject: Re: [os-climate/osc-ingest-tools] Dataframe with datetime64[ns, UTC] is not ingestible (Issue #7)
Out of curiosity, have you tried giving create_table_schema_pairs an override to use timestamp(3) ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://url.emailprotection.link/?blqTwYSLN378y1PV1GhzraV61JvyZZT-6WpI-NXN539m68rlmEHFiTo7CD779urOCB-2Y3OMz_hHdkpE7CnQ2YVbsddz4ZL8iVM8EJmuQpx_6y3HP7kqdOhhJEkXsMoXo, or unsubscribehttps://url.emailprotection.link/?bfETTtMWAFlaRxu8kKY6eB6iJ3gAaDHchYUiuSX6K-ExINGQnK2kEt52IEJ1-zygQmxinP7-3ILOEwyDVSxTMZJ9dQXk-DjaftSexx7wftgx3iJyXZV6qvi_-pMi-236Mwxs57tjReZSHtFn0iaeak9zli8bJPurX2y3eGwLFS6I~.
Triage notifications on the go with GitHub Mobile for iOShttps://url.emailprotection.link/?b93vvrxxFMiIL3aaEZeT1FEDo4ff1nIm3S93M74HChe6oMsjvQxYfvy0INjYS7u2FMe9ddTMaKP63-6NfzGjH_fVMKWZ4WGY8V0YwPxZVTfVo1FS4ve04s9qZpZ4awAgjKFcwCo9eE9NJvenJ6TkrGN_4GTeA60Dx2SZ1BScd4_k~ or Android<https://url.emailprotection.link/?bwU-B7o1UGBPzHAoKA5hX-YU6nt6R47hh3_8HMPKW08nZ4tm-rkZdekFih8
%20adSaIlolX1K6F60Z32RgmcvBYWzPaWnxjzogEdPUobJCj1y-4sZyT_B-NChG2_1GZYBDzw8PDJcos3uXLAK_Haiwidbp-UKkyLlCXKM68K2qknmejC6ePwEZUXvno9Z8XOf5JcrvFAo5kMKScW99QAMxuSjg~~>.

unmanaged_parquet_tabledef interface does not allow me to override the typemap.  For now I've made a local copy of that function where I can add my own typemap.  But this is not a good long-term fix.


Postfix 3.6.2

Motivation for this change
I new minor release has been published: http://www.postfix.org/announcements/postfix-3.6.2.html

I updated the source code url because the current one didn't work.
As a nixos-mailserver maintainer, I added myself as postfix maintainer to better follow nixpkgs updates.

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test postfix

@nlewo please resolve the merge conflict.

@SuperSandro2000 conflicts are resolved.

I updated the source code url because the current one didn't work.

We should add mirror://postfix.


Vision & research section

Make it a bit nicer in place on Foundation page or put on its own page.

leave in place for now, still waiting for a proper text


Update Readme

Quick updates to Readme to reflect the changes appropriate (FLFGC, no longer archived, new URLs, etc.)

Initial updates completed.


Duplicates plugin errors out on merge attempt

First we find several duplicates:
> beet dup -Fp Hazardous Dubs
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\02 Shyne Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\02 Shyne Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\04 Murderer Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\04 Murderer Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\03 Pundjabi Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\03 Pundjabi Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\05 The Anthem.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\05 The Anthem.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\06 Tribes.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\06 Tribes.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\01 No Freaks.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\01 No Freaks.mp3

Then we try to merge them (and I hope beets does the right thing):
> beet dup -M Hazardous Dubs
Traceback (most recent call last):
  File "C:\tools\python2\Scripts\beet-script.py", line 9, in <module>
    load_entry_point('beets==1.3.15', 'console_scripts', 'beet')()
  File "C:\tools\python2\lib\site-packages\beets\ui\__init__.py", line 1163, in main
    _raw_main(args)
  File "C:\tools\python2\lib\site-packages\beets\ui\__init__.py", line 1153, in _raw_main
    subcommand.func(lib, suboptions, subargs)
  File "C:\tools\python2\lib\site-packages\beetsplug\duplicates.py", line 153, in _dup
    merge=merge):
  File "C:\tools\python2\lib\site-packages\beetsplug\duplicates.py", line 329, in _duplicates
    objs = self._merge(objs)
  File "C:\tools\python2\lib\site-packages\beetsplug\duplicates.py", line 316, in _merge
    objs = self._merge_items(objs)
  File "C:\tools\python2\lib\site-packages\beetsplug\duplicates.py", line 274, in _merge_items
    fields = [f for sublist in Item.get_fields() for f in sublist]
AttributeError: type object 'Item' has no attribute 'get_fields'

I don't know what actually should have happened - I've not used beets to merge duplicates before

Thanks! I believe I've fixed this. Could you please try out the fix and see if it works?

OK, now the plugin doesn't crash. But it doesn't merge anything either:
> beet dup -Fp Hazardous Dubs
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\02 Shyne Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\02 Shyne Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\04 Murderer Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\04 Murderer Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\03 Pundjabi Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\03 Pundjabi Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\05 The Anthem.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\05 The Anthem.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\06 Tribes.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\06 Tribes.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\01 No Freaks.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\01 No Freaks.mp3

> beet dup -M Hazardous Dubs
6blocc - Hazardous Dubs EP Part 3 - Shyne Dub
6blocc - Hazardous Dubs EP Part 3 - Murderer Dub
6blocc - Hazardous Dubs EP Part 3 - Pundjabi Dub
6blocc - Hazardous Dubs EP Part 3 - The Anthem
6blocc - Hazardous Dubs EP Part 3 - Tribes
6blocc - Hazardous Dubs EP Part 3 - No Freaks

And the duplicate tracks are still there:
> beet dup -Fp Hazardous Dubs
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\02 Shyne Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\02 Shyne Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\04 Murderer Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\04 Murderer Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\03 Pundjabi Dub.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\03 Pundjabi Dub.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\05 The Anthem.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\05 The Anthem.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\06 Tribes.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\06 Tribes.mp3
G:\Music\Music\6blocc\Hazardous Dubs EP Part 3\01 No Freaks.mp3
G:\Music\Music\6Blocc\Hazardous Dubs EP Part 3\01 No Freaks.mp3

Why does this happen and what was supposed to happen?

Good question. I'll need to defer to someone more familiar with the duplicates plugin. But if you're able to nail down exactly what's supposed to happen, then yes, please do file another bug.

Hi @sampsyo, I guess this would be me. Sorry about that @m0nkey-br4in. Would it be possible for you to send me your beets library file, so I can replicate what's happening?

@m0nkey-br4in: nevermind my earlier request. The plugin is, I think, doing what it should, although perhaps the documentation for the merge option is not clear. See the examples involving merging at https://beets.readthedocs.org/en/latest/plugins/duplicates.html.
Essentially, running --merge on tracks simply copies missing fields from one track to another. It needs to be paired with some other strategy for actually removing the duplicates. For example, --delete, or --move.


Get Labels API doesn't have pagination but pyapi getlabels does

Hi,
According to the GitLab docs, Get/List Labels does not have any pagination. However, this API does and has the page and per_page params. GitLab seems to ignore the extra params and keeps returning the full list of labels. So when getlabels is called with page=2, it still returns the full list,  even with per_page=1.
Note that in a GitLab project Labels web page, it does paginate by 20 but via the API, it remains un-paginated and returns the whole list.

Good catch, probably from the old api docs? Ill remove it.

Fixed on https://github.com/Itxaka/pyapi-gitlab/commit/241c32e3e00af6b0ace3b6e86a46d406733ad46f


Add pyright config, fix data.py

…ters/angeris_cvxpy/data.py:275:20 - error: Operator ">" not supported for "None" (reportOptionalOperand)
/home/legion/Documents/ComposableFi/cvm/mantis/simulation/routers/angeris_cvxpy/data.py:275:38 - error: Operator ">" not supported for "None" (reportOptionalOperand)
/home/legion/Documents/ComposableFi/cvm/mantis/simulation/routers/angeris_cvxpy/data.py:277:25 - error: Operator ">" not supported for "None" (reportOptionalOperand)
/home/legion/Documents/ComposableFi/cvm/mantis/simulation/routers/angeris_cvxpy/data.py:277:44 - error: Operator ">" not supported for "None" (reportOptionalOperand)

@Alliad may be start from smaller file? I feel like data.py may be to start? or some oracle.py


Unable to create or delete NSG rules in the latest version of Azure CLI

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

Describe the bug

In the above command, I am passing an array of strings read from a csv file in which they are stored as space separated values. This command was working fine till azure CLI version 2.39 but it fails for CLI version 2.40 and above. The error says that "The template language function 'int' cannot convert provided value '' to integer value". We are facing the same issue while issuing a delete command for NSG rules also. Attaching screenshots for your reference.
To Reproduce

Expected behavior

When I issue the command, I need the NSG rule to be created in my NSG according to the parameters given.
Environment summary

I am running the script on microsoft hosted agents on Azure Devops Pipeline.
Additional context

The error seems to be occurring due to some interference from our policy assignment. (The policy is in place for more than two years and it didn't interfere with the command in earlier CLI versions)

route to CXP team

@VishalShindeTR Thanks for your screenshot, and could you provide which specific command (with parameters) in the scripts is failing?


[December 15] Regenerate SMCE credentials

The credentials of the deployer on SMCE clusters must be renewed every ~55 days.
Context
AWS credentials for our hub deployer must be renewed for the following clusters:

[ ] nasa-veda
[ ] nasa-ghg
[ ] maap

Resources
Follow the instructions on how to do this.
Definition of Done

[ ] There is a deadline for this task
[ ] The credentials of the deployer on all 3 clusters have been renewed

Yuhuu, this time it worked 🎉
Let's see if we can dinamically add start date and/or end date for these issues


Getting error while trying to login to rail app.

Hi,
I have deployed supergraph, postgres and rails app using https://github.com/dosco/super-graph/blob/master/docker-compose.yml but when I try to sign up rail app getting error. Can you please let me know the default login for this rails app.

ActiveRecord::NotNullViolation in Devise::RegistrationsController#create
PG::NotNullViolation: ERROR: null value in column "full_name" violates not-null constraint DETAIL: Failing row contains (37, null, null, null<EMAIL_ADDRESS>$2a$11$z6h2KUZYGLJz3Otr3BBtoOm8ID.LNPM6bx9qn.zZ12dyOx/EbspMC, null, null, null, 2020-06-15 14:55:10.488498, 2020-06-15 14:55:10.488498).

https://supergraph.dev/docs/intro#try-the-demo-app
# signin to the demo app<EMAIL_ADDRESS>/ 123456)


ListBox - System.StackOverflowException: 'Exception_WasThrown'

Describe the bug
When ListBox contains some buttons in DataTemplate application fail with an StackOverflowException.
To Reproduce
See attached project.
ListBox Exception.zip

Execute and fail
Remove only TextBlocks from DataTemplate and fails
Remove all Buttons from DataTemplate and it works

Expected behavior
It should work
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 1903
Version 0.10.14 - latest

Additional context
Add any other context about the problem here.

You got a combination of these two issues:
https://github.com/AvaloniaUI/Avalonia/issues/724
https://github.com/AvaloniaUI/Avalonia/issues/5726
Basically, DataType is required on a DataTemplate, otherwise it will be applied on everything (thus, stack overflow). That's by design.
CompiledBindings use x:DataType property from a different namespace. And while they should be the same, they are two different properties. That's the issue.

I will close it as a duplicate of https://github.com/AvaloniaUI/Avalonia/issues/5726
As a workaround, you need to use both properties or even better use <ListBox.ItemTemplate>. <ListBox.DataTempates> is useful, when you want to have two or more different templates for different types.


format code with goreturns and allow to process . in format fields

Quick fix to allow to use format fields with . in it (when you have more complex json documents). Only 1 dimension is supported now, but this can be probably refactored to use recursion.
I did not mean to do a full goreturns of the elktail.go, but this automatically occurs when I save the file.

Thank you for the pull request. I reviewed your changes and decided to implement the nested fields a bit differently, mainly to support multiple levels of nesting and to be inline with additional features I plan to add for output formatting. Please take a look at the latest version of the master and let me know if it works for your workflow. It should support multiple levels of nesting.

Yeah, your solution is much better. Thanks for implementing!


vpc sdk public visibility of sdk v1.0.1 is confusing

Initialize this main.go program in an empty directory
vpc $ cat main.go
package main

import (
	"github.com/IBM/go-sdk-core/v5/core"
	"github.com/IBM/vpc-go-sdk/vpcv1"
)

var authenticator = &core.IamAuthenticator{
	ApiKey: "{api_key}",
}

func Get(service *vpcv1.VpcV1) {
	service.GetVPC(service.NewGetVPCOptions("hh"))
}

func main() {
}
vpc $ go mod init powell.com/vpc
go: creating new go.mod: module powell.com/vpc
go: to add module requirements and sums:
	go mod tidy
vpc $ go mod tidy
go: finding module for package github.com/IBM/go-sdk-core/v5/core
go: finding module for package github.com/IBM/vpc-go-sdk/vpcv1
go: found github.com/IBM/go-sdk-core/v5/core in github.com/IBM/go-sdk-core/v5 v5.9.5
go: found github.com/IBM/vpc-go-sdk/vpcv1 in github.com/IBM/vpc-go-sdk v1.0.1

The version of vpc-go-sdk is v1.0.1 - which isn’t the latest: v.0.18
This is confusing.
slack conversation: https://ibm-cloudplatform.slack.com/archives/CHY2EP4JJ/p1648582987813669
I see a year old reference to similar issue: https://ibm-cloudplatform.slack.com/archives/CHY2EP4JJ/p1620154108015500?thread_ts=1619789261.012400&cid=CHY2EP4JJ

Retracted the v1* versions


Ensure containerID and sandboxID fields logged as early as possible

Improve containerID and sandboxID logging by:

Ensuring that setExternalLoggers() saves any previously-set log fields
(this required a SetLogger() API change).

Calling setExternalLoggers() as early as possible, then again once new fields become available.
This is necessary to ensure that all virtcontainers functions atleast have access to the containerID for logging purposes.

PR also performs various log cleanup.
Fixes #519.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

I too wondered if the [1] might get trimmed - maybe when it merges. If it were a kernel thing, then that is exactly what you do for text you don't want to end up in the commit iirc (that or very similar ;-) )

@egernst - fixed.

Rebased!

CI's are green. Ignoring the openstack failing CI + tiny codecov change (0.01% :)...


cap pycodestyle

cap pycodestyle
See https://github.com/PyCQA/pycodestyle/issues/741
cc @joshmoore
https://travis-ci.org/jburel/omero-parade/builds/365548744 is green now using that branch

Seems sensible enough, and is certainly worth a try. Merging and relaunching omero-parade.


AlpineLinux: Update version number

Alpine Linux has been updated to 3.8.1! This PR doesn't actually change how Alpine is booted (netboot.xyz already boots the latest 3.8 version automatically), but it updates the version number that the user sees to avoid confusion.

Thanks!


Error while running playbook

Following error is thrown when running playbook
FAILED! => {"failed": true, "msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute u'service'\n\nThe error appears to have been in '/jtyr.stunnel/tasks/main.yaml': line 30, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n  with_dict: \"{{ stunnel_service_script }}\"\n- name: Install Stunnel service script\n  ^ here\n\nexception type: <class 'ansible.errors.AnsibleUndefinedVariable'>\nexception: 'dict object' has no attribute u'service'"}

Fix is to replace the code with following code
- name: Install Stunnel service script
  copy:
    src: "{{ item.value.src}}"
    dest: "{{ item.value.dest}}"
    mode: "{{ item.value.mode | default('0644') }}"
  when: >
    stunnel_service_install
  with_dict: "{{ stunnel_service_script }}"
  tags:
    - stunnel_service

Thanks for opening this issue. The core should work as it is. You just needs to gather fats from the from the remote machine in order to have the ansible_service_mgr variable available:
- name: My play
  hosts: all
  # This should not be necessary as the facts gathering is enabled by default
  gather_facts: yes
  roles:
    - stunnel

Just so that you know. I already tried that and also defined this variable manually and it didn't work in either case.


Fixed pre_cls_weight

Hi！ Thanks for your great work, I have a small question about the fixed pre_cls classifier weight. Why the weight of pre_cls in the classifier need fixed during the current step?

Hi! Thanks for your question. The main reason is that fixing it will enhance the stability of the method. This is equivalent to adjusting the learning rate on the classifiers. I mentioned this step in the implementation details in the supplementary material.


SecurityGroup removes SecurityGroupRules added later

What happened?
We create a security group and 2 rules added in another component. On fresh deploy  Pulumi sets it all up correctly, but then on the next up it resets the security group to the ingress and egress specified only on the groups component itself and does not realize there are other rules attached since those aws.ec2.SecurityGroupRules are not changing.
Expected Behavior
I would expect that pulumi would understand that the security group rules are attached to a security group managed in the deployment and ensure they are still configured on the security group
Steps to reproduce
include this code in the index.ts

run pulumi up
run pulumi up again
check the aws console and notice the 2  added ingress rules no longer exist

const sg = new aws.ec2.SecurityGroup(sgName, {
            name: sgName,
            vpcId: vpc.id,
            egress: [
                {
                    cidrBlocks: ['<IP_ADDRESS>/0'],
                    description: 'Allows all outbound IPv4 traffic.',
                    fromPort: 0,
                    protocol: '-1',
                    toPort: 0,
                },
            ],
            ingress: [
                {
                    description:
                        'Allows inbound traffic from network interfaces and instances that are assigned to the same security group.',
                    fromPort: 0,
                    protocol: '-1',
                    self: true,
                    toPort: 0,
                },
                {
                    cidrBlocks: ['<IP_ADDRESS>/0'],
                    fromPort: 9443,
                    protocol: 'TCP',
                    self: true,
                    toPort: 9443,
                },
            ],
        })

        pulumi.output(vpc.privateSubnets).apply((ps) => {
            const cidrBlocks: any = ps.map((subnet) => subnet.subnet.cidrBlock)
            new aws.ec2.SecurityGroupRule(`-private-ingress`, {
                type: 'ingress',
                cidrBlocks: cidrBlocks,
                fromPort: 0,
                protocol: '-1',
                toPort: 0,
                securityGroupId: sg.id,
            })
        })

        pulumi.output(vpc.publicSubnets).apply((ps) => {
            const cidrBlocks: any = ps.map((subnet) => subnet.subnet.cidrBlock)
            new aws.ec2.SecurityGroupRule(`public-ingress`, {
                type: 'ingress',
                cidrBlocks: cidrBlocks,
                fromPort: 0,
                protocol: '-1',
                toPort: 0,
                securityGroupId: sg.id,
            })
        })

Output of pulumi about
CLI
Version      3.52.1
Go Version   go1.19.5
Go Compiler  gc
Plugins
NAME                     VERSION
aws                      5.29.1
aws                      5.10.0
command                  0.5.2
docker                   3.6.1
eks                      0.42.7
kubernetes               3.24.0
kubernetes               3.20.2
kubernetes-cert-manager  0.0.3
nodejs                   unknown
Host
OS       darwin
Version  12.5
Arch     arm64
This project is written in nodejs: executable='/opt/homebrew/bin/node' version='v19.7.0'
Dependencies:
NAME                             VERSION
@pulumi/kubernetesx              0.1.6
@pulumi/pulumi                   3.55.0
@types/multimap                  1.1.2
@types/uuid                      8.3.4
@pulumi/aws                      5.29.1
@pulumi/awsx                     0.40.1
@pulumi/command                  0.5.2
@pulumi/eks                      0.42.7
@pulumi/kubernetes-cert-manager  0.0.3
requestretry                     7.1.0
multimap                         1.1.0
@pulumi/kubernetes               3.24.0
axios                            0.27.2
cdk8s-cli                        2.1.135
@types/node                      16.18.12
patch-package                    6.5.1
simple-sha256                    1.1.0
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @jhsinger-klotho thanks for the detail of your issue. Unfortunately, managing a single resource declaritively from two separate sources does lead to these kinds of issues. I'm not aware of a resource that allows you to manage the ingress rules independently of a security group - are you modifying the security group resource directly to achive this separation?
If you really can't centralise the definition of your security group to include all ingress rules there's a two step process you could try as as workaround to the rules being removed:

Refresh the security group before each deployment. This can also be done by using the -r option when calling pulumi up.
Set the ingress and egress properties as ignored using resource options. This will then use the value from your state (which you refreshed in step 1) instead of the value in your program.

I'm going to close this as "won't fix" as this is expected behaviour - resetting a resource back to the declared state if it's been modified outside the program. Let me know you think this needs re-opening though if I haven't correctly understood the scenario.

Coming in a year later on this one, but this does seem like a big gotcha. For instance if I have a NewSecurityGroup that has the Ingress property filled in and then have some separate NewSecurityGroupRule's that add more rules then the ones added later get removed on the next run.

Coming in a year later on this one, but this does seem like a big gotcha. For instance if I have a NewSecurityGroup that has the Ingress property filled in and then have some separate NewSecurityGroupRule's that add more rules then the ones added later get removed on the next run.

I see it is documented and I must have missed it 😞


Fixed: Cron expression resetting on update monitor

Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[ ] I have read the CONTRIBUTING doc
[ ] I certify that the changes I am introducing will be backward compatible, and I have discussed concerns about this, if any, with the Python SDK team
[ ] I used the commit message format described in CONTRIBUTING
[ ] I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
[ ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added unit and/or integration tests as appropriate to ensure backward compatibility of the changes
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)
[ ] I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Overall change looks good.
As a rule, whenever doing a bugfix, add a test case that would fail if this change wasn't there.


munin-hosts.py no scan IP/Domains

HI! After the update you cannot use munin hosts anymore, do not scan or process IP or domain;

If I activate debug mode, the following is displayed;

The munin that checks hash is ok

Oh finally I could find my error, the api were not correctly configured in the munin-host.py file (line API KEYS), although if they were correct in the MUNIN.INI file, I assumed that was enough with it. Curious.

Hi! It happened to me the same. I had to manually add the Key in munin-host.py though its already included in the .ini file.
Thank you


Add previous year entries to business_as_usual phase

Context
Fix Sentry error that has come from #5516
Changes proposed in this pull request

Guidance to review

Link to Trello card

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

Does it make sense to have future dates in the previous year? Otherwise good!

I changed them to historic dates, just to make it a bit less confusing


Output doesn't seem to inline any css

I tried to use the output method but it seems there is no result. I added this line of code in the config.php file that i use to configure kirki:
wp_enqueue_style( 'style_name', get_template_directory_uri().'/css/style_name.css' );

I have the above css file in css/style_name.css.
After this i set the stylesheet_id => 'style_name' and set the output of a select element like so:
...
'output' => array(
   'element' => '#page',
   'property' => 'max-width',
   'units' => 'px'
)

Nothing happens the iframe refreshes but there is nothing happening. Also even if i save nothing happen.
Should i expect something?

Hey there!
Can you please test if this still occurs if you use the master branch?

@aristath nope it seems using the github master it all works. Thanks!! Closing it.

Was the WordPress version updated with this resolved yet?
Not getting any CSS on my site...

That's weird... is that on a publicly available repo I can download to test?


Fix "get involved" links

With no href attribute, the links all point to the current page.

Hi @rrrutledge 👋🏻
Thanks so much for your PR and changes.
I've reviewed the links / preview build and everything is great.
@amaltson - Feel free to merge if you're okay with the changes 👍🏻
James.


HMRC Frontend - Account header - Links are not scaling with browser settings

Library or service name

HMRC Frontend - Account header
https://design.tax.service.gov.uk/hmrc-design-patterns/account-header/
Library version (where applicable)

5.27.0
Description of the issue

When increasing the font size within Chrome's browser settings the text size of all elements on the page increases apart from the account header menu links.
It looks like the links are missing a relative css font value (rem), for example adding font-size: 1rem; appears fix the issue when added via the dev tools.
.hmrc-account-menu__link {
  ...
  ...
  font-size: 16px;
  font-size: 1rem;
  ....
}

Steps to reproduce the issue

Open the example in Chrome:
https://design.tax.service.gov.uk/examples/account-header/example/
Goto Settings --> Appearance --> Customise fonts
Set font size to 24px
View the account header example page and it will look similar to the screenshot below

Actual vs expected behaviour

All text should be increased proportionally to the users preference, similar to the screenshot below:

Environment (where applicable)

Operating system: MacOS 13.1
Browser: Chrome
Browser version: 110.0.5481.100

Apologies for the very late response here - this repo was deprecated some time back but not archived, so we've missed a handful of reported issues.
We'll be tracking a fix for this under PLATUI-2618


Locales

use locales for strings so we can make it translatable

[x] add locales framework (#912)
[ ] user is able to select/set language
[x] decide on translation platform

[ ] Crowdin
[x] Weblate
[ ] pontoon

[x] add ci pipeline that upload & download from translation platform

@anbraten in look to the Vue rewrite - how much do you already had considered it?

@6543 Haven't planned that for the initial rewrite, but I used vue-i18n before which is easy to add.
Do you think translations are really necessary for this kind of software. I had the feeling that most devops are somehow used to English anyways.

For the initial rewrite it's not needed, but it wont hurt and help non native english speaker ...

We know have our own weblate instance: <translate.woodpecker-ci.org>. Language selection will be handled by #1109


Workaround to remove fixed sized stream buffer

Codecov Report

Merging #24 (31680b9) into master (77fb430) will decrease coverage by 0.30%.
The diff coverage is 47.91%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   55.78%   55.48%   -0.31%
==========================================
  Files          17       17
  Lines         760      766       +6
==========================================
+ Hits          424      425       +1
- Misses        284      291       +7
+ Partials       52       50       -2

Impacted Files
Coverage Δ

block/result.go
0.00% <0.00%> (ø)

block/stream.go
0.00% <0.00%> (ø)

blockstream.go
53.52% <0.00%> (ø)

exchange/exchange.go
0.00% <0.00%> (ø)

explore.go
0.00% <0.00%> (ø)

session.go
59.37% <56.41%> (-0.63%)
:arrow_down:

ipld/fetch.go
71.42% <100.00%> (+11.42%)
:arrow_up:

requester.go
75.75% <0.00%> (-1.52%)
:arrow_down:

Continue to review full report at Codecov.

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


Allow Images/Builds resources to select nodes  and configure the scheduling algorithm

This RFC addresses https://github.com/pivotal/kpack/issues/621
Readable

Codecov Report

Merging #812 (b8feb84) into main (c29d0d6) will decrease coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head b8feb84 differs from pull request most recent head 8ddcdfd. Consider uploading reports for the commit 8ddcdfd to get more accurate results

@@            Coverage Diff             @@
##             main     #812      +/-   ##
==========================================
- Coverage   67.78%   67.72%   -0.06%
==========================================
  Files         112      112
  Lines        4836     4840       +4
==========================================
  Hits         3278     3278
- Misses       1190     1194       +4
  Partials      368      368

Impacted Files
Coverage Δ

pkg/apis/build/v1alpha1/builder_types.go
0.00% <0.00%> (ø)

pkg/apis/build/v1alpha2/builder_types.go
0.00% <0.00%> (ø)

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 c29d0d6...8ddcdfd. Read the comment docs.


Treemap support for 'literal' colorType

Hi,
I was able to customize the Sunburst chart by passing in a tree with style built into the data:
{     title: 'John',     size: 1,     style: {       fill: '#FF991F',        fillOpacity: 0.533     } }.
However, Treemap didn't respect the same tree data, the chart showed up with a default color (whatever I specified in the 'style' section has no effect). I don't see the 'colorType' property in the Treemap source. Is it supposed to be supported? If not, does that mean I need create a separate function to build a different tree for Treemap?
Thanks!

Hey @jchenjc,
ColorType is definately supported in Treemap! Each of the nodes checks to see if it has a color value, which is then interpreted via colorType. To wit, copy and pasted from the documentation:
const myData = {
 "title": "analytics",
 "color": "#12939A",
 "children": [
  {
   "title": "cluster",
   "children": [
    {"title": "AgglomerativeCluster", "color": "#12939A", "size": 3938},
    {"title": "CommunityStructure", "color": "#12939A", "size": 3812},
    {"title": "HierarchicalCluster", "color": "#12939A", "size": 6714},
    {"title": "MergeEdge", "color": "#12939A", "size": 743}
   ]
  },
  {
   "title": "graph",
   "children": [
    {"title": "BetweennessCentrality", "color": "#12939A", "size": 3534},
    {"title": "LinkDistance", "color": "#12939A", "size": 5731},
    {"title": "MaxFlowMinCut", "color": "#12939A", "size": 7840},
    {"title": "ShortestPaths", "color": "#12939A", "size": 5914},
    {"title": "SpanningTree", "color": "#12939A", "size": 3416}
   ]
  },
  {
   "title": "optimization",
   "children": [
    {"title": "AspectRatioBanker", "color": "#12939A", "size": 7074}
   ]
  }
 ]
}

That said, you do highlight a valuable insight, we do not support a prop style on each of the nodes, and don't mention colorType directly in the docs. These things can certainly be fixed

Oh also, fill & fillOpacity are not directly relavant here bc the treemap uses DOM elements and not svg

Thanks for the quick response again @mcnuttandrew!
I see, did not realize the Treemap is not svg. Now I guess I have to refactor my 'toTree' function to populate 'color' and 'opacity' directly (instead of using 'style') if I want to share the tree-data between sunburst and treemap.
For sunburst, I can control the width and color of the stroke by passing in different values for data.style = { fill, fillOpacity, stroke, strokeWidth } when the user mouseOver a particular arc. What can I use for Treemap if I want to achieve the same effect (hovering over a square or circle)? Can I use any css property, like passing in borderWidth and borderColor etc.?
Thanks!

@supasonk if you are using SVG mode you can set it via style={{stroke: 'white'}} or if you are using DOM mode you can do style={{border: 'thin sold white'}} check out https://github.com/uber/react-vis/blob/master/showcase/treemap/simple-treemap.js for more details

@jchenjc : I know its a long time, But can you please share your code, I am trying to make it work but no matter what I am not able to color tree-map children's.


Why does libreoffice needs a wrapper that launches dbus if needed?

Issue description
Talking about this:
https://github.com/NixOS/nixpkgs/blob/16608ed2e65d4de4f6804c32ec4cbfbb05af2cd7/pkgs/applications/office/libreoffice/wrapper.sh#L6-L18
Can't we assume that a user running a desktop application like libreoffice would have a working dbus daemon running? Why do we need to add such a wrapper, when there are many environment variables already set in the unwrapped derivations:
$ cat $(nix-build -A libreoffice-unwrapped)/bin/soffice

Besides dbus, there's JAVA_HOME environment variable set there, which we can set in default.nix with makeWrapper's --set-default.
cc @7c6f434c

Well, lack of DBus leads to quite unobvious failure modes, so it's better to start a local instance if there is no global one. And it actually conditionally starts something, not just sets a variable, so I think it could be in its own wrapper for simpler debugging. And Nixpkgs is not only for NixOS with KDE.
$ cat $(nix-build -A libreoffice-unwrapped)/bin/soffice

Besides dbus, there's JAVA_HOME environment variable set there, which we can set in default.nix with makeWrapper's --set-default.
Yeah, that could probably be moved to unwrapped.

Fair enough. It would have been nice to prevent double wrapping though, but it's not that important to me personally. Maybe I'll open a PR that will fix that.. Thanks for the reply.


Full Numerical Type Support

Currently, most of the API returns Double due to the implementation of Apache-Common-Math. There already is support to input Int, Long, BigDecimal, Float, etc and handle all the necessary conversion. But the same type needs to be outputted as well even if the implementation uses Double.
This way type-specific implementations can be used down the road if we decide to scrap Apache-Common-Math for something else.

There are some parts of the APi that should be able to simply accept a Number without being too costly.

Decided I'm going to cancel this effort. Many operations will return a Double regardless of the inputted numeric type, just because it makes a far easier implementation.
We can consider full consistent numeric type support in a 2.0 version.


gnustep.base: use multiple outputs and support multiple outputs in hook

had to move the installFlagsArray in preInstallPhases so that ${!outputX} work because they're set by the multiple-output hook
unar before:
closure size 1.4G
unar after:
closure size 138.9M
Closes https://github.com/NixOS/nixpkgs/issues/117208
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

this breaks at leas the sogo and projectcenter build

fix for the sogo build:
--- a/pkgs/servers/web-apps/sogo/default.nix
+++ b/pkgs/servers/web-apps/sogo/default.nix
@@ -51,7 +51,7 @@ gnustep.stdenv.mkDerivation rec {
     sed -i "s:${gnustep.make}:$out:g" $out/share/GNUstep/GNUstep.conf

     # Link in GNUstep base
-    ${lndir}/bin/lndir ${gnustep.base}/lib/GNUstep/ $out/lib/GNUstep/
+    ${lndir}/bin/lndir ${lib.getLib gnustep.base}/lib/GNUstep/ $out/lib/GNUstep/

     # Link in sope
     ${lndir}/bin/lndir ${sope}/ $out/

projectcenter failed to build before

It's a xmas miracle! Nicely done!!


Solar System Sandcastle Example

I just noticed that all the planets are actually present in cesium along with the moon - it would be an awesome demo if while viewing earth the other planets were highlighted / labeled and if you clicked on them you "zoomed" to them by either manual movement or switching out earth with the sizing/shape/ellipsoid of whatever planet was clicked or at least showed the label of which planet is which - since currently there's nothing highlighting this feature

Hi, thanks to a European funding (EuroplanetH2020), a generic Solar System bodies implementation in Cesium has been drafted.
The code is not in shape for a pull request and future funding is uncertain, but it is available on github and a demo is available here.
Feel free to use it or add it in the demo page.

Thanks or sharing @cmarmo! Yeah, this looks like a great project to share on our Demos page.  @slchow what do you think?

Yes, let's showcase! I'm in touch with @cmarmo


added link Node.js Framework Overview in Web Frameworks

Please read and adhere to the contributing guidelines.
One PR per entry.


rollout: two more pages

@dbentley : A common complaint about adoption/usage I hear a lot is that "adoption is just about people using it, it doesn't really say if people are really enjoying it or finding it valuable". In fact, one customer had a similar concern in a recent customer meeting.
I think the correct response to this, is that the "adoption metric" is good enough. And that it's really hard to measure perceived value directly. Adoption does measure perceived value, but with a delay. But that's still good enough, because adoption is much easier to measure. Adoption is easy to measure because:

Usually there's a metric to do so. The tooling is straightforward.
But more importantly, an adoption metric is easy to understand and fairly objective. And so it's easy to use.

Some of these points are already covered in https://docs.tilt.dev/rollout/active-hours. But I think it might be worth anticipating the concern of "adoption isn't good enough" as a metric, and have a one or two sentence explanation, and point to https://docs.tilt.dev/rollout/active-hours.

@dbentley : Why did you use "could" instead of "should" in "Why could this be fast?" In earlier iterations you used "should" and that makes more intuitive sense to me. Why "could"?


Fix ultrametricity is some bespoke files

GitHub doesn't make it very easy to grasp the diff when looking at long line, but it can be looked at in other tools (e.g. VS Code at least highlights the changes).
I used a tool-guided but still somewhat manual process to fix up a selection of files to make them ultrametric. I will get some of the tools in later, along with some tests, but this is a first step, if only for discussion.
The summary is:

AfrotheriaPoulakakis2010.phy: change 26.110648 to 26.110649. All leaves are now at exactly 84.2
BicherSuzuki2010.phy: had (Polypterus_bichir_bichir:0.0,Polypterus_bichir_lapradei):0.0. Added the missing :0.0 on lapradei
GalapagosFinchesLamichhaney2015.phy: changed 0.355 to 0.364 to make all leaves exactly at 0.91 (some were before at 0.901)
GarsDeepfin.phy: changed 16.241946 to 16.241947. All leaves now at 267.9
PalaeognathaeMitchell2014.PHY: subtract the extinction prop on 3 species, as the lengths was clashing with rest of tree. Also adjusted a few other lengths.
PrimatesOneZoom.phy: fixed what was very likely a typo. It had Eulemur_rufus:7.6:0, and I removed the bogus :0 at the end. This file is actually not being used, but may as well fix this.
 Insignificant, but made the whole file ultrametric at exactly 65.0.
Tetrapods_Zheng_base.PHY: fixed the 37k extra year issue we discussed.

Notes:

There are more files to be fixed. Some are more problematic and will require discussion
This is all separate from the discussed absolute handling of @ syntax length, which is orthogonal.

This is great. Thanks a lot for doing this @davidebbo. You have no idea how long it would have taken me to hunt those down.

Happy to merge this.

For cross reference, this change is part of the discussion in #18


Mahaboob_RPAChallenge

Mahaboob_RPAChallenge.1.0.2.zip

Hi@shaikMahaboob22 Kindly upload the package by adding your rpa excel inside the project.


Support RGB(A) array as colormap input type

Description
This changes the implementation of ensure_colormap so that it converts RGB(A) array-likes (i.e. tuple, list, np.ndarray) as user input that can be coerced into a napari Colormap object. If one color value is provided, this creates a colormap that interpolates between black and the specified single color. If multiple color values are provided, this creates a colormap from those. In both cases, the colormap is unnamed (i.e. Colormap.name has the default custom) and so the AVAILABLE_COLORMAPS dictionary is not updated.
The logic of ensure_colormap is already complex, and this adds more behavior and complexity to it, but I didn't see obvious ways to simplify things.
In #2936, I mentioned that sometimes it could be useful to raise instead of warn when the input colormap type is unsupported (e.g. for some Pydantic model validators). I thought about making a change here to make that configurable (e.g. by adding a boolean parameter to condition warning vs. erroring), but I think a better approach is to add that configuration when actually needed.
I did however update and change some documentation appropriately.
I also moved the definition of ColorType to avoid some import dependency cycles - it's new location seems appropriate, but let me know if you have better ideas or locations for that. I also redefined ColorType to use str instead of AnyStr, as bytes strings are not currently supported by transform_color.
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References
Closes #3141
How has this been tested?

[ ] I added new tests in test_colormaps to check that the single color and multi color array likes are supported.
[ ] example: all existing related tests continue to pass.

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

@andy-sweet do you know why the CI isn't running for this PR? Can you maybe make another commit? Thanks!

@andy-sweet do you know why the CI isn't running for this PR? Can you maybe make another commit? Thanks!

Yep, I used [no ci] in my last commit, which was just a change to docstrings/typing. I just merged main into this and pushed to trigger all the checks.

Yep, I used [no ci] in my last commit, which was just a change to docstrings/typing. I just merged main into this and pushed to trigger all the checks.

Ah, i got it know
@tlambert03 any thoughts on this before merge?


GDAX Heartbeat Channel

Hi there.  I'm looking at the documentation and trying to subscribe to the heartbeat channel, with this code:

var websocket = new Gdax.WebsocketClient(
    ['BTC-USD'],
    'wss://ws-feed.gdax.com',
    {
        key: API_KEY,
        secret: API_SECRET,
        passphrase: API_PASSPHRASE,
    },
    { heartbeat: true }
)

  webSocket.on('message', data => {
    console.log(data);
  });

But I keep receiving the following error, despite passing BTC-USD
//error
{ type: 'error',message: 'Failed to subscribe',reason: 'You need to specify at least one product ID for channel heartbeat'
I'm guessing this is more to do with my ignorance on how the code works than anything else.  Is there a more appropriate, centralized place to put these comments?

I did like below and don't have any issues.  Are you passing API KEY's properly?
const websocket = new Gdax.WebsocketClient(['LTC-USD']);

websocket.on('message', function(data) {
	/* work with data */
	if (data['type'] == 'received') {
		console.log(data);
	}
});

This is a shortcoming of the backwards compatibility for heartbeats. If you authenticate the heartbeat will be processed before the other request, resulting in a failed subscription. This library's heartbeat implementation should be switched to use the heartbeat channel instead.

Occasionally, GDAX web socket connection will be disconnected even subscribing to heartbeat channel.   @fb55 could you please let me know how to reconnect when it is disconnected?

Please review #150 for a fix of the heartbeat subscription.

could you please let me know how to reconnect when it is disconnected?

@nengine You can reconnect by calling WebsocketClient#connect(). For example, in its simplest form:
const client = new Gdax.WebsocketClient(...);
client.on('close', () => client.connect());

Of course, you might want to delay the re-connect and/or continue to retry if the connection continues to fail.

Fixed in #150


Quansight's 0.8.1.dev

This is a tracking PR for all of the updates we have been making, in case others need them later. CC @dharhas

Codecov Report

Merging #87 into master will decrease coverage by 3.33%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
- Coverage   92.11%   88.78%   -3.34%
==========================================
  Files           2        2
  Lines         279      312      +33
==========================================
+ Hits          257      277      +20
- Misses         22       35      +13

Impacted Files
Coverage Δ

conda_mirror/conda_mirror.py
88.67% <66.66%> (-3.36%)
:arrow_down:

Continue to review full report at Codecov.

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

@xhochy - what do you mean including them?

@xhochy - what do you mean including them?

Someone pressing merge here.

Oh yeah, I think that is not going to happen, unfortunately.  If you need the dev version of this package, we are maintaining it over at https://anaconda.org/quansight/conda-mirror

Oh yeah, I think that is not going to happen, unfortunately. If you need the dev version of this package, we are maintaining it over at https://anaconda.org/quansight/conda-mirror

Why? Because the project here is abandoned or are the changes objected by the maintainers?

it seems to be abandoned ☹


Doesn't Work

I setup this module, but it doesn't seems anything. I create an API key and I add config.js and MovieInfo.js folder. However an error occurred because of api_key variable is boolean so it can be  just true or false. Please help me for fix this issue

@Mert4531 sorry for the late reply.
You shouldn't modify anything else than the config.js file.
The minimum config to get this module working is:
{
  module: 'MMM-MovieInfo',
  position: 'top_right',
  config: {
    api_key: 'cb1b7f012d18667a0ef851d8ab1980b5'
  }
}

Just tested it successfully with the newest version.


Milestone 1

Preliminary data collection is complete and all stored in Datastore.
Bare bones webapp is set up with the pixelation feature on sample images

@googlebot I consent.

@googlebot I consent.


Fix single channel png e2p

pillow does not accept h x w x 1 numpy arrays, added squeeze to make it h x w for single channel image.

thank you!


YouthMappers at UAM and YouthMappers ITU

Hi,
in transifex, we have to translate the text "YouthMappers at UAM" (string 4981).
There isn't another text contening UAM and in wikipedia I found 6 universities with these 3 letters UAM.
It should be great to know which one it is.
edit : same question with "YouthMappers ITU" (string 4997) which could be 4 universities.
Best regards

The source strings are in i18n/en.yaml:
YouthMappers at UAM:
https://github.com/osmlab/osm-community-index/blob/4db24896d04c7463f4d968c5780df5c4909abec8/i18n/en.yaml#L1082-L1084
YouthMappers ITU:
https://github.com/osmlab/osm-community-index/blob/4db24896d04c7463f4d968c5780df5c4909abec8/i18n/en.yaml#L904-L907

These strings do become part of the "key" used for translation:

OK, thanks a lot.


Add write support in S3FileSystem

Description
S3FileSystem must implement the WriteFile API.

PR required for this work are
https://github.com/facebookincubator/velox/pull/6309
https://github.com/facebookincubator/velox/pull/6021