[Bug]: Leaking fluid pipes

Forge Version
<IP_ADDRESS>60
Galacticraft Version
<IP_ADDRESS>4
Log or Crash Report

Reproduction steps
TickHandlerServer.fluidNetworks has 1640136 instances of FluidNetwork most of which only reference one TileEntityFluidPipe, with hundreds of thousands of tile entities with the same position. Ticking all of these causes massive amounts of server lag, solved by a restart.
My working theory is that unloading the chunk is supposed to delete the tile entity, but it doesn't get GC'd due to the reference from TickHandlerServer.fluidNetworks then the chunk gets loaded and creates a new tile entity, which makes a new network, which gets saved to the list. Its also possible that when the network is ticked, it loads the chunk, making the issue worse.

Can you iterate what you mean by leaking pipe please. Is this happening on every pipe system in your world? What are the coords in reference too in the attached screenshot? (if they are coords). I didn't write the network code and have tried to not touch as much of it as possible as its a sheer mind-f^&k to go through, and this issue is the first i've heard of seen of this. Also re-produce this on the most current version please to continue any further.

Can you iterate what you mean by leaking pipe please.

Memory leak, tile entities that minecraft has removed are being prevented from being cleared by hard references in galacticraft code

What are the coords in reference too in the attached screenshot? (if they are coords).

Coords are locations in which fluid pipe tile entities exist, the count is the number of tile entities with those coords.

and this issue is the first i've heard of seen of this

Likely the same bug as micdoodle8/Galacticraft#3448. This issue most likely will only affect players who don't restart every 24 or so hours, and have pipes they load and then unload, making it not noticeable, but almost certainly present.

Also re-produce this on the most current version please to continue any further.

The relevant line of code has not been touched in 6 years.

Bug still exists, not stale

@adamsong do you still have the world this happened in?

I broke all the pipes that were causing issues


token sometimes is twice in the cookies

I suggest that this issue needs to be investigated

i have an idea for a quick fix, no idea if its good. no idea if it works. does make sense tho however should check out if this issue appears again or not.
=> idea: when joining a lobby b4 storing tokens and stuff just delete everything. This is already done when the user actually goes to the home page, but in some occasions this does not work so we can force to delete it before joining.


npm run-script serve.prod error

Trying to deploy on Ubuntu 14.14.
"npm start" - works OK.
Run the command "npm run-script serve.prod", getting below error.
Am I doing something wrong?
...
[11:47:44] Using gulpfile ~/projects/angular2-seed/gulpfile.ts
[11:47:44] Starting 'serve.prod'...
[11:47:44] Starting 'build.prod'...
[11:47:44] Starting 'clean.prod'...
[11:47:44] Deleted dist/prod
[11:47:44] Deleted dist/tmp
[11:47:44] Finished 'clean.prod' after 36 ms
[11:47:44] Starting 'tslint'...
[11:47:48] Finished 'tslint' after 4.05 s
[11:47:48] Starting 'css-lint'...
[11:47:50] Finished 'css-lint' after 1.75 s
[11:47:50] Starting 'build.assets.prod'...
[11:47:50] Finished 'build.assets.prod' after 73 ms
[11:47:50] Starting 'build.html_css'...
[11:47:51] Finished 'build.html_css' after 791 ms
[11:47:51] Starting 'copy.js.prod'...
[11:47:51] Finished 'copy.js.prod' after 41 ms
[11:47:51] Starting 'build.js.prod'...
[11:47:54] Finished 'build.js.prod' after 3.2 s
[11:47:54] Starting 'build.bundles'...
[11:47:57] Finished 'build.bundles' after 2.71 s
[11:47:57] Starting 'build.bundles.app'...
[11:48:15] Finished 'build.bundles.app' after 18 s
[11:48:15] Starting 'build.index.prod'...
[11:48:15] gulp-inject 2 files into index.html.
[11:48:15] gulp-inject 1 files into index.html.
[11:48:15] Finished 'build.index.prod' after 103 ms
[11:48:15] Finished 'build.prod' after 31 s
[11:48:15] Starting 'server.prod'...
[11:48:15] Finished 'server.prod' after 14 ms
[11:48:15] Finished 'serve.prod' after 31 s
Error: spawn ENOMEM
at exports._errnoException (util.js:949:11)
at ChildProcess.spawn (internal/child_process.js:302:11)
at exports.spawn (child_process.js:367:9)
at Object.exports.execFile (child_process.js:139:15)
at exports.exec (child_process.js:99:18)
at open (/home/safeuser/projects/angular2-seed/node_modules/open/lib/open.js:58:10)
at Server. (server.ts:54:5)
at Server.g (events.js:286:16)
at emitNone (events.js:86:13)
at Server.emit (events.js:185:7)
at emitListeningNT (net.js:1279:10)
at _combinedTickCallback (internal/process/next_tick.js:71:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
npm ERR! Linux 3.13.0-85-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run-script" "serve.prod"
npm ERR! node v6.1.0
npm ERR! npm  v3.8.6
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>serve.prod: gulp serve.prod --color
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>serve.prod script 'gulp serve.prod --color'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-seed package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp serve.prod --color
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-seed
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-seed
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/safeuser/projects/angular2-seed/npm-debug.log

@AntonyBaasan Have you tried just: npm run serve.prod ?
Is run-script a particular ubuntu thing?

@NathanWalker run is actually an alias of run-script npm command.

@ludohenin Thanks 👍  Seems strange that ubuntu would be that different than OS X... not sure what's going on with his issue.

Today I'll make a Vagrant script that anyone can test my situation.
@NathanWalker, what module is "open" /home/safeuser/projects/angular2-seed/node_modules/open/lib/open.js:58:10 ? I think I can dig from there.

@AntonyBaasan

ENOMEM

is an OS error code , as defined in kern/include/kern/errno.h ,which is returned due to insufficient memory.

Are you running the production build on a VM with limited memory? This seems like environment specific issue.

Hi @mgechev, Yep you are right. It is environment specific.
Bad hardware: DigitalOcean 5$ stack - 1gb CPU, 512mb RAM
Good hardware: DigitalOcean 10$ stack - 1gb CPU, 1gb RAM

Still, it is weird that 512MB RAM are not enough, there's also swap space on disk..

Might just be a bad node config, Max RAM should be set below the available amount (to keep a bit for te OS :))


Category tile changes the whole time and causes memory problems

When we open the category page we noticed that our browser had some issues because of an infinite loop.
We did some research and saw that the children() inside createCategoryView is called an enormous amount of times.
To reproduce this:

go to CategoryTileComponent
checkout the changes by implementing ngOnChanges inside this component.

ngOnChanges(c: SimpleChanges) {
    console.log('c', c);
  }

We have also included a screenshot to make this more clear. It would be great if you can have a look at this. Thank you in advance!

Hi Nicky.  We will have a look at this after the 0.20 release.

@nickyhaze we have now integrated the refactoring and it will be available in the upcoming version 0.22. There should be significantly less view updates now 😉

Great, thank you!


Scaffold with differences

The scaffold deliverable is great for getting an initial build set up on a new machine. However, it feels like it needs to go one step further to fit my use case.
I envisage that I would be able to create document types/templates etc. in my local copy of Umbraco, then scaffold the output which would result in a delivery file containing only the differences since last scaffold, that I can then commit to source control that CI/CD will then deploy onto a production environment.
I have tried doing a second scaffold (without the install step) and that runs through, but it only "installs" new document types and doesn't amend existing document types should I have made any changes to it.
Could it be that I am I going about things all wrong? Is there a suggested way to achieve what I'm trying to do?

Short answer - I'd love to do this, it's just really hard.
I had a look at trying to do this a few years back, had a crack (the branch is still around) but ultimately wasn't able to get it working to the point I was happy to merge it in. The challenge that you have is how do you determine the current state to perform the "diff" against.
Let's say you've got 5 deliveries, and you're preparing to do the 6th, well you need to work out what has changed since the last time. But the last time only contains the doctype News Item, because that was added then, and since then you've edited News Item and you've edited Home Page. Now we need to basically build up a picture of "last known state", so we have to replay all the DocType/Data Type/Macro/Template/etc. creates and edits so we know what it looked like and compare that to the current database.
So you can see where the complexity resides, the last delivery represents the last set of chances which is a delta on all the previous ones, meaning to get last complete state requires analysis of all the previous deltas.
Now you can get around this, or at least simplify it somewhat, by going down the route that uSync does, having only a single file for each thing you're tracking, and just keep updating that one file. But as I've said in the talks I've given on automating Umbraco with Chauffeur I don't agree with the uSync model as it makes it a lot harder to understand at what point in time a change happened, I prefer that level of granularity.
Maybe there are other ways to approach it that are possibly not as foolproof but can be close to reliable like looking at the timestamp in the tracking table and finding items that have update dates since then, but that then means you have to ensure you run all your deliveries locally too.
Yeah, it's tricky 😛.

😄 Thanks for the quick response.
This is similar thinking to mine... how to know if an item has been added or edited since the last Chauffeur scaffold was done.
I'll try creating a new deliverable to see if I can figure something out. But first I'll see if I can dig out your old branch first to make sure I don't go end up down the same path.

Consider the old branch dead, it's quite diverged from master so I probably should just delete it 😝

I opted to go with a tracking table that will be updated on saving an item (doc type/template etc.) and added a ChangesDeliverable to output the changes (which mostly mimics the ScaffoldDeliverable in many ways). I'm expecting to be able to run the DeliveryDeliverable as per usual, and it mostly works.
Any change to a template is carried across no problems (presumably as it's just replacing the file contents on disk).
However, a document type will only update if something is added. If a field is removed, it doesn't remove it at the destination. I suspect that what is happening here is that the DeliveryDeliverable is only appending and not replacing things that are in the database?
Am I a million miles away with my thought process?
(Apologies if I'm missing a blatantly obvious point, I've only been working with Umbraco for about a fortnight, so I'm still pretty new 😄 )

@jonifen I can see the direction that you're going with that approach, but I disagree with it slightly. I'm not a fan of adding another database table if I don't have to, so I wouldn't want to have that additional tracking table that you've introduced.
Instead, I want to follow what Umbraco provides on determining updates of the stuff that it tracks. Now I originally (and probably naively) assumed you could just rely on the umbracoNode table to get an updateDate and just follow from there. But of course, there isn't an updateDate in that table is there, meaning that we can't rely on it 🤦♂.
Not to be deterred through I did some digging and it looks like you can follow from the audit trail backwards (since it has a datestamp that you can work from). So I'd go with the approach of:

Check get all the document type ID's from the umbracoNode table
Cross-join them to the umbracoLog table
Where-clause against that for the appropriate date range
Use the newly found ID's in the Umbraco API to get the stuff for serialization
Repeat for other things

Ultimately this means that I wouldn't have to have Chauffeur running with the Umbraco event system, and I wouldn't have to worry about tracking with my own database table. Also, it's easier to write tests for that 😉.

Thanks for the pointers on this, I hadn't even noticed the UmbracoLog table 😊
I'll look into moving towards that idea instead.
I think either way of building the deliveries should still produce deliveries with the same expected content though, which leads me back to the delivering part of the process.
It doesn't appear to be importing the document types etc. correctly if things have been removed from it prior to the package being created. Templates etc. seem OK as they just get copied in and replace the existing file, but the types that are stored in the database seem to only get added or appended to, but never removed or truncated. Any ideas around that?

Ah sorry, missed that part of the comment.
Yes, removals are a bit... special. The export that you get from Umbraco of a doctype will only contain what the doctype looks like right now, so it doesn't know about any removed properties, that's simply not stored anywhere in the database. I addressed this as part of issue #58 by providing a way which you can remove a property from a doctype. It's a separate step which obviously makes it a bit more annoying.
Trying to automate the creation of that step though? Well, that becomes tricky. As I said originally the only truly foolproof way that you can do a "generate diff" step is that you have to replay all deliveries up until the present to know what "last known state" looks like, then compare the properties against each doctype and find the ones that were removed.
Otherwise, you have to do a bunch of stuff that tracks things while Umbraco runs, which as I said, I'm not really keen on doing.

I've just realised that this issue is still open and I've not touched Umbraco in quite some time and can't remember my thought process at the time. With that in mind, I think this issue can be closed 🙂
Given the time passed, I'll close it as stale.


Window Menu Bug on macOS 10.15.2

Describe the bug
The Window menu has many repeated, duplicate entries. This is similar to #566 and #593.
To Reproduce
TBD
Expected behavior
The menu should not have so many duplicates.
Screenshots

Environment (please complete the following information):

Vim version:

VIM - Vi IMproved 8.1 (2018 May 18, compiled Oct 30 2019 11:57:56)
macOS version
Included patches: 1-2234
Compiled by<EMAIL_ADDRESS>Huge version with MacVim GUI.  Features included (+) or not (-):
+acl               +file_in_path      +mouse_urxvt       -tcl
+arabic            +find_in_path      +mouse_xterm       +termguicolors
+autocmd           +float             +multi_byte        +terminal
+autochdir         +folding           +multi_lang        +terminfo
-autoservername    -footer            -mzscheme          +termresponse
+balloon_eval      +fork()            +netbeans_intg     +textobjects
+balloon_eval_term +fullscreen        +num64             +textprop
+browse            -gettext           +odbeditor         +timers
++builtin_terms    -hangul_input      +packages          +title
+byte_offset       +iconv             +path_extra        +toolbar
+channel           +insert_expand     +perl/dyn          +transparency
+cindent           +job               +persistent_undo   +user_commands
+clientserver      +jumplist          +postscript        +vartabs
+clipboard         +keymap            +printer           +vertsplit
+cmdline_compl     +lambda            +profile           +virtualedit
+cmdline_hist      +langmap           +python/dyn        +visual
+cmdline_info      +libcall           +python3/dyn       +visualextra
+comments          +linebreak         +quickfix          +viminfo
+conceal           +lispindent        +reltime           +vreplace
+cryptv            +listcmds          +rightleft         +wildignore
+cscope            +localmap          +ruby/dyn          +wildmenu
+cursorbind        +lua/dyn           +scrollbind        +windows
+cursorshape       +menu              +signs             +writebackup
+dialog_con_gui    +mksession         +smartindent       -X11
+diff              +modify_fname      -sound             -xfontset
+digraphs          +mouse             +spell             +xim
+dnd               +mouseshape        +startuptime       -xpm
-ebcdic            +mouse_dec         +statusline        -xsmp
+emacs_tags        -mouse_gpm         -sun_workshop      -xterm_clipboard
+eval              -mouse_jsbterm     +syntax            -xterm_save
+ex_extra          +mouse_netterm     +tag_binary
+extra_search      +mouse_sgr         -tag_old_static
-farsi             -mouse_sysmouse    -tag_any_white
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/Applications/MacVim.app/Contents/Resources/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_MACVIM -Wall -Wno-unknown-pragmas -pipe  -DMACOS_X -DMACOS_X_DARWIN  -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang   -L. -fstack-protector-strong -L/usr/local/lib -L/usr/local/opt/libyaml/lib<EMAIL_ADDRESS>-L/usr/local/opt/readline/lib -L. -fstack-protector-strong -L/usr/local/lib -L/usr/local/opt/libyaml/lib<EMAIL_ADDRESS>-L/usr/local/opt/readline/lib  -L/usr/local/lib -o Vim -framework Cocoa -framework Carbon       -lm  -lncurses -liconv -framework AppKit   -fstack-protector  -L/System/Library/Perl/5.18/darwin-thread-multi-2level/CORE

OS: macOS 10.15.2
Terminal: GUI

Additional context
This is very reminiscent of a previous bug in this menu; however in this case I did not open windows that were already open (which is my reliable way of triggering the previous bug). I will update this thread once I find a way to reliably reproduce this.

@isaki Thank you for an issue,
This is a known bug (#566). The original issue was about the Full Screen, this is an additional menu which is broken. I bet the source and the way to reproduce is the same

I can reproduce it when no windows are open. Keep opening and closing the same file from the command line.

Steps Taken:

touch a file (call it foo.txt)
Repeatedly open and close foo.txt with /Applications/MacVim.app/Contents/bin/gvim
See the above behavior when all windows are closed

I'm still working on reproducing the version with Windows open. Will update when I have more.

I've got it for windows open as well.

Steps Taken:

touch 2 files (foo1.txt, foo2.txt)
Open both files with /Applications/MacVim.app/Contents/bin/gvim
Repeatedly open and close foo2.txt
See the above behavior

@isaki It's true, this menu items are connected to the same source.
These additional menus are added and removed by macOS event handlers. The additional check should be done here. AFAIK, any other menus are not duplicated accidentally

@eirnym Please let me know if there is something you want me to test or if you need more information. I'm more than happy to help in any way I can!

This seems to be the same bug as the other listed ones (#566), except in Catalina it is worse. I actually haven't upgraded to Catalina yet, but I probably should, but it's really the same issue (I think).


Is there a model for more landmarks?

Thanks a lot for your kind sharing. But the model you shared is for 29 landmarks. Could you share a model for more landmarks, say, 64 or 83 landmarks?

It seems the sample results you gave have more landmarks. What dataset did you use for training? Can you please send the trained model for the sample results to me<EMAIL_ADDRESS>Thank you!

Hello, can you provide a model with more than 50 and less than 100 landmarks?Thank you very much!


Add WebSocket support to IUserContextBuilder

@sungam3r Let me know if you want to review this here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

80 of 82 (97.56%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 95.676%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/GraphQL.AspNetCore3/UserContextBuilder.cs
19
21
90.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
1661

Relevant Lines:
1697

💛  - Coveralls


Stack toggle with left-alignment

I mapped NERDComment as follows:
map <leader># :call NERDComment(0,"toggle")<C-m>

Which works as expected except that it violates my own debug convention: temporarily-commented line should have comment start before indentation. This convention allows to easily parse the file visually and determine which lines need to be uncommented/removed before checking in the code (because they won't match the indent).
I noticed that NERDCommenter also has a LeftAlign option, which does exactly the kind of alignment I want. Problem: it doesn't toggle. How can I stack the two options?

As of #24 being merged there is now a config option for this. Try let g:NERDDefaultAlign = 1 or similar, and see the help documentation for details.


Add missing example as a service

Hi, I think it would be great to add some documentation in relation to add menu as a service like explained in the official KnpMenu documentation: Creating Menus as Services.
Here is an example of my configuration:
app.menu_builder:
        class: AppBundle\Menu\MenuBuilder
        arguments<EMAIL_ADDRESS><EMAIL_ADDRESS>
    app.main_menu:
        class: Knp\Menu\MenuItem # the service definition requires setting the class
        factory<EMAIL_ADDRESS>createMainMenu]
        tags:
            - { name: knp_menu.menu, alias: main } # The alias is what is used to retrieve the menu

If it can help people to do the same and not spent hours to find the right configuration!

You're right.  This would help.  (I take it that you "spent hours to find the right configuration".  If so, I sympathize.)
I'll work on this sometime, or you can submit a pull request to augment the documentation.  Either way this will get into the docs eventually.
Thanks, Kevin

Hi Kevin, I'll send you a PR with the updated documentation with this example as soon as possible.


Cursor overlay: only swallow pointer events when the cursor is set

Fixes #[ticket_number]
Problem:
Due to a code change introduced by ..., CursorOverlay swallows pointer events whenever there's no image drag session in progress (either dragging from Finder or form UTFB)
Fix:
Only set pointerEvents to all on the cursor overlay if cursor is set to a non-null value and no image drag is in progress
Commit Details:

Update cursor-overlay.tsx according to Fix

@bkrmendy it's probably nothing, but I have vague memories that the overlay was at one point also used to to the right thing w/r/t dragging around sliders and colour picker controls in the inspector. I.e. if you do drag those around and over-drag onto the canvas, the mouse movement shouldn't trigger canvas highlights.
I don't think we had test coverage for that, so it's entirely possible that "unrelated" changes like this could regress is - just give it a quick check before opening up the PR as non-draft please.


Maintenance: review and restore PR automation

Summary
As a result of the work done in #1799 some of the automation related to PRs is broken and is no longer running.
For example, the workflow that is expected to run when a new PR is opened is failing due to permission issues:
https://github.com/aws-powertools/powertools-lambda-typescript/actions/runs/7540911226
Why is this needed?
So that workflows related to the PRs is automated as it was before.
Which area does this relate to?
Automation
Solution
We should review the workflows and related permissions so that they run as intended.
Acknowledgment

[X] This request meets Powertools for AWS Lambda (TypeScript) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

This was fixed with #1978 & #1979


Proposal to Restart Static Site Migration Using Eleventy and CloudCannon

Dear community members,
A project to migrate the StackStorm WordPress website to a static site hosted on GitHub Pages was initiated last year but remains to be completed. Today, the website exists as a static site. However, the HTML files contain leftover artifacts from WordPress that make new content additions or changes to existing content extremely tedious and time-consuming. The original project intent was to replicate the StackStorm WordPress website's visual design by developing a custom Hugo template/theme along with Netlify integration (headless CMS) for content preview prior to GitHub Pages deployment.
A year has passed in which no community member has volunteered to create a custom Hugo template/theme nor has any member volunteered to work out the Netlify integration. In order to achieve our ultimate goal of a community website open to content contributions from across the StackStorm community, I would like to propose an alternative approach.
Here is what I am proposing:
1. Restart the Project from Scratch:
I have access to an exact copy of the original WordPress site and I am ready to restart the WordPress to static site migration project in a local environment whilst the current public website remains - "as is".
2. Adopt Eleventy (11ty) for Static Site Generation:
Given its flexibility and simplicity, Eleventy would more easily facilitate custom template/theme creation with its nearly unlimited support for templating languages that can be used together and independently in a single project. This templating support includes Nunjucks as a port of Jinja2 (a favored templating language among community members).
3. Utilize CloudCannon's CMS:
CloudCannon's user-friendly CMS interface would make it easier for non-technical contributors to contribute content to the website. This aligns with our goal of welcoming content contributions from across the StackStorm community, i.e. sharing use cases, case studies and varied forms of content that tell the story of StackStorm's impact. CloudCannon's native GitHub integration facilitates a streamlined collaborative workflow while reducing or eliminating barriers to broader community engagement. And, we can leverage existing work on CI/CD GitHub Actions pipelines that deploy automatically to GitHub Pages.
4. Address All Currently Open Issues:
Acceptance of this proposal allows us to consolidate and ultimately close the (7) existing open Issues related to the unfinished WordPress site migration, clearing the way for a fresh start.
I understand the community's affinity for Go and Hugo (as opposed to JavaScript-based Eleventy) and I respect that. However, given our current challenges and stalled progress on the Hugo front, I believe Eleventy and CloudCannon offer a practical and efficient path forward with benefits that extend well beyond underlying technology preferences.
I kindly invite the community to consider these points and share your thoughts, concerns, or questions. Together, I'm confident we can revive the website in a way that aligns with the original vision of creating a welcoming space for broader community engagement.
Thank you (in advance) for your consideration.

1 - Restart the migration: ✅ Awesome. Sounds good to me 👍
2 - Using Eleventy vs Hugh: This is where I think we need to focus discussion.
3 - CloudCannon. Let's punt on this until later - If something like CloudCannon requires a particular kind of template, that would be good to know. But other than that, I think we can table that until we've finished migrating to a Static Site Generator.
4 - 🎉 Hooray for closing issues and making progress on stackstorm.com migration to github pages.

To recap a few notes from the previous TSC meeting and adding more details about pros/cons:

2. Adopt Eleventy (11ty) for Static Site Generation

Based on the discussions, Eleventy is a Javascript-based tool (npm).
Historically, the Javascript-based tools didn't work well with the StackStorm Python opensource community. It could be because the average st2 user is DevOps/SRE/Security/backend folk and so the skillset is different in this group.
At this moment I don't know any TSC members who's happy or interested to work with the Javascript tools.
Three examples:

st2chatops - abandoned, despite being critical core component of the StackStorm.
st2web - stackstorm core component, not maintained, just a few contributions by the partners. Lots of security issues and update problems.
api.stackstorm.com generator - something broke in the upstream dependencies resulting in a broken build. It was fixed once, but nobody was ever curious to look at it and fix for 2 yrs.
^^ all this Javascript-based repos have huge amount of technical debt already like updates, security, bugs, etc.

From the other side, with the most recent talks, what I liked about Eleventy is that it can work with the Jinja templating. I also took a look closer and, sadly, despite our conversations, it doesn't work like a React-like in-browser rendering, so just a static site generator as any other tool.
My impression is that Eleventy is a technically superior, but technically ideal is not always the best choice, considering the team/expertize context. What would work for sure is the toolset that's more likely to be used long-term by the TSC, Maintainers and Contributors. Every tool/process that's introduced is something that's needed to be then maintained by the TSC and so people optimize for the lower maintenance burden where possible.
Usage statistics by the platform shows Eleventy as unpopular one compared to Hugo:

In the past when someone proposed Hugo, - it got immediate agreement as a first go-to system candidate, because its popularity, Go/toolset and wide familiarity working with it.
^^ Something to keep in mind when making the informed pros/cons decision.

3. Utilize CloudCannon's CMS:

a)
Yeah, the advantage with the CMS is the ability to do nice management of the website content.
A few concerns about CloudCannon:

Security. We're trying to make sure any 3rd party tool can't write to the repo. Is it a Github pull-based service only or needs write Github integration/access?
CloudCannon is paid. We're trying to avoid any 3rd party or paid services for maintenance/bus factor reasons as well. See https://github.com/StackStorm/community/issues/36. Any paid system (even sponsored) is considered a risk as it increases dependency and amount of moving pieces, harder to manage.
With paid system onboarding/offboarding users is a difficulty. It should be an outside contributor and community friendly, which is not the case for CloudCannon.
Ideal fit for OSS community like StackStorm is allowing people to run the CMS as an (optional) helper locally on their machine and propose a PR with the changes based on that. Examples: https://github.com/julianoappelklein/hokus, https://quiqr.org/, https://github.com/tinacms/tinacms

Overall it feels more like an optional thing, wondering to know how the workflow may look like with this new CMS.
b)
Netlify, mentioned in the Enable Netlify integration (preview) #3 is free for a starter plan and if we'd integrate it, - we need it just for the PR preview as a quick visual check (pull-based, no write permissions needed).
See: https://www.netlify.com/pricing/#pricing-table-full-feature-list
Overall I agree with @cognifloyd that website generation tool goes first.

My take on this discussion is always making sure we're acting in the best interest of the StackStorm community/project long-term. Based on history dealing with various tools around StackStorm, there are more widely adopted tools. What was proposed by the previous maintainers in the past and after taking a closer technical look today, - I agree those would fit the project better 💯

@armab With all due respect, the logic of your argument would suggest that a particular technology, in this case JavaScript, should be held responsible for StackStorm project neglect.
Is it really necessary to remind everyone that Hugo is NOT a JavaScript-based technology. Yet, despite this fact, the StackStorm website SSG project has been no less neglected for well over a year now. How is it possible for this neglect to occur in the absence of JavaScript as the culprit?
I would suggest that blaming project neglect on specific technologies that some individuals personally dislike is unlikely to positively contribute to the openness, growth and vitality required for a Community to thrive and remain relevant into the future.
Furthermore, the puzzling and potentially misleading equivocation between technologies involved with StackStorm core and technologies involved with the StackStorm website are beyond comprehension. These are (2) completely independent, separate and distinct technology domains. It is no secret that the (3) technologies fundamental to pretty much every website in existence are - HTML, CSS and yes - JavaScript!
If one were to take seriously the argument put forth above, then the StackStorm website should be reconstituted in Golang for no other reason than to appease the personal biases and technology comfort zones of some individuals involved with StackStorm core.
In all honesty, I can easily visualize that project neglect within an open source Community has high correlation with the "openness" of that Community to contributors willing to invest their time, skills, energy, blood, sweat and tears to further the aims and goals of that Community. When contributor willingness to contribute is met with obstacles and obfuscations, then they will obviously (and rightly) move on to other projects offering less friction and resistance. It is as simple as that.
Today, a casual observer of the StackStorm Community could rightly question whether this Community honestly intends to remain relevant well into the future. The current trajectory of creeping obscurity will eventually be succeeded by an obvious and irreversible atrophy, at which point it will be too late to identify a culprit technology to hold accountable for pervasive project neglect.
Lastly, I am no less motivated, no less interested and no less enthusiastic to proceed with the StackStorm SSG project as proposed above.
However, I certainly respect that this Community's leadership has every right to consciously prefer that the SSG website project remain neglected rather than accommodate contributors whose skill sets and technology choices (specific to this website project) may differ from the leadership's technology preferences (specific to StackStorm core).

the logic of your argument would suggest that a particular technology, in this case JavaScript, should be held responsible for StackStorm project neglect.

No, there is no connection between the specific technology (Javascript, Golang, Python, etc) and why the initial https://github.com/StackStorm/community/issues/76 project wasn't finalized. The project was left in between because people might switch focus to other priority tasks, have no time anymore, interest, didn't see enough value, or leave the project, whatever it is. By migrating out from Wordpress to GH static pages the most important achievement was done and we solved a lot of problems already like security, access control, transparency, ability for external collaborators to contribute and change the website via PR. The next task was to add a Hugo theme, which would make the process of adding new blog posts easier and friendlier. The task is https://github.com/StackStorm/stackstorm.com/issues/4.
Speaking of the choices. Every team is unique and has its unique technology radar where they prioritize specific languages and technologies they're already familiar, have experience with or want to work with in the future. Check out https://dzone.com/articles/what-is-tech-radar-why-teams-need-to-have-one, an interesting read. Javascript is lowest in our list and we're not just trying to avoid it, but also replace some of the Javascript projects with the other technology where possible. Your effort to bring another node framework doesn't fit that strategy plan and vision. We reviewed your proposal and several people gave feedback that initial Hugo still fits better compared to Eleventy. It's not that technology or language is bad, it's just something that will be easier and higher chances to develop as a larger group and reduce the bus factor and maintenance burden (check out https://www.getclockwise.com/blog/what-is-your-bus-factor). Every adopted tech is something that automatically becomes a responsibility and will be maintained by the members and contributors and where team members should be able to collaborate on. This is an important aspect.


Drop Destination not available during Import process

Hi.
It happens to me frequently that my drop destination isn't available because I forgot to turn on my NAS or other reasons and if thats the case JMM Server will leave the files at the drop source which is good, but JMM Server seems to never check when my drop destination is available and will try to move the files to its destination.
So I am using the following SQL Script:
insert into CommandRequest (Priority, CommandType, CommandID, CommandDetails, DateTimeUpdated)
select 3, 1, 'CommandRequest_ProcessFile_' + cast(VideoLocalID as nvarchar(max)), '<CommandRequest_ProcessFile><VideoLocalID>'+cast(VideoLocalID as nvarchar(max))+'</VideoLocalID><ForceAniDB>false</ForceAniDB></CommandRequest_ProcessFile>', GETDATE()
from VideoLocal where ImportFolderID in (select ImportFolderID from ImportFolder where IsDropDestination = 0) and VideoLocal.Hash in (select xref.Hash from CrossRef_File_Episode xref)

and restart JMM Server upon start JMM Server will start copying the files to the destination.

Scan drop folder or Run Import, will do what you want.

I tried "Run Import" not really sure where I could find "Scan drop folder" ?!?

Click the import folder icon to the right of the import folder's name.


Automount volumes

What type of PR is this:
/kind feature
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes partially #6549
PR acceptance criteria:

[x] Unit test

[x] Integration test

[x] Documentation

How to test changes / Special notes to the reviewer:

/hold until consensus on label/annotation names

/hold cancel

/override windows-integration-test/Windows-test
[FAILED] [211.331 seconds]
E2E Test starting with non-empty Directory add Binding [It] should verify developer workflow of using binding as env in innerloop
/go/odo_1/tests/e2escenarios/e2e_test.go:337

/override OpenShift-Integration-tests/OpenShift-Integration-tests
Errors on files test cleanup
[FAILED] [128.660 seconds]
odo devfile deploy command tests [AfterEach] deploying devfile with long-running exec when Automount volumes are present in the namespace should mount the volumes
  [AfterEach] C:/Users/Administrator.ANSIBLE-TEST-VS/3649/tests/integration/cmd_devfile_deploy_test.go:36
  [It] C:/Users/Administrator.ANSIBLE-TEST-VS/3649/tests/integration/cmd_devfile_deploy_test.go:663

/unhold
I changed the labels /annotations with the new ones

LGTM, but I quickly wanted to check how this would behave with Service Binding, which, I remembered might also alter the Pod spec to inject envFrom using a secret:
odo init --devfile nodejs --starter nodejs-starter --name automount-volumes-nodejs
odo add binding --service edb-cloud-native-postgresql.Cluster.postgresql.k8s.enterprisedb.io \
  --name automount-volumes-nodejs-edb-cloud-native-postgresql \
  --service-namespace shared-services \
  --bind-as-files=false

If I don't have any automounting configmap/secret, it works as expected:
$ kubectl get pod automount-volumes-nodejs-app-68d4c554b-kldjk -o yaml
[...]
    envFrom:
    - secretRef:
        name: automount-volumes-nodejs-edb-cloud-native-postgresql-fb57c24f

$ kubectl exec -it automount-volumes-nodejs-app-68d4c554b-kldjk -- /bin/sh -c 'env | grep CLUSTER | sort'
Alias tip: k exec -it automount-volumes-nodejs-app-68d4c554b-kldjk -- /bin/sh -c 'env | grep CLUSTER | sort'
CLUSTER_CA.CRT=-----BEGIN CERTIFICATE-----
CLUSTER_CA.KEY=-----BEGIN EC PRIVATE KEY-----
CLUSTER_CLUSTERIP=<IP_ADDRESS>
CLUSTER_DATABASE=app
CLUSTER_HOST=edb-cloud-native-postgresql-rw
CLUSTER_PASSWORD=3IxAlTt9AUeVNvPS1TVrptKCUxUMXWIkE37cQgq3orNfZcbKHX4JJOHV8pqD0xK6
CLUSTER_PGPASS=edb-cloud-native-postgresql-rw:5432:*:postgres:3IxAlTt9AUeVNvPS1TVrptKCUxUMXWIkE37cQgq3orNfZcbKHX4JJOHV8pqD0xK6
CLUSTER_PROVIDER=enterprisedb
CLUSTER_TLS.CRT=-----BEGIN CERTIFICATE-----
CLUSTER_TLS.KEY=-----BEGIN EC PRIVATE KEY-----
CLUSTER_TYPE=postgresql
CLUSTER_USERNAME=postgres

If I stop the previous Dev session and apply some automounting configmap/secret (I applied the resources you used in the tests of this PR), it looks like the environment variables expected from the ServiceBinding are no longer injected.
$ kubectl apply -f /path/to/odo/tests/examples/manifests/config-automount/
$ odo dev
$ kubectl get pod automount-volumes-nodejs-app-67f467566c-gjwxx -o yaml
[...]
    envFrom:
    - secretRef:
        name: automount-env-secret
    - configMapRef:
        name: automount-env-configmap

$ kubectl exec -it automount-volumes-nodejs-app-67f467566c-gjwxx -- /bin/sh -c 'env | grep CLUSTER | sort'
$

Do you have the same issue? Or is that intended? Or I might have missed something too.

It works as expected when binding as volumes (bindAsFiles to true):

With no automounting configmaps/secrets:

  volumes:
  - name: odo-projects
    persistentVolumeClaim:
      claimName: odo-projects-automount-volumes-nodejs-app
  - emptyDir: {}
    name: odo-shared-data
  - name: automount-volumes-nodejs-edb-cloud-native-postgresql
    secret:
      defaultMode: 420
      secretName: automount-volumes-nodejs-edb-cloud-native-postgresql-caf68dfb

With automounting configmaps/secrets:

volumes:
[...]
  - name: auto-secret-automount-mount-path-secret
    secret:
      defaultMode: 420
      secretName: automount-mount-path-secret
  - name: auto-secret-automount-subpath-secret
    secret:
      defaultMode: 420
      secretName: automount-subpath-secret
  - configMap:
      defaultMode: 420
      name: automount-default-configmap
    name: auto-cm-automount-default-configmap
  - configMap:
      defaultMode: 420
      name: automount-mount-path-configmap
    name: auto-cm-automount-mount-path-configmap
  - configMap:
      defaultMode: 420
      name: automount-subpath-configmap
    name: auto-cm-automount-subpath-configmap
  - name: automount-volumes-nodejs-edb-cloud-native-postgresql
    secret:
      defaultMode: 420
      secretName: automount-volumes-nodejs-edb-cloud-native-postgresql-caf68dfb
[...]

If I stop the previous Dev session and apply some automounting configmap/secret (I applied the resources you used in the tests of this PR), it looks like the environment variables expected from the ServiceBinding are no longer injected.

I'm pretty sure the service binding operator is using an Update command to set the envfrom field. Because of using Update and not Apply, the values set by the operator are overwritten by odo.
$ kubectl get deployments.apps automount-volumes-nodejs-app -o yaml --show-managed-fields
[...]
  managedFields:
  - apiVersion: apps/v1
    fieldsType: FieldsV1
    fieldsV1:
      f:spec:
        f:template:
          f:spec:
            f:containers:
              k:{"name":"runtime"}:
                f:envFrom: {}
    manager: manager
    operation: Update
    time: "2023-04-17T09:53:51Z"

If I stop the previous Dev session and apply some automounting configmap/secret (I applied the resources you used in the tests of this PR), it looks like the environment variables expected from the ServiceBinding are no longer injected.

I'm pretty sure the service binding operator is using an Update command to set the envfrom field. Because of using Update and not Apply, the values set by the operator are overwritten by odo.

Just to make sure I understand - is this an issue in odo or SBO ?

$ kubectl get deployments.apps automount-volumes-nodejs-app -o yaml --show-managed-fields
[...]
  managedFields:
  - apiVersion: apps/v1
    fieldsType: FieldsV1
    fieldsV1:
      f:spec:
        f:template:
          f:spec:
            f:containers:
              k:{"name":"runtime"}:
                f:envFrom: {}
    manager: manager
    operation: Update
    time: "2023-04-17T09:53:51Z"

@rm3l I think it would be better to create a separate issue to track this problem. I don't think I would be able to fix it as part of thir PR

Sure anyway - let's track this in a separate issue.
Thanks.

/override windows-integration-test/Windows-test
Flaky tests

If I stop the previous Dev session and apply some automounting configmap/secret (I applied the resources you used in the tests of this PR), it looks like the environment variables expected from the ServiceBinding are no longer injected.

I'm pretty sure the service binding operator is using an Update command to set the envfrom field. Because of using Update and not Apply, the values set by the operator are overwritten by odo.

Just to make sure I understand - is this an issue in odo or SBO ?

I think the problem is in SBO side, which should use Server-Side Apply insteof of an Update.

/lgtm

/override windows-integration-test/Windows-test

/lgtm


Custom Resource values not passed through to job input

Using the custom resource specification
#PBS -lselect=1:ncpus=256:mem=900gb
#PBS -lwalltime=72:0:0
the created Gaussian input file at https://openondemand.rcs.ic.ac.uk/pun/sys/chemistry_portal/directory/116/ has the headers
%NProcShared=
%Mem=0MB
%chk=methane.chk
where the first two original values present in the  uploaded .gjf file of  e.g.
%nprocshared=128
%mem=20GB
%chk=methane.chk
have been replaced by null values which induces an error.
PS The only allowed value for ncpus in Express queues is 256 (128 is invalid)  whereas the  Gaussian code only allows up to  128 (256  is invalid).  Matt assured me however that the job would only be charged for 128 cpus and not 256.  Can you in fact query whether this is correct?

Get
• Invalid line: 'export NCORES=256 '

On 12 Aug 2021, at 10:32, Christopher Cave-Ayland @.***> wrote:
You'll need to add the below lines to the resource specification.
export NCORES=256
export MEMORY_GB=900gb
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Ah, restart and try it again.

Get
Job submission failed /opt/ood/ondemand/root/usr/share/gems/2.7/ondemand/2.0.13/gems/ood_core-0.17.2/lib/ood_core/job/adapters/pbspro.rb:418:in rescue in delete': qdel: illegally formed job identifier: (OodCore::JobAdapterError) from /opt/ood/ondemand/root/usr/share/gems/2.7/ondemand/2.0.13/gems/ood_core-0.17.2/lib/ood_core/job/adapters/pbspro.rb:414:in delete' from /var/www/ood/apps/sys/chemistry_portal/main/ruby_scripts/delete.rb:8:in `
' /opt/ood/ondemand/root/usr/share/gems/2.7/ondemand/2.0.13/gems/ood_core-0.17.2/lib/ood_core/job/adapters/pbspro.rb:188:in call': qdel: illegally formed job identifier: (OodCore::Job::Adapters::PBSPro::Batch::Error) from /opt/ood/ondemand/root/usr/share/gems/2.7/ondemand/2.0.13/gems/ood_core-0.17.2/lib/ood_core/job/adapters/pbspro.rb:163:in delete_job' from /opt/ood/ondemand/root/usr/share/gems/2.7/ondemand/2.0.13/gems/ood_core-0.17.2/lib/ood_core/job/adapters/pbspro.rb:415:in delete' from /var/www/ood/apps/sys/chemistry_portal/main/ruby_scripts/delete.rb:8:in
But the job does enter the queue, although it then claims its running but produces no outputs?

On 12 Aug 2021, at 13:27, Christopher Cave-Ayland @.***> wrote:
Ah, restart and try it again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

For
https://openondemand.rcs.ic.ac.uk/pun/sys/chemistry_portal/directory/121/

On 12 Aug 2021, at 13:27, Christopher Cave-Ayland @.***> wrote:
Ah, restart and try it again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Looks like the underlying error was:
Job submission failed qsub: Job resource selection does not match any permitted configuration. Please review the job sizing guidance on: https://www.imperial.ac.uk/admin-services/ict/self-service/research-support/rcs/computing/
I've tweaked things so you should get the more informative error after a restart.
As far as I can see the job doesn't actually make it to the queue.

Chris,
Ah,  the  Custom job script and the Custom resource are not  cross-checked.
Thus if one requests the latter which exceeds normal limits  without also specifying the former, the job will fail.
BOTH have to be specified for success, but there are no warnings if an inappropriate combination of the two is used.  A hole a user can easily fall into.
Is there any code that could trap this condition that could be inserted?
The issue now  is that one has to request 256 cpus from the resource (because 128 is not allowed) but only  128 in Gaussian (because 256 is not allowed).  I have put in a normal help desk query to sort that one out!
I am now getting the following
https://openondemand.rcs.ic.ac.uk/pun/sys/chemistry_portal/directory/128/
The gjf file submitted initially contains
%nprocshared=128
%mem=20GB
%chk=methane.chk
wB97XD/STO-3G opt nmr freq output=wfn
Test for methane
0 1
C                 -3.71007372    1.65847663    0.00000000
H                 -3.35341930    0.64966663    0.00000000
H                 -3.35340088    2.16287482    0.87365150
H                 -3.35340088    2.16287482   -0.87365150
H                 -4.78007372    1.65848982    0.00000000
a.wfn
but when it runs it has been re-written to
%NProcShared=256
%Mem=0MB
%chk=methane.chk
wB97XD/STO-3G opt nmr freq output=wfn
Test for methane
0 1
C                 -3.71007372    1.65847663    0.00000000
H                 -3.35341930    0.64966663    0.00000000
H                 -3.35340088    2.16287482    0.87365150
H                 -3.35340088    2.16287482   -0.87365150
H                 -4.78007372    1.65848982    0.00000000
a.wfn
which fails because Gaussian cannot cope with  256 and also the memory is reset to 0MB.
So its the re-writting of the input file that is the issue here.
SUCCESS (of a sort) at https://openondemand.rcs.ic.ac.uk/pun/sys/chemistry_portal/directory/129/
Specifying the following
#PBS -N portal_job_00000129
#PBS -q express
#PBS -P exp-00049
#PBS -lselect=1:ncpus=256:mem=900gb
#PBS -lwalltime=72:0:0
export NCORES=128
export MEMORY_GB=200gb
produces the following input file
%NProcShared=128
%Mem=0MB
%chk=methane.chk
wB97XD/STO-3G opt nmr freq output=wfn
Test for methane
0 1
C                 -3.71007372    1.65847663    0.00000000
H                 -3.35341930    0.64966663    0.00000000
H                 -3.35340088    2.16287482    0.87365150
H                 -3.35340088    2.16287482   -0.87365150
H                 -4.78007372    1.65848982    0.00000000
a.wfn
So the CPUs is correct, but the memory is  NOT.
Can you check the code that rewrites the memory in the  GJF file?
Henry

On 12 Aug 2021, at 14:57, Christopher Cave-Ayland @.***> wrote:
Looks like the underlying error was:
Job submission failed qsub: Job resource selection does not match any permitted configuration. Please review the job sizing guidance on: https://www.imperial.ac.uk/admin-services/ict/self-service/research-support/rcs/computing/
I've tweaked things so you should get the more informative error after a restart.
As far as I can see the job doesn't actually make it to the queue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Ah sorry that's my bad. The line in the resource config should just be:
export MEMORY_GB=900

without the gb at the end. That should sort the memory.


remove comma "," when not using proxy

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/9095
Tests showed that comma after URL is not needed when no proxy is used.

@JohanFreelancer9 please do the copy editor review for this PR.
Thank you

#sign-off

@Dansimp PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

#sign-off

@illfated could you please help me with the conflicts? I don't know what i need to remove/change to resolve the conflicts.

https://github.com/VLG17/windows-itpro-docs/tree/patch-126 : This branch is 1 commit ahead, 1024 commits behind MicrosoftDocs:public.
It may just as well be easier to close this PR and create a new one based on the existing document page.
Had you been using any Git tools to keep your branches updated, it might have been easier to give advice.

I mean, I could try editing your PR branch document and see if it would be possible to update it to current content, but it could also cause a lot of merge conflicts in my tree/fork/branch tool "Git Extensions", simply because of the amount of changes this page has been through in the meanwhile.


Fix blood group order and minimum dropdown height

Puts 'I dont know' to the top and then adds 20px to the minimum drop down height in order to partially expose any additional items.

@andyguest reverted the order


Don't accept str in FromPyObject for Vec<&str> and Vec<String>

As per the title, it would be nice if we didn't accept Python str in FromPyObject for Vec<&str> and Vec<String>. Some background below.
I'm not sure how hard this will be (without specialization).
Discussed in https://github.com/PyO3/pyo3/discussions/2341

Originally posted by atoav April 27, 2022
In python one can iterate over a str as if it was a List[str]. And while this is occasionaly useful it can also be a major footgun if users pass in a "string" instead of a ["string"] and suddenly have a function operate on single characters instead of the whole string, as they might (admittedly, errouniously) assumed one that one string.
In python I'd prevent this user error  by ensuring incoming str are never treated as a list of characters by wrapping a lone str in a list as follows:
if isinstance(strings, str):
    strings = [strings]

In pyo3 this is an issue as well. Let's say i have a simple function that should take a Vec<String> and print out each string in the vec:
/// Print out each string in a list of strings
#[pyfunction]
fn print_strings(strings: Vec<String>) {
    for s in strings {
        println!("{}", s);
    }
}

If I now run this in the python interpreter with a string that is not a list each character is printed out seperately:
>>> from my_module import print_strings
>>> print_strings("string")
s
t
r
i
n
g

Is there any simple way to ensure single str are treated as [str] using pyo3? Even an Error would be acceptable, as long as users don't pass a string and have it iterate over characters without them noticing.

Hi! Shouldn’t this be closed as #2500 was merged?


Layer Validation: Add Valid Usage Cases and Tests for INVALID_MEM_OBJ and vkFreeMemory

The layer validation code does not validate against application challenges of usage cases mentioned in the spec for vkFreeMemory, nor do related tests exist for layer verification

The referenced valid usage checks are complete.


Segmented padding

Due to the refactor, there's a bug where if a bottom/top inset it set when using a top segmentedControl part of the tableView is cut off at the bottom. I couldn't figure out why this is happening since it seems the contentSize is correct. Should be easy to fix if you can determine the right place the UIScrollView frame/contentSize is being set.

Thanks @cerupcat !
I have fix the bug, it was due to the parallax header which change the pager frame.


[RFR] Refactor datagrid pagination

[x] Add some tests on datagrid pagination directive
[x] Fix pagination issue when too many pages (see #296)
[x] Separate infinite pagination and normal pagination into 2 directives
[x] Move click on page link action to the caller (listController) and pass it as a parameter

Here is how the pagination nav looks for a 9 page list:
at page 1:   1 2 … 9 Next »
at page 2:   « Prev 1 2 3 … 9 Next »
at page 3:   « Prev 1 2 3 4 … 9 Next »
at page 4:   « Prev 1 2 3 4 5 … 9 Next »
at page 5:   « Prev 1 … 4 5 6 … 9 Next »
at page 6:   « Prev 1 … 5 6 7 8 9 Next »
at page 7:   « Prev 1 … 6 7 8 9 Next »
at page 8:   « Prev 1 … 7 8 9 Next »
at page 9:   « Prev 1 … 8 9

Missing:

it should display prev page except on the first one
it should display next page except on the last one

Any progress?

Switching to RFR


Add cnft (npm package)

Showcase addition

Name: cnft
Description: A library for validating metadata for the 721 metadatum property. Complies with community standards. Future plans to add more cnft utility.
Link:   cnft
Source: null
Tags:

javascript
typescript
nft

The tool parses NFT Metadata... here is an example of it in use

Nav to: https://ada9000.io/t/nft-meta
Paste in some metadata

{
  "721": {
    "f0ff48bbb7bbe9d59a40f1ce90e9e9d0ff5002ec48f232b49ca0fb9a": {
      "ada9000": {
        "augmentations": [],
        "core": {
          "handleEncoding": "utf-8",
          "og": 0,
          "prefix": "$",
          "termsofuse": "https://adahandle.com/tou",
          "version": 0
        },
        "description": "The Handle Standard",
        "image": "ipfs://QmaZ56m6ScGyzpYnGdSbp3z6jkMU7UZSuy7Azjnw8gzQMm",
        "name": "$ada9000",
        "website": "https://adahandle.com"
      }
    }
  }
}

@Jack-0 I think this is generally a good addition but your code baseline for this branch is very old and doesn't build (see instructions on front page), I think due some recent changes to the site blog & authorship structure:
[ERROR] Error: Blog author with key "cexplorer" not found in the authors map file. ...
    at getAuthorsMapAuthor (/home/rphair/archive/build/developer-portal/node_modules/@docusaurus/plugin-content-blog/lib/authors.js:89:23)

Can you please update your fork from the repo upstream and merge these changes in with your working branch, or some equivalent process?  This should also take care of the merge conflict currently reported for this PR.

Synced the fork on mobile and it deleted the PR (sorry) :see_no_evil:
New PR -> https://github.com/cardano-foundation/developer-portal/pull/817

understood; I've also heard that if you push a deleted or renamed branch it kills the PR with no way to recover 🤪


rootston: Keep older anchored layers near anchor points

Layer surfaces are attached to edges of the screen starting with the youngest, causing new ones to always displace existing ones. This changes the order to oldest first, keeping the positions more often.
With this change, layers attached to edges have a more intuitive feeling of "building upon" the edge.

Thanks!


Ping sounds stop after a certain amount of time

This is fairly basic, not so much a bug report as I need clarification. I've started leaving my pc on overnight; now either it's because the displays turn off or something - but by the morning, the ping sounds no longer work. Will get to testing at some point.

I've figured it out. In the power options in windows (10?), set the screen off time to a value. Leave the screen to turn off, one fully off; wiggle the mouse and get someone to ping you. Your ping sound should no longer make a sound. Now restart chatterino and they should work again.
v2.1.5 (commit 556c2aef2, 02.11.2019)

I don't really use sounds anymore and I don't see anyone else who has reported this. I'll just assume this was a Windows bug.


[WIP] Add experiment logging support for BrainTrust

Not strictly necessary to include in the MVP, but if it's possible to emit an event each time a node executes for each test case, we can incrementally log span progress to BrainTrust, and then you'd be able to see performance metrics across the whole execution tree.

Ah yeah I can just pipe in the events from the graph execution!


Docs: Remove _getConnectingEdges() as it's not implemented

It's unclear how this function was intended to behave, and it was decided not to implement it for now
Internal ref:

https://github.com/arangodb/planning/issues/3252

Can not verify if it builds because something is broken with the program_options stuff in new Jenkins for branches <= 3.3

okay - I am publishing a new 3.3 version of the doc with the old jenkins, build 365.  This one will be included in a future build. merging for now, thanks


[Common] Added Alignment Requirements for CuBLAS heuristics

Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch

Could we add some test for it?

LGTM, thanks. I verified that my failing case now works.

/te-ci pytorch

Hi, thanks for the feedback. Glad to hear that it works for you now @deepakn94.
@ptrendx I added the unit test to capture this bug. If there is no issue, I will merge this PR. Thanks

/te-ci pytorch


Notify Url for recurring payments

Is there any place we can input notify_url for recurring payments? While the recurring profile gets set up properly, there is no way to extract transaction ids.

There does not appear to be.
Looking at the code comments, there's one stating: "recurring payment doesn't support dynamic notify_url".
If this is not correct [anymore], please can you paste a link to PayPal documentation. Would you be willing to look at a patch & tests if this is the case?


[BUG] Neofetch preset error

General description of bug:

What happened: when using the neofetch preset, theme line has weird characters:
Theme: Breeze (Dark) [QT], Breeze [GTK3]{?3} {3} [Plasma], {?}{7}

What should happen: should not have weird characters.

Fastfetch version used: 2.9.1

Did it work in an older version: I didn't use any version before this one.

Where did you get the binary: Fedora Linux repositories

Repository   : @System
From repo    : updates

Does this issue still occurs in the latest dev build? I don't know. I know it also happens on 2.11.3 on NixOS (tested by a friend of mine).

Often helpful information:
Screenshot:

The content of the configuration file you use (if any):
Default config, but with "-c neofetch".

Output of fastfetch -c ci.jsonc --format json:
I don't think it's necessary for this case.

Output of fastfetch --list-features:

threads
vulkan
wayland
xcb-randr
xcb
xrandr
x11
drm
gio
dconf
dbus
imagemagick7
chafa
zlib
xfconf
sqlite3
rpm
egl
glx
osmesa
opencl
libpulse
libnm
libddcutil
videodev2
Proprietary GPU driver API

It has been fixed in newer versions


Added zh_cn course file and a Golang course.

What does this PR do?
Added resources.
For resources
Description
Added a course zh_cn file.
Added first Golang cource in it.
Why is this valuable (or not)?
This course is free!
How do we know it's really free?
It is provided freely by Microsoft on their website.
For book lists, is it a book? For course lists, is it a course? etc.
It is a course.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists  in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Followup

Check the output of Travis-CI for linter errors!

remember to add a link in the Readme

remember to add a link in the Readme

Fixed, let me know if the name is correct in the readme, since there are a lot of different Chinese languages (variations?).

Hmmmm. I guess I didn't think through the spoken vs. written language issues. the zh_CN locale is usually for Chinese (simplified), while zh_HK and zh_TW are Chinese (traditional). But the dominant spoken languages are Mandarin and Cantonese. So courses may need to be separated by spoken languages while books and interactive tutorials would be separated by the simplified vs. traditional. so I think your original suggestion to create an interactive tutorial page is the better option. currently we have a single chinese page bor books, labeled Chinese / 中文 so maybe we should do the same for interactive tutorials, and only divide by local when there are more items to divide. We have a HOWTO in traditional characters - the label is modified by the rendering of HOWTO.

So, should I create free-programming-interactive-tutorials-zh.md and place this Golang course there?

yes.

Fixed.


script to pull prerequisites for gio on Arch linux

There is now simple scripts to install gio prerequisites in prereqs/ for deb/rpm based linux, the equivalent for Arch-based platforms should be there too I think (and eventually a build for AUR).

after much gnashing of teeth a bug involving https git urls (must have .git suffix) a small lan testnet was run involving machines running windows and manjaro in addition to ubuntu 20.10, scripts are now correct in place for setting up on arch linux


.Net: Plan creation doesn't utilize file-imported Functions

Describe the bug
I am using the ImportSemanticFunctionsFromDirectory feature of the kernel to import function configs & prompts from a directory. I am also tying in Memory using the TextMemory plugin.
I am able to make the memory + LLM query work when using .RunAsync() but am not able to do it using Plan. When I use Plan, I see the plan only ever contain functions to retrieve & store to memory, not to hit the LLM w/ the prompt from the imported functions.
It seems like the Planner is only considering functions in the _GLOBAL FUNCTIONS_ value of the Kernel's Functions property.
To Reproduce
Steps to reproduce the behavior:

ImportFunctions from TextMemoryPlugin
ImportSemanticFunctionsFromDirectory
Use RunAsync() to Recall, run an imported function, and Save the result to memory
Attempt to make a Plan that does the same.
Observe imported functions won't be considered as part of the plan

Expected behavior
I'd expect the imported functions to be considered as part of plan creation & execution
Platform

OS: Windows,
IDE: Visual Studio
Language: C#
Source: NuGet package version 1.0.0-beta6

Additional context
Add any other context about the problem here.

appears I just have to be super-specific in my "goal" in order to make it consider them as part of the plan. I am wondering if "goal" makes it seem like you don't have to do a ton of prompt engineering to get it to do the right thing, when in fact you do ;)


Adding release note about how spotty internet issues no longer cause “IDE session timed out” message

Description & motivation
Adding new release note and Loom  describing how spotty internet issues don't cause the  “IDE session timed out” message.
To-do before merge

Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects

@schottj @davidharting Here's the PR for review!

LGTM!

I find the duality of release notes and product updates confusing - where should I look as a casual visitor? but leona tells me she's talked to @jasnonaz/@krevitt so I'm not going to re-litigate it!

Yea maybe they remember what we had said exactly, but I think they felt like each served a separate purpose.


Throttle resize/scroll events especially on frameEl

We should throttle resize events. Particularly when the resize event is coming from window. If resize events are firing as often from elements then ditto there.
Currently frameEl equals window and when resizing occurs the app grinds down in performance.

The same issue applies to scroll events. In the scroll case the only element we need to care about however is frameEl.


[new/auto-layout] Implement Responsive design with updated auto-layout capabilities

The new version of autolayout has been for a few weeks. learn more here
This enables us to make layout-perfect interpreting logic wihout infering from context, but as-is designed by designers.
We will now support plain html + css for starters, interpreting auto layout to flexbox properties.
We might also need global interpreter (reflect layouts) in which for converting layout configurations to flutter, web, and many other platforms in an intuitive way, through the unified logic gate.

Autolayout Implemented.
Future issues will be tracked at https://github.com/gridaco/designto-code/issues/32


src_duckdb() for R package

Add convenience function to create DuckDB dbplyr sources

src_duckdb() ?

added this in master, behaviour should mirror src_sqlite


Biallelic pairwise distances and neighbour joining trees

Work towards implementing neighbour-joining trees.

Building on proof of concept here.

Going to merge soon here as the njt feature is needed for an analysis project, but there's a lot going on in this PR so very happy to revisit anything here.


fix(GridForm): cancel + submit button

Overview

Fixes cancel button alignment in GridForm + add cancel button alignment to Storybook

PR Checklist

[x] Related to JIRA ticket: [web-2295]
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change
[x] This PR includes testing instructions tests for the code change
[x] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions
Vist GridForm / Submit Button Position + Gridform / Cancel button sections in Storybook + observe how they are fixed
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.291-alpha.ec3a6e.0@codecademy/gamut-labs@35.7.1-alpha.ec3a6e.0@codecademy/gamut@52.1.1-alpha.ec3a6e.0@codecademy/monaco@0.7.10-alpha.ec3a6e.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.293-alpha.8b70bf.0@codecademy/gamut-labs@35.8.0-alpha.8b70bf.0@codecademy/gamut@52.2.0-alpha.8b70bf.0@codecademy/monaco@0.7.12-alpha.8b70bf.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6412368611290d11cb3468ae--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.294-alpha.8bf9ce.0@codecademy/gamut-labs@35.8.1-alpha.8bf9ce.0@codecademy/gamut@52.2.1-alpha.8bf9ce.0@codecademy/monaco@0.7.13-alpha.8bf9ce.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://641367d86211813457beff63--gamut-preview.netlify.appDeploy Logs


MWS Ingress rewrite problem

https://github.com/SoftwareAG/webmethods-helm-charts/blame/20fc1518f7ac358c9caea7fc51cd9395cc7a423f/mywebmethodsserver/helm/values.yaml#L90
Can't find proper values for ingress path rewrite. Both MSR and UM ingresses work with path (although they are slightly different), but I couldn't make MWS path work no matter what I did. Here is what I did for the other products and tried to apply MWS.
UM server didn't require a rewrite, I added the path and the host and Enterprise manager was able to connect using path and nhp protocol.
IS/MSR didn't work with this setting, I added the annotations below and the regex below.
 annotations:
    nginx.ingress.kubernetes.io/x-forwarded-prefix: /wm-poc-is/
    nginx.ingress.kubernetes.io/rewrite-target: /$1
    nginx.ingress.kubernetes.io/use-regex: "true"
    nginx.ingress.kubernetes.io/session-cookie-name: wm-poc-is-cookie
    nginx.ingress.kubernetes.io/session-cookie-path: /
    nginx.ingress.kubernetes.io/affinity: cookie
    nginx.ingress.kubernetes.io/affinity-mode: persistent
  hosts:
    # -- Hostname of Ingress. By default the defaultHostname is used. For more complex rules or addtional hosts, you will need to overwrite this section.

    - host: # DNSname of the vip, without the path. (it doesn't accept path here or on default host)
      # --  Address the backend
      paths:
        # -- Path to address the backend
        - path: /wm-poc-is/(.*)
          # -- Path type to address the backend
          pathType: Prefix
          # -- Port of service
          port: 5555

This setup works for IS but it doesn't for MWS. I also compared the generated yaml files in kubernetes, didn't notice anything odd. MWS responds to first query with the path, but the subsequent requests don't have the path rewritten in them. If I add the path manually and make the call from postman, I get the proper response.

I didn't get the issue here.

Rewrite doesn't work properly. If I make calls including path, it successfully returns, but the MWS pages that are rendered are missing the path.  Same parameters work as intended for MSR, but subsequent requests are not rewritten in MWS. Check screenshot for an example.

But I doesn't get the issue for the mywebmethodserver Helm Chart. Regarding Nginx or LB configuration for My webMethods Server, you should look into the documentation.

I don't think I understand you. I am already using the same load balancer and ingresses that I am using with UM and IS. They both work as intended. When I apply the same ingress/rewrite configuration to MWS, MWS ingress doesn't rewrite the path. There is nothing in the documentation for nginx or ingress. If I remove the path and rewrite, it works without any problems. Why do you think it is load balancer or nginx problem?

Hi, can you please confirm that MWS work correctly without the ingress? (ie. access it over k8s port proxying)
I suspect something else might be at play here where MWS is not starting correctly in the first place... eg. DB connectivity, or local disk persistence, etc...
Especially asking because I have MWS running fine in my prod-grade K8s setup behind a Traefik Ingress...and there is no need for any regex rewrites (at least with Traefik... but I don't think NGINX should be any different)

With only the settings below, no path no annotation,
    - host: # F5 LB address is here but masked
      # --  Address the backend
      paths:

        - path: / #wm-poc-mws(.*)
          # -- Path type to address the backend
          pathType: Prefix
          # -- Port of service
          port: 8585

this is how my login screen looks like. Only 1 module failed to load. It was loading yesterday, it may be caused by my endless configuration changes.

after adding the annotations below,
    nginx.ingress.kubernetes.io/session-cookie-name: wm-poc-mws-cookie
    nginx.ingress.kubernetes.io/session-cookie-path: /
    nginx.ingress.kubernetes.io/affinity: cookie
    nginx.ingress.kubernetes.io/affinity-mode: persistent

I get the same results, same module didn't load. This is after login with Administrator with the annotations above. All requests are forwarded to my vip, there is no path involved.

The screenshot below is my login screen after adding the path, without any path rewrite annotations. This configuration works for UM but it doesn't work for IS without path rewrite annotations.

If you check the network queries, only the first query was successful. The rest are routed to / instead of /wm-poc-mws/. They should have been rewritten as the path included, but it didn't. When I call these services using postman and manually adding my path, I get the correct response, so if I send the request to the correct path it is successful, but it can't rewrite automatically.
Now I add these 3 annotations too
    nginx.ingress.kubernetes.io/rewrite-target: /
    nginx.ingress.kubernetes.io/rewrite-target: /$1
    nginx.ingress.kubernetes.io/use-regex: "true"

and it made no difference. This is weird because these annotations fixed my IS path rewrite issue.

I used incognito window and restarted the browser after every deployment. If I don't do it, it loads the missing files from the cache.

I think the curious login screen is a result of Nginx settings, I have this in 1 of 2 environments. Would you uninstall MWS with deleting the PV and re-install it?

I have done that several times. What is the ingress configuration you want me to test when I do it from start?

After the first MWS deployment, the logon screen was OK in my environment. The small login screen comes later but I don't know when. Do you have the same behavior?

Additional, I think it is not a problem of Ingress (configuration). If I access the MWS in service level with kubectl port-forward, the login screen is also small.
I didn't see the small MWS login screen on Docker. Therefore, I create Helm Chart of mywebmethodsserver version 1.0.2. Here, you can disable PVCs with storage.enabled=false


Questions about the pipeline of the essay

The author has done a really great job, I am wondering whether the sequence of the training phase is

Train AUnet w/o VAE
Train VAE
Train AUnet+VAE

I think you are correct:

Train AUnet w/o VAE
Train VAE
Train AUnet+VAE (load 1 or not)
Train AUnet+VAE+TDN (load 3 or not)

What's the meaning for (load 1 or not) and (load 3 or not). I guess you mean that load the weight in the first step and the weight in the third step.
Moreover, If your pipeline is what we mentioned above, the code for function data_load is not correct, especially in the train_vae.py. The dataLoader there return data and label, but actually the data_load function should output a dict.


Support pointer flags

Change-Id: I97d37bdaeab5b878b8064a555e3a07a131b175f0

Codecov Report

Merging #22 (c80b4d6) into main (e4eb835) will increase coverage by 0.21%.
The diff coverage is 90.24%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
+ Coverage   86.15%   86.36%   +0.21%
==========================================
  Files          11       11
  Lines        1950     1988      +38
==========================================
+ Hits         1680     1717      +37
- Misses        190      192       +2
+ Partials       80       79       -1

Files Changed
Coverage Δ

flag.go
86.68% <88.57%> (+0.59%)
:arrow_up:

cli.go
79.22% <100.00%> (ø)

flag_unsafe.go
87.75% <100.00%> (+1.08%)
:arrow_up:


Not Working

UPI payment is not working please check and update now

duplicate of #35

Not working now because of the Some Privacy Policy issue is getting.


Update Who-Is-Using-CAS.md

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

Would this be OK to include the URI of the main website?

@chrismyers81 Do you have updates on the set of changes asked here?

Closing due to inactivity. Feel free to review and resume.


Puies

https://github.com/aetilius/pHash/blob/9997dab79134121a82ad7a2cc0c0fd4f8da99fce/bindings/python/setup.py

closing as spam


The shippingAddress and billingAddress types in the CartDraft should be AddressDraft

Hi,
Working with CartDraft I notice that the shippingAddress and billingAddress types are wrong.
Currently, these fields are defined as BaseAddress, an interface that doesn’t support the custom fields on the creation.
https://github.com/commercetools/commercetools-sdk-typescript/blob/master/packages/platform-sdk/src/generated/models/cart.ts#L299
https://github.com/commercetools/commercetools-sdk-typescript/blob/master/packages/platform-sdk/src/generated/models/cart.ts#L303
The correct type is AddressDraft, which allows the custom fields!
Thanks!

But if you are creating a new cart, using the CartDraft interface, and in this creation, you want to set a default or initial address (shippingAddress and billingAddress), and you want to define some custom fields on this address, you can't.
Because the CartDraft expects a BaseAddress for the shippingAddress and billingAddress fields, and the BaseAddress doesn't support the custom fields definition.
If you want, I can create an example with this issue.
Best regards.


pkg: controllers: move MCP code in its own package

Extract MCP utilities from the controllers to its own package,
so the test code doesn't depend on the controllers anymore.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

/lgtm

replaced by https://github.com/openshift-kni/numaresources-operator/pull/113


Table Components

From our conversation here: https://github.com/revelrylabs/dcm/issues/245 We need to create two types of table components for Rev Core.

A table component that actually uses the <table> tags

A table component that mimics a table, but uses Rows and Cols (or listed items). A reference for this type of table is already built out for DCM.

Also, make sure to follow the instructions for the mobile table view from the DCM reference above.

Picking this up to work on next

Update:
I paired with some of the devs to resolve more env issues on Friday. Now I'm actually able to work on building out the table component. Hoping to finish this up today or tomorrow.

Update:
I finished up the first Table component (using Rows and Cols). Tomorrow I'm going to work on the second Table component (using the  tags) and then I'll be able to push up my updates for review.

My PR is now in review. More info and screenshots here: https://github.com/revelrylabs/possum/pull/198
I'll give another update once my PR is merged!

I spoke with @CheetoMao and we looked at my PR together. I'm about to do the following:

Break out the second Table option as a new component with a new name (name still undecided)
List this new component under the Containers section

Get the PR for the first Table merged and then this new component will be in a separate PR to keep everything organized.


Is it possible to use the same images as plex?

My Jellyfin and Plex servers share the same media library, is it possible to use the same images generated by Plex with Jellyscrub?

Not at the moment as the structure and networking is jellyscrub-specific. What's the folder structure like for bif files in plex? Are they stored with the media?
If there's a similar structure it's definitely something I could look in to.

Thanks for replying!!

What's the folder structure like for bif files in plex? Are they stored with the media?

Plex seems to store the bif files in plex/Library/Application Support/Plex Media Server/Media/localhost/
after that is has s folders named "0","1", ... , "e", "f"
inside which a folders like 9ecdc7be94ec634a91d11db9d5e3f616fcfb3d.bundle/
then it goes Contents/Indexes/index-sd.bif
so the entire structure looks like this plex/Library/Application Support/Plex Media Server/Media/localhost/6/50aaead4e410082eddaa9f98fd463bc8bbf4567.bundle/Contents/Indexes/index-sd.bif
Not sure if that helps...
Cheers!

Looks like the folder structure uses the media ID for as the folder name to store metadata, similar to jellyfin. The problem is that once the media is imported to jellyfin they will have completely new IDs as generated by jellyfin so I see no easy way to map them automatically--though it would be technically possible with some sort of plugin that could sync/map the IDs between both platforms.

This python script can generate bif files for plex. It finds the bif location for the movie files using the plex api and some config information.
https://github.com/stevezau/plex_generate_vid_previews


Support the requireAdmin function in test fixtures

Feature Request
Description
The release of Sync Gateway 2.1 included a new addition to the sync function API: requireAdmin. Update the test-fixture-maker module to support requireAdmin so that tests can be written for document definitions that make use of it in custom actions.
Examples
customActions: {
  onValidationSucceeded: function(doc, oldDoc, customActionMetadata) {
    if (doc.x === 'foobar') {
      requireAdmin();
    }
  }

Addressed by https://github.com/Kashoo/synctos/pull/333.


calculate the inner volume of an object?

Hi, Marco Musy,
How are you doing there ?
I am new using vtkplotter, trying to calculate the inner volume of my .stl files.
didn't see a ready use method yet.
Is there one ready developed by you ?
tks.
Henry

@henryliangt
Hi Henry you should be able to get it this way, Eg.:
from vtkplotter import *

#c = Cube(side=2).triangle() # make triangular mesh
c = load(datadir+'shuttle.stl').scale(10)

print('volume is:', c.volume())

c.show(axes=1)

this is implemented natively in VTK:
https://vtk.org/doc/nightly/html/classvtkMassProperties.html


Generic over states and transitions

PR for my suggestion under your reddit post.
Type changes:

Removed all any types and as casts (except for one). This means that everything is now verified by TS.
Removed the KeysOfTransition helper type. TypeScript can infer all state and transition types, so this helper is no longer necessary.
Removed BaseStateConfig and BaseConfig. These were purely internal types. I don't know why you added them in the first place. I just used MachineStateConfig and MachineConfig everywhere.
The type parameter of useStateMachine is no longer constrained. From what I see, this only got in the way of TS inferring the type. I'll re-add it if it's important.
useStateMachineWithContext is now generic over states and transitions and not over the config type. This allows TS to infer all states and transitions. However, this is a breaking change for TS users.

Behavioral changes:

reducer: I changed the currentState?.on to currentState.on. Since the state machine is now type-checked, users will get a compilation error if they try to transition into an undefined state. Therefore currentState must be non-nullable.
Context will no longer default to {}. This was actually a bug of the old implementation. I fixed this bug here because TS would not compile (that's the advantage of not using any and as :) ). Example:const [state, send] = useStateMachine<{ foo: { bar: number } }>()({
    initial: 'inactive',
    states: {
        inactive: {
            on: { TOGGLE: 'active' },
        },
        active: {
            on: {
                TOGGLE: {
                    target: 'inactive',
                    guard(context) {
                        // no type error but this will crash at runtime
                        return context.foo.bar > 4;
                    }
                }
            },
        },
    },
});

console.log(state); // { value: 'inactive', nextEvents: ['TOGGLE'] }

send('TOGGLE');

console.log(state); // { value: 'active', nextEvents: ['TOGGLE'] }

send('TOGGLE'); // crash

Feel free to request changes.
You can try out everything here.

Wow, @RunDevelopment, this is spectacular; thanks so much for your time!

I don't know why you added them in the first place.

Because I lack knowledge. This is the first project where I use TypeScript beyond the basics...

The type parameter of useStateMachine is no longer constrained. From what I see, this only got in the way of TS inferring the type. I'll re-add it if it's important.

It makes sense now. I don't think we need to re-add.

Therefore currentState must be non-nullable

👍

Context will no longer default to {}

👍
I also noticed you changed the reducer to always accept a type in the action and expose send as a function that dispatches { type: "Transition" ... instead of exposing the useReducer dispatch directly. A clever solution, very obvious, I don't know why I didn't think of that from the beginning 😅.
One change I will do in this case is renaming the internal send to dispatch to be more idiomatic - but I can do that separately, you already contributed with a lot.
Overall, thanks again a lot for your PR - this is as much an improvement for the library as a learning opportunity for me.
Please let me know if you think of any other suggestions in the future.

Oh, I remembered why I this all that trickery with the send method: I wanted to completely reuse dispatch because it's stable across re-renders. There is the possibility that some user will pass send to child components - if these components use PureComponent or React.memo, an unstable send method will make them useless.
Not a big issue, can be definitely dealt with on a separate occasion.

Ahh, that's true. The methods we give out might behave the same but they aren't the same object.
I wonder, is there an idiomatic way to do this in the React world or will it be a WeakMap solution?

Yes, I can use either useMemo or useCallback.

@all-contributors please add @RunDevelopment for code, test and ideas.


[Bucket Mod] Error on launch

LogOutput.log

Thanks for reporting. However I'm aware of this issue and the fix is hopefully is coming soon.

blessed
thank you!


Change Horizontal units title and feet selection

Step 2 of Grid coordinates entry
Change data field title:
From: Horizontal units
To: Units

Change feet units selection:
From: Feet
To: International Feet

Ok, make up your mind. https://github.com/agrc/plss/issues/104

I know...busted!

validated


package.json: remove unused dependency

through2 was added to package.json but it was never used

Coverage remained the same at 100.0% when pulling c420af50dacfa1506cc6cb86da90c9819e00d3c1 on remove-unused-dependency into 9bf9d023df27200debfee40de2dadf5628140a8d on master.

@mccraveiro Why did you closed this PR? Would better to merge it?


Removing xDscResourceDesigner tests

We are moving them to common tests: https://github.com/PowerShell/DscResource.Tests/pull/6

LGTM

:+1: Awesome, this resolves #5 as well.


Update the prerequisite Node.js versions in manual.mdx

Description

Updates the prerequisite Node.js versions in manual.mdx, so that they're consistent with auto.mdx

Related issues & labels

#7846

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
install/manual.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Added color gain controls, changed 'brightness' to 'luminance', updated help/README.md

Based on the availabe MCCS VCP Codes (Quick summary here: https://www.ddcutil.com/getvcp_known_u3011_output/), I added three commands for adjusting the Red, Green, and Blue video gain which are available on my monitor and I believe relatively common.
I also changed instances of the word 'brightness' to 'luminance', both to match the official VESA terminology in that MCCS document and to allow the short flag 'b' to refer to 'blue'. The help text has been updated to match these changes, README.md was updated to match the help text, and it looks like my Atom editor removed several instances of extra whitespace on blank lines.

Thanks @agentirons


HoodieDeltaSteamer cannot deserialize avro source correctly

As quick start document, I run HoodieDeltaStreamer.java with params
--storage-type COPY_ON_WRITE --source-ordering-field ts --target-base-path /tmp/hoodie/hive/warehouse/stock_ticks_cow --target-table stock_ticks_cow --props /home/leiline/Documents/code/hudi/data/kafka-source.properties --source-class com.uber.hoodie.utilities.sources.AvroKafkaSource --schemaprovider-class com.uber.hoodie.utilities.schema.FilebasedSchemaProvider

the context of my kafka-source.properties is
# base properties
hoodie.upsert.shuffle.parallelism=2
hoodie.insert.shuffle.parallelism=2
hoodie.bulkinsert.shuffle.parallelism=2
# Key fields, for kafka example
hoodie.datasource.write.recordkey.field=key
hoodie.datasource.write.partitionpath.field=date
hoodie.deltastreamer.source.dfs.root=file:///tmp/hoodie/hoodie-dfs-input
# schema provider configs
hoodie.deltastreamer.schemaprovider.source.schema.file=file:///home/leiline/Documents/code/hudi/data/binlog-util/src/main/avro/BinlogRecord.avsc
hoodie.deltastreamer.schemaprovider.target.schema.file=file:///home/leiline/Documents/code/hudi/data/binlog-util/src/main/avro/BinlogRecord.avsc
# Kafka Source
hoodie.deltastreamer.source.kafka.topic=mytopic
#Kafka props
metadata.broker.list=10.202.34.***:9092,10.202.34.***:9092,10.202.34.***:9092
auto.offset.reset=smallest
group.id=mygroup
zookeeper.connect=10.202.34.***:2181,10.202.34.***:2181,10.202.34.***:2181

when running the main file, i got a error
Caused by: io.confluent.common.config.ConfigException: Missing required configuration "schema.registry.url" which has no default value.

that means my source data cannot deserialize  with avro schema.
How can i deserialize my avro data without "schema.registry.url" configuration by HoodieDeltaStreamer?  (I want to deserialize my avro data with a avro schema file)
I checked hudi code and not found refrence of "schema.registry.url"
I found the FilebasedSchemaProvider.java is to load avro file by hoodie.deltastreamer.schemaprovider.source.schema.file and
hoodie.deltastreamer.schemaprovider.target.schema.file
I cannot understand how HoodieDeltaStreamer loads avro schema file , please give me some advices.
thank you!

The Confluent Kafka integration that we have in Hudi requires Schema Registry service (See https://docs.confluent.io/current/schema-registry/docs/schema_registry_tutorial.html)  to be running alongside Kafka when using Kafka as an Avro Source.
DeltaStreamer is a generic tool in the sense that it can support other types of sources than Avro (JSON, RDD). The configuration you mentioned (hoodie.deltastreamer.schemaprovider.source.schema.file) is used internally in DeltaStreamer to register schemas in Hudi.
In your case, it is better to use SchemaRegistry service approach instead of FileBasedSchemaProvider  as that can avoid duplicating schema
An example config (kafka-source.properties):
hoodie.deltastreamer.schemaprovider.registry.url=http://localhost:8081/subjects/impressions-value/versions/latest
Kafka Source
hoodie.deltastreamer.source.kafka.topic=impressions
#Kafka props
metadata.broker.list=localhost:9092
auto.offset.reset=smallest
schema.registry.url=http://localhost:8081

@leilinee :  We have created the jira (https://issues.apache.org/jira/browse/HUDI-73) to provide direct support to Avro Kafka Source.
I am not sure if you already have schema registry setup. Let us know if you run into any issues using Hudi.

thanks for your answer.
I have solved this problem. The  HoodieDeltaSteamer consume avro data from topic and then create spark RDDs , in this process,  HoodieDeltaSteamer deserialize avro data by KafkaAvroDecoder.
protected JavaRDD<GenericRecord> toAvroRDD(OffsetRange[] offsetRanges, AvroConvertor avroConvertor) {
  JavaRDD<GenericRecord> recordRDD = KafkaUtils
      .createRDD(sparkContext, String.class, Object.class, StringDecoder.class, KafkaAvroDecoder.class, kafkaParams, offsetRanges)
      .values()
      .map(obj -> (GenericRecord) obj);
  return recordRDD;
}

I rewrited KafkaAvroDecoder with my deserialize avro data code .
It have to implement  Decoder class and declare a construction method.
my deserialize code is in this.deserialize().
public class KafkaAvroDecoder  implements Decoder<Object>   {
    public KafkaAvroDecoder(VerifiableProperties props) {
        props = null;
    }

    public Object fromBytes(byte[] bytes) {
          return this.deserialize(bytes);
    }
}

It seems my program works well.

@leilinee there is a JIRA open for this.. https://issues.apache.org/jira/browse/HUDI-73
Is this assigned to you? If not, if you could help address this in the main code line. it would be awesome!

@vinothchandar  ok, i will try my best do that.


Feature latex

Adaptions for Latex Compiling

@jehov4 The branch has conflicts. May you check them?

New Request


Problem with permissions

Not sure if it's just me goofing up here, or there's some issue.
I have permissions installed, setup via McMyAdmin.
I've installed multiverse, and I can use any command through the console. But as soon as I try anything ingame, I get a list of permissions I don't have, even though I've added "multiverse.*" in my permissions file.
Any idea what's going wrong here?
sorry if I'm just wasting your time.

I think you're not using a SuperPerms-compatible Permissions-plugin. Use one and everything should be fixed. If that was not your problem, please reopen this issue.

I'm using "Permissions [3.1.6]" afaik it should be compatible, but I may well be wrong.

You're wrong. You need a plugin that uses the new SuperPerms-API because your old Permissions 3.x doesn't do that.

I see. Do you have any recommendations?

PermissionsBukkit sine it is the official one.

Ok. Thank you.

I installed permissionsbukkit and it's all working now, thanks a lot.