Tech Debt

Fix these things please

[ ] Serial port timeouts are not implemented correctly in gps.py.
[x] Use python logging instead of print statements

[ ] main state tests are too unit testy, we should swap the during_<> for a few seconds of run..but we have to make the rocket a thread first


Is it possible to use the Single Hotswap plugin with modern syntax?

When I enable the modern syntax, I can no longer use the hotswap feature provided by the Single Hotswap plugin, nor the one from the IDE itself, which ends up reloading several classes. I tested it, and just by marking it as false, I can use hotswap again, so I don't think it's a problem with the JDK used or the agent I added. I'm talking about simple hotswap, like commenting out two lines in a method and trying to reload the class, but with modern syntax enabled, the hotswap fails.

You'll have to report to SingleHotswap about incompatibility between it and jabel, it is possible that a workaround exists but I am not aware of it.

You'll have to report to SingleHotswap about incompatibility between it and jabel, it is possible that a workaround exists but I am not aware of it.

IDE hotswap doesn't work either


Return additional fields along with co-ordinates of location

@chrismattmann : How about returning country code, state, county along with co-ordinates? This information might make sense when we analyse this data in a context. It will allow us to group locations based on location's country, state and county.
[
{"Pasadena" : [
"Pasadena",
"-74.06446",
"4.6964",
"US",
"CA",
"37"
]}
]

+1 @smadha

@smadha shall we return these fields above?

Yeah sure, I will take this up next. Will raise a JIRA in tika for contextual analysis on these results.

great thanks @smadha

handled in #6


Add privacy policy and fix bno price

Fix price of BNO and link in multiple places
Add page with privacy policy for Android app

I have no idea how to add the new page to the bottom of the navigation panel, halp 🥺

this is ready to be merged, right?


Write first round of unit tests

Progress in https://github.com/philipmw/phrase.shop/commit/46ebb48f5ada7596af498a0746f3b9fac3f60d12 but we could use a lot more.


attempt at twitter follow button

And added rachael's about me blurb

Slack will not load.  I am on my way.  Something came up
On Thu, Jun 29, 2017 at 12:34 PM, Madeline Stevens<EMAIL_ADDRESS>
wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/madhubs/dreamteam/pull/24
Commit Summary

attempt at twitter follow button

File Changes

M about.html
https://github.com/madhubs/dreamteam/pull/24/files#diff-0 (3)
M style.css
https://github.com/madhubs/dreamteam/pull/24/files#diff-1 (1)

Patch Links:

https://github.com/madhubs/dreamteam/pull/24.patch
https://github.com/madhubs/dreamteam/pull/24.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/madhubs/dreamteam/pull/24, or mute the thread
https://github.com/notifications/unsubscribe-auth/AXO_badnLw98Kp3fW3xjpC2gFk7uUDmlks5sI_xTgaJpZM4OJxOz
.


Option to auto-suppress errors

Correct me if I am wrong but currently there is no way to gradually adopt tslint to an existing project.
You are flooded with errors. Therefore adding it to your CI and enforce it is not possible. Also all workarounds are not practical in my eyes.
So here is my suggestion:
Add a command line option like --fix which automatically suppresses all warnings by adding a
// tslint:disable-next-line:rule1 rule2 rule3
comment and a description stating that this is a legacy error. Now if you want to adopt tslint you can enable all the rules you want. Run it once with the new flag enabled and add it to your CI. Jobs done. All new code has to follow the rules because it is enforced. Nobody has to touch code which is not required to change and whenever you come across such a comment in your daily work you can fix it to make your code base better over time.
Any opinions on this?

I've started something here (quite a hack at the moment, but will clean it up going forward): https://github.com/paulkoerbitz/tslint-auto-disable

I think tslint-auto-disable is in a usable state now. @stefan-m  if you're still interested, I'm interested in feedback. It's also on npm now: https://www.npmjs.com/package/tslint-auto-disable

It's still unclear whether this should be integrated into TSLint itself or stay as a separate package (tslint-auto-disable). Since there's a bit of demand for it, it'd be good to see a concrete proposal for how this could be integrated into TSLint.
-0 on this rule until then?

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Creating a logfile similar to "--download-archive" (URL, ID, Name, Date and Time)

[x] I've verified and I assure that I'm running youtube-dl 2016.05.10
[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[x] Question
[ ] Other

Description of your issue, suggested solution and other information
I really enjoy using this tool - thank you very much!
Is there a way to log some (accessible) details of the downloaded videos? I'm thinking of somethink like the current output of "--download-archive" but with more details.
I like how I can configure the file output name, but having all these possibilities to create a own logfile-template would be awesome!

You can use the --write-info-json and use the information stored in the *.info.json files. Isn't that enough?

Thank you for the answer!
--write-info-json is very good, and includes nearly all of the things I want.
The only thing I'm missing is a timestamp when the was finished downloading. If it's possible, I'd like to add it somewhere in the json file, so I just have to collect all the information from one file later on...

The only thing I'm missing is a timestamp when the file was finished downloading. If it's possible, I'd like to add it somewhere in the json file, so I just have to collect all the information from just that file later on...

I think you can just check the change time (ctime) of the video file itself.


Old Forms vs. Railsy Forms

Dear TSE,
README
Model:

User
with columns: username, email, password

Form Types:

HTML
form_tag
form_for

Extras:

Index page: localhost:3000/
Error messages
Red bordering style added for the inputs that returns validation error

With all our respect...

Dear TSE,

Routes reconfigured.
We know that the index page is not needed on this project, we wanted to provide an extra feature.
email_field_tag corrected

With all our respect...


Can't Find variable: config

Using the latest wow.js minified. I get these type of errors in browser:
ReferenceError: Can't find variable: config

Everything runs fine, just the annoying error I'd like to fix. Nothing seems to fix it.

Can anyone help with this error?

Could you please link to a site that throws this error? I couldn't reproduce this.

I have the same problem with the last release

@attilaolah You can check it out on the site http://imhired.com There main page uses wow.js and throws the error if you check in web inspector.

Any update? Still throwing there errors after a few months.

I haven't posted any updates because this issue seems to have been fixed in 0d7d1c4adbd4f09eace7d7abba2fdffa366fbe3e. The problem is, I can't tell for sure which version is active on your website because it is compiled in together with other scripts, and there's no way to tell the version.
I wish we had a WOW.version or something.


Marshal (non-const) char* as StringBuilder

There may be some gotchas such as ANSI/Unicode so some reading is needed but this can and should be done.

@ddobrev char* is NOT always meant to work as placeholder for a string. So how can it be differentiated if it is meant to be pointer (hence better a 'ref char') or a string (so 'StringBuilder').
In my opinion, it would however be better to marshal std::string as StringBuilder...

It's impossible to determine. So we have to use the solution which works in both cases and that is StringBuilder because in the case of a pointer to a char it would simply be a StringBuilder with a single character.
std::string represented with a StringBuilder might or might not be a good idea but in any case it's unrelated to the current issue. Also it's quite difficult because std::string is a typedef to a template type and we don't have proper support for templates yet. So if you want to develop your idea, please file another issue.


(Not for submission) Google.Cloud.Location integration

Closing for now for tidiness, but I'll leave the branch available.


Tag the docker image on release

When we upload the binaries to github for a release, we also need to tag a docker image.

We also currently upload a new canary image for each build. We should tag a "release" build that's stable at a tag as well.


Update Apache Kafka to 2.7.0

align Scala 2.13.x versions to match Kafka

there are no compatibility concerns that I am aware of. The Java clients always try to maintain backwards compatibility with older broker versions.


New Feature: elm-get docs

I'm not sure if this is a new feature or not. Is there a way to build all of the docs to see what the site will look like when it gets published to library.elm-lang.org? I've messed up docs a few times but I didn't know how to see what they would look like until I publish, at which point I can fix and repush.
I'm imagining a feature where I could run elm-get docs in my project and get a directory called docs or something that has the pages generated for the website.

Any news on this?

Finally this exists http://package.elm-lang.org/help/docs-preview thanks to @burabure!

I would still like to have something like this. Basically a way to view the Elm docs offline

I would also like to have this. Especially as elm-make --docs=documentation.json seems to return a file containing just the two characters []...

I think it only generates docs for modules in the exposed-modules list in
elm-package.json. Try adding them there
On Thu, Jun 30, 2016 at 8:54 AM Deimos28<EMAIL_ADDRESS>wrote:

I would also like to have this. Especially as elm-make
--docs=documentation.json seems to return a file containing just the two
characters []...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/elm-lang/elm-package/issues/20#issuecomment-229703075,
or mute the thread
https://github.com/notifications/unsubscribe/AAAcrd0PCKAKgAU4bArWbVfiU7nzwjMzks5qQ-bQgaJpZM4BZCA5
.

Thanks! I'll try that.
Le 30 juin 2016 6:27 PM, "Pete Vilter"<EMAIL_ADDRESS>a écrit :

I think it only generates docs for modules in the exposed-modules list in
elm-package.json. Try adding them there
On Thu, Jun 30, 2016 at 8:54 AM Deimos28<EMAIL_ADDRESS>wrote:

I would also like to have this. Especially as elm-make
--docs=documentation.json seems to return a file containing just the two
characters []...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/elm-lang/elm-package/issues/20#issuecomment-229703075>,
or mute the thread
<
https://github.com/notifications/unsubscribe/AAAcrd0PCKAKgAU4bArWbVfiU7nzwjMzks5qQ-bQgaJpZM4BZCA5
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/elm-lang/elm-package/issues/20#issuecomment-229712522,
or mute the thread
https://github.com/notifications/unsubscribe/ADYDvODLfX5CaJprz4S1wO6zfx2w0xozks5qQ-5lgaJpZM4BZCA5
.


AnalogNFet has no drain element

We were trying to reproduce the 5-transistor OTA.  When we get an error: AttributeError: drain not defined in Element
Here is the link to our col-lab code: https://colab.research.google.com/drive/1cwWHiGRlmKUBavCjhQEOL7jpQqm7cqef?usp=sharing
Any help is greatly appreciated.

I think you copied the example from the development documentation, which will not be supported under schemdraw 0.17 in pypi. Use the example from the stable version.
The difference is a new experimental feature that would automatically add elements to the drawing, so the d += becomes unnecessary. In 0.17,  d += or d.add is required to add each element, and the anchor attributes defined only after one of those adds the element to the drawing.

0.18 was released. Closing this issue.


Does pt.save_match work?

I have two MIDI files  - one is performance midi and one is score midi.
After aligning them with parangonar the alignment seems to succeed and shows that the notes are mostly matched.
However I can't seem to run pt.save_match() successfully. It fails with the following error:
  File "/Users/xavriley/anaconda3/lib/python3.9/site-packages/partitura/io/exportmatch.py", line 595, in save_match
    matchfile = matchfile_from_alignment(
  File "/Users/xavriley/anaconda3/lib/python3.9/site-packages/partitura/utils/misc.py", line 88, in wrapper
    return f(*args, **kwargs)
  File "/Users/xavriley/anaconda3/lib/python3.9/site-packages/partitura/io/exportmatch.py", line 184, in matchfile_from_alignment
    ptime_to_stime_map, _ = get_time_maps_from_alignment(
  File "/Users/xavriley/anaconda3/lib/python3.9/site-packages/partitura/musicanalysis/performance_codec.py", line 770, in get_time_maps_from_alignment
    [np.mean(perf_onsets[u]) for u in score_unique_onset_idxs]
  File "/Users/xavriley/anaconda3/lib/python3.9/site-packages/partitura/musicanalysis/performance_codec.py", line 770, in <listcomp>
    [np.mean(perf_onsets[u]) for u in score_unique_onset_idxs]
IndexError: arrays used as indices must be of integer (or boolean) type

Stepping through this code it does look like the values in score_unique_onset_idxs are arrays of integers. Am I doing something wrong or is there a bug in this part of the code?
My full code for this is below, along with a link to the file I am using to test.
import parangonar as pa
import partitura as pt
import pretty_midi as pm
from tempfile import NamedTemporaryFile

mid_path = "/Users/xavriley/Dropbox/PhD/Datasets/Filosax/Participant 1/01/Sax.mid"

with NamedTemporaryFile(suffix=".mid") as score_mid_path:
    with NamedTemporaryFile(suffix=".mid") as perf_mid_path:
        mid = pm.PrettyMIDI(mid_path)
        del(mid.instruments[1])
        perf_mid = pm.PrettyMIDI(mid_path)
        del(perf_mid.instruments[0])
        mid.write(score_mid_path.name)
        perf_mid.write(perf_mid_path.name)
        score = pt.load_score_midi(score_mid_path.name)
        perf = pt.load_performance_midi(perf_mid_path.name)[0]

        score_part = pt.score.unfold_part_maximal(score[0])

        score_na = score_part.note_array(include_grace_notes=True)
        perf_na = perf.note_array()
        sdm = pa.DualDTWNoteMatcher()
        pred_alignment = sdm(score_na,
                     perf_na,
                     process_ornaments=False,
                     score_part=score_part)
        print(pred_alignment)
        matchfile = pt.io.exportmatch.save_match(pred_alignment, perf, score_part, assume_unfolded=True)

File: https://www.dropbox.com/scl/fi/xyfe3yo7e5ni3e1muf72w/Sax.mid?rlkey=t34r5t0mgjjf2r0os6mvwk5ra&dl=0

Hi @xavriley
thanks for the excellent issue including MIDI file! I found the bug in get_time_maps_from_alignment where arrays of idx is cast to numpy dtype objects if all score and performance idx are unique. I created a PR to the develop branch here: #360 with these changes, save_match' should work again. You may use the "fix 358" branch or the "develop" branch as soon as the PR is merged (for instance clone the branch, navigate to the repo  and then install with pip install .`). I don't know when the fix will be in a proper release though. Let us know if you encounter any more issues!


Slight fix for Visual Studio Community Mac

I am using Visual Studio Community for Mac. I know its not very common, but it is useful to have a visual studio that integrates setting pulled from the .sln file. Pulls dependencies fast and more. Anyway, I got this backend warning while trying out a fork of this project. Might want to consider adding await() to the homecontroller.cs for the sake of showing how it works and commenting how its useful.
In this business, if you aren't constantly learning, your not gonna last long.

Outside of that, I'd say this project is fully mac certified for use in Visual Studio Community Mac!!!
Also @MarkPieszak, I have seen so many of your projects and must say you are awesome. Would love to learn from you!!

Thanks @inunotaisho26 I really appreciate the kind words !! ❤
I'm glad it's all working, yeah I've been working on everything locally with a Mac for over a year now. I haven't tried VSCommunity in a while, I typically use VSCode just because of all of the great tooling.
I haven't noticed that little warning, if you want to get a PR in I can merge it into Master!
Thanks again 💯


Audio always max length

I have successfully trained this model on a custom non-English dataset with 8 GPUs for 48 hours and the dataset is about the same size as LJSpeech and all the speech alignments are really good (almost perfect diagonal lines) yet the utterances synthesized are too long and the later audio alignments after the good audio are terrible (multiple horizontal lines, random patterns, etc...) after the initial audio has completed. If I edit the output audio to fit the correct length then the outputs are awesome.

I have included punctuation in all the training and test data as well as applying only the translliteration text filter.

Is there any way to guarantee that the output gating mechanism will actually truncate the synthesis process properly at the correct point? I'll see if I can post some TensorBoard output if I still have it available.

The gate outputs are horizontal lines during training. Adding additional TensorBoard debugging seems to really slow down the code, so I don't have many screenshots to post (and can't at the moment either way).

Gate output from TensorFlow during a retraining session:

Do your audio samples have silence at the end? I usually add about .3s of silence to the end of my files

Do your audio samples have silence at the end? I usually add about .3s of silence to the end of my files to combat that. Too much and it also won't stop.
A stop token (i.e ~) also helps

can you give more detail about how you do it? I also facing this problem. Sometime the audio end at right time, sometime it have a long silence at the end, another time it speak some "alien language" after finished the input sentence
I tried reduce gate_threshold but it only reduce the error rate, not completely solved the problem

Do your audio samples have silence at the end? I usually add about .3s of silence to the end of my files to combat that. Too much and it also won't stop.
A stop token (i.e ~) also helps

can you give more detail about how you do it? I also facing this problem. Sometime the audio end at right time, sometime it have a long silence at the end, another time it speak some "alien language" after finished the input sentence
I tried reduce gate_threshold but it only reduce the error rate, not completely solved the problem

In my tests the silence in the end makes a big difference, the model starts to learn when to stop. I did it normalizing silence with librosa trim effect and then concatenating 3*hop_length of silence (0's vector), also the stop token you just need to preprocess your tokens to make this:
If your phrase is "my name is Lucas." you process it to be " my name is Lucas ", where  is a token to indicates that the sentences is starting and  where the sentence ends.

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence

what kind of token can I use? a special symbol?
can I do this: "~ this is a sentence ."
do they need to be different or I can use 1 symbol for both SOS and EOS?

Do your audio samples have silence at the end? I usually add about .3s of silence to the end of my files to combat that. Too much and it also won't stop.
A stop token (i.e ~) also helps

can you give more detail about how you do it? I also facing this problem. Sometime the audio end at right time, sometime it have a long silence at the end, another time it speak some "alien language" after finished the input sentence
I tried reduce gate_threshold but it only reduce the error rate, not completely solved the problem

In my tests the silence in the end makes a big difference, the model starts to learn when to stop. I did it normalizing silence with librosa trim effect and then concatenating 3*hop_length of silence (0's vector), also the stop token you just need to preprocess your tokens to make this:
If your phrase is "my name is Lucas." you process it to be " my name is Lucas ", where is a token to indicates that the sentences is starting and where the sentence ends.

I am not clear how long of the silence I should add (in seconds) and how to do it. My model using the default value of hop_length=256
can I read all my audio data as numpy array, add zeros in the end and rewrite file?

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence

what kind of token can I use? a special symbol?
can I do this: "~ this is a sentence ."
do they need to be different or I can use 1 symbol for both SOS and EOS?

I didn't make ablations on the token itself, so by now im using the default text processing of this repository and it is working.

Do your audio samples have silence at the end? I usually add about .3s of silence to the end of my files to combat that. Too much and it also won't stop.
A stop token (i.e ~) also helps

can you give more detail about how you do it? I also facing this problem. Sometime the audio end at right time, sometime it have a long silence at the end, another time it speak some "alien language" after finished the input sentence
I tried reduce gate_threshold but it only reduce the error rate, not completely solved the problem

In my tests the silence in the end makes a big difference, the model starts to learn when to stop. I did it normalizing silence with librosa trim effect and then concatenating 3*hop_length of silence (0's vector), also the stop token you just need to preprocess your tokens to make this:
If your phrase is "my name is Lucas." you process it to be " my name is Lucas ", where is a token to indicates that the sentences is starting and where the sentence ends.

I am not clear how long of the silence I should add (in seconds) and how to do it. My model using the default value of hop_length=256
can I read all my audio data as numpy array, add zeros in the end and rewrite file?

Check out the get_mel and get_audio functions in my repo, i use trim to normalize initial and ending silences and add 5*hop_length at the beginning and ending of the raw wavform data (just putting 0's vector as you said) https://github.com/lucashueda/pt_etts/blob/master/data_preparation.py , i get this functions from other issue and just change it a little to my project.

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence

what kind of token can I use? a special symbol?
can I do this: "~ this is a sentence ."
do they need to be different or I can use 1 symbol for both SOS and EOS?

I didn't make ablations on the token itself, so by now im using the default text processing of this repository and it is working.

where is the default text processing file? I cant file anything related to EOS SOS tokens

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence

what kind of token can I use? a special symbol?
can I do this: "~ this is a sentence ."
do they need to be different or I can use 1 symbol for both SOS and EOS?

I didn't make ablations on the token itself, so by now im using the default text processing of this repository and it is working.

where is the default text processing file? I cant file anything related to EOS SOS tokens

The text processing is here https://github.com/lucashueda/pt_etts/blob/master/text/init.py , in fact it doesn't put eos or sos tokens, but if you put a "." in the every ending of sentences i think it is sufficient to make the model work.
As i already said im still new to this field, so my intuition is that the trim helps attention alignment by normalizing the start and ending of all audios . And the silence in the end linked to a '.' pattern in every ending help the models to learn when to finish the decoder steps

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence

what kind of token can I use? a special symbol?
can I do this: "~ this is a sentence ."
do they need to be different or I can use 1 symbol for both SOS and EOS?

I didn't make ablations on the token itself, so by now im using the default text processing of this repository and it is working.

where is the default text processing file? I cant file anything related to EOS SOS tokens

The text processing is here https://github.com/lucashueda/pt_etts/blob/master/text/init.py , in fact it doesn't put eos or sos tokens, but if you put a "." in the every ending of sentences i think it is sufficient to make the model work.
As i already said im still new to this field, so my intuition is that the trim helps attention alignment by normalizing the start and ending of all audios . And the silence in the end linked to a '.' pattern in every ending help the models to learn when to finish the decoder steps

I also think about adding "." in every sentences before, but I looked at the example dataset they used for the published model, not all of their sentences have end symbols; so I havent tried
but if you tested and see it worked, I will give it a try. Thanks for your help!

I chose a pretty arbitrary small amount of silence that seemed to work for
me, too much and you’ll have other issues. In front I did 150ms and in back
I did 300.
I do not use a start token, but I do use a stop token “~”. I modified the
text processing code to always include that symbol so I don’t need to
manually add it to training days or during inference.
On Sat, Oct 24, 2020 at 12:08 AM EuphoriaCelestial<EMAIL_ADDRESS>wrote:

Oh man the tokens doesn't appear.
"SOS my name is Lucas EOS"
SOS is the start of sequence token and EOS is the end of sequence
what kind of token can I use? a special symbol?
can I do this: "~ this is a sentence ."
do they need to be different or I can use 1 symbol for both SOS and EOS?
I didn't make ablations on the token itself, so by now im using the
default text processing of this repository and it is working.
where is the default text processing file? I cant file anything related to
EOS SOS tokens
The text processing is here
https://github.com/lucashueda/pt_etts/blob/master/text/__init__.py , in
fact it doesn't put eos or sos tokens, but if you put a "." in the every
ending of sentences i think it is sufficient to make the model work.
As i already said im still new to this field, so my intuition is that the
trim helps attention alignment by normalizing the start and ending of all
audios . And the silence in the end linked to a '.' pattern in every ending
help the models to learn when to finish the decoder steps
I also think about adding "." in every sentences before, but I looked at
the example dataset they used for the published model, not all of their
sentences have end symbols; so I havent tried
but if you tested and see it worked, I will give it a try. Thanks for your
help!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/tacotron2/issues/407#issuecomment-715669858,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABICRILLD2I3ZFOEDFF4JHLSMJHLFANCNFSM4P2CL5OQ
.

@xDuck and how did you add silence to audio files? can I have the code?

I do not have the code on hand anymore, but it was super short python code
using one of the many audio libraries. Just remove all silence at start and
end then add silence back in length you want.
I think the library I used was PyDub.
On Sat, Oct 24, 2020 at 12:35 AM EuphoriaCelestial<EMAIL_ADDRESS>wrote:

@xDuck https://github.com/xDuck and how did you add silence to audio
files? can I have the code?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/tacotron2/issues/407#issuecomment-715672132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABICRIKXPXTXIO5EH3Z4NSDSMJKRBANCNFSM4P2CL5OQ
.

Thank you, I will try to do it


minor fix to how branch name is used for image_push_tag

IMPORTANT: Please do not create a Pull Request without creating an issue first.
PR switches image_push_tag in version script to use formatted branch name format.
Problem:

during pr reviews, checking out the PR creates a branch name of format pr/username/${pr-number}
building from this branch results in issues as branch name is used for image tags, which fails as docker image tags cannot contain /.
Solution:

The pr switches to use COMMIT_BRANCH_FORMATTED for image tag, which swaps out / with - in the tag name, allowing images to be built and push successfully.
Related Issue:
Test plan:

@mergifyio backport v1.2 v1.3


Only draw positive radius arcs Fixes #592

Looks good

Hi, could this be merged into v1?

@FeiTeng-Alfred merged.


Best way to use custom animation for slide along with translate

Hi,
I would like to use a different animation for the swiping. I would like to add custom animation. I would like to scale the slide from 100%(size) to 50% gradually for half of the slide transition time and then i would like to scale the slide from 50% to 100% gradually for the next half of the slide transition along with the normal translation of slide along X axis. How and where can i add this animation?

Look how effects are done https://github.com/nolimits4web/Swiper/blob/master/src/js/effects.js Basically you just need to listen onSetTranslate/Transition callbacks and add all required logic there related to translate value


you probably have an older CPU than required

Starting configserver on 'node0.vespanet'
Problem running program /opt/vespa/bin/vespa-runserver => died with signal: illegal instruction (you probably have an older CPU than required)
1714356717.028282	node0.vespanet	15	-	vespa-start-services	info	Too low vm.max_map_count [65530] - trying to increase it to 262144
1714356717.028475	node0.vespanet	15	-	vespa-start-services	warning	Could not increase vm.max_map_count - current value [65530] too low, should be at least 262144
Starting config proxy using tcp/node0.vespanet:19070 and tcp/node1.vespanet:19070 and tcp/node2.vespanet:19070 as config source(s)
Waiting for config proxy to start on 'node0.vespanet'
runserver(configproxy) running with pid: 41
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
1714356718.687518	node0.vespanet	15	-	vespa-start-services	warning	Could not ping configproxy: signal: illegal instruction
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)
Problem running program vespa-ping-configproxy => died with signal: illegal instruction (you probably have an older CPU than required)```

you probably have an older CPU than required

What kind of CPU is recommended to use

you probably have an older CPU than required

What kind of CPU is recommended to use

I know the problem now, it's the instruction set of the CPU that doesn't support it

Vespa builds binaries for both arm64 and x864, For the latter we set haswell as the minimum cpu-type (approx 10 years old, 2014), so you need a newer CPU than that.
From x86 options

Generate instructions for the machine type cpu-type. In contrast to -mtune=cpu-type, which merely tunes the generated code for the specified cpu-type, -march=cpu-type allows GCC to generate code that may not run at all on processors other than the one indicated.

If you don't have that, you can use the generic x86 image. Using the generic target architecture comes with worse performance.

I know the problem now, it's the instruction set of the CPU that doesn't support it

[root@localhost ~]# gcc -march=native -Q --help=target | grep march
-march=                     		core-avx-i
[root@localhost  ~]#

Vespa builds binaries for both arm64 and x864, For the latter we set haswell as the minimum cpu-type (approx 10 years old, 2014), so you need a newer CPU than that.
From x86 options

Generate instructions for the machine type cpu-type. In contrast to -mtune=cpu-type, which merely tunes the generated code for the specified cpu-type, -march=cpu-type allows GCC to generate code that may not run at all on processors other than the one indicated.

If you don't have that, you can use the generic x86 image. Using the generic target architecture comes with worse performance.

Okay, thank you for your reply

Will try to improve the error message for this case!

Added doc in https://docs.vespa.ai/en/cpu-support.html and then the error message will link to this in coming versions.


fix: populate tables and test ReST table

ReST, markdown, and confluence tables are
tested with the actual installed packages.
Furthermore, the ReST table is being parsed
by docutils for errors.

This PR is a preparation for issue #73, because currently the ReST tables are tested without actual packages and the resulting ReST string is not being parsed by docutils.

Thank you for P-R.
I have confirmed running the test including evaluating the rest parser.
We maintain the code style according to PEP 8.
Could you fix the following?
============================================================= FAILURES =============================================================
________________________________________________________ pycodestyle-check _________________________________________________________
/git/pip-licenses/test_piplicenses.py:54:80: E501 line too long (95 > 79 characters)
/git/pip-licenses/test_piplicenses.py:59:80: E501 line too long (93 > 79 characters)

Sure. Done.

@pohlt OK, thank you for your contribution!!


the features table image was swapped for a compressed one

I'm not sure why it's not showing up here, but the original image was like ~200kb and this new image is 54kb, so much smaller = faster loading time. I saved a space in the salesforce to make GitHub see a change

I don't think you added the binary file to the commit?

@richarddli you're right, see 146.


Uncaught Exception: UNKNOWN: unknown error, open 'C:\Users\Conor\AppData\Roaming\Google Play Music Desktop Player\json_s

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.2.4
Time: Sun Jun 05 2016 21:24:31 GMT 1000 (AUS Eastern Standard Time)
Error:
UNKNOWN: unknown error, open 'C:\Users\Conor\AppData\Roaming\Google Play Music Desktop Player\json_s
Error: UNKNOWN: unknown error, open 'C:\Users\Conor\AppData\Roaming\Google Play Music Desktop Player\json_store\playback.json'
    at Error (native)
    at Object.fs.openSync (fs.js:584:18)
    at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:158:20)
    at Object.fs.writeFileSync (fs.js:1234:33)
    at PlaybackAPI._save (C:\Users\Conor\AppData\Local\GPMDP_3\app-3.2.4\resources\app\build\main\utils\PlaybackAPI.js:101:22)
    at PlaybackAPI._setTime (C:\Users\Conor\AppData\Local\GPMDP_3\app-3.2.4\resources\app\build\main\utils\PlaybackAPI.js:170:12)
    at EventEmitter.<anonymous> (C:\Users\Conor\AppData\Local\GPMDP_3\app-3.2.4\resources\app\build\main\utils\PlaybackAPI.js:56:15)
    at emitTwo (events.js:100:13)
    at EventEmitter.emit (events.js:185:7)
    at EventEmitter.<anonymous> (C:\Users\Conor\AppData\Local\GPMDP_3\app-3.2.4\resources\electron.asar\browser\api\web-contents.js:130:25)

This issue was created automatically inside the uncaughtException handler

I think this might be caused by the same thing that caused #1142. The stack traces look identical. Error 122 in the other thread I think is labeled something to the effect of quota reached but it varies based on the kernel in use. Maybe were writing too fast to the playback API file?

This error I have seen before on Windows machines due to strange installation permissions.  I.e. Someone installing the program as Admin and then running as a standard user.
I'm closing this for now because #1142 will track it if it is a write issue and if it isn't then it's the permissions thing mentioned above


Allow partial name match for service ls --filter

This fix tries to address the issue raised in
https://github.com/docker/docker/issues/24270
where it was not possible to have a partial name match when list services with name filter.
This fix updates swarmkit and allows prefix search when name is provided as the filter for listing services.
Related tests have been updated.
This fix is related to
https://github.com/docker/docker/issues/24270
https://github.com/docker/docker/pull/24620
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 55.06%

Merging [#1168][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.38%

@@             master      #1168   diff @@
==========================================
  Files            77         77
  Lines         12099      12089    -10
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6709       6657    -52
- Misses         4483       4511    +28
- Partials        907        921    +14

[][cc-pull]

Powered by Codecov. Last updated by [356c5b2...3757a3c][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/swarmkit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/swarmkit/compare/356c5b2daddefc4a19d54fd5dd0d22ea37aa56d7...3757a3c62f572ec32c05eb7b7681f7188458c4bd?src=pr
[cc-pull]: https://codecov.io/gh/docker/swarmkit/pull/1168?src=pr

Thanks @yongtang. This is a good start but I have a few comments:

I think it's important to be consistent between the different object types. So if services list names by prefix, so should clusters, networks, nodes, and tasks.
If names are matched by prefix, we should rename Filter.Names to Filter.NamePrefixes (to match IDPrefixes). Perhaps we should have both Filter.Names and Filter.NamePrefixes to support both exact matches and prefix matches (is there a use case for both? cc @aluzzardi @vieux).

Thanks @aaronlehmann @stevvooe for the review and feedback. Let me update the pull request.

@aaronlehmann @stevvooe I updated the pull request to includes name prefix search to network, node, cluster, task, and services. The exact name search has also been kept in the update. Please take a look and let me know if there are any issues.

added this to the 1.12 milestone, because the linked docker/docker issues are on the 1.12 milestone, but @stevvooe @aaronlehmann let me know if there's issues with design

Design LGTM

LGTM + nits
I'm fine with prefix matching, as long as we take care to use it in the right place. Anywhere uniqueness is required, we must fail on more than one match. I don't think that is an issue in the use cases we have.
I'd actually prefer we have an operator for prefix matching, but the ship may have sailed there: image -f name=exact and -f name^=prefix.

Thanks @stevvooe @aaronlehmann @thaJeztah for the review. The pull request has been updated. Please let me know if there are any issues.

I misunderstood the comment about the name filterContainsPrefix. I thought that was the name of a function you were adding, not an existing function. So I thought filterContainsNamePrefix, was a better name for the new function, but I don't see a need to duplicate the existing function.
Anyway, it doesn't really matter. LGTM.

I misunderstood the comment about the name filterContainsPrefix. I thought that was the name of a function you were adding, not an existing function. So I thought filterContainsNamePrefix, was a better name for the new function, but I don't see a need to duplicate the existing function.

Apologies. I thought this was part of a carte blanche rename. This doesn't need to be duplicated.
LGTM, here, either way.

@aaronlehmann @stevvooe no problem. The pull request has been updated. And thanks a lot for the review and help.

LGTM


JavaScript

JavaScript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

Completed, had issues with the PR. Kept getting that no changes were made


ConnectionResetError when enabling SSL

Hello,
So far I've been working without SSL to get my application up and running. I used version 8.1 up to now and so far everything was fine.
Now that I enabled SSL I'm getting errors:
2021-07-02 17:58:02.492 ERROR    asyncio: Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
  File "C:\Users\rando\AppData\Local\Programs\Python\Python39\lib\asyncio\events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "C:\Users\rando\AppData\Local\Programs\Python\Python39\lib\asyncio\proactor_events.py", line 162, in _call_connection_lost
    self._sock.shutdown(socket.SHUT_RDWR)
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

Upgrading to 9.1 didn't help.
Gist: https://gist.github.com/Randelung/c1dfbeeb87ba1a1cc0c66424cb67b8f3
It comes with a cert and a private key that get written to disk, please be sure it's not overwriting anything important. Apparently you can't load certificates or keys from strings.
Run, go to http://localhost:8080/ and https://localhost:8443/. The http one will work fine, the ssl one throws one or two errors per load.
The errors seem harmless since the webpage still works, but the log is being spammed to death.
Thanks!

Sounds like an asyncio thing tbh since there's no reference to websockets code in the trace.

... Firefox does not produce the issue, Chrome/Edge do. Reproduced on two systems using two different projects.
Sigh.
Any chance this error can be bubbled to user code as a callback or something in order to catch and ignore it? Or, depending on where it happens, just ignore it outright?

I would recommend creating a socket in the most basic way possible:
import socket, ssl
sock = socket.create_server(("localhost", 1234))
context = ... # your SSL context
with context.wrap_socket(sock, server_side=True) as ssock:
        conn, addr = ssock.accept()
        conn.recv(1024)

If it works at this layer, it should work at all upper layers (hopefully...)
If it doesn't work at this layer, you'll get the best error message Python can give you here.

I'm closing the issue https://docs.python.org/3/library/asyncio-eventloop.html?highlight=create_server#asyncio.loop.create_server doesn't provide an option doing what you want and I doen't wish to stop using it.

I wouldn't have expected you to switch API at all, I was hoping for insight on the issue since you're more experience on the subject.
However, I have a workaround, I think... The issue seems to be a combination of asyncio and Python strangeness.
Googling a bit lead me to https://bugs.python.org/issue30319, which states that socket.shutdown will not ignore errors because the object is still usable and some code might use the signal to trigger something. The only problem is that the event is triggered by asyncio and thus you can't try/catch the event.
However, asyncio lets you set a global exception handler on the event loop, using which I can suppress the exception.
You might have known about this, I'm still learning. 😅 I'm writing it down for posterity.
Your FAQ holds true once again - it's an asyncio issue (or rather misuse of it) and has nothing to do with websockets.

OK - I wasn't aware of the global exception handler. Nice work!


Add documentation to add Graph for issue #19

Documentation update for issue #19

Thanks for your input, I already worked on it myself and now this is unnecessary.


Validate Loaded content

As a user I would like content that is not valid to be loaded to not load.

Resolved https://github.com/jfehrman/decision_json_editor/commit/d41d88b9b5d0858c0002a73f4904ea0451798d93


Fix links to NativeCall

There is currently no /type/NativeCall page, everything seems to be in /language/nativecall
The problem
Broken links on /language/nativetypes
Solution provided
Link to /language/nativecall

FWIW: you're a member of perl6 github org and you have sufficient permissions to just commit these changes directly.

@zoffixznet Thanks! As I'm getting more familiar with the contribution process, I'll start taking those actions. Just waiting now for the checks to finish ...

FWIW, currently in release announcements you're listed as dmaestro. if you want to be listed under a different name, add an entry into https://github.com/perl6/doc/blob/master/CREDITS file


[Feature Request] Add support for Deepstream-6.0 (maybe?)

Hey devs, deepstream-6.0 now releases and the old makefile does not work any more as it will give some error on CFLAGS, can you add support for the new version?

++

++

+++

Do you have this error too? https://github.com/NVIDIA-AI-IOT/yolov4_deepstream/issues/22

Finally, I have been able to build the model and start a pipeline. However it seems that no object is detected. Are you having the same issue too?

Finally, I have been able to build the model and start a pipeline. However it seems that no object is detected. Are you having the same issue too? I am also noticing about 100 FPS on a T4 with FP16 which seems pretty low since I used from 1 to 6 times faster speed using FP16 with yoloV5s with resolution 608x608.

Hi @mfoglio I have the same issue with Deepstream6.1. The pipeline does not output any bounding boxes.

Hi @Harish1810 . I ended up using another model. I didn't find a way to get this one to work.

When I used the custom parser from /opt/nvidia/deepstream/deepstream-5.1/sources/libs/nvdsinfer_customparser this model worked for me in DS6 @mfoglio


Feature Request: Webdav crawler

Hi all,
i'd like to have a new crawler available to access webdav folders.
Please let me know if you need any additional information about what i'd like to see, but i hope the general "webdav crawler" is explicit enough to understand what is needed.
Thank you very much

Hi all,
any update on this or anything I can do to help?

Hi @speedmann, we are working on this feature.
In the mean time, can you please describe what services you will connect to Ambar via WebDav crawler?
Also it will be nice if you can share some feedback about Ambar usage (main purpose/cons and pros).
We need this info to optimize our marketing campaign.
Thank you!

Hi,
I would second this enhancement request. Having webdav crawler would allow me to easily crawl locally running dropbox alternative ownCloud which only supports webdav interface. Besides owncloud there are lots of systems that provide access via webdav interface.
At the moment I'm evaluating ambar for personal use to search thru tons of files that are stored on my computer.
Thank you.

sorry @sochix didn't see your answer.
I'd like to connect a nextcloud instance (and possibly the telekom cloud instance, which provides webdav as interface) to ambar.
The main usage of ambar is for private use and to organise private documents in an order and searchable way.

You can mount webdav folder to our local fs crawler in th elatest release


dbt run test command error get_result_from_cursor() takes 2 positional arguments

dbt test  command error get_result_from_cursor() takes 2 positional arguments
Core:

installed: 1.6.0
latest:    1.6.0 - Up to date!

Plugins:

glue:  1.6.0 - Up to date!
spark: 1.6.0 - Up to date!

dbt --debug  test  --profiles-dir profile --target dev
Compilation Error in test source_not_null_cdc_hudi_db_product_id (models/source_tables.yml)
 get_result_from_cursor() takes 2 positional arguments but 3 were given

#204 https://github.com/aws-samples/dbt-glue/pull/204

I'm also facing the same error while trying to use run_query for some custom macros. Glad to see that a PR's already up.
May I ask when this update be available for use?

The PR has been already merged, and we expect that the PR solved the issue.
Closing this issue but feel free to reopen.


Docker : "not found"

Hello.
Tried your command and when accessing IP:8000, it gives a blank page with "Not found" only.
Any idea ?
Log :
GoAdmin 启动成功。
目前处于 "debug" 模式。请在生产环境中切换为 "release" 模式。
2023-02-10T16:42:04.120Z	INFO	engine/engine.go:402	=====> 初始化数据库连接
2023-02-10T16:42:04.131Z	INFO	engine/engine.go:402	=====> 初始化配置
sqlite is not officially supported, running under compatibility mode.
2023/02/10 16:42:04.139 [I]  http server Running on http://<IP_ADDRESS>:8000

You can try access this curl http://{ip}:8000/admin/login


Introduction of remote gradnorm downsampling

Data selection policy based on gradient norm (based on idiap/EPFL importance sampling)

feel free to merge


lfmm input format

Hi TheWangLab!
What confuses me is that I found that many published articles' public codes use lfmm format as lfmm2 input not genotype dosage matrix.
I also tried emailing the author of lfmm2 to understand this issue, but I didn't get a reply.
I appreciate your input and await your response.
Best!

Hi,
Apologies for the late response! The LFMM format is the same as a genotype dosage matrix: individuals are rows and each SNP is a column in the matrix. In the example code you sent me, the imputed_geno object is the same as the gen object made here in the algatr LFMM vignette (except that there are no row or column names in imputed_geno like there are in gen). You can confirm this by looking at the functions that run the method in the lfmm package documentation (e.g., for the lfmm_ridge() function): they require a "response variable matrix" of genotypes, not any particular LFMM-type file.
So, if you have an LFMM-type file, you should be able to read it into R using LEA's read.lfmm() function (as the authors do in the link you sent over), and then you can proceed to use algatr as normal!
Let me know if you have any other questions.
Best,
Anne

Hi,Anne
Thank you very much for your answer, it helped me a lot!
When I use this function, no matter I use the vcf generated by Beagle imputa or plink --recode vcf-iid, this error will be shown.
I can only import vcfR objects
test1 <- vcf_to_dosage("test.vcf")
Error in x@fix :
  no applicable method for `@` applied to an object of class "character"

Best!

Hi,
Thanks for pointing this out-- this is something we missed adding to the vcf_to_dosage() function. I've just fixed it so if you go ahead and re-install the package, you should be able to use the function on a path to a vcf file now.
Anne

Hi,Anne
Thank you very much for your answers.
I also have a doubt that tess3r can call multi-threads, but I see that this option is not available in the package
Best!

Hi,
You're right that we didn't code the openMP.core.num arg that the tess3() function uses for multithreading into tess_ktest() (or tess_do_everything()). I'd suggest just running tess3() directly as we do in the vignette here; that way you could specify that argument and the rest can be done within algatr. You could also copy the tess_ktest() function within your own environment and modify the line here  that runs TESS to add that argument.
Good luck!
Anne

Hi,Anne
Thanks for the answer, I think you've done a great job of integrating the landscape genome packages.
I'm going to use it heavily in a while. If you have any questions, I will give you positive feedback and I would love you to answer them.
Best!


Create Posts “20221006-explain-like-im-5-tls-cert”

Automatically generated by Netlify CMS

👷 Deploy Preview for monetaboot processing.

Name
Link

🔨 Latest commit
8349bc0e5a82326f591be16e40aae6d70592a6c1

🔍 Latest deploy log
https://app.netlify.com/sites/monetaboot/deploys/633e277bcdc1950008ce7e76


[WIP] [ClangImporter] pick one property decl and stick with it.

In certain situations (which I have yet to pin down) the members of an Objective-C class extension (a special kind of category) can get imported before the members of the class itself. Class extensions are often used to redeclare publicly readonly properties as readwrite. Unfortunately, if the class extension's property has a different type than the original property, the AST verifier (rightfully) flags the getter and setter as having different types. This usually takes the form of mismatched nullability, or providing Objective-C generics in the main interface but not in the class extension.
This first attempt at a fix follows Clang's lead in preferring the version in the class extension. That seems a little odd since that's the version that developers are less likely to be careful about, but it matches the idea that the class extension version is supposed to supersede the one in the main @interface when visible.
(This PR is only supposed to be about the second commit. I'll rebase before merging.)

@DougGregor, what do you think about this? It's not a wonderful answer but it's a very practical one, and it does fix the issue (rdar://problem/30785976 and related).
@swift-ci Please smoke test

I agree that it makes sense to defer to Clang's choice to prefer the redeclaration in the extension.

Hm, this isn't going to be good enough. This changes where the property is declared, which will break cross-references. We really do need the declaration to end up in the original module to get a consistent answer.
I may land something like this anyway to fix the current set of crashes, though.

See also the contradictory results in the test cases I just added.

Abandoning this approach in favor of #8444, which gives much more consistent behavior.


fd_advise on macOS

Use fcntl with F_RDADVISE flag on macOS as a substitute for posix_fdadvise.

Can you add a one-line summary of the RDADVISE semantics?

Sure, that's a good idea!

What about other BSD derivatives, do they support RDADVISE?

No, AFAIK it's macOS specific. See here for FreeBSD man page.

No, AFAIK it's macOS specific. See here for FreeBSD man page.
Please mention this in the comment too.

No, AFAIK it's macOS specific. See here for FreeBSD man page.

Please mention this in the comment too.

Comment added. Have a look!


docs(mongo_client): indicate that ssl defaults to true for srv connection strings

Re: Automattic/mongoose#9511
Description
If you're using mongodb+srv://, the ssl option is true by default, here's where it is set: https://github.com/mongodb/node-mongodb-native/blob/79df55383b5ba3f081b96513253e566793166199/lib/core/uri_parser.js#L78-L84 . I think it's worthwhile to document this.
What changed?
One line of documentation
Are there any files to ignore?
No

Tracked by https://jira.mongodb.org/browse/NODE-2891

Hello, thanks for the suggestion, there's been some back and forth about documenting defaults in drivers vs. in the manual, and the thinking is we want to avoid additional documentation that could be inconsistent with the manual since that covers all drivers.
There's a specific sections that mentions the tls options when using an SRV record here.
https://docs.mongodb.com/manual/reference/connection-string/#dns-seed-list-connection-format

Hello, thanks for the suggestion, there's been some back and forth about documenting defaults in drivers vs. in the manual, and the thinking is we want to avoid additional documentation that could be inconsistent with the manual since that covers all drivers.
There's a specific sections that mentions the tls options when using an SRV record here.
https://docs.mongodb.com/manual/reference/connection-string/#dns-seed-list-connection-format


Fix warnings when documenting with --document-private-items

Use hyperlinks for URLs
Fix broken intra-doc links

This doesn't fix the following warning, since I wasn't sure what change
was appropriate:
warning: public documentation for `rustc_process` links to private item `self::core::compiler::Context::primary_packages`
   --> src/cargo/core/compiler/compilation.rs:164:22
    |
164 |     /// flag), see [`crate::core::compiler::Context::primary_packages`].
    |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ this item is private
    |
    = note: `#[warn(rustdoc::private_intra_doc_links)]` on by default
    = note: this link resolves only because you passed `--document-private-items`, but will break without

To avoid noise, this doesn't add an allow either.

:pushpin: Commit bf62ae9a5a201ea64c92c72711d46b5003e08bc9 has been approved by alexcrichton

:hourglass: Testing commit bf62ae9a5a201ea64c92c72711d46b5003e08bc9 with merge 1cf8c9d102385f80835414a1407b2c3efb6d9e92...

:broken_heart: Test failed - checks-actions

I think the warnings are from a new version of Rust being released last week?
 error: field is never read: `f1`
   --> tests/testsuite/config.rs:660:9
    |
660 |         f1: i64,
    |         ^^^^^^^
    |
note: the lint level is defined here
   --> tests/testsuite/main.rs:6:45
    |
6   | #![cfg_attr(feature = "deny-warnings", deny(warnings))]
    |                                             ^^^^^^^^
    = note: `#[deny(dead_code)]` implied by `#[deny(warnings)]`

I think the warnings are from a new version of Rust being released last week?
 error: field is never read: `f1`
   --> tests/testsuite/config.rs:660:9
    |
660 |         f1: i64,
    |         ^^^^^^^
    |
note: the lint level is defined here
   --> tests/testsuite/main.rs:6:45
    |
6   | #![cfg_attr(feature = "deny-warnings", deny(warnings))]
    |                                             ^^^^^^^^
    = note: `#[deny(dead_code)]` implied by `#[deny(warnings)]`

That sounds right; I know Miri's tests had to be updated recently to account for the newly-increased precision of dead_code.

I'll see if I can fix the error :)

@bors: retry

:hourglass: Testing commit bf62ae9a5a201ea64c92c72711d46b5003e08bc9 with merge ed8590e8c0f68ceb7bb61efc1a9dcfe6d20a6607...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing ed8590e8c0f68ceb7bb61efc1a9dcfe6d20a6607 to master...


Installation fails on Travis for node versions 4+

I have a test repo with just nodehun in at 2.08: https://travis-ci.org/zaccolley/nodehun-travis-test
It works locally when I use 5.0.0 which is why I'm confused.
Will continue to try find the issue but opening here if anyone knows.

I am no longer maintaining this project

fixed #32


Create container for mailing via remote smtp

Note there is already a Makefile in this directory. This means that both
> APP=shame-mailer make
> APP=shame-mailer make push

work. I have not run APP=shame-mailer make push yet.
cc @lavalamp @ixdy @fejta

Meant to be run with environment variables bound like
> docker run -e GITHUB_API_TOKEN=$GITHUB_API_TOKEN -e SHAME_FROM=$SHAME_FROM -e SHAME_REPLY_TO=$SHAME_REPLY_TO -e SHAME_CC=$SHAME_CC -e SMTP_SERVER=$SMTP_SERVER -e SMTP_USER=$SMTP_USER -e SMTP_PASS=$SMTP_PASS gcr.io/google_containers/shame-mailer:TAG

@krousey can you put that in a comment or README somewhere?

@ixdy, comment in the Dockerfile ok?

LGTM - but need to make sure APP=shame-mailer make push doesn't do something awful before merging. :)

@krousey Dockerfile sgtm

@lavalamp agreed, just didn't want to push a container that didn't get an initial review first. :smile:
I'll update the docker file with a comment, push, and update the PR.

Added comment. Fixed flag validation error. Pushed the image to gcr.io/google_containers/shame-mailer:2016-03-25-d0e88f1.
@ixdy you should be able to add the job now. Or if you want, you can give me some pointers on how to do it and I can look at it after lunch.

I wonder how 'deploy' falls down. Doesn't matter I guess, this is fine with me.

Tested on Jenkins, everything seems to work.


Timezone ignored on Docker container

First of all, thanks for Silverbullet!
I have a command to create a note with name "{{ today }} {{ time }}". However, time is UTC instead of local.
My compose.yaml:
services:
  silverbullet:
    image: zefhemel/silverbullet
    container_name: silverbullet
    restart: unless-stopped
    environment:
      - SB_USER=${USERNAME}:${PASSWORD}
      - PGID=1000
      - PUID=1000
      - TZ=America/Sao_Paulo
    volumes:
      - /mnt/dietpi_userdata/silverbullet:/space
    networks:
      - proxy-net

networks:
  proxy-net:
    external: true

Running date inside the container returns UTC time.
I've also tried, with no luck:
   volumes:
      - "/etc/localtime:/etc/localtime:ro"
      - "/etc/timezone:/etc/timezone:ro"

More info: {{ today }} seems to respect timezone fine, while {{ time }} doesn't.

I encountered the same problem

I don't think this is a docker issue, it's indeed a UTC issue. The current time function gives you UTC time, you can define a space function that gives you the local time. Try this:
Somewhere in your space:

```space-script
silverbullet.registerFunction({name: "localTime"}, () => Temporal.Now.plainTimeISO().toString().split('.')[0])
```
Then call it somewhere:
```template
{{localTime}}
```

I don't think this is a docker issue, it's indeed a UTC issue. The current time function gives you UTC time, you can define a space function that gives you the local time. Try this:
Somewhere in your space:

```space-script
silverbullet.registerFunction({name: "localTime"}, () => Temporal.Now.plainTimeISO().toString().split('.')[0])

Then call it somewhere:
{{localTime}}

This is useful for me, thank you very much

I opened #981 but was redirected to this original issue, which is related.
While creating a space script function is an acceptable workaround in some cases, where it falls short is that the out-of-the-box Journal features in Library/Core/Journal use both {{today}} and {{time}} in its filename generation, which creates misleading and potentially confusing names.
This is demonstrated here, where the files were created in the order specified in the annotation:

A note created at 9:57 AM AEST (23:57 UTC) and another created at 10:05 AM AEST (00:05 UTC) are created out of order.
So, the issue in my mind isn't that {{time}} doesn't respect timezones, but that it is used in concert with {{today}} which does appear to. The dates in the journal should be strictly UTC or strictly local by default.

Ok, I'm now formatting all dates and times to whatever the local setting is. Let's hope this works well. A space reindex may be required for this to take effect.


Code refactoring

These slight code adjustments aim to enhance Pythonic quality by incorporating python str.join

Thank you for your interest and your suggestion!


fix: cast support fields nested in lists and maps

Description
The current implementation of cast only works for structs nested in structs.  This PR adds supports for structs contained in other types (lists and maps).  This PR also prevents cast from adding nullable column if the field is not nullable, instead it will throw an error.
Note: This is only a partial solution which would let you merge schema with nested missing columns, it does not allow delta-rs to read the merged schema (though Spark can).  To read the merged schema will require another change where delta-rs defines its own datafusion parquet schemaAdapter.

Please add also a bunch of python tests

@ion-elgreco I'm confused as to what a Python set of tests would do that the tests in Rust don't already do?  :confused:

Please add also a bunch of python tests

@ion-elgreco I'm confused as to what a Python set of tests would do that the tests in Rust don't already do?  :confused:

We mostly need to check the reader side

@HawaiianSpork thanks for the PR! can you rebase please so we can merge

@ion-elgreco thank you.  Sorry, I did not get around to writing python tests.  The code has been rebased.


[Do Not Review] GEM-2928: Added test output for debug

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

last activity was some while ago, please close this PR if no longer needed

Closing due to [Do Not Review] and lack of activity


xmake require grpc fail

Xmake Version
v2.8.9
Operating System Version and Architecture
ubuntu22.04
Describe Bug
xmake require grpc fail
run
$ rm -rf ~/.xmake/
$ xmake require grpc
checking for platform ... linux
checking for architecture ... x86_64
updating repositories .. ok
error: package(abseil/latest:{"cxx_standard=17"}): conflict dependences with package(abseil/latest:{}) in grpc!

it seems caused by https://github.com/xmake-io/xmake-repo/pull/3667/files#diff-c75113795568cbea15dfb1ea883ddc9982f4013aefd2a13a7a32b508f39a356cR43
protobuf-cpp used abseil/latest:{"cxx_standard=17"}, but re2 used  abseil/latest:{}
Expected Behavior
build success
Project Configuration
add_rules("mode.debug", "mode.release")

set_languages("cxx23")
add_requires("grpc")

Additional Information and Error Logs
no more

it seems caused by  https://github.com/xmake-io/xmake-repo/pull/3667/files#diff-c75113795568cbea15dfb1ea883ddc9982f4013aefd2a13a7a32b508f39a356cR43
I will have a quick fix by set all of abseil use c++17

is there any solution that automatically merges options with the same dependencies?
or toplever configurations specified in the project files (projectdir/xmake.lua) to manually designate the dependency project's settings?

@Chi-EEE @c8ef We should remove {configs = {cxx_standard = "17"}} in add_deps and use the default config.
and We can use add_requireconfs("**.abseil", {override = true, configs = {cxx_standard = "17"}}) to modify it in user project.

@Chi-EEE @c8ef We should remove {configs = {cxx_standard = "17"}} in add_deps and use the default config.
and We can use add_requireconfs("**.abseil", {override = true, configs = {cxx_standard = "17"}}) to modify it in user project.

remove all config and use default config will be build fail (  multiple definition ) in my pc, it seems caused by abseil default use c++14 but others default use c++17, maybe some gcc bug here
so i have to change abseil default cxx_standard to c++17 @Chi-EEE @waruqi


Increase standard benchmark regression tolerance

Benchmarks are currently over-sensitive and regularly flag false positive regressions.

I suspect that we might also want to reduce the number of test cases we use as well for the sake of time, though given the kind of variability we're seeing this is probably also necessary.


Add missing FR spree.admin_login translation

spree.admin_login is only provided in four languages. I was a bit confused when the string started displaying in Spanish.
I'm happy to add other languages to this PR if that'd be preferred. But I don't speak any other languages.

Merci beaucoup


Fh 4294 tags and release

This PR sets up the process for doing a development release of mcp and its dependencies.
It also removes the confusing install directory so that we only have installer now.
The concept is to tag the repos and mcp with a consistent tag. Build a set of images with the same tag and push them to the feedhenry org.
Then when using the ansible installer, a docker_hub_tag can be provided, this is consumed by the ansible service broker and will ensure all images used for apbs are installed from that passed tag.
Current issue:
recently we stopped allowing pushing to feedhenry org, so it is unclear how we can tag and push a new image

:eye: at it !

One more change, here https://github.com/feedhenry/mcp-standalone/pull/163
will check the three APBs (android, ios, cordova), when in the train

The scripts did work. I images got pushed over to dockerhub:

mcp:0.0.5
android-app-apb
cordova-app-apb
ios-app-apb

See:
https://hub.docker.com/r/feedhenry/ios-app-apb/tags/


operator: change default MachineInventory name

Right now when the MachineRegistration is missing a 'spec.MachineName' value we set the following hostname:
m-${System Information/Manufacturer}-${System Information/Product Name}-${System Information/UUID}
This causes an invalid hostname if config.registration.no-smbios is set to true, failing the registration.
When no-smbios is set to false instead, we may still fail the registration as the hostname derived by no-smbios data is truncated to 58 chars, which may end up truncating the UUID, which is the only parameter ensuring uniqueness.
So, when SMBIOS data is available (no-smbios == false) the default MachineInventory name got the UUID parameter as the first one (so it will not be truncated); when no SMBIOS is are available instead, the hostname will be set to a random DUID.
Fixes: https://github.com/rancher/elemental-operator/issues/144

we probably should just use the UUID as name? Not sure what the product name and manufacturer provide here, we could simply auto add some of those values as labels directly on the registration using the smbios data like elemental.cattle.io/manufacturer: X instead.
In any case, just having the uuid which should be unique as a default name if the registration does not provide an override, looks good to me.

Codecov Report

Merging #145 (74f5dd0) into main (6a547a7) will decrease coverage by 10.43%.
The diff coverage is 12.53%.

@@             Coverage Diff             @@
##             main     #145       +/-   ##
===========================================
- Coverage   33.77%   23.33%   -10.44%
===========================================
  Files           5        7        +2
  Lines         379      617      +238
===========================================
+ Hits          128      144       +16
- Misses        246      457      +211
- Partials        5       16       +11

Impacted Files
Coverage Δ

pkg/register/register.go
0.00% <0.00%> (ø)

pkg/server/server.go
0.00% <0.00%> (ø)

pkg/server/register.go
13.65% <9.39%> (-10.33%)
:arrow_down:

pkg/register/websocket.go
34.21% <34.21%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Please rebase

Looks ok to me. Please add a test case !
Added unit tests

@fgiudici remeber the docs! they need to be changed as the default has changed!

@fgiudici remeber the docs! they need to be changed as the default has changed!

🤦🏼 thanks, I forgot! Will fix now 😄


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #18.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

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

ℹ No valid build backend found for Python recipe for package glad2 using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ The recipe is not parsable by parser conda-souschef (grayskull). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12104983497. Examine the logs at this URL for more detail.

I didn't understand the organization sorry.

I didn't understand the organization sorry.

For context on the history of the feedstock, check https://github.com/conda-forge/glad-feedstock/pull/5#issuecomment-1455119324

Just to understand, where is this needed?

Trying to build out imgui bundle and getting stuck on OpenGL for osx. I saw something bout glad

see: https://github.com/conda-forge/staged-recipes/pull/28413


feature support insert ignore

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

We already have another implementation with a different semantics https://github.com/apache/doris/pull/21773

We already have another implementation with a different semantics #21773

ok


Resolve the conflict run button in editor context menu

Problem
Since the contribution point: editor/title is open for all the extensions, sometimes different extensions may have conflicts at this area. For example, in Java, such conflicts affect the run experience when the user installs both the Java extensions and the Code Runner extension:

This is a very open and big topic, as a author of VS Code extensions, it will be great if VS Code as a platform, can provide solutions for this issue.
Potential Solutions
Below are just rough ideas on this, it's open for discussion!
Editor Metadata
Extensions can register context value per document/editor, and register command on the editor title area according to the context value. For example. the Java Debugger can use this to mark if the current Java file is executable or not. And register the run/debug command into the editor context area if it's executable.

This can somehow achieved by using the in expression of the when clause, something like resource in hasMainMethodFiles. But we also need to get the context value dynamically from the code to handle with the corporation between multiple extensions.

Meanwhile the Java feature team can contribute changes to the Code Runner extension to align the UX (for example, hide the run button from Code Runner if the current Java file contains an executable Main class).
Official Support for the run experience in the editor title area.
This also may have some opportunity since I believe it's somehow related with #85759, if VS Code team will consider provide official run/debug functionality area in the editor title.

//CC @weinand

//CC @weinand

From where I stand this looks like a feature for "Run & Debug", e.g not only have those commands in the views but also in editor titles. Tho, it would still mean that you have multiple "runners", just behind a nice UI.

From where I stand this looks like a feature for "Run & Debug", e.g not only have those commands in the views but also in editor titles. Tho, it would still mean that you have multiple "runners", just behind a nice UI.

@jrieken Yes, if the VS Code platform can provide a UI to better organize different runners, that's great!

@jrieken Yes, if the VS Code platform can provide a UI to better organize different runners, that's great!

Currently VS Code debugging has no notion of "runners" because regular commands are sufficient to "run" something and there is no additional semantics associated with those commands.
Adding the notion of "runners" as a debug specific tagging mechanism for commands in order to get a special UI treatment, sounds a bit like a hack to me.
Couldn't we not try to find a general solution to this problem?
E.g. something like this:
if multiple command contributions use the same icon, they are grouped together in a drop down menu that uses the same icon.
If this "implicit" grouping mechanism appears a bit too "magical", we could introduce a new "explicit" grouping property.

Currently VS Code debugging has no notion of "runners" because regular commands are sufficient to "run" something and there is no additional semantics associated with those commands.
Adding the notion of "runners" as a debug specific tagging mechanism for commands in order to get a special UI treatment, sounds a bit like a hack to me.
Couldn't we not try to find a general solution to this problem?
E.g. something like this:
if multiple command contributions use the same icon, they are grouped together in a drop down menu that uses the same icon.
If this "implicit" grouping mechanism appears a bit too "magical", we could introduce a new "explicit" grouping property.

If this "implicit" grouping mechanism appears a bit too "magical", we could introduce a new "explicit" grouping property.

Well, that would be a submenu, e.g we would define a submenu that sits in an editor title area and commands are contributed into that (instead of the editor title area directly).

Adding the notion of "runners" as a debug specific tagging mechanism for commands in order to get a special UI treatment, sounds a bit like a hack to me.

Assuming we have said submenu I think it would still be beneficial if debug contributes to that menu directly

If this "implicit" grouping mechanism appears a bit too "magical", we could introduce a new "explicit" grouping property.

Well, that would be a submenu, e.g we would define a submenu that sits in an editor title area and commands are contributed into that (instead of the editor title area directly).

Adding the notion of "runners" as a debug specific tagging mechanism for commands in order to get a special UI treatment, sounds a bit like a hack to me.

Assuming we have said submenu I think it would still be beneficial if debug contributes to that menu directly

And the submenu only appears if two or more commands are contributed to it?

Assuming we have said submenu I think it would still be beneficial if debug contributes to that menu directly

individual debug extensions may contribute to the menu, but VS Code does not (because there is no generic "run")

And the submenu only appears if two or more commands are contributed to it?

Assuming we have said submenu I think it would still be beneficial if debug contributes to that menu directly

individual debug extensions may contribute to the menu, but VS Code does not (because there is no generic "run")

And the submenu only appears if two or more commands are contributed to it?

@joaomoreno knows the details here but it's up to us how we define the behaviour of submenus in toolbars

individual debug extensions may contribute to the menu, but VS Code does not (because there is no generic "run")

Yeah, that's what I mean

And the submenu only appears if two or more commands are contributed to it?

@joaomoreno knows the details here but it's up to us how we define the behaviour of submenus in toolbars

individual debug extensions may contribute to the menu, but VS Code does not (because there is no generic "run")

Yeah, that's what I mean

@joaomoreno does this proposal makes sense?
I propose that we address the issue of "conflicting run buttons" by a mechanism that automatically creates a submenu if two or more commands are contributed to the same "location" (or same contribution path, same ID, same icon etc.)
Since the resulting submenu shows the label instead of the icons, it makes the commands clearly distinguishable. If only a single command is contributed, no submenu is used and the command is directly shown in the editor's title area toolbar.

@joaomoreno does this proposal makes sense?
I propose that we address the issue of "conflicting run buttons" by a mechanism that automatically creates a submenu if two or more commands are contributed to the same "location" (or same contribution path, same ID, same icon etc.)
Since the resulting submenu shows the label instead of the icons, it makes the commands clearly distinguishable. If only a single command is contributed, no submenu is used and the command is directly shown in the editor's title area toolbar.

It makes sense, but requires implementing. This isn't the first time it's been asked for btw. Specifically:

If no actions are in the submenu, it doesn't appear at all
If one action is in the submenu, there is no submenu, only action
If more than one action is in the submenu, they appear as a submenu

Now, I wonder if this behavior should be an opt-in option on the submenu, instead of the default for all. I can imagine some situations in which having a dynamically changing UX is detrimental, since there's always the motivation to keep UI as stable as possible, especially around menus. Thoughts @jrieken?

It makes sense, but requires implementing. This isn't the first time it's been asked for btw. Specifically:

If no actions are in the submenu, it doesn't appear at all
If one action is in the submenu, there is no submenu, only action
If more than one action is in the submenu, they appear as a submenu

Now, I wonder if this behavior should be an opt-in option on the submenu, instead of the default for all. I can imagine some situations in which having a dynamically changing UX is detrimental, since there's always the motivation to keep UI as stable as possible, especially around menus. Thoughts @jrieken?

I propose that we address the issue of "conflicting run buttons" by a mechanism that automatically creates a submenu if two or more commands are contributed to the same "location" (or same contribution path, same ID, same icon etc.)

There should be no magic here, no comparing of ids or icons. What should happen is that extensions contribute their commands to a new menu, e.g. editor/title/run. We then render the menu as submenu in the editor title area using the smartness that @joaomoreno has described. Tho, it does mean that extensions need to adopt and contribute to the new menu, not to editor/title anymore

I propose that we address the issue of "conflicting run buttons" by a mechanism that automatically creates a submenu if two or more commands are contributed to the same "location" (or same contribution path, same ID, same icon etc.)

There should be no magic here, no comparing of ids or icons. What should happen is that extensions contribute their commands to a new menu, e.g. editor/title/run. We then render the menu as submenu in the editor title area using the smartness that @joaomoreno has described. Tho, it does mean that extensions need to adopt and contribute to the new menu, not to editor/title anymore

Adding to January for exploration

Adding to January for exploration

Yes, no "magic". That was the conclusion of my proposal:

Since the submenu item needs an icon and probably a label, an open question is where does those come from?
Deriving them from the first command that goes into the submenu would result in an unpredictable UI. A better approach would be to explicitly contribute a submenu item (with icon and label), but keep it invisible as long as the there are no contributions to it.

Yes, no "magic". That was the conclusion of my proposal:

Since the submenu item needs an icon and probably a label, an open question is where does those come from?
Deriving them from the first command that goes into the submenu would result in an unpredictable UI. A better approach would be to explicitly contribute a submenu item (with icon and label), but keep it invisible as long as the there are no contributions to it.

I can imagine some situations in which having a dynamically changing UX is detrimental, since there's always the motivation to keep UI as stable as possible, especially around menus. Thoughts @jrieken?

@joaomoreno I don't worry too much about changing UI. We already have that today, based on file type, being under git, etc. With when-clauses all the good and bad can be made and that won't change here.

I can imagine some situations in which having a dynamically changing UX is detrimental, since there's always the motivation to keep UI as stable as possible, especially around menus. Thoughts @jrieken?

@joaomoreno I don't worry too much about changing UI. We already have that today, based on file type, being under git, etc. With when-clauses all the good and bad can be made and that won't change here.

FYI on the Python extension we actually hide our run button when we detect Code Runner installed; we use the colour of the button to help us tell which one people are using.

@brettcannon you would still be able to do that but we ask you to contribute into the new submenu instead of the editor title menu

This will be available to all extensions via the editor/title/run-menu

This will be available to all extensions via the editor/title/run-menu

@jrieken  Does this apply to the Notebook Editor as well?

Yes - all editors

Thank you @jrieken !  😄

but it's not displayed in the dropdown list. Is it by design?

I guess but I would accept an issue for that ;-)


0.7.3 - Next problem with SAML auth (Google)

All users gets error 403 with response:
invalid_request unknown IdP

Thanks for the report @ihard -- working to reproduce.

Hi @Insidek -- this should be fixed in 0.7.4, along with a few other SAML fixes. I've tested it's working with Google Workspace. Give it a go and let us know if you hit any snags.


Azure: Standard_L8as_v3 not recognized by cloud setup

instance: Standard_L8as_v3
Scylla 2022.2.3 + machine image 2022.2.3
root@gc-scylla-test001:/home/azureuser# sudo /opt/scylladb/scylla-machine-image/scylla_cloud_io_setup

ERROR:root:This is not a recommended Azure Cloud instance setup for auto local disk tuning.
Traceback (most recent call last):
 File "/opt/scylladb/scylla-machine-image/libexec/scylla_cloud_io_setup", line 357, in <module>
   io.generate()
 File "/opt/scylladb/scylla-machine-image/libexec/scylla_cloud_io_setup", line 298, in generate
   raise UnsupportedInstanceClassError()
TypeError: UnsupportedInstanceClassError() missing 1 required positional argument: 'Exception'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
 File "/opt/scylladb/scylla-machine-image/libexec/scylla_cloud_io_setup", line 359, in <module>
   except UnsupportedInstanceClassError as e:
TypeError: catching classes that do not inherit from BaseException is not allowed
root@gc-scylla-test001:/home/azureuser#

@syuu1228 @yaronkaikov ^

actually, even with above, the fallback to iotune should still work, is above an iotune bug?

@tarzanek is it possible to work with 5.2 or 2023.1? all Azure verification started officially from those releases

@gcarmin can you use 2023.1 RC ?

Any Official release starting from 2023.1.X and 2024.1.X should support this type.
Closing this issue, if you facing this issue in an official release, please feel free to open it again


[pulsar-perf] Write histogram files for consume command

Motivation
The consume command for pulsar-perf doesn't generate histogram file (.hgrm files) like its counterpart produce which makes it more difficult to automate collection of consumer performance results and basically necessitates sending metrics to prometheus, etc.
Modifications
Write the contents of the histogram to a .hgrm file like the produce command.
Verifying this change

[ ] Make sure that the change passes the CI checks.
[x] This change is a trivial rework / code cleanup without any test coverage.

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

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

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc

The change brings behaviour inline with the produce command and I'm not sure additional docs are needed.

@eolivelli Please help review the PR.


Crashing while wandering around

https://bytebin.lucko.me/vyNWVnpOuB

Too few information, and such issue will not be adopted.

Too few information, and such issue will not be adopted.

If you need more information, just ask.
The log hast all technical information you need.


Field containing '%' causes formatflagsconversionmismatchexception in db dump

java.util.FormatFlagsConversionMismatchException at TestLoadFilesIT.java:199
Seems to happen when you attempt to dump a database record containing a percentage sign in a column. The text is fed through String.format and percentage characters are treated as a placeholder field.
Literal % signs need to be escaped before calling format().

Thanks for the feedback !
Could you write a small unit test showing the problem ?

During tests, i identify a problem with a column name with %.
It was a format exception with String.format(...).
I close this issue and hope it was the same problem.