Flatten k-d trees.

I read that it's good practice to flatten the binary tree into a flat array structure using a good heuristic. Should look into implementing this. Apparently there is a performance gain!

The new BVH implementation is already a flat array of nodes, so unless you want to also rework the old implementation, I think this issue can be closed.

Yep, effectively resolved by 9010e295e620027d885ebb585abe309816e7c450


Fixed grammatical errors in the reminder emails

Hey :wave:
I noticed there were some inconsistencies and grammatical errors in the reminder emails.
The daily HTML email had:
We've got some suggested projects for you to send pull requests to this today.

and the weekly text email had:
We've got some suggested projects for you to send pull requests to week.

This PR should correct these to read as requests to today and requests to this week respectively.
Hope this is all ok.
Thanks,
.FxN

:+1:

Thanks @efexen :santa:


feat(docs): mark cpp14 as maintenance branch

the main development branch is now cpp17 template

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

ℹ Additional Information

Probably should be merged once C++17 is ready and available


Added support for tracing SQL query through Entity Framework Core

Issue:
Added support for tracing SQL query through Entity Framework Core 3.0 and above. (.NET Core)
Description of changes:
Entity Framework 3.0 starts to include DbCommandInterceptor for interception of database operations.
https://docs.microsoft.com/en-us/ef/core/what-is-new/ef-core-3.0/#interception-of-database-operations
https://docs.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.diagnostics.dbcommandinterceptor?view=efcore-3.0
Integrated DbCommandInterceptor with AWS XRay SDK to enable tracing SQL query through EF Core 3.0 and above.

Extract database_type from CommandEventData.Context.
Subsegment name is correctly collected via DbCommand through EF Core.
CollectSqlQueries option can be enabled locally or globally.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Switched over to #124


format of secret

hi
what is the format of secret? i mean in previous version the secret has this format dd+secret in config file but in this version i can't recognize the format.
how can found the original secret that can connect with it before run the program?
i want load secret from my database for that i want know the format

If you are talking about ee secrets, the format is ee + <original secret> + <hex-encoded fake-tls domain name>.
You can construct ee secret with following python code:
import binascii

original_secret = b"57d843d08a62464d3ce484c573326241"
domain = b"google.com"

print(b"ee" + original_secret + binascii.b2a_hex(domain))

Also, see this webpage http://seriyps.ru/mtpgen.html


Introduce startblock as an lookup option

More info about startblock can be found here

Hey @0xbe1 , thanks for the contribution. Very cool tool! The official Graph docs do not talk about community resources. The best place to market your tool would be https://codex.thegraph.com and https://thegraph.academy


Json-Editors validation-routine as npm package

Hello there. Would it be possible to take the whole validation-routine and create a seperate npm for this? Actually existing validators like tv4 are working too, but not as good as this here. Actually, I have huge problems with every other validator, only this one works for my purposes. An example is shown below:
ExamplePerson.schema.json:
{
"$ref": "http://www.some-site.com/examplePersonAddition.schema.json",
"id": "http://www.some-site.com/examplePerson.schema.json",
"$schema": "http://json-schema.org/draft-04/schema#",
"properties": {
"id": {
"type": "string"
},
"peronName": {
"type": "string"
}
}
}
ExamplePersonaddition.schema.json:
{
"id": "http://www.some-site.com/examplePersonAddition.schema.json",
"$schema": "http://json-schema.org/draft-04/schema#",
"properties": {
"lastName": {
"type": "string"
}
}
}
The problem is that most validators wont allow any additional propertys in the same object as a $ref was set, but the JSON Editor and its validation do.
In this example the ExamplePerson.schema.json would ONLY contain "lastName" as a property and the others are plainly ignored/deleted according to others.
So would it be possible to extract the validation routine and create a node package so that we could use it in a commandline tool?

$ref at top level seems not to be standards compliant
If you restructure the schema a bit it will be compliant to the Json Schema standard:
http://spacetelescope.github.io/understanding-json-schema/structuring.html
See also: https://github.com/json-schema/JSON-Schema-Test-Suite/blob/develop/tests/draft4/allOf.json


🐛 [firebase_ui_auth] localization not implimented for FirebaseAuthException

Bug report
Describe the bug
Firebase FirebaseAuthException caught during submitting forms (sign-in / forgot password / registeration) do not have localization implimented.
Steps to reproduce
Steps to reproduce the behavior:

Follow steps to install firebase_ui_auth to app.
Add SignInScreen widget to app.
Follow steps for adding firebase_ui_localizations to app.
Run app and fill out test email and password for login / test email for registeration / forgot password forms.
'Error' should be visible, however, it will not be localized / translated to target language.

Expected behavior
Error should be translated / localized.
Additional context
Photo added to show no localisation of errors returned from submitting forms:

Flutter doctor
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.3.10, on Microsoft Windows [Version 10.0.22000.1335], locale en-GB)
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.1.0)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.74.2)
[√] Connected device (3 available)
[√] HTTP Host Availability
• No issues found!

 Using steps to replicate, seeing same behavior as reported.

Same here

Ok, so the unknown error code thing is related to this issue #10966, it seems it only happens on web and debug mode.


Marking NavballDockingAlignmentIndicator 1.0.4 compatible

http://forum.kerbalspaceprogram.com/threads/54303?p=1967443&viewfull=1#post1967443

You rock. Thank you so much. :)


View data summary as part of dbt run.

Describe the feature
DBT has proved to be a great tool in organizing our data science project ETLs. However, in order to view the results of our intermediate transformations, or even the resulting transformation after an entire pipeline, data scientists have to log in to snowflake and get a summary of the data, using a LIMIT clause or doing some basic statistics on their features. Further, as a Data Scientist, sometimes we want to experiment with a feature and view the results then and there, pretty frequently, and this requires us to get into the data warehouse UI again and again.
While the current dbt project structure allows us scientists to focus more on code rather then boiler plate create statements, what could be additionally useful is a feature or a flag that allows to display intermediate results from views or tables created as part of the dbt run command.
An additional thought would be to return the intermediate results as a dataframe like structure, allowing us to manipulate or play around with that data after performing a dbt run. As I visualize it, dbt view-data <model_name>
Describe alternatives you've considered

dbt run --view-data True
dbt view-data <model_name>

Additional context
This feature should not be database-specific, however we use dbt with snowflake and it would be great to start off right there.
I understand that this might be difficult to implement when it comes to larger analytics jobs, especially the ones that involve huge data volumes , maybe in GigaBytes. However we were hoping to get around that using some sampling strategy.
Who will this benefit?
Data scientists, Engineers and everyone how makes frequent changes to their dbt models.
Are you interested in contributing this feature?
Yes.

Really cool ideas @akashshah59! There's a lot you're touching on here, I'll try to break some of it down.
Option 1: Show table samples in the terminal while dbt is running
This follows your proposal for dbt run --view-data True.
There's actually a way to do this in dbt today, but it's exclusively limited to dbt seed, using a --show command that's sort of undocumented, and was originally implemented a long time ago (https://github.com/dbt-labs/dbt/pull/618).
$ dbt seed --show
Running with dbt=0.21.0-rc1
Found 1 model, 1 test, 1 snapshot, 0 analyses, 350 macros, 0 operations, 1 seed file, 0 sources, 0 exposures

19:56:50 | Concurrency: 5 threads (target='dev')
19:56:50 |
19:56:50 | 1 of 1 START seed file dbt_jcohen.country_codes...................... [RUN]
19:56:50 | 1 of 1 OK loaded seed file dbt_jcohen.country_codes.................. [INSERT 247 in 0.16s]
19:56:50 |
19:56:50 | Finished running 1 seed in 0.46s.

Random sample of table: dbt_jcohen.country_codes
------------------------------------------------
| name             | two_letter_iso_code | three_letter_iso_... |
| ---------------- | ------------------- | -------------------- |
| Bolivia          | BO                  | BOL                  |
| Swaziland        | SZ                  | SWZ                  |
| Christmas Island | CX                  | CXR                  |
| Uganda           | UG                  | UGA                  |
| Switzerland      | CH                  | CHE                  |
| Andorra          | AD                  | AND                  |
| Montenegro       | ME                  | MNE                  |
| Chile            | CL                  | CHL                  |
| Comoros          | KM                  | COM                  |
| Nauru            | NR                  | NRU                  |
| ...              | ...                 | ...                  |

Completed successfully

Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

In practice, this isn't so useful, but it is pretty cool. There's a related feature request (https://github.com/dbt-labs/dbt/issues/3265) to do this for test failures, which is conceivable in conjunction with the --store-failures feature: the results are already in a table, ready to be sampled. I just managed to hack on this a bit locally, and I left a comment with some details: https://github.com/dbt-labs/dbt/issues/3265#issuecomment-931639169. Again, this doesn't feel tremendously useful, or like it would be a game-changing feature, but it is fun.
Option 2: Use dbt to run interactive queries against your database
This is more like your proposal of a dedicated view-data command. I think this is the real answer—it's the ability to run arbitrary bits of dbt-SQL against your data warehouse, leveraging your existing connection details. The workflow looks a bit like:
$ dbt run -s my_model

Followed by:
$ dbt run-sql "select * from {{ ref('my_model') }} limit 10"

Or, to follow your example, a handy-dandy CLI command that shortcuts the above:
$ dbt sample my_model
$ dbt view-data <model_name>

That's exactly the kind of development workflow that the dbt Server exists to enable. The initial version (dbt rpc) is a bit janky today, so we're actually in the process of reworking it currently. Hoping to share more details soon.

An additional thought would be to return the intermediate results as a dataframe like structure

Agree! The existing Server sort of does this, by storing results as an agate table (dataframe) and then returning them as a JSON object. I think this kind of workflow would be most compelling in a notebook application, where the notebook persists SQL cell results as data frames (available to R/python context), and a dbt Server makes it possible to run dbt-SQL in that notebook, with access to your full project context.
If you're really committed to doing it all from the command line, and avoiding a data warehouse console, there are some cool SQL-from-CLI tools (e.g. whale).


Can't start VLC on Wayland

Describe the bug
Packaged VLC does not seem to support wayland.
Steps To Reproduce
Steps to reproduce the behavior:

Install vlc
Start with QT_QPA_PLATFORM=wayland vlc on e.g. sway
VLC opens as a XWayland window

Expected behavior
VLC opens as a Wayland native window
Screenshots
Not applicable
Additional context
Possibly related to VLC not depending on qtwayland.
Notify maintainers
@AndersonTorres
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.91, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.5.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

EDIT: Typo

It is strange. I built it and verified, it is not linked with QT libraries even if they are put as inputs.

Is qtwayland also pulled in as an input?

Nope. Working on it now.

Can you test the patch?

Works like a charm, thanks a lot! :)

I'm on NixOS 23.05 and discovered that vlc prefers to use XWayland if DISPLAY var is set. If I unset DISPLAY then vlc starts in Wayland mode.
Does anyone know how to make it to start on Wayland by default?

For now I hacked it with:
environment.systemPackages = with pkgs; [
    ...
    (pkgs.symlinkJoin {
      name = "vlc";
      paths = [ pkgs.vlc ];
      buildInputs = [ pkgs.makeWrapper ];
      postBuild = ''
        wrapProgram $out/bin/vlc \
          --unset DISPLAY
      '';
    })
];

Special thanks to https://nixos.wiki/wiki/Nix_Cookbook#Wrapping_packages

Well, if this is hardcoded on VLC, there is nothing to do (besides maybe searching some hidden --force-wayland-and-please-ignore-xwayland-at-all-costs commandline flag).

VLC still opening with xwayland, started with QT_QPA_PLATFORM=wayland vlc
VLC version - 3.0.20
OS - Arch linux 6.6.7

The "use $DISPLAY if set, $QT_QPA_PLATFORM be damned" behavior is apparently just what QT does, and is not specific to VLC

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/wrapping-a-desktop-file-force-vlc-to-use-wayland/49548/1

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/wrapping-a-desktop-file-force-vlc-to-use-wayland/49548/4

Here is the updated wrap that also addresses the desktop file
    # VLC starts on XWayland if DISPLAY is set (which we need to have for legacy apps)
    # Wrap it
    (pkgs.symlinkJoin {
      name = "vlc";
      paths = [ pkgs.vlc ];
      buildInputs = [ pkgs.makeWrapper ];
      postBuild = ''
        wrapProgram $out/bin/vlc \
          --unset DISPLAY
        mv $out/share/applications/vlc.desktop{,.orig}
        substitute $out/share/applications/vlc.desktop{.orig,} \
          --replace-fail Exec=${pkgs.vlc}/bin/vlc Exec=$out/bin/vlc
      '';
    })


[Feature Request] Add Map Tilting

I would love the ability to tilt the map so that we are not looking straight down on the map, one such reason to do this is to more easily examine the building heights. Being able to tilt the map at a bout 45 degrees and then zoom out a bit to visualize building heights would allow us to examine if there are any building height outliers.
I know this is probably a big ask, but it would be very cool.

Duh... double mouse click and drag gets map tilting!!!! yay! Closing

Yeah this is easy to miss- in fact we're working on a 'first-touch' nav experience that will let you click around to various POIs to inspect the data, and that would adjust the tilt automatically depending on which POI you click. For example, if you click on the 'New York City Buildings' POI, it would set the pitch to 60 degrees, really highlighting the 3D-ness.


feat(api): Assume OT_SMOOTHIE_ID is AMA if environmental variable is unset.

overview
All(?) standard OT2 robots set an environmental variable OT_SMOOTHIE_ID to "AMA". This is used to match the comports and identify /dev/ttyAMA0. However if this environmental variable is unset (e.g. on a custom install) the assumed value is FT232R. I do not see what purpose this serves, and it requires an extra step of setting this environmental variable.
This PR changes that default which should allow graceful handling where OT_SMOOTHIE_ID is unset. I'm not clear of any situation where the current default (FT232R) would be useful and I presume it is vestigial but I may be unaware of a situation in for example old robots.
An alternative would be to raise an exception if this variable is unset but the current behaviour seems to be suboptimal as you get the obscure error:
File "/home/pi/bare/lib/python3.6/site-packages/opentrons/drivers/serial_communication.py", line 126, in connect
    port = get_ports_by_name(device_name=device_name)[0]
IndexError: list index out of range

changelog
Changes default pattern match if OT_SMOOTHIE_ID is unset from FT232R to AMA.
Related to #4397.

I believe the historical reason for FT232R is because with the OT-1 and during early development of the OT-2, the API server was frequently running on developers' laptops with a FTDI USB-to-UART cable connected from the computer to the Smoothieboard
I'm pretty sure this is a good change; development config shouldn't leak into the codebase like this, but I'll let someone on @Opentrons/py sound off

Thanks - at the moment having the OT2 server running is not a priority for me because I am in a sense replacing it with my own thing. I will investigate that at some point though and obviously I have stuff to figure out about calibration etc. Had a suggestion re: blinkenlights in https://github.com/Opentrons/opentrons/issues/4397#issuecomment-551233783 . Happy to make the PR for that if you want it (having that one bit of important API-like code in buildroot seems a shame) but I understand that weird use cases won't be a priority.

Thanks @theosanderson! This change will be present in 3.15.0.


[tutorial] The basics of using the Tkinter module

Description of the tutorial
Tutorial on Tkinter
Do you want to be assigned to work on this?
(mark [x])

[ ] YES
[x] NO

Optional: Any additional info you want to add
...

Open for anyone to write a tutorial about

@JatinGera-noob do you still want to work on it ?

Sorry for replying late, actually I am done with only two files, was planning to add more, thus didn't commit them. doing this now


Error "$ is not a function" in Wordpress

jQuery library included with WordPress is set to the noConflict() mode. You must use jQuery instead $.

@etiennetalbot Wordpress now using jQuery v 1.11.3.
This line don't work with jQuery from WP:
return $(element).prop('tagName');

In the noConflict() mode, the global $ shortcut for jQuery is not available

return jQuery(element).prop('tagName'); can resolve problem.

@etiennetalbot, Thanks :+1:


"Environment xxx is not setup for deployments from branch xxx"

We deploy to this environment all the time. We are attempting to deploy from a feature branch here, to do testing prior to merging that branch into main. Is there something we actually have to do to that environment to enable that kind of deployment? Can we in fact deploy feature branches to test Sandbox environments in this way?  The environment is set up with "No Restriction" with regard to which branches are allowed to deploy to it. Full log is below.
logs_22017573766.zip

https://aka.ms/algosettings#deployto
If Branches are not defined in GitHub settings or AL-Go settings - we default to main.
If Branches are defined in GitHub settings and not in AL-Go settings - we use that
If Branches are defined in AL-Go settings and not in GitHub settings - we use that
If Branches are defined in both places - we use both.

Ah yes. I get it. The explanation under DeployTo is crystal clear--now that I know the real source of the problem.
Would it be possible to word the error message so that it implicates Settings, rather than either the environment or the branch? That might eliminate a lot of looking in the wrong places.
In the meantime, thank you very much for the enumeration of the Settings order of priorities. That will go in my collection of AL-Go Post-it notes for quick reference!


[CT-1520] [Bug] dbt-postgres:1.3.1 image throws on startup

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
The latest 1.3.latest image thows the following exception immediately upon startup:
> docker run -it ghcr.io/dbt-labs/dbt-postgres@sha256:e224ca10626e484a0d5dea072f13a910e0d8b34ee4959fb4a7db76892c852ee1
Traceback (most recent call last):
  File "/usr/local/bin/dbt", line 5, in <module>
    from dbt.main import main
  File "/usr/local/lib/python3.11/site-packages/dbt/main.py", line 3, in <module>
    from dbt.logger import log_cache_events, log_manager
  File "/usr/local/lib/python3.11/site-packages/dbt/logger.py", line 17, in <module>
    from dbt.dataclass_schema import dbtClassMixin
  File "/usr/local/lib/python3.11/site-packages/dbt/dataclass_schema.py", line 37, in <module>
    class dbtClassMixin(DataClassDictMixin, JsonSchemaMixin):
  File "/usr/local/lib/python3.11/site-packages/mashumaro/mixins/dict.py", line 16, in __init_subclass__
    builder.add_from_dict()
  File "/usr/local/lib/python3.11/site-packages/mashumaro/core/meta/builder.py", line 419, in add_from_dict
    dialects_feature = self.is_code_generation_option_enabled(
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/mashumaro/core/meta/builder.py", line 594, in is_code_generation_option_enabled
    type_name(ancestor)
  File "/usr/local/lib/python3.11/site-packages/mashumaro/core/meta/helpers.py", line 150, in type_name
    elif is_generic(t) and not is_type_origin:
         ^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/mashumaro/core/meta/helpers.py", line 217, in is_generic
    raise NotImplementedError
NotImplementedError

Previous version prints the usage message as expected:
docker run -it ghcr.io/dbt-labs/dbt-postgres@sha256:58a78ffd8e0f59f620b8cc9ea66904fa46e09059b2ee93167775fab345e9387f
usage: dbt [-h] [--version] [-r RECORD_TIMING_INFO] [-d] [--log-format {text,json,default}] [...]

Expected Behavior
Not crash and burn :)
Steps To Reproduce
1: Run commands as shown above.
Relevant log output
No response
Environment
- OS: Docker on Ubuntu on WSL, and on Google Kubernetes Engine.
- Python: 3.11 (in provided image)
- dbt: 1.3.0 (in provided image)

Which database adapter are you using with dbt?
postgres
Additional Context
This upgraded to Python 3.11 in the image: https://github.com/dbt-labs/dbt-core/pull/6180/files
This report with a similar exception, claims mashumaro does not support Python 3.10 (and I presume it might have problems with 3.11 too then): https://github.com/dbt-labs/dbt-core/issues/5976

@torkjel Thanks for opening! Good call - I think we probably ought to revert https://github.com/dbt-labs/dbt-core/pull/6180 until dbt officially supports Python 3.11 (https://github.com/dbt-labs/dbt-core/issues/6147), and rebuild the Docker images for the affected versions in the meantime.


is there a way to use openAI/chatGPT without a API key - but as chatGPT plus user?

I have this question for a while, do you have some ideas on this?
I am wondering if someone using rust, figured out some way to interact with the chatGPT plus user account, without using the API key (as a chatGPT user, it seems paying twice? - pay for API and at the same time pay for chatGPT plus user account?)
Thanks

try https://github.com/gngpp/ninja


Can HTTP 302 redirect which is being used now be changed to HTTP 301 redirect?

HTTP 301 redirect is a permanent redirect. Means that link cannot be used for other redirections.

@sandeepsajan0  you can work on this?

@modihere  yeah I am working on this.


Use cog for the README

(not related to the issue, but using cog + symbex was where my brain immediately went to for docs)


New release + "support" for Python 3.13

The planned magic numbers should work for Python 3.13 unless they will change something in the marshal format.

We might want to add new classifiers (not a blocker for the release):
https://github.com/fedora-python/marshalparser/blob/16f28936dfe98e69757ac4e9f16381938c56730a/setup.cfg#L19


Error in sample Python Events code.

In the Events section of the Python reference.md, an error is caused by the fact that the body of the "def start(host):" function is empty.
What should be there?
The error is: "IndentationError: expected an indented block".

Hi, thanks for reporting the issue. The start function was a leftover from an old version. Just fixed the doc. It reads now:
from durable.lang import *

with ruleset('risk'):
    @when_all(c.first << m.t == 'purchase',
              c.second << m.location != c.first.location)
    # the event pair will only be observed once
    def fraud(c):
        print('Fraud detected -> {0}, {1}'.format(c.first.location, c.second.location))

post('risk', {'t': 'purchase', 'location': 'US'})
post('risk', {'t': 'purchase', 'location': 'CA'})


Task path value substitution fixes.

Fixes issue where path for wordpress installation is invalid.

@michaeldove Thanks for the PRs!


unlock_keychain fails using Jenkins - exit status 181

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
On a mac with macOS Sierra, running jenkins.
unlock_keychain is failing with a exit status of 181. Performing the same commands via terminal (as the jenkins user) works correctly.
This also occurs when using the setup_jenkins action.
However if I log in as another user and copy the .keychain file to  ~/Desktop then the unlock_keychain command works well.
"Shell command exited with exit status 181 instead of 0."
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

bash-3.2$ fastlane run unlock_keychain path:*******/Jenkins.keychain password:"*****" add_to_search_list:replace --capture_output
Captured Output
Command Used: run unlock_keychain path:*******/Jenkins.keychain password:******* add_to_search_list:replace
Output/Log

DEBUG [2017-05-04 14:39:54.51]: Checking if there are any plugins that should be loaded...
DEBUG [2017-05-04 14:39:54.54]: Using path: *******/Jenkins.keychain
DEBUG [2017-05-04 14:39:54.54]: Using password: *******
DEBUG [2017-05-04 14:39:54.54]: Using add_to_search_list: replace
INFO [2017-05-04 14:39:54.55]: -----------------------------
INFO [2017-05-04 14:39:54.55]: --- Step: unlock_keychain ---
INFO [2017-05-04 14:39:54.55]: -----------------------------

✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.3

Ruby
2.0.0

Bundler?
false

Git
git version 2.10.1 (Apple Git-78)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.12.3 (16D32)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib

OpenSSL Version
OpenSSL 0.9.8zc 19 Mar 2015

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode8.2.1.app/Contents/Developer/

Xcode Version
8.2.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.28.8
💥 Check failed

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
1.5.1

CFPropertyList
2.3.4

claide
1.0.1

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.4.4

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

unicode-display_width
1.1.1

terminal-table
1.7.3

plist
3.2.0

addressable
2.3.8

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.4

excon
0.54.0

faraday
0.9.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20160826

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
1.9.0

gh_inspector
1.0.2

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

multi_json
1.12.1

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.9.20

json
1.7.7

mini_magick
4.5.1

multi_xml
0.5.5

rubyzip
1.1.7

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.1.1

bundler
1.13.7

faraday_middleware
0.10.0

generated on: 2017-05-04

/Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane_core/lib/fastlane_core/ui/interface.rb:150:in `user_error!': [!] Shell command exited with exit status 181 instead of 0. (FastlaneCore::Interface::FastlaneError)
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane_core/lib/fastlane_core/ui/ui.rb:14:in `method_missing'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/helper/sh_helper.rb:55:in `sh_control_output'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/helper/sh_helper.rb:10:in `sh'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/actions/unlock_keychain.rb:26:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/runner.rb:252:in `block (2 levels) in execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/actions/actions_helper.rb:50:in `execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/runner.rb:230:in `block in execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/runner.rb:226:in `chdir'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/runner.rb:226:in `execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/one_off.rb:41:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/one_off.rb:22:in `execute'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/commands_generator.rb:192:in `block (2 levels) in run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/command.rb:153:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/runner.rb:476:in `run_active_command'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:40:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/commander-fastlane-4.4.4/lib/commander/delegates.rb:15:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/commands_generator.rb:303:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/commands_generator.rb:42:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/fastlane/lib/fastlane/cli_tools_distributor.rb:66:in `take_off'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-2.28.8/bin/fastlane:20:in `'
	from /usr/local/bin/fastlane:23:in `load'
	from /usr/local/bin/fastlane:23:in `'

Some more information:
If I run the following commands instead of unlock_keychain then match seems to work correctly and the build continues successfully.
/usr/bin/security unlock-keychain -p "${KEYCHAIN_PASSWORD}" "${KEYCHAIN_PATH}"
/usr/bin/security -v list-keychains -s "${KEYCHAIN_PATH}"
/usr/bin/security -v set-keychain-settings -t 600 "${KEYCHAIN_PATH}"
/usr/bin/security -v default-keychain -s "${KEYCHAIN_PATH}"

You wrote that Jenkins fails to do this. Usually Jenkins is setup to run as a different user. Are you the using the Jenkins account when you get the error? Also, Jenkins will usually log into a slave with a different shell environment than a regular user, are you certain that you are using the same environment (in terms of which ENV variables are setup).

Yes the errors occur both when triggering the jenkins job (which runs as the jenkins user) and also when acting the the jenkins user in the terminal (via su jenkins).

I've also found the using the following fastlane commands also work since they mirror the commands mentioned in a previous comment:

Can you run printenv for the Jenkins user and a user that can run this command without the failure and post the results here (taking out any sensitive info)?

I have a similar error caused by a (it seems) different scenario. I'll create an issue and link it here.

@agustindc-rga in #9462 the heuristic was incorrectly identifying the keychain path, causing 181 errors. I fixed the issue.
Maybe your issue was caused by a similar problem. If not, I just added some more logging which coud help us understand why it fails for you.
If you are able to try the fastlane github HEAD and let us know how it goes.


Failure to load team chat channel / app on constant loop of crash and restart

Keybase GUI Version: 5.8.0-20210920151148+ac2177aec7
Hi,
Earlier today I posted a message to a channel in teams, and the message was too large. Instead of the cancel / try option (which has displayed only once), the app freezes, and then crashes or disconnects, then reloads. It's a constant cycle. Happy to send logs, just let me know where they are located on the system.
OS: Windows 10

uninstalling and clearing everything that was leftover in C:\Users[name]\AppData\Local\Keybase resolved the issue once reinstalled.


Add CMakelists

Following https://github.com/ARMmbed/mbed-os/pull/13566
Adding CMakelists to our examples
I still need to fix travis, circleCI should work.
Please note, this is still in development (for up to date status, see mbed-os/cmake/README.md)

I updated travis to use master branch. It will fail now but once this gets to master, it should be green (will retest via new PR to make sure travis config is correct).

@hugueskamba Please review. I can squash these into one commit, as the changes are simple and nothing important in the history to be kept.

@hugueskamba Please review. I can squash these into one commit, as the changes are simple and nothing important in the history to be kept.

Thank you and yes please squash.

Squashed

Following ARMmbed/mbed-os#13566

It was merged.
I fixed travis to use master branch, once CI is all green, this should get in.

Fixing the memmap requirements, we will need to figure out the python deps and how to use the scripts.


Add Samsung TV config flow

Breaking Change:

Description:
Add config flow for Samsung TV integration. (next step to split up https://github.com/home-assistant/home-assistant/pull/26865)
Related issue (if applicable): fixes #
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):
not needed, use the UI
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

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

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@tulindo For your information.

Excellent so you choosed to go for the common code.
Simple yet effective

@MartinHjelmare Don't know if you're the right one, but for me the tests seem ok. Two other tests from a former PR are failing with py36 because of too high log level. Should I fix this here?

Please fix the existing flaky tests in a separate PR so we can merge that asap. Thanks!

@MartinHjelmare It's working now.

@MartinHjelmare Is it correct to remove setup_platform now?

@MartinHjelmare What do you think, can we merge?

A broadcast address config option was recently added in another PR. If we want to keep that we need to add it to the component config schema.

@MartinHjelmare fixed your comments

There are merge conflicts.

Shouldn't we save the once detected method and/or port? It's very very unlikely this will change ever in the TV lifetime.

What do you mean?

I've added automatic detection of legacy or websocket devices. Currently this always has to be determined after HA start. We could avoid this.

Yeah the check could be part of the config flow and then save the type in the config entry. :+1:

@MartinHjelmare What do you say is missing to get this merged?

I think we're pretty close. Some clean up around the config validation. Solve the merge conflict.
Everything we can postpone to a later PR we should skip.

Please don't merge, I try to add the mentioned update mechanism.

And there is also still an open point: when starting HA with TV off, the discovered data is added, not the configured when the TV is switched on later

Let me know when you're happy with testing.

Can you explain why this is failing now? https://dev.azure.com/home-assistant/Home Assistant/_build/results?buildId=22596&view=logs&j=67510a0b-4400-5e74-cb95-9572caf5ca54&t=f8fe93fb-5ebb-59cc-455e-c0f2d9ad3a42&l=240

OK, I think I'll postpone this topic. I'm fine now to merge this finally.

Same to you. First a was a bit frustrated that with every commit you find another line to complain about. But after trying to review the code of somebody else I have to say this is really hard work. Thanks for your help.


Bug: Video is not aligned with background in Safari

Is there an existing issue for this?

[X] I have searched the existing issues

Which Mux Elements/Packages does this apply to? Select all that apply
mux-player-react
Which browsers are you using?
Chrome, Safari, Firefox
Which operating systems are you using?
macOS
Description
I'm finding that the media background is peeking out from behind my videos in Safari - specifically a few pixels are visible at the top. I see the same behavior on various devices and in your docs, so I suspect it isn't just me. It is possible to hide the background with "--media-background-color": "transparent". That would be a solution except that my designer wants the video to have rounded corners, and the misalignment of elements is resulting in a subtly ugly chopped-off look. The issue also goes away if I remove the aspectRatio prop from the player, but then the video loads in an unpleasant way.
Top of my video in Chrome:

Top of my video in Safari:

In Safari with a transparent background:

Reduced test case
No response
Steps to reproduce

Visit https://docs.mux.com/guides/player/customize-look-and-feel in Chrome
Visit https://docs.mux.com/guides/player/customize-look-and-feel in Safari
See that the top of the featured video looks different in each browser

Current Behavior
The media background is not vertically aligned with the video in Safari (and in Chrome on mobile).
Expected Behavior
Alignment of player elements is consistent across different browsers.
Errors
No response
What version of the package are you using?
v1.14.1

Hey @kelihansen thanks for filing the issue! I did some preliminary digging, and I'm not sure this is something we'll be able to resolve. It appears to be an issue at the browser level 💔. Here's what both the underlying <video> element and the <media-controller> (which is the custom element that "owns" the background) are reporting for their bounding ClientRect:
> JSON.stringify(document.querySelector('mux-player').media.shadowRoot.querySelector('video').getBoundingClientRect(), null, 2)
< "{
  \"x\": 392,
  \"y\": 93,
  \"width\": 896,
  \"height\": 507.46875,
  \"top\": 93,
  \"right\": 1288,
  \"bottom\": 600.46875,
  \"left\": 392
}"
> JSON.stringify(document.querySelector('mux-player').mediaController.getBoundingClientRect(), null, 2)
< "{
  \"x\": 392,
  \"y\": 93,
  \"width\": 896,
  \"height\": 507.46875,
  \"top\": 93,
  \"right\": 1288,
  \"bottom\": 600.46875,
  \"left\": 392
}"

Note that the positional information is identical. I'm also not seeing any obvious additional CSS that may be impacting the position. Between this and the fact that it is not occurring on other browsers, I suspect this bottoms out in the way WebKit/Safari is managing the rendering plane for the media itself. We may still be able to figure out a way to do some CSS tightrope walking to code around the issue without also breaking other sizing/layout cases, but I am hesitant to make any promises. I'll be sure to get a second set of eyes on this ASAP just in case I missed something. Wish I had better news 😞.

Hey @kelihansen, there might be a way to mask this behaviour.
Unfortunately we can't apply this fix generically from within mux-player as there are scenarios where changing the object-fit property for all cases doesn't make sense. For your use case though you should be able to use --media-object-fit to change it to cover instead of the default contain. See this codesandbox example.
This assumes your videos and player will always be 16/9 and so setting the object-fit property to cover shouldn't have any noticeable affect except covering up that space that Safari adds. Let me know if this works for you.

Looks like that will meet our needs, at least for now! Thank you both so much for digging in!

@kelihansen that makes sense, we're telling it to always expand the video to fill the space. You could maybe control it using an orientation media query: https://developer.mozilla.org/en-US/docs/Web/CSS/@media/orientation


Convert to Loot does not set the Sheet to a LootSheet

Describe the bug
When converting a token to loot the token sheet stays the same/the global default.
To Reproduce
Steps to reproduce the behavior:

Drag an NPC actor into the canvas
Select the token created
Either use the macro or click in the "convert to Loot" button on NPC sheet
See that features, special itens and such are removed, the permission is set to observer and the overlay icon is set, but the token sheet does not change to a loot sheet

Expected behavior
When using the convert to loot option, the sheet type should change to a loot sheet automatically, as per the documentation.
Screenshots
Here is the output on console about the loot generation:

Additional context
I am using FoundryVTT v10, build 291, with Dnd5e system version 2.1.5
I also have the latest version for both Tidy 5e sheet and lootsheet NPC

Result after conversion:

I noticed that with the migration for V10, the name fo lootsheet NPC module changed to lootsheet-simple, maybe this is something that is interfering?

I just found out that the identifier for npc loot sheet has changed to dnd5e.LootSheet5eNPC, while in the code is using dnd5e.LootSheetNPC5e as value. I think I can make a PR fixing it, but I am not very well versed on JS.

I'm having the same issue.

Awesome, thanks for reporting and fixing!


Add daily mean ecco dataset to catalog

Add the catalog for daily ecco

The error in CI you are getting is related to issue #332 , independent of the changes within this PR.
I got another fix for the issue in #332 , but I am having some trouble trying to install a testing environment in Sciserver to check that the error goes away. Mitya is helping me, hopefully will get a PR fixing issue #332 later today...

@MaceKuailv don't forget to run pre-commit to get all the formatting right

I am doing the pre-commit right now. I didn't realize that this part also have format constraints.

Thanks @MaceKuailv and @Mikejmnez!  I'll make an announcement on https://www.poseidon-ocean.net/.


Feature/obj c

Adds Objective-C file generation with the addition of "objc" as the final argument.
Defaults to Swift file generation otherwise.

closing this PR, as it's made redundant by the latest PR (named parameters)


Give names to VCCs

This is a little tricky because many VCs can be created for a given application, and it's not clear which one contributes to a SAT result.
One possibility is to add a bunch of dummy VCs of the form vc_name = vc_body, at each VC creation time, (where vc_name is some fresh boolean) and maintain the map of actual names to vc_names.
Finally given a SAT result, we could check the assignments to see which one was refuted.
Note, we need to generate (vc_name = vc_body) => vc_body instead of just vc_body to get the intended result.

Is this still open?

I believe this was handled in d6b465194978d3e1e084d0893eef0875dc7929c3


[EDMTP-191] Added maven profile for automatic installation of AAT-assets

profile name is install-assets

So cool ! Thanks!
But I think we need to update documentation for that change.
CC: @Konyaka1 , @smiakchilo


Question about multi-modal GT

Hi, thank you for sharing the code and I have a few questions:

Where can we find the generation of multi-modal ground truth (MM GT) as descripbed in the paper?
Is there an implementation of Wasserstein Loss with MM GT as described by Eq. 12 in the paper?
Did DSGN+CDN (the best 3D AP on KITTI) use MM GT or not? I'm using DSGN and trying to reproduce DSGN+CDN+Wasserstein Loss.

Hi, thanks for the interest in our paper:
We haven't released the MM GT version yet but plan to do it soon.
For DSGN+CDN, we don't use MM GT as we find that that MM GT helps improve training earlier but results in similar final accuracies.
To reproduce results for DSGN, it is easiest to compare CDN-PSMNet to original PSMNet model and apply the changes to DSGN.


Resuming the StreamingDataLoader from a checkpoint makes the iterator hang

🐛 Bug
In LitGPT, we currently aren't able to resume correctly from a pretrained checkpoint. The state the dataloader loads causes a stall in the iterator.
To Reproduce
Studio for reproduction.
I made a minimal repro without LitGPT involved:
import torch
from litdata.streaming import CombinedStreamingDataset, StreamingDataLoader, StreamingDataset, TokensLoader

def get_dataloader():
    slim_train_data = StreamingDataset(
        input_dir="s3://tinyllama-template/slimpajama/train",
        item_loader=TokensLoader(block_size=2049),
        shuffle=True,
        drop_last=True,
    )
    train_data = slim_train_data

    train_datasets = [
        slim_train_data,
        StreamingDataset(
            input_dir="s3://tinyllama-template/starcoder",
            item_loader=TokensLoader(block_size=2049),
            shuffle=True,
            drop_last=True,
        ),
    ]

    # Mix SlimPajama data and Starcoder data with these proportions:
    weights = (0.693584, 0.306416)
    train_data = CombinedStreamingDataset(
        datasets=train_datasets, seed=42, weights=weights, iterate_over_all=False
    )
    train_dataloader = StreamingDataLoader(
        train_data, batch_size=4, pin_memory=True, num_workers=8, drop_last=True
    )
    return train_dataloader

def dataloader_state():
    """Dataloader state extracted from the real checkpoint, for reproduction"""
    return {'dataset': {'0': {'num_samples_yielded': 852336, 'num_workers': 8, 'batch_size': 4, 'current_epoch': 1, 'input_dir_path': None, 'input_dir_url': 's3://tinyllama-template/slimpajama/train', 'item_loader': {'block_size': 2049}, 'drop_last': True, 'seed': 42, 'world_size': 1, 'shuffle': True}, '1': {'num_samples_yielded': 376464, 'num_workers': 8, 'batch_size': 4, 'current_epoch': 1, 'input_dir_path': None, 'input_dir_url': 's3://tinyllama-template/starcoder', 'item_loader': {'block_size': 2049}, 'drop_last': True, 'seed': 42, 'world_size': 1, 'shuffle': True}}, 'current_epoch': 0, 'latest_worker_idx': 7, 'num_samples_yielded': {0: [106542, 47058], 1: [106542, 47058], 2: [106542, 47058], 3: [106542, 47058], 4: [106542, 47058], 5: [106542, 47058], 6: [106542, 47058], 7: [106542, 47058]}}

if __name__ == "__main__":
    train_dataloader = get_dataloader()
    # checkpoint = torch.load("dataloader-state.pt")
    train_dataloader.load_state_dict(dataloader_state())

    print("start")
    for batch in train_dataloader:  # hangs here
        print(batch)

If executed in a Studio, to avoid authentication errors, the following patch is needed in /home/zeus/miniconda3/envs/cloudspace/lib/python3.10/site-packages/litdata/streaming/client.py, line 40:
        if has_shared_credentials_file or not _IS_IN_STUDIO or True:
            self._client = boto3.client(
                "s3", config=botocore.config.Config(retries={"max_attempts": 1000, "mode": "adaptive"}, signature_version=botocore.UNSIGNED)
            )

Expected behavior
Continue fetching data from where we left off.
Environment
LitData: 0.2.15
cc @sanyalsunny111

In this line of code
https://github.com/Lightning-AI/litdata/blob/14a47f21575abd045115ad3c43a766d08b102f02/src/litdata/streaming/dataloader.py#L444-L448
The range becomes empty because self._loader._latest_worker_idx == self._loader.num_workers == 8. With self._indexes being an empty list, this then causes this if-else logic
https://github.com/Lightning-AI/litdata/blob/14a47f21575abd045115ad3c43a766d08b102f02/src/litdata/streaming/dataloader.py#L462-L476
to go into infinite recursion, because it calls itself when self._indexes evaluates to False.

thanks for pinpointing the point of concern. We'll try to fix the issue ASAP.

I found the problem, but it is not in the main process like I showed above. The problem is simpler to see/debug if we reduce the number of workers:
import torch
from litdata.streaming import (
    CombinedStreamingDataset,
    StreamingDataLoader,
    StreamingDataset,
    TokensLoader,
)

def get_dataloader():
    slim_train_data = StreamingDataset(
        input_dir="s3://tinyllama-template/slimpajama/train",
        item_loader=TokensLoader(block_size=2049),
        shuffle=True,
        drop_last=True,
    )
    train_data = slim_train_data

    train_datasets = [
        slim_train_data,
        StreamingDataset(
            input_dir="s3://tinyllama-template/starcoder",
            item_loader=TokensLoader(block_size=2049),
            shuffle=True,
            drop_last=True,
        ),
    ]

    # Mix SlimPajama data and Starcoder data with these proportions:
    weights = (0.693584, 0.306416)
    train_data = CombinedStreamingDataset(
        datasets=train_datasets, seed=42, weights=weights, iterate_over_all=False
    )
    train_dataloader = StreamingDataLoader(
        train_data, batch_size=4, pin_memory=True, num_workers=1, drop_last=True
    )
    return train_dataloader

def dataloader_state():
    """Dataloader state extracted from the real checkpoint, for reproduction"""
    return {
        "dataset": {
            "0": {
                "num_samples_yielded": 852336,
                "num_workers": 1,
                "batch_size": 4,
                "current_epoch": 1,
                "input_dir_path": None,
                "input_dir_url": "s3://tinyllama-template/slimpajama/train",
                "item_loader": {"block_size": 2049},
                "drop_last": True,
                "seed": 42,
                "world_size": 1,
                "shuffle": True,
            },
            "1": {
                "num_samples_yielded": 376464,
                "num_workers": 1,
                "batch_size": 4,
                "current_epoch": 1,
                "input_dir_path": None,
                "input_dir_url": "s3://tinyllama-template/starcoder",
                "item_loader": {"block_size": 2049},
                "drop_last": True,
                "seed": 42,
                "world_size": 1,
                "shuffle": True,
            },
        },
        "current_epoch": 0,
        "latest_worker_idx": 0,
        "num_samples_yielded": {
            0: [106542, 47058],
            # 1: [106542, 47058],
            # 2: [106542, 47058],
            # 3: [106542, 47058],
            # 4: [106542, 47058],
            # 5: [106542, 47058],
            # 6: [106542, 47058],
            # 7: [106542, 47058],
        },
    }

if __name__ == "__main__":
    train_dataloader = get_dataloader()
    # checkpoint = torch.load("dataloader-state.pt")
    train_dataloader.load_state_dict(dataloader_state())

    print("start")
    for batch in train_dataloader:  # hangs here
        print(batch)

What I found so far is that the reader is waiting for a chunk to be available here, but it never is:
https://github.com/Lightning-AI/litdata/blob/c4c9117134e20c6b3f9ca7b071932475ab12da80/src/litdata/streaming/item_loader.py#L132-L134
Chunks are being downloaded, but it seems their names are not the same as what the item loader is waiting for, so it will wait forever
For example, the downloader thread will download a file
'/cache/chunks/c8829d9f5b65f28afca2a0e7e207a8a7/chunk-95-280.bin'

corresponding to a chunk index 6748, but the reader will wait in the while loop for a different file
/cache/chunks/c8829d9f5b65f28afca2a0e7e207a8a7/chunk-120-15.bin

with chunk index 8034
This deadlock suggests that the way the filename/chunk index is computed inconsistently is the problem.

A quick question and a suggestion? Can we try to make a threshold time say for 50 ms for a particular chunk and move on to the next chunk? I know this can reduce the number of tokens used for training.

@sanyalsunny111 It's a systematic problem. We can't just skip chunks and hope it gets better :)
I'm trying to understand the code. One thing I don't understand is what the chunk.chunk_indexes are there fore. We put 17000 of them onto the queue here, but why: https://github.com/Lightning-AI/litdata/blob/c4c9117134e20c6b3f9ca7b071932475ab12da80/src/litdata/streaming/reader.py#L262-L263
I doubt that we want to download 17k chunks.


[SelectField][TextField] Fix error styling issue

@oliviertassinari regarding #2532 i've prepared some simple PR that fixes issue in both SelectField and TextField, for sure it can be done better, but i have limited time now and i can't dig into deeper, so maybe it'll help someone to do it well

Thanks for it, I was also working on it (and deprecating some imperative method). This is going to help me.

@patrykkopycinski Could you see if https://github.com/callemall/material-ui/pull/2539 is correctly solving your issue?


2 out of 8 backbones throw size errors

Hi, apologies for creating multiple issues, but really liking this model!
At a default resolution of 1270x736 I'm able to train the following backbones without error:

resdcn_18
resdcn_34
resfpndcn_34
dla_34
hrnet_18
cspdarknet_53

However resdcn_50 and hrnet_32 both throw sizing errors:
resdcn_50:

RuntimeError: The size of tensor a (216) must match the size of tensor b (215) at non-singleton dimension 3

hrnet_32:

RuntimeError: Given transposed=1, weight of size 36 36 2 2, expected input[4, 64, 68, 120] to have 36 channels, but got 64 channels instead

Do you happen to recognize what the problem might be for either of these? Not sure if it's simply an unallowable input size or if network modifications are required.

Also I'm curious if you have a backbone recommendation for best detecting very small objects (areas of ~50 pixels) which also have high counts and are very clustered together (tinyperson crowd counting)?
And is there value in upsampling my 1280x720 resolution videos to something like 1920x1088 to assist small obj detection?
Really appreciate any advice you have time to offer!

@austinmw I think a higher resolution will help to detect small objects, but make sure the resolution will be integer-divisible by 4 which is the pre-set down-sampling factor for the feature map used to do ReID.


Sentry does not play along with Paramiko's SSHException

Hello!
We're using paramiko for one of our projects. We sometimes get an exception that the library is throwing: paramiko.ssh_exception.SSHException.
Basically, sentry is publishing each line of the traceback as individual errors.

I've read this stack overflow over and over, trying to figure out what is wrong and I've got a working example.
lilja in ~/c/u/customs-readjob-setup on new-lambda-handler λ cat paramiko_test.py
#!/usr/bin/env python
import sys
import socket
import paramiko
import logging

logger = logging.getLogger(__name__)

import sentry_sdk

sentry_sdk.init(dsn="")

_sys_stderr = sys.stderr

host = ""
username = ""
password = ""
port = ""

class StdOut(object):
    def __init__(self, stream, _type):
        self.txtctrl = stream
        self.type = _type
    def write(self, string):
        self.txtctrl.write(f"{self.type} -> {string}")

    def flush(self):
        self.txtctrl.flush()

sys.stdout = StdOut(sys.stdout, "out")
sys.stderr = StdOut(sys.stderr, "err")

try:
    logger.info(f"Logging in to {host} as {username}")
    transport = paramiko.Transport((host, port))
    transport.banner_timeout = 500
    transport.auth_timeout = 500
    transport.connect(None, username=username, password=password)
    k = paramiko.SFTPClient.from_transport(transport)
except paramiko.ssh_exception.BadAuthenticationType as e:
    print(f"Bad authentication type. {e!r}")
    exit(1)
except (socket.error, socket.gaierror, paramiko.ssh_exception.SSHException) as e:
    if "Error reading SSH protocol banner" in str(e):
        print(
            "Error reading ssh protocol banner. This might be a temporary error, "
            + "or the server is a FTP(not SFTP) server."
        )
        exit(1)
    raise

Output:
err -> Exception: Error reading SSH protocol banner
err -> Traceback (most recent call last):
err ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/transport.py", line 2211, in _check_banner
err ->     buf = self.packetizer.readline(timeout)
err ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/packet.py", line 380, in readline
err ->     buf += self._read_timeout(timeout)
err ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/packet.py", line 622, in _read_timeout
err ->     raise socket.timeout()
err -> socket.timeout
err ->
err -> During handling of the above exception, another exception occurred:
err ->
err -> Traceback (most recent call last):
err ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/transport.py", line 2039, in run
err ->     self._check_banner()
err ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/transport.py", line 2215, in _check_banner
err ->     raise SSHException(
err -> paramiko.ssh_exception.SSHException: Error reading SSH protocol banner
err ->
out -> Error reading ssh protocol banner. This might be a temporary error, or the server is a FTP(not SFTP) server.out ->
err -> Sentry is attempting to send 18 pending error messages
err -> Waiting up to 2 seconds
err -> Press Ctrl-C to quit

Notice how sentry wants to send 18 error messages! I think that might be a bit much.
According to the stack overflow post, the reason is because paramiko.Transport is doing its work in a thread, which is posting its errors to stderr. The proposed solution is to pipe errors to stdout.
try:
    logger.info(f"Logging in to {host} as {username}")
    sys.stderr = sys.stdout
    transport = paramiko.Transport((host, port))
    transport.banner_timeout = 500
    transport.auth_timeout = 500
    transport.connect(None, username=username, password=password)
    k = paramiko.SFTPClient.from_transport(transport)
except paramiko.ssh_exception.BadAuthenticationType as e:
    print(f"Bad authentication type. {e!r}")
    exit(1)
except (socket.error, socket.gaierror, paramiko.ssh_exception.SSHException) as e:
    if "Error reading SSH protocol banner" in str(e):
        print(
            "Error reading ssh protocol banner. This might be a temporary error, "
            + "or the server is a FTP(not SFTP) server."
        )
        exit(1)
    raise
finally:
    sys.stderr = _sys_stderr

Which brings the output:
out -> Exception: Error reading SSH protocol banner
out -> Traceback (most recent call last):
out ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/transport.py", line 2211, in _check_banner
out ->     buf = self.packetizer.readline(timeout)
out ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/packet.py", line 380, in readline
out ->     buf += self._read_timeout(timeout)
out ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/packet.py", line 622, in _read_timeout
out ->     raise socket.timeout()
out -> socket.timeout
out ->
out -> During handling of the above exception, another exception occurred:
out ->
out -> Traceback (most recent call last):
out ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/transport.py", line 2039, in run
out ->     self._check_banner()
out ->   File "/Users/lilja/code/umbrella/customs-readjob-setup/.venv/lib/python3.8/site-packages/paramiko/transport.py", line 2215, in _check_banner
out ->     raise SSHException(
out -> paramiko.ssh_exception.SSHException: Error reading SSH protocol banner
out ->
out -> Error reading ssh protocol banner. This might be a temporary error, or the server is a FTP(not SFTP) server.out ->
Sentry is attempting to send 18 pending error messages
Waiting up to 2 seconds
Press Ctrl-C to quit

Exactly the same output.
Why is sentry sending so many events, supposedly splitted by each newline?
The core issue in this paramiko example is that we're trying to connect to an FTP, instead of an SFTP. The FTP is a windows filezilla server, which when attempting connect send the response  500 Syntax error, command unrecognized because paramiko only supports SFTP connections. We've patched this internally but still are dealing with the logs.

Those events seem to have a logger tag paramiko.transport. Based on that I went into their sourcecode and found this: https://github.com/paramiko/paramiko/blob/ae3d0febef17a8ece5268bbf6c210a30573ce800/paramiko/transport.py#L2130
This calls _log, a helper function that logs each line individually using that logger, with the ERROR level, which creates a sentry event for each line.
You can ignore that logger using the mechanism described at https://docs.sentry.io/platforms/python/logging/#ignoring-a-logger, or disable event creation from error logs entirely using the information found on that page as well.
TLDR:
from sentry_sdk.integrations.logging import ignore_logger
ignore_logger("paramiko.transport")

Works great!


Fix "-> Type" on a single line indentation

Plus a few minor spelling changes in comments
Fixes #8

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Can I just say thank you to the people who wrote this? Incredibly well commented, easy to edit, awesomeness.

Ah! This is definitely an improvement. I find that there are a lot of other cases where indentation is at least somewhat strange in rust-mode, which is perhaps an argument for switching to a more "tried and true" way of dealing with indentation, e.g. based on SMIE. (To see what I mean, find a random if statement with a moderately complex condition, where the { is after the condition rather than on a line by itself, and insert a newline between any random pair of tokens, then reindent the block. This will almost certainly mess up the indentation of the entire if block. Several other types of expression are similarly fragile to users adding newlines in "unexpected" places.)
Nevertheless, this PR works around one of the most common bad cases, so it is still quite helpful.

Yes, we can work on the rest of the indentation issues later, this is just an issue that I had that was an easy fix.

@quantheory I'm setting up a new repo in order to work on an SMIE grammar for Rust. Would you like to join me? (I've never done any SMIE work before (and in fact, am an Emacs newbie), so I'd like someone who actually knows what they're doing.

@GBGamer I would, but I'm afraid I am really the wrong person to ask. I actually only have a dimly-remembered mental picture of how SMIE works from several years back, and I'm pretty overextended in any case. I really only check in here because I like to whine hope that some of the quirks in the current rust-mode will get ironed out soon.

Pinging @nikomatsakis

Okay, normally I'd say "please add a test case."
But I want this badly enough that I'm probably going to just merge it.  (I just want to quickly double-check myself that it behaves as advertised -- if so, then I'll merge it and then work on a test case myself.)

Yes, awesome, it fixes both of my most important cases:
    fn foo(a: i32)
           -> i32 {
               // whoops
           }

    fn foo<T>(a: i32)
              -> i32 where T:Foo {
                  // bad too
              }

@pnkfelix Thanks for the regression test, I didn't even think about that :stuck_out_tongue_closed_eyes:


Possible support for pgx v5 and compatibility with v4

What do you want to change?
I recently tried out sqlc with pgx v5 which is currently in alpha.
I want to enquire on the path forward for v5 upgrade and if there's a plan to accommodate the changes.
If there is none kindly consider this as a placeholder. Feel free to close this issue if it is in the works already.
I will jot down a few things that I noticed that needed to be changed.

Line 120 in codegen.golang.imports must change to import from v5/pgconn since pgconn inside the v5 package itself as described in pgx changelog.
Currently we have provision for one pgx package only here. We will need to alter it if compatibility with v4 must be maintained.
pgtype imports and enhancements

What database engines need to be changed?
PostgreSQL
What programming language backends need to be changed?
Go

We will need to alter it if compatibility with v4 must be maintained.

Compatibility is the most important thing; we;ll need to continue to support v4. Thankfully we named the pgx sql_package identifier pgx/v4, so it should be straightforward to add support under pgx/v5.

Sure makes sense.


when quick fixing await also change the return type of the function

When quick fixing this await to add the async modifier:
void main() {
  await 1;
}

It would be nice if the return type could change to Future

https://codereview.chromium.org/1652963003

https://github.com/dart-lang/sdk/commit/86106d7a6d1ea479b6ec139d7c54e84f6630eede

Thanks :+1:


drivers: i2c: sam0: add support for I2C_MSG_RESTART

This PR reinstates support for I2C repeated starts, that was accidentally removed in the re-shuffle in de18f97...
Fixes #55172

@nandojve - thanks for the suggestion, I've reworked the comment to discuss this subtlety.


Create ISSUE and PR templates

Issue and PR templates are a necessity. It ensures that people stick to the guidelines also makes it easy for reviewers to get their reviews done.

@sneakycr0w @collinsnji I have them ready, will it be alright if I add them ?

Go for it
Zachary E Sohovich
Full Stack Web Developer
https://sneakycr0w.com
<EMAIL_ADDRESS>
On Oct 2, 2017, at 11:06 AM, Anshuman Verma<EMAIL_ADDRESS>wrote:
@sneakycr0w @collinsnji I have them ready, will it be alright if I add them ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


docker image

Would it be possible for you to make a docker file/image for this bot. For fast deployment?

I have tried once to do an image but was not able to do it because I don't have much experience with docker.
But would probably try to make one in the future.

If there is still interest I could try to get this going with a little help, I've worked with docker a bit


[Feature Request] Add support for Cloudflare Web Analytics

Feature Request:
Duplicate of https://github.com/gohugoio/hugo/issues/8482 , which got prematurely auto closed by the github bot. Can we please have a direct integration with Cloudflare Analytics in Hugo, just like the existing support for Google Analytics: https://github.com/gohugoio/hugo/blob/master/tpl/tplimpl/embedded/templates/google_analytics_async.html ?
Cloudflare analytics are privacy preserving and non-obtrusive form of analytics for static websites.

Hi @jmooring , saw the issue was closed as completed. Are there any docs relevant to the CF Analytics integration? Or any code pointers pointing to the CF integration, if you could point out that'd be very helpful!

@ashutoshsaboo
You created two identical issues. https://github.com/gohugoio/hugo/issues/10464 is still open.


fix: enable toggle collapsible without providing v-model

[x] enable toggle collapsible without providing v-model
[x] enable custom visibility components inside with v-slot

Before: we need to pass v-model to toggle collapsible state
After: we dont need to pass v-model value, and if want to further provide custom appearances visibility of inside component, we use v-slots to get the state instead

Before: we need to pass v-model to toggle collapsible state

Not sure what you mean by this, because it should work with and even without v-model?
Original version for reference:
<script setup lang="ts">
import { provide, toRef } from "vue";

const props = withDefaults(defineProps<CollapsibleRootProps>(), {
  open: false,
});
const emit = defineEmits<{
  (e: "update:open", value: boolean): void;
}>();

provide<CollapsibleProvideValue>(COLLAPSIBLE_INJECTION_KEY, {
  contentId: "1",
  disabled: toRef(() => props.disabled),
  open: toRef(() => props.open),
  onOpenToggle: () => {
    emit("update:open", !props.open);
  },
});
</script>

<template>
  <div
    :data-state="props.open ? 'open' : 'closed'"
    :data-disabled="props.disabled ? 'true' : undefined"
  >
    <slot :open="props.open" />
  </div>
</template>

Also v-model doesn't work with this PR. Reverting this change in #81 (with added v-slot).


fix: missing signcolumn highlight group

Hi! recently I found one missing part while customizing the theme of the neo-tree.

Like the picture above, The background color of the neo-tree can be changed using NeoTreeNormal. And this overrides Normal.
However, because there were no highlight groups defined in the neo-tree for SignColumn, NeoTreeNormal was unable to create a consistent theme.

Thanks!


fix: unique name in load script runtime module

Summary
Support unique name in LoadScriptRuntimeModule
Test Plan

Add packages/rspack/tests/runtimeDiffCases/runtime-load-script/unique-name

Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

!diff

!diff

!diff

!diff


🎉 3.0.0

ℹ About this release

Version: 3.0.0
Type: major
Last Release: 2.1.0 (10/18/2022, 8:40:44 AM) [?]
Commits to merge: 80 [?]

Build System

Deprecate node.js 12 (426588b)

⚡ BREAKING CHANGES

From now on, only node.js ^14.8.0 || >=16.0.0 are supported

📦 Dependencies

Update execa from ^5.1.1 to ^7.1.1

📦 Development Dependencies

Added @semantic-release/npm ^9.0.2
Added tsup ^6.7.0
Update @qiwi/semantic-release-gh-pages-plugin from ^5.2.3 to ^5.2.5
Update @semantic-release/changelog from ^6.0.1 to ^6.0.2
Update @types/node from ^18.7.16 to ^18.15.3
Update @types/semantic-release from ^17.2.4 to ^20.0.1
Update @typescript-eslint/eslint-plugin from ^5.40.0 to ^5.54.1
Update @typescript-eslint/parser from ^5.40.0 to ^5.54.1
Update c8 from ^7.12.0 to ^7.13.0
Update eslint from ^8.25.0 to ^8.36.0
Update eslint-plugin-jsonc from ^2.4.0 to ^2.7.0
Update mocha from ^10.0.0 to ^10.2.0
Update typedoc from ^0.23.16 to ^0.23.28
Update typescript from ^4.8.4 to ^5.0.2
Removed @amanda-mitchell/semantic-release-npm-multiple

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

npm package (@latest dist-tag)
GitHub release
v3.0.0

Your semantic-release bot :package::rocket:


Add support for typed arrays

I didn't see a mention of typed arrays in the shims. For a react-native project I'm working on I added
support for the polyfills from typedarray to the shims.js file generated by rn-nodeify.
This fixed an error I got importing blake2b

Scratch that, it didn't completely fix the error since typedarray doesn't completely support ArrayBuffer. Do you know of a polyfill that does?

@Firescar96 as far as i know, typed arrays are supported natively in most recent ios and android versions, and you shouldn't have to do anything special to add support for them. What error are you hitting?

I got an error on this line that parameter_block is undefined. Above parameter_block is defined as Uint8Array

interesting, but it's unlikely new Uint8Array would return null if they were unsupported. More likely it would crash. Can you add some print statements and check that the line you linked definitely runs second?


R3det is not in the detector registry

Failed to train on R3Det* mmdetection version based on PyTorch. I got an import error when running sh rtools/train.sh
Traceback (most recent call last):
File "tools/train.py", line 161, in
main()
File "tools/train.py", line 134, in main
cfg.model, train_cfg=cfg.train_cfg, test_cfg=cfg.test_cfg)
File "/home/xs/mtl/r3det-on-mmdetection-master/mmdetection-2.2.1/mmdet/models/builder.py", line 67, in build_detector
return build(cfg, DETECTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/home/xs/mtl/r3det-on-mmdetection-master/mmdetection-2.2.1/mmdet/models/builder.py", line 32, in build
return build_from_cfg(cfg, registry, default_args)
File "/home/xs/.conda/envs/torch140/lib/python3.6/site-packages/mmcv/utils/registry.py", line 157, in build_from_cfg
f'{obj_type} is not in the {registry.name} registry')
KeyError: 'R3Det is not in the detector registry'
Besides, I install mmdetection from https://github.com/open-mmlab/mmdetection, because your setup.py installation failed in my device and I found the reason that mmdetection 2.2.1 is needed. So I guess my problem may caused by not using your setup.py?
My environment is :
Pytorch: 1.4.0
Python: 3.6.10
CUDA: 10.0
GCC: 7.3.0

And if I use your setup.py, it shows as :
gcc: error: mmdet/ops/nms/src/rnms_ext.cpp: No such file or directory

you can try to solve this by installing  mmcv0.5.9

u should run setup.py, not just mmdet

您好 我也是这个问题 但是我是run的setup.py 也没有报错 我猜测会不会是显卡的问题？作者使用的是20系显卡和10的cuda 我的环境是30系显卡和11的cuda。那应该怎么解决？


card.source.topic: allow CDS Service to specify a topic for high-level categorization

Adding a ruleCode field to Card.source, of type FHIR Coding. This allows the CDS Service to expose metadata about the rule it used to generate the Card. Knowledge of this information can allow the CDS Client to be more nuanced in its presentation of the Card.
Discussion: https://chat.fhir.org/#narrow/stream/179159-cds-hooks/topic/CDS.20Hooks.20feature.20request

@jmandel , @isaacvetter , @brynrhodes , @dennispatterson - requesting reviews :)

See review comment https://github.com/cds-hooks/docs/pull/505#pullrequestreview-299885651

@bvdh
I can see the point behind adviceCode or adviceId. However, how is that distinct from Card.uuid which is being proposed in #496? I was trying to keep Card.uuid as the identifier of the individual instance of the advice, and Card.source.ruleCode as the identifier of the specific "CDS Magic" in the CDS Service that generated this card. Open to other recommendations on name, but I wish to maintain that distinction.
How is that problem of coding any different from the opacity problem for a code in any other situation? Why would we propose that CDS Hooks have a specific-case solution to what is a generic problem with coding?
I want to push back against the 'behavioral requirement' suggested.

You are making assumptions about the general behavior of the EHR in the first place, and then suggesting constraints on that assumed general behavior. However, at least in our EHR, the assumption about the general behavior itself does not hold true.
In our implementation, if a new CDS request is made against that service, all the currently displayed cards from that service are dropped, before showing the new set of cards returned. So the behavior requirement proposed is moot to begin with. (Put another way, there is no session or card stack maintained between subsequent calls to the same service)
Additionally, I would say CDS Hooks should not mandate the behavior of the EHR in rendering decision support. It should only standardize the exchange of codified decision support. Different EHR implementations vary widely in how CDS Is presented, and each want to integrate CDS Hooks into their existing decision support frameworks, in the best fashion that fits.

I like the idea, and it dovetails nicely with some suggested approaches to tracking the source guidance that are currently using the link element and pointing to the underlying guidance. Would it make sense to align with that trajectory and say it should be guidanceUri or something like that? With the idea being that it may reference a URL on which the guidance is based (like a reference to a published guideline), or it may be the canonical URL of a knowledge artifact in FHIR, like a PlanDefinition or Library, or it may even be just a web page with more information about the rule?

I also agree that it shouldn't make any assumptions about the behavior and duplicate detection/elimination; that's a much broader problem that this could provide input into, but it's only one factor of many.

Are we talking about a "category"/"topic"?: http://build.fhir.org/plandefinition-definitions.html#PlanDefinition.topic

@brynrhodes - if it is a "coding" then a CDS vendor could use URIs / URLs as code values. So I would prefer we keep it a "coding" with a code-system and a code-value on it rather than a URI. Another system might use GUIDs for all their categories/topics?
I'm open to calling it topic, as long as it is still agreed between the CDS Service and CDS Client, and not enumerated by the Spec.

If we're changing label --> display here, we should do the same in #513. Both data structures basically represent a FHIR Coding, and I don't care much what we call the text bit, but we should be consistent :)

@jmandel - I just changed #513 (add override reasons) so that this PR's card.topic.display and that PR's card.overrideReasons.display use the same term.


Auto completion not working and needs some improvements

I'm using a computer with AMD Ryzen 5 5600G with integrated Radeon Graphics × 6, and 32GB RAM, on Linux Mint 21.
This is the code I'm using to try Wingman-AI:
def isPrime(n):
    """ Test if n is a prime number """
    if

def hello(i):
    for j in range(i):
        print(f"{j} is a value")

At the isPrime function, I've been trying to type either if or for to see if Wingman-AI generates a code completion.
From what I understand, while typing code, if I pause, Wingman-AI is supposed to suggest a code completion. In my case, it isn't working. Here's the output:
27/02/2024, 22:01:01 - [info] Ollama - Code Completion submitting request with body: {"model":"deepseek-coder:6.7b-base-q8_0","prompt":"<｜fim▁begin｜>\ndef isPrime(n):\n    \"\"\" Test if n is a prime number \"\"\"\n    for i <｜fim▁hole｜>\n\n        \n\n\ndef hello(i):\n    for j in range(i):\n        print(f\"{j} is a value\")        \n\n<｜fim▁end｜>","stream":false,"raw":true,"options":{"temperature":0.6,"num_predict":-1,"top_k":30,"top_p":0.2,"repeat_penalty":1.1,"stop":["<｜end▁of▁sentence｜>","<｜EOT｜>","\\n","</s>"]}}
27/02/2024, 22:01:01 - [error] Ollama - code completion request with model deepseek-coder:6.7b-base-q8_0 failed with the following error: AbortError: This operation was aborted
27/02/2024, 22:01:01 - [info] Ollama - Code Completion execution time: 0.298 seconds
27/02/2024, 22:01:02 - [info] Ollama - Code Completion submitting request with body: {"model":"deepseek-coder:6.7b-base-q8_0","prompt":"<｜fim▁begin｜>\ndef isPrime(n):\n    \"\"\" Test if n is a prime number \"\"\"\n    for i in<｜fim▁hole｜>\n\n        \n\n\ndef hello(i):\n    for j in range(i):\n        print(f\"{j} is a value\")        \n\n<｜fim▁end｜>","stream":false,"raw":true,"options":{"temperature":0.6,"num_predict":-1,"top_k":30,"top_p":0.2,"repeat_penalty":1.1,"stop":["<｜end▁of▁sentence｜>","<｜EOT｜>","\\n","</s>"]}}
27/02/2024, 22:01:02 - [error] Ollama - code completion request with model deepseek-coder:6.7b-base-q8_0 failed with the following error: AbortError: This operation was aborted
27/02/2024, 22:01:02 - [info] Ollama - Code Completion execution time: 0.143 seconds
27/02/2024, 22:01:02 - [info] Ollama - Code Completion submitting request with body: {"model":"deepseek-coder:6.7b-base-q8_0","prompt":"<｜fim▁begin｜>\ndef isPrime(n):\n    \"\"\" Test if n is a prime number \"\"\"\n    for i in <｜fim▁hole｜>\n\n        \n\n\ndef hello(i):\n    for j in range(i):\n        print(f\"{j} is a value\")        \n\n<｜fim▁end｜>","stream":false,"raw":true,"options":{"temperature":0.6,"num_predict":-1,"top_k":30,"top_p":0.2,"repeat_penalty":1.1,"stop":["<｜end▁of▁sentence｜>","<｜EOT｜>","\\n","</s>"]}}
27/02/2024, 22:01:21 - [error] Ollama - code completion request with model deepseek-coder:6.7b-base-q8_0 failed with the following error: AbortError: This operation was aborted
27/02/2024, 22:01:21 - [info] Ollama - Code Completion execution time: 18.563 seconds

I noticed "num_predict":-1 and checked Wingman config. Sure enough, Code max tokens is -1 by default. Made the value 100, closed vscode, opened it again and tried. This time:
27/02/2024, 22:16:51 - [info] Ollama - Code Completion submitting request with body: {"model":"deepseek-coder:6.7b-base-q8_0","prompt":"<｜fim▁begin｜>\ndef isPrime(n):\n    \"\"\" Test if n is a prime number \"\"\"\n    <｜fim▁hole｜>\n        \n\ndef hello(i):\n    for j in range(i):\n        print(f\"{j} is a value\")        \n\n<｜fim▁end｜>","stream":false,"raw":true,"options":{"temperature":0.6,"num_predict":100,"top_k":30,"top_p":0.2,"repeat_penalty":1.1,"stop":["<｜end▁of▁sentence｜>","<｜EOT｜>","\\n","</s>"]}}
27/02/2024, 22:16:51 - [error] Ollama - code completion request with model deepseek-coder:6.7b-base-q8_0 failed with the following error: AbortError: This operation was aborted
27/02/2024, 22:16:51 - [info] Ollama - Code Completion execution time: 0.112 seconds
27/02/2024, 22:16:52 - [info] Ollama - Code Completion submitting request with body: {"model":"deepseek-coder:6.7b-base-q8_0","prompt":"<｜fim▁begin｜>\ndef isPrime(n):\n    \"\"\" Test if n is a prime number \"\"\"\n    if <｜fim▁hole｜>\n        \n\ndef hello(i):\n    for j in range(i):\n        print(f\"{j} is a value\")        \n\n<｜fim▁end｜>","stream":false,"raw":true,"options":{"temperature":0.6,"num_predict":100,"top_k":30,"top_p":0.2,"repeat_penalty":1.1,"stop":["<｜end▁of▁sentence｜>","<｜EOT｜>","\\n","</s>"]}}
27/02/2024, 22:17:30 - [error] Ollama - code completion request with model deepseek-coder:6.7b-base-q8_0 failed with the following error: AbortError: This operation was aborted
27/02/2024, 22:17:30 - [info] Ollama - Code Completion execution time: 37.988 seconds

So what would I need to do to get code completions?
Few other humble suggestions:
Conservative use of CPU required:
To save on power, Users could prefer to have Wingman-AI query Ollama for a code completion only when using a key combination like perhaps Ctrl+i or something. If Wingman-AI is going to try generating completions everytime the User pauses, it consumes a huge amount of CPU power even when the User doesn't want it to. This not only adds up in terms of the electricity bill, it also puts the User's CPU fan under constant duress/wear. I understand that some people would prefer not having to press a key combo, so this could be a setting that Users could choose. Either to use a pause or a key combo.
Readme update required:
It'd help to update the readme to show users how they can view the output logs and show a screenshot of the wingman config tab, to show how easy it is to change settings. Also, most users won't know what the consequence of changing settings like the code context window, code max tokens, chat context window etc is. So it'd be nice to explain those, and also explain why there's a separate code model and a separate chat model.

Yes, I noticed the time difference when using Twinny. Since I'm on a CPU-only system, the larger models take time to respond. But the smaller models respond fast. This is the response time when using Twinny.

deepseek coder takes 18 seconds to generate a code completion.
codellama takes 36 seconds.
stable-code takes just 2 or 3 seconds.

So I switched to stable-code a few days ago, and Wingman-AI displayed an error message that it's not supported. I uninstalled Wingman-AI.
Today I reinstalled it, and it left me hanging (nothing loads. only the blue loading icon keeps moving horizontally):

I checked the output, and it shows the codeModel is stable-code:

But when I look at the settings, it's not stable-code. So I'm unable to switch back to deepseek coder.

I know I could solve it by clearing some cache, but that's not what matters here. The design of the extension needs to safeguard from such situations.
Few suggestions:

Please add support for smaller models.
You could have a look at Twinny's interface design. It makes it easy to switch models, and is reasonably robust. Even the settings are more accessible. There are some things that Twinny can learn from Wingman-AI too.

Thanks for the suggestions! I will log an issue related to the extension crashing with invalid configuration/non supported models.
and a second enhancement to switch code complete to some form of hotkey instead of a simple on or off

Covering in #27

Since we added issues to add the suggestions I'm closing this ticket.  For the hotkey I'll be releasing that soon so keep an eye out for that.


Text cut off on new user message

The Issue
See screenshot below. Not sure when this started to occur - possibly with some of the recent css/font changes?

Expected behaviour
format properly
Actual behaviour
text cut off
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Fixed in c33b7df7


cmd/compile: "" instead of "..." in errors with [...]T

What version of Go are you using (go version)?

$ go version
go version go1.15 linux/amd64

Does this issue reproduce with the latest release?
It doesn't reproduce with Go 1.13.  It does reproduce with Go 1.14, 1.15, and +721819e7ba.  Going to rebuild a more fresh Go, try the tip, and update the issue in a moment.
Update: also reproduces with:
$ go version
go version devel +5cc030aa19 Sun Sep 6 20:23:50 2020 +0000 linux/amd64
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ainar/.cache/go-build"
GOENV="/home/ainar/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/ainar/go/pkg/mod"
GONOPROXY="REDACTED"
GONOSUMDB="REDACTED"
GOOS="linux"
GOPATH="/home/ainar/go"
GOPRIVATE="REDACTED"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/ainar/go/go1.15"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/ainar/go/go1.15/pkg/tool/linux_amd64"
GCCGO="/usr/bin/gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build723473563=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://play.golang.org/p/rHDhklq4E7D
func f() [2]int {
	return [...]int{2: 0}
}

What did you expect to see?
In Go 1.13:
# command-line-arguments
go/tmp.go:4:17: cannot use [3]int literal (type [3]int) as type [2]int in return argument

What did you see instead?
# command-line-arguments
go/tmp.go:4:17: cannot use [<node DDD>]int literal (type [3]int) as type [2]int in return argument

I saw that on @davecheney's Twitter and tried looking up “node DDD” in existing issues.  There didn't seem to be one, so I filed this one.

I think this change is intentional, see this CL for mode details.

I’m pretty sure @mdempsky did not intend for  to show up in user-facing error messages.

Oops. Yeah, that error message is wrong.
It should say either [3]int literal or [...]int literal.

That was quick!  Now the message is:
# command-line-arguments
go/tmp.go:4:17: cannot use [...]int literal (type [3]int) as type [2]int in return argument
Thanks, everyone!


RiemannSurface instantiation hangs on Windows

This doesn't work on James' computer:
from abelfunctions import (RiemannSurface, RiemannTheta)
#                           RiemannConstantVector, puiseux)
from sympy.abc import x,y

f = x**2*y**3 - x**4 + 1
X = RiemannSurface(f,x,y)
g = X.genus()

omega = X.holomorphic_differentials()
print 'Differentials:'
for omegai in omega:
    print omegai
print 'genus:', g

This hang actually occurred on all OSes because of a bug in the way Sympy numerically approximates RootOf objects. This issue has been resolved in Sympy, though, most package managers have versions that don't contain this fix.


Add lorem files, modify data, change the format

Why before inspections department through?Submit this time?

Compiled automatically checks strictly, really helpless

Lorem is never localized ; don't add one or two elements in a data set. Instead, replace the entire dataset, but reference an external (and open-source) dataset.

Have changed, thank you for your prompt.Really want to do some contribution!

Thank you, very clear tell my mistake, I will carefully read contributors guide. I will also continue to contribute.


DockerImageOperations.BuildAsync - Throwing meaningful exception if docker image name is not lower case

Currently this scenario isn't properly handled which leads to the exception with low level message that doesn't give any clues what went wrong.

I wanted to add unit tests to this, but it will require more time than I can afford, The BuildAsync method invokes methods from static classes, so it will be necessary to create additional abstractions to make it mockable.

easier to test too. What do th

Cool, thanks. I'll do the review and merge later that day 👏.

I fixed the unit test, thanks (https://github.com/testcontainers/testcontainers-dotnet/commit/dcd1858046059478bb3cd1bca8acfb6a48d69854).


feat(snapshot creator): Syncing from snapshots

What ❔
Draft of snapshot creator with new API for fetching snapshots metadata and very simple snapshot creator binary. I did not want to make this PR too big, so there are some TODO items
Why ❔
Snapshot-based node syncing allows for a newly created node to initialize without having a recent full backup of the Postgres storage or creating a Merkle tree from scratch.
Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[ ] Code has been formatted via zk fmt and zk lint.

It's still WIP, though it works relatively well already:
2023-10-30T12:50:25.782786Z  INFO cross_external_nodes_checker::checker: No divergences found
One thing that I'm not super happy with is that with current implementation, I additionally fetch L1BatchWithMetadata for last L1 batch from main node.
Oh, and the query to fetch logs looks extremely ugly now as tree depends on providing keys in increasing order and to sort numbers encoded in little endian, you have to sort "from the end" like, so I ended with a query like:
ORDER BY ARRAY(
SELECT get_byte(storage_logs.hashed_key, 32 - generate_series)
FROM generate_series(1, 32)
)
But that almost certainly can be fixed by modifying tree recovery code

@tomg10

But that almost certainly can be fixed by modifying tree recovery code

Eh, I'm not entirely sure about this. Sure, a Merkle tree technically can be recovered with keys inserted in a random order (since we don't care about exact node version equality for the original and the recovered tree); the recovery will lose some nice properties (e.g., straightforward cursor for fault tolerance), but it seems manageable. OTOH, Merkle range proofs will only work for tree chunks if chunks, well, cover ranges of the tree keys, which IIUC won't work if we change key ordering, e.g. to non-reversed hashed_keys.
@RomanBrodetski @popzxc What's this reverse ordering of hashed_key bytes for the tree is caused by, btw? I guess we can't change it at this point, but was there a specific reason for it, or is it just a quirk?

I believe It's caused by the endianness, if a number is 0x0A0B0C, then in little-endian it will be represented in memory(and stored in postgress) as [0x0C, 0x0B, 0x0A], least signifficant number is stored at the smallest address

What's this reverse ordering of hashed_key bytes for the tree is caused by, btw? I guess we can't change it at this point, but was there a specific reason for it, or is it just a quirk?

@slowli Most likely, because the crypto team implemented it this way; and there most likely because it was more efficient proving-wise. There is also a chance that it just happened to be written this way, without a particularly strong reason, though. I agree that it's unlikely that we can change it now.

@tomg10 please resolve the comments that you already taken care of - will make it a little easier to review 🙏

@tomg10 you'll be opening other PRs right - shall we close this?


install pyrealsense error

Required Info

Camera Model
R200

Firmware Version

Operating System & Version
ubuntu 16.04

librealsense version

pyrealsense version

when i install pyrealsense
i use  sudo pip install pyrealsene
and i already put the pyrealsense into the pyton2.7
the result :
/usr/local/lib/python3.5/dist-packages/numpy/core/include/numpy/npy_1_7_deprecated_api.h:15:2: warning: #warning "Using deprecated NumPy API, disable it by " "#defining NPY_NO_DEPRECATED_API NPY_1_7_API_VERSION" [-Wcpp]
#warning "Using deprecated NumPy API, disable it by "
^
pyrealsense/rsutilwrapper.cpp:614:16: fatal error: rs.h: No such file or directory
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
Command "/usr/bin/python -u -c "import setuptools, tokenize;file='/tmp/pip-install-t6fgyL/pyrealsense/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-record-bTg0ac/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-t6fgyL/pyrealsense/
plz help me to install pyrealsense ,thx
and i also have another question
how to set up the environment  in windows10 :
Windows specifics: set environment variable PYRS_INCLUDES to the rs.h directory location and environment variable PYRS_LIBS to the librealsense binary location. You might also need to have stdint.h available in your path.
how to do that?

Just check if you have installed the librealsense on your computer. If so, then add the path of rs.h to the environmental variable, that is export PYRS_INCLUDES=path/to/rs, for me the path is /opt/ros/kinetic/include/librealsense/.


Add reward balance to transactions view

API endpoint for transactions now has a new value for the current rewards balance.  Adding a new column to display the rewards balance on the transaction page to make this displayable.

Not sure how to display this intuitively because the rewards are integrated into the global balance.  After adding a secondary column, it was overly confusing.


make eslint a real dependency / bump versions / rename deprecated rule

Before this PR, eslint (and babel-eslint, eslint-plugin-react) were devDependencies, which means they would not be installed into a different module via npm install. It required these modules to be installed separately, and all kinds of issues arose when the versions did not match with those assumed here.
Adding eslint-config-mongodb-js (this module) as a devDependency to a module should take care of all the eslint (and plugins) dependencies. It does now, by moving the required dependencies to the dependencies section.
Additional changes:

bumped versions to something current
renamed deprecated react/wrap-multilines to react/jsx-wrap-multilines rule

This change is

LGTM


Release/2.1.1

This PR bumps the version to 2.1.1
Release steps:

Inspecting the CHANGELOG.md to determine if the release was major, minor or patch. Make sure you align with the rest of the team first.
Use npm version --no-git-tag-version <major|minor|patch> to update package.json.
Update the CHANGELOG.md has been updated to show version and release date.
Add any @since X.Y.Z annotations if there have been any added or deprecated APIs.
Commit the changes as release: prepare vX.Y.Z on a branch named release/X.Y.Z
Create a PR, once CI passes, merge as a squash merge.
Create and publish the release via GitHub, this will create the tag and trigger the npm publish.

Just to confirm, before this release, solid-client-js was also released and updated here, right?

Absolutely: https://github.com/inrupt/solid-client-access-grants-js/commit/701292dfdbb54dd537002a631f93dbe7f464c43b#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519

Absolutely: 701292d#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519

Okay, before we do this release, I'd like to get the two fixes out in 1.25.1 that we need to land, that way we've only gotta release this once.


Change tests to accomodate non blocking regular sync

Superseded by https://github.com/matrix-org/complement/pull/588


Remove custom bits from Scalapack test

using /apps/common/UES/reframe/resources/scalapack/

@jgphpc Please open that internally. It is valid.

moving to https://jira.cscs.ch/browse/UES-1295