Is it possible to run VPTQ in CPU only?

Given the small size of the quantised models, I am wondering if it is possible to run VPTQ in CPU only?
Currently, this does not seem to be possible as one of the functions require CUDA.

The current stage of VPTQ can technically run on a CPU. You can disable the support for vptq.ops on CUDA and ROCM, which will make it natively run on PyTorch's CPU backend.  (only use self.dequant(), for example)
https://github.com/microsoft/VPTQ/blob/main/vptq/layers/vqlinear.py#L649-L651
# qweight = self.fast_dequant()
#             if qweight is None:
    qweight = self.dequant()

However, it is highly discouraged to do so because the FLOPS of CPUs are still relatively low, and there is a lack of adequate optimizations at this stage.

Recently, some colleagues asked if it is possible to use the latest CPUs (which usually have very large LLCs). I think this could be a very feasible system optimization. I'll close this issue for now, and I will seriously consider this. Welcome to contribue to CPU support if you like. Thank! Yang

I’ve been trying to run VPTQ in CPU-only mode following the instructions provided. However, I encountered an issue where the fast_gemv function appears to require CUDA as well, preventing the model from functioning on a CPU-only setup. Could you kindly advise on any possible solutions or workarounds to run VPTQ without CUDA? Thanks in advance!

I’ve been trying to run VPTQ in CPU-only mode following the instructions provided. However, I encountered an issue where the fast_gemv function appears to require CUDA as well, preventing the model from functioning on a CPU-only setup. Could you kindly advise on any possible solutions or workarounds to run VPTQ without CUDA? Thanks in advance!

Hi @zianttt,
I'm very sorry, I missed your message and only just saw it. I will follow up with the CPU method soon. Please wait a moment as I need to implement a CPU version of the dequant function.
Thanks!
Yang

Hi Yang,
No worries, thank you so much for your reply! Looking forward for the CPU
support.
Once again thank you for your work in developing this!
Regards,
Tan Zi An
On Thu, 7 Nov 2024, 18:25 Yang Wang, @.***> wrote:

I’ve been trying to run VPTQ in CPU-only mode following the instructions
provided. However, I encountered an issue where the fast_gemv function
appears to require CUDA as well, preventing the model from functioning on a
CPU-only setup. Could you kindly advise on any possible solutions or
workarounds to run VPTQ without CUDA? Thanks in advance!
Hi @zianttt https://github.com/zianttt,
I'm very sorry, I missed your message and only just saw it. I will follow
up with the CPU method soon. Please wait a moment as I need to implement a
CPU version of the dequant function.
Thanks!
Yang
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/VPTQ/issues/67#issuecomment-2461859320, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWBYOSLXWAOMZGSUDSBN7XTZ7M52FAVCNFSM6AAAAABQBEVOE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRRHA2TSMZSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


leverage hashHistory with react router

This leverages the HTML5 history API and is more standards compliant

in fact, hashHistory is the HTML5 API, so the real solution here is to implement #65


Implementing an alert before user tries to exit the tab/browser to avoid losing changes made in the editor

This PR tries to provide the solution for the following issue #86

Thank you for your contribution 🎉


Tutorial needs to focus, too scattered and difficult to get started

I hope redux authors can take this as constructive criticism.
I am using redux, thank you for that.
I take issue with the tutorial. The number of steps and backtracking to run your first Redux application was unacceptable. The tutorials I found on Medium were significantly better at getting Redux started quick, and explaining the components needed in an action, reducer, etc...
The tutorial was written with the understanding that you had experience with Flux. The tutorial was far too long before you could run a program.
People need to play to fully understand. By the time I was 'playing', the tutorial example never worked and I moved onto Medium for learning redux.
I think the tutorial needs to be completely re-written. Redux in 20 minutes and redux with networking would both be very useful.
I hope you can find value in this comment. I find the package great to solve my JS problems, but the tutorial needs improvement.

The main tutorial structure was written when Redux first came out.  At that time, many people were familiar with Flux, and so part of the intent was to explain how Redux was different from other Flux libraries.  Dan Abramov wrote some tweets describing how hard it was to balance the different audiences for the docs.  It's also hard to balance for people with different learning styles.  Dan's teaching style is very much "explain things from first principles".
I'd agree that the situation has probably changed by now.  Almost no one remembers what "Flux" was, and the target audience is likely to be different.
I've already got an open issue at #2590 suggesting we rethink the docs structure as a whole.  Unfortunately, we don't have time to focus on a docs revamp ourselves at the moment, but I'd be happy to work with you or anyone else who wants to contribute improvements.

Hi mark, I would love to help you out with this issue. Let me take a look at the existing tutorial, the suggestions in #2590 and come up with a list of things to discuss with you :)

I noticed today that bulleted items in the docs render with the bullet aligned with the last line of text for each bulleted item. This is not a big deal but can be distracting. I also thought that this didn't require a new issue. I found this "docs" issue was pretty recent and may be a good place to point this out.

What's the current status on this?  I'll walk through it and see if I can improve it.

@hedgerh : don't think anyone's actively doing anything with this.  Would be happy to have someone get involved.

What i had in mind is making a quick start guide with react-redux and with something to handling side effects (be it redux-thunk, ...)
As for the flux refs, i'm not really finding that much of them.

@JoviDeCroock : yes, a quick-start guide would be great!

I agree with the OP, the tutorial is unacceptable, and since I don't have any Flux experience at all, I have no idea what the comparison is about. Also, the todo example is waaay too large for the first timer.
When I was learning Vue.js at vuemastery.com I went through their tutorials in no time, they completely stole my attention with small step by step tutorials. Have a look at that, as I think it's one of the best I've seen around.
It doesn't always have to be text to make it clear and easy to start with, videos of IDE with explanations + code on the page are much much better, as I can code what they talk about at the same time.

I came here to open an issue about the docs, found this one. I'm not sure if I should post in #2933 or #2590 but I chose here because users are posting here vs contributors there.
I'm a newbie just went through react beginner tutorial. Then I came here and clicked Basic Tutorial.
I am very grateful there are docs. That in itself is fantastic. Thank you.
That said, I had some big problems with the "basic tutorial"

There is near-zero time spent explaining environment setup for the tutorial itself. There are output screenshots from browsers running the code but no time dedicated to telling you how to get this running in the browser. I'm not saying it's impossible to figure out on my own, but coming from cra, which this tutorial does not use, a basics tutorial really should cover this. If you want to avoid it by linking to sandboxes only that's fine, then explain how to get it running there and link there.
The code in the tutorial doesn't clearly indicate what file you are editing so you can follow along. They often only say at the end of the section.
The start builds on top of knowledge it assumes you have of flux by just telling you how it differs from the flux concept, not explaining the concept itself.
There is a ton of "Squirrel!" distractions from a simple streamlined flow through the tutorial for a beginner. A basics tutorial should not be so concerned at linking me to hour long talks, other tutorials, complete other libraries so much. Some "extra info this way" is expected, but I felt constantly conflicted if I should continue this tutorial or I needed to go spend several days elsewhere and come back.

While I understand there are a lot of other tutorials out there that do a good job, I came to the official page first and I saw an official basics tutorial so that's what I tried to start with.
It wasn't a good experience.

@ofbeaton : thanks for providing this feedback!  I appreciate you taking the time to comment, and those are some good thoughts.
FWIW, skimming your comment, I think I agree with most of your points.  We do actually already have plans to completely revamp the docs.  Unfortunately, there's other priorities atm, and we haven't been able to get started on that task yet.
If you're interested, I've written up my proposed approach for reorganizing and rewriting the docs here:
https://github.com/reduxjs/redux/issues/3313#issuecomment-450601554
All that said: our usual recommendation is that most folks should focus on learning React first, and then pick up Redux after you're already comfortable with React.  Based on your comment, it sounds like you're still very new to React.  You're totally welcome to learn Redux whenever you want, it's just usually easier to focus on one thing at a time.

@markerikson Thank you for being so welcoming. I know it can be frustrating having users complain.
I've read your comment there (it was long!) Overall I think it is a great plan. However.
It seems insanely ambitious and huge. Given how your first response was that you have other priorities atm (understandable), my suggestion would be to try to pare it down to something that is actionable and has value in the shorter term.
For example you might decide that given current efforts it is not realistic to keep a basic tutorial for beginners, and an advanced tutorial. That as the official documentation, you are there as a reference and source of truth, not to teach people and hold their hand. You could choose to point people to community efforts (maybe react-redux or redux-starter-kit) or not. Just host reference documentation.
If you decided to keep the tutorials, then I would expect them to be opinionated. I don't want to know the 10 different ways of doing this, the why it was designed a certain way, etc. I want you to pick one path, even if it's not "more correct" and show me how to use it while making as few assumptions of me as is reasonable (ha!).
Maybe you can have an info bubble with some links to more advanced docs but generally just saying there are other ways of doing things is enough. That is what the advanced docs are for.
Your focus on trying to get everything into a js sandbox I think will force you to make sure that people can actually follow along with the examples, and run them. Which were 2 of my complaints.
Good luck, and I hope to be around long enough to be able to contribute more substantially in the future!
I'm about 4 weekends into my react forray and loving it so far. Can't wait to see what Redux has to offer me. (My path so far)

Closing in favor of #3592


allow users to configure basic auth by providing basicAuthCredentials

By default, nothing needs to be configured. If the user tries to visit a URL which is behind basic auth, and exception will be with instructions on how to configure the basic auth credentials.
The user will have to implement this in his TurboNavigatorDelegate
var basicAuthCredentials: URLCredential? {
    get {
        URLCredential(user: "username", password: "password", persistence: .forSession)
    }
}

This feels like we are taking over too much control. I like the other approaches we went over in the original PR.


README/AuthoredArticlesWebsites: add minimaUSCGamecockSandstorm

| RalphHightower/minimaUSCGamecockSandstorm | USC Gamecock Sandstorm theme derived from Jekyll/minima, a one-size-fits-all Jekyll theme for USC Gamecock writers. (in dev/testing) |

#1520 closes #1512


Comparing two reports

Hello, is there any facility to compare two reports?

Not any 2 reports, but for now it can compare two consecutive oci360 executions. For more details, check this link

Much appreciated


Support per API-Key Limits

We need to support alternate limits for specific clients. Instead of listing specific IP addresses (like with -nolimit) some sort of api-key included in the header would be more general and flexible (e.g. changing IPs, sharing a single key between multiple machines). The rpm per key could then be specified via a flag whose value is a csv of : separated pairs of key and rpm - <key>:<rpm>. For example:
-limits AKXOXLEK:100,LXKFGODK:10

Perhaps an entry without an rpm value could mean -nolimit, and then we could deprecate the -nolimit flag.
-limits AKOXLEK,LXKFGODK:10

The API key format should remain as flexible as possible (any UTF-8 plaintext?) to maximize compatibility with any generation method.

Which HTTP header should we use for the api-key? Is there a standard one that is appropriate?

Authorization header typically.
Authorization: bearer TOKEN


Client Allocation Request for: SCVSoft

Client Allocation Questions
Core Information

Name: SCVSoft
Website / Social Media: www.scvsoft.net
Region: Asia minus GCR
DataCap Requested: 2 PiB
Addresses to be Notarized: TBD
Notary Requested: IPFS Japan Consortium, BlockchainWorld

wtf

amen brother

@kernelogic HELP ME

It seems you applied for 2PB datacap but this is regular application for 1~100TB.
You can try re-apply here https://filplus.storage/apply it should route you to the proper channel.

@kernelogic Oh tanks

@kernelogic https://github.com/filecoin-project/filecoin-plus-large-datasets/issues/1956 thx


can you provide the sulution about photo?

Hello, can you provide a flame that identifies the photo?

I'm not sure what you're asking. Could you please rephrase the question?


feat(CI): workflows for building site and verifying sample code

Summary
This PR implements CI workflows that verify changes do not break builds.
GitHub Workflows:

builds the workshop site
builds, synths, and tests sample code in each language

[x] Typescript
[x] Python
[x] .NET
[x] Java

Update guides to reflect the fixes I was required to implement

[ ] Typescript
[ ] Python
[ ] .NET
[ ] Java

Fixes #259, https://github.com/aws-samples/aws-cdk-intro-workshop/issues/189

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This is becoming pretty large. I'm going to split this into separate PRs for each language.


GitSync Errors Cause Duplicate Records

When the Git Sync fails for whatever reason, it seems to attempt to resync over and over again which causes it to duplicate things like scripts and variables, etc.

Note: Aggressively Test the GIT SYNC!

I think is has been resolved. I haven't seen it in the latest versions.


Avoid hard-coded URI

The hard-coding of the Socket.io URI silently breaks users of this library that pass in a full URI.
https://github.com/socketio/socket.io-client-cpp/blob/6063cb1d612f6ca0232d4134a018053fb8faea20/src/internal/sio_client_impl.cpp#L218
I'll see if I can whip up a patch for this when I get some time.

Yeah, this is a pain. I believe there are multiple threads related to this open currently. I'll also look into making a pull request.


pycrypto-->pycryptodome

inspired by Jeshuran's troubles. See: https://stackoverflow.com/questions/41813030/problems-with-installation-pycrypto-in-python-3-6

I don't think we are using this library anymore, could we remove it?

fine with me


Marks

Easy if not follow deletes for line number change..

Ok, kinda works with vim-style ones
TODO:

change locations based on changes in file ( I have no idea yet )
named marks


deployment().properties doesn't contain template.contentVersion

Is your feature request related to a problem? Please describe.
Currently with my ARM templates I like to be able to add the contentVersion as a tag onto resources. I am currently doing this through ARM deployment() function, such as:
"tags": {
            "displayName": "operations",
            "VersionTag": "[deployment().properties.template.contentVersion]"
         },

However, when trying this in Bicep, template is not an available properties option in the deployment() function, and therefore I can't get access to the contentVersion.
Describe the solution you'd like
Can the template.contentVersion be made available in the deployment().properties. Such as:
VersionTag: deployment().properties.template.contentVersion

We should get to this on 0.5 as a completeness thing. Will never stop being surprised at the variety of uses of contentVersion :)

We need to confirm if the runtime evaluates expressions in contentVersion or if it is always constants. If the answer is yes, this may require more than just a new decorator.

Much appreciated, thank you both.

Just wanted to poke on this a little more to make sure I understand the requirement. Can this be achieved by setting a variable at the top of your Bicep file, or does using deployment().properties.template.contentVersion provide other benefits?
e.g. something like:
var contentVersion = '<IP_ADDRESS>'

...

resource myRes '...' = {
  name: ...
  tags: {
    displayName: 'operations'
    versionTag: contentVersion
  }
  ...
}

The problem with using the contentVersion as a variable is that its constant, and it introduces high maintainability from our end. Currently we set the ARM template contentVersion as part of a versioning step on our CI/CD pipeline. Using the deployment() function is a really nice low maintenance method of retrieving that content version so we can add it as a tag.

Currently we set the ARM template contentVersion as part of a versioning step on our CI/CD pipeline

Ah, so you run the bicep build, then pass the JSON output through the custom versioning step? Thanks for providing the extra context!

That is correct.

We've used the contentVersion field in ARM templates to contain the version number for a long time (it is what it's there for, right? ;) ). The version is added through the build pipeline in the same way as we stamp our binaries and other artefacts, so everything aligns. Whilst there are other ways that we could pass that version into a resource tag, having support for deployment().properties.template.contentVersion in bicep just makes the whole process more robust, as the ARM fabric ensures the tag value matches the version of the template being used.
Having this supported in a future release will be much appreciated.

Piling on here as we also use contentVersion in our JSON ARM templates to correlate the source versions with the deployed resource by adding a tag to the resource. Same as the others above, we use the same methods of obtaining the version in order to apply the tag during the deployment.

I can try this one


examples: Move examples to their own self-contained repos

Similar to the helia examples repo each example is automated into it's own self-contained repo to give users the ease of getting started by simply forking and then building.

resolved with https://github.com/libp2p/js-libp2p/pull/2171


[CLD-7064] Mattermost Operator + Helm Documentation Updates

Summary
Adjusts documentation for deployment of Mattermost through the Kubernetes Operator. Brings Helm forward as the primary method of installation for the Kubernetes Operator as that moves into v1.0. I went through the documentation and added information where it may have previously been assumed, as well as consolidated information from other sources (README's, etc). Worked on aligning the various YAML snippets so that entity names match up, allowing someone to get Mattermost going almost entirely from copy+pasting snippets rather than having to manually line up things like secret names.
Ticket Link
CLD-7064

@nickmisasi - Should this PR be merged as part of a specific Mattermost release?

@cwarnermm nothing holding this one up - good to go right away.


Debugging a docker container on a remote machine

Environment data
dotnet --info output:
Host (useful for support):
  Version: 2.1.0-preview2-26406-04
  Commit:  6833f3026b

.NET Core SDKs installed:
  No SDKs were found.

.NET Core runtimes installed:
  Microsoft.NETCore.App 2.1.0-preview2-26406-04 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

VS Code version:
Version 1.22.2
Commit 3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9
Date 2018-04-12T16:34:44.222Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
C# Extension version: 1.14.0
I have an azure iot edge deployment on a remote linux-arm machine and am trying to work out the steps to debug the container modules with visual studio code. I modified my launch.json to ssh into my remote machine and then execute docker exec on the container that I want to debug. Here's an example of my modified launch.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Debug IoT Edge Module SampleModule (.NET Core)",
      "type": "coreclr",
      "request": "attach",
      "processId": "${command:pickRemoteProcess}",
      "pipeTransport": {
        "pipeProgram": "ssh",
        "pipeArgs": [
          "-T",
<EMAIL_ADDRESS>          "docker",
          "exec",
          "-i",
          "SampleModule",
          "sh",
          "-c"
        ],
        "debuggerPath": "/root/vsdbg/vsdbg",
        "pipeCwd": "${workspaceFolder}",
        "quoteArgs": true
      },
      "sourceFileMap": {
        "/app": "${workspaceFolder}/modules/SampleModule"
      },
      "justMyCode": true
    },
    {
      "name": "Debug IoT Edge Module SampleFunction (.NET Core)",
      "type": "coreclr",
      "request": "attach",
      "processId": "${command:pickRemoteProcess}",
      "pipeTransport": {
        "pipeProgram": "docker",
        "pipeArgs": [
          "exec",
          "-i",
          "SampleFunction",
          "sh",
          "-c"
        ],
        "debuggerPath": "~/vsdbg/vsdbg",
        "pipeCwd": "${workspaceFolder}",
        "quoteArgs": true
      },
      "sourceFileMap": {
        "/app": "${workspaceFolder}/modules/SampleFunction"
      },
      "justMyCode": true
    }
  ]
}

For now I'm only trying to debug SampleModule so ignore the SampleFunction section.
When I start the debug this works initially as VS code brings up the UI to choose what process to attach to, and I can select my running module. Once I select to attach to my module then the debug ends with the following output:
Starting: "ssh" -T<EMAIL_ADDRESS>docker exec -i SampleModule sh -c "/root/vsdbg/vsdbg --interpreter=vscode"
=message,text="--------------------------------------------------------------------------------\nYou may only use the Microsoft .NET Core Debugger (vsdbg) with Visual Studio\nCode, Visual Studio or Visual Studio for Mac software to help you develop and\ntest your applications.\n--------------------------------------------------------------------------------\n",send-to="output-window"
(gdb)
^error,msg="Undefined MI command:

I also get a pop-up error from VS code with the following message:
Invalid line ending character in protocol message. Ensure that the pipe program 'ssh' has pseudo-terminal (pty) disabled.
What should I do to properly configure my setup for debugging a container on a remote system.

@jsrc27 What is happening is that --interpreter=vscode is not being passed as an argument to vsdbg. You could try setting quoteArgs to false.

You might also try:
      "pipeTransport": {
        "pipeProgram": "ssh",
        "pipeArgs": [
          "-T",
<EMAIL_ADDRESS>          "docker exec -i SampleModule sh ${debuggerCommand}"
        ],
        "debuggerPath": "/root/vsdbg/vsdbg",
        "pipeCwd": "${workspaceFolder}",
        "quoteArgs": true
      },

@gregg-miskelly
Thank you so much that seems to have worked!

@jsrc27 Great! I will close this for now then. Feel free to reopen if you discover some issue with it.


Add async-timeout

Latest aiohttp now only includes async-timeout if running on Python 3.10 or lower.
Include async-timeout explicitly for now until we bump minimum Python version. Python 3.11 has asyncio.timeout.

Hm, I see that is why #435 failed. Should we maybe pin aiohttp? :thinking:

For libraries, which this is, we normally pin all requirements in development, but not in production. This library currently doesn't pin the production requirements in development.


Fixed grammar

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Thank you for finding and fixing that!


Doesn't work with Deferred render mode

Doesn't work with Deferred render mode.
Would it be possible to fix this?
Thank you very much for your great work.

I may be wrong. Sometimes it stops working even with Forward rendering. I'll investigate.


Deprecate styling

Prelude
Styling support in NvDialog was added in v0.5.0 for better integration with Gtk3 and for GNU/Linux developers to use their app's theme (In request of https://stopthemingmy.app). This worked, but it was actually a hassle for cross-platform support. Imagine this code:
NvdCSSManager* mgr = nvd_css_manager_new();
if (nvd_css_manager_attach_string_stylesheet(mgr, "css code here") != 0)
    exit(EXIT_FAILURE);
else nvd_css_manager_use_style(*mgr, NULL);

This:

May work on macOS X, depending on the backend.
Will 99% work on GNU/Linux
Will definitely not work on Windows.

You see, Cocoa, the WinAPI and Gtk have nothing in common when it comes to styling, meaning NvDialog never bothered to work for those platforms. In addition, the reason NvDialog is called NvDialog is just a corruption of the words "Native Dialogs". In other words, CSS styling shouldn't be here in the first place, and removing it seems natural.
This pull request will be a two stage merge, the first one to deprecate code and add compiler warnings, and the second one to completely remove the code, targeted for the v0.8.0 release.
TODOs

[x] Mark related functions as deprecated
[x] Remove the implementations directly and replace them with error returns (NULL, -1, ...)
[x] Update documentation

Stage 2

[ ] Remove implementations entirely (Only allow them to be available during linking, but no contents within the functions).
[ ] Remove declarations from header files.
[ ] Remove functions entirely (v0.8.0)

Merging in preparation of v0.7.0. The rest of the PR will be done in single commits.


metal: use a texture cache for onscreen msaa

reproduction steps
see https://github.com/flutter/flutter/issues/148496
observed results
(notice the purple exclamations in the workload to the left)

expected results
I would expect these textures to be reused across frames, and/or reused for the one frame if that's possible with the correct synchronization.
notes
Jonah said that we do this sort of caching for save layers (https://github.com/flutter/engine/blob/main/impeller/entity/render_target_cache.h).

@jonahwilliams @chinmaygarde it looks like texture allocation is taking 5% of the raster thread

This is unrelated to onscreen MSAA though. You should file a new bug or use https://github.com/flutter/flutter/issues/156503 for render target stability.

onscreen is a single transient attachment, whereas the saveLayer stability is causing re-allocation of a few dozen textures a frame. Xcode warnings aside, if you care about CPU usage the latter is a bigger problem, and has a completely different solution.


Change operator capabilities

 IBM Flash operator is not setting all the flags: https://catalog.redhat.com/software/operators/detail/62eb70efe35cf4509a72b8bf Seamless upgrades and Full lifecycle should be marked. It should also mark as supporting disconnected deployments.


deposit: DynamicItemWidget remove button fix

Do not show the remove button on the first item of a
DynamicItemWidget.

Signed-off-by: Javier Martin Montull<EMAIL_ADDRESS>

Can you have a look @lnielsen ?

In the airport, so didn't have much time to look, but I would think it makes sense to have a remove button even for the first line?

Well it looks like a bug if there is only one element:

But I see your point, if there are several elements in the list, then it can make sense to remove the first element...

Closing this as for the moment I am not sure what the best approach is.


Set up issue template

I'm tempted to convert all issues to discussions to remove the confusion between Issue, PR, and Discussion. However, I'm not sure that is a good idea.
Another approach is to create an issue template which makes it more clear what is required before creating an issue:

PR that reproduces the issue using this repository's example site
a failing test case
a sample repository and steps to reproduce the issue

I don't want to prevent people from raising up issues they find, but it can be hard to replicate bugs sometimes because there are so many variables at play, i.e. Django settings, different browsers, different versions of Python, etc.

I could pick this up whilst I wait for the review on the other PR 🤷🏽♀.   Check this out
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository

Sounds good to me! 👍


[federation]resolve_reference not working with strawberry.federation.field(external=True)

Describe the Bug

Apollo federation entities may have an issue with using both resolve_reference and strawberry.federation.field(external=True) together. If both are present, strawberry may raise an error graphql.error.graphql_error.GraphQLError: Unable to resolve reference for <class ...>.
@strawberry.federation.type(keys=["upc"], shareable=True)
class Product:
    upc: str
    # #################################
    # # resolve_reference don't work with @external together???

    # inStock: bool

    # @classmethod
    # def resolve_reference(cls, info: Info, upc: str):
    #     p = info.context.repo.find(upc)
    #     if p:
    #         return Product(
    #             upc=p.upc,
    #             inStock=p.inStock,
    #         )

    #################################

    weight: int = strawberry.federation.field(external=True, default=0)
    price: int = strawberry.federation.field(external=True, default=0)

    @strawberry.federation.field(requires=["weight price"])
    def shippingEstimate(self) -> Optional[int]:
        if self.price > 1000:
            return 0
        return int(self.weight * 0.5)

    @strawberry.field
    def in_stock(self, info: Info) -> bool:
        p = info.context.repo.find(self.upc)
        return p.inStock if p else False

https://github.com/filwaline/Apollo-Federation-Mutation-Demo/blob/main/services/inventory/inventory/stubs.py
System Information

Strawberry version (if applicable):  0.151.2

Additional Context

A simple setup for reproducing this issue can be found in the repository filwaline/Apollo-Federation-Mutation-Demo. By uncommenting these lines and starting the server, you should be able to reproduce the error.
query ExampleQuery($upcs: [String!]!) {
  findProducts(upcs: $upcs) {
    upc
    weight
    price
    shippingEstimate
  }
}

http://localhost:4000/?explorerURLState=N4IgJg9gxgrgtgUwHYBcQC4QEcYIE4CeABAKIAeAhnAA4A2CAiroQBQAkM1UAzukQNoBlFHgCWSAOYBCALpSAlEWAAdJESIAzcWAAKeCGBhQU3Fpx58OXbopVr1Rc6odEA7glESAFimcPqYlAIfurcXqLU1OISJNwoonAUKMH2AL6q6UggADQgAG4UYhQARvTcGCB26sog5tw1fDUAjDUZIKlAA

@filwaline when using external, the external fields are passed to resolve reference:
    @classmethod
    def resolve_reference(cls, info: Info, upc: str, **kwargs):
        print(kwargs)
        p = info.context.repo.find(upc)
        if p:
            return Product(
                upc=p.upc,
                inStock=p.inStock,
            )

I tried with your repo and works, I'll close the issue, let me know if you still have issues 😊

@classmethod
    def resolve_reference(cls, info: Info, upc: str, **kwargs):
        p = info.context.repo.find(upc)
        if p:
            return Product(upc=p.upc, inStock=p.inStock, **kwargs)

Your advice worked, thank you! If you could document this, it would be even better!

@filwaline I'll try in the weekend :) feel free to send a PR too if you want 😊

@patrick91 I have discovered that when using a custom resolve_reference function, the parameters are not automatically validated. Is it my responsibility to ensure that the types of the parameters are correct?

@filwaline validate as validating the types of the data coming from the gateway/router?

@patrick91  Yes, for example, a datetime object may be automatically converted to an ISO-formatted string by the router, but the receiving side may not convert it back to a datetime object.

@filwaline I think that's a missing feature on our side then! can you open a new issue? 😊

@patrick91 #2478


[Feature Request]: Implementing dijkastra Algorithm using Set

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Adding code for Dijkastra using set
Use Case
Learners will be able to learn different way of Implementing dijkastra Algorithm
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Ajay-Dhangar sir kindly assign me this task under GSSOC 2024

@Ajay-Dhangar  sir i have done it sir please check my PR


Enable dependabot for github-actions

dependabot automatically checks for dependency and create pull request. We are enabling this feature to automatically update github actions.

Approved


Demo Site is down

Is this demo page related with your project?
http://demo.telescopeapp.org
If the URL has changed, please open a pull request at
https://github.com/Kickball/awesome-selfhosted or create an issue. Thank you!
https://github.com/Kickball/awesome-selfhosted/issues/1767

See #10


fix: build race in @agnos-ui/svelte

This fixes stackblitz for svelte on certain conditions.

thx for the review @ExFlo !


Why can't I get a right transaction history in the test net?

My address: AR4iFxFdbjTkHKtStSKRvu5bgXXBcpZx9D
https://neoscan-testnet.io/api/test_net/v1/get_address_abstracts/AR4iFxFdbjTkHKtStSKRvu5bgXXBcpZx9D/1

Because testnet is resyncing. Always check the last block to see if the chain is fully synced
Get Outlook for Androidhttps://aka.ms/ghei36

From: AlleniCode<EMAIL_ADDRESS>Sent: Monday, April 23, 2018 7:52:34 AM
To: CityOfZion/neo-scan
Cc: Subscribed
Subject: [CityOfZion/neo-scan] Why can't I get a right transaction history in the test net? (#144)
My address: AR4iFxFdbjTkHKtStSKRvu5bgXXBcpZx9D
https://neoscan-testnet.io/api/test_net/v1/get_address_abstracts/AR4iFxFdbjTkHKtStSKRvu5bgXXBcpZx9D/1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/CityOfZion/neo-scan/issues/144, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA9Vh8Qxn1KsLVSSSeNyr5evtNUDwUUdks5trbJygaJpZM4Tfqo5.


cblas.h:30:61: error: unknown type name 'cpu_set_t'; did you mean 'sigset_t'?

I get this error while building a project which uses OpenBLAS. Any ideas?!

Does adding an#include <sys/resource.h help ?

cpu_set manual says #include <sched.h> and OpenBLAS source code does that already.

@mshadian can you be more specific about the build environment please ? The cpu_set_t parts are only used on Linux, where I believe they should be available in both glibc and uClibc

A possibility could be that you used the "raw" cblas.h from the build directory, rather than the processed copy that a make install would produce in the installation directory. Does the cblas.h that you used include "common.h" or openblas_config.h ?
The former is only used during the build of OpenBLAS itself, the latter has several build-time defines added, including the information that it is a Linux build (OPENBLAS_OS_LINUX) which triggers the automatic inclusion of sched.h


module "impacket.dcerpc.v5.nrpc" has no attribute "NetrServerPasswordSet2"

Hi!
The title shows every time I run the "set_empty_pw.py" script, but right after the 'error' your script says: "success! dc should now have the empty string as its machine password". But btw, it doesn't!
I've reinstalled impacket, with
"pip3 install impacket"
downloaded the impacket repo from source
"python3.6 setup.py install"
"python3.6 setup.py build"
"sudo apt install python-impacket"
I've tried to change all the "#!/usr/bin/env " to "!/usr/bin/python3.6"
and the script keep complaining on that "has no attribute". I wrote a python3.6 script using NetrServerPasswordSet2 and then it doesn't get error.

Hi There,
I`m facing the below issue please provide fix for this
Traceback (most recent call last):
File "set_empty_pw.py", line 3, in
from impacket.dcerpc.v5 import nrpc, epm
ModuleNotFoundError: No module named 'impacket.dcerpc'

Hi!
The title shows every time I run the "set_empty_pw.py" script, but right after the 'error' your script says: "success! dc should now have the empty string as its machine password". But btw, it doesn't!
I've reinstalled impacket, with
"pip3 install impacket"
downloaded the impacket repo from source
"python3.6 setup.py install"
"python3.6 setup.py build"
"sudo apt install python-impacket"
I've tried to change all the "#!/usr/bin/env " to "!/usr/bin/python3.6"
and the script keep complaining on that "has no attribute". I wrote a python3.6 script using NetrServerPasswordSet2 and then it doesn't get error.

You should use a virtual environment.
Make sure you’re using Python 3.6+
cd zerologon/
python3 -m venv venv
./venv/Scripts/activate

Ensure the env is active
python3 --version
pip install impacket just in-case venv doesn’t work, use --user
python3 setup.py install

Hi There,
I`m facing the below issue please provide fix for this
Traceback (most recent call last):
File "set_empty_pw.py", line 3, in
from impacket.dcerpc.v5 import nrpc, epm
ModuleNotFoundError: No module named 'impacket.dcerpc'

Follow the same instructions in my previous comment


Bugfix TraCI crash

See CHANGELOG.md

This should put down the vast majority of issues with SUMO.


Map

DON'T MERGE!!
Just a test PR for map

I believe the pan/scroll behavior is done and we just need to add the overlay.

Are mouse users able to rotate?


Update 2017-12-00-Using-Ebpf-In-Kubernetes.md

Signed-off-by: Gopi<EMAIL_ADDRESS>corrected  Architecture Image missing url.

Hi @gopihc, please note that SIG Docs policy is not to revise blog posts more than 1 year old.
I see that this is your first contribution here, thank you for your effort! I'd like to invite you to take a look at our set of "good first issues," I'm sure there is something there that could use your help.
/close


MUMMNG-3062 Send app lifecycle in layout feed

Updates our mock feeds to include lifecycleState

:+1:


Double // @flow comment added to generated file

Describe the bug

Hi, when I enable flow and flow-operations plugins at the same time, two // @flow headers are added to the graphql.js file.
To Reproduce
Steps to reproduce the behavior:
I tried to edit the template but it doesn't seem to work with the Flow plugins.
https://codesandbox.io/s/wizardly-paper-wsqgl?file=/document.graphql

My GraphQL schema:

Not relevant for the issue

My GraphQL operations:

Not relevant for the issue

My codegen.yml config file:

overwrite: true
schema: 'allSchema.graphql'
generates:
  ./src/generated/graphql.js:
    documents: 'queries/*.js'
    plugins:
      - flow
      - flow-operations

Expected behavior

Environment:

OS: macOS Catalina
@graphql-codegen/flow: same as codesandbox
@graphql-codegen/flow-operations: same as codesandbox
NodeJS: 14.5.0

Additional context

I think this was introduced in: https://github.com/dotansimha/graphql-code-generator/pull/4609
Does it's breaking anything @FezVrasta ? or it's just semantics?

It errors in my create-react-app project

@FezVrasta  build error or lint error? As far as I can tell, the Flow compiler works perfectly fine with that...

Having the the same issue. This is where flow throws:
| MultipleFlowAttributes ->
  [
    text "Unexpected ";
    code "@flow";
    text " declaration. Only one per ";
    text "file is allowed.";
  ]

What if we remove // @flow prepend from flow operations since flow plugin already has it?

Fixed in<EMAIL_ADDRESS>


Enable more tests for QEMU

If we replace URL from: https://github.com/Dasharo/open-source-firmware-validation/pull/318/files#diff-69e6355529b2609a8ddcccd04f8cd299d0fe65c19c246c853a31b9eb96fe6818R194 with the build to be tested, it should pass?

It should, although I haven't tested it, I always had coreboot.rom in place when running qemu-run.sh.

Secure Boot test resets the platform before settings are actually saved to flash in suite setup (causing first test to fail if platform hasn't SB in default state on entry) and teardown (which causes following test to fail). I'm working on fix/workaround.

@krystian-hebel Not sure if it's problem or not but I noticed that I can't set active PCR banks in BIOS when running QEMU from your branch. In TCG2 Configuration menu both TPM2 Active PCR Hash Algorithm and TPM Hardware Supported Hash Algorithm is [Unknown] and there is no option to choose active PCR banks.
I can set sha256 banks to be active by using tpm2_pcrallocate in Ubuntu and from what I can see after doing it once both SHA1 and SHA256 are active even after starting QEMU again (shouldn't all configuration reset to default between launches?).
In image below top column is QEMU from this branch and below is QEMU from develop

Related to: https://github.com/Dasharo/dasharo-issues/issues/521 ?

Related to: Dasharo/dasharo-issues#521 ?

@macpijan not exactly because I don't even have possibility to change active PCR bank. In issue you linked it was possible to change active PCR in BIOS but changes weren't applied.

Looks like with rebased EDK and DasharoPayloadPkg Tpm2GetCapabilityPcrs() succeeds in PEI but fails in DXE with "Unsupported" error, but works fine with OvmfPkg.  Maybe coreboot leaves TPM is some weird state.

OVMF used LAPIC for a timer, DasharoPkg uses HPET. This makes booting much longer and may also cause timeouts, because the only source of interrupts that is enabled at some points is the RTC, once a second. Using OvmfPkg/LocalApicTimerDxe/LocalApicTimerDxe.inf instead of HPET makes it boot faster, but it also makes the delays shorter, up to a point where tests can't traverse iPXE menu before autoboot, which should normally take 3 seconds. It may be controlled by modifying gEfiMdePkgTokenSpaceGuid.PcdFSBClock (MdePkg defines it as 200M, while most of OVMF platforms use 1G), however I haven't tested how it would work on real hardware.

after starting QEMU again

@m-iwanicki how do you start it, using qemu-run.sh or manually? Can you check if TPM settings are restored after removing /tmp/osfv/tpm?

how do you start it, using qemu-run.sh or manually? Can you check if TPM settings are restored after removing /tmp/osfv/tpm?

@krystian-hebel By using qemu-run.sh. Yes, after deleting /tmp/osfv/tpm settings go back to default (all PCR banks enabled).
Just checked and the same thing happens on develop branch, I just assumed that state should return to same default state each time QEMU is started.

@krystian-hebel I've added https://github.com/Dasharo/edk2/pull/129/commits/b5eca829af8a64b9bbe739c59683b66b6b89129b that resolves the issue pointed out by @m-iwanicki above.


Difference between NLIST and mapdl.mesh.nodes

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment
[X] I have a fully updated virtual environment (i.e. pip install --upgrade --upgrade-strategy eager ansys-mapdl-core)

Description of the bug
Basically, in beam elements (and I presume any kind of element which has hidden nodes), the mapdl.mesh.nodes returns those hidden nodes.
This is not desirable.
Steps To Reproduce
from ansys.mapdl.core import launch_mapdl
mapdl = launch_mapdl()

mapdl.mute = True
mapdl.finish()
mapdl.clear()
mapdl.input(examples.verif_files.vmfiles["vm10"])

mapdl.post1()
mapdl.set(1, 2)
mapdl.mute = False

text = "\n".join(mapdl.nlist().splitlines()[5:])
nlist = np.array([np.fromstring(each, sep=" ") for each in text.splitlines()])

assert np.allclose(nlist[:, 1:4], mapdl.mesh.nodes, rtol=1E-3)

Traceback (most recent call last):
  File "C:\ProgramData\Miniconda3\envs\pymapdl_0\lib\site-packages\numpy\core\numeric.py", line 2375, in isclose
    return within_tol(x, y, atol, rtol)
  File "C:\ProgramData\Miniconda3\envs\pymapdl_0\lib\site-packages\numpy\core\numeric.py", line 2356, in within_tol
    return less_equal(abs(x-y), atol + rtol * abs(y))
ValueError: operands could not be broadcast together with shapes (3,3) (5,3)

So, it seems to me that the NLIST command does not retrieve the midside nodes, whereas mapdl.mesh.nodes (which use the grpc method) does it.
For more info, mapdl.mesh.nodes returns:
>>> mapdl.mesh.nodes
array([[  0.        ,   0.        ,   0.        ],
       [100.        ,   0.        ,   0.        ],
       [  0.        ,   1.        ,   0.        ],
       [ 33.33333333,   0.        ,   0.        ],
       [ 66.66666667,   0.        ,   0.        ]])

The extra nodes, are indeed, in the middle elements.
The FE element is 188 (a 2D beam element), hence these middle nodes are supposed to be hidden. Hence, they should not be retrieved (at least by default) by mapdl.mesh.nodes.
@FredAns do you think you can implement an optional argument to the gRPC method that allows us to choose if we want midside nodes or not?
Pinging @akaszynski for visibility.
Which Operating System are you using?
Windows
Which Python version are you using?
3.9
PyMAPDL Report
NA
Installed packages
NA

Closing issue, we can obtain same result using:
nlist = mapdl.nlist(kinternal="internal")
assert np.allclose(nlist.to_array()[:, 1:4], mapdl.mesh.nodes)


workbench.action.navigateUp  should navigate to the last focused view, not the rightmost one

Steps to Reproduce:

Open two split editors
Navigate from the left editor to the terminal using workbench.action.navigateDown, and then navigate from the terminal using workbench.action.navigateUp

Does this issue occur when all extensions are disabled?: Yes
Right now, when using workbench.action.navigateUp, the rightmost view is focused. This is annoying if you have two split editors and you quickly navigate between the left editor and the terminal, because when using workbench.action.navigateUp from the terminal, the right editor is focused, instead of the left one.
VS Code version: Code 1.49.1 (58bb7b2331731bf72587010e943852e13e6fd3cf, 2020-09-16T23:23:33.049Z)
OS version: Linux x64 5.8.10
Remote OS version: Linux x64 4.15

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 3917)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: unavailable_offvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
31.03GB (1.45GB free)

Process Argv
--no-sandbox --unity-launch --crash-reporter-id 39f0e56d-047c-4994-990f-e6e79755483c

Screen Reader
no

VM
0%

DESKTOP_SESSION
custom

XDG_CURRENT_DESKTOP
undefined

XDG_SESSION_DESKTOP
custom

XDG_SESSION_TYPE
x11

Extensions (10)

Extension
Author (truncated)
Version

vscode-neovim
asv
0.0.62

remote-ssh
ms-
0.55.0

remote-ssh-edit
ms-
0.55.0

vscode-todo-highlight
way
1.0.4

gitlens
eam
10.2.2

error-gutters
Igo
1.0.1

statusbarerror
Joe
0.4.0

python
ms-
2020.8.109390

vscode-pylance
ms-
2020.9.5

vscodeintellicode
Vis
1.2.10

(1 theme extensions excluded)

Is there a way to implement this in an extension?

@bpasero why is this a feature request and not a bug?
This behavior is what users expect and get in all programs I'm aware of with window splits.
Also, can someone please respond on my question above?


Fix docs quickstart

codingame.Client() doesn't have param remember_me_cookie
Update example code to pass cookie to login()

Codecov Report

Merging #17 (756cd36) into master (5dd6f7b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   99.46%   99.46%
=======================================
  Files          28       28
  Lines        1307     1307
  Branches       64      119   +55
=======================================
  Hits         1300     1300
  Misses          5        5
  Partials        2        2

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 5dd6f7b...756cd36. Read the comment docs.


Controller disconnects after closing the "Change Grip/Order" menu

I tried to get nxbt working on Manjaro Linux using sudo pip install nxbt. Install runs successful. However, when I run the demo and go to the "Change Grip/Order" menu, the controller instantly disconnects after first connecting.
The tui connects, but also disconnects after closing the menu. The webapp does not connect at all.
Any ideas?
Here is the error code:
[2020-12-22 15:32:18,026] ERROR in server: [Errno 107] Transport endpoint is not connected
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/nxbt/controller/server.py", line 145, in mainloop
    itr.sendall(msg)
OSError: [Errno 107] Transport endpoint is not connected

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/nxbt/controller/server.py", line 189, in save_connection
    itr, ctrl = self.reconnect(self.switch_address)
  File "/usr/lib/python3.8/site-packages/nxbt/controller/server.py", line 357, in reconnect
    test_ctrl.connect((reconnect_address, 17))
OSError: [Errno 113] No route to host

edit: After some more fiddling, I got to connect the controller in demo and tui-mode, IF another controller is connected as first controller. Delay is insanely long, though.

Hi @jollaWellbuur and thanks for your interest in nxbt. This is a rather interesting issue.
What your error log is telling me is that the connection process was successful, interrupt/control sockets were obtained, and packets were exchanged. The connection seems to be getting dropped when the input frequency (15Hz on the Change Grip/Order Menu) is increased to the full 120Hz. Unfortunately, I'm not able to reproduce the error/behaviour you've mentioned from some initial testing on my own Manjaro 20.1 installation, however, there might be something I've missed.
To get the ball rolling on solving this issue for you, I've got a few questions:

Which version of Manjaro are you using? You should be able to get this info by running cat /etc/lsb-release.
Are you using an external Bluetooth dongle for Bluetooth capability? If so, how old is this dongle?
Do you use any Bluetooth peripherals with the PC you're using nxbt on? Eg: Bluetooth Headphones, Keyboards, Mice, etc.
In your edit, you mention some delay is occurring? Is this in the order of milliseconds or seconds? Does the delay increase with time?
Does nxbt's reconnect functionality work without another controller connected? Eg: With your Switch on the home menu, are you able to run sudo nxbt tui -r and a successful connection is made?

Hi, thanks for your reply. To answer your questions:

Manjaro Version:

DISTRIB_ID=ManjaroLinux
DISTRIB_RELEASE=20.2
DISTRIB_CODENAME=Nibia
DISTRIB_DESCRIPTION="Manjaro Linux"

Internal Bluetooth (ThinkPad from 2017)
I do not use other peripherals paralel to nxbt, but have used bluetooth peripherals in the past without issue.
The delay is about one second. It does not appear to increase with time.
Reconnection is not possible.

After some more testing I was able to establish a connection using switchremoteserver. The problems are the same, however; I am disconnected after leaving the grip order menu.

Thanks for answering my questions. Your point regarding Switch Remote Server is specifically intriguing since that utility uses a different controller emulation backend than nxbt. Since you're encountering a similar issue while using both utilities, I think we might be dealing with a problem slightly lower in the stack or possibly something hardware related.
Since your machine seems happy during the connection phase of emulation, I've created a new branch on the repo that maintains the connection phase's 15Hz frequency throughout the entire emulation process. If possible, would you be able to test this branch? I'll outline some instructions below on how to clone/install a working copy of the branch.

Clone the repo into a location of your choice: git clone https://github.com/Brikwerk/nxbt
Uninstall the pypi version of nxbt: sudo pip3 uninstall nxbt
Navigate inside the cloned repo and checkout the new branch: git checkout frequency-test
Install the local variant of nxbt: sudo pip3 install -e .
Try to connect to your Switch again on the Change Grip/Order Menu: sudo nxbt tui or sudo nxbt demo (keep in mind the demo may not work perfectly since the refresh rate is quite a bit lower with this branch).
To uninstall the local version of nxbt, run sudo rm -rf nxbt.egg-info at the root of the cloned repo

Let me know if this helps with the dropped connection after navigating off of the Change Grip/Order Menu.

Closing issue due to lack of activity.


OAuth2 plugin: not dealing with external IDP

The current OAuth2 plugin doesn't support the use of an external Identity Provider such as Keycloak: http://www.keycloak.org
This would be useful specially for:

Client credentials flow
Implicit grant flow

+1

This would be great for us too, we are using DEX.

An OpenID Connect Relying Party plugin would make it possible to delegate the authentication to external Identity Management. Such a plugin would make it possible to use kong in enterprise environments where an IAM Service already exists and user duplication is not allowed or possible.
Some efforts where made towards a solution. see #2154 or https://github.com/mogui/kong-external-oauth
There is also an OpenResty module that would do the heavy lifting.
https://github.com/pingidentity/lua-resty-openidc
Implementing such a plugin would also reduce the need for the kong team for implementing further auth provider because with OID Connect one can always delegate the complex special cases to dedicated IAMs such auch keycloak, gluu, anvil.io, Azure AD, OpenIAM and many more.

@lg1712, @PLBrick, @rsoletob,
From open source Kong you have currently one or two options:

Kong JWT plugin (https://getkong.org/plugins/jwt/)
Nokia's Kong OIDC Plugin (https://github.com/nokia/kong-oidc)

With JWT plugin you can verify 3rd party issued JWT tokens such as those from Keycloak or Dex (e.g. tokens issued with implicit flow). But you have to configure the public keys on Kong side and keep them in sync with your identity provider as the plugin doesn't support OpenID Connect Discovery.
With Kong Enterprise you have more options:

OpenID Connect plugins (which also work with plain OAuth 2.0)
OAuth 2.0 Introspection plugin

Kong Enterprise comes with 4 OpenID Connect plugins:

OpenID Connect (all-in-one) plugin
OpenID Connect Authentication plugin (applied to global endpoint to initiate the authorization code flow and set up a session between Kong and the client)
OpenID Connect Protection plugin (applied to invidual apis to protect access to those authenticated with the OpenID Connect Authentication plugin)
OpenID Connect Verification plugin (stateless JWT verification, both access and id tokens)

The first one is rather interesting as it supports many authentication methods in one single plugin:

Password Grant
Client Credentials Grant
Authorization Code Flow
JWT Bearer Tokens with signature and claims verification
Opaque Bearer Tokens by introspecting them
Kong OAuth 2.0 Authentication plugin issued tokens

also:

Session cookie based authentication (this plugin sets up a HTTP only session cookie between Kong and client, after the client has authenticated with one of the ways mentioned above).
Refreshing access tokens automatically by using refresh token (if available)

The plugins have been tested with many 3rd party providers:

Auth0
Okta
Google
Microsoft Live Connect
Microsoft Azure AD (both 1.0 and 2.0 versions)
Yahoo!
Salesforce
Paypal
Connect2id
PingFederate
IdentityServer4
OpenAM
Gluu
Keycloak
Dex

It supports auto-configuration by using OpenID Connect discovery, and automatically reloads the public keys as well for stateless JWT (actually JWS) signature verification with several algorithms:

HS256
HS384
HS512
RS256
RS512
ES256
ES384
ES512
PS256
PS384
PS512

Learn more about Kong Priority Support, Products, HA, Demo, Training, API Certifications and Professional Services.
As this will in my opinion answer the question, I will close this.


[bug][demo app] Tag with long attributed text is not cropped

GIVEN I'm using UI kit Tag on demo app on an iPhone
WHEN I select a content with "long attributed text"
THEN tag doesn't fit the size of the iPhone but goes beyond and is not cropped as it should
note: this bug is only related to the demo app, it should not happen on Polaris or other clients

More Information:
The problem is on this file: ComponentUIView
let widthConstraint = self.contentStackView.widthAnchor.constraint(
    equalTo: self.widthAnchor,
    constant: -spacing * 2
)
widthConstraint.priority = .init(999)
widthConstraint.isActive = true

This code must be replaced by:
NSLayoutConstraint.activate([
    self.contentStackView.widthAnchor.constraint(equalTo: self.widthAnchor, constant: -spacing * 2)
])


No support for multi word strings

When I give avvvars a fist and a last name, it only takes the first 2 letters. Suggestion, take the first letter of the first word and the laat word and use those of there are multiple words

That's a good idea, I've added into my to-do list, it's going to be on v2 (Which I'm working on atm)


No updates via Update Manager

I noticed that your plugin doesn't have any reference back to this repo in OctoPrint's Update Manager, so you don't get any updates unless you install them manually. I'm guessing that's fixed easily with some additional info in setup.py, but i can't offer a solution right away ;-)

this is because the release names start with 'v'. Had the same problem with telegram plugin some time ago.

Thanks folks for initiating/commenting on this issue. And sorry for the delay as I was just back from a long vacation.
@derpicknicker1 I tested release names (tags) without the starting 'v' and somehow they still didn't show up in OctoPrint's "Settings" -> "Software Update". Please see attached screenshot. Will appreciate it if you can shed some light on what I'm still missing.

@jangrewe I just tested it and it seems to work - thanks to the tip from @derpicknicker1 . However I'm not 100% confident so I'm not closing this issue now. Can you test it to see if it works for you and let me know?

@kennethjiang all my OctoPrint instances updated themselves to 0.8.1, looks like you fixed it! =)

Thank you @derpicknicker1 . I dropped 'v' from my tag and started using pure numeric version numbers a while ago. However when I pushed a new version it almost never showed up immediately in my other OctoPrint installations. How long do you have to wait for the new versions to show up?

This is because of Octoprint is caching the update informations.
go to Settings -> Software Updates select Advanced Options at the bottom of the dialog and force the refresh of update data.
I don't know how often Octoprint refreshes the update data. Maybe every 24h?

cool I will try "force refresh" when I have next version. Thanks @derpicknicker1


AttributeError: module 'scarches.models' has no attribute 'sagenet'

Dear authors,
I followed the sagenet tutorial but got an error when trying to define the sagenet model object:
>>> sg_obj = sca.models.sagenet(device=device)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'scarches.models' has no attribute 'sagenet'
>>> dir(sca.models)
['Adaptor', 'EXPIMAP', 'SCANVI', 'SCVI', 'TOTALVI', 'TRVAE', '__builtins__', '__cached__', '__doc__', '__file__', '__loader__', '__name__', '__package__', '__path__', '__spec__', '__warningregistry__', 'attach_adaptors', 'base', 'expiMap', 'expimap', 'scgen', 'trVAE', 'trvae', 'vaeArith', 'warnings']

I have scArches version 0.5.4, which was git cloned from this repository.
Could you please take a look at this? Thanks!

Do you use master branch of this repo? Do you have a warning when importing prompting you to install pytorch geometric?


Server side data processing infinite loop

Hi,
I'm currently using react-table version 6.5.3 and trying to move to server side data processing. I have added the prop manual and implemented onFecthData.
My problem is that the table pulls the data in an endless loop even if no filter, page, sort or anything else has been updated. After some investigation I found that in lifecycle.js fireFetchData is always triggered:
return this.setState(newResolvedState, () => {
        cb && cb()
        if (
          oldState.page !== newResolvedState.page ||
          oldState.pageSize !== newResolvedState.pageSize ||
          oldState.sorted !== newResolvedState.sorted ||
          oldState.filtered !== newResolvedState.filtered
        ) {
          this.fireFetchData()
        }
      })

It appears that oldState.sorted !== newResolvedState.sorted return true even when these objects have the same properties. Here are the dump in chrome console:
// oldState.sorted
[{id: "update_timestamp", desc: true}]
// newResolvedState.sorted
[{id: "update_timestamp", desc: true}]

A workaround could be to compare JSON.stringify() of both. Does it make sense or am I missing the point here ?
Thank you for your feedback.

This might come from my implementation.
I have a default configuration for the table (sort, filter, column width, column hidden ...) which is overriden by a user configuration object. To simplify the state manipulation I clone the resulting object and inject it back to the react-table at each render.
So this is a specific implementation issue but I think a shallow comparison here is not enough.

@Raphael67 I also faced similar situation, if I use redux store and make server call via action, pass the data to reducer and then access the data through mapStateToProps and feed it to the react table then react table makes infinite api calls in a loop.
If I maintain the state locally and make server call within component and update the state then it works fine.

@swp44744 do you mind to try if my fork fix your issue ? for convenience I pushed a branch with the compiled version. You should be able to use it in your project by adding this line in your package.json:
...
"dependencies": {
    "react-table": "https://github.com/Raphael67/react-table.git#build",
    ...
}

Be sure that the react-table module has been overwritten with this version.

@Raphael67 Sure I will try your fix and let you know!!

@Raphael67 Hey your fix worked just fine for me..!! thank you.. any plans to merge your code changes???

We're glad you were able to find a workaround for the time being.  The intended usage is to store sorted and filtered as stateful immutable objects. There have been previous issues about this, and is easily solved by not constructing these object on-the-fly in their corresponding props.
Simple fix for both sorted and filtered
constructor () {
  this.state = {
    sorted: [...]
  }
}
updateSorted (newSorted) {
  // if you want to updated sorted, use this.setState
  this.setState({
    sorted: newSorted
  })
}
render () {
  return <ReactTable
    sorted={this.state.sorted}
  />
}


volunteered to be a maintainer of kubevela

@sunny0826 is bootstrap contributors of kubevela, Initiators of the website https://kubevela.io
volunteer @sunny0826 as maintainer of kubevela，management docs and Installation.

+1  I'm glad to see @sunny0826  to be a maintainer to maintain our repo and contribute more!

+1, thank you for continuous contribution to vela!
We will also need to publish a blog post for adding new maintainer
/cc @hongchaodeng for review

+1. Excellent work on KubeVela official site. Thanks. Looking forward to your more contribution to the new Dashboard.

Hi @sunny0826 ,
I would fully support you to become a maintainer. But your contribution so far has not reached a certain level to have impact on any specific component. Comparatively, when @captainroy-hy was promoted maintainer, he has contributed a couple of features and bug fixes to impact the project.
I would like to make the process more transparent and fair to the whole community -- to become a maintainer requires commitment and contribution.
By all means I have seen your commitment to the project. I would highly encourage you to continue working on dashboard component. Once that's done your contribution will be demonstrated as well. At that time I would like to grant you ownership as well.
Thanks for your persistence and hard work!

closing as @sunny0826  has already be one of the reviewer.


check vim version before attempting vimshell install

@LISTERINE Just vim-shell doesn't work on vim 7.2?

The only error I received when running it in 7.2 was from vimshell. There may be others that don't work, but once vimshell was excluded vim started without forcing me to acknowledge anything.

@LISTERINE thanks for send this pull request ;)

+1

Thanks for letting me contribute. I really love vim bootstrap; it's a staple for me on a new system.

Welcome to the collaboration team of vim-bootstrap, you came to add value!


cant create a new cluster with '--jmx-port'?

create doesn't support this arg
so when i am using nodetool, it throws me
nodetool: Failed to connect to '<IP_ADDRESS>:7199' - ConnectException: 'Connection refused'

Can you list the full commands you are using, including the nodetool call?

$ ccm create test -v 2.2.8 -n 3 -s
$ ~/.ccm/repository/2.2.8/bin/nodetool status
objc[20899]: Class JavaLaunchHelper is implemented in both /Library/Java/JavaVirtualMachines/jdk1.8.0_77.jdk/Contents/Home/bin/java and /Library/Java/JavaVirtualMachines/jdk1.8.0_77.jdk/Contents/Home/jre/lib/libinstrument.dylib. One of the two will be used. Which one is undefined.
nodetool: Failed to connect to '<IP_ADDRESS>:7199' - ConnectException: 'Connection refused'.

It appears you didn't use --jmx_port in your create call? The issue you're having (at least in the above steps) is the default jmx port that nodetool points to is 7199, but the ccm nodes have 7100, 7200, and 7300 (in a three node cluster). If you use ccm node1 nodetool status, it will point to the correct port. If you use the normal nodetool binary, you'll have to specify the port manually.

@ptnapoleon
that's actually my original question:)
$ ccm create test -v 2.2.8 -n 3 -s --jmx-port=7199
Usage: ccm create [options] cluster_name
ccm: error: no such option: --jmx-port


requireDotNotation: fix non-latin bug by improving identifier support

The regex is not pretty. We should consider using a library instead,
eg https://github.com/joelarson4/CharFunk.
This fix revealed an incorrect test case: while is not a valid identifier
so it needs to be made a literal and should not cause an error.
Fixes #920

I'm a bit surprised, that we don't do this correctly already... So I think we should do this.
But I don't like to have the giant regexp in the code, as there is no way to check if it is correct and debugging would be a nightmare. We already have a dependency on regenerator, so it should be easy to do something like: https://gist.github.com/mathiasbynens/6334847

Completely agree that the regex is gross. I didn't see the existing dependency on regenerate... I'll give it a whirl.

I can't repro that build failure on node 0.10.40 or 4.1.2. I assume it's a flaky test that times out sometimes.

Yeah don't worry about the time out ones - either flaky or we have a perf issue - we should figure that out / increase the timeout.

Don't want to leave this PR hanging but I think someone else should review since I don't know much about it - although tests look good

Sorry, I meant to review this sooner :-/
The getDialect method can return 'es3', 'es5' and 'es6', but you are only handling 'es5' and 'es6', see here. So I think we should also have an generateIdentifierRegexES3 method, which is generating the appropriate ES3 Regexes.
Apart from that, everything looks good to me.

Nice work!


Why not schedule crawls and deploy to Scrapyd from local machines?

To schedule crawls, we presently require analysts to connect to the server (as the correct user) and run a curl command. We also require analysts to follow a multi-step process to update spiders.
However, I can already do:
curl<EMAIL_ADDRESS>-d project=kingfisher -d spider=test_fail
(We can provide instructions on how to create shell aliases, so that analysts don't need to find the password every time.)
Similarly, if I configure scrapy.cfg in the same way (in which case we'd probably remove it from version control), then I can run scrapyd-deploy from my own machine.
So:

Does anyone prefer having to login to the server, instead of the above?

If not, then we can also remove the local copy of the kingfisher-scrape repository from the server, after closing #295 and #294. I also like that this means there will be no reason for analysts to regularly login as the ocdskfs user.

Tagging @yolile @romifz @duncandewhurst @pindec @mrshll1001 for comment.

We take the files scrape stores on disk and try and archive them for reloading historical data later - this would be lost (or much more complicated) on local machines
Increased chances of errors communicating with process (tho in fairness, that one could be written as "there needs to be better error recovery between the two".)
The server has a much faster and higher capacity internet connection than most people's home internet
Also considering that running locally would involve API -> Local -> Server instead of API -> Server -> Server, so roughly doubling the size of data we send across the internet.
For any scraper that takes longer than a tea break, people may prefer to run it on a stable machine and internet connection. I turn my work laptop off each night, for instance.
On the server it's easier to make sure analysts use a consistent environment, it takes less set up time per analyst (especially for less-technical ones) and it's easier to debug someone else's problem for them.

It could be an option we offer, but don't think removing it from the server should be considered.

I think there's a misunderstanding. The issue description is about sending a remote request to Scrapy from a local machine, instead of logging into the server and sending a local request to Scrapy. The latter just seems like extra work. Can you respond to that proposal?

Oh I see, sorry. I think taking that in 2 parts: sending the request to Scrapy to start from the local machine is fine, but I would be wary of getting people to update scrapers direct from local machines as it may be harder to know for sure which version of the scrapers we have loaded at any one point. Also the point about the consistent environment, less technical users and debugging issues for people is I think still relevant.

I added a step for a user to ensure that they are about to deploy the latest spiders. I retained abbreviated instructions for how to do this from the server. https://ocdsdeploy.readthedocs.io/en/latest/use/kingfisher-collect.html#update-spiders-in-kingfisher-scrape
Only Romi, Yohanna, Andres, and you are expected to deploy scrapers (since no one else writes scrapers), whom I don't consider to be less technical users.
The only part of the environment that needs to be consistent is the scrapyd-client package, which hasn't seen a release in 3 years.


Mapbox studio orphan processes

somehow, somewhere, mapbox studio didn't exit properly:

cc @yhahn

@springmeyer One thing that seems to happen is that if you quit Mapbox Studio/Atom Shell from the dock that doesn't trigger the window-all-closed event for atom and the server + process aren't killed, but the shell is.


Code Analysis warning C26440 TIXML2_MAJOR_VERSION etc.

I admit I am not using the latest version of TinyXML2 but it is fairly recent.
In Visual Studio 2019 running Code Analysis says:

warning C26814: The const variable 'TIXML2_MAJOR_VERSION' can be computed at compile-time. Consider using constexpr (con.5).

It says similar for for others and has a barage of entries like:

warning C26440: Function 'tinyxml2::XMLNode::GetLineNum' can be declared 'noexcept' (f.6).

Please advise.

I've issued a PR to address the issue. Things tend to get merged about once a month around here.

Nevermind. The MacOS compiler they are using is so old it doesn't know what constexpr is. Expect this to never be fixed.

It will never be fixed, as frustrating as @cugone finds that, he is correct.


add testing mock

add mock for testing purpose

Hi @Goxiaoy
Why is mock implementation needed? doesn't Mockito cover this case?

Because when I try to do unittest within codes rely on this package，error comes with no implementation on this platform, which means on my computer

And it is learned from package shared_preferences

Would it work in your case?

import 'package:flutter_secure_storage/flutter_secure_storage.dart';
import 'package:mockito/mockito.dart';
import 'package:test/test.dart';

void main() {
  test('Sample mock', () async {
    final storage = MockStorage();
    when(storage.read(key: 'name')).thenAnswer((_) => Future.value('Stub'));

    final name = await storage.read(key: 'name');
    expect(name, 'Stub');
  });
}

class MockStorage extends Mock implements FlutterSecureStorage{}

It works for me. I am using dependency injection in my flutter project. so it's quite easy for me to replace FlutterSecureStorage to the mock class. But for those who did not, it's hard to replace it.
It's fine to close this PR. Thank you so much for your introduce of mockito.

Would it work in your case?
import 'package:flutter_secure_storage/flutter_secure_storage.dart';
import 'package:mockito/mockito.dart';
import 'package:test/test.dart';

void main() {
  test('Sample mock', () async {
    final storage = MockStorage();
    when(storage.read(key: 'name')).thenAnswer((_) => Future.value('Stub'));

    final name = await storage.read(key: 'name');
    expect(name, 'Stub');
  });
}

class MockStorage extends Mock implements FlutterSecureStorage{}

How to do that for write method?


enable setting server option from VS Code when used as extension by Markdown PDF extension

The example shows how to set options from javascript, but this is an awesome extension for VS Code using Markdown PDF extension.  It would be great to be able to set a setting for this extension so it could direct to a private PlantUML server.  I have to export to HTML, replace all URLs, and then print to PDF at the moment as a work around. In addition, the ability to strip the "/plantuml" from the url would be great.  This would allow easy use with plantuml docker images as shown in the following examples:
docker: docker run -d -p 8080:8080 plantuml/plantuml-server:tomcat
currently generated: "http://www.plantuml.com/plantuml/svg/---sensitive---"
desired generated: "http://localhost:8080/svg/---sensitive---"

I think this issue should be reported to the vscode-markdown-pdf project, because this is the plugin which uses markdown-it-plantuml and needs to support the configuration of the server option which is provided here.
Therefore I open a corresponding issue there see #139.

Closed as per the above.


Explicitly covert body to bytes

Although there is a default for body to be bytes, if
overridden then encode it

@jaitaiwan Requestinig review

No longer a maintainer.

@joestump requesting review


Add audio device selector to transcribe + take a stab at Delete/Retry models

Addresses #13 + adds audio device view in both sections.

New delete model button

Restart download of model if connection lost

Move audio input selection between the buttons on transcribe.

Nice idea with the delete and retry! This is probably a good time to handle canceling the previous download task in resetState() because it's a static method. As-is, if you are in the middle of a download and tap the restart button, the previous download will continue and a new one will start. It may require something like this:
        let downloadTask = Task {
            folder = try await WhisperKit.download(variant: model, from: repoName, progressCallback: { progress in
                DispatchQueue.main.async {
                    loadingProgressValue = Float(progress.fractionCompleted) * specializationProgressRatio
                    modelState = .downloading
                }

                // Check for cancellation here
                try Task.checkCancellation()
            })
        }

or something similar to how the progressbar task gets cancelled:

                let progressBarTask = Task {
                    await updateProgressBar(targetProgress: 0.9, maxTime: 240)
                }

                // Prewarm models
                do {
                    try await whisperKit.prewarmModels()
                    progressBarTask.cancel()
                } catch {
                    print("Error prewarming models, retrying: \(error.localizedDescription)")
                    progressBarTask.cancel()
                    if !redownload {
                        loadModel(model, redownload: true)
                        return
                    } else {
                        // Redownloading failed, error out
                        modelState = .unloaded
                        return
                    }
                }

What do you think? This would also solve a few edge cases with the model selection, like if you select a new model while the previous one is downloading.

Also played around with the mic selector placement, what do you think of this setup? Added a PR to your repo: https://github.com/cgfarmer4/WhisperKit/pull/1

Beautiful, merged your PR and will take a look at cancelling the download tonight.

I removed the retry, getting downloads to cancel is going to require a larger refactor of this function.
let downloadTask = Task {
            folder = try await WhisperKit.download(variant: model, from: repoName, progressCallback: { progress in
                DispatchQueue.main.async {
                    loadingProgressValue = Float(progress.fractionCompleted) * specializationProgressRatio
                    modelState = .downloading
                }

                // Check for cancellation here
                try Task.checkCancellation()
            })
        }

Going for this path, you get an error that Mutation of captured var 'folder' in concurrently-executing code.
Then I tried this which fixes the warning: Mutation of captured var 'folder' in concurrently-executing code; this is an error in Swift 6 but doesnt properly cancel the download.
DispatchQueue.main.async {
    folder = downloadFolder
}

Next approach I tried was assigning the main Task similar to how transcriptionTask is assigned but that requires a pretty another refactor around the do/catch statements.
How about I leave in the delete + your update and call it on this one? The download cancellation needs a rethink or Im totally missing something.

No problem! We can get to that later on, but this is a good addition as-is 👍


RSS workshop feed, link problem

Hello,
I subscribed the rss feed for the coderefinery workshops. There is a problem with the links to the event pages. Here is for example the id of the first item on the feed list:
<id>https://coderefinery.org/https:/coderefinery.github.io/2020-05-25-online/</id>
I think that the problem comes from the fact that for the new workshops, the permalink variable in the markdown file is set to the absolute address, while older workshops only use relative address. I am not a jekyll expert, but I guess that somewhere "https://coderefinery.org/" is added to the permalink variable to create the event.url variable (used in feed-workshops.xml for example).
If one explains me how this event.url works, I can try to fix this, or if someone fixes this, this is fine!
Best
Romain Caneill

And I now realized that by porting the page to Zola I have possibly broken the already broken RSS.
The thing is that Zola has good RSS support (https://www.getzola.org/documentation/templates/feeds/), but this would be for pages that have a date. But this is not anymore how we list new workshops in the current layout.
In other words, sorry for breaking it and we should maybe think how we can re-enable it.

And I now realized that by porting the page to Zola I have possibly broken the already broken RSS.
The thing is that Zola has good RSS support (https://www.getzola.org/documentation/templates/feeds/), but this would be for pages that have a date. But this is not anymore how we list new workshops in the current layout.
In other words, sorry for breaking it and we should maybe think how we can re-enable it.

Reported via support email:

It seems that the RSS feed of the workshops and events pages do not work anymore. Is it possible to make them work again?

https://coderefinery.org/feed-workshops/
https://coderefinery.org/feed-events/

Problem also reported by another person so we better reactivate it again.

Once #608 is in, RSS will be again available under coderefinery.org/atom.xml. Please let me know if this is unexpected or non-standard. We can change the filename. I have personally close to zero experience with RSS so I don't know what clients expect.

This new feed works now!
I have seen RSS feeds with the format like coderefinery.org/atom.xml. A minor issue for this new feed, though, is that one cannot directly subscribe to the RSS feed by clicking browser extensions like the following one:
https://chrome.google.com/webstore/detail/rss-subscription-extensio/nlbjncdgjeocebhnmkbbbdekmmmcbfjd
The RSS feed cannot be found by the Chrome extension when on the following page or on the home page:
https://coderefinery.org/workshops/upcoming/
On the other hand, for feeds like the following one, one can discover (and quickly subscribe) it by clicking the Chrome extension above.
https://github.com/coderefinery/git-intro/releases

https://github.com/coderefinery/coderefinery.org/pull/612 might fix
this, based on my previous understanding of stuff.

Reopening until we iron out the smaller issues.

@wenlzhang can you check whether discover-ability got better?

@bast Thanks for the update! It works like a charm now.

Super. Closing. Thanks to Richard for fixing the discoverability.


Autocompletion while typing in searchbar seems to be broken

Suggestions/autocompletion while typing in search bar does not work even if it is enabled in settings.
Running docker image bpatrik/pigallery2:nightly-alpine
Digest:sha256:24c6864f03ddc72d930c5b61174d443cfa897dbf28cefbf3af27751185f8f5fd

Hi,
I know about the issue. Its beacause of #58, please use v1.8.2 while it gets fixed.

With the latest commits, it should work.

Yes, works very well now. Well done!
I am not sure if I should raise a bug concerning a difference in behavior now.
If i click with mouse, or hit enter on keyboard on one of the autocomplete suggestion - I would prefer that the search is done directly, instead i have to click on the search icon or hit enter again. Is this expected behavior?
Also, another issue, if i start typing something and then press backspace and remove - there is some corrupted texts that is printed on same.

i checked search bars from major companies and indeed hitting enter or selecting auto complete item, runs the search.  So updated the behavior. Also fixed the placeholder overlap issue.

Thank you for the changes! I added a pull request https://github.com/bpatrik/pigallery2/pull/248 for search on click. I hope you are ok with it.


[BUG] Use new Discord invitation link

Description of the bug
In the issue templates configuration, there's a link to WebXDAO's Discord community.
However, the file's using the direct Discord link, which makes it look unclean and does not match the new, clean link being used.
Steps to contribute

Fork the repository here.
Create a new branch named new_discord_link.
Open the .github/ISSUE_TEMPLATE/config.yml file.
Edit the Discord invitation link to https://discord.webxdao.xyz.
Commit and push your changes.
Create a pull request to compare your changes.

If you need any help, let us know and we'll gladly help!
Please add screenshots (if applicable)
No response
Add any other context about the problem here
This is a good first issue. Please leave it open to first-time contributors and beginners. 🙂
Checklist

[X] I have read the Contributing Guidelines
[X] I have checked the existing issues
[ ] I am willing to work on this issue (optional)
[ ] I am a GSSoC'23 contributor

NOTE: This should be updated in the .github repository to reflect all the other repositories as well.

@Panquesito7 Sir Can I be assigned this issue under GSSoC'23
Thanks in advance.
Jeyavishnu @vishnu9865

@Panquesito7 Sir Can I be assigned this issue under GSSoC'23 Thanks in advance. Jeyavishnu @vishnu9865

Hey Vishnu 👋🏼  Sorry for the late reply, this issue is solved by the PR #429 . I am sorry for not replying on your comment related to assigning of the issue.


Support for Kaggle P100 GPU

First and foremost congratulation of a great R&D work here. Wonder if there is any plan to support Kaggle GPUs?
I suspect there will be a great deal of interest from the community there to test this.
Cheers
Dr.Patrick

Thanks for the interest!
We're currently prioritizing support for Turing (T4) & Volta (V100) architectures as mentioned in the roadmap.
We've been talking to the xformers team and they're also working on something similar (targeting fp32 instead of fp16).
There's a chance we might be able to borrow what they have to support P100 eventually.
https://github.com/facebookresearch/xformers/pull/267
https://github.com/facebookresearch/xformers/pull/281


Doesn't appear in menus

I've installed it twice and restarted Atom, closing all windows, but the package still does not appear in the menu and cannot be searched for nor run.

That seems to do with Atom itself possibly. But the actual plugin itself could not be installing correctly. Did you install via command line or via the Atom client?

I originally installed (globally) via the command line... but when it didn't appear in the menu, I re-installed it via Atom's package manager (where it appears as installed, and after several restarts "added time" to the Atom start, but still doesn't show up anywhere as an option to run/use).
  From: Jack Puttock<EMAIL_ADDRESS>
To: jas-chen/atom-live-server<EMAIL_ADDRESS>Cc: Gabrielle<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Wednesday, July 26, 2017 3:27 PM
Subject: Re: [jas-chen/atom-live-server] Doesn't appear in menus (#82)
That seems to do with Atom itself possibly. But the actual plugin itself could not be installing correctly. Did you install via command line or via the Atom client?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Compatibility50: Fix for high-bit characters.

char might be signed. rdar://problem/55130240

@swift-ci please test


Feature Request: Add "full entry" as well as the widget for just the url

Adding a new item forces an immediate check, which muddies its first check because you aren't able to create CSS rules or filters or anything.
I would like to either add an item with initial state of paused, or be able to add an item with the same detail as the edit screen.

or be able to add an item with the same detail as the edit screen.

I dont understand

The edit screen has more detail than adding something. When you add, you can only type in URL and tags.
Edit has filters, title, tag, request headers, etc. etc..

Maybe shift+click on the submit button could add it paused and redirect you to the edit page

Maybe shift+click on the submit button (or shift enter) could add it paused and redirect you to the edit page

And on mobile?

I'm using APIs, where the URL is the same (and cd.io complains).
It'd be neat, if next to the 'Add' button was 'Advanced', what opens up the 'Edit' screen. Submitting on the edit screen would Add the new thing, without duplicate URL check.


Add this to lambda capture list

The implicit capture of 'this' by default by-value capture is deprecated in C++20.  This revision is intended to address this change (and to eliminate numerous warnings when building code).

@ShmuelLevine unfortunately adding this leads to warnings for older standards (i.e. older than 20). @hkaiser's suggestion might help with some of them. If that's not an option we might need something like this to disable warnings with older standards. For reference the warnings are: clang and gcc.

@msimberg We already have HPX_CAPTURE_MOVE and HPX_CAPTURE_FORWARD. Why not add HPX_CAPTURE_THIS?

This looks good now. The APEX builder refers to an outdated commit.


Fix Conflict between HarmonyX and Harmony2 Implementations

MonoMod seems to fail at Generating IL in certain situations.
An example of this is with JustEmuTarkov.
Error Log for Reference
Possibly from an existing 0Harmony.dll being present and loaded before MelonLoader's Assembly Resolver can redirect it.
May relate to https://github.com/LavaGang/MelonLoader/issues/94

After further research and testing of the almost complete version of https://github.com/LavaGang/MelonLoader/issues/94 I can conclude that it was not an assembly resolving issue related to https://github.com/LavaGang/MelonLoader/issues/94.
The Conflict has to do between HarmonyX and Harmony2.
When MelonLoader and JET uses Harmony2 everything works fine.
When MelonLoader and JET uses HarmonyX it fails.
Possibly do to some differences in IL generation. More Research is needed.

Needs Retesting


Add Parcel support

Parcel is a builder with great DX and performance. And it has built-in CSS Modules support too.

Since Parcel have the same API as webpack for CSS Modules, maybe we need only fix the docs.
I will investigate it today.

waiting for support :)

@TryHardNinja Astroturf already have the support
https://github.com/4Catalyzer/astroturf/commit/d71686ce1dc2da5a6768befb7f1b4eeb8870ae62


Object-hash leaking?

Hello,
suspecting memory leaks, I ran the following test over object-hash:
const hash = require('object-hash');
console.log(process.memoryUsage());
while (true) {
  const pre = Date.now();
  for (let i = 0; i < 1000; i++) hash(data);
  const post = Date.now();
  console.log(process.memoryUsage(), "runtime: "+(post-pre)+"ms");
}

These were the results:
{ rss: 21213184, heapTotal: 10522624, heapUsed: 4672656 }
{ rss: 85860352, heapTotal: 70365184, heapUsed: 38203144 } 'runtime: 329ms'
{ rss: 118218752, heapTotal: 102469632, heapUsed: 79757256 } 'runtime: 325ms'
{ rss: 151285760, heapTotal: 135122944, heapUsed: 112984104 } 'runtime: 443ms'
{ rss: 178872320, heapTotal: 161873920, heapUsed: 126826120 } 'runtime: 513ms'
{ rss: 206196736, heapTotal: 189136896, heapUsed: 165380888 } 'runtime: 553ms'
{ rss: 244736000, heapTotal: 227192832, heapUsed: 189656008 } 'runtime: 664ms'
{ rss: 272257024, heapTotal: 254455808, heapUsed: 224160496 } 'runtime: 738ms'
{ rss: 315011072, heapTotal: 297373696, heapUsed: 263990928 } 'runtime: 812ms'
{ rss: 342827008, heapTotal: 325685248, heapUsed: 294446712 } 'runtime: 885ms'
{ rss: 370958336, heapTotal: 352948224, heapUsed: 315126088 } 'runtime: 991ms'
{ rss: 399020032, heapTotal: 381259776, heapUsed: 341713960 } 'runtime: 1107ms'
{ rss: 412856320, heapTotal: 395227136, heapUsed: 362916912 } 'runtime: 1422ms'
{ rss: 440881152, heapTotal: 422490112, heapUsed: 384875336 } 'runtime: 1213ms'
{ rss: 468836352, heapTotal: 450801664, heapUsed: 412514440 } 'runtime: 1328ms'
{ rss: 496558080, heapTotal: 478064640, heapUsed: 445826144 } 'runtime: 1412ms'
{ rss: 525045760, heapTotal: 506376192, heapUsed: 469118536 } 'runtime: 1499ms'
{ rss: 553041920, heapTotal: 533639168, heapUsed: 498784816 } 'runtime: 1632ms'
{ rss: 613945344, heapTotal: 594788352, heapUsed: 550526600 } 'runtime: 1754ms'
{ rss: 641900544, heapTotal: 623099904, heapUsed: 575188672 } 'runtime: 1862ms'
{ rss: 669708288, heapTotal: 650362880, heapUsed: 605594576 } 'runtime: 1967ms'
{ rss: 697405440, heapTotal: 677625856, heapUsed: 641817448 } 'runtime: 2096ms'
^C

Seems like the there are serious leak issues. Could you advise?
Do note that I've tried all versions 1.1.0 - 1.1.4 (inclusive).
Thanks.

I’ll try to look into this. Can you provide a full reproduction, ideally with your data object or an approximation to it?
I can confirm the increasing run times but not the leak itself.

Sure. data is in fact a gibberish object that I created:
const data = {
  data_arr: [
    {
      name: "wegierngheirh",
      desc: "wergiojwegho",
      phone:<PHONE_NUMBER>
    },
    {
      name: "Weg24g24",
      desc: "r0-u3-t23t",
      phone: 1111
    }
  ],
  globalValue: 124135,
  moreContent: {
    withSubStuff: "wet9h23t92-23t-23t0=gj243g0j"
  }
}

But it is really irrelevant: This problem will also be introduced even if you set data={}

But it is really irrelevant: This problem will also be introduced even if you set data={}

I am still not so sure about the memory leak – inserting a gc(); call after the loop and running the test script with --expose-gc seems to keep the resource usage pretty stable.
The timing thing seems to only occur on Node >= 6.3.0, so I’ll dig into that.

So, yeah, this comes from nodejs/node@2c7804ad9eedad7f5e996a06af4c59f8a4bd7103 but in a way that’s not really to blame on Node itself. I’ll have a patch up for object-hash in no time.

+<EMAIL_ADDRESS>It would be great if you could confirm that everything works as expected now. :)

Amazing! Thanks for this super-fast round-trip. I'll look into it ASAP.
Just to be sure -- the fix has no effect on the resulting hash codes, right?

Just to be sure -- the fix has no effect on the resulting hash codes, right?

Yes, exactly. :)

Double & triple tested, seems good. Can't thank you enough!

My pleasure! :smile:


Component template top comment makes component disappear

I tried your 'counter' example from the blog. Every time I click add/subtract buttons, the component disappears from the screen. I checked on the dev screen. Every time it keeps the component's comment and the rest of the component HTML disappears. I removed the components' top comment  and it started to work. Please investigate.

(.env) crysanthus@crysanthus-dev:~/nphapps/ratr$ pip freeze
beautifulsoup4==4.10.0
bidict==0.21.4
click==7.1.2
dnspython==2.1.0
email-validator==1.1.3
eventlet==0.33.0
Flask==2.0.2
Flask-Login==0.5.0
Flask-Meld==0.13.1
Flask-SocketIO==5.1.1
Flask-SQLAlchemy==2.5.1
Flask-WTF==1.0.0
gevent==21.12.0
gevent-websocket==0.10.1
greenlet==1.1.2
gunicorn==20.1.0
idna==3.3
itsdangerous==2.0.1
Jinja2==3.0.3
jinja2-simple-tags==0.3.1
MarkupSafe==2.0.1
orjson==3.6.5
pkg_resources==0.0.0
python-engineio==4.3.0
python-socketio==5.5.0
six==1.16.0
soupsieve==2.3.1
SQLAlchemy==1.4.28
Werkzeug==2.0.2
WTForms==3.0.0
zope.event==4.5.0
zope.interface==5.4.0

(.env) crysanthus@crysanthus-dev:~/nphapps/ratr$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 21.10
Release:	21.10
Codename:	impish
(.env) crysanthus@crysanthus-dev:~/nphapps/ratr$ uname -a
Linux crysanthus-dev 5.13.0-22-generic #22-Ubuntu SMP Fri Nov 5 13:21:36 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
(.env) crysanthus@crysanthus-dev:~/nphapps/ratr$

Thank you! This is a known bug #27. I'll update the article to remove the comment to help reduce the number of occurrences until the bug is fixed.


feat: Add sorting and filtering to Fuel Codes

Hook up filtering and sorting methods
Setup backend filtering lgic
Change SetFilter to use custom so it avoids ag-grid filtering
Don't convert filters to int so we can preserve float values

Looking good.
Is there no 'contains' for floating point filters? i found it a bit confusing that when i put in 37, the record with a CI of 37.21 didn't show up. Default is Equals which is pretty specific.
Sorting works great.

Currently there is no contains option for numbers. Lindsy wants a start with functionality so that if I enter 32, it would returns 32.1 but not 132. I will talk to Al tomorrow but I feel like we should make this its own ticket and apply it everywhere.

Looking good.
Is there no 'contains' for floating point filters? i found it a bit confusing that when i put in 37, the record with a CI of 37.21 didn't show up. Default is Equals which is pretty specific.
Sorting works great.

Currently there is no contains option for numbers. Lindsy wants a start with functionality so that if I enter 32, it would returns 32.1 but not 132. I will talk to Al tomorrow but I feel like we should make this its own ticket and apply it everywhere.

Created a new issue in TODO to add this new behaviour.


fix: check for empty amount strings in balance inc/dec

Background
Apparently it's valid for some kinds of txs to trigger coin_spent and coin_received events with empty string values for the amount (which would otherwise be a list of Coins).
Changes

check for empty string amount values in handleNativeBalanceIncrement and handleNativeBalanceDecrement

:sweat_smile: hehe - I will pass your message along to the mainnet preview node :stuck_out_tongue_closed_eyes:


remove akka/lightbend refs in docs

I didn't rewrite the docs when I switched over to Pekko jars
I did some global replaces but left a lot of stuff behind to keep the PR size manageable

largely done now