Fix exception thrown when two classes with the same name (but not the same fully-qualified name) are registered in a Schema

EDIT: There is no crash, only an exception thrown

Description
This fix aims to mitigate exception thrown when registering two types that have the same name but not the same fully qualified name.
enum UserView {
  struct Public {
    // ...
  }

  struct Private {
    // ...
  }
}

enum OrganizationView {
  struct Public {
    // ...
  }
}

Since the actual implementation of AnyType.hashValue uses String(describing:) to get the type name, both String(describing: UserView.Public.self) and String(describing: OrganizationView.Public.self) would return "Public":
https://github.com/GraphQLSwift/Graphiti/blob/927ebb174825cf8188190fb9eb38d9de84806256/Sources/Graphiti/Definition/AnyType.swift#L8-L10
The same happen for types that aren't nested but in different modules – such as two libraries/frameworks/SPM-targets – as long as the type name themselves are the same.
Proposed solution
With my limited knowledge of the Graphiti codebase, I can see two ways to fix this (always happy to discuss a better implementation with guidance from the maintainers):

Use String(reflecting:) to compute the hash code (which returns the fully qualified class name, i.e. "NameOfTheSwiftModule.UserView.Public").
Change the equality operator implementation to compare the types instead of the hash code.

I preferred solution 2 as the String(reflecting:) documentation states suitable for debugging. Therefore I avoid building production ready solutions on this.
Swift version
swift-driver version: 1.109.2 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.300 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0

Oh, I just realized there's a crash involved. We should probably throw an error with an error message explaining what happened and suggest the user should disambiguate the names manually through the initializers.

Do you mean this initializer (and its overloads) ?
https://github.com/GraphQLSwift/Graphiti/blob/927ebb174825cf8188190fb9eb38d9de84806256/Sources/Graphiti/Type/Type.swift#L109-L115
Because I specify the type name using the as name: String and the crash persists, since it is not used in Type. Let me know if there's something I missed – this is part of the Schema<> object:
  Type(UserView.Public.self, as: "UserView_Public") {
    Field("id", at: \.id)
    Field("displayName", at: \.displayName)
  }

  Type(
      UserView.Private.self, as: "UserView_Private"
  ) {
      Field("id", at: \.id)
      Field("displayName", at: \.displayName)
      Field("mainEmail", at: \.mainEmail)
      Field("createdAt", at: \.createdAt)
      Field("updatedAt", at: \.updatedAt)
  }

  Type(EmailView.Private.self, as: "EmailView_Private") {
      Field("address", at: \.address)
      Field("optInNotifications", at: \.optInNotifications)
  }

It's basically the whole schema for now (if we omit for a Scalar(Date.self) and the queries/mutations).

The last Type(EmailView.Private.self) declaration throws the following error:

Duplicate type registration for GraphQL type name "EmailView_Private" while trying to register type Private

here in the code:
https://github.com/GraphQLSwift/Graphiti/blob/927ebb174825cf8188190fb9eb38d9de84806256/Sources/Graphiti/Definition/TypeProvider.swift#L13-L27
I placed a breakpoint on the throw statement inside the guard block, and after inspecting the value of graphQLNameMap I couldn't find EmailView_Private. See by yourself:
(lldb) po graphQLNameMap
▿ 7 elements
  ▿ 0 : 2 elements
    ▿ key : <AnyType: 0x6000007c30a0>
    - value : "String"
  ▿ 1 : 2 elements
    ▿ key : <AnyType: 0x6000007c3080>
    - value : "Float"
  ▿ 2 : 2 elements
    ▿ key : <AnyType: 0x6000007c3640>
    - value : "DateTime"
  ▿ 3 : 2 elements
    ▿ key : <AnyType: 0x6000007c30c0>
    - value : "Boolean"
  ▿ 4 : 2 elements
    ▿ key : <AnyType: 0x6000007c3660>
    - value : "UserView_Public"
  ▿ 5 : 2 elements
    ▿ key : <AnyType: 0x6000007c3680>
    - value : "UserView_Private"
  ▿ 6 : 2 elements
    ▿ key : <AnyType: 0x6000007c3060>
    - value : "Int"

The name parameter is not used as a key. The AnyType instance is.
That's why no matter if I specify a name or not, it still crashes.

I've been tempted to refactor the code to use name as the key of graphQLNameMap, or maybe forward the name to the AnyType initializer and use it in the hash computation, but it's my first dive in Graphiti's code base so... I proceed cautiously 😅

Oh, I just realized there's a crash involved. We should probably throw an error instead.

Sorry, my bad, there's no crash in Graphiti ! We had a try! on our side.
I clarified and edited the title and description of the issue.

Tests are all green on my local environment

Tests passing on a Mac Studio M1 Ultra running macOS 15.0 beta 1 and Xcode 16.0 beta 1

@paulofaria Is there anything specific in the CI environment that could cause the tests to crash?
I also think it might be the first PR that runs tests using Xcode 16. I checked the last merged PR checks and the CI ran with Xcode 15.4.
It shouldn't cause any trouble since I ran them successfully with latest macOS/Xcode/Swift toolchain. But maybe the GitHub action doesn't fully support it yet ?

Ah yeah, I've validated that this CI fails on main with the same error, so it doesn't appear to be related to your changes: https://github.com/GraphQLSwift/Graphiti/actions/runs/9586143166/job/26433471298?pr=138

OK, yeah you were exactly right on the v16 XCode version. We were using XCode latest (potentially unstable), which resulted in v16.0.0, whereas using latest-stable results in v15.4.0 and passes. I have an MR to set it to latest-stable here
From this (totally official) site, it looks like XCode 16 is using Swift 6, which would explain some significant behavior changes. We've got a bit of work to do to become Swift 6 compatible, but that's a separate effort.
Once the MR linked above is merged, feel free to rebase on it and we'll get these changes in.
Thanks for contributing!!

Alright the linked PR is merged. You should be good to rebase!

@NeedleInAJayStack Rebased! 😊

@abidon This change has been released in tag 1.15.1: https://github.com/GraphQLSwift/Graphiti/releases/tag/1.15.1


Making the version 3.3.0 compatible with Django 4

Hi
can you please make the version 3.3.0 compatible with django 4 (just changing the "ugettext_lazy")
thanks

I've manually circumvented our issues with Jazzband to issue a 4.0.0 release to resolve this.


MemoryPool.AllocateSlab calculation

From MemoryPool.AllocateSlab:
var firstOffset = (int)((_blockStride - 1) - ((ulong)(basePtr + _blockStride - 1) % _blockStride));

var poolAllocationLength = _slabLength - _blockStride;

Shouldn't firstOffset be substracted from poolAllocationLength?
Is the '_blockStride - 1' needed in the firstOffset calculation?

firstOffset is the amount to discard to page align the blocks post allocation

Since poolAllocationLength doesn't take firstOffset into account, I wonder if the last block is completely in the Slab when firstOffset>0.

It allocates var slab = MemoryPoolSlab.Create(_slabLength); but only uses poolAllocationLength which is _slabLength - _blockStride so there is _blockStride free in the slab to allow for alignment

Did you notice the block that is allocated after the for loop?
https://github.com/dotnet/corefxlab/blob/master/src/System.IO.Pipelines/MemoryPool.cs#L159-L164

The for loop test is one less offset + _blockLength < poolAllocationLength

You're right. The loop leaves enough space for the last block. Thanks.


Possible fix for GCC 7.3.1 c++17 compilation error

Hello and thank you for creating and maintaining this great library.
I haven't had any problems using the library with c++11 but I wanted to also use it with more recent c++ standards. However, I got the following error when compiling the libsimdpp headers for SSE2 using GCC 7.3.1 with the -std=c++17 option (c++14 works).
..../include/simdpp/types/empty_expr.h: In instantiation of ‘simdpp::arch_sse2::mask_int32<N, simdpp::arch_sse2::expr_empty>::operator simdpp::arch_sse2::uint32<N>() const [with unsigned int N = 4]’:
..../include/simdpp/detail/insn/cmp_neq.h:195:52:   required from here
..../include/simdpp/types/empty_expr.h:220:52: error: could not convert ‘((const simdpp::arch_sse2::mask_int32<4, simdpp::arch_sse2::expr_empty>*)this)->simdpp::arch_sse2::mask_int32<4, simdpp::arch_sse2::expr_empty>::e’ from ‘const simdpp::arch_sse2::mask_int32<4>’ to ‘simdpp::arch_sse2::uint32<4>’
     SIMDPP_INL operator uint32<N>() const { return e; }
                                                    ^

The same line also produces an error on Clang 6.0.0 on c++17.
I managed to find a possible fix for the problem by removing the cast to uint32x4 of the return value in line cmp_neq.h:195. I believe that cast is not needed anyway and all tests still pass without it.
Best,
Hannes

Looks good to me.


Using a target size (torch.Size([16, 1, 64, 64])) that is different to the input size (torch.Size([15, 1, 64, 64])) is deprecated. Please ensure they have the same size.

Hi again, @TobiasSunderdiek
During the trainning process, this error raised and I have no Idea of what is causing it. I am using the default dataset (safebooru) and default params for the trainning.
Do you have any idea on how to fix it?
Regards

Hi @bdlneto,
thanks again for the issue, every issue makes this repo better!
I think this is caused by processing a batch smaller than 16. Does this error occur while processing the training cell in the notebook?
At some places in the code the actual_batch_size is used to avoid this, but it seems I missed another place.
I think during the download of the images, some error occured. Can you please check if all images in the cartoons/, cartoons_smoothed/ and photos/ folder contain the same amount of 4000 images?
Thanks and best regards!

Hi!
I also found it to be some problem during the download process.
I'll check the folders and, if necessary, re-download them
Thanks again!

Great, thank you for helping me making this repo better by having opened this issue!
I'll close it now, best regards


text_recognition, I export my trained model to OpenVINO™ Intermediate Representation (IR) ,some errors occur!

[ ERROR ]  List of operations that cannot be converted to Inference Engine IR:
[ ERROR ]      FusedBatchNormV3 (8)
[ ERROR ]          shadow/BatchNorm/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_1/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_2/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_3/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_4/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_5/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_6/FusedBatchNormV3
[ ERROR ]          shadow/BatchNorm_7/FusedBatchNormV3
[ ERROR ]  Part of the nodes was not converted to IR. Stopped.

Where did the errors occur? Which tool did you use, how, which command-line parameters? Can you describe your trained model a bit, please?
What do you mean with "export the model to OpenVINO IR", you mean you converted it using the model-optimizer? Which format was it before, TensorFlow?

The version of openvino is 2019 R3.I solve this issue when I change the version to 2020.1


Add section about 'presentation request'

This PR adds a section about presentation requests says this spec doesn't define them, and then points out two properties it may be nice for a presentation request to have.
This PR fixes #23 #29 #51 #52 and may also address #34
Signed-off-by: Brent Zundel<EMAIL_ADDRESS>

Is this a gap in the W3C spec? As they define Verifiable Presentations, one would think they define Verifiable Presentation Requests too.

Is this a gap in the W3C spec? As they define Verifiable Presentations, one would think they define Verifiable Presentation Requests too.

It is an artifact of the limitations under which the VCWG was operating, which prevented them from touching on anything that related in any way to a credential and proof exchange protocol. So, we were able to define Verifiable Credentials and Verifiable Presentations, but not the messages that might be used by Issuers, Holders, and Verifiers to offer or request them.


docs: Optimization docs addition

Fixes #1076

[x] Documented

The majority of this task was completed beforehand on previous tickets. However I thought it would be nice to atleast have some documentation about it.


Add support for Reader API to convert HTMLs into Documents

Is your feature request related to a problem? Please describe.
There's no component to use Jina's Reader API with Haystack.
Describe the solution you'd like
A new JinaHTMLtoDocument (name TBD) component to use Jina's Reader API to convert URLs into Haystack Documents. This component should accept a URL and output a Haystack Document.
Describe alternatives you've considered

This component can output a markdown file and users might use MarkdownConverter to use that component in a pipeline (not Haystack intuitive but might have advantages)
Depending on how the Reader API works, it can accept a list of URLs and return a list of Haystack Documents

Additional context
Add any other context or screenshots about the feature request here.
### Tasks
- [x] The code is documented with docstrings and was merged in the `main` branch
- [ ] Docs are published at https://docs.haystack.deepset.ai/
- [x] There is a Github workflow running the tests for the integration nightly and at every PR
- [x] The package has been released on PyPI
- [ ] Integration tile in https://github.com/deepset-ai/haystack-integrations has been updated
- [ ] The integration has been listed in the [Inventory section](https://github.com/deepset-ai/haystack-core-integrations#inventory) of this repo README
- [x] There is an example available to demonstrate the feature
- [ ] The feature was announced through social media

### Tasks

I was the one who proposed this component.
Unfortunately, I tried the service and it is quite unstable at the moment.

Hey there @bilgeyucel @anakin87!
Is this still a thing? I just toyed around with the API and got good results. Would be happy to knock this out if you guys think its valuable.

@jlonge4 I think the API improved over time.
I see they now have different endpoints for converting a page into markdown, searching the web and grounding (experimental).
What's your idea?

I would say it is just another nice option.
Are you thinking of a single component or more than one?

@anakin87 I agree! Would passing modes at init time to a single component make sense?
Like reader = JinaReader(mode="read") or something to designate which endpoint to use.

I'm thinking of something like:
@component
class JinaReader():

    def __init__(
        self,
        api_key: Secret = Secret.from_env_var("JINA_API_KEY"),
        mode: Union[Mode, str],
        ...
    ):
    ...

    @component.output_types(document=Document)
    def run(self, input:str):

    # check input depending on mode
    ...

Mode can be an Enum like this (with a convenient from_str method): https://github.com/deepset-ai/haystack-core-integrations/blob/ac0e4c2f8c8d0dce7a32e8e3a3fe74362b0686dd/integrations/nvidia/src/haystack_integrations/components/embedders/nvidia/truncate.py#L4

@anakin87 @jlonge4, what are the exact features of Jina Reader API now? I'm asking because we use reader components for extractive QA tasks, and I don't think the JinaReader component will fit well into that category. Does it make sense to name it JinaReaderConverter, maybe?

Convert URL into Markdown
Search the web and convert results to Markdown
Ground a statement with web knowledge (only paid, haven't tried)

https://jina.ai/reader/

@anakin87 looks great, I'll get it cooking asap!
@bilgeyucel you have a great point, it definitely is more of a converter or fetcher vs a reader.

@jlonge4 I have added a tasklist to https://github.com/deepset-ai/haystack-core-integrations/issues/663#issue-2245585294.
Could you maybe help with opening a PR to mention the JinaReaderConnector in https://github.com/deepset-ai/haystack-integrations/blob/main/integrations/jina.md?
(I see the focus is on embedding models, so maybe a brief mention + link to examples is OK)

@anakin87  you've got it, no problem 😎

@anakin87 https://github.com/deepset-ai/haystack-integrations/pull/288

Closing this issue.
(Only social media announcement is missing.)
Added an item for this component to Weekly Announcements - https://github.com/deepset-ai/devrel-board/issues/533


Migrate Pagination to TypeScript

This PR migrates the Pagination component to TypeScript as part of the TypeScript refactor.

forgot to export PaginationProps! done in 43c79d6

8fb049d should fix the last failing TS error 🤞


Update to mason in README only

Code untested.
part of #39
I'd recommend testing these changes. I'm pretty sure it should work, but not confidant.

@Lazerbeak12345
I am very sorry for the delay in responding.
I am unfamiliar with Neovim these days and it has been very helpful.
Thank you for your response.


Try fix the problem using .Net 7 and KUbuntu 22.10

On my installation, KUbuntu 22.10, installed .Net 6 and 7 from Microsoft Package repository, the line count of both dotnet version are 5 instead of 4. It just an empty line, removing last line will solve the problem. However, I personally,  would like to not modify dotnet SDK.

Hello. Thanks for PR and testing on KUbuntu. Maybe better use trim() and empty lines will be removed from start&end, and will be not issue if MS add another new lines. I will check it later. Thanks :0

You are welcome. :D

Reimplemented in actual branch. Thanks


Tutorial: Customize view for EF Database First with ASP.NET MVC app

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @FatWaterAudio.

From @FatWaterAudio on Tuesday, March 9, 2021 6:47:11 PM
the page at this link  --> https://docs.microsoft.com/en-us/aspnet/mvc/overview/getting-started/database-first-development/customizing-a-view?source=docs     does not work as described.  Adding the custom code to the Views/Students/Details.cshtml  causes the project to fail with a 400.0 IIS message.  ...basically a bad request. Once the failure occurs, there doesn't seem to be a way to recover without deleting the Views/Students folder and the StudentsController.cs file.  All prior code up to that point in the tutorial is auto-generated.      All prior code up to that point in the tutorial seems to run correctly.
To Reproduce
Steps to reproduce the behavior:

Go to  this page   -->https://docs.microsoft.com/en-us/aspnet/mvc/overview/getting-started/database-first-development/

follow all instructions up to this page  --> https://docs.microsoft.com/en-us/aspnet/mvc/overview/getting-started/database-first-development/customizing-a-view?source=docs

Expected behavior
The tutorial exercise where the error occurs is intended to show custom (not auto generated) output in the Views/Students/Details.cshtml page.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Win10 64 bit
Browser Chrome
Version 88.0.4324.190

Additional context

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @welcome[bot].

From @welcome[bot] on Tuesday, March 9, 2021 6:47:13 PM
Thank you for creating the issue! One of our team members will get back to you shortly with additional information. If this is a product issue, please close this and contact the particular product's support instead (see https://support.microsoft.com/allproducts for the list of support websites).

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @keziamicro.

From @keziamicro on Wednesday, March 10, 2021 5:29:15 PM
@FatWaterAudio Thank you for taking the time to let us know the article isn't working as you expected. I've reached out to the authors to let them know.

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @FatWaterAudio.

From @FatWaterAudio on Wednesday, March 10, 2021 5:36:34 PM
@keziamicro thank you for your reply.   In my own continuing efforts, I was able to find a nearly identical tutorial using Razor Pages style development instead of MVC style development.  My current project is a new development effort.  Which of these two approaches should be preferred for new development?

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @keziamicro.

From @keziamicro on Wednesday, March 10, 2021 5:38:29 PM
Looping in @Rick-Anderson for the above ^

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @Rick-Anderson.

From @Rick-Anderson on Wednesday, March 10, 2021 6:58:23 PM
@keziamicro  please transfer to https://github.com/dotnet/AspNetDocs

Issue moved from MicrosoftDocs/feedback#3544

Please respond to @Rick-Anderson.

From @Rick-Anderson on Thursday, March 11, 2021 6:07:02 PM
@alligreubel who can transfer this issue to https://github.com/dotnet/AspNetDocs ?

@FatWaterAudio

Once the failure occurs, there doesn't seem to be a way to recover without deleting the Views/Students folder and the StudentsController.cs file.

You never need to delete code. I recommend debugging to see what you did wrong. I recommend you try the ASP.NET Core versions such as https://docs.microsoft.com/en-us/aspnet/core/data/ef-rp/intro and https://docs.microsoft.com/en-us/aspnet/core/data/ef-mvc/intro


Add entity codec instances for Argonaut codecs.

Fixes #156.

Look at #169 first, but I think this is better on top of that.  If people disagree, I'll revert that commit.

I like this a #169. Are there any downsides I'm not seeing? It could be argued that we are starting to make the messages congested, but I think its a pretty nice pattern so its worth it to me.
If Travis is cool with it, I say go ahead and merge it.


Extend application with templates

We need an ability to have an impress application template as a dependency.
It should allow us to write plugins locally so that it would be loaded into template server located in node_modules.

{
  extends: [
    'repoName', // find in `node_modules/repoName`
    'repoName/application', // find in `node_modules/repoName/application`
    '../repo/application', // relative path
    '/home/marcus/projects/repoName/application', // absolute path
  ],
}


version: Correct API feature level.

API feature level should have been increased along with #17010
which increased the community topic editing time.
I think I increased the version but it got lost when resolving merge conflict when another PR got merged with increased API version.

Merged, thanks @amanagr


push monitor: increase token security

Description
As push monitor tokens are not short-lived I propose to raise their length from 10 to 32.
There are no best practices or RFCs recommending specific webtoken lengths but 10 chars seems unnecessarily short. Calculation times for a 32-char token are super fast and tokens are mostly set-and-forget strings and also won't be typed in by hand but rather copy&paste.
Additionally it should be easier to rotate them (think enterprise policies requiring to regularly change/rotate credentials). Therefore I added a "Reset Token" button in the "Edit Monitor" dialog. This also allows users with existing push monitors to upgrade their tokens easily.
This is a non-breaking change but if released, the release notes should advise resetting the token (not a MUST though).
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and test it
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

i don't think this actually works tho?   You'll have to do the old trick of duplicate and drop table.
[EDIT] Nevermind, the length value in SQLite doesn't actually do anything, so it's fine.

I believe that it is still not working? Because the sql add a duplicate field.
Also I prefer to keep the length of 10, because I want it shorter and it is not a top secret. Just like a youtube link's video id (e.g. https://www.youtube.com/watch?v=WyLyc5txdfE)

i don't think this actually works tho?  You'll have to do the old trick of duplicate and drop table.
[EDIT] Nevermind, the length value in SQLite doesn't actually do anything, so it's fine.

I believe that it is still not working? Because the sql add a duplicate field.

Wasn't aware that it ever was not working. Did you test if it is really adding a duplicate field? As @chakflying mentioned, we could also just remove the change in db/patch-monitor-push_token.sql as the length in SQLite is irrelevant.

Also I prefer to keep the length of 10, because I want it shorter and it is not a top secret. Just like a youtube link's video id (e.g. https://www.youtube.com/watch?v=WyLyc5txdfE)

This depends on how you assess security vs ease of use. As this token is the one and only authentication required to submit an uptime information and (as I mentioned in the initial post) is not short-lived it requires some protection. As it is also very rare that someone enters these by hand, there's no reason to have short tokens. I know that these tokens are merely used for heartbeats and that might seem uncritical to protect with strong tokens. Then again I find it weird to have long and strong tokens for notification services like e.g. a Teams notification when the initial information to send out notifications is secured with less strong tokens.

@andreasbrett thank you for this!
I totally agree with making the tokens higher entropy, there's no reason not to do that.
Regarding the DB migration, although SQLite does not have length constraints, it makes sense for it to be documented correctly and not misleading. The simplest way would be to change the existing migration but completely remove the length.


Added metadata to File and Directory Rename

This is not a breaking change because this feature has not been publicly released yet.

@jhendrixMSFT, could we merge this?


Add ability to mask sensitive data

This would be perfect if there was a way to mask the CC number (but for the last 4) after they've entered it.  It is generally bad UX practice to display sensitive data permanently on the screen.  This is in case the info is entered in a public/shared space, you don't want to display sensitive data on the screen.
I'd be happy to help if someone can point me in the right direction

@codenamev,
Please give an example website where CC numbers are masked during input.
This will help identify the priority & need.

https://stripe.com/checkout

@codenamev - wrong. They are not masked.

No need to be so short - we're all friends here :D
@codenamev I don't think this is a common use case, so I'm not sure it has a use in the library. That said, if you're interested in showing some examples where the UX pattern is used, I'd be happy to consider it!
Thank you both for your time and thought here!

Sorry for the vague response!  The downsides of mobile GitHub ;-)
Masked input is a common UX pattern for "staged" payments (e.g. when confirmation is needed), stored CC info (as referenced in the screenshot from Stripe below), and in cases where screen size is an issue due to keyboard overlays (see single field payment used by Square's Wallet).  Although the later may be outside the scope of masked inputs, it's still justification for it's need I think.
I've seen CC masking (except for the last 4 digits) on many other sites I can't think of off the top of my head; but they're out there.  Mostly useful for staging/storing payment info.

Thanks for the additional information, happy to merge in a PR for this support if someone wants to add it.

OK, I've opened up a more general issue for this here and I'm confident that it's a feature I'd like to add.
Would really appreciate continued feedback and conversation there — thank you again for your time and thought!

@jessepollak So, I owe @codenamev an apology. I was short. I didn't clearly communicate my intention and perhaps left an unkind feeling amidst.
@codenamev it was a valid feature request and concern. It's fantastic to see it open into a possible future component of yhe software. Keep up the thoughtfulness and creative thinking.

Aw, thank you for the thoughtfulness, I really appreciate it and I bet @codenamev does as well :D
Thank you both for your contribution to this project!


Update acorn for dynamic import

Perhaps an idea to update acorn and remove acorn-dynamic-import now when https://github.com/acornjs/acorn/pull/834 is released (acorn 6.2.0)?

@westn please use issue template, also it is done for webpack@5

Actually it's not yet done for the next branch, so send a PR

Seems it is solved https://github.com/webpack/webpack/pull/9370

Yep, we can close :+1: need rebase for next too

need rebase for next too

We don't need that, as master is merged into next after each release.


Bael 2330

Adding removeDuplicatesUsingDistinct
Adding more asserts to the unit tests

Can you resolve your merge conflicts by merging in the master tutorials branch?


Can't select TypeScript version from command palette

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.71.0
OS Version: 12.5.1

Steps to Reproduce:

Press ⌘⇧P (open command palette)
Search for Select Typescript version
Nothing appears
Though, if you assign a keybinding to typescript.selectTypeScriptVersion you can open the popup correctly.

This is by design. The command only shows in the command palette when you are in a js or ts file

The built-in extension which contributes this command sets a when-clause to control whether or not it appears in the Command Palette:


[Feature] add dependabot gh action

Description
we are using npm dependencies & also using gh actions so if we add the dependabot gh action we don't need to update dep & gh actions manually.
dependabot will update these automatically by creating the PR
I want to work on this issue
Screenshots
No response
Additional Information
No response

New Updates may introduce breaking changes or compatibility issues.

ok then we should close this PR.


[Snyk] Security upgrade pillow from 9.5.0 to 10.0.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

909/1000  Why? Mature exploit, Has a fix available, CVSS 9.6
Heap-based Buffer Overflow SNYK-PYTHON-PILLOW-5918878
pillow:  9.5.0 -> 10.0.1
No
Mature

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Uncontrolled Resource Consumption ('Resource Exhaustion') SNYK-PYTHON-PILLOW-6043904
pillow:  9.5.0 -> 10.0.1
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Uncontrolled Resource Consumption ('Resource Exhaustion')

Vulnerability fix


[DO NOT MERGE] Dev containerservice microsoft.container service 2020 09 01

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

cc @allenjzhang since this PR contains adding privatelink support.

since new api added, add label "WaitForARMFeedback"

@marwanad synced with me offline, changes in this PR is reviewed before merging to feature branch. For private link, here's the PR https://github.com/Azure/azure-rest-api-specs/pull/10548/files. I do see some issues there. Will trigger an email for further discussion.

/azp run automation - sdk

#sign-off


Can you bump the version and push a new release?

There have been a few fixes in the past few months and we'd like to be able to take advantage of them. Is there any reason not to push out a new version?

@twalling hi, no sorry, I don't have so much time but I will push a new release now.

Thanks!


auto-fir perl script does not run in Ubuntu 18.04

There seems to be a difference between the perl packages that are included (by default) with Ubuntu 16.04 vs Ubuntu 18.04.
The perl script relies on the verilog::gen module which does NOT seem to be part of the Verilog-Perl package (which can be installed in 18.04).
As of the writing of this issue, I cannot find the verilog::gen module for Perl in 18.04.

Remove legacy shell script dependency (from Freedom U500 days), but iirGen.pl depends on the verilog::gen module which doesn't seem to exist anymore (couldn't find a way to install on Ubuntu 22.04)


Add once() equivalent

Motivation
The ability to synchronize on an observed condition is extremely handy, and we use it quite often in Atom. It would be nice if we could define this upstream in the stream interface instead of the atom. That way, there really wouldn't be anything special about the once() method in the atom.
Approach
Originally, I set about adding a once() method for stream that was just a composition of filter(predicate).first(). But there are cases that you want to have the actual matched value, and not just synchronize on it:
let match = yield stream.once(({ status }) => status === 'ready');

In those cases where you want the return value, this method should behave like expect(), raising an exception if the stream is closed before a value is received.
On other occasions however, you don't necessarily need the value, you just want to synchronize, and in those cases it's ok for the stream to close. In order to support both of these cases, we would need both a once() and onceExpect() so I opted instead to allow the first() and expect() methods to accept a predicate against which to match. This lets us not any any methods to the API and yet cover all of our use-cases. In essence, first really means "first matching predicate" and expect means "expect a value matching predicate".
Alternate Designs
It is very easy and straightforward to implement once() in terms of filter() and first() and this was the temptation. It does keep the first() and expect() methods simpler which is nice.
Possible Drawbacks or Risks
I don't like optional parameters as a rule as they make methods difficult to curry later on.

Is there any functional difference between filter(someFunc).expect() and expect(someFunc)? I don't think there is, right? In that case, even if we want to add this shortcut, we should be able to use filter internally, rather than adding a loop?

We actually don't need this if in @effection/atom we change the stream implementation to always emit the current  state. If saying atom.filter(matcher).expect() becomes too onerous, then we'll revisit.


Multiple template proposal

Four templates demonstrating use of extension templates. The kitchensink will most likely be used to demonstrate new features as we add them. Right now it's a copy of my original API testbed. I will most like clean it up.
This is just a proposal. I'm okay with multiple repositories but I feel this approach is easier to manage.

I think we should make a hard minimum of 7.0 anyway – the only people building on a version before that are partners and internal folks and we can help them stay up to date.

And I agree the template should be as simple as possible because it's something we want people to just fork and make minimal changes too. Many projects in the JS ecosystem have way too much included in their templates and it makes it hard to figure out what's really required. Plus, we want people to think it looks really simple!


Update collections, command, completion, ... to 1.9.3

About this PR
📦 Updates

org.scala-sbt:collections
org.scala-sbt:command
org.scala-sbt:completion
org.scala-sbt:core-macros
org.scala-sbt:main
org.scala-sbt:main-settings
org.scala-sbt:sbt
org.scala-sbt:task-system
org.scala-sbt:util-logging
org.scala-sbt:util-position

from 1.4.9 to 1.9.3
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

💡 Applied Scalafix Migrations

org.scala-sbt:sbt:1.5.0 (created no change)

https://gist.githubusercontent.com/eed3si9n/57e83f5330592d968ce49f0d5030d4d5/raw/7f576f16a90e432baa49911c9a66204c354947bb/Sbt0_13BuildSyntax.scala
Documentation: https://eed3si9n.com/syntactic-scalafix-rule-for-unified-slash-syntax

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, scalafix-migrations, commit-count:1

Superseded by #210.


IGNITE-5569 history of node ids to prevent failed nodes to rejoin the…

… cluster

IGNITE-5569
Resolved


Adds step by step disk space info in build logs.

I know I'm printing in the loops and that this might be too much, but until we find what it is, might be for the best?
I'll run kokoro on this branch before merging though.

OK, that sounds good. On it.

Done.

Just added one last change, removed the logs I had added in #5192, they are now redundant


ENH: Add orthogonalize argument to DCT/DST

Reference issue
Ref gh-15033
What does this implement/fix?
This adds an orthogonalize=None parameter to the real transforms in scipy.fft which controls whether the modified definition of DCT/DST is used without changing the overall scaling. So for example, you can now call
dst(..., norm="ortho", orthogonalize=False)

for 1/sqrt(N) scaling of the unmodified DST. Or,
dst(..., norm=None, orthogonalize=True)

for an unscaled variant of orthogonal DST. Neither of which were possible before.
For backward compatibility, orthogonalize defaults to true when norm="ortho" and false otherwise.

Fine with me! The complex interplay between the various flags is a bit unfortunate, but probably unavoidable for compatibility's sake.
I'll try to integrate this change upstream as well, and perhaps also in my other FFT libraries, but it might take some time.

thanks!

I just checked the changes, and it seems pretty fine to me. Awesome!
I agree that, due to backwards compatibility, the complexity of flags might be unavoidable.


Original files still appear even after deleting them

Describe your bug here.
I deleted all of the original FNF files, for example all the songs starting from Tutorial --- Stress and replaced them with my mod's songs, but they still appear in the exported game's files. Same with characters and weeks.

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Yes, I replaced every vanilla FNF file with my mod's files.

This isn't a bug or anything, the source code adds the OG files to the assets. in order for this to work, you have to go in the export's assets folder, and replace the stuff.

This isn't a bug or anything, the source code adds the OG files to the assets. in order for this to work, you have to go in the export's assets folder, and replace the stuff.

Oh, thanks. I just thought its a bug that it adds og files.


needed: videos/updated videos

If we want to sell this program, would be great to have a few good videos.

A good, govt.-specific API explainer. Like this one, updated and tweaked to be evergreen: https://www.youtube.com/watch?v=BVeiRCEwJx8. Overall, there are surprisingly few good API explainer videos in the world.

Overview of what this specific program offers (you know, once we figure that out). For example like these startup explainers.

short
high production

The NZ API link that @gbinal sent around this week does a good job outlining how APIs can benefit agencies (no doubt those points are also somewhere in /developer): https://api.business.govt.nz/benefits-apis
Just noting in case we need to borrow material for a video script


Server Status page totally messed up on apache 2.4.25 with MPM Event

I have Apache 2.4.25 running with MPM Event, but as soon as I load mod_security the Apache status page goes all bonkers: I see stuff like:
Total accesses:<PHONE_NUMBER>595812052 - Total Traffic:<PHONE_NUMBER>6.0 GB
CPU Usage: u5.40831e+11 s1.87 cu0 cs0 - 1.56e+11% CPU load
2.89e+16 requests/sec - 35834876.0 GB/second - 1 B/request
Also the requests have incorrect IP's sometimes (73.101 as example, it is missing 2 dots)

@tjikkun what version of mod_security are you using? Do you have any SecRule/Actions applied or are you just loading the module?

@bostrt Ah good questions. Sorry I forgot to mention this happens with both 2.9.0 and 2.9.1, haven't tested older versions.
As to your second question, I hadn't really thought about it but I did some digging and as it turns out just loading the module does not show the problem, minimum config to reproduce needs to have
SecConnReadStateLimit

I found the issue, will try to put in a pull request tomorrow, should also fix #786


jumping into statement expression can cause weird behavior

Consider:
#include <stdio.h>

int main(void)
{
	goto bar;
	printf("%d %d %d\n", 12, ({bar: puts("hello"); 34;}), 56);
	return 0;
}

This program will show:
hello
12 34 <arbitrary integer>

GCC simply disallows that:
test.c: In function ‘main’:
test.c:5:9: error: jump into statement expression
    5 |         goto bar;
      |         ^~~~
test.c:6:36: note: label ‘bar’ defined here
    6 |         printf("%d %d %d\n", 12, ({bar: puts("hello"); 34;}), 56);
      |                                    ^~~

Jumping into an inner scope is not allowed. See https://www.sigbus.info/n1570#<IP_ADDRESS>p1. It is up to a compiler to report it as an error or just do whatever it wants to do.

Aren’t statement expressions a GCC extension? (Rather than being standardized, I mean.) Because in that case, I think chibicc should try to match GCC’s behavior. If you don’t think it is worthwhile to fix this, I do, and I’d be willing to work on a patch for it if you are interested! (Likewise for #83.)
See:

https://gcc.gnu.org/onlinedocs/gcc/C-Extensions.html
https://gcc.gnu.org/onlinedocs/gcc/Statement-Exprs.html

From GCC’s docs:

Jumping into a statement expression with goto or using a switch statement outside the statement expression with a case or default label inside the statement expression is not permitted. […]

Compare this wording with what it considers undefined behavior:

[…] Jumping into a statement expression with a computed goto […] has undefined behavior. […]

As a side note, I think the link you shared is not talking about statement expressions (since those are GCC extensions), but rather about jumping into the scope of a variable that is or includes a VLA (hence the wording “variably modified type”). Something like this:
int main(int argc, char **argv)
{
	goto foo;

	int vla[argc];

	foo:
	vla[0] = 0;

	return 0;
}

@rui314: I have created a patch! I’m really happy about it, as I think it is very succinct. Thoughts would be appreciated!


Add SnapshotCreator API

This PR allows users of v8go to use the existing API from V8 to create HeapSnapshots.
The v8go API is similar to the one provided by V8 and the SnapshotCreator class.
The API still need some work to have the full support that V8 provides, such as passing options to NewSnapshotCreator as well as allowing to store data arbitrary data on the snapshot AddData
In other to support passing the created snapshot_blob to the NewIsolate method, we had to allow passing options when creating a new isolate.
The old API:
v8go.NewIsolate()

The new API:
// data == *v8.StartupData
v8go.NewIsolate(v8go.WithStartupData(data))

Allowing to pass options to NewIsolate allows us to keep the existing API with breaking existing codebases 🎉
Creating a context from v8go now can be done using two methods:

The already existing one v8go.NewContext(iso)
The one introduced here v8go.NewContextFromSnapshot(iso, index)

The snapshot creator API
For every context that gets added to the snapshot creator, we need to make sure the Isolate is the same as the one associated with the snapshot creator class itself. That is the reason we have the GetIsolate() method, so we can create a new context using that one. This isolate is allocated by V8 when calling SnapshotCreator* creator = new SnapshotCreator;
When creating the snapshot blob we have two methods to add existing contexts to the snapshot creator:

SetDefaultContext -> This will set the context as the default one. From there every new context created from that isolate will have that context loaded, so calling v8go.NewContext(iso) will return a context with the default context loaded.

AddContext -> This will append to the list of additional contexts that a snapshot creator can have. The method returns an index, which we will be able to use to indicate to V8 when creating a new context from it. v8go.NewContextFromSnapshot(iso, index)

The V8 code that I used as a reference for creating the API is the one in the v8/test/cctest/test-serialize.cc file.

We could allow passing SnapshotCreatorOptions struct from the beginning even if we currently do nothing with it, that way in the future whenever we start adding support for the options on the snapshot creator class we would not break the existing API.

@rogchap If you have sometime I would appreciate your feedback on the new API for snapshot creation 😄

On a very quick glance, the API remains very close to the CPP equivalent, and remains mostly backward compatible which is great.
I'll need to spend a little more time to look at this in detail.
Appreciate your contributions 💪

Hi @rogchap 👋
By any chance have you got some time to look into it?

Hi,
this is something i'm interested in as well.
would appreciate to know what's the current blocker :)


Navbar: width of parent-entity  retains natural width of scaled down child-img

Instead of displaying the brand in the navbar as text I included the img of the brand-logo which I scaled down for responsive reasons with "max-width 50%". the parent of the img is a hyperlink tag . the issue: the width of  remains displaying the original width of 100% of the img (being 350px instead of 50% of 350px). I have not been able to find a solution and kindly ask for your help.

Please direct how-to questions and general HTML/CSS questions to StackOverflow (or other similar forums). Thanks!


swarm not load balancing to other node

I have a process running in my swarm's IP, that causes the CPU to jump to 100%, shouldn't swarm be load balancing? The other node never goes about 0.3%. Is swarm not configured correctly?

swarm's default scheduling strategy is bin-packing, so it will always try to load up the most packed node to avoid fragmentation. There are other strategies in discussion such as #227, though.

@bacongobbler now I'm wondering if I am just running one node, because I should be going over the threshold. My 'start' command is "docker -H tcp://<IP_ADDRESS>:2375 run cadvisor" where <IP_ADDRESS> is a node

@128keaton: docker -H should point to swarm, not a node.


disable FP lib testing for FP add

Disabling testing of #199 because of the bug in the implementation of FP add. Can re-enable it once we fix it.


Fixing Database emulation

I've added comments for all my code changes to better explain them.
Overall

Make auth_request signal values match expected results
fix database emulation + workaround for events
some small cleanup things

Related issues:
(https://github.com/firebase/firebase-tools/issues/3329)
#154

@WolfgangSenff I ran this through our tools and it seems to be working fine. No errors when running the test harness


[New architecture] Upgrade @react-native-clipboard/clipboard

We need to upgrade @react-native-clipboard/clipboard in order to use the new architecture in the App.
More context: https://expensify.slack.com/archives/C01GTK53T8Q/p1707293299589539
Discussion here: https://expensify.slack.com/archives/C01GTK53T8Q/p1707293347869349

Sorry, this isn't a new library, so we don't need to use the AutoAssignerAppLibraryReview label on this one

This is just forked off of the larger ongoing initiative to enable the New Arch in E/App

Hey! I’m Wojtek from Software Mansion, an expert agency, and I’d like to work on this issue.

Adding BZ so we can pay @aimane-chnaif

I think this should be paid out at the standard rate of $500, assuming no regressions


Crash when pressing U on an item

When I press U on an item in NEI (Seared Bricks in this case) it crashes my game and points to Mekanism.
Full crash log:
http://pastebin.com/5U9kCkFy

This is an integration issue - some mod is attempting to utilize the Mekanism recipe handler in the outdated way.  I assure you that if you tested this with only Mekanism you would not experience this issue.


Don't adjust startDate for full day events if endDate is in the past

When including past events in the event list for broadcast, the adjustment to make multi-day events appear as happening today had the side effect of promoting single-day full-day events from yesterday to today.  Adding a check to ensure that the event hasn't ended before updating the start date fixes my use case, however, I think a proper fix needs to perform the change in calendar.js when building the dom, as it is, multi-day events will still have their start date updated until after they've ended, which could be confusing for CALENDAR_EVENTS consumers.  Alternatively, a second field (originalStartDate?) could be added for broadcasting, but that seems like a hack.

Codecov Report

Merging #2818 (d588fab) into develop (81ae95e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2818   +/-   ##
========================================
  Coverage    64.91%   64.91%
========================================
  Files            8        8
  Lines          285      285
========================================
  Hits           185      185
  Misses         100      100

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 81ae95e...d588fab. Read the comment docs.

Thanks for your contribution! My apologies for my slow response. Life got in the way.


Urivalidator Implementation.

This will validate Uri on basis on the Uri passed, whether its valid or not and return boolean or UStatus.
Note : 1. 8 test case differ in result when comparing to Java.
2. LongUriSerializer deserialize API doesn't provide result as per Java sdk.

Pr has gone stale, closing


size marked area indicator based on grid column size and/or zoom level

It seems like it's currently possible to zoom out to a maximum level but zoom in infinitely until things looks really strange or break until you reset the zoom.
Would this change implement a maximum zoom in level? How many levels of zoom are you thinking of adding? What is the default level of zoom currently being set to?

It seems like it's currently possible to zoom out to a maximum level but zoom in infinitely until things looks really strange or break until you reset the zoom.
Would this change implement a maximum zoom in level? How many levels of zoom are you thinking of adding? What is the default level of zoom currently being set to?


Markdown table rendering does not respect escaping of |

Hello, like I mentioned before I generate jsdoc to markdown. A standard way to specify multiple types allowed for a parameter is something like
* @param {String|Array} id the document id / key or an array of keys

which gets converted into markdown table like so:
| Param | Type | Description |
| --- | --- | --- |
| id | String \| Array | the document id / key or an array of keys

This is valid markdown and renders correctly:

Param
Type
Description

id
String | Array
the document id / key or an array of keys

With docsify it gets rendered as pictured:

The escaping \| is not respected it seems.
I'll try and dig into this some more.

This is marked bug, I think you should report a issue to marked repo.

Right wasn't sure if it was the markdown parser / renderer issue or docsify. There is already an [issue for this bug[(https://github.com/chjj/marked/issues/595). Although marked doesn't seem to be maintained according to issue (724)[https://github.com/chjj/marked/issues/724]. Others are suggesting using markdown-it.
A workaround is to use | instead of \|.

markdown-it size is very big. I do not think it's a big problem. But you can custom markdown parser by markdown method https://docsify.js.org/#/markdown


deployEnvironmentX mutations don't always report errors correctly

Describe the bug
When running a deployEnvironmentX mutation, if any errors are encountered it doesn't always respond with an actual API error, just a string representation of an error.
Example:
# first update a project to use an invalid build system
mutation updateProject {
  updateProject(input: {
    id: 32,
    patch: {
      activeSystemsDeploy: "lagoon_kubernetesBuildDeploy"
    }
  }) {
    standbyRoutes
  }
}
# then deploy the latest of an environment within that project
mutation deployLatest {
    deployEnvironmentLatest(input: {
      environment: {
        id:14
      }
    }
  )
}
{
  "data": {
    "deployEnvironmentLatest": "Error: Unknown active system 'lagoon_kubernetesBuildDeploy' for task 'deploy' in for project ci-features-control-k8s"
  }
}

Running the same against an environment/project that has no access, we get an actual error from the API.
{
  "errors": [
    {
      "message": "Unauthorized",
      "locations": [
        {
          "line": 1412,
          "column": 5
        }
      ],
      "path": [
        "deployEnvironmentLatest"
      ]
    }
  ],
  "data": {
    "deployEnvironmentLatest": null
  }
}

Additional context
This was reported against the CLI, but the actual issue exists in the API (CLI has a possible workaround)
https://github.com/amazeeio/lagoon-cli/issues/189

resolved in #3568


Adding common functions which are not a part of Power Automate, such as pow() or sqrt()

The format and basic function offering are based heavily on Power Automate and I think we should keep it that way.
However, to the purpose of this project is to have functionality used by PowerAutomateMockUp and to make it easier for others to use the same expressions in their projects, why it is a standalone project.
To not break the Power Automate version and to still offer simple functions to other users, we could add an extension NuGet, where some extra functions could be added to the Dependency Injection, such as pow() or sqrt(). This could be done by:
// ...
services.AddExpressionEngine();
services.AddExpressionEngineExtensions();3
// ...

Then AddExpressionEngineExtensions could be located in Delegate.ExpressionEngine.Extensions namespace.
@pksorensen what do you think?

One NuGet package and one extension method, then add an option to not add additional functions.


cage$encaged$encage.js:8-10: Implement...

The puzzle 3-5ab7f66f from #3 has to be resolved:
https://github.com/objectionary/eo2js/blob/05f2c31c54ef4fff7087c74e6774a74ccefb1687/eo2js-runtime/src/objects/org/eolang/cage$encaged$encage.js#L8-L10
The puzzle was created by maxonfjvipon on 22-May-24.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 3-5ab7f66f has disappeared from the source code, that's why I closed this issue.


Add NG Explorer

What is the purpose of the change?

Describe the changes to the documentation

Notes

@cordt-sei Done


Showing backdrop

v4
Will there be scripts for showing backdrop?

This would be a feature request, but I'm not sure adding in a polyfill for a Webkit CSS filter falls under the role of this CSS framework. So, my guess would be: probably not.

Not likely, it's pretty new and unsupported.


[RESOLVED] High ICMP ping response time

Problem started at 14:50:14 on 2023.04.28
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Operational data: Value: 642ms
Original problem ID: 20162
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23132&eventid=20162

Problem has been resolved in 2m 0s at 14:52:14 on 2023.04.28
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 20162


style(img): pure css lightbox

Fix RM-2076

🧰 Changes
For our lightbox effect, we render each image twice which can cause memory issues when rendering image-heavy pages on iOS. We can cut this back by 50% by simplifying our lightbox to use a pure CSS implementation instead.
🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.60.1


Move the tool section on the new character sheet

Even by default, one has to scroll to even notice the tiny section. And with the portrait section collapsed, this particular tab of the sheet has a lot of dead space that could have made use of it. The above image is on a standard size monitor with the sheet at maximum height.
For games that make actual use of tools often, this new sheet makes it very inconvenient to put into practice.

Adding a quick toggle button to swap the skill section between showing skills vs tools would go all the way

There is space to flex the tools onto the right if the sidebar is collapsed, but doing it purely with CSS is non-trivial I think. If the sidebar isn't collapsed then I don't think there's space as having a lot of tools and proficiencies would make the right-side column look unbalanced.
Toggling tools to the top is an option, but this is a very niche and table-dependent preference I would say, so more module territory.

Toggling tools to the top is an option, but this is a very niche and table-dependent preference I would say, so more module territory.

I wouldn't think anyone would want tools displayed above skills, but the ability to hit a quick toggle to swap the view between skills and tools would be quite helpful.
An example here.
https://github.com/foundryvtt/dnd5e/assets/50169243/99171d40-4782-4255-b31b-90efff3a00d3

Maybe replace the Icon for the toggle with something like "Tools" that switches over to "Skills" when you swap over? Would be a bit clearer

Toggling tools to the top is an option, but this is a very niche and table-dependent preference I would say, so more module territory.

I wouldn't think anyone would want tools displayed above skills, but the ability to hit a quick toggle to swap the view between skills and tools would be quite helpful.
An example here.
Recording.2024-02-01.165426.mp4

Maybe instead of an icon it could just say "Tools" or "Skills" depending on the view? Would make it visually obvious to a degree


Swaps: Fix Formik warning

Also get this when going to the confirmation page

Sorry, not sure how I missed those but will fix.


Provide examples for pd.DatetimeIndex etc.

[ ] Add tests of conversion to pd.DatetimeIndex and pd.PeriodIndex.
[ ] Update HOWTO docs page with examples.
[ ] Add kwargs to to_pandas().

Closes #80.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #102   +/-   ##
=======================================
  Coverage   94.63%   94.63%
=======================================
  Files          17       17
  Lines        1847     1847
=======================================
  Hits         1748     1748
  Misses         99       99

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 7f4f131...a05e807. Read the comment docs.


bug: Most videos are bookmarked, without explicitly being done.

Describe the bug
Videos till week-12 are bookmarked by default, and errors out if bookmark being removed
To Reproduce
Steps to reproduce the behavior:

Uncheck bookmarks, which are previously bookmarked
See error

Expected behavior
Expected:
a) No previous bookmarks, which I didn't do.
b) No error if uncheck it.
Screenshots or GIFs

Works fine for new bookmarks

This is a pretty weird bug. we can't reproduce it in development.
We could speculate why this error is happening.

There is no Bookmark data in the database

but on the client side we are manual assigning some bookmark data like

{
    id: 5,
    userId: '1',
    contentId: 3,
    createdAt: new Date(),
}

when you click on bookMark button, the deleteBookmarkHandler gets triggered and tries to run the following code to delete the bookmark data from db. since you don't have a bookmark record with id:5 and userId:'1'  we are getting the prisma error.

  const userId = session.user.id;
  const { id } = data;
  console.log(id);
  try {
    const deletedBookmark = await db.bookmark.delete({
      where: { id, userId },
    });

The only way to reproduce  this bug in dev  would be to modify the addedBookmark state with wrong data that doesn't exist in database.
! Not sure why this bug would appear in production, we'll probably have to look through the data in prod. Right now in dev we are using less example data and in production we have more data so maybe the the current code which works for a single data item, might be producing this bug, when used with a lot of data.
export const useBookmark = (bookmark: Bookmark | null, contentId: number) => {
  const newBookmark = {
    id: 5,
    userId: '1',
    contentId: 3,
    createdAt: new Date(),
  };
  const [addedBookmark, setAddedBookmark] = useState<Bookmark | null>(
    newBookmark,
  );

Were you able to get the cause of this problem?
If you haven't started working on it, I can pick it up.

Were you able to get the cause of this problem?
If you haven't started working on it, I can pick it up.

I don't see this problem in development, it probably has something to do with the production database.
Go ahead and work on it, if you'd like to.

So the issue here is even my userId is 40541,  the server action fetches the bookmark with any userId (exists in db) as you can see in below screenshots.

This is because there isn't any constraint while fetching all the contents.

There must be a constraint that makes sure we are fetching the bookmarks of the currently logged-in user only.

cool. nice job bro.
Did you insert more data into the database to test it out?

@uttamsutariya how did you found out that issue is with bookmark:true by adding more videos or by creating a diff user?

Initially, I didn't do anything in my local cms.
I saw the prisma error message, then I tried to see the network request to remove the bookmark.
While doing this I saw that for each request the userId is not unique & not mine. So I felt that there is some issue with all the the fetched bookmarks.
Then I saw all the bookmarks have different different userId.
Now at that point I understood the problem that while fetching the content, the bookmarks are not being fetched of only mine, it's being fetched without any constraint of currently logged-in user.
So this way then I tried to create few users in my local db & resolved this issue.
So this kinda debugging skills you will eventually learn while working with prod databases & stuffs.
@hkirat Can you please review this PR as many people are struggling with the unexpected behaviour of bookmarks that they haven't added.


Update MSAL 4.64.0 and add WAM Broker

Updates MSAL to 4.64.0 and adds the WAM Broker to meet new security standards

Contributor

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


Add Ruby 2.3 to .travis.yml

Pending resolution for https://github.com/travis-ci/travis-ci/issues/5361#issuecomment-167517718

@jrafanie please see correspondence here https://github.com/travis-ci/travis-ci/issues/5361#issuecomment-167517718
how would you suggest to proceed? will you add 2.3.0 to miq travis or wait till resolving 2.3 will be done?

We'll have to use 2.3.0 in the .travis.yml here and in miq until they update the build environment.

We'll have to use 2.3.0 in the .travis.yml here and in miq until they update the build environment.

that's possible, I was just wondering how urgent this is for miq in regards of waiting till 2.3 is sorted to keep the naming alignment.


small style/typo edits to readmes

label should be closed by label

I reverted the last commit, so I think you should be able to merge now if it looks good.

I see that the tag typo was fixed here: https://github.com/mavoweb/mavue/commit/463d9489e691e59692e4ce0f8ab398cee0a4c98a
Should this PR be closed or rebased & merged?


Question about initiating multiple NDPs

Hi all,
I have read through the related paper and this work is impressive.
My research is on near-cache computing.
Since I want to use several NDPs near the L2 cache, can current NDPmulator support multiple NDP initiation?
If I want to modify it to support multiple NDPs, what would be the suitable steps to achieve this goal?
Thanks again for your valuable work!
Best,
Jingyao

Hi, Jingyao,
Thank you for your interest in NDPmulator.
NDPmulator allows to add as many NDP devices as you want to the processing system. After having described your NDP device models, you just have to instantiate them in the Python simulation script as follows (for the purpose of this example, consider se-run-ndp_dev_a.py as a base).
Let us assume that you have two NDP devices (ndp_accel1 and ndp_accel2), and you want to couple both of them to the L2 cache.

First, you will have to instantiate both devices. Make sure that the addresses of the programming interfaces do not overlap (ndp_ctrl).

# Create NDP devices
system.ndp_accel1 = NDPDev1(ndp_ctrl=("0x40000000", "0x40001000"), ndp_data=("0x40001000", "0x80000000"))
system.ndp_accel2 = NDPDev2(ndp_ctrl=("0x40001000", "0x40002000"), ndp_data=("0x40001000", "0x80000000"))

Connect the control interface of both NDP devices.

# Connect NDP devices to the CPU and L1D to NDP devices
system.ndp_accel1.cpu_side = system.cpu.dcache_port
system.ndp_accel2.cpu_side = system.ndp_accel1.mem_side
system.cpu.dcache.cpu_side = system.ndp_accel2.mem_side

Connect the NDP devices to the L2 XBar.

# Connect NDP device to L2
system.ndp_accel1.dma_port = system.l2bus.cpu_side_ports
system.ndp_accel2.dma_port = system.l2bus.cpu_side_ports

Take into account that by having multiple NDP devices with independent LSUs coupled with the same memory level may generate unnecessary contention, and you may have tweak parameters such as max_rsze and max_reqs to make the system work (since cache will allways accept requests but the actual packet queue may grow beyond the limit, causing the simulation to crash. It would be preferrable to encapsulate all the NDP devices into the same model and instantiate said model just once.

Hi, Joao,
Thanks for your detailed instructions!
I noticed that two NDPs are cascaded since only ndp_accel1.cpu_side connects to the cpu.dcache_port.
If I want these two NDPs directly controlled by CPU and two NDPs can issue memory requests at the same time, should I add extra modules between parallel NDPs and cpu.dcache_port, and also modify the cache controller to make it able to handle simultaneous requests?

Hi again, Jingyao,
In order to connect both NDP devices to the CPU and avoid dealing with some complicated overlaps in the memory mapping (which I tried before), the easiest way to to couple multiple NDP devices is to mount them like so.
In reality, NDPDev1 will bypass all requests to NDPDev2 with latency 0, so what is actually simulated is a scenario where both the NDPDev1 and the NDPDev2 are connected directly to the CPU.
If you want to pursue the alternative way, you can connect the cpu.dcache port to a XBar and connect both NDP devices and the L1 data cache to that XBar. However, this will require you to redesign the memory mapping of the entire system (which is a rather complex task and there is virtually no advantage of doing so).
I hope this aswers your question!

Hi Joao,
Thanks for your quick and valuable reply!
There is no question from my side (currently?).
Really appreciate your opensourced work for helping future researchers (like me)!


Output Size

@yysijie @yjxiong
I am confused about output size.
I think it is  [class_num, X,  joints_num, person_num]
Does X mean every 4 frames we have a prediction of action, as #120 described?
If so, is the prediction of action for the 4 frames independent or relayed on the predictions of  latter frames.
Cause I printed the label_name_sequence
label_name_sequence = [[label_name[p] for p in l ]for l in label_sequence]
and I found out label_name_sequence  didn't stay the same when the input frames number changed.
for example, in ta_chi.mp4
when frames_num = 300, the first output of label_sequence is 'throwing discus'
but frames_num = 40, the first output of label_sequence is 'stretching arm'

Hi @LucyLu-LX,
Yes, every 4 frames we have a prediction of action. However, it does not mean the receptive field of each prediction includes only 4 frames. They overlap.


You can get the correct words from http://www.chinesetest.cn/index.do

First ， You may login in the http://www.chinesetest.cn/index.do
Then, You can find the window .click it.

There is only this http://www.chinesetest.cn/standardsAction.do?means=standardInfo but no downloadable files right?

It seems someone did download that files: https://github.com/shawkynasr/HSK-official-Query-System

It seems someone did download that files: https://github.com/shawkynasr/HSK-official-Query-System

Yes . Maybe it contains some errors.

There is only this http://www.chinesetest.cn/standardsAction.do?means=standardInfo but no downloadable files right?

Yes. We can get the data by  Web Crawler .  Very easy

Sorry  for my late reply. This is HSK website by official http://www.chinesetest.cn/standardsAction.do?means=standardInfo. There is no download files. But its the best wordlist in the internet. In your last email, you said someone in Github supply the wordlist for HSK. He said that he got the wordlist by OCR. Maybe some wrong message in his wordlist. Any question please e mail to me?
------------------ 原始邮件 ------------------
发件人:                                                                                "elkmovie/hsk30"                                                                                @.>;
发送时间: 2021年7月17日(星期六) 中午1:34
@.>;
<EMAIL_ADDRESS>主题: Re: [elkmovie/hsk30] You can get the correct words from http://www.chinesetest.cn/index.do (#6)
There is only this http://www.chinesetest.cn/standardsAction.do?means=standardInfo but no downloadable files right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It seems someone did download that files: https://github.com/shawkynasr/HSK-official-Query-System

Yes . Maybe it contains some errors.

it seems now they're fixed.

Have anyone tried to get the list too? I'm getting some mistakes out of it

I fixed it according official book
You can check now


Display SSP Data Flow

As a user of the OSCAL Viewer, I want to see the Data Flow of a System Security Plan (example) including any specified inline images so that I can understand how data moves in and out of the system boundaries.
Acceptance Criteria

A Data Flow section is displayed in System Characteristics if present in the OSCAL SSP
The Data Flow description field is displayed
The Data Flow remarks is displayed
Any Data Flow diagram images are displayed with any defined caption

Implementation Notes

For sizing purposes, we will assume #225 has already been completed
We're likely to develop a Diagram component that can be re-used across Authorization Boundary, Network Architecture, and Data Flow components
We could potentially modify the existing oscal-rest-service test SSP to include an authorization boundary section pointing to an actual image, but any example SSP with that data will suffice

@kylelaker, would you mind duplicating the oscal-rest-service test data changes to the oscal-demo-content repo as well?
In the future we'll look to pull the demo content as the test data rather than duplicating.


Building for iOs fails with "ld: 44 duplicate symbols for architecture armv7" error

Hi all,
Trying to build Tensorflow for iOs. The step tensorflow/contrib/makefile/build_all_ios.sh after around 20 minutes of building returns errors including duplicate symbols for architecture armv7. Please find the details bellow.
Environment info
Operating System: Mac OS X 10.11.6 (15G31)
Xcode 7.3.1 (7D1014)
Steps to reproduce
In terminal:

git clone git@github.com:tensorflow/tensorflow.git
cd tensorflow
tensorflow/contrib/makefile/download_dependencies.sh
tensorflow/contrib/makefile/build_all_ios.sh

What have you tried?

Removing the repository directory and going through the steps again.
Going through "Building by hand" steps described here. Error: ld: 44 duplicate symbols for architecture armv7s (Error log).

Error log (Full log)
...
ld: 44 duplicate symbols for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [/Users/b0915218/Xcode/tensorflow/tensorflow/contrib/makefile/gen/bin/ios_ARMV7/benchmark] Error 1
+ '[' 2 -ne 0 ']'
+ echo 'armv7 compilation failed.'
armv7 compilation failed.
+ exit 1

Hi,
I'm having the exact same error message.
OS: OS X 10.11.6
Xcode 7.3.1
Many thanks,
Joe

Today tried to pull the project again and go through all the steps from the beginning. It worked. Thanks for fixing!


Nowhere to insert quality module

Mod source
Factorio Mod Portal
Operating system

=Windows 10

What kind of issue is this?

[X] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
No machines support to insert module quality, so crafting higher quality product is impossible.
Steps to reproduce
No response
Additional context
No response
Log file
No response

when use pycoalprocessing,all machine start cant use the quality module

maybe the reason is the machine prototype allowed_effects doesnt have "quality"


How to predict the first frame or last frame of a video

Thanks for your great work!
I have a question that how to predict the first or the last frames that no former or latter frames?
In your work, it seems you use neighbor frames around the key frames to help predict the target frame.

In the paper, specifically Table 4, you can change the setup.
For example, on the last frame, you can change the context using only previous frames.
There are performance drop but not significant.


Correct clustermesh identity sync kvstore backend usage (to actually use the remote)

This PR is easier to review commit-by-commit. There are many changes but they are mostly moving behaviour around, or passing in the backend variable instead of using the global.
Clustermesh was broken. Routing, ipcache and node sync worked but policy did not. This meant policy drops because policy maps never saw the remote identities, but ipcache did and the datapath would match correctly.
Clustermesh identity sync accidentally used the cilium-agent local etcd backend. This was because kvstoreBackend, the middle-layer between the identity allocators and etcd/consul, accidentally called the pkg/kvstore functions. These functions then used the global etcd client, which is the one talking to the cluster-local etcd.
Clustermesh identity sync also accidentally never started. This was because CachingIdentityAllocator.WatchRemoteIdentities ignored the etcd backend passed in and called the watch functions on the global identity allocator. This had the added effect that it the allocator added itself to itself as a remote cluster allocator, causing identity list to double print identities.
ipcache and node sync worked because they used the correct, remote, etcd backend. The wiring around allocators did the wrong thing. This has likely been the case since the abstractions around CRD identities were added.
Edit 2020-02-14
Something is broken, but I don't know what. When testing with our clustermesh tutorial (https://cilium.readthedocs.io/en/latest/gettingstarted/clustermesh/) and these steps (https://gist.github.com/raybejjani/1272bea3e9d2beb40bcf4d115768b785) I see identities propagate across clusters but the policymap never gets remote identities. I even see log lines like level=debug msg="ToMapState: Allowed remote IDs" EndpointSelector="&LabelSelector{MatchLabels:map[string]string{k8s.io.cilium.k8s.policy.cluster: cluster2,k8s.io.kubernetes.pod.namespace: default,k8s.name: rebel-base,},MatchExpressions:[]LabelSelectorRequirement{},}" policyID="[]" subsys=policy trafficDirection=Egress but it never uses the remote IDs :/

This change is

never-tell-me-the-odds

never-tell-me-the-odds

Coverage decreased (-0.01%) to 44.491% when pulling b2a6131dfbb38c2ceb0364e1232df663b61f614a on raybejjani:clustermesh-kvstore-fix into 1810709a25e50d8db9101d1ede71be8568220317 on cilium:master.

never-tell-me-the-odds

never-tell-me-the-odds

never-tell-me-the-odds


Device is being created without field errorCode leading to fault in smart-home-java backend

After creating Television device, an issue started happening on smart-home-java backend as the data model of television on Cloud Firestore was created without errorCode field.
After adding it manually in fire Cloud Firestore, the error was solved. Probably other devices will have the same issue.

Does this only happen with TVs?

No, I just created some other devices and it is happening with all of them, errorCode is not added in the data model.


Issue with Ollama and Pydantic

Hi guys,
Are you also facing issues when using Ollama with Pydantic? It seems that the response from the LLM is often not put in the output format as requested by Pydantic. Hence, the subsequent code in our program throws an error.
In our tests, we used Codestral-22B and the following function to call the LLM inference. We also tested the same with the API from Mistral, and this works. So it seems there is a problem in the interface between Ollama and Pydantic.
Do you have good experience with Ollama and Pydantic? Do you know how we can solve this?
Many thanks for your help!
Pseudo code (not functional, just to show which functions we call)
from langchain_experimental.llms.ollama_functions import OllamaFunctions

llm = OllamaFunctions(base_url=Config.HOST_LLM, temperature=0, model="codestral:latest", format="json")

llm = llm.with_structured_output(MyPydanticOutputClass)

llm.invoke({...}) # pass data required by the LLM to perform inference

We are using Pydantic through the call with_structured_output:
llm.with_structured_output(MyPydanticOutputClass)
So, sometimes there are some fields of the Pydantic object that are not parsed, so the output of the LLM is incomplete, for example:
Got: 3 validation errors for CodeGenerationOutput
dev_mode
field required (type=value_error.missing)
rtos
field required (type=value_error.missing)
is_func_empty

And sometimes producing a more general error:
raise ValueError(
ValueError: Failed to parse a response from codestral-latest output: {}

Hey @jonabeysens
do you know from Langchain how it instructs the LLM with the Pydantic model you provide for the response format and how it processes it?
I do have some experience with Pydantic and Ollama within my own library (ollama-instructor), where I instructed the LLM to adhere to the JSON schema of the Pydantic model. And yes, sometimes the models are no able to provide the properties of Pydantic model correctly. For this reason I added retries for malformed responses and the possibilities to receive partial models. But I do not know how Langchain handles the response.
You could try to instruct the LLM with the JSON schema while using the ollama client instead of Langchain and see what is coming back from Codestral. And you could try it with my library if this is working for your use case.
To be honest, without knowing the exact response from the LLM it is not that easy. I hope my comments helps anyway. 😊


CI is broken due to nightly bug

https://github.com/rust-osdev/uefi-rs/actions/runs/4439093815
Looks like https://github.com/rust-lang/rust/issues/109209, which may be the same as https://github.com/rust-lang/rust/issues/109199. Hopefully next nightly will resolve this.

Fixed in latest nightly.


Support for multiple languages (incl. English)

This PR adds support throughout the site for Localisation and Internationalization, and also provides the first translation into English.
A few notes:

The base language is German - that is, further i18n changes should use the German form to create the i18n tag. The main website will continue to be served in German, with an /en sub-site.
This whole thing is quite scrappy, especially around the live-telemetry shortcode. See comments.
English translation is quite rudimentary, but I've tried my best! Feel free to alter as you see fit.
The language picker in the heading needs major improvement.  A dropdown is probably the right way to do this, but I don't have a clue when it comes to CSS beyond the basics. Maybe someone else could take a whack? I've added a simple implementation that lets you switch between English and German.

This can be shipped in this state if we want to get it out of the door, hence why it's not marked as a draft, but improvements would be appreciated.

Whoah.... thank you! As for the picker: I'm no ux guy, but many websites just show EN|DE. We probably won't translate this into 20 languages any time soon.

As for the picker: I'm no ux guy, but many websites just show EN|DE. We probably won't translate this into 20 languages any time soon.

I feel like making it a picker would encourage further translation into other languages later on down the line - would be trivial to just make it a toggle for the time being though.

Don't supposed you've had any chance to take a look at this @naltatis?

@duckfullstop sorry for letting this wait for so long. I'll review this today with the goal to get the English version live very soon.

Thanks for your work @duckfullstop. I've made a couple of adjustments to the code and the content.
I've tried to change the configuration in a way that /en and /de would both be served from subdirs with / redirecting to the appropriate language (browser lang). But this (smart redirect) does not seem to be possible with gh-pages. In addition, moving German to /de would break our short urls /sticker and /slack and I didn't find an easy and non-hacky way to preserve this. Thats why I decided to stick with your proposed "German stays in /" configuration for now. This is non invasive and we can always improve this later.
Thanks for your works and again sorry for the delayed response 🙌


Add unwrap_or_default method to the Map.

unwrap_or_default should return a default value: Value::Null. Exists places, where's duplicating, and these places are annoying.

I'm not clear on exactly what signature you have in mind, but the standard BTreeMap and HashMap do not have an unwrap_or_default method, so I would prefer not to add one to serde_json's map.


Error if using a column that is not id as primary key

I have found an issue, that if I set a different column as the primary key, it says that the MySQL syntax is incorrect.
[info] Hash calculation:
CONCAT(COALESCE(LOWER(CONV(BIT_XOR(CAST(CONV(SUBSTR(MD5(CONCAT_WS('#', `username`, `groupname`, `priority`)),17,16),16,10) AS UNSIGNED)), 10, 16)), 0),COALESCE(LOWER(CONV(BIT_XOR(CAST(CONV(SUBSTR(MD5(CONCAT_WS('#', `username`, `groupname`, `priority`)),1,16),16,10) AS UNSIGNED)), 10, 16)), 0))

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') < ()) t limit 1' at line 1

Same here. It seems to be a bug. Did you find a workaround?


Error spam

What Happened?
Happened after starting a new save, getting launch codes and linking with the Nomai statue. Went down to 2 FPS
What was supposed to happen?
No response
Platform
Steam
Mods
New Horizons examples (0.16.0 dev branch)

Logs

Nevermind this is just because of the eye branch


[FR] Request an advice on Internet-draft about new EDE codes

Describe the desired feature
The Internet draft draft-bortzmeyer-more-edes creates three new EDE (Extended DNS Errors) codes. At the IETF meeting in Dublin in november, Petr Špaček suggested to request feedback from implementors.
Therefore, I would like to know if you would consider to implement all or some of these error codes and if you find them useful.
Note that the policy for the EDE registry is just "first come, first served" so consensus is not strictly necessary but would obviously be cool.
Potential use-case
Debugging and information

For the sake of discussion my concerns are:

IP tailoring, this can already be covered by one of the available blocked, censored, prohibited, forged answer. Not sure if operators would like to be specific about IP. For the example of ECS, you can tell by the returned option if the answer is tailored for your IP range, or available for all networks (i.e., <IP_ADDRESS>/0).
Minimal response, for operators that configure minimal responses it seems counter intuitive to enlarge the response with EDE codes because then Unbound would almost always reply with the given EDE code.
Local root, not clear to me if that would be attached to root replies specifically or any answer that started iterating from the local root down.

With my implementer's hat on if these were to be introduced they would probably be configurable and not turned on by default when ede: yes is used.

* IP tailoring, this can already be covered by one of the available blocked, censored, prohibited, forged answer.

I do not see the relationship.

For the example of ECS, you can tell by the returned option if the answer is tailored for your IP range, or available for all networks (i.e., <IP_ADDRESS>/0).

No, the reply just says that the server understands ECS, not that this specific address was tailored.
% dig +subnet=<IP_ADDRESS>/24 @<IP_ADDRESS> www.bortzmeyer.org

; <<>> DiG 9.18.28-1~deb12u2-Debian <<>> +subnet @<IP_ADDRESS> www.bortzmeyer.org
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 61275
;; flags: qr rd ra ad; QUERY: 1, ANSWER: 2, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags: do; udp: 512
; CLIENT-SUBNET: <IP_ADDRESS>/24/0
;; QUESTION SECTION:
;www.bortzmeyer.org.	IN A

;; ANSWER SECTION:
www.bortzmeyer.org.	3445 IN	A <IP_ADDRESS>
www.bortzmeyer.org.	3445 IN	RRSIG A 13 3 86400 (
<PHONE_NUMBER>1111<PHONE_NUMBER>3526 21439 bortzmeyer.org.
				GwUhBuO4kgz4Qt55Lpq2CykhkdMowgxxTMhOX32wHQxO
				dcI2HTwD6WWy5iBJTJZtovMFVWB2YpoTXY1jQoa/0g== )

;; Query time: 20 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>) (UDP)
;; WHEN: Tue Dec 03 11:47:52 CET 2024
;; MSG SIZE  rcvd: 184

(Here, there is just one address so no tailorisation.)

I do not see the relationship.

I mean that based on IP an answer could be tailored (forged answer, censored), or denied (blocked, censored, prohibited). And it could rely on configuration per local data or RPZ file you load.

No, the reply just says that the server understands ECS, not that this specific address was tailored.

That answer has a SCOPE PREFIX-LENGHT of 0 which means that the reply is good for all networks. If that was higher it would mean alternate answers exist for other networks. The FAMILY and SOURCE PREFIX-LENGTH are always echoed back from the auth name server.
I maybe was not precise by using the '<IP_ADDRESS>/0' notation earlier, instead I should have used just '/0'.


Rewards jumped to 0 after withdraw the rewards

@kanniec826 can you elaborate on this? i think it makes sense for rewards to be 0 after withdrawing it

@calvinkei Please ignore this story card, I put in wrong page, suppose to be for my reference only


Input System - Create Input Profile System

Create Input Profile System: This system will use the DeviceSourceManager and create a series of input configurations that the user can customize for easy input organization.

This might completely change with the new input architecture, We ll reopen when ready.


How can I add a new translation (greek)?

Hello, I would like to add a new translation for Greek as I endorse this document and a greek translation is missing. How can I do that?
Thanks,
Nikos

@foo123 just fork this repo, create corresponding directory in content (like content/fr) and translate all *.md files.
Here is example of Ukrainian translation: https://github.com/heroku/12factor/commit/3aed9f77cdd9d8648892bbf03489c36126d4b332

Ok will do, thanks. It might take me a while..


Adding Dameng database support

中国达梦数据库官网 http://www.dameng.com/
表和表字段修改不了
1：表字段修改不了
2：表注释和列注释修改不了

Please write comments in English.

Please write comments in English.

ok thank you

Hello, big man, the English version of China's dream database official website has been online, please support as soon as possible, thank you
http://en.dameng.com/products.aspx?TypeId=97&;FId=t3:97:3

Thank you for the news. It is in our plans to do it, but we don't know when exactly.

现在支持达梦数据库和瀚高数据库吗？


*ImplExt traits are not exported from gtk::subclass::prelude

The *ImplExt traits are required to call parent methods. Currently this requires explicitly importing them since they're not part of the prelude (with the exception of ObjectImplExt). Calling parent methods is a common enough thing that it might be worth to consider putting them into the prelude module.
I brought this up in the Matrix room yesterday and was instructed to open an issue here.

Yes it should be in the subclass prelude :) That wasn't noticed so far because usually people import the whole module with the subclass (e.g. use gtk::subclass::widget::*) in their code, but that shouldn't really be required.

Do you want to provide PRs for that (here and in gio and maybe in glib)?

@sdroege We currently don't have "subclass prelude", better to add it to base "prelude.rs" or add new file?

We do :) here: https://github.com/gtk-rs/gtk/blob/1e26d0ac57f3d71c5ed2c00fe913d106f0eb272a/src/subclass/mod.rs#L29

Thanks, updated gio's PR

@sdroege Sure, it only involves adding a bunch of pub use statements anyhow. So I suppose it should be added for all existing ImplExt traits then?

@tsahyt Yeah. For gio, @EPashkin did it already here https://github.com/gtk-rs/gio/pull/284

@EPashkin This is done now or was something forgotten?

As I remember all done

@GuillaumeGomez Please close :)


Check what funds are locked for governance, and hence re-usable

... and show a "max" button, or at least some info about the max amount that can be used for delegation

bumping the prio of this one, we should show above the input:

free balance
locked balance

The locked balance is the one that can be re-used.


ModuleNotFoundError: No module named 'ipynb_convert'

Hi all,
I'm trying to set up the extension on Windows 11 (Windows Terminal). I have checked the requirements:

NVIM: v0.7.2
Python: 3.9.13
IPython: 8.2.0
Matplotlib: installed

The error I am given is the following:

I would be very grateful to receive your suggestions.
Many thanks for considering my request.

Hi, yeah the plugin was not developed with Windows in mind. The error you're seeing is probably due to some of the paths being hardcoded with '/' instead of '', thus it fails to add the directory of the plugin modules to your system path in python. This should be an easy fix, but I'll still have to test whether all the features work as expected on Windows.
I'll get to it sometime this week as soon as I can get a Windows VM up and running

You can try again and see if it works as expected now, I tested the basic functionality on Windows 10 in PowerShell and got it working, let me know if you have any issues. Also make sure python3 is a valid command in your terminal (and not just python), otherwise you'll need to specify let g:_jukit_python_os_cmd = 'python' in your vim config.


Hair colour black to other colour is not working why?

changing hair-colour from one colour to other it's working except black to other colour, something has to change , please explain

I have a same question with you. Have you found any solution for this?


Azure pipelines tasks terraform has been unpublished

Hi,
I just wanted to mention that the extension you are using/linking to, has been unpublished from the market place.
I believe this is the one to use instead.

Thanks @Panzerbjrn, I was unaware. I do know the extension is still being maintained...I am guessing they switched it over to the one you described based on the old extension history.
I did uninstall and re install the new task and will require some updates to @version of the task. I will submit a PR once confirm everything is working.


samsung.com in list?

Hi! 'samsung.com' is in list https://raw.githubusercontent.com/7c/fakefilter/refs/heads/main/txt/data.txt
It's strange

we will remove it, upset providers (tempr.email in this case) try to inject some stuff into our list, this is why we have introduced https://github.com/7c/fakefilter/issues/29 which covers 98% of regular email hosts, but sometimes we. need your help to cleanup. Since our system is fully automated it is not easy to detect them. Thanks for reporting.

Thank you!


Snapshots: FX Chain not working on recall while recording

In recording state: All other aspects of Snapshots recall fine but FX Chain does not recall.
Works fine during normal play and stop states.

For some background info:
Yes, some SWS functionality (such as this one) was disabled while recording 'for safety reasons'.
There was a discussion started if it's actually still needed but it didn't conclude to something definitive (so far)
See https://github.com/reaper-oss/sws/issues/913

hi, I just faced the same problem. unfortunately during a concert that I wanted to record. thanks @nofishonfriday nofish for clearification and links.
but is there a solution for this problem? the only thing which comes to my mind right now is: add for each channel to be recorded a playback output and a new input in reaper preferences and link both (I'm on linux and use qjackctl so I can simply link both ports via the graph/patchbay). this way I can add the new outputs as hardware outputs in the track's routing dialog. finally, in a new project tab, I define exactly this track's output as recording input. now I can record in the new project and sws actions still work in the main project since here I'm not recording.
works but is quite cumbersome...
does anybody know of other solutions?
thanks very much!

Is there any way we can make this work? I'm using snapshots to change FX parameters for live use as well as practicing/recording. I find it's much less of a headache to maintain than programing multiple ReaLearn parameters. I discovered today though that I cannot use this Snapshots to change FX parameters while recording.