Adds NetIf functions for filtering MAC addresses

Description
Thanks you @htibosch for suggesting this functionality be placed in a PR of it's own, so here it is...
In summary, this PR adds functionality that can be used on it's own and is also the basis of #1019
There are 3 parts to this PR

Adds ipconfigMULTICAST_MAC_FILTERING. When this is enabled, NetworkInterface_t gets two new function pointers. Those are used to control which multicast MAC addresses should be received by the EMAC hardware.
Adds functions to the SAME70 network driver that allow modification of what multicast MAC addresses are being received by the hardware. This implementation utilizes the 64bit hash match register that is present in the SAME70/V71 microcontrollers. Those changes to the SAME70 driver can serve as a primer for implementing the MAC filtering feature on other platforms. Please read the comments and try to contribute support for other platforms. I can only work on the SAME70 port.
Moves the registering of the solicited-node multicast address from the network driver to vIPNetworkUpCalls(). This again applies to the SAME70 driver. Other network drivers need to be updated to NOT register the solicited node MAC address.

More details about bullet point 3 above.....
IPv6 end-points that have a link-local or global address must be able to receive the so-called solicited node address. This is a multicast IPv6 address that corresponds to the end-point's own IP address. As far as I understand things, being able to receive this special multicast address allows detection of duplicate addresses. I believe the same duplicate IP detection is also used in DHCPv6 as well. In any case, the solicited-node address can change any time the end-point's IPv6 address changes and because of this we cannot "set and forget" the solicited-node MAC address in the network driver's init function.
In this PR, I have added some code to the vIPNetworkUpCalls()  Any time an IPv6 end-point goes UP, I calculate the solicited node multicast MAC address and instruct the network driver to begin receiving it. I have not added the complementary code that stops receiving the "old" multicast MAC. This is in my todo list.
If this PR is received well, I'm hoping to base #1019 onto this PR because #1019 depends on this code.
Hopefully, we can expand this PR with more network drivers. That would in turn ensure that #1019 also supports as many hardware platforms as possible.
p.s. I'm leaving this a work-in-progress for now and as usual, let me know what you think.
Questions 1:
According to the coding guidelines, which one of these is considered better?
#if ( ipconfigMULTICAST_MAC_FILTERING == ipconfigENABLE )

or
#if ( ipconfigMULTICAST_MAC_FILTERING == 1 )

or something else maybe?
Question 2:
I have a feeling that hiding the changes of this PR behind ipconfigMULTICAST_MAC_FILTERING  is kind of useless. I think eventually, all platforms should support this functionality and I don't thnk ipconfigMULTICAST_MAC_FILTERING  should be optional and disabled by default. Multicasts are highly needed especially with IPv6 and it feels like I'm adding a useless defines and bloating the code.
Another reason I think we don't need ipconfigMULTICAST_MAC_FILTERING is because even if a platform doesn't support the two functions that this PR adds, NetworkInterface_t is always memset to zero which causes the pointers to the functions to be NULL and every code in this PR and in #1019 checks those pointers before calling them. In essence, nothing bad will happen if the function pointers are just added without an ipconfig define.
Thoughts?
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Update:
I modified the prvAddAllowedMACAddress() and prvRemoveAllowedMACAddress() functions and added a bunch of notes and documentation. The reason for the extensive comments is in case those functions get used as templates for other ports. I can of course get rid of them if the team thinks they are excessive.
I also modified prvGMACInit() to clean up all specific match and hash match registers and then iterate over all end-points of the network interface and call prvAddAllowedMACAddress() for their MAC address. That last change means that we now just have to properly fill-out our end-points and the network driver will handle the MAC addresses whether all end-points use the same MAC or unique MAC addresses.
Let me know what you all think
p.s. @htibosch is this close to how you were envisioning these filtering functions?

Thanks for the input @htibosch
I completely rewrote the comment describing the two filter functions. I tried striking a balance between what you wanted to say and what I wanted to say.
I also removed the ipconfigMAC_FILTERING altogether. Just to reiterate, All calls to the new functions are checked for null before the call is made, so adding the function pointers to NetworkInterface_t is safe even for ports that don't implement the filter functions yet.
I believe this PR is ready for review.
I am not able to implement any CI testing and I'd be very thankful if someone can help with that.

In FreeRTOS_IP.c and in FreeRTOS_IP_Utils.c you are assuming that ipconfigUSE_IPv6 is always enabled, please make it conditional. The variable xAddressType shall only be define when ipconfigUSE_IPv6 is enabled.
FreeRTOS_IP.c
void vIPNetworkUpCalls( struct xNetworkEndPoint * pxEndPoint )
{
-        IPv6_Type_t xAddressType;
+    #if ( ipconfigUSE_IPv6 != 0 )
         if( pxEndPoint->bits.bIPv6 == pdTRUE_UNSIGNED )
         {
             /* Now that the network is up, pxEndPoint->ipv6_settings should hold the actual address of this
              * end-point. For unicast addresses, generate the respective solicited-node multicast address.
              * Note that the check below guards against the loopback address, the unspecified address,
              * and against the weird scenario of someone assigning a multicast address to the end-point. */
-            xAddressType = xIPv6_GetIPType( &( pxEndPoint->ipv6_settings.xIPAddress ) );
+            IPv6_Type_t xAddressType = xIPv6_GetIPType( &( pxEndPoint->ipv6_settings.xIPAddress ) );
             ...
         }
+    #endif /* ( ipconfigUSE_IPv6 != 0 ) */

FreeRTOS_IP_Utils.c
+    #if( ipconfigUSE_IPv6 != 0 )
         if( pxEndPoint->bits.bIPv6 == pdTRUE_UNSIGNED )
         {
-            xAddressType = xIPv6_GetIPType( &( pxEndPoint->ipv6_settings.xIPAddress ) );
+            IPv6_Type_t xAddressType = xIPv6_GetIPType( &( pxEndPoint->ipv6_settings.xIPAddress ) );
             ...
         }
+    #endif /* ( ipconfigUSE_IPv6 != 0 ) */

Unit testing: after the above changes, the UT's will run well. We do have to look at the coverage scores though:

Got it.
I'm currently working on ipv4/ipv6 separation in the multicast PR and just forgot to do it here. Thanks for reminding me.
Hopefully my changes make it better. I also added some more comments to the else cases with the hope that it helps with the unit tests.

@evpopov , here are changes needed to satisfy unit-testing and to reach a coverage of 100%:
Unit-testing_PR_1065.zip

Thanks @htibosch for the unit test fix. @evpopov can you help in updating the PR.

Updated xIndex to uxIndex
Moved vManageSolicitedNodeAddress() to FreeRTOS_IPv6_Utils.c
I'm ready for the next round of nit-picking.

I am implementing the stm32 network interface multicast stuff as you go, and there are a couple things that can be used across Network Interfaces. I know this is really trivial but can we get the ucIsMulticast = pucMacAddress[ 0 ] & 1U; as a helper somewhere? Making pcLOCAL_ALL_NODES_MULTICAST_MAC accessible was another but you already got that. Maybe providing an implementation of the CRC calculation as well?

@HTRamsey
We sure can. Just to make sure I understand you correctly, do you mean something like this?
#define isMulticastMAC( MacBytes )     ( ( MacBytes[ 0 ] & 0U ) != 0U )
#define isUnicastMAC( MacBytes )       ( ( MacBytes[ 0 ] & 0U ) == 0U )

If this is what you mean, I think the best place would be FreeRTOS_IP_Common.h or FreeRTOS_IP_Utils.h.... or maybe NetworkInterface.h
Now that you brought that up, I think I will also benefit from such a macro in a future PR that I'm working on.
Thoughts anyone?

....Maybe providing an implementation of the CRC calculation as well?

Are you asking about the protocol checksum calculation or are you referring to the crc function that is used calculating the hash match register bit?

The hash match. I'm gonna show my naivety here but is there any reason all of the interfaces wouldn't just use CRC32 with 0x04C11DB7 as the poly? Why is DriverSam using CRC16?

@HTRamsey wrote:

The hash match. I'm gonna show my naivety here but is there any reason all of the interfaces wouldn't just use CRC32 with 0x04C11DB7 as the poly? Why is DriverSam using CRC16?

Here is a third one: LPC18xx, which uses a 32-bit CRC but with a different poly (0xEDB88320).
STM32 indeed uses:
/* IEEE 802.3 CRC32 polynomial - 0x04C11DB7 */

So I think that the hash/CRC calculations should stay in their networkinterface.c

@htibosch I believe that is the same as the rest, just manually writing the bit reversal. The current stm32h7 interface does it like this instead prvRevBits32( 0x04C11DB7 ). If they are all different in general then yeah I agree leaving it in the network interface is best but if it's like a 99% use the standard 0x04C11DB7 then I figured a helper would be nice.

I added a MAC unicast/multicast check macro to NetworkInterface.h as requested by @HTRamsey
I decided to put it there because NetworkInterface.h is the lowest level I could find. Everything else is IP and up. NetworkInterface.h is also pretty empty,
While I was at it, I added a bunch of macros and defines to the SAME70 driver in order to greatly improve the readability of all hash register manipulation code.
@AniruddhaKanhere and @htibosch let me know if you disagree with my changes to NetworkInterface.h or if you feel they need improvement.

@evpopov One more suggestion, I think probably NetworkInterface_t * pxInterface should be a parameter to prvAddAllowedMACAddress and prvRemoveAllowedMACAddress as well like the other interface functions have. I like the idea of storing an interface's data in a struct in its pvArgument, so having the interface as a parameter could let you retrieve that data there if necessary (like the counters and index and such).

@HTRamsey, I don't think this is needed. These are static functions inside NetworkInterface.c and they are only exposed to the outside world through pointers inside the NetworkInterface_t struct. Therefore, if you somehow have a pointer to the network interface, you have pointers to those functions.
Hopefully I understood you correctly. If not, please give me some more details.

@evpopov It's just for the purpose of being able to access pvArgument inside of the NetworkInterface struct. Take a look at the Zynq NetworkInterface and you will see why this is necessary. It has multiple instances of the same interface, and stores the index in pvArgument (Although it probably should have been implemented with a struct pointer stored in pvArgument instead, especially if we assume there can be multiple types of interfaces, which would require an index per type of interface rather than a general one). So to get the particular counters, register, etc for that particular interface instance you need access to pvArgument from the interface struct.

@HTRamsey wrote:

Take a look at the Zynq NetworkInterface and you will see why this is necessary.

I agree that it is useful to pass a pointer to NetworkInterface_t to the MAC filtering functions.
typedef void ( * NetworkInterfaceMACFilterFunction_t )
(
    NetworkBufferDescriptor_t * const pxNetworkBuffer,
    const uint8_t * pucMacAddressBytes
);

Some devices like Xilinx have multiple independent EMAC's, each with their own MAC filter rules.

@HTRamsey I see. I must have been tired last night but now I understand exactly what you mean.
@htibosch I think you mean to say:
typedef void ( * NetworkInterfaceMACFilterFunction_t ) ( struct xNetworkInterface * pxDescriptor,
                                                         const uint8_t * pucMacAddressBytes );

but I get the idea.
@HTRamsey I agree with calling the parameter pxInterface, but for consistency with the other functions, I will stick to pxDescriptor.
Gentlemen, since this PR adds functions to the TCP stack's API, I believe it is of critical importance to get things right the first time. If we don't fix these details now we will have to deal with them as backwards compatibility when we fix them later.
I greatly appreciate this type of input.

@htibosch I hate to bother you, but when you have a moment, would you please take a look at the unit tests. I updated the function definitions and declarations, but I have no idea what I'm doing.
Thanks in advance!

@evpopov you check for overflow when incrementing the hash counters but not the match counters.

@HTRamsey Nice find! Thanks!

@evpopov  and everyone active on this PR, we are just wrapping up a round of testing on the main HEAD and will be pushing out a v4.1.0  either this week or early next week. Till then we will close the merge window. We are excluding this PR out of v4.1.0  and will club it potentially in a next release together with your main multicast PR #1019 and some other contributions and features/bug fixes.  Thank you.

@tony-josi-aws sorry it took me two weeks to address your latest comments. I have been busy with other tasks lately. Let me know if there is anything else that I can change to improve this PR.

/bot run formatting


Incorrect Gitenterprise URL in Vitis AI Library Bitbake Recipe - v2.0

The file:  https://github.com/Xilinx/Vitis-AI/blob/v2.0/tools/Vitis-AI-Recipes/recipes-vitis-ai/vitis-ai-library/vitis-ai-library_2.0.bb
Contains an internal gitenterprise link and needs to be updated with the public link.
Thanks!
--Quenton

Hi @quentonh
We have fixed it in the 2.0 branch not in the v2.0 tag.
https://github.com/Xilinx/Vitis-AI/blob/2.0/tools/Vitis-AI-Recipes/recipes-vitis-ai/vitis-ai-library/vitis-ai-library_2.0.bb


Table: allow passing top-level Header again

Changes
I spent a few days trying to improve the typings for columns but I am convinced now that this is somewhat of an impossible task to do in a backwards compatible way. I tried a few different ideas and felt like I was making progress, but all of them were producing at least one type error in some scenario. The main difficulty here is that our types need to be compatible with the Column that the user might import from react-table.
So I've instead decided to reintroduce compatibility with the top-level dummy Header, effectively undoing the breaking change from https://github.com/iTwin/iTwinUI-react/pull/935 while still supporting the new shape added in that PR. In dev mode, there will now be runtime warning when the legacy columns shape is used, but it will not break anything (meaning we officially support both shapes in v2). As an added benefit, this change actually improves compatibility between v1 and v2, thus making migration easier.
I do not think this is worth spending any more time on, as we are going to migrate to react-table v8 in the next major.
Also, this change locks us out of supporting grouped column headers in v2 (#887), which was the original reason for the breaking change, but I do not think we would have done that in v2 anyway because no one has requested it.
Testing
Tested in vite playground. When passing a top-level Header, there is no TS error, the UI looks correct, and there is a console warning only in dev mode.
Also added unit test for top-level Header.
Docs
Added JSDoc, changeset, and console warning (in dev environment) linking to the migration guide.

@adamhock @LostABike Can you review this?


fix bug: src config could be string or array, and update readme

if the src config is a string type, then will throw an error: { Error: EISDIR: illegal operation on a directory, read errno: -4068, code: 'EISDIR', syscall: 'read' }, and the src type in readme is also wrong

Coverage decreased (-0.5%) to 98.413% when pulling 9294dee9a4c1e9a414d40667a834f559f984ea04 on lovetingyuan:master into c8c702f95af383ebd80bf6701c80737e5592f1ea on selaux:master.

Coverage decreased (-0.5%) to 98.413% when pulling 9294dee9a4c1e9a414d40667a834f559f984ea04 on lovetingyuan:master into c8c702f95af383ebd80bf6701c80737e5592f1ea on selaux:master.

Coverage decreased (-0.5%) to 98.413% when pulling 9294dee9a4c1e9a414d40667a834f559f984ea04 on lovetingyuan:master into c8c702f95af383ebd80bf6701c80737e5592f1ea on selaux:master.

Coverage decreased (-0.5%) to 98.413% when pulling 9294dee9a4c1e9a414d40667a834f559f984ea04 on lovetingyuan:master into c8c702f95af383ebd80bf6701c80737e5592f1ea on selaux:master.


Photovoltaic Model Parameters extraction

could you please send me python code

Hi @MohammedQaraad, this project doesn't have any ready-to-use code.  I think #11 has a starting point you could potentially use to build your own in the meantime.  Otherwise, you can use github's "watch" feature on this repository so that you get updates if/when complete code is added to this repository.


Login Expiration should take you to login page

From John:
My book mark is for my individual board. But, if I got to that link with an expired token, I get a 403 error. It may be better to send me to login. For now, will just update my link to go to the dashboard screen.

For ADFS... What does an expired password do? Does it take you back to Ford login?


feat: reth stage command

Adds a reth stage command to run stages in isolation.
Stages will read the database at the specified path. Some stages may require pre-existing data, which is out of scope for this tool: it has to be acquired elsewhere.
The command uses the minimum number of flags possible to keep the maintenance burden low - specialized config for some stages will not be possible. At some point when we have node config, we should revisit and possibly load the config for stage-specific config as well.

Should this be closed, Stages are mostly interconnected (As in Stage1 needs to run before Stage2) is it possible to run it independently?

@rakita this is for debugging reasons. for example, let's say I backed up the database, and wanted to rerun some changes in the Execution Stage, or the Headers stage, in isolation without running the entire pipeline

Superseded by #716


[V1.1] Watcher runs V1.1

Problem
Ensure Watcher runs V1.1
Impact
Reduce cost and have reliable operation of the the system in V1.1 Op Mode
Proposed Solution
Confirm with Defi Wonderland team that Watcher is running
Acceptance Criteria

[ ] Defi Wonderland confirms successful operation of Watcher

@preethamr updates on this?


feat: Add support for copying files in and out of stopped containers

What does this PR do
This adds support to copy files in and out of non running containers thereby  improving feature compatibility with podman and docker. It fixes https://github.com/containerd/nerdctl/issues/1058
High level design
This PR adds logic in 4 packages

main(Modifies parsing logic for nerdctl cp command)
types(Modifies ContainerCpOptions struct)
container(Adds logic to search for container by long/short Id)
containerutil(Adds logic to copy files out and into non running containers)

Fine level Details
cmd/nerdctl/container_cp_linux.go

Modifies processCpOptions function to include GlobalCommandOptions. Deletes logic for container inspect as we don't rely on inspect to find pid of the running container.
Modifies cpAction to include containerd.Client which is used for searching container by long/short id and by SnapshotService to get the underlying Snapshotter.

main_linux.go

Modifies appNeedsRootlessParentMain to return true for nerdctl cp and nerdctl container cp. This is necessary to access containerd.sock in rootless mode.

pkg/api/types/container_types.go

Modifies ContainerCpOptions to add GlobalCommandOptions and ContainerReq.  GlobalCommandOptions are necessary for determining namespace, conatinerd socket address and name of underlying snapshotter. ContainerReq required for finding container by short/long id.

pkg/cmd/container/cp_linux.go

Searches for container specified in nerdctl cp <containerid> command using ContainerWalker. If multiple containers are found it returns an error.
It returns an error when the specified container for copy is not found.

pkg/containerutil/cp_linux.go

Use containerd snapshot service to copy files into and out of created/stopped containers.
It tries to find [Task](https://github.com/containerd/containerd/blob/main/task.go#L154) from containerd. If it's not found(which is the case on nerdctl container create, it mounts the snapshot onto a temporary directory for copying. Any other type of error returns an error.
If the status of the containerd task is Running, then the root filesystem of the running process is used for copying files into /out of the container. (unchanged from previous commits)
If task is found and status is not Running(Exited container), then we fallback to the snapshot service for copying.
For rootless mode it deletes the nsenter command to enter the namespace of the container process since it's already in the namespace of the rootlesskit child pid.
Introduces a helper function mountSnapshotForContainer which returns the temp directory that the container shapshot is mounted on.

Testing
Added tests for stopped containers for exactly the same existing test cases as running containers. They pass in both rootful and rootless modes.

Please try rebasing to make the CI green


Update beta-F248.md to include bug fix 46

Your PR was made after the release of beta-F248 and will be credited differently.

My bad pimp


Developers who are not in the Windows organization should never use the [contract] attribute.

The [contract] attribute is an artifact of the mechanisms that the Windows organization uses to implement our versioning story and manage the process of creating multiple editions of Windows. There is no scenario for a user outside the Windows organization to use contract attributes. The reason you see this in the auto-converted IDL files is because the base IDL files for those are from the internal-to-windows sources.

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

ID: 6ba1cf74-199b-e28e-34ef-40251faf3137
Version Independent ID: 0512dc30-d181-ff2f-2d2f-9d9a5b41dfb4
Content: Predefined attributes (MIDL 3.0) - UWP app developer
Content Source: winrt-related-src/midl-3/predefined-attributes.md
Service: unspecified
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, Larry, thanks for the feedback/info. I'll work with you via email to make this correction and have it reviewed. In the meantime, I'll close this issue work item. Thanks! ~Steve


READ_PHONE_STATE permission

Hi. Your library added permission READ_PHONE_STATE and app has a problem during install (e.g. avast antivirus detect it how dangerous app).
Please take a look my log manifest-merger-release-report.txt
android:uses-permission#android.permission.WRITE_EXTERNAL_STORAGE
IMPLIED from AndroidManifest.xml:2:1 reason: com.felipecsl.quickreturn has a targetSdkVersion < 4
android:uses-permission#android.permission.READ_PHONE_STATE
IMPLIED from AndroidManifest.xml:2:1 reason: com.felipecsl.quickreturn has a targetSdkVersion < 4
android:uses-permission#android.permission.READ_EXTERNAL_STORAGE
IMPLIED from AndroidManifest.xml:2:1 reason: com.felipecsl.quickreturn requested WRITE_EXTERNAL_STORAGE

Thank you for the heads up, will remove that
On Fri, Jan 16, 2015, 00:20 AlexeyFreelancer<EMAIL_ADDRESS>wrote:

Hi. Your library added permission READ_PHONE_STATE and app has a problem
during install (e.g. avast antivirus detect it how dangerous app).
Please take a look my log manifest-merger-release-report.txt
android:uses-permission#android.permission.WRITE_EXTERNAL_STORAGE
IMPLIED from AndroidManifest.xml:2:1 reason: com.felipecsl.quickreturn has a targetSdkVersion < 4
android:uses-permission#android.permission.READ_PHONE_STATE
IMPLIED from AndroidManifest.xml:2:1 reason: com.felipecsl.quickreturn has a targetSdkVersion < 4
android:uses-permission#android.permission.READ_EXTERNAL_STORAGE
IMPLIED from AndroidManifest.xml:2:1 reason: com.felipecsl.quickreturn requested WRITE_EXTERNAL_STORAGE
—
Reply to this email directly or view it on GitHub
https://github.com/felipecsl/QuickReturn/issues/27.

I don't see this anywhere in the repo. Are you sure about it?

Please take a look at answer https://code.google.com/p/android/issues/detail?id=81287#c11

the targetSdkVersion is defaulted to the minSdkVersion (3 in this case) which prompted the manifest merger to add those permissions.

Could you add uses-sdk to the AndroidManifest.xml:
<uses-sdk android:minSdkVersion="10" android:targetSdkVersion="21"/>

Released to maven central with version 1.5.1
It should be available in a few minutes

Works perfect now, many thanks.


Losing site description after ApplyTemplateProvisioning

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you. If you have a actual question, we would ask you to use PnP Yammer group at http://aka.ms/OfficeDevPnPYammer. Thanks!
Which PnP repository to report the issue?

PnP Sites Core - https://github.com/OfficeDev/PnP-Sites-Core

You can delete this section after you have confirmed target repository for your submission.
Category
[V ] Bug
[ ] Enhancement
Environment
[ ] Office 365 / SharePoint Online
[ ] SharePoint 2016
[ V] SharePoint 2013
I try to create first a subsite in Office Dev Pnp CSOM code:
private Web CreateSubSite(ClientContext clientContext, Web hostWeb, string relativeUrl, string template, string title, string description, int language)
        {
            // check if site already exist
            if (!clientContext.Web.WebExists(relativeUrl))
            {
                // create new subsite
                Web newWeb = hostWeb.CreateWeb(
                    new OfficeDevPnP.Core.Entities.SiteEntity()
                    {
                        Title = title,
                        Url = relativeUrl,
                        Template = template,
                        Description = description, // todo, deze beschrijving wordt niet opgeslagen in de site
                        Lcid = (uint)language
                    }, true, true);
                return newWeb;
            }

            return null;
        }

After executing this method above, the subSite has a description. Everything is working fine at this moment.
But when I do a ApplyTemplateProvisioning Im losing the description.
private bool ApplyProvisioningTemplate(Web newWeb)
        {
            XMLTemplateProvider provider = new XMLFileSystemTemplateProvider(String.Format(@"{0}\Templates\", AppDomain.CurrentDomain.BaseDirectory), "");

            if (provider != null)
            {
                var template = provider.GetTemplate("PeriodeTemplate.xml");

                if (template != null)
                {
                    newWeb.ApplyProvisioningTemplate(template);

                    return true;
                }
            }

            return false;
        }

This is my template xml file:
<?xml version="1.0"?>
<pnp:Provisioning xmlns:pnp="http://schemas.dev.office.com/PnP/2016/05/ProvisioningSchema">
  <pnp:Preferences Generator="OfficeDevPnP.Core, Version=2.7.1609.3, Culture=neutral, PublicKeyToken=3751622786b357c2" />
  <pnp:Templates ID="CONTAINER-TEMPLATE-819C564282654030A12C73A642964170">
    <pnp:ProvisioningTemplate ID="TEMPLATE-819C564282654030A12C73A642964170" Version="1" BaseSiteTemplate="BLANKINTERNET#0">
      <pnp:WebSettings NoCrawl="false" WelcomePage="Paginas/default.aspx"
                       SiteLogo="" AlternateCSS=""
                       MasterPageUrl="{masterpagecatalog}/BM.Intern.Werkruimte/Masterpage.Intern.master"
                       CustomMasterPageUrl="{masterpagecatalog}/BM.Intern.Werkruimte/Masterpage.Intern.master"
                       />
      <pnp:RegionalSettings AdjustHijriDays="0" AlternateCalendarType="None" CalendarType="Gregorian" Collation="25" FirstDayOfWeek="Monday" FirstWeekOfYear="0" LocaleId="1043" ShowWeeks="false" Time24="true" TimeZone="3" WorkDayEndHour="5:00PM" WorkDays="62" WorkDayStartHour="8:00AM" />
      <pnp:SupportedUILanguages>
        <pnp:SupportedUILanguage LCID="1043" />
      </pnp:SupportedUILanguages>
      <pnp:Navigation>
        <pnp:GlobalNavigation />
        <pnp:CurrentNavigation />
      </pnp:Navigation>
    </pnp:ProvisioningTemplate>
  </pnp:Templates>
</pnp:Provisioning>

It looks like my issue is related to an existing issue: https://github.com/OfficeDev/PnP-Sites-Core/issues/737

Hello khha,
I just tried this yesterday by creating a subsite with a description and then applying a template without a description. This is not giving me any problems. Can you confirm if this is still a problem?
I used the november release.

Hi bartdonninger, I updated pnp to the latest version from nuget. And now it is working fine! Thanks everybody. I will close this issue.


Reference file not indexed

When I run metaSV, I get the following error: Reference file hg19_reference/hg19_multifasta is not indexed
What kind of indexing does the reference fasta file need?
Thank you,
Madeline

See http://www.htslib.org/doc/samtools.html and look for faidx. This generates an index of the fasta file so that it can be easily seeked.

That's interesting, I already generated a fa.fai index file with faidx. It's in the same folder as the the multifasta file. I'm not sure what the issue is, then.
Thank you for your rapid reply!

Can you paste a listing of the contents of the directory that contains your reference  hg19_reference/hg19_multifasta?

Ah, I identified the problem- an error in the file name. Sorry about that!

No worries!


Adds a random example

Added an example on random. Wasn't sure of the category to put it in, so I put it under a new category "state"

The fact that this hasn't been merged in all the months it's been open, and without discussion too, indicates that maybe there needs to be more people with repository access...

Or it indicates that I want to promote the random library after the examples has been cleaned up by ongoing work http://elm-lang.org/examples/random


FileLock requires write access to root owned directory /var/lock/

Required information
Operating system:
Gentoo, Archlinux
Observed result or behaviour:
/var/lock/ is only accessible by root but the FileLock requires access to it. We should switch the FileLock directory to /tmp since everyone is allowed to write data here.

I don't know if that's possible for QNX. We have limited write access on those images


Consider using workspaces instead of multiple packages

Using packages leads to PRs like https://github.com/cloudflare/lol-html/pull/143 and https://github.com/cloudflare/lol-html/pull/142 where the same dependency needs to be bumped in multiple places.

Definitely a good idea


Fix find_modal (and accept_modal) to work with nested dismiss_confirm/accept_alert calls.

I use code like this:
$(".delete").on "click", ->
  confirmation = prompt "Are you sure you want to delete the page?", 'Type "delete" (without quotes) to confirm'
  confirmed = confirmation == "delete"

  alert "The page will not be deleted." unless confirmed

  confirmed

With test code like this:
page.accept_alert do
  page.dismiss_prompt do
    page.click_button "Delete"
  end
end

This works in Selenium, but Poltergeist would clear the @processed_modal_messages array when #dismiss_prompt finishes, leaving no modal for #accept_alert to find.

Could this be merged in, please?

Hello @DouweM.
Thanks for your work.
Can you rebase first please? Also test case for this feature would be handy to ensure we will not broke this in the future.

Right you are, I'll address both of those things later today.

Looks like it's not even relevant even more since the entire functionality has been reverted c5cd3300629a8926a8436a6b4b16d0d40d6119d2 :/
Too bad.


Invoking by code to change slowQueryThresholdMs

It's well documented that it's possible to change slowQueryThresholdMs value by code, as it's public static variable.
For example as in the below example:
io.opentracing.contrib.jdbc.JdbcTracingUtils.slowQueryThresholdMs = 100
Unfortunately, the class containing this static variable has a package scope (JdbcTracingUtils), so I'm afraid it's not possible to do so.
The only way of setting this value is just via setting environment variable, but didn't find any way to do it dynamically.
Maybe I'm missing here something?

The class JdbcTracingUtils is not public so you can't change it easily.


changes to 3_1.R and revisions to files with code errors

I believe I finalized the if_else statements that needed to be done, but I believe there is also a merge conflict that needs to be addressed before merging with master.

Also, I am not sure if I linked the files correctly to the correspondin citations.


add Session and SessionTemplate resources

Add these resources to allow the URLs for logging into/out of the server to be auto-discovered.

Things to be done to finalize the code:

[ ] Bring the code up to date with the current master.
[ ] Implement the SessionTemplates for all of the supported access methods.
[ ] Follow the 'describe' pattern used with ConnectorTemplates to allow the UI to discover the required fields.
[ ] Find a way for the clojure/script client to determine the session being used by the client.

The other authentication methods will be added once we have feedback on the session templates for the internal authentication.  The implementation may need some significant refactoring to make it flexible enough to support all of the authentication methods with configuration.


RocksDB information in Slow Query and Statement not displayed

Bug Report
Please answer these questions before submitting your issue. Thanks!
What did you do?
Open statement or slow query UI using TiDB master version (5.0).

What did you expect to see?
See newly added RocksDB related fields which is added in https://github.com/pingcap/tidb/issues/20256.

What did you see instead?
RocksDB related fields are not displayed.

What version of TiDB Dashboard are you using (./tidb-dashboard --version)?

Considering that both master and v4.0.x are using the same TiDB Dashboard release, needs some design to display RocksDB information in 5.0 while not display error in 4.0.x.

Fixed by #872


Implement Unet

I assume this PR was done by accident :)

Yes, sorry please ignore:)

Van: Hoel KERVADEC @.>
Verzonden: zondag 29 september 2024 23:22
Aan: HKervadec/ai4mi_project @.>
CC: AnneLouise @.>; Author @.>
Onderwerp: Re: [HKervadec/ai4mi_project] Implement Unet (PR #2)
I assume this PR was done by accident :)
—
Reply to this email directly, view it on GitHubhttps://github.com/HKervadec/ai4mi_project/pull/2#issuecomment-2381607248, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARBOXU6M7ED47WFLIG26LZLZZBVQXAVCNFSM6AAAAABONWTSYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGYYDOMRUHA.
You are receiving this because you authored the thread.Message ID: @.***>


Can 3ML perform multiple detector data analysis at the same time?

Hello all,
Can threeML analyze BAT, XRT, GBM, LAT data at the same time? Or if I want to do analysis like this, how can I use threeML?

Yes. Please read the documentation. There are multiple examples.

But I didn't find an example of analyzing LAT data in the documentation?

https://threeml.readthedocs.io/en/stable/notebooks/Fermipy_LAT.html

thanks,I have seen this example, but it seems that there is no example of using 3ML to analyze LAT GRB in the document, so I take the liberty to speak here.

There is. Please see the documentation.

ok goodbye


Non-ascii characters in media names cause issues

Describe the bug

From the forum: Importing media files with non-ASCII characters in the file name causes a number of issues because in some cases the characters are URL-encoded and in other cases are not.
Affects

[x] Designer
[ ] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior

Users should be able to import any media file they might have on their computer, regardless of the name.
Steps to reproduce

Create a file with non-ASCII characters in the name
Import the file into App Inventor
Note that the name will be urlencoded in the media drawer

@ewpatton can i work on this ?

@ewpatton, should user allowed to upload a file which having non-ascii name or we should convert it to the equivalent ascii character.
But If we want to convert to ascii character than we need to use java.txt.Normalizer which giving compile time error ?
please need your input


Instructions with link statements

Allow VM instructions to use the link notation, unifying the way machines are linked from VMs and block machines.
Previous syntax for "external instructions" not allowed anymore, and should use the new link syntax.

Can you also change the documentation accordingly?


Fix spelling errors in markup files

read through the documentation and found a number of spelling errors

Thanks. Merged the pull request into internal development branch.


Trigger dragging for newly created component

I'm working on creating a new React library to help build workflows (tentatively titled react-workflow) similar to node-red or one of the demos of jointjs but I'm having trouble figuring out how to use DraggableCore to drag a newly created component (it's created as part of the mousedown on another component).
If you go to the demo page, what I'm trying to accomplish is clicking on the red output connector on the left node and then dragging it to the green input connector on the right node.  Currently onMouseDown on the connector just creates the new <Link /> component, but I'd like to be able to keep the mouse pressed down and then pass all the mouseovers to the onDrag of the Links endpoint.
I've seen a demo by a Facebook engineer for an unreleased PoC called combine, and he appeared to be using Dan Abramov's react-dnd and a flux architecture, but it wasn't obvious to be how to reproduce this using react-dnd.
I'm likely going to need to go with a flux/pubsub type architecture to facilitate other concepts (attaching the links to nodes and updating the link's paths as the nodes are moved), but maybe it's required to do that here as well (have a single global state and just pass the x/y positions to the dumb components).

The more I've thought on this, I think a flux architecture will be needed.  I'll close this for now and report back if I was successful.

I made progress using Redux/global state to support creating and dragging links as part of the same mousedown/drag operation.  There is still a lot of work to do, but things are progressing.


Error:Program type already present: org.apache.http.ContentTooLongException

Error:Program type already present: org.apache.http.ContentTooLongException
I get this error when I try to compile my app after importing the library (with nothing written in the code for coinbase).
`
AGPBI: {"kind":"error","text":"Program type already present: org.apache.http.ContentTooLongException","sources":[{}],"tool":"D8"}
FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:transformDexArchiveWithExternalLibsDexMergerForDebug'.
`

httpcore and httpclient-android lib both have ContentTooLongException.java check the version or do not implementation at same time

Anyone found any solution to it.


Pressing esc in terminal navigation mode navigates to previous line

Issue Type: Bug

Launch orca.
Press ctrl+` to activate an integrated terminal.
Type commands in the terminal.
Press ctrl+up two or thre times.
Press the escape key to return to input mode.

Instead of returning to the input mode, the previous line is displayed as if I had pressed ctrl + up.
I also can't type commands in the terminal.
CC @webczat @isidorn
VS Code version: Code - Insiders 1.45.0-insider (a3083e58e2a9e77f60e531acaa32e18527ae8439, 2020-04-17T08:09:45.292Z)
OS version: Linux x64 5.6.4-arch1-1

@jvesouza thanks for filling this!
Yes, I see the same on macOS. When in terminal navigation mode pressing esc navigates to the previous line. It should exit the terminal navigation mode.
@Tyriar can you maybe investigate?

This was a regression in a big refactor I did, thanks for finding 😃

I tested it here and it is working as expected. Thanks.

@jvesouza awesome. Adding verified label.


Bump: 0.18.0 -> 0.18.0-next

Proposed changes
This PR is a preparation for the next release.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update docker image version of FIWARE GE.
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[X] I have updated the change log (CHANGELOG.md)
[X] I send this pull request to the v0.18.0-next branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Codecov Report

:exclamation: No coverage uploaded for pull request base (v0.18.0-next@84cf8ee). Click here to learn what that means.
 Consider uploading reports for the commit c4ed6d8 to get more accurate results

Additional details and impacted files
@@               Coverage Diff                @@
##             v0.18.0-next      #209   +/-   ##
================================================
  Coverage                ?   100.00%
================================================
  Files                   ?         2
  Lines                   ?      1746
  Branches                ?         0
================================================
  Hits                    ?      1746
  Misses                  ?         0
  Partials                ?         0

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

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


[Feature] Ability to specify the description in workflow_class annotation

Motivation: Why do you think this is important?
Flyte UI currently has the ability to show the description of a launch plan. However, updating the description is not completely straight forward. If we could define the metadata at the same point where we define the schema, it would simplify things greatly. The ask here would be to include a description as part of the @workflow_class annotation.
Goal: What should the final outcome look like, ideally?
Provide a fast and simple way for launch plan descriptions to be updated via the same/preferred mechanism used to define the schema, which is in code.
Describe alternatives you've considered
Calling the API directly is an alternative. However, additional scripts would be required, and significant code would need to be written to couple those descriptions with launch plans.
Flyte component

[ ] Overall
[ ] Flyte Setup and Installation scripts
[ ] Flyte Documentation
[ ] Flyte communication (slack/email etc)
[ ] FlytePropeller
[ ] FlyteIDL (Flyte specification language)
[X] Flytekit (Python SDK)
[ ] FlyteAdmin (Control Plane service)
[ ] FlytePlugins
[ ] DataCatalog
[ ] FlyteStdlib (common libraries)
[] FlyteConsole (UI)
[ ] Other

[Optional] Propose: Link/Inline
Not at this time, but I can look into it.
Additional context
None.
Is this a blocker for you to adopt Flyte
Not a blocker, just a really really nice to have. Especially since the capability already exists, folks can take full advantage of the features if they are simplified.

cc @wild-endeavor @matthewphsmith

@joannepolsky - is your interest in having a description be available on the launch plan?  on the workflow?  or both?  They are different entities.  Is the description for workflow ever displayed anywhere?
@katrogan could use your input here.  Currently the named UpdateNamedEntity endpoint takes any named entity.  This means tasks/workflows/launch plans can all have descriptions right?  Does Admin handle all of them correctly today?

@wild-endeavor yes, admin handles description updates for all named entities (the cli can handle updating them all as well)

I would not be opposed to having the workflow_class decorator take a description field, but it should only be used for the workflow, not launch plans derived from the workflow.  For those, it should be part of the launch plan constructor.
What's unclear however is how this would be implemented.  The best way would probably involve a change on the Admin side - currently the registration endpoint does not allow specifying a description I don't think.
We'd have to either change the admin registration endpoints to allow specifying the NamedEntityMetadata field, or we'd have to make this two separate calls to admin in the registration process.  Not sure which one I'd prefer.
I don't think this is super high priority for us though, as the functionality to assign the description already exists, if manual.  Unless you want to work on it @joannepolsky I don't see us getting to this this quarter or possibly even next.

My vote would be for having two API calls and avoiding a single frankenstein endpoint since registration isn't really network-constrained.

I think this is better represented and we should instead do the documentation story https://github.com/lyft/flyte/issues/531


Bug: intermittent flash of loading state while using MacOS build

The loading state, I believe for authentication, flashes intermittently while using the built MacOS version of the app. I'm unsure whether it's due to the token being refreshed and wiping away all page contents after that, or if it's just because authentication is failing but retrying in some way. Here is a demonstration, the first flash occurs around 13s in:
https://github.com/KittyCAD/modeling-app/assets/23481541/fa780607-b6f3-453d-9410-7a49b46d820b

Hey hey no this doesn't happen anymore I'll close sorry


Added delay before drag, issue  #75

Adds an delay for the touchstart event. This fixes item click and scrolling.
Uses config holdToDrag in ms.

Feature works for me. Thanks

Now that a workaround #77 is available for iOS10 I'm fine with merging this.
@timruffles What's your take on this PR?

LGTM!


Add _multiprocessing stub and rename datetime.cs

@slide Yes, good catch.

Currently getting this on test_datetime
======================================================================
2018-11-25T01:37:13.8933857Z ERROR: test_datetime_datetime_pickled_by_cpy (__main__.DatetimeTest)
2018-11-25T01:37:13.8934224Z ----------------------------------------------------------------------
2018-11-25T01:37:13.8934663Z Traceback (most recent call last):
2018-11-25T01:37:13.8935039Z   File "D:\a\1\s\Tests\modules\misc\test_datetime.py", line 1114, in test_datetime_datetime_pickled_by_cpy
2018-11-25T01:37:13.8935644Z     pickled_cpy_dt = pickle.load(f)
2018-11-25T01:37:13.8936163Z   File "D:\a\1\s\Src\StdLib\Lib\pickle.py", line 1548, in _load
2018-11-25T01:37:13.8936441Z     return _Unpickler(file, fix_imports=fix_imports,
2018-11-25T01:37:13.8936710Z   File "D:\a\1\s\Src\StdLib\Lib\pickle.py", line 1038, in load
2018-11-25T01:37:13.8937014Z     dispatch[key[0]](self)
2018-11-25T01:37:13.8937287Z   File "D:\a\1\s\Src\StdLib\Lib\pickle.py", line 1167, in load_string
2018-11-25T01:37:13.8937559Z     self.append(self._decode_string(codecs.escape_decode(data)[0]))
2018-11-25T01:37:13.8937827Z TypeError: expected str, got bytes

Test case is also failing with CPython (although with a different error message), will need to look into it.


Source Control replaces the message that user just typed for next commit.

Issue Type: Feature Request
In "Source Control" view, using GIT, the message that was just typed in, ready for the next commit, was replaced with message from the history of commits when you press 'up arrow' at top of message box. Even though I discovered I could get it back, this was an unnerving experience. Should at least show visually the list of check-ins been stepped over with a way to immediately go back to the "unknown next check-in", or better yet remove the feature.
VS Code version: Code 1.53.2 (622cb03f7e070a9670c94bae1a45d78d7181fbd4, 2021-02-11T11:48:44.518Z)
OS version: Linux x64 4.15.0-135-generic

Previously reported in #111555
I posted a workaround there, but maybe the underlying issue needs reviewing.

You can get back to the typed message by navigating back "down". So in this case it is by-design.


[BUG]: Does not work behind reverse proxy.

I am attempting to use bginx to proxy connections to this running in a Docker container and I keep getting a:
"Status: Not Connected"
Message when loading the UI. I am able to see and interact with the full UI when running with flag --ui-ip=<IP_ADDRESS> but when accessing the same message appears. Additionally, in the STDOUT I see the error "Missing client id". I've tried with both --ui-pass and without. I also do not get a client connected in the STDOUT of the container this is running in at any point.
I am using Nginx Proxy Manager to serve the UI.

Update: Seems this is related to https://github.com/logdyhq/logdy-core/discussions/21 although I am unable to get this to work on my end yet.

Adding the following to Edit > Custom Locations worked for me. This is the edit proxy host modal in NPM.
sub_filter_types text/xml text/css text/javascript; sub_filter 'new WebSocket("ws://"+window.location.host+"/' 'new WebSocket("wss://"+window.location.host+"/'; sub_filter_once off; proxy_buffering off;


fix filtering by included resource

when using of custom filter verifier - context is missing
All Submissions:

[x] I've checked to ensure there aren't other open Pull Requests for the same update/change.
[ ] I've submitted a ticket for my issue if one did not already exist.
[x] My submission passes all tests. (Please run the full test suite locally to cut down on noise from travis failures.)
[ ] I've used Github auto-closing keywords in the commit message or the description.
[ ] I've added/updated tests for this change.

New Feature Submissions:

[ ] I've submitted an issue that describes this feature, and received the go ahead from the maintainers.
[ ] My submission includes new tests.
[ ] My submission maintains compliance with JSON:API.

Bug fixes and Changes to Core Features:

[x] I've included an explanation of what the changes do and why I'd like you to include them.
[ ] I've provided test(s) that fails without the change.

Test Plan:
Reviewer Checklist:

[ ] Maintains compliance with JSON:API
[ ] Adequate test coverage exists to prevent regressions

@lgebhardt do I need to add some tests here?


Better testBasicPersistence

Pull Request Test Coverage Report for Build 31

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.462%

Totals

Change from base Build 29:
0.0%

Covered Lines:
10

Relevant Lines:
26

💛  - Coveralls


(chores) camel-kafka: fix commit integration tests not running by default

❌ Finished component verification: 1 component(s) test failed out of 1 component(s) tested

Finally it's working!


Adding a new service to manage loading/progress bar manually

I think we also need a service which handles showing loading/progress bar manually in some other common use cases too which are asynchronous in nature but may not necessarily be the XHR requests which is what this project is currently aimed at to provide a solution for.
I think it wouldn't be a complete solution until we have a full control on showing and hiding a loading bar even manually in some usecases such as using setTimeOuts, reading data from the offline browser storage, page transitions etc;
Thanks,
Bhasker

you can now use LoadingBarService to manage loading bar manually,
see https://github.com/aitboudad/ng-loading-bar/blob/master/demo/app/app.ts#L42


Enhancement Support read sync materialized view in async materialized view

Identify the column in async materialized view definition. If it selected a column from a sync materialized view, find the real column from the olap table and check if it is the partition column of the olap table.
Proposed changes
Issue Number: close #37394

Will add regression tests after reviewed.

run buildall

@morrySnow Hi, Do you have time to see this PR an give some suggestion?

Could you give a more specific example and put it into the regression test?

run buildall

run buildall

run buildall

run buildall

run buildall

run p0

run p0

in the pr desc, it's better to write the wrong case if create partition mv when set enable_sync_mv_cost_based_rewrite=false;

in the pr desc, it's better to write the wrong case if create partition mv when set enable_sync_mv_cost_based_rewrite=false;

i see that the https://github.com/apache/doris/issues/37394 exists, this is good!

run buildall

@seawinde @morrySnow Hi, I have added some regression tests, please review again.

LGTM


Bump oras modules and remove containerd replace

Description of changes:
Fixes CVE in github.com/containerd/containerd
We had an old replace directive, so I removed it. It wasn't necessary, but I also bumped oras, which are the modules that need containerd.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/retest


Allow map-layer requiring authentication to be correctly initialized after a saved view restore.

When loading a saved view, containing a map-layer definition that requires authentication, the UI needs to be able to:

Provided feedack to the end-user indicating a layer has been attached but requieres credentials to be provided:

Once end-user has provded crdentials, UI needs "revive" the map-layer that has been updated: when a map-layer's tileTree failed to initialize, it's TreeOwner object remains with an TileTreeLoadStatus.NotFound state, and cannot be re-used.  To solve this, a new method Viewport.resetMapLayer() has been added to remove the previously created TeeOwner, and start over the load process.

@pmconne Needs your feedback on this one.  I also dont like the design where the UI has to subscribe to every map-layer's provider status event, in case its authentication status  changes during the session.

@pmconne @MarcBedard8 Please review, I need this for completing another incoming PR.

No unit tests?

@Mergifyio backport release/3.5.x


tdesktop: 4.6.2 -> 4.6.3

Release: https://github.com/telegramdesktop/tdesktop/releases/tag/v4.6.3
Changes: https://github.com/telegramdesktop/tdesktop/compare/v4.6.2...v4.6.3
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Telegram not working since 4.6.2 version. I use nixos-stable and +unstable packages with flakes.
$ telegram-desktop -workdir ~/tmp
/run/current-system/sw/bin/telegram-desktop: /nix/store/b13h86pg7lbf6vpc1vwzw6akmakyw1bs-gcc-11.3.0-lib/lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /run/current-system/sw/bin/telegram-desktop)
/run/current-system/sw/bin/telegram-desktop: /nix/store/b13h86pg7lbf6vpc1vwzw6akmakyw1bs-gcc-11.3.0-lib/lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /nix/store/wajjav2jabaaq0laky9bdkmnhxi259wr-openal-soft-1.22.2/lib/libopenal.so.1)
/run/current-system/sw/bin/telegram-desktop: /nix/store/b13h86pg7lbf6vpc1vwzw6akmakyw1bs-gcc-11.3.0-lib/lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /nix/store/b541v954rvqb8ab74j1vzxhv456p7k9l-qtbase-6.4.2/lib/libQt6Core.so.6)
/run/current-system/sw/bin/telegram-desktop: /nix/store/b13h86pg7lbf6vpc1vwzw6akmakyw1bs-gcc-11.3.0-lib/lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /nix/store/jlx2nfpi73sjb0f3096cly5ik8arw9k9-icu4c-72.1/lib/libicuuc.so.72)
$ steam-run telegram-desktop -workdir ~/tmp
/run/current-system/sw/bin/telegram-desktop: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /run/current-system/sw/bin/telegram-desktop)
/run/current-system/sw/bin/telegram-desktop: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /nix/store/b541v954rvqb8ab74j1vzxhv456p7k9l-qtbase-6.4.2/lib/libQt6Core.so.6)

But, looks like libs are OK
$ strings /nix/store/b541v954rvqb8ab74j1vzxhv456p7k9l-qtbase-6.4.2/lib/libQt6Core.so.6 | grep GLIBCXX_3.4.30
GLIBCXX_3.4.30
<EMAIL_ADDRESS>$ strings /nix/store/jlx2nfpi73sjb0f3096cly5ik8arw9k9-icu4c-72.1/lib/libicuuc.so.72 | grep GLIBCXX_3.4.30
GLIBCXX_3.4.30
<EMAIL_ADDRESS>

I made it to work:
$ LD_LIBRARY_PATH=/nix/store/k88zxp7cvd5gpharprhg9ah0vhz2asq7-gcc-12.2.0-lib/lib/libstdc++.so.6 telegram-desktop -workdir ~/tmp

Please check your environment variables, namely $LD_LIBRARY_PATH, as well as whether telegram-desktop and libraries in /run/opengl-driver are linked to the same libstdc++.so.6.

Backporting tdesktop updates to release-22.11: https://github.com/NixOS/nixpkgs/pull/217154


Separate width and height parameters for Image widgets

This change makes it easier to defined only the width or the height of an image
and let the other value be filled in from the image's intrinsic aspect ratio.
Fixes #175

lgtm


WIP - Feature/module breakdown

feat: created transaction.event.service and adjusted transaction module to have all dependencies imported which required moving the sender/handler and logger into an injectable as we have previously done with config and cache, but was able to step through and it looks like it is injecting things properly

In the screenshot above you will see the module with all of the dependencies injected - however, the code is not finalized. I only made the changes to transaction module so far so leaving the PR as WIP.
In addition, some of the dependencies that are being injected would change because right now, I am simply injecting the new TransactionEventService only to handle _handleTransactionEvent but other event handlers may have a mix of other services. I did not touch the other handlers yet.
Using this approach, things can me imported where needed so the modules can only import the services and services can import their appropriate repositories and feign clients etc... This will allow us to easily break down the handlers into multiple classes as creating classes will become significantly easier with being able to inject any dependency as arguments into the constructor as long as it is appropriate and makes sense and not like a circular dependency...
Creating WIP PR for early exposure

I just updated the rest of the modules aside from OcppRouter to also use dependency injection - coming along smoothly, but the rest of the refactoring with breaking down the handlers into separate smaller pieces / services will have to happen gradually!

Outdated.


Undefined name: F in modules.py

https://github.com/vsitzmann/siren/blob/master/modules.py#L484

Thank you, it has been fixed!


[terra-table] Prevent right click from selecting row in selection mode

Summary

What was changed:
Prevents right click from selecting rows in row selection mode.
Why it was changed:
Unintended selection of row through right click.
Testing

This change was tested using:

[x] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[ ] Accessibility review
[ ] Functional review

Additional Details

This PR resolves:
UXPLATFORM-10312

Thank you for contributing to Terra.
@cerner/terra

@saket2403 @allanxavierm Added DNM label, as there was new requirement along with right click fix. Will include new fix in same PR


Don't install robe if lsp is enabled

Signed-off-by: Huy Duong<EMAIL_ADDRESS>

Good catch! Thanks!


Quote stats in MB/gas

State access count is likely linearly related to gas used. This metric would give more extrapolatable findings

Resolved by https://github.com/perama-v/archors/commit/094d07cc7af56c0f6f1696cc5e87ed38971da2ff


[JENKINS-50394] Add ability for JenkinsRule tests to insert custom GitClient implementation

As what is returned from Git.getClient
JENKINS-50394 - When new commits are pushed to the repository while GitSCMSource is discovering branches then MissingObjectException
To be able to verify a fix for JENKINS-50394 I needed some way of detecting various phases in the descovery process of AbstractGitSCMSource, an easy way would be if I can use a custom "mocked" version of GitClient during the test that injects some more refs in the middle of the fetch/discoverBranches sequence.
Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[X] I have referenced the Jira issue related to my changes in one or more commit messages
[X] I have added tests that verify my changes
[X] Unit tests pass locally with my changes
[ ] I have added documentation as necessary
[X] No Javadoc warnings were introduced with my changes
[X] No spotbugs warnings were introduced with my changes
[ ] I have interactively tested my changes
[ ] Any dependent changes have been merged and published in upstream modules (like git-client-plugin)

Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[X] Infrastructure change (non-breaking change which updates dependencies or improves infrastructure)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Further comments
Added a guard so that this only activates during JenkinsRule tests so that normal production code won't be affected of able to be otherwise "persuaded".
Upstream to https://github.com/jenkinsci/git-plugin/pull/702

@MarkEWaite @stephenc @bitwiseman @dwnusbaum @abayer @kshultzCB @car-roll


React variant of generated App UserManagement Broken

Describe the bug
Install as normal, but select react for the front end
To Reproduce
Trying to get to the User Management screen results in this

Hello @deepu105 do you have some hints to troubleshoot the above issue. I'm not familiar with React at all and since it's based on the default generation I'm wondering if there is no obvious "configurations" we could have missed.

I use React, and with current version of generator UserManagement works fine

Thats strange. You are not modifying the front end right. Also it could be that there is an issue but it only happens when certain conditions are met. I'll take a look when I have time. In the meantime may be compare the exact same config with a default app without blueprint

I think this has been overcome by events. Seems like this is working with the latest generator.
@sirAlexanderT Please try the latest and let us know. I'm closing now but we can reopen if you are still running into issues.


How to set up on my own server?

Hi, I want to set it up on my Ubuntu Server - can you give me a "howto"?

Hello,
Yes. I apologize for the lack of documentation in the readme.
You're going to want to ensure that you have nodejs installed. If not look into downloading nvm (node version manager)
Once you get node setup, you're going to want to fork this repository. Then git clone it to your server.
after you've cloned it, run yarn. This will install all of the required packages.
configure it by adding a .env file in the root directory of the project. See the configuration section of README.md
after you've configured it.
run yarn build, and then yarn preview. This will build and then run the server.
I will look into writing better documentation for those not using vercel.

Fantastic! Thank you, it works. How to add some content now?

Post from a client like https://habla.news and the post will appearSent from my iPhoneOn Apr 4, 2023, at 3:46 PM, Georg Kaser @.***> wrote:
Fantastic! Thank you, it works. How to add some content now?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>


Implement new block-level documentation diagnostics

This pull request includes the implementation of three brand-new StyleCop rules, along with code fixes for each. A description of each of these rules can be found on the associated issue:

Implement new diagnostic SA1653 PlaceTextInParagraphs (fixes #601)
Implement new diagnostic SA1654 UseChildBlocksConsistently (fixes #602)
Implement new diagnostic SA1655 UseChildBlocksConsistentlyAcrossElementsOfTheSameKind (fixes #603)

I rebased and renumbered these rules to account for SA1652 (EnableXmlDocumentationOutput) which was added in the meantime. I also fixed some warnings which appeared due to new style rules being enforced. :smile:

Some statistics on number of violations:

Project
SA1653
SA1654
SA1655

StyleCopAnalyzers
48
6
9

Roslyn
997
65
6

Code Contracts
60
497
6

:+1:

I'm going to move this to a new project.


Incorrect page title alignment on 'edit page'

As reported on #4222, the page title on the 'edit page' view is incorrectly aligned. git bisect shows that this was introduced in commit 5f08013ad2d0a4c31f49fe4177eca4fad96539d5.
Testing against bakerydemo:

Correct rendering:

Fixed in #4229 (not merged yet).


minor feature: add recorded trades time from first trade made in !trades command

chat1
moment.js: Time from now

Instead of calling it operation time, just say that it has traded recorded from x days ago

Alright, I'll change that to "All trades made were recorded from".

chat2

It will show the exact days.
chat3-final

This will do?
another final

as you wish

Do I need to close this an make another pull request with the latest commit?

Yeah.. this look nicer with true on the third argument of the pluralize function.

Yeah.. this looks nicer with true on the third argument of the pluralize function.

You need to remove where you add the number yourself.
Also, why did you close the issue?

moved to #382
Well... I've done too many commits that aren't necessary and might be just increasing my commit point. better open the new one with the good one.


Simplify tag list reconstruct

continuing of https://github.com/ManageIQ/manageiq/pull/12608 (created on top of this PR - reason for WIP)
🎁 @isimluk
@miq-bot add_label wip
@miq-bot add_label chargeback, refactoring

:+1: Thanks!

Rates cache is now in. We can rebase and unwip. 🍔

@miq-bot remove_label wip

👍 LGTM

Backported to Euwe via https://github.com/ManageIQ/manageiq/pull/13419


chore: adding Compatibility Matrix

and some other minor changes

@Aiee @jievince  plz review this PR !


Update Readme

We need to update the readme to reflect an accurate description of Multinet and anything else future users/contributors might find helpful.

Closed with this PR merge - https://github.com/ManifoldRG/MultiNet/pull/69


[TEST/SANDBOX] docker start failure

What does this PR do?

Possible solution: add retry option to docker_run
Motivation

        if check and process.returncode != 0:
            raise SubprocessError(
>               'Command: {}\n' 'Exit code: {}\n' 'Captured Output: {}'.format(command, process.returncode, stdout + stderr)
            )
E           SubprocessError: Command: ['docker-compose', '-f', '/home/vsts/work/1/s/oracle/tests/docker/docker-compose.yaml', 'up', '-d']
E           Exit code: 1
E           Captured Output:

../datadog_checks_dev/datadog_checks/dev/subprocess.py:78: SubprocessError
---------------------------- Captured stdout setup -----------------------------
Attaching to
---------------------------- Captured stderr setup -----------------------------
Creating network "docker_default" with the default driver
Pulling oracle (container-registry.oracle.com/database/enterprise:<IP_ADDRESS>-slim)...
Get https://container-registry-phx.oracle.com/v2/database/enterprise/manifests/<IP_ADDRESS>-slim: Get https://container-registry.oracle.com/auth?account=dev%40datadoghq.com&scope=repository%3Adatabase%2Fenterprise%3Apull&service=Oracle+Registry: net/http: request canceled (Client.Timeout exceeded while awaiting headers)

https://dev.azure.com/datadoghq/integrations-core/_build/results?buildId=13609&view=logs&jobId=c3a06451-088b-587c-b2b9-012b146267bb&j=c3a06451-088b-587c-b2b9-012b146267bb&t=e44c90e6-e662-54c2-0387-1cf630531015

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

v

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes

/azp run PR Changes


The Story of Grass

Sarah Wang / Sarinara
My game is basically about the journey that starts with grass. Basically you turn from a lawnmower to a cow, and then to a car, and then to a milk carton, and finally into a shopping cart.
You use WASD to move. In the first level, you have to mow all the unmowed long grass without touching the flowers. When you turn into a cow, you have to eat all the grass and even the flowers until everything turns into dirt. Next, you have you transport the milk the cow made to the market. On level 4, navigate the milk carton out of the storage room and into the shopping cart. Finally, bring the shopping cart to the cashier and buy it!
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Hey! Are you here from Angelhacks?


Add markdown link-checker to Github workflow

https://github.com/agencyenterprise/neural-data-simulator/issues/29
Introduction
To make sure that we don't have any broken links in our repo, we should add an action to CI to check for that.
Changes
Add the same link-checker as in neurotechdevkit: https://github.com/agencyenterprise/neurotechdevkit/pull/153
Behavior

New GitHub Workflow action on pull requests
We can check that it passes here (or catches any broken links).

Codecov Report
All modified lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


Cohttp 3.0.0 potentially breaking changes

The upcoming version of cohttp contains many fixes and improvements, some of which are breaking the api: https://github.com/ocaml/opam-repository/pull/17571
Clients should work without change and automatically get tls by default, but may be worth checking that all still works. If some changes are affecting you and you need any help feel free to ping me

Tested locally, it all works nicely 👍
Nice library!


Bearer token "sub" is returning action performer object id now instead of email address

In our actionable email implementation, when user take action on the actionable email, we are calling a Power automate flow which then takes the  current user bearer and does the validation. It was working fine till yesterday as the bearer token "sub" was returning the user email address, but now the same property it is returning user object id which is breaking the validation of our code.
Which one is the trusted one user object id or email address? so that we can go ahead with that to change the code. In the MS document code sample also it was mentioned as email address.
https://docs.microsoft.com/en-us/outlook/actionable-messages/security-requirements#verifying-that-requests-come-from-microsoft

Please email<EMAIL_ADDRESS>as this is an Outlook actionable message issue; they should be able to provide more guidance. Thank you!

Hi Raj kumar/Jon,
I also faced the same issue today, its working for some of the users (Sub is showing email address after encoding the bearer  token) but not working for some users( As after encoding bearer token the sub value is showing as OBJECTID)
Which is working fine earlier in all the environments
Could you please help me on it , if we find any fix on it earlier ?
Do we


core: Improve late init callbacks after startup

Summary
Should fix #584
@nazar-pc can you try this branch? I'm not entirely sure whether this fixes your specific issue, but this change should be good in general.
With this PR, the following order of operations should start working:

Build a component
Call transient_for with a widget that is not connected yet
Add the widget to a window (or any other widget somehow connected to a window)
Return (from init, update or similar)

Checklist

[ ] cargo fmt
[ ] cargo clippy
[ ] cargo test
[ ] updated CHANGES.md

I should add that I was testing this with a synchronous component, but I don't think it will work with an async component, am I correct?

I think solution might be in case top most node is not Window to subscribe to an event when it that widget is attached to something and traverse the tree further until Window is finally reached.
Another edge-case that I think is not handled right now is when dialog is set as transient for some widget, but then that widget is moved to a different window (it is possible to do so, right?), then dialog will remain a transient of a wrong window after such thing happens.

I should add that I was testing this with a synchronous component, but I don't think it will work with an async component, am I correct?

I think it should. It should work in general as long as you attach the root widget of a child component within your init() function. Only if you await between calling transient_for and attaching the widget this would still not work.

I think solution might be in case top most node is not Window to subscribe to an event when it that widget is attached to something and traverse the tree further until Window is finally reached.

I thought there is no such event, but connect_parent_notify might work. I'll investigate this further.

Another edge-case that I think is not handled right now is when dialog is set as transient for some widget, but then that widget is moved to a different window (it is possible to do so, right?), then dialog will remain a transient of a wrong window after such thing happens.

Yes, but only if you do this after init, at which point this is not reasonable to take this into consideration from the side of Relm4.

I think solution might be in case top most node is not Window to subscribe to an event when it that widget is attached to something and traverse the tree further until Window is finally reached.

I thought there is no such event, but connect_parent_notify might work. I'll investigate this further.

It turns out that doesn't work reliable. The widget could be attached to another widget, but that widget has no window yet. We could move up to the "highest" widget without attachment and wait for it to be attached, but that attachment does not guarantee that now there is a window available at the top. Overall, I think the current approach is the best solution.
Also, the code should be ready now.

Can you rebase this on main? It seems to be quite a bit behind.

FYI clippy failed in CI and still fails on main, I think that recursion needs to be fixed

I think that's a false positive. The code looks correct to me, there's no recursion. Unfortunately, it's common for new clippy lint's to create false positives. We've had that several times before and even disabled nightly or beta CI once just because of that.

Looks like it was fixed already (https://github.com/rust-lang/rust-clippy/issues/12133), is some caching involved in CI?

Maybe the fix took some time to propagate to nightly. The CI is fixed now without changing this specific issue.


Typo "storage emulator"→"Storage Emulator"

https://docs.microsoft.com/en-us/azure/cosmos-db/table-storage-how-to-use-python

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@SnehaGunda: I'm sorry - only the author of this article, @sakash279, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Migrations issue: Reference to non-Existen migrations

@sebastian-code please review if the migrations run for you.
I had to modify the migrations files to make it work.
Otherwise we will need to start fresh migrations.

I had the same problem always, I'll check but I think is going to replicate again on my own setup because that issue appears to be related to the PyDanny template. I'll let you know how it went.

I have been testing this @andresgz and it works just fine, with additional test I found than the main issue is related to the makemigrations command. I just ran the migrate and every went smoothly.
To be fair, I'm running now with Docker, so that's something to be kept in mind.

Cool @sebastian-code so It means that we can close this issue. Thanks.


task table(s) for animation frames

Let's use a scipiper task table or two to save image files for the animation frames generated by the functions described in #13 and #26. I think it may be simplest to have one task table for #13 and one for #26...but also be thinking about how these frames are going to get bundled up again into a single movie after they've been generated.
Rows=tasks of the task table should be timesteps. There could be one column=step, which would be to generate an animation frame, or there could be multiple columns=steps where each column is to generate an animation frame for the same timestep but with different resolution, aspect ratio, image format, etc -
whatever is needed to allow us to produce all the different types of GIFs/movies we want to produce.

Should this live in 2_process or 6_visualize? or make a new folder like 3_gif_tasks or something?

6_visualize

We're rolling with the task table now.


crypto-square: missing test-case

Hi,
there is a missing testcase for size() that tests that the square is calculated against the filtered input, not the original input.
the size of ("1 2 3 4") should be 2 not 3.
all tests which test size are with valid chars only.
if wished, i could write this test and create a pull-request.

That would be great, thank you!

This seems to have been addressed. This test is in the current suite:
  xit("size of small square with additional non-nuber chars",function() {
    var crypto = new Crypto('1 2 3 4');
    expect(crypto.size()).toEqual(2);
  });

Yepp, that looks right. Thanks!


command 'extension.relativePath' not found

Hello,
Thank you for your plugin, it is very useful.
However, since the last update, it no longer works.
If I try to use it with CMD + Shift + H or via the command palette, I see the message "command 'extension.relativePath' not found."
I tried uninstalling and reinstalling but it is useless.
Regards

Sorry about that. If you uninstall you should have the previous version. I'll fix it asap
On Thu, Sep 8, 2016 at 2:51 PM +0200, "Guillaume"<EMAIL_ADDRESS>wrote:
Hello,
Thank you for your plugin, it is very useful.
However, since the last update, it no longer works.
If I try to use it with CMD + Shift + H or via the command palette, I see the message "command 'extension.relativePath' not found."
I tried uninstalling and reinstalling but it is useless.
Regards
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

any update about this?

Thank you for your quick response @jakob101.
I'll stay on the previous version pending the fix.

Hey, I'm not able to use current extension because of this issue, either. I couldn't figure out how to go to the previous version, though. Uninstalling it just removes it completely for me.

Does the newest version work for you guys?

same here

Do you have 0.10.0? And have you restarted VSCode?
On Thu, Sep 15, 2016 at 9:57 AM +0200, "Nikos Efthias"<EMAIL_ADDRESS>wrote:
same here
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

yes i had 0.10.0 and yesterday it didnt work and it still doesnt work @jakob101

Which version of vscode are you using? It's tough to debug this.

1.5.2

OS?

ubuntu 16.4

same here; hope @jakob101  fix this bug quickly

Viggo you are on Ubuntu too? Tried a vm with win, pure osx and ubuntu. Can repro only on ubuntu.
On Fri, Sep 16, 2016 at 1:23 AM +0200, "Viggo"<EMAIL_ADDRESS>wrote:
same here; hope @jakob101  fix this bug quickly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Fixed in 0.11.0, please verify @qetee @NikosEfthias

its fixed for me @jakob101

Amazing! Closing :) Thanks for help guys.

Same problem.
VSCode v1.18.1
RelativePath v1.0.0
Re-install etc. doesn't help


[dv/csr_utils] update unmapped_addr calculation

Addresses #6594 and #6596.
This PR fixes how we calculate whether a RAL model has unmapped
addresses by utilizing a new method inside of csr_utils_pkg that
essentially "deletes" all mapped address ranges from the full address
range covered by the RAL block, and splitting whatever remains into a
set of addr_range_t structs.
We place this functionality inside the csr_utils_pkg so we can take
advantage of the get_csr_addrs() and get_mem_ranges() tasks that are
already implemented.
New fields in the env_cfg are created to memoize this information during
RAL initialization time such that we do not have to keep running this
expensive computation repeatedly during simulation.
This approach provides some greater flexibility in
tl_access_unmapped_addr() in the tl_errors_vseq, as we can randomly
pick an unmapped address range to target, which also helps to simplify
things by a fair amount.
Signed-off-by: Udi Jonnalagadda<EMAIL_ADDRESS>

I've been looking at similar things while adding the rom_ctrl RAL model. Have you also seen issues with the tl_protocol_err sequence?
It looks like it drives completely unconstrained addresses which then mismatch with the rtl on is_tl_access_mapped_addr

I've been looking at similar things while adding the rom_ctrl RAL model. Have you also seen issues with the tl_protocol_err sequence?
It looks like it drives completely unconstrained addresses which then mismatch with the rtl on is_tl_access_mapped_addr

@tomroberts-lowrisc can you elaborate the issue?
does it happen at the test tl_errors? If so, that test will trigger all kinds of error cases, it never drives non-error cases (all transaction should have d_error = 1). If you see unmapped addresses are sent from that test, that could be expected.

Hi @weicaiyang (welcome back!) - yes I see cases where the address is unmapped, but d_error == 1'b0 from that sequence. It's something to do with having two RAL models in the design as I didn't see it before.
I'll wait for the fixes from @udinator to go in, then I'll push my dual RAL changes as a draft PR to give an example

Hi @weicaiyang (welcome back!) - yes I see cases where the address is unmapped, but d_error == 1'b0 from that sequence. It's something to do with having two RAL models in the design as I didn't see it before.
I'll wait for the fixes from @udinator to go in, then I'll push my dual RAL changes as a draft PR to give an example

I see. Thanks for the explanation.

Update - I tracked down my issue to traffic being driven on the wrong interface - fixed as part of #6830
With that fix and the changes from this PR, everything looks good on the tl_error sequences for me


Getting transpose symbols to be along the same line

For my expression,
\dot{R}(t)R^{\top}(t) + R(t){\dot{R}}^{\top}(t) = 0
The transpose symbols are not along the same line and this is bugging me. It seems to work on share latex though. How do I correct this?
output

Thanks for the report. It definitely looks like we do not match TeX's layout here.
@dpvc is this expected behavior?

@pkra, yes, this is the expected result for MathJax.  See issue #712 which discusses the same problem (with \hat in that case, but it is the same reason).
As for working around it, the following does get the alignment to be consistent:
\dot{R}(t)R^{\top}(t) + R(t){\smash{\dot{R}}}^{\top}(t) = 0

Note that for your second example in the image, the output is consistent with TeX (if your input is the following:
\dot{R}(t)R^{\top}(t) = - {(R(t){\smash{\dot{R}}}^{\top}(t))}^{\top}

If you remove the outermost braces on the right
\dot{R}(t)R^{\top}(t) = - (R(t){\smash{\dot{R}}}^{\top}(t))^{\top}

you will get the alignment that you desire in both MathJax and TeX.  See the discussion in the issue linked above for details of why these differ.

Thank you for the quick response! I've fixed them now. The problem for me was only the first equation though.


Kubernetes version and nginx helm chart version listed are out of date

From the Slack:

https://ranchermanager.docs.rancher.com/v2.6/getting-started/installation-and-upgrade/install-upgrade-on-a-kubernetes-cluster/rancher-on-aks
This says NOTE: If the version of Kubernetes is updated to v1.22 or later, the version of ingress-nginx would also need to be updated.
AKS doesn't support anything older than 1.22 anymore and the helm version of nginx ingress suggested in step 5 doesn't work.
I would suggest updating step 5 to use the latest helm chart which is 4.6.0

In addition, I think the link in the note should point directly to https://kubernetes.github.io/ingress-nginx/user-guide/k8s-122-migration/

It's surprisingly hard to find out which helm chart to match with which version of Kubernetes. Ideally, we'd have a chart or other resource so users could find out for themselves. I think we should update the code block to the latest helm, in the latest docs page (v2.7) but leave the rest alone as an unknown. I'm going to keep digging to see if there is some way to know.

Looks like they list helm chart Kubernetes correspondences in https://github.com/kubernetes/ingress-nginx/blob/helm-chart-4.6.0/charts/ingress-nginx/CHANGELOG.md

Looks like they list helm chart Kubernetes correspondences in https://github.com/kubernetes/ingress-nginx/blob/helm-chart-4.6.0/charts/ingress-nginx/CHANGELOG.md
...but only as far back as 4.2
3.x compatibility is listed here: https://docs.nginx.com/nginx-ingress-controller/technical-specifications/#supported-kubernetes-versions and overlaps with 4.x compatibility.
I sent an email asking which page users should consult and why 4.x isn't listed on that page

The answer was that those are two separate projects. The charts that are referenced in our code block are a version of ingress-nginx which is maintained by the Kubernetes community. The charts described by the compatibility table are maintained by NGINX directly. See https://www.nginx.com/blog/guide-to-choosing-ingress-controller-part-4-nginx-ingress-controller-options/


[turborepo] CLI is not able to find/detect npm

What version of Turborepo are you using?
latest
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Linux
Describe the Bug
When I am using "npx create-turbo@latest" it is not able to detect npm
But when I replace 'latest' with '1.8.8', it's working
i.e Using "npx<EMAIL_ADDRESS>instead of "npx create-turbo@latest" works
I am using (ubuntu 22.04), (node v18.18.0),  (npm [10.1.0)

Expected Behavior
"npx create-turbo@latest" should produce this behaviour

To Reproduce
Use "npx create-turbo@latest" command in terminal

Reproduction Repo
No response
TURBO-1400

Can you run which -a npm and how you installed npm?

run  which -a npm

Interesting, we determine if a package manager is installed by just calling npm --version (and yarn --version etc) in the OS temporary directory. Can you see what happens if you run COREPACK_ENABLE_STRICT=0 npm --version in whatever directory cd $TMPDIR takes you to?

I'm at a loss, that's the exact command we run to determine if a package manager is installed. I've tried your specific npm/node version combination and I'm unable to reproduce.
A few things:

I could add some debug logging to this codepath to figure what exactly about your setup is causing us to only miss these binaries.
Could you try running npx create-turbo@latest in /tmp?

tried npx create-turbo@latest in /tmp

npx<EMAIL_ADDRESS>works properly
This issue only happens when I run npx create-turbo@latest
I don't know what is causing the problem.

@OMzaiswal can you help us bisect to the last working version for you? Here is the list of versions which had changes:
1.9.0-canary.1
1.9.2-canary.0
1.9.2-canary.1
1.9.4-canary.11
1.9.5
1.9.9
1.10.2-canary.1
1.10.4-canary.0
1.10.9
1.10.13-canary.0
1.10.13-canary.1
1.10.13
1.10.14-canary.0
1.10.14-canary.1
1.10.14-canary.2
1.10.15-canary.0

If you can help us identify when it stopped working for you we can maybe isolate the behavior difference.

I tried all the version that you have mentioned, 1.10.9 is the last working version for me.

Thank you! The change in question:
https://github.com/vercel/turbo/pull/5279/files#diff-37e68db2edd31eaeecf3e31f1da27f873acb031157f134708b10d5b45e0f3de8

There was an issue with my npm, but it is now resolved.
npx create-turbo@latest is working perfectly.
Thanks.

Out of curiosity, what was the issue with your npm? Asking in case other users hit the same issue

I am not sure about the issue but something was wrong with the installation and packages. The solution that worked for me was to uninstall Node and npm, followed by a fresh installation. Now it's working perfectly.

i was also facing the same issue but reinstalling does not fix it but this 1.10.9 version works fine. Thanks


CODE CHANGE: Layout and Redundancy Issues in Contact Sections

Related Issue
There is redundancy in the contact information across three sections: "Get in Touch with BuddyTrail," the footer, and the navbar. Despite different layouts, they serve the same purpose, leading to a disjointed user experience.
Fixes:  #1045
Description
The "Get in Touch with BuddyTrail" section has been merged with the Navbar, leveraging the former's superior layout and informative content. This consolidation reduces redundancy, streamlines the user experience, and enhances overall usability.

Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Checklist:

[X] I have made this change from my own.
[ ] I have taken help from some online resources.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@Picodes10

This UI is looking good as it is another page . If you want you can change layout of this or Find another issue or feature to contribute to. Thank you!