OSError: You are trying to access a gated repo.

Neither the main Gradio repo nor Comfy will work whilst you have reliance on gated repo/model.

Then, I will change to a non-gated repo.

Super excited to test this out locally and even more excited for the 4096 checkpoints, keep it up!

We updated the code, could you give it a try?
cd ./ComfyUI/custom_nodes/ComfyUI_ExtraModels
git pull

@lawrence-cj I was getting that error on your own Gradio repo, but it would also translate to Comfy.
On Comfy your workflow will not work as it has nothing there to download models. If you just update the readme with links to the models required that should suffice? I'm not aware of a model auto downloader in comfy so you are referencing files that don't and will never exist?
Prompt outputs failed validation
SanaCheckpointLoader:
- Value not in list: ckpt_name: 'ex\Sana_1600M_1024px.pth' not in (list of length 31)
ExtraVAELoader:
- Value not in list: vae_name: 'xp\dc_ae_f32c32_sana_1.0_diffusers.safetensors' not in ['mit-han-lab/dc-ae-f32c32-sana-1.0-diffusers', 'ae.safetensors']
- Value not in list: vae_type: 'dcae-f32c32-sana-1.0' not in ['kl-f4', 'kl-f8', 'kl-f8-d16', 'kl-f16', 'kl-f32', 'vq-f4', 'vq-f8', 'vq-f16', 'Consistency-Decoder', 'SDV-VideoDecoder', 'MoVQ3', 'dcae-f32c32-sana-1.0-diffusers']

cannot import name 'cached_download' from 'huggingface_hub' (/miniconda3/envs/comfy/lib/python3.12/site-packages/huggingface_hub/init.py)

Then I think you need to upgrade the huggingface_hub?

I updated but still the same error
Installing collected packages: huggingface_hub
  Attempting uninstall: huggingface_hub
    Found existing installation: huggingface-hub 0.26.2
    Uninstalling huggingface-hub-0.26.2:
      Successfully uninstalled huggingface-hub-0.26.2
Successfully installed huggingface_hub-0.26.5

@lawrence-cj  I downgraded to 0.25.0 which fixed that first import issue, but now I have another:
cannot import name 'FromOriginalModelMixin' from 'diffusers'

seems like more testing needed, as now I'm messing around with global comfy packages and i've never had a single issue running comfy!

Seems like you need to use a correct diffusers. What about
pip install -U diffusers
We use Gemma and some newest models, which may need a newer version of diffusers, transformers and so on.
I tested in our env setting here which is available. https://github.com/NVlabs/Sana#-1-dependencies-and-installation

Seems like you need to use a correct diffusers. What about pip install -U diffusers We use Gemma and some newest models, which may need a newer version of diffusers, transformers and so on. I tested in our env setting here which is available. https://github.com/NVlabs/Sana#-1-dependencies-and-installation

No, I also tried lowering the version and the latest version of this as per a google link in the hope that would fix it. It doesn't.
Your own code/repo doesn't even work, regardless of ComfyUI.
You reference an "environment" in your link, but I don't see anything apart from "environment_setup.sh" which doesn't include any diffusers install version, there is no pip requirements file or anything here.

You reference an "environment" in your link, but I don't see anything apart from "environment_setup.sh" which doesn't include any diffusers install version, there is no pip requirements file or anything here.

About this, you need to know. pip install -e . here: https://github.com/NVlabs/Sana/blob/bcad148693affa4f6a971516b57e75d2ee0b3221/environment_setup.sh#L27
It will include the pyproject.toml to prepare all the env we need. Refer to https://github.com/NVlabs/Sana/blob/bcad148693affa4f6a971516b57e75d2ee0b3221/pyproject.toml#L1

Could you push the huggingface cached changed to your own standalone app repo as well?

We will update it after everything is ready soon.


Alternate linking strategy

In the heatmap we provide target URLs.  These are published files that aren't present during development, which throws an error:

Is it possible to supply the target URL by referring to the DITA object?

Output under #32 resolve this issue
Please check the output

Fixed in #32


*TVAL should capture the effective address on an exception

This is a clarification that needs to be added based on the following email thread:
https://lists.riscv.org/g/tech-privileged/topic/94419723
(I keep forgetting to update the spec, so capturing this change for posterity)

@wmat, almost at the end of the "Page Fault, Guest-Page Fault, and Access Fault Exceptions" section (before the non-normative note at the end), add the following one sentence paragraph:
When a page fault, guest-page fault, or access fault exception is taken, the relevant *tval CSR is written with the faulting effective address (i.e. the same faulting address value as for other causes of these exceptions).


StatusBar height unexpected with iPhone X

Problem description
StatusBar height is commonly used in conjunction with displayMode: "float" in order to offset the app content. However, on iPhone X, height reports 44, i.e. the height of the StatusBar including the area around the notch outside of the usable app view area.
Expected behavior
height should report the height of the StatusBar inside the app view area.
Environment

Tabris.js version: 2.5.1
Device: iPhone X
OS: iOS 11

This issue is invalid. You can set background colour of statusBar to transparent and content will rendered under the status bar. Status bar is still visible hence the non-zero value of it's height. This is correct behaviour. You can see it in effect in Google Maps or Weather app for example.


ci: Make the link-liveness job output easier to skim

Before:
Spider mode enabled. Check if remote file exists.
--2024-05-13 23:26:46--  https://bazel.build/
Resolving bazel.build (bazel.build)... <IP_ADDRESS>, 2a00:1450:400f:803::200e
Connecting to bazel.build (bazel.build)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 77416 (76K) [text/html]
Remote file exists and could contain further links,
but recursion is disabled -- not retrieving.

Spider mode enabled. Check if remote file exists.
--2024-05-13 23:26:47--  https://broken-link.robinlinden.eu/
Resolving broken-link.robinlinden.eu (broken-link.robinlinden.eu)... failed: Name or service not known.
wget: unable to resolve host address ‘broken-link.robinlinden.eu’

After:
2024-05-13 23:26:55 URL: https://bazel.build/ 200 OK
wget: unable to resolve host address ‘broken-link.robinlinden.eu’

Yay, it works:


Need an nginx level announcement

Currently we show this

But we do have a page,

https://github.com/cidvbi/patric3_website/blob/master/webRoot/patric/html/50x.html


DS2 does not work through wine in the client

If you use any version after 28.0.0 the injector gets stuck at installing hooks, and the game just loads into the regular fromsoft servers,

With version 28.0.0, although it gets past installing hooks, it fails at patching the port for some reason and always fails to go online. I tried hosting a server with the same version to no avail, I get the same result.

This is not an issue with DS3 for some reason, I can join servers no problem, I even got some server notification from the discord server while I was testing it yesterday.

edit: re-uploaded the correct picture.

I'm having this issue too

Looking into it, the memory protection preventing DS2_ReplaceServerAddressHook::PatchKey from running never turns off, so the hook never finishes running. The check wasn't there in 0.28 so it didn't hang on this stage but obviously the memory was still protected so it doesn't actually connect.

You think this is something solvable by some proton tinkering, or we stuck with this for the long run?

Actually, it doesn't seem to be memory protection. The memory protection is off, but VirtualQuery is returning 0. GetLastError gives 2, but I can't figure out what that actually means.

#220

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

#220

Can we get a build of this, i built it but was getting a missing .net 5 dependency in linux when trying to open the loader. Not sure if it is me or some other reason, cause I installed all the .net in available and still got told i was missing .net 5.

on TroncoNinja's  there is a patched version build throuth github action it must be without dependency error.

https://github.com/TLeonardUK/ds3os/releases/tag/v<IP_ADDRESS>-main
There you go


Dockerhub: Free Team Organization sunset

Hi,
this is the email that I just received from Dockerhub

Currently, RSS-Bridge is such a free tier organization on dockerhub, consisting of me, @em92 and @captn3m0 . It looks like dockerhub wants to monetize these organizations, thats why they are killing the free tier.
We do have to option to apply for the "Open Source Project Program" with dockerhub, which gives us all the benefits of the paid tier but for free. It does involve an application here wher they want some personal info.
I'm okay in going through the application if you want @dvikan but if you want to do it, go for it.
If we dont do this:
Option A: Pay 9$ per user per month.
Option B: Leave Dockerhub and only host on GH
Option C: Use a personal account to host the official image (meaning rss-bridge/rssbridge would change to dvikan/rssbridge or so).
In my opinion, A is not going to happen.
B would be doable but its going to leave a bunch of people stranded on the version on Dockerhub and after 30 days (so mid May), dockerhub would just straight up delete the repo and image. We could do a final push of the image with an adapted version that would just show "Please change the image source, dockerhub will be abandoned" or so, so that hosters at least notice the change.
C would mean a big hassle if the owner of the image would leave the project for good and not care about it (or drop off the earth). Its also the reason why these community organizations were so nice.

There seems to be some confusion as to what features will go away (we don't use private repos, but what happens to teams?), and a lot more OSS projects that seem to be impacted:

https://github.com/docker/hub-feedback/issues/2314
https://forums.docker.com/t/docker-team-organization-sunsetting/135089/3
https://news.ycombinator.com/item?id=35154025

My suggestion would be to apply for the DSOS program, and if we get rejected (we should qualify IMO) - we can look at how we can migrate users properly to GitHub Packages.
From what I understand, GitHub requires an authentication token, even for pulling images - so it makes self-hosting slightly cumbersome (compared to pulling public images from Docker Hub).

I'm okay with you go ahead and apply for oss program.

From what I understand, GitHub requires an authentication token, even for pulling images - so it makes self-hosting slightly cumbersome (compared to pulling public images from Docker Hub).

Thats not correct. You can pull images without any authentication from dockerhub, github and gitlab. The only annoyance is that the docker daemon automatically assumes you mean dockerhub, thats why "rss-bridge/rssbridge" goes to dockerhub. If you want to use a gh image, you need to add the full repo name (ghcr.io/rss-bridge/rss-bridge in our case).
This is quickly becoming a big issue and will probably result in some kind of response from docker.
From the dockerhub issue you linked, I took away three infos for now:

At least in the past, it took a long time (sometimes years) to be approved for the DSOS program.
Orgs in review will not be purged (which could potentially give us a few years of free service ;D )
Namespace squatting will not be a thing because orgs namespaces will not be cleared (which should at least limit the risk of malicious software hosted under our "old" name, if we decide to not use dockerhub anymore).

At this point, I would wait a few days to do the application. We can still do it, but the whole topic is very much in movement right now, so lets see where the wind blows.

If, for some reason, the Docker-Sponsored Open-Source Program isn't working out, we are happy to help you out. We operate a Harbor-based container registry as a service that has many benefits over most of the other registries out there.
There are also features regarding containerized image distribution, that might be valuable too as well.
Don't underestimate the value of owning and controlling your data, it is easier to migrate if the domain is under your control.

https://www.docker.com/blog/no-longer-sunsetting-the-free-team-plan/
Plan canceled on docker end. No current action needed, we can apply for the OSS program later if its actually needed.


Can this be altered to use on tiktok?

No, tiktok.com does not have categories for live broadcasts.


Refactor: extract dependency resolution from postal.Service

Describe the Enhancement
I would like to propose that we extract the service.Resolve method out of the service object and into a package function. It has no dependencies on the service object and it requires no additional injected objects (and hence no mocked dependencies).
This is a pure aesthetic change, and one could argue that there's no functional cost to instantiating a cargo.Transport and the postal.Service object, but I think it is confusing to do so just to get functionality that does not depend on those objects.
Motivation
The reason I care about this, is because in https://github.com/paketo-buildpacks/cpython/pull/398 we want to leverage this logic to resolve the dependency versions from buildpack.toml during integration test setup, but we do not need or want to instantiate the cargo.NewTransport{} struct which is a required dependency of postal.NewService().

The priority of this refactor is increasing for me for a couple of reasons:

I am seeing opportunities to refactor integration tests to leverage common dependency-management code rather than re-inventing it across multiple tests.
It will make any feature additions (like #401) easier to implement, test, and consume.

Therefore I will probably take a pass at this soon.
I'd like to get some opinions from @paketo-buildpacks/tooling-maintainers about where this new function should live. Adding it at the package-level won't be a breaking change, as we can preserve the existing service.Resolve method (with a deprecation notice), but I'd prefer to pick a package that we all agree on first time.
Currently I'm leaning towards making a new package called dependency with just this one package-level function in it. The signature would therefore be dependency.Resolve(). I'd rather not make a new object (e.g. dependency.NewResolver().Resolve()) but I'm open to that if people feel strongly. It would make it slightly easier to implement interfaces on the consuming side, but only slightly. And a package-level function (where practical) is more idiomatic in Go.
@paketo-buildpacks/tooling-maintainers thoughts?

All in all, I agree that dependency resolution is pretty different from the other stuff that postal currently does. (Arguably, GenerateBillOfMaterials, too.) I like the idea of it being simpler to call the Resolve() function in a variety of scenarios. However, I think Resolve is pretty tightly coupled to  the definition of postal.Dependency, its return type. Are you also looking to move the definition of postal.Dependency into another package?
To me, it feels odd to have a new package dependency with a function Resolve() that returns a type that's defined in postal.

I'd prefer that we avoided making a third Dependency type to go alongside those already existing in postal and cargo. If we did any refactoring, I'd like to see us move dependency resolution into cargo since its the package having the most connection to the underlying buildpack.toml descriptor file. We could then replace the internal bits in postal with the cargo implementation and mark the overall postal.Service.Resolve method as deprecated.

Great call @ryanmoran - I forgot we already had two definitions.
I like the idea of creating cargo.ResolveDependency() with the canonical definition of Dependency living in the cargo package and postal leveraging that as appropriate.
I'll take a look at putting a PR together to address this.

I'm going to close this as we aren't actively reviewing this potential refactor. The various PRs are also closed, but the code still exists on them if anyone wants to take a look.


Fix #3580: Remove relativePath from VirtualIRFiles

I prefer the option including the second commit. WDYT?


[Windows] Deleting slash after disk drive behaves weird

This is a bit more complicated, there should be some validator for that cases, as it is behaves weird for internals paths too


Different diagnostics for proveably vs unproveably non-exhaustive switch.

Sometimes the compiler cannot check the exhaustiveness of a switch statement in a reasonable amount of time, and it just gives up. In that case, the user receives the same diagnostic "switch must be exhaustive" as of the compiler were able to positively disprove exhaustiveness. This PR introduces a new diagnostic: "cannot promptly guarantee swift is exhaustive" and issues this one when appropriate, so that the user can tell the difference.
Communicating this distinction is even more important for two reasons:

If the user adds a "default" case to an (unproveably) exhaustive switch, the compiler will warn about the default never being executed, and
We are likely going to increase the frequency of unproveable exhaustion in order to limit exponential compile times.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 044b9f41981cccf2c8d64e39a32ee368f0e0bd84

@CodaFi Thanks for looking at this. I don't understand your comment, though. The proposal would allow the user to tell the difference between a provably non-exahaustive switch and an un-provably non-exahaustive switch. Are you contending that there is little value in conveying this information to the user?
In the case where it is provable, the user would want to consider what to do in the default case and allow for its execution. In the case where it is not-provable, the user could reason on his own that it is exhaustive and simply invoke fatalError. In other words, the user's actions are different in the two cases: Even though in both cases the user adds a default, the code that follows the "default:" is likely to be different. It seems to me that, since the action taken in response to the two situations is likely to be different, that this is a valuable signal to transmit to the user.

Are you contending that there is little value in conveying this information to the user?

Yes.  Because I doubt whether or not we are currently able to discern the exhaustiveness of a pattern matrix actually has the affect on user behavior you mentioned.  Especially for the patterns that the current heuristic arbitrarily rejects as "too big" - reasoning privately about whether the pattern is exhaustive or not is a terrifically error-prone process.
There's a number more semantically meaningful error paths we can lay down here: e.g. GHC computes a partial covering when they hit these large cases and report that rather than going through the full-monty.

I find it unacceptable to report that a statement is exhaustive when the software has not made that determination. It will mislead and confuse users, especially when we tighten up the test.
I have found a very simple example, a switch with 20 cases, the checking of which calls Space::minus 15 million times, takes several minutes to compile, and has caused auto-builds to fail by timing out.
Are you suggesting an alternative to the both this PR and the status quo? One that can be done at low risk in the short term? If so, could you be more specific? I’m not sure what your last paragraph meant?

Here's the example I mentioned above, anonymized from actual code. I don't find it too hard to see that it is exhaustive, but would be unhappy and confused for the compiler to tell me it was not exhaustive. Even if I had to work to see that it is exhaustive, that would make the message "switch not exhaustive" even worse, because I would spend more time trying to find the error in my reasoning.
enum X {case case1X, case2X, ... case 5X}
enum Y {case case1Y, case2Y, ... case 4Y}
... // Assume x is an X, y is a Y.
switch (x, y) {
   case
           (.case1X, .case1Y),
 	   (.case1X, .case2Y),
	   (.case1X, .case3Y),
 	   (.case1X, .case4Y),
 	   (.case2X, .case1Y),
 	   (.case2X, .case2Y),
 	   (.case2X, .case3Y),
 	   (.case2X, .case4Y),
 	   (.case3X, .case1Y),
 	   (.case3X, .case2Y),
 	   (.case3X, .case3Y),
 	   (.case3X, .case4Y),
 	   (.case4X, .case1Y),
 	   (.case4X, .case2Y),
 	   (.case4X, .case3Y),
 	   (.case4X, .case4Y),
 	   (.case5X, .case1Y),
 	   (.case5X, .case2Y),
 	   (.case5X, .case3Y),
 	   (.case5X, .case4Y):
 	break
}

Today, the compile can detect that this is exhaustive, but it takes over three minutes to do it.
In the short term, we will have to cap the work done by checkExhaustion because 3+ minutes is too long to spend compiling this one piece of code. If we don't change the diagnostics, our compiler is likely to create the impression in the user's mind that the diagnostics cannot be trusted. Once lost, trust can be hard to regain.

@swift-ci please test

@CodaFi You actually had done this before in 43f06afef228aa9a8be7797476d2ffca1007c562!
The comment was:
Defer analysis of large switch bodies

This presents a regression in diagnostic quality that is definitely
worth it not to lie to SILGen about whether a switch is covered or not.

At the same time, disable SIL’s unreachable diagnostic for ‘default’
clauses which would previously cause a warning to be emitted if the
default was proven to be unreachable.  This analysis is incomplete
anyways and can be done by Sema in the future if we desire.

But @xedin reverted these (and other?) changes in 7034ef92b6de209af43cdcf611feca1cabcc7368.
Why?

@davidungar I think it might be because of this - https://bugs.swift.org/browse/SR-6652 that's the only revert of space engine I remember doing :)

@swift-ci please test and merge

@swift-ci Please test and merge


How to play the video repeatedly？

controller = VideoPlayerController.network(
widget.url)
..initialize().then(() {
// Ensure the first frame is shown after the video is initialized, even before the play button has been pressed.
setState(() {});
});
_controller.setLooping(false)
When the video finishes playing, I need to restore the video state to the initial state. When the button is clicked, the video starts playing from the beginning.
This is a problem with the video_player plugin
https://pub.dev/packages/video_player
video_player 0.10.2 + 5

u can do like this.

In this case, the video starts playing again. I need to restore the initialization state after a video has been played naturally, and the video is paused at this moment.

Hi @Tumbler0809
This platform is not met for assistance on personal code.
Please see https://flutter.dev/community for resources for asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Please provide a UI option to enter commandline arguments when running Python programs

I've worked with a team that created one Python package (which contained an __init__.py file). Inside that package, they'd create multiple .py files, each meant to perform various tasks. Many of those files had their own main functions too. So these guys were running those files from the commandline, and passing commandline arguments to it (like python3 prog1.py --file \somedir\somefile.csv --n 23. This is clearly a misuse of Python, but they had their deadlines to meet.
Now I'm expected to use and debug those programs, and I'm unable to run the program from VS Code, because the IDE does not have an option to specify commandline parameters. I know you offer a method of editing the config file, but there are two issues with this:

There are many files in this project that have their own main function. So even though all these files belong to one package, I have to be able to run it by specifying custom commandline options for each file I have currently opened. So right next to the triangular run button, if you provide a textfield to specify commandline arguments that are specific to the file currently opened, it'd help.
One should not have to open and edit a config file for functionality that should be available via the UI options.

/extPython
The run and debug features are always contributed by language-specific extensions.


Fix #575 (FocusGained / FocusLost functionality)

Listen to BrowserWindow's blur and focus events
Add an executeAutoCommand method to NeovimInstance
Call executeAutoCommand on blur/focus

Fixes #575

Do we redraw on focus gain?
Whenever I resize my window it shows a blank whatever background color is (test on windows of latest build)

@cyansprite - sorry, just saw your comment here!
We do not explicitly redraw on FocusGained - we might though if there is something in an autocmd that causes Neovim to ask for a redraw (like printing a message, changing line number format, etc).
Resize would likely be a separate issue - this PR had some changes related to resize: https://github.com/extr0py/oni/pull/557
When you say latest build, is it the latest master or the latest downloadable build in the Releases tab? If its on master, feel free to open up an issue for it! Otherwise, there have been quite a few changes since the last binary release, so it might be fixed in master now.

@extr0py
Sorry, I meant building the latest source.
I will test it in a bit and get back to you, whether to open an issue or not.


why not to choose mono scripting backend ?

if i want to use this plugin, why can't choose mono scripting backend with unity build ?

Is there any advantage of using Mono over IL2CPP?

the unity built project  can cost  much shorter  timer with mono than IL2CPP, it will save my develop time

@wangjianpeng at this point, the IL2CPP seems to be the most stable way of getting it to work. Something about how unity player is compiled using mono that gets some methods missing. I will still be improving and be writing iOS support, so I am looking to test it against mono intensively till I get a working implementation.

Tested with mono, and everything works well using Unity 2018.3.f12


API failed in run

Dear OpenGPTs,
Thanks a lot for your efforts, and when testing OpenGPTs API according to API.md
It failed in testing "runs" as followings,
when test this case
import requests
requests.post('http://<IP_ADDRESS>:8100/runs', cookies= {"opengpts_user_id": "foo"}, json={
    "assistant_id": "9c7d7e6e-654b-4eaa-b160-f19f922fc63b",
    "thread_id": "231dc7f3-33ee-4040-98fe-27f6e2aa8b2b",
    "input": {
        "messages": []
    }
}).content

the result is.
b'{"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}'
####### and on the other hand, when test the last case:
import requests
import json
response = requests.post(
    'http://<IP_ADDRESS>:8100/runs/stream',
    cookies= {"opengpts_user_id": "foo"}, json={
    "assistant_id": "9c7d7e6e-654b-4eaa-b160-f19f922fc63b",
    "thread_id": "231dc7f3-33ee-4040-98fe-27f6e2aa8b2b",
    "input": {
        "messages": [{
            "content": "have a good day!",
            "type": "human",
        }]
    }
})
res = []
if response.status_code == 200:
    # Iterate over the response
    for line in response.iter_lines():
        if line:  # filter out keep-alive new lines
            string_line = line.decode("utf-8")
            # Only look at where data i returned
            if string_line.startswith('data'):
                json_string = string_line[len('data: '):]
                # Get the json response - contains a list of all messages
                json_value = json.loads(json_string)
                if "messages" in json_value:
                    # Get the content from the last message
                    # If you want to display multiple messages (eg if agent takes intermediate steps) you will need to change this logic
                    print(json_value['messages'][-1]['content'])
else:
    print(f"Failed to retrieve data: {response.status_code}")

And it returns : Failed to retrieve data: 422
Web page is working ok on http://localhost:5173/
please advise how to troubleshoot, and thanks a lot!

Replicated the problems with this script from the API docs:
import requests
import json

# Set the base URL and user cookie for authentication
BASE_URL = 'http://<IP_ADDRESS>:8100'
USER_COOKIE = {'opengpts_user_id': 'foo'}

# Create an assistant
print("Creating an assistant...")
assistant_response = requests.post(
    f'{BASE_URL}/assistants',
    json={
        "name": "bar",
        "config": {
            "configurable": {
                "type": "agent",
                "type==agent/agent_type": "GPT 3.5 Turbo",
                "type==agent/system_message": "You are a helpful assistant",
                "type==agent/tools": ["Wikipedia"]
            }
        },
        "public": True
    },
    cookies=USER_COOKIE
)

print(f"Assistant creation response status: {assistant_response.status_code}")
if assistant_response.status_code != 200:
    print(f"Assistant creation error: {assistant_response.text}")
else:
    assistant_data = assistant_response.json()
    assistant_id = assistant_data['assistant_id']
    print(f'Assistant ID: {assistant_id}')

# Create a thread
print("Creating a thread...")
thread_response = requests.post(
    f'{BASE_URL}/threads',
    cookies=USER_COOKIE,
    json={
        "name": "hi",
        "assistant_id": assistant_id
    }
)

print(f"Thread creation response status: {thread_response.status_code}")
if thread_response.status_code != 200:
    print(f"Thread creation error: {thread_response.text}")
else:
    thread_data = thread_response.json()
    thread_id = thread_data['thread_id']
    print(f'Thread ID: {thread_id}')

# Add a message to the thread
print("Adding a message to the thread...")
message_response = requests.post(
    f'{BASE_URL}/threads/{thread_id}/messages',
    cookies=USER_COOKIE,
    json={
        "messages": [{
            "content": "hi! my name is bob",
            "type": "human",
        }]
    }
)

print(f"Message addition response status: {message_response.status_code}")
if message_response.status_code != 200:
    print(f"Message addition error: {message_response.text}")

# Check the thread for messages
print("Checking thread for messages...")
check_messages_response = requests.get(
    f'{BASE_URL}/threads/{thread_id}/messages',
    cookies=USER_COOKIE
)
print(f"Thread check response status: {check_messages_response.status_code}")
if check_messages_response.status_code != 200:
    print(f"Thread check error: {check_messages_response.text}")
else:
    print(f'Thread messages: {check_messages_response.json()}')

# Run the assistant on the thread
print("Running the assistant on the thread...")
run_response = requests.post(
    f'{BASE_URL}/runs',
    cookies=USER_COOKIE,
    json={
        "assistant_id": assistant_id,
        "thread_id": thread_id,
        "input": {
            "messages": []
        }
    }
)

print(f"Assistant run response status: {run_response.status_code}")
if run_response.status_code != 200:
    print(f"Assistant run error: {run_response.text}")

# Add a new message and run the assistant again
print("Running the assistant with a new message...")
new_run_response = requests.post(
    f'{BASE_URL}/runs',
    cookies=USER_COOKIE,
    json={
        "assistant_id": assistant_id,
        "thread_id": thread_id,
        "input": {
            "messages": [{
                "content": "whats my name? respond in spanish",
                "type": "human",
            }]
        }
    }
)

print(f"New run response status: {new_run_response.status_code}")
if new_run_response.status_code != 200:
    print(f"New run error: {new_run_response.text}")

# Check the thread for new messages
print("Final check for thread messages...")
final_check_response = requests.get(
    f'{BASE_URL}/threads/{thread_id}/messages',
    cookies=USER_COOKIE
)
print(f"Final check response status: {final_check_response.status_code}")
if final_check_response.status_code != 200:
    print(f"Final check error: {final_check_response.text}")
else:
    print(f'Final thread messages: {final_check_response.json()}')

# Stream back responses for a new message
print("Streaming back responses...")
stream_response = requests.post(
    f'{BASE_URL}/runs/stream',
    cookies=USER_COOKIE,
    json={
        "assistant_id": assistant_id,
        "thread_id": thread_id,
        "input": {
            "messages": [{
                "content": "have a good day!",
                "type": "human",
            }]
        }
    },
    stream=True
)

print(f"Stream response status: {stream_response.status_code}")
if stream_response.status_code != 200:
    print(f"Streaming error: {stream_response.text}")
else:
    for line in stream_response.iter_lines():
        if line:  # filter out keep-alive new lines
            string_line = line.decode("utf-8")
            if string_line.startswith('data'):
                json_string = string_line[len('data: '):]
                json_value = json.loads(json_string)
                if "messages" in json_value:
                    print(json_value['messages'][-1]['content'])

@roboticsocialism I faced a similar problem, it could be because the assistant is not assigned any model. Please check on the frontend and see if on your created assistant a model is selected.

@ArneJanning did you use the same cookie as set in your ui to create the assistant?

Thanks a lot @mkorpela , I used @mkorpela script and found the same result like @ArneJanning testing.

Same problem here, using @mkorpela 's code:
Creating an assistant...
Assistant creation response status: 200
Assistant ID: 490a103a-788b-49ba-b581-16c6a45886a8
Creating a thread...
Thread creation response status: 200
Thread ID: 3a281ba0-0508-4fc7-91b4-254703c43eb6
Adding a message to the thread...
Message addition response status: 200
Checking thread for messages...
Thread check response status: 200
Thread messages: {'messages': [{'content': 'hi! my name is bob', 'additional_kwargs': {}, 'type': 'human', 'example': False}]}
Running the assistant on the thread...
Assistant run response status: 422
Assistant run error: {"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}
Running the assistant with a new message...
New run response status: 422
New run error: {"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}
Final check for thread messages...
Final check response status: 200
Final thread messages: {'messages': [{'content': 'hi! my name is bob', 'additional_kwargs': {}, 'type': 'human', 'example': False}]}
Streaming back responses...
Stream response status: 422
Streaming error: {"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}

Strange thing: although the assistant creation seems to have worked, it's not even visible through the UI.

Same problem here, using @mkorpela 's code:
Creating an assistant...
Assistant creation response status: 200
Assistant ID: 490a103a-788b-49ba-b581-16c6a45886a8
Creating a thread...
Thread creation response status: 200
Thread ID: 3a281ba0-0508-4fc7-91b4-254703c43eb6
Adding a message to the thread...
Message addition response status: 200
Checking thread for messages...
Thread check response status: 200
Thread messages: {'messages': [{'content': 'hi! my name is bob', 'additional_kwargs': {}, 'type': 'human', 'example': False}]}
Running the assistant on the thread...
Assistant run response status: 422
Assistant run error: {"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}
Running the assistant with a new message...
New run response status: 422
New run error: {"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}
Final check for thread messages...
Final check response status: 200
Final thread messages: {'messages': [{'content': 'hi! my name is bob', 'additional_kwargs': {}, 'type': 'human', 'example': False}]}
Streaming back responses...
Stream response status: 422
Streaming error: {"detail":[{"loc":["body","input"],"msg":"value is not a valid sequence","type":"type_error.sequence"}]}

Strange thing: although the assistant creation seems to have worked, it's not even visible through the UI.

@roboticsocialism I faced a similar problem, it could be because the assistant is not assigned any model. Please check on the frontend and see if on your created assistant a model is selected.

would you please show me the config of model?
my config was like following provided by @mkorpela
# Set the base URL and user cookie for authentication
BASE_URL = 'http://<IP_ADDRESS>:8100'
USER_COOKIE = {'opengpts_user_id': 'foo'}

# Create an assistant
print("Creating an assistant...")
assistant_response = requests.post(
    f'{BASE_URL}/assistants',
    json={
        "name": "bar",
        "config": {
            "configurable": {
                "type": "agent",
                "type==agent/agent_type": "GPT 3.5 Turbo",
                "type==agent/system_message": "You are a helpful assistant",
                "type==agent/tools": ["Wikipedia"]
            }
        },
        "public": True
    },
    cookies=USER_COOKIE
)

This seems to work
from time import sleep

import requests
import json

# Set the base URL and user cookie for authentication
BASE_URL = 'http://<IP_ADDRESS>:8100'
USER_COOKIE = {'opengpts_user_id': 'foo'}

# Create an assistant
print("Creating an assistant...")
assistant_response = requests.post(
    f'{BASE_URL}/assistants',
    json={
        "name": "bar",
        "config": {
            "configurable": {
                "type": "agent",
                "type==agent/agent_type": "GPT 3.5 Turbo",
                "type==agent/system_message": "You are a helpful assistant",
                "type==agent/tools": [
                    {
                        "type": "wikipedia",
                        "config": {}
                    }
                ]
            }
        },
        "public": True
    },
    cookies=USER_COOKIE
)

print(f"Assistant creation response status: {assistant_response.status_code}")
if assistant_response.status_code != 200:
    print(f"Assistant creation error: {assistant_response.text}")
else:
    assistant_data = assistant_response.json()
    assistant_id = assistant_data['assistant_id']
    print(f'Assistant ID: {assistant_id}')

# Create a thread
print("Creating a thread...")
thread_response = requests.post(
    f'{BASE_URL}/threads',
    cookies=USER_COOKIE,
    json={
        "name": "hi",
        "assistant_id": assistant_id
    }
)

print(f"Thread creation response status: {thread_response.status_code}")
if thread_response.status_code != 200:
    print(f"Thread creation error: {thread_response.text}")
else:
    thread_data = thread_response.json()
    thread_id = thread_data['thread_id']
    print(f'Thread ID: {thread_id}')

# Add a message to the thread
print("Adding a message to the thread...")
message_response = requests.post(
    f'{BASE_URL}/threads/{thread_id}/messages',
    cookies=USER_COOKIE,
    json={
        "messages": [{
            "content": "hi! my name is bob",
            "type": "human",
        }]
    }
)

print(f"Message addition response status: {message_response.status_code}")
if message_response.status_code != 200:
    print(f"Message addition error: {message_response.text}")

# Check the thread for messages
print("Checking thread for messages...")
check_messages_response = requests.get(
    f'{BASE_URL}/threads/{thread_id}/messages',
    cookies=USER_COOKIE
)
print(f"Thread check response status: {check_messages_response.status_code}")
if check_messages_response.status_code != 200:
    print(f"Thread check error: {check_messages_response.text}")
else:
    print(f'Thread messages: {check_messages_response.json()}')

# Run the assistant on the thread
print("Running the assistant on the thread...")
run_response = requests.post(
    f'{BASE_URL}/runs',
    cookies=USER_COOKIE,
    json={
        "thread_id": thread_id,
        "input": [
            {
                "content": "hi! my name is bob",
                "type": "human",
            }
        ]
    }
)

print(f"Assistant run response status: {run_response.status_code}")
if run_response.status_code != 200:
    print(f"Assistant run error: {run_response.text}")

# Add a new message and run the assistant again
print("Running the assistant with a new message...")
new_run_response = requests.post(
    f'{BASE_URL}/runs',
    cookies=USER_COOKIE,
    json={
        "assistant_id": assistant_id,
        "thread_id": thread_id,
        "input": [
            {
                "content": "whats my name? respond in spanish",
                "type": "human",
            }
        ]
    }
)

print(f"New run response status: {new_run_response.status_code}")
if new_run_response.status_code != 200:
    print(f"New run error: {new_run_response.text}")

print("Wait 5 seconds")
sleep(5)

# Check the thread for new messages
print("Final check for thread messages...")
final_check_response = requests.get(
    f'{BASE_URL}/threads/{thread_id}/messages',
    cookies=USER_COOKIE
)
print(f"Final check response status: {final_check_response.status_code}")
if final_check_response.status_code != 200:
    print(f"Final check error: {final_check_response.text}")
else:
    print(f'Final thread messages: {final_check_response.json()}')

# Stream back responses for a new message
print("Streaming back responses...")
stream_response = requests.post(
    f'{BASE_URL}/runs/stream',
    cookies=USER_COOKIE,
    json={
        "assistant_id": assistant_id,
        "thread_id": thread_id,
        "input": [{
                "content": "have a good day!",
                "type": "human",
            }]
    },
    stream=True
)

print(f"Stream response status: {stream_response.status_code}")
if stream_response.status_code != 200:
    print(f"Streaming error: {stream_response.text}")
else:
    for line in stream_response.iter_lines():
        if line:  # filter out keep-alive new lines
            string_line = line.decode("utf-8")
            if string_line.startswith('data'):
                json_string = string_line[len('data: '):]
                json_value = json.loads(json_string)
                if isinstance(json_value, list) and "content" in json_value[-1]:
                    print(json_value[-1]['content'])
print("END")


fix: Prevent passing a NULL pointer to strncmp when creating an empty string

While trying to find a conststr for a zero-length (empty) it might happen that the string pointer is NULL (for example during parsing a JSON string). This caused berry to pass the NULL pointer to strncmp which is Undefined Behaviour, and caused the address sanitizer to print a warning (also this could cause crashes on some configurations).
Additionally a test case has been added for this behaviour.

Thanks, well done.


Add Gitea Account

Hi,
Thank you so much for this wonderful work on the recovery of stackedit :)
We failed to create a workspace from gitea.
We tried with gitea versions 1.18.3 and 1.19.0
We are redirected to the gitea but after authorizing the stackedit apps nothing happens (:
In the account part we have this: "StackEdit+ has no access to any external account yet."
StackEdit Compose
# https://github.com/mafgwo/stackedit-plus
version: "3.7"

services:
  stackedit:
    image: mafgwo/stackedit-plus:6.0.0-2

    environment:
      - LISTENING_PORT=8080
      - ROOT_URL=/
      # redirect url https://<url>/oauth2/callback
      - GITEA_URL=https://gitea.[url]
      - GITEA_CLIENT_ID=XXX-XXXX-XXX...
      - GITEA_CLIENT_SECRET=<long_secret>
    ports:
      - 10889:8080/tcp

Gitea Config:

Apps name: StackEdit+
Redirect URL: https://stackedit.[url]/oauth2/callback

Could you help us to solve this problem
/Cyril

Make sure you have setup core correct in gitea.
https://tesh.at/blog/gitea-cors/

First, your Gitea instance needs to support cross-origin requests (CORS). The CORS configuration on Gitea's official website may have some issues. If this is the case, you can use an Nginx proxy to support CORS. Second, if your Gitea instance is using HTTP instead of HTTPS, you need to add it via http://stackedit.net/app, otherwise it will not be added successfully.
  add_header                 Access-Control-Allow-Headers *;
  add_header                 Access-Control-Allow-Origin $http_origin;
  add_header                 Access-Control-Allow-Methods GET,POST,PUT,DELETE,OPTIONS;

  if ($request_method = 'OPTIONS') {
    return 204;
  }

Hi,
Txu !!! it's works :+1:

Hi,
Txu !!! it's works :+1:


Support custom keyboards including decimal numbers UIKeyboardTypeDecimalPad

https://github.com/mnill/cordova-custom-keyboard (as mentioned in https://github.com/driftyco/ionic-plugin-keyboard/issues/74#issuecomment-203509710) has support for various iOS custom keyboards. It would be great if Ionic would add support for custom keyboards. Note that the https://github.com/mnill/cordova-custom-keyboard plugin has a significant limitation in that it uses a hidden text view https://github.com/mnill/cordova-custom-keyboard/blob/master/src/ios/CDVCustomKeyboard.m#L11 which apparently results in the cursor not being visible. That is, the cursor is in the hidden text view instead of the input element the user sees as they type.
Would this feature be appropriate to add to the Ionic keyboard?

+1; would love for a Decimal Pad in Ionic 2

+1:  Decimal Pad in Ionic 2 would be great.  Directive attribute to specify keyboard type would be ideal.

+1: Very wonderful why developer team didn't implement this till now!!


⚠ Moving Control Plane Provider and Infra Provider

This PR separates the Control Plane Provider from the Infrastructure provider allowing them to be deployed separately which will help us down the line if we end up wanting to support #44 but also helps us to better the experience for clusterctl once we get our first release out to help fulfill #54.
Testable Steps
Create kind cluster
kind create cluster

Install cert-manager
kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.3.1/cert-manager.yaml

Clone CAPI and Deploy Dev release
We need to deploy the unreleased version of CAPI for v1alpha4 API support.
git clone git@github.com:kubernetes-sigs/cluster-api.git
cd cluster-api
make release-manifests
# change feature flags on core
sed -i'' -e 's@- --feature-gates=.*@- --feature-gates=MachinePool=false,ClusterResourceSet=true@' out/core-components.yaml
kubectl apply -f out/core-components.yaml

Create Docker Images, Manifests and Load Images
export PULL_POLICY=Never
export TAG=dev
make docker-build release-manifests
kind load docker-image gcr.io/cluster-api-nested-controller-amd64:dev
kind load docker-image gcr.io/nested-controlplane-controller-amd64:dev

Deploy CAPN
kubectl apply -f out/cluster-api-provider-nested-components.yaml

Apply Sample Cluster
kubectl apply -f config/samples/

Get KUBECONFIG
clusterctl get kubeconfig cluster-sample > kubeconfig

Port Forward
In a separate shell, you will need to port-forward the apiserver service.
kubectl port-forward svc/cluster-sample-apiserver 6443:6443

Connect to Cluster
To use the KUBECONFIG without modification we first need to setup a host record for the
apiserver service name, to do this we can append that to /etc/hosts
cat /etc/hosts
# Normal hosts file
<IP_ADDRESS> cluster-sample-apiserver

Connect to the Cluster! 🎉
kubectl --kubeconfig kubeconfig get all -A

Related
Closes #45
Signed-off-by: Chris Hein<EMAIL_ADDRESS>

/cc @vincepri we chatted about this before and we didn't get the first workable version out before we did this reorganization.

/assign @charleszheng44 @Fei-Guo

@charleszheng44 the description of this PR has most of the steps laid out for the initial tutorial, I didn't write these up in docs yet if you'd still like to do that for #52

LGTM

@charleszheng44 the description of this PR has most of the steps laid out for the initial tutorial, I didn't write these up in docs yet if you'd still like to do that for #52

Yes. I will try this out and write up the tutorial for the first release.

/lgtm


Update configure-workload-identity.md

Calrifying that the values will change later in the second case as well.

@juliakm
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


10/31/2018 17:58 : Create Pull Request for branch master -> prTesting1_2018103117

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon1.md
:white_check_mark:Succeeded

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


Webpack 5 compatibility

Hi!
First off, thank you for this plugin - it is much easier, more convenient and more flexible to use than the official Sentry plugin.
Unfortunately, it doesn't seem to work with Webpack 5 due to this issue. Would it be possible to provide a fix?
Thank you!

Hi! I am happy to accept a PR implementing that change. Otherwise I will try to look into it myself when I get time, but it is not a super urgent priority for me since webpack 5 is still in beta with no release date announced as far as I know.


Implement realnum::rand

Implement rand. Use the new derive [#default]
Fixes #214
Checklist

[x] My branch is up-to-date with development branch.
[x] Everything works and tested on latest stable Rust.
[x] Coverage and Linting have been applied

Current behaviour
realnum::rand returns 1.0
New expected behaviour
random number is generated

thanks @morenol
I need a 0.3.1 release asap! 👍🏼

LGTM but not sure why ignore was added to one of the doc tests

that test uses datasets feature so it fails, it is replicated among the tests with the right flag.
I may have more changes in the next hours, I am adding anti-panicking checks to fit methods.

It seems that there is conflicts, it says to me that there are 83 commits on this PR, could you rebase?

It seems that there is conflicts, it says to me that there are 83 commits on this PR, could you rebase?

that is because I pushed the wrong branch at the creation of the PR, then I corrected it.


/routing/v1/router_groups returns a 401 instead of 403 if user doesn't have the correct scope

It appears that when listing router_groups, the routing-api is returning a 401 if the client doesn't have the correct scope (router_groups_handler.go:43). This causes cf-java-client to think that the user token is bad (See cf-java-client issue 826).
Ideally you would return a 401 wherever you encounter an invalid token and a 403 whenever you encounter a client that is missing the required scope.

So to provide a little bit of context around this issue, I was interacting with the routing api via the cf-java-client and discovered that the client hangs when creating a shared tcp domain in Cloud Foundry if the user doesn't have access to read the router groups. When I looked into the issue with them, it was discovered that the routing API is returning a 401 when the client doesn't have the correct scope to read the router groups. Currently, they don't have a way to work around this issue since they expect a 403 to tell them that they don't have access. For them a 401 means that they need to refresh their token.
The cf-java-client team has made some improvements but ultimately this is something that will probably need to be fixed in the routing API.

Hi @forbushbl ,
The routing team does not have the bandwidth currently to work on this. Would you be able to make a PR with this change?
Regards,
Shubha

Hi @forbushbl ,
Going to close this issue since we haven't heard back in over a year. If you still would like us to prioritize this, feel free to reopen this issue. Again, we are open to PR's, but still do not have the bandwidth to address this.


Allow to deprecate enums

Similar to https://github.com/strawberry-graphql/strawberry/pull/530
We should be able to deprecate enums :)

Hi @patrick91, I am looking for an Issue to contribute to for Hacktoberfest since another I started towards it will take longer than October. Please can I take this one on, if someone isn't planning to do it already? I am fairly sure I know what to do and can strive to put up a PR before the end of tomorrow.

Hi @patrick91, I am looking for an Issue to contribute to for Hacktoberfest since another I started towards it will take longer than October. Please can I take this one on, if someone isn't planning to do it already? I am fairly sure I know what to do and can strive to put up a PR before the end of tomorrow.

Awesome! No one is working on this as far as I know!
Feel free to join our discord in case you have questions! https://discord.gg/ZkRTEJQ

Excellent, thanks :) I'll ask on the forum if I get stuck in some way otherwise put up a PR shortly.

Hi, just to provide an update, I've missed the chance to finish this for Hacktoberfest because I was busier than I thought over the weekend, but I still plan to address this with a PR. I am perhaps halfway there.
Due to other priorities I estimate it will be towards the end of the week (most likely Friday) before I can finish it and open the PR. I hope that is still okay and sorry for the delay relative to my original plan. Thanks.

@sadielbartholomew no worries! Thanks for the update!
I think we will still send Strawberry stickers to new contributors if they want 😊

@patrick91 What do you mean by deprecating enums?
GraphQLEnumType doesn't have deprecation_reason - https://github.com/graphql-python/graphql-core/blob/3359fe60d9940b174270426438a89d629b6cc6e2/src/graphql/type/definition.py#L1044-L1052
I believe we already can deprecate field that is using enum.
@strawberry.field(deprecation_reason="Deprecated Enum")
def enum_field(self, info, id: int) -> IceCreamFlavour:
    return IceCreamFlavour.STRAWBERRY

@msobas you can deprecate enum fields:
enum Direction {
  UP
  DOWN
  LEFT
  """
  The other left
  """
  RIGHT
  SIDEWAYS @deprecated(reason: "Replaced with Left or Right")
}

Apologies for not being clear in the issue description ☺

Hi, sorry this fell off my radar after November, I'm happy to still give it a go in the next month or so, but if this is urgent then by all means someone else can step in to do this (not sure if that is the case now given these new comments in this thread), in which case please accept my apologies for not getting it done. (I was a over-optimistic with the time I would have to devote to other open-source projects).

@sadielbartholomew no worries at all :) feel free to come back to this task when you have time, don't think it is urgent :)

I looked at this a little bit before @patrick91 updated description and I implemented this kind of API:
@strawberry.enum(
        values_config=dict(
            CHOCOLATE=dict(deprecation_reason="Strawberry is so much better"),
            STRAWBERRY=dict(description="This is obvious winner in all tastes"),
        ),
    )
    class IceCreamFlavour(Enum):
        VANILLA = "vanilla"
        STRAWBERRY = "strawberry"
        CHOCOLATE = "chocolate"

What do you think about this approach?
If it's working for us, I could create pull request @sadielbartholomew (I'm only missing some better tests) if not, than it probably would be quite easy to covert to API suggested by @patrick91

Hmm that feels a little heavy on the enum decorator. Personally, I'd like to keep the value metadata closer to the actual values

@msobas oh, didn't realise you were working on it ☺
I had a short chat with @BryceBeagle on discord and, while we have different preferences, we kinda settled on the api in the description, I think it's consistent with what we have at the moment :)
Feel free to join our discord too https://strawberry.rocks/discord

Well it was more of a case of I have a free evening :D and wanted to play around something. Sorry for not saying anything before ;) probably a good practice so we don't do stuff twice. I'll search for your discussion and think about API again :)


Use native "open" Raycast command for Wikipedia extension

Description

Use native "open" Raycast command for Wikipedia extension to remove an unnecessary dependency
Type of change

Bug fix / improvement for my extension

Screencast

https://user-images.githubusercontent.com/36263538/152693451-c5bffc60-3f1c-4a0e-9ec4-eccbdbc31a81.mp4
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast

Very cool to see that our built-in open function reduces your dependencies :v:


Use default remote branch instead of master

Default to cloning the template repository's default branch, instead of hard-coding a default of master if --branch isn't given.
This has two benefits:

For repositories with a default branch other than master, --branch is no longer required when using cargo generate
If the user has init.defaultBranch (new in Git 2.28.0) set to something other than master, the newly-created repository has that branch name.

As an added bonus, when I if a user with init.defaultBranch set to something other than master runs the integration tests, they don't all fail because the master branch can't be found.
(Re-do of #240 with renamed branch)

Any updates on the additional review? I'm in favor of getting that merged.

@sassman I am in favor of getting that merged too :+1:

@sassman I will update the MR to match the current master, and merge this as soon as the tests are fine.

Done, thanks @9999years for your work :)


Support for setting the service name on Log4j2's EcsLayout

What does this PR do?
Helps with #1626
Checklist

[x] This is a new plugin

[x] I have updated CHANGELOG.asciidoc
[x] My code follows the style guidelines of this project
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have updated supported-technologies.asciidoc
[x] Added an API method or config option? Document in which version this will be introduced
[ ] Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.

:grey_exclamation: Build Aborted

There is a new build on-going so the previous on-going builds have been aborted.

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Reason: Aborted from #2

Start Time: 2021-11-29T15:30:09.795+0000

Duration: 22 min 53 sec

Commit: fb5c1cfc718c40ffd0d20121e17b8b8ee86da906

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

run elasticsearch-ci/docs

run elasticsearch-ci/docs

run elasticsearch-ci/docs

Thank You!

Thank you too!


Add AWS "sm" service shortcut for Secrets Manager

Not to be confused with AWS Systems Manager, aka "SSM". 🙄

Merged in #422


the decelerating at RunForward blocks briefly

Hi,
I have a problem with the deceleration in the RunForward command. I change the speed with a potentiometer in the loop as follows:
void loop() {
potValue = analogRead(potpin);
targetSpeed = map(potValue, 0, 1023, 1, 40000) ;// Targetspeed pro Intervall
stepper->setAcceleration(30000);
stepper->setSpeedInHz(targetSpeed);  // the parameter is Hz !!!
stepper->runForward();

}
The acceleration works wonderfully, but with the deceleration I have errors. There seems to be no acceleration function behind the deceleration, and when I turn the potentiometer slowly the stepper sometimes brakes briefly to a standstill and then accelerates again.
Can anyone help me with this problem?
Greetings
Flo

Hi Gin66,
Thanks a lot for the very fast response. I will try that in the afternoon. What I am also confused of is the missing deceleration function. The stepper speed follows direkt the potentiometer position. It should not behave like this, right? If I speed up, it works like expected.

Hi,
so i tested the above code today.
int average = 0;
void loop() {
potValue = analogRead(potpin);
average += (potValue - average)/4;
targetSpeed = map(average, 0, 1023, 1, 40000) ;// Targetspeed pro Intervall
stepper->setAcceleration(30000);
stepper->setSpeedInHz(targetSpeed);  // the parameter is Hz !!!
stepper->runForward();
delay(1);
}
Unfortunately it does not bring any improvement. I am using version 0.30. It is conceivable that I have the same error as described in #178, but I could not reproduce it with the code in issue #178.

Thanks for the update.
So there are two possibilities:

You stick with your solution, which works for you.
Manually update to 0.30.4 and hope, that fix works in your situation, too.

While you are at it, perhaps you leave a comment here


RDM-4392: Read replicas db

https://tools.hmcts.net/jira/browse/RDM-4392
https://jdbc.postgresql.org/documentation/head/connect.html
Configure Def Store with 2 data sources one pointing to the Master DB the other one to load balance between replicas.
If read replica Data source is enabled, all db requests are by default directed to the read replicas (read replicas made the default because Def Store has only
1 endpoint that produces write operations, the import endpoint. All other endpoints are read endpoints).
To route write/read flows to the master DB, use the @RouteToMaster annotation
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x] No

Codecov Report

Merging #372 into master will decrease coverage by 0.27%.
The diff coverage is 40%.

@@            Coverage Diff            @@
##           master    #372      +/-   ##
=========================================
- Coverage   89.58%   89.3%   -0.28%
=========================================
  Files         331     334       +3
  Lines        6203    6238      +35
  Branches      413     415       +2
=========================================
+ Hits         5557    5571      +14
- Misses        564     585      +21
  Partials       82      82

Impacted Files
Coverage Δ

...store/rest/endpoint/DraftDefinitionController.java
100% <ø> (ø)
:arrow_up:

...inition/store/excel/endpoint/ImportController.java
80% <ø> (ø)
:arrow_up:

...nition/store/rest/endpoint/UserRoleController.java
100% <ø> (ø)
:arrow_up:

...v/hmcts/ccd/definition/store/DataSourceConfig.java
0% <0%> (ø)

...d/definition/store/database/RoutingDataSource.java
100% <100%> (ø)

.../store/database/MasterDBConnectionInterceptor.java
80% <80%> (ø)

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 613d1c8...b65f1aa. Read the comment docs.


Getting value errors on 8th username

Using latest version.
I have removed and changed users around and always the same resutl
Traceback (most recent call last):                                                                                File "/usr/local/bin/instagram-scraper", line 11, in <module>                                                                                load_entry_point('instagram-scraper==1.5.11', 'console_scripts', 'instagram-scraper')()                                                                                File "/usr/local/lib/python3.5/dist-packages/instagram_scraper/app.py", line 754, in main                                                                                scraper.scrape()                                                                                File "/usr/local/lib/python3.5/dist-packages/instagram_scraper/app.py", line 347, in scrape                                                                                user_details = self.get_user_details(username)                                                                                File "/usr/local/lib/python3.5/dist-packages/instagram_scraper/app.py", line 405, in get_user_details                                                                                raise ValueError('Error getting user details for {0}'.format(username))                                                                                ValueError: Error getting user details for

The user is either private or is not valid. Fixed handling of these two cases in v1.5.12 so the app does not crash.


Please add support for Swift Package Manager (Dependencies)

Many people will be moving away from CocoaPods and Carthage and using Swift Package Manager (Dependencies). Can you please add support? Thank you!

+1

Pull request here: https://github.com/SVProgressHUD/SVProgressHUD/pull/993

JK, The library depends on an internal Bundle (3 different images). I don't think that's currently possible to include into SPM. These images are very simple so the easiest solution may be to draw them directly anyway rather than using pngs.

Hi any update on this? I know another library with simply images that now draws them, and was able to implement SPM easily after that. With most of my library’s now supporting SPM it would be nice to get this one there as well. Thank you so much!

Xcode 12 and Swift 5.3 (both available as dev betas) now include binary support. Hopefully SPM can now be done for this?
https://www.hackingwithswift.com/articles/218/whats-new-in-swift-5-3

+1

+1


Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.
Please see the contribution instructions for more information.

@bors r+

:pushpin: Commit a6a065181bad2fbb1bb35ee5bcc4ec25c06d484a has been approved by Amanieu

:hourglass: Testing commit a6a065181bad2fbb1bb35ee5bcc4ec25c06d484a with merge d8da5cfcde4c7c67df6988341b8415095f040147...

:broken_heart: Test failed - checks-actions

I'll update this to drop support for the emscripten targets for now, but in parallel to that I've reached out to the emscripten folks on Discord to see if they'd be willing to stub out these functions without implementing the rest of sched_* (like OpenBSD does).

@bors try

@rtzoeller: :key: Insufficient privileges: not in try users

@bors r+

:pushpin: Commit b68bd77e3130115a5312d856ce881e0ec8533b0e has been approved by Amanieu

:hourglass: Testing commit b68bd77e3130115a5312d856ce881e0ec8533b0e with merge e1eb9721dc4534cea84e9e3bf591e7cb257e679c...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing e1eb9721dc4534cea84e9e3bf591e7cb257e679c to master...


Other reasons turning on ReplSeqMem is broken?

So I've been meaning to diagnose @bwrcbcr 's problem with Legalize, but on a simpler test circuit. The one I used was generated directly from Chisel [Trying to get OptionsManagers to work is another story...]:
;buildInfoPackage: chisel3, version: 3.1-SNAPSHOT, scalaVersion: 2.11.7, sbtVersion: 0.13.11, builtAtString: 2017-01-26 12:23:07.403, builtAtMillis:<PHONE_NUMBER>403
circuit CustomMemory :
  module CustomMemory :
    input clock : Clock
    input reset : UInt<1>
    output io : {flip rClk : Clock, flip rAddr : UInt<3>, dO : SInt<16>, flip wClk : Clock, flip wAddr : UInt<3>, flip wEn : UInt<1>, flip dI : SInt<16>}

    io is invalid
    io is invalid
    smem mem : SInt<16>[7] @[SRAMSpec.scala 42:19]
    read mport _T_17 = mem[io.rAddr], clock
    io.dO <= _T_17 @[SRAMSpec.scala 45:21]
    when io.wEn : @[SRAMSpec.scala 56:31]
      write mport _T_18 = mem[io.wAddr], clock
      _T_18 <= io.dI
      skip @[SRAMSpec.scala 56:31]

Only when I enable ReplSeqMem does the Verilog generation fail with
scala.MatchError: (SIntType(IntWidth(16)),UIntType(IntWidth(16))) (of class scala.Tuple2)
[info]   at firrtl.passes.InferWidths$.firrtl$passes$InferWidths$$get_constraints_t$1(InferWidths.scala:224)
[info]   at firrtl.passes.InferWidths$$anonfun$firrtl$passes$InferWidths$$get_constraints_s$1$1.apply(InferWidths.scala:265)
[info]   at firrtl.passes.InferWidths$$anonfun$firrtl$passes$InferWidths$$get_constraints_s$1$1.apply(InferWidths.scala:263)
[info]   at scala.collection.TraversableLike$$anonfun$flatMap$1.apply(TraversableLike.scala:252)
[info]   at scala.collection.TraversableLike$$anonfun$flatMap$1.apply(TraversableLike.scala:252)
[info]   at scala.collection.immutable.List.foreach(List.scala:381)
[info]   at scala.collection.TraversableLike$class.flatMap(TraversableLike.scala:252)
[info]   at scala.collection.immutable.List.flatMap(List.scala:344)
[info]   at firrtl.passes.InferWidths$.firrtl$passes$InferWidths$$get_constraints_s$1(InferWidths.scala:263)
[info]   at firrtl.passes.InferWidths$$anonfun$firrtl$passes$InferWidths$$get_constraints_s$1$6.apply(InferWidths.scala:292)

Aside from the extra io is invalid line and the underscore preceding letters weirdness, I don't think I see anything out of place for this CHIRRTL... (although I'm not fully cognizant right now...)...
Has anyone tried running the transform lately?

The underscores are just to force verilator to ignore those signals. I'd guess there is some bad interplay between the pass and SInts? I've been running it often with a very new firrtl/a51e72fefe1d33a1077e85cf04e1750dad6f8704 and a reasonably new chisel3/ad53161bbb9f67e16b88ca7a508a537f88d77e05 with no issues. But I dont use SInts in my memories

Found the bug. Working on a fix.

@azidar the memory pass does a lot of bit manipulation to chunk up memory contents correctly (i.e. for a Vec of SInts, all of the SInts get concatenated together as the data into the memory). In order for this to work with anything except UInts, you need to do an explicit cast back to SInt, Fixed, etc. for the read out data. Should something like this mess around with range inference if range inference occurs later? (Seems like InferWidths happens in Middle to Low Firrtl, which is after ReplSeqMem). Or is it ok, b/c we're assuming ranges are fixed/defined @ module boundaries? i.e. input range == output range. But that info needs to definitely make it past this pass (worth another thought!)


Unable to remap cc

It has always bothered me that using cc inside of a list, it clears the whole line including the list index -  or 1. . So I wrote my own JS snippet to clear the current line just after those - , 1. , > , - [x]  and whatsoever. It works as expected (tested in Obsidian interactively), but I'm having trouble mapping it in the .vimrc file.
Here's my .vimrc:
unmap cc
exmap clearLine jsfile mdHelpers.js { clearLine() }
nmap & :clearLine
nmap cc &a        " In order to enter insert mode after the line is cleared

It does not change the behaviour of cc. Yet the same mapping with some other key instead (i.e. gc) works perfectly.
Any idea how to fix that?

Seems like the CodeMirror Vim integration doesn't allow to map cc, among its many strange quirks :shrug:


need a way to link into graphics/os selected tabs

When creating links to the tutorial people will want to be able to link to OS / GFX specific pages.
For example
https://openxr-tutorial.com/1-introduction.html#environment-setup?os=Android&gfx=Vulkan would be acceptable, though a  more SEO friendly approach might be better.

Commits 4fdaa3a, dbfb508, 7377e1e and 36ee8d8 through e9c0876 add URL parameters. You can manually edit the URL or click the tabs to update the cookies, which saves the user's selection choices in the URL.

While this now works, I'm a bit dubious about it. We rewrite the address bar to add the parameters, but it's a bit ugly. @rbessems  how about we support the params but don't add them if they aren't there?

The solution we've implemented, there's a "share" button at the top-right of each page, which copies to the clipboard the current page URL, adding in the parameters if not already present.

Yes that is an option. It's a bit cumbersome and I'm not 100% sold on the paragraph symbols in every heading. Let's discuss tomorrow.
This also is making me wonder how well this text is discoverable by the Google spider. I'll open a separate ticket for that.

I agree, the permanent para symbols are ugly. I've made it so they are invisible until hovered.
Hidden text is available to all search spiders and not ranked negatively unless flagged as manipulative, which would not be correct in this case.
https://stackoverflow.com/questions/21997663/hidden-text-negative-for-google-spiders#21997688


*: Update to libp2p v0.33.0

Next to the general Cargo.toml updates this pull request also contains:

client/network: Consistently track request arrival time
With libp2p/rust-libp2p#1886 one is guaranteed
to receive either a ResponseSent or a InboundFailure event for each
received inbound request via RequestResponseEvent::Message. Given this
guarantee there is no need to track arrival times in a best-effort
manner and thus there is no need to use a LRU cache for arrival times.

client/offchain: Adjust to PeerId API changes

With https://github.com/libp2p/rust-libp2p/pull/1874 PeerIds are stack allocated and implement Copy. This makes all calls to clone on a PeerId superfluous. I don't think those calls to clone have a performance or correctness impact, please correct me if I am wrong. I can either remove the calls in this pull request or as a follow up pull request. I am also fine if we do the change gradually piggy-backed on other changes. This strategy would reduce the amount of noise in our git history.
//CC @romanb @tomaka

bot merge

bot merge


@code_warntype output printed out-of-order

In the Julia REPL, the tree is well-formed:

In IJulia, the tree is broken up:

The issue is the grey line annotations in IJulia show up on the left rather than the right, and that's reflected in the HTML:

Body::UnitRange{Int64}
│╻╷╷╷ Type5 1 ─ %1 = (Base.sle_int)(start, stop)::Bool
││╻    unitrange_last  │        (Base.sub_int)(stop, start)
│││┃    -  │   %3 = (Base.sub_int)(start, 1)::Int64
│││    │   %4 = (Base.ifelse)(%1, stop, %3)::Int64
││     │   %5 = %new(UnitRange{Int64}, start, %4)::UnitRange{Int64}
│      └──      return %5

I'm running OS X v10.14.1, Chrome v71, Julia v1.0.3, Jupyter Notebook v5.7.4, and IJulia v1.14.1. I also reproduced this on a Fedora machine at work and in JuliaBox.
e: Doing a bit of digging, it looks like the function doing the actual formatting for code_warntype is show_ir.  Can exercise that with
Base.IRShow.compute_ir_line_annotations(code)

e2: Reading a bit more, I think the problem's with IJulia's - Jupyter's? - handling of escape sequences. The warntype printer uses the \e[1G escape sequence to skip back to the beginning of the row after printing the annotation. In the REPL,
print("\e[7Gsecond"); print("\e[1Gfirst")

correctly prints first second, but IJulia prints second first

Thanks, figured that might be the case. I'll ask around and see if either of those are easy fixes (hah, not a chance).


terra-date-picker momentjs version doesn't support es-us

Issue Description
The minimum required version of momentjs of terra-date-picker, 2.17.1, doesn't support the local es-US. This leads to it giving a placeholder of DD/MM/YYYY on dates. Momentjs 2.19 added support for this locale which fixes this issue. If anyone has another package with an older version of moment preferred, this bug will pop up.

Expected Behavior

Steps to Reproduce

Install terra-date-picker with moment < 2.19 in a test project.
Set the locale for a test application to es-US.
Datepicker will enforce DD/MM/YYYY formatting.

Environment

Component Version:
Browser Name and Version: locale es-US, moment <2.19

This has been released in<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>


Patched values update of controlled fields result into obsolete state

Environment

react-advanaced-form: 1.5.0

Prerequisites
Setup a simple form where all fields are controlled. Create a handler that updates the state with another values.
What
Current behavior
Next values are propagated to the components and are displayed correctly in the UI, but their counterpart in field state is obsolete.
Expected behavior
Field value updates (regardless of controlled, or uncontrolled fields) are propagated properly into field state.
Why
#268

This should be partially solved by #352 (introducing of patched state updates). Concurrency on a field level will be eliminated, yet the buggy dispatch nature of controlled fields onChange handler will remain due to being invoked in componentWillReceveiProps.
The last part to fix this issue would be #324.

This issue relates to #268 and will be solved by it. No need to duplicate.


Support no-std and alloc-only

Based on changes from #33 extended to support alloc-less no-std as well.

Supporting encode without alloc requires implementing the APIs mentioned in #21.

bors r+


local or offline testing

I've been using awsmobile for a couple of months now. Using services cloud logic, cognito, dynamodb and appsync. As my code base become bigger, it's been taking a long time for me to push my changes only to find out that it's not working as intended.
Is there a way for me be able to test my codes locally specially cloud-logic and appsync without pushing my changes?

For now it's not possible.
I will mark this as a feature request.

+1 for now I'll have a look at SAM and see if it works.


How to get and set values in ComponentOne Flex Grid?

Hi, We have a ComponentOne Flex Grid (C1.Win.C1FlexGrid.C1FlexGrid) in our .Net Desktop application. inspect.exe tool is able to identify the grid but not the internal rows and columns. The rows has cells containing text boxes and list boxes.
How can we set and get values in row's edit box and combo boxes using WinAppDriver? Tried lot of google but not able to find anything. We are able to work on same grid using UFT (MicroFocus Automation Tool). It provides methods like SetCellData and GetCellData to work on Cells of Grid. Is there similar solution we have using WinAppDriver ?

If inspect is not able to identify the elements then win apps driver will not work


Features of brackets code editor

Please give the main feature of brackets code editor like Inline Editing,Live Preview,Preprocessor Support.I know there are some extensions for this king of work but there are not as good as brackets and these feature are really useful.so please think about it.these features will really help the web designers.thank you very much.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Devloper-King sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Split main.bundle.js

Is there a way to split main.bundle.js in for example 4 files with predefined names?
For example:

main.bundle.1.js
main.bundle.2.js
main.bundle.3.js
main.bundle.4.js

@shlomiassaf can you pls check this?


upgrade npm packages

Codecov Report

Merging #334 into master will increase coverage by 0.61%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #334      +/-   ##
========================================
+ Coverage    2.59%   3.2%   +0.61%
========================================
  Files          52     52
  Lines        1349   1092     -257
  Branches      187    182       -5
========================================
  Hits           35     35
+ Misses       1313   1056     -257
  Partials        1      1

Impacted Files
Coverage Δ

...oder-problems-frontend/src/pages/StreakRanking.tsx
0% <0%> (ø)
:arrow_up:

...coder-problems-frontend/src/pages/ShortRanking.tsx
0% <0%> (ø)
:arrow_up:

...coder-problems-frontend/src/pages/FirstRanking.tsx
0% <0%> (ø)
:arrow_up:

...ems-frontend/src/pages/UserPage/SubmissionList.tsx
0% <0%> (ø)
:arrow_up:

...er-problems-frontend/src/pages/TablePage/index.tsx
0% <0%> (ø)
:arrow_up:

...der-problems-frontend/src/pages/FastestRanking.tsx
0% <0%> (ø)
:arrow_up:

atcoder-problems-frontend/src/pages/SumRanking.tsx
0% <0%> (ø)
:arrow_up:

...problems-frontend/src/pages/TablePage/TableTab.tsx
0% <0%> (ø)
:arrow_up:

atcoder-problems-frontend/src/reducers.ts
0% <0%> (ø)
:arrow_up:

atcoder-problems-frontend/src/App.tsx
0% <0%> (ø)
:arrow_up:

... and 29 more

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 c1e2ca4...9a76666. Read the comment docs.


The height of header bar in the android platform is being neglected

When I am testing on the android platform when using the ionic framework, the height of the header bar is being neglected when I am trying the demo like this:
  <walkthrough
        id="Demo2"
        walkthrough-type="transparency"
        focus-element-id="focus-item"
        icon="arrow"
        main-caption="{{demoCaption2}}"
        icon-padding-left="30"
        icon-padding-top="0"
        force-caption-location="TOP"
        is-active="demoActive2"
        use-button="true">
  </walkthrough>

I have created a pull request for you just in case.
https://github.com/souly1/ng-walkthrough/pull/28

This solution in the pull is ok, but I'd rather not add another property. I've checked in a new version (https://github.com/souly1/ng-walkthrough/releases/tag/v0.4.2) which should be more automatic and generic, and also solving the server call for icons. It would be great if you could check it out in your code and update if issue solved.
Thanks

Sorry for the late reply. I tried the version 0.4.2 and everything works fine. Thanks.


Discrepancy Between Local and Online Transcription Results in Transkun

Hello, I am a music industry professional with a keen interest in programming. The model you trained has been incredibly helpful to me in music transcription. Thank you so much for your effort.
I have a question. Previously, I used this online platform(https://colab.research.google.com/drive/1Eo_cfCfPVPFpQANuxQBQS6T68kc-tNXD?usp=sharing#scrollTo=O5oJ2O0gSfGa) to transcribe audio files into MIDI. However, I recently encountered a quota limit (haha). So, I visited GitHub, downloaded the package you provided, and deployed Transkun locally (version 2.0.1, I believe).
I noticed that the transcription results obtained locally are not as good as those from the online platform (though still better than other models on the market). I was wondering, is the package you released on GitHub identical to the one used on the online platform? What might be causing the difference in results? Thank you!

The latest version of the PIP package and this github repo are synchronized. Can you be more specific about what kind of "discrepancy" it is?
Do you mean note duration extension?: Compared with the previous release (v0.1.4), there is a significant behavior change: the default checkpoint's behavior, now, is not to extend note durations according to pedal activations. If transcribing notes with pedal extension is the desired behavior, use "Transkun V2 Aug" as in Model Cards.

To make it clearer, I’ve attached images to illustrate. Image 1 shows the results obtained using the online model on Google, while Image 2 shows the results from running the model locally. Although I can’t directly articulate the differences between the two, based on auditory perception, the online model’s results are superior in several aspects: the completeness of the notes (no omissions), the accuracy of rhythm, the precision in handling long and short notes, and the decision-making on pedal usage. Overall, the online model provides better results in terms of listening experience.
I have tested the local model multiple times to avoid errors from a single run, but the results consistently fall short compared to the online model.
So, I was wondering if the difference in results might be due to the two using different models. The online model is shown as follows:

Could you provide the audio file and the content of "PianoTrans/mp3_to_midi_gui.py" so that I can investigate what's happening when you call it locally?


Set labels changed files output

This adds functionality to include the filepath, fieldpath, and
value for each set-label in the output. This behavior is
modeled after the behavior of set-annotations.
An example output:
$ kpt fn render set-labels-advanced
Package "set-labels-advanced":
[RUNNING] "gcr.io/kpt-fn/set-labels:unstable"
[PASS] "gcr.io/kpt-fn/set-labels:unstable" in 3.2s
  Results:
    [info] metadata.labels: set labels: {"color":"orange","fruit":"apple"}
    [info] metadata.labels: set labels: {"color":"orange","fruit":"apple"}
    [info] spec.selector.labels: set labels: {"color":"orange","fruit":"apple"}

Successfully executed 1 function(s) in 1 package(s).

This change is based on a change to bump the version of kustomize consumed by set-labels. That change has been proposed as a separate pull request here: https://github.com/GoogleContainerTools/kpt-functions-catalog/pull/728
Issues: GoogleContainerTools/kpt#2448

This needs rebase

This needs rebase

Thank you, done


Add 'attack' folder and components

Part two of: zaproxy/zaproxy#5972
I propose that his be reviewed/merged, then the repo renamed. Then another PR be raised to address updating URLs, etc (changelog, build kts files, etc).
Also see: https://github.com/zaproxy/zap-extensions/pull/2465
Signed-off-by: kingthorin<EMAIL_ADDRESS>

This addition PR contains a bunch more files than the removal PR because there was a bunch of help and build files to update.

I'd suggest updating the links now so that when the repo is renamed everything is as expected already.

Okay, will tackle that shortly

Done/added

Looks good, may I suggest update the attack files and backdoors to the latest from upstream?

Sure

Thank you!


ADIOS2 data collection leaves out element attributes

Guess this is partially my fault for not noticing this while reviewing the original ADIOS2 PR but I just noticed that the data collection doesn't save off the element attributes. I'm starting to need that value as part of some my post-processing to associate certain data with a given attribute. So, it would be nice if this could be added into that data collection.
In the mean time, I've got a temporary fix where I should just be able to create a GridFunction using an L2 FiniteElementSpace and just save the element attributes off that way.

@williamfgc can you take a look at this when you get a chance?

So, I've been playing around this data collection a bit more, and my temporary fix worked quite well. Also, it appears to me that there could be a potential storage space savings to be had if you have a gridfunction with an L2 FiniteElementSpace of order 0.  Then you could save the data off using the element tag in VTK xml scheme rather than saving off number of element nodes  times as much data associated with point data. Depending on how many variables you're saving off like this, you could see a pretty nice space savings from such an optimization especially once you start using higher order elements or just really large meshes.

@tzanio @rcarson3 apologies for the late reply, I'll set up a meeting with @rcarson3 to work on this. Thanks for the feedback!

@rcarson3 I see the ParaviewCollection is assigning the element attribute always as "material". Is this the preferred name rather than the more generic "element attribute"? Please let me know, so I can add it to the schema like this:
  <CellData>
        <DataArray Name="material" />
  </CellData>

@williamfgc if that's what the ParaviewCollection is using it might not be a bad thing to keep that consistent across the data collections as long as it's documented that the mesh element attribute = material output variable name.

@rcarson3 @williamfgc Do you need any help from the ADIOS2 team to progress on this issue? Is #1704 fixing your issue?

@pnorbert sorry for the delayed response. So #1704 partially addresses the above issues. It does add the element attribute to the output. However, it appears that the current ADIOS2 reader in ParaView v5.8.1 does not recognize the outputted cell data as something that can be visualized.
The other concern I had regarding space savings for gridfunctions of an L2 finite element space with an order 0 (essentially element/cell data rather than point data but is currently outputted as point data) is also not addressed in that PR. However, you would need to ask @williamfgc to see if he'll need help from the ADIOS2 to implement this feature into MFEM.
Also on another note, I didn't notice this until I started deforming my mesh quite a bit, but the mesh is constant through out time, which doesn't follow with the rest of the data collections which output the mesh at each time step. I'd imagine if this was updated then the ADIOS2 collection could also support AMR meshes as well. Now if you have the displacement field outputted then this is fine because you can just use the warp filter with the displacement field to get out the current mesh. However, I don't believe there is any documentation that states this which should probably be added in either #1704 or a future PR with ADIOS2.

@pnorbert just wanted to ping you to see if you'd had a chance to look over my last comment about this issue.

Hi, I thought @williamfgc addressed your issues with #1757

@pnorbert so #1757 addressed the element attribute part of the issue. However, the space-saving optimization wasn't addressed in that PR, and it'd be nice to have that one addressed in-order to make use of ADIOS2 with large simulations with tens of millions of elements and saving off any appreciable number of time steps and state variables/fields.

@rcarson3 let's make a separate PR as the work is very different and required cell centered visualization support in VTK which is work in progress. Let me know what you think. There is also the moving mesh issue.

@williamfgc a separate PR or PRs is fine. I just wanted to make sure this doesn't end up getting forgotten about now that all of the end of the FY madness has calmed down.

@rcarson3 thanks, I just want to make sure at least one piece is done. Yes, it's been madness. We also have a related issue related to linking adios2 v2.6.0 and VTK. Thanks for pushing this, it won't be forgotten from my side :)

@rcarson3 I see the ParaviewCollection is assigning the element attribute always as "material". Is this the preferred name rather than the more generic "element attribute"?

@williamfgc if that's what the ParaviewCollection is using it might not be a bad thing to keep that consistent across the data collections as long as it's documented that the mesh element attribute = material output variable name.

The PrintVTU function (but not yet ParaviewCollection) is now using attribute instead of material as discussed here: https://github.com/mfem/mfem/pull/1825#issuecomment-713868907

@benzwick thanks for pointing it out! I just did a PR. @rcarson3 I'll be doing a PR on ParaView in the next few days for cell data visualization, I'll let you know when it's merged. Thanks!

@rcarson3 just following up on ParaView support I did a merge request related to this issue. I'll keep you posted for when this is merged.

It took a while, but the VTK/#7401 got merged. Hopefully, this is soon available on ParaView nightly :)


Mac OS detected as windows

I am running Mac OS 10.12.6
My hardware is: MacBook Pro (Retina, 13-inch, Mid 2014)
Running the dump example I get this:
I think the host OS is Windows..
If I'm wrong Please submit a GitHub issue with the folowng data at:
https://githb.com/keyboaio/FingerprintUSBHost/issues
Be sure to include your OS and OS version.
80,6,1,0,0,8
80,6,1,0,0,12
80,6,3,2,409,2
80,6,3,2,409,12
80,6,3,1,409,2
80,,3,1,409,16
80,6,3,3,409,2
80,6,3,3,409,10
80,6,3,0,0,FF
Related to this thread:
https://community.keyboard.io/t/mapping-international-characters/556/11

I don't have a Mac to check if MacOSX works.
My tests have been run on the following OS and I can say that it is working:

Windows 10: Detects Windows
Windows 7: Detects Windows
Ubuntu: Detects Linux
Debian: Detect Linux
Kali Linux: Detects OS Linux

MacOs Mojave
I think the host OS is Windows.

If I'm wrong Please submit a GitHub issue with the following data at:
https://github.com/keyboardio/FingerprintUSBHost/issues
Be sure to include your OS and OS version.

80,6,1,0,0,8
80,6,1,0,0,12
80,6,3,2,409,2
80,6,3,2,409,1C
80,6,3,1,409,2
80,6,3,1,409,18
80,6,3,3,409,2
80,6,3,3,409,E
80,6,3,0,0,FF


Fix for navbar style #213

Fixed the height of navbar and underline under text

Navbar view before the changes were applied:

Navbar after style changes are applied:


Fix the new line replacement

When the config.yaml already contains a combination of characters "\n", for example when doing a "sed" inside the file itself, or using a regex, the execution of the script is broken.
This fix solves this problem

Hi @rdiezsj I just tried running your changes on a test config that I have, and it seems to break the validity of the yaml