Can't find local modules

The tool doesn't find modules in the current working directory when it's installed from pip, which is what I would expect it to be used for.
Example of local file:
$ cat lucli.py
import click

@click.group()
def cli():
    """Your standard, everyday lunch CLI."""
    pass

@cli.command()
@click.option('-dish', metavar='thing', envvar='DISH', default='burger',
                show_default=True, help='The meat dish to order')
def order_meat(dish: str):
    """Orders a meat dish."""
    click.echo(f'Meat is neat!

And when running the tool:
$ mdclick dumps --baseModule=lucli --baseCommand=cli --docsPath=docs/_build/md-click/
Creating a new documents from lucli.cli into docs/_build/md-click/
Could not find module: lucli. Error: No module named 'lucli'

If I add the current working directory to the Python path at the beginning of main.py:
import sys
import os
sys.path.insert(0, os.getcwd())

then it works:
$ mdclick dumps --baseModule=lucli --baseCommand=cli --docsPath=docs/_build/md-click/
Creating a new documents from lucli.cli into docs/_build/md-click/
Created docs under docs/_build/md-click/

but I'm not sure that's a good solution.

Running into the same problem.


operator project-quay (3.9.0-nightly.20230309)

Add 3.9.0-nightly.20230309 to community operators

/merge possible

/merge possible


Improve the v1.3.0-alpha.0 changelog

What this PR does / why we need it:
Improve the v1.3.0-alpha.0 changelog.
Does this PR introduce a user-facing change?:
NONE

/assign @kron4eg

/retest


python312Packages.allure-python-commons-test: 2.13.3 -> 2.13.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.allure-python-commons-test is: Just pack of hamcrest matchers for validation result in allure2 json format
meta.homepage for python312Packages.allure-python-commons-test is: https://github.com/allure-framework/allure-python
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.13.3 with grep in /nix/store/fpriayxlbr4d7m1qzk50vmyx73b2h409-python3.12-allure-python-commons-test-2.13.3
found 2.13.3 in filename of file in /nix/store/fpriayxlbr4d7m1qzk50vmyx73b2h409-python3.12-allure-python-commons-test-2.13.3

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.allure-behave
python311Packages.allure-pytest
python311Packages.allure-python-commons
python311Packages.allure-python-commons-test
python312Packages.allure-behave
python312Packages.allure-pytest
python312Packages.allure-python-commons
python312Packages.allure-python-commons-test

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fpriayxlbr4d7m1qzk50vmyx73b2h409-python3.12-allure-python-commons-test-2.13.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.allure-python-commons-test https://github.com/r-ryantm/nixpkgs/archive/c18b5b49b54f15f152231fc6038173d224071039.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c18b5b49b54f15f152231fc6038173d224071039#python312Packages.allure-python-commons-test

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fpriayxlbr4d7m1qzk50vmyx73b2h409-python3.12-allure-python-commons-test-2.13.3
ls -la /nix/store/fpriayxlbr4d7m1qzk50vmyx73b2h409-python3.12-allure-python-commons-test-2.13.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

16 packages built:

python311Packages.allure-behave
python311Packages.allure-behave.dist
python311Packages.allure-pytest
python311Packages.allure-pytest.dist
python311Packages.allure-python-commons
python311Packages.allure-python-commons-test
python311Packages.allure-python-commons-test.dist
python311Packages.allure-python-commons.dist
python312Packages.allure-behave
python312Packages.allure-behave.dist
python312Packages.allure-pytest
python312Packages.allure-pytest.dist
python312Packages.allure-python-commons
python312Packages.allure-python-commons-test
python312Packages.allure-python-commons-test.dist
python312Packages.allure-python-commons.dist

Maintainer pings
cc @evanjs for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.allure-python-commons-test is: Just pack of hamcrest matchers for validation result in allure2 json format
meta.homepage for python312Packages.allure-python-commons-test is: https://github.com/allure-framework/allure-python
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.13.4 with grep in /nix/store/qdgv15vhlfk82sgdis5hqdwnajy8xjlq-python3.12-allure-python-commons-test-2.13.4
found 2.13.4 in filename of file in /nix/store/qdgv15vhlfk82sgdis5hqdwnajy8xjlq-python3.12-allure-python-commons-test-2.13.4

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.allure-behave
python311Packages.allure-pytest
python311Packages.allure-python-commons
python311Packages.allure-python-commons-test
python312Packages.allure-behave
python312Packages.allure-pytest
python312Packages.allure-python-commons
python312Packages.allure-python-commons-test

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qdgv15vhlfk82sgdis5hqdwnajy8xjlq-python3.12-allure-python-commons-test-2.13.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.allure-python-commons-test https://github.com/r-ryantm/nixpkgs/archive/d5b2b1a9abb52328aa774d2f1923c56c8fbc068a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d5b2b1a9abb52328aa774d2f1923c56c8fbc068a#python312Packages.allure-python-commons-test

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qdgv15vhlfk82sgdis5hqdwnajy8xjlq-python3.12-allure-python-commons-test-2.13.4
ls -la /nix/store/qdgv15vhlfk82sgdis5hqdwnajy8xjlq-python3.12-allure-python-commons-test-2.13.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

16 packages built:

python311Packages.allure-behave
python311Packages.allure-behave.dist
python311Packages.allure-pytest
python311Packages.allure-pytest.dist
python311Packages.allure-python-commons
python311Packages.allure-python-commons-test
python311Packages.allure-python-commons-test.dist
python311Packages.allure-python-commons.dist
python312Packages.allure-behave
python312Packages.allure-behave.dist
python312Packages.allure-pytest
python312Packages.allure-pytest.dist
python312Packages.allure-python-commons
python312Packages.allure-python-commons-test
python312Packages.allure-python-commons-test.dist
python312Packages.allure-python-commons.dist

Maintainer pings
cc @evanjs for testing.


Feature/connectivity manager deprecations

Description
This PR updates the usage of the ConnectivityManager API to resolve deprecation warnings related to the use of NetworkInfo and its methods, which have been replaced by NetworkCapabilities starting from Android 10 (API 29). The key updates include:

Replacing NetworkInfo with NetworkCapabilities for checking network type and internet availability.
Using hasTransport(NetworkCapabilities.TRANSPORT_WIFI) to detect if the connection is Wi-Fi and hasCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET) to verify network availability.
Added backward compatibility for Android versions earlier than Marshmallow (API 23) by using the older NetworkInfo approach.
The deprecated allNetworkInfo method has been removed, as it's no longer needed with the newer API.
Issues

Resolved

Removes deprecation warnings related to the use of activeNetworkInfo, isConnected, type, TYPE_WIFI, and allNetworkInfo.
Enhances compatibility with modern Android versions starting from Android 10 (API 29).

Compatibility

Maintains compatibility with pre-API 23 Android devices using the legacy NetworkInfo API conditionally.

Testing
Verified on devices running Android 10+ to ensure that NetworkCapabilities functions as expected.
Tested on older Android devices to ensure proper functionality using the legacy implementation.
Open Question
There remains an open question of whether it is better to completely remove the legacy NetworkInfo API code or keep it alongside the new implementation for backward compatibility. Input and suggestions are welcome on whether supporting older Android versions is still a priority for this codebase. Should we remove support for Android API 21 and move it up to API 23?

Looks good @vicajilau. Thanks. merging.


Minor Improvements

add support for IgnoredBox, with updated documentation

add a debug to print the path to file, for printing the URL of the Diff Image in the console.

@puneet0191 are you working on this issue?
if not let me know I m interested in solving issues.

@prasadgujar sure go ahead and help me with the issues, let me know how it goes

Already started working on this issue. @puneet0191

Hi, guys maybe you have any status about this request? Or know a workaround for ignoredBoxes to work :)

@manciukass  IgnoredBoxes can be used by providing element locator so that the screenshot comparison happens only for that element and ignores the rest of the area on the image.
have you tried using :
seeVisualDiffForElement

Yes, I know that, but imagine a page with lots of elements, and let's say i would like to ignore half of the page from the bottom. I would not like to check every element separately. So idea to have possibility to check only some parts (big) of page would be really nice to have :)

@manciukass hmm you make sense, let me try to add this request as part of the new release which will be 1.6.0 coming out in some days

That would be wonderful :)


Efficient data structure

Implementing issue #38 and #39.

Can you please create new branch from main and move all new commits to this branch to commit structure more obvious


No correct behaivor of modal

From @sebrojas14 on March 28, 2016 9:2
The ion-autocomplete modal doesn't behaves like a normal modal and than gives me problems. For example, I can´t show a Ionic confirmation modal with the ion-autocomplete showing (appears behind the ion-autocomplete)
Copied from original issue: driftyco/ionic-platform-web-client#82

Hello! Thanks for opening an issue with us. As this is an issue related to the framework and not the Ionic platform I will be moving this issue to the correct repo. Thanks again!

This issue was moved to driftyco/ionic#5962


THRIFT-4303 Update ubuntu-xenial to use D 2.087.0

Update the ubuntu-xenial Docker container to use D 2.087.0.

[X] Did you create an Apache Jira ticket?  (not required for trivial changes)
[X] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[X] Did you squash your changes to a single commit?  (not required, but preferred)
[X] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

You can force-push to the branch, this will automatically update the PR.

Merged, thank you!


How do I specify smtp authentication?

I've used node-sendmail for sending emails locally from my server, and that's worked fine until recently, when my emails seem to get bounced more.
So I signed up for an SMTP sending service, and they provide me an SMTP server/port and a username/password, and recommended I configure my code to send through them.
I specified the smtpPort and smtpHost configurations... but how do I provide the authentication info? Is it auth: { user, pass } (as in nodemailer)?
I want to continue using this package so I don't have to change all my code to use a different library.

Any update on this? Is it already implemented or will it be in the future?
Thanks for helping :)

I found this code commented out in the source code:
https://github.com/guileen/node-sendmail/blob/f8f5666b13252e6e5940db59d8083fcf3a102047/sendmail.js#L165-L171
I think that definitively answers that this library doesn't support SMTP auth in its current form. Not sure if it will in the future. Unfortunately, I just had to switch my code to using nodemailer which works just fine. The conversion of the code was pretty straightforward.

Thanks for the help and quick reply. I also switched to nodemailer due to it's simplicity.


[ARCTIC-291] [Improvement]: support access Tables tab only for display table info only

Why are the changes needed?
fix https://github.com/NetEase/arctic/issues/291
people can use  ams table tab page only to display arctic table detail;
Brief change log
First, i will support an api to generate a token with catalog,db,table.
second , all tabls tab request url will be /ams/v1/tables/catalogs/{catalog}/dbs/{db}/tables/{table}, i will extract catalog,db,table from url, and check the token.
if token mismatch , return signatureExecption.
if match, return the result of query.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduces a new feature? ( no)
If yes, how is the feature documented? (not documented)

LGTM


Filling the blanks, plus trivial typos

[1] In "Navigation Fundamentals" / Intro paragraph
"This is why organizing and labeling your navigation links is so critical () usability and findability."
() something is missing here.
For example: " critical for usability and findability."
Or: " critical in purpose of usability and findability."
Etc...
[2] In "Navigation Fundamentals" / "Labeling" / "Consistent"
"When we have a lot of content on sites, we often need to use a combination () category and sub-links on pages."
() missing: "of"
should be: " a combination of category and sub-links "
[3] In "Navigation Fundamentals" / "Link Targets"
(the very last sentence)
"[...] such as the text web part where you can open elect to open a link in a new window."
>> should be: " [...] where you can elect to open [...] "
[4] In "Navigation in the Flat World" / "Local Navigation" / "Link to pages, not individual documents."
[4.1] "But when navigation points to a page with an embedded () instead of directly to a document, [...] "
() missing: "document"
should be: "But when navigation points to a page with an embedded document instead of directly to a document, [...] "
[4.2] (later in the same section)
"The page allows you to provide context for the document and it also and keeps the reader in the context of your site."
>> should be: " [...] and it also keeps the reader [...] "
[5] In "Navigation in the Flat World" / "Hub Navigation"
(below the picture...)
"Hub site navigation is established in the on the site that is declared to be the hub."
>> should be: " [...] established in the site that is declared [...] "

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

ID: e8576791-f019-67e1-3d31-fccefd6a165b
Version Independent ID: 4844c3bf-70cd-e5f4-2380-59db5ad32ae5
Content: Planning Navigation for the modern SharePoint experience
Content Source: SharePoint/SharePointOnline/plan-navigation-modern-experience.md
Service: sharepoint-online
GitHub Login: @LoreenLa
Microsoft Alias: loreenl

@officedocsbot assign @scanum

Greetings,
Thank you @scanum for taking ownership of this submission!
Meanwhile, the same point number "[5]" above has been addressed through Pull Request #935
Accordingly, point "[5]" can be dropped from the above list.
Kind Regards.

@a-zaf Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close

Hello @scanum, thanks for the notification!
So far, the changes have not been published online.
I will verify again when the changes do go live, but by looking at the pull request #946 ,
it seems that from the original list above, all points have been covered except point "[4.2]"
It's just a little typo, if it matters.
Otherwise, if you deem it unimportant, then please close this issue again and just "let it fly" :-)


Fix splitMap types bug

Repl:
https://stackblitz.com/edit/typescript-c5mry8?file=index.ts
Steps:
// Partial destructing argument throw ts error
highLvl: ({ payload }) => payload.lvl >= 50 ? payload.lvl : undefined,

// Full destructing argument works fine
highLvl: ({ settings, payload }) => payload.lvl >= 50 ? payload.lvl : undefined,

@Fl0pZz It is a typescript bug.
Move sample result to a variable:
  const source = sample({
    source: $settings,
    clock: someEvt,
    fn: (settings, payload) => ({ settings, payload }),
  });

  const { highLvl, lowLvl } = splitMap({
    source,
    cases: {
      highLvl: ({ payload }) => (payload.lvl >= 50 ? payload.lvl : undefined),
      lowLvl: ({ settings, payload }) =>
        payload.lvl < settings.MIN_LVL ? payload.lvl : undefined,
    },
  });


Fix missing CHERRY-PICKING, MERGING, REVERTING, etc

Adding tests for #828

Revert "fix CHERRY-PICKING display bug"

@NihilityT your fix does work, but I ended up reverting it to gain some performance. Not using the status branch name if we have it leads to two extra git calls on every prompt (see failures). Since we nearly always have a branch, we should try to use it as an optimization.
So the logic ends up being:

Get branch from status (but only if not in .git or bare)
Always call Get-GitBranch to add the necessary suffix, but with new parameters -Branch and -IsDotGitOrBare (which we know from before calling git status)
Only make the extra git rev-parse calls for .git/bare if we need to


reset settings

Type: Feature Request
i am new to vsstudio i messed up vs code setting and its not returning to normal and i cannot compile my any program please do something about it
VS Code version: Code 1.80.1 (74f6148eb9ea00507ec113ec51c489d6ffb4b771, 2023-07-12T17:22:07.651Z)
OS version: Windows_NT x64 10.0.22621
Modes:

\gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Learning Pathways Site not provisioning

Describe the issue
Learning Pathways Site not provisioning. Getting Unfortunately your site provisioning at least partially failed!
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Details (please complete the following information):**

PnP Correlation ID: [Provide a correlation ID for the provisioning action]
Reference Template: [please provide the URL of the template in the Look book site. It should be something like: https://lookbook.microsoft.com/details/{Template-ID}]
Primary language of the target tenant: [provide the LCID of the primary language of the target tenant]
Are you using a Tenant Global Admin account to provision the template: [Yes/No]

Additional context
Add any other context about the problem here.

Hi,
Thanks for using the Look Book site. In order to investigate the issue and provide feedback, we need the PnP Correlation ID of the provisioning request. Could you please share the Correlation ID? Thanks.

I'm having the same problem, tried 2 times.
PnP Correlation ID: 0e3c467f-ff8f-42be-b0d4-3f56c9bc34c6
Reference Template: https://lookbook.microsoft.com/details/3df8bd55-b872-4c9d-88e3-6b2f05344239
Primary language of the target tenant: not sure what the LCID is but the default language is English
Are you using a Tenant Global Admin account to provision the template: Yes

Hi,
Thanks for using the Look Book site. Based on the event log, we can see that you applied the template providing the URL of a site that was already in the recycle bin of SharePoint Online. Please, provide a different URL or empty the recycle bin. Thanks.


🐛 Continuation for search method not working

I tried the client.search method to get the youtube search results. I could get the first 20 videos from the first request, however, I could not get the next videos by using the continuation where it returned from the first request.
I found the issue, I am guessing it might be a similar problem.

update:
modified code as below works for me.
Endpoint.SEARCH,
# params=utils.filter(
#     dict(
#         continuation=continuation,
#         ctoken=continuation,
#     )
# ),
body=utils.filter(
     dict(
         query=query or "",
         params=params,
         continuation=continuation,
     )
),

Hi @shihs, thanks for opening this issue! I'll take a look into this now

I've been able to reproduce the issue on my end and assert that your code change fixes the issue

innertube version 2.1.10 released to PyPI with the fix :rocket:


example code

I'd like to use this module, but I'm new to firebase. Could you include some example code that I could easily modify?

Hi @shapr and thank you for your interest. This library is far from ready for production use and right now it doesn't target the new Auth mechanism (Oauth2). We're working on it and we'll hopefully reach a solution in that regard.
For now you can take a look at the tests to get an idea of how to use this library, or if you have a use-case in mind I could help you.

@sphaso Is this library still maintained? Are there any plans regarding further development?


Fix initial parameters not being converted based on paramType

Overview
When a value is set by initial parameters, it's always a string. It should be transformed based on the paramType.
Deliverables

[x] update code and test

Work for this issue will begin begin on branch work-liquid-labs/question-and-answer/63.


QuorumPreimage test sometimes fails

Unittesting agora.test.QuorumPreimage..
================================ ASSERT HANDLER ===============================
[source/agora/test/Base.d:1086] Assertion thrown during test: [source/agora/test/QuorumPreimage.d:160] Less than 1 spendables in block:
Height: 19, Prev: 0xa6bd...e299, Root: 0x055b...581e, Enrollments: []
Signature: 0x36ca27541775849c8bc86fcbf79577d1ae89e9c1a68116dd132101f52a2d9f6a0e59146295f78c14f5eeecd094db9ad184a79abdd7d41930671bc657af9cdaa7,
Validators: [252],
Random seed: [0xddfa...75cf],
Slashed validators: [],
Transactions: 1
Type : Freeze, Inputs (1): 0x6071...4a18:0x8482...c7d2
Outputs (2): boa1xza007gl...9g93(30,500,000), boa1xrc00kar...9lkl(30,500,000)
2021-05-12 05:07:06,468 Error [agora.utils.Utility] - Operation timed out: : Request timed-out
Module tests failed: agora.test.QuorumPreimage
----------------

From what I can tell the issue is the usage / implementation of the generateBlocks function. It calls addBlock which internally takes the last block's tx's and tries to spend them. But in the QuorumPreimage test the last block only contains freeze transactions, and those can't be spent.
I'm not even sure how this test didn't always fail.

We should probably use an approach similar to https://github.com/bosagora/agora/commit/77f12a93b10791f8135fe3d15ff2904494060789


feat: add documentation

Implements documentation for the project. We still have to document the dev flow, such as setting up the project and deploying it.
Closes #9.

For the TODOs, we first need to actually implement the App Script stuff, then document the process. I also need to get the permission details from the current implementation to add to the docs.


Add new API iter.CurrentBufferSize()

I am working on a project where it would be immensely useful to know how many bytes (approximately) each JSON object we pull of an iterator used. We can almost do this by passing in a io.Reader that counts the number of bytes read, but we still need to discount for anything in the buffer. With this simple API addition this is possible.
Alternatively we could count all bytes read, and provide an InputOffset() method like the stdlib json.Decoder, but that feels like going over the top?

And thanks for a great project btw! I love it! :-D

Thanks for the pr. Could you pls show an example how you're using this api?

Sure. Concretely the intent is to figure out how many bytes we read for each json object in some stream. To this end I wrap the base reader in a custom ByteCountingReader
// Implements io.Reader and counts total number of bytes read from R
type ByteCountingReader struct {
  N int64 // incremented on every .Read()
  R io.Reader
}

So pseudo-go looks something like:

bcr := &ByteCountingReader{0, r}
iter.Reset(bcr)
for {
beginN := bcr.N
beginBuf := iter.CurrentBufferSize()
obj = parseObj(iter)
endN := bcr.N
endBuf := iter.CurrentBufferSize()
reportBytesForObj(obj, (endN - beginN) + (beginBuf - endBuf))
}

Looking at it I realize that it requires some tight book keeping on the caller side - maybe it is cleaner having the Iterator do the book keeping and expose iter.InputOffset() instead, this would also help bring the API to jsoniter.Decoder and bring it on par with go 1.14...

Superseded by https://github.com/json-iterator/go/pull/493


Questions about tests

Hi @NicolasDorier
Unit tests run and pass perfectly well. However I found some friction in a very few. I would like to know if some of these suggestions make sense for you.

NBitcoin.Tests.Benchmark.BlockDownloadFromNetwork
using(var node = Node.Connect(Network.Main, "<IP_ADDRESS>", new NodeConnectionParameters())
You are running a bitcoin node in <IP_ADDRESS> but I think most devs don't.
Could that be replace by?:

node_ip = string.IsNullOrEmpty(node_id) ? "<IP_ADDRESS>" : node_ip;````

I assume someone who download the NBitcoin source code could be running a node in <IP_ADDRESS>. And even when that could be not true, it is more probable to have a node in localhost than in <IP_ADDRESS>.
So, You could define the "bitcoin_node_ip" with <IP_ADDRESS> to reduce just a bit the friction for new devs.

* **NBitcoin.Tests.Benchmark.BlockDirectoryScanSpeed
NBitcoin.Tests.Benchmark.BenchmarkBlockIndexing
NBitcoin.Tests.Benchmark.BenchmarkCreateChainFromBlocks
NBitcoin.Tests.Benchmark.BenchmarkTemplate**
```BlockStore store = new BlockStore(@"**(Z|E)**:\Bitcoin\blocks", Network.Main);```
Could that be replaced by?:
```var blocksFolder = Environment.ExpandEnvironmentVariables("%appdata%\\Bitcoin\\blocks\\");```

* **NBitcoin.Tests.MnemonicReference_tests.CanCreateBrainAddressFromNetwork**
Needs a node running in the default porr 8333. Other tests fail with:
```throw new InvalidOperationException("No bitcoin-qt or bitcoinq process running with rpc server on test net (\"bitcoin-qt.exe\" -testnet -server -rpcuser=NBitcoin -rpcpassword=NBitcoinPassword )");```
I think it was a good idea and this test could fail with the same kind message.

* **NBitcoin.Tests.MnemonicReference_tests.CanCreateBrainAddress**
Result Message:	NBitcoin.InvalidBrainAddressException : This block does not exists
(I didn't check why, yet)

So for the tests, I advise you to group them by Trait.
The "Benchmark" are not meant to be runned daily, I mostly use them for playing with performance improvement.
Same things with those without Trait, they are in a "testing the test" phase. Or in other words, some stuff that are not yet really stable that I will improve later.
The rest of the tests should run fine if you have a local bitcoind with RPC enabled.

What you say is clear, but you didn't answered my questions. Don't you think that would be better to use *%appdata%\Bitcoin\blocks* instead of Z:\Bitcoin\blocks? Because others are forced to create a map just because it's more comfortable for you (maybe it could be easier for others if some tests don't fail in theirs env. )

I understand, but it would prevent me to run it (I have SSD, not enough space for blockchain)
Since these tests are meant to be runned manually in debug mode, I expect people to just change the path when they need to run it. (If someone commit with their own folder, I don't really care, I will just change it manually when I need to measure some stuff)
I think I will change the traits of these test to something like "DontRunAutomatically" or "PlaygroundCode".

Ok, thank you Nicolas.


Restrict Descriptions of Proposals to 2000 characters

AC
Restrict Descriptions of all proposal types to max 2000 characters
Backend Story:
https://github.com/hypha-dao/dao-contracts/issues/209

@jmgayosso I added over 2000 characters and got the correct error message. Then I deleted most of the characters but the error message didnt go away and I can also not click "next".

@leonieherma1 I will update this to do easy debugger, now will contain a count of characters in the error message

@jmgayosso let me know when I can test

@leonieherma1 done, now you can test it on dev


Fix_RunEveryMinute

Proposed change

Fix RunEveryMinute failed when calls while current time is ##:59
Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[ ] The code compiles without warnings (code quality chek)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

Codecov Report

Merging #455 (d25cf2c) into dev (54b59a1) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #455      +/-   ##
==========================================
+ Coverage   54.52%   54.57%   +0.04%
==========================================
  Files          83       83
  Lines        4616     4616
  Branches      714      714
==========================================
+ Hits         2517     2519       +2
+ Misses       1870     1869       -1
+ Partials      229      228       -1

Flag
Coverage Δ

unittests
54.57% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...pp/NetDaemon.App/Common/Reactive/NetDaemonRxApp.cs
84.25% <100.00%> (ø)

...mon/NetDaemon.Daemon/Daemon/TextToSpeechService.cs
95.65% <0.00%> (+4.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Update README.md and remove obsolete env file

:exclamation: Outdated dependencies error

Installed version of react-dom (17.0.2) is older than required version (18.0.0)
Installed version of react-router-dom (5.2.0) is older than required version (6.0.0)
Installed version of @patternfly/react-core (4.135.0) is older than required version (5.0.0)
Installed version of react (17.0.2) is older than required version (18.0.0)
Installed version of @redhat-cloud-services/frontend-components-config (4.6.15) is older than required version (6.0.0)
Installed version of @patternfly/react-icons (4.11.0) is older than required version (5.0.0)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.42%. Comparing base (969eb57) to head (19256e0).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #399   +/-   ##
=======================================
  Coverage   90.42%   90.42%
=======================================
  Files         117      117
  Lines        2517     2517
  Branches      449      449
=======================================
  Hits         2276     2276
  Misses        239      239
  Partials        2        2


Javascript for..in breaks syntax highlighting

Issue Type: Bug
Using for-in statement breaks syntax highlighting for the for statement (after and including in keyword) and the next statement.
for (let type in attachments) {
  const desc = attachments[type];

is displayed as

Note how in and const are not highlighted.
VS Code version: Code 1.22.1 (950b8b0d37a9b7061b6f0d291837ccc4015f5ecd, 2018-04-06T02:26:57.615Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

Memory (System)
15.95GB (5.90GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.10

prettier-vscode
esb
1.3.1

docthis
joe
0.6.0

sublime-babel-vscode
jos
0.2.10

cpptools
ms-
0.17.3

node-debug2
ms-
1.22.7

debugger-for-chrome
msj
4.4.3

shader
sle
1.1.3

autoimport
ste
1.5.3

vscode-react-native
vsm
0.6.10

Sorry it seems the problem is caused by Prettier plugin, without it the highlighting is as expected.


getsocketname failed

Steps to reproduce
ssh -p port user@host

Expected behavior
connected to host

Actual behavior
get socketname failed:not a socket

Environment data

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      19013  1

@faithfulsco The repository is only for reporting PowerShell Core issues.


Error .stack bug with SwitchStatement / MemberExpression

It seems that when a MemberExpression throws a TypeError inside a SwitchStatement, the TypeError's .stack ends up blaming the SwitchStatement instead of the MemberExpression.  E.g., executing the following throws TypeError: Can't convert undefined to Object (correct) but the .stack property of the error blames the line containing switch (1) instead of the line containing undefined.hasNoProperties (incorrect):
switch (1) {
  case 1:
    return undefined.hasNoProperties;
}

O.K., so there are actually two bugs here:

In the MemberExpression (and Identifier) step functions, we pop the state off the state stack before throwing, so at best the next-innermost subexpression/statement will get the blame.

In Thread.prototype.callers we were accidentally overwrite the first-found value of pos, so we always end up blaming the outermost statement of the function.

Ooops.


[Impeller] Add a benchmark that captures advanced blends / backdrop filters

Currently, we are using the gallery app to benchmark performance. However, we've observed that in apps like Wonderous, the performance profile differs. This is because they frequently use backdrop filters and advanced blends. Advanced blends represent a known weak spot in the Vulkan backend, which we are actively addressing. It's crucial that we capture the results of those improvement efforts.
Integrating Wonderous into CI might not be the best strategy. Instead, we should create a benchmark inspired by the types of scenes they render.
This will also be significant for https://github.com/flutter/flutter/issues/131782. The Vulkan backend required a special investment, and it's essential to ensure that it consistently outperforms the OpenGL ES backend.

Looks like we already have backdrop filter coverage in animated_blur_backdrop_filter_perf__timeline_summary: https://flutter-flutter-perf.skia.org/e/?queries=test%3Danimated_blur_backdrop_filter_perf__timeline_summary
It is only running on moto g4 though.  We want this running on vulkan too, similar to what we have in https://github.com/flutter/flutter/issues/131782

There is also a test that has multiple backdrop filters but it doesn't seem to be run by impeller vulkan or impeller opengles backdrop_filter_perf_impeller_ios__timeline_summary: https://flutter-flutter-perf.skia.org/e/?queries=test%3Dbackdrop_filter_perf_impeller_ios__timeline_summary


When change ratio screen down 85%, copy item still keep init size (in 100% screen)?

Describe the bug
Reproduction
Live Reproduction
Steps to reproduce the behavior:

Turn on Devtool and open mode screen smaller
Drag item
Item copy still keeps its initial size

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Yes but the link to the repo (Your Live Reproduction link) points me to "https://codesandbox.io/...."


Font Features is inconsistent

Windows Terminal version (or Windows build number)
10.0.19043.1237, 1.11.2421.0
Other Software
No response
Steps to reproduce
I'm using the Nerd-Font port of FiraCode as my font for Windows Terminal. Using the new Font Features options, I've configured WT to use the alternate zero glyph which replaces the slash in the middle of the character with a dot. There have been a couple instances in which I've launched a terminal only for the zero glyph to have reverted back to the old style, though launching a new terminal does fix the issue.
Here is my settings config:
"font":
{
  "face": "FiraCode NF",
  "features":
  {
    "calt": 0,
    "ss04": 1,
    "zero": 1
  },
  "size": 12
},

I think I've found a way to reproduce this. It seems like it happens when you first launch windows terminal from a full restart.
Ignore that last one, it doesn't happen all the time. I can trigger the effect by dragging my window from a display at 100% scaling to one with 125% scaling.
Expected Behavior
The font should display with the correct 0 symbol
Actual Behavior
The zero symbol is incorrect.

Updated as I think I have a reproduction case now

Updated with a better reproduction example

Thanks, this is really helpful. I'm betting that when we added the new version of UpdateFont that took font features, we didn't consistently call it in all scenarios (like DPI change!). /cc @PankajBhojwani


Docker issue

0.593 E: The repository 'https://ppa.launchpadcontent.net/kicad/kicad-6.0-releases/ubuntu noble Release' does not have a Release file.
complete log:
[+] Building 1.1s (9/14)                                                                                docker:desktop-linux
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 575B                                                                                0.0s
 => [internal] load metadata for docker.io/library/ubuntu:latest                                                                          0.5s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [ 1/11] FROM docker.io/library/ubuntu:latest@sha256:2e863c44b718727c860746568e1d54afd13b2fa71b160f5cd9058fc436217b30                  0.0s
 => CACHED [ 2/11] RUN apt update                                                                                0.0s
 => CACHED [ 3/11] RUN apt install -y software-properties-common python3 python3-pip npm                                                  0.0s
 => CACHED [ 4/11] RUN apt install -y libgtk2.0-dev openctm-tools                                                                         0.0s
 => CACHED [ 5/11] RUN add-apt-repository --yes ppa:kicad/kicad-6.0-releases                                                              0.0s
 => ERROR [ 6/11] RUN apt update                                                                                0.6s
------
 > [ 6/11] RUN apt update:
0.112
0.112 WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
0.112
0.203 Hit:1 http://ports.ubuntu.com/ubuntu-ports noble InRelease
0.232 Hit:2 http://ports.ubuntu.com/ubuntu-ports noble-updates InRelease
0.264 Hit:3 http://ports.ubuntu.com/ubuntu-ports noble-backports InRelease
0.278 Ign:4 https://ppa.launchpadcontent.net/kicad/kicad-6.0-releases/ubuntu noble InRelease
0.297 Hit:5 http://ports.ubuntu.com/ubuntu-ports noble-security InRelease
0.314 Err:6 https://ppa.launchpadcontent.net/kicad/kicad-6.0-releases/ubuntu noble Release
0.314   404  Not Found [IP: <IP_ADDRESS> 443]
0.324 Reading package lists...
0.593 E: The repository 'https://ppa.launchpadcontent.net/kicad/kicad-6.0-releases/ubuntu noble Release' does not have a Release file.
------
Dockerfile:6
--------------------
   4 |     RUN apt install -y libgtk2.0-dev openctm-tools
   5 |     RUN add-apt-repository --yes ppa:kicad/kicad-6.0-releases
   6 | >>> RUN apt update
   7 |     RUN apt install -y --install-recommends kicad
   8 |     RUN npm install -g easyeda2kicad
--------------------
ERROR: failed to solve: process "/bin/sh -c apt update" did not complete successfully: exit code: 100

Same.


fix #6164: Reduce format failure for non default imports_granularity

Fixed: #6164
This patch reduces format failure for non default imports_granularity and correct the behavior around some edge cases.

Supports too long line of use ... that contains {}.
Fixes the width calculation of too long lines of use ....

Thank you for quick look!
I'll rethink and refactor. I'll mention once I've done.

@ytmimi Could you have a look? Thanks!

@ytmimi Could you have a look? Thanks!

I really appreciate you taking the time to make these changes. As I mentioned in https://github.com/rust-lang/rustfmt/pull/6165#pullrequestreview-2067260236 I likely won't have time to re-review this for a little while, but I'll follow up when I can

np. Take your time.

Rebased and conflict resolved. Updated using RewriteResult/RewriteError.

This is a friendly ping.
Three months passed.
@ytmimi Could you have a look? Let me know if I can help.

I understand your situation. (But... I wonder why don't you add more reviewers? It looks other PRs also are stuck with review.)
OK. Then, I'll pause to make this PR fresh. Let me know when you can review. Then, I'll rebase it.

@kenoss Thanks for taking the time to rebase the PR. At the moment this is a lower priority item for the team and I don't anticipate I'll get around to reviewing the implementation before the end of the year. It's hard to say when exactly I'll be able to revisit this one, but I'll be sure to reach out when I do.

Thank you for taking time!
FYI, it's a blocker for me to upstream it because I need it for CI. I don't want to diverge it.

Thanks! Comment added. All review comments resolved. (I'm not familiar with GitHub and a way of this repository. Which should I or you do Resolve conversation? I'll leave them.)

Yes, you can Resolve conversation the outdated conversation


Resolve logic in generating the root certificate

Problem
When a NOC provider is passed in the root certificate is not generated.
Change overview

Add logic to check if the root certificate is present and generate it when not present.

Rename  variable.

Add new variable to check if root cert is present.

Testing
Ran chip-tool-darwin which passes the NOC signer.

/rebase

Fast-tracking Darwin-specific change that reverts logic to what it used to be.


Create args parser

Make GraspingPlanner available through the command line.

Not doint this since I'm working with ROS arguments now.


Export valid attachment position

I'd like these to be exported so that lib clients can validate position on their side without duplicating this list.
In my case I built a wrapper for a tooltip component and I'd like my wrapper to also declare proptypes.
Also need ability to validate positions at app startup time (ie I create HOC using positions) instead or runtime
Maybe my usecase is not that clear but I'm sure it's not a big deal to export these positions anyway

How would someone consume this if it's being exported like that? Not sure I understand how it would work.

export const OnbooardingHotspot = React.createClass({
  propTypes: {
    hotspotPosition: ReactTether.PositionPropType.isRequired,
  }
})

just if I build a wrapper around react-tether that expose a position prop i'd like to not declare list of position myself in my own code.

Totally. I can't get it to work by calling TetherComponent.attachmentPositions though. Wondering if there is any way to do that? Maybe make it a getter or something. Seems kind of weird though.

These should fall through your component and into the PropTypes of this one if the attachment position is not valid, right? If this is not the case, I'll reconsider exporting them.


Nodes are no longer sending to other nodes

I just updated the nodes, and now the everywhere nodes aren't working.

I just rolled back to commit d9582c7fd878dd96d51b2684b4b2ae71bd601cc5 and that works, so it was commit 02355efc4494254240d1c5b20c9c58c4d318f543 that broke it.

same issue here

Yeah - sorry,  I did a bad commit. Should be fixed now if you update.

@niko2020 @GalaxyTimeMachine - should be fixed now. Apologies.

Now it works again. Thanks

fixed now. thanks for quick update!

Thanks! :)


[Smoke Tests]

Summary
Exclude the unbranded OpenAI library from smoke testing.

/check-enforcer override


any upgrade instructions to 0.2.0 and 0.3.0?

I noticed there are new releases 0.2.0 and 0.3.0 on npmjs.com.  But in this GitHub repo, these 2 new releases are not listed under Releases.  I tried to upgrade to these new releases using yarn upgrade, and met strange errors when running in browser, like:
Cannot read property 'isAllowed' of undefined

I wrote my code when I was using 0.1.3.  Currently I am using 0.1.4 without a problem.  Maybe I need to make some changes to adapt to 0.3.0?  Is there any instructions or release notes on how to upgrade to these new releases?
Thanks!
Zhu Ming

Hey @blackpuppy,
I have updated the Releases section.
Can you share your code with me? Just the definition of your perimeter and the way you use it in your view.
There shouldn't be any noticeable breaking changes in v0.3.0. All e2e tests has passed and I have updated vue-kindergarten in 2 of my projects and didn't have any issue :thinking:

Hi, @JiriChara,
I described my use case below.
I am using Laravel 5.4 with laravel-mix.
// resources/assets/js/app.js
import VueKindergarten from 'vue-kindergarten';
import store from './store/';
import child from './child';

Vue.use(VueKindergarten, {
    child,
});

const app = new Vue({
    el: '#app',
    template: '',
    components: { MainApp },
    router,
    store,
});

And child is defined as:
// resources/assets/js/child.js
export default (store) => {
    return store;
};

The BasePerimeter is using store's getters:
// resources/assets/js/perimeters/BasePerimeter.js
import { Perimeter } from 'vue-kindergarten';

export default class BasePerimeter extends Perimeter {
    isAdmin() {
        const allowed = !!this.child
            && this.child.getters['auth/isLoggedIn']
            && this.child.getters['auth/isAdmin'];
        return allowed;
    }

    isAccountManager() {
        const allowed = !!this.child
            && this.child.getters['auth/isLoggedIn']
            && this.child.getters['auth/isAccountManager'];
        return allowed;
    }
}

And I have only 1 perimeter:
// resources/assets/js/perimeters/userPerimeter.js
import BasePerimeter from './BasePerimeter';

export default new BasePerimeter({
    purpose: 'user',

    govern: {
        'can viewOtherUserTrxns': function () {
            return this.isAdmin() || this.isAccountManager();
        },
    },
});

Eventually I use 2 ways to check the permission in Vue components:
// resources/assets/js/components/TransactionTable.js
<template>

</template>

All is quite normal except this.$user.isAllowed(...) in data().  In 0.1.4, it works fine.
Thanks!
Zhu Ming

Hey @blackpuppy,
this one is little bit tricky! There was an issue that components haven't been updated after user logged in: https://github.com/JiriChara/vue-kindergarten/issues/8. The problem was that all exposed methods and purposes like $user, $isAllowed etc. were not computed properties and therefore not updating.
They are now computed properties and therefore not available in the data() method, but you still have 2 options:
data() {
  if (this.$options.computed.$user.isAllowed('viewOtherUserTrxns')) {
    // ...
  }
  // ...
}

or
import { createSandbox } from 'vue-kindergarten';

// ...
data() {
  const sandbox = createSandbox(perimeters: [
    userPerimeter,
  ]);
  if (sandbox.user.isAllowed('viewOtherUserTrxns')) {
    // ...
  }
  // ...
}

I would recommend you to upgrade to latest vue-kindergarten, because it includes several bugfixes and includes some performance improvements. I will try to find a workaround to make it work again.

Hi, @JiriChara,
The first option looks simpler but does not work.  It gives me an warning:
[Vue warn]: Error in data(): "TypeError: this.$options.computed.$user.isAllowed is not a function"

The second option is a bit cumbersome as I need to import a few stuff.  In addition, I am using store's getters in child.  Maybe I should try the child reactivity fix here.
Thanks!
Zhu Ming

I am closing this as expected behavior, because computed properties are not working in the data() method. I think it's better practice to hide values that users shouldn't see in the template, or pass the correct data from server.


chore: fix quoting for weekly updater

add missing quote to start of body

/cc @hh

/approve
/lgtm

@hh, thank you!


Can't Login on A Mobile Device

Running a simple devise login app. Devise won't login on a mobile device. (specifically ios)
Posted below are the terminal responses I get. Sorry for the limited information, I'm new at this :/
Mobile (failed)
Started POST "/users/sign_in" for <IP_ADDRESS> at Thu Mar 08 11:32:50 -0500 2012
  Processing by Devise::SessionsController#create as HTML
  Parameters: {"authenticity_token"=>"N6r8grQHhY+ckmd9RMoLfLgUyntrWCiI9NbMNduEht8=", "commit"=>"Sign in", "utf8"=>"✓", "user"=>{"remember_me"=>"0"<EMAIL_ADDRESS>"password"=>"[FILTERED]"}}
  User Load (0.2ms)  SELECT "users".* FROM "users" WHERE "users"."email" =<EMAIL_ADDRESS>LIMIT 1
  AREL (0.4ms)  UPDATE "users" SET "last_sign_in_at" = '2012-03-08 16:32:46.804390', "current_sign_in_at" = '2012-03-08 16:32:50.483731', "updated_at" = '2012-03-08 16:32:50.484364', "sign_in_count" = 24 WHERE "users"."id" = 1
Redirected to http://<IP_ADDRESS>:3000/
Completed 302 Found in 121ms

Local
Started POST "/users/sign_in" for <IP_ADDRESS> at Thu Mar 08 11:29:02 -0500 2012
  Processing by Devise::SessionsController#create as HTML
  Parameters: {"authenticity_token"=>"Rt0B0bA4VKDjuMhNNjIiB1OfkFZnm5hKIdiHbhq19tU=", "commit"=>"Sign in", "utf8"=>"✓", "user"=>{"remember_me"=>"0"<EMAIL_ADDRESS>"password"=>"[FILTERED]"}}
  SQL (0.8ms)   SELECT name
 FROM sqlite_master
 WHERE type = 'table' AND NOT name = 'sqlite_sequence'

  User Load (0.4ms)  SELECT "users".* FROM "users" WHERE "users"."email" =<EMAIL_ADDRESS>LIMIT 1
  AREL (0.6ms)  UPDATE "users" SET "current_sign_in_at" = '2012-03-08 16:29:02.907101', "sign_in_count" = 21, "last_sign_in_at" = '2012-03-08 16:23:15.726511', "current_sign_in_ip" = '<IP_ADDRESS>', "last_sign_in_ip" = '<IP_ADDRESS>', "updated_at" = '2012-03-08 16:29:02.907762' WHERE "users"."id" = 1
Redirected to http://localhost:3000/
Completed 302 Found in 338ms

Thanks for any help

The user is being found. I can see that because there is an UPDATE query to the database that updates last_sign_in_at field and sign_in count. The problem is that it is redirecting it by default. You should probably send your request as json or another navigational format. If you still have questions, please use the mailing list.


Ignore test that sends a content body with a 204 response

204 means 'No Content' so this obviously violates the RFC

Linux Build 7605 is now running using a merge of bfd11ad962ff35c5c1ebd0214b50d62a4b16fe9c

Linux Build 7605 outcome was SUCCESS using a merge of bfd11ad962ff35c5c1ebd0214b50d62a4b16fe9c
Summary: Tests passed: 3116, ignored: 365 Build time: 00:49:50


Bug: Lualine setup kills nvim intro.

Self Checks

[ X] I'm using the latest lualine and neovim nightly.
[ X] I didn't find the issue in exsisting issues or prs.

How to reproduce the problem
Put lualine.setup {} in the config
Start nvim
Actual behaviour
Lualine kills the intro screen

Expected behaviour
Starting up nvim should display the intro

Minimal config to reproduce the issue
require('lualine').setup{}

Config

lualine.setup{
options = {
theme = 'gruvbox_material',
},
};

I'm aware of this and looked into it for some time now. Seems like all lua based plugins cause this? I'm not sure tbh but that's all I found while trying to debug it

I'm aware of this and looked into it for some time now. Seems like all lua based plugins cause this? I'm not sure tbh but that's all I found while trying to debug it

Other lua don't cause this as per my experience.
Lualine by default doesn't kill the intro but as soon as just require('lualine').setup{} added to the config, it kills the intro.

I just noticed this as well, I have other Lua packages installed that doesn't cause it.

Seems to be related to the vim.cmd call used here. If I comment that one out the intro is not cleared. https://github.com/hoob3rt/lualine.nvim/blob/9726824f1dcc8907632bc7c32f9882f26340f815/lua/lualine/highlight.lua#L29

No this is realated to async loading lualine

Switched to Lualine recently (thanks!), and noticed the same issue. After some digging, I found it was related to async loading (as @shadmansaleh suggested). I think this is because Lualine does the theme asynchronously and the redraw will remove the intro msg (some info in :h :intro).
I worked around this by adding an option to apply theme synchronously in my fork. Not sure if it's right, but it seems to work for me.

Just solved this for feline.nvim - you might find the information inside helpful.

Fixed in #311

As of 2d6108e07f5888a42dee37598570bac990ca1297 this seems to be an issue again.
(Very) minimal config:
call plug#begin("/tmp/intro/.local/share/nvim/plugged")
Plug 'nvim-lualine/lualine.nvim'
call plug#end()

lua << END
require'lualine'.setup {

}
END

Checking out to 5f68f070e4f7158517afc55f125a6f5ed1f7db47 solves the problem.

I have even more minimal config without lualine. Simply setting statusline from a scheduled function causes it while setting outside of scheduled function doesn't.
-- file: minimal_init.lua
vim.o.stl = 'asdf'
vim.schedule(function () vim.o.stl = 'jkl;' end)

nvim --clean -u minimal_init.lua

With current redraw mechanics we can't quite avoid setting statusline from scheduled context . I guess we could try report it to nvim devs and see what they think about it if it's a bug or intended behavior.

I also experience this issue on edca2b0 . Could we reopen it to indicate it is not resolved?

I encounter this problem with with winbar is setup.

Still an issue for me as well.

I'm also experimenting this issue.

 I got sick of the issue and I just wrote a very simple plugin (~100 lines) that works around this issue and displays a simple neovim logo.
If you are interested: https://github.com/eoh-bse/minintro.nvim

Also experiencing this issue.

I am having this issue only when I set section_separators to something other than ''

I am having this issue only when I set section_separators to something other than '' Modifying component_separators works fine.

I've  tried this and it worked. Setting the section_separators to blank  fix the issue. Guess we now know what is the cause

I added cmdheight=2 then execute intro command, and then cmdheight=1, I have read that the separator for lualine causes a redraw, so I think calling the intro after the setup would work but I'm not sure to be honest but it works for me.

It works but I have the anoying Press ENTER or type command to continue. Please let me know how to disable this when using vim.cmd using lua

I forget about this, and it seems like it was fixed now May 27, 2024

I can confirm that non-empty section_separators work with lualine and neovim 0.10.1, and the intro does not disappear.


[Idea] Allow Plural Versions of Command Line Options

Is your feature request related to a problem? Please describe.
The documentation uses "bundle" and "bundles" so to avoid frustration we should match for both cases. This can even extend to other resources, such as "profiles", "clusters", "clusterspecs".
Describe the solution you'd like
We should match more than the singular for command line options.  Mostly a quality of life improvement.
Also in the CLI we mention plural:
bundle        Manage configuration bundles
cluster       Manage clusters
Describe alternatives you've considered
Make sure all documentation only includes the singular if we are not going to match plural. This isn't an issue in command line options showing how to do something, this is an issue with wording in descriptions of concepts.
Additional context
➜  ~ arlonctl bundles
Error: unknown command "bundles" for "arlonctl"
Did you mean this?
bundle
Run 'arlonctl --help' for usage.
Aha! Link: https://pf9.aha.io/features/ARLON-372

Hey team! Please add your planning poker estimate with Zenhub @bcle @jayanth-tjvrr @Rohitrajak1807 @ShaunakJoshi1407

Backport to v0.10 branch too.

FYI bundles, clusterspecs, profiles, etc... have all been replaced with newer constructs. Not sure how much this still applies. What exactly is being asked here? What commands? What options?

FYI bundles, clusterspecs, profiles, etc... have all been replaced with newer constructs. Not sure how much this still applies. What exactly is being asked here? What commands? What options?

From Mike's descriptions I think he means all the commands which are direct children of the root arlon command

To recap, I think we can do this for arlon app ... in v0.10
@mpetason  wanted this for commands like arlon bundle(s), arlon profile(s) - for v0.9 (deprecated but not removed in v0.10)


chore(package): update cz-conventional-changelog to version 3.0.2

Closes #10

:tada: This PR is included in version 3.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[WIP] Write json

@robbyt could you please have a look at these spec tests for the function

@ghoneycutt please apply this patch:
diff --git a/spec/functions/sensu_sorted_json_spec.rb b/spec/functions/sensu_sorted_json_spec.rb
index df486c5..8b5e1d9 100644
--- a/spec/functions/sensu_sorted_json_spec.rb
+++ b/spec/functions/sensu_sorted_json_spec.rb
@@ -1,11 +1,18 @@
 require 'spec_helper'
-describe 'sensu_write_json' do
+describe 'sensu_sorted_json' do
+  it { is_expected.not_to eq(nil) }
+
   describe 'should error when expected' do
     it { is_expected.to run.with_params.and_raise_error(Puppet::ParseError, /0 for 1, 2 or 3/) }
     it { is_expected.to run.with_params(1,2,3,4).and_raise_error(Puppet::ParseError, /4 for 1, 2 or 3/) }
   end

-  valid_hash = {"bind_addr"=>"<IP_ADDRESS>","client_addr"=>"<IP_ADDRESS>","ports"=>{"http"=>-1,"https"=>8500,"rpc"=>8567},"start_join"=>["<IP_ADDRESS>","<IP_ADDRESS>","<IP_ADDRESS>"]}
+  valid_hash = {
+    "bind_addr"=>"<IP_ADDRESS>",
+    "client_addr"=>"<IP_ADDRESS>",
+    "ports"=>{"http"=>-1,"https"=>8500,"rpc"=>8567},
+    "start_join"=>["<IP_ADDRESS>","<IP_ADDRESS>","<IP_ADDRESS>"],
+  }

   valid_pretty_hash = <<-END.gsub(/^\s+\|/, '')
     |{
@@ -26,7 +33,7 @@ describe 'sensu_write_json' do

   describe 'should return correct results' do
     it 'should run with valid hash' do
-      should run.with_params(valid_hash).and_return(valid_pretty_hash)
+      is_expected.to run.with_params(valid_hash, true, 4).and_return(valid_pretty_hash)
     end
   end
 end

@robbyt Thanks for the patch


FIX error when compiling less

When run grunt for the first time this error appeared :
NameError: variable @zindex-modal-background is undefined in src/less/lib/overlay.less on line 26, column 22

Thank you guys, your issue seems solved by other means.


Schema Manager

Add UI to show Schemas and VSchemas for a given keyspace, shard and tablet.

This change is

Review status: 6 of 8 files reviewed at latest revision, 4 unresolved discussions, some commit checks broke.

web/vtctld2/src/app/app.component.css, line 6 [r1] (raw file):
Previously, enisoc-bot (Anthony Yeh) wrote…
> We should be able to target this better to avoid `!important`.

Done.

web/vtctld2/src/app/schema/schema.component.css, line 8 [r1] (raw file):
Previously, enisoc-bot (Anthony Yeh) wrote…
> Why do these need to be deep selectors (`>>>`)? The name prefix makes them look like they should only be used in this component anyway.

The table is generated by the prime component and so can only be accessed by the deep selector.

web/vtctld2/src/app/schema/schema.component.ts, line 13 [r1] (raw file):
Previously, enisoc-bot (Anthony Yeh) wrote…
> Spaces

Done.

web/vtctld2/src/app/schema/schema.component.ts, line 15 [r1] (raw file):
Previously, enisoc-bot (Anthony Yeh) wrote…
> You shouldn't need to initialize to `undefined`.

Done.-

Comments from Reviewable

LGTM after merge conflicts are resolved.


adding annotation information to set sidecar container's seccompprofiletype

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Adding information about annotation dapr.io/sidecar-seccomp-profile-type introduced with this PR

Issue reference
#dapr/5916

LGTM. Can you also please add a sentence in the Kubernetes production page? https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-production/#recommended-security-configuration

LGTM. Can you also please add a sentence in the Kubernetes production page? https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-production/#recommended-security-configuration

Could you provide a specific location as to where to add the sentence and what the sentence should say? you linked recommended-security-configuration section of the page which has nothing related to sidecar container configurations in it. I see https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-production/#sidecar-resource-settings, but that section is more related to how to define how much resources can sidecar consume, also not related to sidecar's securityContext. so I am a little confused on where and what to add on Kubernetes production page

LGTM. Can you also please add a sentence in the Kubernetes production page? https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-production/#recommended-security-configuration

Could you provide a specific location as to where to add the sentence and what the sentence should say? you linked recommended-security-configuration section of the page which has nothing related to sidecar container configurations in it. I see https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-production/#sidecar-resource-settings, but that section is more related to how to define how much resources can sidecar consume, also not related to sidecar's securityContext. so I am a little confused on where and what to add on Kubernetes production page

@ItalyPaleAle what are your thoughts on this?

@JaydipGabani - What @ItalyPaleAle is referring to, is that there needs to be a section in the docs that describe the "why" and "when" to use this annotation. Having this in the annotation file is not enough since it does not describe a use case and best practices for when to set this. If you read through the whole issue on security settings that brought you to implement this to start with, how do we capture that explanation in the docs for developers.

@ItalyPaleAle @msfussell Thanks for making things clear. I am working on changes will update PR by tomorrow

@ItalyPaleAle @msfussell PTAL, I made the appropriate changes

Please keep this open!

@ItalyPaleAle @yaron2 wondering whats the reason for keeping this PR open?

@JaydipGabani a @dapr/maintainers-docs will need to review this, I am not a maintainer here :(


Ios Bundle assets and other Xcode settings

Hey there.
I have a couple of somewhat basic questions:

How can I move assets ( images, text, scripts, etc) to the to the compiled IOS app?
I know you mentioned on this issue that I should use this generator: https://github.com/floooh/fips/issues/187

But do you have any pointers on how to do it with an MacApp and iOS app?

How can I change Xcode build settings? for example every time I do ./fips gen I need to change the Architectures from armv7 arm64 x86_64..  to $(ARCHS_STANDARD)

Hi, for copying asset files I'd recommend using the (somewhat new) scripts in the fips_utils project:
https://github.com/fips-libs/fips-utils
The fipsutil_copy() macro accepts iOS and macOS specific overrides for the destination directory, so that you can place the asset files into the right bundle subdirectory:
https://github.com/fips-libs/fips-utils/blob/7703b78914aea29910abd3c5242dd2ab4e64231c/fips-files/generators/copy.py#L13-L17
I'm surprised that editing the Architectures Xcode build setting is necessary, what sort of error are you getting?
The value is defined in fips here:
https://github.com/floooh/fips/blob/6fff11c39c60178fd9987913a5684c317bc4db61/cmake-toolchains/ios.toolchain.cmake#L16
You can override the ios.toolchain.cmake file in your own project (basically copy the file from fips into your own project, and modify it as needed), how this works is described here:
https://floooh.github.io/fips/docs/crosscompiling/#under-the-hood
I'll close the issue, if there are any more questions or something isn't working, just open it again :)

Hi, thanks for the reply and information =)
This is the error that I was getting, I think it started after I updated my mac to Catalina:

This is the error that I was getting, I think it started after I updated my mac to Catalina:

I remember seeing a similar error message too, but the problem went away after doing a "fips clean" and "fips gen" again (so basically regenerating the Xcode project files from scratch), so I shrugged it off so far.
It might make sense to remove armv7 from the architectures list though, since 32-bit apps are no longer supported anyway (AFAIK at least).


Add learning progress bar

Add a progress bar showing the status of learning.
This task requires writing custom GridSearchCV function.
I found that this may be achieved this way. I also found this lbrary which seems to be an easy to use progress bar module.

Is there any way to hack or find other implementations of those search functions? Grid search does not seem to be the optimal solution for our use case. We might consider RandomSearchCV or even BayesSearchCV (which can be found here). We might take a look and find an open source implementation of those or (even better) an implementation that takes callback function as a parameter, which we could use to update the progress bar.


Clean and check Android user home

Previously, the Android tests might pollute global ~/.android, resulting
in unexpected failures. This PR set Android user home to test temp directory,
and check Android user home after the build.
Closes https://github.com/gradle/gradle-private/issues/3380

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test and merge

Your PR is queued because #17112 is being built.

OK, I've already triggered a build for you.


Triplet Loss Error

You have a nice implementation however I believe there is an error in your loss metric. You calculate a normal triplet loss instead of that implemented in the paper. They utilized a "double" triplet so to speak:

You just need a second distance and to add the other triplet.
All the Best,
Cade

Thank you for your comment! I fixed this problem now!
All the Best,
Gerard

You have a nice implementation however I believe there is an error in your loss metric. You calculate a normal triplet loss instead of that implemented in the paper. They utilized a "double" triplet so to speak:

You just need a second distance and to add the other triplet.
All the Best,
Cade

Thank you for your comment! I fixed this problem now!
All the Best,
Gerard


Laravel 10.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.
Before merging, you need to:

Checkout the l10-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: The latest version of Laravel has a dependency for phpunit/phpunit of ^9.5.10. If you would like to also upgrade your tests, you may run the PHPUnit 9 Shift for free.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-emojione.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "christofferok/laravel-emojione": "dev-l10-compatibility",
    }
}

Finally, run: composer update

Please let me know when this PR will be deployed to master branch - we are using this package and we want to upgrade our app asap to L10. Thanks!


Zero scores - white codels

In solving problem of coloured pixels remaining after backspace/repaint, created problem of codels turning white after every space between words

Need to discard spaces/ zero scores when repainting canvas


added timeout in accept_friend

The Client.accept_friend method calls the Client.wait_for method without any timeout parameter. As a result, when asyncio.wait_for is called in Client.wait_for, timeout = None is passed which causes a block until the future check=lambda f: f.id == user_id completes. This pull request just adds a 60 second timeout so that, in the event that the future never completes, accept_friend does not block.

Changed timeout default from 60 seconds to None to be backwards compatible with existing implementations of the method.


Implementing Official CVE Feed alpha

Supporting blog to the announcement of the CVE feed
Covers implementation details, design considerations
Links to original blog

Preview: https://deploy-preview-330--kubernetes-contributor.netlify.app/blog/2022/09/12/k8s-cve-feed-alpha/
/sig security docs
/area security
related to https://github.com/kubernetes/website/pull/35608 (Need both to be merged together)
/hold

Pushkar,
I've created this hackmd file because I wanted to rewrite parts of your blog post for clearer language, and that's easier than giving you a bunch of individual edits:  https://hackmd.io/-5mXZZQWQyaTbU45O6AHhQ

Done editing the Hackmd file.  Please review it; I think it improves some of the text around describing your implementation.  Particularly, check how I converted passive to active voice in a lot of sentences.

/hold
due to https://github.com/kubernetes/contributor-site/issues/281

@jberkus can you please grant me access to hackmd link you shared?

Pushkar: it should be public. What are you seeing when you try to access it?

Aha, never mind, found the problem.
Use the view link if you don't have a hackmd account: https://hackmd.io/-5mXZZQWQyaTbU45O6AHhQ?view

Thanks @jberkus and @sftim for the wonderful input via hackmd. I have updated the PR with the suggested changes!

/hold
due to #281

@sftim could you please check this PR: https://github.com/kubernetes/contributor-site/pull/331 fixing the issue for future blog posts with PublishDate variable

Given that this will merge into main which already has the fix from #331
/hold cancel
:crossed_fingers:


Create Random Quote Generator

@Jishnu2608, may I know the reason of closing PR.

Something just went wrong over the code. So I decided to close this PR and make a new one

Sure, make a new pr @Jishnu2608


Make file type exported ('File')

type file struct { ... } is unexported. This means that clients cannot define their own files, nor even FileSettings.
This makes it hard to add small attachments, e.g. vCards.
The workaround is to write every attachment to a temporary file. But this is painful because it's wasteful and it needs extra cleanup code.
Far better would be to export type file as File.

@rickb777 makes sense..

It's not strictly necessary, as I later discovered, but it would still be nice to have because it would be more obvious to new users.


fix(providersModal): Replace TextInput with PF4

closes #181

Deploy preview for project-koku ready!
Built with commit d68b9b891949903702921c80db47d3e2b8cc0e0f
https://deploy-preview-189--project-koku.netlify.com

Codecov Report

Merging #189 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #189   +/-   ##
=======================================
  Coverage   91.86%   91.86%
=======================================
  Files         113      113
  Lines        1230     1230
  Branches      148      148
=======================================
  Hits         1130     1130
  Misses         91       91
  Partials        9        9

Impacted Files
Coverage Δ

src/pages/providersModal/attributeField.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Overview for SDK version and corresponding package versions range

I would like to update from SDK version 9.3.0 to 9.6.0, but i don't know (up to) which Firebase package versions i may enter in my pubspec.yaml in order that the ios build, deploy and run works fine.
Is there an overview which SDK versions correspond to the published Firebase package version ranges?
E.g. when i tried to build with version 9.6.0 i got lots of errors since the package versions of the several Firebase plugins i entered in pubspec.yaml seem to be too new.
Sorry for the dumb question, perhaps i'm completely wrong here, then just correct me. Thank you.

Hey there :wave: - this does not contain quite enough details to help https://stackoverflow.com/help/how-to-ask
Additionally, why wouldn't you want to use current versions (that is, 10.1.0?)
You mention pubspec.yaml so I will assume you mean FlutterFire.
I think  you want to add a variable to your Podfile. that defines the version that you want and all of FlutterFire will use it:
https://github.com/firebase/flutterfire/blob/294e1085ae7af92573657489b78ae0a82ab5e4b4/packages/firebase_core/firebase_core/ios/firebase_core.podspec#L6
10.1.0 should work, assuming you are on the most up to date versions of your FlutterFire dependencies

Hi Mike,
thanks for answering. I tried with version 10.1.0 but get this error when calling
pod install --repo-update
Updating local specs repositories
Analyzing dependencies
Pre-downloading: FirebaseFirestore from https://github.com/invertase/firestore-ios-sdk-frameworks.git, tag 10.1.0
FirebaseFirestore/AutodetectLeveldb: subspec would include leveldb if used directly or by default.
cloud_firestore: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
cloud_functions: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
firebase_analytics: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
firebase_app_check: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
firebase_auth: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
firebase_core: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
Ignoring ffi-1.15.5 because its extensions are not built. Try: gem pristine ffi --version 1.15.5
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/dependency.rb:313:in to_specs': Could not find 'rexml' (~> 3.2.4) - did find: [rexml-<IP_ADDRESS>] (Gem::MissingSpecVersionError) Checked in 'GEM_PATH=/Users/valley/.gem/ruby/2.6.0:/Library/Ruby/Gems/2.6.0:/opt/homebrew/Cellar/cocoapods/1.11.3/libexec:/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/gems/2.6.0', execute gem envfor more information 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1449:inblock in activate_dependencies'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1438:in each' 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1438:in activate_dependencies'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1420:in activate' 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems.rb:223:in rescue in try_activate'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems.rb:216:in try_activate' 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:123:in rescue in require'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:34:in require' 	from /Users/valley/.pub-cache/hosted/pub.dartlang.org/firebase_crashlytics-3.0.4/ios/crashlytics_add_upload_symbols:1:in '
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/dependency.rb:313:in to_specs': Could not find 'rexml' (~> 3.2.4) - did find: [rexml-<IP_ADDRESS>] (Gem::MissingSpecVersionError) Checked in 'GEM_PATH=/Users/valley/.gem/ruby/2.6.0:/Library/Ruby/Gems/2.6.0:/opt/homebrew/Cellar/cocoapods/1.11.3/libexec:/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/gems/2.6.0', execute gem envfor more information 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1449:inblock in activate_dependencies'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1438:in each' 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1438:in activate_dependencies'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/specification.rb:1420:in activate' 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems.rb:217:in try_activate'
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:123:in rescue in require' 	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:34:in require'
from /Users/valley/.pub-cache/hosted/pub.dartlang.org/firebase_crashlytics-3.0.4/ios/crashlytics_add_upload_symbols:1:in <main>' /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in require': cannot load such file -- xcodeproj (LoadError)
from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in require' 	from /Users/valley/.pub-cache/hosted/pub.dartlang.org/firebase_crashlytics-3.0.4/ios/crashlytics_add_upload_symbols:1:in '
firebase_crashlytics: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
firebase_messaging: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
firebase_storage: Using Firebase SDK version '10.0.0' defined in 'firebase_core'
[!] CocoaPods could not find compatible versions for pod "FirebaseFirestore":
In Podfile:
FirebaseFirestore (from https://github.com/invertase/firestore-ios-sdk-frameworks.git, tag 10.1.0)
cloud_firestore (from `.symlinks/plugins/cloud_firestore/ios`) was resolved to 4.0.4, which depends on
  Firebase/Firestore (= 10.0.0) was resolved to 10.0.0, which depends on
    FirebaseFirestore (~> 10.0.0)

And i've checked that i'm using the most recent Firebase package versions in my pubspec.yaml.

The Podfile is the important part, did you put:
$FirebaseSDKVersion = '10.1.0'

in your Podfile? Because right now firebase_core is still not picking it up, it's trying to use 10.0.0 leading to inevitable conflict.

I name the SDK version here in my Podfile (which worked fine for e.g. version 9.3.0):
target 'Runner' do   use_frameworks!   use_modular_headers!   pod 'FirebaseFirestore', :git => 'https://github.com/invertase/firestore-ios-sdk-frameworks.git', :tag => '10.1.0'   flutter_install_all_ios_pods File.dirname(File.realpath(__FILE__)) end
I've never seen the variable you mentioned above. Is this one required from version 10 on?

I link-quoted the code above that uses the variable and incidentally shows how it is used.
It has always been available but not publicly documented as it will require you - the integrator - to bump the version manually as needed if you use it.
But that's your explicit goal, to manually ingest firebase-ios-sdk 10.1.0, so I'm showing you how to get what you want.
How did it go when you tried it?

It actually works, i.e. build and deployment run fine now with the most recent versions.
So that missing variable was the crucial part.
Thank you very much.

Fantastic! Glad that worked - remember to remove it later when you have the version you want, so you are not forever responsible for updating it. The release of the next version this PR will be your clue that you may safely remove it https://github.com/firebase/flutterfire/pull/9869

Does it matter if that line just stays there?
I don't perform these iOS SDK version updates that often.

Yes it matters how long it stays there, it should be considered "for emergency use only" as you are now responsible for all compatibility issues between firebase-ios-sdk and flutterfire modules. I'm not sure I personally would sign up for that responsibility long term. But every project is different. But if you never update it, and you leave it there, you will eventually have a compile failure.
You will decide for yourself how long to leave it there and if that is important for your project.

Ah, so this means declaring the variable FirebaseSDKVersion explicitely was only a quickfix and will be obsolete once the mentioned PR has been merged, i.e. after that action i'm good to go with "only" using
pod 'FirebaseFirestore', :git => 'https://github.com/invertase/firestore-ios-sdk-frameworks.git', :tag => '10.1.0'

in target Runner again?

My previous comments have answered your question


Update app.py

1.added hex encoder and decoder
2.corrected some typo

added my name as contributor also


[feat]: Adjust setup wizard styling

Prerequisites

[X] I have searched for existing issues that already suggest this feature, without success.

Describe the Feature Request

Remove white and increase logo size (suggestion from designer)
Describe the Use Case
Make the setup wizard UI confirm to better design styling practices.
Describe Preferred Solution
No response
Describe Alternatives
No response
Anything else?
No response

For reference, this is how it looks now

Add option to migrate data from a previous drive


return false in request

Describe the bug
To use this library in thinkphp5, the returned information is false, and nothing is returned to me except false; I have set "set OPENAI_API_KEY=sk-gjtv...."
I am sure there is nothing wrong with my code, if this library is not referenced, then it should directly display an error
To Reproduce
1
Code snippets
<?php

namespace app\index\controller;

use think\Controller;
use Orhanerday\OpenAi\OpenAi;

class Index extends Controller
{
    public function index()
    {
        $open_ai_key = config("openai.OPENAI_API_KEY");
        $open_ai = new OpenAi($open_ai_key);
        //dump($open_ai_key); // have

        // $complete = $open_ai->completion([
        //     "model"   =>  "code-davinci-002",
        //     "prompt"  =>  "",
        //     "temperature" => 0,
        //     "max_tokens" => 64,
        //     "top_p" => 1,
        //     "frequency_penalty" => 0,
        //     "presence_penalty" => 0,
        //     "stop" => ["\"\"\""],
        // ]);
        $complete = $open_ai->completion([
            'model' => 'davinci',
            'prompt' => 'Hello',
            'temperature' => 0.9,
            'max_tokens' => 150,
            'frequency_penalty' => 0,
            'presence_penalty' => 0.6,
        ]);

        dump($complete); //that is "false"
        //return $this->fetch(["msg"=>$complete]);
    }
}

OS
win10
PHP version
7.4.3
Library version
3.4

I think cURL is disabled, Can you check this issue; https://github.com/orhanerday/open-ai/issues/12

@orhanerday Thank you, I will refer to it，cURL is enable

@orhanerday It turned out to be an issue with the ssl certificate https://github.com/orhanerday/open-ai/issues/12#issuecomment-1379440261


Add visual mode to viewport for selecting and copying text

I think this is doable, particularly starting with a full-line selection.
Initial impl:

in any viewport-showing page, hit V to enter visual mode
while in visual mode, cursor is visible (even if disabled) but a different color
when moving cursor up and down, selected portion is sticky (i.e. V then k k k j leaves 3 full lines selected). Selected lines show up slightly different color than the cursor line, both colors are different from normal cursor color
on escape, exit visual mode without copying selected lines to clipboard
on enter, copy selected lines to clipboard and exit visual mode
left/right don't do anything (yet) while in visual mode, and no vim-like bindings (no viw, etc.), just up and down full line selection to start
if line is copied but overflows viewport, the entire line is copied (including the parts not shown)

Closing as this adds too much complexity to already complex viewport. Saving file or copying portion directly from terminal view is good enough.


accepted-protocols docs misleading

The docs for the accepted-protocols option are misleading because they suggest the arguments should be "space separated".
I would thus have expected this to work (I've omitted all the other parameters):
az network front-door create --accepted-protocols "Http Https"
However it does not. The error message is BadRequest - "Value 'Http Https' is not allowed for property 'Frontdoor.RoutingRules.AcceptedProtocols'"
To get it work I actually had to provide a comma separated list of values.
az network front-door create --accepted-protocols "Http,Https"

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

ID: 0e1b5ddd-0054-0175-b689-d20a1349bedb
Version Independent ID: 72ccc1b6-814f-b780-99cd-ab48dfdecb66
Content: az network front-door
Content Source: latest/docs-ref-autogen/ext/front-door/network/front-door.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

This is a usage error. You did not provide a space-separated list. You provided a string with spaces.
Space-separated: --accepted-protocols Http Https
Space-separated: --accepted-protocols "Http" "Https"
String with space: --accepted-protocols "Http Https"
Comma separated values are what the service accepts. The CLI accepts space-separated values and internally converts it.

Leaving this as an FYI - I was not able to get the cli to work with space-separated values. Through powershell I needed to provide  --accepted-protocols "Http,Https" for it to work. Notice the casing as well. All other attempts to make it work failed.


Update to Zivid SDK v2.1.0

Can you just change the commit message from 'Modify downsample sample to use Zivid API downsample' to 'Modify downsample sample to use Zivid API'? (easier to read)


Support spaces on mac, fixes #999

Codecov Report

Merging #1001 into master will decrease coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1001      +/-   ##
==========================================
- Coverage   94.94%   94.81%   -0.13%
==========================================
  Files          39       39
  Lines        3483     3667     +184
  Branches      300      538     +238
==========================================
+ Hits         3307     3477     +170
- Misses        176      180       +4
- Partials        0       10      +10

Impacted Files
Coverage Δ

browser/services/platform.js
100% <100%> (ø)
:arrow_up:

browser/pages/install/controller.js
71.3% <0%> (-2.91%)
:arrow_down:

browser/pages/location/controller.js
92.3% <0%> (-2.86%)
:arrow_down:

browser/services/credentialManager.js
93.02% <0%> (-1.72%)
:arrow_down:

browser/main.js
95.58% <0%> (-1.39%)
:arrow_down:

browser/model/devstudio.js
96.8% <0%> (-0.97%)
:arrow_down:

browser/model/helpers/downloader.js
95.27% <0%> (-0.63%)
:arrow_down:

browser/pages/selection/controller.js
98.52% <0%> (-0.47%)
:arrow_down:

browser/model/jdk-install.js
96.61% <0%> (-0.42%)
:arrow_down:

browser/services/data.js
97.43% <0%> (-0.23%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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


cmd/get: support -mod flag and vendoring

What version of Go are you using (go version)?
go version go1.20.4 darwin/arm64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="arm64"
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOOS="darwin"

What did you do?
In https://github.com/kubernetes/kubernetes run go get -d -v -u -mod=vendor github.com/GoogleCloudPlatform/k8s-cloud-provider.
What did you expect to see?
Updated go.mod with the latest version of the package and updated vendoring directory.
What did you see instead?
flag provided but not defined: -mod
usage: go get [-t] [-u] [-v] [build flags] [packages]
Run 'go help get' for details.

The official documentation says that go get supports the -mod flag and describes what it does. So please,

Update the documentation.
Provide vendoring support in go get.

Indeed, in my mind, the vendoring support in go get should run go mod vendor after the module cache is updated.
And it is a good point that go get is already too complicated.
Thank you for actualizing the documentation!
@bcmills, since you got here, can I kindly ask you to visit the two stale reviews? They are waiting for your response:

https://go-review.googlesource.com/c/go/+/432536
https://go-review.googlesource.com/c/go/+/489175