Migrate to Capacitor 5

Updates the plugin to support Capacitor 5.
I've confirmed this is working for Android and iOS, tested in the simulator and through npm run verify

Version 1.0.3 (and lower) will target Capacitor 3
Version 2.0.0 will target Capacitor 4
Version 3.0.0 will target Capacitor 5

I've confirmed the messenger dialog is working correctly with Capacitor 5 on both iOS and Android

Just confirming: do these IOS breaking code changes not affect us? https://capacitorjs.com/docs/updating/plugins/5-0#ios

Sorry just realized I don't get Slack notifications for this repo


Improve Edb.modeler.primitives

Description of the current limitations
primitives is a collection.  Each time the prop is accessed, the full list is recreated. Each item in the list is an instance of Edb.  It would be far more efficient to create an internal self._prims property and update that every time a primitive is created or deleted.  The difficult part will be to catch when primitives are deleted.
Code sample expected
create self._prims as a list or instance of a collection class.  The latter would be more efficient.
Update items as primitives are created or deleted.
Note that each item in Edb.modeler.primitives is an instance of edb_core.edb_data.primitives_data.EdbPrimitives. Here the naming is confusing since the latter is not a collection.  It should probably be renamed to singular.
Useful links and references
https://docs.python.org/3/library/collections.html

I think the current class implementation EdbPrimitives should be renamed to EdbPrimitive and a new collection class called EdbPrimitives should be created as a container for instances of the former.

Edb is very efficient in collection handling. It can handle very efficiently for us.


bring in minimum compiler-rt dependencies

This seems to resolve all of the missing env imports when compiling sqlite. It's a bit janky to include random bits of compiler-rt, but it does seem to work.

I have updated SQLSync to this change and it seems like it's working very well.
https://github.com/orbitinghail/sqlsync/commit/1f985f2247d9d86140c7b7052eec38b51039c7cd


Rewards bar refactor

Silo V3 introduces mowing for individual tokens, which breaks the previous paradigm in the Claim Rewards dialog.  I went ahead and fixed both of these at the same time, and updated the logic to use mowMultiple when necessary.

Future work

Add tooltips to explain why certain things are disabled
Form isn't resetting correctly after submit
Roundoff errors with local enroot calculation, maybe use static call to calculate instead?

Requesting retroactive review


💡 [REQUEST] - Creating a About Us Section

[x] I am from GSSoC'23.
[x] I would like to be assigned this task.

Section
Description

Concise Description
When we click on About in NavBar, nothing happens, thus I want to create an About Us Section where the user is redirected to while clicking on About

Solution Proposed
This would be quite beneficial as the user will get to know about CropForeSight, its significance etc...

Alternative Solutions
We may create a section on the main page for About Us, or a separate page.

Additional Context
It would improve overall user experience and engagement.

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[x] I follow the Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme of this repository.

Hi @abhi03ruchi
Thank you for creating the issue. Please go ahead and start working on it. If you have any questions or need assistance during the process, feel free to ask.
Thank you for your contribution, and I'm excited to see your progress in resolving the issue.

Resolved Issue #37 , I have created a pull request plz review and merge.


chore: resolve warning for bottom sheet navigator

Description
Noticed this warning when running in dev mode a lot. Applied the recommended fix from here.
Looks like you're passing an inline function for 'component' prop for the screen 'DappKitSignTxScreen' (e.g. component={() => <SomeComponent />}). Passing an inline function will cause the component state to be lost on re-render and cause perf issues since it's re-created every render. You can pass the function as children to 'Screen' instead to achieve the desired behaviour.
Test plan
Tested locally on iOS by swapping cUSD to Ethix on Ubeswap.
Related issues
N/A
Backwards compatibility
Yes

Thanks, note that it only hides the warning and doesn't address the possible performance issue.
https://reactnavigation.org/docs/screen/#children


Cleanup http_server_filter.

I ran into this while working on something unrelated.
Prior to this PR, we were intercepting the on_complete callback for the batch containing the recv_message op in order to determine if we had seen recv_initial_metadata before recv_message.  This approach is more complex than necessary, and I don't think it's actually guaranteed to work, because we don't actually have any guarantee that we will see on_complete for recv_message before we see recv_initial_metadata_ready.
This PR simplifies the logic by simply having the recv_initial_metadata and recv_message callbacks be aware of each other, so that things will work properly in either order.

****************************************************************

libgrpc.so

     VM SIZE                                                              FILE SIZE
 ++++++++++++++ GROWING                                                ++++++++++++++

 -------------- SHRINKING                                              --------------
  -0.0%     -12 [None]                                                    -420  -0.0%
  -1.5%     -60 src/core/ext/filters/http/server/http_server_filter.cc     -60  -1.5%
      [DEL]    -166 hs_on_complete                                            -166  [DEL]
      [DEL]     -97 hs_on_recv                                                 -97  [DEL]
      [DEL]     -73 hs_recv_message_ready                                      -73  [DEL]
     -25.9%     -28 init_call_elem                                             -28 -25.9%
      -4.8%     -24 hs_start_transport_stream_op_batch                         -24  -4.8%
     -26.9%     -21 [Unmapped]                                                 -21 -26.9%
     -35.0%     -14 destroy_call_elem                                          -14 -35.0%

  -0.0%     -72 TOTAL                                                     -480  -0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                                                                FILE SIZE
 ++++++++++++++ GROWING                                                                              ++++++++++++++

 -------------- SHRINKING                                                                            --------------
  -1.5%     -60 src/core/ext/filters/http/server/http_server_filter.cc                                   -60  -1.5%
      [DEL] -2.58Ki server_filter_incoming_metadata(grpc_call_element*, grpc_metadata_batch*) [clone .is -2.58Ki  [DEL]
      [DEL]    -211 server_filter_outgoing_metadata(grpc_call_element*, grpc_metadata_batch*) [clone .is    -211  [DEL]
      [DEL]    -166 hs_on_complete                                                                          -166  [DEL]
      [DEL]    -143 add_error                                                                               -143  [DEL]
      [DEL]    -108 init_call_elem                                                                          -108  [DEL]
      [DEL]     -97 hs_on_recv                                                                               -97  [DEL]
      [DEL]     -63 init_channel_elem                                                                        -63  [DEL]
      [DEL]     -40 destroy_call_elem                                                                        -40  [DEL]
      -4.8%     -24 hs_start_transport_stream_op_batch                                                       -24  -4.8%
     -26.9%     -21 [Unmapped]                                                                               -21 -26.9%
      [DEL]      -2 destroy_channel_elem                                                                      -2  [DEL]

 -+-+-+-+-+-+-+ MIXED                                                                                +-+-+-+-+-+-+-
  -0.0%     -12 [None]                                                                                +92  +0.0%

  -0.0%     -72 TOTAL                                                                                +32  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences


[Bug] fix docker deployment

What changes were proposed in this pull request
Issue Number: close #1783

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (no)

Wait a minute, pr is not submitted

Waiting for CodeQL CI


How can I fix a corrupt manifest.nix?

Issue description
I'm really not sure when or what happened, but suddenly my nix-env is unusable!
nix-env -q
error: syntax error, unexpected $end, at /nix/store/0722zxv7zijaljpkrx342rjyjf56k979-env-manifest.nix:1:1

I couldn't find anything to fix it! I tried:
sudo nix-store --verify --repair --check-contents
reading the Nix store...
checking path existence...
checking hashes...
path ‘/nix/store/0722zxv7zijaljpkrx342rjyjf56k979-env-manifest.nix’ was modified! expected hash ‘632ec35215ef2d5e7adc4698e87db5ee331c21633f5a4a351e10a7e249c6dd00’, got ‘77ac62e2629d8e45f624589c0c8bf99e24b3a722349bf1e79bc186008534e246’
error: cannot repair path ‘/nix/store/0722zxv7zijaljpkrx342rjyjf56k979-env-manifest.nix’

I tried to ask on IRC or search the chat logs but couldn't really find any help anywhere. Is anyone knowledgeable enough to fix this kind of issue!?

can you pastebin the contents of the manifest.nix?

It's empty.
On Sun, Sep 4, 2016, 04:16 cleverca22<EMAIL_ADDRESS>wrote:

can you pastebin the contents of the manifest.nix?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/18279#issuecomment-244597232,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdNji1fuSOGxU0klDbVKE4e5i3BbkoSks5qmqhrgaJpZM4J0emV
.

ah, then your only option is nix-env --rollback to switch to the previous version

It says no environment older than the current one exists... :(

To anyone this happens to (corrupt profile + garbage-collected other profiles), you can delete your corrupt profile. First you might want to remember what was installed:
nix-store -q --references ~/.nix-profile

Then wipe the corrupted profiles out:
sudo rm /nix/var/nix/profiles/per-user/yourname/profile
sudo rm /nix/var/nix/profiles/per-user/yourname/profile-*-link

Then start again from scratch.
Thanks to clever and vcunat for helping me!

This was helpful in diagnosing my issue:
# any nix-env command, including the one ran by home-manager
error: syntax error, unexpected $end, at /nix/store/ry2b0qwvhdvdlmrrkawj9c6xiy9d6krs-env-manifest.nix:1:1

For home-manager users, see what is populating your profile:
nix-store -q --references ~/.nix-profile

look for home-manager-path, mine was:
/nix/store/ghj6r1157710pdw113rmfg1nj46f0f7s-home-manager-path

To fix the broken manifest:
$ sudo rm -r /nix/var/nix/profiles/per-user/user/profile-*-link

then I could run home-manager again (using the path from earlier):
/nix/store/ghj6r1157710pdw113rmfg1nj46f0f7s-home-manager-path/bin/home-manager switch

Everything works as intended now

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/home-manager-stopped-working-on-my-non-nixos/6023/15

Sorry to comment on a closed issue, but I ran into the same manifest.nix + garbage collecting problem after a crash mid-rebuild and wanted to add my solution here for posterity in case someone else runs into it.
I fixed it by building directly from the nixpkgs tarball with -I:
$ sudo nixos-rebuild switch -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/20.09.tar.gz

Then you can sudo nix-channel --update again and nixos-rebuild as normal

sudo nixos-rebuild switch -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/20.09.tar.gz

Didn't work for me

Somehow i ran into this issue this morning. For me it was essential to clear
~/.local/state/nix/profiles

then run
sudo nix-collect-garbage -d

then reinstall my home-manager profile as usual. Note that this is dangerous and could eat your dog

I encountered a corrupt manifest.nix for both users. I don't use nix profile myself, the profiles were created by home-manager. Given that I include home-manager in my configuration.nix, the broken manifest.json also broke nixos-rebuild switch. What I did was to remove /nix/var/nix/profiles/per-user/<user>/{profile,profile-xyz-link} and create a new symlink to profile-abc-link where abc = xyz - 1.

I found that instead of changing the symlink, I could have used nix-env rollback, and instead of removing the broken profile I could have used nix-env --delete-generations. That's probably safer than to fiddle directly with the files.

Somehow i ran into this issue this morning. For me it was essential to clear
~/.local/state/nix/profiles

then run
sudo nix-collect-garbage -d

then reinstall my home-manager profile as usual. Note that this is dangerous and could eat your dog

life saver! I was starting to panic.


Iroha api reference link updated

Description of the Change
Iroha api reference link updated

Benefits
navigation to correct page

Possible Drawbacks
none

Usage Examples or Tests [optional]

Alternate Designs [optional]

Thanks for the pull request! It is probably safe to say the the documentation was migrated to ReadTheDocs, and this note can be removed. Could you update the pull request, and sign-off the commits?


Docs: Fix Selections templating example

This pull request includes a

[x] Bug fix
[ ] New feature
[ ] Translation

The following changes were made
The docs page for selections currently does not render the flags as intended in the example. This makes the code the same as is used on the dropdown page, which does work

I now realize this was a misunderstanding of the difference between templateSelection & templateResult.


User-defined data types (and records) not comparable

If I define:
data Day = Sun | Mon | Tue | Wed | Thu | Fri | Sat

I cannot compare Day values with < or use Day as a key in a Dict. This really discourages me from defining (and using) more precise types in my programs. I could use a String instead of Day, but then I lose the typechecker's help when I accidentally do something nonsensical that makes sense for arbitrary strings, but not for days. :(
This would be fine if there were a way to provide a custom comparable implementation for a data type, but there's no way to do that, either. This comes up a lot. It's very common to want to use some ADT as a key in a Dict, or store some ADT in a Set.

Closing in favor of #1008 which will track any issue along these lines and see the different use cases and discussions.

We've run into this a lot with validation errors.
Specifically we want to enumerate all the fields on a form as a union type (e.g. type Field = Username | Password | Email), and then to represent validation errors as a Dict Field String so we can easily look up whether there is an error on a given field with things like Dict.get.
We work around this by using List (Field, String) as a fake Dict and then filter it a lot. You can see a public example of where we're doing this here: http://package.elm-lang.org/packages/NoRedInk/elm-rails/1.1.0/Rails-Decode - we'd definitely prefer if that function had the following signature:
errors : Dict String comparable -> Decoder (Dict comparable (List String))

Right now we can't do that, because if we did we couldn't use union types for our fields.

:+1: running into this one with some game dev stuff. The inability to express mapping from Union Types as the Model is quite frustrating.

couldn't this be solved using something like
module Days exposing (Day, compare)

data Day = Mon | Tue | Wed | Thu | Fri | Sat | Sun

toInt = Day -> Int
toInt day =
    case day of
        Mon -> 1
        Tue -> 2
        Wed -> 3
        Thu -> 4
        Fri -> 5
        Sat -> 6
        Sun -> 7

compare = Day -> Day -> Bool
compare day1 day2 = (toInt day) > (toInt day2)

couldn't this be solved using something like

Depends on what you mean by "this". For example, one of the things mentioned in the opening post is the desire to be able to use Day as the key type in a Dict. Your proposal helps none with that.

@yogsototh sry, i thought Dict part is self explanatory, to use it in Dict i would do, eg:
week = Dict.fromList[(toInt(Mon), [ Event "meeting1" ], ...)
modayEvents = week.get(toInt(Mon))

you might even create a WeekDict which would hide all the repetative toInt parts

In that plain form, what about the type correctness that the original poster wanted here? How is your solution better than one using Strings? How do you prevent someone from accidentally querying entry 8 in that dict? A Dict Day Whatever would have type guarantees that neither the String nor the toInt approach has.
About a less plain approach, somehow creating a WeekDict that hides toInt calls: How are you going to apply all the wonderful functions from http://package.elm-lang.org/packages/elm-lang/core/latest/Dict and http://package.elm-lang.org/packages/elm-community/dict-extra/latest/Dict-Extra on your WeekDict encapsulation?

@jvoigtlaender thank you for explaining it to me, i'm new to elm. i see now what do you mean by type guarantee.

About a less plain approach, somehow creating a WeekDict that hides toInt calls: How are you going to apply all the wonderful functions from http://package.elm-lang.org/packages/elm-lang/core/latest/Dict and http://package.elm-lang.org/packages/elm-community/dict-extra/latest/Dict-Extra on your WeekDict encapsulation?

Which can be solved once Elm gets HKT's.  Or those libraries decide to invert function control (which I doubt they would do, nor should they really do).

Relevant to this discussion: https://github.com/eeue56/elm-all-dict

@rtfeldman Ooo interesting find, though it might be better to make toString wrappers in your own code?  I am curious if there is a Dict that can store any record especially though.  I'd love to be able to store dynamic TEA component models like that in a library I've made...

A type-safe but cumbersome work-around could be to create custom Dict type for Day keys, by wrapping a Dict Int a. E.g.
type DayDict a = DayDict (Dict Int a)

insert : Day -> a -> DayDict a -> DayDict a
insert day a (DayDict dict) = DayDict (Dict.insert (dayToInt day) a)

Running into this.  If tuples are comparable, shouldn't records (which are also an enumerable list of fields) and ADTs (which are are an enumerable list of constructors, each with an enumerable list of fields) also be comparable?
I also just had to downgrade my intended Sets and Dicts to Lists and Lists of pairs, and was unconsciously moving towards @rtfeldman's approach:

We work around this by using List (Field, String) as a fake Dict and then filter it a lot.

but I feel like this will get out of control quickly.
+1 to comparable ADTs, and why not records as well.

Sketch-n-Sketch would benefit from allowing arbitrary ADTs as set members and dictionary keys; also allowing arbitrary ADTs to be comparable would be beneficial as well.
A proposal. Perhaps decomposing comparability into two classes is clearest:
(1) For enumerated ADTs that have a natural ordering, such as the days of the week example above, allow the ADT to be annotated as comparable:
comparable type Day = Sun | Mon | Tue | Wed | Thu | Fri | Sat

In practice, this would be equivalent to Haskell's deriving (Ord), and would allow the user to use < > >= <= on instances of their ADT.
(Ideally, a custom comparison function could be specified, but that is a separate feature.)
(2) It may not make sense to allow the use of < > >= <= on all ADTs. Some ADTs do not have a natural ordering. However, even these ADTs should be usable as set members or dictionary keys.
Requiring "comparability" to be a set member/dictionary key is an implementation detail—if sets were based on hashmaps then they would require "hashability". Because "comparability" is an implementation detail, keep it hidden.
Allow all ADTs to be internally comparable using special methods (e.g. internal_less_than, internal_greater_than) and silently use those methods to implement set membership and dictionary keying.
A user can thus use any ADT in a set or dictionary, but cannot call < > >= <= on their ADT unless they explicitly mark it comparable.

I'm generating an elm package for the AWS SDK (from the apis/*.normal.json files). Some of the types are maps from enums to other things. For example, from
"PurchasedDevicesMap":{
  "type":"map",
  "key":{"shape":"DevicePlatform"},
  "value":{"shape":"Integer"}
},
"DevicePlatform":{
  "type":"string",
  "enum":[
    "ANDROID",
    "IOS"
  ]
},

I would like to represent DevicePlatform as
type DevicePlatform
    = ANDROID
    | IOS

but currently I can't because then PurchasedDevicesMap becomes
Dict DevicePlatform Int

(1) For enumerated ADTs that have a natural ordering, such as the days of the week example above

I think the ability to have ordering on sum types in Elm would be very convenient (and having a way to opt-in would be neat) but i want to note that DoW is a terrible example for it as the ordering is highly locale-dependent, as those paying attention may have noticed from @garbas 's comment:

although the abrahamic tradition starts the week on Sunday, that only remains in the Americas, and a few other specific countries (not necessarily of abrahamic tradition): Japan, India, Israel, South Africa, Taiwan

Most of the world (Europe, Oceania, most of Asia) starts the week on Monday, that's also the ISO-8601 standard

And the Islamic world starts the week on Saturday

So weekdays are mostly an argument towards not ordering sum types.

Some ADTs do not have a natural ordering. However, even these ADTs should be usable as set members or dictionary keys.

To the extent that they are either "empty" or containing suitable items themselves, in fact that's one issue with a comparable annotation (aka deriving(Ord)), it would/will need to handle sum types which can not be made comparable.

Because "comparability" is an implementation detail, keep it hidden.

It's not an implementation detail, it's necessarily part of the API since it's a contract the map key must fulfill for the collection to work properly. Just as hashability (and equatability, and that the two are coherent) is part of the contract for hashmaps.
Adding a "secret" implicit comparability for hashmap would be unlikely to make things clearer, as you'd now get sum types which can and can't be used as map keys without any explanation why e.g. sum types where any constructor has collection or record associated data. Requiring an explicit annotation and only allowing annotated sum types to be used as keys would provide the occasion for the compiler to generate clear error messages in case of incompatibility.


Helidon http server goes down in the scenario being proxy layer for high amount of concurrent requests

The Helidon http server goes down during load test in the scenario of being a proxy layer.
Environment Details

Helidon Version: Helidon MP
JDK version: 21
OS: MacOS Sonoma 14.3.1, M2

Problem Description
The purpose of test was to compare throughput of Helidon http server for the same scenario I had for tomcat/netty/go in past: https://github.com/filipemunhoz/performance-api-webflux-vs-mvc-vs-golang/pull/2
For that the quick startup project from Helidon examples was used and crafted for the same scenario:

vegetal making requests to Helidon http server endpoint, that endpoint in turn makes a request to same http server on another endpoint that should return collection of fictitious data with predefined delay.

With the parameters of 2000 RPS Helidon http server becomes unresponsive and doesn't restore.
Source: https://github.com/RassulYunussov/Helidon-benchmark

@RassulYunussov I suggest that you switch and use Helidon SE instead of Helidon MP. Helidon MP is based on the Jakarta set of specifications and requires CDI and other libraries that are not ideal to test performance (especially not compared to more "basic" frameworks). You can generate a quickstart for SE as well. Having said that, I will still try to reproduce and investigate the problem you report here.

I've tried this on my local Macbook (older model). I don't actually see the server hanging, but I do see connection resets at higher number of rates in vegeta. Seems to work fine below up to 200 or so. After that, some connections are getting closed and you see client errors,
Get "http://localhost:8084/performance-helidon?delay=100": read tcp <IP_ADDRESS>:60968-><IP_ADDRESS>:8084: read: connection reset by peer

and server exceptions,
Caused by: java.net.SocketException: Connection reset by peer
        at java.base/sun.nio.ch.Net.pollConnect(Native Method)
        at java.base/sun.nio.ch.Net.pollConnectNow(Net.java:682)
        at java.base/sun.nio.ch.NioSocketImpl.connect(NioSocketImpl.java:598)
        at java.base/java.net.Socket.connect(Socket.java:751)
        at java.base/sun.net.NetworkClient.doConnect(NetworkClient.java:178)
        at java.base/sun.net.www.http.HttpClient.openServer(HttpClient.java:531)
        at java.base/sun.net.www.http.HttpClient.openServer(HttpClient.java:636)
        at java.base/sun.net.www.http.HttpClient.<init>(HttpClient.java:280)
        at java.base/sun.net.www.http.HttpClient.New(HttpClient.java:386)
        at java.base/sun.net.www.http.HttpClient.New(HttpClient.java:408)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient(HttpURLConnection.java:1304)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect0(HttpURLConnection.java:1237)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect(HttpURLConnection.java:1123)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.connect(HttpURLConnection.java:1052)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1675)
        at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1599)
        at java.base/java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:531)
        at org.glassfish.jersey.client.internal.HttpUrlConnector._apply(HttpUrlConnector.java:423)
        at org.glassfish.jersey.client.internal.HttpUrlConnector.apply(HttpUrlConnector.java:268)
        ... 62 more

We can dig a bit deeper, but given that we should probably be using Helidon SE, it may not be worth our time. We should also try this on Linux. Again, the server does not hang for me, that could be something specific to the M2 HW.
My suggestion is to switch Helidon SE and try this again, and also to try it on Linux. I'm attaching my (slightly simplified) project that uses Helidon 4.0.6.
Helidon-benchmark.zip

Hi @spericas !
Thank you for your suggestion.
But I believe the issue will repeat with Helidon SE. The main problem there is with HTTP-CLIENT.
It looks like it acquires all of the ports on both ends doesn't release.

You're using MicroProfile Rest Client, which means under the hood Jersey's default client connector based on HttpURLConnection.
Performance testing on macOS makes little sense and isn't worth our time.
Please use Linux and also ideally Helidon WebServer + Helidon WebClient. If you can reproduce the issue we can prioritize accordingly.

@RassulYunussov As Romain says, you should use WebClient instead. Let us know if you need help.

Hi @spericas
Yes, replaced by WebClient, now all works.
Thank you!
I've updated my repo with WebClient.
Does it mean, that it is not recommended to use "MicroProfile Rest Client, which means under the hood Jersey's default client connector based on HttpURLConnection"?


5.1 Feedback:  A single protocol can be uploaded many times

Overview
No response
Current Behavior
A single protocol can be uploaded many times  - the app does not reanalyze the file as one file

Expected Behavior
Is the expected behavior supposed to be like the custom labware upload where there is a modal that says do you want to overwrite the file?
Steps To Reproduce

Upload a protocol
Upload the same protocol multiple times

Operating system
Mac
Robot setup or anything else?
No response

See expected behavior comment
@emilywools - do you have any insight on this?

Expected behavior at the moment
Feature request - not in scope for 5.1 currently @mikecardarelli -to ticket

New ticket https://app.zenhub.com/workspaces/app--ui-5f6248dba845b000139aa767/issues/opentrons/opentrons/10181


When press build image button, server return 504 error occasionally

In GitLab by @sokril on Dec 12, 2017, 17:24

In GitLab by @wknet123 on Nov 8, 2019, 07:43
Fixed with PR #790

In GitLab by @sokril on Nov 8, 2019, 08:16
closed

In GitLab by @sokril on Nov 8, 2019, 08:16
assigned to @wknet123


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #39.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #39

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


test: Add unit test for current default/batch.js

Actually as an experiment I picked one file that looked like testable.
Signed-off-by: Denes Pal<EMAIL_ADDRESS>

config.batch() has been removed.


Update protons to 0.0.1a2

This updates protons to the latest tagged version.

Not sure how that broke the builds. Perhaps something changed on the conda side of things..


Github action

Rehost CI to github action

@dominic-mulligan-arm  I think the GitHub Action is good to be merged. Yet AWS Codebuild has failed, which I assume the instance get killed?


多个geodatebase在fragment中添加，全部都加载成功，可以查询，但有时候会有其中一两个图层在地图上不显示

@yinxh2016  以后请使用这个网站报告bug: https://community.esri.com/
Since we don't have enough info for your problem,  here is only my guess: is it possible that the feature layers are scale-dependent?  Please detail the steps to repo your issue, and include data if possible.


trivy: 0.57.1 -> 0.58.0

Diff: https://github.com/aquasecurity/trivy/compare/refs/tags/v0.57.1...v0.58.0
Changelog: https://github.com/aquasecurity/trivy/releases/tag/v0.58.0

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 361775

x86_64-linux

:white_check_mark: 1 package built:

trivy


[build tool] Target aot_android_asset_bundle failed: PathExistsException

Steps to reproduce

Create a working project.
Add an assets directory in the project root.
Add assets  to the pubspec.yaml file under the project root.
Place files in the assets directory.
Build the project using the flutter build apk command.
Repeat step 5.
If I delete the assets file under the output directory, the expected behavior will not occur.

Expected results
Built with no errors and created an installable APK file
Actual results
FAILURE: Build failed with an exception: OS Error: cannot create the file when it already exists, errno = 183
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
$ flutter build apk

Font asset "MaterialIcons-Regular.otf" was tree-shaken, reducing it from 1645184 to 1652 bytes (99.9% reduction). Tree-shaking can be disabled by providing the --no-tree-shake-icons flag when building your app.
Target aot_android_asset_bundle failed: PathExistsException: Cannot copy file to 'C:\learn\flutter\quick_review\build\app\intermediates\flutter\release\flutter_assets\assets/ok.wav', path = 'C:\learn\flutter\quick_review\assets\ok.wav' (OS Error: 当文件已存在时，无法创建该文件�??
, errno = 183)

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileFlutterBuildRelease'.
> Process 'command 'C:\src\flutter\bin\flutter.bat'' finished with non-zero exit value 1

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 19s
Running Gradle task 'assembleRelease'...                           20.4s
Gradle task assembleRelease failed with exit code 1

Flutter Doctor output
Doctor output
$ flutter doctor -v
[√] Flutter (Channel stable, 3.19.2, on Microsoft Windows [版本 10.0.22621.3155], locale zh-CN)
    • Flutter version 3.19.2 on channel stable at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (7 days ago), 2024-02-27 16:51:22 -0500
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\yezi_\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.5)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.8.34511.84
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] IntelliJ IDEA Community Edition (version 2023.2)
    • IntelliJ at C:\Users\yezi_\AppData\Local\Programs\IntelliJ IDEA Community Edition
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart

[√] VS Code (version 1.87.0)
    • VS Code at C:\Users\yezi_\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.84.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[√] Connected device (5 available)
    • Redmi Note 10 5G (mobile) • t4knnjukskjvf6yx  • android-arm64  • Android 13 (API 33)
    • RK3288 I (mobile)         • <IP_ADDRESS>:5555 • android-arm    • Android 7.1.2 (API 25)
    • Windows (desktop)         • windows           • windows-x64    • Microsoft Windows [版本 10.0.22621.3155]
    • Chrome (web)              • chrome            • web-javascript • Google Chrome 122.0.6261.95
    • Edge (web)                • edge              • web-javascript • Microsoft Edge 122.0.2365.66

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

It could be caused by the asset is not copied to build directory for some reason. Could you check this to see if it helps?

It could be caused by the asset is not copied to build directory for some reason. Could you check this to see if it helps?

I appreciate your assistance. I have verified the access rights of the asset file. It has a read-only attribute. If I delete it, the build will succeed.


Feature Request: Push Values on to a MQTT Queue

Hi,
I'm working on a remote data logger, that logs data from my Govee sensors. I would like have the RPI put the values read at a certain interval on to a MQTT queue.
I'm not a CPP developer, so I can't implement it myself and submit a pull request for a patch.
I did look into it, though, and using the Eclipse Paho Library and a few new parameters this should be a straight forward thing to do...
Parameters
[MQTT]
MQTT_URI=myserver.athome.com
MQTT_PORT=1883
MQTT_Username=myMQTTuser
MQTT_Password=myMQTTpassword
MQTT_TOPIC_PREFIX=goveebttemplogger
MQTT_INTERVAL=300

The routine should then every MQTT_INTERVAL seconds send the latest reading from each device as a JSON payload to the MQTT Topic:   MQTT_TOPIC_PREFIX/<<Mac Address of Device>>  (or use the filename prefix used for the log files, for consistency)
JSON Payload
{
  "temperature": 23.20,
  "humidity": 43.70,
  "battery": 100
}

On the other end we can then use e.g. Home Assistant's MQTT Sensor to process the data
Example Home Assistant configuration.yaml entry
mqtt:
  sensor:
    - name: "Temperature"
      state_topic: "goveebttemplogger/gvh-A4C13876CCXX"
      suggested_display_precision: 1
      unit_of_measurement: "°C"
      value_template: "{{ value_json.temperature }}
    - name: "Humidity"
      state_topic: "goveebttemplogger/gvh-A4C13876CCXX"
      unit_of_measurement: "%"
      value_template: "{{ value_json.humidity }}"
    - name: "Battery"
      state_topic: "goveebttemplogger/gvh-A4C13876CCXX"
      unit_of_measurement: "%"
      value_template: "{{ value_json.battery }}"

(Note: This config is just a basic template, needs to be tuned to the respective HA installation)
Just a thought.....
Best regards,
Yonz

I don't see myself adding this to the core program. One of my design principles has been to limit the dependencies as much as possible.
I believe I'm currently only dependent on c++ std libraries and libbluetooth.
You could probably achieve what you want by monitoring the log directory for updates, then using a stream text editor (sed?) to manipulate the logged data and submit it to MQTT. E.g. "tail -f /var/log/govee/gvh-xxxxxx.txt"
I just thought of the tail -f command as an example of monitoring a text file for updates. I'm sure there are better methods in a language such as Perl that are designed to handle text better than C/C++.

Hi,
I see your point. It does make sense to keep the code clean and simple.
I created a script to do exactly what you suggested, with the addition of dynamically create the HA Devices when the GoveeBTTempLogger encounters a new device (writes to a new log file)
My script is here: Datalogger


Some fields are translated as JSON fields even though they shouldn't be

I have the following setup:
A table base model
class TableBase(BaseModel):
    id: int = Field(default=1, gt=0)
    created_by: str
    created_at: datetime = Field(default_factory=datetime.now)
    last_modified_by: str = Field(default="not_yet_modified")
    last_modified_at: datetime | None = Field(default=None)

and the final tables inheriting from the table base and the respective read model, e.g.:
@db.table(pk="id")
class Language(LanguageRead, TableBase):
    pass

When I call db.init() I get an sqlalchemy ProgrammingError and it shows me the following SQL as the culprit:
CREATE TABLE language (
        id JSON,
        created_by VARCHAR NOT NULL,
        created_at JSON,
        last_modified_by VARCHAR,
        last_modified_at JSON,
        name VARCHAR NOT NULL,
        PRIMARY KEY (id)
)

The issue is probably in PydanticSQLTableGenerator._get_column() but I couldn't figure out why my model fields would be caught by one of the cases specified for JSON there. Any help is appreciated!

One initial observation though: at the moment, there seems to be no way to translate date/datetime fields of any kind to their respective sqlalchemy counterparts, this should be an easy addition.


Apply clippy fixes for Rust 1.83

dev/rust_lint.sh no longer passes for me, maybe because of rustup update. This is first portion of fixes suggested by clippy.

for https://github.com/apache/datafusion/issues/13597

I think we need to #allow[missing_docs] as the clippy now complains on that

:) just added

now that https://github.com/apache/datafusion/pull/13598 is merged, let be rebase, otherwise we no longer test with 1.83

clippy job passed with toolchain pin dropped.

Thanks @findepi @comphead @Dandandan


make python <3.5 compatible

Raised by #1, make python sdk to work in all python 3 version.

Given we are (or at least I am) heavily using type hints for functions (PEP-484) and variables (PEP-526), in the mmif-python package, I think we should also drop support for python <=3.5 in this repo and package as well. Closing this issue.


Added additional checks to make sure localStorage is really supported

Fixes private browsing QUOTA_EXCEEDED_ERR issue in Safari.
Currently Safari (in OSX or iOS) will throw an exception if you try to call the setItem method while in Private Browsing mode.

I see the code has been implemented, possibly outside of this PR. Can this PR be closed then?


Paragraph streamed at once in a non-smooth manner + references view is open and then closes as response is streamed in

Testing #217639
In the following example where I ran the command @workspace /explain, I noticed two things which I found a bit unexpected.

After the 1st and the 2nd point were streamed in, there was a pause and the 3rd point was streamed in directly in a block. This looked a bit sudden. I am wondering if this is something we control and if we can make streaming more smooth? The same happened for the 4th point.
Initially the references view was open and then it closed before streaming in the response. I guess this is maybe intentional so that the references view does not take up too much space in the panel, it just looked at bit sudden too.

https://github.com/microsoft/vscode/assets/61460952/84098707-baa3-4ead-b0f7-92fec709f6d7
The output in the Window is appended here in a word file.
testing.docx

This is probably related to the links and the linkification process. Not sure whether it can be smoothed out without delaying the rendering more but I want to investigate some more.

An easy way to see the real stream is to edit getProgressiveRenderRate in ChatListRenderer.ts and add return 10000

Problem appears to the the getDefinitions call here: https://github.com/microsoft/vscode-copilot/blob/90c4864f48cee454e951051e8f025c9aa4c4e980/src/extension/linkify/vscode-node/inlineCodeSymbolLinkifier.ts#L105
Instead we can likely defer this until the link is clicked. If a definition is not found, we can open the matching symbol instead

Should be fixed upstream


exactEarth sample notebooks

Example AIS notebooks with exactEarth data

@normanb I'll push this to the release branch monday.

@normanb I'll push this to the release branch monday.


Added Ridge Classifier

Added Ridge Classifier and tuned it for better accuracy.

@KrishnenduDakshi2002 is this updated correctly?

@kpratik1929 you've submitted two issues in one PR?

@kpratik1929 I'd suggest you reset your repo to a previous point and then separately add the ridge classifier code open a PR and then edit the R2 score and again open another PR. I'm closing this PR.

@kpratik1929 please update the requested changes.

@anuragc2001 you can check it now whether its correct format?

@kpratik1929 yes thank you. Everything looks fine. Thank you for your contribution through out the project.


Fix Mummy Desc

Double Checked by testing in the sim.

There are more; see https://github.com/Zarel/Pokemon-Showdown/blob/master/sim/pokemon.js#L1273
You also have to update the Gen 6 mod with the previous description. Please put them in alphabetical order too.

Thanks!


Add Llamacpp support

Add Llamacpp support

Hi @FredericoPerimLopes, sorry for the late reply. I'm not convinced of the need for this PR as LlamaCpp supports OpenAI-style API.
Is there value in supporting the "native" llamacpp API?

Ok here's why I think this is necessary.

llama.cpp implements the OpenAI chat api, not the legacy completions api.
llama.cpp can, however, accept a request in the form of the legacy completion api via their /completion endpoint (as opposed to /v1/completions for the OAI models).
The legacy AOI completions api repsonse is of the form response.choices, so you get the first item of the choices array.
The llama.cpp completion response is of the form response.content.

So it seems like the easiest option is to either create a separate llama.cpp backend adapter, or create an additional config option for something like llama.cpp_response = True so that it uses the OpenAI adapter, but gets the completion from response.content instead of response.choices.
The end result of all this is I'm able to make a request to the llama.cpp server but none of the suggestion shows in VSCode.

would be cool imo!

Closing in favor of https://github.com/huggingface/llm-ls/pull/94

@FredericoPerimLopes thank you for your contribution, I added you as co-author on the commit that landed in main


Extraneous files in Spring folder

There seem to be a lot of extraneous files in the Spring folder like Data.swift and Info.plist or LoadingView.xib and I'm not sure what is absolutely necessary for Spring to run while integrated with my project. Could you look into separating the absolutely necessary files for animation vs the demo app? Thanks!

You're right, they're remnants of the chapter 4 files for beginners to get started. I'll put them in a different folder and do a cleanup!
Thanks! Meng
On Sun, Feb 1, 2015 at 2:45 PM, Kevin Xu<EMAIL_ADDRESS>wrote:

There seem to be a lot of extraneous files in the Spring folder like Data.swift and Info.plist or LoadingView.xib and I'm not sure what is absolutely necessary for Spring to run while integrated with my project. Could you look into separating the absolutely necessary files for animation vs the demo app? Thanks!
Reply to this email directly or view it on GitHub:
https://github.com/MengTo/Spring/issues/10

Actually I just took a deeper look into some of the files and they're pretty useful actually! Maybe add some documentation on them :)

Thanks, will definitely give some more thought on the organization of the package and document the extra utilities more. Feel free to contribute. :)


python path for django projects

when working on django projects, I get a lot of "Unable to import" errors when importing stuff betwin apps.
Is it possible to add vim's pwd to the python path  when calling linters?

See issue #324.
Are the errors coming from pylint, or some other program? I would like to automatically detect import paths where possible. pylint might require only running it when files are saved in future in order for it to really work, see issue #333.

the errors are comming from pylint (I don't think than flakle8 is working at all, and do not have mypy)

You may have to configure the executable path for flake8. Which OS are you using, how are you running vim, and where is flake8 installed?
For the pylint errors, some work is needed to get that to really work well. I'd turn it off for now.

i'm on funtoo linux in a virtualenv, the flake8 command is available in the PATH in $HOME/.virtualenvs/myvirtualenv/bin/flake8
I tried with let g:ale_linters = {'python': ['flake8',]}
with default values and
let g:ale_python_flake8_executable = 'python3'
let g:ale_python_flake8_args = '-m flake8'

Nothing else is wrong with pylint, so i'll keep it, it's not that big of a deal, only a slightly annoying

Aha, It's in a virtualenv directory. You'll have to use something like this:
let g:ale_python_flake8_executable = $HOME . '/.virtualenvs/myvirtualenv/bin/flake8'
let g:ale_python_flake8_args = ''

There are some plugins which also set up virtualenv for Vim which might work with the options you have above. I would like to eventually automatically detect virtualenv directories in future, but it can be tricky. There's no uniform naming convention for virtualenv directories, they can be in $HOME or in ignored inside of a git repository. It varies a lot.

This kind of detection behaviour exists for Node installed packages for linters like eslint, because the convention there is that the directory is almost always called node_modules and inside a project directory, so it's much easier to detect.

On the subject of detecting virtualenv directories, I suppose I could make the path detection configurable. So I could use some defaults, and then let users set some directories to try in ancestor paths, or just globally, etc.

But why is pylint working and not flake? they are both living in my virtualenv, and not installed in regular system wide python path

It could not be running for any number of reasons. I'd try playing with the executable paths a bit, including using -m flake8 with a Python path inside of a virtualenv directory.

still not working with python3 -m nor flake8 with full absolute path

Hmm, could you copy and paste your settings, and also the output of a successful flake8 run which shows some errors or warnings from the command line? There might be something wrong with either your executable path, or with the output. Also, could you paste the output of flake8 --version?

(venv)$ flake8 sports/models.py
sports/models.py:11:80: E501 line too long (80 > 79 characters)
sports/models.py:30:80: E501 line too long (80 > 79 characters)
sports/models.py:33:80: E501 line too long (84 > 79 characters)
sports/models.py:34:80: E501 line too long (84 > 79 characters)

(venv)$ flake8 --version
2.6.2 (pycodestyle: 2.0.0, naming: 0.4.1, mccabe: 0.5.0, pyflakes: 1.2.3) CPython 3.5.2 on Linux

my config is not really interersting, just plain default values ale-wise, installed with vundle
set nocompatible              " be iMproved, required
filetype off                  " required

" set the runtime path to include Vundle and initialize
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()
" alternatively, pass a path where Vundle should install plugins
"call vundle#begin('~/some/path/here')

" let Vundle manage Vundle, required
Plugin 'VundleVim/Vundle.vim'

" The following are examples of different formats supported.
" Keep Plugin commands between vundle#begin/end.
" plugin on GitHub repo
Plugin 'tpope/vim-fugitive'
Plugin 'tpope/vim-repeat'
" plugin from http://vim-scripts.org/vim/scripts.html
Plugin 'L9'

" Linters
"Plugin 'scrooloose/syntastic.git'
Plugin 'w0rp/ale'

Plugin 'scrooloose/nerdtree.git'
Plugin 'scrooloose/nerdcommenter.git'
Plugin 'ervandew/supertab.git'
Plugin 'altercation/vim-colors-solarized.git'
Plugin 'tpope/vim-surround.git'
Plugin 'kien/ctrlp.vim.git'
Plugin 'majutsushi/tagbar.git'
"Plugin 'davidhalter/jedi-vim'
Plugin 'jmcantrell/vim-virtualenv'
" TODO: find sensible bindings before use
Plugin 'terryma/vim-multiple-cursors'

Plugin 'LaTeX-Suite-aka-Vim-LaTeX'
Plugin 'matchit.zip'

Plugin 'SearchComplete'
Plugin 'delimitMate.vim'
Plugin 'closetag.vim'

Plugin 'vim-airline'

"Plugin 'javacomplete'
"Plugin 'JavaRun'
"Plugin 'Rip-Rip/clang_complete'
"Plugin 'Shougo/neocomplete.vim'
"Plugin 'justmao945/vim-clang.git'

Plugin 'SirVer/ultisnips'
Plugin 'honza/vim-snippets'

" typescript
Plugin 'leafgarland/typescript-vim'
" Plugin 'Quramy/tsuquyomi'
" Plugin 'Shougo/vimproc.vim'
Plugin 'Vimjas/vim-python-pep8-indent'

" Plugin 'MarcWeber/vim-addon-mw-utils'
" Plugin 'tomtom/tlib_vim'
" " Optional:

Plugin 'ConflictMotions'
Plugin 'ingo-library'
Plugin 'CountJump'
Plugin 'mileszs/ack.vim'
Plugin 'airblade/vim-gitgutter'

Plugin 'chrisbra/NrrwRgn'

Plugin 'valloric/youcompleteme'

"TODO
"https://github.com/majutsushi/tagbar

" All of your Plugins must be added before the following line
call vundle#end()            " required
filetype plugin indent on    " required
" To ignore plugin indent changes, instead use:
"filetype plugin on
"
" Brief help
" :PluginList       - lists configured plugins
" :PluginInstall    - installs plugins; append `!` to update or just :PluginUpdate
" :PluginSearch foo - searches for foo; append `!` to refresh local cache
" :PluginClean      - confirms removal of unused plugins; append `!` to auto-approve removal
"
" see :h vundle for more details or wiki for FAQ
" Put your non-Plugin stuff after this line

" make YCM compatible with UltiSnips (using supertab)
let g:ycm_key_list_select_completion = ['<C-n>', '<Down>']
let g:ycm_key_list_previous_completion = ['<C-p>', '<Up>']
let g:SuperTabDefaultCompletionType = '<C-n>'
"
" " better key bindings for UltiSnipsExpandTrigger
let g:UltiSnipsExpandTrigger = "<tab>"
let g:UltiSnipsJumpForwardTrigger = "<tab>"
let g:UltiSnipsJumpBackwardTrigger = "<s-tab>"

"
"let g:jedi#show_call_signatures = "1"
"let g:jedi#use_splits_not_buffers = "left"
"let g:jedi#completions_enabled = 0
"autocmd FileType python setlocal completeopt-=preview

"let g:SuperTabDefaultCompletionType = "context"
" set completeopt=menuone,longest,preview

set background=dark
let g:solarized_termcolors=256
let g:solarized_termtrans=1
let g:solarized_visibility='high'
let g:solarized_contrast='high'
colorscheme solarized

set omnifunc=syntaxcomplete#Complete

let g:airline_powerline_fonts = 1

let g:syntastic_cpp_compiler = 'clang++'
let g:syntastic_cpp_compiler_options = '-std=c++11'

"let g:syntastic_python_checkers=["flake8"]

let g:clang_cpp_options = '-std=c++11'

let g:UltiSnipsJumpForwardTrigger="<c-b>"
let g:UltiSnipsJumpBackwardTrigger="<c-z>"

nnoremap <leader>g :YcmCompleter GoToDefinitionElseDeclaration<CR>

nmap <silent> <C-k> <Plug>(ale_previous_wrap)
nmap <silent> <C-j> <Plug>(ale_next_wrap)

nnoremap <Leader>a :Ack!<Space>

"if has("autocmd")
    "autocmd Filetype java setlocal omnifunc=javacomplete#Complete
"endif

"
" set nocompatible
"
set relativenumber
set number

set cursorline
highlight CursorLine cterm=bold ctermbg=NONE
set showcmd
set showmatch
set ruler
set visualbell
set scrolloff=3

set ts=4 sw=4
set hlsearch
set incsearch
set smartcase
nnoremap <silent> <CR> :nohlsearch<CR><CR>

" set wildmenu

set autoindent
set expandtab
set backspace=indent,eol,start

set splitright
set splitbelow

set encoding=utf-8
set fileencoding=utf-8
set autoread

"set undodir=~/.vim/tmp/undo//     " undo files
"set backupdir=~/.vim/tmp/backup// " backups
"set directory=~/.vim/tmp/swap//   " swap files
"set backup                        " enable backups
set noswapfile                    " It's 2012, Vim.

if has('mouse')
    set mouse=""
endif

"
map Q gq
nmap <F8> TagbarToggle<CR>

nmap <space> i
inoremap <S-CR> <Esc>

inoremap  <Up>     <NOP>
inoremap  <Down>   <NOP>
inoremap  <Left>   <NOP>
inoremap  <Right>  <NOP>
noremap   <Up>     <NOP>
noremap   <Down>   <NOP>
noremap   <Left>   <NOP>
noremap   <Right>  <NOP>

" set history=1000

" set hidden

" if has("autocmd")
"   filetype plugin indent on
"   autocmd FileType text setlocal textwidth=78
"   autocmd BufReadPost *
"     \ if line("'\"") > 1 && line("'\"") <= line("$") |
"     \   exe "normal! g`\"" |
"     \ endif
"
"   augroup END
" else
"   set autoindent
" endif " has("autocmd")
"
" if !exists(":DiffOrig")
"   command DiffOrig vert new | set bt=nofile | r # | 0d_ | diffthis
"             \ | wincmd p | diffthis
" endif
"
"
"
" set textwidth=80
 set columns=80

 augroup vimrc_autocmds
     autocmd BufEnter * highlight OverLength ctermbg=red ctermfg=white guibg=#592929
     autocmd BufEnter * match OverLength /\%80v.\+/
     autocmd BufEnter * set modeline
 augroup END

" set gfn=Consolas

" if &t_Co == 8 && $TERM !~# '^linux'
"     B
"     set t_Co=16
" endif
"

syn on

If you check out the latest code, you can now use :ALEInfo to get a print out of all of the options you have set, and the last few commands that ran for the buffer. You might be able to figure out what is wrong from that. I would recommend turning off other linter plugins, as they might produce some conflicts, but maybe not.
The output you have submitted from flake8 parses just fine. I think there's just something wrong with the executable path.

just to add as a general information, adding this to your vimrc makes you use the flake8 from your virtualenv:
let g:ale_python_flake8_executable = $VIRTUAL_ENV . '/bin/flake8'

The pylint issue has been fixed, and now flake8 will be run from virtualenv paths, when detected.


OSError: [Errno 86] Bad CPU type in executable geckodriver

[WDM] - Driver [/Users/badiead/.wdm/drivers/geckodriver/macos/v0.29.1/geckodriver] found in cache
Traceback (most recent call last):
File "/Users/badiead/Downloads/TradingView_Machine_Learning-master/scripts/takeprofit_linear.py", line 3, in
from functions import click, webdriver, show_me, get, find
File "/Users/badiead/Downloads/TradingView_Machine_Learning-master/functions/click.py", line 6, in
from functions.webdriver import driver
File "/Users/badiead/Downloads/TradingView_Machine_Learning-master/functions/webdriver.py", line 18, in
driver = create_driver()
File "/Users/badiead/Downloads/TradingView_Machine_Learning-master/functions/webdriver.py", line 14, in create_driver
driver = webdriver.Firefox(profile, options=options, executable_path=GeckoDriverManager().install())
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/selenium/webdriver/firefox/webdriver.py", line 164, in init
self.service.start()
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/selenium/webdriver/common/service.py", line 72, in start
self.process = subprocess.Popen(cmd, env=self.env,
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/subprocess.py", line 1823, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
OSError: [Errno 86] Bad CPU type in executable: '/Users/badiead/.wdm/drivers/geckodriver/macos/v0.29.1/geckodriver'

The simple fix to this is to install the latest geckodriver in /Users/badiead/.wdm/drivers/geckodriver/macos/v0.29.1/geckodriver


Big MultiGrid (>1m) scrollbar skips rows

A big MultiGrid (seems to happen starting at 1m rows) - I can see rows 1-9 but cannot get it to show row number 10, either by clicking the scrollbar down icon or with keyboard arrow down or mousewheel. What am I missing?
On Windows, each Browser seems to skip a different number of rows:
Chrome Version 60.0.3112.113 (Official Build) (64-bit) - jumps to row 28
FireFox 54.0.1 (64-Bit) - jumps to row 39
Microsoft Edge 40.15063.0.0 - jumps to row 25
Basically I cannot find a way to see rows 10-24...
PS: my Windows Mouse settings are set to ONE row for the mousewheel, even then it skips multiple rows.
Here is a Plunker sample with 1m rows (my real case scenario actually has over 6m rows where it then skips 120 rows at once :(
Thanks in advance for a great library!!

no ideas or is no one else able to reproduce this?
or is this another "edge case" and I am not really supposed to render >1million rows with the Grid components?
Thanks

I am also able to reproduce this on the Demo site (obviously) in the Grid sample.
I was looking a bit more into this and it happens when 1.5m pixels height are set - after investigating a bit more the sourcecode, it's all related to the ScalingCellSizeAndPositionManager which seems already had a discussion here: #396
Unfortunately it seems so far that ticket got closed without any changes actually being implemented...
Maybe this could have at least been mentioned in the Docs or in the Demo somewhere...
Closing for now as I doubt anything will ever come out of this (and it's way over my programming skills to try and do this by myself).


set aria-hidden for a non-visible control

last time I checked with Brian, he only set the tabindex to -1, that would not stop screen reader from getting to it, adding aria-hidden to fix the issue.
This PR fixes #11546

Pull Request Test Coverage Report for Build 270183910

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 38.083%

Totals

Change from base Build 269776218:
-0.001%

Covered Lines:
19144

Relevant Lines:
45868

💛  - Coveralls


Grid switching, part 3

Implementation of SwitchGridsPipeline pipeline and tasks.
Note: Proper pipeline tests are quite tricky and will be added in the future work.

After you resolve the merge conflicts and decide what to do with the remaining two issues, feel free to merge.


Remove API heavy events from Marathon repository

Remove API heavy events from Marathon repository
Summary:

Removed all the source code that was supporting the heavy events API.
Rewired unit and integration tests to always assume the "light" events API. There is no mention of "light" in the codebase now as it is the (only) default.
Update yamls to reflect plan-format and the default serialization format.

JIRA issues: MARATHON-8458

Changelog entry with a short description of that breaking change would be awesome!

Changelog entry with a short description of that breaking change would be awesome!

I think this is not breaking change per say but I guess we should write to changelog that people cannot use that feature flag anymore as marathon would fail with that feature flag present. Good point!


Support C++20 constant expressions

When C++20 mode is enabled in compiler,
many tests from examples/ will use constexpr for variables.
This requires that library APIs they use are constant
expressions.

Oops, mean to merge against my fork...


Check ci-cd.yml and workflow

Here are the checks on ci-cd.yml. Most of them should be there and for action that currently fails, it requires to install sphinx on the poetry dev.
Overall:

[ ] Workflow in .github/workflows/ci-di.yml is correctly configured so that it does the following while pushes and pulling to the project's deployment branch (e.g. main)
[ ] Runs the test suite
[ ] Style checkers
[ ] Deploy to TestpyPI
[ ] It should be evidenced by a green release button on the package repository's README
[ ] Ensure deployment branch (e.g. main branch) is correctly specified on lines 6, 9, 42
[ ] Ensure correct Python version is listed on line 19 (should match with pyproject.toml)

Hi @lirnish ,
Just checked the latest version of the branch protection bot git repo, they changed the parameter access-token into access_token (https://github.com/benjefferies/branch-protection-bot).
Can you update the ci-cd yaml and try again? thanks!

Thanks @thomassiu ! Let me do a pull request again

It seems like even after the fix the deactivate of the branch protection is not working...

I see one of our classmates also asked the same in Slack. Also Tiff was doing tests too on her repo and with the same result (https://github.com/ttimbers/pycounts_tat). So I'm trying something else too ...

hmm ok, if nothing works, we could maybe just disable branch protection...

seems still not working @thomassiu and getting a same error message

She didn't say but I found she was testing 😆

I made it working!!!
Check my cicd and the actions there...
https://github.com/thomassiu/pycounts/blob/main/.github/workflows/ci-cd.yml

Changes I have made:
In ci:
Added Check style (step 6)
In cd:
Added Step 4 Install poetry
Replace original "Use PSR..." into the one "Bump version and tagging..."
Added Step 8. Build package
Note 1: The step numbers were not fully ordered.
Note 2: I decided to add back the conditions at the top, otherwise there will be a lot of redundant checks
Note 3: in step 10, change back to covizpy when you wanna copy this...

I have also added git push after poetry version in CD step 6 to make sure pyproject.toml version bump has updated to the repo.

@lirnish  and all, tests completed at my repo and you may use the version here...
https://github.com/thomassiu/pycounts/blob/main/.github/workflows/ci-cd.yml

Wow awesome!! Thank you @thomassiu !!! I will update it now

all fixed and closed by #78


Support for UITableView Section indexes

It appears that when trying to add section indexes (the alphabet on the right side of contacts for instance) that it is ignored.. anyone else with some experience on the matter?

@ericlewis UITableView section index titles are controlled via the UITableViewDataSource delegate methods  -sectionIndexTitlesForTableView: and -tableView:sectionForSectionIndexTitle:atIndex:.
Only SSCoreDataSource provides a default implementation for those methods. If you'd like to customize those section indexes or specify your own, you should subclass the data source you are using and override those delegate methods.

So a subclass is the only way to handle it appropriately? I can't just add the delegates to my table view controller and it work? Which I've tried to do in a few ways so far btw. Thx for the help. You need a beer fund :)
—
Sent from Mailbox
On Thu, Jan 15, 2015 at 4:34 PM, Jonathan Hersh<EMAIL_ADDRESS>wrote:

@ericlewis UITableView section index titles are controlled via the UITableViewDataSource delegate methods  -sectionIndexTitlesForTableView: and -tableView:sectionForSectionIndexTitle:atIndex:.
Only SSCoreDataSource provides a default implementation for those methods. If you'd like to customize those section indexes or specify your own, you should subclass the data source you are using and override those delegate methods.
Reply to this email directly or view it on GitHub:
https://github.com/splinesoft/SSDataSources/issues/41#issuecomment-70189673

If you are using SSDataSources, then your UITableView's data source will be a subclass of SSBaseDataSource. You cannot add section indexes to your table by adding delegate methods to your controller because your controller is not your table's data source.
It might be neat to add some block handler properties to SSDataSources so you could specify some section indexes without having to subclass, but until then, subclassing is the way to go.


InternalIdStrategy vs elementId in Neo4J 5

Expected Behavior
Unsure.
Current Behavior
@Id @GeneratedValue long id will use Node ID.
Context
The recommended version for Neo4J OGM 4.x is Neo4J 5.x. When trying to update Neo4J to 5.2 I get deprecation warnings on a test that verifies that the saved entity's id is the Node ID via that("id", node.getId(), equalTo(entity.id)) (where org.neo4j.graphdb.Node node and entity is an OGM mapped entity). The deprecation is on Node.getId() and the recommended action is to use Node.elementId(). But this new one is a string, so they cannot be compared. This also means that the default GeneratedValue.strategy is using a deprecated feature, but itself is not deprecated. Are there any plans to migrate away from InternalIdStrategy?
References:

Neo4J 5 id(): https://neo4j.com/docs/cypher-manual/current/functions/scalar/#functions-id
Neo4J 5 elementId(): https://neo4j.com/docs/cypher-manual/current/functions/scalar/#functions-element-id
Neo4J 5 upgrade notes: https://neo4j.com/docs/cypher-manual/current/#_documentation_updates_for_neo4j_5 (2nd bullet)

Good morning Rob,
thanks for raising this and as you might have guessed, we are very aware of that and have been actively discussing this topic with both the driver and the kernel team. This is something that effects us both at https://github.com/neo4j-contrib/cypher-dsl and more dire at https://github.com/spring-projects/spring-data-neo4j.
Some background: Originally elementId should have been a more stable and reliable replacement for id, available throughout Neo4j 5. The original idea was this:
elementId would return the same value as id, but as String when using the well known store format. There is a new store format in the making for some time now and elementId is needed for id and would have had different content then.
That would have had allowed for an interim solution using elementId on new databases and us internally converting back to long and allow a grace period.
As it is now, this is not the case, not only the format but also the content differs all the time.
OGM and SDN6+ are very much affected by this as we really heavily in both frameworks on some internal, stable identifier. Right now, we didn't find a solution to make this work reliable for Neo4j 4 and 5 together (things we tried among other have been toString(id(n)), but the performance penalty alone was enough to stop us.
What to make of this:

Both this project (Neo4j-OGM) and SDN6 without OGM require the precedes of id() and will use it. If you check for deprecations in the neo4j query log, then you'll see warnings with neo4j 5. There is no way around this.
Right in time before Neo4j 6 will will solve this and provide new versions of SDN6 and most likely of Neo4j-OGM too that will work than only on Neo4j 5 and higher
For your code our recommendation is to migrate away from @Id @GeneratedValue long towards @Id @GeneratedValue UUID. In both Neo4j-OGM and SDN6 there are generators for that. SDN6 will automatically detect UUID fields.

I hope this helps. Please ask, if not and also feel free to raise issues with http://github.com/neo4j/neo4j itself if the id/ elementId change turns out to be problematic.

Thanks for the background, sounds like a big challenge. No questions right now.
Other than maybe consider adding an ElementIdStrategy?
@Id
@GeneratedValue(strategy=ElementIdStrategy.class)
String id;

Also a quick note based on some very minimal observations:
it looked like elementId.substringAfterLast(':').toLong() == id

Also a quick note based on some very minimal observations:
it looked like elementId().substringAfterLast(':').toLong() == id()

Don't take this for granted right now.
We also cannot provide a generator strategy right now because Neo4j-OGM does not know how to query those.
As Michael said:

Right in time before Neo4j 6 will will solve this and provide new versions of SDN6 and most likely of Neo4j-OGM too that will work than only on Neo4j 5 and higher

For now, I will close this issue because the general point was discussed, right?
If not, please feel free to re-open this issue again.


Avoid components list to scroll to top whenever user choose another component

Is your feature request related to a problem? Please describe.
Whenever I go for components at the end of sidebar (i mean the components list), it will always be scrolled to top when navigating to a new component page
Describe the solution you'd like
Keep the components list while users are looking for components

Have raised a PR for this issue please check @Asfak00

Have raised a PR for this issue please check @Asfak00

Done. provide you email we will reach out you

Have raised a PR for this issue please check @Asfak00

Done. provide your email we will reach out you

<EMAIL_ADDRESS>@Asfak00


upgrade postgres.py from 2.1.2 to 2.2.0

So, where is the postgres.py changelog to know what to test here?

Also it looks like downgrade.

Also it looks like downgrade in issue title.

Sorry, fixed. :-)

So, where is the postgres.py changelog to know what to test here?

I almost wrote one for you, but didn't. Sorry. :-(
You'll have to refer to the commit log for now.

@techtonik Here are the two significant changes between 2.1.2 and 2.2.0:
https://github.com/gratipay/postgres.py/issues/26
https://github.com/gratipay/postgres.py/commit/55c69c09339da2bece6baedbcf5aa2d64d87db90

No issues raised, and this is marked as Review - going to merge.

Thanks @rohitpaulk! :-)

Oops! Found a bug, opening a PR..

https://github.com/gratipay/postgres.py/pull/53

cc: @whit537 - ^^ Production is going to break without that.

Roger that.

Blech. I mistook https://github.com/gratipay/postgres.py/pull/53 for a gratipay.com PR. :-(
https://github.com/gratipay/gratipay.com/pull/3807#issuecomment-147012273

Rolled back ...


telegraf: make mdstat monitoring option

It makes telegraf otherwise errors if /proc/mdstat is not available.

cc @zowoq (we might need to explicitly enable this in infra)

bors merge


Usage

You say it is possible to use it with MLC Chat, but I think it needs a config file that is missing ! mlc-chat-config.json ...
Here is the screenshot :

Similar error on iOS as well.

Same error on ios. It's a 4.4GB or so file.
Configure in the iOS app, but early on the llama 7B didn't work for MLC either.
I've seen in their official MLC LLAMA 2, https://huggingface.co/mlc-ai/mlc-chat-Llama-2-7b-chat-hf-q3f16_1
they sharded the model into 100 different shards. I would encourage doing that, maybe 50 as it may have more success.
Do that, and check the mobile config, it's likely not configed for the app yet.

Same problem here.

Same problem. I also tried loading on text-generation-webui and gave same error.

Please fix asap

no config file


rpc tracing compare testing support

Describe the feature
taking this
Additional context
No response

@mattsse i need to write test block in trace.rs right ?

But also which are the two rpc that i need to compare ?

ideally anything.
I think we want a helper type that takes two clients and then some testing support that allows us to compare responses, for example given an input for
https://github.com/paradigmxyz/reth/blob/916b6b3b286e06534fbe1de30ba699b2b40264c5/crates/rpc/rpc-testing-util/src/trace.rs#L107-L114
this will tests that both streams yield the same results etc.

Closing #5148


Resize Window

It should be possible to resize the window of the Emoji Keyboard. That would be fine to set the window on free space on the desktop, not a fixed size and necessary to switch to the window.
Maybe that's an idea for a feature version.
Thanks in advance.

@bueltge Window is resizable in the new version.


new: Support Bazel Remote APIs.

Will start with caching.

Hey,
I have seen that this feature was not introduced into the 1.29 release.
What are the missing pieces, despite the merge conflicts.
Maybe I can support here.

@harlequin The requests from moon to bazel-remote fail with cryptic http2 errors that I've been unable to figure out.

As far as I remember HTTP2 implies that you have SSL enabled. Will try to setup an environment for testing.

Yeah I think the problem has to do with the bazel/tonic libraries being http2, while bazel-remote is http1, and the communication between the 2 just isn't working.

I have played around with the remote service ...
What I have found so far
1. TLS has to be activated and the endpoint has to include  .assume_http2(true)
https://docs.rs/tonic/latest/tonic/transport/channel/struct.ClientTlsConfig.html#method.assume_http2
endpoint = endpoint
            .tls_config(
                ClientTlsConfig::new()
                    .with_enabled_roots()
                    .assume_http2(true)
            )
            .unwrap();

With this setting I can connect to the remote service
2. GZIP, ZSTD compression
So with the enabled connection moon tries to upload the cache into the remote service
bazel-remote-service shows
2024/10/15 07:37:48 POST 400 <IP_ADDRESS> /build.bazel.remote.execution.v2.ContentAddressableStorage/BatchUpdateBlobs
But on moon side I am receiving
[crates\remote\src\cache_api.rs:96:13] &error = Status {
    code: Internal,
    message: "protocol error: received message with invalid compression flag: 114 (valid flags are 0 and 1) while receiving response with status: 400 Bad Request",
    metadata: MetadataMap {
        headers: {
            "content-type": "text/plain; charset=utf-8",
            "x-content-type-options": "nosniff",
            "date": "Tue, 15 Oct 2024 07:37:48 GMT",
            "content-length": "126",
            "set-cookie": "4afd39e673bbfe62f6f7f4a9622d53cd=9c659af26f02f4f78a396f6570abab67; path=/; HttpOnly; Secure; SameSite=None",
        },
    },
    source: None,
}
[crates\remote\src\cache_api.rs:97:13] error.code() = Internal
[crates\remote\src\cache_api.rs:98:13] error.metadata() = MetadataMap {
    headers: {
        "content-type": "text/plain; charset=utf-8",
        "x-content-type-options": "nosniff",
        "date": "Tue, 15 Oct 2024 07:37:48 GMT",
        "content-length": "126",
        "set-cookie": "4afd39e673bbfe62f6f7f4a9622d53cd=9c659af26f02f4f78a396f6570abab67; path=/; HttpOnly; Secure; SameSite=None",
    },
}
[crates\remote\src\cache_api.rs:99:13] error.message() = "protocol error: received message with invalid compression flag: 114 (valid flags are 0 and 1) while receiving response with status: 400 Bad Request"
Error:   × Main thread panicked.
  ├─▶ at C:\Users\rustmec\code\contributions\moon\crates\remote\src\cache_api.rs:101:13
  ╰─▶ Status { code: Internal, message: "protocol error: received message with invalid compression flag: 114 (valid flags are 0 and 1) while receiving response with status: 400 Bad Request", metadata: MetadataMap { headers: {"content-type": "text/plain; charset=utf-8", "x-content-type-
      options": "nosniff", "date": "Tue, 15 Oct 2024 07:37:48 GMT", "content-length": "126", "set-cookie": "4afd39e673bbfe62f6f7f4a9622d53cd=9c659af26f02f4f78a396f6570abab67; path=/; HttpOnly; Secure; SameSite=None"} }, source: None }
  help: set the `RUST_BACKTRACE=1` environment variable to display a backtrace.

So what I have seen bazel-remote supports uncompressed and zstd for communication.
In moon is only gzip available as config.
My best guess by today, that something has to change additional to the compression ...
Hope this helps

Awesome work. I wasn't even aware that the bazel APIs had 2 separate protocols. But that definitely makes sense for the issues I've been having.
I also agree with splitting this up into 2 separate features. I'm going to try and focus on this for 1.30. It would be nice to land this and python support in the next release.


Better handle long tagId in the challenge preview card

The issue is reported by @vpchung in PR #85.
If the challenge has many tags or if its tag has super long length, it will cover the platform icon in the card, like pic below:

The longest tagId we have now is 24 chars and  the tag should not be longer than 60 according to current tagId schema.
Potential solutions:

set shorter length of tagId in schema
make the card layout responsive to long/many tagIds (e.g. vertically align)
both 1 & 2

Moving this ticket to the icebox until we decide to prepare a production version.


Syntax highlighting breaks when text is on the same line

Broken:

Not broken:

Thanks for the report. This should be fixed in 0.22.1.


QC: T2-star

There's two outliers in the GM T2 star values. We should look into if there's a reason we missed as to why we might want to omit them from the results (they're up the intrasubject COVs).
The datapoints are:

sub02-ses010
sub05-ses005

sub-02_ses-005 (excessive motion):

something fishy about sub-05_ses-002's input image:

Great, thanks!
sub02-ses010 looked normal for you when you QC'd?


[High Priority] Add resolve service with information API in the discovery client

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Added resolve_service_with_information API in the discovery client.
Added unit tests for the resolve_service_with_information API.

Why should this Pull Request be merged?

The Measurement Plug-In Client generator lacks backward compatibility. (Context - #946)
This implementation contains the addition of resolve_service_with_information API, which will be used by the client generator to fetch the version of the measurement during client generation.

What testing has been done?

Existing and new automated tests passed.

@Jotheeswaran-Nandagopal Please cherry-pick this PR into releases/2.1.


Allow user-level components to specify @phantom ports

@phantom ports are currently used to model the initiation intervals of modules that don't have an explicit go signal. For example, a combinational component has a @phantom @interface[G, G+1] go: 1 port to model the fact that they cannot be used multiple times in the same cycle.
The idea here is to allow user-level components to specify @phantom ports iff the lowered component will not instantiate a component. This is possible only when:

All sub-components have only @phantom go ports, and
The component does not have any resource sharing

In these cases, we do not need to instantiate an FSM and can additionally make the go port corresponding to the event a @phantom.

Fixed by https://github.com/cucapra/filament/commit/f72e5ec432a783aad6ea0a52b67a7c9c60f9a855


fix: package.json styles export

:tada: This issue has been resolved in version @rubin-epo/epo-react-lib-v2.1.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @rubin-epo/epo-react-lib-v2.1.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add "sampleWithoutDebugFlag" option for tracer to decide set/not set debug flag when a span is forced to be sampled

Which problem is this PR solving?

Resolves #422

Short description of the changes

Add "sampleWithoutDebugFlag" option when initialize tracer
Update setSamplingPriority logic to set debug flag accordingly
Avoid race condition update set firehose flag

@yurishkuro @vprithvi


Vim undo and redo shortcuts not working

Same issue as #4581 but it still isn't resolved and I can't reopen or comment on the old ticket.
Environment data

VS Code version: 1.61.0
Jupyter Extension version (available under the Extensions sidebar): v2021.9.1101343141
Python Extension version (available under the Extensions sidebar): v2021.10.1365161279
Vim (vscodevim) Extension version: v1.21.10
OS (Windows | Mac | Linux distro) and version: MacOS Big Sur 11.5.2
Python and/or Anaconda version: 3.7.10
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pyenv
Jupyter server running: N/A

Expected behaviour
If you make some changes in a cell, e.g. dd to delete a line then normally in vim you can press u to undo the change and ctrl-R to redo the change.
Actual behaviour
But this doesn't work and you have to use the standard Cmd-z vscode shortcut to undo and Shift-Cmd-Z to redo.
Steps to reproduce:
Open a jupyter notebook with vscodevim extension installed. Go into a cell. Make a change. Press Esc. Press u to undo the change.

I actually just found a workaround for this though it would be nice if enabled by default. Following this post I added the following
"vim.normalModeKeyBindings": [
        {
            "before": ["u"],
            "after": [],
            "commands": [
                {
                    "command": "undo",
                    "args": []
                }
            ]
        },
        {
            "before": ["<C-r>"],
            "after": [],
            "commands": [
                {
                    "command": "redo",
                    "args": []
                }
            ]
        }
    ]

to my settings.json (open command palette and search for Open Settings (Json) to edit).

VIM support would be handled by VS code itself. Jupyter extension doesn't do anything special here.


add a Dark Science extension

Hello there!
This one adds a new extension for the webcomic Dark Science. The Dresden Codak website has a few more comics, but i’m only reading Dark Science there. So if demand arises, this could instead turn into a Dresden Codak extension in the future.
Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions

Not a multisrc.

[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")

No open issues.

[x] Added the isNsfw = true flag in build.gradle when appropriate

SFW

[x] Have not changed source names
[x] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio

Ran the extension in private for a long time, so it’s battle-tested.

[x] Have removed web_hi_res_512.png when adding a new extension

Heyo. Some changes happened after i finally learned a bit of how these Observable things work.

fetchChapterList should no longer be blocking.
Switched to pageListParse, like previously with imageUrlParse.

Unfortunately the helper function accumulateChapters has to rely on recursion. Can’t see how to pull an Observable from a suspend fun at the given minimum API level.


add the posibility of specifying a url prefix

this commit implements a prefix for urls, which allows easily deploying
dpaste into a "subdirectory". It adds a url_prefix to be read by default
from the environment which makes it especially suitable for using for
docker containers. For example, running
docker run -e PREFIX_URL=dpaste/ -p 8000:8000 barttc/dpaste
will make dpaste available on
http://localhost:8000/dpaste/
instead of the default http://localhost:8000
This is specially useful if you want to proxy dpaste from another host
and let it live in a sub-url. Without this commit all urls would be
incorrectly generated in this case.

Thank you for your work.
I tested this, it doesn't work

steps that I did
git clone -b "feature/add-url-prefix" https://github.com/jjguti/dpaste.git jjguti

docker build -t dpaste:jjguti .
docker run -e PREFIX_URL=dpaste/ -p 8000:8000 dpaste:jjguti

The URL still only accessible at the root domain

Hi Darren,
I double checked and apparently I mixed up the name of the environment variable, it should be URL_PREFIX and not PREFIX_URL. to run you should use:
docker run -e URL_PREFIX=dpaste/ -p 8000:8000 dpaste:jjguti

Hello jjguti,
I just tested it, it works.
If you can rebase this, I would gladly push it to main.

Hi,
rebased now.


np.linalg.solve unexpectedly returns an F_CONTIGUOUS array for C_CONTIGUOUS inputs

Reporting a bug

[X] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[X] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

Reproducer
import numpy as np
from numba import njit

def f(x, y):
    return np.linalg.solve(x, y)

if __name__ == "__main__":

    x = np.array([[1, 2], [3, 5]], dtype=np.float64)
    y = np.eye(2, dtype=np.float64)

    g = njit()(f)

    numpy_result = f(x, y)
    numba_result = g(x, y)

    print(numpy_result.flags)
    print(numba_result.flags)

This prints the following:
  C_CONTIGUOUS : True
  F_CONTIGUOUS : False
  OWNDATA : True
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

  C_CONTIGUOUS : False
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

I am not sure if this is the expected/desired behaviour. I encountered this when attempting to feed the results of np.linalg.solve into np.dot and saw:
NumbaPerformanceWarning: np.dot() is faster on contiguous arrays, called on (array(float64, 2d, A), array(float64, 1d, C))

Thanks for the report. Given what Numba's implementation of np.linalg.solve does this is not surprising, however it is a bug as it ought to match NumPy.

I actually encountered the same issue, but with np.linalg.qr function. Are there any workarounds for this?


Could you fix your plugin to  be compatible with flutter 3.0?

After I upgrade flutter SDK to 3.0, I can't run my app.
Could you provide a new version to add compatibility with flutter 3.0?
Related: https://github.com/flutter/flutter/issues/103571#issuecomment-1125694094

Fixed in bitsdojo_window: ^0.1.2 - https://github.com/bitsdojo/bitsdojo_window/commit/f320b623db107b03a5976fd88e52ec4fe81fa0c9
Closing this issue. Please re-open if needed.


Refactor Crypto APIs to have a provider scheme

CryptoProvider is a struct which provided functions for performing the various ciphers and hashes Str0m needs. Additionally, it can produce a DtlsIdentity, which in run can be used to create one or more DtlsContext.
CryptoProvider is just function pointers, so it can be cheaply copied as needed. DtlsIdentity holds the "DTLS certificate" instance today, and can then create one or more DTLS Contexts from it. Each DTLS Context manages a DTLS session.
CryptoProvider <- All new thing.
DtlsIndentity <- A trait but the analog was DtlsCertInner
DtlsContext  <- A trait but the analog was DtlsInner
Changes in addition to providing a CryptoProvider in many places:

DtlsContext is used directly (through a Box), vs. the DTLS pass through
RtcConfig can now have a CryptoProviderId set. If OpenSsl is enabled, this defaults to one of the OpenSsl providers.
RtcConfig can still have DtlsCert set, if it is set then the CryptoProviderId matches the cert. You can't set both the cert and provider, they override each other.

@algesten
Martin, here's my first whack at a CryptoProvider. OpenSSL is working, I think wincrypto works as well but since there is no default when wincrypto is used the tests are currently broken. But figured it'd be good for you to see if this was a good approach, or if there is anything I should change.
I can do a cargo test --all-features and things pass, which should be testing openssl w/ sha1 crate.
...
For the tests, I was wondering what your opinion was on injecting the CryptoProviderId into tests that need a cryptoprovider, and executing the test with each provider. Do you have a preferred tool or method to run the same test with an injected value (rstest?, something else?). Ideally we'd enumerate the CryptoProviderId and run the tests with each.

I'm away for the weekend so i haven't been able to process this PR yet. Will get to it next week.

Conceivably the Cert set in RtpConfig and the CryptoProvider set in RtpConfig don't have to match, I think allowing that would be a foot gun.... basically... nothing in the CryptoProvider is stateful, so you should be able to interchange cryptoproviders at any time. The Cert itself creates the DtlsContext so those are guaranteed to be compatible, and the DtlsContext manages the Dtls session, so it's all gauranteed to use the same provider. If SRTP happens to use a different provider, there isn't a reason it wouldn't work.
So maybe instead of setting the cryptoproviderid.. you set a cryptoprovider in RtpConfig. We have our set of 3 standard providers (OpenSSL, OpenSSL w/ SHA1, Wincrypto). In theory people can provide their own provider, or mix and match providers (like OpenSSL w/ SHA1 is essentially OpenSSL with the SHA1 function replaces).
But maybe I'm overthinking this.

For the tests, I was wondering what your opinion was on injecting the CryptoProviderId into tests that need a cryptoprovider, and executing the test with each provider. Do you have a preferred tool or method to run the same test with an injected value (rstest?, something else?). Ideally we'd enumerate the CryptoProviderId and run the tests with each.

I think it would be reasonable if the default could be set / overridden via a build-time cfg like RUSTC_FLAGS=--cfg strom_default_crypto=xyz. That would make it easy to run all tests with different providers.
Also, I just wanted to mention that having no crypto-provider set needs to work as well (full disclosure, I didn't give this a full-review so you may very well cover this anyway). We only use the IceAgent from str0m and disable all other features because we don't want to compile openssl at all.

Thinking about this some more.. maybe we should get rid of the cryptoprovider id... and jsut make cryptoproviders be potentially defined outside of str0m... str0m just defines the Provider struct, and the associated traits.
WinCrypto can then be it's own crate that depends on Str0m .. and when you use RtcConfig you pass in the CryptoProvider that WinCrypto defines.
For the SHA1 case.. we just have a crypto provider with sha1 has defined, and the other crypto functions set to panic/fail. but basically, I think the cryptoproviders can all be defined as exported constants, then passed in. We can leave the SSL, SHA1Only, and SSLwSHA1 in str0m itself since they're already there, and can provide default behavior.
I think other than a cryptoprovider, the only other touch point we have are for those Error Enums,but we can just have a generic CyptoProviderError(String), defined by Str0m, we can still leave OpenSSL with it's own, since it's bulit in, but this would allow more complete flexibility for the crypto provider and potneital future providers.

I would like to understand why we need to make these much changes with traits, dynamic dispatch etc? From my perspective, we already have a structure for abstracting the crypto, and this seems like an abstraction on the abstraction. Can we take a step back and discuss what the PR tries to solve?

I would like to understand why we need to make these much changes with traits, dynamic dispatch etc? From my perspective, we already have a structure for abstracting the crypto, and this seems like an abstraction on the abstraction. Can we take a step back and discuss what the PR tries to solve?

I might need to take a stab at what I'm thinking myself to understand how we end up with the need for CryptoProvider.

Starting to see the problem. Our current abstraction kind of forces us to feature flag on the inside of stuff like DtlsCert.


Istio running on VM doesn't rotate certificates when istio-token expires

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
When running Istio proxy agent on a Virtual Machine (https://istio.io/latest/docs/setup/install/virtual-machine/), certificates are rotating until the istio-token expires and then Istio starts to throw errors.
Version
Virtual Machine:

# pilot-agent version
version.BuildInfo{Version:"1.20.1", GitRevision:"552626bd81f625917bcba6e415cae5aa3d5c61af", GolangVersion:"go1.21.5", BuildStatus:"Clean", GitTag:"1.20.1"}

# Kubernetes:

$ istioctl version
client version: 1.19.0
control plane version: 1.20.2
data plane version: 1.20.1 (1 proxies), 1.20.2 (28 proxies)

$ kubectl version --short
Flag --short has been deprecated, and will be removed in the future. The --short output will become the default.
Client Version: v1.25.10
Kustomize Version: v4.5.7
Server Version: v1.26.7

Additional Information
Using Istio-csr (https://github.com/cert-manager/istio-csr/blob/main/deploy/charts/istio-csr/templates/certificate.yaml) integration with HashiCorp Vault for certificate management. Certificates are being rotated while istio-proxy token is valid (for 1h):
/etc/certs # openssl x509 -in cert-chain.pem -noout -text | grep "Not After"
            Not After : Feb  1 08:18:21 2024 GMT
/etc/certs # openssl x509 -in cert-chain.pem -noout -text | grep "Not After"
            Not After : Feb  1 08:48:24 2024 GMT

After the istio-token expires, certificates are not being rotated anymore and istio service throws these errors:
 warning        envoy config external/envoy/source/extensions/config_subscription/grpc/grpc_stream.h:152        StreamSecrets gRPC config stream to sds-grpc closed: 2, failed to generate secret for default: failed to generate workload certificate: create certificate: rpc error: code = Unauthenticated desc = request authenticate failure

It works when Istio self-signs certificates, but apparently something's up when istio-csr with Vault integration is hooked up.


Change kluster to klaster.

This address https://github.com/jk8s/sig-docs-id/issues/20

/assign

/cc @girikuncoro
/cc @phanama
/cc @danninov

/uncc @davidopp

Address https://github.com/kubernetes/website/issues/15341

/lgtm

@phanama just fix the conflict.

/lgtm

/approve


test-related Gulp tasks refactored to a separate file

Gulpfile.js is growing wildly so we should split it into multiple files - one for each group of related tasks. This is PR extracted all test-related tasks into new module, in next step we could extract all build-related tasks.
@jrichter1, @joshuawilson, @dgolovin can you please review?

released to master

Thanks, I will refactor build-related tasks as soon as all build-related PRs are merged/closed.


Fix global configuration naming for JCasC

Please check if the PR fulfills these requirements

[x] The commit message describes your change
[x] Tests for the changes have been added if possible (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[x] Changes are mentioned in the changelog (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes the naming in the global configuration so that the JCasC is clear.

What is the current behavior? (You can also link to an open issue here)
The current key is pluginImpl which is ambiguous when defining several different plugins in JCasC.

What is the new behavior (if this is a feature change)?
The new behaviour renames the configuration key to be pipelineStepsAWS rather than pluginImpl

Does this PR introduce a breaking change? (What changes might users need to make in their setup due to this PR?)
If JCasC is defined, a rename would be required. pluginImpl --> pipelineStepsAWS

Other information:

Before

After

This looks good. Thanks for the change. Can you please make a note on the changelog that this might be a breaking change?


Updates Molecule workflow for Ubuntu 22.04

This is mostly pipe fitting to get CI/CD tests to pass again on Ubuntu 22.04 runners. Also adds missing PIP dependencies introduced since version Molecule 5.0+.
Further discussion can be found here with regards to permission changes to cgroup volume.

Hello @markagonzales !
Thanks for your contribution.
I am currently working my way through all my public Ansible roles and updating them.
I hadn't managed to update this role yet.
Now I have done this.
For this reason I reject your MR.
Best regards
Felix