bus: ensure we don't depend on messages being zeroed-out

Not entirely sure this is a good idea TBH.
The primary win here is that, if we accidentally depend on stuff being zero, @memset(undefined) will catch us out. The secondady win here is slight perf boost. A lot of our messages are header-only, and zeroing-out the whole body for them seems wasteful.
The primary drawback here I think is that we expose messages to the clients, and our users might not be prepared to deal with genuinely uninit data, as it usually isn't idiomatic thing in non-Zig languages...

Yeah, great idea!


Update list of collaborators

The collaborator list was initially added a few months ago in https://github.com/apache/pinot/pull/13346.
The list needs to be periodically updated (every release or so) to take into account changes in contribution activity as well as existing collaborators becoming committers.
The new list was generated with the same method used previously:

> git shortlog --numbered --summary --since="1 year ago"

   440  dependabot[bot]
   166  Xiaotian (Jackie) Jiang
   108  Xiang Fu
    74  sullis
    57  Gonzalo Ortiz Jaureguizar
    55  Pratik Tibrewal
    55  Yash Mayya
    48  Abhishek Sharma
    43  Xiaobing
    40  Rong Rong
    37  Shounak kulkarni
    27  Christopher Peck
    20  Haitao Zhang
    18  Saurabh Dubey
    18  soumitra-st
    17  Ankit Sultana
    17  Kartik Khare
    14  Vivek Iyer Vaidyanathan
    14  swaminathanmanish
    12  Prashant Pandey
    12  Seunghyun Lee
    11  Rajat Venkatesh

Committers were manually filtered out from the above list (using the source of truth here - https://projects.apache.org/committee.html?pinot)

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

Project coverage is 0.00%. Comparing base (59551e4) to head (a7069e4).
Report is 1090 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (a7069e4). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (a7069e4)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14086       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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


Create a timeline

Add a list of all your activities, achievements, work experience. Add as much info as you can remember. In case you don’t have working experience tell about your pet-projects or student projects. Don’t be shy :)

Initiating the creation of a timeline section. This will display a chronological order of significant events or milestones.


Unexpected Behaviour in RBAC Support for Scopes

Describe the issue:
When trying Role-based access control support for scopes,  The scopes are returning for the users who doesn't even have the specific role assigned.

How to reproduce:

Create a new role -> ex: sample_role
Add a new scope and bind the above role
Create a user and assigned the newly created role
Generate a token requesting that scope for the user with the above mentioned role (sample_role). -> works as expected
Generate a token requesting that scope for a user who doesn't have the sample_role

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: WSO2 Identity Server-7.0.0-m4-SNAPSHOT
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

This issue was a missing config from the guide.
This issue initially stemmed from a missing configuration in the guide. Even after adding the required configuration, the expected behavior did not manifest. The key analysis was that during role binding with the scope, it is necessary to define the domain, especially if it's not a PRIMARY role.
Sample request body
{
    "name": "sample_scope",
    "displayName": "sample_scope",
    "description": "sample info",
    "bindings": [
        "Internal/sample_role"
    ]
}

With these adjustments, the functionality now works as expected.


Unit 1 Sprint 4

Can I just take a moment to say how freaking impressed I am with you right now. The amount of improvement in your python code from last week to this week is actually blowing my mind right now.
na_values=['?'] Love this, great way to catch those values and auto convert them.
for col in df:     dt = df[col].dtype      if df.isna().sum()[col] == 0:       next     else:       if dt == int or dt == float:           df[col].fillna(df[col].mean(), inplace=True)       else:           df[col].fillna(df[col].mode()[0], inplace=True)
slightly wordy, but I love that you're interpolating your missing data values. Solid!
Really cool how you split your barcharts into target classes for positive and negative approved credit. It really helped add a story element to your analysis.
Really, really great work this week


Custom filename for JSON file from Update-DbcPowerBiDataSource

Feature Request
When working with a longer custom list of checks (i.e. specifying individually rather than using all checks in a file) it is easy to generate a filename too long to be saved properly in Windows:
WARNING: [16:44:01][Update-DbcPowerBiDataSource] Failure | The specified path,
file name, or both are too long. The fully qualified file name must be less
than 260 characters, and the directory name must be less than 248 characters.

Being able to specify the target JSON file name would allow this to be avoided.

I know the alternative is to utilize the ExcludeCheck flag, but that ends up being unclear as to what you're actually trying to check in the command and would result in new checks being automatically added to your check cycle when you upgrade.

Hi @LowlyDBA - Sorry I missed this - You can already do this with the -Environment parameter on Update-DbcPowerBiDataFile (The hardest command name to say :-) )
(This code also creates naming convention for the XML for VSTS
Heres how I do this
``        $date = Get-Date -Format yyyMMddHHmmss
$base =  $VMName + "" + $System + "" + $date
$filename = "C:\windows\Temp\DbaStack" + $base + "_" + "PesterResult.xml"
$PesterResult = Invoke-DbcCheck -OutputFile $filename -OutPutFormat NUnitXml -PassThru
$PowerBiFileName = $System +"-" + $VMName+ "-" + $date
$PesterResult | Update-DbcPowerBiDataSource -Environment $PowerBiFileName

@SQLDBAWithABeard  I updated to the latest version and it still seems to append all specific checks in the name.
Using:
Invoke-DbcCheck -Show Summary -SqlInstance $array -Check $envChecks -PassThru -OutputFormat NUnitXml | Update-DbcPowerBiDataSource -Environment $ConfigurationGroup -Path $destPath -Append

Results in:

Where $configurationGroup is either "prod" or "staging" and despite the -Append flag, we are deleting the files before each run of the script.

Don’t use the path parameter. Specify the entire path in the environment parameter instead

Thank you!

Still struggling with this...I think the issue is still that the code appends the checks/tags in addition to the environment name versus instead of:
$basename = "dbachecks_$i"
        if ($InputObject.TagFilter) {
            $basename = "$basename`_$($InputObject.TagFilter -join "_")"
        }

        if ($Environment) {
            $basename = "$basename`_$Environment"
        }

So you still get dbachecks_1_check1_check2_check3_check99_environment.json as a name as soon as you specify -Check or -Tag

of course, I didnt use a checks param - apologies
Let me look at this now.


Added link to CSS Diner under More Information

[x ] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks for contributing, but this link doesn't work - Thanks and happy coding!


"cargo new" executed in a drive root hangs

Executing cargo new directly in a drive root on Windows (eg, C:\ or D:) causes cargo to hang indefinitely at 100% CPU usage. Watching file access using ProcMon shows it attempting to open .git and objects directories, making me think that it's trying to scan upwards for a parent git repo.
Possibly related to #3783?
λ  cargo --version
cargo 0.18.0 (fe7b0cdcf 2017-04-24)

Thanks for the report! This is likely coming from libgit2 and is probably a bug in libgit2, but it'd be great if we could work around it. If you've got a reliable reproduction of this it'd be great if you could help out debugging! If we could get a backtrace that'd be invaluable for figuring out what's going wrong here.

I've traced this back and confirmed that it is a bug in libgit2. It's a loop in git_discover_repository caused by bugs in libgit2s path implementation. It's known and fixed in the latest versions of libgit2:
https://github.com/libgit2/libgit2/issues/4106
I'm not sure if there's any reasonable workaround beyond updating to the latest version of libgit2, short of implementing a manual version of repo scanning.

Oh, it's worth noting that I'm pretty sure the title of the issue, which says this only happens on directories other than C;\ is just wrong - reviewing the fixes, there's no reason I can see why c:\ is special and bugs reported against other libgit2 language bindings correctly identify it happening in any drive root.

Oh excellent! I'll go update libgit2 so we can get that bugfix. Thanks so much for investigating!

Ok I've published libgit2-sys 0.6.11 which should have this fix, I've sent a PR to rust-lang/rust to update the lockfile as well.


Return result set type information in the HTTP response

We currently don't know the exact types of each column in the query result. We should encode the the information as JSON in a response header.

If implementing this feature, remember to also make sure the header is included in Access-Control-Expose-Headers (not to be confused with Access-Control-Allow-Headers, which is about which headers are allowed in the request) so that cross-origin client code can read it from the response.
This is not necessary for CORS-safelisted response headers, which include only Cache-Control, Content-Language, Content-Length, Content-Type, Expires, Last-Modified, and Pragma (deprecated).
Perhaps it would be appropriate to use the "parameters" of the Content-Type response header, which is actually kind of intended for this purpose (but could have unknown effects on proxies like Cloudflare):

The indicated media type defines both the data format and how that data is intended to be processed by a recipient

The type/subtype MAY be followed by semicolon-delimited parameters (Section 5.6.6) in the form of name/value pairs. The presence or absence of a parameter might be significant to the processing of a media type, depending on its definition within the media type registry. Parameter values might or might not be case-sensitive, depending on the semantics of the parameter name.

For example, for this request:
curl 'https://demo.seafowl.io/q/4be9110c4fe720dae8310fde08f028b49d01a5c41b70df21668a143265f6a4bc.csv' \
  -H $'x-seafowl-query: SELECT%20year%3A%3Ainteger%20AS%20year%2C%0ASUM(volume)%20AS%20total_volume%0AFROM%20supply_chains%0AWHERE%20country_of_production%20%3D%20\'ARGENTINA\'%0AAND%20country_of_import%20%3D%20\'AFGHANISTAN\'%0AAND%20commodity%20%3D%20\'CORN\'%0AGROUP%20BY%201%20ORDER%20BY%201%20ASC'

which outputs:
{"total_volume":78.0,"year":2018}

maybe we could return the content-type header like:
content-type: application/octet-stream; total_volume=FLOAT; year=INT;

Another approach might be to include that info in the output itself, like snowflake does, and then nest the actual results under some specific key, e.g. data.
However this would be a breaking change, and possibly hurt readability. Also the header approach is simpler, so I went with that for now, check it out: https://github.com/splitgraph/seafowl/pull/367

Nesting the results under a specific key is how we do it in the web DDN, but that means abandoning jsonlines. I'm not a fan of that approach, because it means we (as a client) cannot support streaming result sets, and must buffer the entire result into memory before we can parse it.
In practice, at least in the madatdata JS client, we still buffer the entire result set into memory right now, but that's only because I haven't implemented readable streams.
The header approach seems hacky at first glance, but I think it's actually quite elegant - I mean after all, that's what a header is for, isn't it?


[Search v2.5] Tax rate field is displayed after disabling Taxes

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v.9.0.60
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Issue was found when executing this PR: https://github.com/Expensify/App/pull/52102
Issue reported by: Applause Internal Team
Action Performed:

Go to staging.new.expensify.com
Log in with a new account
Create a new workspace
Go to workspace settings > More features
Enable Taxes
Go to Search > Filters.
Verify that there is tax filter displayed if user has taxes
Go back to workspace settings > More features
Disable Taxes
Go to Search > Filters.

Expected Result:
There is no tax filter displayed if the user has no taxes.
Actual Result:
There is tax filter displayed when the user has no taxes.
The filter disappears when the user clears the cache (Troubleshoot > Clear cache and restart).
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/f1e2d88d-7f00-4249-be17-856e96f945d7

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @sumo-slonik

Hey! I'm Kuba Nowakowski from Software Mansion, an expert agency, and I'd like to work on this issue!
I talked to @Kicu

CC @luacmartins this looks pretty simple, probably need a more advanced if check around taxes.
I suggested that Jakub here can handle it ☝

Gonna demote since it doesn't break any functionality

@sumo-slonik all yours

@luacmartins do I need to add the Help Wanted label to post on Upwork, or are @sumo-slonik and @Kicu handling it totally separate to Upwork?

@RachCHopkins we work for an expert agency and no Upwork post is needed for us :)

Assigning myself too since I got assigned the contributor's PR for review.

@sumo-slonik any updates here?

It seems to me we a PR was created and successfully was merged 3 weeks ago?
https://github.com/Expensify/App/pull/52462
Perhaps something failed in automation?

Correct, this PR was merged 3 weeks ago

There is no C+ here and the authors/reviewers are either internal or from SWM, so no Upwork payment is due. Closing


fix(warnings): use regex as string refs #253293

Warning: Failed prop type: Invalid prop `path` supplied to `Route`, expected one of type [string].

@nileshgulia1 I think we can leave with the warning :smile:
$ node
Welcome to Node.js v16.20.0.
Type ".help" for more information.
> const notInEn= /^(?!(#|\/en|\/static|\/controlpanel|\/cypress|\/login|\/logout|\/contact-form)).*$/
undefined
> notInEn
/^(?!(#|\/en|\/static|\/controlpanel|\/cypress|\/login|\/logout|\/contact-form)).*$/
> notInEn.source
'^(?!(#|\\/en|\\/static|\\/controlpanel|\\/cypress|\\/login|\\/logout|\\/contact-form)).*$'
>

I tend not to trust this source thing. Maybe we can fix it in Volto Core to allow also regex :thinking:

@avoinea Yes perhaps, We should indeed touch volto source for this. The react-router already converts paths fragments into regex using path-to-regexp and return simply regex when regex is supplied in routes splitting a warning!


Flaky TestDelayedCompaction test

What did you do?
In the CI step for Run make GOOPTS=--tags=stringlabels GO_ONLY=1 SKIP_GOLANGCI_LINT=1, I just got this test failure with a panic in compaction:
--- FAIL: TestDelayedCompaction (2.68s)
    compact_test.go:2042:
        	Error Trace:	/__w/prometheus/prometheus/tsdb/compact_test.go:1965
        	            				/__w/prometheus/prometheus/tsdb/compact_test.go:2042
        	Error:      	"1.982065514s" is not less than "1s"
        	Test:       	TestDelayedCompaction
    --- FAIL: TestDelayedCompaction/delayed_compaction_not_enabled (2.68s)
panic: test executed panic(nil) or runtime.Goexit

goroutine 6180 [running]:
testing.tRunner.func1.2({0x1562940, 0x229b970})
	/usr/local/go/src/testing/testing.go:1632 +0x3fc
testing.tRunner.func1()
	/usr/local/go/src/testing/testing.go:1635 +0x6b6
runtime.Goexit()
	/usr/local/go/src/runtime/panic.go:629 +0x5e
testing.(*common).FailNow(0xc00062e1a0)
	/usr/local/go/src/testing/testing.go:1006 +0x85
github.com/stretchr/testify/require.Less({0x1b27f70, 0xc00062e1a0}, {0x1645680, 0xc0027ec1e8}, {0x1645680, 0xc0027ec1f8}, {0x0, 0x0, 0x0})
	/go/pkg/mod/github.com/stretchr/testify@v1.9.0/require/require.go:1188 +0x107
github.com/prometheus/prometheus/tsdb.TestDelayedCompaction.func2(0xc00064c000)
	/__w/prometheus/prometheus/tsdb/compact_test.go:1965 +0x10a
github.com/prometheus/prometheus/tsdb.TestDelayedCompaction.func3(0xc00062e340)
	/__w/prometheus/prometheus/tsdb/compact_test.go:2042 +0x626
testing.tRunner(0xc00062e340, 0xc001a38270)
	/usr/local/go/src/testing/testing.go:1690 +0x227
created by testing.(*T).Run in goroutine 6179
	/usr/local/go/src/testing/testing.go:1743 +0x826

What did you expect to see?
No response
What did you see instead? Under which circumstances?

System information
No response
Prometheus version
No response
Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

@machine424

I'm not seeing any failures on main in last couple of weeks, I think this was a result of the on space left on device issue as for https://github.com/prometheus/prometheus/issues/14857
I think we can close this.

It could also generally have been related to the race detector being enabled in tests, which we disabled temporarily (and still have disabled now) to make CI tests pass. I guess it's ok to close for now, but it could very well be that it resurfaces once the race detector is enabled again.


Infiray types

hello,
well done on this project @LeoDJ.
I'm trying to work with 2 modules xcore micro 3 lite, but I'm trying to customize a Python code for specific requirements.
does this code work with my modules? does it read two cameras at the same time? also if this code can open an irv video recorded files.
thank you very much.
hope i hear from you soon.
Ajami

The only module I think might be similar to the P2 Pro would be the Tiny1-B module.
And my tool is currently pretty unfinished, I didn't have the time the last few months to work on it.
From the specs of the InfiRay Micro III Lite 640 module, it looks to be quite another type of product. The video interface isn't simply USB or SPI, but rather LVDS/MIPI/BT.656/CDS2 (more complicated).
I'm afraid my tool wouldn't help you with that either, even if it would be working ^^"
But I wish you much success for your project, sounds cool!


Fixed issue with offer/new API call

Method for offer/new API call was silently dropping params. Also, server enforces weird types for specific params, so we coerce the params to required type before making a call.

Hi @arvicco  it makes perfect sense, thank you !


Really Dumb Question

But when you have it all setup, where do you enter the code? I mean do you enter it into terminal and press enter? because that does nothing. So when i want to edit it, how would i run the code of Mewtwo? I already have the code from paste bin but how do i run it?


added lazy loading to images

Description

I have added a lazy loading option to images.
Issue Ticket Number
Fixes [Issue #23 ]
Type of change

[] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[x] I have created an issue ticket for this PR
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[ ] I have run tests locally to ensure they all pass
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

hey @developer-diganta i think this is close, but not quite there
the idea is that the loadingStrategy value is a string wher we can pass that value in to the loading attribute
for instance:
loadingStrategy: 'lazy'

where then
setAttribute('loading', loadingStrategy)

which would yield
loading="lazy"

the goal would be that it's configurable, however, the value would default to lazy:
const { loadingStrategy = 'lazy' ...

additionally while this defines the option in the update function, this doesn't actually make it available to use. currently, there's not a way for someone to specify the loading strategy through configuration, as someone would never touch any of the code itself when using the plugin
please review the details I added to the GitHub Issue prior to your comments, such as: https://github.com/colbyfayock/netlify-plugin-cloudinary/issues/23#issuecomment-1264367515

So if I correctly understood it, I have to add loadingStrategy as input in index.js? And for netlify.toml, shall I give it a value of loadingStrategy : 'lazy'?

@colbyfayock Could please check if it is correct now?

Looks great @developer-diganta! Last thing...
Can you please update the documentation to include the input
https://github.com/colbyfayock/netlify-plugin-cloudinary#plugin-inputs
Then we should be good to go!

@colbyfayock Done✅ And thanks for the help!

Thanks @developer-diganta for the contribution and your patience!
Email me at<EMAIL_ADDRESS>where I'll send you a form to claim your swag :) https://spacejelly.dev/hacktoberfest/

@all-contributors please add @developer-diganta for code, doc, test

Thanks @developer-diganta for the contribution and your patience!
Email me at<EMAIL_ADDRESS>where I'll send you a form to claim your swag :) https://spacejelly.dev/hacktoberfest/

Done✅ Thank You for the swags too!


rollup-plugin-esbuild has import issue which affects 0.5.10 to latest

rollup-plugin-esbuild had an issue with import for es-module-lexer: https://github.com/egoist/rollup-plugin-esbuild/issues/325
this affects all fresh installs from Unbuild:
please upgrade to "rollup-plugin-esbuild": "4.8.2" to resolve this issue

in the background it uses version 4.8.2 so this is not as critical, but it might be nice to update the package either way.

Yes, I also found this problem yesterday, I am going to mention pr to solve it, just to see your issue


bartenderApi Review

Sets up bartenderAPi, and adds a removeOrder to Table for bartenderApi

Looks really good! Can't wait to see it working with the menu and menuItem class

Looks great! Good integration with Stock class


fix: Challenge solution added and broken test updated.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

The beta/rocks page, https://www.freecodecamp.rocks/learn/responsive-web-design/basic-css/use-css-selectors-to-style-elements, does not accept the solution of adding a single style element.  i.e.

CatPhotoApp
...

This PR corrects that and adds a bare-bones solution as well.

All added solutions need to include all original Challenge Seed code and necessary changes to pass solution.  Thank you.

Also, the challenge instructions are quite specific for this challenge:

and instead create a CSS style block

It does not say add more than one block.  This extra test change is not needed.

That's a bit strange, given that the rest of the solution isn't used in the tests.  Isn't including it just going to make the tests slower unnecessarily?

It does not say add one or more style blocks. This extra test change is not needed, so please remove.

Fair enough, I just figured it wouldn't hurt to make the test forgiving.  Having said that, I don't know why someone would want to add a second block. I'll make it an equality.

That's a bit strange, given that the rest of the solution isn't used in the tests. Isn't including it just going to make the tests slower unnecessarily?

It is our standard practice to include all code, so that in case any test added/changed breaks with a full working solution of the challenge, we know it has nothing to do with the solution.  It may not always been relevant with the current set of tests, but on many challenges it would be.  It is quite possible that someone might think they are add all "needed" code, but later tests added/changed could pass when they should not if the full solution was present.  This way, we don't have to worry about that scenario.

Righto.  I was operating under the assumption that the tests were basically a specification for what a solution has to be i.e. something that passes the tests!  Would it be possible to add a clarification to https://github.com/freeCodeCamp/freeCodeCamp/blob/master/docs/how-to-work-on-coding-challenges.md and/or https://github.com/freeCodeCamp/freeCodeCamp/blob/master/docs/style-guide-for-curriculum-challenges.md ?

@ojeytonwilliams Great idea.  Would you mind creating a separate issue for your request, so we don't lose track and also, we can discuss/work out the verbiage which needs to be added before creating a PR for it.

@RandellDawson Sure, I'll do that in a sec.  I've made the changes as well.

@ojeytonwilliams After thinking about it a bit more, I am reversing my original stance on allowing the multiple style elements.  I think your original idea of >= is fine.  Actually, I am not sure how campers were even getting past this test before when it was just >.

@RandellDawson Ah, that's because this bug isn't on live, just on beta/rocks.  I only stumbled across it when I was looking at something completely unrelated.  I'll change it back to >=.

I was looking at this bug, almost didn't see there was a PR already.
I didn't see an explanation for this bug so I just wanted to point out what I found.
Looks like there used to be a style element in head of the #fcc-main-frame iframe which is no longer there on master and the version on freecodecamp.rocks
The test which checks for the style element assumes the style element in head is there and is testing for $("style").length > 1 (the style in head + the style added by the camper in the challenge). I search for other challenge tests and can't find any other that is using $("style") so it looks like it only broke the one challenge.
@ojeytonwilliams @RandellDawson Anyway, would scoping the selector to the body help future proof this test a bit better?
$("body style").length > 0

@lasjorg Good catch!  I wondered why this ever used to work.
As far as future proofing goes, it should be fine as is.  Reason being, this PR adds a solution, which means this challenge will be added to the test suite.  If someone adds a style element to the test frame, the challenge seed would no longer fail the tests (which it should!) and we'd know there's a problem.

LGTM!

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


samples: nffs: New NFFS sample

This is a simple application demonstrating how to read & write persistence data on
flash of the nRF52 SoC, using NFFS file system.
Signed-off-by: vikrant8051<EMAIL_ADDRESS>new file:   samples/subsys/fs/nffs/CMakeLists.txt
new file:   samples/subsys/fs/nffs/README.rst
new file:   samples/subsys/fs/nffs/prj.conf
new file:   samples/subsys/fs/nffs/sample.yaml
new file:   samples/subsys/fs/nffs/src/main.c

Codecov Report

Merging #6155 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6155   +/-   ##
=======================================
  Coverage   52.88%   52.88%
=======================================
  Files         412      412
  Lines       40274    40274
  Branches     7801     7801
=======================================
  Hits        21297    21297
  Misses      15761    15761
  Partials     3216     3216

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 6973d13...fc7cbe0. Read the comment docs.

@vikrant8051 Please do not close this pull request.


Linux support

The reason why this is important that booting live Linux images from USB sticks is easier, and there are lots of working variants, compared to Windows PE.

Any updates on this?

I would like some support for Linux

I would like this too, hopefully I don't run into issues skipping this

I hope we can get a Linux support one hour, is so much easy to use Linux instead of the others operating systems for those type of things

I'm waiting for....


why the open document speed slower and slower？

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:02:52.799Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

csharp
ms-vscode
1.8.1

mono-debug
ms-vscode
0.15.4

Steps to Reproduce:

@pokerchina Please add more details. What files do you open? What language and what size?

Also if you are using the csharp extension, please file directly against https://github.com/OmniSharp/omnisharp-vscode

Closing after no additional info was provided.


Verification docs outdated

The verification documentation is outdated. It says to retrieve keys using contrib/builder-keys/keys.txt, however this file has been removed in favor of having the keys directly in the guix.sigs repo now. The verification docs need to be updated to reflect this.

See #807, 894, #878 #891 etc

Looks like https://github.com/bitcoin-core/bitcoincore.org/pull/878#issuecomment-1377185769 is up for grabs

In the meantime, PGP fingerprints are available here

See #807, #894, #878 #891 etc

All incorporated into incorporated https://github.com/bitcoin-core/bitcoincore.org/pull/964 now


[Perf]  Fast Generate Url for specific cases to avoid string allocations

Reference Issue #4593
loadtest with 3000 requests for the home page of the MusicStore app.
Before

After

Total savings 1.18% savings in total bytes allocated.
Saves 22 string allocations per request to the Home page.

Good perf improvement! :watch:

:shipit: after fixing comment about comments

@rynowak  I added unit tests and has taken Sebastien's advise. One final review please

:watch:

:up::date:

:shipit:

:up::date:


Extremely slow access to epochs after channel remapping.

I have two epoch lists. In one of the lists all channels have to be mirrored to the other side of the hemisphere (flipped). After that I want to concatenate both lists to one epoch file. So I'm using mne.rename_channels() to remap the channels of the one list. After that I'm using mne.equalize_channels() to get a uniform channel mapping of the two lists. After that I'm using mne.concatenate_epochs() to concat the lists. It works perfectly fine but if I try to access a condition after the concatenation in the resulting epoch instance, things become really slow. Before the access is really quick like instant. After it takes up to 20s. How could I resolve that I tried a lot already. Thank you very much!

it's really hard to debug/profile without a script to replicate.

Got it, I will post it tomorrow. Thanks for the reply.

Unfortunately, I can't replicate the issue with sample data. The reason is probably that in my data I have around 400 Event IDs whereas here one has only 4. I'm flipping 200 out of these 400 IDs. Here I'm only flipping 2 out of 4. After concatenation of the epoch list with the 400 IDs of which 200 have flipped channels, the issue arises when I try to create evokeds out of them.
The reason I'm flipping the channels is that it is a tactile paradigm and I want to pool right stimuli with left stimuli by flipping them.
The procedure here is the same as in my code:
`####
Remap-Dict
OriginalChannels = ['Fpz',
'Fz',
'FCz',
'Cz',
'CPz',
'Pz',
'POz',
'Oz',
'Fp1',
'F1',
'FC1',
'C1',
'CP1',
'P1',
'F3',
'FC3',
'C3',
'CP3',
'P3',
'PO3',
'O1',
'F5',
'FC5',
'C5',
'CP5',
'P5',
'PO7',
'F7',
'FT7',
'T7',
'TP7',
'P7',
'Fp2',
'F2',
'FC2',
'C2',
'CP2',
'P2',
'F4',
'FC4',
'C4',
'CP4',
'P4',
'PO4',
'O2',
'F6',
'FC6',
'C6',
'CP6',
'P6',
'PO8',
'F8',
'FT8',
'T8',
'TP8',
'P8',
'FT10',
'AF8',
'AF7',
'FT9']
RemappedChannels = ['Fpz',
'Fz',
'FCz',
'Cz',
'CPz',
'Pz',
'POz',
'Oz',
'Fp2',
'F2',
'FC2',
'C2',
'CP2',
'P2',
'F4',
'FC4',
'C4',
'CP4',
'P4',
'PO4',
'O2',
'F6',
'FC6',
'C6',
'CP6',
'P6',
'PO8',
'F8',
'FT8',
'T8',
'TP8',
'P8',
'Fp1',
'F1',
'FC1',
'C1',
'CP1',
'P1',
'F3',
'FC3',
'C3',
'CP3',
'P3',
'PO3',
'O1',
'F5',
'FC5',
'C5',
'CP5',
'P5',
'PO7',
'F7',
'FT7',
'T7',
'TP7',
'P7',
'FT9',
'AF7',
'AF8',
'FT10']
remappingRtoL = dict(zip(OriginalChannels, RemappedChannels))

Prepare Example Data
from mne.datasets import sample
data_path = sample.data_path()
fname = data_path + '/MEG/sample/sample_audvis_raw.fif'
raw = mne.io.read_raw_fif(fname, preload=True)
info = mne.create_info(ch_names=OriginalChannels, sfreq=500, ch_types='eeg')
events = mne.find_events(raw, stim_channel='STI 014')
event_dict = {'auditory/left': 1, 'auditory/right': 2, 'visual/left': 3,
'visual/right': 4, 'smiley': 5, 'buttonpress': 32}
raw = raw.pick('eeg')
raw.info = info
epochs = mne.Epochs(raw, events, event_id=event_dict, preload=True)

Mirror Channels
Epochs = []
epochs_L = epochs['left']
Epochs.append(epochs_L)
epochs_R = epochs['right']
epochs_R.rename_channels(remappingRtoL) # Mirror Channels
Epochs.append(epochs_R)
Epochs_eq = mne.channels.equalize_channels(Epochs, verbose=True)
epochs with right channels flipped to left side
epochs_remapped = mne.concatenate_epochs(Epochs_eq * 150) # repeating epochs to increase effect
unflipped epochs
epochs_OG = mne.concatenate_epochs([epochs] * 150) # repeating epochs to increase effect

Evoking
import datetime
unflipped
a = datetime.datetime.now()
evoked_a = epochs_OG['auditory'].average()
evoked_v = epochs_OG['visual'].average()
b = datetime.datetime.now()
print(f'Time to evoke unflipped epochs: {b-a}')
flipped
a = datetime.datetime.now()
evoked_remap_a = epochs_remapped['auditory'].average()
evoked_remap_v = epochs_remapped['visual'].average()
b = datetime.datetime.now()
print(f'Time to evoke flipped epochs: {b-a}')
`

@KSuljic you can make fake events using sample data to replicate the issue

Thank you for the hint. I could replicate the issue to some extend but in my code the difference is stronger:
Time to evoke unflipped epochs: 0:00:00.056981
Time to evoke flipped epochs: 0:00:12.620404
It seems not much but in my analysis it makes out of 0.8s -> 3.5h.
Thank you for your aid.
`####
Remap-Dict
OriginalChannels = ['Fpz',
'Fz',
'FCz',
'Cz',
'CPz',
'Pz',
'POz',
'Oz',
'Fp1',
'F1',
'FC1',
'C1',
'CP1',
'P1',
'F3',
'FC3',
'C3',
'CP3',
'P3',
'PO3',
'O1',
'F5',
'FC5',
'C5',
'CP5',
'P5',
'PO7',
'F7',
'FT7',
'T7',
'TP7',
'P7',
'Fp2',
'F2',
'FC2',
'C2',
'CP2',
'P2',
'F4',
'FC4',
'C4',
'CP4',
'P4',
'PO4',
'O2',
'F6',
'FC6',
'C6',
'CP6',
'P6',
'PO8',
'F8',
'FT8',
'T8',
'TP8',
'P8',
'FT10',
'AF8',
'AF7',
'FT9']
RemappedChannels = ['Fpz',
'Fz',
'FCz',
'Cz',
'CPz',
'Pz',
'POz',
'Oz',
'Fp2',
'F2',
'FC2',
'C2',
'CP2',
'P2',
'F4',
'FC4',
'C4',
'CP4',
'P4',
'PO4',
'O2',
'F6',
'FC6',
'C6',
'CP6',
'P6',
'PO8',
'F8',
'FT8',
'T8',
'TP8',
'P8',
'Fp1',
'F1',
'FC1',
'C1',
'CP1',
'P1',
'F3',
'FC3',
'C3',
'CP3',
'P3',
'PO3',
'O1',
'F5',
'FC5',
'C5',
'CP5',
'P5',
'PO7',
'F7',
'FT7',
'T7',
'TP7',
'P7',
'FT9',
'AF7',
'AF8',
'FT10']
remappingRtoL = dict(zip(OriginalChannels, RemappedChannels))

Prepare Example Data
from mne.datasets import sample
data_path = sample.data_path()
fname = data_path + '/MEG/sample/sample_audvis_raw.fif'
raw = mne.io.read_raw_fif(fname, preload=True)
info = mne.create_info(ch_names=OriginalChannels, sfreq=500, ch_types='eeg')
events = mne.find_events(raw, stim_channel='STI 014')
Parameters
moving = ['moving_no', 'moving_yes']
attention = ['attention_no', 'attention_yes']
phase = ['active', 'passive', 'stationary']
stim = ['single', 'empty']
side = ['side_left', 'side_right']
closeness = ['stim_close', 'stim_far']
factors = [side, phase, stim, closeness, moving, attention]
conditions = list(itertools.product(*factors))
conditions_agg = ['/'.join(c) for c in conditions]
condition_no = np.arange(1, len(conditions_agg))
condition_ids_dict = dict(zip(conditions_agg, condition_no))
import random
Events = np.empty((0,3))
for i in np.arange(20):
x = random.uniform(0.9, 1.1)
events_sim = events.copy()
events_sim[:,0] = events[:,0] * x
Events = np.concatenate((Events, events_sim))

import math
no_rep = math.ceil(len(Events)/len(condition_no))
event_ids_list = [condition_no] * no_rep
event_ids_list = np.concatenate((event_ids_list))
Events[:,2] = event_ids_list[:len(Events)]
Events = Events.astype('int')
raw = raw.pick('eeg')
raw.info = info
epochs = mne.Epochs(raw, Events, event_id=condition_ids_dict, event_repeated='drop', preload=True)

Mirror Channels
Epochs = []
conditions_to_flip = conditions_agg[48:]
for k in list(epochs.event_id.keys()):
    if k in conditions_to_flip:
        epoch = epochs[k]
        print('Flipping: ' + k)
        epoch.rename_channels(remappingRtoL)

    else:
        epoch = epochs[k]
        print('Not Flipping: ' + k)

    Epochs.append(epoch)

Epochs_eq = mne.channels.equalize_channels(Epochs)
Epochs_remapped = mne.concatenate_epochs(Epochs_eq)

Evoking
import datetime
unflipped
a = datetime.datetime.now()
evoked_l = epochs['side_left'].average()
evoked_r = epochs['side_right'].average()
b = datetime.datetime.now()
print(f'Time to evoke unflipped epochs: {b-a}')
flipped
a = datetime.datetime.now()
evoked_remap_l = Epochs_remapped['side_left'].average()
evoked_remap_r = Epochs_remapped['side_right'].average()
b = datetime.datetime.now()
print(f'Time to evoke flipped epochs: {b-a}')
`

I confirm the weirdness:
In [6]: len(Epochs_remapped)
Out[6]: 6271

In [7]: len(epochs)
Out[7]: 6271

In [8]: %timeit epochs['side_left'].average()
754 ms ± 67.3 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [9]: %timeit Epochs_remapped['side_left'].average()
3.05 s ± 174 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [12]: len(epochs['side_left'])
Out[12]: 3193

In [13]: len(Epochs_remapped['side_left'])
Out[13]: 3193

@larsoner any hint?

The issue is also reproducible without mne.rename_channels(), only by appending the epochs to a list and concatinating with mne.concatenate_epochs().

The issue is also reproducible without mne.rename_channels(), only by appending the epochs to a list and concatinating with mne.concatenate_epochs().

In that case, can you edit your post above to simplify the code? The more minimal your example is, the easier it is for us to track down the problem.

Done. It seems the issue has something to do with appending the epochs and then using mne.concatenate_epochs(). Is there any way to mirror/flip the channels in place (meaning in the original epoch instance)? Then I wouldn't need concatenate_epochs(). I tried rename_channels() on the original epoch instance and reorder_channels(). After that I tried equalize_channels() in both cases. Both times it didn't work. The channels stay the same:
for k in list(epochs.event_id.keys()):
        if 'side_right' in k:
            epochs[k].reorder_channels(RemappedChannels)

or
for k in list(epochs.event_id.keys()):
        if 'side_right' in k:
            epochs[k].rename_channels(remappingRtoLmovement)

Here is an example that can be copy-pasted:

from copy import deepcopy
import datetime
import mne
data_path = mne.datasets.sample.data_path()
fname = data_path + '/MEG/sample/sample_audvis_raw.fif'
raw = mne.io.read_raw_fif(fname).pick_types(meg=False, eeg=True).load_data()
events = mne.make_fixed_length_events(raw, duration=0.5)
epochs = mne.Epochs(raw, events, preload=True)
epochs_list = [epochs[ii] for ii in range(len(epochs))]
epochs_concat = mne.concatenate_epochs(epochs_list)
assert len(epochs_concat) == len(epochs)

#untouched epochs
a = datetime.datetime.now()
evoked_l = epochs['1']
b = datetime.datetime.now()
print(f'Time untouched epochs:    {b-a}')

#concatenated epochs
a = datetime.datetime.now()
evoked_remap_l = epochs_concat['1']
b = datetime.datetime.now()
print(f'Time concatenated epochs: {b-a}')

a = datetime.datetime.now()
deepcopy(epochs_concat.drop_log)
b = datetime.datetime.now()
print(f'Time deepcopy drop_log:   {b-a}')

The output is:
Time untouched epochs:    0:00:00.046417
Time concatenated epochs: 0:00:00.637247
Time deepcopy drop_log:   0:00:00.632559

i.e., the problem is deepcopy on a list of list of str that has lots of elements.
One fix would be to make it a tuple of tuple of str, which is immutable. I actually like this change because epochs.drop_log is not meant to be modified directly by users anyway.

(and it will speed up all epochs.copy operations, not just the one associated with epochs[...] -> epochs.__getitem__ -> epochs._getitem(..., copy=True) which is the cause of the slowdown here)

+1 for using tuples then

Thank you very much for looking into it.
Is there any direct fix I could apply now?

@agramfort @larsoner Can I fix this by myself anyhow (with tuples?) or do I have to wait for an update? Unfortunately, due to this issue my analysis came to a stop. Thank you!

You can try doing epochs.drop_log = [[]] * len(epochs) before subselection to see if it helps. If it doesn't, then you can do epochs.drop_log = []. Neither are actually correct, but they might allow you to continue your analysis until we fix the slowness.

Unfortunately, both didn't work. If I try to subselect it states:
IndexError: list assignment index out of range

I'll try to install the dev version in another conda env. It seems you fixed it there already.

In the dev version it works perfectly (roughly only double the time as you mentioned). Thank you very much!


ERROR [vpn] redirecting port in firewall: redirecting  port: redirecting IPv6 source port

URL to the Wiki page
https://github.com/qdm12/gluetun-wiki/blob/main/setup/providers/protonvpn.md
What's missing?
Hi guys,
Spent an awful lot of time searching to resolve this error.
2024-05-1615:24:40+02:00 ERROR [vpn] redirecting port in firewall: redirecting port: redirecting IPv6 source port 41874 to destination port 51413 on interface tunO: command failed: "ipotables-legacy -t nat --append PREROUTING -i tuno -d :: 1 -p tcp -- dport 41874 -j REDIRECT --to-ports 51413": ip6tables v1.8.10 (legacy): can't initialize potables table 'nat': Table does not exist (do you need to insmod?)
Everything is set up correctly, added +pmp to my username, PORT_FORWARDED on, with protonvpn and country=netherlands with PAID_ONLY=on.
Can't find a way around this. My transmission container doesn't have the peer so i'm downloading at 50ko/s... so for 4K torrent you should not be in a hurry !
Tried everything but couldn't make it work.
DS224+ up to date with gluetun container, and transmission with network_mode = contrainer-vpn

Hey,
I am having the exact same issue on both OpenVPN and Wireguard with Proton VPN.
DS920+ Synology NAS | Latest Update 5 Firmware
I have tested multiple times and it seems to only happen when I have PORT_FORWARDED=ON and specify a port VPN_PORT_FORWARDING_LISTENING_PORT=51765

Workaround: just leave VPN_PORT_FORWARDING_LISTENING_PORT blank and use whatever port you get assigned in Transmission.
Otherwise, would this help: https://stackoverflow.com/a/22876662
sudo modprobe ip_tables
sudo echo 'ip_tables' >> /etc/modules


add postgres (using new local docker flow)

Package changes:
+libpq5 9.4.8-0+deb8u1
+postgresql 9.4+165+deb8u1
+postgresql-9.4 9.4.8-0+deb8u1
+postgresql-client-9.4 9.4.8-0+deb8u1
+postgresql-client-common 165+deb8u1
+postgresql-common 165+deb8u1

@lrvick package.txt has a bunch of unrelated changes.

The only thing I did was install postgres, which pulled in all those deps automatically.

@KellerFuchs I suspect this might be an artifact of actually doing package installs on -clean- debian systems, which have not been the case in a while
For instance I realized our packages.txt never included ssh (much to my confusion) and did not get installed on build.Turns out the base image atlantic.net ships with it already installed, so we never noticed.
Perhaps there is another higher level dependency that is not on our list so apt-get just cleans up what it sees dependency packages that were never installed/required.

Perhaps something in this is derped? https://github.com/hashbang/shell-server/blob/master/setup.sh#L39-L45

@lrvick I will check when I can manage the spoons & spare time to setup Docker in Whonix.
I suspect this is the same issue as for https://github.com/hashbang/shell-etc/commit/de68cd3b5150fc250c678b2eb8a91bdf75febcc4#commitcomment-18100715

apps3 is not in use at all atm. You could test there if you like.
On Fri, Jul 1, 2016 at 12:27 PM, The Fox in the Shell <
<EMAIL_ADDRESS>wrote:

@lrvick https://github.com/lrvick I will check when I can manage the
spoons & spare time to setup Docker in Whonix.
I suspect this is the same issue as for de68cd3#commitcomment-18100715
https://github.com/hashbang/shell-etc/commit/de68cd3b5150fc250c678b2eb8a91bdf75febcc4#commitcomment-18100715
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hashbang/shell-etc/pull/100#issuecomment-230029185,
or mute the thread
https://github.com/notifications/unsubscribe/AAEOUHtexrd1Ga_WHC1EEsw1U9N-59L-ks5qRWoHgaJpZM4I-yxG
.

--
Lance R. Vick

Cell      -<PHONE_NUMBER>
Gtalk     -<EMAIL_ADDRESS>Website   -  http://lrvick.net
PGP Key   -  http://lrvick.net/0x36C8AAA9.asc
keyserver -  subkeys.pgp.net

ping


How can I ignore something?

I don't want something to receive the decal, but others need to be received.
Stencil test can achieve that?

your "don't want to receive decal" objects need to draw first and write value K into stencil,
then you render the decal, which only pass the stencil test when stencil != K

Cheers!  I Made it!


Fix bigchaidb-tendermint version mismatch

Problem: After tendermint version upgrade one line was left behind unchanged causing bigchaindb/tendermint versions mismatch in installation procedure documentation.
Solution
Changed lines containing 2.0.0b9 reference to 2.0.0.

Codecov Report

Merging #2669 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2669   +/-   ##
=======================================
  Coverage   93.46%   93.46%
=======================================
  Files          46       46
  Lines        2709     2709
=======================================
  Hits         2532     2532
  Misses        177      177


TypeError: Path must be a string. Received undefined webpack_2.2.0-rc.3

webpack snippet:
{
test: /.scss$/,
exclude: /node_modules/,
use: [
'style-loader',
'css-loader',
'postcss-loader',
'sass-loader',
{
loader:'sass-resources-loader',
options:{
resources: './src/assets/sass/_constants.scss'
}
}
When using the loader like the above, i get:
ERROR in ./~/css-loader!./~/postcss-loader!./~/sass-loader!./~/sass-resources-loader/lib/loader.js?{"resources":"./src/assets/sass/_constants.scss"}!./src/assets/sass/style.scss
Module build failed: TypeError: Path must be a string. Received undefined
at assertPath (path.js:7:11)
at Object.resolve (path.js:186:7)
at d:\DEV\monza\mugello-client\node_modules\sass-resources-loader\lib\loader.js:62:31
at Array.map (native)
at Object.module.exports (d:\DEV\monza\mugello-client\node_modules\sass-resources-loader\lib\loader.js:60:34)
@ ./src/assets/sass/style.scss 4:14-290 13:2-17:4 14:20-296
@ ./src/app/app.component.ts
@ ./src/app/app.module.ts
@ ./src/app/index.ts
@ ./src/main.browser.ts
@ multi main
After some investigation i saw that webpackConfigContext in loader.js is undefined so i changed line 61 from: var file = _path2.default.resolve(webpackConfigContext, resource);
to: var file = webpackConfigContext ? _path2.default.resolve(webpackConfigContext, resource): resource;
All is working now....Please fix it...

Please submit a PR if appropriate.

I'm kinda newbee with git....
how can i PR on this branch?
On Wed, Jan 11, 2017 at 10:44 AM, Justin Gordon<EMAIL_ADDRESS>wrote:

Please submit a PR if appropriate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shakacode/sass-resources-loader/issues/17#issuecomment-271810765,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADDSAyzOZ1OMmdO394jJ-HKZN1ZltTr4ks5rRJZmgaJpZM4LgPtw
.

--
ThanksAlon Avgil http://il.linkedin.com/pub/alon-avgil/7/89a/126

Fixed in 1.2.0.

how fix this problem???
I have some problem...

@davodaslanifakor Upgrade to 1.2.0.


Remove containedViewController when adding a new one

Would be cool if we could release this fast on CocoaPods as this issue may be blocking my team soon-ish.

✅ Done. 3.0.2 should be up now. Thanks for the PR!


Tentative update to the .NET HTML project to mark maintenance mode

Due to resource constraints we don't have capacity to support the .NET HTML package. This decision is still underway so please let us know if you heavily depend on this and/or would like to contribute to keep it going
Microsoft Reviewers: Open in CodeFlow

Docs Build status updates of commit ddc47d6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/sdk/rendering-cards/net-html/getting-started.md
:bulb:Suggestion

Details

AdaptiveCards/sdk/rendering-cards/net-html/getting-started.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ddc47d6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/sdk/rendering-cards/net-html/getting-started.md
:bulb:Suggestion

Details

AdaptiveCards/sdk/rendering-cards/net-html/getting-started.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Procedural generation of cities

Is there any tool available that allows for quick generation of cities from scratch? Blender seems to have a tool called ICity (youtube). Would it be possible to support this by importing the cities generated in this tool to Carla or having a similar tool for Carla?

We have https://carla.readthedocs.io/en/latest/adv_digital_twin/ which is very early development


Questions about Flow best practices

I have the following code.
function openMessenger() {
  return {
    type: 'OPEN_MESSENGER'
  };
}

function closeMessenger() {
  return {
    type: 'CLOSE_MESSENGER'
  };
}

module.exports.openMessenger = openMessenger;
module.exports.closeMessenger = closeMessenger;

There are a few different ways that I could add Flow types to this code and I am wondering what is considered best practice.

Use exact string values for the type of OpenMessengerAction & CloseMessengerAction.

/* @flow */
type OpenMessengerAction = {
  type: 'OPEN_MESSENGER';
};
type CloseMessengerAction = {
  type: 'CLOSE_MESSENGER';
};

function openMessenger(): OpenMessengerAction {
  return {
    type: 'OPEN_MESSENGER'
  };
}

function closeMessenger(): CloseMessengerAction {
  return {
    type: 'CLOSE_MESSENGER'
  };
}

module.exports.openMessenger = openMessenger;
module.exports.closeMessenger = closeMessenger;

Use string instead of exact value for type of OpenMessengerAction & CloseMessengerAction.

/* @flow */
type OpenMessengerAction = {
  type: string;
};
type CloseMessengerAction = {
  type: string;
};

function openMessenger(): OpenMessengerAction {
  return {
    type: 'OPEN_MESSENGER'
  };
}

function closeMessenger(): CloseMessengerAction {
  return {
    type: 'CLOSE_MESSENGER'
  };
}

module.exports.openMessenger = openMessenger;
module.exports.closeMessenger = closeMessenger;

OpenMessengerAction & CloseMessengerAction are basically the same except for the value of type so define a common type MessengerAction for them.

/* @flow */
type MessengerAction = {
  type: string;
};

function openMessenger(): MessengerAction {
  return {
    type: 'OPEN_MESSENGER'
  };
}

function closeMessenger(): MessengerAction {
  return {
    type: 'CLOSE_MESSENGER'
  };
}

module.exports.openMessenger = openMessenger;
module.exports.closeMessenger = closeMessenger;

Do you plan to mutate these objects after getting them from the function? If not, any reason not to just use raw strings ('OPEN_MESSENGER', etc) rather than an object wrapper around them?

@jeffmo these are Redux actions, the typical pattern is to have an object with a type and optionally some data. Please see here

@ccoffey got it. In that case, I think your best bet is to type each action individually and use string-literal types for the type key (AKA: option 1 in your list).
This is the only of the 3 options that will let Flow understand when you distinguish the two types later (via a switch or if-statement, etc).

@jeffmo just for completion's sake, can you please explain how option 1 is best for Flow when I have a reducer that uses the above two actions?
module.exports = function(state = initialState, action) {
  switch (action.type) {
  case 'OPEN_MESSENGER':
    return Object.assign({}, state, {isMessengerOpen: true});
  case 'CLOSE_MESSENGER':
    return Object.assign({}, state, {isMessengerOpen: false});
  default:
    return state;
  }
};

So, given that the two objects are identical except for the type, there's not actually a huge benefit in this specific scenario. However -- if ever you decide to add another action (or add any additional keys to any of the existing actions), using disjoint types lets Flow understand the difference between the shapes of the objects for each action.
So imagine this hypothetical change to the OpenMessengerAction type in the future:
type OpenMessengerAction {
  type: 'OPEN_MESSENGER',
  openedFromLink: boolean,
};

switch (action.type) {
  case 'OPEN_MESSENGER':
    // Flow will understand that action.openedFromLink is a boolean here
  case 'CLOSE_MESSENGER':
    // Flow will understand that there are *no* other keys on action other than "type" here
}


the applicability of haphic

Dear Xiaofei,
I would like to seek your advice regarding the applicability of haphic. I am currently planning to assemble the genome of a self-pollinating plant with very low heterozygosity, similar in size to the rice genome. In your experience, would you recommend using haphic for chromosome-level assembly in this case?
Additionally, if the assembly results from hifiasm are already quite close to a complete genome (around 50%-60% of the chromosomes), would you suggest first breaking the contigs with allhic before using haphic?
Thank you for your time and guidance.
Best regards,

In your experience, would you recommend using haphic for chromosome-level assembly in this case?

I usually do not consider heterozygosity as a direct reference metric because different methods and choices of homologous peaks can lead to vastly different results in heterozygosity calculation. Another important factor influencing the scaffolding of haplotype-resolved assemblies is the uniform distribution of heterozygous sites. For instance, although the heterozygosity of the human genome HG002 is relatively low, HapHiC still performs well (refer to Supplementary Figs. 53 and 54 in our paper). Therefore, I suggest starting by using HapHiC to scaffold haplotypes simultaneously. If you find that the filtering with MAPQ>=0 results in insufficient Hi-C links or if they are unevenly distributed along the assembly, then consider using HapHiC to scaffold each haplotype (hap*.p_ctg) separately. After that verify the results through mapping all Hi-C read to the combined chromosome-level haplotypes, filtering multiple mappings with MAPQ>=1, and visualizing it using Juicebox or haphic plot.

Additionally, if the assembly results from hifiasm are already quite close to a complete genome (around 50%-60% of the chromosomes), would you suggest first breaking the contigs with allhic before using haphic?

I do not recommend performing assembly correction before scaffolding for hifiasm assemblies since hifiasm usually does not produce many misjoined (chimeric) contigs. My preference is to manage these contigs in Juicebox to ensure the max length of the contig. In addition, the assembly correction function of ALLHiC is too aggressive, as shown in Supplementary Figs. 10 and 11, and Fig. 5 in our paper. Thus, I do not recommend using it in any situation. If you need to perform assembly correction in HapHiC, please use the --correct_nrounds parameter.


Process resources tweaks etc

Made changes to some of the processes resource ammounts, made the EVA scrubber more efficient, added a waste incinerator chemical process to ISRU's, added shield resource to stock nuclear parts and some antenna tweeks.

I'm trying to understand the changes in processes. I'm not saying they are wrong. On the contrary, I admit to be a complete chemistry noob, so maybe you can help me understand.
For example, I'm rechecking all the reactions on my spreadsheet and lets take the water electrolysis.
Now, going here to balance lead to:
2x H2O -> 2x H2 + O2

.
coeff
molar mass

H2O
2x
18.01528

H2
2x
2.01588

O2
1
31.9988

With these proportions, and using CRP densities for the elements, there is violation of mass conservation: input mass is 8.42% greater than output mass. I want to understand why.
Am I right in assuming that the error is in fact in the difference between CRP densities and whatever the software at the site linked is using? If not, where am I borking instead?

Regarding how you have an 8.42% input mass greater than the output is a mystery to me, I'm no chemistry professor either but as far as I know the error should not be due to the densities, though I could be wrong. I've attached a copy of my Spreadsheet for you to look at, maybe it can help you find your problem.
I was previously using real densities in the spreadsheet and have now changed over to the CRP densities.
This spreadsheet is using the CRP values -> KSP Kerbalism process calc.xlsx
In my calculations for each element in a process. I've used Avogadro's constant and Molar Mass to determine Total Mass.
I've then used a volume of 5 Litres along with density to calculate the mass of one KSP Unit of each element.
Next is to calculate how many iterations of the reaction can be produced using the entire mass of a single KSP Unit of an input resource.
Then I can derive the number of KSP Units for each element needed to satisfy the process.

Why the changes to antennas?

@YaarPodshipnik Re:

Why the changes to antennas?

The power usage of the antennas is very high, i.e. a mobile phone will use approx. 500mW (Half a Watt or 0.0005 EC) and have a +40km range.
Take Voyager-1 for example, it is nearly 20 billion km away and has a transmitter power of 20W (0.02 EC) and can still be received because of the very high gain of the radio telescope array that is used to receive the signal from Voyager.
The problem I have with the antennas in Kerbalism at the moment is the use of the term low-gain and high-gain and its association with the relay function.
The gain of an antenna is how good it is at receiving a signal, the higher the gain the lower/weaker the signal it can hear, it has nothing to do with the transmission power and relay is purely a function of the radio equipment's capability to transmit a received signal back out.
Depending on the type of radio equipment, you can receive (Rx) just like an average music radio receiver (uses gain), you can transmit (Tx) like a radio station transmitter does (uses power), or you can do both (Tx/Rx) like a HAM radio transceiver or a mobile phone, but you cant Rx and Tx at the same time with the same antenna.
Or you could look at it as a High-gain antenna is like having big ears but does not mean you have a mouth, you need power for a mouth. Or you could say a High powered antenna is a big mouth that even low-gain little ears can pickup, some can also be big mouthed and have big ears just like my ex :|

The terms low-gain / high-gain are used in this mod to indicate 'ominidirectional' / 'directional'.
I didn't come up with these terms usage either, rather it is used in general.
The origin of the usage of these terms to indicate directionality could be related to the fact that, at parity of overall capability over the sphere, the most directional antenna will have the strongest signal.
In this mod, when you switch 'relay' on in an antenna, you are enabling data reception from other vessels. Why is this only included in the 'low-gain' (omnidirectional) antennas? Because in this mod the 'high-gain' (directional) antennas are always pointed to the DSN on Kerbin/Earth ground. Therefore, it would be pointless for them to have a flag that enable data reception from other vessels.

To further clarify about the term gain (I had to re-check, I'm not an expert on these matters by any mean), this is what wikipedia says about the matter:

In electromagnetics, an antenna's power gain or simply gain is a key performance number which combines the antenna's directivity and electrical efficiency

A high-gain antenna (HGA) is a directional antenna with a focused, narrow radiowave beam width.

By contrast, a low-gain antenna (LGA) is an omnidirectional antenna with a broad radiowave beam width.

I was assuming the directionless independent gain, I was totally unaware that the specific terms high-gain and low-gain are used in general to refer to directional and omnidirectional antennas.
Anyway, I can now see why the antenna function the way they do with the relay's only on the omnidirectional (low-gain) antennas. ;)

I was checking the waste incinerator, and got totally different values:

Waste Incinerator
Resource
Weight (Kg)
Density(Kg/unit)
Units

H18C82
Waste
0.00008120
0.00075
0.108266667

O2
Oxygen
0.00022408
0.00141
0.158919425

CO2
-> CarbonDioxide
0.00029215
0.00195
0.149743960

H2O
- >Water
0.00001313
1.00000
0.000013126

The weights of the reaction components were obtained from this online calculator here, then normalized against waste production per-second per-crew.
The reaction conserve mass, that is the inputs and outputs weights are equivalent.
Your values, instead, lead to a -36% variation in mass after the reaction.

The previous comment was edited (sorry about that), there was a mistake on the spreadsheet used to check the waste incinerator values of @PiezPiedPy.
Discussion on useful bits of this commit has moved:

Waste Incinerator: #90
EVA scrubber: #91

No worries and I'll keep any commits smaller next time. Thanks for looking.


⚠ ClamAV has degraded performance

In e895109, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in ed7f21d.


Fix type definition for config object

hi 🙂
the currently released npm package is missing the runOnCompile: boolean config option, which is causing typescript errors:

created a quick PR that should fix this: https://github.com/rhlsthrm/hardhat-typechain/pull/10


[bugfix]Replace favicon icon with png, add in18 to user type field

the 'user type' in i18n has been fixed in #3022 ,thx a lot


NumberBox up/down arrows default to being disabled

Describe the bug

When using NuberBox, its increment/decrement buttons are showing in their disabled state by default.
Steps to reproduce the bug

Steps to reproduce the behavior:

Add NumberBox to a sample app <muxc:NumberBox SpinButtonPlacementMode="Inline" />
Run the app
Note that the buttons are disabled

Expected behavior

Buttons should be enabled by default, and only show as disabled when some specific condition (like getting to the min/max value) is encountered.
Screenshots

Version Info

NuGet package version: 2.5.0-prerelease.201027002

Additional context

This issue can be worked around by setting the value of NumberBox to 0 instead of leaving it unset.

@SavoySchuler Value seems to default to quiet_nan which is probably why we are seeing this behavior. Should the default be something else ?

IMO if Value isn't set, then the buttons should remain disabled. Otherwise the first interaction would then also be responsible for initializing Value.
If the buttons don't remain disabled then an assumption is being made about what the initial value should be, unless another property is added to change this.


Move testing from ba to ba2 package

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

Project coverage is 100.00%. Comparing base (8eb4730) to head (77ed207).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #646   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        32
  Lines         2757      2757
  Branches       357       357
=========================================
  Hits          2757      2757

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


TOMEE-3846 improve documentation TomEE v7.1

this is the "per-version" comparison page for TomEE v7.1
i have several questions/issues:

i lack some JSR specs number, i need to dig them, working on it, help appreciated
this comparison page would be a quasi duplicate between 7.0 and 7.1, it is ok for now.

there are 2 TomEE minors (v7.0,v7.1) for 1 Java EE (v7) its ok if it stays like that
it may be a pain later if there is again 2 TomEE for 1 Jakarta (say TomEE v10.0 and TomEE v10.1 for same Jakarta 10.0)
could this happen again ?

i would appreciate feedbacks on this suggested comparison page

is red cross a bad idea to tell EE7 does not include JSON-B ?
do i better remove the lines with red crosses (specs not part of EE7) ?

instead of making a comparison page for every TomEE minors (7.0, 7.1, 8, 9),
could i make a comparison page for every JavaEE/JakartaEE minors (7, 8, 9.1, 10.0) ?
(and put them on the main website instead)
i'm not sure whats the best option.
here is what i'am working on :

tomee.apache.org/

comparison.html
tomee-7.1/docs/comparison.html (more risk ok duplicate)
tomee-8.0/docs/comparison.html
tomee-9.0/docs/comparison.html

here is an alternative :

tomee.apache.org/

comparison.html
comparison-javaee-7.0.html (no risk ok duplicate)
comparison-jakartaee-8.0.html
comparison-jakartaee-9.1.html
comparison-jakartaee-10.0.html

Jakarta EE 8 has RPC, does Jakarta EE 9 and 11 include RPC 1.1.4 ?

@sultan on the spec version questions, some of those are a bit tricky.  For example ActiveMQ only implements JMS 1.1, however there is code in TomEE to implement some of the JMS 2.0 features.  I don't know if there are any gaps.  We're not running the JMS TCK so it's hard to say.
Going to merge these comparison PRs and if we need to adjust the versions we list, we can do that in separate PRs


Memory issue on Navigation stack, Sheet & FullSheet

@obvios Hi Eric, For Tracking Purpose i created a new issue and attaching the sample and the video for reference.
I played the app in switching the tabs and also inside tab navigation, the memory is getting increased when i go over and over
in the same screens (Attached video),
Also when i logged out, by making the login view as rootview, memory is not releasing at all and still there, Can you please have a look on this issue (Attached code as well) and let me know how to release it.
TabViewLatest.zip
https://github.com/obvios/Routing/assets/68360396/afafd855-3cfa-44d4-a3d0-2b6f9dd86279

Thank you,
Will continue investigating this.

Hi Eric, One question I have, How to use a common view in multiple tabs since the view has the router instance, needed for push and pop and we need to initialise all the router instance?I have a case which has tabs and a common view need to trigger from all below tabs. Appreciate your quick response Regards,SureshOn 15-Feb-2024, at 7:26 PM, Eric Palma @.***> wrote:
Thank you,
Will continue investigating this.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Hi,
Apologies for the late response. I will look at the issue over the weekend and get back.

Hello @Suri1044 ,
Unfortunately the Routing library does not support this use case out of the box. Currently views need to specify which Router type they use for the compiler.
The only work around I can think of is declaring two optional router properties in the view, so something like:
struct SharedView: View {
	@StateObject var routerA: Router<ExampleA>?
	@StateObject var routerB: Router<ExampleB>?

	init(routerA: Router<ExampleA>?, routerB: Router<ExampleB>?) {
        _routerA = StateObject(wrappedValue: routerA)
        _routerB = StateObject(wrappedValue: routerB)
    }

	var body:  {
		...
	}
}

You would have to add your own logic to ensure only one router instance is used in the view at any given time, depending on where it was created from. This is not the cleanest code but it should work.
Hope that helps.

@obvios  Thanks for the info.
Any update on the memory issue ?

@obvios: Hi Just checking out any updates on the above. And also Can we enhancement for deeplinks? I believe currently deeplinking is not there right and also popToParticular view also we can enhance. Just suggestions

@Suri1044 @obvios It's not a memory leak in the Library, it's something in NavigationStack.
Demo
I made a small demo to show the issue, please take a look at the code.
NavigationStackLeak.zip
https://github.com/obvios/Routing/assets/1476754/343a66cb-71aa-42ad-9297-f107b92506f2
Solution
I found an article talking about the solution for this issue.
https://yeonduing.tistory.com/47

This is an Apple bug within navigation stack. In my case, I was creating a sheet view and then navigating within it using Navigation Stack and I kept seeing memory issues. Many have logged an Apple bug for this. In my case, for the workaround, I had to create multiple individual sheets instead of navigating to different views within on sheet.

Hi @mansibarodia10057 ,
Thank you for sharing! That was very helpful


Protection against XSS by CSP & mask ReGaHss Servername

the setenv.set-response-header part will overwrite the possibly provided ReGaHss Server name on Lighttpd controlled ports
The HTTP response header will not longer contain Server: ise GmbH HTTP-Server v2.0 --> it becomes Server: Server
If a system is available on open port 80 by portforwarding, it is not longer identifiable by this special server name
Currently there are 8303 open HomeMatic systems listed: https://www.shodan.io/search?query=ise+GmbH+HTTP-Server+v2.0
It's (a little bit) harder for the attacker to find a HomeMatic system, if the Server HTTP header is obfuscated.

The CSP setting protects against XSS attacks. It needs to allow remote host *.homematic.com to be able to load version check java script from http://ccu3-update.homematic.com/firmware/download?cmd=js_check_version&........
You may adjust the host name for RaspberryMatic update check, too.

As your PR is for RaspverryMatic please provide the correct CSP setting and please provide references to appropriate documentation explaining the theory behind your changes in detail.

I don't know the update check URLs used in RaspverryMatic, as the maintainer you should adapt them.
Which theory do you mean? The configuration usage or the help of response obfuscation?
Since 1.4.46 lighttpd is able to use the setenv.set-environment() refer to
https://redmine.lighttpd.net/issues/650
https://redmine.lighttpd.net/issues/2295
https://redmine.lighttpd.net/projects/1/wiki/Docs_ModSetEnv

Thanks for your PR. After some checks I integrated it now and also enhanced it a bit.


TS-4660: Improve remap.config error messages.

Improve some of the confusion about which error message buffer to
use in remap_parse_config_bti. The final message is now constructed
to append the original failure message.

@zwoop This is a stopgap to fix the error messages until we can do the larger refactoring in TS-4660.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/426/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/320/ for details.

:+1:


DSA solution for Queue Implementation using Linked List

A queue is defined as a linear data structure that is open at both ends and the operations are performed in First In First Out (FIFO) order.

hey @RajkumarSony this is my last PR, I would appreciate if you'd merge this PR ASAP.


How can I use the secrets in my Test scope to read property from keyvault @Value("${property}")

Environment

Spring boot starter:

key vault spring boot starter

OS Type: Windows

Java version:

1.8

Summary
I cannot use the KeyVault properties in my Tests, even if i define the test for the dependency in my pom.xml
Reproduce steps
Set test  for azure-keyvault-secrets-spring-boot-starter in pom.xml
I cannot read the KeyVault property in my tests as @Value("${property}")
I get them as ${property}
Expected Results
I would need to retrieve the value from the keyvault
Actual Results
$(property} is returned

Thanks for reaching out. Could you help provide more info, such as how the tests are written or configuration are set?

I was able to get this by getting the context to test by
@SpringBootTest

Close this issue for now.


Added project link to README

I see u ninja. We are same dojo.
I retire guru, animator, artist, teacher: 3D metaverse-filmmaker... U?
What kinda ninja r u amitdutta? : )

d3 data, needs migration from csv to json.

Do u kno ... tricks?

Easy; // render checkbox left (of label), if each node (info:true), display txt:info (on checked), in float:right new displayPanel().

Medium, use NextJS to PAGINATE VIEWS into "page-oriented" urls.

EXAMPLE: /Web3/Community/Public_Good/
txt:info =  "We can do good, for a living, ... now."
GOAL: vibrant films recorded in metaverse studios.
Actual INNOVATION rises.
UR discord handle?


Debugger not showing variables

Godot version:
3.2.3 (Newest official version, Standard)
OS/device including version:
Windows
Issue description:
I have a rather complex project (4 Android Plugins, 2 GDNative scripts, plenty of classes). At some point two days ago, the Debugger stopped working - but not completely. It is hitting breakpoints, but I can see no Stackframes and no Variables. Variable inspection is not possible at all. Also stepping is not possible. I can click on the buttons, but nothing happens.
Is this a known issue? Could this be related to my project setup, or something else?
Steps to reproduce:
Have a complex project. Set a breakpoint. Code exeution stops at breakpoint - no variables, no stackframes to see, stepping does not work.
Minimal reproduction project:
I cannot provide that unfortunately - I am not allowed to upload our project. The problem seems to occur in big/complex projects, imho.

I cannot provide that unfortunately - I am not allowed to upload our project. The problem seems to occur in big/complex projects, imho.

I'm afraid there's nothing we can do without a minimal reproduction project.

Hello again, ragerding the sample project: The problem is that for "small" projects, the problem never occurred for me so far. So it's really hard to reproduce the problem intentionally.
But I found out yesterday that in some cases the debugger works again - mostly in parent classes, but never in inherited classes.  I also did some major refactoring in the beginning of the week  and created several class hierarchies. Could it be that the problem is related to inheritance? Is there already an issue for that or should I investigate further?

I'm having this problem too. My project isn't very much complex, but still I'm having a hard time to find the exact moments when breakpoints are broken like OP described.
Having breakpoints set at the very first functions (i.e. AutoLoad scripts) works fine.
@SilvanaP , is your project making heavy use of Tree/TreeItems ? I think this can be related to signals emitted from Tree (i.e. item_selected, in my case), that's when my callstack doesn't show variables

Managed to make a minimal reproduction project
Looks like it is something about set_meta on a TreeItem and using get_selected()

The null.whatever() is just to call the debugger. It doesn't show variables if line1 is a meta of line1.
If line1 is a meta of line2 and line2 is a meta of line1 (cross-referencing), it also breaks debugger.
if only line2 is meta of line1 or only line1 is meta of line2, debugger shows variables.
debugger_broken.zip

@rilpires Hello, I am not using any Tree / Treeitems at all in my project. I still have the problem and suspect that making heavy use of custom classes or inheritance at some point broke the debugger...but this is only a guess, as I also observed that breakpoints at the very first functions (Autoloads, MainScene.gd) work.

I was about to open a very similar bug. I noticed this was reported multiple times overs the years for example here: #10703
Anyway, I also created a demo for it happening. It's different from what @SilvanaP, but I guess, it could be very related.
Here we go:
Godot version:
3.4.4 (Standard)
OS/device including version:
Windows 10
Issue description:
Local variables won't show up in the debugger for certain code.
Minimal reproduction project:
I took my complex project and cut it down until I ended up with a VERY simple piece of code which triggers the problem:
extends Node

func _ready():
	var visibleVar1 = 1
	var visibleVar2 = "test"
	if visibleVar1 > 0:
		var invisibleVar1 = 0815
		var invisibleVar2 = "invisibleTest"
		return invisibleVar2 # BREAKPOINT HERE, you won't see invisibleVar1 or invisibleVar2 in locals
	else:
		pass # if you remove the else it doesn't happen

Heres a demo project with it happening right a away. Just one empty scene and one gd file:
DebuggerBug.zip

I had the same problem like SilvanaP described. I found the solution and described it here:
https://github.com/godotengine/godot/issues/5510#issuecomment-1302908859


Backport save_widget_settings with 3 arguments from RELENG_2_3

Commit 6f1410582412fe771f51bd8b67bcbb952da97db6 introduced code relying on this.

Is there any call on RELENG_2_3_2 using 3rd parameter?


Exit retry handler earlier if context is done

Retry handler doesn't currently handle context being cancelled or expiring during the retry delay. This can lead to situations where the handler sleeps for a long time due to a long Retry-After value but then exits with a context deadline expired or context cancelled error without actually sending a retry
This PR updates the logic for the sleep to exit early if the context is marked as done

@microsoft-github-policy-service agree company="Alcion"


Design QA: Robot Settings Page

https://github.com/Opentrons/opentrons/issues/10106 (to merge in with this ticket)

#10263


Remove the 'report a vulnerability'

Is this a regression?
Yes
Description
The security issue should be reported using an email, following the rule of Hyperledger labs.
Please provide the exception or error you saw
When creating a new issue, there is 3 choices: bug, feature and report a security issue.

The report of security shall be removed from the choice.

Please provide the environment you discovered this bug in
Not applicable

Anything else?
I cannot find where this is triggered in the code. I thought it was in the .github directory but it seems not there.

@essbante-io , could you provide the reference to the rules of HL to follow?

@mkbreuningIOHK, One of the recommendations, is to Open a GitHub security vulnerability report.

Open a GitHub security vulnerability report: Open a draft security advisory on the “Security” tab of this GitHub repository. See GitHub Security Advisories to learn more about the security infrastructure in GitHub.

We have already covered this process in the security.md file, so I believe we are good.
We may need to review the contents of the security.md file as we work towards becoming a Hyperledger project. Please see the full guideline at https://toc.hyperledger.org/governing-documents/security-template.html.

@mineme0110 , this is ci so I ping you here: do you know how to update this? I am trying to find a way from https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/about-issue-and-pull-request-templates but it does not explain why when I click on 'New issue' I will see 3 choices whereas in the ISSUE_TEMPLATE directory, there are only 2.

@mkbreuningIOHK I am not admin, So I dont have access to settings, I need the access then I can look

As aligned with @essbante-io , this issue can be closed as the 3 options lead to the same interface.


Revert "[Re-land] Exposed tooltip longPress"

Reverts flutter/flutter#118796
See b/267566560. This broke some semantics tests.

This was already reverted in https://github.com/flutter/flutter/commit/1ee87990dc76044ba4b23395eb64ea1416f9c85c


add the FAQ link next to the marianne

On Tchap web v2
Since there is no more homepage, the link to the FAQ has been deleted.
If the FAQ button is placed next to the Marianne, it will be clearly visible

I can do it.
But FAQ & CGU are already present in "Settings" > "Help & About" (the link is presently pointing to riot.im but I will change it).
Do you really want to me to add it ? I can add GCU & user guide link too if you want.
Do you agree wit something like that :

Done.


bundle install will not trigger hooks to rehash

You can see what's wrong in https://github.com/ccmywish/rbenv-for-windows/blob/main/share/rubygems_plugin.rb

Fixed in https://github.com/ccmywish/rbenv-for-windows/releases/tag/v1.4.6


Rails 5.1: Index Creation fails

I kept getting errors  on 'db:migrate' like:
-- add_index(:rapidfire_questions, :survey_id)
rake aborted!
StandardError: An error has occurred, this and all later migrations canceled:

Index name 'index_rapidfire_questions_on_survey_id' on table 'rapidfire_questions' already exists

I ended up commenting out all the add_index lines.

Thanks for the report @drernie We will look into this.

Thanks.  FYI, I've also submitted a PR with all my changes for 5.1, in case
it helps.
On Thu, May 18, 2017 at 10:32 PM, Yuva<EMAIL_ADDRESS>wrote:

Thanks for the report @drernie https://github.com/drernie We will look
into this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/code-mancers/rapidfire/issues/101#issuecomment-302612761,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABNT-QRL7pxr4ADEkORGw5PgX32Ggnuks5r7SltgaJpZM4NeyyK
.

This issue has been resolved #118.


Update npm package versioning script to be compatible with other packages

Description
Currently, the protocol-devchain-anvil CI script publishes the devchain-anvil package with a version equal to the PR number. This can cause issues, as PRs are not always merged linearly. (e.g.: when PR#11034 was merged after PR#11035).
The proposed changes will use the determine-release-version script to determine the version.

I just noticed this PR re-introduced the hardcoded Foundry version in the protocol-devchain-anvil.yml workflow, instead of fetching the version from a global env variable. See git blame. I missed that in the review, is that intentional?
https://github.com/celo-org/celo-monorepo/blob/3a0a771cac2917a703f48a02e5a0859bb0e62344/.github/workflows/protocol-devchain-anvil.yml#L94


Fix how external tests clean up Docker images

External test jobs have been temporarily disabled from upstream pipeline. However if build job is triggered directly (not by pipeline) with default test list ( "sanity.openjdk", "sanity.system","extended.system", "sanity.perf", "sanity.external", https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-x64-openj9/configure) https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-x64-openj9/1103/ , external tests job will be triggered. https://ci.adoptopenjdk.net/view/Test_external/job/Test_openjdk11_j9_sanity.external_x86-64_linux/515/console
External test jobs story has been changed to run as parallel ( or later dynamic?), the old default configuration doesn't set as parallel, which means the job will run all enabled external sanity jobs. However the task  docker_prune will recursively run and delete all earlier generated docker images and failed the job. https://ci.adoptopenjdk.net/view/Test_external/job/Test_openjdk11_j9_sanity.external_x86-64_linux/515/console
To avoid the confusion and prepare for re-enabling external tests, two steps can help

Update ci-pipeline-jenkins job config_regeneration to be consistent with nightlyDefault.
Regenerate external test jobs to be latest and corresponding update to  parallel or dynamic.

@LongyuZhang noticed the failure in both adoptium and openj9 and questioned if any changes in external tests caused the failure

Some more details about the external tests failure:

the docker image for a specific test (e.g. quarkus_quickstarts-test) was built successfully (e.g. adoptopenjdk-quarkus_quickstarts-test:11-jdk-ubuntu-openj9-full), but when building the next test (e.g. scala), its docker prune step deletes docker image for previous tests (e.g. quarkus_quickstarts-test), and caused testing for previous test (e.g. quarkus_quickstarts-test) failed since it cannot find the docker image.
An example on Jenkins pipeline is here, with the following console output.

08:50:17       [exec] Successfully tagged adoptopenjdk-quarkus_quickstarts-test:11-jdk-ubuntu-openj9-full
08:50:17
08:50:17  dist:
08:50:17       [copy] Copying 2 files to /home/jenkins/workspace/Test_openjdk11_j9_sanity.external_x86-64_linux/jvmtest/external/quarkus_quickstarts
08:50:17
08:50:17  build:
08:50:17
08:50:17  move_scripts:
08:50:17       [copy] Copying 1 file to /home/jenkins/workspace/Test_openjdk11_j9_sanity.external_x86-64_linux/jvmtest/external/scala/dockerfile
08:50:17
08:50:17  docker_prune:
08:50:18       [exec] Deleted Images:
08:50:18       [exec] untagged: adoptopenjdk/openjdk11-openj9:ubuntu-nightly
08:50:18       [exec] untagged: adoptopenjdk/openjdk11-openj9@sha256:d61b310bd5fee13c708c20bbdc38ca3a25d2415a1a212fb71b5e7e10bff206a7
08:50:18       [exec] untagged: adoptopenjdk-quarkus_quickstarts-test:11-jdk-ubuntu-openj9-full

FYI @llxia @smlambert

Steps:

remove depends="docker_prune" in https://github.com/adoptium/aqa-tests/blob/master/external/build.xml#L60
add docker_prune  in https://github.com/adoptium/aqa-tests/blob/master/external/build.xml#L41
run Grinder with TARGET=sanity.external on xlinux

i can take this on


Created a Custom moveVelocity System and Added Double Overload for lemlib::avg()

Summary
(a) Created a custom moveVelocity function for motor groups to replace the flawed VEX move_velocity. It uses a TBH controller and the move voltage function inside a task, with a different task for each motor group.
(b) Added a double overload for lemlib;:avg()
Motivation
(a) Sets up #4 to be solved when unit system is implemented, and directly solves #102
(b) Makes working with motor groups easier
Test Plan
(a) Test on robot when accessed
(b) Doesn't really need it
Additional Notes

The tasking could go very wrong
Values like kI and take-back value probably need tuned
PIDs set up but not implemented

Download the template for this pull request:

[!NOTE]
This is auto generated from Add Template to Pull Request

via manual download<EMAIL_ADDRESS>via PROS Integrated Terminal:

curl -o<EMAIL_ADDRESS>https://nightly.link/LemLib/LemLib/actions/artifacts/1072356048.zip;
pros c fetch<EMAIL_ADDRESS>pros c apply<EMAIL_ADDRESS>rm<EMAIL_ADDRESS>

Can you merge this into the refactor branch instead? That's where development is focused atm

Shoot, I might need to change some stuff to work with the refactor then.

You have some merge conflicts that need to be solved b4 this can be merged

Yea I'm going to work on it today, it needs to be reformatted for the refactor.

TO-DO:

abstract settle util
abstract controller (VAP)
gain schedular (steal #110 implementation somehow)


Add support for comment and label manifests.

Changes
Introduces .MANIFEST files to comment and label directories to keep
track of sub-resources when the PullRequest resource is initialized.
This allows us to keep track of what resources the user explicitly
deleted, and not accidentally overwrite new comments/lables that were
introduced during execution.
Minor changes included:

Changed fake GitHub server to make copies of PullRequests to avoid
pointer collision across tests.
Changed uploadLabels to Add/Delete labels explicitly instead of
replacing all.

Change manifest path to .MANIFEST to avoid collisions.
Fixes #1286
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
- Fixes bug where PullRequestResource could accidentally delete newly created upsteam resources in certain race conditions.

/hold
Found some issues with the manifest, so this is not ready for review yet. :sweat_smile:

/hold cancel

/lgtm

Squashed the commits in prep for submission.

/lgtm

/lgtm


Windows builds go OOM and need to be redone

This is a known issue and is "resolved" for PRs only by rerunning them until they pass

Maybe @h-vetinari could take a look at that? Have seen this problem before or are you familiar with this? It seems to be recurrent on polars-feedstock.
Relevant:

https://github.com/conda-forge/polars-feedstock/issues/96
https://github.com/conda-forge/polars-feedstock/pull/87
https://github.com/pola-rs/polars/issues/6624

@timkpaine could you describe in more depth the problem?

Since #121 was closed (seemingly) without action, I wanted to kindly ask again whether the Windows builds for 0.17.2 can be retried 😅

Since #121 was closed (seemingly) without action, I wanted to kindly ask again whether the Windows builds for 0.17.2 can be retried 😅

Sure, go for it. It's unclear to me how you retrigger the builds, and even if you did, polars is released several times a week. A one time retriggering of an individual release doesn't sound like a valuable exercise vs trying to fix the underlying issue.

#129 does not make any meaningful progress either. Since the Linux builds don't seem to be affected: can we use Linux runners to cross-compile the Windows packages? I don't have any experience with setting this up for feedstocks, can someone weigh in here, potentially @xhochy?


Fixes isBuiltInComponent check.

Previously it detected components which included built-in component tags as substrings. E.g. it detected 'my-transition-component' as a built-in.

Thanks, much appreciated!


Semantics related to transient transitions

Bug or feature request?
Seems like a bug because the logic (or ordering) of execution of actions and guards seems wrong.
Description:
Imagine the following simple statechart: it has two states 'a' and 'b' and one conditional eventless transition going from 'a' to 'b'. I would expect the abMachine to land in state 'b' after it is being started, but it stays in state 'a'. Seems like that eventless transitions's guard is executed before entry action of state 'a', which seems wrong to me, as it doesn't reflect the logic of the state machine.
  const abMachine = Machine({
    id: 'abMachine',
    initial: 'a',
    context: {
      value: 0
    },
    states: {
      a: {
        entry: ['aEntry'],
        on: {
          '': {
            target: 'b',
            cond: 'isReadyForB'
          }
        }
      },
      b: {
        entry: ['bEntry']
      }
    }
  }, {
    guards: {
      isReadyForB: (context) => {
        console.log('isReadyForB');
        return context.value > 0;
      }
    },
    actions: {
      aEntry: (context) => {
        context.value++;
        console.log('a');
      },
      bEntry: () => {
        console.log('b');
      }
    }
  });

(Bug) Expected result:
After starting the abMachine I would expect it to land in state b, since the entry action of state 'a' increases the context value from 0 to 1, and then the eventless transition's guard should evaluate to true (since now the context.value is greater than 0) and move the machine to state 'b'.
(Bug) Actual result:
The abMachine stays in the state a, since it seems like the eventless transition's guard is evaluated before the entry action of state 'a', which is semantically wrong. If we are leaving the state it is expected to be able to see the results of state's entry (and exit) actions in the guards that follow, right?
(Bug) Potential fix:
Not really confident about the fix.

You are mutating context, which you should not be doing. Please use assign() and see the docs on how to properly update context: https://xstate.js.org/docs/guides/context.html
If you still are having issues after making the above change, please create a CodeSandbox (XState template) reproduction of this, and we can reopen.
Thanks!

Thanks David, I've tried with updating the context using assign and it works nicely now. Would you be so kind to tell me why is context access (actually context mutation) designed this way, just curious...

Why it is immutable? This prevents bugs (as you have seen), if that's what you were asking.


Will not work with Rails 3.1.x

The gemspec requires:
s.add_dependency 'activerecord', ['~> 3.0.0']
Which prevents using this gem with Rails 3.1.x (current release)
If the gem is otherwise compatible using:
s.add_dependency 'activerecord', ['>=  3.0.0']
Might do the trick.

Same problem in me


small

Is possible to small set?
no server, no files (only add binary data blob) no sqlbase and no curses?
simple main loop
:volume increase,
:pause
:stop
:seek forward
:next track, :set track :set playlist
nothing more

Maybe you're better server with cmus or SoX.

It's possible to use the musikcore library to implement something like this, but it doesn't currently exist and I don't have plans for it. Like ghost mentioned, cmus may be a good alternative for you.