Nested Sub-Bullets missing

Hey David,
Will you be adding sub-bullets to tomboy-ng?
I noticed that it was missing and reformatting my notes on my phone (tomboy clone).
I'm running a Mac 10.13.3
Hoping for something like this:

Ah, I'm home again. Can get to deal with some of these issues !
Thanks XYPD, yep, it does not support nested bullets. No reason why not apart from me not getting around to do it.  Honestly, I am quite unhappy with how bullets work on tomboy-ng anyway and just realised I unconsiously avoid using bullets for that reason !
Long term, its a rewrite of the bullet code, make the inset cnsistent and provide nested bullets.
But, no promise its going to happen quickly....
We'll leave this ticket open to remind me of the need to do it.
Davo

Thanks for the update on the look of the bullets. I have no idea how hard it is, but appreciate you considering tackling it. It something I personally use often. But I have adjust my usage for now. Since  it conflict with my mobile version (conflict ... meaning it removes them any sub :( ).

OK. Interesting.  Can you tell me more about your "mobile version" ?  Application ? Platform ? Sync ?
davo

I use an obscure mobile platform called maemo. It's run debian with a fork on gnome. You can find some info here about Conboy
I sync using owncloud/nextcloud, by choose the same directories on both the desktop and phone. It's been fantastic for getting my notes to the desktop. I might be changing that sync to rsync soon though, since my phone doesn't support newer versions of most stuff now days.

Ah, that makes sense, I used to work with someone who used that, long time ago. It was very unreliable back then......   Hope it works better now.

haha, I love it, both the hardware keyboard and the OS. It's more like a tiny laptop then a phone. It's been great for me so far, issue here and there but like any device I suppose. Some new hardware is coming out Q1 next year so I might actually have something more modern hardware wise :D.

I think I will close this to tidy up a bit.  Multilevel bullets are on my (published) ToDo list but its a hard problem, not one I am likely to commit a lot of time to in the immediate future.
Davo

Since there is only this request from a single person, I want to chime in and voice that at least for me, this is a crucial feature.   Tomboy itself is a surprisingly critical part of my life for going on nearly 20 years now, and without nested bullets, tomboy-ng unfortunately can not be a viable replacement.
I appreciate that others find this type of software so useful, and that you took it on yourself to re-implement it as the problems with GTK# and mono etc made getting tomboy working more and more difficult over the years.  But still I feel I should voice the perspective on the record so to speak, that for the listmaking, scratch planning, brainstorming of ideas and project architecting that I do in tomboy, makes constant use of organizing items under other items -- multiple levels of bullets -- and without them it simply cannot serve the purpose that OG tomboy serves for me.
Here's a snip out of my current todo list

Nearly all of my thousands of notes are like this, and if I can't do this it is a vast downgrade -- I simply cannot use it for the things I have used tomboy for since the early 2000s.
I am always interested in a re-implementation of tomboy, since it's a miracle every time I manage to get old-tomboy installed on a new environment.  Getting a working install usually takes 10-12 hours of trial and error over several days, with the old busted dependancies.  Once installed, it's buggy with the task tray icon being invisible about 1/2 the time, since one of the last GTK updates around 2016.  Since 2016 i've been clicking an invisible space in my task tray to open my notes.  In spite of all this, I persist, because tomboy is the most useful thing ever.  But I know one day I will have a new environment and tomboy simply will not install anymore, even after the multiple days of trial and error.  I honestly don't know what I will do then.  For this reason I perpetually have my eye on tomboy-ng, and I check back every year or so to install tomboy-ng and see if nested bullets have been implemented, but as long as they have not, I simply do not have a viable path away from geriatric tomboy.
Anyway I don't want to sound whiney or entitled, or throw out the baby with the bathwater of your good work.  I just want to put it out there that there are people like me for whom this seemingly small peace of functionality is really the fulcrum of it's viability as a replacement.

OK, I have reopened this issue.  Its a maybe.  As I have mentioned in the past, I am pretty unhappy with how kmemo handles bullets, I have added some pretty ugly code in there to may them work as I think they should. So, I worry that it means adding more ugly code of mine.
If I find myself looking for something to do ....
Davo

Not to complicate your life, but this seems a reasonable feature request. I remember noting the lack of nested bullets at some point, but I've not been keeping very complicated notes lately. For project planning tasks and subtasks, outlines for writers or charting Dante's circles of hell this is an important and useful feature. As you have indicated this is not the first limitation you have encountered in kmemo, so maybe you could work with upstream? (Yeah, pretty cheeky comin' from a non-programmer.)

I wish there was something I could do to help.  I am inferring that kmemo is the engine of the "rich text box" tomboy-ng displays notes through?  And kmemo is one control out of the KControls project here https://github.com/kryslt/KControls
This part of their wiki talks about some of their funniness with bullets https://wiki.freepascal.org/KControls/KmemoNotes#Bullets
I tried to look at their source code to get an idea of how it works, if the bulletedness is a bit when it would need to be an integer, or maybe nested bullets are really just indented bullets, and if indents and bulletness are seperate kmemo block properties.  Just trying to understand the situation.  Sadly my unfamiliarity with any aspect of pascal (I didn't actually realize it was still an active language) is a barrier.

Yes, you have summed it up there GunArm, KMemo does do nested bullets but overall, its bullets lack some of the behaviour we expect bullets to to do so I have added code to make up the difference. Its my code that does no do nested bullets.
And of course, this code is called each time the user presses a key so its critical it be fast.  So, I have taken the approach that it should do as little as possible. Sounds like I will have to reconsider.
I believe to best approach is going to be to go back and fix the underlying issues in KMemo rather that the pretty ugly wrapper I have used.

pascal (I didn't actually realize it was still an active language)

yes, still quite active. Has quite a large user base. While simple Pascal looks quite similar to simple C, the Object Pascal we use here is more to be compared to C++. And they are somewhat more different. The big advantage is the closely intergrated LCL, Lazarus Component Library and IDE that makes the cross platform GUI very easy.
Davo

OK, yes, its possible. And perhaps not as hard as I decided back in the early stages of this project.
But they cannot all be bullets, I must use a different 'character' for each level. I can use things like Bullets, Arabic Numerals, Roman Numerals, Letters. The Roman and Letters can be upper or lower case, this is what I have been playing with -

Your comments welcome !
Davo

Great news. That said I would not mix bullets and letters/numbers if possible. The latter suggest order/priority as well as hierarchy. I recall using bullets, hyphens (-) and then dots (.), even in ancient, pre-word processing days. This could be extended with the em-dash (—), perhaps mathematical signs like =, +, >. The asterisk is also possible, but might best be used at the highest level given its size. Others may have better ideas.

Yep, agree. But to use other than the ones predefined in KMemo, I'll have to patch KMemo.  And that causes problems with my Debian and Ubuntu builds, using (eg) KMemo 'out of the box' makes things a lot easier.
During both builds, I get a live copy direct from TK's repository. If I want to patch it, its a case of applying that patch, completely hands off, during the remote build process.  These builds happen on Debian and Ubuntu remote machines you understand.
Davo

Some nice progress here. So you're saying that your dev setup will break or cause issues if you change the KMemo bullets to something none debiab/ubuntu standard?
Cause I would be happy with some of the following, * ° • - — + > » or even a simple "."
So you can't not compile it on your setup?
This too has made it tough to sync all my note without it reformatting.
x

Yep, agree. But to use other than the ones predefined in KMemo, I'll have to patch KMemo. And that causes problems with my Debian and Ubuntu builds, using (eg) KMemo 'out of the box' makes things a lot easier.

Understood. I assume from the reply there is no numbered list defined in KMemo, which would be better than mixing.
xypd raises a good question about reformatting, which has a further implication: would KMemo actually reformat, or choke??? I see some nasty testing ahead... :-/

xypd, no, its not a case of using debiab/ubuntu standard. Just of using the characters TK has chosen to use in KMemo.  KMemo is an external library linked to tomboy-ng at compile time.  The source is readily available and can be patched when I build -ng locally, on my laptop for example.
But when building as part of a Debian or Ubuntu source build, I submit my source code and scripts that know how to do the building. Those scripts go and get KMemo from its 'official' repository and then use it.  While I can add patching of kmemo to that process, it increases the risk of a build failure. And all that building happens out of my sight.  KMemo is a pretty complicated package, if I hack away at it, I'll need to do a lot of testing !
TK really describes this feature as "paragraph numbering". KMemo allows the use of several 'characters', but only one of them is not sequential, the bullet.  I believe I have to use a different character for each level so we can have only one non-sequential level.
I will log an issue with TK and see if I am wrong about each level having to use its own character. He is a pretty busy guy but has helped me in the past.
Davo

OK, like wow !
Less that 24 hours after I logged this, TK has come back with a solution !  I'll have to do some tests and get it lined up for Lazarus's Online Package Manager but right now and it looks pretty good.
Unlike fixes to Lazarus LCL, fixes to KMemo are available to use almost immediately !
I'll need to do those tests ....
https://github.com/kryslt/KControls/issues/35
Davo

Here is a sample. TK has provided us with two extra 'bullets', I have altered the order a bit here and we still have lower case letters as a fourth level. Now, I don't see  a case for four levels anyway. But I note that xypd's sample does have four levels.  So, xypd, is using sequential lowercase OK at the fourth level ?
Personally, I have only ever used four levels of bullets in legal documents, and I don't think of tomboy-ng as the tool for that sort of work. But I am looking for your thoughts folks !
Davo

This sounds like great news! I would need to look on my other device to see really how many levels I use.
I think however this is very workable for daily use. So if this is what it needs to be I think I can live with that. I'm not sure how possible this requests would be, but could an preference option be added to change the the level order? So meaning I could choose apps wide to have square -> dots -> sequential for example?
I don't have an issue with sequential, since some other might find it useful.
Will response more about details in a few days, when I get to my device.

Been out through the weekend, so I just want to say I'm ecstatic with the progress.  I really appreciate your having taken time out of your life to address this.  I would have been ok with the original nested things turning in to numbering, although it would not be my first choice, but atleast items could go under other items.
As for the current iteration, having checked over a lot of my notes, I saw a few times where I went out to 4 levels, but it was rare.  I'm grateful for the fact it goes out that far regardless of if the sigils turn into ordered letters.
My initial concern about the sequences though, is that I know WYSIWYG editors, even very basic ones, are hard to write and maintain, even google's built into gmail is not always perfect.  And autonumbering is weird even in big software like LibreOffice.  I find it miraculous when they don't do weird things.  For example, what happens if you have two sets of auto numbers, and then you delete the space between them?  Or if you have 1-5 (or a-e) and reduce the third one a level back, or put a space after the third one, do 4 and 5 become 1 and 2 like you'd expect, or does it leave you with a sequence starting at 4?  OTOH maybe it "just works", but I just know it can be rife with gotchas.
Frankly, even having multiple different character sigils for different bullet-levels isn't completely necessary to me, IE, if it was the same sigil at all levels I'd accept that.  It's nice that there are multiple sigils though.  But it's certainly not critical that they are fully unique at each level, so depending on code limitations, if it could just go back to bullet style 1 after level 3.  FWIW, that's how original tomboy works, it has 3 bullet symbols and just repeats the sequence at further levels.  But if, for whatever code-wise reason, it's easier to go to an incrementing 4th level that's fine too.  Just thinking how even though no-one requested incrementing, if it's in the software, someone will get neurotic about it and file bugs about oddities, which might be harder to fix than the original feature was to create .  So just saying looping sigils is another possibility.
Side note: Do you think the migration of notes between old tomboy and ng will be able to convert tomboy bullets to tomboy-ng bullets?

GunArm, I have not tested how well the sequencial numbering works to be honest. I have had to some ubtrusive code around KMemo's "paragraph numbering" system to make it behave as we expect it to, things like backspacing at the beginning of a bullet line. I may need to do more if we have issues with the fourth level. But if thats the case, I think I will keep TK informed, he has been very helpfull here for a guy who said he does not want to work on KControls any more !
The issue is not really about reusing the bullet image itself, the software has a fixed number of 'slots' each has its own image. We could have them all with the same image, the limiting factor is the number of slots or levels defined. But TK has demonstarted that was reasonably easy to add two more....
Migration from Tomboy to tomboy-ng with extended bullets is my problem, should not be too hard.  Hardest thing will getting an instance of Tomboy going so I can see what the html looks like. If you have an existing note with multiple levels, posting it here would be a big help !
Davo

could an preference option be added to change the the level order?
@xypd - I would rather not, it would be another thing that needs to be looked up and dealt with on every key press. And that would slow things down. We have some very fast typists out there ...

Better IMHO, to decide on the best order now and 'hard wire' it that way.
Incidentally, the changes being made here (in my code, not TK's) to accommodate multiple levels should be a touch faster now that i understand a bit more about the levels. Thats nice !
Davo

I made a sample OG tomboy note and exported it to html.
Funny thing, github doesn't allow attaching html files
So I went to put it on pastebin.  And pastebin says it's offensive content lol.  I whittled the doc down, and eventually worked out that it's "http://beatniksoftware.com/tomboy" which is offensive, although not ""http://beatniksoftware.com/" or "tomboy".  Anyway pastebin literally would not let me paste this content.  Never seen that before, anyway I had to find a different pastbin site.
Let me know if this link doesn't work or anything: https://ghostbin.com/paste/GKvZ8

Thanks GunArm, but what I need is the file in .note format, its xml but pretty different to html.  So, just the note please.
It will still have the content offensive to pastebin (??) so, maybe just zip it up ?
There is also an email address in the license file for me.
Davo

My mistake, not sure what I was thinking exporting it.  Here's the .note
note.zip

Ah, excellent news, I'll run with that then.
Been in C19 lockdown here now for a week so I have the time .....
Than GunArm !
Davo

OK, I think I have it working as expected. I say 'think' because there is a a lot of new code in there. Not only new Bullet code but there is also relatively untested new code to do the regular note saves in a separate thread. So, I ask, you want to have a try with it ?  I have made packages I will use for my testing and can post them if you want to try. But there is an element of risk.
xypd - MacOS ?
GunArm - Windows ?
Davo

Yeah windows here.  If you've got a build I'll be happy to beat on it :)

Yeah macOS is fine. I could also try linux in vm or on arm64. But mac would be the easiest at the moment

OK, they are available from -
https://github.com/tomboy-notes/tomboy-ng/releases/tag/v0.33a
Look for the 0.33b in the file names.  I have tested superficially but, as I mentioned, some big changes in there.
@xypd, arm64 ?  What flavour are you using ?  I ship a -ng with armhf, for RasPi on RasPi OS but no 64bit version.  I suspect it might be hard to make both because I cross compile and it seems I can have either but not both on the same system. So, interested in how widely used the 64bit OS's are.
Davo

I would try it on my PinePhone which can run multible distros. I actually already have it install on mobian (debian mobile DE), so maybe the .deb will work fine. The device has an Allwinner A64 chip in it. But on linux I would likely bootup my ubuntu virtualbox vm

Look like Libgtk2.0-0 libc6 libcanberra-gtk-module are miss for the arm64 install. Just fyi.
After installing them, I was not able to install tomboy-ng armhf. Will give the mac OS a tried once I get time to backup my notes or maybe I can add it to another user account.

After installing them, I was not able to install tomboy-ng armhf.

No, the armhf is a 32bit system. You might be able to install all the necessary 32bit libraries but it would be pretty ugly !
It 'should' be possible to build a 64bit arm. But I have not tried, mainly because no demand  :-)

Allwinner64

Is an arm chip too. So, won't work with the intel targeted packages.
Davo

Ah I see thanks for that info. Never knew that about armhf.

I don't see a windows build for 33b.  I'm looking for an exe right?  I see rpm, deb, tgz.  Only one exe (for 33a)

I don't see a windows build for 33b.

No, I don't know what I did wrong there, maybe i uploaded the 0.33a version again ?  Sorry, the one you want is now there, yes, its called 0.33b
Davo

Thanks.
Having the nested bullets is great.  Also my notes from tomboy all migrated and look good from a cursory glance.
Here, in no particular order of significance, are some oddities I noticed:

If you bullet something, and then unbullet it, some indent is left over, which goes away when you close/reopen the note

There seems to be slightly more indent on the 8th bullet than the rest of them

Migrating really deep bullets (deeper than I have ever used), the bullet level gets lost past level 6.  In this screenshot I went out to 9 only to test, knowing it was one more than supported, but would have expected 7 and 8 to increased and I guess 9 to be on level 8.

Here are two .note files related to this one.  tomboy00* is the original tomboy note, tomboyng* is the note after it was opened in tomboy ng, and then I changed a char and changed it back to force a save.
migrate-test.zip

Maybe this is expected and can't be helped, but just to report, it looks like the incrementing numbers are global for the note

If you bullet something, and then unbullet it, some indent is left over,

Yes, it does.I dont know why I did not notice that, I have a fair idea of what causes it.

There seems to be slightly more indent on the 8th bullet than the rest of them

Yep, found it.

the bullet level gets lost past level 6.

That is strange. Calling no bullet level 0, I can go out to 8 and there it stops.  I get 7=Uppercase Roman; 8=Number; 9=Is the same numbers

Great feedback GunArm. Sadly it might be a week before I have time to mess around with out changing my live setup.

Accidental close .....

incrementing numbers are global

Yes, thats what made me go back to TK at KControls and request one more (non-sequential) bullet.  He has now done that and agrees more work is needed so what we see now is temporary.  I have supported all nine levels but think the latter ones are unusable. So, will advertise as having four. After that, its your decision to use them or not. But they will be at least saved as levels.

Is it possible to make the solid circle first, a hollow circle second and then continue with what you have? For my taste the arrow call too much attention to itself. But I can live with that over no nested sub-bullets :).

Yes, I agree that the large arrow is too large. But we have a limited number of slots and I felt that I should use the most prominent one first.
Now, TK has come back to me with the further addition I requested and an observation that it needs more work. IMHO, we need at least four 'medium' bullets, support for beyond that is not a high priority but I want to allow it just in case someone has a specific need and I don't want to risk not saving their work.
I will take this up with TK and see if we can find a reasonable 'set' of four or five medium bullets.
Davo

the bullet level gets lost past level 6.

That is strange. Calling no bullet level 0, I can go out to 8 and there it stops. I get 7=Uppercase Roman; 8=Number; 9=Is the same numbers

Just to be clear: I can go out to 8 the way you describe in ng and it works fine, it was only migrating a tomboy-prime note to tomboy-ng which did that.  I don't want to give the impression that levels 7 and 8 don't work at all for me, they do.
FWIW about the big first arrow: it is big, but I could get used to it.  I also am with you that if one is more prominent than the others it should probably be the first one.

it was only migrating a tomboy-prime note to tomboy-ng which did that

OK, I did no get that initially. Seriously wonder why ?

big first arrow

I keep telling myself I can get used to it too but I suspect I lie.
I have put a new model to TK that, I think, will make things better. Could even let the user choose.....
Davo

Hi folks, TK is clearly pre-occupied at present, thought I would have a play with some alternative arrangements.  Attached is an image showing five levels (with the possibility of an additional three not very functional ones). I have chosen smaller bullet characters and will need to do some testing to ensure that they are widely available. Apparently thats not always the case ...

Davo

Not sure how you did it, but looks nice...

Folks, an issue, TK, the author of KControls has found that in his locale, Czech, the bullet characters I have used do not show up in Windows. Assuming you are using Windows and the the 0.33e beta of tomboy-ng, could you please let me know where you live ?
Davo

Sure, I'm in the US, seems to work for me:

Not a Win user, but decided to test 33e and noted something wonky. Made a mistake and decided to eliminate the level and redo it erasing with the backspace key. At every level except the first bulleted level when the first character of the bullet point is deleted, the cursor moves to the level above, but treats it as if it were the level removed, indenting it more and using the symbol for the lower level. See pic where the lower part shows how the penultimate level is changed with the removal of the one below. (Pic may not be worth 1000 words, but better than the description I just wrote.)
Not sure if that was intended behavior, but is not what I would have expected. The unindent shortcut, of course, quickly undoes this so it is not a catastrophe or (one supposes) an indication of impending Armegeddon.
When the top level is reached, ng removes the bullet point and moves the text to the left margin.
At some point using the keyboard shortcut to indent a level turned on bold. This happened several times, but could not be reproduced consistently, so I assume there is some magic sequence that leads to this -- just don't know what it is at the moment.
Mageia 9 with Enlightenment 24.2 DE (and Spanish keyboard, if that matters).

OK, fixed in newly released v0.34, will close this ticket.
Worth noting that this is, still, somewhat, a temp solution as in some locales, the bullet characters do not all work but until TK comes up with a better model, we are using my hacked version of KControls and it does seem to work for most people. I suspect, but have not proven, for the ones it does not work "out of the box" selecing a different "normal" font in tomboy-ng, will solve the problem.
Davo

David, just want to give you a personal thank you for your work and time sacrifice on this.  I know it was hassle but it is very much appreciated!


Error After Update to 2.0.7

@StorytellerCZ hi
Error After Update to 2.0.7:
I have tested the software many times. This problem is only due to the update to the new version.

These bugs took hours of our time, please follow up.

@StorytellerCZ  ?

I can only see dependencies updates and one fix for Cordova for 2.0.7
In the case of the error you are getting it means that file._source.url is undefined. :thinking:
Can you provide a reproduction repo or form locally and add a log around there to see what is the input you are getting?

I'm having a similar issue too, I can't deploy to galaxy anymore. How do we fix this?
Errors prevented deploying: While minifying app stylesheet:                packages/minifier-postcss/plugin/minify-css.js:137:56: Cannot read property 'replace' of null at packages/minifier-postcss/plugin/minify-css.js:137:56 at Array.map (<anonymous>) at mergeCss (packages/minifier-postcss/plugin/minify-css.js:119:23) at CssToolsMinifier.processFilesForBundle (packages/minifier-postcss/plugin/minify-css.js:89:18)

@flean can you please provide a quick reproduction or add some logging so that we know what input you are getting?

@Saeeed-B forgot to ask if this is resolved for you.

Now yes

Great! I will close this for now then.


GH-3503: Add sources attribute to DotNetCoreTestSettings

Add sources attribute to DotNetCoreTestSettings
Close: #3503

@eoehen your changes have been merged, thanks for your contribution :+1:


python310Packages.hahomematic: 1.8.5 -> 1.9.0

Description of changes
https://github.com/danielperna84/hahomematic/releases/tag/1.9.0

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.hahomematic
python39Packages.hahomematic


Improve thunar/thunarx.

Allows one to register plugins not built directly as
part of thunar by using an environment variable
instead of a build time fixed value.
This could allow one to wrap thunar in a thunar-with-plugins
package where plugins are declared instead of being found (a
bit in the same fashion as gstreamer plugins).
Note that when no environment variable present, simply
fallback to previous behavior.

I currently use nixos to set my environement in the following way.
environment.variables.THUNARX_DIRECTORY = "/run/current-system/sw/lib/thunarx-2";

thunar-dropbox-plugin is now loaded as expected (this is a new plugin yet un-merged upstream).

@eelco

Sorry @jraygauthier, different Eelco, you mean @edolstra.

:+1: but I would call it THUNARX_MODULE_DIR

@lethalman: Done.

Combined the two commits + updated/standardized commit msg.

See tests described in #9935.

@domenkozar: how do/did you use thunar-archive-plugin?

@vcunat: xfce.thunar_archive_plugin in its current incarnation does not work properly. For
some strange reason (mirror related?) it builds the volume manager plugin instead. I have
an upcoming pull request that fixes that (currently running here).

Looks good.

I never got to use the plugin :) Thanks for tackling this!


Docker build is broken

While I had tested #60 I must have missed something as docker no longer builds and as such deploys are broken. I'll try to look into this tonight 👍

Closed by #67.


Discrepances in position, size and color of connectors' names

Connectors within the MSL differ in the position, size and color of their name primitive. In the figure attached, there are shown some of the connectors from different domains.
There should be defined a convention on how to handle the connectors' names, similar to Modelica.UsersGuide.Conventions.Icons. See also discussion on descriptive text in #2471.

I would also prefer to have a consistent way of defining a positive and negative (or what ever their name is) connector by extending from a base connector definitions. This is, however, handled differently in different packages. The base class shall not have an icon. Icons shall be added by the positive and negative connector definitions.
Example 1: Modelica.Mechanics.Rotational.Interfaces

Flange_a is defined without extending from a base defintion
Flange_b is defined without extending from a base defintion

connector Flange_a "One-dimensional rotational flange of a shaft (filled circle icon)"
  SI.Angle phi "Absolute rotation angle of flange";
  flow SI.Torque tau "Cut torque in the flange";
end Flange_a;

Example 2: Modelica.Electrical.MultiPhase.Interfaces:

PositivePlug extends from Plug
NegativePlug extends from Plug
Plug contains the base definition of the potential and flow variable

connector Plug "Plug with m pins for an electric component"
  parameter Integer m(final min=1) = 3 "Number of phases";
  Modelica.Electrical.Analog.Interfaces.Pin pin[m];
end Plug;

connector PositivePlug "Positive plug with m pins"
  extends Plug;
end PositivePlug;


Adding StorSimple1200Series Azure SDK for Net

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please add REST spec PR link to the SDK PR
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@johnemmanuelm  was the code regenerated? There should be a .txt and SdkInfo_*.cs files generated which needs to be checked in

Hello Deepak, As advised re-generated the client and now those two files are generated. Thanks for the help!

No code under the Generated folder was updated. In fact had to update the rest API spec to generate it again to a pass a certain test case. https://github.com/Azure/azure-rest-api-specs/pull/3951

Clsoing in favor of https://github.com/Azure/azure-sdk-for-net/pull/4781


Metacoq is broken with Coq master

Hi folks,
ci-metacoq has been broken in Coq's CI for a while:
Error:
 [Derive EqDec] requires a [NoConfusion] or a [NoConfusionHom] instance for type sort_family to be derived first.
 Command exited with non-zero status 1

we should fix this as there are a few PRs that may need overlays for metacoq and otherwise will go merged untested.

Thanks for the notice, I'll fix it ASAP.

@ejgallego should be good now, at least the pipeline job on coq/coq#12542 goes through now.

Merci @mattam82 !


Composer requirements for 2.0.8

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for ffuenf/ffuenf_deleteorder 2.0.8 -> satisfiable by Ffuenf/Ffuenf_DeleteOrder[2.0.8].
    - Ffuenf/Ffuenf_DeleteOrder 2.0.8 requires ffuenf/ffuenf_common * -> no matching package found.

Your packages are not listed on https://packages.firegento.com/#!/ffuenf

https://github.com/magento-hackathon/composer-repository/blob/master/satis.json
Also not listed there

Same for
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for ffuenf/ffuenf_deleteorder 2.0.8 -> satisfiable by Ffuenf/Ffuenf_DeleteOrder[2.0.8].
    - Ffuenf/Ffuenf_DeleteOrder 2.0.8 requires ffuenf/ffuenf_magetrashapp * -> no matching package found.

Solution:
composer config repositories.ffuenf_magetrashapp vcs git@github.com:ffuenf/Ffuenf_MageTrashApp.git
composer config repositories.ffuenf_deleteorder vcs git@github.com:ffuenf/Ffuenf_DeleteOrder.git
composer config repositories.ffuenf_common vcs git@github.com:ffuenf/Ffuenf_Common.git


Add tctl command to delete workflow

tctl should add a workflow delete command since frontend service already expose such functionality:
Originally posted by @wxing1292 in https://github.com/temporalio/temporal/issues/3570#issuecomment-1307983637

added


tf.lite.Optimize.DEFAULT - Hybrid models are not supported on TFLite Micro.

@tensorflow/micro
System information

Host OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Catalina 10.15.6
TensorFlow installed from (source or binary): 2.3.0
Tensorflow version (commit SHA if source):
Target platform (e.g. Arm Mbed OS, Arduino Nano 33 etc.): ESP32

Describe the problem
I'm trying to convert a model for tflite, but keep hitting:
"Hybrid models are not supported on TFLite Micro."
Please provide the exact sequence of commands/steps when you ran into the problem
Here is my model:
model = Sequential([
    Conv2D(4, 3,
           padding='same',
           activation='relu',
           input_shape=(IMG_WIDTH, IMG_HEIGHT, 1),
           name='conv_layer1'),
    MaxPooling2D(name='max_pooling1'),
    Conv2D(4, 3,
           padding='same',
           activation='relu',
           name='conv_layer2'),
    MaxPooling2D(name='max_pooling2', pool_size=(2,2)),
    Flatten(),
    Dense(
        20,
        activation='relu',
        name='hidden_layer'
    ),
    Dense(1, activation='sigmoid', name='output')
])

And here is the code I am using to convert the mode:
converter = tf.lite.TFLiteConverter.from_saved_model("checkpoint.model")
converter.optimizations = [tf.lite.Optimize.DEFAULT]
model = converter.convert()
open("converted_model.tflite", "wb").write(model)

I have also tried tf.lite.Optimize.OPTIMIZE_FOR_SIZE which has the same issue. Removing all optimisations lets me
Is there any way to avoid triggering this error with my model? Ideally, I would like to optimize my model to make it smaller.

See here:
https://www.tensorflow.org/lite/performance/post_training_quantization
Go for full integer quantization. The default optimization leaves part of the operations as floating point (hence hybrid) and mixing isn't supported in micro.

@yair-ehrenwald
Great! Setting a value for the representative_dataset seems to fix it.
Thanks for the quick response.

See here:
https://www.tensorflow.org/lite/performance/post_training_quantization
Go for full integer quantization. The default optimization leaves part of the operations as floating point (hence hybrid) and mixing isn't supported in micro.

Thanks for this you answered a question that I had been scratching my head about. Do you know why full integer quantization would be failing with Zero Padding ? https://github.com/tensorflow/tensorflow/issues/50725

any update for this bug. I need a fix as soon as possible


Update app.py

I added these as they are needed when running in my environment (maybe because I am running it inside pycharm).

Going to close this out for now, thank you though!


Reworked file copying to use pipe

This solves the event loop time out as streaming closing is pickedup correctly.

While this PR doesn't pass builds, it's a huge step forwards and I prefer to fix the stalled builds in a follow up PR.


Ensure using one version of Node

It is important to ensure using the same version of tools across developers as well as local development and CI systems so that builds can be reproducible. This PR adds a configuration file .nvmrc to specify the version of Node being used, and configures the CI to use the exactly same version in the config file.

@lyuyangh @neubig I don't have a write access to merge this pull request. Would you merge this PR for me?

Sure!


Redis cache key should use column name instead of field name

This is because field renames should be a safe, code-only change and not need a cache key bump.

The issue with doing this is it will cause an invalidation of all existing caches, which could cause a huge spike in DB load. To accomplish this, we'll need to create a way to let users roll out the new cache key format on a per-concrete-entity-implementation basis in one version and then in another version remove the old format.

The issue with doing this is it will cause an invalidation of all existing caches, which could cause a huge spike in DB load. To accomplish this, we'll need to create a way to let users roll out the new cache key format on a per-concrete-entity-implementation basis in one version and then in another version remove the old format.


grammar improvement in article: Filled map in Azure Maps Power BI visual

Per prior arrangement, if this PR needs approval, or if the commits need to be pulled into the private repo, only @eriklindeman should be notified. Thanks.

@eriklindeman

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

cc @deniseatmicrosoft

sign-off

#sign-off


Fix JSON unmarshalling error (non-pointer objects.CommonObjectProperties)

When unmarshalling a stix bundle that contains a custom type (causing the unmarshaller to hit the default case in this switch statement),   the error json: Unmarshal(non-pointer objects.CommonObjectProperties) is returned because it's not being unmarshalled into a pointer type.

Ah I see this is a dupe of #15
@jordan2175 any chance of merging one of these PRs?

Done


[Proposal] Default metadata in BaseMujocoEnv

Proposal
The current code for BaseMujocoEnv requires the env metadata dictionary to have fixed, pre-specified values. While this may be useful for future API changes, it doesn't seem very useful at the moment.

    assert self.metadata["render_modes"] == [
        "human",
        "rgb_array",
        "depth_array",
    ], self.metadata["render_modes"]
    if "render_fps" in self.metadata:
        assert (
            int(np.round(1.0 / self.dt)) == self.metadata["render_fps"]
        ), f'Expected value: {int(np.round(1.0 / self.dt))}, Actual value: {self.metadata["render_fps"]}'

I propose to set the metadata attribute in BaseMujocoEnv to the current fixed values

self.metadata["render_modes"] = [
            "human",
            "rgb_array",
            "depth_array",
        ]
self.metadata["render_fps"] = int(np.round(1.0 / self.dt))

This will enable classes derived from MujocoEnv to overwrite the dictionary (if required, in the future), but it will not force them to write an explicit metadata dictionary if the default values suffice, reducing redundancy and making derived classes more compact and readable.
Motivation
No response
Pitch
No response
Alternatives
No response
Additional context
No response
Checklist

[X] I have checked that there is no similar issue in the repo

The current implementation is to enable definition of metadata["render_fps"] after MujocoEnv.__init__ (see how Ant_v4 and Ant_v5 define metadata for example), if your proposal does not break anything, feel free to make a PR

Perfect, I have sent a pull request. The pull request avoids breaking the existing behavior by defining default metadata as class attribute, and then setting render_fps dynamically (as a function of self.dt) only if the key has not been overridden by a child class.
PR: https://github.com/Farama-Foundation/Gymnasium/pull/1099


Add strip assertions flag

This change temporarily adds iree-opt-strip-assertions to SDXL CI which should fix the regression discussed in https://github.com/iree-org/iree/issues/19002. Assertions within linalg.generic ops can mess with dispatch creation and lead to poorly performing dispatches, despite the fact that they get stripped in later pipelines.
Part 2 (future PR or maybe this one): remove flag usage and make this default.

Could I get a review on this? This might also be affecting Llama performance, there were ~100 dispatches that could be CSEed if the asserts were removed.

The idea was to to fix sdxl regressions first by just using the flag, then make it default after because there may be some side effects of stripping all assertions like this by default. But I can change this to be default if that seems better


Ensure that we set opaque type substitutions on local variables.

We were never setting these opaque type substitutions, but code
generation was silently failing. Now we assert, so move the code into
the proper common location so we always set opaque type substitutions
on properties.
Fixes rdar://86800325.

@swift-ci please smoke test


[Feature]: Add an option to copy CURL of a failed run from the debugger

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
if I setup an api endpoint sometimes I have to re-export and test it inside a  terminal with curl.
Why should this be worked on?
it is really helpful a button to export the text of the api call setup in appsmith

@abeatbeyondlab endpoints in Appsmith cannot be accessed from outside the platform except via our new workflows feature where you can easily get the URL #3378

I am talking about another topic.
I am only saying the export curl featuire that you find for example in api docs

I reopened here
https://github.com/appsmithorg/appsmith/issues/32256

@abeatbeyondlab please explain why you need that feature?

as i already wrote above. the need is an export curl command function
now is already implemented  the curl import.
if we have the "curl import" I dont understand why we dont have the "curl export"

@abeatbeyondlab because the purpose of CURL import is to bring in external APIs into Appsmith but there is no need to export an API inside Appsmith to the outside world because Appsmith APIs are configured with bindings that read values from widgets. So they cannot be triggered from outside. Hope this diagram helps. If you want to trigger a workflow from a 3rd party system, I suggest using our workflows feature

@Nikhil-Nandagopal I think export curl will be helpful in troubleshooting.
Ex : I have a developer who makes apis specifically for Appsmith, now if there's some error and API is not working as expected, he would ask for the curl of it, which then helps developer in testing things out.

@sachans so what you are actually asking for is a CURL of a run that specifically failed in order to debug. Is that correct?

the main goal should be to have a "curl export" button that it allows you to paste on a terminal and then  debug better what is configured on appsmith reading from terminal the result of the curl export string.
it is simple to do

news on that?

@abeatbeyondlab sorry this hasn't made it to our roadmap yet. We'll let you know once we prioritise it.


Falco

What to document

There is not a label identifying the kind of this issue.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@3garbilx is this an error?

i will close this :)


unable to login into SPV Wallet web-frontend

Latest beta release:
➜  spv-wallet-1.0.0-beta.32 ./start.sh
Welcome in SPV Wallet!
 [Y/n]
>
Do you want to run spv-wallet-admin? [Y/n]
>
Do you want to run block-headers-service? [Y/n]
>
Do you want to run spv-wallet-web-frontend? [Y/n]
>
Do you want to run spv-wallet-web-backend? [Y/n]
>
Define admin xPub (Leave empty to use the default one)
>
To login to the admin panel, you will need to provide the admin xPriv.
You choose to use default admin xPub, so you can use the following xPriv:
xprv9s21ZrQH143K3CbJXirfrtpLvhT3Vgusdo8coBritQ3rcS7Jy7sxWhatuxG5h2y1Cqj8FKmPp69536gmjYRpfga2MJdsGyBsnB12E19CESK
What PayMail domain should be configured in applications?
>
Do you want to expose the services on  and its subdomains? [y/N]
>
Do you want to run everything in the background? [y/N]
>
File .env.config updated!
[+] Running 5/5
 ✔ spv-wallet Pulled                                                                                1.6s
 ✔ wallet-frontend Pulled                                                                                1.5s
 ✔ wallet-backend Pulled                                                                                1.5s
 ✔ block-headers-service Pulled                                                                                1.6s
 ✔ spv-wallet-admin Pulled

No domain selected, able to login into SPV Wallet admin.
It seems the SPV Wallet web-frontend does not use the correct url for the api call?
GET http://api/v1/config net::ERR_NAME_NOT_RESOLVED

I cannot reproduce the exact error.
I have similar error for some time after running the start.sh, because the backend isn't ready yet.
Can you ensure in docker that the wallet-backend is running? Or eventually run start.sh and then wait for some time (arround 1 minute), then in start.sh output you should be able to see logs with prefix wallet-backend-1
and then check if the problem still persists?

➜  ~ docker ps
CONTAINER ID   IMAGE                                 COMMAND                  CREATED       STATUS                 PORTS                                               NAMES
08577db944be   bsvb/spv-wallet:latest                "./entrypoint.sh"        5 hours ago   Up 5 hours             <IP_ADDRESS>:3003->3003/tcp, :::3003->3003/tcp           spv-wallet-spv-wallet-1
e5ff915f9401   bsvb/spv-wallet-web-backend:latest    "./entrypoint.sh"        5 hours ago   Up 5 hours             <IP_ADDRESS>:8180->8180/tcp, :::8180->8180/tcp           spv-wallet-wallet-backend-1
0717fcc83721   bsvb/spv-wallet-admin:latest          "/docker-entrypoint.…"   5 hours ago   Up 5 hours             80/tcp, <IP_ADDRESS>:3000->3000/tcp, :::3000->3000/tcp   spv-wallet-spv-wallet-admin-1
6feff59b6323   postgres                              "docker-entrypoint.s…"   2 days ago    Up 5 hours (healthy)   <IP_ADDRESS>:5432->5432/tcp, :::5432->5432/tcp           spv-wallet-wallet-postgresql-1
d827d8e57091   redis                                 "docker-entrypoint.s…"   2 days ago    Up 5 hours (healthy)   <IP_ADDRESS>:6379->6379/tcp, :::6379->6379/tcp           spv-wallet-wallet-redis-1
65303b5500d0   bsvb/block-headers-service:latest     "./entrypoint.sh --p…"   2 days ago    Up 5 hours             <IP_ADDRESS>:8080->8080/tcp, :::8080->8080/tcp           spv-wallet-block-headers-service-1
37212d659c8f   bsvb/spv-wallet-web-frontend:latest   "/docker-entrypoint.…"   2 days ago    Up 5 hours             <IP_ADDRESS>:3002->80/tcp, [::]:3002->80/tcp             spv-wallet-wallet-frontend-1

➜  ~ docker logs --follow spv-wallet-wallet-backend-1
2024-10-21 08:43:42.889 DBG host=wallet-postgresql port=5432 user=postgres password=postgres dbname=postgres sslmode=disable application=spv-wallet-web-backend ecs.version=1.6.0 service=database
2024-10-21 08:43:42.929 INF Successfully connected! application=spv-wallet-web-backend ecs.version=1.6.0 service=database
2024-10-21 08:43:43.212 DBG Websocket server started application=spv-wallet-web-backend ecs.version=1.6.0 service=websocket
[GIN-debug] [WARNING] Running in "debug" mode.

I'm sorry for the delay in answer.
Can you please:

run docker compose down in a root directory of the spv-wallet repository
update the repository to newest version
run ./start.sh -l - to start everything again with previous setup

If the new the problem still persist, please send us:

file .env.config
result from calling http://localhost:3002/env-config.json
please also sand a screenshot from Network tab (please ensure that it's clear just before reloading the page, then reload the page and take a screenshot.


⚠ PrivateHD has degraded performance

In 54a07da, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3658 ms

Resolved: PrivateHD performance has improved in 2231979.


Tekton demo improvements

Couple of changes:

Tekton demo can ran from local source tree (without -g flag).
Running with -g flag ensures modifications to the pelorus git project happens in the temporary directory and not local source tree
Better handling of branches:

if remote branch exists, use it
if remote branch does not exists, create one

Cleanup function added.
Ensure we don't pollute master branch.

Signed-off-by: Michal Pryc<EMAIL_ADDRESS>Testing Instructions
Local directory run:
clone pelorus fork, run demo on existing branch or non-exising, e.g.
git clone git@github.com:mpryc/pelorus.git
cd pelorus
./demo-tekton.sh -r non_existing_remote_branch -c 3
# Create new branch existing_remote_branch using GitHub UI (to ensure script works fine with remote branches not synced to current clone):
./demo-tekton.sh -r existing_remote_branch -c 3

Temp directory

This is better as it works without polluting any of current files/branches from which script is being ran
Ensure -g url is provided to location where local user have commit access, most likely https is not the one...

cd pelorus
./demo-tekton.sh -r non_existing_remote_branch -c 3 -g git@github.com:mpryc/pelorus.git
# Create new branch existing_remote_branch using GitHub UI (to ensure script works fine with remote branches not synced to current clone):
./demo-tekton.sh -r existing_remote_branch -c 3 -g  git@github.com:mpryc/pelorus.git

Failing master, we really don't want to run this script with master branch...
./demo-tekton.sh -r master -c 3 -g  git@github.com:mpryc/pelorus.git

@redhat-cop/mdt

hrm.. still have a bug on the pelorus test.. unrelated to this PR
Test that Pelorus has found the latest git commit:
Build Type is buildconfig
test buildconfig: requires exporter named committime-exporter
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  2674  100  2674    0     0  15916      0 --:--:-- --:--:-- --:--:-- 15916

There is quite a delay in the clean up. I think folks may think the script is hung. Suggestion is to log the cleanup as it works through or add a message it may take 2 minutes or so. :)
/var/tmp/PELORUS/pelorus1/demo
1. Namespace 'demoapp1' already exists
Clean up resources prior to execution:

@weshayutin added some printf statements to be more informative what's happening.

./demo-tekton.sh -b buildconfig -n demoapp1 -a demoapp1 -g https://github.com/weshayutin/pelorus.git  -r demoapp1  -c 12 -t 5

The current_branch is not substituted in the buildConfig in 05-

--- /tmp/meld-tmp3z5luvk2.yaml
+++ /home/whayutin/OPENSHIFT/git/KONVEYOR/PELORUS/upstream/pelorus/demo/tekton-demo-setup/05-build-and-deploy.yaml
@@ -22,8 +22,8 @@
source:
type: Git
git:

     uri: https://github.com/konveyor/pelorus

     ref: master

     uri: ${url}

     ref: refs/heads/${APPLICATION_NAME}
   contextDir: "demo/python-example"
 strategy:
   sourceStrategy:

We would have to make sure we sync the branch name w/ the app name for this to work.. perhaps a fix
--- /tmp/meld-tmp3z5luvk2.yaml
+++ /home/whayutin/OPENSHIFT/git/KONVEYOR/PELORUS/upstream/pelorus/demo/tekton-demo-setup/05-build-and-deploy.yaml
@@ -22,8 +22,8 @@
       source:
         type: Git
         git:
-          uri: https://github.com/konveyor/pelorus
-          ref: master
+          uri: ${url}
+          ref: refs/heads/${APPLICATION_NAME}
         contextDir: "demo/python-example"
       strategy:
         sourceStrategy:

Reworked a bit to include such. It's actually in 2 places, first is the template which takes the URI and REF and then it translates into tekton parameters.
Also added check if the Pelorus installation is suitable to run the demo (commit time properly monitoring namespace where we are building app).

/test 4.9-e2e-openshift

multiple builds working on binary :) woot thanks!
buildConfig and s2i working as designed as well..
Tested w/
demo apps
./demo-tekton.sh -b buildconfig -n demoapp1 -a demoapp1 -g https://github.com/weshayutin/pelorus.git  -r demoapp1  -c 12 -t 5
./demo-tekton.sh -b binary -n demoapp2 -a demoapp2 -g https://github.com/weshayutin/pelorus.git  -r demoapp2  -c 12 -t 5
./demo-tekton.sh -b s2i -n demoapp3 -a demoapp3 -g https://github.com/weshayutin/pelorus.git  -r demoapp3  -c 12 -t 5


How to access config from helpers?

In config.rb I've set a few default config "vars", eg.
set :title, "Website Title"

I seem to be able to access this from helpers using title, however this does not work when the helper is accessed from a partial
What's the proper way of accessing these config vars?
Thanks

Trying random variables I think I landed on the proper solution: config[:title]

These should all work:
title
config.title
config[:title]
settings[:title]

I like to use a site variable for this (same as Jekyll), so as to not pollute the core namespace which you can view here: http://localhost:4567/__middleman/config/#core
set :site, OpenStruct.new(
  title: 'Website Title'
).freeze
# site.title

Thanks @Arcovion - using the site namespace is a good idea!

in v4, config lives in a sandbox so we're going to be a lot safer with naming. That said, site reads pretty well.

@tdreyno can you comment on the current way to do this as of v4.1.2? Here's my approach with the error being raised:
# config.rb

# Configure site host for generating absolute URLs
configure :development do
  config[:site] = { host: "http://localhost:4567" }
end

configure :build do
  config[:site] = { host: "https://jeffcole.github.io" }
end

# helpers/twitter_helpers.rb

module TwitterHelpers
  def twitter_share_link(page)
    # Raises: undefined local variable or method `config'
    # for #<TwitterHelpers:0x007f81b4207e88>
    host = config[:site][:host]
    # ...
  end
end

Thanks in advance for any help you can give!

If you call helpers TwitterHelpers, that should be available in templates and work as written.
Are you calling that helper from a non-template context?

I'm calling it from my blog layout like this:
-# source/layouts/blog.haml

= twitter_share_link(current_article)

I'm taking advantage of the helper auto loading that works via file name convention. I tried manually requiring and invoking helper TwitterHelpers as well and got the same results.

Hmm, that should work fine. Would you have a repo I could look at?

@tdreyno I figured it out, I'm actually using a nested class within TwitterHelpers to generate the URL, and the nested class doesn't have access to config. You're right that config is accessible from within the module, so I can just pass it through.
Very sorry for the trouble and thanks for your very speedy help!

Glad to hear


Enhanced Input Validation and Code Refactoring for Starknet Address Script

Enhanced input validation and refactored address computation logic in Starknet address pre-computation script

Time spent on this PR:
0.5 days

Pull request type:

[x] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)

What is the current behavior?
The original code does not validate hexadecimal inputs for classHash, salt, and deployerAddress fields, which could lead to runtime errors if users provide values in an incorrect format. Additionally, the function and variable names do not fully conform to consistent camelCase styling.

What is the new behavior?

Input Validation: Implemented validation for hexadecimal input values using regular expressions to ensure the correct format for classHash, salt, and deployerAddress.
Improved Naming Conventions: Updated function and variable names to camelCase for readability and consistency.
Error Messages: Added clearer error messages to guide users when input format is incorrect.

Files Changed:

Script File:

Added input validation to check for hexadecimal format in classHash, salt, and deployerAddress.
Refactored variable and function names to camelCase style.
Improved console output for readability and ease of debugging.

README Documentation:

Updated usage instructions to reflect new validation requirements.
Provided example inputs that adhere to expected formats.

Environment Configuration (if applicable):

No functional changes but updated comments to specify input requirements, particularly around the format for private keys and addresses.

This change is

need to fix trunk (linter)

need to fix trunk (linter)

Thanks for the heads-up! I’ll get those fixes sorted and update the PR later today.

Hey @ClementWalter
I've made the requested fixes.Thanks!


Fix root validators & subclasses

Fixes #5388
Selected Reviewer: @dmontagu

please review. I'll fix whatever is wrong with 3.11 tomorrow / before merging.

okay all fixed, please review


(194) View and update actions

https://trello.com/c/CsNfWkNp
Show a checklist of Actions on the individual Task page
Initially, we want to show our YAML Actions as a completable checklist on each individual Task page.
DRY up relationships in existing factories
We can simply use the name of the related factory when declaring relationships in FactoryBot.
Add update action to Task controller
Users need to be able to complete Actions, and for this to be persisted.

 View and update a Tasks Actions

I started to review this and realised that @jacksonj04 is already doing one, so I will stop now.


openssh: 9.2p1 -> 9.3p1

Description of changes
Security and bug fixes: https://www.openssh.com/txt/release-9.3

cc @mweinelt
@wahjava - I have not marked HPN as insecure because there are no CVE IDs afaict, but your HPN patches don't apply to 9.3 anymore
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
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

[x] Fits CONTRIBUTING.md.

pkgsMusl and pkgsCross.aarch64-multiplatform variants build too.


chore: add istio in e2e tests

Explanation
Add istio in e2e tests.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 29.07%. Comparing base (dc82d97) to head (47a304e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #282   +/-   ##
=======================================
  Coverage   29.07%   29.07%
=======================================
  Files          30       30
  Lines        1183     1183
=======================================
  Hits          344      344
  Misses        791      791
  Partials       48       48

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


How to get both LdcEqualsLdd as true and false in rocblas logic yaml?

Each yaml file has 2 groups of solutions under rocBLAS/library/src/blas3/Tensile/Logic/asm_full.
The only difference between the 2 groups is parameter: LdcEqualsLdd.
How did you train tensile to get this yaml?

There is a massage script under Tensile/Logic/archive that does the replication.  If a logic file needs the replication, the original logic file is archived in the Tensile/Logic/archive directory.  Place the logic files to be massaged in a working directory e.g. /tmp/raw, and create an output directory e.g. /tmp/massaged, then just run python massage.py /tmp/raw /tmp/massaged.  Then copy /tmp/massaged/* over to Tensile/Logic/asm_full.
Just FYI, this aspect of the logic files is evolving.  A better approach may be introduced in the future.  Hope this answers your question.  Thanks for your interest.

There is a massage script under Tensile/Logic/archive that does the replication.  If a logic file needs the replication, the original logic file is archived in the Tensile/Logic/archive directory.  Place the logic files to be massaged in a working directory e.g. /tmp/raw, and create an output directory e.g. /tmp/massaged, then just run python massage.py /tmp/raw /tmp/massaged.  Then copy /tmp/massaged/* over to Tensile/Logic/asm_full.
Just FYI, this aspect of the logic files is evolving.  A better approach may be introduced in the future.  Hope this answers your question.  Thanks for your interest.


mcpwm_ll_timer_get_count_mode() return type is wrong (IDFGH-6407)

https://github.com/espressif/esp-idf/blob/23a03a3bdbaab1fe0ba427dcfcc82ef391cc06f4/components/hal/esp32/include/hal/mcpwm_ll.h#L352
I'm using "ESP32 Arduino 2.0.1 based on ESP-IDF 4.4"
I found this error.
mcpwm_ll.h: In function 'mcpwm_timer_count_mode_t mcpwm_ll_timer_get_count_mode(mcpwm_dev_t*, int)':
mcpwm_ll.h:352:50: error: invalid conversion from 'unsigned int' to 'mcpwm_timer_count_mode_t' [-fpermissive]

If you specify the type, the error will disappear.
return (mcpwm_timer_count_mode_t) mcpwm-> timer [timer_id] .timer_cfg1.timer_mod;

Thank you for reporting, we will fix it ASAP and backport to 4.4 release branch.


Release Google.Cloud.NetworkSecurity.V1Beta1 version 2.0.0-beta03

Changes in this release:
New features

Enable REST transport in C# (commit cdb518c)

Created release for Google.Cloud.NetworkSecurity.V1Beta1-2.0.0-beta03


feat: hide datasource legacy based on featureflag

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11371 into master will decrease coverage by 5.89%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #11371      +/-   ##
==========================================
- Coverage   65.77%   59.88%   -5.90%
==========================================
  Files         838      394     -444
  Lines       39838    24940   -14898
  Branches     3655        0    -3655
==========================================
- Hits        26203    14935   -11268
+ Misses      13534    10005    -3529
+ Partials      101        0     -101

Flag
Coverage Δ

#cypress
?

#javascript
?

#python
59.88% <100.00%> (-1.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/config.py
90.18% <100.00%> (ø)

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
31.91% <0.00%> (-59.58%)
:arrow_down:

superset/db_engine_specs/hive.py
53.90% <0.00%> (-30.08%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
69.76% <0.00%> (-12.32%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/api.py
83.49% <0.00%> (-7.08%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

... and 460 more

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 225c905...b733eb4. Read the comment docs.

@lilykuang would you mind giving this a look?

LGTM once green

retriggering CI, since https://github.com/apache/incubator-superset/pull/11433 merged


lint-doc stage does not account for constraints on Odoc

Currently Irmin's documentation does not build correctly when using Odoc 2.0.{1,2} (https://github.com/ocaml/odoc/issues/793), and as a result we have added an incompatibility with these versions to irmin.opam (here).
Unfortunately, ocaml-ci does not account for the project's constraints on Odoc when installing it. This is causing Irmin's CI to fail consistently (e.g. here). Perhaps Odoc should be installed along with the project's dependencies to ensure that this doesn't happen.

Related https://github.com/ocurrent/ocaml-ci/issues/264


Add tests for updating/deleting shared service instances

As an app dev (receiver), I cannot update a service instance that has been shared with me #150973390
As an app dev (receiver), I cannot delete a service instance that has been shared with me#150973376
NOTE: This PR builds on top of #982 which should be merged first. The actual changes on top of #982 can be viewed in this diff.
What
This PR adds additional test to ensure that only users who have write access to the space in which the service instance was created are able to update/delete it.
PR

[X] I have viewed signed and have submitted the Contributor License Agreement
[X] I have made this pull request to the master branch
[X] I have run all the unit tests using bundle exec rake
[X] I have run CF Acceptance Tests on bosh lite

Thanks, sapi (@jenspinney and @samze)

Please resolve the conflict in spec/unit/controllers/services/service_instances_controller_spec.rb


Added compose.yml, adds a config for the server

The config defines an audience and scopes for the meshwith directory. With this config file, we now must use make requests to the meshwith path or the request will fail.
For the time being, the config JSON is stored in compose.yml because I couldn't figure out how to add a relative path to  compose.yml.
The program will fail if the aud claim does not match the aud claim defined in compose.yml. The config file seemingly ignores the scopes we give it (ie doesn't force it to match) despite the server requiring us to provide scopes in our requests.
Should resolve #3 and maybe #1. I'm not entirely sure if the behavior described above is acceptable for our use cases.

So, this is great progress.
I did make a modification whereby audience is sent in a parameter to /authorize, not as a custom claim (because we want to match the standard OAuth flow as closely as possible).  I believe this will resolve our issue with audiences.
With regard to the scopes, I there's more work to do still.  I see that what you've done is added some custom scopes to the compose.yml file, but what I want to be able to do is request certain scopes when I log in (for example, meshwith:read, but not meshwith:write), and have whichever claims I request added to the Access Token.   This would simulate users logging in and allowing different levels of access to the client.

In that case, this problem should have a very easy solution with the built-in dynamic variables that the server supports for expressly this purpose. Emphasis on should - I've spent the past three hours trying to fiddle with the systems with no success. By all accounts it should work, but I seem to be missing something which is making the program completely refuse to swap out the variable.
Regardless the progress I've made so far is on my branch - this new version seperates config.json to it's own file so Docker isn't trying to replace the variable with an environment variable. Which might actually be worth looking into as an alternate solution should I fail to figure out why the server refuses to insert our scopes into the variable.

Or at least I DID update compose.yml. It looks like I forgot to hit save. The updated compose.yml should be up shortly

Added a possible solution, but I'm not sure if this solution is even a legitimate one.
For some reason, our mock server doesn't support dynamic variables for specifically the scope variable. However, it doesn't seem to care about scope(s).
This solution does indeed return the scopes we feed it, but I'm not sure if this is even a valid solution given that, in the eyes of the program, the scope parameter is not the same as the scopes parameter. Keep in mind that for our authorization_code request, the parameter MUST be scope or else the mock server will refuse our request - it only accepts scopes in the access token request.


renamed Remove & altered popover description

Description

Renamed 'Remove' button in dropdown menu.
Altered description in popover dialog accordingly.
Fixes 1009

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Merged! Thank you @eugen-soliar !
Thanks for the review @Its-treason !
I also made some small UX updates


Made grass step sounds, rain, and portals quieter.

Not sure if I'm doing this right. Sorry.

Hey, thank you for your contribution! I'll take a look at it as soon as possible, likely next week. Perhaps we can use reflection to reduce changed base classes. I agree with the changes (the "annoyances"), especially the portal loudness.

Thanks.

Hey, I'm afraid something went wrong with committing your changes. The source code you committed is unchanged from what I retrieve from 1.7.3-LTS. Could you perhaps re-check, and perhaps even add comments on where you made changes in the files?


Can the extension sign in also sign in for Azure CLI in the integrated terminal

There's a longstanding issue where the interactive az login doesn't work with GitHub Codespaces in browser https://github.com/Azure/azure-cli/issues/20315
This extension has Azure sign-in that works properly with Codespaces in browser, but it doesn't seem to sign-in for Azure CLI in the terminal. It would be great if it could, like the GitHub extension and gh CLI both work seamlessly in this aspect.

Thanks for the suggestion. I can't make any promises, but this functionality is on our radar and is something we'd like to achieve.


Create a sample event for GCI summit and generate an App for it

We need to create a sample event on eventyay.com for the Google Code In summit and create a sample app for it.
App Name : GCI 2016
Icon : Use the GCI Icon
Tracks : Use mock data for this
Sessions : Use Mock data
Schedule : Something Like this,
8:30-9: breakfast
9-10: keynote
10-11: sessions
11-12: sessions
12-1:30: lunch
1:30-2: sessions
2-3: sessions
3-4: break
4-5: sessions
5-6: lightening talks
6-8: dinner, party

Location : Google Office, MV
Sponsors : None
Speakers : Use Mock data
We need the GooglePlay variant of the app, so please feel free to create this and share the apk link down below for us to test.

@the-dagger Can I work on this?
On Apr 16, 2017 12:18, "Harshit Dwivedi"<EMAIL_ADDRESS>wrote:

We need to create a sample event on eventyay.com for the Google Code In
summit and create a sample app for it.
App Name : GCI 2016
Icon : Use the GCI Icon
Tracks : Use mock data for this
Sessions : Use Mock data
Schedule : Something Like this,
8:30-9: breakfast
9-10: keynote
10-11: sessions
11-12: sessions
12-1:30: lunch
1:30-2: sessions
2-3: sessions
3-4: break
4-5: sessions
5-6: lightening talks
6-8: dinner, party
Location : Google Office, MV
Sponsors : None
Speakers : Use Mock data
We need the GooglePlay variant of the app, so please feel free to create
this and share the apk link down below for us to test.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/1418, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AMHNbHr13LxvjwRq98qFtca7y1gXhn3Wks5rwbnMgaJpZM4M-jtA
.

Sure, go ahead.
On Sunday, April 16, 2017, Harshita Gupta<EMAIL_ADDRESS>wrote:

@the-dagger Can I work on this?
On Apr 16, 2017 12:18, "Harshit Dwivedi"<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

We need to create a sample event on eventyay.com for the Google Code In
summit and create a sample app for it.
App Name : GCI 2016
Icon : Use the GCI Icon
Tracks : Use mock data for this
Sessions : Use Mock data
Schedule : Something Like this,
8:30-9: breakfast
9-10: keynote
10-11: sessions
11-12: sessions
12-1:30: lunch
1:30-2: sessions
2-3: sessions
3-4: break
4-5: sessions
5-6: lightening talks
6-8: dinner, party
Location : Google Office, MV
Sponsors : None
Speakers : Use Mock data
We need the GooglePlay variant of the app, so please feel free to create
this and share the apk link down below for us to test.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/1418, or mute
the
thread
<https://github.com/notifications/unsubscribe-auth/
AMHNbHr13LxvjwRq98qFtca7y1gXhn3Wks5rwbnMgaJpZM4M-jtA>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/1418#issuecomment-294337104,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQtluZooo5fM3iOLCJgzQp-Q8ShbjN-ks5rwbuYgaJpZM4M-jtA
.

--
Best,
Harshit Dwivedi

@the-dagger
we have to use the apk generator for this right where we provide the required json files?
Or building it from scratch?

@the-dagger
actually i used the event generator and provided it with the json files having content as you provided above but it gives an error saying the app cannot be built.

You can do it either way.
But priority is to use the apk generator.

@the-dagger
as the app generator showed error while building the app i built it from scratch as seen below. Please provide inputs so as to improve it further!
app-release-unsigned.zip

The apo generated is corrupt.

@the-dagger Tried to create a sample application. Please check it out. Currently the schedule is displaying null but I will try to look into it.
apk.zip

There are just very few people in that event. I would not call it summit. Also most participants are students and there is no extensive program. I dont think they need a dedicated app for that, but you can spend your time as you like. However, I this it is not really the right place to open an issue here. It is not really relevant for Android development. You can open an issue on the open event repo, if you want to create another sample https://github.com/fossasia/open-event. I am closing this therefore.


Add using arrow keys to move the box.

This tool save a lot time for me, really. Thank you @tzutalin !
In my work, sometimes, I need to label tiny object. When I do some precise operations, arrow keys become very useful. I think may be other people have the same demand, so I create this PR.
In this PR, the program allow user use arrow keys to move selected RectBox pixel by pixel.
For the detail in this PR, I add 4 trigger keys in keyPressEvent and implement the bound check function as well as move function.
I hope you like it!

You are right, this is a very useful tool.


datasignal dsclk returns 1 to all e2 in same tick even if e2 was executed by something else

While debug my os as always i found that dsClk returns 1 to all e2 in same tick even if e2 was executed by something else like input or something else. Here is short instruction of about what i mean:

Spawn one e2 with this code:

@name DS Check 1
@inputs Send SEnt:entity
@persist
@trigger

if (~Send & Send) {
    print("DS Check 1")
    dsSendDirect("TEST",SEnt,"Test Message")
}

Spawn second e2 with this code:

@name DS Check 2
@outputs Ent:entity TrigMe

if (first()|duped()) {
    Ent = entity()
}

if (dsClk()) {
    print("DS Check 2", dsGetString())
    TrigMe = !TrigMe
}

Spawn third e2 with this code:

@name DS Check 3
@inputs TrigMe

if (~TrigMe) {
    print("DS Check 3", dsClk(), dsGetString())
}

Connect Send input on DS Check 1 to button.
Connect SEnt:entity on DS Check 1 to Ent:entity from DS Check 2.
Connect TrigMe input on DS Check 3 to TrigMe output from DS Check 2.
Press button.

If everything connected right you will get:
DS Check 1
DS Check 2	Test Message
DS Check 3	1	Test Message

So basically on DS Check 3 it reports that it was also triggered by datasignal, but it was triggered by input only! I'm not sure if this is a feature or bug but still reporting about this.

This is because the datasignals use a shared variable (currentsignal, set here: https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/core/datasignal.lua#L91).
Storing that value in the target E2 context instead of as a shared variable should fix it.

This shouldn't be possible because datasignals are executed on the next tick https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/core/datasignal.lua#L110

I've got a fix for this
-- old code:
e2function number dsClk()
    return currentsignal ~= nil and 1 or 0
end

-- new code:
e2function number dsClk()
    if not currentsignal then return 0 end
    return currentsignal.to == self.entity and 1 or 0
end

this means you will still be able to run dsGetString() or whatever but that shouldn't matter because you won't be doing that if dsClk returns 0 anyway.
I'll merge this tomorrow if no one has anything against it

runOnTick works the same way btw and has always worked the same way since the beginning
https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/core/gametick.lua#L40
so if you erase if (dsClk()) { in your 2nd E2 and add runOnTick, then tickClk() will return 1 in the 3rd E2.

Yeah, these should definitely use self.data, I'm happy to review changes.

While we are at refractoring, maybe have a single variable that stores trigger reason like "input"/"tick"/"duped", instead of flags for everything?. And a clkType() or whatever function that allows getting that var in E2.


Try to conditionalize app id [SATURN-1267]

I have no clue how to test that this works. If anyone else has ideas, I would love to know. Is there a dev data warehouse? How would I go about accessing it?

Closing out this pull request because we want to try to maintain the consistent saturn header, in order to be able to aggregate information around 'all traffic from a terra-ui hosted site'. The new approach is to try to use Origin, which already is sent from the UI, but would need to be propagated through to kibana from the services.


MatDatepicker - WCAG 1.4.11 Issue

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
Present in all versions
Description
The color of the secondary date focus state indicator does not meet WCAG 1.4.11 requirements. It's currently at 1.6:1 and it needs to be 3:1 minimum.
https://www.w3.org/WAI/WCAG21/Understanding/non-text-contrast.html
Success Criterion 1.4.11 Non-text Contrast (Level AA): The visual presentation of the following have a contrast ratio of at least 3:1 against adjacent color(s):
User Interface Components
Visual information required to identify user interface components and states, except for inactive components or where the appearance of the component is determined by the user agent and not modified by the author;
Graphical Objects
Parts of graphics required to understand the content, except when a particular presentation of graphics is essential to the information being conveyed
WCAG 1.4.11 - Mat Datepicker.docx
Reproduction
Steps to reproduce:

Open the datepicker component
Tab into the individual dates
Select  to pick an active date
Go back into datepicker and use arrow keys to move to between non-selected dates

Expected Behavior
Color contrast should be 3:1 or higher
Actual Behavior
Color contrast for the secondary date when in focus is 1.6:1
Environment

Angular: 12.2
CDK/Material: 12.2
Browser(s): Edge and Chrome
Microsoft Edge - Version 103.0.1264.71 (Official build) (64-bit)
Chrome - Version 103.0.5060.134 (Official Build) (64-bit)
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10 Enterprise

Hello,
Thank you for the detailed write-up. Does enabling strong focus indicators fix this problem?
-Zach
P.S. it's generally difficult for us on the Angular team to open .docx and .zip files. If you provide an image or Google doc, we will be happy to read what you wrote.

Hi Zach,
Thanks for following up. I added two images that show the current situation.... (to replace the .docx file with screenshots).
One of our developers is looking into where we apply the strong focus indicator setting.
For clarification, is the strong focus indicators "theme" setting intended to replace or override the default theme colors? I'm not sure I understand how this setting will replace the existing hex value with something that is WCAG compliant. Are we supposed to declare a color, or a new hex value?

Hello @tleck1 ,
I agree with the original bug report that the blue circle for focus state would be better if it had more contrast.
I would suggest enabling strong focus indicators as a work-around. It doesn't fix the NTC issue, but it will help make the focus state more clear. I would recommend the focus indicators regardless of this bug. I'm leaving this bug open, and hopefully a work-around is helpful.
For instructions, see strong focus indicators documentation. For support requests, the best place to reach out is on stackoverflow or even starting a github discussion. We hope the documentation meets your needs, but feel free to file a documentation bug if it does not.
Thanks,
Zach

Also, we appreciate that the original bug report lists a WCAG rule. That helps make the bug report easier to take action on, and we generally require WCAG rules for a11y bugs 👍


Received an error with invalid JSON from Parse: Cannot POST /classes/_User\n

I'm using PFUser-Digits resource to authenticate parse users using Twitter's Digits API.
My parse server is hosted on App Engine with MongoLab database.
I get this error when trying to signup/login
RESOURCES USED
Link to Swift File
Link to Cloud Code handling Authentication

2016-03-01 23:59:49.646 Bars[10323:15114219] [Error]: {
    code = 107;
    message = "Received an error with invalid JSON from Parse: Cannot POST /classes/_User\n";
} (Code: 141, Version: 1.12.0)
2016-03-01 23:59:49.647 Bars[10323:15113659] -[__NSCFDictionary length]: unrecognized selector sent to instance 0x7fe0adc33a10```

SOLVED
In server.js, I neglected adding ``/parseinjavascript serverURL: process.env.SERVER_URL || 'https://projectname.appspot.com/parse```
because I thought it was taken care of here
javascript app.use(process.env.PARSE_MOUNT_PATH || '/parse', parseServer);


the memory usage of the wordpress machine

I use the free and default memory.  After I deployed the wordpress apache docker, I received an email like this once a day
Hello! Your “xxx” application hosted on Fly.io crashed because it ran out of memory. Specifically, the instance 81857550a10598. Adding more RAM to your application might help!

You should know that adding RAM does cost money. Not much, but some. Figure about $5/mo per extra GB. Our docs have [complete pricing details](https://fly.io/docs/about/pricing/?utm_campaign=oom-notification&utm_medium=email&utm_source=flyio#virtual-machines).

When you’re ready, add more RAM by running this from the command line or the [Fly Web CLI](https://fly.io/terminal):

fly scale memory [memoryMB] -a xxx

(replace [memoryMB] with the amount of memory you want to use, in megabytes eg. 2048)

Here’s the raw log message:

Out of memory: Killed process 1272 (apache2) total-vm:322300kB, anon-rss:24052kB, file-rss:0kB, shmem-rss:34308kB, UID:33 pgtables:312kB oom_score_adj:0

the graph, while I was trying to restart the machine

just share this until I find some solution.

That's interesting. Thanks.
I wonder what default values WordPress uses ... It would be a case of looking at the php.ini wp-config.phpetc and see if any of them use a value that assumes more memory is available.
The easy fix would indeed be to up the RAM to 512MB, however it should work with 256MB.


Git Module Icons Looks Bugged

The icons of Git module can look bugged depending on the font you are using. This issue happens due to the characters width that expands for more than one character. This issue can make it hard to see the status of git.

Fixed by commit 7f2ddb42e648b0bc0ac118b1c48ee863d6c88224.


Mlstubblefield fix/stream closed too early

In the company I work for, we usually log out the request body if an error has occurred. We do this by reading the respones stream off of the http context in a separate middleware. This isn't possible currently in hotchocolate due to the HttpPostMiddleware closing the stream (arguably when it should not). I made a change so that the stream remains to be open and owned by asp.net.
Happy to build upon this/give more context as needed :)
Also random question, would this impact other middleware handlers in the solution? I did some initial searching, but only found this place where I needed to change it.

First of all thank you for fixing that. We will include that into the next 10.4 hotfix release. For version 11 we will rewrite the middleware completely.

@PascalSenn @rstaib I will bring this into the master and than copy this change over to 10.

@mlstubblefield  thanks for contributing:


Add overview to core code structure

Added what I think we should have for some (not all) of the core bigchaindb files, as far as laying out the structure.
@vrde @r-marques @sbellem have a look.  Feel free to add more documentation directly.
@libscott would be great if you could take a look -- since you're new to the code, you can tell me what's understandable and what's not, what is missing that you would expect etc.

Current coverage is 95.85% (diff: 100%)

Merging #678 into master will not change coverage

@@             master       #678   diff @@
==========================================
  Files            21         21
  Lines          1157       1157
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1109       1109
  Misses           48         48
  Partials          0          0

Powered by Codecov. Last update 2acf37e...e063f53

I can add links to backlog etc. good catch.
Not sure where these belong.  Not sure if the readthedocs is the right place, since these are only really relevant to people contributing to the development of bigchaindb.  I wrote them as a response to the meeting monday, it seemed like a natural place.

I like the idea to have small README.mds to guide the developer through the dirs in the code.
:+1:


Use of @import

I mentioned this awhile ago and found a work around, but more and more frameworks appear to be using @import which fails with "Use of @import when modules are disabled." the failing library was imported with cocoapods and I looked at the targets and made sure the setting "Allow Non-modular includes in Framework Modules" was set to yes.  When building my project normally the build is successful. It's only my cedar specs target thats failing.
How can I test classes that use or have dependancies that use @import?

Hey @jkunzelman, the @import feature requires clang modules to be enabled. Clang modules are not well-supported for C++, so they are disabled by default, however it is possible to explicitly turn them on, which may or may not resolve your issues. To turn this on, add -fcxx-modules to the Other C++ Flags build settings in your spec target.
There's a real chance you will run into compiler bugs when doing this, but I have seen this work successfully in some basic smoke testing.

Is there a radar that we could duplicate so that apple is aware that people want to use @import from a C++ context?
Another alternative, albeit less exciting answer, is to switch to using Quick and Nimble, which are available for both Objective-C and Swift. It probably won't be easy, and it's not a magic bullet, because you'll still have some strange cases where things fail to compile for one reason or another, but it certainly would resolve your pain with @import, @jkunzelman.


Portal should have link to WebJobs dashboard

Not sure if we came up with exact UI yet.

35a331270d4e94dffc56eba40024fc14179446dd


RYA-377 Rya Streams

This is a first draft of the Rya Streams feature. It's a single node Kafka Streams implementation of an incremental SPARQL processor. It supports most of the query use cases of the Fluo app we've written, but it does not have any periodic support yet. We still need to hook historic import and PCJ/Construct output into the source Rya instance using Kafka Connect.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/580/


added /sys/class/power_supply/

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 99.69040% with 1 line in your changes missing coverage. Please review.

Project coverage is 93.42%. Comparing base (90dd3cf) to head (73a7a7f).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

src/sysfs/class_power_supply.rs
99.69%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   92.85%   93.42%   +0.56%
==========================================
  Files          26       27       +1
  Lines        3737     4060     +323
==========================================
+ Hits         3470     3793     +323
  Misses        267      267

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.