STYLE: Reformat Python code with black

Implemented with
find . -name "*.py" -print0 | xargs -0 black

Manually wrap comment text to 88 characters too.

... I suggest to install follow up on the work initiated ...

Is it reasonable to defer this to a different pull request?

I don't like that it changed 2-space indentation to 4-space. Is that something you can configure?

I don't like that it uses a line length of 88 instead of something like 120 that I find much more reasonable.  Yes, these things are configurable.  However, the argument usually goes that we should accept the black defaults so that it looks like other people's code.

Yeah I agree also 88 seems really small.
Line length 120 sounds good!

Considering there is already infrastructure to address styling, I suggest to update it. Looking at the changes we introduced in upstream Slicer (e.g see here, here and here) may be helpful.
Generally speaking, once styling change are integrated, the file .git-blame-ignore-revs should be updated to ensure git blame (either from command line or from within GitHub UI) ignore these and allows to follow the history independently of style changes.
That said, in the future, adopting black is definitely possible. I would recommend to gradually converge to it instead of applying this huge set of changes.
For example, like we have done in Slicer, updating the .flake8 configuration removing exception and addressing them would be a good first step.

That said, in the future, adopting black is definitely possible. I would recommend to gradually converge to it instead of applying this huge set of changes.
For example, like we have done in Slicer, updating the .flake8 configuration removing exception and addressing them would be a good first step.

I think I agree with this.
@Leengit Instead of the one big static style update, would you want to look into removing some of the flake8 exceptions for the aspects of style that could be improved? Or if this is becoming too much trouble to be worth it for you then feel free to close the PR? I just think that it's much better when sweeping style changes occur via the linting configuration

Generally speaking, once styling change are integrated, the file .git-blame-ignore-revs should be updated to ensure git blame (either from command line or from within GitHub UI) ignore these and allows to follow the history independently of style changes.

ah thank you I was wondering how the blame ignoring worked!

was wondering how the blame ignoring worked

And to illustrate, see https://github.com/Slicer/Slicer/pull/6306#issuecomment-1092426901

I prefer automated approaches such as black.  I find that easier to implement than addressing flake8 exceptions and that it provides more consistency in the code.  Would it help if we go for black with a line length of 120 and an indent of 2?  We can then proceed to install hooks to automate formatting on every commit, so that no one need think about this much going forward -- at least in theory; I haven't actually done this before, though ITK does something similar with clang-format of C++ code.)

Would it help if we go for black with a line length of 120 and an indent of 2?

I'm okay with it as a one time adjustment
I don't mind running flake8 corrections locally and it is kind of nice to have full control over what gets commited, so I think I ultimately prefer the approach of managing code style enforcement via flake8 config, with no automated formatting

I have just pushed a version of the code formatted with find . -name "*.py" -print0 | xargs -0 black -l 120 -C.  That sets line length to 120 instead of the default of 88.  Also -C says to put each list on one line if it fits, even when the last list entry has a trailing comma (which is usually the signal to black for "don't collapse to one line").  I could not find a way to override the indentation from 4 spaces to 2 spaces.  Thoughts ... merge, change, or close?

Thanks, I guess I can live with 4 spaces :slightly_smiling_face:
Good to merge :+1:

Transitioning to 4 spaces makes sense, could this be done in a first PR ? Then move forward with the other updates.

See here to find out which comment to run:
https://github.com/Slicer/Slicer/commit/9390be6bf76e048e6fd384d8bfd607eba2857b5a

With this latest force-push I hacked the black source code so that it uses 2-space indents.  I also squashed this series of commits into a single commit.  Once Pull Request #52 is merged, I can update this pull request again -- or submit a new pull request if this one has already been merged.

I just saw @jcfr comment about transitioning to 4-space indents first.  Unfortunately, black won't do just that, so we'd have to manage that some other way.  The philosophy about black is to be opinionated and not flexible.  That gets uniformity in the code, but perhaps not the same format as we have aimed for in the past.  black is mighty convenient, which is why I favor it, but if we go with an alternative then we can simply close this pull request without merging.

Since the current infrastructure used flake8 et alike, changes should be done considering the existing infrastructure. For now, I suggest to close this pull request and move forward with #52, the proposed changes are aligned with black style and a step in the "right" direction.

Once Pull Request https://github.com/KitwareMedical/lungair-desktop-application/pull/52 is merged, I can update this pull request again -- or submit a new pull request if this one has already been merged.

For now, this pull request could be closed, and eventually re-opened afterward.


adjust clueFilter minion buttons

Description:
Changed so that it creates up to 3 buttons for the highest tiered clue scrolls that are owned.
Changes:
Adjusted the filter for ClueTiers on the minion command.
Other checks:

[X] I have tested all my changes thoroughly.

resolves #3953


Non-root file access oddity

Hello, using the latest goofys and catfs versions (as of today, that is)
I think this will be clearer than a lot of text:
# root @ nginx-s3 in ~ [5:13:13]
$ cat /etc/fuse.conf
# mount_max = 1000
# user_allow_other
user_allow_other

# root @ nginx-s3 in ~ [5:13:16]
$ cat /etc/fstab | grep goofys
goofys#s3-bucket-shared-seed /var/run/goofys fuse _netdev,allow_other,--file-mode=0777,--dir-mode=0777,--endpoint=https://s3.fr-par.scw.cloud/,--cache=--free:20%:/tmp/goofys 0 0

# root @ nginx-s3 in ~ [5:13:22]
$ mount -v | grep goofys
s3-bucket-shared-seed on /run/goofys type fuse (rw,relatime,user_id=0,group_id=0,default_permissions,allow_other)
/dev/fuse on /run/goofys type fuse (rw,nosuid,nodev,relatime,user_id=0,group_id=0,default_permissions)

# root @ nginx-s3 in ~ [5:13:29]
$ getfacl /run/goofys
getfacl: Removing leading '/' from absolute path names
# file: run/goofys
# owner: root
# group: root
user::rwx
group::rwx
other::rwx

# root @ nginx-s3 in ~ [5:13:35]
$ su tristan

# tristan @ nginx-s3 in /root [5:13:38]
$ ls /run/goofys
ls: cannot access '/run/goofys': Permission denied

Point being that despite allow_other being configured correctly (afaik, only /etc/fuse.conf and fstab option are needed?) and using file/dir mode as 0777, non-root users get permission denied.
Was using S3FS before and it worked with the umask option, if that helps.
Now it seems goofys is doing the right thing as post-mount the mountpoint has the correct ACL (we do see others::rwx), so I'm wondering what could be the issue here...
It might also be an artifact of catfs for all I know since it is being used here, but I wouldn't expect the permissions of the underlying cache to impact the parent mounting...
Would you have any idea of what's going on?
Thank you very much

Seems like it might be related to catfs being mounted after goofys, but without allow_other...
With fstab line:
goofys# [... irrelevant] allow_other,--cache="allow_other,--free:20%:/tmp/goofys" 0 0
Tried no quotes, double quotes, single quotes, using escape sequence for , (\054 as fstab uses ascii octal escapes) to no avail
Would alwyas yield back a flag provided but not defined: -free:20%:/tmp/goofys
Might be wrong though, but when mounted (with allow_other in goofys params, but not catfs') I do see the following mounts:
s3-bucket-shared-seed on /run/goofys type fuse (rw,relatime,user_id=0,group_id=0,default_permissions,allow_other)
/dev/fuse on /run/goofys type fuse (rw,nosuid,nodev,relatime,user_id=0,group_id=0,default_permissions)

where clearly the latter one (so catfs), which is mounted on the very same mountpoint, lacks the allow_other setting

Worked around it after some source code inspection:
goofys#s3-bucket-shared-seed /var/run/goofys fuse _netdev,allow_other,--file-mode=0777,--dir-mode=0777,--endpoint=https://s3.fr-par.scw.cloud/,--cache=-o:allow_other:--free:20%:/tmp/goofys 0 0

# root @ nginx-s3 in ~ [2:01:36]
$ su tristan

# tristan @ nginx-s3 in /root [2:01:43]
$ ls /tmp/goofys

will add a PR to README as that wasn't quite trivial :)

instead of doc update, what if goofys automatically pass along -o allow_other (and --uid, --gid) to catfs?

That sounds like a good alternative indeed


Where to put section files

Environment

macOS
site_prism 2.15 / capybara 3.0.3
rspec
chromedriver 2.40.565386

Expected Behavior
Section classes seeing each other and be seen by Page instances
Actual Behavior
I'm not sure where to put the section definitions. All my page definitions are in spec/support/pages which is autoloaded by rails_helper. However, adding sections directory there and putting my section definitions there results in "unitialized constant " errors thrown by pages.
I tried taking the prism code out of support directory (now they are in spec/page_object/pages and spec/page_object/sections) to take control of require order and it seemed to work locally but, on my CI setup, im getting one exception per each spec file (even if that file is not using the page_prism) complaining that i'm trying to use, say, SecondSection in FirstSection (the error is thrown in FirstSection and says that SecondSection is an unitialized constant).
The reason for this is wrong order of require of the section classes so they don't know about each other but i'm not really sure on how to improve that nor why the hell they are initializing the section objects on require - it shouldn't behave like that afaik.
To require my site_prism stuff, i'm using following code in rails_helper.rb
require 'site_prism'
Dir[
  Rails.root.join("spec/page_objects/sections/**/*.rb"),
  Rails.root.join("spec/page_objects/**/*.rb")
].each {|f| require Rails.root.join(f)}

Steps to reproduce
It's mostly about running it on test environment, you can see an example failing spec here: https://gitlab.com/mbajur/prismo/-/jobs/83171464

From initial triage this seems to be a user-error. The way ruby works is that you need to define things backwards. I.e. most granular classes (Small Sections), then less granular classes (Larger Sections / Pages)
i.e.
require_relative 'smallest_section'
require_relative 'medium_section'
require_relative 'my_page'

For
class MyPage < SitePrism::Page
  section :medium_section, MediumSection, 'med-sect'
end

class MediumSection < SitePrism::Section
  section :smallest_section, SmallestSection, 'small-sect'
end

class SmallestSection < SitePrism::Section; end

if you can upload a sample script that fails to execute then please do so. I've only had a quick look at the job but it doesn't show me your source code. If you can produce us an SSCCE then we'll triage further.

yes, that's exactly what i'm trying to do - to load it "granularly" but without specifying each require by hand. I would love to auto-load all the sections just like all the pages are auto-loaded.
Does using the require snipped i used:
Dir[
  Rails.root.join("spec/page_objects/sections/**/*.rb"),
  Rails.root.join("spec/page_objects/**/*.rb")
].each {|f| require Rails.root.join(f)}

should theoretically work out of the box and prism sections should don't care about the order or there are some internals in site_prism i don't know about ? If so, this issue can be safely closed as it seems to be an issue with my setup rather than with site_prism

Or if you want a really "quick" way.
A -> Have a folder of all of your bottom level sections (i.e. sections which don't call other sections).
B -> Have a folder of all of your other sections (That only call sections defined in A)
C -> Have a folder of all of your pages (Already done)

Thank you! I'm definitely gonna use one of these approaches :)


added awards page to menu

added awards page under "conference"

Thank you very much, @annakijas1


Unable to write to Workspace Settings because no workspace is opened. Please open a workspace first and try again.

Repro steps:

Action: azureFunctions.deploy
Error type: Error
Error Message: Unable to write to Workspace Settings because no workspace is opened. Please open a workspace first and try again.
Version: 0.10.0
OS: win32

You need to add your function app folder to workspace first.

Offhand I'm not sure what settings we're trying to write while deploying. As @JohnReeson mentioned you can just open the folder as a workaround. However, we can investigate getting rid of this error completely

@EricJizbaMSFT  It seems to write azure-functions-related settings(in ~/.vscode/settings.json), same as when we initialize a new function project. Looks like any function related operation requires this workspace settings and we may get the prompt when no workspace is opened(no settings available).


Page update required

Hi there,
This page needs updating since Gen2 went GA. Finding the CapacityID was a bit of guesswork! Thanks, Angus

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bcddef48-ab1d-cdd2-900a-9b9f3500a06f
Version Independent ID: 42ce0182-3006-4271-7981-a9f8a984ca53
Content: Install the Premium Gen2 metrics app - Power BI
Content Source: powerbi-docs/admin/service-premium-install-gen2-app.md
Service: powerbi
Sub-service: powerbi-premium
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Hi @Withnail99,
We're in the process of revamping our documents, and our table of contents. However, this might take some time :-)
Can please describe what you struggled with and why? So that I can update the article accordingly.
Thanks,
Kesem

Hi Kesem
Well, the image is of the preview and it has now gone GA. So it needs revamping because the process is somewhat different. If you run through it yourself, then you’ll find that parts of the document need re-writing and new images taken.
Kind regards
Angus
From: Kesem Sharabi @.>
Sent: 01 December 2021 14:01
To: MicrosoftDocs/powerbi-docs @.>
Cc: Angus Fookes @.>; Mention @.>
Subject: Re: [MicrosoftDocs/powerbi-docs] Page update required (Issue #3451)
Hi @Withnail99https://github.com/Withnail99,
We're in the process of revamping our documents, and our table of contents. However, this might take some time :-)
Can please describe what you struggled with and why? So that I can update the article accordingly.
Thanks,
Kesem
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/powerbi-docs/issues/3451#issuecomment-983668693, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOJ3HC3XCFL5HEWYWMSM2ELUOYTDPANCNFSM5JDXPCZA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi @OxfordPowerBI,
I updated the Install the Gen2 metrics app article.
Hope this helps,
Kesem


Some Atari Jaguar games are unplayable due to button mapping issues

I have tested all 50 retail cartridge games as well as 4 CD games. The following games have a button mapping issue:

Atari Karts
On PS4/5 controller Pause = A, L1 = D-Pad R, D-Pad L = 4, R Thumb = D-Pad L. D-pad R, U, D and other buttons not working
Breakout 2000
Same as Atari Karts
Missile Command 3D
Same as Atari Karts
Myst
L1 acts like D-Pad Left and R3 acts like D-Right

My test was with a PS4 and PS5 controller. I've tried to remap these games in the advanced web config but it didn't seem to have any effect.

Breakout 2000 issue look different than Atari Kart & Missile Command.
If you simply press right and B to start the game right away everything look fine to me.
If you press down in the menu, then the game goes in an infinite loop.
v1.2.3_wip1.zip
Try this build, it will fix the infinite loop in breakout 2000, other than that, this game look completely fine to me.
Will look into the other next.

I can't really fix the other 3 games, the games is simply reading back the JOYSTICK/JOYPAD register too quickly after writing to it. TTL chip can handle it since they probably transition output in the 10ns, Using pretty much all the trick I could think off the ESP32 can only do between 300ns and 400ns transition.

This make those game read the previous buttons bank set. This look quite stable.
As a workaround, I a made a preset called "Jaguar cycle shift" (see https://blueretro.io/blueretro_presets.html) and using that both myst and atari kart are playable properly. You can't use that in the working games however. To help navigate the GameDrive I mapped the non-shifted d-pad on the left joystick.
Missile Command is still a bit broken, the main menu up/down dont work at all. The timing of bank transition is quite brutal at 1us (vs 8us for Atari Kart & Myst), but once in gameplay it kind of work?

An alternative workaround for GameDrive people would be to make rom hack of the games with some NOPs after setting JOYSTICK register.


Make pause button show play

The default behavior now makes the model not rotate, but the play/pause button still shows pause instead of play. It should be changed to be set to play when shdr starts

The icon shows play on start now but still requires two clicks to actually change the behavior of the model.

Ditto. Did anyone every try to change it?


🐻❄bump prom operator version to latest 🐻❄

looks like the latest prometheus operator now in catalog. bump to support this version

LGTM


[EventRequest] Codeunit 90 “Purch.-Post”, PostItemLine

Please add a new publisher event OnBeforeInsertDropShptBuffer in Codeunit 90 on function PostItemLine and call it as shown in the image below:

Please add the corresponding Integration Event In the bottom of the codeunit
LOCAL [IntegrationEvent] OnBeforeInsertDropShptBuffer(VAR DropShptPostBuffer : Record "Drop Shpt. Post. Buffer";PurchLine : Record "Purchase Line")

This event already exist
OnBeforeTempDropShptPostBufferInsert(TempDropShptPostBuffer, PurchLine);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Question About Command Handler Return.

Hi,
I'm just new in design patter of CQRS.
Now while reading articles about this pattern.
It says that the command handler must not return data.
In this case how i will pass the generate data from command handler to controller.
I found articles that they use GUID that was generated before it will pass to command.
But in my case since I'm using Auto-increment in database, how i will know the generated ID.
Thanks.

@cydrickn Tnx for taking the time to write your comment. But it is indeed the case that you can't return anything inside the handler. The reason behind this is that the command bus should be able to handle the command on a later moment. If you think about in that case it wil not happen at that direct point in time and you will not be able to retrieve the Auto-increment as it is not created yet.
I know it is a mind shift and to be honest I had to work on it for a while to wrap my mind around the concept. If you really need a identification of what you are about to create the only way is to start using a UUID and send that with the command to the handler. In our case we implemented both a Auto-increment and we are using a UUID for cases where we really need to have that reference.

https://github.com/SimpleBus/MessageBus/issues/47

Thanks for the reply @cmodijk,
I will start using UUID now so that i can easily reference my entity.


microservices inside - not up to date

A lot has happened around the APIs over the last few weeks.
Unfortunately the project is not up to date with the latest Microservices.
Please update the project accordingly. (i miss, for example:  "config-thing-sap")
I don't like to make changes in the source code of you
Thanks

Hi @DanielPretzsch ,
You're right, the micro service list is not up to date. But you can set the second parameter of the method to false, which disables the micro service check.
setBaseUrl("config-thing-sap", false)
This should do it.
Regards
Jan


Is there a mechanism or pattern for Block Style Literal Strings

I have a string content that is muliline and currently I can use pydantic-yaml with ruamel to dump it as yaml.
It uses the quoted string format.
Is there an option to use the block style ?
Or can suggest some options to consider.
https://yaml-multiline.info

Hi, improving the flexibility of dumping YAML is something that's been on my wanted list for a while. It might wait until Pydantic 2.0 releases and I'll need to totally rewrite this... I'll probably base it around ruamel's or even (shudders) do the YAML part myself.

Hi @Bengreen, in terms of user experience - would you prefer to do this configuration within the model class itself (using Pydantic-like config class) or within the "dumping engine" as a global configuration? How did you use it in the case above?

Currently, limited support for this is now available within to_yaml_str and to_yaml_file using dumping keywords (since 1.1.0), or with a custom YAML dumper instance. For example, you could try configuring text longer than some length to use a block style. You'll have to go through the internals of ruamel.yaml though.


Limit Line Length and Destroy on Collide

Limit line length so user is unable to draw without limit. Also add functionality to destroy line when ball collides with it.

Done.


ci(android): update java requirement for cordova-android@11

Platforms affected
CI: Android
Motivation and Context

Paramedic which fetches and uses the latest releases of the platform is now using<EMAIL_ADDRESS>which requires Java 11.
Description

Update the GitHub Action workflow

Use latest version of the actions/setup-java GitHub Action plugin
Use Java 11
Use the Java Distro temurin which is the replacement of adopt

Testing

See Action

Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[x] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[ ] I've updated the documentation if necessary

The failing tests are unrelated to the change.

Error: There was a conflict trying to modify attributes with  in plugin cordova-plugin-paramedic.

The change of this PR's is safe to merge.


ReactNative WebView content height calculation working fine on iOS but on Android it gives more height than the actual content

I am using web view to load my rich text editor html content in react native. The problem is I need to render multiple webviews one after the other vertically. And webview need a explicit height to load content. So, after searching for some time I found https://medium.com/@fxwio/set-webview-height-to-the-height-of-html-content-react-native-2d696128c181 and it helped me to identify height based on content. The content height on iOS is working fine. But on Android if the actual content height is "296" it calculates as "2785". Please help me out of this.
Environment:

OS: Android
OS version: Pie
Expo SDK 37
react-native-webview version: 9.2.1

the same problem

<meta name="viewport" content="width=device-width">

What do we do with this?

Where should the meta tag be added to the WebView?

@sreekanth-14 @lkwjohn @buffge Might want to try this: https://formidable-webview.github.io/webshell/docs/autoheight/


⚠ Bofrost.fr has degraded performance

In 541096f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 825 ms

Resolved: Bofrost.fr performance has improved in 9ac4b82 after 9 minutes.


Custom module icon only used in VB

Problem Description
Adding a custom svg for a module icon works in the Visual Builder but the BB still shows the default font icon, is there a way to add it to both?
Steps To Reproduce
        $this->icon_path        =  plugin_dir_path( __FILE__ ) . 'icon.svg';

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[x] Detailed description
[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

Since I want my modules to pop out a bit more so users can immediately distinguish them from core modules, I use my own CSS rather than just relying on the icon path. For that you need two CSS locations. For the backend builder, you put it in your admin.css:
.et-pb-all-modules .et_pb_jt_counter:before,
.et_pb_saved_layouts_list .et_pb_jt_counter:before {
  content: "\58";
}

.et-pb-all-modules .dss_post_meta_parent:before,
.et_pb_saved_layouts_list .dss_post_meta_parent:before {
  content: '\70' !important;
  font-family: 'ETmodules';
}

.et-pb-all-modules .et_pb_jt_counter,
.et_pb_saved_layouts_list .et_pb_jt_counter {
  background-color: #3949ab;
  color: white;
}

.et-pb-all-modules .et_pb_jt_counter:after,
.et_pb_saved_layouts_list .et_pb_jt_counter:after {
  content: url('../divi-sensei-64.png');
  position: absolute;
  top: 6px;
  right: 0px;
}

For the visual builder, you put the CSS inside the modules own CSS file (as it is loaded on the frontend, not the admin area):
    .et-fb-all-modules .dss_post_title:before {
        content: '\61' !important;
    }

    .et-fb-all-modules .dss_post_title {
        background-color: #3949ab;
        color: white;
        overflow: visible;
    }

    .et-fb-all-modules .dss_post_title:after {
        content: "";
        background-size: 40px 40px;
        background-image: url('./../../../admin/assets/divi-sensei-64.png');
        background-repeat: no-repeat;
        position: absolute;
        top: -10px;
        left: -15px;
        width: 40px;
        height: 40px;
    }

Thanks for this. I have been doing much the same with the Admin CSS already but it didn't even occur to me to use the Module CSS for the Visual Builder. That's good to know.


⚠ Bofrost.de has degraded performance

In 1161fef, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1162 ms

Resolved: Bofrost.de performance has improved in 8af42e9.


DCN support?

Thanks your great works! If neck part were implemented by DCN, how to do FGD? Thank you.

Hello, I meet the same question, have you solved it?


Notify customers if they are using an old version of the CLI

Description
Would require the CLI to call out to the CLI repo to check for latest version.

Possibly even prevent the usage of old versions entirely.

CLI version was published on 3/28/22


Update optimize-models.md

Suggested update from the D365 Guides team.  Jonathan Cobb<EMAIL_ADDRESS>and Gavin Thorton<EMAIL_ADDRESS>can help with the wording.

@BryceHolmes  can you please take a look at this?  It is to address a customer issue with D365 Guides HL, not rendering transparency correctly in some 3D models.

Added directly in private repo.


Broken citation link

In notebook, the cited link http://citeseerx.ist.psu.edu/viewdoc/download?doi=<IP_ADDRESS>.6858&rep=rep1&type=pdfor is broken.

That has been fixed in the latest version.


make file for hello.c

Hi, I am wondering if you can provide a makefile for the hello.c example. As obvious as this might be to me or to the developers of rbld, I think many people will be discouraged by the absence of a such simple example, and simply giving rbld a try.
Regards,

you can find it in here:
`cli/tests/hello
https://github.com/rbld/rebuild/tree/master/cli/tests/hello

Thanks!


[Aura-Safe][Safe Management] Some UI issues

Serenity

[x] ISSUE 11: Chỉnh màu text #FFFFFF -> Design #E6E7E8

[x] ISSUE 25: Vị trí button "Continue" canh phải

[x] ISSUE 26: Amount ngang hàng với Address

PASS in Serenity 04/08/2022
ISSUE 11:

ISSUE 25:

ISSUE 26:


Elements not tappable on real Android device

We're having issues where some elements are not tappable on a real Pixel 4a device. Taps work on the same device when emulated and also work on iOS simulator. Using 1.17.2.
When inspecting the view using studio we get the warning This element's width or height is 0 so you won't be able to tap on it.
Here's an example:

You'll notice the buttons seem duplicated as well. I also cannot tap on them in the view on the left:

We don't have this issue on the same Pixel 4a device when emulated:

Nor on iOS:

+1

any updates on this issue? we might have the same problem..

Hi everyone, can you update to the latest Maestro version 1.20.0 and try again? We released a fix

Hi everyone, can you update to the latest Maestro version 1.20.0 and try again? We released a fix

@ArthurSav what was the solution to this issue?
please advise.
thanks in advance,
rey malahay

Hi @reymalahay please open a new ticket describing your issues and the steps necessary to reproduce it.

Hi @reymalahay please open a new ticket describing your issues and the steps necessary to reproduce it.

thank you, @ArthurSav
i just cut issue ticket #1544
kind regards,
rey malahay


svtplay-dl: mark as not broken

Motivation for this change
2.4 builds OK for me.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build svtplay-dl

Rebased to master in c69ebd2883dfca8621b34e95a4e006b0c34ee7b9.


Hacktoberfest 2021 🍻

Hacktoberfest 2021 🍻     Between October 1 and October 31
MudBlazor will officially participate in Hacktoberfest 2021 as a maintainer.
Don't know what Hacktoberfest is?
Read more about it here or just take a quick look at this text from their website.

Hacktoberfest, in its 8th year, is a month-long celebration of open source software run by DigitalOcean. During the month of October, we invite you to join open-source software enthusiasts, beginners, and the developer community by contributing to open-source projects.

What will it mean for MudBlazor?
If you are participating in Hacktoberfest or would like to do so you just have to sign up on their website as a participant.
We as maintainers will start to label issues with the hacktoberfest label if we think they are good candidates to be worked on.
Participants that do a PR that we merge will get the label hacktoberfest-accepted and we decide on merging it into the library.
Other than that our rules still apply, read our contributing guidelines and our code of conduct.
PR's needs to be in line with the rest of the library, we don't want unnecessary refactoring (changing code but no bugfix/feature) We value and appreciate bug/issue fixes before new features.
If you want, plan or have thoughts about putting in allot of hours into something, it might be a good idea to ask us first so you don't spend 100 hours coding and we have to decline it.

Hacktoberfest - Participation
Hacktoberfest - Beginners
Hacktoberfest - Quality Standards
Hacktoberfest - FAQ

Please note that if a PR contains breaking changes and is something we would like to accept it will not be accepted until its time for a major/minor release. but we will mark it with hacktoberfest-accepted label
If a PR is in general really well done, documentation, code, examples, tests and allot of changes and so on we will mark it hacktoberfest-accepted label even if we are going to decline it, but this has to be a very special case.
PR's that are spam or invalid (random stuff) will be reported to hacktoberfest organizers.

can I contribute

can I contribute

Of course! Note that every PR will count for the hacktoberfest. Even issues without the hacktoberfest label.

can I modify some files for better efficiency


Incorrect library path for MacOS tools?

Bug description
espup on MacOS is looking for libraries in an incorrect place for my system.

Would you like to work on a fix? n

To Reproduce
Given install:
cargo install espup
espup install
. ~/export-esp.sh

I get an error running cargo:
$ cargo version
dyld[76858]: Library not loaded<EMAIL_ADDRESS>  Referenced from: '/Users/ted/.rustup/toolchains/esp/bin/cargo'
  Reason: tried<EMAIL_ADDRESS>(no such file), '/Users/ted/.rustup/toolchains/esp/lib/libcrypto.3.dylib' (no such file), '/Users/ted/lib/libcrypto.3.dylib' (no such file), '/usr/local/lib/libcrypto.3.dylib' (no such file), '/usr/lib/libcrypto.3.dylib' (no such file)
Abort trap: 6

my libssl.3.dylib & libcrypto.3.dylib live in /opt/local/lib/
Expected behavior
cargo works
Environment

OS: MacOS 12.6
espup version: 0.2.5

Additional context
Maybe they could be statically linked? Or shipped with the binary and put in ~/.rustup/toolchains/esp/lib/ ?

(I've copied the files over myself and have working cargo, but this could possibly block others)

Hi @tschundler!
Thanks for reporting this issue, could you try the installation with the PR branch to verify that it solves the issue?
cargo install espup --git https://github.com/esp-rs/espup --branch fix/mac-openssl
espup install
. ~/export-esp.sh

oops. I probably should have been more specific about running cargo.
I create a new esp32s3 project, eg
cargo generate --git https://github.com/esp-rs/esp-template -n esps3-test
cd esps3-test

then run
cargo version

and that's what isn't statically linked, when it uses cargo in ~/.espressif/dist/rust/rust-nightly-x86_64-apple-darwin/cargo/bin/cargo
$ otool -L ~/.espressif/dist/rust/rust-nightly-x86_64-apple-darwin/cargo/bin/cargo
/Users/ted/.espressif/dist/rust/rust-nightly-x86_64-apple-darwin/cargo/bin/cargo:
        /usr/lib/libiconv.2.dylib (compatibility version 7.0.0, current version 7.0.0)
        /System/Library/Frameworks/Security.framework/Versions/A/Security (compatibility version 1.0.0, current version 59306.101.1)
        /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1675.129.0)
<EMAIL_ADDRESS>(compatibility version 3.0.0, current version 3.0.0)
<EMAIL_ADDRESS>(compatibility version 3.0.0, current version 3.0.0)
        /usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11)
        /usr/lib/libcurl.4.dylib (compatibility version 7.0.0, current version 9.0.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1281.100.1)

(on an M1 mac, it's fine. otool -L ~/.espressif/dist/rust/rust-nightly-aarch64-apple-darwin/cargo/bin/cargo doesn't list libssl, libcrypto or libz, so they must be statically linked)
...digging more, /usr/local/opt/lib seems to be a homebrew thing, so I tried brew install openssl and that put it in the expected location. Still, it should probably be static linked.
I'm not sure who owns building that special nightly rust toolchain - maybe this bug should be in a different place?

oops. I probably should have been more specific about running cargo.
I create a new esp32s3 project, eg
cargo generate --git https://github.com/esp-rs/esp-template -n esps3-test
cd esps3-test

then run
cargo version

and that's what isn't statically linked, when it uses cargo in ~/.espressif/dist/rust/rust-nightly-x86_64-apple-darwin/cargo/bin/cargo
$ otool -L ~/.espressif/dist/rust/rust-nightly-x86_64-apple-darwin/cargo/bin/cargo
/Users/ted/.espressif/dist/rust/rust-nightly-x86_64-apple-darwin/cargo/bin/cargo:
        /usr/lib/libiconv.2.dylib (compatibility version 7.0.0, current version 7.0.0)
        /System/Library/Frameworks/Security.framework/Versions/A/Security (compatibility version 1.0.0, current version 59306.101.1)
        /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1675.129.0)
<EMAIL_ADDRESS>(compatibility version 3.0.0, current version 3.0.0)
<EMAIL_ADDRESS>(compatibility version 3.0.0, current version 3.0.0)
        /usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11)
        /usr/lib/libcurl.4.dylib (compatibility version 7.0.0, current version 9.0.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1281.100.1)

(on an M1 mac, it's fine. otool -L ~/.espressif/dist/rust/rust-nightly-aarch64-apple-darwin/cargo/bin/cargo doesn't list libssl, libcrypto or libz, so they must be statically linked)
...digging more, /usr/local/opt/lib seems to be a homebrew thing, so I tried brew install openssl and that put it in the expected location. Still, it should probably be static linked.
I'm not sure who owns building that special nightly rust toolchain - maybe this bug should be in a different place?

Sorry, is this the output of testing the fix/mac-openssl branch? In this branch, openssl should be statically linked

After updating to fix/mac-openssl branch, while having the esp toolchain active, the command cargo version has the following output on mac:
dyld: Library not loaded<EMAIL_ADDRESS>  Referenced from: /Users/paazmaya/.rustup/toolchains/esp/bin/cargo
  Reason: image not found
Abort trap: 6

The problem isn't with espup itself, but with the rust-nightly that espup installs. That cargo, that it puts in ~/.rustup/toolchains/esp/bin/cargo is what doesn't have openssl static linked.
The location it expects is where http://brew.sh puts it, not a standard MacOS location. I have no problem otherwise with cargo from https://rustup.rs/, only what espup installs.
Is there somewhere better for the bug to be?

Is there somewhere better for the bug to be?

Probably, could you open up an issue in https://github.com/esp-rs/rust-build with your conclusions? I will still merge #153 as I think its an improvement

Ack, found existing bug https://github.com/esp-rs/rust-build/issues/157
For @paazmaya , using homebrew to install openssl is a work-around that is working for me. (All the LLVM stuff for xtensa is resole, right? So we shouldn't need a special toolchain much longer?)

Ack, found existing bug https://github.com/esp-rs/rust-build/issues/157

Sorry I completely forgot about that issue, where I also participated

All the LLVM stuff for xtensa is resole, right? So we shouldn't need a special toolchain much longer?

Yes, the first 10 patches have now been accepted, hopefully we will have Xtensa support in LLVM and Rust in the future!


Deployment

Deployment
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN a Prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can access my website from any browser

https://sergluna.github.io/prework-study-guide/


Provide a way to easily create CircuitBreakerRule with RequestLog

ResponseHeaders, HttpStatus and an exception is used to decide if a request fails and the result is reported to CircuitBreaker.
https://github.com/line/armeria/blob/5b384fbe27e7e6f9225d6db91cbb684d09dfbb5e/core/src/main/java/com/linecorp/armeria/client/circuitbreaker/CircuitBreakerRuleBuilder.java#L127-L143
That would be sufficient if users may want to build a fine-grained rule based on the information in RequestLog such as response duration.
I proposed to add new builder methods to create a rule with RequestLog
class CircuitBreakerRuleBuilder {

    @Override
    public CircuitBreakerRuleBuilder onRequestLog(
            Predicate<RequestLog> requestLogFilter) {
        ...
    }
}

I'll handle this issue soon. @ikhoon nim, would you please assign this to me?

it would be consistent to have the new method receiving both ClientRequestContext and RequestLog. Should I do that like below?

Right, my example was a draft. We need to add the both version.

should add BiPredicate<ClientRequestContext, RequestLog> requestLogFilter to AbstractRuleBuilder as a member field and implement the onRequestLog method in all the subclasses, right?

It sounds good.

@ikhoon nim, I'm writing some unittests after onRequestLog implementation. Would you please take a look at below?
@Test
    void openCircuitWithRequestLog() {
        final CircuitBreakerRuleWithContent<HttpResponse> rule =
                CircuitBreakerRuleWithContent
                        .<HttpResponse>builder()
                        .onRequestLog((ctx, requestLog) -> requestLog.responseDurationNanos() > 100) // (2) RequestLogProperty#RESPONSE_END_TIME is unavailable and it throws RequestLogAvailabilityException.
                        .thenFailure();
        final BlockingWebClient client =
                WebClient.builder(server.httpUri())
                         .decorator(CircuitBreakerClient.newDecorator(CircuitBreaker.ofDefaultName(), rule))
                         .build()
                         .blocking();

        assertThat(client.get("/delayed").contentUtf8()).isEqualTo("Hello, Armeria!"); // (1) "/delayed" endpoint responds 1 second late.
        await().untilAsserted(() -> {
            assertThatThrownBy(() -> client.get("/")).isInstanceOf(FailFastException.class);
        });
    }

I want to write a unittest that circuit becomes OPEN due to delayed response, and verify that the failfastexception is returned as the response of the next request. However, in onRequestLog, I'm facing an issue where the responseDurationNanos isn't available because the response hasn't been aggregated yet. How could I make the RequestLogProperty#RESPONSE_END_TIME value available in onRequestLog?

Good question. CircuitBreaker waits until the response headers are available by default so RequestLogProperty#RESPONSE_END_TIME isn't recorded when the predicates are called.
https://github.com/line/armeria/blob/ab7488bafc2912445f026fb6d02420f134937073/core/src/main/java/com/linecorp/armeria/client/circuitbreaker/CircuitBreakerClient.java#L321-L335
We may change 1) CircuitBreakerRule.requiresResponseTrailers() to return true when onRequestLog is configured and then 2) make CircuitBrekear use ctx.log().whenComplete() if requiresResponseTrailers is true.
For example:
final RequestLogProperty property =
        rule.requiresResponseTrailers() ? null
                                        : RequestLogProperty.RESPONSE_HEADERS;

...

private void reportResult(ClientRequestContext ctx, CircuitBreakerCallback callback,
                          @Nullable RequestLogProperty logProperty) {
    final CompletableFuture<RequestLog> logFuture;
    if (logProperty == null) {
        logFuture = ctx.log().whenComplete();
    } else {
        logFuture = ctx.log().whenAvailable(logProperty);
    }
    logFuture.thenAccept(log -> { ... });
}

I got it. I'll try that then. Thank you for your explanation in detail.


Add Wikity plugin

Wikity is a plugin that allows you to use Wikitext as a templating language.
For instance you can copy-paste Wikipedia pages in and the result pretty much works.
(Since Eleventy is at 0.12 now you can use this plugin to get a true Twelvety :p)

This is awesome! Can you add a code sample to show how to use it with Eleventy’s addPlugin feature?

A code sample to the repo or as a comment?

If the latter, then first the following must be added to .eleventy.js:
const wikity = require('wikity');
module.exports = function (eleventyConfig) {
    eleventyConfig.addPlugin( () => wikity.eleventyPlugin() );
}

Then add .wiki files to your website. For example:

'''Wikitext''', also known as '''Wiki markup''' or '''Wikicode''', consists of the syntax and keywords
used by the [[MediaWiki]] software to format a page.

This will compile into:

Wikitext, also known as Wiki markup or Wikicode, consists of the syntax and keywords
used by the MediaWiki software to format a page.

This is exactly what I meant thanks! sorry if I missed it the first go-round!


Allow building all contracts from dependency package at once

Problem
Some projects grew quite sizeable groups of contracts to include in build-external-contracts listing in their users. For instance, in Dojo, users have to include this behemoth in their projects:
[[target.dojo]]
build-external-contracts = [
    "dojo_erc::erc721::components::Balance",
    "dojo_erc::erc721::components::OperatorApproval",
    "dojo_erc::erc721::components::Owner",
    "dojo_erc::erc721::components::TokenApproval",
    "dojo_erc::erc721::erc721::ERC721",
    "dojo_erc::erc721::systems::erc721_approve",
    "dojo_erc::erc721::systems::erc721_burn",
    "dojo_erc::erc721::systems::erc721_mint",
    "dojo_erc::erc721::systems::erc721_set_approval_for_all",
    "dojo_erc::erc721::systems::erc721_transfer_from",
]

Missing one of the contracts here will lead to problems.
Proposed Solution
Allow the build-external-contracts field to allow pulling all contracts from a given package/crate at once, in a glob fashion.
The expected syntax would look like this:
[[target.dojo]]
build-external-contracts = [
    "dojo_erc::*",
]

If this happens to be trivial to implement, it would be cool to also allow importing all contracts prefixed by a module. But this is not mandatory, and just pulling everything from a crate is sufficient. For instance:
[[target.dojo]]
build-external-contracts = [
    "dojo_erc::erc721::*",
]

Notes

The dojo target in this example inherits source code from starknet-contract. This task is about adding this logic to starknet-contract.
The following parts of code will require amending:

https://github.com/software-mansion/scarb/blob/bf927194941f6c0ce62677e7e2ef4f9122489ff6/scarb/src/compiler/compilers/starknet_contract.rs#L69-L92
https://github.com/software-mansion/scarb/blob/bf927194941f6c0ce62677e7e2ef4f9122489ff6/scarb/src/compiler/compilers/starknet_contract.rs#L300-L325
➕ I guess somewhere else

Don't forget about adding tests!

Please leave a comment that you are willing to contribute a PR fixing this issue before starting doing so, in order to avoid having two people working on this independently 😄

@tarrencev what do you think about this proposed syntax (with * usage)? Scarb team uniformly likes it :)

I’d like to take this

Go on :) Do you have any questions right now?

Go on :) Do you have any questions right now?

Yes, could you point me to any external examples?

The docs and the example there are the only use case that we directly aimed for: https://docs.swmansion.com/scarb/docs/extensions/starknet/contract-target.html#compiling-external-contracts
As in issue description, this feature is used by Dojo. Here is the only example I could find: https://github.com/dojoengine/dojo/blob/9774d4b3670733636485d8e7bbee6801a8d7b925/examples/ecs/Scarb.toml#L13-L25. Dojo is maintaining a copy-paste or our code FYI


Share internal identifiers and message types with the JavaScript client

Move internal messages to ActionCable::INTERNAL and sync  window.Cable so we don't have to maintain them in two places.
cc: @lifo

Looks great to me!


[FEATURE] Fasttanh

Implement activation mirroring fast_tanh_ in NeuralAmpModelerCore

Update: it's slower to compute in PyTorch, and the approximation is actually really good.
I'm going to close this and simply have the plugin use Fasttanh instead of the "real" tanh function. Simpler all around 🙂


Fix: Mark test classes as final

This PR

[x] marks test classes as final

💁♂ There's probably no reason to extend from them.

Good for me to with 1.9.1 indeed

Thanks!

Thank you, @fzaninotto and @pimjansen!


Video texture stops playing when recording begins.

I have a sphere in three.js and its texture is a video, my camera is inside the sphere. Everything works fine until I use CCapture to record my scene. The video stops playing at that point (even though video.currentTime continues to update) and the scene continues to update (it spins).
I made a QuickTime screen capture which demonstrates what is going on: http://rwoodleymedia.org/scratch/DemoForCCapture.mov
Thoughts?

OK - so this issue is related to issue #20 . I was using video.currentTime to get the current time for display purposes and that paused the video. I don't really need to display the current time, so it was easy enough to comment out.


Error in the pyproject.toml file

It looks like there is a spacing issue in the default description value for the pyproject toml.
description = "A project that adds numbers.
"

please note that i haven't tested this to see if it's something that I did when i entered the description or something native to the template!

closing this. it's because you have to hit esc to add the description. i accidentally added a line break!

Either way, it should definitely have triple quotes for being multiline =]

thank you @Midnighter  !!


Why implementation of Driver::runInternal is so complicated？

I checked out the implementation of  Driver::runInternal，it looks so complicated:

why not use coroutine to simplify the block of pyhsical thread
why not push data from bottom to top

Both 1 and 2 can make the code more easier, more maintainable, and also with high concurrency

Hi @yvxiang,
thank you for the questions.
(1) coroutines are available only in C++ 20, but Velox requires only C++ 17.
(2) we want to allow for early termination by operators like Limit and Hash Join. Limit doesn't need to receive all its input before producing full output. As soon as it has seen N rows, it is done. Similarly, Hash Join can finish early if build side turns out to be empty. We also would like to allow for the downstream operators to apply back pressure and prevent production of too much data. The design there processing id driven by demand from the downstream operators satisfies these requirements.
Hope this helps. Let me know if I misunderstood your questions, my responses are not clear or you have follow-up questions.

OK，any plan to use coroutines in the future?


Warning Querying ID3D11Device for unknown/unhandled interface: GUID {26c5dc23-e49c-4b0a-8f79-e7b1ac804d32}

I am new to this software (thanks for writing it!) and apologize in advance for lack of clarity. I am trying to capture a few buildings from Google Maps into Blender via RenderDoc.
I start Chrome in Debug mode without "OK" to the PID modal message box (PID 6232).
I inject the Process with RenderDoc, which works fine (11:07). At this point there is connection with the process but no API associated to it in RenderDoc
When I then OK the PID modal messagebox (11:10), the connection is closed by remote host
I tried this several times with no difference.
The log is below. I use Win 10 with a Dell Latitude e7240 and an Intel HD Graphics 4400 card with drivers just recently updated.
RDOC 005052: [11:07:40]    win32_process.cpp( 597) - Log     - Injecting renderdoc into process 6232
RDOC 006232: [11:07:40]             core.cpp( 440) - Log     - RenderDoc v1.11 Windows 64-bit Release (c5480aa622a2f5447c87d23ccc0b08fb320b0169) capturing application
RDOC 006232: [11:07:40]             core.cpp( 282) - Warning - Disabling crash handling server due to detected chrome.
RDOC 006232: [11:07:40]         settings.cpp( 460) - Log     - Loading config from C:\Users\giuli\AppData\Roaming\renderdoc\renderdoc.conf
RDOC 006232: [11:07:40]   win32_libentry.cpp(  66) - Log     - Loading into C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
RDOC 006232: [11:07:40]  sys_win32_hooks.cpp(  82) - Log     - Registering Win32 system hooks
RDOC 006232: [11:07:40]      d3d11_hooks.cpp(  44) - Log     - Registering D3D11 hooks
RDOC 006232: [11:07:40]        amd_hooks.cpp( 151) - Log     - Registering AMD hooks
RDOC 006232: [11:07:40]      nvapi_hooks.cpp( 117) - Log     - Registering nvidia hooks
RDOC 006232: [11:07:40]      d3d12_hooks.cpp( 173) - Log     - Registering D3D12 hooks
RDOC 006232: [11:07:40]       d3d8_hooks.cpp(  36) - Log     - Registering D3D8 hooks
RDOC 006232: [11:07:40]       d3d9_hooks.cpp(  45) - Log     - Registering D3D9 hooks
RDOC 006232: [11:07:40]       dxgi_hooks.cpp( 250) - Log     - Registering DXGI hooks
RDOC 006232: [11:07:40]        egl_hooks.cpp( 941) - Log     - EGL hooks disabled by RENDERDOC_HOOK_EGL environment variable - if GLES emulator is in use, underlying API will be captured
RDOC 006232: [11:07:40]         gl_hooks.cpp( 263) - Log     - Registering OpenGL hooks
RDOC 006232: [11:07:40]        egl_hooks.cpp( 941) - Log     - EGL hooks disabled by RENDERDOC_HOOK_EGL environment variable - if GLES emulator is in use, underlying API will be captured
RDOC 006232: [11:07:40]        wgl_hooks.cpp( 616) - Log     - Registering WGL hooks
RDOC 006232: [11:07:40]         vk_layer.cpp(  99) - Log     - Registering Vulkan hooks
RDOC 005052: [11:07:40]   target_control.cpp( 585) - Log     - Got remote handshake: chrome [6232]
RDOC 006232: [11:10:40]      dxgi_common.cpp(1633) - Warning - Querying IDXGIAdapter for unsupported/undocumented interface: IDXGIAdapterInternal2
RDOC 006232: [11:10:40]             core.cpp(1798) - Log     - Adding D3D11 device frame capturer for 0x000001CDD2775798
RDOC 006232: [11:10:40]     d3d11_device.cpp( 159) - Log     - New D3D11 device created: Intel / Intel(R) HD Graphics Family <IP_ADDRESS>63
RDOC 006232: [11:10:40]      dxgi_common.cpp(1659) - Warning - Querying ID3D11Device for unknown/unhandled interface: GUID {26c5dc23-e49c-4b0a-8f79-e7b1ac804d32}
RDOC 005052: [11:10:55]    win32_network.cpp( 261) - Warning - recv: WSAECONNRESET: An existing connection was forcibly closed by the remote host.
Thanks in advance for any help you can provide!

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


⚠ Bofrost.lu has degraded performance

In 28ae1ff, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1845 ms

Resolved: Bofrost.lu performance has improved in e3a5ee5 after 15 minutes.


Consider further generalizations to the switch statement

Personnel

[x] Owner: @jafingerhut
[x] Supporters: @mbudiu-vmw

Design

[x] Document: Union Types in P4_16

Implementation

[x] p4-spec: https://github.com/p4lang/p4-spec/pull/887
[x] p4c: https://github.com/p4lang/p4c/pull/2575

Process

[x] LDWG discussion: https://docs.google.com/document/d/1XSgdXeG1UuF1FM_XAqxDrHeN4dHZWBnJPKVS6SnGNwM/edit#heading=h.lggdh8yt82ei, at the September LDWG
[ ] LDWG approved
[ ] Merged into p4-spec
[ ] Merged into p4c

That is, over and above the generalizations proposed here: https://github.com/p4lang/p4-spec/pull/887
That PR proposed allowing the switch expression to have types bit<W>, int<W>, enum (either with, or without, an underlying type), or error, and required that all switch labels be exact match values of those types, with no duplicates.
It also describes a mechanical translation from the generalized switch statements to the more restricted one that existed before in the P4_16 language specification versions 1.2.1 and earlier.
That transformation also makes it fairly straightforward to consider generalizing switch statements even more, allowing switch expressions to be tuples/lists of one or more sub-expressions, where each tuple/list member has one of the types listed above.
If a tuple/list member had type bit<W>, int<W>, or enum with an underlying bit<W> or int<W> type, it seems reasonable to allow switch labels to use ternary expressions like 0x0ea0 &&& 0x0ff0 for a member with type bit<16>.  The corresponding field of the synthesized table would have match_kind ternary instead of exact.  It seems most efficient if the compiler chose match_kind exact whenever there were only expressions that were single values like 0x0ea0 for that tuple/list member in all switch labels, only using ternary if there was at least one expression like 0x0ea0 &&& 0x0ff0 for the tuple/list member among the switch labels.
One could also consider allowing range expressions like 5 ... 15, which seems like a useful convenience for P4 developers.  As far as the suggested transformation that P4 compiler implementers can use, that works well if the target architecture supports match_kind range for tables.  If not, one approach is to use ternary and expand ranges into multiple ternary entries when necessary, as was done in p4c's bmv2 back end for transition select statements in this p4c commit: https://github.com/p4lang/p4c/commit/28b5258bc8a310af4d7e38c8e6a534c8ed4593f4

Continuing the previous comment, if a tuple/list member had any other type, i.e. enum without an underlying type, or error, then the simplest proposal is to only define exact match values for such members.  Such values have no portable promises on how they are implemented on target devices, so ternary expressions do not really make sense.  Range expressions do not, either.

I am experimenting with migrating the LDWG discussions into items. How about this standard format for an issue?

I made a few typographical tweaks. See what you think?

Looks good. We should have somewhere this template to make it easy to copy-and-paste.
Also, I find that references to the google doc are rather weird.

Stashed it here
https://github.com/p4lang/p4-spec/wiki/LDWG-Issue-Template
Does that work?


Big noise in the wave start

I trained the model on my own data-set.  The generated voice is very good, except the problem of noise in the wave start.

The reason I think is that, the training data is a random clip from the merged voice, so there are many training data without silence start, any only few data is start with silence.
Does anyone also have this problem?

@candlewill @hdmjdp
I also have the same, synthesized by C on CPU inference.
In Python-GPU inference, there is no noise in the wave start.
LJ050-0269.zip


Added school name

Hi, I wanted to apply for the Jetbrain educational license.

@flaxeneel2 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

https://www.cosmopolitanschool.de

https://www.cosmopolitanschool.de
Rückerstraße 9, 10119 Berlin, Germany
Link: https://www.cosmopolitanschool.de/secondary-school/information-technology-it/
Sorry, if you would go to http://cosmopolitanschool.eu/ (the domain used for student emails) then it would say the site has moved, however it is still owned be the school, just that there are only student emails with that domain, all the teacher emails are with the .de domain. To provide proof i can show an email to me from a teacher who has the correct domain.

(My name is there in the licensing in one of my github repos which was made long ago so i believe that should be sufficient to show that that is my actual name. Two repos having it are haste-server-express and aio-java)

@flaxeneel2 Thank you for the clarification. Pull request merged.


Stop building function every time when "fn run" called

fn run always builds the image even if it was build previously, build option should be configurable, i.e.
fn run --no-build or some sort of.

fn run is dead


module 'sadedegel' has no attribute 'load'

In the api document(https://sadedegel.ai/detail/), usage is given as follows:
import sadedegel
from sadedegel.dataset import load_sentence_corpus, load_raw_corpus

nlp = sadedegel.load()
tokenized = load_sentence_corpus()
raw = load_raw_corpus()

summary = nlp(raw[0])
summary = nlp(tokenized[0], sentence_tokenizer=False)

This will produce error "AttributeError: module 'sadedegel' has no attribute 'load'".

You are perfectly right. This was based on a very early release :/
Correct way is available in README.md
We will also fix Website as soon as possible.


[dart2wasm] Will dart2wasm support deferred as to split main.dart.wasm like dart2js

As title.

@natebiggs is looking at implementing this.
Nate, would you like to repurpose this as a tracking bug for the implementation?

This feature is a work in progress.
Today the dart2wasm compiler is already able to emit multiple modules splitting a program using deferred imports. We have tests executing fairly complex programs in this multi-module state, some with hundreds of modules.
However, the optimizing part of our toolchain (binaryen) is not currently set up to be able to optimize this split output. So we are unable to perform most of the optimizations that we would perform for normal, single module compilations.
We're working with the binaryen team to improve the tooling to a point where they can support these multi-module optimizations. But we don't have a strict timeline to get there.
The lack of optimizations makes this impractical for production builds at this point. However, if you're just interested in trying it out the feature is available via a flag today, --enable-deferred-loading. In order to disable optimizations you must also pass -O0 to the compiler.

Thanks for answering.


FEAT: TypeORM currently using auto synch in production not migrations

Currently TypeORM is using autosynch in production which might lead to issues with data loss.
Need to establish initial migrations for the database tables
https://orkhan.gitbook.io/typeorm/docs/migrations

Moved to schema less mongo


feat(docker): Reproducible asterisc prestate

Overview
Adds a dockerfile and just recipe for building the reproducible prestate
artifacts for kona-client compiled towards the asterisc FPVM target.
Usage
just docker-build-client-prestate-asterisc <kona_tag> <asterisc_tag>
[opt: output_dir]

#357  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @clabby and the rest of your teammates on  Graphite

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

Project coverage is 77.9%. Comparing base (8915dcd) to head (02dba72).

Additional details and impacted files

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


Bump py-xgboost CI dep to 2.0.3

Testing the viability of this suggestion from @jakirkham:
https://github.com/rapidsai/dask-build-environment/pull/87#issuecomment-1974225587

Thanks Charles! 🙏
Looks like it worked 🎉
Can see it is installed here

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.47%. Comparing base (6706433) to head (86a5a85).
Report is 13 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1314   +/-   ##
=======================================
  Coverage   85.47%   85.47%
=======================================
  Files          77       77
  Lines        4242     4242
  Branches      790      790
=======================================
  Hits         3626     3626
  Misses        448      448
  Partials      168      168

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


choose bytes or str based on python 2 or 3

Code failed to run on my python 3.6 environment, and it's caused by using bytes for places where str is expected for python 3. So change to choose bytes or str based on python 2 or 3. Now I can run advisor server through docker, then use python 3 advisor client to communicate with it. In other words, I can use python3.6 to "advisor run -f ./advisor_client/examples/python_function/config.json"

Great and thanks @colinzuo . Looks good to me 👍


is "description in Model property" supported in swagger 2.0 Yaml spec?

I tried to run this in swagger ui, it doesn't display the description.
Is the below spec correct?
Here is an definition of Amount Model:
Amount:
properties:
currency:
type: string
default: "USD"
description: "This is currency unit"
value:
type: number
format: double
required:
- currency
- value

If you have an issue with SwaggerUI please open a ticket there:
https://github.com/swagger-api/swagger-ui

Closing as this was reopened on the right project.


Move the ODH Operator subscription to the fast channel for the v2.x release of ODH

Description
Updates the ODH Operator subscription to use the fast channel for v2.x installations of ODH.  Additionally adds the DataScienceCluster installation to deploy ODH in the cluster
Closes #60
How Has This Been Tested?
Local deployment into my test cluster

I'm not sure how we should handle the installation of acm/odh-core/apps since this is dependent on the DataScienceCluster CRD to be installed by OLM in order for it to work.  Given that we will eventually have access to ArgoCD features, we could use ArgoCD syncwaves to stagger the install of the operator before the ODH CRD


Compatibility with ES2.2

ERROR: Plugin [langdetect] is incompatible with Elasticsearch [2.2.0]. Was designed for version [2.1.1]
Could you provide an update, pls?

+1 - Thanks for all of your hard work.

I released Langdetect plugin version <IP_ADDRESS> which is compatible to Elasticsearch 2.2.0
https://github.com/jprante/elasticsearch-langdetect/releases/download/<IP_ADDRESS>/elasticsearch-langdetect-<IP_ADDRESS>-plugin.zip

Woop..  Thanks @jprante.


Getting '401' unauthorized in foxx service after disabling all authentication in config file

Team,
I was getting http 401 unauthorized in foxx service after disabling authentication in all the config files.
Later I commented out the below lines in foxxes.js and it started working.
anonymousRouter.get('/docs/*', module.context.apiDocumentation(
(req, res) => {
//console.warnLines("Test Message "+req.suffix);
if (req.suffix === 'swagger.json' && !req.arangoUser) {
//res.throw('unauthorized');
}
return {
appPath: decodeURIComponent(req.queryParams.mount),
indexFile: 'index-alt.html'
};
}
));
I realised that req.arangoUser was coming 'undefined'.
Regards,
Sajeev

One needed to check whether:
internal.authenticationEnabled()
is set too. Thanks for reporting.

Also I could see that eventhough the service was throwing 401, it was executed in the server despite 401.

the throw() line you commented out is directly responsible for the 401 - so thats the place where processing will stop.

Yes agreed. But its worth validating.

@sajeevpr the web interface is a Foxx service, so ArangoDB doesn't prevent access to it when a request contains no credentials. However it still parses and validates the credentials, so the Foxx service can check the arangoUser (which is the username ArangoDB has verified, or undefined if no credentials were provided or they weren't valid) to protect against unauthenticated access where needed. This check of course happens within the Foxx service itself.
This is why the Foxx service that provides the web interface can serve you a login page at all -- otherwise instead of a login page you'd only see a 401 straight from ArangoDB itself with no clue on how to authenticate.
Foxx handles HTTP error responses by allowing you to throw a specific error representing the error status code you wish to respond with using res.throw. So throwing the error is the correct behaviour when authentication is enabled. It passes back through the routing stack and is translated into the appropriate (JSON) error response by Foxx itself.

ok. Thank you all


A placeholder error message is printed when parsing files with ambiguous options

I've noticed that the csv gem produces a placeholder error message when attempting to parse a TSV with ambiguous parsing options.
If one were to parse the string
foo\t\tbar

when directed to strip whitespace (i.e., to treat whitespace as insignificant) and use tabs as field separators (i.e., to treat whitespace as significant), then should one parse it as ["foo\t\tbar"], ["foo", "bar"], or ["foo", "", "bar"]? There's no way to choose a parsing strategy that wouldn't cause a reasonable surprise to someone.
I think an error should be produced that informs the user that this is a problem; at the moment, this doesn't happen. With Ruby 3.0.2 and csv 3.2.1, the file
require "csv"

sep = "\t"
tsv_file = "example.tsv"

File.open(tsv_file, "w") { |f| f.puts("foo#{sep}bar") }
CSV.read(tsv_file, col_sep: sep, strip: true)

produces the error
csv-3.2.1/lib/csv/parser.rb:935:in `parse_quotable_robust': TODO: Meaningful message in line 1. (CSV::MalformedCSVError)

I think this is misleading; the file is not really "malformed".
Do you agree with the above? If so, I'm happy to offer up a merge request around this (e.g. to check the options before beginning the parsing, to make sure that if strip is set to true then col_sep must not be whitespace).

Good catch!
Could you open a pull request?

Thanks for getting back to me so quickly—I'm glad I'm not going mad and that you agree!
I'll work on it and raise a merge request soon.


Make the app work without magnetometer

Apparently this sensor is reasonably uncommon. We could support a lot more users if we could deal with it not being available.

I have a original Moto G phone which doesn't have many sensors. The proximity detector seems to be the favourite for detecting phone to ear. The Phone app must use it. It triggers as about 1cm distance.There's a very old app called Sensor Readout on F-Droid, if you haven't already found it, which is good for finding out what does what.

The problem with that is that this app also offers a feature to hang up calls by flipping your phone over. With just the proximity sensor you obviously can't differentiate between those.

https://github.com/TheLastProject/RaiseToAnswer/commit/7acb12e436438e066f4e6c7b0a90715589c2cc71 reinstates the support for only the Raise To Answer feature. Because we can't reliably differentiate between holding against your ear or flipping over without magnetometer, flipping over is disabled on devices without magnetometer.


Encrypted journals do not carry over well

Describe the bug
Creating a new encrypted journal entry, whether it be the first entry of the day or not, always prompts me if I would like to kill the previous day's entry because it has been modified.
To Reproduce
Steps to reproduce the behavior:

Enable encryption (refer to the Emacs Config component at the bottom).
Create a new journal.
Observe the mode-line message for the prompt to kill the previous day's buffer because it has been modified.

Expected behavior
The buffer for the previous day's journal should be saved and killed without a prompt.
Screenshots
Prompt to kill previous day's buffer
The whole message log
Desktop (please complete the following information):

OS: Arch Linux
Emacs Version: 27.2 (Spacemacs develop 0.300)

Your Emacs Configuration
The part relevant to org-journal.
(setq org-journal-enable-encryption t
      org-journal-encrypt-journal t
       )

Leads
I have deduced that this is from org-journal-carryover-items. It is in this function that the kill-buffer function is called to kill the previous journals after checking for items with the carryover value. When not encrypted, this works fine just as it is. There is no prompt to kill the previous day's journal entry and any carryover items if present will be carried over without an issue.
I believe that the reason for the discrepancy is that the carryover is done when the previous journals are de-encrypted and need to be re-encrypted and saved again before saving (the org-journal-encrypt-on variable does not seem to trigger the encryption process at all here for some reason). This is may be a small issue but it keeps popping up every time I start a new Emacs session (when I accidentally close my daemon) and want to add in a new journal entry.

Thank you for your bug report and analysis. Would you like to investigate this issue yourself? Pull requests are certainly welcome!

Sure thing, I'll look more into it once I have the time. I needed to put it out here for documentation and in case there are others out there who were facing the same issue.
By the way, this package is a wonderful tool for my workflow, I love it! Thank you for making this!


Add no-js class to html element

Add
<html class="no-js" lang="en">

See http://stackoverflow.com/questions/6724515/what-is-the-purpose-of-the-html-no-js-class

See https://github.com/mdotasia/mdotasia-theme/issues/11.

No modernizr support planned at this time.


For Save/Restore State, what do we really want to save/restore?

This is more of a discussion that hopefully will lead to some code changes.  I'm using save / restore state features to save various table states to a users profile.  The user can reload the app and still access and reload their saved states.  This leads to a problem because the saveColumns function simply does an angular.copy() on each column's "filters" object, which will attempt to save the filter's condition function.  This works fine until you reload the app.  On page reload, the closures for those functions are no longer valid so restoring that table state leads to a broken column filter.
I was thinking of creating a pull request for just saving and restoring each column filter's "term" but I think that may be too naive.  Are their other things that should be saved for each filter?  My thought is that we only save "term" since that is generally the only thing the user can modify during runtime and is not purely a config option.

The editors are customisable, and some editors that people might right would use things other than the term (for example, one of my editors creates a regEx and puts it in the condition, which lets me do an OR - it aggregates items selected from a multi-select dropdown).
I'm not quite clear on why on page reload anything would remain - surely the entire grid gets rebuilt, and anything from the past is overwritten?

To better explain the problem, the current implementation is to do an angular.copy() on the entire filters object for each column.  This copies the condition function for each filter with all the closures and object references it uses at that runtime instance.  When the page reloads, those references are no longer available.  So initially the table gets rebuilt and everything works fine as you stated.  But when the user restores their previously saved table state, the filter's condition function gets overwritten from the saved state and it breaks the filtering for that column.
I just created a pull request for my original thought.  After reading your comment about using other filter properties, maybe it would be better to remove only the condition function instead of only saving the "term" property.

I realized this is a closed issue, but I personally don't feel this is the correct solution. If you're saving state, you should be persisting it in some form. It should not be the libraries duty to handle that sort of situation. If I attach a property to an object, I expect it to be there when I call for it. It doesn't make sense to remove the condition functions because of this one edge case. It completely breaks expected results from an API perspective.
I came across this myself when writing custom filters and found it to be extremely frustrating to deal with when others are actually trying to persist conditions.
https://github.com/angular-ui/ui-grid/issues/5250
Above is another issue explaining the same issue.

Omg, why would you leave condition? I use custom templates for filtering and user can chose not only term but also a condition. Now I have to reimplement the whole saving of filters...


docs: update Vite version

The project is now using Vite 5, here's a screenshot from running latest main:

@all-contributors add @tomholford for doc.


fix(behavior_velocity_planner): vehicle points lateral calculation

Signed-off-by: Hasanzfdn<EMAIL_ADDRESS>Description
closes #1734
In addition to the issue, the util.cppcode has also been changed

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

LGTM for obstacle avoidance planner
@taikitanaka3 Could you check for behavior velocity planner

Ok I will check this later.

@hasanzfdn lateral polygon including this PR is something wrong. please check with occlusion spot / run out detecion area polygon

I tried again and again but I have not encountered such a problem. Screenshot like this.

@taikitanaka3 have you tested it again?

@xmfcx
Sorry I didn't test this again yet,  @xmfcx can you help me testing this PR if you have time to?

Thank you @hasanzfdn, can you rebase PR for review? There are some conflicts on obstacle_avoidance_planner.

I tested with different maps and vehicles, I think occlusion_spot problem is solved.

Another map and vehicle:

@taikitanaka3 @TakaHoribe It looks like LeoDrive member has done a review for this PR. Would you like to run Planning team CI internally before merging?

LGTM for me @TomohitoAndo  can you take a look at run out module?

@hasanzfdn
It seems variables of wheel_tread, right_overhang and left_overhang are not assigned.
Could you modify the manager.cpp for run out module like you did in occlusion spot module?

@hasanzfdn will you be able to work on this or shall we assign this issue to @ismetatabay?

I will update the lateral calculation of the vehicle in the run out module.

@hasanzfdn Can you update the PR to the draft stage?

@TomohitoAndo Hi,  I updated the run out model as you said and it works as in the video you shared.

@ismetatabay
Thanks! I checked it works in my environment and CI.
LGTM

@taikitanaka3 a code owner approval is also required, can you check it please?


Add documentation

The docs should include:

basic info for integrators
description of non-portable features
"How to contribute"

Closing as implementation has been moved to https://github.com/resteasy/Resteasy/tree/master/resteasy-client-microprofile


Checkbox machine props TypeScript error

🐛 Bug report
Checkbox machine props for label give TS errors with SolidJS. It occurs only with the strict mode enabled.
💥 Steps to reproduce

Copy a checkbox example from docs for SolidJS
Enable TypeScript strict mode
Label - span element, props give TypeScript error

💻 Link to reproduction
https://stackblitz.com/edit/solidjs-templates-jkxx7d?file=src/Checkbox.tsx
🧐 Expected behavior
No TS error regardless of TypeScript settings
🧭 Possible Solution
I see that it's using label element for props type despite using span in examples. Maybe it's something off here one way or another?
🌍 System information

Software
Version(s)

Zag Version
0.2.3 (checkbox), 0.2.2 (solid)

Browser
N/A

Operating System
N/A

📝 Additional information
--

While digging into I see that there are props like for which probably shouldn't be on span element 🤔

Thanks a lot for solving the issue, awesome work 🚀


[Bug] [Module Name]

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
3.0.0版本，上传到Hadoop资源中心中的文件，在工作流中执行，下载后归属于root用户而不是当前用户，导致无法执行。
What you expected to happen
从资源中心下载的文件别的用户也有权限执行，或者下载的文件归属于当前登陆用户。
How to reproduce
1、上传文件
2、在工作流中执行。
Anything else
No response
Version
3.0.x
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This problem is about the permissions of Linux. Please use the correct title. I'll close this issue.

This problem is about the permissions of Linux. Please use the correct title. I'll close this issue.

how can I fix this question?

how can I fix this question?

You can ask it in stackoverflow.


Add initialisation logging and test coverage

refs #28
As a first step to debug this issue this patch adds some logging for when initialisation for the extension fails. For that I did some minor refactoring to make code a bit more testable.

@christian-bromann this branch won't run for me. looks like test fail too?

Yeah, have to rebase. One sec.

@sourishkrout should be ready for review now.


[now-cli] Fix now rm -h

Currently, now rm -h outputs:

This PR fixes it:

Codecov Report

Merging #3581 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3581      +/-   ##
==========================================
+ Coverage   14.04%   14.09%   +0.04%
==========================================
  Files         274      274
  Lines       11098    11145      +47
  Branches     1528     1529       +1
==========================================
+ Hits         1559     1571      +12
- Misses       9370     9405      +35
  Partials      169      169

Impacted Files
Coverage Δ

src/util/metrics.ts
61.53% <0%> (-6.65%)
:arrow_down:

src/util/prefer-v2-deployment.ts
55.1% <0%> (-4.9%)
:arrow_down:

src/util/projects/get-project-by-id-or-name.ts
19.04% <0%> (-4.49%)
:arrow_down:

src/util/dev/router.ts
69.33% <0%> (-3.91%)
:arrow_down:

src/util/dev/yarn-installer.ts
78.12% <0%> (-3.55%)
:arrow_down:

src/util/output/create-output.ts
57.4% <0%> (-2.6%)
:arrow_down:

src/util/dev/builder-cache.ts
77.88% <0%> (-1.62%)
:arrow_down:

src/util/get-files.ts
89.61% <0%> (-0.89%)
:arrow_down:

src/util/get-update-command.ts
9.37% <0%> (-0.63%)
:arrow_down:

src/util/dev/builder.ts
65.29% <0%> (-0.6%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


Local pull request success status testing

Docs Build status updates of commit 6e1950e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


docker-compose.yaml variables possible?

Is it possible to set the FQDN of the images in a variable? Our server is not allowed to connect directly to the internet, we pull the containers via our jfrog artifactory.
Example:
services:
apigw:
container_name: "vc_dev_apigw"
image: docker.sunet.se/dc4eu/apigw:latest
restart: always
volumes:
- ./dev_config_docker.yaml:/config.yaml:ro
- /var/log/sunet:/var/log/sunet
depends_on:
- redis
- mongo
networks:
vc-dev-net:
ipv4_address: <IP_ADDRESS>
environment:
- "VC_CONFIG_YAML=config.yaml"
The bold formatted "docker.sunet.se" would have to be in a variable that we have to set at startup. The same for redis/mongo/jaeger.

As stated in the README file in this repo, this repo and its tools and scripts is intended for developers. However I have made this repo (https://github.com/dc4eu/vc_up_and_running) intended for running the code.
I will update the README in this repo with this info and add url in variable in vc_up_and_running repo.

Seems fine


registration assist does not read both functional scans

I hope i can explain this issue accurately.
When using the --registration-assist flag for contrast enhanced (ce) functional images, the MOVING and REFERENCE files are incorrectly read into the appropriate space. essentially, from what i understand, when this flag is active the pipeline tries to register the normal bold scan (task-rest) to itself.
It appears to be doing this because the script is only looking for the phrase "task-rest" and once it finds it, it stops. Instead, it needs to continue and include more of the name to enable the pipeline to tell the difference. In the images below, the first one details the submission command and allows you to orient yourself. The latter describes where both the regular bold and contrast enhanced are using the same reference image.

after additional troubleshooting, using this flag causes terminal errors that within FMRIVolume.


Good day, I have a dynamic table on Bootstrap

Good day, I have a dynamic table on Bootstrap, is it possible to use TableFilter for such tables? Now it simply reports that at least 1 row is needed for the work, although the data is loaded into the table. Data is loaded via AJAX. Thanks and waiting for reply<EMAIL_ADDRESS>

@grad23 make sure you instantiate TableFilter when the async request to populate has completed and the table rows are effectively rendered. From what you describe it seems to be a timing issue related to AJAX.

Ok, more or less managed to make friends filter and table, but when pagination from bootstrap-table data selection is not for all values of the table, but only for those displayed on the page. For example, in the table of 100 values from 0 to 99, the pagination bootstrap-table = 10 values on the page, respectively, the "5" filter finds and shows, and 11 (which on the next page) does not see.
And the second, after setting the value of col_2: 'select', (col_2 = field_id as the simplest without using the formatter) instead of the list of values gives an error.
Uncaught TypeError: Cannot read property 'cells' of undefined
at dropdown.js:218
at tablefilter.js:2828
at Dropdown.build (dropdown.js:188)
at Dropdown.onSlcFocus (dropdown.js:60)
at HTMLSelectElement. (dropdown.js:127)

I put script for table-filter in to .on('load-success.bs.table', function (e, data, row, value) { , so all data are loaded by AJAX before...

If you publish an example reproducing the issue on codepen or equivalent, I will be able to understand a little bit more.

can be re-opened if needed


Need to have an option in the management console UI to download public certificate of the Identity Server

Currently, in order to get the public certificate of the super tenant, we have to access the file system and use the keytool command. It is better to have a UI option as well to support the users who don't have access to the file system.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Traefik crash on --cluster=true

What version of Traefik are you using (traefik version)?
1.1.2
What is your environment & configuration (arguments, toml...)?
The configuration is in the script below.
What did you do?

Traefik working in clustered mode inside Kubernetes, with a pod running on each node in the cluster.
Without the 'cluster=true' flag, one node claims the consul store, and the others don't work properly.
With the 'cluster=true' flag, the Traefik pods crash with the error message shown below.
What did you see instead?
The nodes crash, and produce the following stack trace:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x63872a]

goroutine 1 [running]:
panic(0xd2c680, 0xc420014020)
	/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/containous/traefik/cluster.NewLeadership(0x7fd5b349a388, 0xc42042fb80, 0xc4203f4ee0, 0x7fd5b349a348)
	/go/src/github.com/containous/traefik/cluster/leadership.go:27 +0x7a
main.NewServer(0xa, 0x100, 0x0, 0x0, 0x0, 0x0, 0xc420431324, 0x5, 0xc4202d3a70, 0xc4203f4ee0, ...)
	/go/src/github.com/containous/traefik/server.go:88 +0x423
main.run(0xc420318120)
	/go/src/github.com/containous/traefik/traefik.go:285 +0x6e3
main.main.func1(0x613866332d323201, 0xc420369480)
	/go/src/github.com/containous/traefik/traefik.go:52 +0x2a
github.com/containous/traefik/vendor/github.com/containous/staert.(*Staert).Run(0xc4204bfeb8, 0xc93960, 0xc420318120)
	/go/src/github.com/containous/traefik/vendor/github.com/containous/staert/staert.go:83 +0x2e
main.main()
	/go/src/github.com/containous/traefik/traefik.go:204 +0x12ad

Do you still have the same problem with the latest version?

@ldez latest as in 1.3, or 1.2 branch?

@salzig it's the 1.3.0-rc2, look at https://github.com/containous/traefik/releases

I'll close the issue due to inactivity but feel free to re-open if needed.

I have the same issue with latest Traefik, here's what I did:
(I had a previous traefik instance running, with a config file but would now like to switch to a 3-node cluster)

# traefik storeconfig
- >consul kv contains config just fine

# traefik --cluster=true -d

INFO[0001] Using TOML configuration file /etc/traefik/traefik.toml
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x83500e]

goroutine 1 [running]:
github.com/containous/traefik/cluster.NewLeadership(0x2e08560, 0xc420557840, 0xc4202a1340, 0x0)
        /go/src/github.com/containous/traefik/cluster/leadership.go:28 +0x6e
github.com/containous/traefik/server.NewServer(0x2540be400, 0x100, 0x0, 0x0, 0x0, 0x0, 0x0, 0xc42035b930, 0x5, 0xc4205ef740, ...)
        /go/src/github.com/containous/traefik/server/server.go:105 +0x63e
main.run(0xc4205678c0)
        /go/src/github.com/containous/traefik/cmd/traefik/traefik.go:307 +0x6f6
main.main.func1(0xc42016cdc0, 0xc4202b31a0)
        /go/src/github.com/containous/traefik/cmd/traefik/traefik.go:61 +0xd9
github.com/containous/traefik/vendor/github.com/containous/staert.(*Staert).Run(0xc4206c1f30, 0x1aa1940, 0xc420496300)
        /go/src/github.com/containous/traefik/vendor/github.com/containous/staert/staert.go:83 +0x2e
main.main()
        /go/src/github.com/containous/traefik/cmd/traefik/traefik.go:218 +0x1bf1

@beniwtv Thanks for your interest in Traefik 😃
This issue is closed. Please discuss this in :

Stack Overflow using the "traefik" tag
the Traefik community Slack channel


Adding Icons in the navbar

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
The current navbar lacks icons, which impacts its visual appeal and usability. Users might find it less engaging and intuitive to navigate through the website without visual cues provided by icons.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I propose adding respective icons to the navbar to enhance its visual appeal and improve user navigation experience. These icons will align with the theme of the website and provide users with clear visual cues for each navigation option
** Add screenshot

Checklist
Example Fill x { [x] } into bracket

[ x] I have checked the existing issues *
[x ] I have read the Contributing Guidelines *
[ x] I have starred the repository *
[ x] Working under Gssoc24*

I would like to work on this issue, can you please assign it to me?

@harshmishra19  already someone is working on this issue.


Incorrect command in section "Install and configure the proxy service"

Step 5 of this section lists the command Test-AzureADPasswordProtectionDCAgentHealth -TestAll
The correct command to use is Test-AzureADPasswordProtectionProxyHealth -TestAll
Step 7 mentions the same incorrect command. It's unclear whether the output of Test-AzureADPasswordProtectionProxyHealth -TestAll has any diagnostic which verifies the successful completion of step 6.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 23e000f0-84ce-7ec7-e5ed-4bd9c49f5d49
Version Independent ID: e636b9d1-be81-0f00-97c4-ff4fa9078be3
Content: Deploy on-premises Azure AD Password Protection - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-password-ban-bad-on-premises-deploy.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@interro-bang
Thanks for your feedback! We will investigate and update as appropriate.

@interro-bang
Thanks for reporting this! We have created a PR for this issue once the author will approve the changes should go live soon.


New Python and Node.js Samples

Push Button Sample written in Node.js
Push Button Sample written in Python
Web Cam Sample written in Node.js
WiFi Connect Sample written in Node.js

Hi @trmck, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

cc. @alanch-ms for the Python sample

Really cool. Are you also adding pages and links from windowsondevices.com?

In progress ... eta 6/6


feat(Analysis/Convex): Radon's convexity theorem

Add Radon's theorem on convex sets

Thank you very much for your review and advice, I've learned a lot of things from this! I did everything you suggested and make a few more small simplifications in both radon_partition and radon_set_partition.

Thanks everyone! I am really happy about my first commit!
bors r+


Add vela uninstall

Currently vela supports install command. The uninstall command is also needed to clean up things at the end in tutorial section. Note that this also requires uninstalling dependency components.

fixed in #467 .
This is handled in server side.


Config yaml error

Config issue:
When I was processing my own video data, I encountered the following message: "assert os. path. exists (args. config)"
"AssertionError" error, I checked and found that there is indeed no config folder for my video in the egs/datasets/videos folder. How can I obtain this config, or can I directly use May's to replace it?

You can copy config files from May, but remember to edit the config files and replace all occurrence of "May" to your own video name


Add VS Code Keyboard Shortcuts

Warning
Only Add 1 Keyboard Shortcut Per Pull Request.

Note
Follow Me: https://www.linkedin.com/in/anshsinghsonkhia

@AnshSinghSonkhia Wouldn't there also need to be one issue per PR, so the PR can use fixes #11?

@AnshSinghSonkhia Wouldn't there also need to be one issue per PR, so the PR can use fixes #11?

@brandonmcconnell Issues are just to mention an issue in this GitHub Repo. It's not necessary to be an assignee. You can submit the PRs without being an Assignee.