New-NavTopMenu and Navlink

Hello,
I was testing some code and found that this New-NavTopMenu used and found that these Menus are very impressive and useful.
I was trying if I click on Sub-menu link and give some URL link, can that open in same page with continuing Top Menu as header, without leaving the page?
Sample code
$time = (get-date)
    $CompanyLogo = "P:\InfraWeb\Home_files\image002.png"
    New-HTML -TitleText 'Report' -ShowHTML -Online -FilePath $SavePath {
        New-HTMLNavTop -Logo $CompanyLogo -MenuColorBackground LightGray -MenuColor Black -HomeColorBackground Chardonnay -HomeLinkHome  {

            New-NavTopMenu -Name 'Domains' -IconRegular building -IconColor black {
            New-NavLink -IconRegular file-image -IconColor blue -Href 'https://infraweb.abc.com/VMInventory/' -Name 'VM Inventory'
                New-NavLink -IconSolid users -Name 'Groups' -InternalPageID 'Groups'
                New-NavLink -IconSolid users -Name 'Groups_Empty' -InternalPageID 'Groups_Empty'
                New-NavLink -IconMaterial folder -Name 'OU' -InternalPageID 'OU'
                New-NavLink -IconSolid scroll -Name 'Group Policy' -InternalPageID 'GPO'
                New-NavLink -IconSolid print -Name 'Printers' -InternalPageID 'Printers'
            }
            New-NavTopMenu -Name 'Objects' -IconSolid sitemap {
                New-NavLink -IconSolid user-tie -Name 'Users' -InternalPageID 'Users'
                New-NavLink -IconSolid laptop -Name 'Computers' -InternalPageID 'Computers'
            }

        }

    }

Here if I click on Vm Inventory sub-menu it will redirect to that page but if there a way to open the target page with same menus so I can navigate to other menus/tabs?

There are 2 ways to do it:

Is to use InternalPageID along with New-HTMLPage. This means you don't use Href, but the module automatically tracks this for you and links to a different page will be done automatically creating necessary navigation in the new page
If you want to use Href directly the thing is the page you're linking to has 0 awarness about that you have some menu. What you can do is create html page with the same menu as above, inside the html page put a frame on it and link the href to that href. The problem is putting stuff into frame has drawbacks if pages have protection against linking or the pages have security against showing content from different domains.

If you are in control you can work around things. I use either New-HTMLPage or New-HTMLFrame all the time.

I tried to use New-HTMLPage but had no luck.
Maybe I am not using the correct syntaxes.

I think it's a bug. I'll fix it.

Actually no bug.
$DataTable1 = @(
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 1.42 }
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 1.72 }
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 1.22 }
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 2.73 }
)

New-HTML -TitleText 'Testing Navigation' {
    New-HTMLNavTop -Logo 'https://evotec.xyz/wp-content/uploads/2021/04/Logo-evotec-bb.png' -LogoLinkHome {
        New-NavTopMenu -Name 'Domains' -IconRegular address-book {
            New-NavLink -IconMaterial airplane -Name 'This is internal link 1' -InternalPageID 'Page 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link to evotec' -IconColor Pink -NameColor Red
            New-NavLink -Name 'This is link 3' -IconBrands apple-pay {
                New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                New-NavLink -IconSolid address-book -Name 'This is link 6' {
                    New-NavLink -Name 'This is link 3' -IconBrands confluence {
                        New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                        New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 6'
                    }
                }
            }
        }
        New-NavTopMenu -Name 'Domain Controllers' -IconSolid ambulance {
            New-NavLink -IconMaterial airplane -Name 'This is link 1 (internal)' -InternalPageID 'Page 2'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 3'
        }
        New-NavTopMenu -Name 'Office 365' {
            New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 4'
        }
    } #-MenuColor Black -MenuColorBackground Pink -HomeColorBackground Black -HomeColor Red
    New-HTMLHeader {
        New-HTMLText -Text 'In header - main page'
    }
    New-HTMLTab -Name 'Test 21' {
        New-HTMLText -Text "This is first page - tab 1" -Color Red -FontSize 40px -Alignment center

    }
    New-HTMLTab -Name 'Test 22' {
        New-HTMLText -Text "This is first page - tab 2" -Color Red -FontSize 40px -Alignment center
    }
    New-HTMLFooter {
        New-HTMLText -Text 'In footer - main page'
    }

    New-HTMLPage -Name 'Page 1' {
        New-HTMLHeader {
            New-HTMLText -Text 'In header'
        }
        New-HTMLTab -Name 'Test 1' {
            New-HTMLSection -HeaderText 'Test 0' {
                New-HTMLTable -DataTable $DataTable1 -Title 'Test2' -PagingLength 2
            }
            New-HTMLSection -HeaderText 'Test 1' {
                New-HTMLTable -DataTable $DataTable1 -Title 'Test2' -PagingLength 2
            }
        }
        New-HTMLTab -Name 'Test 2' {

        }
        New-HTMLFooter {
            New-HTMLText -Text 'In footer'
        }
    }
    New-HTMLPage -Name 'Page 2' {
        New-HTMLSection -HeaderText 'Test2' {
            New-HTMLTable -DataTable $DataTable1 -Title 'Test2' -PagingLength 2
        }
    }
} -ShowHTML -Online -FilePath $PSScriptRoot\Example41-Navigation03.html

Thanks for sharing this Example.
Now what is I need to load the HTML page which is output of Dashimo.
I want to load the exported HTML output in the Page 1 content body, so by that it will not leave the make Menu page.
I tried replacing
New-HTMLPage -Name 'Page 1' {
        New-HTMLHeader {
            New-HTMLText -Text 'In header'
        }
        New-HTMLTab -Name 'Test 1' {
            New-HTMLSection -HeaderText 'Test 0' {
                New-HTMLTable -DataTable $DataTable1 -Title 'Test2' -PagingLength 2
            }
            New-HTMLSection -HeaderText 'Test 1' {
                New-HTMLTable -DataTable $DataTable1 -Title 'Test2' -PagingLength 2
            }
        }
        New-HTMLTab -Name 'Test 2' {

        }
        New-HTMLFooter {
            New-HTMLText -Text 'In footer'
        }
    }

With this as testing, I know this is incorrect but just looking to see if we can use any command to load the other HTML page here without creating a new HTML page from this Master script.
So the idea is to have this as the Master page with small checks, but other heavy scripts will run separately and generate the output then we can call those from this master page
    New-HTMLPage -Name 'Page 1' {
        New-HTMLHeader {
            New-HTMLText -Text 'In header'
            New-HTMLContent -Content "P:\_Scripts\Dashboards_Testing\VMinventory.html"
        }

        New-HTMLFooter {
            New-HTMLText -Text 'In footer'
        }
    }

Here is the complete code
$DataTable1 = @(
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 1.42 }
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 1.72 }
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 1.22 }
    [PSCustomObject] @{ Test = 'Name'; Test2 = 'Name2'; Test3 = 'Name3'; Test4 = 2.73 }
)

New-HTML -TitleText 'Testing Navigation' {
    New-HTMLNavTop -Logo 'https://evotec.xyz/wp-content/uploads/2021/04/Logo-evotec-bb.png' -LogoLinkHome {
        New-NavTopMenu -Name 'Domains' -IconRegular address-book {
            New-NavLink -IconMaterial airplane -Name 'This is internal link 1' -InternalPageID 'Page 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link to evotec' -IconColor Pink -NameColor Red
            New-NavLink -Name 'This is link 3' -IconBrands apple-pay {
                New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                New-NavLink -IconSolid address-book -Name 'This is link 6' {
                    New-NavLink -Name 'This is link 3' -IconBrands confluence {
                        New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                        New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 6'
                    }
                }
            }
        }
        New-NavTopMenu -Name 'Domain Controllers' -IconSolid ambulance {
            New-NavLink -IconMaterial airplane -Name 'This is link 1 (internal)' -InternalPageID 'Page 2'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 3'
        }
        New-NavTopMenu -Name 'Office 365' {
            New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 4'
        }
    } #-MenuColor Black -MenuColorBackground Pink -HomeColorBackground Black -HomeColor Red
    New-HTMLHeader {
        New-HTMLText -Text 'In header - main page'
    }
    New-HTMLTab -Name 'Test 21' {
        New-HTMLText -Text "This is first page - tab 1" -Color Red -FontSize 40px -Alignment center

    }
    New-HTMLTab -Name 'Test 22' {
        New-HTMLText -Text "This is first page - tab 2" -Color Red -FontSize 40px -Alignment center
    }
    New-HTMLFooter {
        New-HTMLText -Text 'In footer - main page'
    }

    New-HTMLPage -Name 'Page 1' {
        New-HTMLHeader {
            New-HTMLText -Text 'In header'
            New-HTMLContent -Content "P:\_Scripts\Dashboards_Testing\VMinventory.html"
        }

        New-HTMLFooter {
            New-HTMLText -Text 'In footer'
        }
    }
    New-HTMLPage -Name 'Page 2' {
        New-HTMLSection -HeaderText 'Test2' {
            New-HTMLTable -DataTable $DataTable1 -Title 'Test2' -PagingLength 2
        }
    }
    #} -ShowHTML -Online -FilePath $PSScriptRoot\Example41-Navigation03.html

}-ShowHTML -Online -FilePath "P:\_Scripts\Dashboards_Testing\Home.html"

So the thing is you can't modify existing HTML document without heavy parsing. When I created the NavBars (3 kinds) what I did is actually thought on how to address the issue that you are having. Essentially all my modules GPOZaurr/Testimo and so on, all generrate HTML reports and how to combine them into a single Dashboard.
The problem is:

each html has all the tags body, scripts, links, headers, footers
a lot of html files will overlap with scripts and links which if you just merge them go into conflict

So I did entertain the idea that I would have PSWriteHTML use PSParseHTML to get current scripts, compare with new scripts, merge NavBar at the top which may or may not work. But then I was thinking - ok, this will require the reports to be generrated by PSWriteHTML and if you try to merge anything else it will be bonkers. There's just too many unknowns. Like I wanted to integrated PingCastle to it.
So what I did instead is use New-HTMLFrame. Basically you create pages as you do above, that link to existing HTML files that are displayed within a frame. And it works.
I've created project TheDashboard which takes existing files, and history of the reports and merges it into a dashboard

And then if you choose report, you go to next page, which then shows you newest report, but with ability to go back in time and check older reports (if present). The marked rectangle is basically a frame that displays HTML that sits somewhere else. So the main page is small (if you use links, not include full CSS/JS), that links to a much larger document.

I guess you're trying to build something similar, and I have this working on IIS/SharePoint for a long time now where I've learnt what works, what doesn't. I do plan on releasing TheDashboard, but I'm thinking on it as "sponsorware" at the moment. But I have not decided yet on how to do it. If you want to do it yourself, New-HTMLFrame is what you need.

Thanks.
Let me digest this and test it.

That worked, as I wanted to. Thanks for giving this life saver Module.
So what I tried and what is the concern now.
I tried the below code I working perfectly fine as the mentioned page opens in the same master page panel.
> Working
New-HTML {
    New-HTMLNavTop -Logo 'https://evotec.xyz/wp-content/uploads/2021/04/Logo-evotec-bb.png'   -LogoLinkHome -MenuColorBackground LightBlue -HomeColor black  -MenuColor Black -HomeColorBackground White -HomeLinkHome {
    #endregion Top Menu part

    #region  All the Menus
        New-NavTopMenu -Name 'Report' -IconSolid receipt {
            New-NavLink -IconMaterial accounts-alt -Name 'VM Inventory' -InternalPageID 'Page 1'
            New-NavLink -IconMaterial alarm-check -Name 'Server reboot' -InternalPageID 'Page 2'
            New-NavLink -Name 'Inventory' -IconBrands   blogger {
                New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                New-NavLink -IconSolid address-book -Name 'This is link 6' {
                    New-NavLink -Name 'This is link 3' -IconBrands confluence {
                        New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                        New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 6'
                    }
                }
            }
        }
        New-NavTopMenu -Name ' Bookmarks' -IconSolid bookmark {
            New-NavLink -IconSolid address-book -Href 'https://portal.azure.com' -Name 'Azure'
            New-NavLink -IconBrands codiepie -Href 'https://www.vmware.com/support/services.html' -Name 'VMware Support'
            New-NavLink -IconBrands codiepie -Href 'https://github.com/' -Name 'GitHub'
        }
        New-NavTopMenu -Name 'IntraLinks' {
            New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 4'
        }
    #endregion All the Menus

   } #-MenuColor Black -MenuColorBackground Pink -HomeColorBackground Black -HomeColor Red
   New-HTMLPage -Name 'Page 1' {
        New-HTMLSection {
            New-HTMLFrame -SourcePath "P:\InfraWeb\VMInventory\Home.html" -Scrolling Auto -Height 1500px
        } -HeaderText 'VM Inventory'
        }
    New-HTMLPage -Name 'Page 2' {
        New-HTMLSection {
            New-HTMLFrame -SourcePath "P:\InfraWeb\RebootCheck\Home.html" -Scrolling Auto -Height 1500px
        } -HeaderText 'Server reboot'
        }

<#
        New-HTMLSection {
            New-HTMLFrame -SourcePath "https://google.com"
        } -HeaderText 'Test' -Height 100px
#>
    } -Online -TitleText 'Test Inline' -ShowHTML -FilePath "$PSScriptRoot\Example-InlineHTML01.html" -AddComment

Now if I am trying to add the Tab or Section its not working properly and data also not showing in formated way.
New-HTML {
    New-HTMLNavTop -Logo 'https://evotec.xyz/wp-content/uploads/2021/04/Logo-evotec-bb.png'   -LogoLinkHome -MenuColorBackground LightBlue -HomeColor black  -MenuColor Black -HomeColorBackground White -HomeLinkHome {
    #endregion Top Menu part

    #region  All the Menus
        New-NavTopMenu -Name 'Report' -IconSolid receipt {
            New-NavLink -IconMaterial accounts-alt -Name 'VM Inventory' -InternalPageID 'Page 1'
            New-NavLink -IconMaterial alarm-check -Name 'Server reboot' -InternalPageID 'Page 2'
            New-NavLink -Name 'Inventory' -IconBrands   blogger {
                New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                New-NavLink -IconSolid address-book -Name 'This is link 6' {
                    New-NavLink -Name 'This is link 3' -IconBrands confluence {
                        New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                        New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 6'
                    }
                }
            }
        }
        New-NavTopMenu -Name ' Bookmarks' -IconSolid bookmark {
            New-NavLink -IconSolid address-book -Href 'https://portal.azure.com' -Name 'Azure'
            New-NavLink -IconBrands codiepie -Href 'https://www.vmware.com/support/services.html' -Name 'VMware Support'
            New-NavLink -IconBrands codiepie -Href 'https://github.com/' -Name 'GitHub'
        }
        New-NavTopMenu -Name 'IntraLinks' {
            New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 4'
        }
    #endregion All the Menus

   } #-MenuColor Black -MenuColorBackground Pink -HomeColorBackground Black -HomeColor Red
   New-HTMLPage -Name 'Page 1' {
        New-HTMLSection {
            New-HTMLFrame -SourcePath "P:\InfraWeb\VMInventory\Home.html" -Scrolling Auto -Height 1500px
        } -HeaderText 'VM Inventory'
        }
    New-HTMLPage -Name 'Page 2' {
        New-HTMLSection {
            New-HTMLFrame -SourcePath "P:\InfraWeb\RebootCheck\Home.html" -Scrolling Auto -Height 1500px
        } -HeaderText 'Server reboot'
        }

    New-HTMLTab -Name 'Dashboard' -IconRegular chart-bar   {
       New-HTMLSection  -Name 'Block info' -Width 10% {

       New-HTMLSection  -Name 'Sub-Block info' -Margin 10 -Width 15%    {
                New-HTMLPanel -Margin 10 {

                    New-HTMLPanel -BackgroundColor lightgreen -AlignContentText right -BorderRadius 10px {
                        New-HTMLText -TextBlock {
                           # New-HTMLText -Text $UserDisabled -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 50 -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 'Active Test' -Alignment justify -FontSize 15
                            New-HTMLTag -Tag 'i' -Attributes @{ class = "fas fa-user-slash fa-3x" }
                        }
                    }

                    New-HTMLPanel -BackgroundColor yellowgreen -AlignContentText right -BorderRadius 10px {
                        New-HTMLText -TextBlock {
                            #New-HTMLText -Text $userinactive -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 130 -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 'Failed Test' -Alignment justify -FontSize 15
                            New-HTMLTag -Tag 'span' -Attributes @{ class = "fas fa-user-clock fa-3x" }
                        }
                    }

                }
                }

           }
    }

    } -Online -TitleText 'Test Inline' -ShowHTML -FilePath "$PSScriptRoot\Example-InlineHTML01.html" -AddComment

Just move New-HTMLPage to the end, as it's supposed to be processed as last.
New-HTML {
    New-HTMLNavTop -Logo 'https://evotec.xyz/wp-content/uploads/2021/04/Logo-evotec-bb.png' -LogoLinkHome -MenuColorBackground LightBlue -HomeColor black -MenuColor Black -HomeColorBackground White -HomeLinkHome {
        #endregion Top Menu part

        #region  All the Menus
        New-NavTopMenu -Name 'Report' -IconSolid receipt {
            New-NavLink -IconMaterial accounts-alt -Name 'VM Inventory' -InternalPageID 'Page 1'
            New-NavLink -IconMaterial alarm-check -Name 'Server reboot' -InternalPageID 'Page 2'
            New-NavLink -Name 'Inventory' -IconBrands blogger {
                New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                New-NavLink -IconSolid address-book -Name 'This is link 6' {
                    New-NavLink -Name 'This is link 3' -IconBrands confluence {
                        New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 5'
                        New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 6'
                    }
                }
            }
        }
        New-NavTopMenu -Name ' Bookmarks' -IconSolid bookmark {
            New-NavLink -IconSolid address-book -Href 'https://portal.azure.com' -Name 'Azure'
            New-NavLink -IconBrands codiepie -Href 'https://www.vmware.com/support/services.html' -Name 'VMware Support'
            New-NavLink -IconBrands codiepie -Href 'https://github.com/' -Name 'GitHub'
        }
        New-NavTopMenu -Name 'IntraLinks' {
            New-NavLink -IconMaterial airplane -Href 'https://evotec.xyz' -Name 'This is link 1'
            New-NavLink -IconSolid address-book -Href 'https://evotec.xyz' -Name 'This is link 2'
            New-NavLink -IconBrands codiepie -Href 'https://evotec.xyz' -Name 'This is link 4'
        }
        #endregion All the Menus

    } #-MenuColor Black -MenuColorBackground Pink -HomeColorBackground Black -HomeColor Red

    New-HTMLTab -Name 'Dashboard' -IconRegular chart-bar {
        New-HTMLSection -Name 'Block info' -Width 10% {

            New-HTMLSection -Name 'Sub-Block info' -Margin 10 -Width 15% {
                New-HTMLPanel -Margin 10 {

                    New-HTMLPanel -BackgroundColor lightgreen -AlignContentText right -BorderRadius 10px {
                        New-HTMLText -TextBlock {
                            # New-HTMLText -Text $UserDisabled -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 50 -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 'Active Test' -Alignment justify -FontSize 15
                            New-HTMLTag -Tag 'i' -Attributes @{ class = "fas fa-user-slash fa-3x" }
                        }
                    }

                    New-HTMLPanel -BackgroundColor yellowgreen -AlignContentText right -BorderRadius 10px {
                        New-HTMLText -TextBlock {
                            #New-HTMLText -Text $userinactive -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 130 -Alignment justify -FontSize 25 -FontWeight bold
                            New-HTMLText -Text 'Failed Test' -Alignment justify -FontSize 15
                            New-HTMLTag -Tag 'span' -Attributes @{ class = "fas fa-user-clock fa-3x" }
                        }
                    }

                }
            }

        }
    }

    New-HTMLPage -Name 'Page 1' {
        New-HTMLSection {
            New-HTMLFrame -SourcePath "P:\InfraWeb\VMInventory\Home.html" -Scrolling Auto -Height 1500px
        } -HeaderText 'VM Inventory'
    }
    New-HTMLPage -Name 'Page 2' {
        New-HTMLSection {
            New-HTMLFrame -SourcePath "P:\InfraWeb\RebootCheck\Home.html" -Scrolling Auto -Height 1500px
        } -HeaderText 'Server reboot'
    }
} -Online -TitleText 'Test Inline' -ShowHTML -FilePath "$PSScriptRoot\Example-InlineHTML01.html" -AddComment

The way it works that way is that when you add New-HTMLPage, it basically treats anything that comes before New-HTMLPage, as not defined New-HTMLPage that is the primary page. If you mix it up it gets confused what goes where. I am not sure if it's worth my effort to do anything about it. Probably not.

That worked.
Thank you so much once again.
Now I will build my Dashboard and see what other challenges I face.


Demo Setup: Cannot find a model for Dall-e or one that is "plug-in capable"

I have access to Azure Open AI and I am able to deploy models. I created an instance in East US. I was able to deploy gpt4 as per the instructions but I  do not see "dalle and a plugins-capable model." I looked at the list of models and nothing here helped me understand which model to choose. Am I in the wrong region or is there some step I missed?
List of Models: https://learn.microsoft.com/en-us/azure/cognitive-services/openai/concepts/models#model-summary-table-and-region-availability
Do any of these models generate images or support plug-ins?

The DALL-E models are still in private preview.  For the DALL-E model, you must request access using the form at https://aka.ms/oai/access and in question #22, request access to the DALL-E models for image generation.
The plugin-capable models are not publicly released yet.

I've updated the README with this information in this commit: https://github.com/microsoft/PodcastCopilot/commit/56db5b1beaac65b0f8e11c2dd695e2cb21457c56


myScroll fails to reinitialize after calling .destroy()

We are using iScroll lite (v4.1.1) and are calling the .destroy() method, then setting iscroll to null and then calling the loaded method again to reinitialize iScroll.
However, we keep on running into this error TypeError: Result of expression 'that[dir + 'ScrollbarWrapper'].parentNode' [null] is not an object.
Any thoughts as to why this might be happening?

I got this same error message the other day but found that it was because I had modified the iScroll area prior to destroying iScroll. If you append or remove an element from the top level scroll element you have to call destroy or refresh before doing so.


Had to change requests writes to binary to fix dropbox-node issue.

Hello,
Ran into an issue with file uploads using dropbox-node which depends on node-oauth oauth.js. I had to change the request.write calls to write in binary to get non-ascii file uploads (images, pdfs etc) to work. View the issue here. Not sure if this is something you want to adopt or if it will have implications with other oauth APIs, but the Dropbox Oauth api will not work with non-ascii files unless the request is written in binary.
Thanks,
Ralph

Still an issue, I have a 3rd party api that returns a PDF from a get request. Will see how hard it is to fix and submit a PR. Is this project still being maintained?

@Dakuan Get it working?

@philals sure did: https://github.com/Dakuan/node-oauth/commits/master

Thanks for the @Dakuan
From what I see your changes make the returned data always binary rather than utf8. Is that the case? (https://github.com/ciaranj/node-oauth/compare/master...Dakuan:master#diff-84fad1d3b3913ec86805e857580c5d54R404)
I'm trying to use the same lib for application/json and application/pdf

we are doing just that for in our case - for the xero api at it happens :) - i sent you a message on linkedin too

Well.. looks like we have things in common.
For those that get here from google. Here is how I have managed to GET an invoice.

You must also create an Oauth object and pass in the accept header = application/pdf see here: https://github.com/ciaranj/node-oauth/issues/106


retain cycle in SDWebImageDownloader.m

SDWebImageDownloader.m :154
capturing 'self' strongly in this block is likely to lead to a retain cycle
inSession:self.session

I found in SDWebImageDownloaderOperation.m :103
        __weak __typeof__ (self) wself = self;
        UIApplication * app = [UIApplicationClass performSelector:@selector(sharedApplication)];
        self.backgroundTaskId = [app beginBackgroundTaskWithExpirationHandler:^{
            __strong __typeof (wself) sself = wself;
             ..........

}
Will this lead to retain cycle?
On other place like this,the block is not hoding by self.

This is fixed on the 4.x branch.


[Suggestion] you can use langchain.js in Unity

So you can access OpenAI's API more easily.
https://github.com/puerts/unity-langchain-demo

https://hwchase17.github.io/langchainjs/docs/overview/
https://langchain.readthedocs.io/en/latest/index.html

My understanding is that LangChain is useful for making the app API-agonistic (or should I say LLM agonistic?)
I made this project for the Unity-ChatGPT tutorial series, so I wouldn't like to extend it further.
Anyway, thanks for the suggestion. It looks very interesting. I'll consider integrating it in future projects.


Is this even helping?

I was going through source code and found that rating data (rating and votes) is appended into id and inserted into fst::Set. My question is why not stick to fst::Map instead?
https://github.com/BurntSushi/imdb-rename/blob/f2a40bfcc39ac01201b896044f31318c0b82db82/imdb-index/src/index/rating.rs#L98
To my understanding, this new set element with 18 bytes (first nine for imdb_id, followed by 0x00, and 4 bytes each for rating and votes) will bloat the fst as last 9 byte tail will always have it's own separate state transformation chain. Instead, if map is used, u64 space of value can store both rating and votes without bothering about extra space wasted in storing fst transformations. Help me understand, if I'm thinking the right direction?
I came to this observations after I did a similar thing on Have I Been Pwned dataset.
-rw-r--r--   1 vivek  admin<PHONE_NUMBER>1 Oct 18 17:49 pwned-passwords-ordered-by-hash.set.fst
-rw-r--r--   1 vivek  admin<PHONE_NUMBER>2 Oct 17 23:46 pwned-passwords-ordered-by-hash.map.fst
-rw-r--r--   1 vivek  admin<PHONE_NUMBER>5 Jul 11 06:07 pwned-passwords-ordered-by-hash.txt

.txt file is sha1 hash and count data like following:
000000005AD76BD555C1D6D771DE417A4B87E4B4:4
00000000A8DAE4228F821FB418F59826079BF368:2
00000000DD7F2A1C68A35673713783CA390C9E93:630

.map.fst uses 20 bytes sha1 as key, count as u64 value. In .set.fst 20 bytes of sha1 are appended with u32 as big-endian bytes. My original intent was saving it as sets would save some space which would reduce file size and therefore would incur lesser page faults when reading mmaped fst data. Instead I ended up without and practical saving (0.17%?).
If there is something else that I'm not seeing, let me know.

@yoursvivek Sorry, I'm having trouble parsing your comment here. It sounds like you're saying an fst map will use less space, but your example shows the exact opposite? That is, your fst set is using 18,648 MB but your fst map is using 18,679 MB. Honestly, that's not much difference at all.
Or... maybe you're asking a conceptual question here? Are you saying that you expected the map to use less space, and you're wondering why it isn't? If so, then yeah, that is fairly easy to explain. The issue with a map is that maps store the output values on the transitions themselves, which means there is less opportunity for reusing states. Remember, a state can only be reused if all of its outgoing transitions are the same. In the case of a map, the output values on those transitions are part of determining equality between transitions. So yeah, the set does need to encode more states naively, but you're not accounting for the limits on state reuse in maps. It's a bit hard to reason about without going and looking directly at the finite state machine.
Also, you might be overestimating how much extra space is used by the set in this case. I can't remember all the specifics, but in the very best case, a state in an fst is represented by a single byte. I think that case is for states with exactly one transition, which would probably correspond to the tailing states in the set.

Let me try again.
Preface: I tried to save space in my pwned dataset by moving from map to set but using 4 extra bytes in keys, instead of 8 byte value of map. I forgot about bytes required in transition states. Also given that all sha1 hashes in my file were unique, the byte representation of count will be a tail with no chance of reuse of state. I learned my lesson.
Now over to imdb-rename code, I see it as a similar data set. Unique keys (imdb id), and same sized value payload (4+4 bytes). Since keys are unique, there are no chances of reuse of state when appending value payload to keys themselves. Set representation will also incur cost of saving state transitions for each byte of value payload appended to key. then there is a nul byte too.
That begs the question: Why did you use sets instead of maps in saving ratings fst in imdb-rename?
I'm just curious. Is there some other gain?

I don't know. Maybe I started with maps and then switched to sets because it uses less space? That's unlikely though, since ratings are I think the smallest fst in the index, so I doubt I would care.
Another possibility is that I thought it might be possible for one IMDb title to be associated with multiple ratings. The type signatures don't encode that possibility, but I wasn't sure if the data guaranteed it.
Another possible explanation is that I just copied the code from representing the TV show index, which does require this sort of representation. For example:
https://github.com/BurntSushi/imdb-rename/blob/f2a40bfcc39ac01201b896044f31318c0b82db82/imdb-index/src/index/episode.rs#L100-L113

Hmm, Ok.
So in general, when should one use fst sets and when should one use fst maps?
PS: I really need to finish reading your Automata article.

So in general, when should one use fst sets and when should one use fst maps?

I don't think there exists a general answer. There is probably also an argument about access time here. It's probably faster to find a value in a map using get than it is to execute a range query and decode the tail of your key to get a value.


invoices+sqldb: small fixes to address some inconsistencies between KV and native SQL invoice DB implementations

This PR fixes a few inconsistencies between KV and native SQL invoice DB implementations that caused numerous test failures on our CI:
  --- FAIL: TestLightningNetworkDaemon/tranche05/107-of-168/bitcoind/sendpayment_amp_invoice_repeat (19.61s)
        lnd_amp_test.go:300:
            	Error Trace:	/home/runner/work/lnd/lnd/itest/lnd_amp_test.go:300
            	            				/home/runner/work/lnd/lnd/lntest/harness.go:386
            	            				/home/runner/work/lnd/lnd/itest/lnd_test.go:139
            	Error:      	Not equal:
            	            	expected: 1
            	            	actual  : 2
            	Test:       	TestLightningNetworkDaemon/tranche05/107-of-168/bitcoind/sendpayment_amp_invoice_repeat

    --- FAIL: TestLightningNetworkDaemon/tranche02/47-of-169/bitcoind/list_payments (24.16s)
        --- FAIL: TestLightningNetworkDaemon/tranche02/47-of-169/bitcoind/list_payments/invoice_exact_end_date (0.00s)
            lnd_payment_test.go:616:
                	Error Trace:	/home/runner/work/lnd/lnd/itest/lnd_payment_test.go:616
                	Error:      	"[]" should have 1 item(s), but has 0
                	Test:       	TestLightningNetworkDaemon/tranche02/47-of-169/bitcoind/list_payments/invoice_exact_end_date
                	Messages:   	req: creation_date_end:1725010429

Now includes changes from https://github.com/lightningnetwork/lnd/pull/9022:
The PR fixes the following test:
make itest backend=btcd dbbackend=postgres nativesql=true icase=sendpayment_amp_invoice
Note that the sendpayment_amp_invoice_repeat test is not fixed by this. Also note that this depends on https://github.com/lightningnetwork/lnd/pull/9021 to unmask the test failure.

Native sql tests were green in the last run! Kicking the other failed tests to see if they're flakes.


Sorting function.

At present, this plug-in can only be sorted according to the addition order. Classifying similar commands can help us find the required operations faster.

+1 for this. It's really required.


Service worker does not work with trusted types

Issue and Steps to Reproduce
We use a content security policy with "require-trusted-types-for 'script';" to increase security of our application. Unfortunately the OidcServiceWorker does not work with it.
Steps:

integrate react-oidc in application as described in the documentation and verify it is working
add the policy "require-trusted-types-for 'script';" to the Content-Security-Policy response header
try to access application in the browser (possibly a reregistration of the service worker in the browser is needed to start from clean)

Versions
3.7.1
Results
Expected
Login in application works as before.
Actual
Browser shows: "An error occurred during authentication"
Javascript console shows:
OidcServiceWorker.js:240 This document requires 'TrustedScriptURL' assignment.
(anonymous) @ OidcServiceWorker.js:240
OidcServiceWorker.js:240    Uncaught TypeError: Failed to execute 'importScripts' on 'WorkerGlobalScope': This document requires 'TrustedScriptURL' assignment.
    at OidcServiceWorker.js:240:7
(anonymous) @ OidcServiceWorker.js:240
mydomain.com/:1 Uncaught (in promise) TypeError: Failed to register a ServiceWorker for scope ('https://mydomain.com/') with script ('https://mydomain.com/OidcServiceWorker.js'): ServiceWorker script evaluation failed

Additional Details
We temporarily fixed the issue by adding the following lines of code after the first line of OidcServiceWorker.js file:
if ((typeof trustedTypes !== 'undefined') && (typeof trustedTypes.createPolicy == 'function')) {
  trustedTypes.createPolicy('default', {
    createScriptURL: function(url) {
      if (url == scriptFilename) {
        return url;
      } else {
        throw new Error('Untrusted script URL blocked: ' + url);
      }
    },
  });
}

Thank you very much @Bewalticus for your issue.
I have to read about this security.
i'am in holiday for 3 weeks. I can manage this in september.

@guillaume-chervet, do you have any update here?

hi @Bewalticus , very sorry for the delay :)
version 7.6.0 implement it :)

@guillaume-chervet , thank you for putting it into the release. It works like a charm!

Thank you for your feedback @Bewalticus  !


VSR: Remove unnecessary commit

This code is left over from ce554c2b9f56d7b3959b09e67e7bb539c58b8a2e, over a year ago! Since MessagePool.get_message() now succeeds unconditionally, the only way the head prepare's prepare.ok_quorum_received is true in on_prepare() is if we are already committing.
Pre-merge checklist
Performance:

[x] I am very sure this PR could not affect performance.

bors r+
This commit is definietly useless.
But the overall timeout flow feels a bit weird here. If we don't fire prepare timeout while we are committing, then why do we need to keep it ticking at all? It would feel more logical to do either of the two:

stop the timer commit_pipeline, reset the timer in commit_ops_done
keep the timer ticking, but, in on_prepare_timeout search for a prepare which doesn't have a quorum, instead of looking only at the first prepare in the pipeline.

It would feel more logical to do either of the two:

stop the timer commit_pipeline, reset the timer in commit_ops_done
keep the timer ticking, but, in on_prepare_timeout search for a prepare which doesn't have a quorum, instead of looking only at the first prepare in the pipeline.

Great idea! I'm going to implement the second, since it perfectly complements the upcoming "primary steps down if it doesn't receive prepare_oks" feature.


fix: allow persistent storage to be extended

Summary of Changes
Allow persistent storage to be extended by adding to to the config useConfiguration() overrides.
Related Issues
Please reference here any issue #'s that are relevant to this PR, or simply enter "N/A" if this PR does not relate to any existing issues.
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

Will re-create


Implement pipeline for downsampled morphometrics maps for large scale microscopy

When segmenting a large scale sample (e.g. full spinal cord slice), users need to obtain downsampled morphometrics maps. This can currently be done by the AxonSeg pipeline. The implementation of this functionality should also be available in AxonDeepSeg in order to remove the dependency with AxonSeg.

Stale


Update outdated terms

"contribute": "Contribute on GitHub"
"pullrequest": "Make a Pull Request on GitHub"

Thanks!

Complete vi.json in #493

Updated the checklist, cheers!


Updates Dev Wallet v2 for local dev with harness

Updates for compatibility with harness app

Updates default port to 3000 as defined in docs, tests, Dockerfile
fixes env vars bug
Updates some text for clarity and add links to README

Ok, I'll update the harness


No issue, just appreciation

Hi David, as the title suggests there are no issue (that I've found so far!), I just wanted to say thanks and well done on a useful utility. The beauty of it for me as a novice Golang developer is that it will be a valuable learning resource, especially on the usage of command line flags.
Thanks again,
Carl

Thank you for the kind words @carlca!
@alsm and I are very pleased both with what we've accomplished, and the reception kurly is currently getting.  Thank you!
For a simple application the default Golang flag package is likely all you need, but indeed for a more complex tool, something like github.com/urfave/cli is certainly the way to go. 👍
Thanks again for the feedback; feel free to let us know of any issues you encounter.
Cheers,
David & Al


[LLM Runtime] integrate AVX_VNNI

Type of Change
NO API changed or not
integrate AVX_VNNI
detail description
JIRA ticket: xxx
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

@VincyZhang can you deploy extension tests on one of our client machine? @yuchengliu1 can help

CPU: 12900
compute_type: int32
| first token | next token
-- | -- | --
llama-7B_q4j_perN | 17046.92 ms (16.65 ms per token) | 169.06 ms per token
llama2-7B_q4j_perN | 16952.96 ms (16.56 ms per token) | 166.27 ms per token

I remembered latency of AVX2 is 141.69ms.
Is 166 ms good enough?

I remembered latency of AVX2 is 141.69ms. Is 166 ms good enough?

compared with https://github.com/intel/intel-extension-for-transformers/pull/493 161ms
BTW it will be better to list memory bandwidth of your machine @yuchengliu1

I remembered latency of AVX2 is 141.69ms. Is 166 ms good enough?

141.69ms is the latency of llama.cpp not ITREX, and it was performed on a different machine. Considering the different CPU and memory, a direct comparison would not be appropriate. The machine from #493 seem have a better performance. The next token latency of ITREX q4_0 179.03ms(this PR) VS 136.37ms(#493 ). @VincyZhang and I will run a complete CI on this machine.

https://inteltf-jenk.sh.intel.com/job/nlp_toolkit_cpp_graph_test/1445/

Model
AVX VNNI First token  (ms)
AVX VNNI Next token  (ms)
AVX2 First token  (ms)
AVX2 Next token  (ms)

LLAMA2-7B-Chat
17046.92
169.06
25337.58
250.01

https://inteltf-jenk.sh.intel.com/job/nlp_toolkit_cpp_graph_test/1445/
Model	AVX VNNI First token  (ms)	AVX VNNI Next token  (ms)	AVX2 First token  (ms)	AVX2 Next token  (ms)
LLAMA2-7B-Chat	17046.92	169.06	25337.58	250.01

llama.cpp ( set thread=16 manually ) performance  on the same matchine.
first token:  39511.79 ms /  1024 tokens (   38.59 ms per token,    25.92 tokens per second)
next token: 5337.70 ms /    31 runs   (  172.18 ms per token,     5.81 tokens per second)


How to speed up cursor's movement on error lines

Information
VIM version
:version  gvim8.1 1-26.
Operating System: Windows 10.
:ALEInfo
Current Filetype:
Available Linters: []
Enabled Linters: []
Linter Variables:
Global Variables:
let g:ale_cache_executable_check_failures = v:null
let g:ale_change_sign_column_color = 0
let g:ale_command_wrapper = ''
let g:ale_completion_delay = v:null
let g:ale_completion_enabled = 0
let g:ale_completion_max_suggestions = v:null
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%code: %%s'
let g:ale_echo_msg_info_str = 'Info'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 1
let g:ale_fixers = {'coffeescript': ['coffee', 'coffeelint'], 'javascript': ['prettier', 'eslint'], 'python': ['flake8']}
let g:ale_history_enabled = 1
let g:ale_history_log_output = 1
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_filetype_changed = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_lint_on_insert_leave = 0
let g:ale_linter_aliases = {}
let g:ale_linters = {'coffeescript': ['coffee', 'coffeelint'], 'javascript': ['eslint'], 'python': ['flake8']}
let g:ale_linters_explicit = 0
let g:ale_list_window_size = 10
let g:ale_list_vertical = 0
let g:ale_loclist_msg_format = '%code: %%s'
let g:ale_max_buffer_history_size = 20
let g:ale_max_signs = -1
let g:ale_maximum_file_size = v:null
let g:ale_open_list = 0
let g:ale_pattern_options = {}
let g:ale_pattern_options_enabled = 0
let g:ale_set_balloons = 1
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_info = '--'
let g:ale_sign_offset = 1000000
let g:ale_sign_style_error = '>>'
let g:ale_sign_style_warning = '--'
let g:ale_sign_warning = '--'
let g:ale_statusline_format = v:null
let g:ale_type_map = {}
let g:ale_use_global_executables = v:null
let g:ale_warn_about_trailing_blank_lines = 1
let g:ale_warn_about_trailing_whitespace = 1
Command History:
What went wrong
I've noticed this plugin slow down movement of cursor (h, l) on lines that marked as errors. Also feel a bit slow down on cursor's movement in general when enable ale.
errors show on status bar constantly blinking together with cursor's movement (h, l). Is this caused by async process?
I also disabled signcolumn to speed up apart.
vimrc config portion:
let g:ale_linters = {
  \ 'javascript': ['eslint'],
  \ 'python'    : ['flake8'],
  \ 'coffeescript'    : ['coffee','coffeelint'],
  \}
let g:ale_fixers = {
  \ 'javascript': ['prettier', 'eslint'],
  \ 'python'    : ['flake8'],
  \ 'coffeescript'    : ['coffee','coffeelint'],
  \}
let g:ale_sign_column_always = 0
let g:ale_fix_on_save = 1
set signcolumn = 0

See this FAQ entry. https://github.com/w0rp/ale#5xiii-will-this-plugin-eat-all-of-my-laptop-battery-power
See :help ale-lint.
Try playing with some of the options mentioned in both places so ALE will run whatever less often.

I have a question to add to this. I've noticed that when writing, I get serious delays in neovim in an OCaml file with the merlin backend as ALE updates the display. My question is, why does that even happen? Shouldn't ALE be running asynchronously and therefore not impacting the editor experience itself?


WIP: integration of unity_ros2 improvements and port to crystal

Work in progress: this is not a master-ready material, but please create a working branch so we can coordinate work between 3 people easily and make other pull requests to it.
This is a natural step towards Dashing port, which should follow once we get this to a satisfying level.
Current status:

Examples confirmed to run. Tested with fresh build, no other testing done.

How to run:

Following prerequisites are required: existing Crystal install, colcon.
Use ros2_dotnet_crystal.repo with vcs, this is a minimal set of repos.
Source your Crystal env.
colcon build

Required work:

Update README with option of build with existing Crystal install, colcon etc.
Produce and test .repo file for standalone build (aka no Crystal distro installed). Remember to add colcon dependency, remove unnecessary ones.
Test with UWP
Update CI to test for new minimal build
Update NOTICE files so they reflect contributions properly

Commit message:
Integrated improvements from unity_ros2 project:

nested types
unit tests
rclcs overhaul to crystal version and structure changes

Also added my own fixes to build and remaining code

extended interfaces to provide something useful
replaced obsolete rcl_ok() call
adapted examples to new library semantics
introduced ros2_dotnet_crystal.repo, which supports a smaller build with existing Crystal install

@esteve

@adamdbrw thanks, I really appreciate the effort, but this is impossible to review. Can it be broken into smaller pull requests? Also, I'm not comfortable with including work from someone else's, it should be Samuel who submits the pull request, to ensure that all is ok, copyright-wise.

@esteve
I would really like to make it easier for you.

I am cooperating with Samuel on this and I emailed him the pull request so hopefully he can comment himself.  Please note that the integration was not straightforward and took some significant effort since you don't have the same root, and he has limited time while I have some hours for this in my project. I kept all the NOTICE and authorship stuff intact, didn't even add my name anywhere.
Lots of stuff is interdependent here and hard to break into smaller pull request without plenty of creative effort to do so (and I can't be convinced to do it). If I could make it easier for you without much work on my side, I would (ideas?). However I am working in a frame where I can't afford to do much "extra" work, unfortunately.

@esteve I believe these are not normal circumstances, since we are creatively integrating two bodies of work with plenty of differences. There are methods to handle this kind of case, i.e. a dedicated code-base review after we pass Unit Tests and CI.
I like small pull request just like any other guy, and I seek no glory.

FWIW my current thinking is to focus on getting a 'minimal' standalone build of dashing running in a UWP container, using the C++/WinRT native layer (no C# required). This ought to serve as a foundation for deploying ROS2 within UWP (including the various patches to rcl and others by @esteve and anything else that pops up).
My understanding is once the the rcldotnet/rclcs stuff should ride on top of that rather independently, right?
By the way @esteve is there any significant need to fully support crystal now that dashing is out? I'd rather skip straight to the latest.

@esteve I believe these are not normal circumstances, since we are creatively integrating two bodies of work with plenty of differences.

Except that one of the bodies of work started two years ago, whereas the other is a fork that could have been integrated as patches from the beginning. This pull request wasn't created in a vacuum completely separate from the rest of the world.

There are methods to handle this kind of case, i.e. a dedicated code-base review after we pass Unit Tests and CI.

Why not just contribute the test suite? That's what I told you by email.

I also understand there's anxiety when your child mutates into something alien.

There's nothing of that, look at https://github.com/ros2-rust/ros2_rust for example. A user saw that ros2-rust already existed and instead of forking it and evolve it on their own without contributing back, he decided to submit a pull request (https://github.com/ros2-rust/ros2_rust/pull/3) that included a lot of improvements, but in a way that was humanly reviewable. The outcome? I added that user to the team and he and I are now collaborating, he's actually taking the lead on the project.
Moreover, what's the point of replacing one project with another? I just don't understand. If you like the other project, seriously, use it, I will not force anyone to use any of the projects I've written. I'm doing this in my spare time for the fun of it.

It is of course a question whether the benefit is worth the effort for you and other ros2_dotnet users.

However, why should I make the effort of integrating a third person's work (Samuel)? Why couldn't that person submit this pull request? Your effort is laudable, but I just don't understand any of this process. Why isn't Samuel doing this work instead of you?

@theseankelly that makes the most sense and it's much easier to iterate that way. Ensuring that ROS 2 Dashing as is (up to rcl) can be built for UWP would pave the way for any .NET implementation to support the HoloLens.
I'll set up the CI to use colcon and then we can update the repos files to build the rcl-based subset for UWP.

@esteve
Should we close the pull request then?
If you have an idea how I can help you otherwise I would be happy to.


Adding a way to add group and user in case environment doesn't provide it by default. It is controlled by property.

The property introduced to enable group and user addition is: azkaban.add.group.and.user.for.effective.user

Could you please elaborate on the property's use case? On what scenarios should be set? How is it set? What user is supposed to set it?

Pull Request Test Coverage Report for Build 6677

5 of 37 (13.51%)  changed or added relevant lines in 3 files are covered.
221 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 45.293%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/JobRunner.java
4
14
28.57%

azkaban-common/src/main/java/azkaban/utils/ExecuteAsUserUtils.java
0
22
0.0%

Files with Coverage Reduction
New Missed Lines
%

azkaban-web-server/src/main/java/azkaban/webapp/servlet/ProjectManagerServlet.java
221
8.58%

Totals

Change from base Build 6662:
-0.03%

Covered Lines:
17976

Relevant Lines:
39688

💛  - Coveralls


HTML5 Font Issue

It seem like font embedding isn't working with OpenFl 8.8.0 / Lime 7.2.1 while targeting html5
This can be seen by running the OpenFL TextAlignment sample

Can confirm this issue is affecting me as well.

Just to paste here what I shared on the discord channel.
The demo for me looks fine.

What can be noted that during building the build process said:
Warning: Could not generate *.woff web font for "Assets/nokiafc22.ttf"
This however doesn't prevent the font from showing in the final build since the html css changes accordingly
(see how the "Liberation" font could create the woff and includes it, but the "nokia" one failed and thus doesn't include it)
@font-face {
    font-family: 'Liberation Serif';
    src: url('assets/LIBERATIONSERIF-REGULAR.eot?#iefix') format('embedded-opentype'),
    url('assets/LIBERATIONSERIF-REGULAR.woff') format('woff'),
    url('assets/LIBERATIONSERIF-REGULAR.TTF') format('truetype'),
    url('assets/LIBERATIONSERIF-REGULAR.svg#Liberation%20Serif') format('svg');
    font-weight: normal;
    font-style: normal;
}
@font-face {
    font-family: 'Nokia Cellphone FC Small';
    src: url('assets/nokiafc22.eot?#iefix') format('embedded-opentype'),
    url('assets/nokiafc22.ttf') format('truetype');
    font-weight: normal;
    font-style: normal;
}```

(Also, I can't see the image from the original post so I don't know what the issue was)

This is the error the Firefox console spits when I compile the sample to HTML5, which is also the same error I get with my other projects. So it's definitely something on our end.
For the record I am using:
Lime 7.1.2
OpenFL 8.8.0
and Haxe 4 rc 1

So sharing this info with @peteshand , you may want to reinstall Haxe. seems there's a weird bug going around where Haxe files are getting corrupted. After reinstall, the problem went away.


callysto: cleanup functionality now bundled with oauthenticator 16.2.0+

Removes callysto's custom authenticator by adding allowed_domains_claim: email to the config.

Fixes #3268.

Thank you for reviewing @GeorgianaElena !!!
I see that callysto has no active users atm, I'll merge and verify function manually a bit further!

Verified to function and resolved with #3465


update code.gitea.io/git in vendor to fix delete branch fails

Fixes #2245
Replaces #2248

LGTM

LGTM


Handle session expiry in the event there is already another

this fixes the problem experienced in issue #6

:+1:

I'll close this PR since it's more than 10 years old at this point 🙂


Global Trends Dataset vs Social Media Usage --> final with conclusions

Work on notebook, add text, notes, polish code and comments
Write conclusions
Write README.md with description of the work done and conclusions

That's really great work,
Thanks Denys


Evolve and docs.

✨

Current coverage is 97.31% (diff: 100%)

Merging #135 into master will decrease coverage by 2.68%

@@           master       #135   diff @@
========================================
  Files           8          8
  Lines         504        522    +18
  Methods         0          0
  Messages        0          0
  Branches      110        102     -8
========================================
+ Hits          504        508     +4
- Misses          0         11    +11
- Partials        0          3     +3

Powered by Codecov. Last update 1481227...4dbd8fa

Sure thing, the changes will be done tonight.

Done!

Awesome, thanks!


Initial version of Joint Fabric implementation

Initial version of the Joint Fabric implementation 34635
Added support for JCM (5.6.3. Joint Commissioning Method)
Fixes https://github.com/project-chip/connectedhomeip/issues/34635

@vijs I think the huge delta is due to the kotlin formatter not being run: all kt files changed and all look unformatted (quite ugly)

Please split this up to something more manageable. A PR of 300K lines added and 240K removed is not reviewable in any practical sense.
I imagine a lot of this is code generated files, so please start with a PR for those (XML for data first, then maybe some commands individually)

Thank you for your suggestions. I’ve regenerated and restyled the files. Now, only 73 files have changed.


docs: update instructions on how to contribute to dashboards

Motivation
After using the download script a bit more myself noted that instructions are not that precise for current workflow.
It is no longer required to import a temporal dashboard with different name but more importantly it must be ensured that the UID is not modified, else the download script will not pick up the changes.
It should be pretty clear to existing maintainers that when using our Grafana cloud instance there is no need to import the dashboard, in fact, it is not possible to import it with same UID. Point 1. and 2. are rather meant for (new) contributors that modify dashboards on their own / local Grafana instance.
Description
Update instructions on how to contribute to dashboards to better reflect current workflow.

:tada: This PR is included in v1.10.0 :tada:


Placeholder wait for SSHKit 1.10.x

Placeholder here to remind me that we're waiting for SSHKit 1.10.

We still have to cut a final release of 1.9.0.

Indeed, but I don't feel right changing behaviour in 1.9x when we already
cut an RC, hence 1.10
(sent from my phone, please excuse typos)
On 7 Mar 2016 4:36 p.m., "Matt Brictson"<EMAIL_ADDRESS>wrote:

We still have to cut a final release of 1.9.0.
—
Reply to this email directly or view it on GitHub
https://github.com/capistrano/capistrano/issues/1628#issuecomment-193302700
.

1.10 was released today.


YQL parsing of large multi-valued query operators (weightedSet/dotProduct/wand/weakAnd)

This is based on an observation from the wild where a Vespa user was using weightedset query operator for large sets. With about 1000 keys the YQL parsing took around 10 ms which compared to backend execution was close to 3x.
I think this is a problem with many of the multi-valued query operators like weightedSet, dotProduct where YQL parsing becomes a bottleneck with large inputs.
I think we should have some native parameter/searcher implementation to accept large set filter inputs outside of YQL, maybe similar to this PR

Perhaps we need a more robust search API that allows separating the actual query from parameters so we could cache query parsing (similar to preparedStatements in traditional DBs)

#20513 should reduce the time spent parsing that YQL by a factor ~2-2.5. It affects parsing of large wand, dotProduct, and weightedSet items. Using a custom encoding is still quite a bit faster, though.

https://github.com/vespa-engine/vespa/pull/20665 adds support for supplying the term list in those operators externally to the YQL expression. That brings parse time for a weighted set with 1000 items down to about 0.5 ms. The remainder is number parsing cost.

Documentation: https://github.com/vespa-engine/documentation/pull/1743


Change data representation in collate_fn to use lists of integers

There is no need to use padding for cell membership in the loader representations. Cell membership padding is not used for spatial tasks, which also have huge differences in cell sizes. The data collider supports lists of integers. Padding can be created efficiently on the fly with torch functions for qm9.
This issue should:

Represent internally as a list of integers in the collate fn
For qm9, restore the original padded format for backward compatibility

After this issue is solved, other issues/PRs will integrate the spatial benchmarks.

Maybe the solution is to have two collators

Fixed.


Issue 164, pipdeptree doesn't work with python 3.11

Hi, I'm trying to fix issue https://github.com/naiquevin/pipdeptree/issues/164 which makes pipdeptree does not work with python 3.11.
Since python 3.11, pip try to use importlib.metadata.Distribution instead of pkg_resources.Distribution.
https://github.com/pypa/pip/blob/e2c9c40edeb65b9d26c13a12dbe5642dc2e00fd0/src/pip/_internal/metadata/__init__.py#L45-L49
Currently there is a workaround which is set _PIP_USE_IMPORTLIB_METADATA=0 as mentioned in https://github.com/naiquevin/pipdeptree/issues/164#issuecomment-1145482711.
But I think it should be better to explicit import pkg_resources and use it.

Sorry for the mess with git, I created a new one https://github.com/tox-dev/pipdeptree/pull/173


Block comment can result in incorrect minification

IMPORTANT: clean-css is now in a maintenance mode. PRs are still welcome, and I will try do an occasional bugfix relase.
Precheck

Do a quick search and make sure a bug has not yet been reported;
do a quick check if the bug still exists in the latest patch version;
finally, be nice and have fun!

Environment

clean-css version - npm ls clean-css<EMAIL_ADDRESS>node.js version - node -v: v16.14.2
operating system: Windows 10

Configuration options
// rollup_config.js
import CleanCSS from 'clean-css';
...
transform: (contents) => new CleanCSS().minify(contents).styles,

Input CSS
.toggle--active {
  /* NOTE: We need to clear `transform` since it makes the fixed overlay background (.toggle::after)
     relative to the container's boundaries instead of the viewport. For more info, see
     https://achrafkassioui.com/blog/position-fixed-and-CSS-transforms/ */
  transform: none;
  visibility: visible;
}

Actual output CSS
.toggle--active{visibility:visible}

Expected output CSS
.toggle--active{transform:none;visibility:visible}

Fixed alongside #1224.

The fix is out in 5.3.2.


Different parent

The native sticky behaviour constraints the element to dimensions of its direct parent element.
This can be an issue sometimes, in this case the stickied
element should use the body element itself as parent.

I do not fully understand your issue. You can use the scrollEl.

    stickybits('.sticky', {
      useStickyClasses: true,
      stickyBitStickyOffset: stickyOffset,
      scrollEl: '.main',
    });

The element is stickied, but still constrained by dimensions of its direct parent.

Please provide a codepen so we can directly see and test this.

@DanielRuf: https://codepen.io/strarsis/pen/MVgeKm

Upon further inspection I notice that apparently just position: sticky is applied to it (on Chrome) but no further toggling (like position: fixed) for a different scroll element.

position: sticky is what is used in browsers that support it so this is right.

@DanielRuf: But doesn't this library state it is not a polyfill and that it thereby offers additional, different behaviour?

Is there an update and/or fix that needed to made @strarsis? If so, please submit a pull request—with a readme or code update. If the update/fix makes sense, I will happily support it being merged.
Thanks! ~Jeff 🙏


Update Stack Exchange / StackOverflow link

old version has a banner saying it is depreciated and points to its new site

Ahh sweet, thanks for updating!


403 Access Denied when use a query param

Describe the bug
I got 403 Access Denied when use the query param '?location' in my Amplify Hosted App
To Reproduce
Steps to reproduce the behavior:

Go to https://my-domain.com/?location or  https://my-domain.com/signup?location or any  route with the query  param '?location'
Get 403 error

Expected behavior
It should show the page as it is
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version 88.0.4324.190

Hi @cristihandev, thanks for raising this. Are you still running into this?
If so, can you please share the type, structure, and redirect rules of your application to provide a bit more context? Thanks!

Hi @siegerts , yap I'm still running into that.
Type: A React Application (SPA)
Structure: Is based on create-react-app but I don't think it is because of that.
Redirect rules: Is setup base on the documentation for SPAs  https://docs.aws.amazon.com/amplify/latest/userguide/redirects.html
Address: </^[^.]+$|\.(?!(css|gif|ico|jpg|js|png|txt|svg|woff|ttf|map|json)$)([^.]+$)/> DestinationAddress: /index.html RedirectType: 200
But I can't see the restriction for https://example.com?location
for example I did test with the following URL
Suppose my site is: https://example.com
https://example.com --> works https://example.com?anyotherparam=anyvalue --> works https://example.com/ --> works https://example.com/anyurl --> works https://example.com/anyurl?anyotherparam=anyvalue --> works https://example.com/any/sub/url -> works https://example.com/any/sub/url?anyotherparam=anyvalue -> works https://example.com?location --> returns 403 https://example.com?location?anyvalue --> returns 403 https://example.com/anyurl?location --> returns 403 https://example.com/anyurl?location=anyvalue --> returns 403 https://example.com/any/sub/url?location --> returns 403 https://example.com/any/sub/url?location=anyvalue --> returns 403

Thanks for the follow up. Can you copy the redirect/rewrite config for the above (without any sensitive info)? You can snag it by clicking the open text editor in App settings -> Rewrites and redirects.

Yap, I got this.

Check out the docs around using query params @ https://docs.aws.amazon.com/amplify/latest/userguide/redirects.html#query-strings-and-path-parameters
For example, using something like below checks for the query param before the React app catches the match and moves navigation into the app (via react-navigation, etc.):
[
    {
        "source": "/?location=<location>",
        "target": "/<location>",
        "status": "301",
        "condition": null
    },
    {
        "source": "</^[^.]+$|\\.(?!(css|gif|ico|jpg|js|png|txt|svg|woff|ttf|map|json)$)([^.]+$)/>",
        "target": "/index.html",
        "status": "200",
        "condition": null
    }
]

Hope this helps!

Thanks @siegerts !, the thing is I really need the location query Param because I do an API call with it, I would like to get rid of the 403 ERROR instead of redirecting some where else in the APP

Is it possible to handle this in the React app since you only need to capture the param value?  So everything goes to index.html -> then parse in the app?
I have an example doing this @ https://main.d1m5x5zjik90i7.amplifyapp.com/test/one/two/account?name=modus-create using https://reactrouter.com/web/example/query-parameters

I have exactly the same problem.
Amplify "catch" the request and returns 403 error.
If location param is used, react is never executed. (It works fine locally when Amplify is not involved)

@cacharrin is using the router in the app an option as I did above? That seems to catch it as you would expect.

Yup... your example has exactly the same problem...
Try this yourself in your own repo: change the param from name=modus-create to location=modus-create...
(https://main.d1m5x5zjik90i7.amplifyapp.com/test/one/two/account?location=modus-create)
BOOOOM!!!
The problem is caused only by a param called location is not caused by any configuration on the react part... react is never executed, amplify catch it and throws error, request never get to react.
At this time I have tested it in 4 amplify projects (2 owned, 2 externals - including yours), all are affected by this bug.

Thanks for testing, @cacharrin. Marking this as a bug.
Steps to reproduce:

Create application using create-react-app
Add react-router and query parameters example in App.js
Test locally, in-app nav + direct nav

http://localhost:3000/account?location=modus-create

Works as expected.
Deploy to Amplify:

Set up with default SPA rewrite rule
Test in app nav + direct nav

https://<domain.com>/account?location=modus-create

Direct nav throws a 403.


find reverse dependencies for distributions other then unstable

Sometimes I would like to be able to find all reverse dependencies, say from wheezy, and rebuild them.
AFAIK Currently you can rebuild packages on wheezy, but this will cause errors if unstable has more packages then what was available on wheezy.

Have you tried just string-replacing “sid” and “unstable” with “wheezy” in ratt.go? Does that work? If not, what goes wrong?
Also, do you have an example I could use to reproduce the issue you’re mentioning?

I tried it on Jessie and it didn't work.
However I think that is because apt-get on Jessie doesn't have the "apt-get indextargets" command, and the fallback didn't work. /var/lib/apt/lists/*_InRelease doesn't get downloaded for a distribution just because the sources are referenced, it appears.
So now I have upgraded to testing, I really should try it again.
The package in question was libav - I did a security update for it for wheezy and want to know what packages in wheezy will break if I convince the security-team to include it. So as I result I need to be confident I am including all the relevant packages.

For some reason the following command produces no results, not sure why:
"dose-ceve" "--deb-native-arch=amd64" "-T" "deb" "-r" "libav-tools,libav-dbg,libav-doc,libavutil54,libavcodec56,libavdevice55,libavformat56,libavfilter5,libswscale3,libavutil-dev,libavcodec-dev,libavdevice-dev,libavformat-dev,libavfilter-dev,libswscale-dev,libavresample-dev,libavresample2,libavcodec-extra-56,libavcodec-extra" "-G" "pkg" "debsrc:///var/lib/apt/lists/ftp.au.debian.org_debian_dists_wheezy_main_source_Sources" "debsrc:///var/lib/apt/lists/ftp.au.debian.org_debian_dists_wheezy_contrib_source_Sources" "debsrc:///var/lib/apt/lists/ftp.au.debian.org_debian_dists_wheezy_non-free_source_Sources"

So it looks like I need to download the binary package indexes, not just the source indexes for this to work.
However just noticed that the --extra-package= flag to sbuild doesn't appear to be working. So while it is building the required packages, it is building against the libav that is already in wheezy, not my new version :-(

I submitted http://bugs.debian.org/820882 on this problem.

ratt needs to make sure that when it uses the same architecture for building the packages as in the source *.changes file. e.g. if the *.changes is i386, we need to use an i386 chroot. Otherwise it won't work. Still not entirely sure I understand in my case why it chose amd64 as the default, so will think about this and open up a new issue if required.


Captcha fixed, its working fine in both login & register pages, close…

Description
The captcha which was not working is fixed, now the users can login or register verifying the captcha.
Fixes #132
Type of change
Authentication

Thanks @tusharzalte for merging, can you label and assign this PR✌

Yess
Can you make it responsive?
On Sat, Jun 10, 2023, 14:57 Debsourya Datta @.***>
wrote:

Thanks @tusharzalte https://github.com/tusharzalte for merging, can you
label and assign this PR✌
—
Reply to this email directly, view it on GitHub
https://github.com/tusharzalte/tusharesume/pull/143#issuecomment-1585582963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yess Can you make it responsive?
…
On Sat, Jun 10, 2023, 14:57 Debsourya Datta @.> wrote: Thanks @tusharzalte https://github.com/tusharzalte for merging, can you label and assign this PR✌ — Reply to this email directly, view it on GitHub <#143 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY . You are receiving this because you were mentioned.Message ID: @.>

Which part specifically??

Like login and register page and also templates
On Sun, Jun 11, 2023, 09:34 Debsourya Datta @.***>
wrote:

Yess Can you make it responsive?
… <#m_8525000024726928252_>
On Sat, Jun 10, 2023, 14:57 Debsourya Datta @.> wrote: Thanks
@tusharzalte https://github.com/tusharzalte
https://github.com/tusharzalte https://github.com/tusharzalte for
merging, can you label and assign this PR✌ — Reply to this email directly,
view it on GitHub <#143 (comment)
https://github.com/tusharzalte/tusharesume/pull/143#issuecomment-1585582963>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
. You are receiving this because you were mentioned.Message ID: @.>
Which part specifically??
—
Reply to this email directly, view it on GitHub
https://github.com/tusharzalte/tusharesume/pull/143#issuecomment-1586004381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTK64KOEHCNXMRFE5PFR3DXKU7T5ANCNFSM6AAAAAAZBR24QY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Like login and register page and also templates On Sun, Jun 11, 2023, 09:34 Debsourya Datta @.> wrote:
…
Yess Can you make it responsive? … <#m_8525000024726928252_> On Sat, Jun 10, 2023, 14:57 Debsourya Datta @.> wrote: Thanks @tusharzalte https://github.com/tusharzalte https://github.com/tusharzalte https://github.com/tusharzalte for merging, can you label and assign this PR✌ — Reply to this email directly, view it on GitHub <#143 (comment) <#143 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY . You are receiving this because you were mentioned.Message ID: @.> Which part specifically?? — Reply to this email directly, view it on GitHub <#143 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVTK64KOEHCNXMRFE5PFR3DXKU7T5ANCNFSM6AAAAAAZBR24QY . You are receiving this because you were mentioned.Message ID: @.**>

Okay I will open an issue for this, till then please label this PR!

Yes!
On Sun, Jun 11, 2023, 09:56 Debsourya Datta @.***>
wrote:

Like login and register page and also templates On Sun, Jun 11, 2023,
09:34 Debsourya Datta @.
> wrote: … <#m_6429983990887973322_> Yess Can you make it responsive? …
<#m_8525000024726928252_> On Sat, Jun 10, 2023, 14:57 Debsourya Datta @.>
wrote: Thanks @tusharzalte https://github.com/tusharzalte
https://github.com/tusharzalte https://github.com/tusharzalte
https://github.com/tusharzalte https://github.com/tusharzalte
https://github.com/tusharzalte https://github.com/tusharzalte for
merging, can you label and assign this PR✌ — Reply to this email directly,
view it on GitHub <#143
https://github.com/tusharzalte/tusharesume/pull/143 (comment) <#143
(comment)
https://github.com/tusharzalte/tusharesume/pull/143#issuecomment-1585582963>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
https://github.com/notifications/unsubscribe-auth/AVTK64KBAS3FSKWY7S7HNYDXKQ4ZXANCNFSM6AAAAAAZBR24QY
. You are receiving this because you were mentioned.Message ID: @.> Which
part specifically?? — Reply to this email directly, view it on GitHub <#143
(comment)
https://github.com/tusharzalte/tusharesume/pull/143#issuecomment-1586004381>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTK64KOEHCNXMRFE5PFR3DXKU7T5ANCNFSM6AAAAAAZBR24QY
https://github.com/notifications/unsubscribe-auth/AVTK64KOEHCNXMRFE5PFR3DXKU7T5ANCNFSM6AAAAAAZBR24QY
. You are receiving this because you were mentioned.Message ID: @.**>
Okay I will open an issue for this, till then please label this PR!
—
Reply to this email directly, view it on GitHub
https://github.com/tusharzalte/tusharesume/pull/143#issuecomment-1586007861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTK64OMQTUINP3S4VTE7TDXKVCHNANCNFSM6AAAAAAZBR24QY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello @tusharzalte , can you label the PR as discussed on the GSSOC Discord server.

Hello @tusharzalte @ZapeeoSheikh 👋, please label the gssoc & level2 tags on this PR since it is not being counted on the GSSOC points table without the tags.


Feed slug validation is not consistent with GetStream docs

I am getting a feed_slug validation error with this slug: "user-aggregated".
I was able to create this feed from the dashboard and the docs don't mention much about the naming convention.
Issue at: BaseFeed::validFeedSlug
Exception received: StreamFeedException(feed_slug can only contain alphanumeric characters or underscores)

Dash is fine for feed id but not for feed type slug.
That's why validFeedSlug (for feed type) is correct, which you created in dashboard (because there is no API support to create it from SDK). The other one is validUserId (for feed id), where you're creating from SDK.


noEmit compilerOption not supported

I have a project with an existing grunt workflow and I'd like to add the conversion from TypeScript to JavaScript using grunt-ts. This should also allow folks who do not use atom-typescript to have a watcher task generate the JavaScript code.
Of course I'd like to also allow usage of all the cool features while editing ts with atom-typescript. To avoid friction I wanted to switch of JavaScript code generation by adding the noEmit compiler option to the tsconfig.json file.
{
    "version": "1.5.0-alpha",
    "compilerOptions": {
        "noImplicitAny": false,
        "noEmit": true,

However I am getting an error which comes from atom-typescript at this line.
The related schema can be found at http://json.schemastore.org/tsconfig which is linked from https://github.com/Microsoft/TypeScript/wiki/tsconfig.json.
Could this be added easily that is by simply changing the validation or are deeper reasons why atom-linter should generate the JavaScript?
On the hand it might be even better if this option could be enabled in an atom-typescript specific configuration file in the project, so that the grunt-ts can use the same tsconfig.json file.

Could this be added easily that is by simply changing the validation or are deeper reasons why atom-linter should generate the JavaScript?

Nope. Just didn't come across it. Pushed with 4.6.2 but haven't tested.
FWIW you probably want https://github.com/TypeStrong/atom-typescript/blob/master/docs/tsconfig.md#compileonsave  Setting it on tsconfig.json will disable it for tsc command line tool as well and you probably don't want that.
:rose:

Thank you!


Dead link https://go.microsoft.com/fwlink/?linkid=843471

Points to https://github.com/Microsoft/vsts-tasks/blob/master/Tasks/AzureRmWebAppDeployment/README.md but the folder is renamed

Currently pointing to https://github.com/Microsoft/azure-pipelines-tasks/blob/master/Tasks/AzureRmWebAppDeploymentV4/README.md now


fix: use zap logger

Reason for This PR
closes: https://github.com/roadrunner-server/roadrunner/issues/2028
Description of Changes

Properly convert raw payloads with the updated logger.

License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

[x] All commits in this PR are signed (git commit -s).
[x] The reason for this PR is clearly provided (issue no. or explanation).
[x] The description of changes is clear and encompassing.
[x] Any required documentation changes (code and docs) are included in this PR.
[x] Any user-facing changes are mentioned in CHANGELOG.md.
[x] All added/changed functionality is tested.


Title: Adding Log In and Sign Up button with proper styling and functionality

Is your feature request related to a problem? Please describe.
I think a log In and Sign Up button will be more user friendly.
Describe the solution you'd like.
I will probably add those buttons inside the navbar with proper styling using html,css and js.
Describe alternatives you've considered.
Please assign this issue to me under GSSOC'23. I would like to work on it.
Add any other context or screenshots about the feature request here.
No response

Hey, as an community website, login signup isn't required as of now


[SPARK-22903]Fix already being created exception in stage retry caused by wrong at…

…temptNumber
What changes were proposed in this pull request?
This PR fix the wrong attemptNumber in stage retry, it will solve the probem of AlreadyBeingCreatedException thrown by executor when failedStages already created the taskAttemptPath.
Details see: https://issues.apache.org/jira/browse/SPARK-22903
(Please fill in changes proposed in this fix)
How was this patch tested?
manual
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Release 0.2.24

add an option to keep mangled name, thanks to @osiewicz
add syntax highlight for mangled names
bump dependencies

@osiewicz 0.2.24 is out, thank you for your contribution.


string() in List of Builtin Tests table of contents links to wrong section of document

Expected Behavior
Clicking on the string() item in the List of Builtin Tests table of contents items takes the user to the documentation for string() under List of Builtin Filters instead.
Actual Behavior
Clicking on the string() item in the List of Builtin Tests table of contents items should take the user to the documentation for string() under List of Builtin Tests.
This seems to be due to a name collision where the documentation for string() under both sections would generate the same document identifier (id attribute). I was rather confused when trying to read the documentation until I realised what was going on and had to manually scroll to the correct section.
Template Code
N/A
Full Traceback
N/A
Your Environment
N/A

Duplicate of https://github.com/pallets/pallets-sphinx-themes/issues/29, but I'll leave this open since it's more likely to be seen.

Is this still an issue? I just checked the link given above and as far as I've seen it looks fine, unless I am missing something.

yes, it's still an issue

can you elaborate more?

@Enigma04 The issue still exists as outlined in my initial submission. I'm not sure what else there is to elaborate on.

I want to work on this issue, where can I find the code of the webpages?

pallets/pallets-sphinx-themes#29


ENH: added bids-validator options

Same addition as in the freesurfer BIDS-App. When I'm testing cpac runs on our cluster, the bids-validator is a large time sink and this helps me get the testing done quickly with our (already BIDS organized) datasets.
Best,
James

It might be also worth updating the validator - latest version is way faster.

@chrisfilo, you're referring to version 0.24.0, right? That version is way faster, but I think the bottleneck is the i/o between the compute node of our cluster and the server we have the data collected on. when I run the command locally it takes less than 10 seconds, but on our cluster it appears to take minutes if not longer.

Yup - that's the one!


Fix issues submitting to App Store

The App Store seems to have some new requirements which was in conflict with the framework's build settings. I was getting all kinds of errors. This fixes it.

[x] Fix (frustrating) issue getting Bit Code = YES to propagate to Release Build
[x] Fix "Unsupported Arch"
[x] Fix "Non-PIE Binary"
[x] Fix "Invalid Segment Alignment"

Thanks


added support for S3 tags

I've verified the AWS documentation for createMultipartUpload which accepts Tagging as a param. Not sure the best way to test this though.


Whenever the BMLT Format Code TC is used alone, have YAP respond with"Meeting is temporarily closed".

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I think it would be a good feature if [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Mention any other of the Github handles that are also interested in this enhancement
Additional context
Add any other context or screenshots about the feature request here.

this seems like a reasonable request.  i have to think about how to implement it

Currently TC meetings are excluded, with no option to display them.   The thought process here is that the target audience for the phoneline would be folks not familiar with the existing set of meetings.  I'm not sure how to proceed on this.

https://github.com/bmlt-enabled/yap/pull/454

Hi Danny,
Thank you for replying, but I am still receiving these TC meetings in my YAP response.    I just called the helpline and selected number 1, meeting  information. Then selected meetings by city or county, I then spoke Brooklyn NY.  YAP Replied with the top 5 meetings and four of the meetings had the TC format used, yet YAP replied that the meetings were open today in Brooklyn NY.    I also called the helpline and selected meetings by zip code and 4 of the 5 meetings returned had the TC Format code used.   Could this be I'm missing an update in the version of BMLT we are using?  thanks, Maureen

Note:   I have only one meeting physically open today in Brooklyn NY, zip code is 11201   Free At Last,  Maureen

What version are you on?

That's a good question.  I have no idea.   I do not update our web.  I will have to ask Lou, who has taken the reins form Chris Marshall.  What is the current version?   Maureen

The most recent version is always listed here:
https://github.com/bmlt-enabled/yap/releases/latest
—Danny

On Jul 26, 2020, at 1:40 PM, Brooklynmaureen<EMAIL_ADDRESS>wrote:
That's a good question. I have no idea. I do not update our web. I will have to ask Lou, who has taken the reins form Chris Marshall. What is the current version? Maureen
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/bmlt-enabled/yap/issues/442#issuecomment-664018683, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOD3O5E3NXXJQCJDUXHXB3R5RTARANCNFSM4OR55DJQ.

thank you.

If you go into your Twilio account, you will see your call webhook under the phone number.  You can take the URL and replace the PHP page with upgrade-advisor.php and paste that in your web browser to get the version.
Example:
https://example.org/input-method.php?Digits=2
would be
https://example.org/upgrade-advisor.php

On Jul 26, 2020, at 1:42 PM, Danny Gershman<EMAIL_ADDRESS>wrote:
The most recent version is always listed here:
https://github.com/bmlt-enabled/yap/releases/latest https://github.com/bmlt-enabled/yap/releases/latest
—Danny

On Jul 26, 2020, at 1:40 PM, Brooklynmaureen<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
That's a good question. I have no idea. I do not update our web. I will have to ask Lou, who has taken the reins form Chris Marshall. What is the current version? Maureen
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/bmlt-enabled/yap/issues/442#issuecomment-664018683, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOD3O5E3NXXJQCJDUXHXB3R5RTARANCNFSM4OR55DJQ.

Danny, Are all BMLT changes made this way, in our Twilio account?         I emailed these instructions to Lou, but this seems like something I can do.

Twilio has nothing to do with BMLT changes.  It’s the call circuits that connect to Yap.

On Jul 26, 2020, at 1:56 PM, Brooklynmaureen<EMAIL_ADDRESS>wrote:
Danny, Are all BMLT changes made this way, in our Twilio account? I emailed these instructions to Lou, but this seems like something I can do.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/bmlt-enabled/yap/issues/442#issuecomment-664020435, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOD3O655BM4UGBDUFPJMILR5RU6LANCNFSM4OR55DJQ.

Danny, I'm in my Twilio account.  Can you baby step me through this.   I need the senior citizen instructions LOL.   What I am seeing is the dashboard, do I select settings?  I do not see the call webhook under the phone number?

You will need to find someone who is a litle more technical to help you at this point.  This is not the right forum to ask for this level of support.  Perhaps the facebook group is a better place.
With regards to the specific request, TC meetings are filtered out automatically (as of version 3.7.0) if they are not tagged as VM in the BMLT.
I am implementing a feature that would allow someone to include descriptions for formats if they should choose to (for example indicating a meeting is meeting virtually or is a hybrid).

On Jul 26, 2020, at 2:13 PM, Brooklynmaureen<EMAIL_ADDRESS>wrote:
Danny, I'm in my Twilio account. Can you baby step me through this. I need the senior citizen instructions LOL. What I am seeing is the dashboard, do I select settings? I do not see the call webhook under the phone number?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/bmlt-enabled/yap/issues/442#issuecomment-664022275, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOD3O45C4IOLAVGQBVG2KLR5RW5PANCNFSM4OR55DJQ.

thank you


Add product block programmatically

I'm trying to simplify the use of product pages in a larger Community Commerce site by adding the product block to the page template programmatically.
I'd expect it work something like this but it doesn't.  Should I expect this to work?
<?php
$product = BlockType::getByHandle('community_product');
$product->controller->productLocation = 'page';
$product->controller->showProductName = 1;
$product->controller->showProductDescription = 1;
$product->controller->showProductDetails = 1;
$product->controller->showProductPrice = 1;
$product->controller->showWeight = 1;
$product->controller->showImage = 1;
$product->controller->showCartButton = 1;
$product->controller->showIsFeatured = 1;
$product->controller->showGroups = 1;
$product->controller->showDimensions = 1;
$product->render();
?>

Nothing immediately comes to mind why this approach wouldn't work.
What happens when you try, do you get error messages or just no output?
Have you tried putting some debugging statements within the product block controller file to see how far it gets along, whether it finds the product, etc?

I've found that the product list block does not show anything when put on a page and told to retrieve the products from below that page (after moving the product in the sitemap of course).

I believe that this will only show products if the category page is a designated category in Community Store and has products set up in the Product > Categories tab.
This raises an interesting issue, I would prefer to arrange my products as pages in the sitemap as noted in #175 rather than storing all pages under a single Products page and using the Community Store categories system.
It would be great if the product list block could accommodate this and allow products to be selected either from 'Under current Page' (meaning product pages actually located beneath the page) or  'Under current Category' (meaning located using the category system).

@katalysis I was just about to suggest such an option. I think the current options in the 'List Products' drop down could be better described to avoid this current confusion. Perhaps it should be worded more like 'Categorised under current page'. (or your wording may be better as it's simpler)
Then further options could something like 'With a product page under current page'.
I think if someone tackles this extra option it should be in conjunction with the option we discussed here: https://github.com/concrete5-community-store/community_store/issues/175