Fixing Bugs in xGroup Resource (Fixes #180)

This should fix the Administrator issues occurring in #175 as well as the issue occurring in #180.
I made several small (but important) bug fixes:

Test-TargetResource will no longer check for group members if they are not specified.
You can now specify only one member.
Set-TargetResource no longer uses group.Clear() when using the Members parameter, so the built-in Administrator account should not complain if it is being included properly.
Get-Group will now properly return null when the specified group does not exist rather than throwing an error.
Length --> Count for arrays. No functionality change. They are the exact same thing, but in case one variable gets overwritten with a string by some weird accident, the script will properly throw an error since a string does not have Count but it does have length.
More null checks

I added 1 new unit test to make sure Set-Target resource will properly create an empty group, and I also added 2 new integration tests to check for these issues, one that creates a new empty group, and one that ensures that the built in Administrator (username 'Administrator') is in the Administrators group.

This change is

@PlagueHO Can you please look this over before I merge it?

Looking great to me! I've put in comments on the bugs I found. You could include the fixes in your PR if you want or I could submit another one after yours. If you want me to submit my fixes in another PR, give me a shout. After you've merged this PR I'll look at adding some more tests (although the coverage is getting much better now).
I did also want to do a bit of work bringing the code into line with the styleguidelines, but if you're working on that then I won't. Feel free to let me know what parts I can help out on!

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 373 [r1] (raw file):

                $whatIfShouldProcess = $pscmdlet.ShouldProcess(($LocalizedData.GroupWithName -f $GroupName), $LocalizedData.SetOperation)

                $actualMembersAsPrincipals = Get-MembersAsPrincipals `

This line has one of the bugs I found: if Get-MembersAsPrincipals only returns a single record it won't be an array and so the .Count property used later on won't exist. I resolved by wrapping the cmdlet with an @( ) to explicitly cast to an array.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 753 [r1] (raw file):

                    -Credential $Credential )

                if (($null -eq $expectedMembersAsPrincipals -xor $null -eq $actualGroupMembers) -or (($null -ne $expectedMembersAsPrincipals -and $null -ne $actualGroupMembers) -and $expectedMembersAsPrincipals.Count -ne $actualGroupMembers.Count))

This line makes my brain hurt :smile: Any chance we could split it with back ticks to under 100 chars to make it easier to read? Although I think the logic is sound.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 778 [r1] (raw file):

        elseif ($PSBoundParameters.ContainsKey('MembersToInclude') -or $PSBoundParameters.ContainsKey('MembersToExclude'))
        {
            $actualMembersAsPrincipals = Get-MembersAsPrincipals `

This line has one of the bugs I found: if Get-MembersAsPrincipals only returns a single record it won't be an array and so the .Count property used later on won't exist. I resolved by wrapping the cmdlet with an @( ) to explicitly cast to an array.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 2303 [r1] (raw file):

    if ($GroupName.IndexOfAny($invalidCharacters) -ne -1)
    {
        ThrowInvalidArgumentError -ErrorId 'GroupNameHasInvalidCharacter' -ErrorMessage ($LocalizedData.InvalidGroupName -f $GroupName, [String]::Join(' ', $invalidCharacters))

I found a bug in this line: ThrowInvalidArgumentError does not exist - should be:
New-InvalidArgumentException `
            -ArgumentName 'GroupNameHasInvalidCharacter' `
            -Message ($LocalizedData.InvalidGroupName `
                -f $GroupName, [String]::Join(' ', $invalidCharacters))

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 2320 [r1] (raw file):

    if ($nameContainsOnlyWhitspaceOrDots)
    {
        ThrowInvalidArgumentError -ErrorId 'GroupNameHasOnlyWhiteSpacesAndDots' -ErrorMessage ($LocalizedData.InvalidGroupName -f $GroupName, [String]::Join(' ', $invalidCharacters))

I found a bug in this line: ThrowInvalidArgumentError does not exist - should be:
New-InvalidArgumentException `
            -ErrorId 'GroupNameHasOnlyWhiteSpacesAndDots' `
            -Message ($LocalizedData.InvalidGroupName `
                -f $GroupName, [String]::Join(' ', $invalidCharacters))

Comments from Reviewable

Review status: all files reviewed at latest revision, 6 unresolved discussions.

README.md, line 347 [r1] (raw file):

    * Fixed logic bug in MembersToInclude and MembersToExclude.
    * Fixed bug when trying to include the built-in Administrator in Members.
    * Fixed bug where Test-TargetResource would check for members when none specified.

Also, if you include my fixes, this is the Readme.md changes I put in:

     Fix bug in Test-TargetResourceOnFullSKU function when group being set to a single member.

     Fix bug in Set-TargetResourceOnFullSKU function when group being set to a single member.

     Fix bugs in Assert-GroupNameValid to throw correct exception.

Comments from Reviewable

Review status: 2 of 4 files reviewed at latest revision, 6 unresolved discussions.

README.md, line 347 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
> Also, if you include my fixes, this is the Readme.md changes I put in:
> +          Fix bug in Test-TargetResourceOnFullSKU function when group being set to a single member.
> +          Fix bug in Set-TargetResourceOnFullSKU function when group being set to a single member.
> +          Fix bugs in Assert-GroupNameValid to throw correct exception.

Done.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 373 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
> This line has one of the bugs I found: if Get-MembersAsPrincipals only returns a single record it won't be an array and so the .Count property used later on won't exist. I resolved by wrapping the cmdlet with an @( ) to explicitly cast to an array.

Done. I also added the array wrapper around any other calls to Get-MembersAsPrincipals that I found.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 753 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
>  This line makes my brain hurt :smile: Any chance we could split it with back ticks to under 100 chars to make it easier to read? Although I think the logic is sound.

Simplified it. The null checks here are actually no longer needed since we now have array wrappers around the calls to Get-MembersAsPrincipals (which populates both these variables), so I just removed them.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 778 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
> This line has one of the bugs I found: if Get-MembersAsPrincipals only returns a single record it won't be an array and so the .Count property used later on won't exist. I resolved by wrapping the cmdlet with an @( ) to explicitly cast to an array.

Done.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 2303 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
> I found a bug in this line: ThrowInvalidArgumentError does not exist - should be:
> ```powershell
> New-InvalidArgumentException `
>             -ArgumentName 'GroupNameHasInvalidCharacter' `
>             -Message ($LocalizedData.InvalidGroupName `
>                 -f $GroupName, [String]::Join(' ', $invalidCharacters))
> ```

Done.

DSCResources/MSFT_xGroupResource/MSFT_xGroupResource.psm1, line 2320 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
> I found a bug in this line: ThrowInvalidArgumentError does not exist - should be:
> ```powershell
> New-InvalidArgumentException `
>             -ErrorId 'GroupNameHasOnlyWhiteSpacesAndDots' `
>             -Message ($LocalizedData.InvalidGroupName `
>                 -f $GroupName, [String]::Join(' ', $invalidCharacters))
> ```

Done.

Comments from Reviewable

@PlagueHO Is this good to go?

Hi @kwirkykat, sorry,  I forgot to do this last night. I'll review when I get to work in about an hour. Sorry about the delay.

 Thanks @KwirkyKat!

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


New icon: Cryptohopper

[X] I have carefully read and followed CONTRIBUTING document.

URL of the site: cryptohopper.com

Proof of the Aegis-compatible 2FA: https://docs.cryptohopper.com/docs/user-profile/two-factor-authentication-2fa/

Logo / icon info: https://www.cryptohopper.com/about/press

Background color info: Background color is from the logo (#00B2C8)

The site's logo was way too wide, but this is their main logo and favicon so I had to work with it despite it being literally just a circle. If there's any ideas to make this look better, I'm all for it.

Just for heads up, please check up first for the press page before taking logo from header (as already instructed on the CONTRIBUTING), since Cryptohopper has one with logos and everything.
These can have important information about logo usage, colors ect.

I rather have logo / icon info URL to press page, if it has info about logos: https://www.cryptohopper.com/about/press

I rather have logo / icon info URL to press page, if it has assets and/or info about logos: https://www.cryptohopper.com/about/press

I actually had no idea there was a press page, my bad.

I rather have logo / icon info URL to press page, if it has assets and/or info about logos: https://www.cryptohopper.com/about/press

I actually had no idea there was a press page, my bad.

It's usually at the bottom of the page, that's the first thing I look before anything else. Then I check the top navigation.
Also I recommend using icon grid (can be found from "_templates" dir), I haven't written the documentation yet for it, but it has the now standard size for 1:1 circle / square logos.

I rather have logo / icon info URL to press page, if it has assets and/or info about logos: https://www.cryptohopper.com/about/press

I actually had no idea there was a press page, my bad.

Link is usually at the bottom of the page if there's one, that's the first thing I look before anything else. Then I check the top navigation.
Also I recommend using icon grid (can be found from "_templates" dir), I haven't written the documentation yet for it, but it has the now standard size for 1:1 circle / square logos.

Okay I'll use that from now on!


Implement after() for analytics whenever possible

https://nextjs.org/docs/app/api-reference/functions/after
Note: only after next's been upgraded to 15+

awaiting for next 15 PR

get pros and cons between waitUntil and after and use cases


Prevent global functions from overriding completion of subscript

Prevents global functions (clamp, ceil, abs etc) from overriding completion of subscripted class functions with the same names.
Before:

After:

Bugsquad edit:

Fixes #58403

Thanks!

Actually I may have merged a bit too fast, see issue #58403 and PR #59261 where that bug had been discussed. Seems like just reordering the checks might not be the perfect solution here, to be re-discussed with @vnen.


Added upgrade note about default options for message console command

Docs fix related to #18941

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues

Thanks.


How to display an asset name ?

Can we display an asset name in https://pyportfolioopt.readthedocs.io/en/latest/_images/ef_plot.png ?

That isn't supported by default, but it's a reasonable feature request!

I will try and handle this!


Hello ! I added MD file and user profile in jpg format

i added myself, review PR #270

Thank you for merge but this repo is exclued with respect to hacktoberfest so, it will not be counted .

@sanamkandar yes, this is true, I realised this. #271 . See if this label  helps.

@CodingSpecies  so should i again created PR ?

No, this repo isn't part of hacktoberfest, https://github.com/Binary-Cumates/Hacktoberfest-21-Open-First-PR/issues/271 this issue explains. Sorry for the confusion. @sanamkandar

It's okay no worries!


state transition issues

Hello there, I have an application that so far has been using regular ionic transitions. I just discovered your library and it looks very promising.
My app uses $state.go, and ionic state triggers (beforeEnter/etc) extensively in many cases. What I am seeing is that the states get stuck in transitions in display, also in many cases transitioning between menu to views get stuck. I am testing on and Android 5.0 Moto G
all I have done so far is this:
In index.html
 <script src="lib/ionic-native-transitions/dist/ionic-native-transitions.min.js"></script>

added in angular.module
'ionic-native-transitions'

I also have
$ionicNativeTransitions.enable(true, false);

in app.js in device ready
Looking at logs, I see the native transitions being called. I don't understand if I need to specifically add code to $state.go or whether this should be automatic. Can you advise?
I am looking at the bare minimum required to replace all ionic transitions with this plugin.
thanks

whoops - it was an error in my code. For whatever reason having this code in some of my controllers was messing up native transitions - in controllers that had the code, the view never got  removed but the new view went to the bottom and if you switched to landscape mode, the view at the bottom would show up while the top view showed in portrait orientation :-)
$controller('zmApp.BaseController', {
        $scope: $scope
    });


Custom base path on FastAPI server

Hello everyone,
We have a situation here in which we are deploying our LLMs behind a ALB on AWS using path prefix, in this case it is deployed on a path that looks like my.domain.com/my-model. By default ALBs on AWS doesn't support path rewrite so it will forward the request to the API server including the /my-model on it, breaking the API because this URL doesn't exist there given that the API only listens in the /generate path. I implemented a custom API server here, but I think we could add an optional parameter (or ENV VAR) that we can use to set the base path of the server (or keep the default behavior if it is not there). I can open a PR doing this, but I wanted to confirm if this is something that makes sense for other people before I actually do this.

any update here?

FastAPI supports this through the --root-path parameter, but it'd have to be exposed via fastAPI.
https://fastapi.tiangolo.com/advanced/behind-a-proxy/
Rand into the same issue.  Here's how I run a similar fastAPI service behind ALB
      command: [ "uvicorn", "api.main:app", "--host", "<IP_ADDRESS>", "--port", "8000", "--root-path", "/basepath" ]

Would need to get this parameter propogated intot he FastAPI server.

@hughesadam87 I've been using kubernetes so I just changed my route to be my-model.mydomain.com instead of api.mydomain.com/my-model. No need to create another ALB.  Another option is to do what I did and to create a custom python server that is identical to the one used by vLLM and just change the routes to have your prefix, it worked too.
Ideally we could just have an env var on VLLM that adds the prefix to the routes when set


PYIC-7482 Add nested journey for KBVs

Proposed changes
What changed
Pull all KBV journeys into a nested journey
Why did it change
The KBVs are highly duplicated, with 6 different copies - making it quite well-suited to pulling out into a nested journey.
One consideration is that this ties our KBV priority together - we'll have to add duplication back in if we want different priorities depending on the route.
Screenshots
KBV routing with HMRC/DWP disabled:

KBV routing with HMRC enabled:

KBV routing with DWP enabled:

P2 identity:

P1 identity:

The KBV CRI states don't have use the evidence_requested parameter, instead it is calculated from the targetVot. This means P1 & P2 journeys can both use this shared journey and they will both request the correct verification score (1 & 2 respectively).
Using targetVot is a bit of a latent method of keeping track of which journey you are on - and actually could be used to deal with the CHECK_FRAUD_SCORE issue from WEB_DL_OR_PASSPORT? I think it'd be good to choose one method of containing the P1/ P2 choice, so if targetVot isn't suitable then maybe it isn't for KBVs either?


Run time crash while trying to get model from model zoo

I have installed the requirements as mentioned in the beginner tutorial in  Google Colab and once I am running the following lines runtime is crashing:
from d2go.model_zoo import model_zoo
model = model_zoo.get('faster_rcnn_fbnetv3a_C4.yaml', trained=True)
Any ideas ?

plz see #141


Completed Task 849 : Design System > Element > Progress Bar > Default : Enhancements in style

Description
1] update the style of the default single progress bar,
2] percentage shifted to right hand side of progress bar as per wireframe

Fixes # (issue)

New style for default state implement as per wireframe

Type of change

Please delete options that are not relevant.

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

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.

Test A
Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Colovariant list also updated from colorvariant.ts file. now we can see two colors only that is primary and error


Add nikhil96.json

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ x] All new and existing tests passed.
[ x] This PR does not contain plagiarized content.
[ x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Deleted the nikhil96.json file.


Cannot edit styles in chrome devtools

Sometime the css styles becomes uneditable in chrome devtools like this:

Instead of being editable as usual like this:

Steps to reproduce:

Use exploring
Open a file using a css file with chrome
Open devtools
Update a style from that css file
Open the css file in your editor
Save file to trigger livereloading
When you inspect element again it is disabled.

This seems caused by a feature of chrome devtools called local overrides. Chrome devtools remember the changes made on the css file and when iframe is replaced chrome devtools are in strange state where it seems unhappy that there is unsaved local overrides.
This situation can be fixed by refreshing the page.
It can also be prevented by setting up a folder where to save chrome local overrides.

Should be fixed now that exploring reloads the whole page on file change + file is execute on top level window instead of an iframe.


Allow v-if with transition

Added a willRender prop to social-sharing comp, and key prop to network comps
for easily transitioning the social sharing icons with the normal Vue v-if and transition options.

well I tried.. no idea what the current lint errors mean. sorry.

Hello @eybarta,
I fixed the build on the master branch but your PR still breaks it, as the willRender property default to false the component isn't created anymore in the tests.
Couldn't you just wrap this package components in any HTML tag to add this kind of behavior like:

  <network network="line">
     Line
  </network>

Or does this fail to meet your use case?

@nicolasbeauvais hey man, thx for responding.
First, you're totally right about the default, it should be TRUE, incase you don't use the feature..
Regarding the solution you suggested with wrapping spans around every network, I think the downside
of this approach is adding additional unneeded markup to the template, especially if there are many
networks. Also, I tried to test this approach with my code instead of the 'will-render' and the wrapped
network component never rendered regardless of the state of the bool prop.. so I'm not sure if I'm doing something wrong or this doesn't work for some other reason.
So, I do think the ability to transition in/out the social share icons is very nice to have feature and easy
to implement with Vue.. I like my implementation because it's just adding few lines of code to the plugin and I was able to use 1 prop to nicely stagger animate my social icons in.
Except for changing the default value for the prop, do you see any other problem with it?
If you insist wrapping spans solution is better, could you show me a working example (jsfiddle) ?
thx a lot for your work bro!


Does not Dump offsets

When i start the dumper it dumps everything except the offsets. #

Look inside of the generated folder for offset/{} files. You can also run the binary with the -v flag to see more detailed log messages.


fix: Add 1 sec wait before sending second build request

Description
Adding 1 sec wait before sending second build request, which will avoid the issue occurred here
Issue ticket number and link
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/test all

/approve
/lgtm


Fix selecting network

Fixes https://github.com/oasisprotocol/oasis-wallet-ext/issues/180

can I ask though, from a react newb: why do we have these two copies of the network list, one in the props and one in the state?

It seems just so that netConfigAction appends key, value fields to items for <Select options=

ah for a change in format then. thanks for the explanation


fix album/track assignment queries

Resolves #143
Small bug in the code for load.py that meant the pipeline runs were always failing. This bugfix replaces bad table and column names with correct ones.

Updated branch but they don't conflict


Don't special-case the spelling of Api.

According to @jorendorff, Rust best practices are to avoid capitalizing acronyms.

This is documented here:

In UpperCamelCase, acronyms and contractions of compound words count as one word: use Uuid rather than UUID, Usize rather than USize or Stdin rather than StdIn. In snake_case, acronyms and contractions are lower-cased: is_xid_start.

Examples in the standard library include OsString, TcpListener, IpAddress, Cow, AsciiExt, Utf8Error. The ecosystem really does follow this rule: the uuid crate really does use Uuid; hyper has Http instead of HTTP; git2 has Oid and Odb; and so on.


[WIP] Feature: Python script and module to check dataset readiness for data preservation

Description

This is WIP. Do not merge!
This is a new feature to support preparation for preservation checks.

Closes #235
ToDo List
@astrochun:
As of 07/26/2021, here is the current status of this PR:
While most methods have been tested at a minimum level for the preservation check workflow, it is still not fully tested/explored. The methods of Preserve that have been tested include: get_metadata, save_metadata, check_files, make_symbolic_links, delete_old_readme_files, delete_hidden_files, delete_files (used by delete_hidden_files).
I have not tested:

[ ] The Preserve.update_files method
[ ] I have not done full end-to-end testing with instances where there is (1) no issue (i.e., curation version matches published version), (2) cases where the are differences between curation version and published version (we were not immediately aware of such a version for testing).
[ ] I have not also tested this against dataset that does not have data (e.g., NPN datasets). Presumeably the --metadata_only option should work for this purpose.

In addition, we should have the script run as a "dry run" by default and to have a --update option. This will ensure that files are not replaced and to not download metadata. This would be a keyword input option that would need to be available to all method, probably through a (self) instance variable in Preserve.
Test plan

Update Changelog

[ ] CHANGELOG.md

Resources

Screenshots or additional context

Preservation work is now in the ReBACH  repo. Closing this PR


“check-translator” CI is failling with 403 status code

It looks like all the last PR that ran “check-translator”, that CI is failing with
    url: 'https://api.github.com/repos/expressjs/expressjs.com/issues/1613/labels',
    status: 403

@UlisesGascon worked on it, can you confirm if it's a real bug/issue, please?
few examples:

https://github.com/expressjs/expressjs.com/pull/1623
https://github.com/expressjs/expressjs.com/pull/1619
https://github.com/expressjs/expressjs.com/pull/1613
https://github.com/expressjs/expressjs.com/pull/1620

Hi, this is an issue that already has an open discussion. If you have any ideas that could help resolve it, feel free to comment on #1553.


Page animations

I've been checking closed issues if they'd lead me in the right direction, however looking at a few I've still been hitting some hurdles.  In my containers/App/App.js I've wrapped my this.props.children within my ReactCSSTransitionGroup, however I haven't been able to get it to work. Styles related to animation have been wrapped within the :global{} in App.scss.  Any ideas of what I may be doing wrong here? Thanks guys!
            <ReactCSSTransitionGroup
              transitionName="moveLeft"
              transitionEnterTimeout={500}
              transitionLeaveTimeout={500}
              >
              {this.props.children}
            </ReactCSSTransitionGroup>

Most probably your styling is not align, check that both the actual rendered style and component class style name are aligned

Turned out I needed to add a component="div" for whatever reason within the grouping.

hi jasonjchang, can u give me a small demo code for transition. It would be helpful i m unable to achive it.|
`

          {this.props.children}

      </ReactCSSTransitionGroup>
    </div>`

this is like i used in mine app.js file. But it not working

@AnuragSinghTomarr sure. so within app.js seems like you'll have to stipulate a key as well.
        <ReactCSSTransitionGroup
          component="div"
          transitionName="moveLeft"
          transitionEnterTimeout={500}
          transitionLeaveTimeout={500}
          >
          {React.cloneElement(this.props.children, {
            key: this.props.location.pathname
          })}
        </ReactCSSTransitionGroup>

Also make sure that your CSS3 page animations are within your global scope if you are using css-modules

Keep in mind that my transition name is moveLeft please change it to your respective css animation class.

Also due to the boiler plate linter, you may also need to add this in your propTypes
    location: PropTypes.shape({
      pathname: PropTypes.string
    })

As i tried yours and it worked i am thankful to you. Want to know that
 {React.cloneElement(this.props.children, {             key: this.props.location.pathname           })}
why we need to clone  element. If instead of this use  {this.props.children} than the animation does not works.

It doesn't have to do so much with React.cloneElement() as it does providing the key attribute.  There are other ways of implementing without React.cloneElement() but I just decided to stick with the example pattern provided by react-router.
For more reference:
https://facebook.github.io/react/docs/animation.html
Skip down to the red Note:

I wanted to use the low level API for transition in react i.e. ReactTransitionGroup
`  render() {
const {user} = this.props;
const styles = require('./App.scss');
// console.log(React.DOM);
return (
    <ReactTransitionGroup transitionName="example">
      <div className={styles.appContent}>
        {this.props.children}
      </div>
    </ReactTransitionGroup>

);

}`
this is the code in app.js, but i am getting error as  Warning: React.createElement: type should not be null, undefined, boolean, or number. It should be a string (for DOM elements) or a ReactClass

Has anyone figured out how to use ReactTransitionGroup in this project? Would also really like to use the low level API.
Any Help Greatly Appreciated.


Jenkins improvements for local testing.

Setup a docker-compose environment to run jenkins and have a couple build options to build with local (current) changes. I'm also setting up/testing some changes for loading the Jenkinsfile from the repo.
Testing
manual testing
Check List

[ ] New functionality includes testing

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The conents of this PR were merged in another squashed merge by mikaylathompson.


Unable to install Scipacy using anaconda navigator in Mac

I am using Mac with latest Catalina version 10.15.3.  Anaconda is suggesting using the terminal option under environment i dedicated to scispacy.  However, when I start the terminal under that environment.  All i got is below message that i cannot proceed further.
Last login: Fri Feb  7 17:11:38 on ttys000
ka-mantse@Ka-mans-MBP ~ % /Users/ka-mantse/.anaconda/navigator/a.tool ; exit;
/Users/ka-mantse/.anaconda/navigator/a.tool: line 1: syntax error near unexpected token `('
/Users/ka-mantse/.anaconda/navigator/a.tool: line 1: `bash --init-file <(echo "source activate /opt/anaconda3/envs/scispacy;")'

[Process completed]

I encountered no such problem using Windows.  I am confused.

Hi, this appears to be a problem with anaconda and the terminal you are using, rather than a problem with scisapcy - maybe take a look at this issue:
https://github.com/ContinuumIO/anaconda-issues/issues/1831
If you can't fix it and figure out that it is a scispacy issue directly, please re-open!


Finds 0 tests when using parameterized test

I have a particular test case that overrides defaultTestSuite, this seems to trip up xctool, works with xcodebuild.
When running with xctool, I get 0 tests. Other tests from different schemas run fine.

Unfortunately xctool relies on default XCTest framework defaultTestSuite behavior. If you use it to create tests on the fly, it is your responsibility to return same set of test cases when xctool queries them.

Could you please point me to the direction on what additional I might have
to do to "conform" to the default behavior? I am already returning existing
test suite with additional tests.
-John
On Tue, Jun 7, 2016 at 3:30 PM Aleksey Kononov<EMAIL_ADDRESS>wrote:

Unfortunately xctool relies on default XCTest framework defaultTestSuite
behavior. If you use it to create tests on the fly, it is your
responsibility to return same set of test cases when xctool queries them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/xctool/issues/701#issuecomment-224434292,
or mute the thread
https://github.com/notifications/unsubscribe/AAk6dEbvpwLHwOD-kHYTSRMTl9V-MteWks5qJfEJgaJpZM4Iwa3U
.


Added Lowest Common Ancestor Of Binary Tree Cpp

added #1614
Description
Added Lowest Common Ancestor Of Binary Tree Solution In C++
Fixes #1614
Type of change

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

Checklist:

[ ] My code follows the style guidelines(Clean Code) of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@Amit366 please review

@sonamgupta136 this is similar to https://www.geeksforgeeks.org/lowest-common-ancestor-binary-tree-set-1/

U can send another PR free from plagiarism.


Improve the format of the emails being sent

Make the subject line unique for each email, so Gmail won't group them all together.
Use the reply-to header, so replies will go to the user instead of to the club's gmail.

emails should be sent using a managed framework. this will make them more accessible and less spam looking.
there are a few good open source project to help send  emails that conform to standards

I'll consider it. What do you have in mind?

Look into using django-mailer for our email backend, so emails can be batched periodically instead of on demand.
Also find (or create) a email sending framework, so we aren't constantly maintaining HTML and TXT versions of each email template.

Original issue fixed by commit 402d370257. We'll open a separate to-do item for using a mailer framework.


Support for new Kiosk link format

Hi,
Huawei has recently changed the format of the Kiosk link, now it's like:
https://eu5.fusionsolar.huawei.com/pvmswebsite/nologin/assets/build/index.html#/kiosk?kk=XXXXX
Any plans to support that change?

Actually, the link is still the same AFAIK, if you monitor the traffic in a browser you can see that it does a call on the "old" link. Hence I guess something else is the issue here. Unfortunately I am not savvy enough to debug python applications.

Hi,
I have tried to put Kiosk url into iframe card but doesn't show.
I have 5 entitates and I use them in entitate card.
What is the problem?

Is this addon still working?

It works fine for me, I had some issues in the beginning but then it started working all of a sudden. Not sure what I did / if I did anything, to make it work though.

I've added the custom repo in HACS as an integration, restarted, opened and installed version 2.0.0 from there. I then go into integrations to add this, cannot find it in the list.

I integrated too.
You can see in config file.
I put the entities into entities or glance card.
After that I add an iframe card and I put the kiosk url, but doesn't work.
What did I miss?
On Thu, 1 Jul 2021, 09:28 punjabiz, @.***> wrote:

I've added the custom repo in HACS as an integration, restarted, opened
and installed version 2.0.0 from there. I then go into integrations to add
this, cannot find it in the list.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tijsverkoyen/Home-Assisant-FusionSolar-Kiosk/issues/4#issuecomment-871997607,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJEZZT54ATL4XUMEDBMMJCDTVQKJFANCNFSM44K5G3PQ
.

Ratkai Imre

I integrated too.
You can see in config file.
I put the entities into entities or glance card.
After that I add an iframe card and I put the kiosk url, but doesn't work.
What did I miss?

I integrated too.
You can see in config file.
I put the entities into entities or glance card.
After that I add an iframe card and I put the kiosk url, but doesn't work.
What did I miss?

Seems like your integration works then? As for the iframe-card I have no idea, I have not tried this out.


Need to upgrade to GeoServer 2.12.4

This new release includes upgrades to some libraries that have known vulnerabilities. GeoWave should upgrade to this newest version to benefit from those changes. Their release notes indicate: "This release is made in conjunction with GeoTools 18.4 and GeoWebCache 1.12.4." Not sure if GeoServer uses either of these, but if they do, then they should all be upgraded together.

We released v0.9.8 with updated support for GeoServer 2.13.2 and GeoTools 19.2.


Moved the Base64 alphabet into PROGMEM to save RAM

Storing the whole alphabet in RAM is not needed since it does not change. The string is quite long and thus uses up a lot of valuable RAM.

Good point. Not sure why I didn't do that before.


Fix @mention notifications

I'm not sure I understand your comment. The terminators are relevant both before and after the @mention. Prior to this change a mention not followed or preceded by a space would fail.

I'm not sure I understand your comment. The terminators are relevant both before and after the @mention. Prior to this change a mention not followed or preceded by a space would fail.

My apologies... Just wanted to make sure.
Thank you for your efforts!


v22.7.1 convert AppKernel into PHP7

Current coverage is 99.54% (diff: 100%)

No coverage report found for master at 506ee97.

Powered by Codecov. Last update 506ee97...b64d83a


[Question] is this component need use arc?

i don't found "alloc" used in your code?
is that necessary to use arc?
do i miss some thing?

@sanhue0302 interface to the BZIP2 algorithm in this source code is via the singleton pattern. We don't make any instances of classes and no objects. That's why there's no alloc

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here. Hurry, applications close on March 6, 2024.
Job Description:

Position: Senior Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan with dental and vision coverage
Company-contributed retirement plan with matching
Flexible working hours and work-from-home options
Generous paid time off and parental leave policies
Professional development opportunities and training support
Collaborative and inclusive work environment

Requirements:

Minimum of 5 years of experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Extensive experience with frameworks such as React, Angular, or Vue.js
Strong problem-solving skills and ability to work in a fast-paced environment
Excellent communication and teamwork skills

Note: This position is open to candidates from anywhere in the world. Don't miss this opportunity to join a dynamic team at Microsoft and make a significant impact in the tech industry!
no-decaf, @TheDeveloperFarm, @hubingliang, @pcranaway, @cbin, @huzaifamk, @mrmean15, @VozmakGervant, @DNPZ91TSN72L, @timleader, @jdickman00, @MarkVillacampa, @anorudes, @0xKMG, @nickbytes, @muturgan, @jamesblackwell, @arden, @gabrieledm, @gringonivoli


Add app central integrations to product list

Purpose of this pull request
This pull request adds Integrations in App Central to this article:
https://help.sumologic.com/docs/integrations/

Per Asana ticket "Add automation integrations to the Product List article"
Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions, .clabot
[x] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

PLEASE READ
Following a recent back-end update, all contributors with a local clone or fork of our repository are required to run yarn install. This does not apply to direct page edits.

[x] Yes, I've run yarn install
[ ] No, does not apply to me

This is a work in progress.


Updated grep command for 4.5.0 in docker-image.yml

Can one of the admins verify this patch?

Not required


Blacklist check to make blacklists command independent

Summary
A blacklist check where you would be able to put a list of userids into and those people wouldn't be able to use the command.
Problem
This makes blacklists command independent and you won't have to check in the command itself.
Ideal implementation
@checks.blacklist(user_ids=[...])
Checklist

[x] I have searched the issue tracker and have made sure it's not a duplicate.
[x] If it is a follow up of another issue, I have specified it.

The implementation might get a bit messy if you are going to add support for Guild, Channel and Role IDs

Why? Can't you just add extra arguments? (didn't think of guild and role ids before but seems like a good idea)

I don't think the implementation would get too messy.

ability to pass a callable that returns a sequence of ids here would be more useful - although at that point its almost just a custom check, but I don't see much use for it other than for initial development but you can just use owner_only()

ability to pass a callable that returns a sequence of ids here would be more useful - although at that point its almost just a custom check, but I don't see much use for it other than for initial development but you can just use owner_only()

premium bots

premium bots

even so, most bots aren't premium and a callable would be more useful

I think being able to pass a callable could be useful

may as well just use a custom check at that point


[QA] Adapter refurbish required

[ ] adapter should use adaptercode instead ob liob/utils
[ ] code should use let/const instead of var
[ ] lint should be activated
[ ] problems reported by adapter checker should be solved
[x] missing dependencies (fs, http, path) should be added

done with re-new


Running error

Hi! I run the runscript.py and return errors. Is there any solutions? Thanks
Traceback (most recent call last):   File "/home/os/yuehuang/MMASim/runscript.py", line 6, in <module>     from main import Auction   File "/home/os/yuehuang/MMASim/main.py", line 250, in <module>     model.step()   File "/home/os/yuehuang/MMASim/main.py", line 217, in step     self.schedule.step()   File "/home/os/.local/lib/python3.10/site-packages/mesa/time.py", line 123, in _wrapped_step     self.model._advance_time()   File "/home/os/.local/lib/python3.10/site-packages/mesa/model.py", line 122, in _advance_time     self._steps += 1 AttributeError: 'Auction' object has no attribute '_steps'. Did you mean: 'step'?

Try a lower version of python (3.8 or 3.9) and a lower version of Mesa (before 2.0.0).

thanks, my problem is solved!


[Flake] Interop ALTS java->go server time out

https://source.cloud.google.com/results/invocations/45c5d7af-84de-4ac3-8f27-ed0acb9b4c6d/targets/grpc%2Fcore%2Fpull_request%2Flinux%2Fgrpc_interop_alts/log
This looks very similar to #15434
2019-02-20 12:16:43,025 Running test ping_pong
Exception in thread "main" java.lang.AssertionError: Timed out waiting for response
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.assertTrue(Assert.java:41)
	at org.junit.Assert.assertNotNull(Assert.java:712)
	at io.grpc.testing.integration.AbstractInteropTest.pingPong(AbstractInteropTest.java:725)
	at io.grpc.testing.integration.TestServiceClient.runTest(TestServiceClient.java:269)
	at io.grpc.testing.integration.TestServiceClient.run(TestServiceClient.java:212)
	at io.grpc.testing.integration.TestServiceClient.main(TestServiceClient.java:70)
Shutting down
2019-02-20 12:16:43,025 FAILED: cloud_to_cloud:java:go_server:ping_pong:alts [ret=1, pid=2259, time=7.2sec]

Spotted again in the wild: https://source.cloud.google.com/results/invocations/c2359a46-d3d2-49a1-a5e8-9bdd5a24767b/log

based on go/grpc-flaky dashboard, I don't think we are seeing this anymore

How flaky is the test? The timeout for this test is 5 seconds, so there is a very small chance that timeout could happen.

Two instances since May 14th that we've noticed. I don't have a number to put on it at the moment.

Another instance: https://source.cloud.google.com/results/invocations/6dc06a00-8d77-469c-979b-da6ace70fdd8/log

another instance: https://source.cloud.google.com/results/invocations/29f20b8e-1031-490a-bc0d-edb3a12be523/targets/grpc%2Fcore%2Fpull_request%2Flinux%2Fgrpc_interop_alts/log

@temawi would be the candidate to triage the issue.

@temawi   I'm guessing this is stale, as we've just gone through a fixit sprint, but I'll let you evaluate and decide.

I went through a bunch of the most recent failures for this target and this error did not appear. I believe this one is stale and I'll close it.

This test has been failing for about a week again with the same error log. For example: https://source.cloud.google.com/results/invocations/6a70fbad-1029-4fad-b80a-f930ab46f5f2/targets/grpc%2Fcore%2Fpull_request%2Flinux%2Fgrpc_interop_alts/log

As we decided some time back, we don't want to track interop flakes here. (It should go to internal buganizer.)


5.3.x x509 rest backport

this is an attempt at cherry-picking the PR#3457 commit 4fccc264826114e50ec48538ec322b0fd87f3e84 from master, and backporting onto 5.3.x while ensuring everything it touches still builds/checkstyles/findbugs cleanly.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Curtis W Ruck seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks excellent, thank you.
One small failure:
org.apereo.cas.support.x509.rest.X509RestHttpRequestHeaderCredentialFactoryTests > createX509Credential FAILED
    java.lang.AssertionError
        at org.junit.Assert.fail(Assert.java:86)
        at org.junit.Assert.assertTrue(Assert.java:41)
        at org.junit.Assert.assertFalse(Assert.java:64)
        at org.junit.Assert.assertFalse(Assert.java:74)
        at org.apereo.cas.support.x509.rest.X509RestHttpRequestHeaderCredentialFactoryTests.createX509Credential(X509RestHttpRequestHeaderCredentialFactoryTests.java:48)

...and minor comments here and there.

I can't figure out why the test is failing, and for some reason gradle is skipping tests locally...

OK, no worries. I'd wait for Travis CI to run through this batch once more if it still fails, we'll merge anyway and look into it later. Thanks!

Yea, even updating the test and adding an @ToString shows the test "should" be succeeding.  its just a simple instanceof test like the other tests class.

org.apereo.cas.support.x509.rest.X509RestHttpRequestHeaderCredentialFactoryTests > createX509Credential FAILED
java.lang.AssertionError: cred is not right type: X509CertificateCredential(certificates=[[

You may want to update the assert statement to indicate the credential type received; that way you can possibly try to figure out the reason for the mismatch.

Also, shouldn't be assertTrue instead? You want to make sure the credential is in fact of that type, and else throw an error, right?

your right, it should be assertTrue, on both test cases.


Module.exports = {} not recognized by IntelliSense

TypeScript version: 3.5.1
The CommonJS construct module.exports = {} does not provide import suggestions in JS files via VSCode IntelliSense.
IntelliSense provides a TS error handle on the module.exports = {} construct:

File is a CommonJS module; it may be converted to an ES6 module.ts(80001)

Pressing the Quick Fix option converts the exports to ES6 format. After this, IntelliSense import suggestions work in other scripts.
What also works is formatting the export as follows:
module.exports.showText = showtext;

My files are:
lib.js
function showText() {
  console.log('Text');
}

module.exports = {
  showText: showText,
}

myCode.js
showTe...;  // <- no import suggestion provided

Project structure:
./lib.js
./myCode.js

I would like to get the IntelliSense import suggestion without change of the module.exports = {} construct (I do not have the authorization to do this for the entire project code base).
I am not using a jsconfig.json file.

For what it's worth, I'm seeing this same issue with the same version of TypeScript in a different type environment.
Specifically, the exports are defined in this declaration file. I'll copy that line here for convenience:
export = {shell, storage};

The coding environment is JavaScript with a jsconfig.json file (uxp.d.ts). The following code causes the issue:
/** @type {import('uxp').storage.Folder} */
let folder;

Specifically, VSCode underlines the storage portion of that in red and reports the following error:

"Namespace .../uxp..export=' has no exported member 'storage'.

Which... clearly it does.
I have a jsconfig.json file configured in the project but none of the module-related settings I've tried changes the outcome.

As this issue is tagged as "Awaiting more feedback", I will chime in.
I imported a fairly large project into VSCode that makes extensive use of the module.exports = { ... } syntax. I was very disappointed that the IntelliSense autocomplete appears unable to index those exports. Refactoring the entire project to conform to the parser is not a palatable option.

Facing the same issue in our project where module.exports is extensively used with spread operator.
index.js
module.exports = {
   ...require('./module1'),
   ...require('./module2')
}


Datapath: 1.14 Pre-Release Regression Testing

This issue is to track pre-1.14 regression testing for datapath throughput and latency. The goal is to perform a check to ensure that Cilium with a default install does not contain a regression between 1.12, 1.13, and 1.14-pre.
Goal: Compare throughput and latency on a default helm install of Cilium between 1.12, 1.13, and 1.14-pre.
Non-Goal: Gather independent, publishable numbers regarding throughput and latency performance.
Test Environment: Two-node GKE cluster.
Test Setup: Pod-to-pod netperf test for TCP stream, CRR, and RR with the same number of parallel connections as cores.
CC: @aanm, @marseel, @borkmann

I got a first round of results today. I had trouble finishing the CRR tests due to an issue with the conntrack table filling up on each node. I haven't dug into the results yet, I'll do that next week and report the findings here.
Here are the results from each test. Note that the CRR and RR tests I ran only gave percentiles, so rather than averaging them, I'm taking the results from the netperf instance with the largest 99th percentile.
CRR Latencies (ms) by Version (lower is better)

Version
50th
90th
99th

1.14.0-rc.0
848
2536
8380

1.13.4
864
2357
8073

1.12.10
762
2319
7080

RR Latencies (ms) by Version (lower is better)

Version
50th
90th
99th

1.14.0-rc.0
144
216
467

1.13.4
149
235
504

1.12.10
148
226
513

Stream Average Throughput (10^6 bits/s) by Version (higher is better)

Version
Throughput

1.14.0-rc.0
992

1.13.4
987

1.12.10
991

This looks like it is within parameters for the v1.14.0 release. Thanks for testing @learnitall!


⚠ Bofrost.at has degraded performance

In 53d260d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1220 ms

Resolved: Bofrost.at performance has improved in f258e78.


Provide configurable options to make "parameter select" report smaller via Border options

Is your feature request related to a problem? Please describe.
Parameter select should be smaller or allow to control their size better.
Describe the solution you'd like
A way to control the size would be to have a configurable top/bottom/left/right border in px
Other types of reports already offer this so we should align.
Describe alternatives you've considered
None possible
Additional context
Add any other context or screenshots about the feature request here.

We have plans to refactor Parameter Select to a dedicated area in the reports, making them easier to use, and taking up more space.
That is planned as part of the next step of building NeoDash's new design using Needle.
So we will close this, unless you think there is a quick temporary fix (I'm not sure what you meant about the configurable top/bottom/left/right border in px in other reports, can you explain ?)


FATAL ERROR

FATAL ERROR:
/home/BLaDE-master/src/main/blade.cxx:23
Running with 20 omp threads but only 1 GPUs
Hello, I have encountered the above problem. Can you provide some suggestions？

Yes, BLaDE tries to find a GPU for every OpenMP thread, and crashes if it cannot. Launch it with fewer OpenMP threads. In you bash script, precede your blade command with
export OMP_NUM_THREADS=1
If that doesn't solve your problem, let me know.


Title garbled in Time-series Table

A clear and concise description of what the bug is.
Expected results
the title of Time-series Table displayed correctly.
Actual results
Title of Time-series Table garbled.
Screenshots

How to reproduce the bug

create a Time-series Table
See error

Environment
(please complete the following information):

superset version: 0.28.1
python version: Python 3.4

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x]  I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x]  I have reproduced the issue with at least the latest released version of superset.
[x]  I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

is this issue fixed? we are using superset version built from the master branch of july, same issue

same issue  +1

chart edit
http://yourhostname/chart/edit/{chart_id}
edit Parameters   key

Hi ,
@fazhang  the problem is you have to compile production build
npm run build

same issue +1


Wassaf/341 newsletter page

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

[x] Changes have been manually tested

What are the relevant tickets?
(Required)
Closes #341
What's this PR do?

Adds newsletter basic data
Adds mailchimp form
Adds mailchimp css overrides

How should this be manually tested?

Open /newslestter
Verify newsletter is designed as per invision
Verify newsletter form works/

Any background context you want to provide?
A few issues

The current form does not have a reCaptcha embedded in it.
The error and success messages should be designed in mailChimp designer as they are dynamically inserted using JS

Screenshots (if appropriate)

@pdpinch
FYI

The current form does not have a reCaptcha embedded in it.
The error and success messages should be designed in mailChimp designer as they are dynamically inserted using JS

The current form does not have a reCaptcha embedded in it.

That's fine.

The error and success messages should be designed in mailChimp designer as they are dynamically inserted using JS

Can you add some screenshots of the current state of these messages? I'm hoping they are good enough for now.

@pdpinch Updated screenshots

@pdpinch Redesigned footer

@pdpinch Redesigned error and success messages.

Styling looks fine to me. Did you have to change that at mailchimp.com, or were you able to override it in the page?

Styling looks fine to me. Did you have to change that at mailchimp.com, or were you able to override it in the page?

I dont have access to this particular form i was able to override it the css from my side.
its does have some code duplication which i was hoping to avoid.


Type ...Non-Micorosoft

To make Non-Micorosoft apps compatible with the SMS sign-in feature
should be
To make Non-Microsoft apps compatible with the SMS sign-in feature

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

ID: 5ee9d7f4-1d8d-adc5-c00b-ddccd0225328
Version Independent ID: 4eb876ad-f2b6-997f-5413-90205504c34f
Content: App support for SMS-based authentication in Azure Active Directory
Content Source: articles/active-directory/authentication/how-to-authentication-sms-supported-apps.md
Service: active-directory
Sub-service: authentication
GitHub Login: @AanjuSingh
Microsoft Alias: justinha

@gbms311 Thank you for your feedback. We will check and update the thread.

@gbms311 Thank you for your contribution to enhance the documentation. I have updated the document. Once changes will be approved, same will be reflected in the document.


Support recursive search for tfstate in s3 bucket?

I am testing steampipe against tfstates.
I am managing multiple tfstate files in one s3 bucket.
After testing, it seems that formats such as "$bucketname/**/*.tfstate" are not supported.
In practice, we manage a lot of tfstates, and it would be very helpful if recursive search is possible.
Found some test cases which looks related to this https://github.com/turbot/steampipe-plugin-sdk/blob/develop/plugin/get_source_files_test.go#L165
But it never work as I expected

Hi @keyolk,
I apologize for the delayed response.
Here are a few key points for you to review:

This plugin supports searching Terraform resources from an S3 bucket.
Have you had a chance to look at the plugin documentation?
I tested it locally with .tf file resources stored in an S3 bucket, and I got the expected results using the following configuration in the terraform.spc file:

connection "tf_bucket" {
  plugin = "terraform"

  configuration_file_paths = [
    "s3::https://test-delete90.s3.us-east-2.amazonaws.com//**/*.tf?aws_profile=default"
  ]
}

I haven’t yet tried it with a tfstate file in an S3 bucket. Could you please give it a try and let us know if you encounter any further issues?
Thank you!

@ParthaI Hi! basically, it works. But extreamely slow and most of case, it returns error like below
steampipe query "select name, type, mode, address, arguments, attributes, path from tfstate.terraform_resource where mode = 'managed' limit 10" --output json
Error: tfstate: runtime error: index out of range [0] with length 0 (SQLSTATE HV000)

Any idea?

And seems it just follow AWS_REGION env even I set aws_region in its file_path which make much confusing

Hello @keyolk, thank you for testing these things out!

steampipe query "select name, type, mode, address, arguments, attributes, path from tfstate.terraform_resource where mode = 'managed' limit 10" --output json
Error: tfstate: runtime error: index out of range [0] with length 0 (SQLSTATE HV000)

It seems the error occurs while parsing the content of a specific state file. It would be great if you could figure out which state file content is failing.

Could you kindly share the connection configuration details (with any sensitive information removed, of course) from the terraform.spc file?
How many state files do you currently have in your bucket? If it's a large number, could you try testing with a smaller set of state files?
Additionally, could you please clarify what you meant by "And it seems to just follow AWS_REGION env even though I set aws_region in its file_path, which is quite confusing"?

The information you provide will greatly assist me in replicating the issue you're encountering.
Thank you!

it took about 70sec on my m1 macbook
connection "tfstate" {
  plugin = "terraform"

  state_file_paths = [
    "s3::https://<mybucket>.s3.<my_region>.amazonaws.com//<my_object_prefix>/**/terraform.tfstate?aws_profile=<my_profile>&aws_region=<my_region>"
  ]
}

$ time steampipe query "select name, type, mode, address, arguments, attributes, path from tfstate.terraform_resource where mode = 'managed' limit 10" --output json
...
real    1m14.606s
user    0m4.186s
sys     0m4.547s

$ steampipe --version
Steampipe v0.23.5

$ du -chs state.json
256K    state.json
256K    total

for single file like the above it can get result but takes too much time about 80 sec
$ find . | grep tfstate | wc -l
      45
$ du -chs *
...
4.2M    total

And the above case it ahs 45 tfstate with size of 4M
it got error
Error: tfstate: runtime error: index out of range [0] with length 0 (SQLSTATE HV000)

Also some error logs from plugin.log
2024-09-10 08:26:05.015 UTC [WARN]  initializePlugin failed: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference (0x140005f36c0)
2024-09-10 08:26:05.019 UTC [WARN]  PluginManager Get failed for hub.steampipe.io/plugins/turbot/aws@latest: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference (0x14000d10c00)
2024-09-10 08:27:11.809 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>220: readAndStreamAsync failed to read previous rows from cache: context canceled publisher tfstate-1725956798220 (tfstate-1725956798220)
2024-09-10 08:27:11.809 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>220: QueryCache AbortSet - aborting request  with error set request has no subscribers (0 subscribers) (tfstate-1725956798220)
2024-09-10 08:27:11.809 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>220: queryData.streamRows returned error: context canceled
2024-09-10 08:27:11.809 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>220: executeForConnection tfstate returned error context canceled, writing to CHAN
2024-09-10 08:28:19.991 UTC [WARN]  failed to set connection config for hub.steampipe.io/plugins/turbot/aws@latest: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference
2024-09-10 08:28:19.991 UTC [WARN]  initializePlugin failed: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference (0x14000bb6100)
2024-09-10 08:28:19.995 UTC [WARN]  PluginManager Get failed for hub.steampipe.io/plugins/turbot/aws@latest: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference (0x14000ba6108)
2024-09-10 08:28:58.695 UTC [WARN]  failed to set connection config for hub.steampipe.io/plugins/turbot/aws@latest: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference
2024-09-10 08:28:58.695 UTC [WARN]  initializePlugin failed: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference (0x14003566cc0)
2024-09-10 08:28:58.697 UTC [WARN]  PluginManager Get failed for hub.steampipe.io/plugins/turbot/aws@latest: setAllConnectionConfigs failed: runtime error: invalid memory address or nil pointer dereference (0x14000f240a8)
2024-09-10 08:29:31.244 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>334: readAndStreamAsync failed to read previous rows from cache: context canceled publisher tfstate-1725956938334 (tfstate-1725956938334)
2024-09-10 08:29:31.244 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>334: QueryCache AbortSet - aborting request  with error set request has no subscribers (0 subscribers) (tfstate-1725956938334)
2024-09-10 08:29:31.244 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>334: queryData.streamRows returned error: context canceled
2024-09-10 08:29:31.244 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>334: executeForConnection tfstate returned error context canceled, writing to CHAN
2024-09-10 08:30:18.276 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]  failed to delete the temp directory /var/folders/7g/1dw9yqgj4mzcrfgcsqj9vs6w0000gq/T/steampipe-plugin-terraform: unlinkat /var/folders/7g/1dw9yqgj4mzcrfgcsqj9vs6w0000gq/T/steampipe-plugin-terraform/tfstate/2024-09-10T08:30:13Z/aws/chat-dev/apne1: directory not empty
2024-09-10 08:31:33.674 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.674 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: streamRows execution has failed: tfstate-1725957057522 - calling queryCache.AbortSet (tfstate: runtime error: index out of range [0] with length 0)
2024-09-10 08:31:33.674 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryCache AbortSet - aborting request  with error tfstate: runtime error: index out of range [0] with length 0 (1 subscriber) (tfstate-1725957057522)
2024-09-10 08:31:33.674 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: queryData.streamRows returned error: tfstate: runtime error: index out of range [0] with length 0
2024-09-10 08:31:33.674 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: executeForConnection tfstate returned error tfstate: runtime error: index out of range [0] with length 0, writing to CHAN
2024-09-10 08:31:33.674 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: error channel received tfstate: runtime error: index out of range [0] with length 0
2024-09-10 08:31:33.691 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.700 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.708 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.729 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.729 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: readAndStreamAsync failed to read previous rows from cache: tfstate: runtime error: index out of range [0] with length 0 publisher tfstate-1725957057522 (tfstate-1725957057522)
2024-09-10 08:31:33.772 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.845 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:33.878 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:34.003 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:34.051 UTC [WARN]  steampipe-plugin-terraform.plugin: [WARN]<PHONE_NUMBER>522: QueryData StreamError runtime error: index out of range [0] with length 0 (tfstate-1725957057522)
2024-09-10 08:31:38.355 UTC [WARN]  refreshConnections failed with err
        failed to create transaction to perform update query: failed to connect to `user=root database=steampipe`: <IP_ADDRESS>:9193 (<IP_ADDRESS>): server error: FATAL: the database system is shutting down (SQLSTATE 57P03)
2024-09-10 08:31:38.391 UTC [WARN]  setAllConnectionStateToError failed to acquire connection from pool: failed to connect to `user=root database=steampipe`: <IP_ADDRESS>:9193 (<IP_ADDRESS>): tls error: read tcp <IP_ADDRESS>:51540-><IP_ADDRESS>:9193: read: connection reset by peer
2024-09-10 08:31:38.392 UTC [WARN]  failed to send error notification, error

Hello @keyolk,
Thank you for providing such detailed information!
I was able to replicate the error: Error: tfstate: runtime error: index out of range [0] with length 0 (SQLSTATE HV000). To address this, I’ve submitted a PR that I believe should resolve the issue.
In my test environment, the tfstate files in my bucket were around 80KB, and the query executed successfully in 1.4s. However, I haven't been able to test with larger file sizes.
Could you please try the PR branch issue-105 in your environment and let us know how it performs? Your feedback would be greatly appreciated, and I hope this resolves the issue you're experiencing.
Thank you!

I was able to take states in 90 sec for the 4M states. And also 90 sec for the 256K as well. Seems it is not related to the size of state files. Seems it is not directly related to the terraform. Thanks!
I'll see more why the latency happens


Improve generators.toPretty

Motivation for this change
For big Nix expressions, toPretty can be very hard to read, as it doesn't output any newlines. This implements the multiline option for toPretty, which when enabled makes it output newlines with proper indentation. Example output:
{
  foo = {
    bar = 10;
    baz = [
      "baz"
    ];
  };
}

In addition, I made some other improvements:

Don't quote attribute names if they don't need to be quoted by using escapeNixIdentifier, introduced in https://github.com/NixOS/nixpkgs/pull/82461/commits/f9eb3d158a47dccb1e4762f3ed91f224fc96dc7e

Support prefixing the entire output with the prefix option. This e.g. allows using
throw (generators.toPretty { multiline = true; prefix = "error: "; } { foo = "foo"; })

to generate
error: {
error:   foo = "foo";
error: }

Refactor the implementation to have less if statements, replacing them with an attribute switch on the value type

Ping @Profpatsch @rycee
Best reviewed commit-by-commit. Specifically there's one commit in there that just refactors the function without changing its behavior.
Things done

[x] Added tests for the multiline and prefix options

Did some changes:

Multiline strings are printed correctly now
Switch away from the δ and λ symbols for derivations and functions respectively
Went back to the big if else. I don't think it's nicer, but it works out well for another change:
To print __functor's as functions
Removed the prefix argument again. This can be achieved with builtins.replaceStrings [ "\n" ] [ "\nprefix" ] as well

I decided to not render single-element lists/attrs without newlines, because I fear this makes it more complicated to understand the output, as the indentation can't be used to see the nesting level anymore.
Here's a sample output for this Nix expression:
{
  value = {
  fun = lib.id;
  funargs = lib.types.submoduleWith;
  multiline = {
    multi = ''
      hello
      there!
    '';
    multi' = ''
      hello
      there!'';
    single = "hello there!";
  };
  pkg = pkgs.hello;
  single = { foo = [ { bar = [ null ]; } ]; };
}

Which renders as
{
  fun = <function>;
  funargs = <function, args: {modules, shorthandOnlyDefinesConfig?, specialArgs?}>;
  multiline = {
    multi = ''
      hello
      there!
    '';
    multi' = ''
      hello
      there!'';
    single = "hello there!";
  };
  pkg = <derivation /nix/store/2nnf3klixf4j2ih8x0zx4albfzlk74l9-hello-2.10.drv>;
  single = {
    foo = [
      {
        bar = [
          null
        ];
      }
    ];
  };
}

I suppose eventually we'd want this to be a builtin function in Nix, since it's needed in the UI anyway, and it's easier on the C++ side to handle circular values etc.


ARROW-13670: [C++] add virtual destructors

This prevents warnings for non-virtual destructors in base classes.

nope probably just compiler warnings, still useful to reduce warning spam.

Rebased.

The two CI failures seem unrelated.


DOC: Broken link in Quickstart page

Issue with current documentation:
Issue
On the documentation page: https://python.langchain.com/docs/get_started/quickstart
In the Next Steps section:

Explore end-to-end use cases results in page not found

Screenshot

Idea or request for content:
I think that the above issue can be resolved by replacing the above link with this: https://python.langchain.com/docs/use_cases/qa_structured/sql

I am willing to work on the above issue


Fix #288: Skip sha3 taint analysis after call stack exceeds depth

The issue is about Sha3TaintAnalysis middleware.


feat: add position to mod doc

This adds position info to module doc comments, which is essential for doc-gen4 to render them properly. As mod doc is not used in Lean itself, this pull won't affect other parts of the code.

The moduleDoc.lean segfaults at printDoc when tests are run in stage1, but no such error in stage2, and updating stage0 code may fix this.

@xubaiw Yes, the failure is due to staging issues. I will merge manually and perform the update-stage0.

Merged manually.


Want to add more problems on tree and graph.

Select the Day of Content
Day 25
What information is missing?
Question on trees and graph data structures are missing in blog segment.

I would like to work on this issue, please assign this to me.
@subhadipbhowmik


🚧 Allow flushing by database name.

Fixes #287

Coverage decreased (-2.5%) to 91.074% when pulling 9406673f4ace08c7c34701458ce95ad4caa6a16a on feature/allow-flushing-by-database into 527005f04439e94144b03f214cc3617a5401ff2b on master.

Currently not possible.


Add the tap_action option

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Some Lovelace cards have the option of having a tap_action specified. I use browser mod popups to display additional cards when I want more information on something. Tap action would be nice on this card so I can pull up a radar view iwith a dofferent custom card if desired desired.
Additionally, I can see there isn't really anywhere to tap on this card except the weather icons on the forecast section, which pulls open the weather entity. It would be nice if the tap action can also be called from elsewhere on the card, maybe near the main weather icon for the current weather.
Additional context
Add any other context or screenshots about the feature request here.


Confusing error message when passing malformed boolean value in scala-cli config command

Is your feature request related to a problem? Please describe.
Right now, when passing a malformed boolean to a scala-cli config command, we get an error like
$ scala-cli config power true.
[error]  Config DB error: Malformed values true. for power, expected value

Describe the solution you'd like
The expected value message is confusing. Something like expected a boolean value "true" or "false" would work better for example.

Maybe the original passed value should be quoted too, like
[error]  Config DB error: Malformed values "true." for power, expected a boolean value "true" or "false"


Use macos dependencies for skia

Nixpkgs Skia does not build with fontconfig or freetype support on darwin by default, even if those packages are available. The change to enable these libraries in Skia allows the ladybird web browser to compile further on nix-darwin.
I'm not sure but should skia_use_metal=true be gated behind some sort of sdk version check?

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

fontconfig is required on macos now in ladybird as of https://github.com/LadybirdBrowser/ladybird/commit/dd1b8de671747f0a5efc84b77933e13208136507

I took the liberty of writing a more descriptive commit message


adding new biblio module

Addressing #2827, this is a minimal biblio module that provides ivy-bibtex or helm-bibtex (which were recently removed from the latex module in #2838) depending on completion feature, and provides a single global keybinding.
The idea is this provides the core functionalty, and the community can iteratively build it up as appropriate.

This is probably a matter of taste, but I would personally favor this being a global binding, rather than conventionalized on particular modes. Ivy/helm-bibtex essentially provides a UI for, e.g., opening pdfs, opening note files, etc. Doom already includes bindings under SPC n for, e.g., navigating to org-roam files, so I think this is defensible.

No objection from me.

I updated to make the binding global. But it only works if ivy-bibtex or helm-bibtex is first loaded. I assume I need a hook, but not sure how best to do that.

I added global keybindings for evil and emacs and rebased. Hope this is OK.

Good question.
What should happen?
I'm getting to the edge of my depth here with elisp/emacs/doom :-)

Can/should we include one as a requirement in the docs?

That seems like a reasonable solution. Once/if we add additional tools such as ebib, this will be redundant in any case.

Note that #2888 adds an additional package (org-ref) and the README.org file.

I just pushed the requested changes.

I'm not sure how notification works on these tickets, but I think I'm done.

Looks good to me, thanks!

Thank you! I appreciate your patience.


Rename .devcontainers for CUDA 12.5

Follow up to PR: https://github.com/rapidsai/kvikio/pull/401
Partially addresses issue: https://github.com/rapidsai/build-planning/issues/73
Renames the .devcontainers for CUDA 12.5
cc @KyleFromNVIDIA @jameslamb (for awareness)

/merge


1.9 Port

I have ported the mod to 1.9 here. If you create a branch for me to target, I can PR it here.

I've created a 1.9 branch. I looked through your commit, and it looks good. PR it and I'll merge it in.

To avoid some extra commits, you could re-create the branch from the 1.8 branch instead of from master.
Otherwise, you'll get a couple duplicates like so:


fix(source-contentful) add possibility to define search parameters in config

This will solve #748
Previously the Contentful source only returned the default locale of a field. As mentioned in the Contentful docs, the search parameter locale=* has to be provided to return all locales of a field.
With the changes in this pull request, you can now define search parameters in the plugin configuration.
{
   use: '@gridsome/source-contentful',
   options: {
      space: process.env.CTF_SPACE_ID,
      accessToken: process.env.CTF_ACCESS_TOKEN,
      host: 'cdn.contentful.com',
      environment: 'master',
      typeName: 'Contentful',
      parameters: {
         locale: '*'
      }
   }
}

Looks good now, @helloiamlukas :) Thanks for the PR!

What is the latest with #1341 ?


'bin/server' Command is not Available

I successfully installed localtunnel/server on an Ubuntu 18.04 VPS through ssh.
When I typed bin/server to start the server, this message came out:
/usr/bin/env: ‘node -r esm’: No such file or directory
How to fix it?

Same here. Spent some time trying to troubleshoot but no dice.

This matter has been discussed here: https://github.com/localtunnel/server/issues/90
Seems like running node -r esm ./bin/server instead of npm run start works for a few cases.

Try using Node 10 instead of later versions and it should work.

Fixed this issue here https://github.com/StyleT/mytunnel-server It now runs on latest Node.js LTS via npm start.


Exporter/Importer: Blender crash on import/export

I am trying to export or import an xml file and blender closes.
Console output:
Read prefs: /home/user/.config/blender/2.93/config/userpref.blend
jit_cuda_init(): could not find symbol "cuDevicePrimaryCtxRelease_v2" -- disabling CUDA backend!
mitsuba-blender v0.1alpha registered (with mitsuba v3.0.1)
Read blend: /home/user/Blender-Projects/dao.blend
Scene File: /home/user/Blender-Projects/dao.xml
Scene Folder: /home/user/Blender-Projects
Writing: /tmp/dao.crash.txt
Segmentation fault (memory stack flushed to disk)

OS: Ubuntu 22.04.1
Blender Version: 2.93.10
Add-on Version: latest

dao.crash.txt
dao.zip
Rendering existing xml files by Mitsuba 3 works well. Blender file has no dependencies, such as textures, etc., or we are talking about other dependencies?

This seems to be the same issue as #81


Retrieve value from json array element and copy it to the rest of the array

Given this JSON array:
[
{
"a": "blue",
"b": "red",
"c": "green"
},
{
"x": "foo",
"y": "baz"
},
{
"q": "quux",
"r": "yaaar"
}
]
I'm struggling with being able to take the value of "a" (blue) and add that value to the other elements as well, ideally renaming the original, ending up with this:
[
{
"Color": "blue",
"b": "red",
"c": "green"
},
{
"Color": "blue",
"x": "foo",
"y": "baz"
},
{
"Color": "blue",
"q": "quux",
"r": "yaaar"
}
]
I've been playing with Benthos less than a month and having a blast!

Hey @ericforte, if I've understood correctly then something like this could work as a bloblang mapping:
pipeline:
  processors:
    bloblang: |
      let color = this.0.a
      root = this.map_each(ele -> ele.merge({"Color":$color}).without("a"))

Thank you so much.  Even this little snippet showed me a few things (especially this.0.a).
Anyway, this worked.  I realized that I had drilled too far down into my array and that's why it wasn't working.
Thanks again and best regards!


[DT-468] Updated indexing builds and pointed to inside secure perimeter project ids

Ran indexing build for RT and CT test AoU datasets
Updated projectId and queryProjectId to values for projects inside the secure perimeter for RT and CT

@marikomedlock / @freemabd Closing this PR as we now have a new synthetic CDR that is enabled for Tanagra in the AoU test cdr-config file. These are: all-of-us-ehr-dev.SR2023Q3R1 and all-of-us-ehr-dev.SC2023Q3R1 for RT and CT respectively


If the issue still persists, please recreate a new issue or reactivate the existing one


A few wiki fixes

Fixed a few typos and added some missing steps. Link to my fork.

Section 1.4.1 - 'git clone' command missing '.git'
Section 3.2.1 - Spelling, added steps to create the referenced GameObjects, formatting

Link I found useful for merging wikis:

Thank you @mcevoyandy for these clarifications. I just pushed your changes to the upstream Wiki. @roehrlverena fyi

Fixed the path for a couple of files in section 3.3:
     JoyAxisJointTransformWriter.cs (/PATH_TO_YOUR_UNITY_PROJECT/Assets/RosSharp/Scripts/RosBridgeClient/MessageHandling/)
JoyAxisInputPasser.cs (/PATH_TO_YOUR_UNITY_PROJECT/Assets/RosSharp/Scripts/RosBridgeClient/MessageHandling/)
Link to my fork.

Thanks for the info @m-talha and for providing the corrections.
I just updated the whole wiki page.
I hope this tutorial becomes better understandable now.
Please do open a new issue for new topics in the future. New issues in closed topics often get lost.


Update local environment setup commands

After #3050 is merged, there is no need to clear volumes using --volumes anymore when spinning down the project.
The localenv command should be updated to just pnpm localenv:compose down and the issue regarding this bug should be removed from "Known issues".

3050 has been merged. We can work on this now.


[infra] Fix labler

Trying to address:
Error: Error: found unexpected type for label 'type-infra' (should be array of config options)
Error: found unexpected type for label 'type-infra' (should be array of config options)

Similar PR:

https://github.com/flutter/flutter/pull/129676

@devoncarew I'm not sure why this is failing, any ideas?

Duplicate of https://github.com/dart-lang/native/pull/83


Cache compiled extensions on Travis

Compiled extensions should be cached for reduced test run times.

Compiling doesn't take that long and the builds are fast enough, closing.


Disable testRepositoryCacheDoesNotDerailResolution

This tests seems to access remote dependencies, we should change it so that it can work without network access. This disables it for now.

@swift-ci please test

@SDGGiesbrecht could you take a look at changing this test? I missed that this test requires remote dependencies when reviewing #3006

I can look into other ways of tripping it. I cannot promise I will find a way though, because the point of the test was that the mocking infrastructure used elsewhere was bypassing the problem entirely.
Either way, you may as well outright remove the test instead of just disabling it, because if I do find a way, it will look very different.

#3047 has added a new test that replaces this, we can close.


PHPUnit Code Coverage for runned process

Is it possible somehow to run PHPUnit Code Coverage on the process which is run with Process? In my application, I run a couple of PHP sub-processes of my code, but I want to check Code Coverage on that.

@Kolesar Welcome to ReactPHP and thanks for you excellent question! Unfortunately this is somewhat out of scope of this project, so I'm not sure this is the right place for this question.
The same question applies to running any external process with PHP, be it a exec() call or accessing things via HTTP etc., so perhaps StackOverflow etc. could help here? :+1: If anybody has an answer for this, feel free to respond to this ticket!
Other than that, I believe this has been answered, so I'm closing this for now.


fix: using simple string match replace regex match for contains udf

Which issue does this PR close?

Closes #12930.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

ci failure looks unrelated to changes.

ci failure looks unrelated to changes.
yeah, I add test case for retring ci.😂

LGTM

🚀


Front page not updating when event is edited

Bug or Problem
The front page not updated when event is edited until the user goes to a different route and comes back or the page is refreshed.
Steps to Reproduce

Click on one of the events
Click the edit button at the bottom
Change the title of the event, then press Submit
Exit the popup.
The entry on the menu is not changed :(

Environment

System: Arch Linux
Browser: Firefox

Possible Solution
idk, probably something with updating props on the client/src/pages/Home.js page (I'll find a solution)

Another Problem
Same issue when adding events.
Page refresh is required for it to display.

Another Problem
Same issue when adding events.
Page refresh is required for it to display.


[Firefox] [Import/Export] The file is generated but is not exported

The file is generated but is not exported.

Test OK.

Tested on version: 62.2-20190524


Error in slow5tools merge

Hi,
I tried to use fast5toslow5 to convert the fast5 files to slow5 and then merge them into a single file. I am getting the following error during the merge step
[list_all_items] Looking for '*.fast5' files in /hpf/projects/bkalish/MIA_m6A/nanopore/new_samples/raw_data/Sample1
[list_all_items] Looking for '*.fast5' files in /hpf/projects/bkalish/MIA_m6A/nanopore/new_samples/raw_data/Sample1/fast5
[f2s_main] 2747 fast5 files found - took 1.053s
[f2s_main] Just before forking, peak RAM = 0.000 GB
[f2s_iop] 8 proceses will be used.
[f2s_iop] Spawning 8 I/O processes to circumvent HDF hell.
[f2s_child_worker::ERROR] Bad fast5: Fast5 file '/hpf/projects/bkalish/MIA_m6A/nanopore/new_samples/raw_data/Sample1/fast5/PAW00730_6e7a5de0_3b3dcc69_14.82_4.fast5' could not be opened or is corrupted.
[f2s_child_worker::ERROR] Bad fast5: Fast5 file '/hpf/projects/bkalish/MIA_m6A/nanopore/new_samples/raw_data/Sample1/fast5/PAW00730_6e7a5de0_3b3dcc69_3.67_5.fast5' could not be opened or is corrupted.
[f2s_iop] Child process 2794779 exited with status=1.
[list_all_items] Looking for '*.slow5' files in /hpf/largeprojects/ccmbio/acelik_files/kalish/nanopore/nanopore/lauren_test/debug_nanopolish/try_f5c/slow5_dir/Sample1
[merge_main] 158 files found - took 0.049s

[slow5_hdr_init::ERROR] Malformed blow5 header. Failed to read the magic number. EOF reached. At src/slow5.c:801
[slow5_init::ERROR] Parsing slow5 header of file '/hpf/largeprojects/ccmbio/acelik_files/kalish/nanopore/nanopore/lauren_test/debug_nanopolish/try_f5c/slow5_dir/Sample1/PAW00730_6e7a5de0_3b3dcc69_27.20_2.blow5' failed. At src/slow5.c:153
[slow5_open_with::ERROR] Exiting on error. At src/slow5.c:369

I have slow5 files output but the merged.blow5 file is empty.
I am using the most recent binary version of slow5tools. The original fast5 files are converted from the pod5 files with the pod5 program. The basecaller is dorado.
Thanks!
Laur

The error suggests that the input fast5 files to slow5tools  are not in the expected schema.
Could you try directly converting the pod5 files directly to blow5 using https://github.com/Psy-Fer/blue-crab.
I have a feeling that the pod5 to fast5 converter could be creating fast5 in an unexpected fashion which the slow5tools cannot recognise. Let's see if the direct conversion works.

Thank you so much for the quick response! I will get the pod5 files and try the blue-crab. Will keep you posted! Thanks a lot!
Laur

@lyj95618 Has this problem been sorted out?

Yes thanks! Everything solved after using the original pod5 files with blue-crab. Thank you for all the helps!


CI changes for running makefile

This PR is for triggering the makefile using github actions.
What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind new feature
/kind bug fix
/kind cleanup
/kind revert change
/kind design
/kind documentation
/kind enhancement

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Test Report Added?:

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind TESTED
/kind NOT-TESTED

Test Report:

Special notes for your reviewer:

Please also squash all commits


Add basic event actions to analytics

Click around in: https://excalidraw-git-events.excalidraw.vercel.app
Version 1.0 (this PR)

[x] MVP analytics.md
[x] Don't run analytics during tests
[x] Disable analytics for localhost
[x] Fix tests

Version 2.0

Library
Collaboration
Export dialog
Layer
Align
Content menu

Fixes #1701

If needed to debug locally, add REACT_APP_INCLUDE_GTAG=true to your .env.local file (not in VSC).


Description of configuration file

I started a Q&A in the discussion here https://github.com/Nickwasused/dnoise/discussions/4, but it seems that you did not receive a notification about it.
It would be very helpful if you describe the properties in the configuration file. For example, what does multiplier mean?

I have added https://github.com/Nickwasused/dnoise/wiki/Config-File


Add hash table saturation metrics

Added hash table saturation metrics added in https://github.com/ironcore-dev/dpservice/issues/438
Metrics are exported in following format (each table has its capacity and entries exported):
hash_table_saturation{stat_name="capacity",table_name="conntrack_table"} 850000
hash_table_saturation{stat_name="entries",table_name="conntrack_table"} 0

Added testing Grafana dashboard.

Not needed after merger with dpservice https://github.com/ironcore-dev/dpservice/issues/554
Functionality merged directly in dpservice repo.


Datastore entrypoint with IFluidLoadable instead of IProvideFluidHandle

Description
Version of https://github.com/microsoft/FluidFramework/pull/11887 using IProvideFluidLoadable instead of IProvideFluidHandle
@anthony-murphy FYI. Built locally without issues, let's see if the CI tests pass.

These changes have been integrated into the main PR https://github.com/microsoft/FluidFramework/pull/11887.


Week 0 - Challenges

Individual GitHub pages (3 points)
Navigates notes and plans for each Tech Talk
Contains Test Prep plans and participation for success on AP exam (required for test takers)
Navigates to Data Structures work, contains code snippets on problems solutions, links to GitHub for Code, Replit for runtime, or alternatives

1.5/1.5 Notes & plans for TT w/ diagram
(not taking AP Exam)
1.125/1.5 Links to code commits but could have been easier to see w/ code snippets.

Also I think you have to use Jekyll instead of Wikis but I would suggest that you ask.


yarn check prints out two warnings when installing the latest Jest 24.4.0 packages

🐛 Bug Report
yarn check prints out two warnings when installing the latest Jest 24.4.0 packages.
To Reproduce
Install the latest Jest 24.4.0 packages. Observe these warnings:
warning "jest > jest-cli > @jest/core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "jest > jest-cli > jest-config ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>
Expected behavior
No warnings.
Run npx envinfo --preset jest
Paste the results here:
$ npx envinfo --preset jest
Error: EPERM: operation not permitted, mkdir <redacted>
TypeError: Cannot read property 'get' of undefined

Having the same issue over here.

same here!
We are using Windows 10

Also seeing this on our Linux build machines.
npx: installed 1 in 2.049s

  System:
    OS: Linux 4.14 CentOS Linux 7 (Core)
    CPU: (48) x64 Intel(R) Xeon(R) Gold 5118 CPU @ 2.30GHz
  Binaries:
    Node: 10.15.0 - /usr/local/bin/node
    Yarn: 1.13.0 - /usr/local/bin/yarn
    npm: 6.4.1 - /usr/local/bin/npm

Same for me using macOS with Jest 24.5.0.
System:
  OS: macOS Mojave v10.14.3
  CPU: 2,6 GHz Intel Core i7

Binaries:
  Node: 10.15.1 - /usr/bin/node
  Yarn: 1.13.0 - /usr/bin/yarn
  npm: 6.4.1 - /usr/bin/npm
  npmPackages:
    jest: ^24.5.0 => 24.5.0