V0.0.8 to mainw

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

173 of 173 (100.0%)  changed or added relevant lines in 25 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 99.857%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
569

Relevant Lines:
569

💛  - Coveralls


refactor: move scott_post_helper below scott_post, use Xhad's one line join in scott_post to parse the args and pass that to the helper function, don't forget to add that to the command invocation terminal output

Issue created by TheSkeward#1111.

' '.join(f'"{arg}"' if ' ' in arg else arg for arg in args)

Done!


Compatibility with napalm-base 0.18.0 and release 0.4.3

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

I've tested this against a couple of devices & run the unit tests locally.
For some reason Travis does not like a detail and the log is not very useful...

Coverage decreased (-0.2%) to 84.38% when pulling acbe0d2f2e61e5691f20d797f90fdac45ad02f5a on mirceaulinic:NB0.18.0-COMPAT into a81346338d4b1b806d25fe58439a152b9548243c on napalm-automation:develop.

+1


STYLE: Upgrade python syntax to 3.6 and newer

Some syntax upgrading is neccessary for SlicerSegmentEditorExtraEffects as I noticed in https://slicer.cdash.org/viewBuildError.php?buildid=2569128 that there were syntax warnings. These warnings are present because Slicer recently upgrade from Python 3.6.7 to Python 3.9.10 in https://github.com/Slicer/Slicer/commit/34e48e8aef5dad19ec8a955d6f48a1940846e3f3.
The syntax warning in question started as of Python 3.8.
https://docs.python.org/3.8/whatsnew/3.8.html#changes-in-python-behavior

The compiler now produces a SyntaxWarning when identity checks (is and is not) are used with certain types of literals (e.g. strings, numbers). These can often work by accident in CPython, but are not guaranteed by the language spec. The warning advises users to use equality tests (== and !=) instead. (Contributed by Serhiy Storchaka in bpo-34850.)

This PR uses pyupgrade to automatically upgrade python syntax to newer versions. The script listed below was run for Python 3.6+ syntax with changes committed, followed by Python 3.7+, then Python 3.8+ and finally Python 3.9+.  There were no changes made when specifying Python 3.7+, 3.8+, or 3.9+ which is why there are no commits for those iterations.
Using pyupgrade

Install: PythonSlicer -m pip install pyupgrade
Running:

On 1 file: PythonSlicer -m pyupgrade --py36-plus MyPythonFile.py
On multiple files: Here is my pyupgrade-script.py written to automate running pyupgrade across all python files in the SlicerSegmentEditorExtraEffects repo. It was run by PythonSlicer pyupgrade-script.py.

# pyupgrade-script.py
import os
import subprocess

search_directory = "C:/Users/JamesButler/Documents/GitHub/SlicerSegmentEditorExtraEffects"
for root, _, files in os.walk(search_directory):
    for file_item in files:
        file_path = os.path.join(root, file_item)
        if os.path.isfile(file_path) and file_path.endswith(".py"):
            subprocess.call(["PythonSlicer", "-m", "pyupgrade", "--py36-plus", file_path])

cc: @lassoan for review


GamePass(insiderUpdate)

Hello. I'm fairly new to these things and am at a lost on as of how to fix this error. I'm playing the GamePass version but Yakuza 0 and Kiwami 1 have worked flawlessly with your manager. Kiwami 2 does not however work, so the only method for me to Mod it is manually placing and replacing (using ParTool/Parc-Archive-Importer) downloaded mods from Nexus. Is there any possible way you could fix this? and BTW xbox app (insider) on PC now allows you to download games wherever you choose and allows file editing so that is big. The manual method I'm using is time consuming and sometimes a hit and miss so although not being able to use RMM for Kiwami 2 is a small inconvenience, It still would be awesome to have things automated with your handy tool again. Thanks for taking the time if any to read this.

Hey there. Does the mod manager work fine with Yakuza 0 and Kiwami 1 with the latest xbox app update? i.e. did you have to dump the games, or are you directly accessing the game files (without dumping)?
As for Kiwami 2, there is a possibility that the game's executable is different from the one in the latest steam version, so that might be causing this issue. Only way to fix this is to add support by finding some specific instructions in the UWP version of the game, which I don't own currently. I won't be able to fix this right now, but I'll keep it in mind for later.

Yes both games worked perfectly with the new GamePass (Insider) update, you can edit files directly no UWP dumping needed. And okay that sounds promising, I tried to fix it myself having no knowledge of coding or languages. After a few dozen debugging and coding videos I realized I bit off more than I could chew. Sorry for the late reply I was exhausted.

I'm still going to try and fix it on my end, It's just going to take me time to learn these things and apply them. Do you have any suggestions on where to begin? I honestly want to learn these things so I could help in the future with projects such as these.

Did you find a fix for this yet?

anything yet?

Problems continue to arise even now.

Has this been fixed?


Add DG changes to PPP

Codecov Report

Merging #540 (12487fc) into master (aa247aa) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 12487fc differs from pull request most recent head d6e9f8f. Consider uploading reports for the commit d6e9f8f to get more accurate results

@@            Coverage Diff            @@
##             master     #540   +/-   ##
=========================================
  Coverage     65.25%   65.25%
  Complexity      737      737
=========================================
  Files           147      147
  Lines          2590     2590
  Branches        282      282
=========================================
  Hits           1690     1690
  Misses          749      749
  Partials        151      151

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


"status" variable dropped in bibtex -> json conversion

Here's a one-liner showing a BibTeX record sent to pandoc-citeproc via stdin:
echo "@article{mweInPress, title = {The title}, author = {Author, First and Other-Author, Second}, journal = {The Journal}, status = {in press}}" | pandoc-citeproc -j -f bibtex

in the output, the "status" field is lost:
[
  {
    "author": [
      {
        "family": "Author",
        "given": "First"
      },
      {
        "family": "Other-Author",
        "given": "Second"
      }
    ],
    "container-title": "The Journal",
    "id": "mweInPress",
    "title": "The title",
    "type": "article-journal"
  }
]

meaning that even if I customize my CSL style to use the "status" field when "issued" is missing, it won't work because "status" is no longer present.  A work-around is to convert the whole BibTeX database to JSON in advance, manually edit the relevant record to restore the "status" variable, and then provide pandoc with the JSON bibliography file instead of the BibTeX one...  but it would be nice if that wasn't necessary.

AFAICT, BibTeX does not support a status field (no such thing mentioned in Patashnik 1988a, 1988b; or Markey 2009).
Biblatex, on the other hand, has a pubstate field (Lehman 2018, 24) which is supported by pandoc-citeproc.

Lehman, Philipp. 2018. “The Biblatex Package.” Version 3.11. http://mirror.ctan.org/macros/latex/contrib/biblatex/doc/biblatex.pdf.

Markey, Nicolas. 2009. “Tame the BeaST: The B to X of BibTeX.” http://tug.ctan.org/info/bibtex/tamethebeast/ttb_en.pdf.

Patashnik, Oren. 1988a. “BibTeXing.” http://mirrors.ctan.org/biblio/bibtex/base/btxdoc.pdf.

———. 1988b. “Designing BibTeX Styles.” http://mirrors.ctan.org/biblio/bibtex/base/btxhak.pdf.

Ah. That would explain why it gets dropped. I was assuming that any syntactically valid BibTeX file would get converted to corresponding syntactically valid JSON, it didn't occur to me that there would also be a lexical check on the BibTeX field names.  But of course there would have to be something in place in order to parse authors / editors differently from titles, etc.  Is there a particular reason why it's good to drop fields that aren't defined in the BibTeX specification?

Actually, pandoc-citeproc does map pubstate to CSL status even when the source is bibtex. Couldn’t you switch to using pubstate rather than status in your bibtex data?

:man_facepalming: of course.  Thanks for pointing to the obvious solution that I was missing.


Physical keys not working on Android

If i get focus in flutter, the physical keys on Android will not work, including the back key and volume keys

Can you confirm if this happens if you remove Flame's GameWidget from the widget tree?
Flame games are just widgets and we don't access those button at all, so I can't see this being cause by flame.

Closing due to inactivity, feel free to add more comments and we can re open this in any case.


chore: upgrade jest-junit to avoid older jest dependencies

Summary
Upgrade jest-junit to the latest version (breaking change between 5 and 6 was reverted). In particular, 6.2.0 removes the dependency on jest-config which pulls in babel-core@6 etc.
I noticed this since react-native had jest-junit in dependencies until #23276. So not as big of an issue. Still a nice cleanup, I'd say
/cc @palmerj3
Btw, any reason Jest is still at alpha?

Changelog

[General] [Fixed] - upgrade jest-junit to avoid older jest dependencies
Test Plan
I guess check that the junit report is still picked up by CI? 😀

babel-core is still in the lockfile since jest is at version alpha.6, btw. Upgrading to alpha.9 (or just stable) will include the internal upgrade to babel@7

Yeah, @mjesun is trying to upgrade to the latest release of Jest but he is having trouble with a jest-worker issue that we haven't figured out yet.


Enable pyright reportMissingModuleSource

Depends on #613

Codecov Report
Base: 82.46% // Head: 81.54% // Decreases project coverage by -0.93% :warning:

Coverage data is based on head (1547df1) compared to base (9ca4d12).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #614      +/-   ##
==========================================
- Coverage   82.46%   81.54%   -0.93%
==========================================
  Files          78       78
  Lines        6085     6085
==========================================
- Hits         5018     4962      -56
- Misses       1067     1123      +56

Impacted Files
Coverage Δ

quetz/tasks/workers.py
76.41% <100.00%> (ø)

quetz/pkgstores.py
42.73% <0.00%> (-11.50%)
:arrow_down:

quetz/tasks/mirror.py
89.54% <0.00%> (-1.05%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Release 3.1.0

Merge 3.1.0 to release branch. Done by below:
git co -b release upstream/release
git pull upstream release
git co -b release-3.1.0
git merge indy-parent-3.1.0 -X theirs

LGTM


Getting 404 when trying to use plugin for IntelliJ

What repository URL are you using, and which version of IntelliJ?

hi ,
intellij:
2016.1.1
repo url:
http://downloads.ceylon-lang.org/ide/intellij/development/updatePlugins.xml

That's odd, the files are here. Does the error occur every time?

working now !
weird... this time i clicked first on 'ceylon runtime' then 'ceylon IDE'

1 min I'm going to delete intellij and try again clicking first on 'Ceylon IDE'...
if works then was bit instability of the repo ?

IDE depends on Runtime, so in theory when you install IDE it should automatically install the Runtime. Perhaps it doesn't find the dependency because I'm using relative paths. I'll take a look when I have time.

yup same error by clicking on 'Ceylon IDE' first.

I've removed CeylonRuntime, everything is now contained in the plugin named "Ceylon IDE". You can check for updates, and disable/remove CeylonRuntime from your IDE.


add new program: inDrive

URL
https://hackerone.com/indrive
Contact
https://hackerone.com/indrive/reports/new?type=team&report_type=vulnerability
Bounty
Yes
Additional Information
https://t.me/indrive_bbp - the official inDrive Bug Bounty Updates channel! Here you'll find all the latest news and updates about our Bug Bounty. We are happy to share with you information about the launch of new services, features, promotions and other information

Hi! Thank you for raising this. I've added the program in https://github.com/disclose/diodb/commit/965ca3cdc1d64b297f976cfb9d279ce549ffc178.


No way to configure shop id for sync storage.

There is no way to configure shop id for command ongr:sync:provide magento
while ongr:sync:execute magento.product requires shop id.
So provide puts data into ongr_sync_storage but execute expects data in ongr_sync_storage_{shop_id} .

After more inspection DoctrineExtractor and PassthroughExtractor does not provide shop id for storage->save

Found requirements to solve this solution:

[ ] Implement shop ids getting from configuration, and passing it to SyncStorageFacility.
[ ] Design item skip pattern in Pipeline #53, to allow skiping items from wrong shops.


MWPW-156388 [MILO][MEP] fix for martech=off when page has manifest with enablements

pages with enablements cannot finish rendering when martech is disabled.

add a check to disable the feature when martech is off

Resolves: MWPW-156388
Test URLs:

Before: https://main--cc--adobecom.hlx.page/products/illustrator?martech=off
After: https://main--cc--adobecom.hlx.page/products/illustrator?martech=off&milolibs=martechofffix

Psi check: https://martechofffix--milo--adobecom.hlx.page/?martech=off

@robert-bogos lint issue fixed
@seanchoi-dev can you review again?  I realized the function in question is used in another location besides MEP.  So the better solution is to update the function.


Fix Rails path mutation

ActionDispatch will mutate the uri of a request before the application
actually gets a chance to see it. This can lead to auth failures where
the client signed with the original version of the path, and the server
is comparing against a mutated version.
Instead for ActionDispatch::Request we should use #original_fullpath,
which is passed to Rails from Rack.

Can you write a spec that can get run through all the request drivers to proves we maintain the trailing slash in each implementation of #request_uri


Is coreos integration broken in v0.11.0?

Currently i can't find any up to date docs about how to running heapster on coreos.
If i try to put --coreos parameter or --fleet_endpoints heapster print help with:
flag provided but not defined: -coreos

Is coreos support dropped or I need to use specific environment variables?

I apologize for the confusion. The support is still intact. The
configuration has changed. Take a look here
https://github.com/GoogleCloudPlatform/heapster/blob/master/docs/source-configuration.md
.
We had to update the configuration semantics to make it possible to support
more sources in the future.
On Mon, May 4, 2015 at 7:36 AM, Vadim<EMAIL_ADDRESS>wrote:

Currently i can't find any up to date docs about how to running heapster
on coreos.
If i try to put --coreos parameter or --fleet_endpoints heapster print
help with:
flag provided but not defined: -coreos
Is coreos support dropped or I need to use specific environment variables?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/issues/273.

Thank you, it's now works.
systemd unit file now looks like this:
[Unit]
Description=Heapster monitoring tool for cluster
After=docker.service
Requires=docker.service

[Service]
ExecStartPre=-/usr/bin/docker kill heapster
ExecStartPre=-/usr/bin/docker rm -f heapster
ExecStart=/bin/bash -c "source /etc/environment && /usr/bin/docker run --name heapster -p 8082:8082 -e SINK=influxdb -e INFLUXDB_HOST=http://<IP_ADDRESS>:8086 -e FLAGS=\"--source=cadvisor:coreos?fleetEndpoint=http://$COREOS_PUBLIC_IPV4:4001\" kubernetes/heapster:v0.11.0"
ExecStop=/usr/bin/docker stop -t 2 heapster


chrore: prepare for v1

@all-contributors please add danielfsousa for code, doc, infra and maintenance


rollingstone.com banner ads hang Firefox when using Shumway 0.8.303

Shumway 0.8.6 (the version currently bundled with Firefox Nightly 30) can play the following page's ads correctly, but Shumway 0.8.303, the current addon version, will hang Firefox (I tested 27–30) or OS X, requiring a hard reboot.
http://www.rollingstone.com/culture/news/the-flight-of-the-birdman-flappy-bird-creator-dong-nguyen-speaks-out-20140311

That particular ad doesn't exist anymore, but the current one also doesn't work. Filed bug 1142518.


mtime should be accurate to nanoseconds after sync

What happened:
root@bench-01:~/juicefs# rm xxxx yyyy zzzz -rf && touch xxxx  && ./juicefs sync xxxx yyyy && cp -a  xxxx zzzz  && stat xxxx yyyy zzzz
2023/12/07 14:34:39.250032 juicefs[471227] : Prometheus metrics listening on <IP_ADDRESS>:9567 [mount.go:158]
2023/12/07 14:34:39.250385 juicefs[471227] : Syncing from file:///root/juicefs/xxxx to file:///root/juicefs/yyyy [sync.go:1099]
Scanned objects: 1/1 [==============================================================]  974.2/s used: 1.033498ms
Skipped objects: 0                 0.0/s
Pending objects: 0                 0.0/s
Copied objects: 1                 1230.5/s
Copied bytes: 0.0 b (0 Bytes)   0.0 b/s
Failed objects: 0                 0.0/s
2023/12/07 14:34:39.251453 juicefs[471227] : Found: 1, skipped: 0, copied: 1 (0 B), failed: 0 [sync.go:1138]
File: xxxx
Size: 0         	Blocks: 0          IO Block: 4096   regular empty file
Device: fc01h/64513d	Inode: 4325528     Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2023-12-07 14:34:39.250585739 +0800
Modify: 2023-12-07 14:34:39.218585555 +0800
Change: 2023-12-07 14:34:39.218585555 +0800
Birth: -
File: yyyy
Size: 0         	Blocks: 0          IO Block: 4096   regular empty file
Device: fc01h/64513d	Inode: 4325543     Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2023-12-07 14:34:39.218586000 +0800
Modify: 2023-12-07 14:34:39.218586000 +0800
Change: 2023-12-07 14:34:39.246585717 +0800
Birth: -
File: zzzz
Size: 0         	Blocks: 0          IO Block: 4096   regular empty file
Device: fc01h/64513d	Inode: 4325544     Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2023-12-07 14:34:39.218585555 +0800
Modify: 2023-12-07 14:34:39.218585555 +0800
Change: 2023-12-07 14:34:39.250585739 +0800
Birth: -
What you expected to happen:
mtime should be accurate to nanoseconds
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?
Environment:

JuiceFS version (use juicefs --version) or Hadoop Java SDK version:
Cloud provider or hardware configuration running JuiceFS:
OS (e.g cat /etc/os-release):
Kernel (e.g. uname -a):
Object storage (cloud provider and region, or self maintained):
Metadata engine info (version, cloud provider managed or self maintained):
Network connectivity (JuiceFS to metadata engine, JuiceFS to object storage):
Others:

Most of the storage does not support mtime in nanosecond, so we may not fix it.


spark.sql.shuffle.partitions option should be set

In the Spark properties in UserDefinedInstanceProperty there is a reference to setting spark.sql.shuffle.partitions but it is not actually set.

Merged into main.


add data package specification validation tool

add data package specification validation tool

Thanks!
What do you think of this description?
[GFTS Data Package specification](https://github.com/Stephen-Gates/GTFS) - A [Data Package specification](http://specs.frictionlessdata.io/data-packages/) with validation accomplished with [Good Tables](http://goodtables.okfnlabs.org/). Includes a data package, schemas, tests, and uses South East Queensland GTFS data as an example.

Great description.
What do you think about moving some of this inventory activity to TransitWiki?
I started a category here: http://www.transitwiki.org/TransitWiki/index.php?title=Category:GTFS-consuming_applications http://www.transitwiki.org/TransitWiki/index.php?title=Category:GTFS-consuming_applications
GTFS page: http://www.transitwiki.org/TransitWiki/index.php?title=General_Transit_Feed_Specification http://www.transitwiki.org/TransitWiki/index.php?title=General_Transit_Feed_Specification

On 30 Sep 2016, at 9:25 AM, Luqmaan Dawoodjee<EMAIL_ADDRESS>wrote:
Thanks!
What do you think of this description?
GFTS Data Package specification - A Data Package specification with validation accomplished with Good Tables. Includes a data package, schemas, tests, and uses South East Queensland GTFS data as an example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/luqmaan/awesome-transit/pull/51#issuecomment-250789290, or mute the thread https://github.com/notifications/unsubscribe-auth/AGMdl2zISi5RIZNwIyEtBkDwtgSUfneCks5qvTgKgaJpZM4KLNQy.

Thats up to @barbeau

@antrim that's a good question,  I've been debating it myself. Seems like developers are more likely to stumble on Github,  while Transit wiki is turning into a great resource.  Maintain two lists is obviously more work.

To revisit moving to Transit Wiiki - one thing I think Github is good at is facilitating conversations surrounding organization of ideas.  For example, if someone wants to add something in a new section to the list, on Github they can open an issue or we can comment on pull requests.  You don't get that on TransitWiki - one person makes the edit, and for the most part no one else knows until you stumble on it later, without a simple way to publicly discuss those edits.
So I think keeping this repo is a good idea - it helps build a community.  This repo should continue to be a simple list, though, with TransitWiki holding more extensive content.  My two cents.


[Enhancement] Development Container (devcontainer)

Hey,
I just started with MARLlib and I like the project a lot. However, it was a quite painful experience for me to not only get it to run on my Windows machine (with the updated Dockerfile) but to also have an editable/developing version.
I know that Windows/Mac support is on the Roadmap and I think that's a good move (even though I personally would prioritize dependency upgrades more). Anyway, you can already now have full functionality/support on your Windows/Mac system with the help of Docker and the magic of a development container (devcontainer).
What are devcontainers?
Development containers are specifically designed containers for developers, basically emulating a nice virtual machine with a magic setup. A general overview, including supporting tools like IDEs (VSCode, etc.) can be found here at containers.dev. Github also has a nice introduction in their github docs.
Why devcontainer?
As I already mentioned, this way you can work on any platform. Moreover, if anything breaks (dependencies or similar) that I can't easily fix, I simply rebuild the container and everything works again. Bonus, I can add any code, have editable library versions, control resources, simplify collaboration, and more. Finally, integration with VSCode (my preferred IDE) is seamless and as easy as it gets.
My proposal
I created a devcontainer setup for the MARLlib project. With the extra .devcontainer folder, if someone clones this project and uses a supporting IDE, the IDE will directly ask if this project should run in a devcontainer that certainly comes with a compatible system+dependencies.
I will create a pull request soon and link it here for discussion. Please let me know if you've got questions or concerns.

Thanks for your contribution. Pull request #168 has been merged.


Improve InvokeApiAsync documentation

This is was originally raised here, but got automatically closed when azure-mobile-services was deprecated. In case the archive link gets 404'ed, I took a screenshot of the original issue.
@Injac made this comment:

If you set a custom route-prefix like:
[RoutePrefix("Mothership")]
on a custom controller and you try to call an action, that is attributed as well like this:
[Route("connectioncheck")]
The usual way would be:
var result = await App.client.InvokeApiAsync("Mothership/connectioncheck",HttpMethod.Get,null);
Which will not work, until you change the controller-wide attribute to:
[RoutePrefix("api/Mothership")]
Maybe this should be documented somewhere. Client adds the prefix "/api/..." to every call. Therefore the call will fail, if you add a custom attribute, because the route lacks the "api" part.

I made this comment:

I agree with @Injac this should be documented a bit more explicitly.
I had to dig through the code for MobileServiceClient to understand what was going on.
Like @Injac, I had a routing problem and found out InvokeApiAsync prepends api/, this is implicit behaviour and I feel should be documented explicitly.
I also found out that I can override this implicit behaviour by making sure my usage starts with / - again this is not documented anywhere. I had to read the source code  to find out I could do that.
Cheers,
Indy

I know in the grand scheme of things this isn't a super important issue, but it's an issue I lost a few hours to. And would like to see resolved. If pull requests are welcome, then I'd be happy to do one.
Thanks again,
Indy

Thanks for the report - totally agreed here.  I'll get it updated.

Corrected in the azure docs repo.  Will be live with the next publication to docs.microsoft.com.  Reference: https://docs.microsoft.com/en-us/azure/app-service-mobile/app-service-mobile-dotnet-how-to-use-client-library


Adding exacaster

@Pasalietis could you please sign the CLA (if that's OK on your end) so I can merge this?

@Pasalietis could you please sign the CLA (if that's OK on your end) so I can merge this?

Already signed in


"Mark as unpaid" button not appearing

According to #2324 host admins should have the ability to unpay expenses, but the button is not showing up for most expenses.
I can only see it for Collectives where I am an admin of the Collective. As host admin, I should be able to see the button for all Collectives under my host.
Collective where I am the admin:

Collective where I am not the admin (but I am host admin):

Question: should "mark as unpaid" be available for all paid expenses, or has it only been implemented for manual payment type (not Paypal API transactions)?

    const canMarkExpenseAsUnpaid =
      LoggedInUser &&
      LoggedInUser.canPayExpense(expense) &&
      expense.status === 'PAID' &&
      expense.payoutMethod === 'other';

Just looked at it and it's only available for manual payment type.

We need this paypal expense to be marked as unpaid as the funds were returned: https://opencollective.com/keepassxc/expenses/12164
Can someone please do that?
@znarf it would be very helpful to have the mark as unpaid button appear to fiscal host admins for all expenses, manual or paypal.

We have now re-paid https://opencollective.com/keepassxc/expenses/12164 so we don't need it marked as unpaid anymore.
However it would be useful to have this button available for all expenses regardless of payment type.

Resolved by https://github.com/opencollective/opencollective-frontend/pull/4383 + https://github.com/opencollective/opencollective-api/pull/4020


Add explicit rule for uninitialized variables

Our team is seeing this error while running bslint:
error BSLINT1001: Using uninitialised variable 'isSecondaryTemplate' when this file is included in scope 'components/core/Pages/BasePage.xml'

This is for a situation that should 100% be an error for a production deploy, because it will always cause a crash if this code is executed, but our current situation involves porting a project between repositories piecemeal and these errors are occurring in what's temporarily dead code. It would be nice to have this be an explicit rule that could be disabled or set to a warn severity.

You can ignore that rule on a per-line basis by adding `bs:disable-next-line above the lines in question. Would that help your issue temporarily?


add microphone support for video recording

ability to hear my own voice through my mic when recording arenas

Added in 4.8.1


Added code-you-can-play

Added Definition of Done: Fun - Building Code You Can Play  entry to ReadMe.md and pdf to talks.

Thanks!

Thank you!
On Tue, May 16, 2023 at 10:10 AM Steve Smith @.***>
wrote:

Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/stirtrek/2023MayStirTrek/pull/14#issuecomment-1549755692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBSKB7AWJCMSMJ73XIGWGDXGODEZANCNFSM6AAAAAAYC4VADU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Krista Campbell
@.***


migrate entity pages to use relative routes for react-router v6 stable compatibility

Hey, I just made a Pull Request!
More work towards #9913, on top of #10450
This migrates the absolute entity paths to be relative, which we'll for react-router v6 stable
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@freben yep, although will do another round of verification once all builds are green. Pretty much all of the beta.0 -> stable changes seem to be possible to express in ways that are compatible with both, so I think that's how we'll do the migration. Migrations, deprecations, and optional version bump going out in 1.1, and then in some later release we drop support for beta.0


Provide more than 300 results

It looks like the maximum return of files for this project is 300. It would be wonderful to have the return be higher than that so truly all of the data is surfaced through this tool.
If not in browser, as a downloadable file.

One person typing in data.gov would give us a $1000 API bill from Bing. We're always going to need limitations on this, because it can theoretically cost an unlimited amount to run.

Would caching results and searching for the next 300 datasets over and over be a possibility? If folks want more results, should they just use the Bing or Google UI (does that max out too)?

No, we're charged for every query. We cap results at 300—Bing doesn't. 600 just costs us twice as much as 300. :) It's a setting in the config file—anybody can download LMGTDFY, set the cap at whatever they like,  run it themselves, and pay Bing for that API access.

Here's Bing's API pricing, incidentally. It's my understanding that a "transaction" provides up to 50 results. We're on the free tier right now, and that'll get us a maximum of 5,000 domains entered a month (assuming that every search returns 0–50 results). It'll get us a minimum of 833 domains a month (assuming that every search returns 300 or more results). How many results we're going to average, it's too soon to say—it hasn't even been 36 hours. But once we have some baseline usage data, it'll be possible to tweak the number of returns.
With project funding, I'd make the cap much higher—perhaps 1,000 results? But right now, U.S. Open Data is in no position to make a long-term financial commitment to keep the site operating—it's best to keep it on the free tier.
One more point, for the record: I'm just talking about our installation of this software. Anybody's free to install it on their own system, get a Bing API key, and set the cap at as many results as they want!

Ah thank you for providing that information here!

So much fun! Also, as an FYI and not an issue (at least from my perspective), I'm getting 600+ results from my state .gov domain searches.

Huh. That's odd. Looking through the records, I'm not seeing any searches yielding more than 300 results. I wonder if that's resulting from duplicates, per #18?

That looks like it's probably it - at least for the most part. For
Virginia.gov I got 650 records, of which 335 were duplicates.
On Tue, Feb 24, 2015 at 6:22 PM, Waldo Jaquith<EMAIL_ADDRESS>wrote:

Huh. That's odd. Looking through the records, I'm not seeing any searches
yielding more than 300 results. I wonder if that's resulting from
duplicates, per #18 https://github.com/opendata/lmgtdfy/issues/18?
—
Reply to this email directly or view it on GitHub
https://github.com/opendata/lmgtdfy/issues/26#issuecomment-75870665.

--
Emily Shaw
National Policy Manager | Sunlight Foundation |
(o)<PHONE_NUMBER> x 282 | (c)<PHONE_NUMBER>
@emilydshaw http://twitter.com/emilydshaw

Ugh.


boxslider - only first slide loads

Hi.
Congrats for the great plugin.
We create multiple widgets with it in wordpress. We want for a client dashboard at backend. The thing is that we use also boxslider and only the graphs at first slide appear.
we use the followin code:
var slider=$('.bxslider').bxSlider({
mode: 'horizontal',
easing: 'ease',
preventDefaultSwipeY:true,
onSliderLoad: function(i){
create_them($('li[data-slide="'+i+'"]:not(.bx-clone) .tocanvas'),1);
},
onSlideAfter: function($slideElement, old, nw){
create_them($('li[data-slide="'+nw+'"]:not(.bx-clone) .tocanvas'),1);
}
});
on slide after should be the next graphs created. But the function is running fine, chart object is being returned but the canvas shows nothing. What is more, attributes like width and height are not being added. It is obvious that is a problem with render. We have tried everything and we have gone deep inside the code but we cannot find a solution.
Thanks for your time.

So after some more debugging we figured out that only the last slide is not displaying.
Box slider does not use display none. It places the slider vertically and changes their position.

So we found a fix around not a solution.
We call first the function to create all the graphs and after 100 ms we initialize the bxslider. It is not what we wanted to achieve but for now it works. The render issue although should be reported for the combination of chartjs and slider.


New page in quickstart derived types

A new introductory tutorial about derived types placed as the last page in the Quickstart tutorial.

#build_preview

Hi @smeskos, this is great! Many thanks for adding this. I will go over and review in detail soon :+1:
Quick observation: one of your commits seems to modify a lot of files, and I can't work out why?

Thanks a lot, Stavros! This is a great addition.
However, I'm sorry if this wasn't clear (we need to improve the contributing guide): _site/* directory and any files built by Jekyll shouldn't be committed to the repo. They are built and deployed automatically by GitHub.
I think the easiest way to fix this is to close this PR and open a new one which would include only your additions to the Markdown file where the tutorial is, and the associated metadata (.yml).

Ah I see what's happended.
_site/ files are added in 82487cee6fdcf03d305c262ba46a9442541de6c9.
So git revert 82487ce should fix this branch.

I will try to fix this.

#delete_preview


nvim 0.10 inlay hints broken

I see in a recent commit you added support for nvim 0.10 inlay hints.
On the event LspAttach with clangd, the field client.server_capabilities.inlayHintProvider is not set to true. However, clangdInlayHintsProvider is - I think clangd may be calling the inlay hint field something other than what the standard expects? Either way, the error stems from when a lsp request is sent in this line and the provider is not found in the LSP config.
I get the following error when opening up an empty c++ file in nvim 0.10:

Let me know if I can provide more info or if you think my environment is set up incorrectly.

Also, notably, changing the "textDocument/inlayHint" back to "clangd/inlayHints" works like a charm:

You need to update clangd

In any case this is the wrong place if you're using LSP inlay hints - this plugin is only for off-spec features
To reiterate, either -

Update clangd, use 0.10's LSP inlay hints and disable inlay hints in clangd_extensions.nvim
Use clangd_extensions.nvim and don't enable 0.10's LSP inlay hints

Thank you for explaining the issue clearly. Apologies for my misconceptions.


'NoneType' object has no attribute 'get'

Latest dev version (2f91fd0410eeda430aeb9cb956885eda8bf04e22), when trying to catch a pokemon I get:
[11:03:54]  Starting PokemonGo Bot....
[11:03:54] Something rustles nearby!
[11:03:54]  A Wild Voltorb appeared! [CP 177] [Potential 0.71]
[11:03:54] IV [Attack/Defense/Stamina] = [14/10/8]
[11:03:57]
[11:03:57]  Ran for 0:00:11
[11:03:57]  Total XP Earned: 0  Average: 0.00/h
[11:03:57]  Travelled 0.00km
[11:03:57]  Visited 0 stops
[11:03:57]  Encountered 0 pokemon, 0 caught, 0 released, 0 evolved, 0 never seen before
[11:03:57]  Threw 0 pokeballs
[11:03:57]  Earned 0 Stardust
[11:03:57]
[11:03:57]  Highest CP Pokemon:
[11:03:57]  Most Perfect Pokemon:
Traceback (most recent call last):
  File "pokecli.py", line 446, in <module>
    main()
  File "pokecli.py", line 71, in main
    bot.tick()
  File "/home/nepoxx/dump/PokemonGo-Bot/pokemongo_bot/__init__.py", line 92, in tick
    if worker.work() == WorkerResult.RUNNING:
  File "/home/nepoxx/dump/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_visible_pokemon.py", line 24, in work
    return self.catch_pokemon(self.bot.cell['catchable_pokemons'].pop(0))
  File "/home/nepoxx/dump/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_visible_pokemon.py", line 36, in catch_pokemon
    return_value = worker.work()
  File "/home/nepoxx/dump/PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 71, in work
    if not self.should_capture_pokemon(pokemon_name, cp, pokemon_potential, response_dict):
  File "/home/nepoxx/dump/PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 321, in should_capture_pokemon
    cp_iv_logic = catch_config.get('logic')
AttributeError: 'NoneType' object has no attribute 'get'

I've seen a few issues on here as well but there's no information on the resolution: https://github.com/PokemonGoF/PokemonGo-Bot/issues/2464 https://github.com/PokemonGoF/PokemonGo-Bot/issues/2146 https://github.com/PokemonGoF/PokemonGo-Bot/issues/1994 https://github.com/PokemonGoF/PokemonGo-Bot/issues/1399
I tried capturing a pokemon the "old fashioned way" with my phone/legit app and it worked, so I don't this is a server issue.

The way I ended up fixing it was by copy and pasting most of the "release" section in the config.json over top my own after playing with some of the values, and that seemed to fix it.
When it doubt, just git pull and redo your config from scratch.

I can't figure out what was wrong with my config/repo, but that did the trick. Much appreciated!


Refactor to generalize

This moves the policy rules to policies package so we can generalize over all policies. That is, in main we no longer need to call out each policy by name.
The test for main is now a bit more fragile as changes to the policies could fail the test, this is to assert that the output from main hasn't changed in it's shape (say from a set to an array).

Lgtm. Could be out of scope in this commit, but did you consider ditching the "hacbs.contract" name space? Not sure if it's bringing much value.

Need to remember to update the task as well...

Lgtm.


[SPARK-20114][ML] spark.ml parity for sequential pattern mining - PrefixSpan

What changes were proposed in this pull request?
PrefixSpan API for spark.ml.
Note: Currently there's no sequential rule mining support in mllib.PrefixSpan so we cannot implement PrefixSpanModel.transform. In this PR, the transform do nothing. Or we can throw out unsupport-operation exception. I am open to both of them.
How was this patch tested?
Testsuite added.

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/testing-k8s-prb-make-spark-distribution/1487/
Test PASSed.

Test build #88207 has started for PR 20810 at commit c1c5338.

Test build #88207 has finished for PR 20810 at commit c1c5338.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/88207/
Test PASSed.


Crash after copying linux files to folder then selecting next to list files to pick as an exe

When i select a linux gameimage and i copy the relevant files to the linux folder then clicking next so gameimage lists the exe file to pick or script file gameimage will randomly just close. Maybe something to do with the amount of files it is reading? Also when it randomly does show a list of files and i select the script i want to run, gameimage doesnt run the script.
When i run the script without gameimage from file explorer everything runs fine.

Thanks for reporting this issue @Blad3forc3 ! I'll check it out.

Hello @Blad3forc3 , let me know if v1.5.1 fixes this issue with crashing on selection.

It doesnt appear to RUan. I am trying to get the linux version of civilization v into a gameimage and after copying the files over it pauses then just closes.
The log file doesnt really give anyything interesting either

This is happening still.
I ran gameimage from the command line and it gives me a bit more when it closes by itself
I :: Access 'Monster-Hunter-Stories'
I :: Access 'path_dir_project'
I :: Access 'path_dir_project_root'
gameimage.rs:58: I :: Search directory '/media/ade/Temp/Temp/build/Monster-Hunter-Stories/opt/gameimage-games/Monster-Hunter-Stories/wine/drive_c'
gameimage.rs:58: I :: Exclude directory '"/media/ade/Temp/Temp/build/Monster-Hunter-Stories/opt/gameimage-games/Monster-Hunter-Stories/wine/drive_c/windows"'
gameimage.rs:58: I :: Exclude directory '"/media/ade/Temp/Temp/build/Monster-Hunter-Stories/opt/gameimage-games/Monster-Hunter-Stories/wine/drive_c/ProgramData/Microsoft/Windows"'
gameimage.rs:58: I :: Exclude directory '"/media/ade/Temp/Temp/build/Monster-Hunter-Stories/opt/gameimage-games/Monster-Hunter-Stories/wine/drive_c/users/gameimage/AppData/Local/Microsoft/Windows"'
gameimage.rs:58: I :: Exclude directory '"/media/ade/Temp/Temp/build/Monster-Hunter-Stories/opt/gameimage-games/Monster-Hunter-Stories/wine/drive_c/users/gameimage/AppData/Roaming/Microsoft/Windows"'
gameimage.rs:58: I :: Starting IPC for "/opt/gameimage/bin/gameimage-cli"
gameimage.rs:58: I :: key identifier: /opt/gameimage/bin/gameimage-cli
gameimage.rs:58: I :: Generated message_queue key:<PHONE_NUMBER>
gameimage.rs:58: I :: Message queue id: 32810
ipc.rs:91: Frontend key is:<PHONE_NUMBER>
ipc.rs:112: Closed existing message queue
search.rs:27: Finished reading messages
search.rs:32: Backend exited successfully
wine.rs:716: Could not read input args: No such file or directory (os error 2)
Error sending request: Broken pipe
Fatal error while communicating with the Wayland server: Broken pipe

This happens with wine games too if i add files to the wine directory.
It doesnt happen every time. It seems to crash when a large amount of separate files are added and it tries to read the list


GRADING: Dress Rehearsal

Usage

Frontend?

Elaborate:

Backend?

Elaborate:

APIs?

Elaborate:

Agile Methodology?

Elaborate:

Hook

Points: 3.6-4.0
Why that score?:

Knowledge

Points: 3.6-4.0
Why that score?:

Value

Points: 0.6-1.0
Why that score?:

WOW Factor:

What & why?:

I liked the user interface, it was very user friendly and interesting. The program idea was good too.
Hook: 3.9, good catch of attention
Knowledge: 3.95 showed good reflection of understanding program
Value: 1
Wow Factor: Very interesting idea and design

USAGE OF:

[ ] Frontend
[ ]  Backend
[ ] API
[ ] Agile

HOOK:
Points: 3.7
Reason: Had good reasoning as to why I should like his project
KNOWLEDGE:
Points: 3.8
Reason: Talked well about how everything worked. Seemed like he knew what he was talking about
VALUE:
Points: .9
Reason: I like their project, was cool to see initiative into learning
WOW FACTOR: designed a similar game to my childhood
Reason:
SCORE: 8.4/9

Criteria/Key Features:

[x] Frontend
[ ]  Backend
[x]  API
[x]  Database
[ ]  Agile, several students interacting with answering questions.

HOOK:
Points:
Reason:
KNOWLEDGE:
Points:
Reason:
VALUE:
Points:
Reason:
WOW FACTOR:
Reason:
8.9/9

Scores

8.5/9
8.4/9
8.85/9

Average: 8.58/9 = 95.37%

Criteria/Key Features:
Frontend
Backend
API
Agile, several students interacting with answering questions.
HOOK:
Points: 3.7
Reason: Very passionate about their project
KNOWLEDGE:
Points: 3.8
Reason: Explain thoroughly about how their project worked
VALUE:
Points: 1.0
Reason: Useful for learning about geography
WOW FACTOR:
Reason: Cool idea and implementation
8.5/9


[Localization]: Add localizers for validation attributes

Add localizer classes for all dataannotation validation attributes.
If a dataannotation attribute does not have a resource type specified, we should call validation localizer class which in turn calls IStringLocalizer

@DamianEdwards

@DamianEdwards do we need this for beta8? Or just push to backlog?

Talked to @DamianEdwards - let's hold on to this in beta8 for now.

merged 0889b18f9523c4a1418e74a446814eea9c9084d0


azure ai lab polyu2022

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/d365community/forum/79b1327d-d925-ec11-b6e6-000d3a4f06a4.

@yiuyu2357 Thanks for raising this GitHub issue. Do you have any feedback related to any Azure documentation? If yes, please share the Azure document link and also elaborate your feedback so we can try to take action if appropriate. Thanks.

@yiuyu2357
Since we have not heard back from you, we will now proceed to close this thread.
If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Added data structure for unique constraints in create_statement.h

Modifications:

This commit adds the data structure for multi-column unique constraint in create_statement.h.

This aligns with the current foreign key constraints. It will be needed by the parser team to pass column names of each unique constraint.

The parser could also parse a multi-column unique constraint as a ColumnDefinition and put it in the columns list, just like the foreign key constraint.

Coverage increased (+0.008%) to 71.464% when pulling 75dec0c9e710c2744a6615451d862e4a0606157d on vittvolt:multi-unique into f9238733916b3f42c0ed7e947b4a1c3ce24f846e on cmu-db:master.

@mengranwo @camellyx @AngLi-Leon Can you guys check this and merge it in?

Sure, I'll start to look at it today

Merged by hand in #749


Close session terminated popup and re-enable studysession buttons once users sees the popup

Issue is circumvented because now users cannot nav to another tab with an ongoing session in the first place


Docking feature and persisted window layout

One thing with Docking is not super clear yet: what's the 'right' (as in idiomatic) way to load the layout from the *.ini file that persists user modifications to the layout, after closing and reopening the application?
Right now on startup I populate dockingParams.dockableWindows programmatically every time so it gets rid of whatever customization was made the previous time the app was run.
Also, would be nice to have a way to customize the name of the ini file (imgui.ini is not appropriate in the presence of several co-existing imgui-based apps).
I have done it before with:
ImGuiIO& io = ImGui::GetIO();
io.IniFilename = ...;

but maybe you want to expose that in the RunnerParams ?

One thing with Docking is not super clear yet: what's the 'right' (as in idiomatic) way to load the layout from the *.ini file that persists user modifications to the layout, after closing and reopening the application?

I need to work on this. This will take a couple of days

I'm on holidays this week. I'll try to,find a few minutes for this tonight.
I pushed a modif that enables to store the ini filename in params : https://github.com/pthom/hello_imgui/commit/c037e98ed76bb1d2981f6cb0d21aecaefa4b413e
Concerning the persistence of user settings, it still needs some adaptations

a58f14c44404fd4e2590af60911a8916f16d890a adds some settings to reload user settings for the docking layout.
Also see comments in the related commit of ImGui bundle:
https://github.com/pthom/imgui_bundle/commit/d9c55952f1df035abf0062ed4adfff08f6978985#diff-6b423f8761a5a9441f8502a922eb99740ce2ecf0b621a37977f09c452f931671
(the name of the ini file is derived from the window title by default)

Thanks, I tested the latest changes in my app and that's looking good.
I made a number of observations:

application window size is not restored (although it does appear in the ini file). I am not sure it's a problem on the hello_imgui side though.

in a given dock division, while the docked windows tab order is preserved, which docked windows is visible is not.
Let's take an example : if I select the Config tab docked window below and close the app, when I next reopen it, Plot is still the one that shows up on top. I do see that the ini file is changed in that case so I am thinking we might be throwing away information?
Note: In my code when I list the dockable windows, Plot is the last one, which could explain why it's always on top.

Also, if I close a window so it doesn't even appear in the tabs, it always gets restored on a restart. Example here when closing the Config window:

In that case the ini file does not appear to change though.
However, the 'Restore Default Layout' View menu entry does not restore the close tab. Only the restart does, which feels the wrong way around.

Some partial answers.

application window size is not restored (although it does appear in the ini file). I am not sure it's a problem on the hello_imgui side though.

You can restore the application window size and position with HelloImGui specific params:
    runnerParams.appWindowParams.restorePreviousGeometry = true;

They will be stored in a separate ini file (e.g. imgui_appWindow.ini), and they are managed by HelloImgui since they deal with the native platform window.

in a given dock division, while the docked windows tab order is preserved, which docked windows is visible is not.

Handling this is very much on the imgui side. I'm not sure if this situation is handled at the time.

Also, if I close a window so it doesn't even appear in the tabs, it always gets restored on a restart. Example here when closing the Config window

This could be handled from HelloImGui. I will keep this in mind.

Important note: lots of adaptation were made to store and persist several different window layouts.
See demo video on youtube for more explanations: https://www.youtube.com/watch?v=XKxmz__F4ow


P1: qatlib sample application fails with access to /dev files

Summary
Priority1 issue for GA 1.0.1: QAT workload with qatlib and executing sample cpa_sample_code with resource qat.intel.com/dc fails with access to /dev with container_device_t selinux. Only works when pod executes as privileged
Detail
QAT qatlib workload pod with cpa_sample_code sample application fails with access to dev/ files with container_device_t selinux policy i.e custom scc with access to host volume and selinux container_device_t does not work.  It only works when executed as a privileged pod.
Error below:
type=AVC msg=audit(1693324314.875:9260): avc:  denied  { read write } for  pid=3334307 comm="cpa_sample_code" name="526" dev="devtmpfs" ino=46020946 scontext=system_u:system_r:container_t:s0:c19,c27 tcontext=system_u:object_r:vfio_device_t:s0 tclass=chr_file permissive=0
Solution

workload needs access to vfio_device_t. Can add this in user-container-selinux project to support this option for current release and work with RH to merge in OCP later

We are seeing an issue even after using an workaround (setsebool container_use_devices on). It failes to allocation memory during initialization. checking with qatlib team for more details.
[error] SalCtrl_ServiceInit() - : Failed to initialise all service instances
[error] SalCtrl_ServiceEventStart() - : Private data is NULL
qaeMemInit started
ADF_UIO_PROXY err: adf_init_ring: unable to get ringbuf(v:(nil),p:(nil)) for rings in bank(0)
ADF_UIO_PROXY err: icp_adf_transCreateHandle: adf_init_ring failed
ADF_UIO_PROXY err: adf_user_subsystemInit: Failed to initialise Subservice SAL
ADF_UIO_PROXY err: adf_user_subsystemStart: Failed to start Subservice SAL
ADF_UIO_PROXY err: icp_adf_subsystemUnregister: Failed to shutdown subservice SAL.
quickassist/lookaside/access_layer/src/sample_code/performance/cpa_sample_code_main.c, main():479 Could not start sal for user space
[error] SalCtrl_AdfServicesStartedCheck() - : Sal Ctrl failed to start in given time

[error] do_userStart() - : Failed to start services

Mikko:
container runtimes by default limit capabilities given to processes. QAT apps need to add IPC_LOCK capability to deployments in order to DMA from user space.

Thanks @mythi @mregmi  IPC_LOCK with container_device_t selinux option works for qatlib sample

Thanks @mythi @mregmi  IPC_LOCK with container_device_t selinux option works for qatlib sample

Does it also mean that no new selinux configs are needed?

Does it also mean that no new selinux configs are needed?
Yes, tried without also container_device_t. IPC_LOCK was the only one missing. @mregmi please correct if I am wrong


[WHISPER] Shared post message is cut of for smaller images

If the post has small images it cuts of the info

before:

after:


Remove debug lines

Closing since this has been addressed in https://github.com/hmrc/transit-movements-trader-eis-router/pull/18


[Typescript] FormSubscription interface should have optinal keys

Hi,
As i was testing the performance optimizing features of react-final-form using react-define-form i found it a bit cumbersom to defined all of the options in the subscription props in order to optimize the rendering.
I would suggest that the FormSubscription interface either had optinal keys or maybe used Partial in order to make it easier to use :-)

I'm happy to accept a pull request to add the ? before each : in https://github.com/ForbesLindesay/define-form/blob/master/packages/define-form/src/index.d.ts#L14-L31

Alright :-)
I am a bit new to the open source community and so on :-)
Please let me know if i didnt do it the right way :-)


[Feedback] Document correction request - Avoid Parameter not defined error when using Parameterized Azure KeyVault linkedservice

Document Correction Proposal: Error when using same parameters for both Service principal ID and Secret name in the pipeline dynamic expression of the Azure KeyVault linked service
Overview
This request provides a step-by-step explanation of the below error when using Azure KeyVault linked service with parameters in another linked service - ADF.

The parameters and expression cannot be resolved for schema operations. Error Message: { "message": "ErrorCode=InvalidTemplate, ErrorMessage=The template function 'linkedService' is not defined or not valid." } Activity ID: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxxxxxxx

As per the error details, the generic reason for this error is not defining the linked service parameter but this error can arise when using same parameters for both Service principal ID and Secret name in the pipeline dynamic expression of the Azure KeyVault linked service. The documentation lacks the error details about the Azure KeyVault linked service. This request aims to offer a structured approach using the linked service parameters in Azure KeyVault linked service.
Correction Details:
This guide will allow users to:
Handle the above error when using Azure KeyVault linked service with parameters.
Steps:

Prerequisites:

Create the parameters in the Azure Keyvault linked service, parameterize the Service principal ID and Secret name in it. Use this linked service for any other linked service for the purpose of secret retrieval.

Reason for the above error:

The above error can occur for two reasons.

A generic reason for this can be not defining the linked service parameter.

When the same parameters are used for both Service principal ID and Secret name in the pipeline dynamic expression.

Steps to avoid this error:

Making sure to use different parameters for the Service principal ID and Secret name or Use string interpolation @{} when using these parameters in the linked service.

Conclusion
By following this guide, users can avoid the parameter not defined error and can use Parameterized Azure KeyVault linked service effectively.
Reference:
https://stackoverflow.com/questions/79085888/how-to-parametrize-the-keyvault-secret-field-in-a-linked-service-using-function/79086418#79086418
This Correction request aims to improve user experience and security by providing clear documentation and example scripts to avoid the error in the Parameterized Azure KeyVault linked services.

@GovindulaRakesh
Thanks for sharing the feedback. Since this channel is for driving improvements towards Docs, could you please provide the URL of the specific documentation that your feedback is applicable to? That way, we can make sure your feedback is addressed by the right team.

@GovindulaRakesh
Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.

@GovindulaRakesh
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


fix: bugs found while indexing

Changelog
New

N/A

Changed

Per our asset schema docs (https://next.carbondesignsystem.com/contributing/schema#asset-keys), tags is optional. Patterns don't have tags (yet). This PR updates the schema package to allow 0 tags on an asset and not discard it as invalid.
To accommodate that change, the web app now sets default tags to an empty array, and checks for an empty array when merging inheritable properties.
I added an en-dash to the Tags column in the assets table, when the asset has no tags. (E.g. https://next.carbondesignsystem.com/libraries/carbon-styles/latest/assets, after the new patterns get merged in.)
The default framework per the asset schema docs is design-only, but we weren't applying that default when ingesting indexed data. This PR fixes that.
I updated the space between the framework icon and name, to match the space between the status icon and name. It looked like this before:

The asset overview pages weren't showing demo links, probably from a copy/paste error? I fixed that.
The asset overview page other frameworks list was using framework keys and capitalizing the first character instead of using the framework display names. That resulted in seeing Design-Only. I fixed that.

Removed

N/A

Testing / reviewing
Main thing to test is that the inherited asset properties still work. Everything else is just visual.

I got code-smelled by SonarLint.

@jdharvey-ibm @francinelucca me adding the default design-only and [] I guess added too much complexity... is there an easy fix besides moving those out to new functions to set defaults?

I got code-smelled by SonarLint.

@jdharvey-ibm @francinelucca me adding the default design-only and [] I guess added too much complexity... is there an easy fix besides moving those out to new functions to set defaults?

Don't know if this will work but I'd try doing get(asset, ['framework'], 'design-only')  instead of asset?.framework ?? 'design-only'  (get from lodash). and same for the tags

@francinelucca get(asset, 'framework', 'design-only') worked. The best way to test that is carbon-styles library seeing all the Design only tags, that aren't in main branch: https://next.carbondesignsystem.com/libraries/carbon-styles/latest/assets

status should be 4px padding according to the status indicator pattern:

if possible we can use the CircleSolid icon for a true icon. it also has padding built into it.
Since we going use the 4-x padding for the status we should also use it for the framework icon.

@kingtraceyj we're using CircleFill. See: https://github.com/carbon-design-system/carbon-platform/blob/main/services/web-app/data/status.js
We're using the official "status" icon for this:

Where CircleSolid is in the formatting section:

The SVG looks to have the proper internal padding:

Thanks @kingtraceyj. I resolved @alisonjoseph, since 4px spacer is good per @kingtraceyj.

is there an easy fix besides moving those out to new functions to set defaults?

@mattrosno Moving some chunks of functionality out into separate functions would 100% be the best approach. Is there something preventing that from being a viable solution? The lodash function really doesn't improve the situation at all. If anything, the complexity is now even higher because the lodash function is a bit esoteric.

Moving some chunks of functionality out into separate functions would 100% be the best approach. Is there something preventing that from being a viable solution?

@jdharvey-ibm just time? I can certainly refactor into new functions if necessary. We use the get lodash functions in ~6 web-app files, so we have precedent there on using it (get a value in an object and specify a default if there are any errors getting that value)?

@jdharvey-ibm here's my refactor commit to not use Lodash get and not add SonalLint complexity: https://github.com/carbon-design-system/carbon-platform/pull/1188/commits/4050b862d697f50366baf87832b6833360452066


Add option to fall back to GET when using IE8-9

Because those browsers don't support CORS requests with cookies.
http://stackoverflow.com/questions/10941281/make-a-cors-request-in-ie9-with-cookies

This has been part of v2 for quite a while now.


[Bug]: rspack not support codemirror

System Info
System:
OS: macOS 14.4.1
CPU: (10) arm64 Apple M1 Max
Memory: 6.55 GB / 32.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.17.1 - ~/.nvm/versions/node/v18.17.1/bin/node
Yarn: 1.22.18 - /usr/local/bin/yarn
npm: 9.6.7 - ~/.nvm/versions/node/v18.17.1/bin/npm
pnpm: 8.14.0 - ~/Library/pnpm/pnpm
Browsers:
Chrome: 123.0.6312.124
Safari: 17.4.1
npmPackages:
@rspack/cli: ^0.6.1 => 0.6.2
@rspack/core: ^0.6.1 => 0.6.2
@codemirror/view: => ^6.26.3
Details
rspack dosen't work
Reproduce link
No response
Reproduce Steps

pnpm install @codemirror/view
typo new EditorView({ ... }) on code
pnpm rspack --watch

then show error in shell
[rspack] Panic occurred at runtime. Please file an issue on GitHub with the backtrace below: https://github.com/web-infra-dev/rspack/issues
[rspack] Message:  internal error: entered unreachable code
[rspack] Location: crates/rspack_plugin_javascript/src/dependency/esm/provide_dependency.rs:83

tried to reproduce with a minimal setup and a typical ts setup with builtin:swc-loader with no luck.
please provide a repro repo.
btw, does the build using ProvidePlugin?


CommandDotNet.TestTools

@bilal-fazlani  I extracted the testing tools into a separate project because I would like to use them in other projects.  It's handy to be able to test at the API level like this.  It will also help when building middleware in other projects.
The only thing missing from this is the update to our build process that would allow us to publish targetted NuGet packages based on the project-specific tags, as discussed in #154.  I'll merge this without that and we can advantage of it when we get to it.  The build process is the part I'm least familiar with so if you were able to update that in the next week or two, I would do the work of creating the other 3rd party projects in the repo.  And no worries if you don't have the time.  I'll figure it out.

I will take a look a t build process today. I will have to update deploy.sh and .travis .yml. This should not be a big, I guess.
Just to make sure I am on the same page - this is what we are looking at..
This repo will contain multiple projects.

Every time we make a commit, the build runs usually compiling everything and running tests for the entire solution.
But when there is a tag present which is of format project1_2.4.3-beta, in addition to compile and test, we also want to runt the deployment stage of the pipeline for that project.
Is this understanding correct?

Also, my understanding is that if 'project2' needs 'project1' it will have to depend on nuget package of 'project1' and can't directly have a project dependency. What do you think?

Deployment scripts updated. Now to deploy a new version we need to provide tag in new format.
PeojectName_Version

I am thinking this should allow us to bring the 2 DI repositories in this repo under the same solution... I am gonna try that today

Hi @bilal-fazlani.  I'm sorry I couldn't reply sooner.  I was away from tech for the weekend.
Yes, that's the pattern I was thinking.  And yes, we can bring the DI projects into this repo.

@bilal-fazlani Just looking at the updates you pushed to master.  This... is... FANTASTIC!.  Thank you.  I'll merge those into the beta_v3/master branch and then update the PRs


Fix grainy slope face material assignment

The thresholds don't work correctly on all parts. This can probably also be done using shader nodes. The advantage of nodes is not needing to assign materials per face, which adds to import times and complicates the code. A working node setup can easily be ported to shader code for applications if necessary. This may require adding a "normals" attribute to access non transformed normals from the shader nodes.

[x] apply angle threshold using material nodes
[ ] use an attribute for normals without transform
[ ] fix grainy slopes being applied to technic pins

The attributes should also contain a float attribute for whether a face is a stud. This can also be packed into the normals attribute. This avoids slow material lookups and assignments for each face for slope parts.


Library not loaded: @rpath/libcudart.8.0.dylib

Hi! Tensorflow is not properlly installed.
Here's the output of: python -c "import tensorflow; print(tensorflow.version)"
dyld: warning, LC_RPATH $ORIGIN/../../_solib_darwin/_U@local_Uconfig_Ucuda_S_Scuda_Ccudart___Uexternal_Slocal_Uconfig_Ucuda_Scuda_Slib in /Library/Python/2.7/site-packages/tensorflow/python/_pywrap_tensorflow.so being ignored in restricted program because it is a relative path
dyld: warning, LC_RPATH ../local_config_cuda/cuda/lib in /Library/Python/2.7/site-packages/tensorflow/python/_pywrap_tensorflow.so being ignored in restricted program because it is a relative path
dyld: warning, LC_RPATH ../local_config_cuda/cuda/extras/CUPTI/lib in /Library/Python/2.7/site-packages/tensorflow/python/_pywrap_tensorflow.so being ignored in restricted program because it is a relative path
Traceback (most recent call last):
File "", line 1, in
File "/Library/Python/2.7/site-packages/tensorflow/init.py", line 24, in
from tensorflow.python import *
File "/Library/Python/2.7/site-packages/tensorflow/python/init.py", line 60, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "/Library/Python/2.7/site-packages/tensorflow/python/init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "/Library/Python/2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 28, in
_pywrap_tensorflow = swig_import_helper()
File "/Library/Python/2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow', fp, pathname, description)
ImportError: dlopen(/Library/Python/2.7/site-packages/tensorflow/python/_pywrap_tensorflow.so, 10): Library not loaded: @rpath/libcudart.8.0.dylib
Referenced from: /Library/Python/2.7/site-packages/tensorflow/python/_pywrap_tensorflow.so
Reason: image not found
OS: OSX 10.11.6
Tensorflow pip version: tensorflow-gpu==0.12.1
My "cuda" libs under /usr/local/cuda/lib/*
/usr/local/cuda/lib/libcublas.8.0.dylib		/usr/local/cuda/lib/libcusparse.8.0.dylib	/usr/local/cuda/lib/libnppim.8.0.dylib
/usr/local/cuda/lib/libcublas.dylib		/usr/local/cuda/lib/libcusparse.dylib		/usr/local/cuda/lib/libnppim.dylib
/usr/local/cuda/lib/libcublas_device.a		/usr/local/cuda/lib/libcusparse_static.a	/usr/local/cuda/lib/libnppim_static.a
/usr/local/cuda/lib/libcublas_static.a		/usr/local/cuda/lib/libnppc.8.0.dylib		/usr/local/cuda/lib/libnppist.8.0.dylib
/usr/local/cuda/lib/libcuda.dylib		/usr/local/cuda/lib/libnppc.dylib		/usr/local/cuda/lib/libnppist.dylib
/usr/local/cuda/lib/libcudadevrt.a		/usr/local/cuda/lib/libnppc_static.a		/usr/local/cuda/lib/libnppist_static.a
/usr/local/cuda/lib/libcudart.8.0.dylib		/usr/local/cuda/lib/libnppi.8.0.dylib		/usr/local/cuda/lib/libnppisu.8.0.dylib
/usr/local/cuda/lib/libcudart.dylib		/usr/local/cuda/lib/libnppi.dylib		/usr/local/cuda/lib/libnppisu.dylib
/usr/local/cuda/lib/libcudart_static.a		/usr/local/cuda/lib/libnppi_static.a		/usr/local/cuda/lib/libnppisu_static.a
/usr/local/cuda/lib/libcudnn.5.dylib		/usr/local/cuda/lib/libnppial.8.0.dylib		/usr/local/cuda/lib/libnppitc.8.0.dylib
/usr/local/cuda/lib/libcudnn.dylib		/usr/local/cuda/lib/libnppial.dylib		/usr/local/cuda/lib/libnppitc.dylib
/usr/local/cuda/lib/libcudnn_static.a		/usr/local/cuda/lib/libnppial_static.a		/usr/local/cuda/lib/libnppitc_static.a
/usr/local/cuda/lib/libcufft.8.0.dylib		/usr/local/cuda/lib/libnppicc.8.0.dylib		/usr/local/cuda/lib/libnpps.8.0.dylib
/usr/local/cuda/lib/libcufft.dylib		/usr/local/cuda/lib/libnppicc.dylib		/usr/local/cuda/lib/libnpps.dylib
/usr/local/cuda/lib/libcufft_static.a		/usr/local/cuda/lib/libnppicc_static.a		/usr/local/cuda/lib/libnpps_static.a
/usr/local/cuda/lib/libcufftw.8.0.dylib		/usr/local/cuda/lib/libnppicom.8.0.dylib	/usr/local/cuda/lib/libnvToolsExt.1.dylib
/usr/local/cuda/lib/libcufftw.dylib		/usr/local/cuda/lib/libnppicom.dylib		/usr/local/cuda/lib/libnvToolsExt.dylib
/usr/local/cuda/lib/libcufftw_static.a		/usr/local/cuda/lib/libnppicom_static.a		/usr/local/cuda/lib/libnvblas.8.0.dylib
/usr/local/cuda/lib/libcuinj.8.0.dylib		/usr/local/cuda/lib/libnppidei.8.0.dylib	/usr/local/cuda/lib/libnvblas.dylib
/usr/local/cuda/lib/libcuinj.dylib		/usr/local/cuda/lib/libnppidei.dylib		/usr/local/cuda/lib/libnvgraph.8.0.dylib
/usr/local/cuda/lib/libculibos.a		/usr/local/cuda/lib/libnppidei_static.a		/usr/local/cuda/lib/libnvgraph.dylib
/usr/local/cuda/lib/libcurand.8.0.dylib		/usr/local/cuda/lib/libnppif.8.0.dylib		/usr/local/cuda/lib/libnvgraph_static.a
/usr/local/cuda/lib/libcurand.dylib		/usr/local/cuda/lib/libnppif.dylib		/usr/local/cuda/lib/libnvrtc-builtins.8.0.dylib
/usr/local/cuda/lib/libcurand_static.a		/usr/local/cuda/lib/libnppif_static.a		/usr/local/cuda/lib/libnvrtc-builtins.dylib
/usr/local/cuda/lib/libcusolver.8.0.dylib	/usr/local/cuda/lib/libnppig.8.0.dylib		/usr/local/cuda/lib/libnvrtc.8.0.dylib
/usr/local/cuda/lib/libcusolver.dylib		/usr/local/cuda/lib/libnppig.dylib		/usr/local/cuda/lib/libnvrtc.dylib
/usr/local/cuda/lib/libcusolver_static.a	/usr/local/cuda/lib/libnppig_static.a

This issue has been reported before, if you search issues you'll find solutions (disable SIP, set DYLD_LIBRARY_PATH, etc)


Set up doc directory for admin stuff; write instructions for new fund contracts

I rule and have completed some work on Case Manager that's ready for review!
Adding here two things:

stands up an administering directory, so we can keep that out of onboarding docs
write contract steps down

Will be doing some other doc work in advance of my bouncing, but this gets us started I think. I'm going to merge first and ask questions later.
no view changes
no issues

godmode because docs only


Create NSM CRD bundle yaml

Today NSM registers its CRDs in NSMD-k8s here: https://github.com/networkservicemesh/networkservicemesh/blob/51295d0cd3bfdbe8c3a99e05ad8974d19c6d9a3a/k8s/cmd/nsmd-k8s/main.go#L112
They are removed when NSMD-k8s is killed here:
https://github.com/networkservicemesh/networkservicemesh/blob/51295d0cd3bfdbe8c3a99e05ad8974d19c6d9a3a/k8s/cmd/nsmd-k8s/main.go#L134
This add/remove procedure is not as robust as we would like. The proposed (and established) way of registering the project CRDs would be to have a dedicated yaml bundle that describes them.
These would become part of our cluster configuration, as we do with the RBACs here:
https://github.com/networkservicemesh/networkservicemesh/blob/master/.k8s.mk#L29
We also want NSMD-k8s code to deal with CRDs add/remove to be deleted.
Note that the initializing the Packet cluster for the CI needs to be adapted for your CI to pass. It is done here:
https://github.com/networkservicemesh/networkservicemesh/blob/master/scripts/prepare-circle-integration-tests.sh#L31

I can work on this


text color is yellow instead of white on android

https://github.com/alexrainman/SegmentedControl/blob/master/SegmentedControl/SegmentedControl.FormsPlugin.Android/SegmentedControlImplementation.cs#L100
i believe this should be
v.SetTextColor(Color.FromHex("#00ffffff").ToAndroid());

the developer can explicitly provide an 8-digit number that completely specifies the Alpha, Red, Green, and Blue channels, in that order.

I just made the Android version behaves exactly like ios where unselected tabs are transparent a selected one text is transparent also.

I understand your point now. Thanks

just to add, i think you should consider making the Tint color be the Accent color by default, it looks much better than the default color being used right now (which i believe is the default radio button colors)

I am using TintColor. I dont know what you mean by accent.

https://developer.xamarin.com/guides/xamarin-forms/working-with/colors/
Color.Accent
Color.Default
these are named colors that are defined by the android OS theme.

I still dont understand what you want :). I am using TintColor in unselected tabs text color, borders and selected tan background. Exactly as iOS component.

set the Tint color to default to Accent

@julesx I agree we need an additional property for unselected background color and maybe for text color (selected normal).
For instance: http://prntscr.com/diq6y7
Looking for work around to make the unselected items background transparent and text color black.

I am using TintColor in unselected tabs text color, borders and selected tab background. Exactly as iOS component and that wont change. I am adding a property to set selected tab text color, normally to provide the hability to match container background color, as transparent doesnt works.

UISegmentedControl uses TintColor for selected segment background color, unselected segments text color and border color. I am matching that.

If you set the text colour to Color.Default it should be visible both selected and not. A property certainly wouldn't hurt though.

As i said, Android control is intent to mimic iOS counterpart and that's the way it should be.

If you set the background color (to white for instance), it colors outside the border of the control. It means if I want white buttons with blue selected button and blue borders, there is white drawn outside the button itself.
                                    <segControl:SegmentedControl x:Name="YearSplitButton"
                                        TintColor="Blue"
                                        BackgroundColor="White"
                                        SelectedTextColor="White"
                                        SelectedSegment="0" Grid.Row="0" Grid.ColumnSpan="3"  HeightRequest="30" Margin="0,10,0,10">
                                      <segControl:SegmentedControl.Children>
                                        <segControl:SegmentedControlOption Text="2016" />
                                        <segControl:SegmentedControlOption Text="2017" />
                                        <segControl:SegmentedControlOption Text="2018" />
                                      </segControl:SegmentedControl.Children>
                                    </segControl:SegmentedControl>


using fun() in breaks follow-up specs attributes formatting

As I remember I fixed some of these highlight issues in https://github.com/pgourlain/vscode_erlang. Unfortunately this repo is based on the same source that was used by vscode_erlang so the updates are now gone from vscode_erlang. I plan to create a pull request for this repo and then uplift this submodule in vscode_erlang to get back the updates.

I'm quite sure we merged the fixes from vscode_erlang into this repo and
use the grammar here for both plugins.
Also this issue might be fixed already (it's quite old).
On Tue, Jun 29, 2021, 13:17 Kornel H @.***> wrote:

As I remember I fixed some of these highlight issues in
https://github.com/pgourlain/vscode_erlang. Unfortunately this repo is
based on the same source that was used by vscode_erlang so the updates are
now gone from vscode_erlang. I plan to create a pull request for this repo
and then uplift this submodule in vscode_erlang to get back the updates.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/erlang-ls/grammar/issues/1#issuecomment-870548171,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACHLD7HW2Q7YYKHGWQ3Z2LTVG2WPANCNFSM4XFAZVJA
.

Hi, even if the idea behind https://github.com/pgourlain/vscode_erlang/issues/181 was to port the vscode_erlang grammar improvements to the shared repo, I don't think that was ever done. vscode_erlang simply switched to this repo, so it's very likely that some improvements there are still not ported here.
It should be possible to diff the grammar at https://github.com/pgourlain/vscode_erlang/commit/27cbea3dde1b96ab399c1e6903f3d9f242293bc9 and port other improvements here.

Current Erlang.plist has 2 small differences compared to the erlang.tmLanguage in pgourlain/vscode_erlang commit 6975d9a that I used to add syntax updates for OTP 23 and fix some highlighting glitches like spec lines, record definitions, etc. (see commits 921bb3e and db9ef8b.)
When my time will allow I will merge those updates here (unless somebody else is more eager to do it ;)). I still know about some additional issues in syntax highlighting but I'm not a pro of TextMate language files and Oniguruma regular expressions and unfortunately I couldn't fix those.

If I'm right this issue was fixed by pull request #4.


Max request header size?

Hi there,
We are running into an issue where we go over the max total request header size of 4K. Can you please either change it from 4K to 8K, or make it configurable?
https://github.com/uNetworking/uWebSockets/blob/f1b10ed221b36c0d8848998e3e939823f66496eb/src/HttpParser.h#L175
Change From:
const size_t MAX_FALLBACK_SIZE = 1024 * 4;

To:
const size_t MAX_FALLBACK_SIZE = 1024 * 8;

Note: there is no real RFC I think on what the proper max size is, but it seems some http servers do honor the 8K as well. See for example this old discussion: https://stackoverflow.com/questions/1097651/is-there-a-practical-http-header-length-limit
It mentions "So to conclude: To be accepted by all web servers above, a request’s request line plus header fields should not exceed 8190 Bytes.".
Thanks!

See also: https://github.com/uNetworking/uWebSockets/pull/1660

As additional information; I think more than 4K is big, but... we have no control over this size where we are in the development chain. The infrastructure adds multiple big load-balancer cookies, and we have an authorization token with a JWT user context in them. We can not make the request size smaller.

As additional information; I think more than 4K is big, but... we have no control over this size where we are in the development chain. The infrastructure adds multiple big load-balancer cookies, and we have an authorization token with a JWT user context in them. We can not make the request size smaller.

Read

Hi @gillsoftab, thanks. But my question remains. The 4k is an arbitrary limit, and we can not make our code go under the 4k. I see no reason why we could not go to the more modern value of 8K, which is supported by many servers. None of the other components in our landscape have an issue with our headers.

Hi @gillsoftab, thanks. But my question remains. The 4k is an arbitrary limit, and we can not make our code go under the 4k. I see no reason why we could not go to the more modern value of 8K, which is supported by many servers. None of the other components in our landscape have an issue with our headers.

If you ask us, we would have forced max header length to 2k or even 1k. If larger max header size is needed it's an indication the implementer(s) did something wrong or constructed a suboptimal solution. It might be a good idea for them to talk to the yellow rubber duck until progress is made.

Hi @gillsoftab ok, no doubling of the setting then. How about we make it configurable via for example an environment variable (with default to 4K)? Then you can lower it to 1K, and we can increase it to 5K. Would that be ok with you?

Hi @gillsoftab ok, no doubling of the setting then. How about we make it configurable via for example an environment variable (with default to 4K)? Then you can lower it to 1K, and we can increase it to 5K. Would that be ok with you?

Hi, I't not us, rather @uNetworkingAB who is the authority regarding such changes. We as an example use this repo as a base, then we apply security and compliance related changes in our codebase.

@gillsoftab at the moment, we indeed do build the code ourselves before we include it in the final service, with the buffer increase, but it would be easier if we can skip doing search/replaces, and having it as just a config option.
@uNetworkingAB would you be OK with making the max header setting configurable via for example an environment variable? (with default to 4K). I don't mind having a go at a PR, but if it won't be considered an option, it's not worth the time.

A quick test in a separate hello world.cpp file indicates this line would work fine:
const size_t MAX_FALLBACK_SIZE = getenv("MAX_HEADER_LENGTH") ? atoi(getenv("MAX_HEADER_LENGTH")) : 1024 *4;

But as I'm not a CPP programmer, you might have preferences/ways to make it look nicer. Or maybe you already have standard functions in the uWebSockets to solve this differently.

Thanks for raising it @gillsoftab , we have the same problem and getting more and more serious, affecting production.
@uNetworkingAB could the max header setting be configurable for both MAX_FALLBACK_SIZE and MAX_HEADERS?
We are using a 3rd party framework graphql-mesh that is using your webserver. As our consumer base grows they starting facing errors due to the current header limits, both randomly 500s or 431s in production and e.g. when they what to add a new feature that require aditional headers, they figure out that they cannot add more headers. It makes the situation worse that

The cloud infrastructure also adds additional headers, without having much influence on it
The client applications are also not in control of all headers, the browsers are sending many of them automatically
Some headers have inheretly big value, like JWT
At bigger companies there can be different teams that they need to add different headers for their on purpose to the same request.
You can imagine that in today's cloud based web apps, considering the end-to-end flow, it is easy to reach the 50 header and 4k size limit, and it is difficult, if not impossible, to solve it by keeping the header limits within these values, since it would require controlling and limiting the functionality of the different parts.

SInce we use uWebSockets <- uWebSockets.js <- graphql-mesh <- our app, it would be also much more difficult to have our own version of all these dependencies, rather than e.g. having an environment variable for each of these limits as configuration option.

@janos-mands I have altered my PR to add your wish; https://github.com/uNetworking/uWebSockets/pull/1661
Now you only need to get @uNetworkingAB to merge it some time...
We also needed the increase of the max values for graphql-mesh in our enterprise cloud environment. However it seemed the graphql-mesh we were using (possibly the same one? not sure) had multiple issues, among which some memory leak. We moved to try just stitching, instead of using mesh, which seems much more stable.

The PR doesn't compile

Ah, yes, sorry ;-) I didn't have my coffee yet perhaps. This would require a dynamic allocation. I'll have another look at it.

@atkaper thanks for adding MAX_HEADERS to your PR. I can see that your primary concern of MAX_FALLBACK_SIZE has been addressed and so the PR is closed. I'll propose the same for MAX_HEADERS, but wondering if you guys do not get errors when the max header count is exceeded or it is just never the case for you. Worth noting that when the max size is exceeded, we are getting 431 back, but when the max count is exceeded, we are getting 500 from our cloud service, that is, based on our investigation by checking the process directly, caused by socket hung up/ timeout.

@uNetworkingAB thanks for making MAX_FALLBACK_SIZE configurable. Really appreciate it. Wondering if you could agree with the same kind of change for MAX_HEADERS as well. That would be a big help for us, since after that we could use uWebSocket as is, and it could avoid a chain of custom builds for us. I am happy to make a separate issue and PR for that.

Wondering if [...] the same kind of change for MAX_HEADERS as well.

No, since a compile-time constant used to size an array.
Making MAX_FALLBACK_SIZE configurable at runtime via getenv() cannot be done in this case,
w/o changing the allocation of the headers array to be dynamic (i.e. heap allocated), which affect performance,
and this make is less likely for @uNetworkingAB to accept it. Just a guess though. My $0.02. --DD

@atkaper thanks for adding MAX_HEADERS to your PR. I can see that your primary concern of MAX_FALLBACK_SIZE has been addressed and so the PR is closed. I'll propose the same for MAX_HEADERS, but wondering if you guys do not get errors when the max header count is exceeded or it is just never the case for you. Worth noting that when the max size is exceeded, we are getting 431 back, but when the max count is exceeded, we are getting 500 from our cloud service, that is, based on our investigation by checking the process directly, caused by socket hung up/ timeout.

@janos-mands The same solution is not possible, as the header array size is a compile time thing/constant. Simplest is to just double the max in the source, and rebuild. The best solution would be to get the code changed to do dynamic header allocation (without extra limit apart from the existing total header bytes). I might have a look at it some day (no promises), but it's probably much faster if @uNetworkingAB could do it as my C has gotten a bit rusty (I moved to java around 2001) ;-)
As for the question if we ran into the max # of headers; I'm actually not sure, it could quite likely indeed have been the case without us knowing this. A lot of requests worked fine, but a part of them did fail. The number of request headers differs in our case depending on if end-users are anonymous, returning recognized users, or logged on users, and it depends on what browsers or proxies or providers they use. Perhaps we indeed went over the 50 headers in some cases. And our infra indeed adds quite some headers as well. So probably yes, this might have been one of our issues also.
What we did for building this graphql-mesh, is use a multi stage "Dockerfile", like this (all in one file):
# Temporary setup, we do a full checkout+build of the uWebSockets.js server project, to increase buffer size.
FROM ubuntu as uWebSocketsJsBuilder

RUN apt update && apt install -y  git build-essential clang zlib1g-dev libuv1-dev libssl-dev curl cmake golang
WORKDIR /app
RUN git clone --recursive https://github.com/uNetworking/uWebSockets.js.git --depth=1 --shallow-submodules
WORKDIR /app/uWebSockets.js
RUN sed -i 's/MAX_FALLBACK_SIZE = .*/MAX_FALLBACK_SIZE = 1024 * 8;/g' uWebSockets/src/HttpParser.h && grep 'MAX_FALLBACK_SIZE = 1024 \* 8;' uWebSockets/src/HttpParser.h
RUN make

# Normal build of mesh container
FROM node:18

WORKDIR /app
COPY package.json /app
RUN yarn install
COPY .meshrc.ts forward-headers-plugin.ts version-resolver.ts /app

# Overwrite uWebSockets.js server with results of uWebSocketsJsBuilder
RUN rm -rf /app/node_modules/uWebSockets.js/uws*
COPY --from=uWebSocketsJsBuilder /app/uWebSockets.js/dist/uws* /app/node_modules/uWebSockets.js/
RUN ls -la /app/node_modules/uWebSockets.js/uws*

CMD ["yarn", "start"]

Of course you could do something similar to override the max headers setting. Just add an SED line to above example.
The use of the multi stage docker file allowed us to do the build in a single step in our build pipelines (in github action). Although it was much slower than needed due to doing the extra build in stage 1.

Thanks @ddevienne, @atkaper - you guys are right, I have missed that point about static -> dynamic array allocation. (You can tell I am not a C++ programmer, for a long time.) I understand the additional, possible performance cost of making it dynamic, but unfortunately the original problem still remains. I assume we are not the only one who can reach this limit now, and given the different trends in IT, I belive we will not be last either, so I'll make a separate issue for it (instead of highjacking this issue), to make it more visible, and debate it further there , whatever will be the outcome.
@atkaper  Thanks a lot for the build example, it could help us to make it working in the meantime, faster.

Max header count issue has been moved to https://github.com/uNetworking/uWebSockets/issues/1689 and max request header size (in bytes) has been fixed in master. You can now set environment variable UWS_HTTP_MAX_HEADERS_SIZE to override the default max of 4096 bytes.
Closing this issue.


How to adjust row spacing

You can specify line spacing with CSS like so:
.paragraph-text {
    -fx-line-spacing: 15;
}

I tried to do that, but it didn't seem to work. Did I set it up wrong
`package test;
import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.layout.AnchorPane;
import javafx.stage.Stage;
import org.fxmisc.flowless.VirtualizedScrollPane;
import org.fxmisc.richtext.InlineCssTextArea;
public class consoleText1 extends Application {
@Override
public void start(Stage primaryStage) throws Exception {
AnchorPane anchorPane = new AnchorPane();
InlineCssTextArea textArea = new InlineCssTextArea();
VirtualizedScrollPane virtualizedScrollPane=new VirtualizedScrollPane(textArea);
anchorPane.setTopAnchor(virtualizedScrollPane, 0.0);
anchorPane.setBottomAnchor(virtualizedScrollPane, 0.0);
anchorPane.setLeftAnchor(virtualizedScrollPane, 0.0);
anchorPane.setRightAnchor(virtualizedScrollPane, 0.0);
    textArea.setStyle(" -fx-line-spacing: 15");

    anchorPane.getChildren().add(virtualizedScrollPane);
    Scene scene = new Scene(anchorPane, 500, 600);
    primaryStage.setScene(scene);
    primaryStage.setTitle("test");
    primaryStage.show();
}

public static void main(String[] args) {
    launch(args);
}

}
`

Unfortunately it doesn't work with setStyle,  it has to be added through a stylesheet like:
scene.getStylesheets().add( "path-to/my-style-sheet.css" );

wo,It's a bit awkward, or it's a failure. No matter how I adjust, the spacing between each line is still so large
package test;
import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.layout.AnchorPane;
import javafx.stage.Stage;
import org.fxmisc.flowless.VirtualizedScrollPane;
import org.fxmisc.richtext.InlineCssTextArea;
public class consoleText1 extends Application {
@Override
public void start(Stage primaryStage) throws Exception {
AnchorPane anchorPane = new AnchorPane();
InlineCssTextArea textArea = new InlineCssTextArea();
VirtualizedScrollPane virtualizedScrollPane=new VirtualizedScrollPane(textArea);
anchorPane.setTopAnchor(virtualizedScrollPane, 0.0);
anchorPane.setBottomAnchor(virtualizedScrollPane, 0.0);
anchorPane.setLeftAnchor(virtualizedScrollPane, 0.0);
anchorPane.setRightAnchor(virtualizedScrollPane, 0.0);
//        textArea.setStyle(" -fx-line-spacing: 15");
    anchorPane.getChildren().add(virtualizedScrollPane);
    Scene scene = new Scene(anchorPane, 500, 600);
    scene.getStylesheets().add( "path-to/my-style-sheet.css" );
    primaryStage.setScene(scene);
    primaryStage.setTitle("test");
    primaryStage.show();
}

public static void main(String[] args) {
    launch(args);
}

}
my-style-sheet.css

.paragraph-text {
-fx-line-spacing: 100;
}

It only applies to lines inside a paragraph that wraps over multiple lines and not between paragraphs that only have one line. In RichTextFX a paragraph is any text that ends with enter / new line.
In the image you provided it looks like each line ends with enter without wrapping and so no line-spacing is seen.
I suppose this is an oversight since if you have for example a numbered or bulleted list of single lines then one would expect the list to have the same spacing as a multi-lined wrapped paragraph. I'll have a look and see if I can change it.

You can also try the following CSS to achieve what you want:
.paragraph-box {
    -fx-padding: 0 0 15 0;
}


Fix incorrect metric conversions for rain and pressure

Describe what the PR does:
This PR fixes an incorrect conversion of rain and pressure data when using metric units.
Does this fix a specific issue?
Fixes https://github.com/bachya/ecowitt2mqtt/issues/43
Checklist:

[ ] Update README.md with any new documentation.
[ ] Add yourself to AUTHORS.md.

@oztourer Can you test this PR for me and verify that it fixes #43?

@oztourer Can you test this PR for me and verify that it fixes #43?

@oztourer Can you test this PR for me and verify that it fixes #43?

The new code is an exact copy of my operational code so no need for me to test separately, but I can confirm that the changes are correct.

@oztourer Can you test this PR for me and verify that it fixes #43?

The new code is an exact copy of my operational code so no need for me to test separately, but I can confirm that the changes are correct.


Create a Reader abstraction for Morphir similar to a DataReader.

Is your feature request related to a problem? Please describe.
Create a Reader abstraction for Morphir similar to a DataReader.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Is this for Morphir authoring or backend processing?


axtls upgrade to svn r251 (1.5.3+)

This is an upgrade of axtls to svn revision r251, including various fixes and support of sha256 (not hmac_sha256), sha384 and sh512. Fixes #9.

:+1: awesome, thank you for the pull request. Will review and merge on Monday.

Ok, here is a small guide how to track further updates of axtls-svn by your own in your local git clone:
git svn init svn://svn.code.sf.net/p/axtls/code --trunk=trunk --tags=tags --branches=branches --prefix=axtls-svn/
git svn fetch
git checkout -b upstream axtls-svn/trunk


How to use when multiple instances of vault need unseal (HA mode)

Hi
I have 3 instances of vault running.
currently I'm running vault unsealer against the service endpoint, according to the vault docs, the unseal needs to happen on each instance.
It is important to note that only unsealed servers act as a standby. If a server is still in the sealed state, then it cannot act as a standby as it would be unable to serve any requests should the active server fail.

I prefer not o run it as another container within the vault pod.
I can either use a service account and query the endpoints:
kubectl get endpoints vault \         -o jsonpath="{.subsets[*].addresses[*].ip}" | xargs
or put the unseasler as a container in the vault pod.
What do you think is the recommended best approach?
is there a way to achieve the unseal per vault pod?

I also have this problem. Today I set up 3 instances of vault-unsealer, each pointed at a single vault node. Ideally I would like to pass a list of targets into the vault-unsealer instead of a single VAULT_ADDR environment variable, and the single process could monitor all 3 endpoints.

you can create a headless service (which will return all the pods IP's) and use that
I ended up attaching the unsealer into vault pod as a sidecar container - still need to only run on startup which i didn't get to that part (its currently checking constantly) and the unsealer is talking to vault on localhost.