feature: A way to disable auto-select in nvim-cmp

Did you check the docs?

[X] I have read all the LazyVim docs
[X] This is not a request for a new or existing extra (see above)

Is your feature request related to a problem? Please describe.
The auto-completion behavior hard-wired into LazyVim is that the first option is always auto-selected:
https://github.com/LazyVim/LazyVim/blob/c848cfddef6f4417b133382cae6fda49e8df990c/lua/lazyvim/plugins/coding.lua#L26
I was looking for a more "non-invasive" behavior where I have to explicitly select an option. This took me quite long to figure out because all solutions that came up online suggested setting vim.opt.completeopt or nvim-cmp's preselect option or using cmp.mapping.confirm({ select = false }), neither of which seems to solve the issue on its own.
Describe the solution you'd like
It would be great to make auto_select a LazyVim option that can be changed in the user config instead of having it hardcoded.
Additionally, I would disable the ghost-text feature when auto_select is false.
Describe alternatives you've considered
It seems that something like this results in the behavior I was looking for:
  {
    "hrsh7th/nvim-cmp",
    ---@param opts cmp.ConfigSchema
    opts = function(_, opts)
      opts.preselect = cmp.PreselectMode.None
      opts.completion = { completeopt = "menu,menuone,preview,noinsert,noselect" }
      opts.experimental.ghost_text = false

      opts.mapping = vim.tbl_extend("force", opts.mapping, {
        ["<CR>"] = cmp.mapping.confirm({ select = false }),
      })
    end
  }

This could be added as a recipe in the docs.
Additional context
No response

#301 is related.

That's up to the user to change.

@folke That's fair but it's non-trivial as it requires setting a combination of options. Why not add it as a recipe as also suggested here?

I second the addition of an easy toggle; can't express how many combinations I've tried, as a person unfamiliar with neovim ecosystem to get this going :[
In other words: yes, user can toggle it. The toggle has an unusual shape, and the advice about flipping it suggests many ways of flipping it, most of them not working :D

Thanks everyone. Just created a PR to add this as a recipe based on this comment.
https://github.com/LazyVim/lazyvim.github.io/pull/109


Support paths that end with trailing slashes

This didn't appear to be supported by buildpipe and has puzzled me for a while. Since directories are usually indicated with a trailing slash, buildpipe should probably support this as well.

@ksindi I need to spend a bit more time looking into why this one particular testcase became unhappy after this change.


BUG wrong selector for __glibc virtual package

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

license_file entry is missing, but is required.

For recipe:

License exception is not an SPDX exception.

@conda-forge-admin rerender

We should disable x86_64 builds because of the current_repodata.json issue.

The only reason I care about this is for x86_64 builds. Let's keep this open and blocked on fixing that.


Check for path

Fixes: Invalid 'path'. Must be either string or array.

I have the same issue with an undefined path on the overly complicated model.
Sub-model without path is marked as $isSingleNested and maybe it is related to the fact that this  childSchema is used in the model multiple times.
Unfortunately, I had no time to debug it further yet. But later I'll try to provide more info and not working example.
For now, ignoring empty _path works for me well

@wanjas I'll merge and release this for now. Normally we wouldn't without a repro script, but we'll make an exception because this is such a minor change.


[WIP] Restructure main for hub_tool

Signed-off-by: shiyu22<EMAIL_ADDRESS>

Codecov Report

Merging #440 (fd19a76) into main (81ee200) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #440   +/-   ##
=======================================
  Coverage   61.99%   61.99%
=======================================
  Files         141      141
  Lines        5420     5420
  Branches      805      805
=======================================
  Hits         3360     3360
  Misses       1838     1838
  Partials      222      222

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 81ee200...fd19a76. Read the comment docs.

Submit another PR #451


Can't detect a creeper

Hi,
robot.detect() doesn't detect anything.
Is this normal?

chip needed

I already have the chip bro.

It's not working.


CVE-2021-28165-Jetty

In Eclipse Jetty 7.2.2 to 9.4.38, 10.0.0.alpha0 to 10.0.1, and
11.0.0.alpha0 to 11.0.1, CPU usage can reach 100% upon receiving
a large invalid TLS frame.
Upgraded Jetty from 9.4.35.v20201120 to 9.4.39.v20210325.
https://nvd.nist.gov/vuln/detail/CVE-2021-28165

(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@dinghram Thanks for the PR, this was already being addressed on #10177 which just got merged.

@dinghram I would like to close this PR, since 10177 solved this issue.  Feel free to open other PRs for other enhancement.


Vercel deployment fails with  ERR_PNPM_OUTDATED_LOCKFILE

Summary
Vercel deployment fails with ERR_PNPM_OUTDATED_LOCKFILE  error.
The same error while:

using Saleor CLI saleor storefront deploy command
following the guide in docs https://github.com/saleor/react-storefront/blob/main/docs/vercel.md

In both cases Turborepo authentication is omitted - ie without pnpm dlx turbo login
[16:52:15.492] Cloning completed: 738.443ms
[16:52:16.271] Looking up build cache...
[16:52:16.581] Build Cache not found
[16:52:16.610] Running "vercel build"
[16:52:17.117] Vercel CLI 28.2.2
[16:52:17.575] Detected `pnpm-lock.yaml` generated by pnpm 7...
[16:52:17.621] Installing dependencies...
[16:52:18.192] Scope: all 10 workspace projects
[16:52:18.345]  ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with packages/eslint-config-checkout/package.json
[16:52:18.345]
[16:52:18.345] Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"
[16:52:18.352] Error: Command "pnpm install" exited with 1

Steps to reproduce
Using Saleor CLI
saleor storefront create
cd your-created-app
saleor storefront deploy
Using guide https://github.com/saleor/react-storefront/blob/main/docs/vercel.md
omit pnpm dlx turbo login and pnpm dlx turbo link
A link to a reproduction repository
No response
Output from envinfo
System:
    OS: macOS 12.4
    CPU: (8) arm64 Apple M1
    Memory: 81.39 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.8.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 8.18.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 104.0.5112.101
    Firefox: 103.0.2
    Safari: 15.5

pnpm -v
7.5.2

Maybe related
https://github.com/saleor/react-storefront/blob/main/turbo.json#L86

This problem is unrelated to turborepo @lkostrowski. It's a matter of pnpm versions incompatibility – 7.5.2 needs to be used as 7.6.0+ introduced a breaking change.


few changes

Hi,
Thanks for the PR!  Is there anything else going on here besides PSR code style changes?  Are you using the cloudflare-pluign-backend for anything?
Thanks,
John

Hello John.
No, I do not use cloudflare-pluign-backend. Just I saw a opened vacancy of PHP developer in CloudFlare, and I viewed public php project on github to understand what kind of work you doing by PHP.

Hey,
We don't except outside changes on this repository but that being said I appreciate the initiative and agree with the changes you made.  We have PSR-2 formatting in our backlog.
I see you already applied to one of our PHP positions, I made a note about your PR.
Thanks!
John

Hi John,
Thanks :)


[BUG] No items to display for cases

I really really like your website I've been looking for one for many many while but can you fix stuff including like on some cases it says "No items to display" because when i get some case and it says that i cant  open and and also is it impossible to pull knifes because ive been playing for a couple days and I've never pulled one how can i change the pull rate ands last thing. can you make accounts that have their own data and not just on steam like make a simple account with username password etc. that saves the guns and stuff so you don't have to have many steam accounts to log in with. like i said i really like your website and i would love for it to be further worked on thank you a lot my friend.

Also maybe have a pin feature to pin items to the top and maybe price system and trade? and yes I'm aware that would take lots of time

Whoa whoa, fist things first - regarding the issue you are reporting: Kilowatt cases require keys, did you craft its keys, too? The app simulates the game, you can't open cases only by adding them to your inventory.
Currently only Steam authentication will be supported, no plans to add another method sorry. And regarding your other suggestions, please open an issue for each suggestion using the [REQ] prefix in the title, detail your request and I'll look over them.


[RESTEASY-1696] ignore @PreMatching on ClientRequestFilter and give a…

… warning msg

Thanks @rsearls . I've tried resolving a simple conflict here using the webeditor but that messed up the PR a bit, so I've manually pushed your commit to master. Closing this PR then.


Use foreach instead of for, for code readability.

This greatly improves the code readability, at it does not need a "counter" variable.

Probably because of the better performance.
Shouldn't be a issue, because all the Lists are pretty small.

Huh, I somewhere read that when you reference the iterated variable more than twice in the loop, you should use foreach, else it's faster to use for... Which is correct now?

http://www.dotnetperls.com/for-foreach
Maybe you can adjust this to when a variable was only acessed once, leave it as for, and when it's acessed multiple times (or sometimes even the cached value), change it to forach? That's best for performace. But as pmhpereira said, doesn't really matter anyway.
I'm simply used to use for loops instead of foreach most of the time;)

In fact the performance issue truely depends on the structure you are iterating on. But however, at this scale we are not talking at all about performance.
I think that the code would greatly benefit from such modification, not because it would perform better, but because that is FAR more readable. I think at this point it is more important than performance. :)

^^ Can you fix these three occurances to a for loop again? I think that is the only way:/

I'll have a look this evening. :)
By the way, concerning about the performance, according to this and this,
when a foreach loop is applied to an array, it is compilated into normal indexing loop (well a good compilator should...) .
Unless they are wrong, both solutions must be equivalent in term of performance. :)

Unity's old mono compiler has a bug where foreach loops generates (40 bytes of) garbage on anything that's not a normal array (usually a List, like in the conditions list here).
This has caused a huge amount of confusion on the forums about what's good practice and what's not.  Generally, I'd say that that this is not an issue, and the readability wins out. This is a plugin though, and it's nice to be able to use plugins like a black box without having to worry about their memory usage.
Up to you guys, I just figured I'd chip in the facts.

Thanks for the clarification:) I'd be ok with the for loops besides the above mentioned three exceptions where a for loop is needed. Then I would merge if everyone is oke with that.

Done. :)

Anyway, can be added at a later date;)

Seems like one of the commits throws an error where you replaced the for with a foreach loop, but the cnt is still referenced in the loop. Reverting back to for loop, this is necessary.


Appstore limited to US

Sadly enough the app store version seems limited to the US...
How to reproduce: try to download it from the appstore with an Non-US account.
Solution: Make it available world wide or add an IPA file

Yes, we had to do this due to the export restrictions (in the US) on encryption. I will look at making it available world wide soon when we submit an update.

Sroske,
The export restrictions do not include source code according to EAR part 772.1

Excellent, I was not aware of that.

Whoops, forgot to add a comment. If that is the case I will open it up worldwide right now. Thanks!

I have taken a look into the law itself...
"The Adoption of Note 4 Decontrols
The decontrols note appears as Note 4 to Category 5, Part 2 of the CCL and provides as follows:
Note 4: Category 5, Part 2 does not apply to items incorporating or using "cryptography" and meeting all of the following:
(a) The primary function or set of functions is not any of the following:
(1) "Information security";
(2) A computer, including operating systems, parts and components therefor;
(3) Sending, receiving or storing information (except in support of entertainment, mass commercial broadcasts, digital rights
management or medical records management); or
(4) Networking (includes operation, administration, management and provisioning);
(b) The cryptographic functionality is limited to supporting their primary function or set of functions; and
(c) When necessary, details of the items are accessible and will be provided, upon request, to the appropriate authority in the exporter’s
country in order to ascertain compliance with conditions described in paragraphs (a) and (b) above.
Note 4 completely removes the decontrolled items from control under Category 5, Part 2 of the CCL.
Please note that certain products may be controlled under an ECCN elsewhere in the CCL even if they are no longer controlled for encryption reasons.
"

https://www.bis.doc.gov/index.php/policy-guidance/encryption/identifying-encryption-items

So Yes, it is legal to export.

Okey, More in dept:
Because it is not a finished product but for research and development...
Call it a "Beta" and you can safely classify it as 5D002, without the need of paperwork :)


haskellPackages.x509-system: Override based on the host, not target platform for 18.09

Motivation for this change
Backport of #47096.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: haskellPackages.x509-system
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-darwin (full log)
Attempted: haskellPackages.x509-system
Partial log (click to expand)
copying path '/nix/store/x1jx15ai3slxil6myyamwlk2jb5ja9jp-x509-1.7.3-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/apssvxzh1n9wqzlff7q9f1bmbiksn6j0-asn1-types-0.3.2' from 'https://cache.nixos.org'...
copying path '/nix/store/j40h6bskbxjdg7jamyli0ivyflfwc1lr-x509-store-1.6.6-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/iqicidb1p4kwsq0gz8ijf6g9aksbhvfk-x509-system-1.6.6-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/80asis68hwlwj2gqnfwghkadka9b5k53-asn1-encoding-0.9.5' from 'https://cache.nixos.org'...
copying path '/nix/store/d55avb1ykm0dixpmrym6lll91l7yz3ky-asn1-parse-0.9.4' from 'https://cache.nixos.org'...
copying path '/nix/store/li1rqmb8lc0w8zsrmhywyxgh3vga5r3f-x509-1.7.3' from 'https://cache.nixos.org'...
copying path '/nix/store/dn09rjkr2qc5i5m7db75nhkxqr5bgla8-x509-store-1.6.6' from 'https://cache.nixos.org'...
copying path '/nix/store/2hjxvgwv70d9yc4yb0s7f5x4krskya68-x509-system-1.6.6' from 'https://cache.nixos.org'...
/nix/store/2hjxvgwv70d9yc4yb0s7f5x4krskya68-x509-system-1.6.6

Success on x86_64-linux (full log)
Attempted: haskellPackages.x509-system
Partial log (click to expand)
copying path '/nix/store/58b6jq7snqsygabn04f172l985d30pjy-asn1-encoding-0.9.5' from 'https://cache.nixos.org'...
copying path '/nix/store/1vjq0alnbm97fa22afyah7fwnwpyavzr-pem-0.2.4' from 'https://cache.nixos.org'...
copying path '/nix/store/k7bvg1r35836bvb4xwxcx0z9b9637i0c-asn1-parse-0.9.4' from 'https://cache.nixos.org'...
copying path '/nix/store/8b2p93rcrlvmvkq8388719qywzvpsnpz-x509-1.7.3-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/sn24lkc69ykgabw3h02ammgif02m1hav-x509-1.7.3' from 'https://cache.nixos.org'...
copying path '/nix/store/9wi5ajx63wsjj7zgh1yggg7c3d3gnfs9-x509-store-1.6.6-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/0ba4krrxhzgg93bg6cwbq9a9crjjlclw-x509-store-1.6.6' from 'https://cache.nixos.org'...
copying path '/nix/store/s3gn0wga9yls0jyqw92cf11565p8qv3n-x509-system-1.6.6-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/nfilhbkq42mp18jfa2y1wxqyh90pf3mg-x509-system-1.6.6' from 'https://cache.nixos.org'...
/nix/store/nfilhbkq42mp18jfa2y1wxqyh90pf3mg-x509-system-1.6.6


Support helm chart to use a Secret resource to hide metadata storage db credentails.

Motivation
I have tried to deploy druid using helm chart.
the helm chart doesn't support to inject environment variables related to metadata storage db credentials using Secret resource.
There's no way to inject the credentials unless using ConfigMap defined in values.yaml.
Proposed changes

Add a way to pull some configs like db credentials (druid_metadata_storage_connector_user, druid_metadata_storage_connector_password) to each process (broker, historical, ...) from Secret resource.
It can be custom Secret user made outside this helm chart.

Rationale
As I said on the above, it supports only to pull configs from ConfigMap.
It's hard to operate and manage values.yaml of this druid helm chart.
Operational impact
It's just add changes. so, it keeps backward compatibility.

@imcheck I was able to accomplish this by adding the following to the helm chart values:
  env:
    - name: druid_metadata_storage_connector_password
      valueFrom:
        secretKeyRef:
          name: druid-pg-app
          key: password

@jwitko Thank you for replying 👍 However, it seems that the druid chart (0.3.5 chart version) doesn't support it.
I also tried adding the values in values.yaml, but it didn't change anything.

Sorry you are right, I forgot I was using a vendored copy of this chart that has the fixes in the above linked PR.

@imcheck @jwitko Thanks for your contribution. As mentioned in PR https://github.com/apache/druid/pull/15904, we have migrated the Helm Chart to a new repository, you are welcome to raise up new one there and we can maintain the Druid Helm Chart together in new repository. And this PR will closed soon. Thank you again for your understanding and collaboration.


No stopping on breakpoints in the desktop web extension host

create a web extension with yo code --insiders -t=web
open in VS Code,
set breakpoint in extension.ts
run the 'Run Web Extension in VS Code' launch configuration
In the Extension Development Host window, run the hello world command

-> break point is not reached

Our team had a similar sounding issue this week, and eventually found this workaround:

The built in Node Debug extension ms-vscode.node-debug2 is default using a new version of the js debugging extension as of a new update. Even rolling back vscode seems to use the new Node Debug extension. For whatever reason, that's broken the breakpoints in my services. The solution was to revert to the old behaviour by adding this to my vscode/settings.json
"debug.javascript.usePreview": false

@peter-dolkens you are not hitting the same issue since the old debugger does not debug web extensions, and bear in mind that the usePreview option will go away in the next release. Please file an issue: https://github.com/microsoft/vscode-js-debug/issues

Thanks @connor4312  - was trying to reduce duplicates, but I'll file a new ticket for you as it's a new issue.
Cheers


KAFKA-5186: Avoid expensive log scan to build producer state when upgrading

@junrao Maybe you can take a look and see if this approach is reasonable?

retest this please

@hachikuji : Thanks for the update. LGTM. I will let you merge it when the test completes.


Opt out of warning if a task is skipped because of re-invocation

Are we accepting pull requests to disable the following warning?
If you really meant to run this task again, use invoke!("rails:docker:build")
THIS BEHAVIOR MAY CHANGE IN A FUTURE VERSION OF CAPISTRANO. Please join the conversation here if this affects you.
https://github.com/capistrano/capistrano/issues/1686

Due to the way I've developed my tasks, they sometimes overlap but I'd still like them to only be invoked once. Or is there a better way to check if the task has been invoked before invoking?

We may tweak the working of the warning, but I don't see us removing it right now.

is there a better way to check if the task has been invoked before invoking?

Assuming you are using Rake 11 or newer, you can do this:
invoke("rails:docker:build") unless Rake::Task["rails:docker:build"].already_invoked

@mattbrictson Thanks!
Defined this helper method for future reference
def invoke_once(task)
  invoke(task) unless Rake::Task[task].already_invoked
end


returned months wrong in 6.6.4 (double March for 2023)

Hi,
our automated UI tests flagged an error after we updated the @rehookify/datepicker package from 6.6.1 to 6.6.4
The returned months from the hook suddenly contains March twice for 2023, here a screenshot:

Any update regarding this error? I am also facing the same issue in version 6.6.4

hey @gunters63 and @VaibhavGupta-creditas for reporting this issue
Please upgrade to v6.6.6 👿


Question: Caching?

Does Elm-live cache files? I'm getting a module import error, I've modified the file so there is no mention of this module but I'm still getting the same compile error. Restarting elm-live does not help, also deleting and recreating the module file doesn't register either.
`-- MODULE NOT FOUND --------------- /home/user/Code/Elm/Swapi/src/CustomScalarCodecs.elm
You are trying to import a Swapi.Scalar module:
5| import Swapi.Scalar
`
The module code:
`module CustomScalarCodecs exposing (Id(..), PosixTime, codecs)
import Json.Decode as Decode
import Json.Encode as Encode
import StarWars.Scalar
import Time
type Id
= Id Int
type alias PosixTime =
Time.Posix
codecs : StarWars.Scalar.Codecs Id PosixTime
codecs =
StarWars.Scalar.defineCodecs
{ codecId =
{ encoder = (Id raw) -> raw |> String.fromInt |> Encode.string
, decoder =
Decode.string
|> Decode.map String.toInt
|> Decode.andThen
(\maybeParsedId ->
case maybeParsedId of
Just parsedId ->
Decode.succeed parsedId
                            Nothing ->
                                Decode.fail "Could not parse ID as an Int."
                    )
                |> Decode.map Id
        }
    , codecPosixTime =
        { encoder = Time.posixToMillis >> Encode.int
        , decoder = Decode.int |> Decode.map Time.millisToPosix
        }
    }`

I'm being an idiot!


Update Cancel Proposal Tooltip Copy

App

[X] Admin
[ ] Summon
[ ] Contracts
[ ] Contracts
[ ] SDK

Feature Type

[ ] New Feature
[X] Enhancement

Feature

Proposals can only be cancelled by the sponsor....
Screenshots and Context

Complete
https://github.com/HausDAO/monorepo/pull/521


Reject packages with a SUSE-Firmware license in the BCI repo

Dirk Mueller @.***> writes:

tiny nit: atm it's not possible to find out which package is the offender from the logs. This could be worked around by not using grep -c and instead only grep and then counting the lines in python (should be reasonably cheap)

doesn't work because the package name is not on the line that is being greped, but somewhere around it. it's not difficult to find the offender, so this should be fine.

Ah, right! Fair enough then


[Map] War-TombOfAmunrah

Player Count = 12-24
Source = Readme
The authors are not 's Notes (surprise!) but Simion (Oldskool0482) Korbra & Cryspy
Source = Readme
[As an aside, Cryspy is Bryan Crisp in the credits section of the readme]

URL: https://unrealarchive.org/maps/unreal-tournament-3/warfare/T/war-tombofamunrah_13e6ef87.html
Hash: 13e6ef87050c58ffdf5289bcc343d295dabad8c3
Current name: War-TombOfAmunrah

Fixed


[SPARK-27419][FOLLOWUP][DOCS] Add note about spark.executor.heartbeatInterval change to migration guide

What changes were proposed in this pull request?
Add note about spark.executor.heartbeatInterval change to migration guide
See also https://github.com/apache/spark/pull/24329
How was this patch tested?
N/A

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10103/
Test PASSed.

Test build #104792 has started for PR 24432 at commit f37ab0d.

Test build #104792 has finished for PR 24432 at commit f37ab0d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104792/
Test PASSed.

thanks, merging to master/2.4!


Specify the Census search space with an obs_value_filter parameter in find_nearest_obs

Description
Currently find_nearest_obs finds the closest Census cells to user's data. It would be a great addition to have the ability for the search to be constrained to a subset of user-defined Census cells instead of all Census cells.
To provide consistency with other census API, a simple parameter obs_value_filter (as used in the get_anndataAPI) can be added to limit the search to Census cells meeting the filter criteria.
Context
The find_nearest_obs functionality is great and working as intended, however many times I can be interested on searching the most similar cells against a specific subset of Census defined by a biological context, specially when I know my query cells are from the same/similar biological context.
Impact
This limits the ability to utilize the find_nearest_obs to its full extent in its current form.
Ideal behavior
A parameter obs_value_filter in  find_nearest_obs to limit the search to Census cells meeting the filter criteria.

Hmm, will have to check with @cathystoli to see if we're still accepting feature requests from you 😉

I think this makes a ton of sense as a feature. I've noticed that we probably want to filter out all is_primary==False cells from queries, since you just end up getting a bunch of cells with the exact same embedding.
I've consulted with the TileDB folks and I think this should be quite doable.

This notebook demonstrates the metadata_df_filter_fn which basically does exactly what you want
Ideally we would be able to use the same query string you are suggesting, but we need to figure out how to wrap that as a function which can be applied per row.

cc: @mlin


Add support for downloading from Unity Catalog volumes

Description of changes:
User would run:
mcli create secret env DATABRICKS_HOST=...
mcli create secret env DATABRICKS_TOKEN=...

Env vars consistent with https://databricks-sdk-py.readthedocs.io/en/latest/authentication.html
Issue #, if available:

Merge Checklist:
Put an x without space in the boxes that apply. If you are unsure about any checklist, please don't hesitate to ask. We are here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[ ] I have read the contributor guidelines
[ ] This is a documentation change or typo fix. If so, skip the rest of this checklist.
[ ] I certify that the changes I am introducing will be backward compatible, and I have discussed concerns about this, if any, with the MosaicML team.
[ ] I have updated any necessary documentation, including README and API docs (if appropriate).

Tests

[ ] I ran pre-commit on my change. (check out the pre-commit section of prerequisites)
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate).
[ ] I ran the tests locally to make sure it pass. (check out testing)
[ ] I have added unit and/or integration tests as appropriate to ensure backward compatibility of the changes.

Overall looks great. Few minor comments:

Can you also add the similar functionality in upload.py which basically uploads the shard files in UC during dataset conversion? You can look at PR 311 and 265 for reference.

Done

Can you also add a unit test for the same in test_download.py and test_upload.py ?

Done

Can you also update the documentation with UC Volume section on how user would provide a credentials ?

Done

Thanks!

The PR looks good to me, Can you rebase the PR and then fixed the lint issue which I mentioned in PR 362? Thanks!


Header and suspend icon have size problem

The suspend-icon is not correctly scaled, also the header about icon has some problems.

FTR This was recently introduce in master, ie in 0.16.


error

Hello
after running it im getting an error in this line
assertionFailure("Could not copy DefaultList: (error)")
its line 37 in adblocker.swift
anyclue thank you

This should print out the error in the console. Do you get any output?

no output at all ?

I'm unable to reproduce this. Can you add a print(error) in the line above assertionFailure("Could not copy DefaultList: \(error)")? Also, what's your phone's model and iOS version?

fatal error: Could not copy DefaultList: Error Domain=NSCocoaErrorDomain Code=260 "The file “DefaultList.json” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Users/Apple/Library/Developer/CoreSimulator/Devices/8FE486CE-29AB-4EAB-878A-37D9CA4587EF/data/Containers/Bundle/Application/91D2E079-986E-44F7-96ED-8D3BDF59A74E/OpenAdBlock.app/DefaultList.json, NSUnderlyingError=0x7fd350d88370 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}: file /Users/Apple/Downloads/OpenAdblock-master/OpenAdBlock/OpenAdBlock/AdBlocker.swift, line 60
(lldb)

i have iphone 6 with ios 9.0.1

Can you try installing OpenAdblock, and reinstalling? If that fails, you may have an old copy of OpenAdblock, try downloading a new one.

ok i will b doing tht,
does it work on the simulator


[Bug]: Unable to set explode to false for a query parameter

Describe the bug
According to the OpenAPI 3.0 guide, the default for the explode option is true (unlike for path parameters and headers).
Starting with TypeSpec 0.59.0, there is no way to produce explode: false for a query parameter with the OpenAPI 3.0 emitter.
When explicitly defining @query(#{ explode: false }), TypeSpec wrongly considers that the default value is false and therefore does not generate the explode: false property in the OpenAPI file. As a result, the explode option is always understood as true in Swagger UI.
Reproduction
Playground Link
Checklist

[X] Follow our Code of Conduct
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

oh ouch

Will be release in openapi 0.59.1

Great, thank you very much 😃


Generated Typescript definitions for Unions are null or undefined

const AuthSuccess = objectType({
  name: "AuthSuccess",
  definition(t) {
    t.string("token");
    t.field("user", { type: CurrentUser });
  }
});
const NoUser = objectType({
  name: "NoUser",
  definition(t) {
    t.string("message");
  }
});
const WrongPassword = objectType({
  name: "WrongPassword",
  definition(t) {
    t.string("message");
  }
});
const AuthResult = unionType({
  name: "AuthResult",
  definition(t) {
    t.members(AuthSuccess, NoUser, WrongPassword);
  }
});
const AuthPayload = objectType({
  name: "AuthPayload",
  definition(t) {
    t.field("result", { type: AuthResult });
  }
});

generates the following typescript definition for AuthPayload in nexus.ts:
  AuthPayload: { // root type
    result: undefined | null; // AuthResult!
  }

This is the same for any other union I define. Am I doing something wrong?
And while I'm here, is there a better way to resolve types than looking at what properties a root field has? In the example above, some of the types that are part of the AuthResult union have exactly the same fields, and what I'm really interested in is their type identity.

This is the same for any other union I define. Am I doing something wrong?

Nope, this is a bug. Will have a patch up shortly.

And while I'm here, is there a better way to resolve types than looking at what properties a root field has? In the example above, some of the types that are part of the AuthResult union have exactly the same fields, and what I'm really interested in is their type identity.

I'm not sure I follow the question.
Are you asking how you can choose between WrongPassword and NoUser in the union's resolveType given that they have the same root fields?
The answer is to define custom types for the root field and have them picked up by the typegenAutoConfig. See this comment for a little more explanation, though I need to add more documentation on type configuration in general.


celebA RuntimeError: The MD5 checksum of the download file does not match the one on record.

🐛 Describe the bug
Running
from torchvision.datasets import CelebA

data_dir = "./celeba/data"
data = CelebA(root=data_dir, split='train', download=True)

yields the error
RuntimeError: The MD5 checksum of the download file ./celeba/data/celeba/list_landmarks_align_celeba.txt does not match the one on record.Please delete the file and try again.
Deleting the file and running the code again does not solve the problem.
Versions
PyTorch version: 2.0.1
Is debug build: False
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A
OS: macOS 11.7.8 (x86_64)
GCC version: Could not collect
Clang version: 12.0.5 (clang-1<IP_ADDRESS>)
CMake version: version 3.17.3
Libc version: N/A
Python version: 3.11.3 (main, May 15 2023, 10:43:03) [Clang 14.0.6 ] (64-bit runtime)
Python platform: macOS-10.16-x86_64-i386-64bit
Is CUDA available: False
CUDA runtime version: No CUDA
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Intel(R) Core(TM) i7-8559U CPU @ 2.70GHz
Versions of relevant libraries:
[pip3] numpy==1.23.5
[pip3] pytorch-lightning==2.0.4
[pip3] torch==2.0.1
[pip3] torchmetrics==1.0.0
[pip3] torchvision==0.15.2
[conda] blas                      1.0                         mkl
[conda] mkl                       2023.1.0         h59209a4_43558
[conda] mkl-fft                   1.3.6                    pypi_0    pypi
[conda] mkl-random                1.2.2                    pypi_0    pypi
[conda] mkl-service               2.4.0                    pypi_0    pypi
[conda] mkl_fft                   1.3.6           py311hdb55bb0_1
[conda] mkl_random                1.2.2           py311hdb55bb0_1
[conda] numpy                     1.23.5                   pypi_0    pypi
[conda] numpy-base                1.25.0          py311h53bf9ac_0
[conda] pytorch                   2.0.1           cpu_py311he1daf47_0
[conda] pytorch-lightning         2.0.4                    pypi_0    pypi
[conda] torch                     2.0.1                    pypi_0    pypi
[conda] torchmetrics              1.0.0                    pypi_0    pypi
[conda] torchvision               0.15.2                   pypi_0    pypi
cc @pmeier

CelebA is notoriously hard to download since it is hosted on GDrive, which has no API that we can interact with. We are trying to catch common mistakes, but that process is not as robust as I would like it to be. Could you please have a look at the downloaded file and see if it actually contains the data we would expect or something else?

I'm currently getting
RuntimeError: The daily quota of the file img_align_celeba.zip is exceeded and it can't be downloaded. This is a limitation of Google Drive and can only be overcome by trying again later.

As the error implies, there is nothing we can do about that. There are countless threads on this tracker (rightfully) complaining that they can't download the files. The only solutions are a) wait until the quota is no longer exceeded or b) ask the authors to host the files elsewhere that doesn't have this restriction.

Thanks for the quick reply! I have been receiving your error for the last three days and today was the first time it showed a different one. I just took a look at the file (list_landmarks_align_celeba.txt) and indeed, it seems it does not have the desired content:
<meta http-equiv="content-type" content="text/html; charset=utf-8"/><title>Sorry...</title>

Google
Sorry...

We're sorry...

... but your computer or network may be sending automated queries. To protect our users, we can't process your request right now.

See Google Help for more information.

Google Home

We're sorry...

... but your computer or network may be sending automated queries. To protect our users, we can't process your request right now.

That is a new one :roll_eyes: Let me see if we can guard against it and raise a proper error.
But in general, my statement from https://github.com/pytorch/vision/issues/7850#issuecomment-1683559793 holds. There is nothing we can do. Other than convincing the author that GDrive is a suboptimal solution to host datasets, you can't do anything but wait until the quota resets. AFAIK, it is unknown when exactly that happens and thus it is impossible to predict at what time of the day your chance for actually getting through is higher.

All of a sudden, the code ran successfully. I did not do anything other than wait even more time and run again...


Add 3 utility functions & some typing fixes

Added functions
.extends() & .extend()
typesafe role extension methods

extend(permissions) creates & returns a new role based off of this., with permissions
extends(...roles) copies permissions from roles to this., returns this.

const baseRole = new Role([
    ['can', 'create', 'reviews']
]) // not used auth, only used for permissions

const reader = baseRole.extend([
    ['can', 'read', 'books']
])

const writer = baseRole.extend([
    ['can', 'create', 'books']
])

const readerAndWriter = new Role().extends(reader, writer)

.clone()
Clones this. and returns it
Typing changes
.set() now updates typings of role
Using .set on a role does not give a type error when setting an action or subject that didn't exist at the time of instance construction. .extends() & .extend() do this as well, since they do .set() under the hood
const user = new Role()

const developer = user.clone().set('can', 'create', 'app')

const admin = developer.clone().set('can', 'update', 'developers')

.#permissions is no longer typed by Action:Subject, but string:string
This was not needed internally, and was removed to support the new .set() typing changes

i'm alive and i will review it soon

I've also been experimenting with a new "is" clause, not sure what the use for that would be though.


test that camput does not upload files more than once

Test that with the help of the local (stat/have)cache or even without (because the
server reports so), camput never uploads a file again after it has already done so.

https://camlistore-review.googlesource.com/6044

7fa4f9a2eeb61488f63b5f0b46aac35e413383a6


Istanbul integration

In this branch, the jasmine gem has been replaced with the teaspoon and teaspoon-jasmine gems. the jasmine-jquery-rails is no longer required, because we have jasmine-jquery in spec/helpers.
The jasmine-jquery-2.1.0.js (which was actually was version 1.5.1 and improperly named) file in spec/helpers has been updates to jasmine-jquery-2.1.1.js.
spec/teaspoon_env.rb configures teaspoon, and code coverage options and spec/javascripts/spec_helper.js defines what resources need to be loaded in the particular order required for the jasmine specs to run.
The teaspoon test runner uses istanbul for code coverage, so you will need to install istanbul 'npm install -g istanbul' in order to generate coverage reports.
See the bottom of [(https://github.com/projecttacoma/bonnie/wiki/Guide-to-Bonnie-Rake-Tasks)] for teaspoon usage instructions.

When running the teaspoon rake task it does not find phantomjs. Do I need to manually install it?


Retirement Calculator

Retirement Calculator can help a person plan the financial aspects of retirement. Each calculation can be used individually for quick and simple calculations, or in chronological order as a more comprehensive walkthrough of retirement planning.
I would like to work on this under JWOC pls assign to me

@Priyamakeshwari Please describe a more what you want to implement.

I would like to implement like

How much do you need to retire?
How can you save for retirement?
How much can you withdraw after retirement?

Basically i want to implement the 10%, 80%, 4% rule.

@Priyamakeshwari Go for it. Add necessary js inside html file or a separate js file. Avoid adding in `script.js

I want to work on this issue under JWOC @Srijita-Mandal


fix error on null result, bump dependency ver

thanks for making this :+1:
while trying to work with the Web of Science SOAP API, i ran into 2 problems that i've fixed in this PR:

newer versions of vpulim/node-soap allow to set HTTP-Headers on clients
there was a TypeError thrown upon empty (but successful) SOAP responses

HI @krugar, thanks for the PR! I didn't see this and commited already the soap library version bump, could you please rebase it so I can merge?
Thanks!

there you go

Great, thanks a lot!!


Update acorn actions used

We are ignoring these nuget packages since we do not contain license information at least not any that conforms to the tool we are checking with:

jose-jwt
Microsoft.AspNet.WebApi.Client

These ignored licenses have do have licenses that are in the allowed licenses file. We just cannot automate checking them.
They should properly be checked for alternatives that can be validated, or perhaps make pull requests to them so they can be fixed.
Also note, that no nuget packages used in test projects are validated still - there is a TODO about re-enabling them.

@tnickelsen - While the check-license is disabled I cannot really know if this PR fixes everything, but I have run the health check to see if the updated actions work there.

@tnickelsen - While the check-license is disabled I cannot really know if this PR fixes everything, but I have run the health check to see if the updated actions work there.

You may reenable them to test this :)

Testing it lead to few changes...
The input for validating licenses is now the solution file, this however will also target test projects, but will also check everything in one go.
This means the packages that are ignored are now:

jose-jwt
MockQueryable.NSubstitute
Microsoft.AspNet.WebApi.Client
Testcontainers*
origin/*
xunit.categories

The origin/* entry are local dependencies.


Permit overriding buildRustCrate arguments

buildRustCrateWithFeatures makes its implementation overridable.
However in doing so, it hides the override function of buildRustCrate.
With this change attributes other than features are passed through to
buildRustCrate's override, so that its arguments are overridable too.
This addresses #15.

@kolloch  I made this a draft PR to ask if this is something you would accept. If so, I'll add a test for this as well.

In principle, that looks good to me. Could you provide tests in tests.nix, i.e. an example that makes use of that? Thanks.

Oh, and explaining a meaningful example in README.md/CHANGELOG.md would be greatly appreciated (but not required).

Superseded by https://github.com/kolloch/crate2nix/commit/184d3ddb158e47af2014a94b95676036464bf9b7 .
Great to have this functionality now!

I actually didn't think of this when making my change.
I kind of like keeping an interface somewhat restricted so that I know what to test and support. But the generality of your CL has obvious advantages as well.
Thank you.


Removed trailing space for test

Critical Changes
Changes

Removed trailing space for access test.

Issues Closed

Just a test. Cleaning up.


Bogus map entries in RMapCache

Using version 3.5.0, we've noticed a very strange behavior in RMapCache map-iteration APIs (keySet,  entrySet, etc). It seems that the map sometimes contains "inverted" entries in which the key looks like a (bogus) instance produced by a value encoder, and the value looks like an (again) bogus instance produced by the key encoder*. When they do, these entries show alongside the normal entries. It's only because of Type Erasure that the code doesn't break until one actually calls an iterartion API: Because the original entries are still in place, get, put, etc. works as expected even when the map is in this broken state. We've managed to workaround this issue by using readAllKeys rather than keySet.

We have a custom extremely type-aware jackson-based codec, which makes this bug very easy to spot.

Could you provide any test case to reproduce it?

It has the characteristics of a race condition, so it's going to be hard to provide a test case here. We don't do anything special: it's a RMapCache in which values are put with a 3-hour TTL, bound to a jackson-based codec in which the keyEncoder is set to a specific type, and the valueEncoder is set to some other type (using readValue(v, TypeReference> and writerForType). The map entries are read using keySet in a periodic task running once a minute. Values are written from multiple processes, but keySet is used only in one of those processes.

Unable to reproduce


README: Link to godoc reference

What do you guys think of this?

Coverage remained the same at 46.58% when pulling 43838f684e45b4656ddefab29f88af9f081798b6 on readme-godoc into 9920b547e0224b4a9fca97c92fe83af70daca642 on master.

Coverage remained the same at 46.58% when pulling c156ee6dd323a52e0162df7d10f3cf2e7afebe0b on readme-godoc into 9920b547e0224b4a9fca97c92fe83af70daca642 on master.

Good idea! LGTM

LGTM


IIS install

Hello,
Great CMS
I install it on IIS server. Everything went fine except there is no index.php file to point in root dir.
Which index.php file should i use in backend/web/index.php or frontend/web/index.php
I tried both but site showed nothing
Thanks

YeeCms doesn't support IIS server.


Allow the configuration object to be inject into the mesos-cli component.

Currently the mesos-cli uses a global configuration object that gets statically imported into many components of the CLI. Look at the ls command for an example.
We would like to change this to something that is dynamically loaded/bound so that we can change the implementation in the future.

:+1:


Class 'MongoClient' not found

Hi
$connection = new \yii\mongodb\Connection([
'dsn' => 'mongodb://localhost:27017 ',
]);
$connection->open();
error Class 'MongoClient' not found

looks like you have not installed the required php extension: https://github.com/yiisoft/yii2-mongodb/blob/master/composer.json#L22

installed extension mongodb in php
and has yii2-mongodb in vendor/yiisoft/

Could be that it is installed in console but not for web.

sorry
I did not catch

Often console uses different php.ini than web. Check phpinfo() for mongo.

in command :  php -m | grep mongo
result => mongo


feat: Added unit test templates

Templates for unit tests
#71
XUnit
NUnit
MSTest

Nice to see this PR. Thank you. I see that it was also easy to add the new menu items.
Well done.

The PR has conflicts that need to be addressed before it can be merged.


Is skipset was removed from 1.2.0?

I noticed skipset stuff was commented out from source code.
Also could you catch the ConnectionRefusedError and TimeoutError in download()?

skip uses now the csv instead of an extra file. thats why skip_write was commented out :)

I noticed skipset stuff was commented out from source code.
Also could you catch the ConnectionRefusedError and TimeoutError in download()?

this errors should already been catched. have you got one unhandled?

I noticed skipset stuff was commented out from source code.
Also could you catch the ConnectionRefusedError and TimeoutError in download()?

this errors should already been catched. have you got one unhandled?

sry, I mean ConnectionResetError and TimeoutError.


Allow supplying TypeDBOptions when creating a TypeDB client object

What is the goal of this PR?
It is now possible to supply TypeDBOptions when constructing a TypeDB or TypeDB Cluster client object. The option will be inherited by sessions and transactions opened from the client object, unless deliberately overridden by the user.
What are the changes implemented in this PR?

Make TypeDB::coreClient and TypeDB::clusterClient accept TypeDBOptions and TypeDBOptions.Cluster, respectively.
Make TypeDBOptions.Cluster accept TLS parameters: tlsEnabled and tlsRootCA.

Superseded by https://github.com/vaticle/typedb-client-java/pull/333


what is inputFileGenerated?

hi every one what is what is inputFileGenerated and what's it usage?

It should be used whenever not yet existed file needs to be sent. For example,

A compressed photo, generated from an existing one.
A compressed video, generated from an existing one.
A file, which first needs to be downloaded from an HTTP URL.
A currently recording voice/video note.
A file outside library cache that needs to be copied and stored in the cache.
To achieve that the client specifies two pretty arbitrary strings original_path and conversion and should be able to generate the file based on value of these parameters whenever it receives updateFileGenerationStart.
For example, a client can use for the mentioned cases:
Path to the original photo as original_path and "compress_photo" as conversion.
Path to the original video as original_path and "compress_video" as conversion.
URL of the file as original_path and "download" as conversion.
".ogg" as original_path and "record_voice" as conversion.
Path to the original file as original_path and "copy" as conversion.

There are a lot of more complex cases when inputFileGenerated can be used. Basically all cases except sending a local/already sent file should be covered by inputFileGenerated.

tanks

is there any list of conversions? Or should dev implement each?

There is no built-in conversions except "#url#" used for files, which need to de downloaded from an HTTP/HTTPS URL. But even that conversion needs to be handled and implemented by the client.
Also, all conversions beginning with '#' are reserved for internal TDLib usage.

How to register conversion so tdlib can know which method to call?

Should I use  updateFileGenerationStart and updateFileGenerationStop as wrapper?

You must handle updateFileGenerationStart and updateFileGenerationStop updates from TDLib, and then provide progress about the generation using setFileGenerationProgress and finishFileGeneration.

Since TDLib 1.4.0 you can also use the method writeGeneratedFilePart if you have no direct access to TDLib's file system.


(ChatLiteLLM - Together AI) AttributeError: 'ValueError' object has no attribute 'status_code'

System Info
platform: Vagrant - Ubuntu 2204
python: 3.9.18
langchain: 0.0.350
langchain-core: 0.1.1
langchain-community: 0.0.3
litellm: 1.15.1
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction

Run the code:

from langchain.chat_models import ChatLiteLLM
from langchain.schema import HumanMessage

# Initialize ChatLiteLLM
chat = ChatLiteLLM(
    model="together_ai/mistralai/Mixtral-8x7B-Instruct-v0.1",
    verbose=True,
    temperature=0.0,
)

text = "Write me a poem about the blue sky"
messages = [HumanMessage(content=text)]

print(chat(messages).content)

2: Error message:
AttributeError: 'ValueError' object has no attribute 'status_code'

Expected behavior
I can't get ChatLiteLLM to work with Together AI. I expect ChatLiteLLM to work correctly and to output the result.

Having this issue too
i ended up using togetherai from langchain directly
from langchain_community.llms.together import Together

llm = Together(
   model="mistralai/Mistral-7B-Instruct-v0.2",
   temperature=0.7,
   top_k=1,
)


Generator support for URL parameters, updated Meetings endpoint family, new MeetingRegistrants endpoint family

Generator support for URL parameters
Some endpoint families require additional parameters to be passed as part of the URL. Examples:
https://webexapis.com/v1/meetings/{meetingId}/registrants
https://webexapis.com/v1/locations/{locationId}/floors
This adds support for URL parameters to the SDK generator.
For endpoints requiring URL parameters, YAML template:

must include the parameter(s) in the endpoint definition:  endpoint: meetings/{meetingId}/registrants
must have URL parameters described under url_parameters key, including name, description, and type.
URL parameters are always treated as mandatory

New MeetingRegistrants endpoint family
Adds support for MeetingRegistrants endpoint family generated with the updated generator.
Updated Meetings endpoint family
Small updates to the Meetings endpoint family. It was re-generated with the updated generator.

Hello @zhenyamorozov Thanks for the PR! Since you are creating some new feature or functionality, please also write a test to verify that your code works as expected.
We have an upcoming update from @cmlccie that will get the tests working in master. But if you could get this queued up that would be great.

Hi,
I've added tests for meetings and meeting_registrants endpoints.
I hope they are alright. I had nearly zero knowledge about pytest or testing in general before writing these. All tests pass for me. I am getting some warnings, but they seem unrelated to these particular tests.


failed to resolve lj_bcwrite

failed to resolve lj_bcwrite
error loading lua_shared
i think pattern is dead "\x83\xEC\x24\x8B\x4C\x24\x2C\x8B\x54\x24\x30\x8B\x44\x24\x28\x89"

resolved.

I'm not sure if you ever figured it out (or even care still as this issue was opened a long time ago), but the symbol now is:
\x83\xEC\x24\x8B\x44\x24\x2C\x8B\x4C\x24\x28\x89\x44\x24\x10\x8B

I'm not sure if you ever figured it out (or even care still as this issue was opened a long time ago), but the symbol now is:
\x83\xEC\x24\x8B\x44\x24\x2C\x8B\x4C\x24\x28\x89\x44\x24\x10\x8B

error C2390 visual studio 2019 windows.

Hmm, I didn't have this error yesterday when I wasn't using VS 2019 (I was simply compiling it with msbuild manually using god knows what version) - but now that I've installed VS 2019 I am experiencing this error as well.

Hmm, I didn't have this error yesterday when I wasn't using VS 2019 (I was simply compiling it with msbuild manually using god knows what version) - but now that I've installed VS 2019 I am experiencing this error as well.

try it
#undef__cplusplus

or

#define offsetof(s,m) ((size_t)&(((s*)0)->m))

Yeah, that works


switched the beta function sampling to jStat

the rbeta function was incorrect and so switched it to jStat. the tests and such still need to be updated.

Hello, @astronomeralex, thanks for raising this issue.  The code you are proposing to replace is based on this file:
https://github.com/CamDavidsonPilon/Probabilistic-Programming-and-Bayesian-Methods-for-Hackers/blob/master/Chapter6_Priorities/d3bandits.js
It sounds like you believe this code is in error.  I personally don't have the expertise to say one way or the other.  I've taken the algorithm from that file, trusted it to be correct, and exposed the algorithm as a node.js library.
Perhaps the best way to get this resolved would be if you raise this "upstream" in that repository.  I think that might be the best way for you to get an answer as to why this code is the way that it is.  You might also find someone who is better able to tell you why the code is the way that it is.  Also if there is an error in the algorithm, I imagine the maintainers of the "upstream" repository might appreciate learning about this.

I'm following up on this issue, @astronomeralex.
According this comment, d3bandits.js is be based off "Simulation and MC, Wiley".  There appear to be a few related tomes that potentially match this description - I guess different editions.   I dug up the 2007 edition of "Simulation and Monte Carlo - With applications in finance and MCMC" by J. S. Dagpunar and found a Maple implementation that appears to match what's in d3 bandits on page 250.  I'm reproducing the Maple code here:
beta:=proc(a,b) local z,r1,r2,w,rho,m,la;
if a>0 and b>0 then
  rho:=a/b;m:=min(a,b);if m<=1 then la:=m else
la:=sqrt((2*a*b-a-b)/(a+b-2)) end if;
do;
  r1:=rand()/10^12;
  r2:=rand()/10^12;
  z:=(1/r1-1)^(1/la);
  if
evalf(4*r1*r1*r2-evalf(z^(a-la)*((1+rho)/(1+rho*z))^
(a+b)))<0 then break end if;
  end do;
  evalf(rho*z/(1+rho*z));
else ERROR("a and b should be positive") end if;
end proc:

Looking it over, the algorithm in bayesian-bandit.js and in d3bandits appears to match the algorithm given in the source text book.
In case it's helpful to anyone reading this, Dagpunar in turn cites the following as the source of this algorithm:

Cheng, R.C.H. (1977) The generation of gamma variables with non-integral shape parameter.
Applied Statistics, 26: 71–76

Regarding bayesian-bandit.js's repo in particular - I'd rather not leave an ambiguous issue like this open.   I feel like I've done sufficient diligence on this for now, reviewing the source of this algorithm (d3bandits) and comparing against the source of the source's algorithm (Dagpunar).  If you believe there is still a problem, I am happy to reopen this issue, but I ask you open an issue in the probabilistic programming repo, link it to this one, and please expand on your comment, preferably in the "upstream" repo.
Again I want to say thank for you reaching out and reviewing my code.  However this ends up getting resolved I believe a potential problem like this is worth investigating.
@CamDavidsonPilon FYI.

A+ issue resolution, @omphalos - specifically diving into the books and comparing implementations.


Fix compilation error for unit tests

What this PR does / why we need it
This PR updates some IDs in context_test.go which didn't compile.
Note that the unit test still fail, but it is because of a change in https://github.com/vmware-tanzu/tanzu-plugin-runtime/pull/103.
At least the tests compile, which also fixes the linter.
Which issue(s) this PR fixes

Fixes # N/A
Describe testing done for PR
make lint
Ran the tests in context_test.go: they now fail but they compile; they used to not compile.
Release note

Additional information
Special notes for your reviewer

@marckhouzam thanks, I noticed as well and I am taking care of it in my change

Resolved by #510


Delegate or add maintainers

In light of how a PR took 8 months to be accepted, and how the library published on npm is a whole 3 years behind what it is on here, I think it might be time to add additional maintainers on both github and npm in order to ease things up and allow more people to contribute to what should be a lively project.
I would be more than happy to shoulder at least part of that burden as I use elasticlunr in production on two projects, and authored about half of the rewrite that just went through the latest PR. I'm also pretty sure I would not be the only one.

@weixsong Thoughts on this? This is used on production for many https://www.gatsbyjs.org/ websites.
Gatsby Plugins:

https://github.com/gatsby-contrib/gatsby-plugin-elasticlunr-search (over 11k downloads)
https://github.com/andrew-codes/gatsby-plugin-elasticlunr-search

@srenauld , I have added you to the Collaborators, please try it. And really thanks very much for the contribution.

@weixsong Thank you for the change, that is one positive step forward!
As I see it, there are a few ways to make this mostly self-managing, but it will require you to do a few more things. They don't have to be done right now.
Step 0 is proper gitflow, which I can set up myself. As I doubt we will need to backport changes across multiple versions, I am pretty confident we can keep a single development and staging branches, and as such, the flow will be from dev to staging, staging to master in order to release.
The first is continuous integration, which we don't have at the moment, but the PR I threw at this project 9 months ago allows this (as I changed the underlying test framework to something most CI/CD tools understand). I will get to making a travis configuration and checking whether it fits our needs to validate a build.
While doing this, I will include e2e tests such as a basic gatsby setup, as the gatsby community relies on this to function.
The added bonus of both changes is that, if they're in and gitflow is obeyed, we can then deploy to npm automatically when something happens on master.

@srenauld , great, as I haven't been working on JS for a long time, I do not know what it the best way to develop the JS project. You could do whatever you think is right and make the repo popular again. :)
Thanks again.


AttributeError: 'PublicKeyInfo' object has no attribute 'issuer'

running these scripts in sequence.
python cert-make.py
python pdf-make.py
python pdf-sign-cms.py

produces

Traceback (most recent call last):
File "pdf-sign-fpdf.py", line 26, in
main()
File "pdf-sign-fpdf.py", line 23, in main
doc.output('pdf-signed-fpdf.pdf', "F")
File "C:\HOMEWARE\Anaconda3-Windows-x86_64\envs\abc\lib\site-packages\endesive\pdf\fpdf\fpdf.py", line 1065, in output
self.close()
File "C:\HOMEWARE\Anaconda3-Windows-x86_64\envs\abc\lib\site-packages\endesive\pdf\fpdf\fpdf.py", line 246, in close
self._enddoc()
File "C:\HOMEWARE\Anaconda3-Windows-x86_64\envs\abc\lib\site-packages\endesive\pdf\fpdf\fpdf.py", line 1673, in _enddoc
self.pkcs11_sign()
File "C:\HOMEWARE\Anaconda3-Windows-x86_64\envs\abc\lib\site-packages\endesive\pdf\pdf.py", line 65, in pkcs11_sign
signed_md)
File "C:\HOMEWARE\Anaconda3-Windows-x86_64\envs\abc\lib\site-packages\endesive\signer.py", line 23, in sign
'issuer': cert.asn1.issuer,
AttributeError: 'PublicKeyInfo' object has no attribute 'issuer'

After a bit of digging, it seems that the information has been lost during the export function in the pkcs12.py
serial_number is also lost during the process which are required in the sign stage later.
Would you be able to take a look? Thanks.

The information is in the key itself.
It seems like the information is lost during the key construction in the oscrypto library, i.e. asymmetric.py
Probably a windows issue, will test it on a Mac later.

I have removed dependence on oscrypto, now the library also works on MSW


pip-sync 7.0.0 reinstalls packages due to case difference

Description
pip-sync 7.0.0 reinstalls all packages which have non-lowercase letters in their names
For instance in *.in and *.txt files we have the following data:
django

and
django==4.2.0
  ...

When we run pip sync, it will uninstall and install django because in pip-freeze the same package mentioned as Django
Environment Versions

Required
Information

OS Type
MacOs/Linux

Python version: $ python -V
Python 3.11.4

pip version: $ pip --version:
pip 23.2 from /home/user/venv/lib/python3.11/site-packages/pip (python 3.11)

pip-tools version: $ pip-compile --version
pip-compile, version 7.0.0

Steps to replicate

Create *.in file with django requirement
Compile requirements to generate *.txt file
Run pip-sync on the txt file
Run pip-sync on the txt file again

Expected result
The output should be Everything up-to-date
Actual result
The requirement is reinstalled every time you run pip-sync

Thanks for the issue! See https://github.com/jazzband/pip-tools/issues/1917#issuecomment-1638197374 for details.

@sshishov could you test this fix #1919? Does it resolve the issue?
pip install git+https://github.com/atugushev/pip-tools@fix-broken-sync-diff

@atugushev going to test it now, sorry for the long reply


Added Comments on functions which needed comments in ag_label.c and label_commands.c

Closed as unnecessary. This is a repackage of #633 with additional comments that aren't necessary.


Tutorial text is different on exported release vs in editor

On the left is in editor, on the right is exported release.
The left is how it should look but for some reason it is now spaced out quite a bit in exported release.

I think this has something to do with the ui element settings.
https://docs.unity3d.com/2018.4/Documentation/Manual/HOWTO-UIMultiResolution.html


Prod: log in sometimes fails silently on zaino.cc

Broken on https://zaino.cc/
fine on https://zaino-2e6cf.firebaseapp.com/

Implemented temporary workaround: use signInWithPopup.


Implementing use of PSR7

This PR implements PSR7 and PHP-HTTP. The benefit of this is that we don't need to care about the HTTP transport implementation. The user of this library can choose that HTTP adapter to use. So, we automatically support Guzzle6.
This PR will fix #57.

:+1: please merge

I've just made some major changes to this PR. The public API has not changed much. Our dependencies are starting to get stable. I hope to merge this within a couple of weeks.

This PR is a big rewrite of the code. The public API has not changed much. See the Upgrade.md docs.
I want to focus on readable code and being SOLID as much as possible. Please try this out and tell me what you think.


Change mso_mdoc signing implementation

Closes #153
This PR removes the dependency to the external service and implements mso_mdoc signing internally.
Now, issuer is able to use a single signing key (loaded by external key store) for all formats.

[x] Migrate PID in mso_mdoc to the internal signing service
[x] Migrate mDL in mso_mdoc to the internal signing service
[x] Remove dependency to the external signing service
[x] Load a single signing key from external keystore (Perhaps remove support for randomly generated keys)
[x] Possibly, add a section in the README, providing details on how to create the keystore

@babisRoutis could you please check and let me know of any comments?


Hi PLanning to Implement Shamapper not an issue

Hi
I landed here when google for EntityFrameweork for Sharepoint. currently I developed an application and using it without any mapper.
Do u u see any issues in long run, if I implement in my project.
planning to figure out a way to implement it

Please let me know ur thoughts, how stable is this product ?

Hi.
It is stable, but not all field types supported (all general types are supported and this is enough in most cases). I would recommend first try to work without this library, maybe you don't need it at all. Then if you realize, that you need some sort of mapper/linq-to-sharepoint, then it is good choice to use this lib.


Guide feedback

Following the guide and there are some bits that seem wrong.

'Compute Engine Virtual Machine Setup' -> Customize the instance
This has the exact same picture as above. There is also not an option to customise the instance.

Machine type. There is no dropdown. It instead looks like this. I assume this is the right selection.

It is not clear that after setting the external IP to a static IP that you should press Create before you edit the firewall and can SSH in.

The part about specifying upstream DNS providers is unclear. I think the PiVPN and Pi-hole instructions are mixed.

The question about entering a custom domain/custom search domain is during PiVPN not Pi-hole.
The bit about a default DNS is in Pi-hole not PiVPN.

Installing PiVPN, default is now Wireguard so if you want people to use OpenVPN then they need to be told to change that to OpenVPN. The part about the custom search domain comes in here.

That is as far as I got in the guide.
edit:
I tried setting the DNS to custom and <IP_ADDRESS> during installation of PiHole and got the following
[i] Pi-hole blocking is enabled
[✗] DNS resolution is currently unavailable
[✗] DNS resolution is not available

Things appear to have changed with both the Google Cloud Console and the PiVPN setup. I will be making some updates soon. The demand for Wireguard is much higher, so I will be focusing my efforts on getting a Wireguard guide drafted before coming back to this OpenVPN one.

Thanks for your contributions @rajannpatel and team, another minor yet annoying issue I encountered because I went into zombie copy paste mode, regarding the guide for your older repo OpenVPN + Pi Hole - the instruction for adding the NAT table rule. The rule "-A POSTROUTING -s <IP_ADDRESS>/24 -o eth0 -j MASQUERADE" is I'm guessing out of date since the default public interface created on my GCP vm was named "ens4". Verified this was what indeed what should have been used in the NAT rule based on your new setup.sh script where you pull the public nic info using the "ip -4 route ls | grep default | grep -Po '(?<=dev )(\S+)' | head -1" command.
Having the incorrect public ipv4 interface in the rule declaration resulted in no internet connection while Pi Hole worked fine :| Just throwing this out there in case someone else runs into the same issue. Thanks again for this, really awesome guide.

So when installing PiVPN you will be asked to choose a DNS Provider, we don't want to push any 3rd party DNS services to clients connecting via VPN. We want to push <IP_ADDRESS>, which is our Pi-Hole DNS Server.
Pi-Hole can also act like a Router, and that is why during its installation you have the option of choosing a Custom Domain / Search Domain.

For whatever reason, PiVPN was unable to install and would exit the installer before I could get to the prompt for DNS provider. Based on your comment above with changes in GCP with PiVPN, I assume this could be part of the issue. I had to improvise a bit and followed some parts of the guide here - https://docs.pi-hole.net/guides/vpn/overview/.
Side note - your Pi Hole + Wireguard config script is great and it did work for me, but I realized that Windows users must have admin privileges to run WG client, and not all my users necessarily have that. Thus had to go this route. It's working great so far. Thanks again.

I will run through the steps again tonight and figure out what may have changed, thanks for the heads up!

@rajannpatel Much much appreciated this OpenVPN guide. Please consider updating this OpenVPN Guide
Doing everything step by step I dont get connection via TCP nor UDP

Wireguard is also free why even bother with not working OpenVPN
All go for Wireguard ;)

Recommending folks use the Wireguard tutorial going forward: https://github.com/rajannpatel/Pi-Hole-on-Google-Compute-Engine-Free-Tier-with-Full-Tunnel-and-Split-Tunnel-Wireguard-VPN-Configs


Fixed .gitignore file

Basically no-one actually uses composer.phar per project these days, and if they do, then can ignore it in their system ignore file. As for .DS_Store, that 100% belongs in the system ignore file, not the project one.

:+1:

:+1:


ember build source map failure with<EMAIL_ADDRESS>

Our project uses a custom command line flag to ember build which causes this stack trace.
We are using<EMAIL_ADDRESS><EMAIL_ADDRESS>start /myapp
> ember build --environment realprod

WARNING: Node v8.0.0 is not tested against Ember CLI on your platform. We recommend that you use the most-recent "Active LTS" version of Node.js.
DEPRECATION: Addon files were detected in `/myapp/node_modules/pagination-pager/addon`, but no JavaScript preprocessors were found for `pagination-pager`. Please make sure to add a preprocessor (most likely `ember-cli-babel`) to in `dependencies` (NOT `devDependencies`) in `pagination-pager`'s `package.json`.
DEPRECATION: Addon files were detected in `/myapp/node_modules/ember-cli-copyable/addon`, but no JavaScript preprocessors were found for `ember-cli-copyable`. Please make sure to add a preprocessor (most likely `ember-cli-babel`) to in `dependencies` (NOT `devDependencies`) in `ember-cli-copyable`'s `package.json`.
[WARN] (broccoli-uglify-sourcemap) Minifying: `assets/vendor.js` took: 22942ms (more than 20,000ms)
cleaning up...
Build failed.
Cannot set property stack of SyntaxError which has only a getter
TypeError: Cannot set property stack of SyntaxError which has only a getter
    at Class.buildError (/myapp/node_modules/ember-cli-less/index.js:82:19)
    at addonPromises.project.addons.map.addon (/myapp/node_modules/ember-cli/lib/models/builder.js:139:34)
    at Array.map (native)
    at Builder.processAddonBuildSteps (/myapp/node_modules/ember-cli/lib/models/builder.js:137:43)
    at processAddonBuildSteps.then.then.then.then.then.then.catch.error (/myapp/node_modules/ember-cli/lib/models/builder.js:172:14)
    at tryCatch (/myapp/node_modules/rsvp/dist/rsvp.js:539:12)
    at invokeCallback (/myapp/node_modules/rsvp/dist/rsvp.js:554:13)
    at publish (/myapp/node_modules/rsvp/dist/rsvp.js:522:7)
    at publishRejection (/myapp/node_modules/rsvp/dist/rsvp.js:457:3)
    at flush (/myapp/node_modules/rsvp/dist/rsvp.js:2414:5)

This is in a private app; I will try to update later with a test app.

resolved by #59


Add chat.getPermalink support

Fixes #411 by providing a GetPermalink function. Included tests validate a successful request and associated response.

mind just rebasing against master? I made a mistake this morning when moving some personal branches around and had to fix master.

No problem, will try and get it done today.


Add "Custom JSON Serializer"

While browsing docs, I couldn't see a mention of JSONSerializer. While browsing the source code of Echo, by accident, I found the JSONSerializer. I think it should be documented.

thanks!


Kubernetes plugin is not correctly reading the external id from the aws config

The kubernetes plugin is not correctly reading the external id from the aws config
Expected Behavior
It should read external id and use it to pass to the sts assume role call.
Current Behavior
Currently it does not read the external id.
Possible Solution
Pull request 6891 fixes this.
Steps to Reproduce
Setup a cross account role in AWS with an external id
Configure the Backstage k8s backend with an external id with eks
You should expect backstage to work with the role. However it does not.

Your Environment
N/A

https://github.com/backstage/backstage/pull/6891 has the fix for this issue.

6891 is merged, so Ill close this one.


Merge pull request #1 from ddd-edm-online-course/master

[LAB 3] submission

Pulling Lab 3 Submission


Refresh, Chart Legend and t value synchronizing fix

Fixes corresponding to #114, #127, #122 and #128.

Observations for fixing refresh crashes:

ScaledFrameLayout exception catching onDispatchDraw fixes the issue in nearly all of the cases
NotifyDataSetChanged changing to data.setValues() from copy of entries and waiting for refresh to finish fixes the issue for non-Pie Charts


[WIP] apiserver: update APF documentation for GA

Promote APF flowcontrol API to v1 - https://github.com/kubernetes/kubernetes/pull/121089
set APF featuregate to stable - https://github.com/kubernetes/kubernetes/pull/121638
KEP: https://github.com/kubernetes/enhancements/tree/master/keps/sig-api-machinery/1040-priority-and-fairness
Enhancement issue: https://github.com/kubernetes/enhancements/issues/1040

/cc @sftim @MikeSpreitzer @wojtek-t

LGTM

Please do not add api-ref-assets/api/swagger.json or change the toc.yaml file in this PR.
Those files will be updated in a batch when 1.29 is out.

/lgtm

This PR should only change one language (English)
/lgtm cancel

English only now.
/lgtm

Thank you for replying to comments
/approve


Includes homebrew lib path

Allows this to build in OSX.

@mwhansen PR and :+1: please. This works for me in vagrant and on mac with the sqlite homebrew package installed.

@bshelton229 this works fine for me in vagrant.

@mwhansen Can you think of any reason why this wouldn't be OK to merge into our rover branch? I tested it here.
https://github.com/roverdotcom/web/pull/8078


Input under-scrolling header & not returning using accessory bar

I extracted a view from an app I was working in to a fresh ionic project and the bug still exists.

ionic-plugin-keyboard 2.2.0 "Keyboard"
ionic v1.3.1

All when using Accessory bar:
When progressing to a field that is hidden by the keyboard at the bottom the view scrolls to behind the header bar.
When returning back to a previous field no attempt to return the field into the view is made and the input if focused off the top of the screen.

Apologies for the low res gif, full video file at: https://drive.google.com/file/d/0BwXpo6QvBD3VWU5CR2xsMnBMYVU/view?usp=sharing
Used code in the above test environment is:
`

<ion-pane>
  <ion-header-bar class="bar-stable">

Ionic Blank Starter

  </ion-header-bar>
  <ion-content>

      <ion-list>
        <ion-item class="item-divider">
          Parent / Guardian Info
        </ion-item>

        <ion-radio ng-model="guardianDetails.relationshipType" ng-value="'parent'">Parent</ion-radio>

        <ion-radio ng-model="guardianDetails.relationshipType" ng-value="'guardian'">Guardian</ion-radio>

        <ion-item class="item-divider">
          Details
        </ion-item>

        <ion-item class="item item-input">
          First Name:
          <input type="text" placeholder="First Name"
                 required
          >
        </ion-item>

        <ion-item class="item item-input">
          Last Name:
          <input type="text" placeholder="Last Name"
                 required
          >
        </ion-item>

        <ion-item class="item item-input">
          Contact Number:
          <input type="text" placeholder="Contact Number"
                 required
          >
        </ion-item>

        <ion-item class="item item-input">
          Extra field:
          <input type="text" placeholder="Contact Number"
                 required
          >
        </ion-item>

        <ion-item class="item item-input">
          Email:
          <input type="email"<EMAIL_ADDRESS>                 required
          >
        </ion-item>

            Nationality:

          <select
            required
            >
              <option value="">Select nationality</option>
            </select>

            Language:

          <select
          required>
            <option value="">Select language</option>
          </select>

            Language:

          <select
          required>
            <option value="">Select type</option>
          </select>

        <ion-item class="item item-input">
          Address:
          <textarea
            id="" cols="30" rows="10"
            required
            >
          </textarea>

        </ion-item>
  </ion-content>
</ion-pane>

`
Any help greatly appreciated.

If you aren't using cordova.plugins.Keyboard.disableScroll(true) try that. If that doesn't work, I'd try overflow-scroll="true" on your ion-content, since the translated scroll view isn't actually scrollable in the browser's mind, so it really messes up the layout to bring the input into view.

Adding the overflow-scroll="true" to the ion-content block seems to work, though any fields that are on the border of view / keyboard strangely get doubled up and this stops the input from working screenshot of DOM attached

Sounds like a bug in the framework. As a workaround though I would try and keep inputs above the keyboard as much as possible, by breaking the different form sections into separate pages.

any other solutions for this problem.. If my input is at bottom of the page, then on focusing it doesn't scroll up into view(above keyboard) automatically in android.
Then If i scroll page up then page moved up by some extra space and it never comes down by any mean neither by closing keyboard, neither by scrolling page down manually, nor by clicking somewhere else.
Below settings are in my app.js file for keyboard
        if (window.cordova && window.cordova.plugins.Keyboard) {
            cordova.plugins.Keyboard.hideKeyboardAccessoryBar(false);
            cordova.plugins.Keyboard.disableScroll(false);
        }

<ion-content has-header="true" has-footer="false" scroll="false">
	<ion-scroll ng-if="multiplePagesProposal"
		overflow-scroll="true"
		direction="xy"
		locking="false"
    	zooming="true"
		has-bouncing="false"
		scrollbar-x="false"
		min-zoom="zoomOptions.minimumZoom"
    	delegate-handle="zoomScroller"
		class="proposal-ion-scroll proposal-template-scroll">

 	</ion-scroll>
</ion-content>

This directive fixed it for me:
.directive('form', function () {
      return function () {
        if (window.cordova && window.cordova.plugins.Keyboard && navigator.userAgent.match(/(iPod|iPhone|iPad)/)) {
          cordova.plugins.Keyboard.hideKeyboardAccessoryBar(false);
        }
      };
    }
  );

I've accomplished a different workaround on this.
angular.module('app')
  .run(function($rootScope, $ionicPlatform, $ionicScrollDelegate){

      $ionicPlatform.ready(function () {
          if (window.cordova && window.cordova.plugins.Keyboard){
              cordova.plugins.Keyboard.disableScroll(true); // This will prevent the view to bounce when inputs are on focus
          }
      });

      $rootScope.$on('$ionicView.loaded', function () {
          if (window.cordova && window.cordova.plugins.Keyboard) {
              cordova.plugins.Keyboard.hideKeyboardAccessoryBar(false); // This makes the accessory bar visible and it only works when the view is loaded and DOM ready
          }
      });

      window.addEventListener('native.keyboardshow', function () {
          $ionicScrollDelegate.scrollBy(0, 1); //This will return focus to the current input once the keyboard slides-up in the view
      });

};)

@alejandro-heredia-coderoad-com  what if I tap in one input field then tab to the second input field. In that case  $ionicScrollDelegate.scrollBy(0, 1); this will not work :-(


Fix issue with PHPUnit test paths on linux

First of all, thank you @DumbergerL for all the work you've put in with 2.0! It's looking great!
It seems like calling PHPUnit fails (at least on Linux), because phpunit.xml still has the lowercase directory names instead of the ones you've changed in #165. Perhaps it didn't fail on Windows because directories there are often treated case-insensitive (I didn't validate that however). It's a good step to make tests PSR-4 compliant though!
I wondered why PHPUnit didn't fail in CI and saw that for some reason it doesn't throw an error in Github Actions, but it says "No tests executed!". Perhaps PHPUnit could be setup to fail when no tests are executed? https://github.com/5pm-HDH/churchtools-api/actions/runs/6744038583/job/18333071116

Seems like the integration CI is broken and I'm not sure why. It seems like it's looking in the right place for the injected .ini but the credentials are not there:
https://github.com/5pm-HDH/churchtools-api/blob/04f7700e89b96ef54fa8e676c16cb8e8c5991dfd/tests/Integration/IntegrationTestData.php#L11-L15

I can't understand the behavior of the github-actions pipelines 100%. When I push commits to this PR, the credentials file obtained from the github secrets is empty. As soon as I push the commits to a new branch and create a fresh pr, the pipeline runs through and this PR also turns green.
I close this PR. All changes are continued on pr #186 .


Argument names in Roc documentation

What is the plan around argument names in Roc standard library docs?
For example, looking at List.reserve, it will "Enlarge the list for at least capacity additional elements". The function signiture is [reserve](https://www.roc-lang.org/builtins/List#reserve) : List a, Nat -> List a.
This is essentially implicitly naming the second argument "capacity". This may be confusing in some situation with more arguments. So two main questions:

Can/should we add some sort of function parameter name documentation to the standard library docs?
If we aren't going to add name information, how should we refer to each argument/what would be better wording?

I think adding parameter name documentation is a really good idea, but I don't have the greatest sense of the right way to do it.
One nice property of the docs today is that you can copy and paste a type signature into Roc code; that is, the signature docs are pure Roc code.
If we care about preserving that property, I think there are a few options here.

We could add hover tooltips over the types of parameters to show their displayed names. That is, something like this, but prettier:

I think this solution has some downsides. One is that you can't see all of the parameter names at once, you have to hover over each, which can make reading the docs harder. The second is that this interactive aspect could be hard on mobile, and I don't think you necessarily want that much interactivity to read docs.

Add in-line parameter names that are smaller and not selectable. That is, something like this

And when you select the type signature, the parameter names will not be selectable. That way you can still copy-paste the type signature. I like this option more but it has the downside that it's no longer pure Roc syntax. OTOH I don't think it's likely anyone will think this is proper syntax.

There are other options like having two lines,

which doesn't look too bad in this example, but wouldn't play well with multi-line type signatures.
From the technical side we'd also need to figure out how to implement these kinds of docs for signatures missing implementations, like straight-to-lowlevel functions in the stdlib. I guess we could just hard-code those.


[$1000] Display name not shown for new user in mention

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open staging website on chrome (User 1)
Login with User 2 as a NEW user on another chrome and change display name
From user 1, click on FAB icon and click on new chat and input the User 2 email
From user 1, on chat composer, mention User 2 display name and notice that mentioning user 2 display name doesn't show the suggestion (only works if mentioned email)

Expected Result:
Mentioning display name of User 2 should show the mention suggestion
Actual Result:
Display name not shown for new user in mention
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.37-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/507e3db8-68bb-4ab5-9cb3-ee1126c75ec9
https://github.com/Expensify/App/assets/43996225/a1f41142-8de0-42db-b21d-24f1eb46bc6a
Expensify/Expensify Issue URL:
Issue reported by: @avi-shek-jha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688571414432459
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0136ede1fc01c6d55b
Upwork Job ID:<PHONE_NUMBER>224551936
Last Price Increase: 2023-07-06

Was able to recreate, so going to get eyes on this

We might need to handle this on the backend. I see that when we search for the user, they are given a temporary ID in Onyx personalDetails. When we open the report we receive a number of Onyx actions, one of which includes a merge on personalDetails with the user's correct ID, but it only includes their avatar and display name, as shown in ginsuma's comment.
We don't really have a way to connect the two on our side that I can see. Perhaps the merge sent in the response data should include the user's email as well, or a SET onyx action before the merge... something to give us a connection between that email and that ID.

@ginsuma and @rain2o Thank you! So let me summarise this from my point of view based on what you said:
When we create a new chat we set an optimistic personalDetails object that contains login and that explains why we can mention using the email. Then later on we get another personalDetails object (with a different key since the accountID cannot be pre-determined) that contains the display name but not login and that explains why we can't mention using the display name.
So we have two personalDetail objects that are meant to refer to the same thing but they don't since the key is not the same?

@ginsuma Yes and we already allow to be mentioned by the display name. In this case it's not working just because the data is split into two objects (optimistic object and the real server response object).

Not overdue 👍

Based on the Slack discussion https://expensify.slack.com/archives/C01GTK53T8Q/p1688995928956959 we can close this. cc @CortneyOfstad

Sounds good — thanks @s77rt!


Add bail! and ensure! macros

PR for #72. I copied and adapted the macros from error_chain. I used the  err_msg function, which means that anything that works for err_msg will also work. Editor did some unintended formatting also.

Merged in through #86. Thanks!


Updated the file with latest spec file.

Updated the file with latest spec file to fix the issue # 352

Would this be visible on the page? Look at blue-highlighted section in attached screen-shot :


Use hasOwnProperty to check for object properties

I've been doing some performance profiling of Aphrodite and noticed that
even after updating to inline-style-prefixer 3.0.0, the prefixValue
function was still more expensive than I expected. After looking at the
CPU profile and the lines highlighted here, it seems that checking for
the existence of these object properties using foo[bar] is much more
expensive than checking using foo.hasOwnProperty(bar).

In my profiling of Aphrodite's css(), this PR plus #115 and https://github.com/rofrischmann/css-in-js-utils/pull/1 dramatically improve the performance of prefixValue().
Before, 429ms (21.41% of total css() time):

After, 75ms (6.36% of total css() time):


Replace the current shunts with needed 75 milliohm resistors.

Manufacturer sent steel shunts rather than resistive shunts.

Shunts replace. Current readings still didn't work, so the shunts were bypassed for now.


Add auto merge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/icet-feedstock#9.


Fix circuit breaker empty response, fixes #170

Vendor in new oxy and scroll

@r0mant can you take a look at the PR? It's very simple

lgtm


feat: ping metadata

Closes #445

@mpaulucci I tried some options to correctly run the CI / Start and stop the node (pull_request) but it doesn't work seems like you have approved with its failure is existing is this something which doesn't need to be fixed?

@karasakalmt that should work. Have you tried running the node using make checkpoint-sync?


[FLINK-5702] [doc] At-least-once configuration info for FlinkKafkaProducer

Adds information about the setLogFailureOnly and setFlushOnCheckpoint methods with regards to at-least-once guarantees of the producer, and warns that at-least-once is compromised if configured inappropriately.

Thank you for the review @uce! I'll address all of them except the last one (please see my inline reply).

The comments have been addressed. Apart from the minor fixes for grammar, all sections were moved up the hierarchy by one to show a better ToC. The at-least-once info for the producer is moved to a sub-section called "Kafka Producers and Fault Tolerance" (coherent with the naming for exactly-once info on the consumer side).

Makes sense, +1 to merge.

Merging to master and release-1.2 ..


Clean up timers on receipt of a non-0x78 UDS error

Description
The end user complaint (simplified) was that openpilot master stays in "waiting for controls to start" for almost a minute at startup. The same car works correctly on openpilot <IP_ADDRESS>.
With #24724, the UDS fingerprinting code now handles "request received - response pending" errors by extending timeouts to wait for a successful response. However, queries can still fail at this stage.
In the case of a certain snowflake VW Polo engine ECU, we can get 0x7F2278 response pending quickly followed by 0x7F2231 request out of range. When this happened, the query was marked done but the timer wasn't cleaned up, which prevented the query loop from proceeding until the 10 second timer elapsed. This happened a few times to build up the larger total delay.
This fix makes sure failed queries are always completely terminated.
Verification
The end user reports this patch fixed his problem.
Route
Before: 0bbe367c98fa1538|2022-08-10--14-28-52
After: 0bbe367c98fa1538|2022-08-12--00-11-15
0bbe367c98fa1538|2022-08-12--00-06-36
0bbe367c98fa1538|2022-08-11--23-56-30

The patch appears to work as intended, released from draft.


Context menu

Right / Ctrl clicking an element should pop out a context menu

Copy
Delete
Paste
Move Forward
Move Backwards
Send to Back
Send to Front

—-
Right clicking a selection with multiple elements should have the option to group, right clicking a group should have option to ungroup.

Agreed, this way we don't have to put all those things on the left menu which is getting crowded!

I'll take it


feat(dpp)!: use blake3 for hashing

Issue being fixed or feature implemented

Trying to use blake3 for hashing
What was done?

installed polymorphic @dashevo/blake3
async init of the blake
updated imports

How Has This Been Tested?

Unit tests
Breaking Changes

old hashes are incomatible

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

The reason checks are not passing is that in Drive, an initialization of DPP (dpp.initialize) is hanging during loading of blake3 WASM.

@thephez This is posponed for v0.23


Avoid concurrent or useless runs

Avoid concurrent or useless runs

please reword the commit message and update PR title 🙂