mTLS example from docs broken: bug with cert updates

Describe the bug
Following the TLS and mTLS portions of the docs here: https://preliminary.istio.io/docs/tasks/traffic-management/secure-ingress/#configure-a-mutual-tls-ingress-gateway
For mTLS, after creating the new secret and updating the Gateway rule, no changes are pushed to the ingress gateway.  The additional cert file has been mounted to as show by
$ kubectl exec -it -n istio-system $(kubectl -n istio-system get pods -l istio=ingressgateway -o jsonpath='{.items[0].metadata.name}') -- ls -al /etc/istio/ingressgateway-ca-certs
total 4
drwxrwxrwt 3 root root  100 Mar  1 22:01 .
drwxr-xr-x 1 root root 4096 Mar  1 22:01 ..
drwxr-xr-x 2 root root   60 Mar  1 22:01 ..2019_03_01_22_01_21.609145900
lrwxrwxrwx 1 root root   31 Mar  1 22:01 ..data -> ..2019_03_01_22_01_21.609145900
lrwxrwxrwx 1 root root   24 Mar  1 22:01 ca-chain.cert.pem -> ..data/ca-chain.cert.pem

There are no failed LDS updates in pilot/ingressgateway, it looks like pilot did not process the updated Gateway rule and therefore did not push the updated listener to the ingress gateway
Expected behavior
The curl from step 3 of mTLS section should fail after creating the new secret
Steps to reproduce the bug
https://preliminary.istio.io/docs/tasks/traffic-management/secure-ingress/#configure-a-mutual-tls-ingress-gateway
Version
images built from release-1.1 branch 2/28

tested with latest release candidate and verified issue has been resolved


Refactor Clipboard's GUI to behave like the Trash

We have this wonderful Drag'n'Drop Trash. We should refactor the Clipboard to act like the Trash.

Why not rethink the whole Clipboard and implement a user based clipboard model in the db.
The Clipboard could hold pages, elements and would be persistent.
Would be a nice feature.


Index column named 'Unnamed: 0' on .to_csv() after reading DataFrame with .read_cvs()

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.10
Modin version (modin.__version__): 0.9.1
Python version: 3.7.4
Code we can use to reproduce:

import numpy as np
import pandas as pd
import os
os.environ["MODIN_ENGINE"] = "ray"
import modin.pandas as pd2

df1 = pd.DataFrame(np.arange(12).reshape(4, 3), columns=list('abc'),
                   index=[f'i{i}' for i in range(1, 5)])
df2 = pd2.DataFrame(np.arange(12).reshape(4, 3), columns=list('abc'),
                    index=[f'i{i}' for i in range(1, 5)])

df1.to_csv('df1.csv')
df2.to_csv('df2.csv')

df1_re = pd.read_csv('df1.csv', index_col=0)
df2_re = pd2.read_csv('df2.csv', index_col=0)

df1_re.to_csv('df1_re.csv')
df2_re.to_csv('df2_re.csv')

This gives
$ head *re*csv
==> df1_re.csv <==
,a,b,c
i1,0,1,2
i2,3,4,5
i3,6,7,8
i4,9,10,11

==> df2_re.csv <==
Unnamed: 0,a,b,c
i1,0,1,2
i2,3,4,5
i3,6,7,8
i4,9,10,11

Describe the problem
After reading a CSV and writing it again the resulting file has Unnamed: 0 as header of the index column, whereas a file that has been written with pandas does not (compare df1_re.csv and df2_re.csv after running the example). This only occurs with Ray, while Dask gives UserWarning: `to_csv` defaulting to pandas implementation. and writes the file as expected.

Thanks @julibeg for the report!
cc @anmyachev

Some details.
After creating df2 by the constructor,  df2 has 1 partition:
    a   b   c
i1  0   1   2
i2  3   4   5
i3  6   7   8
i4  9  10  11

After creating df2_re by read_csv, df2_re has 2 partitions:
(Pdb) df2_re._query_compiler._modin_frame._partitions[0][0].to_pandas()
            a  b  c
Unnamed: 0
i1          0  1  2
i2          3  4  5
(Pdb) df2_re._query_compiler._modin_frame._partitions[1][0].to_pandas()
            a   b   c
Unnamed: 0
i3          6   7   8
i4          9  10  11

It turns out in some cases that a dataframe created from the same data but in different ways has a different internal representation. Need to understand why.


Revert "refactor(manager): output folder cleanup (#2083)"

This reverts commit ba59dd46a8885bb89ff2aced73502ea0c7b2e323.

This seems to be a big revert. But I'm OK if this is the only way to fix the issue.

The fix may be easy, but it's safest to rollback and debug later. Fixing forward is risky and we should really try to avoid it. Once reverted we can attempt this change again!


BS_roformer multistems

When using BS_roformer and mel_band_roformer models for inferencing, I found that only target_instrument is returned and the other stem is generated as (mixture - target_instrument).
I tried to change some codes and removed the "target_instrument", but it turned out that both stems were the shape of the target_instrument.
I noticed that from https://github.com/ZFTurbo/Music-Source-Separation-Training/blob/main/configs/config_musdb18_bs_roformer.yaml, in the config there are multi-stems and one target_instrument, however I didn't find the weight of the model and from you code in inference.py I guess that there will only be two stems output: voclas and instrumental. I'm curious that if that the model has the ability to return multi-stems: vocal, bass, drums and other, instead of the two according to the target_instrument. If it does, how to change the inference code?
Additionally, @SUC-DriverOld suggests me to use his aspiration model whose config's target_instrument is null, I found it works fine and did return two stems, whcih may suggest mel_band_rorformer (I guess BS_roformer too) can handle multi-stem cases. And I'm not sure if the pretrained models (such as model_bs_roformer_ep_368_sdr_12.9628, whose config's target_instrument is vocals) can have the same effect.

maybe this will help
https://bit.ly/3TCbXO6
Pass: changeme
If you don't have the c compliator, install it.(gcc or clang)

I think it was related issue:
https://github.com/ZFTurbo/Music-Source-Separation-Training/issues/56

For pre-trained model it will return only one stem until you train/finetune them with different config.

For pre-trained model it will return only one stem until you train/finetune them with different config.

I see. I find the pre-trained models only contain the weights of the target_instrument. So I think there's no way to get the other stem(s) directly from the pre-trained models. Thanks for your reply.


Upgrade to windows 0.51

This is still "draft" as the necessary "bingen" utilities were set to be published (https://github.com/microsoft/windows-rs/issues/2607, https://github.com/microsoft/windows-rs/pull/2609), but have not yet made it to crates.io (though they are not needed to consume this crate!).
Still on the fence whether to go the cargo install riddle; riddle --etc bindings.txt route, or whether to keep the cargo r -p api_gen crate now that windows-rs decided to keep a windows-bindgen library crate that we can instead give these arguments to [^1].
[^1]: Meaning that the riddle utility does not need to be separately installed, because running the api_gen crate through cargo automatically downloads and compiles dependencies.

Decided to stick with windows-bindgen to save on a manual cargo install riddle, and because it's not published yet, requiring an install from git (and cloning the windows-rs repo takes ages).


2023.07.06 Youtube - Video Not Available

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
All
Provide a description that is worded well enough to be understood
I translate the description of the problem through Google translator, so it may not be clear the first time.
When I try to get information about the video via yt-dlp embed, it says that the title of the video is - Video Not Available, and preview - https://i.imgur.com/JxUlEUA.jpg. But the video downloads without any problems
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://www.youtube.com/watch?v=WoWzAVNDH2M']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.10.6 (CPython x86_64 64bit) - Linux-5.4.0-105-generic-x86_64-with-glibc2.31 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[youtube] Extracting URL: https://www.youtube.com/watch?v=WoWzAVNDH2M
[youtube] WoWzAVNDH2M: Downloading webpage
WARNING: [youtube] unable to extract initial player response; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[youtube] WoWzAVNDH2M: Downloading ios player API JSON
WARNING: [youtube] Skipping player response from ios client (got player response for video "M5t4UHllkUM" instead of "WoWzAVNDH2M"); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[youtube] WoWzAVNDH2M: Downloading android player API JSON
WARNING: [youtube] Skipping player response from android client (got player response for video "M5t4UHllkUM" instead of "WoWzAVNDH2M"); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[youtube] WoWzAVNDH2M: Downloading iframe API JS
[youtube] WoWzAVNDH2M: Downloading player da7c2a60
[youtube] WoWzAVNDH2M: Downloading web player API JSON
WARNING: [youtube] Skipping player response from web client (got player response for video "M5t4UHllkUM" instead of "WoWzAVNDH2M"); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [youtube] unable to extract yt initial data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [youtube] Incomplete data received in embedded initial data; re-fetching using API.
[youtube] WoWzAVNDH2M: Downloading initial data API JSON
WARNING: Extractor failed to obtain "title". Creating a generic title instead
ERROR: [youtube] WoWzAVNDH2M: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "/home/eyesaver/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1560, in wrapper
    return func(self, *args, **kwargs)
  File "/home/eyesaver/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1709, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/home/eyesaver/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1768, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/home/eyesaver/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 2732, in process_video_result
    self.raise_no_formats(info_dict)
  File "/home/eyesaver/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1073, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [youtube] WoWzAVNDH2M: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

Can you try installing master branch and share a new log if the issue persists?

tengo el mismo problema utilizo los servidores de Hetzner  en Finlandia , pero tambien me sucedio con un servidor que tengo en Canada.

Can you try installing master branch and share a new log if the issue persists?

[debug] Command-line config: ['-vU', 'https://www.youtube.com/watch?v=WoWzAVNDH2M']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481]
[debug] Lazy loading extractors is disabled
[debug] Python 3.10.6 (CPython x86_64 64bit) - Linux-5.4.0-105-generic-x86_64-with-glibc2.31 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.07.22, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1863 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[youtube] Extracting URL: https://www.youtube.com/watch?v=WoWzAVNDH2M
[youtube] WoWzAVNDH2M: Downloading webpage
[youtube] WoWzAVNDH2M: Downloading ios player API JSON
WARNING: [youtube] Skipping player response from ios client (got player response for video "M5t4UHllkUM" instead of "WoWzAVNDH2M"); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[youtube] WoWzAVNDH2M: Downloading android player API JSON
WARNING: [youtube] Skipping player response from android client (got player response for video "M5t4UHllkUM" instead of "WoWzAVNDH2M"); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [youtube] Skipping player response from web client (got player response for video "M5t4UHllkUM" instead of "WoWzAVNDH2M"); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: Only images are available for download. use --list-formats to see them
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] WoWzAVNDH2M: Downloading 1 format(s): sb0
[debug] Invoking mhtml downloader on "https://i.ytimg.com/sb/M5t4UHllkUM/storyboard3_L2/M$M.jpg?sqp=-oaymwENSDfyq4qpAwVwAcABBqLzl_8DBgi867OjBg==&sigh=rs$AOn4CLD6zz6wUMPtwBuO9zjkyt-TR6iF_g"
[mhtml] Total fragments: 4
[download] Destination: Video Not Available [WoWzAVNDH2M].mhtml
[download] 100% of   20.07KiB in 00:00:00 at 58.00KiB/s

Video Not Available, what i was talking about

If you don't mind answering: In what region is the IP you are using? Is it a data center IP?

Many regions. With proxy. IP - datacenters

Norway
Italy
Portugal
Slovakia
Netherlands
Russia
Russia
Switzerland
Bulgaria
Spain
Latvia
Sweden
Poland
Moldova
France
Estonia
Turkey
Kazakhstan
Great Britain
Ukraine
USA

So installing master branch resolved the unable to extract initial player response error - confirming that was due to #7594
The issue that persists looks like it is #7143 minus the http error 429

So installing master branch resolved the unable to extract initial player response error - confirming that was due to #7594
The issue that persists looks like it is #7143 minus the http error 429, in which case the tl;dr is youtube has banned the IP range of your cloud host/proxy/vpn

I believe you are wrong. When blocking IP addresses, nothing would be downloaded.

But in my case, the video is downloaded, but with the name that I already said earlier.

Take a look at #7143 -- that is exactly what youtube is doing to the blocked IPs

But in my case, the video is downloaded, but with the name that I already said earlier.

That is not the video, but storyboards, since no other formats were found

WARNING: Only images are available for download.

But in my case, the video is downloaded, but with the name that I already said earlier.

That is not the video, but storyboards, since no other formats were found

WARNING: Only images are available for download.

It doesn't matter, it happens with any video.

In my bot the subtitles are downloaded, it means that the ip is not blocked. it must be a different error.

@fermani10 You are experiencing #7860

And as of ~2 days ago, Youtube has started with a new kind of IP blocking that only applies to the videoplayback endpoint (format/stream URLs), which results in the 403 errors

@fermani10Estás experimentando #7860

Y desde hace ~2 días, Youtube comenzó con un nuevo tipo de bloqueo de IP que solo se aplica al videoplaybackpunto final (URL de formato/transmisión) , lo que da como resultado los errores 403

I contracted another alternative ip but I still could not activate it on my server. I hope that solves it


Change Sink to be Algebraic of Appender and BufferedTextWriter.

Avoids creation of OutputRange object.
BufferedTextWriter is a helper struct that accumulates writes to avoid excessive system calls.

Update: The class layout is now Logger -> TypedLogger!Sink -> actual class. There's a limited set of supported sinks.
I'm really not sure how else this could possibly work with OOP.

Now the loggers are pre-sorted by sink.


Revert DragonFly BSD 4.8 Support in gbenchmark

These changes serve no real purpose in gbenchmark where there are no
differences between FREE BSD and DragonFly BSD.
I didn't realize that when I created PR #2573 "Update gbenchmark" and
merged in the gbenchmark changes from PR #2183. This reverts back to
just using BENCHMARK_OS_FREEBSD in lib/gbenchmark and thus a few less
differences between lib/gbenchmark and google/benchmark.

Note, the failiure in travis-ci is most likely unrelated to this change. I've create issue #2580 to track the problem.

The CI failure is indeed unrelated.


hmtl-loader 1.* compatibility

This add compatibility with html-loader 1.0+. Also, replaces html with more limited disruption of original code to avoid breaking other loaders that add code to html-loader (like file-loader).

Thanks @peterpoe !
I took some inspiration from your code, but decided on a slightly different approach that is backwards compatible and supports different variable names:
https://github.com/WearyMonkey/ngtemplate-loader/pull/88


Thats really weird.. and slightly scary.


"Așchileu Mare", "Așchileu Mic", "Cristorel", "Dorna", "Fodora", are listed in the drop-down menu

Description:
When user writes"Așchileu Mare", "Așchileu Mic", "Cristorel", "Dorna" and "Fodora" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Așchileu Mare"
Expected results
The location appears in the drop-down menu as "Sat Așchileu Mare, CLUJ (Așchileu)"

Step 3
Write in the search bar "Așchileu Mic"
Expected results
The location appears in the drop-down menu as "Sat Așchileu Mic, CLUJ (Așchileu)"

Step 4
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Cristorel"
Expected results
The location appears in the drop-down menu as "Sat Cristorel, CLUJ (Așchileu)"

Step 7
Write in the search bar "Dorna"
Expected results
The location appears in the drop-down menu as "Sat Dorna, CLUJ (Așchileu)"

Step 8
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 9
Write in the search bar "Fodora"
Expected results
The location appears in the drop-down menu as "Sat Fodora, CLUJ (Așchileu)"


GLB texture from PNG only shown with --raytracing

If I just open the GLB without any options no texture is shown. When enabling ray tracing it works as expected, but with on-board graphics on my notebook that's painfully slow. Is this a bug or am I doing something wrong? My model is a reconstruction by OpenMVS (353M scene_dense_mesh_texture.glb, 244M scene_dense_mesh_texture.png). Google's model viewer renders these just fine.
I'm on Arch linux with f3d 1.2.1.
Best, A.

can you share your data ?

Sure, you can download them here:
scene_dense_mesh_texture.png
scene_dense_mesh_texture.glb
If you're patient you can also see how model viewer is rendering this here:
aljoscha.geo.uni-potsdam.de

I confirm that the texture does not appear correctly, everything appears as white without raytracing.

The data is not available anymore. Please share it again @Rheinwalt

Thank you for looking at this again. I reactivated the links above, they should work again. For me the issue remains, also with my current version:
$ f3d --version
F3D 2.3.0
F3D - A fast and minimalist 3D viewer
Version: 2.3.0.
Build date: 2024-01-21 20:04:22.
Build system: Linux 64-bits.
Compiler: GNU 13.2.1.
External rendering module: ON.
Raytracing module: ON.
VTK version: 9.3.0.

Sorry, not closed.

Here is a link that will stay alive: https://drive.google.com/file/d/1ngJNyHeJhf6VjqkiDPsyPwXTWsujI5-v/view?usp=sharing
Issue is still present in F3D 2.3.0


Is there any way to implement list cards in bot composer?

Hi, I am trying to implement list cards in bot composer as we are having this type of card implementation in MS teams, is there any way to implement list cards in bot composer?
List card provides functionality like scroll bar, etc. is there any alternative for similar to list card?
Sample List Card
{
"contentType": "application/vnd.microsoft.teams.card.list",
"content": {
"title": "Manage your sevice desk related activities here.",
"items": [
{
"type": "resultItem",
"title": "Incident",
"subtitle": "",
"tap": {
"type": "imBack",
"value": "Incident"
},
"icon": "{{url}}"
},
{
"type": "resultItem",
"title": "Service Request",
"subtitle": "",
"tap": {
"type": "imBack",
"value": "Service Request"
},
"icon": "{{url}}"
},
{
"type": "resultItem",
"title": "Home ",
"subtitle": "",
"tap": {
"type": "imBack",
"value": "Home"
},
"icon": "{{url}}"
}
]
}
}

List card with scroll bar, image and title

Our requirement is:
1)List card should contain items like title and image
2)Action should happen when we click on the item
3)Need scroll bar if the items are more than the adaptive card height

Closing as resolved.


Villager Trading Menu

Currently there is no way to scroll down in the menus for villager trading. I'd expect it to be right stick, but nothing happened. All my controls are default.

Fixed in 0.16.5


[HSDK-371] Added Huma Onboarding Android docs

https://humatherapeutics.atlassian.net/browse/HSDK-371
Added Huma Onboarding Android docs

@valeriy-huma @andriib-huma Please check this PR its failing


Regex does not match when using a complex capture group

The default regex is (\d+) which will match MyProject.MyNamespace.MyTest.MyTest_123.
However, if your test project or namespace contain numbers, this will not match correctly. e.g. MyProject1.MyNamespace1.MyTest1.MyTest_123
If the test is renamed to TestCase123_MyTest and the regex pattern TestCase(\d+) is used, the regex does not match.

released in 0.1.16


Unbreak master

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Fixes merge conflicts from #269
Tests
Tested locally

What's the post-mortem for this? How was it broken, and is there a way we can stop it happening again?

Rough timeline

#269 was mostly adding new code, it had passing tests on CI
#247 was merged, changed some variable names
#269 had no merge conflicts but used the variable names that were changed
I merged #269
Checked out master, it was broken

I knew #247 was in and had changed a bunch of things, but I saw "no merge conflicts" so I thought "it should be fine". After the merge, checked out master, it was not fine.
There are ways from preventing it happening:
What I should have done: merge master into #269, run tests then merge into master. This is what we usually do after big changes anyway.
Some other ways - at various levels of overkill and negative side effects, could be considered more once we're gold:

have robots check if master is ahead of branch and prevent merging unless branch is ahead of master
have merges be done automatically by robots that check various things e.g. things compile - basically circle++
merge into a staging branch and e.g. nightly tests transition staging to master


fixed calendar, issue 1696, ics file start date is not date type

Fixes issue #1696
Symptom: calendar does not sync. But has not relation to authentication. The start date was simply not a date so toISOString() failed.
Same fix applied to exdates[i]
Tested with this calendar:
personal.zip
On the other hand might be that the code above, L33-38, failed, unfortunately I can't debug yet:
    if (curr.start.length === 8) {
        var comps = /^(\d{4})(\d{2})(\d{2})$/.exec(curr.start);
        if (comps) {
         curr.start = new Date (comps[1], comps[2] - 1, comps[3]);
        }
      }

Thanks.
Unfortunately the build has failed due to lifting errors of the changelog. Are you able to fix this?

Sure. I didn't notice, that you even have an integration build set up. That's great.

Thanks!

My PR #1701 (that I was writing just as this was merged) also updates node-ical.js, but uses the "official version" which I believe offers equivalent levels of error handling, but also fixes several other calendar-related issues.
I resolved the conflict on my PR by simply using peterbraden's updated version, but want to make sure that I'm not missing anything or stepping on any toes. Thanks!

My PR #1701 (that I was writing just as this was merged) also updates node-ical.js, but uses the "official version" which I believe offers equivalent levels of error handling, but also fixes several other calendar-related issues.
I resolved the conflict on my PR by simply using peterbraden's updated version, but want to make sure that I'm not missing anything or stepping on any toes. Thanks!

Indeed, looks much better. I just wonder what happened to curr.exdates, isn't that previous logic required anymore?

My read on why the exdates logic is no longer needed is this (but as with all things, especially date-related things, I heartily endorse a decent probability I'm off-base):
These objectHandler overrides at the end of node-ical.js are cleaning up / smoothing over some rough edges in ical inputs, specifically removing any milliseconds (. followed by 3 digits) added on to dates in DTSTART and EXDATE fields.
(Potential sidebar): Because the DTSTART and EXDATE dates are used as indexes in the resultant parsed ical object, being overly precise isn't particularly helpful and potentially a source of strange bugs. And I am fairly certain there isn't a use case for events repeating more than once a second.
As to why EXDATEs no longer need to be cleaned up: There was a recent-ish change in ical.js that stores EXDATE date keys as just the date part of the event, all hour/minute/second delineations are dropped. This means events that repeat more than once a day can't both have exceptions (and/or exceptions might get applied to the wrong one of the repeating events on that EXDATE??), but I don't think that events repeating more than once a day is at all common, as far as I know (at least for me, neither Fantastical nor Apple's calendar would allow a frequency smaller than daily). So, since all time information is dropped anyways, no need to partially clean it up beforehand.

@mattdb I just checked the ical project and it looks not that active. There are open pull requests of 2013! I'm not that sure if it's a good idea to rely on the "official repo". The vows unit tests of master don't work either: "callback not fired".


feat(staking): add BTC support

Mofidied the contract to work with EVM and Bitcoin.

@andresaiello can you, please, take a look and see if this look right?

Will merge after I check it out on the real testnet.

@SocketSecurity ignore<EMAIL_ADDRESS>

Thanks for all the comments, @lumtis! Really appreciate it!

Thanks for reviewing, @andresaiello & @lumtis!


Add support for styled-components

styled-components is a library for styling React applications that's fairly new (a few months old) and already quite widely used in the community.
A simplified example of its usage looks like this:
const Box = styled.div`
  color: palevioletred;
  background: papayawhip;

  @media screen and (max-width: 500px) {
    color: papayawhip;
    background: mediumseagreen;
  }

  &:hover,
  &:focus {
    color: red;
  }

  > h1 {
    color: palevioletred;
  }
`

As you can see, you can write actual CSS in JavaScript. The issue, as you can also see, is that there's no syntax highlighting, since there's no native support from syntax highlighters. (there's now support in Atom, Sublime and Vim, and other editors are being worked on)
It would be awesome if highlight.js supported styled-components. I use highlight.js heavily on my website (and I know a lot of other people do too, thanks for making an awesome project!) and having styled-components support would be huge. In case you can see this as something worthwhile having let me know and I'll lay out the specifics of what's needed. (the API surface is a tiny bit bigger than just styled.x, but not much)

In case you want to know about the benefits of this, admittedly slightly unusual, approach see this article in SmashingMag and this talk from ReactiveConf. If you want to see a comparison with all other CSS in JS frameworks, see this talk from ReactNL. (note: this is where styled-components was released, its first mention happens at ~25 minutes. "One more thing...")

I can see that there is still no support for styled-components, or at least this is not working for me

I think for something like this the React community may have to contribute a React grammar much the same as we have the VueJS grammar for Vue.js.  I'm sure we'd be open to reviewing a PR for this, but long-term it might be better to split JSX out into it's own thing.  Grammars with their own maintainers (or group of maintainers) are going to be easier to keep up-to-date with the latest trends like this I think.
Ref: https://github.com/highlightjs/highlightjs-vue
@marcoscaceres Any thoughts on this one?

Actually if this is just the ES6 custom string literal stuff then we definitely should support it in our Javascript language.

As you can see, you can write actual CSS in JavaScript.

Well, you can actually write anything in JS as long as it's enclosed in a string, which is all that is happening here.
Actually can you be 100% clear what you are requesting here?  We already highlight template literals as strings... so the first example you gave should work (and does in my testing).  Of course the internal content is JUST a string, it's not going to get magic highlighting as anything else (css, HTML, etc)...
That's not a part of Javascript, so wouldn't be something we'd add to the JS language highlighter.
The second example looks to be a known issue with our JSX support, and not related to styled components specifically (that I can see).

Closing the original request as out of scope and the follow-up as a duplicate of https://github.com/highlightjs/highlight.js/issues/1915.
Example showing we do highlight string templates:
https://jsfiddle.net/ajoshguy/ocq8sad9/

Actually I just leaned we do highlight HTML and CSS if you use html or css as the prefix:
  var HTML_TEMPLATE = {
    begin: 'html`', end: '',
    starts: {
      end: '`', returnEnd: false,
      contains: [
        hljs.BACKSLASH_ESCAPE,
        SUBST
      ],
      subLanguage: 'xml',
    }
  };


iOS accessibility focus not returned to textfields after keyboard closed when other widgets are present

Steps to Reproduce

Enable VoiceOver on iOS
Open flutter gallery app and navigate to material > text fields
Tap on any textfield, double tap to start editing
Dismiss the keyboard with done
The accessibility focus does not return to the edited textfield, but move to other widget and start to read (if there's something to read).

e.g
Completes editing name textfield -> starts read Email textfield
Completes editing password textfield -> accessibility focus goes to submit button
This really causes a mess and makes it very difficult to complete multiple text inputs for VoiceOver users. But everything works fine on Android.
Logs
[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Mac OS X 10.14.5 18F132, locale en-TW)
    • Flutter version 1.7.8+hotfix.3
    • Framework revision b712a172f9 (2 weeks ago), 2019-07-09 13:14:38 -0700
    • Engine revision 54ad777fd2
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/fiona/Dev/android_sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • CocoaPods version 1.7.1

[✓] iOS tools - develop for iOS devices
    • ios-deploy 1.9.4

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 36.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.36.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.2.0

[✓] Connected device (1 available)
    • iphone X  • ios           • iOS 12.2

Code Sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.spaceAround,
          children: [
            TextField(),
            Semantics(
              label: 'Flutter Logo',
              child: FlutterLogo(
                size: 100,
              ),
            )
          ],
        ),
      ),
      floatingActionButton:
          FloatingActionButton(child: Icon(Icons.add), onPressed: () {}),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-7.1.pre, on Mac OS X 10.15.6 19G73, locale
    en-GB)
    • Flutter version 1.20.0-7.1.pre at /Users/taha/Code/flutter_dev
    • Framework revision 7736f3bc90 (6 days ago), 2020-07-10 16:33:05 -0700
    • Engine revision d48085141c
    • Dart version 2.9.0 (build 2.9.0-21.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google
      Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (5 available)
    • SM M305F (mobile)      • 32003c30dc19668f          • android-arm64  •
      Android 10 (API 29)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            •
      iOS 13.6
    • macOS (desktop)        • macos                     • darwin-x64     •
      Mac OS X 10.15.6 19G73
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 83.0.4103.116

• No issues found!

+1
This also creates an accessibility issue when using SearchDelegate. When the search is submitted from the keyboard, accessibility focus is placed at the end of the page. Depending on how long the search results build takes, focus is either placed at the very bottom of the page or in the middle of the results list.

This issue is reproducible on the latest stable and master channels with sample code at https://github.com/flutter/flutter/issues/36910#issuecomment-659409793.

Demo
https://user-images.githubusercontent.com/104349824/203755105-8d8dc202-84b2-4b36-b1fb-bc80a43c6f09.MP4

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.3.9, on macOS 13.0 22A380 darwin-x64, locale en-VN)
    • Flutter version 3.3.9 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (31 hours ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Community Edition (version 2022.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.4167.21

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.73.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.52.0

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 12 (API 31)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 13.0 22A380 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 107.0.5304.110

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[!] Flutter (Channel master, 3.6.0-7.0.pre.45, on macOS 13.0 22A380 darwin-x64, locale en-VN)
    • Flutter version 3.6.0-7.0.pre.45 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d0352f55db (3 hours ago), 2022-11-23 20:01:26 -0500
    • Engine revision 9b2895724c
    • Dart version 2.19.0 (build 2.19.0-424.0.dev)
    • DevTools version 2.19.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Community Edition (version 2022.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.4167.21

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.73.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.52.0

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 12 (API 31)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 13.0 22A380 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 107.0.5304.110

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Need to verify the native behavior, but it seems like jumping to a random botton on the screen after keyboard close is not good.


Software/Hardware Trigger option for each device

FEATURE:
Is your feature request related to a problem? Please describe.

We are aware that we can use either software or hardware trigger for many devices of interest. Currently, we don't have a systematic way to address trigger source selection.
Describe the solution you'd like

I would propose having a base class property that chooses the trigger type and all device adapter implementations can use a unified API to adjust their behavior according to trigger type. This can also have a counter-part on GUI, where we can automatically show a trigger source field for devices.
Describe alternatives you've considered

Alternative could be having independent trigger source selection APIs for each device adaptor class but that would likely lead to inconsistencies.
This discussion might be related to #136 .

I think this depends on how you want to drive the hardware that supports triggering, but I believe we might benefit from a property class that sets or retrieves information about the triggering so that this can be handled by a main timing controller. For a coordinated synchronized triggering, the timing controller needs information about the devices like the type of input/output (i.e digital, analog, voltage ranges), delays (i.e LC switching times, stage settling time, trigger response time, etc), and if the device is ready signals. It would be helpful to know these parameters to automate the sequence of pulses that should be generated.
However, from previous experience, having each device set to trigger mode has worked well considering most device types (i.e cameras, stages, mirrors, etc) have shared triggering functionalities.  Since there is quite a lot of overlap between devices, there is also a benefit to having these properties implemented in the device abstract class, and perhaps, what has to be done is to ensure that the new devices that support triggering implement such functionality.


[ESRGAN] Setup Second Phase Training

Setup training of the model (initialized with weights from 1st phase) using combined loss.

Resolved in: https://github.com/captain-pool/GSOC/commit/d83d4e4086fd462386e21fa9c90408c54bcfc67f


Adnuntius Bid Adapter: Removed native from Adnuntius

This is a pull request that removes that Adnuntius supports native as per this PR.
https://github.com/prebid/Prebid.js/pull/9458
🏷 Type of documentation

[ ] new bid adapter
[ ] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[x] bugfix (code examples)
[ ] new examples

📋 Checklist

[ ] Related pull requests in prebid.js or server are linked
[ ] For new adapters check submitting your adapter docs

@mikael-lundin this PR includes no changes :sweat_smile: . Not sure how you managed to do this

Closing this PR. Will open a new one to remove native from adnuntius


[feature request] get python.pythonPath from cwd rather than system wide.

Environment data
VS Code version:  1.8.1
Python Extension version: 0.8.0
Python Version:  3.6.3
OS and version:  macOS 10.13.2
Actual behavior
use system python if I leave python.pythonPath blank
Expected  behavior
find python at current path

use python set py pyenv
I can set .python-version in some parent-folder

Steps to reproduce:

pyenv local <some-version> at ~/dev
open ~/dev/project1 as a project
current python.pythonPath is the system-wide, not read from current folder.

Not quite sure what you are after here as I don't use pyenv.  Or do you not want the changes made by pyenv to be detected?

@brettcannon Thanks, this plugin doesn't pick-up the python-version which pyenv customized for current folder, it will use the version set for the system.

pyenv is a tool that replaces python and installed python tools (pylint, flake8, etc.) with symbolic links and routes requests to different commands based on global/local configuration.
It is used as an alternative to virtualenvs and is one of the more common ways of managing python versions on OSX.
So, for example:
pyenv install 2.7.14
pyenv install 3.5.0
pyenv install 3.6.3
pyenv virtualenv 3.5.0 somepython
pyenv global 3.6.3
which python  # /Users/waksman/.pyenv/shims/python
python --version  # Python 3.6.3
pyenv which python  # /Users/waksman/.pyenv/versions/3.6.3/bin/python
mkdir -P two/subdir threefive/subdir

cd two
pyenv local 2.7.14
# Inside two/** python, pip, etc. will point to pyenv 2.7.14
python --version  # Python 2.7.14
pyenv which python  # /Users/waksman/.pyenv/versions/2.7.14/bin/python
cat .python-version  # 2.7.14
cd subdir
python --version  # Python 2.7.14
cd ../..

cd threefive
pyenv local somepython
# Inside threefive/** python, pip, etc. will point to pyenv somepython, which is 3.5.0 but with package installs specific to somepython
python --version  # Python 3.5.0
pyenv which python  # /Users/waksman/.pyenv/versions/somepython/bin/python
cat .python-version  # somepython
cd subdir
python --version  # Python 3.5.0
cd ../..

The crux of it is that executing a command from a given folder will result in pyenv routing it to the correct environment (assuming bash_profile/bashrc has been run and set up pyenv).
So, the ask (I think) is that when linting/executing/whatever a given source file, execute the command from the location of that file. Alternatively, the interpreter could be set by using pyenv which python for any given path.

@gwax Thanks @brettcannon I find that this plugin now works as I expected. So this issue can be closed.

I continue to have trouble with this, requiring me to manually select interpreter whenever I look at a different workspace and would appreciate if this issue could be re-opened.

I assume you're executing pyenv before you launch VS Code from the shell? I.e. your example is missing a code . command or something?

I am not opening VS Code from the terminal, I am opening it via a shortcut. I am also using a multi-root workspace and different root folders have different pyenv environments. On top of that, I occasionally have single projects that
My examples were to illustrate the behavior of pyenv.
Essentially, pyenv knows how to route to the appropriate virtualenv and it would be great if the interpreter could be taken from pyenv instead of having to be specified manually. After pyenv init (usually in .bash_profile or .bashrc), running pyenv which python from the same folder as an open file yields the full path to the expected interpreter.

For prioritization context: pyenv is (to my knowledge) on of the more common ways of managing Python environments on OSX; and, it is possible to work around this by manually associating the appropriate interpreter with the project root.

So the issue is that pyenv mutates your environment, so if you're aren't launching code from that same shell that pyenv mutated we won't pick up what version of Python you chose implicitly (unless you create a virtual environment in the project which we will pick up).
But we could bake in explicit support for pyenv. That would mean using pyenv root to notice what versions of Python are installed, pyenv global to detect when a global version of Python has been set, and pyenv local to pick up a local specification of a Python version.

Yeah, and problematically, the environment mutation is working directory dependent so even if I launch code from a given folder, if I open a file from a different folder, that folder's pyenv might be entirely different.
Explicit pyenv support seems reasonable and it's probably a good way to avoid having to pull pyenv setup from an enclosing environment (running code from a terminal or sourcing .bash_profile).

Same issue here, and it's currently preventing me from using VSCode as my Python IDE since I always use pyenv and virtualenvs (so instead I'm using Spacemacs, which handles all this just fine). More specifically, I don't just use pyenv but pyenv with the pyenv-virtualenv plugin. So you have installed pyenv versions, which VSCode picks up as valid interpreters in its list (though not automatically the way pyenv would do it), but then you have virtualenvs under each version. pyenv handles this by using shims. The shims are first in the path so you always go through them to get to pyenv, which then handles interpreting your environment to find the right version and location. I tried setting my python.pythonPath to /usr/local/opt/pyenv/shims/python but VSCode always assumes I'm in pyenv's "global" state, which means that when pyenv is doing whatever it does to settle on a version, it lacks sufficient information inside of VSCode to do anything but settle on the default global setting for pyenv. What it needs to do (not just for the interpreter but for all executables like pylint) give pyenv all the information it needs to do the right thing, which probably just means calling the shim as if it were being called from the folder the open file is in.

Yes, pyenv does mutate the environment but I don't think that mutation is essential to how it works (though I could be wrong). Instead, as I said, it works through a shim executable (which you could set in your python.pythonPath) and when that shim is executed, it goes through a four step process that depends on the directory you're in, particularly steps two and three. This also raises the question of why VSCode doesn't just automatically choose the first interpreter on your path rather than searching a pre-specified list of paths (that don't happen to include $(pyenv root)/shims). That way there would be no need to even touch the user settings out of the box.

Having lived with this a bit longer, I find it's a lot less difficult to work around than I previously thought but there are a few minor issues that make things annoying and some other possible nice to haves.
pyenv, pylint, flake8, and VSCode works perfectly well together if you set the appropriate interpreter (Python: Select Interpreter). If you're using pyenv all of your python versions and pyenv-virtualenv virtual environments will show up on the list (with a caveat detailed below). All of this works without touching python.pythonPath.
Annoyances and nice to haves:

If you add a new pyenv interpreter, it won't show up in VSCode until you restart the application
I have to set an interpreter for every one of my project folders (which creates a .vscode folder in each)

it would be great if this was autodetected from pyenv or the .python-version file

interpreter can only be set at the project level

sometimes I have single projects that use different Python versions in project subfolders

See https://github.com/Microsoft/vscode-python/issues/1167 for tracking the request to add support for .python-version to auto-select the interpreter. And I think these issues are now duplicates of each other so I'm going to close this one and move the discussion over.

@gwax All of mine aren't showing up, so I'm curious how you accomplished (or lucked into) that. I'd be fine with most of the issues you mention if I at least got a complete list, but look at this:
* system (set by /usr/local/opt/pyenv/version)
  3.6.5
  /usr/local/Cellar/pyenv/1.2.3/versions/3.6.5/envs/NDF
  /usr/local/Cellar/pyenv/1.2.3/versions/3.6.5/envs/PonyGE2
  /usr/local/Cellar/pyenv/1.2.3/versions/3.6.5/envs/nrf
  NDF
  PonyGE2
  nrf

Now:
Screenshot
Notice that the PonyGE2 and nrf venvs are nowhere to be seen. It's almost like it's taking the first one in order under the 3.6.5 parent and ignoring the rest.

@neverfox I see the same behavior. I created a pyenv virtualenv and set that to global. vscode does see that as an interpreter. However, any other virtualenvs do not show up.
At present I use the Select Interpreter to reference the exact python version for the workspace/directory which does work. Will start tracking the other open issue for completion.

@gadams999 Thanks for confirming. Sounds like this should be its own issue.

@gwax I'm actually really curious how that's able to do the trick for the other executables like pylint, flake8, etc since you're not tell the extension about where to find the right copy like you are with the interpreter. Perhaps it works because the extension sets something in the environment that effects the shims for them to be directed appropriately?


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
a66adc9 -> 0809302

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.21.7 -> v1.23.0

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.23.0
Compare Source
Features

added json comparison functions (#593) (1781c81)

v1.22.0
Compare Source
Features

cbr cleanup (#658) (7721cbe)

v1.21.10
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.50.1 (#651) (df3e66d)

v1.21.9
Compare Source
Bug Fixes

deps: update gomod (#650) (1835de4)

v1.21.8
Compare Source
Bug Fixes

fixes bug in workspace path when using upgrade path and implicit destroys.* Fixed a bug where the temp directory could be created in the wrong place (did not affect the functionality of the code).* Removed unused internal variable. (#648) (71af06a)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.4.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


⚠ SceneTime has degraded performance

In 3448ad4, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1225 ms

Resolved: SceneTime performance has improved in 94bdd23 after 6 minutes.


Access to IModelParamsExtensions

Can we set this class as public, need access to this method for building multiple contexts outside LLamaSharp
I stood up a wee demo of the web UI too :)
https://llamaweb.chainstack.nz/

Hopefully it won't be needed public long term, but for now this makes sense 👍

No it wont, once LLamaSharp have multiple context support it can be all locked down
I think @AsakusaRinne was looking at supporting this via the executor classes, that is a way to big of a change for a non project owner, so I have created a shim project to enable multiple context so I can continue with the UI projects as they all rely on more than 1 context

I'm playing around with some ideas for how best to expose it as well. At the moment I have a (half built) demo scene with the LLM talking to itself using 2 contexts. My plan is to build that out and improve whatever APIs I need to do it.
Do you have a link to your shim project? Might provide me some helpful pointers on how to achieve it :)

My shim is just the implemetation deom the PR I made las week
I just splits LLamaModel into LLamaModel + LLamaContext
The shim project is LLamaHub  it just copies a bunch of classes accross to allow the minimum I need to continue
Its pretty much tthe same implemetation as PR 76

The demo is very cool by the way! Just checked it out.

Each session is its own context, so you can open multple tabs, set differnt parameters and compare the output
way easier than using the commanline, thgere is also a Tag package there if you want to run it locally


Add LICENSE

The license of CLAIRE is found in the main README-document, which is generally speaking a bad idea. As previous CLAIRE iterations are licensed under a permissive license, I recommend the Apache license.

Hello Mirko,
it's been a while since I last looked at the issue :).
My previous understanding was that there is a difference between:

the copyright, which mostly is there to prevent someone from blocking me to use CLAIRE
the license to use, which is granted to everyone - use and redistribution - in the README.

I'll be happy to switch to something more modern/liberal. I tried to look into GPL but I got confused and did not have the time to pursue.
If you have a reference about how to use the Apache licence (i.e. what to put where, README, header, ...), please let me know,
Best regards,
Yves

I agree with you, the situation is quite complicated.
As you are not using third party code (as far as I see), the thing is quite easy for you. Generally speaking, most developers add a LICENSE file to the code. The text for the Apache licence can be found here: https://apache.org/licenses/LICENSE-2.0.txt. It is a bit longer than e.g., the BSD-licenses, but it protects users and developers from the misuse of patent claims, as it grants everyone the right to use a patent if the code is patented.
However, unlike the GPL, it allows the CLAIRE compiler to be added to a proprietary product, like another CLAIRE compiler with some added features. No matter if you use the GPL or Apache, you can still develop and run proprietary programs with the help of the interpreter. If you are considering to use the GPL, things might get a bit tricky for the compiler part of CLAIRE, as AFAIK, compilers insert parts of their own bytecode into the produced binary. Therefore, you need to add a so-called compiler exception, so you can compile non-free code with the CLAIRE compiler. This was done for the GCC. An example of this exception (that needs to be adapted for CLAIRE is here: https://gcc.gnu.org/onlinedocs/libstdc++/manual/license.html)
I would like to add that I am no lawyer and have no experience in developing compilers, I am just a computer science student.
I hope this is helpful. If you need more help concerning the GPL, the legal team of the FSF is quite helpful. See here: https://www.fsf.org/licensing/
I am also happy to help.

Many thanks for the help !
I have added a LICENCE.txt to the repository, under the Apache licence.
This states much more nicely the rights to use and redistribute.

I think that everything is fine now.
I'll close the issue.


body from file

fix #16
with option start with @
$ cat mocks/create.json
{
 "mock": "data"
}

$ node index.js --options<EMAIL_ADDRESS>

Thank for this. I'll test it out and merge it soon.

@jayair how soon? 🙃 would love this!

@patrykkrawczyk Thanks for the bump. I'll have a look at it in the next few days.

Pushed!


Document doesn't apply the update

Nothing in this document details actually apply the update from Azure security Center.

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

ID: e2480656-ebe3-d20a-4c9f-ae9ed1c483c1
Version Independent ID: dca55042-e02d-f48f-24cb-d4691bed1502
Content: Apply system updates in Azure Security Center
Content Source: articles/security-center/security-center-apply-system-updates.md
Service: security-center
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

@Placid7
Thanks for your feedback! We will investigate and update as appropriate.

Hi @Placid7 ,
That information is provided in the links under the doc. These updates are on the operating system level and when you click into the message, the portal itself provides instructions for how to do the updates.
See the screenshot in this article: https://docs.microsoft.com/en-us/azure/security-center/security-center-update-os-version
CC: @rkarlin

Hope this helps!
We will proceed to close this thread but if you have further questions feel free to tag me in the comments or open a new issue.

Ok, but the title of the document doesn't make sense then. You aren't using Azure Security Center to update the VMs. You are using Azure Security Center to monitor VMs that need to be updated.


Error messages are vague when types have the same name

Consider the following:
module ts {
    export interface Node {
        kind: SyntaxKind;
        pos: number;
        end: number;
    }
}

function f(n: Node) {
    ts.forEachChild(n, () => { /* ... */ });
}

The subtlety is that the user meant to declare ts.Node as the type of n.
Here you'll get the error on n within the forEachChild call, with the message
TS2345 Argument of type 'Node' is not assignable to parameter of type 'Node'.

which is unhelpful because it makes the user feel like they've completely lost their mind.
We should use a qualified name in such cases.

@DanielRosenwasser /@chrisbubernak,
I ran into this gem today using 1.4:

I mean Types of parameters 'other' and 'other' are incompatible might be a candidate for the error messages hall of fame? :smile:

That error looks like a bug. Do you have a repro?

@RyanCavanaugh, there's only one DomElement. While implementing the interface there was a parameter clash with a type in lib.d.ts. Seems to be in the same ball park as this issue, but will take a look at producing a repro.

We're almost there; #1798 is probably the next pain-point to tackle.


Test effect of noise on Raykar (toy data)

Simulate a labelling task with different alpha and beta values. When changing alpha, fix beta = 1, and vice versa. Plot estimated alpha against true alpha and balanced accuracy, and repeat for beta.

(Considering using a filled-between line plot for these)

Same data, filled-between line plot with fill representing standard deviation. I think this is clearer for the bottom plots, but I'm not sure about the top ones.


Added support for viewport units

fix ben-eb/cssnano#12
Can you give me access for fast responsiveness?

:+1:, although, I would prefer to use the npm version command to tag the release. So if you can rollback that version number change I'll tag a release after that.


Semaphore::create_binary can't be used

Quoting the docs:
[...] binary semaphores created using xSemaphoreCreateBinary() are created in a state such that the the semaphore must first be 'given' before it can be 'taken'

The Semaphore API only allows locking, and then dropping the guard, which means there is no way to "give" a binary semaphore. This API also makes using Semaphores as synchronization tools unwieldy.

Good point. PR is welcome. Since I'm not maintaining any active code using this repo there is not much activity from my side, but any help is welcome.

One solution would be to directly implement give and take, per the FreeRTOS API, which does not hand back a SemaphoreGuard. lock, which does, can still exist, too. This would also be "familiar" for people familiar with FreeRTOS.
Even looking at the std::sync::Mutex, doing a directly unlock requires unstable features to call drop directly:
https://doc.rust-lang.org/src/std/sync/mutex.rs.html#337-339
If you really wanted completeness, some properly named version of new_binary like new_binary_ready could be implemented to just call the shim function to give it at the time of creation.

This issue seems to have been fixed in #46 in exactly this way:

One solution would be to directly implement give and take, per the FreeRTOS API, which does not hand back a SemaphoreGuard.


Align Makefile- kubectl or oc

In the process of moving FAR to Medik8s organization we align the Makefile - fourth PR.
Use kubectl and fallback to oc.

lgtm


something wrong in batchnorm between ctx.is_train in false and true

I use lstm to train and test, use "cpu only"
I set BatchNormPara with:
eps: 0.001
momentum: 0.9
fix_gamma: false
use_global_state: false
In train phrase, all are good; but in test state,
when use exec->Forward(false),  I get result with some data -nan(0x400000)
but use exec->Forward(true), all are good;
I check the code, and find that in batch norm phase,  when ctx.is_train == false, it just assign out with "moving_var" and "moving_mean";
and i check the data in "out" after assign, some part is -nan(0x400000) when the data in "data" and "slope(gamma)" and "bias(beta)" are good.
why this?  why not calc var and mean when ctx.is_train == false?

In bn we need to use the global stats from training phase in testing according to the original paper.
Could you check that in the saved model the BN stats (namely, the auxiliary variables) are not nan?
Also, could you confirm that it works in GPU mode?

重新看了一下batch norm在lstm上应用的文章， 并且做了一下试验，发现了一个问题：
例如使用unrolled lstm， 序列长度为m， 对lstm的输入数据做batchnorm， 则对应m个batchnorm， batchnorm在初始化的时候可以指定gamma和beta 两个symbol，因此可以控制这m个batch norm都使用同一组gamma和beta参数，可是对于aux_states， 却仍然有m组， 因为目前在batch norm初始化的时候无法指定aux_states。 请问这个应该如何解决呢？

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


[FRONTEND] Add activation functions

I don't think this should be part of triton tbh. If someone makes a DNN kernel library on top of Triton then it should reside there


Support for .NET Standard

Xamarin app development in Visual Studio 2017 is going to adapt .NET Standard libraries more and more. Most of the Xamarin.Plugin libraries that I'm using in my project were already ported to use .NET Standard (Geolocator, External Maps, Permissions, etc.). FilePicker is still using PCL, so I set out to port it to .NET Standard.
Here is a pull request with my porting efforts. Please check each commit and accept it, request modifications or cherry-pick changes as you like.
Unfortunately when developing with Visual Studio 2017, Microsoft dropped support for Windows 8.1 and Windows Phone 8.1, so I also had to remove it from the .sln file. The projects are still there.
I also moved and modified the .nuspec file, as in the current location no package could be built. I set the version to "2.0.0-beta", which should allow publishing the package to NuGet.org without disturbing users of previous versions.
The projects build without errors and the "nuget pack" generates a NuGet package that works in my Android-only project. As the platform-dependent projects weren't touched, I expect that other platforms work OK, too.
Finally, maybe this Pull Request should go into a "2.0" or "net standard" branch, since maybe more 1.3.x or 1.4.x versions are needed. I hope my contribution is useful for someone!

@ArtjomP maybe you should to a "beta" package first (I already set version 2.0.0-beta" in the .nuspec file) and ask for feedback, before releasing the package. I definitely will update my project and try it out :-)


java-openliberty:  default server start timeout too low when trace is enabled

Default timeout of 30 seconds is too short, causing the container to exit if we take a little longer, eg if trace is enabled.

Fixed by #732


Fix the meaning of conv{1,2}d_transpose output_padding parameter.

I finally had time to work on it again so this is a re-attempt at https://github.com/apache/incubator-tvm/pull/4318, which was reverted.

For some context the tvm meaning for output_padding is to pad the output shape with zeros.
For every other deep learning framework, this parameter name is used to disambiguate between the possible output shapes in the presence of padding (Unless they have an explicit output_shape parameter like tensorflow, which doesn't have output_padding).
The current behaviour is in my mind redundant since you can always zero-pad manually if you need to. Also there is no way to compute the gradient of a convolution with strides that did not have the minimum possible input shape.
The new behaviour makes this possible and brings us in-line with the other frameworks.

I've rebased the code and added checks to make sure an invalid output_padding is refused.
As for the comments by @icemelon9:

The aformentioned check should cover that case and prevent going out of bounds.
Since the preprocess function handles everything that we need output_padding for, we don't need to pass it forward.

My best guess for why the CI didn't catch the cases is that going just a little bit out of bounds (like off by one) is unlikely to trigger any memory problems.

The only thing probably missing is an update of tophub for the missing output_padding arguments.

The tophub PR has been merged, please restart CI to try again

ping @abergeron

Sorry about the delay, I was distracted by another project.

I saw that @siju-samuel added the conv3d_transpose op but with the wrong definition for output_padding.
I would rather make a follow-up PR to fix that since I want this one to be merged if it's ok and I don't have a lot of free time currently.
I will make a follow-up PR to fix the definition of output_padding in conv3d_transpose later if nobody else does it before me.

@abergeron Thanks for bringing it up and the PR. I  followed the existing conv2d_transpose implementation.  You can add another PR to fix for fixing this issue.


undefined method `unsigned' for ActiveRecord::ConnectionAdapters::ColumnDefinition

When using Rails 5.0 with the mysql adapter via seamless_database_pool, a standard add_column change migration fails with the following stack trace (running bundle exec rake db:migrate --trace):
undefined method `unsigned' for #<ActiveRecord::ConnectionAdapters::ColumnDefinition:0x0055b0ff865280>
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/mysql/schema_creation.rb:15:in `visit_ColumnDefinition'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:14:in `accept'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:39:in `visit_AddColumnDefinition'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/mysql/schema_creation.rb:20:in `visit_AddColumnDefinition'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:14:in `accept'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:26:in `block in visit_AlterTable'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:26:in `map'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:26:in `visit_AlterTable'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_creation.rb:14:in `accept'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/schema_statements.rb:549:in `add_column'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:845:in `block in method_missing'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:814:in `block in say_with_time'
[RUBY]/benchmark.rb:288:in `measure'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:814:in `say_with_time'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:834:in `method_missing'
[RAILS]/db/migrate/20160812010138_add_processed_at_to_workshops.rb:3:in `change'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:788:in `exec_migration'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:772:in `block (2 levels) in migrate'
[RUBY]/benchmark.rb:288:in `measure'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:771:in `block in migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/connection_adapters/abstract/connection_pool.rb:398:in `with_connection'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:770:in `migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:950:in `migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1211:in `block in execute_migration_in_transaction'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1281:in `ddl_transaction'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1210:in `execute_migration_in_transaction'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1183:in `block in migrate_without_lock'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1182:in `each'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1182:in `migrate_without_lock'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1131:in `block in migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1298:in `with_advisory_lock'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1131:in `migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:1005:in `up'
[GEMS]/activerecord-5.0.0/lib/active_record/migration.rb:983:in `migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/tasks/database_tasks.rb:161:in `migrate'
[GEMS]/activerecord-5.0.0/lib/active_record/railties/databases.rake:58:in `block (2 levels) in <top (required)>'
[GEMS]/rake-11.2.2/lib/rake/task.rb:248:in `call'
[GEMS]/rake-11.2.2/lib/rake/task.rb:248:in `block in execute'
[GEMS]/rake-11.2.2/lib/rake/task.rb:243:in `each'
[GEMS]/rake-11.2.2/lib/rake/task.rb:243:in `execute'
[GEMS]/rake-11.2.2/lib/rake/task.rb:187:in `block in invoke_with_call_chain'
[RUBY]/monitor.rb:211:in `mon_synchronize'
[GEMS]/rake-11.2.2/lib/rake/task.rb:180:in `invoke_with_call_chain'
[GEMS]/rake-11.2.2/lib/rake/task.rb:173:in `invoke'
[GEMS]/rake-11.2.2/lib/rake/application.rb:152:in `invoke_task'
[GEMS]/rake-11.2.2/lib/rake/application.rb:108:in `block (2 levels) in top_level'
[GEMS]/rake-11.2.2/lib/rake/application.rb:108:in `each'
[GEMS]/rake-11.2.2/lib/rake/application.rb:108:in `block in top_level'
[GEMS]/rake-11.2.2/lib/rake/application.rb:117:in `run_with_threads'
[GEMS]/rake-11.2.2/lib/rake/application.rb:102:in `top_level'
[GEMS]/rake-11.2.2/lib/rake/application.rb:80:in `block in run'
[GEMS]/rake-11.2.2/lib/rake/application.rb:178:in `standard_exception_handling'
[GEMS]/rake-11.2.2/lib/rake/application.rb:77:in `run'
[GEMS]/rake-11.2.2/exe/rake:27:in `<top (required)>'
[GEM_PATH]/bin/rake:23:in `load'
[GEM_PATH]/bin/rake:23:in `<top (required)>'
[GEMS]/bundler-1.12.5/lib/bundler/cli/exec.rb:63:in `load'
[GEMS]/bundler-1.12.5/lib/bundler/cli/exec.rb:63:in `kernel_load'
[GEMS]/bundler-1.12.5/lib/bundler/cli/exec.rb:24:in `run'
[GEMS]/bundler-1.12.5/lib/bundler/cli.rb:304:in `exec'
[GEMS]/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
[GEMS]/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
[GEMS]/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
[GEMS]/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
[GEMS]/bundler-1.12.5/lib/bundler/cli.rb:11:in `start'
[GEMS]/bundler-1.12.5/exe/bundle:27:in `block in <top (required)>'
[GEMS]/bundler-1.12.5/lib/bundler/friendly_errors.rb:98:in `with_friendly_errors'
[GEMS]/bundler-1.12.5/exe/bundle:19:in `<top (required)>'
[GEM_PATH]/bin/bundle:23:in `load'
[GEM_PATH]/bin/bundle:23:in `<main>'

I tracked the issue down to the create_table_definition private method in the MySql2 connection adapter not being proxied properly, so the generic ActiveRecord::ConnectionAdapters::TableDefinition object is being passed around instead of the (required) ActiveRecord::ConnectionAdapters::MySQL::TableDefinition.
To work around this issue temporarily, I added the line:
            master_methods.concat(connection_class.private_instance_methods(false))

to seamless_database_pool_adapter.rb:101, then the migration completed properly.

fixed by #23.


Add image previews to directory list

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

When viewing a picture album on the WebUI, we have to click on each and every single image in order to view them.
Describe the solution you'd like

A clear and concise description of what you want to happen.

It would be better if there was card view where we could see multiple images at once.
Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Showing all medias at the top of the list of files could also be interesting.
Additional context

Add any other context or screenshots about the feature request here.

https://github.com/ipfs/dir-index-html/issues/17

Thanks, @NatoBoram! We'll add this to our features backlog for further prioritization (and, of course, if you'd be interested in tackling a PR for this, that would be awesome!) It's a nice idea -- would be interested to hear from others how useful they might find this.


[IDEA] Add instructions on how to build TF from source to  make notebooks more accessible to those who have different GPUs

Thanks for helping us improve this project!
Before you create this issue
Please make sure you are using the latest updated code and libraries: see https://github.com/ageron/handson-ml3/blob/main/INSTALL.md#update-this-project-and-its-libraries
Also please make sure to read the FAQ (https://github.com/ageron/handson-ml3#faq) and search for existing issues (both open and closed), as your question may already have been answered: https://github.com/ageron/handson-ml3/issues
Is your feature request related to a problem? Please describe.
Please indicate the notebook name and cell number where the problem occurs (or the chapter and page number in the book), and provide a clear and concise description of what the problem is. Ex.  In chapter 1, cells 200-220, I think the code could be clearer [...]
Describe the solution you'd like
Include instructions for building tensorflow from source for specific GPUs in the installation guide in the GitHub repository. As well as where to find alternative builds for the ROCm variation of the package.
Describe alternatives you've considered
Using the CPU version of tensorflow for unsupported cards/drivers. This may be okay for the examples in the book, but will become a problem for any real world examples. It would be good for this repository to include instructions on building tensorflow so those with different graphics cards or drivers can get the most out of this book.
A second alternative would be to provide .whl files that target specific GPU/Driver combos in the repo. But that may be a burden to maintain.
Additional context
The current version of tensorflow is built for CUDA 11.8 and cuDNN 8 only. This is problematic for many users, as the current version of CUDA is 12.4 and cuDNN is on version 9. Tensorflow will not work properly with these drivers. It can cause problems for users who are following along on their own systems to have to downgrade their GPU drivers, as they may need the up to date drivers for other activities.
There is a similar problem for AMD users. They can also solve this problem by building tensorflow for their specific GPU/Driver. However, AMD also maintains their own distribution which AMD readers may be able to use. There alternative builds can be found here.
Tensorflow provides a solution, with instructions on how to build the project from their github repo to target your GPU/Driver combination. However, these instructions are not entirely up to date and there are some pit falls. Mainly

On linux, using clang on ubuntu based system gives an error (NvInferVersion.h missing )

This can be solved by building with gcc instead by selected n when asked to use clang

The "build" command is wrong for the most recent version of tensorflow. You need to use the v2 build like so:

bazel build //tensorflow/tools/pip_package/v2:wheel --repo_env=WHEEL_NAME=tensorflow --config=cuda
This will also output the build to .../v2 inside of the directory the instructions tell you it will be output to

You need to set the following env variables

export LD_LIBRARY_PATH=/usr/local/cuda-12.4/lib64:/usr/local/cuda-12.4/extras/CUPTI/lib64:$LD_LIBRARY_PATH
export PATH=/usr/local/cuda-12.4/bin:$PATH
export CUDA_HOME=/usr/local/cuda-12.4
export XLA_TARGET=cuda120
export XLA_FLAGS=--xla_gpu_cuda_data_dir=/usr/local/cuda-12.4
export TF_CUDA_VERSION='12.4'

You need to specify what version of python you want to target by setting an env variable

export TF_PYTHON_VERSION=3.12

I believe adding these instructions would be very helpful for new readers. Especially those who are not strong programmers and are taking this more as a data science exercise, who may not be familiar with the tools needed to set this up, if they want to run the projects on their own machine.

When these installation instructions are already part of the Tensorflow repository itself, a link might make more sense to not become outdated too quickly. If you think that there are issues inside the official Tensorflow docs, you should consider reporting them upstream and might even provide a corresponding patch/fix to let all Tensorflow users benefit from this.


findup-sync security updates

findup-sync should be updated to the most recent version to raise the version of minimatch (included with findup-sync) to remove a potential RegEx DOS vulnerability.

Would you mind submitting a PR? I'd be happy to land and cut a release if all the tests pass.

These bumps were major so we need to be careful. The vulnerability doesn't really effect this module.

These dependencies have been updated and published as 2.0.0


Bug Report: converter.structure(d['selector'], sel2cls[d['selector']['selector_type']])

Hi, Gupta.
I came across a bug when trying to run the codes.
I was running a testing shell file:
CUDA_VISIBLE_DEVICES=1 python experiments.py --label "test" --seeds 0 \
   --datasets "COGS" \
   --selectors "cosine" \
   --lms "llama2-7b" \
   --n-shots 8 --baselines-exp \
   --paramsfile "params/all.jsonl" --run \
   --no-collate-results \
   --preview "logfiles"

CUDA_VISIBLE_DEVICES=1 python run.py --paramsfile "params/all.jsonl" --gpus "1"

As you can see, it contains two python commands, where the first one python experiments.py succeeded while the second one python run.py failed.
the error location: "in-context-learning/src/params.py:123 in from_dict"
│   120 │   │   # DP = converter.structure(d['data'], DataParams)                                  │
│   121 │   │   DP = converter.structure(d['data'], ds2cls[d['data']['dataset']])                  │
│   122 │   │   LP = converter.structure(d['llm'], LLMParams)                                      │
│ ❱ 123 │   │   SP = converter.structure(d['selector'], sel2cls[d['selector']['selector_type']])   │
│   124 │   │   return cls(EP, DP, LP, SP)

the error message: TypeError: CosineCoverageSelectorArgs.init() missing 1 required positional argument: 'n_shots'.
I tried to decompose the error so that anyone can reproduce the error with running the following minimal example code:
from selector import (
    CommonSelectorArgs,
    CosineCoverageSelectorArgs,
    BertScoreSelectorArgs,
    GistBertScoreSelectorArgs,
    StructuralCoverageSelectorArgs,
    LFCoverageSelectorArgs,
)

import attr
import cattr

converter = cattr.Converter()
d = dict()

d = {
'selector_type': 'cosine',
'n_shots': 8,
'emb_lm': 'sentence-transformers/all-mpnet-base-v2',
'coverage': False,
'reorder': False
}
SP = converter.structure(d, CosineCoverageSelectorArgs)
# Here we met up with the same ERROR!
# TypeError: CosineCoverageSelectorArgs.__init__() missing 1 required positional argument: 'n_shots'
print(SP)

To be more specific, SP = converter.structure(d, CosineCoverageSelectorArgs)  will assign the whole dictionary d as a string (maybe) to the first class variable of CosineCoverageSelectorArgs (which is selector_type in this case), but it makes no sense! Hence will be nothing to be assigned to the second class variable (shot here).
Do you have some thoughts about this bug?
Much appreciation for your time reading my question!

Hi, this happened because of some breaking changes in the latest versions of the cattrs and/or attrs libraries. The simple fix is to install version 23 of both: mamba install attrs=23.2 cattrs=23.2. I'll update the requirements.txt.

It works. Gupta thanks for the response.


Most client commands should only modify one chain.

E.g. currently changing the resource policy or adding or removing validators changes all owned chains, because it makes them receive the cross-chain message sent by the admin chain.
This should only be the case for commands that explicitly say so. It's probably sufficient to add a sync-all command (or an --all-chains option to the sync command).

Passing to @afck to check for outliers

I went through the commands in linera/main.rs and couldn't find any that modify more than one chain.


batching fsync

we do not batch fsync today. It might cause issues on some cloud env which has a very limited number of iops.

a fsync must be called before:

leader sends out a raft message with a larger commit index.
follower sends out a msgAppResp after appending new entries.

If we want to batch fsync, we need to batch the messages basically.
/cc @yichengq

Somewhat related (#2486), writes and fsync calls occur continuously on an idle single-node cluster with no data and no api calls being made to it. strace indicates that no network syscalls are called other than bind and accept, yet pairs of write + fsync syscalls occur about 4+ times per second to the WAL.
I suspect write frequency is driven EtcdServer.Ticker.

@extemporalgenome the idle cluster will do sync too. and it is 4 times per second(sync/500ms, which involves two writes and one fsync).

@xiang90 --cluster-sync-interval doesn't seem to have an effect on the write rate.

@extemporalgenome It is an internal thing to sync the TTL. You cannot tune it. But I guess 4 iops is fine... If you want to run etcd smoothly, you should at least prepare for 100s iops even with fsync batching.

Now we already start batching message and disk io seems not to be the bottleneck.


Add Python version of setup script

Adds a Python version of the setup script and updates the README.md to reflect this new addition.

Implements setup_python_env.py: Adds a new Python script that mirrors the functionality of the existing shell script. This script prompts the user for project name and location, creates a project directory with src and tests subdirectories, adds requirements.txt and README.md files, sets up a Python virtual environment, and opens the project in Visual Studio Code.
Updates README.md:

Adds information about the availability of the Python script version, setup_python_env.py, alongside the original shell script.
Modifies the "Prerequisites", "Installation", and "Usage" sections to include instructions for both the shell and Python scripts, highlighting the flexibility in project setup options for users.

For more details, open the Copilot Workspace session.

Introduce a Python script for macOS Python project setup and update README.md to reflect new script options and prerequisites.

Python Script Addition: Adds setup_python_env.py, a Python script for setting up a Python project environment on macOS. This script creates a project directory, subdirectories (src and tests), a blank requirements.txt file, and a README.md file. It also sets up a Python virtual environment within the project directory and opens the project in Visual Studio Code.

Implements optimization by checking if the virtual environment already exists before attempting to create a new one.
Adds error handling for the subprocess call to open Visual Studio Code, catching and printing any errors encountered.
Includes comments explaining the optimization technique and error handling strategy.

README.md Updates: Reflects the addition of the Python script alongside the existing shell script, specifying that the repository now offers both shell script (setup_python_env.sh) and Python script (setup_python_env.py) versions for setting up Python environments. Adjusts the prerequisites and usage instructions to accommodate the new script option.

For more details, open the Copilot Workspace session.


Ionic + Webpack Sub-Entry Points failing

Type of Issue
[X] Bug Report
[ ] Feature Request

Description
When trying to create a repository with the following package structure below, Ionic (through Webpack) notes that it cannot find module @eh/test/components.
Package Structure:
@eh/test
@eh/test/components
How To Reproduce
Please find a sample repo here: https://github.com/ehorodyski/ng-packagr-issue
To run, npm i then ionic serve.
I do path linking in tsconfig.json which the IDE resolves to, but for the Webpack build, the path linking is in config/webpack.config.js.
Expected Behaviour
To run the sample app.
I think there might be an issue with the project structure itself. Current information on this lib is hard to find, and the examples are not very intuitive to follow. For instance, when does one use ng-package.json or not? What are the possible values of the package.schema.json, and what do they control?
Version Information
$ node_modules/.bin/ng-packagr --version
ng-packagr: ^3.0.3
@angular/*: 5.2.11
typescript: ~2.6.2
rxjs: 5.5.11
node: 8.11.2
npm: 6.1.0
ionic-angular: 3.9.2

You have a couple of configuration issues. You should point to the bundled code and not typescript library code.
To get this working you need to update the below in your tsconfig;
    "baseUrl": ".",
    "paths": {
      "@eh/test": [
        "libs/dist"
      ],
      "@eh/test/*": [
        "libs/dist/*"
      ]
    }

And in webpack.config.js the following;
useDefaultConfig[env].resolve.alias = {
    "@eh/test": path.resolve('./libs/dist')
};

Thanks for the response @alan-agius4 -- I'm almost there!
My last issue is that VSCode will toss this error in my libs folder:
[ts] Experimental support for decorators is a feature that is subject to change in a future release. Set the 'experimentalDecorators' option to remove this warning.

Would placing a tsconfig.json at the libs level help?

Awesome, thank you so much @alan-agius4!
I've made a few deviations:

I left the src paths in tsconfig.json -- it wasn't picking up the modules if I didn't do it this way.
I changed the entryFile in both the package and sub-package to index.ts. I can't explain why, but it seems to be how Ionic will correctly pick up my custom components.
I need to change the entry file from public_api.ts to index.ts for VS Code to run experimental decorators in the libs folder. Not sure why...

It's a lot for me to ask, but do you mind checking my latest commit and seeing if I'm packaging up my dependencies correctly?