NEXT-11875: Fix sorting of order line items

1. Why is this change necessary?
The order line items are in a different order than the cart line items.
2. What does this change do, exactly?
Adds sorting (position) of lineItems association.
3. Describe each step to reproduce the issue or behaviour.
Buy some stuff and compare the order of the line items from the cart to the order of the line items of the order.
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-11875
https://issues.shopware.com/issues/NEXT-9869
New PR for #1512 because the source branch got deleted.
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-13190
Please use this issue to track the state of your pull request.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-13190
Please use this issue to track the state of your pull request.

Hey @claudiobianco, please add some tests.

Hey @claudiobianco, please add some tests.

Hey @J-Rahe can you give me a hint what and how to test it?

Hey @J-Rahe can you give me a hint what and how to test it?

basically add some test that fail if the sorting of the orders is changed. If you look at e.g. the CartOrderRoute.php there is already an CartOrderRouteTest.php for it with an testOrderOneProduct which might help you. To test the sorting you would order multiple items in the same order and check that that order is correct.

basically add some test that fail if the sorting of the orders is changed. If you look at e.g. the CartOrderRoute.php there is already an CartOrderRouteTest.php for it with an testOrderOneProduct which might help you. To test the sorting you would order multiple items in the same order and check that that order is correct.

But the PR does only add the sorting for specific views/use cases. It does not change the sorting in the order per se. So I would have to add the sorting for the test and then check if it is correctly sorted? That would just test if the DAL sorting works correctly? Or am I missing something? ;)

But the PR does only add the sorting for specific views/use cases. It does not change the sorting in the order per se. So I would have to add the sorting for the test and then check if it is correctly sorted? That would just test if the DAL sorting works correctly? Or am I missing something? ;)

Well it would test if the position field is correctly saved from cart line item to the order line item. Would that be sufficant @J-Rahe?

Well it would test if the position field is correctly saved from cart line item to the order line item. Would that be sufficant @J-Rahe?

Hey @claudiobianco, sorry that i didn't respond before the weekend, what the test should do is to ensure that we will notice when the sorting is changed again. e.g. the test checks that if your order is sorted A,C,B,F with the change, the test would compare the sorting against that.

Hey @claudiobianco, sorry that i didn't respond before the weekend, what the test should do is to ensure that we will notice when the sorting is changed again. e.g. the test checks that if your order is sorted A,C,B,F with the change, the test would compare the sorting against that.

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline


Added z_indexto the DebugMenu scene. Closes #6

Added z_index and set it to max value in _init() so that the menu does not go under the current scene

Well, you can do that too, I thought it would be easier this way instead of having new people adjust that manually.

Well, you can do that too, I thought it would be easier this way instead of having new people adjust that manually.

It can be done in the autoloaded scene so people don't have to adjust it manually :slightly_smiling_face:

Superseded by https://github.com/godot-extended-libraries/godot-debug-menu/commit/2d1bf27e5901186080e0fde5adc56df65dd9c583, which also fixes the issue with Camera2D nodes making the debug menu appear offset. The demo project has been modified to demonstrate this use case.
Thanks for the contribution nonetheless!


Use absolute links in the sidebar ToC

This fixes #335 by just avoiding relative links in the sidebar that might pick up a trailing / and cause the confusion described there. It feels like a bit of a hack, but it seems to work.

There's something bothering me: on the preview, clicking on the link given as a repro of the issue, the browser first very quickly shows a 404 page, and then redirects to the right one :sweat_smile: . I'm not sure what happens here, if it's due to caching somewhere. However, this change doesn't sound unreasonable, and it still strictly improve the situation. Maybe we should have a look at having better redirect rules afterward.

It seems the redirect from /user-manual/<page>.md to /user-manual/<page> isn't applied immediately. Maybe we should just look into processing the markdown links with remark hooks or something like my brother suggested...


trace config invalid

Hello, I got this error when I turn on the 'Scheduling details' on CPU Probes.
could you help me?
-:69:13 error: No field named "symbolize_ksyms" in proto FtraceConfig
symbolize_ksyms: true
^~~~~~~~~~~~~~~
[452.354] perfetto_cmd.cc:476     The trace config is invalid, bailing out.

Please provide the Android version of the device you are collecting the trace from
@octaviantu - maybe you can take a look at this as part of your refactoring.

This seem simly an issue of passing a config to an older device that doesn't support symbolize_ksyms

Actually, my device is quite new.. galaxy Z Fold 3. and the Android version is 11.

Unfortunately symbolize_ksyms was added in Android S/12 so even Android 11 is too old for this flag.
As Primiano said, we should just not enable this option if you specify/we detect an older version in the UI when generating the config.

That's why there was a problem. Thank you!

I think we'd still keep this issue open to track the change in the UI to stop specifying this flag.

symbolize_ksyms is added when I turn on 'Scheduling details'. If so, is there any other way to get CPU Scheduling details on the Android version 11?

If you mean collecting directly through the UI (i.e. without using the command line), then this is not possible until we fix this bug.
However, uou can just remove that line from the generated config and run the adb command the UI gives. Scheduling info will still be collected in that case (the only thing you'll miss is some information like the kernel blocked function in sched_blocked_reason).

Fixed in https://github.com/google/perfetto/commit/327ad962a59d149ad6662fcbd18e38d808f91a3f

For now the fix is only available on the Autopush branch.
(you can get to Autopush by writing  localStorage.perfettoUiChannel = 'autopush' in the dev console)
It will be rolled out to canary and stable in the next few weeks.


Modified Date is bugged

Hello! I'm using Quartz to host my worldbuilding stuff, and I'm currently using the date modified type to indicate the last time I updated this particular page from Obsidian. In Obsidian, I use the Linter plugin to automatically create/update the frontmatter tags, which are only the "date created" and "last-modified" tags. Every time I save, the Linter would updated the modified tag to the present time if I had modified the file at any point.
I saw through other closed issues that there are some problems concerning the dates not showing accurately if this is hosted through Vercel. I'm under the belief that it is fixed, so I'm a bit confused as to why, even in the local dev build, the dates aren't accurate. The screenshots will show this.
Now, according to quartz.config.ts, the priority of where the dates should be taken from should be first from the frontmatter, then the filesystem. In lastmod.ts, this else-if statement should gather the dates necessary (except publishedDate) from the frontmatter.
else if (source === "frontmatter" && file.data.frontmatter) {
      created ||= file.data.frontmatter.date
      modified ||= file.data.frontmatter.lastmod
      modified ||= file.data.frontmatter.updated
      modified ||= file.data.frontmatter["last-modified"]
      published ||= file.data.frontmatter.publishDate
    }

The modified variable does take in the correct date from the frontmatter (refer to screenshot 2 as an example). However, somewhere in between that and the page being created, the date shows up as the current date (refer to screenshot 1).
I've narrowed down the problem to the coerceDate() function in lastmod.ts, where I read the output of the dates gathered from the "last-modified" frontmatter property. In it, I notice that a majority of the dates that are being put into the coerceDate() function fails the isNaN() check with the getTime function, and it's outputting a brand new Date object at the present time, and setting that as the modified date, which is absolutely not what I want to happen.
function coerceDate(d: any): Date {
  const dt = new Date(d)
  return isNaN(dt.getTime()) ? new Date() : dt
}

Screenshots
This screenshot here shows that this page is updated on Friday, Sep 22, 2023, however I haven't touched this page at all.
The screenshot below this shows what the Obsidian YAML frontmatter shows.

I do want to note that everything I have shown here is with a local dev build. The site via Vercel shows the same thing, but I figured if I could figure out how to fix the dates in the local build, I'd be able to fix it in the production build.
Now, I could absolutely be putting the frontmatter wrong, and the way it's trying to read it is messing up the dates on the page. If so, please let me know how to remedy this issue. If I am missing anything crucial that would help me understand this better, let me know!
My repo for reference: https://github.com/SheathedBlade/The-World-of-Luciradis
The production website page that corresponds to the example: https://luciradis.arczenpulse.dev/The-World-of-Luciradis/Kallie-Myrai

i think part of the problem is that your obsidian plugin sets "last-modified", even if you haven't touched the file. The date on quartz however should still be consistent with the frontmatter entry, it being a day in the future sounds like a bug with quartz.

What date format does your template use? It needs to be a standard date format

The linter is, by default, set the date format to a moment date format (momentjs): dddd, MMMM Do YYYY, h:mm:ss a

Quartz doesn't use Moment style dates, just regular JS date which only accepts dates roughly similar to YYYY-MM-DDTHH:mm:ss.sssZ

though @benschlegel we should also add a warning for this lol

yeah good idea, I could take a look at this but have a few other things on my backlog atm, if you have a quick fix for it, you can add it too

I see, won't be a problem to change it. But I'm still not sure how my file created dates are set into the future after my modified date, I'm at a complete loss as to how that happened.

I'll keep this issue open until I can confirm the modified date being correctly updated, so I'll make some changes to a page tomorrow. If everything is OK, then I'll close the issue.

Added a warning in https://github.com/jackyzha0/quartz/commit/13c867322629f5ee59c54b95679585b370e1442b. Closing as otherwise working as intended


Added transactional shortcuts

New commands added

tstart
Shortcut \TS
Does a START TRANSACTION in the background.
tcommit
Shortcut \TC
Does a COMMIT in the background.
trollback
Shortcut \TR
Does a ROLLBACK in the background.

Thanks, a lot, yeah, it makes sense
but please move it to another file
https://github.com/caretdev/irissqlcli/blob/main/irissqlcli/packages/special/dbcommands.py

Will do and request again when done.
[upload image]
Stefan Cronje
Director
comprehensoft
M: +27 82 641 1478
W: https://www.comprehensoft.comhttps://www.comprehensoft.com/
E<EMAIL_ADDRESS>[linkedin]https://www.linkedin.com/company/comprehensoft
IMPORTANT: The contents of this email and any attachments are confidential. It is strictly forbidden to share any part of this message with any third party, without a written consent of the sender. If you received this message by mistake, please reply to this message and follow with its deletion, so that we can ensure such a mistake does not occur in the future.
From: Dmitry Maslennikov @.>
Sent: Sunday, 12 February 2023 12:39
To: caretdev/irissqlcli @.>
Cc: Stefan Cronje @.>; Author @.>
Subject: Re: [caretdev/irissqlcli] Added transactional shortcuts (PR #5)
Thanks, a lot, yeah, it makes sense
but please move it to another file
https://github.com/caretdev/irissqlcli/blob/main/irissqlcli/packages/special/dbcommands.py
—
Reply to this email directly, view it on GitHubhttps://github.com/caretdev/irissqlcli/pull/5#issuecomment-1426997774, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5MGA7LUJVHGVK4BCXBSG6TWXC4UHANCNFSM6AAAAAAUZHIXT4.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Changed and pushed.


Consider adding a no-op instruction to SSA

Problem
Certain passes like resolve_is_unconstrained go through the ssa to handle only a few instructions (in this case only vN = call is_unconstrained()) but don't have a good way to remove these instructions without re-inserting every instruction into the block as they go (and choosing not to re-insert these handled instructions) or iterating through the block again afterward and retaining only the non-handled instructions.
Happy Case
We can consider adding an Instruction::Nop { result_types } instruction which conceptually "returns" value(s) of any type but in practice will always be removed from the program or skipped over in acir-gen.
This Nop instruction can be used to replace instructions like call is_unconstrained() with so that they will later automatically be removed by calling Instruction::Nop { ... }.simplify() which will always return SimplifyResult::Remove.
Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

Certain passes ... don't have a good way to remove these instructions without re-inserting every instruction ... or ... retaining only the non-handled instructions.

Just looked at resolve_is_unconstrainted at it's doing neither re-insertion of all instructions nor retaining them per-se, rather, it disconnects the results of the call from their original return value and assumes the DIE pass will remove the now-unused call. Does that imply that in this case there is a way, but it's not a good way?
To make sure I understand correctly, what you're suggesting is not only to replace the original result with a constant, but to replace the call instruction itself with a different Nop instruction in-place, to avoid having to shift the instruction vector in the block, and then, similar to how resolve_is_unconstrained relies on DIE removing unused instructions, that there will be some other pass that will actually re-insert all instructions, while calling simplify.
IIUC the DIE pass would not eliminate Nop because it still has return values, unless those are similarly disconnected from the originals. If I just return None from simplify, wouldn't that leave the original result IDs unassigned and somehow invalid?

The general idea is this no-op instruction is a way that we can tell compiler that we don't care about the results. The component can then whenever it says reinserting instructions ignore this instruction and so we automatically filter them out without needing a whole pass devoted to that.
The current situation were the instructions persist until die is a method but not ideal as we have to keep these instructions through all the other SSA passes

What you say in your comment is correct. We would want the diepass to assert that any noop instructions are used however - it would be a compiler bug if this instruction had its results used by other instructions.

... assert that any noop instructions are used however - it would be a compiler bug if this instruction had its results used ...

You mean assert that they are not used?

Ah yeah, typo there. "are not used" is correct.

Tom is right, but I thought I'd add my own clarifications:

Just looked at resolve_is_unconstrainted at it's doing neither re-insertion of all instructions nor retaining them per-se, rather, it disconnects the results of the call from their original return value and assumes the DIE pass will remove the now-unused call. Does that imply that in this case there is a way, but it's not a good way?

I originally made this issue while working on the remove value ids PR in which I had to rewrite the resolve_is_unconstrained pass so that it didn't need to call this replace results method which was no longer possible in this PR. I'd still like to remove replace results because I don't think it should be needed if we're already calling set_value_from_id on the results, and I think adding a Nop instruction could be one way to do it.
I mentioned a Nop instruction returning possibly any dummy values but maybe what we'd want instead is for the instruction to always return nothing, since if you insert an instruction you should always be relying on the return values being mapped away already. This'd provide us a bit greater guarantee that these return values which should be removed aren't sticking around I think. resolve_is_unconstrained also isn't the only PR which has this "touch only a few things, then call .retain() on the instructions vec afterward" pattern. IIRC at least mem2reg does it as well with removing unneeded stores.
These Nop instructions would be expected to be removed automatically in a few cases:

Calling .simplify() on them will always remove them. So any pass which re-inserts instructions like unrolling, constant folding, etc would remove them.
They have no return values and aren't side-effectful so DIE will always remove them
Failing all the above (e.g. if they're inserted by the last SSA pass that is run), acir-gen and brillig can simply ignore them since they have no results.


Detect UDS client disconnection

Feature Request
Motivation
We have a UDS server that looks for DHCP leases for a client. When the client disconnects we would like to be able to recognize this and stop DORA on the server side - otherwise the server just keeps searching for a lease. It would be useful in this case to recognize that the client disconnected and that we can do clean up on the server side.
I understand this issue has been talked about before in issue number #377, however, that was related to streamed connections.
Proposal
Im unsure of how this would be implemented. I do think having some sort of channel which sends messages about disconnected clients which would give specific information on which client disconnected would be helpful.

bump.  does anyone have information about this?

Without using streaming currently the server doesn't support this. You could work around this by implementing your own connection type that uses drop to do clean up. Though you would have to use hyper manually and not use the tonic transport since this isn't a feature that comes out of the box.

@LucioFranco Can you explain what you mean by

Though you would have to use hyper manually and not use the tonic transport since this isn't a feature that comes out of the box.

Can you guide me in the right direction for how to do this?

@Jackbaude There should be a uds example that newtypes the connection and you can use that and implement drop to send a message.
You can always add a separate streaming request/response that you use to detect disconnection. Then you have your regular unary request that does the other work.


pygame_learning_environment env implemented

Great project! After going over the source code, I feel that the project is really nicely built. I've decided to try and go ahead an implement the pygame_learning_environment to get familiarized with the api.
My main takeaways are that the api is somewhat confusing (issue #438) and also that some libraries (doom) are deprecated.

Nevermind, something's wrong.

Thanks for integrating!


Administration endpoints and dependancies

Adds endpoints for mno-hub (sync status, launch sync, disconnect account)
Adds a dependancies endpoint (dependancies of mno-components)
Adds a method to get the current connec! version of the tenant

@BrunoChauvet Can you take a quick look? Especially the json response for the endpoints to check that it fits what we need in MnoHub

@BrunoChauvet I've also added a commit for a batched first synchronization as discussed on slack. I don't have a good way right now to store the state to be able to continue in case of error... if you have any miracle idea

@BrunoChauvet I've ETE it, almost worked.
I've also change the endpoint for a toggle one, good idea 👍


TypescriptGoToSourceDefinition failing

Hello 👋🏼,
For certain projects I'm seeing failures to use the source definition mapping. I'm seeing the following issue:
Error executing vim.schedule lua callback: vim/shared.lua:0: Expected table, got nil
stack traceback:
        [C]: in function 'assert'
        vim/shared.lua: in function 'tbl_isempty'
        .../site/pack/packer/opt/typescript.nvim/lua/typescript.lua:2679: in function 'handler'
        /usr/share/nvim/runtime/lua/vim/lsp.lua:1390: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>
Press ENTER or type command to continue

I just installed the plugin, and I'm seeing the same error

Can you try #38?

@jose-elias-alvarez Going to soruce definiton on is here replicates it for me: https://github.com/sQVe/lika/blob/main/src/string.ts#L11


Bookmarks Persist After Toggle in Profile

this PR does the title and also removes localstorage usage since users can't see bookmarks unless they're logged in anyway.

Other than that one comment this lgtm

@ezwang I fixed the bug, was using the list of favorites updated by the user instead of the list of favorites cached in state to check whether to display the placeholder text


Could you please cut a new release

Hi,
thank you for fixing the latest bugs on the RDS logs parser. Do you think you could cut a release with the new changes ?
Thank you in advance,
R

Thanks for the kick Riccardo, release v11.2 have been just published.


Updates to main navigation for the token dashboard dapp

The token dashboard dapp main navigation is updated. This version:

Reduces the visual clutter of line breaks between menu items.
Removes double highlight of arrow and colored bar
Removes the footer bar and adds links to the bottom of the main navigation: Staking docs on Gitbook, Keep website + Join Discord link

@pdyraga @r-czajkowski I added this to the Keep Dashboard column in the Process & Client backlog.

Design work in Figma file


[ALLUXIO-1907] fix and unignore free tests

open this PR to see if the tests are still flaky..

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10077/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.shell.command.FreeCommandTest.freeWildCardTest
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10078/
Test PASSed.

jenkins, test this please

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10079/
Test PASSed.

jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10093/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10100/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10103/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.shell.command.FreeCommandTest.freeWildCardTest
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10106/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.shell.command.FreeCommandTest.freeWildCardTest
Test FAILed.

Merged build finished. Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10108/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.shell.command.FreeCommandTest.freeWildCardTest
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10109/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10110/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10112/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10121/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10123/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10124/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.master.lineage.LineageMasterIntegrationTest.lineageRecoveryTest
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10126/
Test PASSed.

@yupeng9, this is probably ready for review. I tried rerunning build a few times after the fix and the only failure was lineageRecoveryTest, which might be a different flaky test.

Thanks for the fix! Could you briefly describe what caused the flakiness?

Updated the flakiness reason in PR description.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10128/
Test PASSed.

Merged build finished. Test PASSed.

Thanks. I left one comment.

LGTM. It's more for consistency of the utility function use.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10130/
Test PASSed.

Merged build finished. Test PASSed.


W1D1 notebook 1 - text cells.

Fixed two typos in the notebook. 2) Changed 'excluding movie' to 'excluding video'.

@EitanHemed Please file an issue instead, do not merge from forks as we use a specific pipeline. Thank you. I am fixing this now.


luasnip-here in docs

Looks like node_opts has referenced to |luasnip-here| which does not exists. Could you point me to node_opts?

Oh wow, this has gone unnoticed for some time.. Maybe panvimdoc has never been able to properly parse these :/
Anyway, you can check the online-documentation on github instead, DOC.md in the repo-root, I'll look into what's going wrong with the helpfile-generation

@L3MON4D3 Check here, is unfortunate but intentional
https://github.com/kdheepak/panvimdoc/blob/edeefb362ed38da61c6e404e367345b455c489e9/doc/panvimdoc.md#markdown-links

Ah, okay I can kind-of see the sense in that..
Guess we'll have to rephrase those parts such that it works, seems like the path of least resistance


Markdown issue in SECURITY.md file

Running rst-lint will raise the following error:
WARNING .github/SECURITY.md:17 Inline literal start-string without end-string.
WARNING .github/SECURITY.md:21 Inline literal start-string without end-string.
WARNING .github/SECURITY.md:21 Inline interpreted text or phrase reference start-string without end-string.

Based on the articel
https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html#inline-literals
only two "``" shall be used, but this will not great a block !!

Looks like the "-" as first character in first line is raising this error...


Revert "Fix GitLab glitch in tests when changing an MR's target branch. (#29)"

This reverts commit 61c4fbf32e04d523ddb43172bf0bd6f47927c4d0 as now #36 #33 have fixed the GitLab issue.

Fingers crossed!


Frontend API integration on file upload .

This PR includes the API integration of Upload files (/sources) and GetFiles (/sources_list)

https://github.com/neo4j-labs/llm-graph-builder/issues/23


Team update - Jun 22, 2022

This is a @Quansight-Labs/czi-a11y-grant team sync 🎉🎉🎉! This is a way for the Team Members to provide status reports on what they've been up to this week and request help and attention for things they are working on. This issue will be closed at the end of the day.
Copy and paste the template below, and answer questions as you wish!

Response Template
**Thanks I'd like to give 🙌**
- So-and-so helped me out a lot with foo...

**Updates from last week :heavy_check_mark:**
- I worked towards deliverable: <link-to-deliverable>

**Challenges I faced and things I'd like assistance with 🙏**
- I had a hard time figuring out ...

**My availability for next week**
- I'll be off on foo day...

**Important items for discussion 💬**
- I have a question about goal <link-to-goal>
- Can @foo give a comment on issue #NN ?
- I opened #NN for discussion, please chime in
- I need to discuss foo

:mag: Needs Triage
The Needs Triage issues require an initial assessment and labeling.
No issues need triage! 🎉
:dart: Project boards
Please make sure the boards reflect the current status of the project.
Jupyter a11y grant project

As I mentioned in the last update, this week I took a break from CZI work to focus on the website. The only thing I would like to do this week related to CZI is to try to upstream my fork of expect-axe-playwright, which we discussed on Monday.
My availability for next week

I'll be available all next week.


[Feature Request] Convert to ES Modules

With all major browsers now supporting ES modules and node support on its way, I think it would be a great time to migrate this popular lib and help the community move forward.
To be clear, the goal would be to be able to do:
In node:
import {
  after,
  afterEach,
  before,
  beforeEach,
  context,
  describe,
  it
} from 'mocha'

In browsers:
import {
  after,
  afterEach,
  before,
  beforeEach,
  context,
  describe,
  it
} from '/node_modules/mocha/mocha.js'

Mocha supports Node.js v6+.
However, node.js supports ES modules since node 8.5.0 and still Experimental. I believe we can't migrate it.

@outsideris  even if the build step outputs an AMD build for backward compat?

@pdesjardins90 mocha is a test framework. It should be transpiled?

Mocha is used in multiple environments that don't support all the same features, i.e. browsers and node. The repo's current strategy regarding that is outputing a single transpiled script file that can be ran into browsers, while using the CommonJS require() syntax in its core files. The browser script relies on global variables, which is widely considered a bad practice, but was a necessary evil in browsers until ES modules came out.
ES modules bring a native, language-based way of managing modules that will be compatible on both environments. You're right about it being currently experimental in node, but even if it lands in the next version, you're probably going to want to support older runtimes for a while, unless you make a breaking change and force all of your users to upgrade to the latest version of node and ES modules, which may be a little too rough for such a popular and trusted library.
Hence my suggestion to include a backward-compatible version of mocha's CommonJS modules, while updating its source to ES modules.

I'll add that the browser script should still be packaged for the same reasons

@outsideris any chance you'll reconsider?

@boneskull perhaps as top contributor to this repo, you may have a different opinion?

@pdesjardins90
sure, I'd like to modernize the codebase and use new ES2015+ features.
But as long as we target IE11 (still not an insignificant % of market share), that means we will need to transpile our code to support it.
So that's two things:

Configure transpilation (for production and our test suites) which results in both a CJS-based distribution pkg for supported Node.js versions and an IE11-compatible, AMD-compatible bundle
Refactor the codebase to use new JS features (this can be a gradual process)

It's nontrivial and introduces the potential for a lot of regressions.  I don't know if it's worth it.  Maybe better to wait until IE11 drops off of the radar entirely and Node.js has stable native ES module support so we can avoid the headaches?
If someone wanted to do the work in 1., assuming it didn't break things, I'd approve it, but the team would still need to consider the added build and test complexity (maintenance burden) to be worth it.

This might be a duplicate of something.  If it's not, we could leave it open as a "modernize Mocha's build"-type ticket.

(@outsideris is correct, but there's more to the story than just Node's experimental  ES module support)

Ok thanks for the explanation, I understand your predicament better. I'll do a POC on my end, see how it goes and how it may impact maintenance. I'll open a PR if I come up with anything that would check everyone's boxes.
Thanks again for your input

@pdesjardins90 I couldn't find duplicated issue with this. If re-opening this issue and it is helpful to your work, I can re-open it.

Oh I think I’ll work on my end first and when I have something to show for we could reopen this and reference it in the PR for documentation, I’ll keep you posted!


BUG: Fixed parenthesis

As stated in #9882, there is a small formatting error.

thanks @PershingSquare


Fix typo in module.nix

Describe your PR, what does it fix/add?
This PR fixes a small typo in module.nix, which caused builds to fail.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Hyprland built successfully after applying this patch, so there should not be any problems with it.
Is it ready for merging, or does it need work?
This should be ready for merging.

Thanks!


[iOS] Add static linking pod example

prepare for #1159

What's the purpose to have two separate pod specs? I find it hard to support both. What are the benefits of this?

This one is only for development, easier to test compiling from source code, there is one step s.prepare_command = 'tools/install-dependencies && tools/generate-files' in TrustWalletCore.podspec which takes ages to install
there is another probject.yml and binary podspec in ios-release
I can try to simplify protobuf files later

can we rename file to WalletCore-development.podspec? so we know it's only used for development purpose and not for production use.

good suggestion, will do

I will merge this one into pr #1269

I will merge this one into pr #1269


refactor(alert): separate css vars

fixes https://github.com/patternfly/patternfly-next/issues/1162

Deploy preview for pf-next ready!
Built with commit e8f15b6a8607f669b24a0aad873950c022bdbe03
https://deploy-preview-1163--pf-next.netlify.com


Install PHPUnit in PHP container

ISSUE TYPE

Feature request
add phpunit to the php container

SUMMARY

would be useful to have phpunit globally installed in the php container like codeception is.

https://github.com/devilbox/docker-php-fpm/pull/37

Merged, will be available in ca. 30 minutes


importScripts is not defined

I faced this issue using importScripts option.
I want to add additional logic to generated service worker.
So I have a simple script for additional handling fetch event:
self.addEventListener('fetch', function(event) {
  console.log(`One more handler for fetching url: ${event.request.url}`);
});

sw-precache-config.js
module.exports = {
  staticFileGlobs: [
    'docs/**.css',
    'docs/**.html',
    'docs/Images/**.*',
    'docs/**.js',
    'docs/**.js'
  ],
  runtimeCaching: [{
    // exclude googleapis, analitics, drive requests
    urlPattern: /^((?!(googleapis|analytics|googleusercontent)).)*$/,
    handler: 'networkFirst'
  }],
  importScripts: ['test-worker.js'],
  stripPrefix: 'docs/',
  verbose: true
};

Env:

sw-precache v5.0.0
sw generated using sw-precache --config=./sw-precache-config.js --root=docs command

Thanks :)

That should be a supported use case, and I tried something similar just now and the generated service worker file seemed to be okay.
Where exactly are you seeing the "importScripts is not defined" error message reported? Are you seeing it when running the command-line tool to generate the service worker file, or are you seeing it logged in the JS Console when your web app attempts to register and use the service worker file?

@jeffposnick thanks for reply. Yes, file generated correctly, but after sw registration in browser I'm getting this error in console. It's really strange because importScripts part of web workers specification...
In addition sw generated without importScripts works totally fine.

That should be a supported use case, and I tried something similar just now and the generated service worker file seemed to be okay.

Have you tried it running in browser? Does it work correctly?

Yes, it works fine in Chrome 59, and I'd assume other versions as well:

Which browser are you seeing this in? As you say, importScripts() is a standard part of the web platform, so it's odd. The only thing that I could think of is that somehow your generated service worker file is being run from the context of the page and not the service worker?

@jeffposnick It was just me - dumb thing. sw also became part of the bundle and invoked as usual js file, and missed that 😞
Thank you for your help and sorry for waisted time.

@jeffposnick It was just me - dumb thing. sw also became part of the bundle and invoked as usual js file, and missed that 😞
Thank you for your help and sorry for waisted time.

Hi I am facing the same issue. Can you help


Softmax as logistic function

Hi,
I have two questions:

After training mnist, the testing it gets me average accurage of 0.3, any ideas why?
Activations are updated by calculating the cost and applying a logistic function, why is a softmax used instead of sigmoid?

Hi,

Basically, you can use the hyper parameters given in the readme file for similar performance on mnist. Other parameters may lead to failure in training.
Sigmoid function can give a similar performance according to our experiments. Softmax here just theoretically ensures a distribution over capsules.


Investigate the use of compiler macros for Orchestration DSL

The orchestration DSL is currently achieved by using the sbt boilerplate plugin. It could be made far more powerful and flexible by implementing the same using compiler macros. We should investigate and implement if feasible.

Macros will be retired, so marking as wontfix.


Empty categories no longer cause NPEs

Fixes https://issues.jboss.org/browse/AGPUSH-1594
@edewit mind taking a look?

:eyes:

:+1: works like a charm

landed on master (7a3733d) and 11.1x (1d7b0b9886ab69278ad034b1748b683cafc61e66)


Fix broken links to refer a latest version.

Coverage remained the same at 84.0% when pulling beb0ad98782467dfa5142c985800bd3953ad04df on Kroisse:patch-1 into c2fff20c87fdd42eab8654717ee9358b8438640d on dahlia:master.


Variable expression evaluation system

It should be possible to evaluate compound expressions like this in an if statement:
((PlayerName=="Chris" && Score==2) || (PlayerName=="John" && Score==4))
It would also be very powerful to be able to do this in an Event Handler. This is essentially rule based block selection as described here

Requested on the forums: http://fungusgames.com/forum/#!/general:more-complex-ifthenandor

This can now be done via Lua scripting.

Use LuaBindings to bind a Flowchart component
Add an ExecuteLua command to evaluate the expression. Store the return value in a Boolean variable.
Add an If command which checks the value of the Boolean variable.

In the Execute Lua command, you use the Flowchart.GetVariable() method to get the variables to use in the expression. e.g. In Lua,
v1 = flowchart.GetVariable("VarName1").Value
v2 = flowchart.GetVariable("VarName2").Value

return v1 == v2


Multiple SLN within a project

I've a project with multiple SLN's.
These are:

Website
Database
Etc.

Every time I open a new CS file it will ask again what solution file I would like to use.
Is there any way to remember this open?

Try adding this to your .vimrc:
let g:OmniSharp_autoselect_existing_sln = 1

Thank you for your response, after setting the variable you mentioned it is still asking to select the SLN every time I change to a new CS file.

I haven't tested as I'm currently on holiday, but it looks as if that would work - I haven't ever actually used that setting before.
What are you expecting? What happens in other editors like Visual Studio and VSCode when you have 2 solutions in the same space? I can't think of any possible reason to do that?

I hope you have/had a good holiday!
In Visual Studio you can have multiple solutions for including projects or not.
As an example, one solution for the database and an web API and another with only the web API.
I would expect when you enter a CS file for the first time you select the SLN you would like to use.
After that that SLN will be remembered when you change to another (new) CS buffer.
Now, wat is looks like, it will scan for the SLN every time you change to another CS buffer.


Give boss bars a usable identity

At the moment, platform implementations have to rely on hacky maps and wrapper objects to properly use and manage boss bars which is horrible and not fun to use. Additionally, it makes it incredibly hard for plugins to share boss bars around.
It would be nice to make some of the following changes to boss bars:

[ ] Give them an identity (most likely a Key to mimic Minecraft).
[ ] Allow a boss bar to be looked up based on it's identity (either using the API or some form of SPI).
[ ] Add a method to check if an Audience is viewing a boss bar (this would always default to false on ForwardingAudience).

Give them an identity (most likely a Key to mimic Minecraft).

Not all bossbars in Minecraft have a Key-based identity. All bossbars do have a UUID, though.

Add a method to check if an Audience is viewing a boss bar (this would always default to false on ForwardingAudience).

I am not sure about this - I don't think I like the idea of such a method on Audience.

Not all bossbars in Minecraft have a Key-based identity. All bossbars do have a UUID, though.

Yes, but we're not providing a way to obtain boss bars not created by Adventure. Therefore we should be preferring to use an identity that is better for the user and dev experience. If necessary, a key could be created from a UUID, although as I don't think we should provide methods to obtain non-Adventure boss bars as Adventure boss bars, this isn't really necessary anyway.

I am not sure about this - I don't think I like the idea of such a method on Audience.

Why?


Create onboarding guide

Background

We need a standard process we follow when onboarding new team members.
Proposed user journey

Approach

Create runbook
Acceptance Criteria

[ ] Runbook created

Reference
How to write good user stories

Done


How to disable cache

The setting of the cache control parameter can only range from 1d (1 day) to 1y (1 year). When I set it to 0d, the return value is default value one year. How can I change cache-control HTTP-header to no cache, to disable cache?

Hello, did you see https://github.com/weserv/images/issues/292#issuecomment-873653028?

We don't want to have shorter max-ages than 1 day, as it would be unfeasible to host the free service without any caching.

I hope this information helped. Please feel free to re-open if questions remain.


get positon of listview item from adaptor to delete particular item

I am trying to delete particular list row but unable to get positon.

listView.setOnMenuItemClickListener(new SwipeMenuListView.OnMenuItemClickListener() {
        @Override
        public boolean onMenuItemClick(int position, SwipeMenu menu, int index) {
            int id = (int) adapter.getItemId(position);
...

I implemented a delete behavior too. This is the way I used to get position. adapter.getItem(position) allows you to get an object also.


Register PS Repository fails with 'SourceLocation' is an invalid Web Uri

Registering of a private powershell gallery fails with 'SourceLocation' is an invalid Web Uri. even so the URI seems correct and corresponds to the one provided by my Nexus3 server.

I have direct access so no proxy
Changing the url from https to http gives the same error
I can successfully browse the repository

Steps to reproduce
Register-PSRepository -Name TestRepo -SourceLocation https://nexus3/repository/nuget-group -PublishLocation https://nexus3/repository/nuget-psgallery/ -PackageManagementProvider nuget -InstallationPolicy Trusted

Expected behavior
PS Repository is registered successfully
Actual behavior
Error is thrown
Register-PSRepository : The specified Uri 'https://nexus3/repository/nuget-group' for parameter
'SourceLocation' is an invalid Web Uri. Please ensure that it meets the Web Uri requirements.
At line:1 char:1
+ Register-PSRepository -Name TestRepo -SourceLocation https://nexus ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (https://nexus..ory/nuget-group:String) [Register-PSRepository], Arg
   umentException
    + FullyQualifiedErrorId : InvalidWebUri,Register-PSRepository

Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      5.1.14393.1884
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.1884
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

>Get-PackageProvider

Name                     Version          DynamicOptions
----                     -------          --------------
msi                      <IP_ADDRESS>          AdditionalArguments
msu                      <IP_ADDRESS>
NuGet                    <IP_ADDRESS>        Destination, ExcludeVersion, Scope, SkipDependencies, Headers, FilterOnTag...
PowerShellGet            <IP_ADDRESS>          PackageManagementProvider, Type, Scope, AllowClobber, SkipPublisherCheck, ...
Programs                 <IP_ADDRESS>          IncludeWindowsInstaller, IncludeSystemComponent

@papanito Could you repo with latest PowerShell Core build?

Same error for  v6.1.0-preview.518 but that is maybe because the given Uri cannot even be pinged:
ping https://nexus3/repository/nuget-group
Ping request could not find host https://nexus3/repository/nuget-group. Please check the name and try again.

Sometimes I feel it would've been better if this repo was named PowerShellCore or something similar but I understand that it is probably too late now.  But I can also understand that people rather post here because UserVoice is frankly speaking a dead letter box for most MSFT products (I do the same for VSTS issues tbh.). What about having at least a separate WindowsPowerShell repo just for issues that concern only Windows PowerShell (similar to how WiX does it)? @SteveL-MSFT

@iSazonov I can't update my PS. More interestingly, I could do the same command successfully last Friday on the same host. Still try to figure out what is different from Friday to Monday ....
@bergmeister you have to ping your host e.g. ping nexus3 not ping https://nexus3

@papanito PowerShell Core don't replace Windows PowerShell. You can safely install it side-by-side (and just simply copy-paste in any folder from zip distro).

Same problem with PowerShell-6.0.0-rc.2-win-x64

Ok probably not a but at all.... When I enter the url from source location into the browser I get the following error from Nexus3:
Repository path must have another '/' after initial '/'

So after adding a trailing / to the url, no more error
 https://nexus3/repository/nuget-group/

@markekraus Could you please comment the Issue?

@iSazonov I'm not sure what help I can be. The PowerShellGet module and Register-PSRepository don't use the Web Cmdlets. It directly calls to either System.Net.Http.HttpClient or System.Net.WebRequest. the InvalidWebUri error is thrown for a bunch fo different reason in that module.
Only thing I can suggest to @papanito  is to use the server's FQDN instead of just https://nexus3/. But I'm no expert on this module.

@markekraus Thanks! Maybe the question is for PowerShellGet repository.

I agree, maybe @rebro-msft  can help out?

I believe this issue is already being tracked in PowerShellGet repo: Issue 180
And has a  Stack Overflow post.
I'm not familiar with nexus repository, so not sure if the following applies to your scenario.  But when I saw something similar (registering a debug session of PowerShell Gallery locally) it was complaining because the SSL cert used by HTTPS was not in the Trusted Root.  When I fixed that, the issue resolved.
Another workaround is to  I registered the http endpoint, then edit the repositories xml file to use the https endpoint instead.


Scrape decisions from the Trial Decision Library.

This scrapes the table at https://nypdonline.org/link/1016 . Most of these should already be included in the profile data, but in the case a trial decision gets posted when an officer no longer works for the department (e.g. they got fired) it'll show up here but not in the profile data (because their profile goes away when they leave).

Thanks for the addition the functionality looks good. I only have a couple trivial comments on style preferences.


Retinue Touched not giving the spells

retinue_touched_bug.zip
I have taken the feat but the spells are nowhere to be found. My character is Farryn.

All these feats require an existing spell repertoire. Your hero is a Monk without any.

I did not know that. Could you please add that to the description?


[Question/Issue] Draw_editor example with segmentationLUT/Freesurfer

Hello!
I have a question/issue regarding how to access and modify the raw data content of a stack. Particularly, my scene is data + a freesurfer labelmap generated with the segmentationLUT, which can be seen on this live fiddle.
What I want to do is to modify the contents of the stack (as a drawing tool would do, for instance). To do so in the most basic way, I access the contents of the stack2 after the stack2.pack() line and try to change some values. For instance the segmentation label 7 (voxel value present on the stack).
stack2.prepare();
  // pixels packing for the fragment shaders now happens there
  stack2.pack();
  for(let i = 0; i < stack2._rawData[0].length; i++){
    if(stack2._rawData[0][i] != 7) stack2._rawData[0][i] = 0;
  }
  var textures2 = [];

When I do this everything turns to 0 (transparent), except the 7 label, which works as intended. HOWEVER, when trying to do other things such as:
 for(let i = 0; i < stack2._rawData[0].length; i++){
       stack2._rawData[0][i] = 7;
  }
  var textures2 = [];

which should turn the entire stack to one color (the one of the segmentation label number 7) or
 for(let i = 0; i < stack2._rawData[0].length; i++){
       if(stack2._rawData[0][i] == 0) stack2._rawData[0][i] = 6;
  }
  var textures2 = [];

Which should turn everything transparent to color label number 6 for instance, etc. etc. it is not working, everything turns transparent as the labelmap is not applied. The stack values are changed though, what can be the issue there? can it be due to the pixel packing, the shader, the model stack...? Which would be the best way to modify the contents of the stack/texture?
Cheers,

_rawData is the data to be passed to the shaders, encoded in 8 bits.
The data you are looking is encoded in 32 bits (stack2.bitsAllocated). In order to directly update the "packed" _rawData, you'd have to do something like:
   const value = 7;
   for(let i = 0; i < stack2._rawData[0].length; i+=4){
       stack2._rawData[0][i] =  (value) & 0x000000FF;
       stack2._rawData[0][i+1] = (value >>> 8) & 0x000000FF;
       stack2._rawData[0][i+2] = (value >>> 16) & 0x000000FF;
       stack2._rawData[0][i+3] = (value >>> 24) & 0x000000FF;
  }

That is a shortcut not to have to run "pack()" another time as it is expensive.
Depending on the format of the data you want to update, you would have to adjust the approach:
https://github.com/FNNDSC/ami/blob/dev/src/models/models.stack.js#L523
http://jsfiddle.net/2f4bL2w3/19/
For reference, I've been doing something similar in the past but somehow it seems very slow now:
https://fnndsc.github.io/ami/#editor_draw
(was much faster before :/)
Please re-open in needed -

Thank you very much for your quick answer! Refering to the editor_draw example, I have seen that it's slow on Chrome but not on Firefox (draws almost instantly on Firefox) :+1: . I suspect it has something to do with the context.getImageData() method...

Yes that is what I saw in the Chrome dev tools when I had a quick look at the issues but that is actually good if fast in Firefox - hopefully there is a good workaround or Chrome will fix it!

Hello @NicolasRannou ,
I have been fiddling around with the draw example. I have taken the example as it is and changed the data for the freesurfer labelmap (.css , .html , .js can be found on a .zip here (would have made a fiddle but events were not working...)).
On this example I basically changed the data, get rid of the information panel (editorStats) and the segmentation labels that were before for simplicity, and assigned a known label value to the cursor (6), and a color, so basically you are editing the colormap adding the value label 6.
let cursor = {
  color: '#28c928',
  value: 6, //For testing purposes, we paint the segmentation number 6!
  size: 15,
  shape: 'round',
  segment: 'erase',
};

Also in the mapCanvasToData function I changed the old line of updating the rawData for the workaround that you put above.

            if (oldValue != newValue) {
              // update raw data NEW
              stack2._rawData[rawDataIndex][inRawDataIndex] =  (newValue) & 0x000000FF;
              stack2._rawData[rawDataIndex][inRawDataIndex+1] = (newValue >>> 8) & 0x000000FF;
              stack2._rawData[rawDataIndex][inRawDataIndex+2] = (newValue >>> 16) & 0x000000FF;
              stack2._rawData[rawDataIndex][inRawDataIndex+3] = (newValue >>> 24) & 0x000000FF;
              //update raw data OLD
              //stack2.rawData[rawDataIndex][inRawDataIndex] = newValue;

              // update texture that is passed to shader
              textures2[rawDataIndex].image.data = stack2.rawData[rawDataIndex];// tex;
              textures2[rawDataIndex].needsUpdate = true;

            }

When editing, though, nothing is drawn below the cursor... only random pixels from random frames are drawn, it seems that there is an issue on the mapCanvasToData function, when getting the inRawDataIndex... maybe you know what causes this, the textureSize = 4096 maybe has changed or the coordinate changes are messed up (although I do not know, everything is exactly the same as the drawing example but the data...).
Cheers and thank you for your patience,

so therewere a couple of issues:
1- the voxel increment is 4  because we are in 32 bits:
            let voxelIndex = i
                            + j * stack2._columns
                            + k * stack2._rows * stack2._columns;
            voxelIndex *= 4;

2- get the old value:
            let oldValue =
              (stack2.rawData[rawDataIndex][inRawDataIndex + 3] << 24) |
              (stack2.rawData[rawDataIndex][inRawDataIndex + 2] << 16) |
              (stack2.rawData[rawDataIndex][inRawDataIndex + 1] << 8) |
              (stack2.rawData[rawDataIndex][inRawDataIndex]);

And that should be it.
Finally, I think context.getImageData should be moved out of the crazy i/j/k for loops:
-> Get all image data into an array at the very beginning of mapCanvasToData.
-> access the right index from array instead of let pixel = <get right index>.

Thank you very very much Nicolas!! Been bothered by that for quite the time, just an issue of bit packing then, phew.
Greetings,

Hello @NicolasRannou ,
I have been testing how the texture drawing is stored into the data array and if it is consistent with the coordinates etc. and I have a problem accessing the data from the labelmap. All the code can be seen
here. Namely I have implemented this method on mouse move
function raycastLabelmap(event) {
      const canvas = event.target.parentElement;
      const id = event.target.id;
      const scene = sceneLayer0;

      if(typeof lutLayer1 == "undefined") return; //If no labelmap loaded, no point on raycasting it

      const mouse = {
          x: (event.clientX - canvas.offsetLeft) / canvas.clientWidth * 2 - 1,
          y: -((event.clientY - canvas.offsetTop) / canvas.clientHeight) * 2 + 1
      }

      const raycaster = new THREE.Raycaster();
      raycaster.setFromCamera(mouse, camera);

      const intersects = raycaster.intersectObjects(scene.children, true);
      if (intersects.length > 0) {
          let ijk = Utils.worldToData(
              stackHelper.stack._lps2IJK,
              intersects[0].point
          );
          let voxelValue = Utils.value(stack2, ijk);

          if (voxelValue != prevVoxelValue) {
              prevVoxelValue = voxelValue;
              let NameLabelmap = lutLayer1.segmentation[voxelValue];
              if (voxelValue !== 0 && typeof NameLabelmap != "undefined") {
                  document.getElementById("structure-info-label").innerHTML = "Structure name:
" + NameLabelmap['label'];
              } else {
                  document.getElementById("structure-info-label").innerHTML = "Structure name:
 undefined";

              }
          }
      }
    }

Which raycasts the scene and returns the voxel value of the stack labelmap at a given ijk coordinate and gets the structure name when hovering over. It works perfectly on the segmentation loaded. However, when drawing a new segmentation, the stack labelmap value seems not to be updated correctly, as hovering the newly drawn texture do not show its corresponding structure, but seems that, although visually it is drawn, the values are not updated hence the getting the ijk value from the stack gives you the old value at that position even though the new labelmap is drawn correctly.
Do you know what that would be caused for? is it an issue on how do we put the new data into stack2 at the mapCanvasToData function?

Yes, the reason is that when drawing, we directly draw into the "stack.rawData". Its only purpose (at least yet) is to be sent to the GPU for visualization. The actual value of the data is not stored there. It is stored in the frame.pixelData.
In addition of updating the stack2.rawData, you have to update the value of the data stored in the frame (frame.pixelData) - see there how we use it to get data from a frame. (https://github.com/FNNDSC/ami/blob/dev/src/core/core.utils.js#L246-L254)
We will need something similar to set the data of the frame.
Does it make sense?


⚠ Bofrost.be has degraded performance

In c956019, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 884 ms

Resolved: Bofrost.be performance has improved in de84659 after 9 minutes.


Missing filters for product endpoint

Problem Statement
Need to filter down to product by

contains product codes
does not contain inactive ingredient UNIIs

Criteria for Success
Product endpoint filter for product_codes and not_inactive_ingredient_uniis
Additional Information
Desired workflow is probably this:

User searches for RXCUI via a user-friendly search
Query RxNorm for an RXCUI https://rxnav.nlm.nih.gov/REST/ndcproperties.json?id=314200
Gather distinct product codes from this response
User also searches for inactive ingredient UNIIs that they can't tolerate via a user-friendly search
Query DailyMed API for all products that contain ANY of the product codes from RxNorm which also do NOT contain ANY of the inactive ingredient UNIIs

@jrlegrand Why do we not want to return inactive ingredient UNIIs?  Also, by product code, do you mean entire 11 digit NDC or only first two sections?

@jrlegrand Why do we not want to return inactive ingredient UNIIs?  Also, by product code, do you mean entire 11 digit NDC or only first two sections?

It's not that we don't want to return UNIIs. It's that we want to return products that do not have UNIIs that patients are allergic to or can't tolerate. By product NDC I mean the first two segments.

It's not that we don't want to return UNIIs. It's that we want to return products that do not have UNIIs that patients are allergic to or can't tolerate. By product NDC I mean the first two segments.


upgrade macro(keepalived_writing_conf)

Signed-off-by: pablopez<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
me and @pmusa found that some k8s clusters are using kube-keepalive instead of keepalive. This macro is affecting the katacoda lab for k8s environments. This change is not going to break anything, as it just extends the proc.name included in the rule condition. More info about it here: https://github.com/aledbf/kube-keepalived-vip
Example of the logs:
10:37:16.732178071: Error File below /etc opened for writing (user=root user_loginuid=-1 command=kube-keepalived --services-configmap=kube-system/vip-configmap parent=containerd-shim pcmdline=containerd-shim -namespace moby -workdir /var/lib/containerd/io.containerd.runtime.v1.linux/moby/8d542604c0f88c7e6756ba3a95e6cb59da06100c57be29490adedc2b2a9069e2 -address /run/containerd/containerd.sock -containerd-binary /usr/bin/containerd -runtime-root /var/run/docker/runtime-runc -systemd-cgroup file=/etc/keepalived/keepalived.conf program=kube-keepalived gparent=containerd ggparent=systemd gggparent=<NA> container_id=8d542604c0f8 image=gcr.io/google_containers/kube-keepalived-vip) k8s.ns=kube-system k8s.pod=kube-keepalived-vip-h542z container=8d542604c0f8 k8s.ns=kube-system k8s.pod=kube-keepalived-vip-h542z container=8d542604c0f8

@pmusa tested in our environment, and it fix the error (lots of false positives in the output like the one copied above).
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@pabloopez: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pabloopez
To complete the pull request process, please assign kaizhe after the PR has been reviewed.
You can assign the PR to them by writing /assign @kaizhe in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.31.0

@pabloopez: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

d6676c3 Merge branch 'falcosecurity:master' into master

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


Retrieve AWS ALB with Ingress info

Have a use case to retrieve AWS ALB after using Ingress to create it. We need a few information such as hostedzone, DNS hostname, ALB ARN, ALB name etc. The ingress status will return hostname/IP only. Using ALB SDK will need either ALB ARN or Name. However ingress controller will generate ALB name based on namespace, ingress name and hash prefix.
What is the recommendation to resolve this? thanks,
For example, in GCP ingress, additional annotation are added by ingress controller,
 ingress.kubernetes.io/backends: '{"k8s-be-30389--11111":"HEALTHY","k8s-be-32649--11111":"HEALTHY"}'
If we can add the similar annotation and add the alb name, that will be helpful,
 ingress.kubernetes.io/ingress-lb-name: 'hash-ns-ing-name'

Open this again since we still look for this.

Hi,
Can you share your use case for such information?
Personally, I think these information are implementation details, and kube-native applications running inside k8s cluster shouldn't rely on these information.
There might be use cases like management tools outside k8s cluster utilizing such information.

We have a use case to poll ALB status once it is being created.

We use AWS SDK for this purpose now.
AWS ALB can be retrieved using either ALB ARN or Name(ingress controller will generate ALB name based on namespace, ingress name and hash prefix.). Otherwise we have to loop through all the list and find the match for hostname/IP.
We can add annotation for alb name, that will be helpful,
ingress.kubernetes.io/ingress-lb-name: 'hash-ns-ing-name'

If ALB Ingress controller can provide health check and update the status in annotation, we can use the ingress SDK to do this.
For example, for GCP ingress controller we used below,
```ingress.kubernetes.io/backends: '{"k8s-be-30389--11111":"HEALTHY","k8s-be-32649--11111":"HEALTHY"}'````

If you can support 2# then we do not need 1# as we can query ingress only.
If you can not support 2# then at least 1# can help to retrieve ALB.

Hi,
For now, you can still directly get the alb without looping through 😄
alb created with be tagged with below tags:

kubernetes.io/cluster/{your cluster name}:  owned
kubernetes.io/namespace: {your namespace}
kubernetes.io/ingress-name: {your ingress name}
So you can directly get that alb by query using resource tagging api with above tags:  https://docs.aws.amazon.com/resourcegroupstagging/latest/APIReference/Welcome.html

BTW,
I think the above tags are not using a consistent style, i think it might be better to change to below:

kubernetes.io/cluster:  {your cluster name}
kubernetes.io/namespace: {your namespace}
kubernetes.io/ingress: {your ingress name}

Cool, resource tagging API should solve the current problem. thanks,

BTW, it's an good idea to add alb information to ingress, i'll add it in later PR

Yes, those will be nice feature to have so user can use ingress resource to track ALB other than using cloud SDK.


Error building wheel for mujoco-py

Hello everyone,
I'm trying to install Safety Gym on Conda virtual environment (Ubuntu 20.04.4 LTS and Python 3.6.13). I'm having issues with mujoco-py, since it looks like some error is happening while its wheel is building. The error code is the following one:
**ERROR: Command errored out with exit status 1:
command: /usr/bin/python3 /tmp/tmpp4lzfm1t build_wheel /tmp/tmpakrcnohb
cwd: /tmp/pip-install-85me5y04/mujoco-py
Complete output (21 lines):
running bdist_wheel
running build
Removing old mujoco_py cext /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/cymj_<IP_ADDRESS>_38_linuxcpuextensionbuilder_38.so
Compiling /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/cymj.pyx because it depends on /tmp/pip-build-env-9h71wxn_/overlay/lib/python3.8/site-packages/Cython/Includes/libc/string.pxd.
[1/1] Cythonizing /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/cymj.pyx
running build_ext
building 'mujoco_py.cymj' extension
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp/pip-install-85me5y04
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp/pip-install-85me5y04/mujoco-py
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp/pip-install-85me5y04/mujoco-py/mujoco_py
creating /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp/pip-install-85me5y04/mujoco-py/mujoco_py/gl
x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -Imujoco_py -I/tmp/pip-install-85me5y04/mujoco-py/mujoco_py -I/home/ikmobility/.mujoco/mujoco200/include -I/tmp/pip-build-env-9h71wxn/overlay/lib/python3.8/site-packages/numpy/core/include -I/usr/include/python3.8 -c /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/cymj.c -o /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp/pip-install-85me5y04/mujoco-py/mujoco_py/cymj.o -fopenmp -w
x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -Imujoco_py -I/tmp/pip-install-85me5y04/mujoco-py/mujoco_py -I/home/ikmobility/.mujoco/mujoco200/include -I/tmp/pip-build-env-9h71wxn/overlay/lib/python3.8/site-packages/numpy/core/include -I/usr/include/python3.8 -c /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/gl/osmesashim.c -o /tmp/pip-install-85me5y04/mujoco-py/mujoco_py/generated/_pyxbld_<IP_ADDRESS>_38_linuxcpuextensionbuilder/temp.linux-x86_64-cpython-38/tmp/pip-install-85me5y04/mujoco-py/mujoco_py/gl/osmesashim.o -fopenmp -w
/tmp/pip-install-85me5y04/mujoco-py/mujoco_py/gl/osmesashim.c:1:10: fatal error: GL/osmesa.h: No such file or directory
1 | #include <GL/osmesa.h>
|          ^~~~~~~~~~~~~
compilation terminated.
error: command '/usr/bin/x86_64-linux-gnu-gcc' failed with exit code 1
ERROR: Failed building wheel for mujoco-py
Successfully built gym
Failed to build mujoco-py
ERROR: Could not build wheels for mujoco-py which use PEP 517 and cannot be installed directly**
I saw that there are other topics open with related issues, but sadly none of them solutions worked for me. I would apreciate if someone could help me. Thanks in advance. Bye.

I faced the exact same issue and this worked for me.

I faced the exact same issue and this worked for me.

Thanks, it worked. Have a nice day!


connection retry

Is your proposal related to a problem?
Some time fails create connection between tape and peer.
Describe the solution you'd like
add a general 3 times retry for connection creation before error.
Describe alternatives you've considered
N/A
Additional context
N/A

Would connection retry cost more time?

Would connection retry cost more time when calculate the tps?

no， just retry to set up connection.
Tape works as set up connection and send test data.
the calculation starts form data send.

this looks good to me

https://github.com/grpc/grpc-go/blob/02cd07d9bb5609fe76f8bb8d948037e79a8b29c7/examples/features/retry/README.md
https://github.com/hyperledger/fabric/blob/master/internal/pkg/comm/client.go#L178
for my understanding... it seems that fabric client code invokes grpc from it's vendor folder.
which is a/an old version of grpc code which doesn't support retry feature in latest grpc?
@guoger

we could always retry at application layer, or
we could always switch to latest grpc-go if necessary

ok, let's make it at application layer.


The days are straying

i have next options:
  selectDate() {
       this.calendarCtrl.openCalendar({
           title: 'Календарь',
           from: new Date(),
           disableWeekdays: [0, 6],
           weekStartDay: 1,
           weekdaysTitle: ["Вс", "Пн", "Вт", "Ср", "Чт", "Пт", "Сб"],
           monthTitle: 'MMMM yyyy',
           closeLabel: '',
       }).then(res => {
           this.selectedDate = new Date(res.date.time).toLocaleDateString();
       });
   }

And the following happens:

How do I fix this issue in my project?


Allow for additional hyperlink formats

Currently (to my knowledge) Zulip only supports http://, https:// and (if an config option is enabled) file:/// (and probably mailto). We'd like for zulip to allow several additional link formats to be converted into links so they can be dealt with by our system's protocol handlers. Perhaps this could be a settings file only configuration?
More information on protocol handlers
I tried finding this code to provide a helpful link, but couldn't hone in on what is causing the linkification.

This change is planned.; there's an open PR somewhere.  See https://github.com/zulip/zulip/pull/13076.

Oh awesome! Thanks Tim!


Troubleshooting

We have already heard of problems with the data set. Especially with the timing of orders. I.e. the orders cannot be processed in one day because the plants can't handle all the orders even when they are fully busy.
A method to mitigate this problem would be to extend the decision space to incorporate multiple days. The binary variables would then become 3-D matrices.
Another idea is to relaxate the constraints that are making the problem infeasible by slack variables. This however requires the determination of the infeasible constraints. How the relaxation works is also not fully understood.

We realized that there is an infeasibility within the dataset:
Customer of order number 141 (Order ID:<PHONE_NUMBER>) expects product to be produced by plant 2 (VMI) but only plant 3 is capable of producing product no 1699333


Use a dropdown for finishing review?

Here is how GH shows it:

And here is ours:

I find it strange that we use multiple buttons and pick "Comment" as the default, while in 99% of the cases the action picked is likely "Request Changes" or "Approve".
I would suggest turning this into a dropdown button where you then pick your choice.

@alexr00 I am not sure, but should we be using a dropdown here as well?

Good point. I will not be able to add it for this release, but I will add it in December.


[Question] How to test using llava-llama2?

Question
I failed test the llava-llama2 model with the lateset version.
No response

Hi, please follow the instructions here to launch the demo, thanks.
https://github.com/haotian-liu/LLaVA#demo


Hud fill overflow

Hud fill will overflow if current state goes into negative or beyond the maximum value. Given that these values are not supposed to be exceeding said range, this should not be an issue, however, it would be nice to still cover this edge case

Should probably also add clip to these groups. Currently only l-dungeon has clip borders, but it would have been much better to have border clip on every element - just to be sure


"warning: circular argument reference - score" in 1.9.2

I bundle installed gem 'vanity', '1.9.2' configured for Postgres, and tried to run rake db:migrate. Got the following error:
$ rake db:migrate
.../.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/vanity-1.9.2/lib/vanity/experiment/ab_test.rb:305: warning: circular argument reference - score

Line in question: def conclusion(score = score). I started looking into forking, but I see the master branch has def conclusion(score = score()) so this might be in flux. What's the right way to proceed?
Completely possible I just Postgres config-ed wrong and this is all a red herring, but figured I'd check with the experts.

Hm, if there was a specific patch that fixed it, let's cherry pick it back
to the 1.9.2 branch - a PR would be great!
On Tuesday, September 29, 2015, Wyatt Andersen<EMAIL_ADDRESS>wrote:

I bundle installed gem 'vanity', '1.9.2' configured for Postgres, and
tried to run rake db:migrate. Got the following error:
$ rake db:migrate
.../.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/vanity-1.9.2/lib/vanity/experiment/ab_test.rb:305: warning: circular argument reference - score
Line in question: def conclusion(score = score). I started looking into
forking, but I see the master branch has def conclusion(score = score())
https://github.com/assaf/vanity/blob/master/lib/vanity/experiment/ab_test.rb#L296
so this might be in flux. What's the right way to proceed?
Completely possible I just Postgres config-ed wrong and this is all a red
herring, but figured I'd check with the experts.
—
Reply to this email directly or view it on GitHub
https://github.com/assaf/vanity/issues/273.


htmlspecialchars in Admin messages

Screenshots

Server (please complete the following information):

OS: Linux
PHP version: 7.2.31
MySQL version: 5.7.30
Browser: chrome
Version: 3.1.6

Additional context
Add htmlspecialchars decode or change charset to UTF-8

Hello, good morning, I installed the script on my site, but when a user wants to register or the admin wants to enter the user panel, he gives an error: The security code is wrong.  The code is correct but gives an error again: The security code is incorrect


Bug in RSS feed - root blog link

The root blog link takes from the site url (from the request? not sure) but the actual item creation in the RSS takes from the site_url.
Example, I have a local rails server and I get the following RSS feed:
<rss version="2.0">
  <channel>
    <title>Illegal Numbers</title>
    <description>Life. Code.</description>
    <link>http://localhost:3000/blog/</link>
    <item>
      <title>test</title>
      <description>
wysiwyg

</description>
      <pubDate>Thu, 03 Nov 2016 00:00:00 +0000</pubDate>
      <link>http://www.illegalnumbers.com/blog/2016/test</link>
      <guid>http://www.illegalnumbers.com/blog/2016/test</guid>
    </item>
  </channel>
</rss>```

Ah, interesting.
Any chance you can send a PR fixing that?

I can tinker with it when I get a chance!

So if the rss file that gets copied over is the same one used in the engine this
xml.instruct! :xml, version: "1.0"
xml.rss version: "2.0" do
  xml.channel do
    xml.title Monologue::Config.site_name
    xml.description Monologue::Config.meta_description
    xml.link root_url

    for post in @posts
      xml.item do
        xml.title post.title
        xml.description raw(post.content)
        xml.pubDate post.published_at.to_s(:rfc822)
        xml.link Monologue::Config.site_url + post.full_url
        xml.guid Monologue::Config.site_url + post.full_url
      end
    end
  end
end

Would need to change the xml.link to root_url for everything or change it to Monologue::Config.site_url for everything. Either way would stay consistent, but if I patch it I'll probably do the site_url way.

Yup, sounds good to me, thanks @illegalnumbers!


Typescript compatibility?

Hi there,
Does Angulartics work with Typescript?
Thanks.

Yes, just use it in your project with requirejs or commonjs. Any javascript will work with typescript. I don't know about Angular 2, but I'm assuming it should work with it as well.


turtlebot4_setup: 1.0.0-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository turtlebot4_setup to 1.0.0-1:

upstream repository: https://github.com/turtlebot/turtlebot4_setup.git
release repository: https://github.com/ros2-gbp/turtlebot4_setup-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: null

turtlebot4_setup
* turtlebot4_setup tool
* RPI config updates
* Discovery server files
* Contributors: Roni Kreinin

Holding for Humble sync


TBS Caipi parameters

I tested PX4 on my caipi now for a few months and it works great! The attached parameters worked best for me. Is it possible they become the default parameters for Caipi flying wing? I use the 3s power setup.
Caipi PX4.txt

Can you put the important params you want to change in this file https://github.com/PX4/Firmware/blob/master/ROMFS/px4fmu_common/init.d/3100_tbs_caipirinha and post a test flight to logmuncher?
http://logs.uaventure.com/

How can I reverse PWM outputs? I added this:
param set PWM_MAIN_REV0 1
param set PWM_MAIN_REV1 1
It doesn't seem to accept it.

Set them via QGroundControl instead. Make sure to not have any text files on your SD card.

So it's not possible to reverse the PWM outputs modifying following file?
https://github.com/PX4/Firmware/blob/master/ROMFS/px4fmu_common/init.d/3100_tbs_caipirinha

ah, sorry, in the autostart file its possible and correct.

In v1.8.0 the PWM_MAIN_REV* params are not longer listed/displayed in QGC v3.3.2.  Does this mean that we now need to manually set them up in the init file and build the firmware?

They should load in QGC. Did you look in the PWM Outputs group?

Yes, I did.  Only the AUX and DIS are listed.


Utils.fetchFullArtwork() can cause OutOfMemoryError

From #46:

The getArtwork() method in PlayerController sometimes produces OutOfMemory Error in my device(Api 19). I am guessing you need to add some Bitmap Factory tweaks to keep the image size within memory budget.
http://stackoverflow.com/questions/477572/strange-out-of-memory-issue-while-loading-an-image-to-a-bitmap-object

Fixed in #167


The remote server returned an error: (504) Gateway Timeout

Hey,
I think I ran into a throttle when using the Fluent Api.
Now i'm unable to authenticate, I instead receive the error above.
I'm running this code:
 IAzure azure = Azure                 .Configure()                 .Authenticate(auth)                 .WithSubscription("xxxx");
Should I save the IAzure instance as a Singleton? Now i'm creating new ones for different actions.
I'm running this code in a webapi, so every request generates about 1-5 new authentications depending on the call tree.

probably an ASP.nET issue actually - there was a big fix that may have addressed this in v1.2 - pls reopen this issue if it still reproduces in the latest SDK.


Error: connect EACCES /var/run/docker.sock

1/

Hello i try to run my first container with ContainerNursery, i have some errors.
Why ContainerNursery ? :
I run rrr (petio, radarr, sonarr ...) stack thruhg Gluetun (vpn) continer, than expose Petio to internet using network proxy manager.
Unfortunatly Petio has a bug : 100% CPU usage when nothing happening and nobody connected to it.
So i would like to run this container only when someone want to use it and try to connect to it.

Now my setup is this :
Petio (7777) > network_mode: container:${C_GLU} > Gluetun (glu:27777) > nginx proxy manager (http://glu:7777) > https://dn-petio.monsite.ovh
I can access petio from :
http://<IP_ADDRESS>:27777 (ip for synology nas witch host the docker)
http://dn.l:27777 (dn.l is local dns name from Pihole local dns record)
https://dn-petio.monsite.ovh

I try to add new layer with ContainerNursery who will wake up petio when the url is hit.
But i an not shure of my docker compose and config.yml and not sure what to setup inside nginx proxy manager.
Tryed this but have some errors.

All my containers use :
    environment:
      - PUID=${PUID}
      - PGID=${PGID}
or :
user: ${PUID}:${PGID}
(not sure if can do this with ContainerNursery ?)
Have this error in the logs : Error: connect EACCES /var/run/docker.sock

All my containers are in separate docker compose files for granular run on demand.

Also if no network section in docker compose file then ContainerNursery create a new network, so i added network section.

Petio :
version: "3"
services:
  srv_petio:
    user: ${PUID}:${PGID}
    container_name: ${C_PET}
    image: ${C_PET_IMG}
    restart: ${C_ALL_RESTART}
    volumes:
      - ${REP_APPDATA}/${C_PET}/logs:/app/logs
      - ${REP_APPDATA}/${C_PET}/config:/app/api/config
    environment:
      - PUID=${PUID}
      - PGID=${PGID}
      - UMASK=${UMASK}
      - TZ=${TZ}
    network_mode: container:${C_GLU}

Gluetun :
version: "3"
services:
  srv_glu:
    container_name: ${C_GLU}
    hostname: ${C_GLU_HOST}
    image: ${C_GLU_IMG}
    restart: ${C_ALL_RESTART}
    cap_add:
      - NET_ADMIN
    ports:
      - ${C_GLU_8888}:8888/tcp # HTTP proxy
      - ${C_GLU_8388}:8388/tcp # Shadowsocks
      - ${C_GLU_8388}:8388/udp # Shadowsocks
      - ${C_GLU_6681}:6681/tcp
      - ${C_GLU_6681}:6681/udp
      - ${C_GLU_8000}:8000
      - ${C_GLU_6682}:6682    # qbittorrent
      - ${C_GLU_8080}:8080    # qflood
      - ${C_GLU_3000}:3000    # qflood
      - ${C_GLU_8191}:8191    # flaresolverr
      - ${C_GLU_5076}:5076    # hydra
      - ${C_GLU_9117}:9117    # jackett
      - ${C_GLU_27017}:27017  # mongo
      - ${C_GLU_9696}:9696    # prowlarr
      - ${C_GLU_7777}:7777    # petio
      - ${C_GLU_3579}:3579    # ombi
      - ${C_GLU_5055}:5055    # overseerr
      - ${C_GLU_8686}:8686    # lidarr
      - ${C_GLU_7878}:7878    # radarr
      - ${C_GLU_8090}:8090    # mylar
      - ${C_GLU_8181}:8181    # npm
      - ${C_GLU_8787}:8787    # readarr
      - ${C_GLU_8989}:8989    # sonarr
    volumes:
      - ${REP_APPDATA}/${C_GLU}:/gluetun:rw
    environment:
      # See https://github.com/qdm12/gluetun/wiki
      - FIREWALL_OUTBOUND_SUBNETS=<IP_ADDRESS>/24 # noweb
      - VPN_SERVICE_PROVIDER=${VPN_SP}
      - OPENVPN_USER=${VPN_USER}
      - OPENVPN_PASSWORD=${VPN_PASS}
      - SERVER_REGIONS=${VPN_REG}
      - VPN_TYPE=openvpn
      - PUID=${PUID}
      - PGID=${PGID}
      - UMASK=${UMASK}
      - TZ=${TZ}
    networks:
      - web
networks:
  web:
    external: true

NPM :
version: "2.1"
services:
  srv_npm:
    container_name: ${C_NPM}
    image: ${C_NPM_IMG}
    restart: ${C_ALL_RESTART}
    hostname: ${C_NPM_HOST}
    ports:
      - ${C_NPM_8181}:8181
      - ${C_NPM_8080}:8080
      - ${C_NPM_4443}:4443
    volumes:
      - ${REP_APPDATA}/${C_NPM}/config:/config:rw
      - ${REP_APPDATA}/${C_NPM}/tmp:/tmp:rw
    environment:
      - PUID=${PUID}
      - PGID=${PGID}
      - UMASK=${UMASK}
      - TZ=${TZ}
      - DISABLE_IPV6=true
    labels:
      - "com.centurylinklabs.watchtower.enable=${WATCHTOWER_LABEL_ENABLE}"
    networks:
      - web
networks:
  web:
    external: true

Nursery :
version: "2.1"
services:
  srv_nursery:
    user: ${PUID}:${PGID}
    container_name: ${C_NUR}
    image: ${C_NUR_IMG}
    restart: ${C_ALL_RESTART}
    hostname: ${C_NUR_HOST}
    ports:
      - ${C_NUR_80}:80
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ${REP_APPDATA}/${C_NUR}:/usr/src/app/config
    environment:
      - CN_PORT=${C_NUR_80}
      - PUID=${PUID}
      - PGID=${PGID}
      - UMASK=${UMASK}
      - TZ=${TZ}
    networks:
      - web
networks:
  web:
    external: true

config.yml :
proxyHosts:
  - domain: dn-petio.monsite.ovh
    containerName: g-petio
    proxyHost: glu
    proxyPort: 7777
    timeoutSeconds: 600
    stopOnTimeoutIfCpuUsageBelow: 200

Error log :

INFO [2022-11-29 18:23:33.103 +0100]: (Re)loading hosts, clearing all existing hosts first
node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^
Error: connect EACCES /var/run/docker.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1195:16) {
  errno: -13,
  code: 'EACCES',
  syscall: 'connect',
  address: '/var/run/docker.sock'
}
Node.js v17.9.1
INFO [2022-11-29 18:23:47.880 +0100]: (Re)loading hosts, clearing all existing hosts first
node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^
Error: connect EACCES /var/run/docker.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1195:16) {
  errno: -13,
  code: 'EACCES',
  syscall: 'connect',
  address: '/var/run/docker.sock'
}
Node.js v17.9.1
INFO [2022-11-29 18:23:47.882 +0100]: Added proxy host
    host: [
      "dn-petio.monsite.ovh"
    ]
    container: [
      "g-petio"
    ]

    proxy: {
      "host": "glu",
      "port": 7777,
      "useHttps": false
    }
    timeoutSeconds: 600
INFO [2022-11-29 18:23:47.928 +0100]: Proxy listening
    port: 11180
INFO [2022-11-29 18:23:47.940 +0100]: Proxy placeholder server listening
    port: 8080
    host: "<IP_ADDRESS>"
INFO [2022-11-29 18:24:00.164 +0100]: (Re)loading hosts, clearing all existing hosts first
node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^
Error: connect EACCES /var/run/docker.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1195:16) {
  errno: -13,
  code: 'EACCES',
  syscall: 'connect',
  address: '/var/run/docker.sock'
}
Node.js v17.9.1
INFO [2022-11-29 18:24:15.219 +0100]: (Re)loading hosts, clearing all existing hosts first
node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^
Error: connect EACCES /var/run/docker.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1195:16) {
  errno: -13,
  code: 'EACCES',
  syscall: 'connect',
  address: '/var/run/docker.sock'
}
Node.js v17.9.1
Container stopped

2/
Would be nice to be able to choose to "PAUSE/start" instead of "STOP/start" behaviour :)

Please help , best regards :)

Finally this worked :

docker-compose.yml :

version: "3"
services:
  srv_nursery:
    container_name: ${C_NUR}
    image: ${C_NUR_IMG}
    restart: ${C_ALL_RESTART}
    hostname: ${C_NUR_HOST}
    ports:
      - ${C_NUR_80}:80
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ${REP_APPDATA}/${C_NUR}:/usr/src/app/config
    environment:
      - CN_PORT=${C_NUR_80}
      - UMASK=${UMASK}
      - TZ=${TZ}
    labels:
      - "com.centurylinklabs.watchtower.enable=${WATCHTOWER_LABEL_ENABLE}"
    networks:
      - web
networks:
  web:
    external: true

config.yml :

proxyHosts:
  - domain: dn-petio.monsite.ovh
    containerName: g-petio
    proxyHost: glu
    proxyPort: 7777
    timeoutSeconds: 300
    stopOnTimeoutIfCpuUsageBelow: 200

Nginx Proxy Manager :
http://nur:11180


repository: maybe provide a Repository.get_delegating_role()

I wonder if Repository.get_delegating_role(role: str) -> str would be useful

In my implementations I end up looking up the delegating role  of a role quite a lot.
if this existed, Repository could also offer QOL things like Repository.get_verification_result(role, md)

The reason this does not exist is that we can't really provide a fully generic implementation: Repository does not maintain a delegation tree. So we could

Require all Repository implementations to provide the method OR
Provide a default implementation that assumes max one level of targets delegations (and require more complex implementations to provide their own) OR
Provide a default implementation that adds a delegation tree into Repository

Oh this is going to be a nightmare if we allow "diamond" delegation structure (mutliple roles in different delegating metadata delegating to same name)

cc #660


Add support for target x86_64h-apple-darwin

https://doc.rust-lang.org/nightly/rustc/platform-support/x86_64h-apple-darwin.html

@sunfishcode Hello, can you review this PR please?
We recently added support for x86_64h-apple-darwin in cargo-binstall but since target-lexicon isn't able to parse this target it causes an error in our application
https://github.com/cargo-bins/cargo-binstall/pull/1228

@bjorn3 @sunfishcode Friendly pinging as this PR has been stale for at least one week.

Thanks!

This is now released in target-lexicon 0.12.11.