chore: add configuration for fuzzy metrics

Signed-off-by: Nandita Koppisetty<EMAIL_ADDRESS>Explain what this PR does.

Codecov Report

Merging #54 (f470743) into main (db8b839) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #54   +/-   ##
=======================================
  Coverage   87.16%   87.16%
=======================================
  Files          16       16
  Lines         709      709
  Branches       68       68
=======================================
  Hits          618      618
  Misses         69       69
  Partials       22       22

Impacted Files
Coverage Δ

numaprom/_constants.py
100.00% <ø> (ø)

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


Optional output directory and handling of absolute file paths

Fixes #113 and #114.

Codecov Report

Merging #115 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   96.29%   96.28%   -0.01%
==========================================
  Files           8        8
  Lines        2670     2668       -2
  Branches      418      418
==========================================
- Hits         2571     2569       -2
  Misses         50       50
  Partials       49       49

Impacted Files
Coverage Δ

rflx/cli.py
100% <100%> (ø)
:arrow_up:

rflx/parser.py
96.56% <100%> (-0.02%)
:arrow_down:

Continue to review full report at Codecov.

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


CTRL+P and CTRL+N not working

These keys are not working in the find it faster terminal. Although when running fzf directly it is working.
Kindly let me know the cause for it and how it can be fixed.
I tried running the fzf command in the vscode terminal and the terminal that is left by FindItFaster in the editor both places CTRL+P and CTRL+N are working. But when opening the FindItFaster using the keyboard shortcuts and then trying these keys dont work.
Thanks
@tomrijndorp

Hi, the VS Code terminal is a regular terminal so things should work the
same, but the VS Code might capture key bindings such as Ctrl+P (such as
for showing the internal file picker) before giving them to the terminal.
The FAQ of the extension should give you some pointers on how to change
this (there’s an example on Ctrl+J/K but the same principle applies).
On Thu, 19 Sep 2024 at 20:48 Shivam Rawat @.***> wrote:

These keys are not working in the find it faster terminal. Although when
running fzf directly it is working.
Kindly let me know the cause for it and how it can be fixed.
Thanks
@tomrijndorp https://github.com/tomrijndorp
—
Reply to this email directly, view it on GitHub
https://github.com/tomrijndorp/vscode-finditfaster/issues/112, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADYSIG7LOXKRNEBSDPYTLPTZXOLJFAVCNFSM6AAAAABORF2RE2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZTONZYGQ2DENI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Attaching a video.
Currently i am using ctrl k/j for now and disabled the shortcuts for them.
https://github.com/user-attachments/assets/38c7fc0c-cbfe-4213-b5e1-da8293abb780
Notice that ctrl+P / N are not working when i am opening the terminal using shortcut. But after it is opened and i press esc to cancel search and open the fuzzy finder using the fzf command then it is working fine.
Also shown that ctrl + K / J worked in the starting.

@ShivamRawat0l
Apparently --history option was preventing us from using ctrl+p/n. It should reproduce using fzf --history xxx directly from your terminal.
https://github.com/tomrijndorp/vscode-finditfaster/blob/abd30e7f0d45b02d48b0863afc9a83bf0b1a7b9c/find_files.sh#L55
You can add a config to your zshrc/bashrc to fix this issue:
export FZF_DEFAULT_OPTS="--bind ctrl-n:down,ctrl-p:up"


[BUG] Extreme latency in BulkIndexer

What is the bug?
It seems that with a BulkIndexer with 2 workers, I am getting unexpected latency on BulkIndexer.Add(). It seems that somehow the workers are not consuming the queue within any reasonable sort of timeframe, I'm seeing delays of over 20s!
For example, in the last hour I've 53 cases of >1s latency on just Add() out of a total of 174 calls.
How can one reproduce the bug?
With 2 workers running, adding items from different goroutines and a relatively busy search cluster.
What is the expected behavior?
Sub-millisecond latencies, basically the time it takes to shove something into a channel.
What is your host/environment?

OS: Ubuntu 20.02
Version: 1.1.0 (but nothing has changed to the bulkindexer since the fork from ES)

Do you have any screenshots?

Note; returning to the default of numCPU workers seems to alleviate the issue, but given the significant delays (seconds versus sub-millisecond) I would still strongly argue that there is an underlying issue here.

@VijayanB @VachaShah Any ideas?

Poke!

@dokterbob This looks visible problematic, but doesn't look like the folks here got to looking into it. Let's try to move this forward? First, what's the easiest way to reproduce this? Are you able to bulk load data a lot faster into this instance with other mechanisms (aka is this a client issue for sure)?

@dokterbob Do you mind posting some of the code you were using to help pinpoint this issue?

Sorry, I didn't see the messages. Code is https://github.com/ipfs-search/ipfs-search/ but of course you'll need a more detailed test case.
After increasing the workers it seems the problem has become less severe. Now that it's been picked up I'll see if I can get more concrete feedback over the next couple of weeks.

@dokterbob hey, I want to work on solving the issue, how relevant is this?
Are there any problems now, or was it on the client side?


License?

I noticed there isn't a license associated with this code - at least not clearly. Is it possible to add a LICENSE to the root of the repository or at least the readme to indicate usage permissions?

added MIT License file to the root.
thanks.


[FEATURE] Enhance documents

Is your feature request related to a problem? Please describe.
We need more documents:

[ ] Description of how to use bpftime for uprobe, syscall tracing
[ ] The details of how bpftime works

Describe the solution you'd like
Add documents in eunomia.dev
Describe alternatives you've considered

Provide usage examples

Additional context

Hey @yunwei37,
I would like to contribute to this issue as my first contribution in your org.
So, can you please assign it to me to get the work done?
Thanks.

Thanks!

The documents have been moved to https://github.com/eunomia-bpf/eunomia.dev/blob/main/docs/bpftime
So please update the documents there

Maybe you can first update the documents for bpftimetool?
https://github.com/eunomia-bpf/bpftime/tree/master/tools/bpftimetool

Sure, looking into it,..

Maybe you can first update the documents for bpftimetool?
https://github.com/eunomia-bpf/bpftime/tree/master/tools/bpftimetool
You can create a new markdown file in the eunomia.dev repo, and updated the https://github.com/eunomia-bpf/eunomia.dev/blob/main/mkdocs.yaml for that.
The website will be updated automatically once the PR is merged.

Can you provide what in the documentation should be updated?

You can create a new markdown file like https://github.com/eunomia-bpf/eunomia.dev/blob/main/docs/bpftime/documents/co-re.md in https://github.com/eunomia-bpf/eunomia.dev/blob/main/docs/bpftime/documents/, add details about bpftimetool, and update the https://github.com/eunomia-bpf/eunomia.dev/blob/main/mkdocs.yaml for the new doc.


Add support for local slack install with nginx

This PR builds on @franciclo's work in #361 to support login locally using localtunnel. I choose to use that instead of ngrok because it doesn't require an account.
However, if we encounter any issues with it, we could switch to ngrok.
EDIT: This PR now uses ngrok because I found localtunnel to be unstable. It does require a free ngrok account.
You can manually put in a requested subdomain, and then that will be used to create your public account which you can then create a slack app for.
I tested this and it seems to work. Note that I tried it at first under a slack workspace where I was not an admin and it failed, so make sure to try it with an admin workspace.

If anyone who has Docker installed could try this out locally to make sure it works for them as well that would be much appreciated!


bug: Modal always appears above Drawer

What version of daisyUI are you using?
4.12.13
Which browsers are you seeing the problem on?
Chrome
Reproduction URL
https://play.tailwindcss.com/UEJsJgvLZ0
Describe your issue
I tried to set z-index, but it doesn't work.

HTML <dialog> element is a top-layer element. it means it's outside of <html> tag so you can't put things from <html> tag on top of it.
To show a drawer sidebar on top of the modal you can use the checkbox method of modal:
https://daisyui.com/components/modal/#method-2-using-a-hidden-checkbox-legacy
Example: https://play.tailwindcss.com/YbG3CtMXVz
Or you can use 2 <dialog> modals, one of them placed on the side (looks like a sidebar)
https://play.tailwindcss.com/DX35O1XGxk
But I would't advice doing this in the first place. It's not a good UX. Modals, popovers, dropdown menus, etc. are unstable parts of design. They get open because of an interaction and get closed with another interaction. It's not a good idea to nest them inside each other because when multiple unstable sections are open inside each other, their behavior would be unpredictable for the user.
Instead, I would suggest using tabs to conditionally show different parts or restructure your UI in a way that wouldn't require opening modals and drawers and dropdowns inside each other.
Let me know if you have a question.


Add support for setxor1d

This is the implementation for np.setxor1d from #70 (and #2078 ).
Including this and #5664, all the NumPy set routines listed in #2078 are implemented.

Hi - are you interested in continuing to work on this? Thanks!

Yes, I still am.
Will try to finish it by this week.

Since we decided to explicitly flatten the input arrays to match setdiff1d, union1d Jax implementations, I faced problems similar to https://github.com/google/jax/pull/4635#discussion_r509419926 as the behavior of jax setxor1d is no longer guaranteed to match np.setxor1d (_CheckAgainstNumpy assertions fails for few test cases).
So following setdiff1d, now unique() is called irrespective of assume_uniqe value.

Thanks - I'm not certain I understand the problem re: uniqueness and raveling. What is an example of a case that fails?

Test cases with - non-1d shapes and assume_unique=True fails ( np.setxor1d does not support non-1d with assume_unique=True, https://github.com/google/jax/pull/5665#discussion_r571649983)
[  FAILED  ] LaxBackedNumpyTests.testSetxor1d_float16[3,1]_int16[1,4]_assume_unique=True
[  FAILED  ] LaxBackedNumpyTests.testSetxor1d_int16[1,4]_int8[2,3,4]_assume_unique=True
[  FAILED  ] LaxBackedNumpyTests.testSetxor1d_int32[4]_int16[3,4]_assume_unique=True
[  FAILED  ] LaxBackedNumpyTests.testSetxor1d_int32[4]_int8[1,4]_assume_unique=True
[  FAILED  ] LaxBackedNumpyTests.testSetxor1d_int8[4]_int16[1,4]_assume_unique=True
[  FAILED  ] LaxBackedNumpyTests.testSetxor1d_int8[4]_int8[3,1]_assume_unique=True

On the second thought, we can just flatten arrays before passing them to np.setxor1d when assume_unique=True in tests. This way it will match with jax setxor1d.

I see, thanks! Agreed that pre-flattening arrays (with a comment indicating why) is probably the best.
It might also be worth adding a lax_description to the _wraps to mention this difference in behavior.

comment and lax_description added 👍


Different PCs with same IP - voting duplication

Hi, I'm playing with my girlfriend on two different PCs with same public IP and we investigated that our votes aren't unique. After discover some merchant by us, the other has vote on this merchant, same with earlier discovered merchant.
It's possible to fix it and make our PCs unique?

Hmmm, possibly could make this account-based, if logged in. Would have to make some tweaks to the way it tracks votes behind the scenes though.

Hmmm, possibly could make this account-based, if logged in. Would have to make some tweaks to the way it tracks votes behind the scenes though.

We're both logged in into our discord accounts.

Should in theory allow tracking votes by account as long as you're each logged in. May act a little wonky if one of you is ever logged out, but basically similar to the old behavior of matching by IP.


Add: shx dependency for cross platform commands

Added shx so that you can now develop on Windows as well.
Solves #12

Maybe I should have requested to merge into the dev branch? I hope it doesn't cause any problems.

@AWulkan Thank you!


Won't install on Cygwin

I'm trying to install on windows 10 using cygwin. I've tried 2 methods:

The "official" one, at https://cpputest.github.io/, which errors error: AC_CONFIG_MACRO_DIRS([m4]) conflicts with ACLOCAL_AMFLAGS=-I m4 when I run autoreconf .. -i (which, if I've understood correctly, should have been fixed)
Following this set of unofficial instructions, which becomes useless when running make says *** No targets specified and no makefile found.  Stop..

Any solutions?

I did a quick google for you related to (1). Could you check this: https://stackoverflow.com/questions/47582762/ac-config-macro-dirsbuild-aux-m4-conflicts-with-aclocal-amflags-i-build-aux

Running dos2unix Makefile.am in the cpputest directory fixed it, now it complains that 'configure.ac' or 'configure.in' is required, even though there's a configure.ac right besides it.

Do the same ? It seems your line endings are messed up... but not understand why that is.

Can you run the autoreconf from the directory that the files are instead of the ..

worked, thanks!


Feature req: Option to open tabs as normal when opening bookmark or hitting + button

If possible, it'd be nice to have an option to open tabs from the bookmark toolbar normally when middle clicking, same with the + on the tab bar.

Unfortunately there is no API for knowing how a tab was opened, so it's not possible.


[MRESOLVER-109] - AndDependencySelector should override toString

https://issues.apache.org/jira/browse/MRESOLVER-109
Intentionally omitted tests. These are just for designed debugging and tests would only make it harder to change them. I did test the methods, just not with unit tests

Well, if you insist. Added

waiting for jenkins: https://ci-builds.apache.org/job/Maven/job/maven-box/job/maven-resolver/job/69/

Iterator is a better solution. I've changed it, thanks.
Java 7 means no StringJoiner, String.join or streams. Other resolver modules include lang3, which has StringUtils with a join method, but this module doesn't. So using a list was already my 5th choice.
I don't see the value in pre-calculating it. You shouldn't add complexity to a class (i.e. extra fields) for something that's only intended for debugging. As a performance optimization it's also ineffective because you'd be forcing most users to eagerly compute a value they don't use, actually making performance slightly (though insignificantly) worse.

I deferred to copying Lombok's style, which is the closest thing I could think of to a standard. Most people are happy with the output of that.
I checked quite a few of the other implementations in the project, there didn't seem to be any particular precedent to follow.


Add devise

In this PR I :

Add the devise gem to my app and use it for the registration and login of users.

Project Status: INVALIDATED :red_circle:
Hi @Patfarmurs  ,
While you made a great effort in this project, I can't go ahead and review your code.

Invalid Code Review Request

This is because you have submitted a project that is a pair-programming activity and as a result needs to have commits from both you and your partner or partners. 💯
However, if for some reason you have to do the project alone, then attach a screenshot of the message you received from student success stating that you can do the project alone in the pull request description 👍

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @NahnahAJ in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Thank you @vickymarz


local.md updated for mac os support for mongodb setup

Fixes #518
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Updated local.md for local installations  of mongodb setup on mac

Issue# is missing.

@vkartik97 updated. The issue was raised by me itself, as i was blocked in mongodb installation while installing on mac. So, i raised #518  issue


Existing file gets overwritten on first creation of a branch

https://bg.door43.org/Elsy-org-test/ru_ta already has the majority of the files in Russian

Choose Elsy-org-test as your org
Choose TA
Choose Russian
Open a md file like /translate/figs-metaphor/01.md
The Russian will be overwritten in English in your branch.

In other words, when the branch gets created, an English file is copied to the branch even if the master contains the file.

@birchamp I can replicate the issue. After I deleted my branch, opened ru_ta for figs-ellipsis(for this file a Russian translation exists), it showed English on Target side

Added #223 for tN issue related to this same issue.

Verified on translationCore Create - v1.0.0-rc.2. The issue is fixed with existing file getting overwritten on first creation of branch. Able to see the language translation on the Target side.


Aligning with Bash style guide

Changed variable names to avoid camel casing

@sreekzz
IMPORTANT: 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

@sreekzz
IMPORTANT: 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


Some initial work on updating for 1.99

More to come :)
Will probably need to be squashed into one merge commit or something.

Hang on, there's a problem with my email in one of the commits, let me see if I can fix it.


value in all inputs never receives the value of the field

If I use the register command in hidden fields the value is never updated, so when the form is submitted, all values are empty?

I am using the latest Remix, React, react-hook-form, remix-hook-form, I have tried using fetcher.Form and remix Form, still the values are empty in the html?

OK I'll close this, it looks as though its working after forcing an update on react-hook-form? Must have been an issue with remix-h-f and react-h-f, as I was using react-h-f prior to trying remix-h-f, shall keep testing, thank you

Look like I need to reopen this issue, if I change a value in a form, then the form is rendering, and the values in the hidden fields are cleared?
A bit more I am using MUI controls, with controlled inputs


Tag version on github

Is it possible to tag the relevant git commits when a new version releases?
This would greatly improve Arch linux builds, as I would be able to indicate exactly the version I want when building.

I'll take a look at the Git commit tag field when I do the next release and you can tell me if it works.

Yes, pretty please. It will make automated building so much easier, as many hooks are built to check out release tags for deploy versus commit tags for test builds.

+1

Guess I should close it now?
@gridcoin please continue tagging future versions!


QObject::connect error in debug log.

Describe the issue
I get this error in my debug.log file.  I am not sure how to reproduce the error.

GUI: QObject::connect: No such signal QLabel::clicked() in qt/silkgui.cpp:241
GUI: QObject::connect:  (receiver name: 'RPCConsole')

Steps to reproduce

Not sure.

What platform are you using (Linux, Windows, Mac)
Ubuntu Linux 16.04 64 bit.  Built using Qt5

I cannot replicate on OSX.


Support ImageContent as parameters of methods in AiServices

Sending images in regular chat API
Since the last update of GPT Turbo, the vision API has been merged with the "regular" API.
So a user can send images to the regular gpt-4-turbo model.
One interresting use case with LLM is the ability to extract informations from images. Count the number of cars in this image, read the signs,  etc. So, basically, it's the ability to put images in and get back structured data. This is already possible with the OpenAI API:
 {
        "model": "gpt-4-turbo",
        "messages": [
            {
                "role": "user",
                "content": [
                    {
                        "type": "text",
                        "text": "Count the cars in this image. Respond using JSON like this: { \"numberOfCars\": 3 }."
                    },
                    {
                        "type": "image_url",
                        "image_url": {
                            "url": "data:image/jpeg;base64,{image_data_in_b64}"
                        }
                    }
                ]
            }
        ],
        "max_tokens": 1000,
        "response_format": { "type": "json_object" }
}

Using ImageContent in AiService
Currently, using ImageContent in AiService is not supported.
One could imagine having this kind of AiService:
record CardDescription(int numberOfCars){}

interface CarCountingService {
    @UserMessage("Count the number of cars in this image")
    CarDescription countTheNumberOfCars(ImageContent image);
}

Of course, you could image all kind of combinations:
record CardDescription(int numberOfCars){}

interface CarCountingService {
    @UserMessage("Count the number of cars of the brand {{brand}} in this image")
    CarDescription countTheNumberOfCars(ImageContent image, @V("brand") String brand);
}

Alternative : using tools to specify structure using JsonSchema
I've noticed that when structured data are becoming complex, the model doesn't manage to comply with the example.
What I did in my code was using the tools feature just to be able specify the correct format using JsonSchema, and then pass this format to an input of a function.
So I would pass a function like this:
{
    "model": "gpt-4-turbo",
    "messages": [
        {
            "role": "user",
            "content": [
                {
                    "type": "text",
                    "text": "Count the cars in this image."
                },
                {
                    "type": "image_url",
                    "image_url": {
                        "url": "data:image/jpeg;base64,{image_data_in_b64}"
                    }
                }
            ]
        }
    ],
    "max_tokens": 1000,
    "response_format": {
        "type": "json_object"
    },
    "tools": [
        {
            "type": "function",
            "function": {
                "name": "count_cars",
                "description": "Count the cars.",
                "parameters": {
                    "type": "object",
                    "properties": {
                        "cars": {
                            "type": "array",
                            "items": {
                                "type": "object",
                                "properties": {
                                    "make": {
                                        "type": "string"
                                    },
                                    "model": {
                                        "type": "string"
                                    },
                                    "color": {
                                        "type": "string"
                                    },
                                    "count": {
                                        "type": "string"
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    ]
}

Implementation direction
I was wondering if giving the user the possibility to specify ImageContent as parameters of AiService was the right way to go.
What do you guys think ?

@mmacphail this is what I was thinking about as well, there is https://github.com/langchain4j/langchain4j/issues/938 already, so I will close this issue, please comment in https://github.com/langchain4j/langchain4j/issues/938


How to have multiple colours for the same Treesitter value?

I am not sure, if this is the place to ask for this but; is it possible to have different colours depending not only on Treesitter, but on both Treesitter & Syntax?
Here is a sample code from Ruby on Rails project:
scope :ok_world, -> { where(ok_world: true) }

def hello
  puts "Hello world!"
end

Treesitter Enabled
Treesitter Disabled

Playground data
call [266, 2] - [266, 41]
  method: identifier [266, 2] - [266, 7]
  arguments: argument_list [266, 8] - [266, 41]
    simple_symbol [266, 8] - [266, 14]
    lambda [266, 16] - [266, 41]
      body: block [266, 19] - [266, 41]
        call [266, 21] - [266, 39]
          method: identifier [266, 21] - [266, 26]
          arguments: argument_list [266, 26] - [266, 39]
            pair [266, 27] - [266, 38]
              key: hash_key_symbol [266, 27] - [266, 32]
              value: true [266, 34] - [266, 38]
method [268, 2] - [270, 5]
  name: identifier [268, 6] - [268, 11]
  call [269, 4] - [269, 23]
    method: identifier [269, 4] - [269, 8]
    arguments: argument_list [269, 9] - [269, 23]
      string [269, 9] - [269, 23]
        string_content [269, 10] - [269, 22]

scope
hello

As shown above, Treesitter thinks both scope and hello is a TSFunction –which is technically true–, but they are not only a TSFunction:

scope is a TSFunction and rubyMacro.
hello is a TSFunction and rubyMethodName.

I hoped maybe lua/solarized/solarized-normal/highlights.lua file is similar to how CSS works and assigning Ruby specific colours after TSFunction like
syntax['TSFunction'] = syntax['Function']
syntax['rubyClassBlock'] = {fg=colors.red}
syntax['rubyMacro'] = {fg=colors.red}

can change how words are highlighted, but that did not work.
So, is it possible to override Treesitter highlighting with the Syntax one, whenever it is needed?

This is even an issue for me when highlighting some c code. It most likely can be configured with the treesitter setup but I have no idea as I don't use lsp and treesitter.


Tracking folders with spaces in their names doesn't work

Maybe I'm doing something wrong, but tracking folders with spaces in their names doesn't work for me.
e.g.
git lfs track "Asset Sources"
git add .

After this, according to git lfs status, git lfs still hasn't picked up the files in "Asset Sources".
The .gitattributes file seems to contain the correct pattern:
Asset[[:space:]]Sources filter=lfs diff=lfs merge=lfs -text

Do you have any idea if this is a bug in git-lfs or if it's my fault?

Is Asset Sources a directory?  Try git lfs track "Asset Sources/*". Git LFS is writing to the .gitattributes file correctly (yes, even with the [[:space:]]), but your pattern just isn't matching anything.
Git Attributes is a core git thing, so we have no control over how it works. However, I'd like to add examples for common things that people try to track besides just file extensions.

Interesting! Adding the wildcard makes it work. Thanks for the hint!
I've already had some problems before with getting the patterns in .gitattributes right. If you're interested, I could definitely open a PR explaining the most common examples.

Yeah, I think even some examples in the git lfs track man page would go a long way.


Enhance the Examples Documentation

I changed the Example docs to work without change for vagrant users, under the assumption that if you have spun up a cluster you can make the necessary changes.
Also in the base examples directory I wrote  more rambling discussion of all the steps you need to take that people new to these various tools can find have starting points.  (everyone is new to at least one of these technologies).
You can see the docs easily here https://github.com/larry-svds/microservices-infrastructure/tree/fix-examples

Very nice! There are some good bones here, just need a little copy editing. If you remove the .gitignore stuff, I'll go ahead and merge this and then make the changes we need. I think we'll take this opportunity to deprecate the Minecraft example (it won't work on anything but DigitalOcean and Vagrant with our current setup.) Thank you for your work on this, updated documentation is always a great PR to get. :+1:

revered .gitignore changes that were mac and intellij specific.

oops.  pressed the wrong button.

Merging as @BrianHicks suggested. It does look good!


Input focus is not captured when use Vitest

Describe the bug
I am migrating from Jest to Vitest to do unit test. when I test input focus, it seems not emitted.
By the way, it work with Jest, but not Vitest
Reproduction
StackBlitz: https://stackblitz.com/edit/vitest-dev-vitest-gndsfu?file=test%2Fbasic.test.ts,src%2Fcomponents%2FOptions.vue,package.json,vite.config.ts
In this demo, I try to trigger click and focus on input element, but only get click.
await wrapper.find('input').trigger('focus');
await wrapper.find('input').trigger('click');
console.log(wrapper.emitted()); // [Object: null prototype] { click: [ [] ] }

<input type="text" @focus="handleFocus" @click="handleClick" />

<script>
// ...
methods: {
  handleFocus() {
    this.$emit('focus');
  },
  handleClick() {
    this.$emit('click');
  },
},
</script>

System Info
System:
    OS: macOS 12.5.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 25.82 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 14.18.1 - ~/.nvm/versions/node/v14.18.1/bin/node
    npm: 7.24.2 - ~/.nvm/versions/node/v14.18.1/bin/npm
  Browsers:
    Chrome: 105.0.5195.102
    Firefox: 104.0.1
    Safari: 15.6.1
  npmPackages:
    @vitejs/plugin-vue2: ^1.1.2 => 1.1.2
    vite: ^3.1.0 => 3.1.0
    vitest: ^0.23.1 => 0.23.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hi @justforuse , you may need to add a name to your Options component.

@watonyweng  I tried add a name, but seems not solve this issue.
<script lang="ts">
import { defineComponent } from 'vue';

export default defineComponent({
  name: 'Options',
  // xxx
});
</script>

@justforuse we might want to focus on the following:

It means your parent component will capture this method, when you call this.$emit.
Note the method signature of emitted, and learn more at emitted.

@watonyweng  I just check docs again, and find the tip:

When using trigger('focus') with jsdom v16.4.0 and above you must use the attachTo option when mounting the component. This is because a bug fix in jsdom v16.4.0 changed el.focus() to do nothing on elements that are disconnected from the DOM.

and test case passed. Thanks


chore: Add cspell setup for VSCode

cSpell has a VSCode extension https://marketplace.visualstudio.com/items?itemName=streetsidesoftware.code-spell-checker which is very useful to prevent from doing typos in the code. This is the setup for VSCode.
Later on, they also have GitHub actions or command lines to verify the code doesn't contain any typos.
If you prefer not to add it, that is fine, we can close the PR.

Nice !
I'll test it later but I'm ok with the idea of adding it.


Error with form event listeners

I'm getting these errors with some form event listeners, not sure what $event is being inferred here.
Parameter $event has wrong type 'Symfony\Component\Form\Event\PostSetDataEvent<mixed>|Symfony\Component\Form\Event\PostSubmitEvent<mixed>|Symfony\Component\Form\Event\PreSetDataEvent<mixed>|Symfony\Component\Form\Event\PreSubmitEvent|Symfony\Component\Form\Event\SubmitEvent<mixed>',

should be 'Symfony\Component\Form\Event\PostSubmitEvent' (see https://psalm.dev/141)

        ->addEventListener(FormEvents::POST_SUBMIT, function (PostSubmitEvent $event) {

Can you put your code, i.e. the part of $builder->addEventListener(....)?

With this code:
->addEventListener(FormEvents::POST_SUBMIT, function (PostSubmitEvent $event) {
    foreach ($this->deliveryConditionsFailedToSave as $deliveryCondition) {
        if ($event->getData() === $deliveryCondition) {
            $event->getForm()->get('code')->addError(
                new FormError("Could not change code from '{$deliveryCondition->getCode()}' to '{$event->getForm()->get('code')->getData()}'. Code '{$deliveryCondition->getCode()}' is in use.")
            );
        }
    }
})

I get this error (on symfony 6.2, php8.1)
ERROR: MismatchingDocblockParamType - src/TradeBundle/Form/Type/DeliveryConditionType.php:40:67 -
Parameter $event has wrong type 'Symfony\Component\Form\Event\PostSetDataEvent<mixed>|Symfony\Component\Form\Event\PostSubmitEvent<mixed>|Symfony\Component\Form\Event\PreSetDataEvent<mixed>|Symfony\Component\Form\Event\PreSubmitEvent|Symfony\Component\Form\Event\SubmitEvent<mixed>',
should be 'Symfony\Component\Form\Event\PostSubmitEvent' (see https://psalm.dev/141)

->addEventListener(FormEvents::POST_SUBMIT, function (PostSubmitEvent $event) {

Note this happens when chaining addEventListener
That is
$builder->add()->addEventListener()
Without chaining it seems to work fine like:
$builder->addEventListener()

@aszenz Yes, I see it now. I forgot to @return FormConfigBuilderInterface<T>.
I can make a fix in 2-3 days but can you brute-edit this file and add
* @return FormConfigBuilderInterface<T>

and see if chaining works until I fix this?

It seems to cause other errors like `UndefinedInterfaceMethod - SimpleContractlineType.php:63:15 - Method Symfony\Component\Form\FormConfigBuilderInterface::add does not exist

OK, my bad; I just noticed you were using
$builder->add()->addEventListener()

and I miss-read it as
$builder->addEventListener()->addEventListener()

To help me the fix, can you put entire full calls you make, and all errors? Should be easy fix, just need a way to replicate it.

Didn't really follow this subject but I would say to be careful when introducing generics on a plugin:
Other tools (for instance PHPStan) will not support this generic until someone add the same in phpstan-symfony (but the risk is to have someone providing a different template...)
That's one of the reason why I stopped using Psalm in favor of phpstan. Because they were incompatible
Symfony started to merge PR with generics, so it could be interesting to start doing PR on symfony code base instead.

@VincentLanglet

Other tools (for instance PHPStan) will not support this generic until someone add the same in phpstan-symfony (but the risk is to have someone providing a different template...)

The issue is that PHPStan doesn't yet support nested generics (I think this is the correct term) like one in AbstractController. For example:
$data = $this->createForm(CustomerType::class)->getData(); // mixed

but psalm can correctly resolve to null|Customer.

Didn't really follow this subject but I would say to be careful when introducing generics on a plugin:

Sorry, I didn't understand this one. Are you in favor of resolving it via plugin, or via template annotation (as it is now) or nothing at all (mixed always)?
We could also make an option to disable form stubs if users find this incompatibility with PHPStan a big problem. Or even remove from this repository and put into another.
WDYT; is it worth it? You are more in Symfony circles than I am.

The issue is that PHPStan doesn't yet support nested generics (I think this is the correct term) like one in AbstractController. For example:
$data = $this->createForm(CustomerType::class)->getData(); // mixed

You mean that if CustomerType is a FormType<Foo>,
then $this->createForm(CustomerType::class) should return FormType<Foo>
and then getData will return Foo ?
I'm not sure bout the support of PHPStan, I would say it's possible to do this...

Sorry, I didn't understand this one. Are you in favor of resolving it via plugin event, or via template annotation (as it is now) or nothing at all (mixed always)?

I'm fine with template, but it can now be done directly on Symfony first
like https://github.com/symfony/symfony/pull/47412

We could also make an option to disable form stubs if users find this incompatibility with PHPStan a big problem. Or even remove from this repository and put into another.

There is an issue with FormView template for instance, because it's not generic in PHPStan but it is in Psalm.
Fact is that Psalm supports the syntax
array{value: T}&array<string, mixed>

but PHPStan doesn't, so a stub in PHPStan codebase wouldn't make sens.
I got a recent issue with Psalm 5 since now this tool report "MissingTemplate" errors which didn't exist in Psalm 4.
I didn't take a big look, but it's maybe possible to disable with the IssueHandler, something like
<MissingTemplate>
    <class>
       <FormView error="suppress" />
    </class>
</MissingTemplate>

So I would say there is no need to provide something to disable a stubs.

You mean that if CustomerType is a FormType,
then $this->createForm(CustomerType::class) should return FormType
and then getData will return Foo ?
I'm not sure bout the support of PHPStan, I would say it's possible to do this...

Yes, that one. I tried PHPStan to resolve this, but it always returned mixed. But I am sure eventually it will get fixed.

I'm fine with template, but it can now be done directly on Symfony first
like https://github.com/symfony/symfony/pull/47412

There has been few attempts of merging form stubs into Symfony itself, but stof gave some good arguments.
From my POV, after 2-3 years of using these stubs, I have never had a problem. Doesn't mean they don't exist but I just didn't encounter them even with complex compound forms and transformers.
Did you have problems with them, assuming event listeners were used? If so, can you paste an example?

<MissingTemplate>

Honestly, I didn't even know about this one, thanks. Today I tried V5 and from zero errors, got 212; only 5 coming from me forgetting template on AbstractTypeExtension.
But with this, compatibility with V4 can be kept and not deter new users.

Did you have problems with them, assuming event listeners were used? If so, can you paste an example?

I essentially had problem because stub was added to PHPStan or Psalm, and I had to add it to the other.
Currently the only issue I have is the fact the Form-generics doesn't exists in PHPStan.
But never used the eventListener-related ones.


Deserialisation failing from osu lazer api?

Hi, I apologise in advance if I've made a mistake and this is something on my end.
The wrapper seems to be failing to deserialise responses from the lazer api as of the last day or so. I haven't changed my code at all so I suspect a change may have been made to the lazer resposne format?
I'm currently getting an error upon running this function
pub async fn get_recent_score(osu: &Osu, user_id: impl Into<UserId>, index: usize, include_fails: bool) -> Option<Score> {
    let res = osu
        .user_scores(user_id)
        .recent()
        .mode(GameMode::Osu)
        .include_fails(include_fails)
        .offset(index)
        .limit(1)
        .await;

    match res {
        Err(OsuError::NotFound) => None,
        Err(e) => panic!("failed to get recent score: {}", e),
        Ok(scores) => {
            scores.first().cloned()
        }
    }
}

Which gives me a Parsing Varianet of the OsuError e.g. running this on my own user ID gives:
failed to get recent score: failed to deserialize response: [{"ranked":true,"preserve":true,"processed":true,"maximum_statistics":{"great":547,"legacy_combo_increase":249},"mods":[{"acronym":"EZ"},{"acronym":"CL"}],"statistics":{"ok":76,"meh":8,"miss":51,"great":412},"beatmap_id":3103318,"best_id":null,"id":2732442911,"rank":"B","type":"solo_score","user_id":19357648,"accuracy":0.80195,"build_id":null,"ended_at":"2024-04-26T13:02:02Z","has_replay":false,"is_perfect_combo":false,"legacy_perfect":false,"legacy_score_id":4618270719,"legacy_total_score":575745,"max_combo":136,"passed":true,"pp":0.020622,"ruleset_id":0,"started_at":null,"total_score":127544,"replay":false,"current_user_attributes":{"pin":null},"beatmap":{"beatmapset_id":1512813,"difficulty_rating":6.24,"id":3103318,"mode":"osu","status":"ranked","total_length":134,"user_id":9088487,"version":"Deca's Expert","aclast_updated":"2021-08-15T19:31:21Z","mode_int":0,"passcount":27723,"playcount":172315,"ranked":1,"url":"https:\/\/osu.ppy.sh\/beatmaps\/3103318","checksum":"5546117d00d133dc48ed304e29558859"},"beatmapset":{"artist":"Paramore","artist_unicode":"Paramore","covers":{"cover":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/cover.jpg?1629055899","cover@2x":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/cover@2x.jpg?1629055899","card":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/card.jpg?1629055899","card@2x":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/card@2x.jpg?1629055899","list":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/list.jpg?1629055899","list@2x":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/list@2x.jpg?1629055899","slimcover":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/slimcover.jpg?1629055899","slimcover@2x":"https:\/\/assets.ppy.sh\/beatmaps\/1512813\/covers\/slimcover@2x.jpg?1629055899"},"creator":"attendant","favourite_count":556,"hype":null,"id":1512813,"nsfw":false,"offset":0,"play_count":629594,"preview_url":"\/\/b.ppy.sh\/preview\/1512813.mp3","source":"","spotlight":false,"status":"ranked","title":"Anklebiters","title_unicode":"Anklebiters","track_id":null,"user_id":12416885,"video":false},"user":{"avatar_url":"https:\/\/a.ppy.sh\/19357648?1710019187.jpeg","country_code":"GB","default_group":"default","id":19357648,"is_active":true,"is_bot":false,"is_deleted":false,"is_online":true,"is_supporter":true,"last_visit":"2024-04-26T17:37:56+00:00","pm_friends_only":false,"profile_colour":null,"username":"Dcs"}}]

I'm using this line in my Cargo.toml: rosu-v2-lazer = {package="rosu-v2", git = "https://github.com/MaxOhn/rosu-v2", branch = "lazer", features = ["replay"] }
Apologies again if I've made a mistake, otherwise I'd be happy to help if you need any other information to help solve the issue.

Indeed they added a new field to scores just recently which isn't a problem in of itself but the deserialization is currently configured to be extra sensitive to make sure no change goes unnoticed.
I''ve already pushed a fix so I believe a simple cargo update -p rosu-v2 should do the trick for you.

Ah, thank you! I appreciate the info.


UI changes SWoC20

Improving the UI of the website.

#41

#41

@preeti13456 i want to work on this.

@preeti13456 i want to work on this.

It's already assigned wait for 5 days . Till it get assigned to you in the meantime you can explore the website fix some bugs or raise issue and can provide any valuable feedback to this issue as well.

It's already assigned wait for 5 days . Till it get assigned to you in the meantime you can explore the website fix some bugs or raise issue and can provide any valuable feedback to this issue as well.


The whole PC freezes after running wsl --shutdown with NVIDIA GPU driver

Environment
Windows build number: Version 10.0.22598.200, and also in previous builds.
Your Distribution version: Tested in Ubuntu 20.04, 21.04 and 22.04.
Your WSLg version: 1.0.33

Steps to reproduce

Run some gui apps in WSL, then run wsl --shutdown from cmd or Powershell.
Tested using latest Nvidia drivers.
WSL logs:
No errors in logs since this happens when wsl --shutdown is run and this also makes the user to forcefully shutdown the computer.
Solution seems to be that you need to ensure all gui related apps are killed before running wsl --shutdown , or run wsl -t Ubuntu && wsl --shutdown. But this does not always work.

versions.txt

WSLg ( x86_64 ): 1.0.33+Branch.main.Sha.d57addbe6309d0a453123c91e6117fc21871496c
Mariner: VERSION="1.0.20220226"
FreeRDP: c574044a10003e50453acb4cf42801c5833fb572
weston: 651d33374f9b9e93e1cf1210faff13bcd339f132
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5

Expected behavior
Successfully shutdown WSL after running wsl --shutdown .
Actual behavior
The whole PC freezes (including mouse and all keyboard combinations) and becomes unresponsive, after running wsl --shutdown. The only way to get out of this situation is a force shutdown of PC.

same problem (NVIDIA Drivers 511.65 - RTX A5000)

same issue:

wsl --status

Default Distribution: Ubuntu
Default Version: 2
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.33
MSRDC version: 1.2.2924
Direct3D version: 1.601.0
Windows version: 10.0.22610.1

google-chrome --enable-features=UseOzonePlatform --ozone-platform=wayland

Chrome occasionally freezes, trying to resolve by

wsl --shutdown

The whole system hangs and needs to be rebooted.

@skyblueplus, thanks for reporting the issue. I was able to reproduce the issue and this is the issue in Window's side kernel mode driver issue. (for internal reference, the bug number at OS bug is 39506793), thanks!

@skyblueplus, @brokeDude2901, @inclu-media, if any of you are on Windows insider build, the fix is released to insider last week with Windows build number 25140 (https://blogs.windows.com/windows-insider/2022/06/15/announcing-windows-11-insider-preview-build-25140/). If you have a chance to try out this build and verify this issue is fixed, please let us know, thanks!

Thanks @hideyukn88 I'll guess I'll have to wait till it hits the beta channel. Any idea on time frames?

I am still having the same issue with Windows 11 22622.586...
Any ETA for the release of the fix on Beta channel?

Wlsg kills my 32gb machine. I only have to open a linux distro and the machine is dead - no response. What is wslg? A virus cannot do the damage it does. Does anyone have a fix?

@unixdaddy, to narrow down the root cause, would you please try with Graphics driver disabled? You can disable it from Device Manager, locate GPU driver node, right click on that, then select disable, then restart WSL, thanks!

Same thing is happening to me. Just completely locks up, my mouse won't move can't Ctrl+alt+del, nothing. Just have to physical hold the power button and restart.

@digitalnomad91, do you see the issue just by starting up Linux distro, or by wsl --shutdown? And does disabling graphics driver help? thanks!

I saw some activity so I want to add my experience.
I have a laptop running Windows 11 insider preview on beta channel with the following specs (I will loat generic info):

Intel 10th i7
32gb ram
rtx 2070 super
The laptop is mux-less, so I don't have a direct option to decide which gpu to use.

I started wsl by opening my Ubuntu distro, I executed a normal apt upgrade and then I tried executing wsl shutdown which frozen my laptop
At that time the gpu led was red, so nvidia gpu was running something. I can't guarantee that the process using nvidia gpu was wsl at that time.
I can say that the freeze doesn't happen everything, today I executed a wsl shutdown without freezing

The fix for system freezes by wsl --shutdown is released in KB5017389, which is currently in release preview state. Anyone on Windows insider channel should be able to install, thus if you happen to install it, please let us know how it goes, thanks!

@digitalnomad91, do you see the issue just by starting up Linux distro, or by wsl --shutdown? And does disabling graphics driver help? thanks!

The issue is when I run the shutdown command.  I have my monitors plugged in to my 3070 ti so I'm not sure how to disable the graphics drivers, do I need to plug in to the onboard Intel graphics? I don't use any wsl GUI stuff btw, I just run a lot of dev environments and connect to it with windows terminal & vscode.
Also idk if it's relevant, but I am running the genie systemd hack. I was about to update wsl and remove it and try installing the new Microsoft native systemd implementation when it happened again.

The fix for system freezes by wsl --shutdown is released in KB5017389, https://blogs.windows.com/windows-insider/2022/09/22/releasing-windows-11-build-22621-607-to-the-release-preview-channel/, which is currently in release preview state. Anyone on Windows insider channel should be able to install, thus if you happen to install it, please let us know how it goes, thanks!

Is this fix included on 22623.670 as well?
I am currently on 22622 and I didn't get 22621.607

@LightDestory,

Is this fix included on 22623.670 as well?

Yes, it is.

@digitalnomad91

I'm not sure how to disable the graphics drivers, do I need to plug in to the onboard Intel graphics?

You can temporarily disable the graphics driver from Device Manager.

@hideyukn88 Isn't there another way to solve this problem without joining the Windows Insider program?
In your suggestion I should disable my RTX 2060 for a few moments to be able to run the wsl --shutdown ?

@brunohubner, if everything goes as planned, it will be released to non-insider channel by late November, thanks!

Same happens with Windows 10 21H2 + KB5020030 + NVIDIA graphics. Supposedly it would happen for the other Win10 wslg updates, too.
Any updates planned for the Win10 wslg support?

Same happening to me, also nvidia graphics. I mostly use WSL through a remote SSH connection, and this manifests as "the SSH session suddenly dies, then I have to go hold the power button to turn the PC off".
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.47
MSRDC version: 1.2.3575
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22621.1105

I am also consistently seeing my computer freeze when I run wsl --shutdown. Here is my version info
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.51
MSRDC version: 1.2.3770
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19045.3208


Issue/fatal error requests

Import stops with error when fetching images
The block_builder_defaults.py has been refactored including adjustments to fetch_url()

Testing

[x] CI passes
[x] If necessary, tests are added for new or fixed behaviour
[x] These changes do not reduce test coverage

Documentation.

[x] Documentation changes are not necessary because: no user changes have been made

@nickmoreton I just tried using this PR as I need it for my current project.
From a functionnal point of view, it works as expected :+1:

@nickmoreton I just tried using this PR as I need it for my current project.
From a functionnal point of view, it works as expected 👍

Thanks @fabienheureux I have opened a couple of issues for the outstanding comments. As you have tested it locally I'll merge this in now.


Behavior of SendRef can surprise the user

Hi! In #70 , I commented our workaround to avoiding empty sends after already taking out a SendRef. We used to store the SendRef in an Option for the next loop iteration. This workaround came back to bite us. The behavior of SendRef is correct for all I understand but it can still be surprising.
Let's look at the following situation on a high-level:

Thread 0 creates a SendRef and holds on to it, because its current loop iteration did not yield any data to send.
Thread 1 sends something with a regular Sender::send.
Thread 0 finally finds some data to send 10 seconds later and uses the SendRef (dropping it) that it had stored.

Naively, we might expect that the following is observable on the receiver side:

The item from thread 1 is received immediately.
10 seconds later, the item from thread 0 is received.

Unfortunately, that is not what is happening. Nothing will be received until 10 seconds pass, and then both items are received in succession, but the item from thread 0 first (although it was "sent by dropping the slot" second). With the inner workings of thingbuf, this makes sense since the pointer to the next element to receive cannot advance while the first SendRef is still held.
Though I wonder if it might make sense to mention this in the documentation, so that less people for it. Especially with several tasks / threads, this can lead to ugly situations if someone holds on to a SendRef and thereby inadvertently stalls the receiver.
Thanks to @flxo for helping me in debugging :pray:

An example can be found in the draft PR #89

I personally did expect exactly the actual behavior: I think of creating a sendref as "reserving my spot" in the queue, but I do think it's definitely a good idea to make it clearer in the docs.


[FEATURE] Add Semantic release automation with Github workflow

Is your feature request related to a problem? Please describe
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Currently the EKS Blueprints releases are manual . This feature allows to automate the releases weekly
Check this out https://github.com/marketplace/actions/action-for-semantic-release

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Unfortunately, this is not supported in this org. We will continue with manual releases at this time


Problem running tutorial: Linker warning

Hi,
I'm trying to get the tutorial example to run but I get a warning when running after adding libKiwi.a:
ld: warning: directory not found for option '-L/Developer/Kiwi/build/Release-iphoneos'
I've cleaned the project and rebuilt and I can see that it is building Kiwi but obviously it does not find the files.
Sorry if this is the wrong forum for asking this. Thankful for any help.
Erik

Solved it myself: If you put Kiwi in a place where you do not have write permissions the build will fail (obviously), but it fails silently!
Perhaps there should be a warning about this in the tutorial.

I'll add this to the Wiki :)


Corrects plugin/ removal in rename.py

When users select 'n' option when prompted to keep plugin/ or not, rename.py fails due to incorrect method in line 45. This PR corrects this line.

thanks


Update changelog with version 2 changes

The CHANGES.md doesn't contain any v2 changes. This should be added.

So I've updated the changelog and will open a PR within the next week.
But I want to use this opportunity to discuss how we are going to handle changelogs/releases going forward. I have the feeling maintaining the changelog and releasing are still very ardorous tasks, so I propose we automate it as much as possible. I've been looking into possible solutions and have come across one approach which is being used by the Angular team (and has apparently become very much the norm for larger open source projects recently)
– It's a cli script called standard-version and it relies on the "conventional commit" standard – (short intro, official spec)
From the standard-version docs, when you run the script:

bumps the version in package.json/bower.json (based on your commit history)
uses conventional-changelog to update CHANGELOG.md
commits package.json (et al.) and CHANGELOG.md
tags a new release

Then you can simply publish and push.
It think this would make it much easier and safer to release. Base on the PR description template it would be easy for the people who merge & squash to write the correct commits. (Although it is more work than before of course)
– What do you think @thijstriemstra @katspaugh

@mspae I'm not sure if the average contributor will understand this process..

And btw, did you finish that PR for the changelog?


React 17 support

Hi, please can we have support for React 17 so I don't get this npm warning upon npm install in my React 17 projects. Thanks!
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^15.0.0 || ^16.0.0 but none is installed. You must install peer dependencies yourself.

https://github.com/popperjs/react-popper/releases/tag/v2.2.4 supports React 17


Socratic Seminar 12 Reading List (Jan 2023)

Drop links here!

A few lightning-dev topics we could put on the list..
A proposal for modifying the lightning channel protocol to introduce "Tunable Penalties"
https://github.com/JohnLaw2/ln-tunable-penalties/blob/main/tunablepenalties10.pdf
new lightning channel Factory construction proposals based on the above "Tunable Penalties" idea:
https://lists.linuxfoundation.org/pipermail/lightning-dev/2022-December/003782.html
circuitbreaker: an imperfect way to fight back against channel jamming
https://lists.linuxfoundation.org/pipermail/lightning-dev/2022-December/003781.html

Not really from December but I had missed it initially.  Interesting idea until better privacy solutions are deployed.
lnproxy:
https://github.com/lnproxy/lnproxy

Follow-up on our Full-RBF session last month.  Peter Todd surveyed running bitcoin nodes to find that at least 17% of nodes running 24.x have enabled full-rbf.
https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2022-December/021296.html

MAD Transactions: A new 0-conf tx protocol proposal as a way to enable 0-conf txs in a full-rbf world. a game theoretical solution based on mutually assured destruction.
https://medium.com/@idBrain/mad-transactions-mutual-assured-destruction-transactions-c04f7b5a2fa7

BIP324 v2 P2P Transport Protocol PR merged: https://github.com/bitcoin/bips/pull/1378#event-8155389517
https://bip324.com

Standard for importing labels was merged https://github.com/bitcoin/bips/blob/master/bip-0329.mediawiki


Use persistent clients when fetching remote data

The current version does not maintain persistent (keepalive) connections when fetching data from other servers. It should reuse the same client object when requesting additional resources.

Implemented in 185038aa2515e4bf3732a5730b8f6cc9eca7fc50


Removed unused Oracle DatabaseOperations.date_interval_sql().

Unused since 5ca82e710e2f92b8c5114492205c8764918407d3.

buildbot, test on oracle.

@felixxm, I already created PR for that https://github.com/django/django/pull/9468

@sir-sigurd Sure, you can assign this ticket to yourself. I think it's worth to have separate ticket for that, because it's not related with regression on SQLite (see https://github.com/django/django/commit/7c1f3901bcdabb1340a621e7df9b24f3acd0d6f3).


Git status bad perfomance

go-git status has too bad performance. Should be replaced by direct git call

I've added optimization which allows skipping of invoke of git status if files don't need any migration. For further improvement, we can use https://github.com/libgit2/git2go instead of go-git. (but since it is a C library, it will complicate build for different platforms)


Add DMS support

Add support for AWS Database Migration Service that is addressed with https://github.com/spulec/moto/issues/1577

This is only about half implemented but wanted to put up my work in progress for now.
I need to add implement the response stuff in particular and add tests.

Coverage decreased (-0.4%) to 92.337% when pulling d136ce1261cb528bbb6190014d0b586edfe0f8ac on jzucker2:add-dms into 56a56fc626a5cb9d048d2abc50bb85af17644c07 on spulec:master.

Coverage decreased (-0.4%) to 92.337% when pulling d136ce1261cb528bbb6190014d0b586edfe0f8ac on jzucker2:add-dms into 56a56fc626a5cb9d048d2abc50bb85af17644c07 on spulec:master.

@jzucker2 HI! HI! HI!
This is looking great, man. So delightful to see you here. I help out around these parts as a maintainer :D

@JackDanger hello! I saw your name in the commits but didn't realize you were a maintainer. Will definitely be reaching out with some questions soon.

Hi folks, I was also interested in this and would love to contribute a bit, but wanted to check before doing so. I see that @jzucker2 and others were doing some checks and had some movement more than a year ago, so I wanted to see where I can help and get this ball rolling, at least as a minimum implementation so we can get it out (if it passes the maintainers rules).
Thanks and let me know how I can pick it up and/or help with what its already there

@enriquemanuel if nobody has worked on it recently, feel free to pickup where they have left off.
With Moto, we are always happy with a minimum implementation (basic endpoints, basic functionality). A basic implementation that people can use today and we can iterate on over time is the goal.

Perfect. I will continue with the implementation and tests and see how we can have that first iteration for now.
Thanks @spulec

Most of these features have now been implemented by #3900, so I'll close this one.
Any followup PR's to implement the remaining methods are of course welcome!


Create Timer service and refactor timing code into it

I wonder if month should actually be an enum.

Ooh, I like that idea! Seems like a good use case for enums. number-based, starting from one?
(And then for consistency, I'd probably want to do the same thing for Quarter.)

Oh, I thought it was already an enum since all the options were listed.
Oops.
On Sun, May 24, 2020 at 2:30 PM Joe Moonan Walbran<EMAIL_ADDRESS>wrote:

Ooh, I like that idea! Seems like a good use case for enums. number-based,
starting from one?
(And then for consistency, I'd probably want to do the same thing for
Quarter.)
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/mn-pollinators/buzz-about/pull/29#issuecomment-633283107,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAU7345HGYWPER7VNAKZ54LRTFYWHANCNFSM4NHDLLBQ
.

--
K.K. Lamberty
Associate Professor of Computer Science |
morris.umn.edu/academics/computerscience/
Division of Science and Mathematics, 600 East 4th Street, Morris, MN 56267
University of Minnesota, Morris | morris.umn.edu
<EMAIL_ADDRESS>|<PHONE_NUMBER>
she/her/hers

I had to look up the difference 🙃 It seems like what we're doing is actually a really big union type, which is like an enumeration, but the values don't have names, and it doesn't have as many neat features.
The names are really the selling point here—January, February, March seems like a more intuitive API than either 0, 1, 2 or 1, 2, 3

(Unless anyone objects, I'll plan on making that change tomorrow.)

I don't know if the quarters need to be a enum. With the months being an enum though we can actually use their names as the string returned from monthString I think.

Oh, okay, sounds good! 🙂

I've tentatively decided to make currentTime$ emit whenever the timer is unpaused, in addition to whenever the time changes, for two reasons:
First, if you initialize the timer to a starting value, subscribe to the timer, and then start the timer running, you'll still receive the initial value.
Second, if you only emit when the time changes, you end up in a situation where it's impossible to predict whether the initial TimePeriod is emitted without knowing where the timer left off.
Imagine the following hypothetical scenario:
Suppose someone decides it's a good idea to reset the timer to (1, 1)—maybe in a constructor, or an ngOnInit() method. Now, in a completely different file, let's say round one of Buzz About wants to start from the first quarter of January. It also sets the timer to (1, 1), and starts the timer running. But, because the timer was already on the first quarter of January, (1, 1) is never emitted—from the round's perspective, it will look like that time just got gobbled up.

(But I'm happy to change it back if people prefer the old way! 🙂)

(Pardon me for force-pushing; I was working with a few versions of the file locally and I pushed the wrong one.)

Test Components

Timer Test
Flower Test


flutter app cannot handle detached life cycle.

Engine add to app starts sending detached message to framework. However, framework side change is pending on desktop embedding.
to reproduce
1, run flutter/sample/experiment add to app with ios full screen
2, launch the flutter screen and close it
I/flutter (16072): The following assertion was thrown during a platform message callback:
I/flutter (16072): 'package:flutter/src/scheduler/binding.dart': Failed assertion: line 326 pos 12: 'state != null': is
I/flutter (16072): not true.
I/flutter (16072):
I/flutter (16072): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter (16072): more information in this error message to help you determine and fix the underlying cause.
I/flutter (16072): In either case, please report this assertion by filing a bug on GitHub:
I/flutter (16072):   https://github.com/flutter/flutter/issues/new?template=BUG.md
I/flutter (16072):
I/flutter (16072): When the exception was thrown, this was the stack:
I/flutter (16072): #2      SchedulerBinding.handleAppLifecycleStateChanged (package:flutter/src/scheduler/binding.dart:326:12)
I/flutter (16072): #3      WidgetsBinding.handleAppLifecycleStateChanged (package:flutter/src/widgets/binding.dart:543:11)
I/flutter (16072): #4      SchedulerBinding._handleLifecycleMessage (package:flutter/src/scheduler/binding.dart:342:5)
I/flutter (16072): #5      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:74:49)
I/flutter (16072): #6      _DefaultBinaryMessenger.handlePlatformMessage (package:flutter/src/services/binding.dart:200:33)
I/flutter (16072): #7      _invoke3.<anonymous closure> (dart:ui/hooks.dart:303:15)
I/flutter (16072): #11     _invoke3 (dart:ui/hooks.dart:302:10)
I/flutter (16072): #12     _dispatchPlatformMessage (dart:ui/hooks.dart:162:5)
I/flutter (16072): (elided 5 frames from class _AssertionError and package dart:async)

Hello, Did anyone find a solution for detached state to recognize!

In Application Lifecycle cant check if user closed the application or not, i think detatched in ApplicationState Should handle this.

This depends on whether the app is just in the background or the activity is destroyed. If the app is just in the background (pressing home screen), it will be in AppLifecycleState.paused. If the activity is destroyed (android back button), it will be in AppLifecycleState.detached

@chunhtai okay but what if the application in background and i dragged it to destroy? i tried this but the last state was paused, and my state require to do action before App destroyed!

I don't think there is a way to listen to drags to destroy. Maybe @xster knows more.
May I know what is your use case?


Add forRoot Method to TrimValueAccessorModule

Hi,
When I import this module, components in my child modules are not getting the directive applied. So I import it to each of my module and then it works. I have not tested but I suspect if you add the forRoot method that can fix this issue.
Thanks

Thanks for submitting the PR for this, but before merging it, can you please elaborate more on your issue?
What version of Angular are you using? Are you importing it in the AppModule? Can you share a code sample?
Regarding the fix with the forRoot, here's a quote from Angular's doc page:

Use forRoot() to separate providers from a module so you can import that module into the root module with providers and child modules without providers.

I'm not sure that's exactly what we need here but it's good to know whether you've tested your solution and if it solves the issue.

Hi, I am using Angular 5, the issue I am having is when I import it under AppModule, the value accessor is not working in components in the child modules. Sorry for creating a PR without testing, let I verify and do some testings.

I'm using it currently in Angular 7 and importing this module inside the SharedModule and exporting it back. This way it's accessible through the feature modules.

You are right about the SharedModule export, that was the thing I was missing, thank you.

Is there any other solution for this?
Using the SharedModule is a workaround. Basically the same as importing it directly in every child module, just with less code. It not applicable to a lot of use cases.
For example, I have 3 apps: 1 library project, and 2 apps that use the library.
Inside the library there are some textareas that I want trimmed.
In the 1st app I want to use the trimming, but not in the 2nd app.
So, I can't push this module import all the way down to the library project.
I want to import it only in the AppModule of the 1st app.


Adding .update() to docs

Thanks! Although I'm not sure that's the correct place to put such doc

Thanks! Although I'm not sure that's the correct place to put such doc

I thought about it and at the end decided to put it there cause was more a ref.read specific scenario. You are right, it is not about reading! The other pages I think was too way off, do you suggest any place in which I can put it?

I think the place where it would make the most sense is on a page dedicated to StateProvider

What do you think? better?

Hum no, it's not the correct page either.
Maybe add that to the dartdoc of StateProvider. https://pub.dev/documentation/riverpod/latest/riverpod/StateProvider-class.html

Hum no, it's not the correct page either.
Maybe add that to the dartdoc of StateProvider. https://pub.dev/documentation/riverpod/latest/riverpod/StateProvider-class.html

Ok but I have to take a little bit of time since I have no idea on how to contribute to dartdoc.

I'll try to make a page for all providers this week-end. Then I'll move this to the Stateprovider page

I'll try to make a page for all providers this week-end. Then I'll move this to the Stateprovider page

I can close the pull request?

No need. I'll take care of things from there and rebase/merge it when I'm done with other things

Done, thanks for your contribution!


Adding Credits to Markup

Added a credit section to the markup guidelines.
@jimmynotjim, can you take a look and see if I'm missing anything?

:+1: Looks good. Once @jimmynotjim agrees let's merge!

Looks good to me as well. Merging.


add functionality to add and save property constraints

add functionality to create property constraints:

frontend: add input field to enter xml
backend: add functionality to add constraint as xml

Can one of the admins verify this patch?

Please consider the issue outlined by Codacy and resolve the conflicts with the master branch.


Add CAS1 APPROVEDPREMISESTESTUSER to staff

we should probably do this for all test users, but this at least resolves an issue when creating applications locally

nice: out of interest, which probation area (12) are we setting?

to be honest i just copied jim snow. i'll double check and make sure it aligns with the test user

nice: out of interest, which probation area (12) are we setting?

Currently we're using an 'random' area code of GCS. This doesn't break anything locally because we don't update the users probation region/area if we can't map the code returned by community api.
Ideally i'd update the user in community api to use expected codes but these don't appear to exist in our test instance of community API (e.g. for east of england we're expecting N56, but in the community database this code doesn't exits), so this is going to be a bit of work to fix. I'll create a task capturing my investigation so far.
I'm going to merge now to unblock local deploys


drivers: modem: adding support for power control for gsm ppp driver

This PR address the support of power control mechanism
of modem with GSM PPP driver.The switch on and off
mechanism for modem is left for users of GSM PPP driver.
The driver has been tested with echo_client app with
SARA U201 modem.
The PR address #27622 and #27611
Signed-off-by: Tahir Akram<EMAIL_ADDRESS>

Alternative suggestion: https://github.com/zephyrproject-rtos/zephyr/pull/28475

We went with the alternative solution #28475 which is now merged so I will close this one.


I need to open the chat without the bubble, when the component loads

How to disable the bubble and allow only the Livechat component to be shown?

It is possible but you need to make changes in code. I would recommend to check: https://developers.livechatinc.com/docs/visitor-sdk/#methods  . Here you can find everything you need :)


MBot: Client API

Client (curl requests / postman) can start a chat with MBot, post messages to it and see the message history. These messages are not forwarded to DMR yet, this task is only for setting up the api and in memory storage (hashmap) for chats.
Acceptance Criteria

[x] Client can create a new chat via curl
[x] Client can post a message to a chat
[x] Client can get all the messages in a specific chat

The Chat API controllers implement these: https://github.com/buerokratt/Mock-Bot/blob/main/src/MockBot.Api/Controllers/ChatController.cs


remote stemcell in bosh manifest does not check for existence correctly

When I have this in my manifest:
  stemcell:
    name: bosh-aws-xen-hvm-ubuntu-trusty-go_agent
    sha1: d71a2b0c9cec487931eb5790355a8a6ad187a812
    url: https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160
    version: 3160

I get an infinite loop like this during bosh deploy:
root@07979c888b61:/host# bosh deploy
Acting as user 'admin' on 'my-bosh'
Checking whether release cf/226 already exists...YES
Acting as user 'admin' on 'my-bosh'
Checking if stemcell already exists...
No

Using remote stemcell `https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160'

Director task 19
  Started update stemcell
  Started update stemcell > Downloading remote stemcell. Done (00:00:00)
  Started update stemcell > Verifying remote stemcell. Done (00:00:00)
  Started update stemcell > Extracting stemcell archive. Done (00:00:00)
  Started update stemcell > Verifying stemcell manifest. Done (00:00:00)
  Started update stemcell > Checking if this stemcell already exists. Failed: Stemcell `bosh-aws-xen-ubuntu-trusty-go_agent/3160' already exists (00:00:00)

Error 50002: Stemcell `bosh-aws-xen-ubuntu-trusty-go_agent/3160' already exists

Task 19 error

Stemcell at https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160 already exists.
Acting as user 'admin' on 'my-bosh'
Checking if stemcell already exists...
No

Using remote stemcell `https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160'

Director task 20
  Started update stemcell
  Started update stemcell > Downloading remote stemcell. Done (00:00:00)
  Started update stemcell > Verifying remote stemcell. Done (00:00:00)
  Started update stemcell > Extracting stemcell archive. Done (00:00:00)
  Started update stemcell > Verifying stemcell manifest. Done (00:00:00)
  Started update stemcell > Checking if this stemcell already exists. Failed: Stemcell `bosh-aws-xen-ubuntu-trusty-go_agent/3160' already exists (00:00:00)

Error 50002: Stemcell `bosh-aws-xen-ubuntu-trusty-go_agent/3160' already exists

Task 20 error

Stemcell at https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160 already exists.
Acting as user 'admin' on 'my-bosh'
Checking if stemcell already exists...
No

Using remote stemcell `https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160'

Director task 21
  Started update stemcell
  Started update stemcell > Downloading remote stemcell. Done (00:00:00)
  Started update stemcell > Verifying remote stemcell. Done (00:00:00)
  Started update stemcell > Extracting stemcell archive. Done (00:00:00)
  Started update stemcell > Verifying stemcell manifest. Done (00:00:00)
  Started update stemcell > Checking if this stemcell already exists. Failed: Stemcell `bosh-aws-xen-ubuntu-trusty-go_agent/3160' already exists (00:00:00)

Error 50002: Stemcell `bosh-aws-xen-ubuntu-trusty-go_agent/3160' already exists

Task 21 error

Stemcell at https://bosh.io/d/stemcells/bosh-aws-xen-ubuntu-trusty-go_agent?v=3160 already exists.
Acting as user 'admin' on 'my-bosh'
Checking if stemcell already exists...
No

Sent @cppforlife full manifest in slack.

Are you using latest bosh cli?

Yeah, I'm using 1.3160.0.

FWIW, this is the dockerfile of the image that I'm running inside: https://github.com/mavenraven/cloudfoundry-installer/blob/master/Dockerfile

Any reason why you're not using light stemcells here? That seems to work for me.

btw: your manifest contains bosh-aws-xen-hvm-ubuntu-trusty-go_agent as name but the name in the db seems to be bosh-aws-xen-ubuntu-trusty-go_agent. That's why the first test doesn't find the stemcells but the second says it is already there. Seems like something is wrong with the name for the xen-hvm full stemcells?

Thanks @voelzmo, I got it working. One thing I noticed is that even correcting the urls and names to match, it wasn't working when I had the version set to latest. It seems like bosh should do better validation of name/version, like you said. It also would be nice if bosh.io listed sha1s as well as md5s to make it harder to use the wrong type of stemcell.

@mavenraven see https://github.com/cloudfoundry/bosh-init/issues/41 for the md5/sha1 mismatch.


;roll is broken

;roll 1d6
RoxBotBOT - Today at 8:12 PM
Expression missing. If you are unsure of what the format should be, please use ;help roll
;roll 2d20h1 + 7 (this is the example given in help file)
RoxBotBOT - Today at 8:12 PM
Expression missing. If you are unsure of what the format should be, please use ;help roll

Fixed in 3ff4302a7d9afef32b1692567c3869dedfea221d but one issue that still remains is trailing addition and subtraction isn't working. So I'll have to get @TBTerra on that.


BUG: pipCommand not pinned by hash is not affecting the score

Describe the bug
This pip install is being identified by the Scorecard as a Warn: pipCommand not pinned by hash but it is not affecting the score at all.

Reproduction steps
Steps to reproduce the behavior:

run

scorecard --repo https://github.com/joycebrum/x86_64 --checks Pinned-Dependencies --format json --show-details --commit 9ddec47afa76d8d8de4e21396f3c0256b29dfc02

The score should be 10/10
The details should mention a pipCommand not pinned by hash found.

Expected behavior
Not sure how much it should affect the score, but it should affect it since it is mentioned as a warn.

Probably will be fixed on #2582


[Event Request] Codeunit 90 Purch.-Post PostSalesTaxToGL

In this issue #1809, an event was requested for OnBeforeGenJnlPostLine(GenJnlLine).  It will occur before the below line of code, in the PostSalesTxToGL function of the Purch.-Post codeunit.
GenJnlPostLine.RunWithCheck(GenJnlLine);
We require an additional parameter added to this event as well.  TempSalesTaxAmtLine
Please see below for example:
OnBeforeGenJnlPostLine(VAR GenJnlLine: Record "Gen. Journal Line"; TempSalesTaxAmountLine: Record "Sales Tax Amount Line")

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


fix(fun_prop): bug fix - use Mor.headBeta insted of Expr.headBeta

fix(fun_prop): bug fix - use Mor.headBeta insted of Expr.headBeta
This comes up when using HasUncurry.uncurry with bundled morphisms

I'm closing this as I have more principled and extensive fixes upcomming.

Should I delete the branch? What is the convention?


add context to syntax errors

fixes https://github.com/edgedb/edgedb-go/issues/116
A query snip-it with arrow and hint is introduced for syntax errors.
Caveats
This implementation is very crude and likely to work poorly with characters outside of the ascii range because bytes are construed as characters. It also replaces tabs with spaces in the query to avoid misalignment when printed.
There doesn't seem to be a library in go similar to rust's codespan so the only option seems to be to roll our own.
Example
SELECT (foo (((1 2) 3)) 4) produces
edgedb.EdgeQLSyntaxError: Unexpected token: <Token ICONST "2">
query:1:18

SELECT (foo (((1 2) 3)) 4)
                 ^ It appears that a ',' is missing in a tuple before '2'

If it seems worth while I could extend this to work with arbitrary unicode instead of just the ascii range.

This change is based on https://github.com/edgedb/edgedb-go/pull/119 and is WIP until the upstream branch is merged.

If it seems worth while I could extend this to work with arbitrary unicode instead of just the ascii range.

Yes, we should do that. The updated error messages is a major improvement.

I'll wait until the review until https://github.com/edgedb/edgedb-go/pull/119 is merged.

I could extend this to work with arbitrary unicode instead of just the ascii range.

This is a little better now, but it turns out that character width, even in a fixed width font, is font dependent. 世界 is 2 code points but each character takes up 2 columns in my terminal font. I haven't found a nice way of determining the number of columns a grapheme cluster typically consumes.


A workaround to install vtkplotter in Ubuntu 20.04

Ubuntu 20.04 is shipped with Python 3.8.2 but the vtk library is not yet available in PyPI for python 3.8 hence it not possible yet to install it using pip. Since vtk is the main dependency of vtkplotter it will not be installed.
The workaround, I found is to install vtk7 using apt package manager:
sudo apt install python3-vtk7

clone the vtkplotter library:
git clone https://github.com/marcomusy/vtkplotter.git

in the file setup.py removevtkfrom the dependencies. Change the line 15 to the following:
install_requires=[],

Now you can install vtkplotter as follows:
python setup.py install

I hope this is helpful.

Actually vtkplotter is in the debian repository!
So if you do
sudo apt install python3-vtkplotter
you should get version 2020.2.0 along with vtk7
(i never tested it)

There is a wheels package available ( vtk-9.0.0-cp38-cp38-linux_x86_64.whl ) on the vtk download site. https://vtk.org/download/  Doesn't that work? (no idea why it is not on pypi yet)


updated without extras

Closes https://github.com/conda-forge/argopy-feedstock/issues/15

@conda-forge-admin, please rerender


feat: data-plane self unregistration

What this PR changes/adds
Make the data plane "unregister" itself instead of "delete".
Why it does that
Briefly state why the change was necessary.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #4238
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

Codecov Report
Attention: Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 75.03%. Comparing base (7f20ba5) to head (08dcbf6).
Report is 295 commits behind head on main.

Files
Patch %
Lines

...plane/selector/spi/instance/DataPlaneInstance.java
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4249      +/-   ##
==========================================
+ Coverage   71.74%   75.03%   +3.28%
==========================================
  Files         919     1046     +127
  Lines       18457    20980    +2523
  Branches     1037     1169     +132
==========================================
+ Hits        13242    15742    +2500
+ Misses       4756     4726      -30
- Partials      459      512      +53

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add clip_skip to EmbeddingsProvider

Add a way to directly control how many clip layers should be skipped.

i won't be merging this because it conflicts with (this enum)[https://github.com/Disty0/compel/blob/bf89819d697f7fab1843408d934fb4dfd2d45e01/src/compel/embeddings_provider.py#L21]

perhaps a way to do this is to add extra enum values for different clip_skip values

Ended up hijacking EmbeddingsProvider._encode_token_ids_to_embeddings like this: https://github.com/vladmandic/automatic/blob/master/modules/prompt_parser_diffusers.py#L16


links not displaying

http://bibsoup.net/pitman/peres_yuval
is full of links in the records but none are displaying

fixed in code, will be pushed to live later today.

Mark,  still no links showing in http://bibsoup.net/pitman/peres_yuval
Did you not push yet?

try refreshing the collection, see if it works then.

markmacgillivray<EMAIL_ADDRESS>wrote:

try refreshing the collection, see if it works then.

Yes!  Displays fine now, thanks.
--Jim


Amplify init errors out for appsync event starter app

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

Describe the bug
Create the AppSync API using the Events Starter through the AppSync console.
Clone appsync events starter for Android
Run amplify init
Note: It is recommended to run this command from the root of your app directory
? Enter a name for the project awsmobileappsynceven
? Enter a name for the environment beta
? Choose your default editor: Sublime Text
? Choose the type of app that you're building android
Please tell us about your project
? Where is your Res directory:  app/src/main/res
Using default provider  awscloudformation

For more information on AWS Profiles, see:
https://docs.aws.amazon.com/cli/latest/userguide/cli-multiple-profiles.html

? Do you want to use an AWS profile? Yes
? Please choose the profile you want to use default
⠦ Initializing project in the cloud...

CREATE_IN_PROGRESS awsmobileappsynceven-20190415094023 AWS::CloudFormation::Stack Mon Apr 15 2019 09:40:23 GMT-0700 (PDT) User Initiated
CREATE_IN_PROGRESS UnauthRole                          AWS::IAM::Role             Mon Apr 15 2019 09:40:27 GMT-0700 (PDT)
CREATE_IN_PROGRESS AuthRole                            AWS::IAM::Role             Mon Apr 15 2019 09:40:27 GMT-0700 (PDT)
CREATE_IN_PROGRESS DeploymentBucket                    AWS::S3::Bucket            Mon Apr 15 2019 09:40:28 GMT-0700 (PDT)
CREATE_IN_PROGRESS UnauthRole                          AWS::IAM::Role             Mon Apr 15 2019 09:40:28 GMT-0700 (PDT) Resource creation Initiated
⠇ Initializing project in the cloud...

CREATE_IN_PROGRESS AuthRole AWS::IAM::Role Mon Apr 15 2019 09:40:29 GMT-0700 (PDT) Resource creation Initiated
⠦ Initializing project in the cloud...

CREATE_IN_PROGRESS DeploymentBucket AWS::S3::Bucket Mon Apr 15 2019 09:40:29 GMT-0700 (PDT) Resource creation Initiated
⠴ Initializing project in the cloud...

CREATE_COMPLETE UnauthRole AWS::IAM::Role Mon Apr 15 2019 09:40:46 GMT-0700 (PDT)
CREATE_COMPLETE AuthRole   AWS::IAM::Role Mon Apr 15 2019 09:40:46 GMT-0700 (PDT)
⠴ Initializing project in the cloud...

CREATE_COMPLETE DeploymentBucket AWS::S3::Bucket Mon Apr 15 2019 09:40:50 GMT-0700 (PDT)
⠦ Initializing project in the cloud...

CREATE_COMPLETE awsmobileappsynceven-20190415094023 AWS::CloudFormation::Stack Mon Apr 15 2019 09:40:54 GMT-0700 (PDT)
✔ Successfully created initial AWS cloud resources for deployments.
✔ Initialized provider successfully.
✖ There was an error initializing your environment.
init failed
SyntaxError: Unexpected token  in JSON at position 0
    at JSON.parse (<anonymous>)
    at getCurrentAWSConfig (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-frontend-android/lib/frontend-config-creator.js:84:22)
    at createAWSConfig (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-frontend-android/lib/frontend-config-creator.js:26:28)
    at Object.createFrontendConfigs (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-frontend-android/index.js:23:10)
    at Object.onCategoryOutputsChange (/usr/local/lib/node_modules/@aws-amplify/cli/src/extensions/amplify-helpers/on-category-outputs-change.js:22:33)
    at initializeEnv (/usr/local/lib/node_modules/@aws-amplify/cli/src/lib/initialize-env.js:79:27)
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:228:7)

To Reproduce
Steps to reproduce the behavior:

Create the AppSync API using the Events Starter through the AppSync console.
Clone appsync events starter for Android
Run amplify init

Expected behavior
Amplify init to succeed.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

It turned out the starter repo
https://github.com/aws-samples/aws-mobile-appsync-events-starter-android.git
saved this file app/src/main/res/raw/awsconfiguration.json as a UTF-8 BOM file, probably because it was edited in Windows, causing the json parser to fail.
see this node issue for more information: https://github.com/nodejs/node/issues/20649
I will talk to the team tomorrow to figure out the best solution for this.


Clipboard behavior in text fields is incorrect with selection

If you select "a" in a text field, paste in "b", select "b", and paste again, it will go back to "a" when it should stay as "b".

This is still an issue.

It seems like this issue somehow got fixed at some point.


Get new access token from client after refresh

In "offline" mode when
credentials.authorize(httplib2.Http())
is called, it refreshes the access_token. Is there any way to get this new access_token?
Pretty much the same as what's being talked about here: https://github.com/google/google-api-php-client/issues/901

You can get notified when a new token is obtained via https://oauth2client.readthedocs.io/en/latest/source/oauth2client.client.html#oauth2client.client.Storage

Thanks for your time.
client.Storage.get() expects the Storage object, I believe? I am not using one and I think the authorize function refreshes and stores the credentials in the http object passed to it.

I tried to create a store with
oauth2client.client.Storage
but store.get() is not implemented yet?
Anyway, I ended up doing a refresh of the credentials outside and storing it
from oauth2client import client credentials = client.OAuth2Credentials.from_json(self.credentials) credentials.refresh(httplib2.Http()) auth = credentials.authorize(httplib2.Http()) self.credentials = credentials


Remove test_llama_perf

Proposal: remove models/demos/llama3/tests/test_llama_perf
e2e performance is measured from demo.py, test_llama_perf doesn't measure prefill, doesn't use tracing, device perf can be measured from the demo directly, it no longer has a compelling reason to exist and be maintained.

This was shipped with the latest Llama3 text feature complete PR.


Launching meteor does not show any error logs.

This may be windows' fault. I am on windows 10 64-bit. When I launch my app using atom, it doesn't show anything except for the warning icon continuing to blink. Although it can run when just using the command prompt.

It's also confirmed on OSX. Something has changed in the last Atom's update. Weird.
Thanks for the warning.

open and close developer tools once you launch meteor, the logs will start to show up.

It now gives me the tasklist error

Please see if mongo process was killed properly, solved the same problem for me – it was due to wrongly seth paths and application crashing before killing it.
About those paths – I found that if I already have meteor directory added to system PATH I coud just  set "Meteor Path" in extension settings to:
meteor.bat

and it just worked.
(Please note that you do not need to create or modify any file on your filesystem in this solution)
And if this doesn't work then probably there is no meteor directory in system path variable – you can fix it by adding it permanently by typing in command line:
SET PATH=%PATH%;C:\Users\username\AppData\Local\.meteor\

(You can remove it later if you want of course, but it's really good thing to have; and if you want to use several different versions of meteor, I would suggest just creating many .bat files pointing to them in the meteor directory that was added to path, and just edit their contents to create/overwrite default meteor.bat file to point to different installs inside, and later just delete default meteor.bat, to now use unknown installation in case of something crashing... but these are just my loose thoughts).

Fixed in 0.27.0.

excellent


Add support to dynamically fetch PR if config file not present

Signed-off-by: Arush Salil<EMAIL_ADDRESS>

Hi @arush-sal :)
Can you please provide more information on what this pull request does?

Please provide a screenshot or even better a video of how the new flow looks/works.
What's the use case for this?

@dlvhdr
What this pull request does:

It removes the mandate need to have a static config file defining your repos

How the new flow looks like:

Typically it doesn't change anything in the flow except that now you don't necessarily need to create the config file for this to work.

What's the use case for this?

For people with too large of dynamic repo set that they can't have a stick config file and hence want to track all the recent PRs for them/in their profile.

I'm working on removing the repo list requirement and think it will solve that for you.
The approach you suggest will make it so new users start with prs from their notifications which is nit something I want.
I would like to have a default config file.
Once I allow not to pass the repos list, that will be easy to do.

See https://github.com/dlvhdr/gh-prs/pull/40


Metamask cannot cancel or speed up a transaction.

and will resend the transaction in an infinite loop. Even restarting the browser does not prevent it.

That will means sign a transaction with different gas price, which isn't what we can support at this stage.

But in any case, we need a way to cancel the transaction. Currently there is no way to cancel the transaction other than deleting the account.

yeah I also seen this situation before, but I think this is caused by when using instant-sealing, after metamask send a tx, this tx isn't included in a new block, and node didn't trigger a new block either...
metmask keeps searching for this non-exist tx hash, and ignores following ones

we can go to metamask settings => advanced => reset account to clear metamask cache

I can do more experiment to see how to reproduce this issue deterministically, I have seen this very rarely and randomly

UPDATE: got it, when first time sending token to a brand new account, it won't trigger a new block, but the tx actually went through and succeed.

I think I know what's happening: we have to clear metamask cache every time we restart mandala node... otherwise there will be some nonce issue, i.e. metamask will use cached tx to speculate currently nonce...


Add LinearMatrix.{from_function, from_rotor}

Inspired by conversation on slack, this seems like a fairly common use case, or at least one that is pedagogically useful.
This is tested by its doctests, hopefully. Coverage report will tell.

Will fix CI shortly


Update Decorators.md

fix typo

Fixes #

@tijwelch,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

thanks!