Loading chunk {n} failed

Hello, when loading the page, there is an error loading chunk {n} (n - some number) failed how to fix it? Thanks in advance!

Hi @alekseyLymarev
Please provide a repository to reproduce this error or an example online by forking https://glitch.com/edit/#!/nuxt-hello-world.
Then I'll re-open the issue.

Well as soon as I can reproduce the error I will send you, it appears periodically.

Hi @Atinux
[nuxt] Error while initializing app Error: Loading chunk 6 failed.
I'm getting the same error after build and start
I'm using the express server example and router mode as history
I don't know how to reproduce this on a small project.
I can tell that in Network inspector I see /_nuxt/pages/.37ce4084c1c2cef60da2.js cannot be found (404 Not Found)
I also see this on terminal when building
                        Asset       Size        Chunks               Chunk Names
pages/.37ce4084c1c2cef60da2.js  677 bytes       6  [emitted]         pages/```

Hope this can give a clue on this issue

Thanks!

I have same issue. When the app get bigger, more code will be compiled from webpack.
Then you will get this error more often. Not sure how to fix this!

@vinhtq I was thinking of that, I'm planning to start a clean project and keep adding things one by one to see the breaking point but don't know if it's worth it.
Cheers!

The new version of nuxtjs RC11 talking some fixes about this issue

Fix problem with route chunk names #1461
Fix maximum call stack size exceeded #582
Fix compile error when using scrollBehavior #1517

Try to update to the latest version to see if the issue still exists

I'm still getting the error.
I'll do the decompose thing because it's been a roller coaster experimental project.
Thanks!

Thanks @vinhtq it was the chunk routes name as described in #1461
I had to use a router template, that's the danger of making router manually to support language routes

I also experience it in latest version(V1.3x). In development mode, it is ok, but in production mode, you can meet this error sometimes.
I have no idea how to fix it~

I'm having this issue as well anyone know any solutions?

@campope did you configure your router manually?

Nope running in universal mode using only page and folder names for routes.

Experiencing this issue as well

I'm not sure, but I currently guess that the reason for this bug is wrong caching of your service worker.

Same here @ IE11 Only...

In the past 2-3 months I have got this issue, irregular, mobile devices...

Experiencing this issue as well

Found the reason on our project, verify that you are not importing any nuxt-* (nuxt-link, nuxt-child, etc..) modules inside your components, they're preloaded. (WebStorm imports automatically when completing the typing)

Still have this error on mobile devices with slow connection.
@aviramaz we are used JetBrains IDE too (PyCharm, Python adapted analog of WebStorm), but no nuxt* imports were found.

Same issue.
By checking server log, found that first-time fetching url was '/_nuxt/pages/something.js'.
And from the second time, the url becomes '//_nuxt/pages/something.js', and returns 404.
And that's the error chunk num in manifest.js.

i solved this problem with pm2 kill after i  checked nested page component

@ac88 buddy, what's the solution for that ?

@aviramaz can you please clarify – you say, that problem is that some component is missing ?

maybe chunk routes name
Dynamic Nested Routes must be keep to https://nuxtjs.org/guide/routing#nested-routes

@ac88 i don't have such structure inside project – but problem still exist

Hello I'm getting this issue as well in development mode. I have a small app here on https://github.com/tonycen23/nuxt-routing-bug. The bug can be found on localhost:3000/advertisers.

@tonycen23 @Atinux
I have the same problem here with similar url for personal project => http://localhost:3000/my/adverts
Looks like 'adverts' or 'advert...' whatever path is guilty

@Atinux
Thanks, that was the problem.

I just experienced this on my live site. The site is running in production mode with no service worker, and runs with resourceHints: false to limit the number of . The sequence of events was as follows:

User opened the site home page
New version of the app was deployed to web servers
User linked a link to another page within the nuxt SPA
User saw Loading chunk 2 failed message instead of the new page content

I'm worried that any site deploy could break the SPA for my live users and cause them to need a refresh.

@tctimmeh same situation same problem.

@tctimmeh Yes, I modified error.vue to identify this error and refresh it automatically to fixed this issue.

@yoyo837 i guess something like that should be implemented in nuxt itself.  Can you share your code for identifying this error?

This is due to code-splitting and would produce the same on any Vue application.
The only solution is to keep the old files when deploying the new app, so the best is to upload to a CDN every time you deploy, so the old files are not deleted.
But I guess what we can do is to check this error and do a hard reload to avoid doing so? cc @pi0 @clarkdo

@aldarund
my code:
layouts/error.vue
<template>

      {{ isChunkLoadingFailed ? '页面加载失败' : error.statusCode }}

      {{ isChunkLoadingFailed ? '该页面内容可能已变更并重新实现，将自动刷新页面。如未自动刷新，您的网络可能遭遇劫持，请手动刷新重试' : error.message }}

    <nuxt-link class="button" to="/" v-if="error.statusCode === 404">
      返回首页
    </nuxt-link>

</template>

global :
 window.serverRebuildHook = function() {
    popup.alert('服务器版本已更新，正在刷新本地缓存，请稍后...', {
      duration: 1000 * 5,
      callback: () => {
        location.replace(location.href)
      }
    })
  }

server/index.js :
router.all(/\.js$/, async function(request, response, next) {
  if (!config.dev) {
    const jsFilePath = request.url.split('?')[0]
    if (jsFilePath.startsWith(nuxt.options.build.publicPath)) {
      const srcBuiltPath = path.resolve(nuxt.options.buildDir, 'dist', jsFilePath.substr(nuxt.options.build.publicPath.length))
      const y = await myfs.exists(srcBuiltPath)
      if (!y) {
        // console.log(request.url, request.headers.referer)
        // response.redirect('/')
        response.setHeader('Content-Type', 'application/javascript; charset=UTF-8')
        response.setHeader('Accept-Ranges', 'bytes')
        response.setHeader('Vary', 'Accept-Encoding')
        response.setHeader('Transfer-Encoding', 'chunked')
        response.setHeader('Last-Modified', new Date().toUTCString())
        response.setHeader('Cache-Control', 'no-cache')
        response.send('window.serverRebuildHook && window.serverRebuildHook();')
        return
      }
    }
  }
  next()
}, nuxt.render)

Discussion going on: https://github.com/nuxt/nuxt.js/issues/3389

I solved it on my end by prepending a dot to "publicPath" outuput option in my webpack config file.
output: {
    path: path.resolve(__dirname, './dist'),
    publicPath: './dist/',
    filename: 'build.js'
  },

The same issue.
[vue-router] Failed to resolve async component default: Error: Loading chunk vendors~second-chunk failed.
(error: http://localhost:8080/vendors~second-chunk.js)
[vue-router] uncaught error during route navigation:
Error: Loading chunk vendors~second-chunk failed.
(error: http://localhost:8080/vendors~second-chunk.js)
at HTMLScriptElement.onScriptComplete (app.js:819)

I'm trying to implement the solution provided by @yoyo837 but having some platform specific issues so here's to hoping someone has a better grasp on this..
I'm hosting my app on heroku which runs the "yarn run build"-command in the  heroku-postbuild script, building all the files to the ".nuxt/dist"-folder. When I run it locally I can access the "dist"-folder w/o any problem.
However; runtime in heroku the dist-folder doesn't seem to exist in the ".nuxt"-folder:
[
'App.js',
'client.js',
'components',
'index.js',
'empty.js',
'google-tag-manager.js',
'router.js',
'server.js',
'loading.html',
'middleware.js',
'store.js',
'utils.js',
'views'
]
.. So:
router.all(/\.js$/, async function(request, response, next) {
  if (!config.dev) {
    const jsFilePath = request.url.split('?')[0]
    if (jsFilePath.startsWith(nuxt.options.build.publicPath)) {
      const srcBuiltPath = path.resolve(nuxt.options.buildDir, 'dist', jsFilePath.substr(nuxt.options.build.publicPath.length))
      const y = await fs.existsSync(srcBuiltPath) // <- ALWAYS FALSE SINCE 'DIST' FOLDER DONT EXIST IN THE CONTEXT
      if (!y) {
...
      }
    }
  }
  next()
}, nuxt.render)

I'll admit I'm not all to experienced in Heroku hosting but surely the dist folder must exist because the site works perfectly.
Anyone has a handle on this?


Make Specialize Global Static

Is your feature request related to a problem? Please describe.
Specialize currently exists as a required static member on every class that inherits from Expression and often relies on macros provided by the BinaryExpression, and UnaryExpression classes.
Describe the solution you'd like
We should make Specialize global static and use overloading via concepts to provide alternative implementations between BinaryExpression and UnaryExpression children.
Additional context
We currently need to enforce that Specialize is implemented on every child class of Expression which complicates our macros.

Possibly might take the opportunity to rename it to RecursiveCast, TreeCast or something else.


Decide on environment and deploy pre-release on estate (MO specific)

I don't think that an alpha release is appropriate to put on any branch of the default environment, so I would push for Iris v2.0.0a0 to go into experimental.  Particularly bearing in mind that this page (although it needs updating anyway) states that at any point we could give 2 weeks notice for the default-next environment to become default-current.  This does not seem suitable for an alpha.

Agreed. I don't think this issue should be tracked in SciTools/iris though - it has no meaning outside of the MO's context. Happy to close?

Let's keep this open to capture any further conversation that may arise ...

Why has this been re-opened? I don't think this or any "MO specific" issues should be on github

This has now been done, so I'm going to close it, but again this work has no context externally


fitbit initial push

Added pull method to get lifetime steps and distance travel.
To setup environment:
pip install fitbit

Can you also add the fitbit package as part of the requirements before I merge this? The web app should be able to download those automatically from pip

Cool dude, just did.

Perfect!


#210 Added OpenId module for OpenId/OAuth2 support

I decided to put everything related with OpenId/Oauth2 in a separate module. In order to do so I have removed dependencies adding UserId to OpenIdToken class in this new module instead of having a Tokens array in User class.
The feature still is not completed. I'm testing it using Postman following this post http://kevinchalet.com/2016/07/13/creating-your-own-openid-connect-server-with-asos-testing-your-authorization-server-with-postman/
By the moment I'm facing problems described here https://github.com/OrchardCMS/Orchard2/issues/210 .
It would be also interesting to add demo clients

@sebastienros Well I took a look to Startup.cs in Orchard.Demo module class but I didn't see anything different than what is in Module.cs in Orchard.OpenId module that also derives from IStartup.
I asked ISiteService though Startup constructor but it fails, however using  serviceProvider.GetRequiredService() it works but the by default a tenant doesn't have a base Url.
I though Configure and ConfigureService is called for each tenant having the opportunity of configuring middleware for each tenant. Is this what you mean? If I'm right then using baseUrl won't be a problem because each tenant will have different Authority and Audience when I use the baseUrl instead of the hardcoded url "http://localhost:2918".
I thought current request is not available at the moment of middleware configuration. I will try. But it will be same request for all tenant middlewares innicializataion, isn't it? So, as you point using same url for all the tenants doesn't seem a good idea.
Am I missing something?

That's good. The base url is empty because it's not yet defined in the code. You can change it too if you want when the setup is done to initialize it with the current url.

Done. Now baseurl is set on Setup

As an update, it seems that as I expected the [Authorize] attribute is not checking for the user to be at least authenticated. It's just applying the default authorization policy by default.
c.f.: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Core/Authorization/AuthorizeFilter.cs
And you can see when the change happened here: https://github.com/aspnet/Mvc/commit/f54a9648156d040fa455daffb680001a9928167d
So if you want to check that a user is authenticated, or has a specific permission (as in Orchard concepts) just create an attribute that behaves like this.
Today I am teaching something to @PinpointTownes :tada:

Today I am teaching something to @PinpointTownes :tada:

Nah, it's exactly what I said here: https://github.com/OrchardCMS/Orchard2/issues/210#issuecomment-237649123 :smile:
When you don't specify a policy name, [Authorize] uses the default one... that rejects unauthenticated requests: https://github.com/aspnet/Security/blob/dev/src/Microsoft.AspNetCore.Authorization/AuthorizationOptions.cs#L22.
The behavior should be exactly the same as the one we used to have in MVC 5 (unless Orchard overrides the default policy, of course).
And FWIW, I've personally reviewed the PR that removes the extra "is authenticated check" from MVC's authorization filter: https://github.com/aspnet/Mvc/pull/4385 :sweat_smile:

@jersiovic FYI, the OpenIddict PR that allows you to handle grant_type=password and grant_type=client_credentials token requests was merged and a new package was uploaded on MyGet.org.
You can now do something like that:
[HttpPost("~/connect/token")]
public async Task<IActionResult> Exchange() {
    var request = HttpContext.GetOpenIdConnectRequest();

    // Note: you can use request.IsClientCredentialsGrantType()/IsPasswordGrantType()
    // to distinguish the two types of token requests.

    var user = await _userManager.FindByNameAsync(request.Username);
    if (user == null) {
        return Forbid(OpenIdConnectServerDefaults.AuthenticationScheme);
    }

    // Ensure the password is valid.
    if (!await _userManager.CheckPasswordAsync(user, request.Password)) {
        if (_userManager.SupportsUserLockout) {
            await _userManager.AccessFailedAsync(user);
        }

        return Forbid(OpenIdConnectServerDefaults.AuthenticationScheme);
    }

    if (_userManager.SupportsUserLockout) {
        await _userManager.ResetAccessFailedCountAsync(user);
    }

    var identity = await _userManager.CreateIdentityAsync(user, request.GetScopes());

    // Create a new authentication ticket holding the user identity.
    var ticket = new AuthenticationTicket(
        new ClaimsPrincipal(identity),
        new AuthenticationProperties(),
        OpenIdConnectServerDefaults.AuthenticationScheme);

    ticket.SetResources(request.GetResources());
    ticket.SetScopes(request.GetScopes());

    return SignIn(ticket.Principal, ticket.Properties, ticket.AuthenticationScheme);
}

Ah, great then, everything is well done.

Message received loud and clear ;) ... I will find time tomorrow to submit change supporting it.
Thank you

@PinpointTownes Did you know that the current US flag was designed by high schooler Robert Heft as part of a school project?

@sebastienros haha no, I didn't :sweat_smile:
Now you can say you've taught me something :smile:

@sebastienros  where Orchard is supposed to override the default policy? I expected to found it here https://github.com/OrchardCMS/Orchard2/blob/51e873fb2a78a14d723899f6ed2f8efaa592e4cf/src/Orchard.Security/AuthorizationServiceModule.cs but here nothing is done

We don't have to. I just defined some AuthorizationHandler classes to handle permissions (which are custom Requirement). So the AuthorizeAttribute should just work as is.

Unless the default policy is affected by the fact some IAuthorizationHandler are registered.

On last commit I have added support for password and client_credentials harnessing new hook provided by @PinpointTownes .
When the [Authorize] mystery is solved everything related with flows is finished.
In the mean time I will add roles selection to OpenIdApp edition page.
Related with allowed flows. Don't you think that limiting the allowed flows per OpenIDApp is more interesting than do it per tenant?
I mean, currently OpenIddict doesn't allow Public apps to use Client Credentials or Password flow. This is ok. But why don't we allow to the user say at app level exactly what flows are allowed.
For a confidential app the admin will choose which flows are allowed: Client Credentials or Password flow or both depending if he wants client app is authenticated as an user or its authentication is only based on roles related to the app. Related with Refresh flow I'm a bit confused cause Openiddict allows using scope offline_access with Password Flow but not with Client Credentials Flow? Why this difference?
For a public app, Code flow will be mandatory, but admin could choose if he want to allow Refresh Flow or not.
What do you think of that? Is there a hook in Openidict for controlling when refresh token is allowed?
Looking to offer even more flexibility. Do you think it is useful to allow to the admin to set if JWT token or default ASOS token is going to be used at app level? I mean default ASOS token is safer, so whenever I can use it I would prefer to use it. But maybe for an integration with another app I need JWT token. It would be good to have fine grane control to use JWT token only for that app.
@PinpointTownes do you  think is it possible with OpenIddict?

@jersiovic do you need help with anything, like the Authorize attribute issue, or testing any flow. What is left on your plate?

@jersiovic please consider using git rebase to get rid of the commits that have nothing to do with this PR. You probably used git merge at some point and it created unnecessary noise.

I am squash merging this PR, this way it's clean and we can also contribute to the branch.

Thank you for your answer @PinpointTownes.
@sebastienros I've been busy with the end of my vacation moving to home. Furthermore, first days in the office are very busy. So, I expect to come back with this in the weekend or next week.
Great you merged it, in that way we can focus and discuss separately on each issue.
I tested flows an they worked properly. Current pending issues IMO are the following:

To fix Authorize
To fix problems including inline script with  in Edit.cshtml https://github.com/xkproject/Orchard2/blob/00cdd7dff42dac4b7ab7cc7d5a45b4969c667601/src/Orchard.Web/Modules/Orchard.OpenId/Views/Admin/Edit.cshtml and Create.cshtml https://github.com/xkproject/Orchard2/blob/00cdd7dff42dac4b7ab7cc7d5a45b4969c667601/src/Orchard.Web/Modules/Orchard.OpenId/Views/Admin/Create.cshtml both.
When and app is configured as confidential admin user should also choos if he wants to allow Client Credentials or Password flow or both. Only when Client Credentials is selected it should select roles related to that app.
Add tests for OpenId module.
Add an SPA demo sample using OpenId module.

What do you think of adding them as separate issues?:

Please create the issues, I don't know what they are about in most non-obvious cases, and we can also tag them appropriately.


Removes product team updates

Since the engineering and product teams are now in much closer communication, we don't need this in our standup. And indeed, we haven't had any updates from product in a month or so, so this PR just brings the docs into line with our existing practices.

:+1: there is another mention of an update from the product team in the checklist but i can follow up with that PR.

@cavvia Good catch! A PR would be 👍


recorder for iai_kinect bridge using rosbag

Kinect Studio in windows has the option to record and playback data. For this, I have implemented the kinect2_recorder package that can be used to record rosbag files and playback the data. I think this feature might be useful to many users. Please look into this pull request Thiemo.
I have implemented it similar to the kinect2_viewer that you have written. Let me know if you have any questions. Hope this is helpful :)
Cheers,
Nishanth

Should I have rather sent the pull request to the devel branch?

Hi Nishanth,
thanks for the pull request. I didn't had much time to look into it yet. But I have some questions regarding the pull request.
What is the use-case? Why not simply using rosbag record <topics> and rosbag play <bagfile>? For what is a dedicated recorder needed, what are the advantages?

Hi Thiemo,
Thanks for your reply. For most cases rosbag record and play would do.
However, I wanted to record Kinect2 frames in synchronization with a motion capture system and a humanoid robot for aligned datasets. To solve this problem, I included ZeroMQ interface with the recorder to communicate with a client which was also communicating with the Mocap system.
This could be one specific use-case for the recorder when we want to record frames at specific  time instances. I found very few references for using the rosbag api and it took some time for me. So I thought it might be help to others.
Cheers,
Nishanth

Hi,
I though a lot about this pull request and I decided to not merge it in. I think using the rosbag tool is sufficient for most users. I hope you can understand my decision.
I think the best solution would be, if you place your tools as standalone tools in a own separate public github, so that people who need the functionality could look it up there.


no line numbers found

I have installed appimage . I found no line numbers to use the feature "go to line number" with shortcut Ctrl+g.
is this feature missing or is it an appimage problem ?

Can you share a screenshot?

screenshot attached

I get what you mean now. FromScratch doesn't show line numbers. But with ctrl+g you can still go to e.g. the 10th line.

thanks for the reply.
it is not easy to go to a line number  without counting lines . we can't count line numbers if the count goes up. I am expecting line numbers, as we see on the left side of window in vim and emacs . with visible line numbers,, navigation will become easier.


CI/CD: offline_sev_kbc CI

Signed-off-by: Dov Murik<EMAIL_ADDRESS>
Following #27 , add a build "job" for the offline_sev_kbc. It builds and runs the rust tests with --feature offline_sev_kbc.
How can we test this?
cc: @fitzthum

I'll wait until #31 is merged, then I'll rebase. Hopefully it will solve the CI issues.

rebased on top of main; we can try the github workflow now.


Add support for removing the checkbox column from just the header row

If showCheckboxColumn is true, there is currently no way to hide the checkbox from just the header row, in case you want to not allow "select all" behavior. It would be nice to have this option.

showHeadingCheckBox has been added some time ago, please try it


Command line interface for the conversion of OpenAPI files into Sphinx files

@olivier-heurtier FYI, I updated this PR in the way that if you need a script to generate a reStructuredText snippet from an OpenAPI spec you've got to run the following command:
python -m sphinxcontrib.openapi -i docs/specs/openapi.yml -o foo.rst

not
oas2rst -i docs/specs/openapi.yml -o foo.rst

@ikalnytskyi
This is perfect like that! Thanks.


Use Headers consistently in the different IPDFStream implementations

The Headers functionality is now available in all browsers/environments that we support, which allows us to consolidate and simplify how the httpHeaders API-option is handled; see https://developer.mozilla.org/en-US/docs/Web/API/Headers#browser_compatibility
Also, simplifies the old NetworkManager-constructor a little bit.

/botio test

Thank you for improving this!


DS101-JL SMART SWITCH 1 gang version dump

readResult_BK7231N_QIO_DS101-JL_2023-13-5--18-29-06.zip
That one:
https://allegro.pl/oferta/dotykowy-wlacznik-swiatla-szklany-wifi-tuya-13127485395

Btw, I think you can actually post all of these dump packs in one issue. Not really an issue, but could help with building profiles in batch.

Btw, I think you can actually post all of these dump packs in one issue. Not really an issue, but could help with building profiles in batch.

Separate issues is actually better for if we ever use the github_issues value we're setting.  It also helps me keep track of which ones have been completed.

Added as Tuya Generic DS101-JL 1 Gang Smart Switch


[WiP] Enable compilation with latest Peano

There are (at least) two issues that prevent the current AMD AIE plugin to work with the latest version of Peano (LLVM AIE single core compiler):

Peano now provides initialization code through crt0, so linking against me_basic.o is unnecessary and creates redefined symbols.

Peano expect the entry point to be named main, not _main.

I think this might be a good thing to have just in case folks want to check integration with Peano. With this patch, I'm able to compile and run the matmul examples using direct code generation with the latest Peano.

Very nice! Maybe we should

install latest peano on CI
land this PR with (with an update to the CI files in iree-amd-aie to make them point at the new version installed in 1).
update HOWTO and README

Very nice! Maybe we should
1. install latest peano on CI

2. land this PR with (with an update to the CI files in iree-amd-aie to make them point at the new version installed in 1).

3. update HOWTO and README

Yes! Thanks! I also want to figure out what's the issue with μkernel code path; it compiles, but it hangs and times out when I try to run it. I have no clue whatsoever...

Very nice! Maybe we should
1. install latest peano on CI

2. land this PR with (with an update to the CI files in iree-amd-aie to make them point at the new version installed in 1).

3. update HOWTO and README

Yes! Thanks! I also want to figure out what's the issue with μkernel code path; it compiles, but it hangs and times out when I try to run it. I have no clue whatsoever...

Maybe we should wait with updating the CI machine until the ukernel works as well.

Peano expect the entry point to be named main, not _main.

Here's the line you want to patch:
https://github.com/nod-ai/iree-amd-aie/blob/main/compiler/plugins/target/AMD-AIE/iree-amd-aie/Target/AMDAIETargetLdScript.cpp#L161

Maybe we should wait with updating the CI machine until the ukernel works as well.

For sure, we won't land this until everything relevant is in working order, but it's useful to me (and I assume, others) to have an integration point with the latest Peano. When the time comes, I move this out of draft and it can land.

I have no clue whatsoever...

It's gotta be the _main/main thing?

Peano expect the entry point to be named main, not _main.

Here's the line you want to patch: https://github.com/nod-ai/iree-amd-aie/blob/main/compiler/plugins/target/AMD-AIE/iree-amd-aie/Target/AMDAIETargetLdScript.cpp#L161

Yup, that's there. I also patched a similar one in: https://github.com/nod-ai/iree-amd-aie/blob/9f809c6c27730894275919347f2c44ff40e4a505/compiler/plugins/target/AMD-AIE/iree-amd-aie/Target/AMDAIETargetBCF.cpp#L122
Not sure that one's needed, but it's not breaking anything downstream and I don't want to forget about it.

I have no clue whatsoever...

It's gotta be the _main/main thing?

It's for sure not that (or not only that). The _main/main change happened late last week, and I've been experiencing this for a couple of weeks at least, before that change.

It's for sure not that (or not only that). The _main/main change happened late last week, and I've been experiencing this for a couple of weeks at least, before that change.

Only other thing I can think of is lock intrinsics are borked but then no ukernels would work?

yea so this is what I was seeing (not a hang) when I was testing tip Peano (ie all zeros always).

With ukernel? I'd just like to confirm that the ukernel test failing is a CI test, because there are currently no safety rails with our ukernel pipeline and if the tiling isn't done "just right" then you get bad numbers / crashes.

yea so this is what I was seeing (not a hang) when I was testing tip Peano (ie all zeros always).

How do you know it's not timing out? When it hangs, the driver must have some sort of watchdog that ends the execution after a bit over 16s. The test ending and returning all zeroes does match observed behavior.
Anyhow, I guess there's no way around it, I'll have to take a deep dive into the µkernel architecture. It's very odd that just by updating the back-end compiler, the direct code generation works, but the µkernel doesn't. I suspect it's some silly convention change that needs to be accounted for in that pipeline.

the driver must have some sort of watchdog

you sure about that? lol.
anyway there is a timeout but it causes an error (easy to hit/see if you misconfigure a sync). that's not what you see here (unless our xrt bindings are failing gracefully? @nirvedhmeshram.
anyway locally there's no lag/delay - the results just come back (quickly) as zeros.

dunno why I didn't think of doing this before but here's the info on which version is here in CI
(base) mlevental@mlevental-F7BSC:~/Downloads/LLVM-AIE/llvm-aie-3/llvm-aie/bin$ ./clang --version
clang version 17.0.0 (https://gitenterprise.xilinx.com/XRLabs/llvm-aie 5a044b2342f51cf939e032c349c91c2731c812a2)

https://github.com/Xilinx/mlir-aie/pull/1612

after fiddling with a related issue today, i believe the problem is since we're not linking crt0, we're not correctly setting the stack pointer https://github.com/Xilinx/llvm-aie/blob/aie-public/libc/startup/baremetal/aie2/crt0.S#L13. we need to rewrite the LD script emission (which is currently specifically tuned to chess) to be more like https://github.com/Xilinx/llvm-aie/blob/aie-public/libc/startup/baremetal/aie2/ldfile. I'll get to this soon.

Closed in favor of https://github.com/nod-ai/iree-amd-aie/pull/595


Change the test code to a more spring way

See #37

Pull Request Test Coverage Report for Build 130

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

Totals

Change from base Build 126:
0.0%

Covered Lines:
599

Relevant Lines:
1819

💛  - Coveralls


Exporter raises error when roughness and metallic image textures have different resolution in Blender 2.8

Because currently the GLTF 2.0 exporter in Blender 2.8 will combine the  roughness and metallic maps into one PNG (using seperate channels) they should use the same resolution. When you use different resolutions for these the exporter fails.
Using this node setup:
https://imgur.com/a/GB94aKp
Crash log: https://gist.githubusercontent.com/roxlu/065707a844b890534eb80801e3164c5b/raw/ed54ccddce18e088f5188f772a647605d17d1829/gistfile1.txt

In glTF 2.0, these maps are always combined.  They're part of the same image, and as such they must be the same resolution.
I suppose we could have the exporter up-sample one of the images to match the other, or it could produce a nicer error message.  Would that be worthwhile?  But the resulting file will always have these two channels at the same resolution.

it could produce a nicer error message

Sounds worthwhile to me!

We can leave it open for merging images as a future enhancement.

This was fixed by #1059.


Hash helper update issue 94

Hey @tomazy
needed bit more time on separating out this one. Updated hash helper so that we could use old way (no sub level key conversion if needed).
Please check it out when you have time.
Thanks!

Released other updates as:

1.21.5
1.21.6
1.21.7

We can release this one as 1.22.0 once code review and additional updates if neded are done

Used #112 instead.


Update make_string util to clean up bad values

From exif-samples-master, the type of the seq arguments for a few entries (mostly UserComment) in util.make_string is a list. mypy still succeeds after the type annotation amendment. There is also no value in keeping a list of null bytes.
On another note, many other UserComment only contain spaces and some other fields contained trailing spaces (seen null character as well).
Finally, Nikon MakerNote ISOSetting is int16u[2] and should not be stringified. https://exiftool.org/TagNames/Nikon.html Sample values seen include [0, 0] and [0, 200], which were respectively being converted to '[0, 0]' and '}' by make_string.
The test diff output does not clearly show the effect of .strip(), so here it is along one example of the printable value of the IfdTag:
-  "EXIF UserComment": "                                                                                ",
+  "EXIF UserComment": "",

$ diff -y --suppress-common-lines exif-samples-master/dump exif-samples-master/dump_test
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
MakerNote MakernoteVersion (Undefined): [0, 2, 0, 0]          | MakerNote MakernoteVersion (Undefined): 0200
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined): [0, 0, 0, 0, 0, 0, 0, 0, 0, 0,  | EXIF UserComment (Undefined):
EXIF UserComment (Undefined):                                 | EXIF UserComment (Undefined):

@ianare, if you approve this PR, I can modify the exif-sample-master/dump afterwards to reflect changes :)
:snake: :man_juggling:

Great, thanks! If you can do the PR for the samples, that would be perfect. I'll approve this but wait for the samples to be done before merging to avoid false failures.

Here we go: https://github.com/ianare/exif-samples/pull/5


MDL Molfile v2000: Incorrect padding for M  ALS

Steps to Reproduce

Launch Ketcher
Open molfile-with-als.zip or create same structure
Save it as MDL Molfile v2000
Open any text editor and look at the number of padding for M  ALS

Actual behavior
The amount of padding at the end of the M-block is one space too few
This example uses underscores instead of padding for clarity.
Ketcher:
M ALS 1 2 F C N__
Expected behavior
Three padding
Correct:
M ALS 1 2 F C N___
Screenshots
https://user-images.githubusercontent.com/62464241/209547587-40948e9c-dceb-4a22-9de5-9563456929d6.mp4
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 108.0.5359.125 (Official Build) (64-bit)

Ketcher version
Version 2.7.0-rc.6
Additional context
Add any other context about the problem here.

Between atoms that are designated by one letter (for example C, N) there are 3 spaces as it should be. And between those that have a small letter in the abbreviation 2 spaces


Update code owners

Description
Update code owners
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has documentation added

[ ] Commits are signed as per the DCO using --signoff

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

@VachaShah Do you have permission to merge this PR?

The change is already merged https://github.com/opensearch-project/opensearch-cli/pull/83/files, close this PR.
Thank you


NTR: Anopheles benarrochi B

For new term requests, please provide the following information:
Term Label Anopheles benarrochi B
Synonyms [optional]
Definition [required except for organism name]
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3583324/
Suggested Parent Term [required]
Please look in the ontology hierarchy in a browser such as OLS or choose one from following list:

[ ] Material (independent of other entities and can be maintained through time, such as person, cell, sample)
I think this should go under http://purl.obolibrary.org/obo/NCBITaxon_44543

http://purl.obolibrary.org/obo/NCBITaxon_360405


Warning when building

Hello, when running make html the following warning appears during the build:
search/usr/local/lib/python3.7/site-packages/themes/sphinxbootstrap4theme/search.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a

Getting same thing here
The HTML pages are in ../public.
/usr/local/lib/python3.6/dist-packages/themes/sphinxbootstrap4theme/search.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {% trans %}From here you can search these documents. Enter your search
/usr/local/lib/python3.6/dist-packages/themes/sphinxbootstrap4theme/layout.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.

Same here...
preparing documents... done
/usr/lib/python3.7/site-packages/themes/sphinxbootstrap4theme/layout.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.

writing output... [100%] index
generating indices... genindex
writing additional pages... search/usr/lib/python3.7/site-packages/themes/sphinxbootstrap4theme/search.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {% trans %}From here you can search these documents. Enter your search
/usr/lib/python3.7/site-packages/themes/sphinxbootstrap4theme/layout.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.

I've got a fix for this in the below pull request
https://github.com/myyasuda/sphinxbootstrap4theme/pull/11/commits/2dcf81ff441404a5143891b4ce35aaf8f7b93298

@grbd Thank you.

Is this in master yet ?

no but you could just install the modified one from
pip install git+https://github.com/Hecatron-Forks/sphinxbootstrap4theme


reverse issue fixed for Farsi words

As explained here (https://www.w3schools.com/cssref/pr_text_unicode-bidi.asp), using this style will allow you to keep the order of a word chars when it has a different direction.
This fix if for my issue (#45).
Thank you Ali.

Thanks for contributing :) :heart:


Scaffolding

Adds the start of a new router
Adds a cmdgen tool to use go:generate to generate the interface implementations for new router
Adds a WIP wumpgoctl command to scaffold new bot projects.

Codecov Report

Merging #4 (55c541e) into main (15174b9) will decrease coverage by 42.44%.
The diff coverage is 29.61%.

@@            Coverage Diff             @@
##             main      #4       +/-   ##
==========================================
- Coverage   50.99%   8.54%   -42.45%
==========================================
  Files          73      58       -15
  Lines        6756    3382     -3374
==========================================
- Hits         3445     289     -3156
+ Misses       3228    3035      -193
+ Partials       83      58       -25

Impacted Files
Coverage Δ

router/command.go
0.00% <0.00%> (-100.00%)
:arrow_down:

router/command_router.go
0.00% <0.00%> (-69.67%)
:arrow_down:

router/errors.go
0.00% <0.00%> (ø)

router/options.go
0.00% <0.00%> (ø)

router/parser.go
43.24% <43.24%> (ø)

router/stack.go
76.92% <76.92%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update why-astro.mdx

I corrected the text "Creemos firmemento" to "Creemos firmemente"

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.


Get 2016 Election Data for Foco/Larimer

We have a contact at the City of Fort Collins who has access to the elections people for the city, and potentially county contacts. We need to get election data per precinct for at least the presidential election, and hopefully for state and local issues as well.
Olivia Terrell
<EMAIL_ADDRESS>970-416-2282
215 N. Mason St.

Done!


add core framework of meta protocol proxy

Commit Message: add core framework of meta protocol proxy
Additional Description:
A sub PR to land #18209. This PR is implemented core framework of meta protocol proxy.
This PR is still in active development and needs to wait #18823 to be merged.
Design proposal: https://docs.google.com/document/d/1y_ALyjGp_H1_TRrrRxZxnqENISqeZI6eLn71FAFsZPk/edit?usp=sharing
cc @zhaohuabing
Related API PR: #18823
Risk Level: Mid.
Testing: Wait.
Docs Changes: Wait.
Release Notes: Wait.

still in working.

still in working.


create generic backend to cut dependency on daemon

seeing #17463 gave me the idea that at least somewhat the routers can be 'impl-ized' individually.
If we apply similar changes to the local router, that would successfully split the api.
This is just an sample of what the final version would look like. I am interested in what people think about splitting the different sections into separate routers and separate Backends, vs keeping them all as one, with a single Backend.
Names are subject to change, and I need to write a better commit message.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "network-impl" git@github.com:MHBauer/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@duglin @calavera @icecrime bleh, attention.

yep, designing several interfaces was our plan all together :smile_cat:
/cc @dongluochen

PS: the title could be slightly more descriptive.

This is cool. I'm working on image routes under local.

I'm working on image routes under local.

please, don't touch the image router until #17324 is merged.

LGTM

LGTM

LGTM


[Suggestion] Lighthouse Results

Hi, I'm 4th grade of the waseda university studenets.
I'm belonging to the applied chemistry.
I tried to execute Lighthouse about this website.
The result is the following picture.

The following URL describe the detail of Lighthouse
Lighthouse
The performance and the PWA is not so good.
If you want to provide the better UX, you should try to fix these points.
If you have some motivation to apply the PWA, i may help something!!
(Just, I'm interested in the PWA, I have no experience about the implementation)
In Japan, the Long Holiday(Golden Week!!) will be coming soon,
so I have the plan to try to apply the pwa and submit PR.
How do you think about these suggestion?

Hello @nd-02110114! You must be にっしさん from WRESS 😄Welcome to WaseTime!
I think your proposal is fantastic and would welcome a PR for sure.
However, currently, not much documentation is written (I'm sorry for that 😢), so please let me know if you have any problem developing the app locally. I'm having my final exams soon until 5/10, but I will reply as frequently as possible!
I'd like to say thank you again and please enjoy the upcoming golden week!

I'll never forget about you 😄
Life at NUS is crazy. There is so much work here, but the experience is unforgettably awesome. I hope you're doing well as a 4th year Waseda student!
Absolutely. Please let me know if you have any questions. 😆You think you can set up the environment by following the README.


Using flexbox, first draggable items goes behind latest items

Hi
I have a simple example using DND using flexBox which I don't get right. Just 3  draggable/droppable items in a flexbox container.
When I drag first item into second or third, it goes behind them while dragging. When I drag third item into second or first item, it looks OK.
You could simulate this in: https://codesandbox.io/s/fast-mountain-xt34l9?file=/src/App.js
Do I have missing something?

The problem was that each element has position=relative which creates a new "environment" in order to control zIndex for the element and its children.


[Low] in the end have a note that explains things

Things to add

1/sqrt(t) step size + proof for non-convex

https://arxiv.org/abs/2104.06675


[MINOR] Enable log compaction by default for MDT

Change Logs
Enable log compaction on metadata table by default.
Impact
Will compact log blocks to produce another log file every 5 log blocks.
Risk level (write none, low medium or high below)
medium
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Sure. That's why kept it in draft. Just wanted to check with this PR what all tests are failing.


useReadContracts and useReadContract have different cache behaviour

Describe the bug
While useReadContracts uses the cache and retrieves the latest cached value, useReadContract doesn't.
Just changing a useReadContract to a useReadContracts with just one contract makes my app behave differently, which shouldn't. Or at least, it is not documented.
Link to Minimal Reproducible Example
No response
Steps To Reproduce
Create a localhost project.

Use useReadContract to fetch data. Do it online, it will work.
Disconnect from Internet and refresh. You will get undefined.
Change useReadContract to useReadContracts with the same information. Fetch data.
Disconnect from Internet and refresh. You will get the previous value (which is fine)

Wagmi Version
2.5.7
Viem Version
2.7.19
TypeScript Version
No response
Check existing issues

[X] I checked there isn't already an issue for the bug I encountered.

Anything else?
No response

Could u provide a Minimal Reproducible for it?

I just tested it with  @tanstack/react-query-devtools and couldn't reproduce. All my read queries were marked as stale. Keep in mind that react-query still fetches the data even if stale time did not elapsed, the only difference is that if there's a cache for queryKey, then it's immediatly returned by useQuery, but request is still made on the background and cache is updated


Debug: Show debug state in activity bar?

We are not that aggressive anymore in opening the debug viewlet when you hit a breakpoint. Why not show the debug states as decoration on the debug viewlet itself? I could imagine a yellow pause icon when you are on a breakpoint and the green play icon when you are actually running or debugging a program.
@stevencl @chrisdias @bgashler1 thoughts?

ping @bgashler1 @stevencl
silence I will take as approval and implement this on monday 😊

Sorry, take it as not keeping up :-)
In saying that, I am not opposed to the idea but I wonder why we changed the behaviour and don't switch to the debug viewlet when hitting a breakpoint now?

I am not sure if we should change the current behavior to always reveal the debug viewlet. I think we have a reason to not reveal it all the time because people got distracted. Thinking about viewlet indicators in general, I find it a good idea to show any viewlet state in the activity bar because these indicators are meant for exactly that purpose (the viewlet is not visible but has state that you might be interested in).

I like the arguments ben is making and that is the main reason why I would like to experiment with this to get feedback.

@stevencl or putting it the other way around, can you state concerns you have with showing the debug state in the viewlet? Would people maybe expect they could break/continue from the activity bar? Maybe we should start thinking of making the activity bar more powerful and allowing to invoke actions from it (e.g. the full debug toolbar).

I'm not opposed to the idea. If I had to think up a concern it would be the same one we have about icons in the explorer, that it becomes a bit of a Christmas tree effect. But that is just me trying to think of a concern. So definitely go ahead and experiment with it.
I see also that we switch to the debug viewlet until the user activates another viewlet during debug mode. That makes sense to me too.

What are the badges that we would show in the activity bar? if we showed a play/pause badge it may feel like you can click on it to take an action. If it is multi color (e.g. orange when debugging) then i feel like it becomes a christmas tree very quickly.

Maybe @isidorn could attach screen shots if he has ideas how to present it. Also, I would maybe only show an indication when the execution is currently stopped in a breakpoint because otherwise we indicate debugging state in the status bar.

Is this what you were thinking @isidorn ?

@bgashler1 I want to click on that pause button!

Since we already have all the Debug buttons persist on top (no matter what viewlet we're in), are we certain this is a problem with users?

Are they often running their apps while working in the Explorer and wanting to be notified of hitting breakpoints?
If so, is the "Pause" button turning to a "Play" button not clear enough that they have stopped due to a breakpoint/terminating?

@chrisdias: just an experiment.  Is this less clickable to you?

No, its not :).  One thing to consider, we have a problem where the debug toolbar only shows up when the debug viewlet is visible/active.  i find this problematic if i switch to the explorer while at a break point, where did the toolbar go and why?  we've always discussed having the activity bar buttons be commands, perhaps now is the time to consider.

Yes I was thinking the same when we brought up the concern that the icon looks like you could click it.

@bgashler1 yes I had something in mind like the first picture you attached.
@chrisdias I understand the concern that you might want to click on that pause button, but once the user clicks once and figures it is only a state he will no longer try to click it imho. It looks like the other badges which are also not clickable.
I am fine with pushing this to the backlog and introducing it once we decide we want to move forward on making activity bar buttons be commands

Incidentally, @chrisdias, I don't see the debug toolbar disappear when switching to the explorer viewlet while in debug mode. As @bgashler1 mentions above, the debug toolbar might be enough feedback to communicate the state of the debugger as long as it doesn't disappear when switching to another viewlet when in debug mode?

The debug toolbar does not dissapear

I'm sorry, I was wrong about the toolbar disappearing when you switch viewlets. The thing that I was thinking about was that you have to bring up the debug viewlet to be able to press the "Play" button. When I move away from the debug viewlet, there is no button to press, and I like to press the button rather than hit F5 for some reason. I apologize for the confusion.
Going back to the original question... the problem is that we don't bring up the debug viewlet when we hit a breakpoint as aggressively as before. As brad says we have the toolbar in view and we have the orange status bar.  Does the badge in the activity bar help us here?

Closing this as we have not reached a concenzus and I do not think adding this is solving any issue for the user.


tweaks: Recognizer

Coverage increased (+0.3%) to 68.113% when pulling 9dabd9d8c311052ab8ddb6b15a7a98d576ac0c06 on recognizer-wildcard into 4b502b1be6d7a18f486762dc5155eb727145637d on master.


.NET 7 Version

Hi,
I am also trying to get multiplatform working. I have adapted my dockerfile with the lessons hat you showcase here:
https://github.com/notifo-io/notifo/blob/multiplatform/Dockerfile
But I am running from error to error. Stuff like ArgumentNullException from .NET runtime and now this error:
 > [linux/arm/v7 backend  9/13] RUN dotnet restore --runtime $(cat /tmp/rid):
#53 11.41 Assertion failed: (dc->base.pc_next & 1) == 0 (../target/arm/translate.c: thumb_tr_translate_insn: 9650)
#53 11.77 Aborted (core dumped)

Have you tried it with .NET 7?

I just updated the repo to .NET 7 and seems to be OK, I didn't look too closely at your Dockerfile yet, if you're still stuck I will take a look at the weekend just bump this issue :)

Thanks a lot for your answer.
Interesting, I have this problem now:
 > [linux/arm64 backend  9/13] RUN dotnet restore --runtime $(cat /tmp/rid):
#35 124.3 /usr/share/dotnet/sdk/7.0.100/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.TargetFrameworkInference.targets(54,5): error MSB4184: The expression "[MSBuild]::GetTargetFrameworkIdentifier(net7.0)" cannot be evaluated. Exception has been thrown by the target of an invocation. [/src/src/Notifo.Infrastructure/Notifo.Infrastructure.csproj]

It has been reported before: https://github.com/NuGet/Home/issues/12227... strange that it works for you.


test commit

Pull Request Functional Test Report for #2499 / 2a317b0ed886c60f1be7cb02f5e8e8431bd80de5
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

example-0.1: Topology Test
fd90efda
Log

Cisco 8000E

example-0.1: Topology Test
fd6a96da
Log

Cisco XRd

example-0.1: Topology Test
a2e2cede
Log

Juniper cPTX

example-0.1: Topology Test
dad4fb0d
Log

Juniper ncPTX

example-0.1: Topology Test
12c2ed30
Log

Nokia SR Linux

example-0.1: Topology Test
9e8f10af
Log

Openconfig Lemming

example-0.1: Topology Test
4522ac95
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

example-0.1: Topology Test

Cisco 8808

example-0.1: Topology Test

Juniper PTX10008

example-0.1: Topology Test

Nokia 7250 IXR-10e

example-0.1: Topology Test

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

23 of 187 (12.3%)  changed or added relevant lines in 3 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.9%) to 51.116%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

topologies/binding/options.go
23
29
79.31%

topologies/binding/reset.go
0
10
0.0%

topologies/binding/binding.go
0
148
0.0%

Files with Coverage Reduction
New Missed Lines
%

topologies/binding/binding.go
1
31.93%

topologies/binding/options.go
2
85.45%

Totals

Change from base Build<PHONE_NUMBER>:
0.9%

Covered Lines:
1626

Relevant Lines:
3181

💛  - Coveralls


The problem of H5 keyboard style in flutter's WebView

when i use “”(h5) in android,the keyboard style will change,but in Flutter ， i can't see this change.what's worse，the keyboard style will flash once.
this style is i want to see.

but the webview appear this style.

the h5 is <input type='search' />

the keyboard style of the first pic is i want to see

Hi @libo1223
please follow up on https://github.com/flutter/flutter/issues/56515,
I'm closing the current one as duplicate.
If you disagree please write in the comments and I will reopen it


Allow compendium entities to be displayed on the Grid

Not sure the ramifications of this. I know that fromUuid does not seem to find entities still in Compendiums.

In addition to that a categorisation would greatly enhance the usability. A first drop down menu to select the type (Item, Actor, Compendium, Journal Entry) and a second one that lists all of the selected type.

fromUuid does handle compendium entries, their uuid simply needs to be formatted correctly. Our _onDrop in GmScreenApplication isn't handling compendium drops correctly to store the right Uuid.

This is going to be coming with 08x compatibility update soon:tm:!


Commit new phishing LA and FA

A new version of Yaniv's original User Submitted Phishing Logic App, which is more SCU efficient, and very accurate.
"This solution automates the analysis of user-submitted phishing emails using Copilot for Security. It monitors a shared inbox, triggering a Logic App upon new email arrival. The Logic App processes the email using a Function App, which extracts relevant details, leverages the MDTI Copilot for Security Skills for reputation insights, and sends the results to an email recipient."

@microsoft-github-policy-service agree company="Microsoft"
On Wed, Sep 11, 2024 at 7:25 PM microsoft-github-policy-service[bot] <
@.***> wrote:

@cd1zz https://github.com/cd1zz please read the following Contributor
License Agreement(CLA). If you agree with the CLA, please reply with the
following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:

(default - no company specified) I have sole ownership of
intellectual property rights to my Submissions and I am not making
Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work
for my employer (or my employer has intellectual property rights in my
Submissions by contract or applicable law). I have permission from my
employer to make Submissions and enter into this Agreement on behalf of my
employer. By signing below, the defined term “You” includes me and my
employer.

@microsoft-github-policy-service agree company="Microsoft"
Contributor License Agreement Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the
party signing below (“You”),
and conveys certain license rights to Microsoft Corporation and its
affiliates (“Microsoft”) for Your
contributions to Microsoft open source projects. This Agreement is
effective as of the latest signature
date below.

Definitions.
“Code” means the computer software code, whether in human-readable
or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft
and offered under a license
approved by the Open Source Initiative (www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or
distributing code or other content to any
Project, including but not limited to communication on electronic
mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf
of, the Project for the purpose of
discussing and improving that Project, but excluding communication
that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material
Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement
before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now
or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions
is entirely Your original work.
Should You wish to Submit materials that are not Your original work,
You may Submit them separately
to the Project if You (a) retain all copyright and license information
that was in the materials as You
received them, (b) in the description accompanying Your Submission,
include the phrase “Submission
containing materials of a third party:” followed by the names of the
third party and any licenses or other
restrictions of which You are aware, and (c) follow any other
instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include
Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a
contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your
employer has intellectual
property rights in Your Submission by contract or applicable law, You
must secure permission from Your
employer to make the Submission before signing this Agreement. In that
case, the term “You” in this
Agreement will refer to You and the employer collectively. If You
change employers in the future and
desire to Submit additional Submissions for the new employer, then You
agree to sign a new Agreement
and secure permission from the new employer before Submitting those
Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the
Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive,
royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly
display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all
of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the
Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive,
royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or
the combination of the
Submission with the Project to which it was Submitted to make, have
made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the
Project.
Other Rights Reserved. Each party reserves all rights not
expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without
limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are
legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your
original work (except as You may
have disclosed under Section 3). You represent that You have secured
permission from Your employer to
make the Submission in cases where Your Submission is made in the
course of Your work for Your
employer or Your employer has intellectual property rights in Your
Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You
represent and warrant that You
have the necessary authority to bind the listed employer to the
obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless
You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE
WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED
UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO,
ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of
any facts or circumstances of which
You later become aware that would make Your representations in this
Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to
Projects and information about
contributions may be maintained indefinitely and disclosed publicly,
including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the
laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal
courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in
which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County,
Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire
agreement between the parties, and
supersedes any and all prior agreements, understandings or
communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may
be assigned by Microsoft.

—
Reply to this email directly, view it on GitHub
https://github.com/Azure/Copilot-For-Security/pull/117#issuecomment-2345071451,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKQ754ZXIUWBOC4B5ZAMATZWDUQDAVCNFSM6AAAAABOCDKAHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBVGA3TCNBVGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


fix(tablerow): remove action not checking tableId

Proposed Changes
TableHOCRowActions.remove action was removing all rows with an id that match the id in the action payload no mater what is the tableId specified. This was causing the action to delete rows across tables.
Potential Breaking Changes
None.
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1298 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1298      +/-   ##
==========================================
+ Coverage   99.42%   99.42%   +<.01%
==========================================
  Files         359      359
  Lines        9043     9044       +1
  Branches     2134     2135       +1
==========================================
+ Hits         8991     8992       +1
  Misses         52       52

Impacted Files
Coverage Δ

.../components/table-hoc/reducers/TableRowReducers.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Typing Issues related to amazon-cognito-auth-js

Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
After installing AWS Amplify & AWS Amplify React on a newly created create-react-app Typescript application and importing AWS Amplify in my App.tsx, I get the following error:
C:/Users/***/***/project/node_modules/aws-amplify/lib/Common/Facet.d.ts
(6,34): Could not find a declaration file for module 'amazon-cognito-auth-js'. 'C:/Users/***/***/project/node_modules/amazon-cognito-auth-js/lib/index.js' implicitly has an 'any' type.
  Try `npm install @types/amazon-cognito-auth-js` if it exists or add a new declaration (.d.ts) file containing `declare module 'amazon-cognito-auth-js';`

Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
I'm using the AWS-Amplify ^0.3.3, AWS-Amplify-React ^0.1.43.
Also tried using<EMAIL_ADDRESS>Is this a bug with the library? Is there a way to bypass/fix this?

@vincentsolutions there's now a @types/amazon-cognito-auth-js that can be added to devDependencies. This should suffice until type definitions are added to the amazon-cognito-auth-js package.


Remove "prefer_label_over_filter_text" setting

As discussed in #722 and seen causing confusion with metals in #771.

Coverage decreased (-0.02%) to 34.837% when pulling 09cfb63bc955f6bcd6fa81e914d67e7235e1983f on remove-use-of-filtertext into e72085f848dcda0f283e4ddc3d1753dda0ca733e on master.

I will update the doc in Metals once there is a release.


Unsteady statistics are not initialized by effects

For those folks familiar with statistic effects - do the stats need to be initialized by code, before they can be used? In CBTBE I have a trio of statistics that I want to use to signal a ternary state. If missing, I assume I should use the default value. If present, I look for either a true or false value. I thought I'd achieve this by not initializing the statistics in a Mech.InitEffectStats() patch like I normally would, and allow the effects to create the statistic instead.
However, BTA isn't seeing these effects proc at all. I've poked through StatOp and StatisticEffect and I'm not seeing a 'add if missing' flow. I suspect I've misunderstood how this would work, and it seems like HBS expects the statistics to always be initialized in code before they are referenced by an effect.
Can anybody else confirm this understanding (i.e. you cannot create a statistic via a Set operation in an effect), or share how they achieved this type of flow?

Need to fix the following stats:
    public const string PhysicalWeaponUnsteadyAttackerOnHit = "CBTBE_Physical_Weapon_Unsteady_Attacker_On_Hit"; // bool - if true, attacker will be unsteady on a hit
    public const string PhysicalWeaponUnsteadyAttackerOnMiss = "CBTBE_Physical_Weapon_Unsteady_Attacker_On_Miss"; // bool - if true, attacker will be unsteady on a miss
    public const string PhysicalWeaponUnsteadyTargetOnHit = "CBTBE_Physical_Weapon_Unsteady_Target_On_Hit"; // bool - if true, target will be made unsteady by a hit

Fixed in 0.8.14


Azure Notebooks no longer exist...

[Enter feedback here]
This page mentions Azure Notebooks which is no longer available, but instead spread throughout vscode, AML and Github.

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

ID: 97abbeb4-6c20-4dfd-8a3f-3b448919679c
Version Independent ID: d979d82d-2e80-5a1e-7928-7f8154ed34f3
Content: Choose a data analytics and reporting technology - Azure Architecture Center
Content Source: docs/data-guide/technology-choices/analysis-visualizations-reporting.md
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @EdPrice-MSFT
Microsoft Alias: architectures

@eavanvalkenburg Thanks for the feedback. I have assigned the issue to the content author to investigate further and update the document as appropriate.

Just noticed this is a duplicate of #3172 which was raised a year ago, please fix!

Closing as duplicate of #3172 - Internal tracking: https://dev.azure.com/MSFT-SKILLING/Content/_workitems/edit/24500


Revert "Disable Interpreter/async_fib.swift"

Reverts apple/swift#37498
The failures with debug stdlibs should be fixed by: https://github.com/apple/swift/pull/37461
rdar://78192131

@swift-ci Please smoke test

preset=buildbot_incremental,tools=RA,stdlib=RD,smoketest=macosx
@swift-ci Please test with preset macOS


An error about "donttest"

The version of systemrdl-compiler is 1.71, a Error:
error: Property 'donttest' expects a boolean for non-field types. Got an integer in 'HPS_PAGE'     } HPS_PAGE @ 0x008;
my rdl code is:
   reg HPS_PAGE {         desc = "Provides visibility of the page for host accesses to HPS. Previously this was a configurable value but is now fixed at FPGA image compile time";         donttest = true; // exclude from plain simple hw register test         stat_f hpsPage[31:21]; hpsPage->desc = "If applicable, the address presented to the ARM interface will be prepended by this value.";     } HPS_PAGE @ 0x008;

Thanks for reporting. I have confirmed the issue and will fix.
In the meantime, using an implicit true assignment should work:
reg HPS_PAGE {
  desc = "Provides visibility of the page for host accesses to HPS. Previously this was a configurable value but is now fixed at FPGA image compile time";
  donttest; // exclude from plain simple hw register test
  stat_f hpsPage[31:21];
  hpsPage->desc = "If applicable, the address presented to the ARM interface will be prepended by this value.";
} HPS_PAGE @ 0x008;

Thanks for reporting. I have confirmed the issue and will fix.
In the meantime, using an implicit true assignment should work:
reg HPS_PAGE {
  desc = "Provides visibility of the page for host accesses to HPS. Previously this was a configurable value but is now fixed at FPGA image compile time";
  donttest; // exclude from plain simple hw register test
  stat_f hpsPage[31:21];
  hpsPage->desc = "If applicable, the address presented to the ARM interface will be prepended by this value.";
} HPS_PAGE @ 0x008;

thank you

Fixed in v1.7.2


Fix the build :tada:

We're still waiting on judofyr/temple#86 before we can change back 54a23c5, and cut a release.

:joy: It's beautiful.

:tada:


Fix warning in sgdp and adamp

Codecov Report

Merging #186 into master will decrease coverage by 0.69%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
- Coverage   86.45%   85.76%   -0.70%
==========================================
  Files          20       20
  Lines        1137     1131       -6
  Branches      267      267
==========================================
- Hits          983      970      -13
- Misses         85       92       +7
  Partials       69       69

Impacted Files
Coverage Δ

torch_optimizer/adamp.py
89.87% <100.00%> (-0.38%)
:arrow_down:

torch_optimizer/sgdp.py
83.56% <100.00%> (+5.93%)
:arrow_up:

torch_optimizer/swats.py
72.50% <0.00%> (-15.00%)
:arrow_down:

Continue to review full report at Codecov.

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


opnode: Do not drop all deposits on parsing error

Only drop the affected deposit. This does mean loss of funds, but
this is also a case that should never happen. In addition, halting
the chain means that while funds are safe, we are exposed to a
denial of service attack if it would be possible to cause a parsing
error from the deposit feed contract.
In addition, Change deposit gasLimit from uint256 to uint64
This prevents the user from setting a gas limit that the rollup node
is not able to parse.

I gotta update some more contracts stuff. Will force push when that's done

Codecov Report

Merging #383 (e5c0081) into main (00e65fe) will decrease coverage by 0.63%.
The diff coverage is 60.00%.

:exclamation: Current head e5c0081 differs from pull request most recent head 4f390fc. Consider uploading reports for the commit 4f390fc to get more accurate results

@@            Coverage Diff             @@
##             main     #383      +/-   ##
==========================================
- Coverage   40.77%   40.13%   -0.64%
==========================================
  Files          46       46
  Lines        5791     5890      +99
==========================================
+ Hits         2361     2364       +3
- Misses       3146     3241      +95
- Partials      284      285       +1

Impacted Files
Coverage Δ

opnode/contracts/deposit/deposit_feed_raw.go
0.70% <ø> (-0.05%)
:arrow_down:

opnode/rollup/driver/step.go
62.37% <33.33%> (ø)

opnode/rollup/derive/payload_attributes.go
77.92% <71.42%> (+1.29%)
:arrow_up:

Continue to review full report at Codecov.

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

Can you update https://github.com/ethereum-optimism/optimistic-specs/blob/main/specs/deposits.md to reflect the uint256 -> uint64 gasLimit change?


URL parameter support for lineage report tab

Allow URL parameters to control:

Selected lineages
Selected structure? PDB ID and mode?

Closed by #592


How to use demo as a starter project

Newbie here, please advice how to use this demo part as a starter project.

Interested too.

Interested too.


pull request for e2e test at 02/06/2017 16:15:05

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


make sign in pages accessible

Signed-off-by: susanev<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
we recently did an accessibility pass and there were a bunch on the sign-in pages, this fixes them.
:chains: Related Resources
fixes #3814
:+1: Definition of Done
stuff should work and look the same for the past part, there's one new h1 added on the fist page and one color change to the error message to make it pass the color contrast check
:athletic_shoe: How to Build and Test the Change
rebuild components/automate-dex
some changes to automate-ui were also added so also rebuild that
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

@msorens @srenatus im hoping i can pass this to one of you to finish. prior to this pr once you choose local sign in it takes you to the page and no error is presented at the top unless you enter something bad. after this pr when you use the local sign in button and it takes you to the next page the error shows right away before you have a chance to fill in the inputs. thoughts on how to fix it?

prior to this pr once you choose local sign in it takes you to the page and no error is presented at the top unless you enter something bad. after this pr when you use the local sign in button and it takes you to the next page the error shows right away before you have a chance to fill in the inputs.

That's quite odd, since there's nothing dynamic here -- it's rendered by the server, and Invalid shouldn't be true when you click on "local user". 🤔 I don't see the problem yet.

@srenatus I have never looked at the dex templates before; where is the code that processes those dex templates and fills in the variables (BackLink, Invalid, etc.) ?

@msorens there's nothing special to it, it's using the stdlib template package. The variables are provided when the server renders the template. At any rate, the HTML change here wouldn't possibly change those values. I'll have a look later.

@srenatus after you review and if it looks good, feel free to merge, thank you!


[github-actions] Cert_5_3_10_AddressQuery fails intermittently

Describe the bug
tests/scripts/thread-cert/Cert_5_3_10_AddressQuery.py fails intermittently.
Console/log output

logs_47592.zip
packet-verification-pcaps.zip

From the advertisement the packets, Leader did not expire Router_1's ID after Router_2 power off 580s, add more extra time to test

Add routerselectionjitter=1 for Leader

Add routerselectionjitter=1 for Leader

Have not seen this failure for a while. Closing.


add Apple TV HD dfu instructions

on my device i can get past checkm8 and to kernel boot, but then it panics. waiting for serial cable to check logs

Hmm, not sure if there's much point in adding this right now considering palera1n doesn't currently work on tvOS. It's planned, but we hit an issue with frameworks not loading properly (the error refers to APFS.framework, but it probably just happens to be the first one in the list).
Also, you forgot Apple TV 4K 1st gen (CPID: 0x8011, BDID: 0x2).

@i1p bro stop

it does not let me use palera1n on my Mac

can you just join the discord server (https://dsc.gg/palera1n) instead of commenting on random prs please

How can I jailbreak my iPhone XS Max iOS 16.4.1a

You can't, also you're asking in the wrong place.

on my device i can get past checkm8 and to kernel boot, but then it panics. waiting for serial cable to check logs

Hey! It's been a long time now, but palera1n now has Apple TV DFU instructions starting from: https://github.com/palera1n/palera1n/commit/05e1872ccbcf35ea7ded02edf29da3c0125f4529.
More proper DFU instructions for the ATV were done in a later commit being:  https://github.com/palera1n/palera1n/commit/053a495755008969bb63e409cfe973d32c867af6
Thank you for your initial PR, though we're now gonna close it due to not needing it anymore.


CFR - Request view for Personal requests

As a ministry coord
I want to view details of a personal request
so that I know what records are required

Assumptions & Scope
What are the assumptions for this story?
What is IN scope?
What is NOT in scope?

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

Business area decision needed before we can build this. Creating a separate ticket

Close as duplicate,


Fix data visualizer resizing

Stops the visualizer from limiting the height of the display by creating a second, nested display area with its own set of scroll bars.
This solution is still not perfect, as if the drawing is too tall the user has to scroll to the bottom of the drawing to access the resizing bar and view the REPL.

Pull Request Test Coverage Report for Build 3465

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 36.99%

Totals

Change from base Build 3452:
0.0%

Covered Lines:
2690

Relevant Lines:
6512

💛  - Coveralls


Is it possible to mirror the screen?

Hi, there.
In my university, we've got projectors caged to the ceiling which otherwise work, except that it is very cumbersome to get cables from there.
I've got one of those cheap "Anycast" HDMI dongles that can act as a DLNA/UPnP renderer and would like to use it, instead of a long HDMI cable, to share my screen with students during a lecture. Go2TV works great for casting videos, but I couldn't figure out a way to cast/mirror my screen. Tried piping ffmpeg's x11grab output, but that didn't work ("Broken pipe"). If I dump the screen to a WebM/VP8 file and give that to Go2TV, then mirroring starts (although with some delay), but then breaks off after a few seconds.
Is there an easy way to implement this, or could you give me some pointers as to how to get this working?
Thank you very much for your work!

Hello!
I also tried last month to "trick" ffmpeg to cast my screen but with no luck. Didn't spend too much time on it however. When you dump the screen to a WebM file and pass that to Go2TV, it sees it as a complete video file and only casts the data that already recorded up to this point. Maybe if there was a mode that treated this file as a sequential stream of data rather than a seekable object, maybe we could cast the full thing, but still that would be a hack and the delay would be massive (as you also identified).
Something like changing https://github.com/alexballas/go2tv/blob/main/httphandlers/httphandlers.go#L300 to a io.Copy when in screen casting mode? Just speculating, I would need to run some tests.
The standard way to cast / mirror your screen is with the Wifi Display / Miracast protocol which is an entirely different thing from the UPNP implementation in Go2TV.
I'm a linux user myself but I believe that Windows support wireless display / anycast natively.

I think the only proper solution to that would be to implement the chromecast protocol https://github.com/alexballas/go2tv/issues/38. Any other workarounds would not work properly since the  DLNA protocol is not really designed for that. Closing.


Enhance min frequency checking

Highlighted updates:

Raise a value error when min_frequency is a float, while the minimum row count is set to be less than 1.
Make the item-integer_ID map to be deterministic and the order of output patterns get fixed when sequences use characters.
Add unit tests for the corner cases, such as empty list, one or two rows with min_frequency being a percentage.
Update version to 1.2.1 and the docs.

One question: why the change on HTML files?

The docs also contains a page for the source code of sequential.seq2pat, then we might want to update the html file if there is an update on the code. So I recreate the docs.

@skadio, minor updates have been made at the places regarding the comments, see this commit.

Approved from my end. Feel free to merge as you proceed, no need for another review round.
Does this require recompilation of the backend etc? I don't remember that part.
I assume you will update the PyPI package part once this is merged. And we can follow-up to that issue thread to announce the fix and Close the open Issue.

This doesn't require recompilation since backend hasn't been changed. The Python part takes care of the input validation. And after the merge, I can update PyPI release and follow up the issue.


global: beta release preparation

Ready from our side. Fixed the dot and ensured test cover age 100%

Coveralls is not responding. Otherwise LGTM

LGTM. Coveralls responded now, just LGTM that needs to report status.

LGTM


Fix compile error with TextureButton

Export TextureButton::ResizeMode as its own enum values
Tested on Arch Linux with GCC 6.3.1

It still fails to build on travis: https://travis-ci.org/godotengine/godot/jobs/201105261
scene/gui/texture_button.cpp: In constructor 'TextureButton::TextureButton()':
scene/gui/texture_button.cpp:306:17: error: 'StretchMode' is not a class or namespace

It should be compiled with ancient compilers now.


Zenit3SLBReduzx

8193 [1] FATAL CKAN.NetKAN.Program (null) - Too may .version files located: HorizonAeronautics_V1.32/GameData/AnimatedDecouplers/AnimatedDecouplers.version, __MACOSX/HorizonAeronautics_V1.32/GameData/AnimatedDecouplers/._AnimatedDecouplers.version Build step 'Execute shell' marked build as failure
It gets stuck on a .version file in the AnimatedDecouplers folder. I'm not sure why it's looking there tbh since the CKAN-spec clearly states that Only .version files that would be installed for this mod are considered. and the find directive should only work against the top-most matching directory which shouldn't be in the _MACOSX folder since that makes everything one layer deeper. Not sure how to ping on this but I'm guessing @dbent since it seems NetKAN isn't doing what the spec says it should be doing.

I think that Netkan is downloading and looking for .version files before it gets to the filtering part of the instructions. You can specify the name of the .version file to force the correct file. eg.
"$vref": "#/ckan/ksp-avc/horizonaeronautics.version/",

Thanks, I'll fix that, but isn't this non-spec?

This file doesn't have a .version, so I just removed the $cref line

The spec was missing some details there. I've updated it.
Ah, not duplicate .versions, but no .version at all!

Sorry, meant to merge this earlier, but fatigue drove me off to bed.


feat: Address validations

Description
Complete the following functions under ./src/utils/validations.ts
export function validateEthAddress(ethAddress: string): boolean {
  // TODO
  return true
}

export function validateSnAddress(snAddress: string): boolean {
  // TODO
  return true
}

These parameters will come from rpc requests. You have to verify whether it is valid or not. Maybe a checksum check is also good?
Requirements

Unit tests have to be implemented.
Do not change the existing type structure

Completion Time
The issue is assumed to be completed in 5 days. If you need more time, please also specify that.

https://github.com/starknet-io/starknet.js/blob/8bf55bff9f6edd82b5e427e94a48deb3059ebe05/src/utils/address.ts

Could I be assigned?

i want to work on this issue @ermvrs

@berzanorg I assigned you to other one. I am assigning @ktarun1419

Thank you, @ermvrs! I'll begin addressing the issue immediately. Is there a designated group for discussions?

Telegram group: https://t.me/rosettanet

Completed https://github.com/keep-starknet-strange/rosettanet/pull/44

@all-contributors please add @ktarun1419 for code, tests


feat: Now supports generating events for nodes

Description
Now supports generating events for nodes by adding the attribute tactile-event-name.

This pull request is included in version 1.7.0


Off  noreferrer lint

该规则会修改超链上的ref，但添加noreferrer会对内网统计产生不利影响，经讨论将该规则off

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

LGTM.


VSCode not recognizing @types/jest installed in subfolder

VSCode Version: 1.51.1 (user setup)
OS Version: Window 10 Family 20H2

Steps to Reproduce:

Create a monorepo using lerna
Add a package for frontend using gatsby
Create components with typescript
Add unit tests to gatsby components by following this guide

Does this issue occur when all extensions are disabled?: Yes
I always have this error, even though I installed @types/jest.

Cannot find name 'describe'. Do you need to install type definitions for a test runner? Try npm i @types/jest or npm i @types/mocha.

I try to install it at the root directory and the error has gone. But this is not the solution I want. I don't use typescript in all my packages.

Please share an example project that demonstrates this issue

Hi @mjbvz,
Thanks for your answer.
Here is a link to the sample project: https://github.com/GhassenRjab/sample-project
You will notice that the test is running well but VSCode keeps on unrecognizing @types/jest

This is almost certainly a confirmation issue with your project. See https://stackoverflow.com/questions/42024526/vscode-automatic-type-acquisition-for-jest for example


[Block grid] Remove

Following an internal discussion, the block grid is being removed from the framework. This is because it doesn't work if you try to create multiple rows of items.
Instead, we're going to add a shortcut to the <row> tag in Inky to apply classes for you.
<row small-up="1" large-up="3"></row>

Removed the docs page.

This makes my issue #201 even more important in that case, because IIUIC block grids were the only recommended way to get responsive reflow in the Gmail app

Passing assignment to @rafibomb, as we're going to keep it but explain the caveats.

The block grid doesn't really solve the fact that Gmail doesn't support media queries. It's in the framework and we're open to other solutions. It was suggested and logged on our roadmap for Inky to use the spongy grid method as an option. That could be a great community project!

Calling it "spongy grid" is neat. I am already using this grid in production. That roadmap is a great idea.


Display an evaluation, that is not a curation statement, by Access Microbiology

Sample article with 3 human reviews and 1 editor "curation statement": http://localhost:8080/articles/activity/10.1099/acmi.0.000569.v1

Example of arbitrarily complex body with various nested paragraphs and XML <bold> tags:
https://www.microbiologyresearch.org/docserver/fulltext/acmi/10.1099/acmi.0.000569.v1/acmi.0.000569.v1.xml

Documentation of fast-xml-parser: https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/docs/v4/2.XMLparseOptions.md#alwayscreatetextnode

To provoke a pull for more full text, we ran this:
$ curl -v -H "Authorization: Bearer $SECRET" -X POST https://sciety.org/api/record-evaluation-publication -H "Content-type: application/json" -d '{"groupId": "4d6a8908-22a9-45c8-bd56-3c7140647709", "publishedAt": "2023-03-17", "evaluationLocator": "doi:10.1099/acmi.0.000569.v1.4", "articleId": "doi:10.1099/acmi.0.000569.v1", "authors": [], "evaluationType": "review"}'


Use assert in image package unit tests

Some misc cleanup

extract a function from a rather large function in the builder
remove setup duplication from the tests in image/
use assert.* in the tests to make them more readable, and also for better assertions when an error is expected. Now a specific error message is expected.

SGTM but windowsRS1 failing
00:07:21.216 --- FAIL: TestFSInvalidSet (0.00s)
00:07:21.217 	assert.go:107: fs_test.go:53: Expected error to contain 'is a directory', got 'rename C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store489919157\content\sha256\.tmp-c3ab8ff13720e8ad9047dd39466b3c8974e592c2fa383d4a3960714caef0c4f2553185424 C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store489919157\content\sha256\c3ab8ff13720e8ad9047dd39466b3c8974e592c2fa383d4a3960714caef0c4f2: Access is denied.'
00:07:21.217 --- FAIL: TestFSInvalidRoot (0.00s)
00:07:21.217 	assert.go:107: fs_test.go:80: Expected error to contain 'not a directory', got 'mkdir C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store907166895\root: The system cannot find the path specified.'
00:07:21.218 --- FAIL: TestFSMetadataGetSet (0.02s)
00:07:21.218 	assert.go:107: fs_test.go:120: Expected error to contain 'no such file or directory', got 'open C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store697213506\metadata\sha256\fcde2b2edba56bf408601fb721fe9b5c338d10ee429ea04fae5511b68fbf8fb9\tkey2: The system cannot find the file specified.'
00:07:21.218 --- FAIL: TestFSGetUnsetKey (0.00s)
00:07:21.218 	assert.go:107: fs_test.go:197: Expected error to contain 'no such file or directory', got 'open C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store803968859\content\foobar\abc: The system cannot find the path specified.'
00:07:21.219 --- FAIL: TestFSDelete (0.01s)
00:07:21.219 	assert.go:107: fs_test.go:225: Expected error to contain 'no such file or directory', got 'open C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store710109765\content\sha256\2c26b46b68ffc68ff99b453c1d30413413422d706483bfa0f98a5e886266e7ae: The system cannot find the file specified.'
00:07:21.219 time="2017-03-03T23:54:24Z" level=error msg="invalid image sha256:f1234d75178d892a133a410355a5a990cf75d2f33eba25d575943d4df632f3a4, invalid character 'i' looking for beginning of value"
00:07:21.220 --- FAIL: TestRestore (0.10s)
00:07:21.220 	assert.go:107: store_test.go:43: Expected error to contain 'no such file', got 'open C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store684222093\metadata\sha256\f40b4ddcd4289be71c5f8f305c5b8babb4266ef2c39bc599ac968dfc6c8b720e\parent: The system cannot find the path specified.'
00:07:21.220 --- FAIL: TestAddDelete (0.02s)
00:07:21.220 	assert.go:107: store_test.go:93: Expected error to contain 'no such file or directory', got 'open C:\Users\ContainerAdministrator\AppData\Local\Temp\images-fs-store104089660\content\sha256\8d25a9c45df515f9d0fe8e4a6b1c64dd3b965a84790ddbcc7954bb9bc89eb993: The system cannot find the file specified.'
00:07:21.221 FAIL
00:07:21.221 coverage: 82.6% of statements
00:07:21.221 FAIL	github.com/docker/docker/image	0.237s

I wrapped the errors and fixed the assertions, so part of the error should be consistent between platforms now.


new button installed

Fixes Issue
Closes #issue_number

Changes proposed

Screenshots

Checklist

[x] You've linked this PR to the correct issue.
[x] You have checked that the code is working correctly.
[x] You ⭐ the repository!

Note to reviewers

hey pls merge this pull request

🎉 The GSSOC'23 program has reached its conclusion, prompting us to officially close all pull requests (PRs) and issues. We extend a heartfelt thank you for your amazing contributions! 🙌 Keep the coding spirit alive with the same enthusiasm, and may luck 🍀 favor you in all your future coding adventures!


git push origin requires personal token to login

The tutorial step1 step that requires pushing the step1 branch to origin fails git authentication.
I used my git account password and authentication failed.  Googling led me to learning that you have to create a personal access token and use that to authenticate.  Far from intuitive.
Please add a note of guidance to that step telling the student that they must go to:
User->Settings->Developer Settings->Personal Access Token and create a new token.  Use that token for the login challenge that the push will cause.

I created a PR for the carbon-website to address this.