lstm example and some changes to softmax

This looks nice! I hope you don't mind if I make a few style changes and update the example to work with py3k.

Definitely feel free to do so. I was hoping that you do some style changes!
Currently, I have difficulty getting multi-layer LSTM converging as fast as
currennt software on the same example.
One issue is I do not handle the sequences right since I concatenate all
sequences together which is not the right thing to do but it should not
affect the results too much.
Also, during validation I do not separate sequences but concatenate them
all together.
I think it would help to be able to handle different sequence lengths. I
need to write code that says ignore these frames (or steps) at the end of a
sequence which do not correspond to real data. This can be done using a
mask parameter for the targets. I think this warrants special handling in
theanets since unequal sequence lengths are always going to happen.
currennt converges in less than 100 epochs to about 20% error on the
validation data using a 3 layer bidirectional LSTM  setup.
I tried currennt with LSTM (no bidirectionality) with layers
(39,100,200,76,51) which converges to 28% error in less than 70 epochs.
currennt uses NAG for training. Objective function uses sum of negative
logs instead of the mean of them like in theanets. So, I try to adjust the
learning rate accordingly in theanets.
But, I achieved 45% error in 400+ epochs in theanets which is worse than a
single layer LSTM. Next, I will try layerwise training which should work
much better I think since single layer works.
Any ideas about why this may be happening?
Hakan
On Thu, Jan 8, 2015 at 12:10 PM, Leif Johnson<EMAIL_ADDRESS>wrote:

This looks nice! I hope you don't mind if I make a few style changes and
update the example to work with py3k.
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-69212275.

Well, I checked in a rather large pruning-down of the code in your example. Again, I hope you don't mind.
I expanded the size of the LSTM layer to 500 but didn't see a huge improvement in performance. I'm not sure why this might be happening, but it might be a good thing to discuss on the mailing list?
Also, I see what you mean about supporting masks for network outputs. I've filed an issue to keep track of this: #58.

Also, when you say that you tried curennt with layers (39, 100, 200, 76, 51), what does that mean? Is that the number of units in each layer of a network with an input, 3 hidden layers, and an output? Are all of those layers LSTM, or just one of them?
You can create a network with lots of layers in theanets by giving the layers tuple (39, 100, 200, 76, 51). Then you can specify which of those layers is to be recurrent by providing recurrent_layers=(1, 3) (for example; would make the 100 and 76 layers be recurrent).

Yes, I realized yesterday that I was doing it wrong. Actually, in currennt
it is 3 hidden LSTM layers on top of each other.
In theanets, I was using only one LSTM in the middle and it was not the
same, however I assumed they were the same but found out that they were not
yesterday.
I need to correct this and re-run it later. I guess that may be the reason
of discrepancy in error rate achieved.
Also, a reverse LSTM can be done using go_backwards argument of scan. One
forward and one backward LSTM layer outputs can be concatenated in another
layer on top of them to obtain a bidirectional LSTM. Or one can write a
bidirectional LSTM layer. The mask thing is also very important for the
bidirectional one.
Thanks.
Hakan
On Fri, Jan 9, 2015 at 12:07 PM, Leif Johnson<EMAIL_ADDRESS>wrote:

Also, when you say that you tried curennt with layers (39, 100, 200, 76,
51), what does that mean? Is that the number of units in each layer of a
network with an input, 3 hidden layers, and an output? Are all of those
layers LSTM, or just one of them?
You can create a network with lots of layers in theanets by giving the
layers tuple (39, 100, 200, 76, 51). Then you can specify which of those
layers is to be recurrent by providing recurrent_layers=(1, 3) (for
example; would make the 100 and 76 layers be recurrent).
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-69364772.

There seems to be a problem with training deep networks with more than one
hidden layer, including feedforward ones. I experimented with
mnist_classifier and lstm_chime examples. It may be expected due to
vanishing gradients but even layerwise training does not seem to work. For
the lstm chime example, we know currennt works with multiple hidden layers
and it uses simple stochastic gradient descent with momentum without
further tricks. In theanets, I almost always get better results with
shallow networks as compared to the deep ones. It is very hard to see what
may be the problem though. I am not sure if the problem is with theano or
theanets. Any ideas?
Hakan
On Fri, Jan 9, 2015 at 2:09 PM, Hakan Erdogan<EMAIL_ADDRESS>wrote:

Yes, I realized yesterday that I was doing it wrong. Actually, in currennt
it is 3 hidden LSTM layers on top of each other.
In theanets, I was using only one LSTM in the middle and it was not the
same, however I assumed they were the same but found out that they were not
yesterday.
I need to correct this and re-run it later. I guess that may be the reason
of discrepancy in error rate achieved.
Also, a reverse LSTM can be done using go_backwards argument of scan. One
forward and one backward LSTM layer outputs can be concatenated in another
layer on top of them to obtain a bidirectional LSTM. Or one can write a
bidirectional LSTM layer. The mask thing is also very important for the
bidirectional one.
Thanks.
Hakan
On Fri, Jan 9, 2015 at 12:07 PM, Leif Johnson<EMAIL_ADDRESS>wrote:

Also, when you say that you tried curennt with layers (39, 100, 200, 76,
51), what does that mean? Is that the number of units in each layer of a
network with an input, 3 hidden layers, and an output? Are all of those
layers LSTM, or just one of them?
You can create a network with lots of layers in theanets by giving the
layers tuple (39, 100, 200, 76, 51). Then you can specify which of those
layers is to be recurrent by providing recurrent_layers=(1, 3) (for
example; would make the 100 and 76 layers be recurrent).
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-69364772.

Yeah, not sure what might be the problem here. As far as I can tell the optimization code in theanets is correct (for what it's worth, the optimization code in benanne/Lasagne is basically the same). I unfortunately have a hard time figuring out how currennt works -- I've pored over the lstm code there and it looks to me like it basically shares the same implementation as theanets.
Also, I've noticed that the mnist-autoencoder.py example reaches about the same level of performance with anywhere between 1 and 4 hidden layers (using the rmsprop optimizer), so it does seem like the trainers are working.
But now I am starting to wonder: maybe the problem is in the cross entropy cost or the softmax that defines the output of the classifier networks? I'll see if I can poke a little at that tonight.

These optimizers are pretty hard to get right.  It is a work in progress here if you are curious
https://github.com/kastnerkyle/net/blob/master/net.py
Mostly in the TrainingMixin class. But there are some hacks in there right
now, trying to figure out where the best time to clip is. We have some
small disagreement between code bases in our lab about "where to clip" :)
On Mon, Jan 12, 2015 at 9:51 PM, Leif Johnson<EMAIL_ADDRESS>wrote:

Yeah, not sure what might be the problem here. As far as I can tell the
optimization code in theanets is correct (for what it's worth, the
optimization code in benanne/Lasagne is basically the same). I
unfortunately have a hard time figuring out how currennt works -- I've
pored over the lstm code there and it looks to me like it basically shares
the same implementation as theanets.
Also, I've noticed that the mnist-autoencoder.py example reaches about the
same level of performance with anywhere between 1 and 4 hidden layers
(using the rmsprop optimizer), so it does seem like the trainers are
working.
But now I am starting to wonder: maybe the problem is in the cross entropy
cost or the softmax that defines the output of the classifier networks?
I'll see if I can poke a little at that tonight.
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-69687715.

Yeah, that's some dark magic right there! :)
Hakan, if you want to give it a try, you can add gradient clipping with gradient_clip=10 (or whatever you want the max gradient norm to be) when you train your model. I haven't messed around with that myself, but it's in theanets because I read about it in a paper. :) Kyle, FWIW I've implemented the "clip the gradient before adding to momentum" method.

Cool, I moving there too. Clipping after momentum felt hacky... hopefully
this will work better. Can't wait for LSTM to be in theanets, it should
give lots of people a chance to try interesting stuff. Maybe I can
contribute some demos once I find something interesting that is pretty easy.
On Mon, Jan 12, 2015 at 10:10 PM, Leif Johnson<EMAIL_ADDRESS>wrote:

Yeah, that's some dark magic right there! :)
Hakan, if you want to give it a try, you can add gradient clipping with
gradient_clip=10 (or whatever you want the max gradient norm to be) when
you train your model. I haven't messed around with that myself, but it's in
theanets because I read about it in a paper. :) Kyle, FWIW I've
implemented the "clip the gradient before adding to momentum" method.
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-69689182.

Yes, I found out currennt also does gradient clipping (limitedError
function) for the LSTM gradients which I did not pay much attention before.
Still, I could not get mnist classifier example work with deeper nets in
theanets. That one should work with layer wise training. I suspected
softmax or cross entropy cost as well for that. May be it helps to add a
small number before taking the logarithm in the cross-entropy cost.
Hakan
On Mon, Jan 12, 2015 at 10:10 PM, Leif Johnson<EMAIL_ADDRESS>wrote:

Yeah, that's some dark magic right there! :)
Hakan, if you want to give it a try, you can add gradient clipping with
gradient_clip=10 (or whatever you want the max gradient norm to be) when
you train your model. I haven't messed around with that myself, but it's in
theanets because I read about it in a paper. :) Kyle, FWIW I've
implemented the "clip the gradient before adding to momentum" method.
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-69689182.

I prefer to clip the values before the logarithm instead of always adding a
constant, but same idea. Just want to keep log from blowing up - I use
T.clip(X, 1E-12, 1E12) and it seems ok. though in theory probability should
be between 0 and 1 so T.clip(X, 1E-12, 1.) might be reasonable too...

OK. Could the multi-layer problem related to this subtle issue here:
https://github.com/benanne/Lasagne/issues/97
What I mean is, could it be the case that when computing multiple outputs in a network (each hidden layer output is a different output right?), theanets may be using a different batch each time due to randomness in the input batch selection (when the input is a callable, it may keep calling the function each time with a different batch for each output separately). Also, gradient calculation may be doing something funny as well.
So if this is the case, it would also slow down the computation, since for each output computed, it would run forward pass from the input with a different batch.
Probably, I am wrong. But this may be worth investigating.

This is an interesting thought. I've spent a little time looking into it, but I'm not sure it applies to theanets because we onyl construct the outputs graph once. I will keep looking at it, though, since it would be nice to figure out what's up with this slow training issue.
I noticed one more "trick" in the currennt code: it looks like they bound the cross-entropy error to the range +/- 100: http://sourceforge.net/p/currennt/code/ci/master/tree/currennt_lib/src/layers/CePostOutputLayer.cu#l95 -- this would act like another form of gradient clipping, I think.
There might also be some differences in how the weights of the model are being initialized that could have a big impact on things. I'm about to check in a change to the LSTM forget biases that I just read about, maybe that will help.

Alex Graves uses this with various thresholds for different tasks -
sometimes 10 or 100. I believe there is a PR for theano to make it an
option of grad . Saw it on the mailing list today.
On Jan 14, 2015 11:27 PM, "Leif Johnson"<EMAIL_ADDRESS>wrote:

This is an interesting thought. I've spent a little time looking into it,
but I'm not sure it applies to theanets because we onyl construct the
outputs graph once. I will keep looking at it, though, since it would be
nice to figure out what's up with this slow training issue.
I noticed one more "trick" in the currennt code: it looks like they bound
the cross-entropy error to the range +/- 100:
http://sourceforge.net/p/currennt/code/ci/master/tree/currennt_lib/src/layers/CePostOutputLayer.cu#l95
-- this would act like another form of gradient clipping, I think.
There might also be some differences in how the weights of the model are
being initialized that could have a big impact on things. I'm about to
check in a change to the LSTM forget biases that I just read about, maybe
that will help.
—
Reply to this email directly or view it on GitHub
https://github.com/lmjohns3/theanets/pull/56#issuecomment-70038409.

Nice discussions guys.
Some comments/ideas:

In a backwards recurrent layer, I think the output should be reverted in time, since scan is supposed to produce the output in the backwards order when go_backwards is True. I saw this in the lasagne/nntools implementation of RNN/LSTM. Interestingly the network still learns something even if the output is not reversed, but it is not the best one can do since it uses output at another time to predict the target at another time along a sequence.

The theano function used for learning (trainer.f_learn) is very complex right now with many outputs. May be there may be an option to have as few outputs as possible (e.g. no monitors) which may speed up compilation and runtime due to better optimization. I am not sure if it is possible to have no outputs at all (just have updates that depend on the gradient of the cost for example)

last line of feedforward.py
return self.predict(x).argmax(axis=1) could be
return self.predict(x).argmax(axis=-1)
to make it work for 3D sequence inputs, otherwise not correct for batched sequence data...

Any plans to implement the mask idea for sequence data soon? Then we could directly compare with currennt and even load currennt network parameters into theanets and see if they both work the same.

This comment from craffel https://github.com/benanne/Lasagne/issues/17#issuecomment-70204547 seems to verify my intuition about higher layers recomputing the earlier layer outputs again and again. If each layer goes all the way to the input (when it is a callable which picks data at random), each layer could be using different data for computing their outputs and we would be in for a mess!


feat: allow controllers to call __motoko_stable_var_info query endpoint

This was originally spec-ed, but there was no possibility to implement it (a canister couldn't get it own controller list).
In the meantime canisters can get their controller list (via query), but a query cannot call another query 🫤.
So we had to wait until the ic0.is_controller syscall became available. Now it is there and the implementation is easy.
(It was technically possible to call it before, but it invariably trapped.)

I guess we could now provide a helper in ExperimentalInternetComputer to call this on another canister than self, but not urgent.


MVP-2257: Support previewing view by inputing demoByJSONConfig arg

Have NotifiSubscriptionCard support one more arg demoByJSONConfig to show the demo preview. See the the following.
https://user-images.githubusercontent.com/127958634/227394005-369d9696-c9dc-44aa-83df-7a3a12d9831c.mov

Issue addressed. Avoid conditional call of hook and move the preview check logic into hook.

Refactor note:

Instead of passing demoPreview data from NotifiSubscriptionCard, now using NotifiContext to do so.
Instead of handling demoPreview related logic everywhere, centralising these into new context NotifiDemoPreviewContext.
Make sure all the card components not changing.

Overview :

@eric-notifi hey eric did you mean to merge and close?


Big std::strings in nvarchar(max)s via ODBC fails with HY104

Hi, soci throws odbc_soci_error when the std::string I use is very long (4096 worked but failed at 29906).
The ODBC error code is HY104, so I guess soci does not handle big strings right (to be fair due typical ODBC driver implementation limitations), or I overlooked the way to handle them with soci. The according column is defined as nvarchar(max), the server is an MS SQLServer 2008RC2 and the ODBC-Driver uses the SQL Server Native Client 11.0 on Windows 7 Prof.

If the driver doesn't handle strings of this size, I don't think SOCI should do anything about it. Try using BLOBs instead of the strings instead perhaps?

@vadz AFAIK is supposed to work see https://msdn.microsoft.com/de-de/library/ms130896.aspx

The sole reference of SQLBindParameter parameter ColumnSize does not say it all.
AFAIU, in case a large object exceeds certain range (see below), then the ColumnSize is not set with actual size of the object, but with special constant. See Binding Parameters for SQL Character Types:

If the SQL data type passed in is a character type, ColumnSize is the size in characters (not bytes). If the length of the data string in bytes is greater than 8000, ColumnSize should be set to SQL_SS_LENGTH_UNLIMITED, indicating that there is no limit to the size of the SQL type.
For instance, if the SQL data type is SQL_WVARCHAR, ColumnSize should not be greater than 4000. If the actual data length is greater than 4000, then ColumnSize should be set to SQL_SS_LENGTH_UNLIMITED so that nvarchar(max) will be used by driver.

I may be misunderstanding, so it would be good if someone could confirm that.
Currently, SQL_SS_LENGTH_UNLIMITED case is not handled.

I could try testing this, but what exactly is the test case here? I.e. does it happen when writing a long value to the database or when reading it from it?

Writing

Yeah, it's always helpful if a bug report is accompanied with (pseudo)code sample or, preferably, CATCH-based test case.

I understand here is my preliminary example, closely based on the setup I encountered it (MS SQLServer 2008RC2 and the ODBC-Driver uses the SQL Server Native Client 11.0 on Windows 7 Prof.)
I will try to deliver a catch based test, but I don't visit my workplace until next week, so please wait until next week for a better example, i.e. reproduced with a different setup.
soci::register_factory_odbc();
soci::connection_parameters main_session_parameters(soci::odbc, ODBC_CONNECTION_STRING);
main_session_parameters.set_option("odbc_option_driver_complete", "0");
soci::session current_session(main_session_parameters);

string log = {/*Long string bigger > 8000 */}

current_session << "UPDATE files SET ... uploader_log = :log where ...", ... soci::use(log, "log"); // Will throw soci::odbc_soci_error as mentioned above

Table:
CREATE TABLE files (
file_id uniqueidentifier NOT NULL PRIMARY KEY DEFAULT NEWSEQUENTIALID() ROWGUIDCOL,

...
uploader_log nvarchar(max) NULL,

CONSTRAINT unique_file UNIQUE(directory_id,file_name)
);

I don't see the error but this could be due to ODBC driver difference. I do see the problem with the string being just truncated to 8000 characters silently which is arguable even worse. The PR #328 fixes it for me, please test if it helps with your problem too or if it's a different one.

@Superlokkus  :+1:

Should be fixed by c025cc86968914bd4bd0523dee11cdd9db991aba

Since AFAIK your fix commit c025cc86968914bd4bd0523dee11cdd9db991aba is included in today's master: Is not fixed by your commit. MS SQL still reports
[2015-Sep-30 10:27:56.736424, 0x00001a48] 4: Cought execption when tried to register file: Error binding input parameter #2: [Microsoft][SQL Server Native Client 11.0]Invalid precision value (SQL state HY104) while executing "UPDATE files SET read_time = :time, partially_uploaded = 1,uploader_log = :log where file_name = :name AND directory_id = (select directory_id from directories where full_path = :path)" with :name="16689.VS", :path="\\<IP_ADDRESS>\Data_Share_RHE\RHE_OPERATIONS\01_Fertigung\05_Messtechnik\02_Elektrisch\Novell\TESTDATA\SSR1000", :time=2015-09-30 10:27:56, :log="Error: Invalid data format!

I really try to get a test case working, but I'm still a C++ novice.

Sorry, I'm really not sure how/why did this happen, but it turns out that c025cc8 has never been merged into master. I've just (finally) done it now, sorry for the confusion.

@vadz it was fixed in #383. Please revert your last commit on master. Because it breaks the tests (ref appveiour results)

Was AppVeyor not broken before?

@mloskot Not in odbc tests - thanks for history

@snikulov OK, I might have not paid enough attention

Oops, sorry about this, I didn't realize it had been applied and then replaced by 0723512f39a21cd78ebee017a84fb3f617b3e4ba, reverted now.


Animated stickers don't work

Current implementation doesn't support sending animated stickers even though they are in the getUrl function

only png stickers supported yet. the getUrl function has other types just for future


Toggle Search Details in Search changes height for Search Results

Issue Type: Bug

Search for a term
Toggle Search Details
Search results height become shorter

After toggling the search details a few times, the search results disappear.
VS Code version: Code 1.51.0 (fcac248b077b55bae4ba5bab613fd6e9156c2f0c, 2020-11-05T18:14:40.758Z)
OS version: Darwin x64 19.6.0

That's weird and I can't repro. Do you see any errors in the devtools (Help > Toggle Developer Tools)


Extremely slow Context Menu launch in Windows Explorer

I have a modern laptop (Dell XPS13-9310) with 16Gb of RAM, i7-1195G7 @ 2.9GHz (4 physical cores) CPU, big SSD, Win11 22H2 (22621.2715)
Still, every launch of Windows Explorer takes a second or more - for drawing the file tree, the file icons.
Worse, any first context menu click in Windows Explorer or Desktop is flickering and takes like 500ms up to 1-2 seconds and has flickers (first a smaller viewport gets drawn which later gets expanded to fit all the items and then it flickers one more time). I saw this being reported on answers.microsoft.com for a long time since Win11 release, but this is still present on 22H2 (22621.2715)
Probably this is because various XAML technologies taking too much time somewhere and maybe requiring some uncached disk access or parsing or library loading or icon fetching/drawing. Unfortunately, I don't have skills or time for properly tracing and debugging this (I'll try my chance later to have it video-taped with Win+G), but if this is the proper place, maybe someone will add some traces.

Confirmed with the team that opening a Feedbackhub issue, and repro'ing the issue, is the correct path forward to engage with the team. Feel free to post the url so we can track it.

closing due to lack of activity


Feat/back buttons

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Do not change the following line aside from replacing YOUR-ISSUE-HERE with the issue this PR fixes
fixes #399
Please provide enough information so that others can review your pull request:
Explain the details for making this change. What existing problem does the pull request solve?
These changes add back buttons to SurveyPage2 and SurveyPage3. Navigating back and forth in the survey saves whatever data is filled out in the redux store. After completing a survey and sending the data to the database, the redux store clears for any subsequent survey attempts.

Test plan (required)
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.

Final Checklist

[ ] Have you bumped the version in package.json?

Second decimal for major change, third decimal for minor change. This can go past 10 i.e. 1.0.9 !=> 1.1.0, 1.0.9 => 1.0.10
Read here for more info

[ ] Have you added any new tests necessary?
[ ] Is your PR rebased off the most current master?
[ ] Have you squashed all commits, or if you will merge will you squash all commits?
[ ] Did you use yarn, not npm?
[ ] Did you use Material-UI wherever possible?
[ ] Did you format according to Prettier?
[ ] Did you run all of your most recent changes locally to make sure everything is working?

Made requested changes.


react with nerd emoji when someone says they're a pro

I upgrade your bot pt.2

no 🤓

no 🤓

yes


chore(main): release 0.0.6

:robot: I have created a release beep boop
0.0.6 (2024-07-26)
Features

Azure plugin - remove Palette presets, reading permission set files (#97) (95787db)

Other

bump validator and plugin versions (#106) (a3863aa)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.0.6
:sunflower:


Update dependencies and translations to the most recent versions

/botio-linux preview

/botio test

What happened with the Firefox ref-tests on Windows, since there seem to be movement in every single test!?
Has something been changed on the bot, since I don't understand how this patch could "cause" this?

It looks like this started two days ago already in https://github.com/mozilla/pdf.js/pull/18918#issuecomment-2423015380, so indeed it doesn't seem related to this patch. Five days ago in https://github.com/mozilla/pdf.js/pull/18905 there were no issues yet, which was the last successful run I could find, and in all cases we used Firefox 133.0.

/botio-linux preview

/botio test

I have created #18928 to track the bot issues found above because they also happened on a dummy PR, which shows that they are not caused by this PR. In the interest of preparing for the next release (and because I plan to do more with the dependencies, and keeping package-lock.json rebased is a bit of a pain) I'd therefore propose to move on with this PR as-is, unless there are objections of course.

/botio integrationtest

What happened with the Firefox ref-tests on Windows, since there seem to be movement in every single test!? Has something been changed on the bot, since I don't understand how this patch could "cause" this?
/cc @calixteman

See my answer: https://github.com/mozilla/pdf.js/issues/18928#issuecomment-2425146808


feat(resources): adding document load resources

Which problem is this PR solving?

Fixes #456

Short description of the changes
It adds spans for each of the resources from  Resource Timing API

Codecov Report

Merging #478 into master will increase coverage by 0.01%.
The diff coverage is 100%.


Fix the parameters of first few layers

There are good examples on how to finetune from a pretrained model for MXNet:
http://shuokay.com/2016/09/02/transfer-learning/
https://ysfalo.github.io/2016/04/01/mxnet之fine-tune/
However, in these examples, actually the gradient of the first few layers were still computed during the training(just the learning rate is different). But I actually only want to train the last layer, and fix the parameters of the first few convolution layers, which would save a lot of time.
Is there anyway to do it in MXNet?

search for fixed_param_names in module

@piiswrong Thanks very much!!! Awaresome!

I have not got the search result, what do you mean search in module?


New Mode Setting Request

Hello Dear,
Request for adding a new model setting 'quiet' in English.
{"setting_name":"quiet","setting_values":[{"setting_synonym":["quiet", "silent'],"lang":"en"}]}
#295

issue #295 is how I found the problem, I marked it as related and closed.

In English, the setting_name is "Turbo_Silence", with "quiet" and "silent" as the values.


Release a new version

Lots of changes have happened after release# 32. Lets release a new version 33.
I am looking forward to the ipv6 fix.

@pronix probably a good idea, most notably with the fog version pinning commit.

done

Thanks :)


Instructions for C++ compiling?

When compiling C++, I've got an error:
$ cat hello3.cpp
#include <iostream>
int main()
{
  std::cout << "hello world from C++.\n";
  return 0;
}
$ gcc -o hello3 hello3.cpp
Undefined symbols for architecture x86_64:
  "__ZNSt8ios_base4InitC1Ev", referenced from:
      __Z41__static_initialization_and_destruction_0v in cc2q2AwP.o
  "__ZNSt8ios_base4InitD1Ev", referenced from:
      __Z41__static_initialization_and_destruction_0v in cc2q2AwP.o
  "__ZSt4cout", referenced from:
      _main in cc2q2AwP.o
  "__ZStlsISt11char_traitsIcEERSt13basic_ostreamIcT_ES5_PKc", referenced from:
      _main in cc2q2AwP.o
ld: symbol(s) not found for architecture x86_64
collect2: error: ld returned 1 exit status

Environnement:
$ gcc --version
gcc (GCC) 13.1.0
$ xcodebuild -version
Xcode 14.3.1
Build version 14E300c
$ uname -srm
Darwin 22.6.0 x86_64

What instructions are needed ?

You need to use g++ instead of gcc so as to get libstdc++ included. Or clang++, which pulls in libc++.

Arrgh, that's it, I'm really not used to C++ ;-).
Thanks Simon.


Added plotting to the case sample workflow closes #519

closes #519 @davidbenjamin Can you review this please?

@asmirnov done with review; just little comments to address.  Looks good!

@asmirnov239 Ready to rebase and merge.


chore(main): release 3.0.0

:robot: I have created a release beep boop
3.0.0 (2024-11-13)
⚠ BREAKING CHANGES

Remove classic greeting.

Features

Remove classic greeting. (957cbcb)

Bug Fixes

Make droids louder. (df5b61a)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v3.0.0
:sunflower:


Try to use Net-a-porter parser if httpclient fails

PR to use the Net-a-porter parser in case URLEncodedUtils.parse() fails

Cool! Scheduling...


fix: Adding missing permissions in the ECS service module

Description
ECS Cluster tasks can't start because this lack of permission on creating Log Group.
Motivation and Context
This fixes an error where a task that is started is allowed to create the AWS CloudWatch log group it needs to log too.
ResourceInitializationError: failed to validate logger args: create stream has been retried 1 times: failed to create Cloudwatch log group: AccessDeniedException: User: arn:aws:sts::xxxxxxxxxxx:assumed-role/xxxxxxxxx-idnumber00000xxxxx/uuid is not authorized to perform: logs:CreateLogGroup on resource: arn:aws:logs:us-east-1:xxxxxxxxxx:log-group:/ecs/xxxxxxxxxxxx:log-stream: because no identity-based policy allows the logs:CreateLogGroup action status code: 400, request id: xxxxxxxx-xxxxx-xxxxx-xxxxx-xxxxxxxxxxxx : exit status 1
Breaking Changes
It should not break any backwards compatibility,
How Has This Been Tested?
This was deployed to a Dev environment, the task failed to start up.
In the module in the .terraform folder I added the missing permission, re-ran the terraform apply which updated the ECS Role with the correct permission and the tasks started successfully.
I then ran a terraform destroy to remove everything, removed the .terraform folder and ran terraform apply again and got the same error as above. re-applied the above test steps and it succeeded

closing until more details can be provided


Assign specific programs to specific zones

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Assign a window (Chrome, Discord) to a zone. So, if I change zone layouts, program windows move accordingly to fill the zone they're associated with.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Have you tried these settings enabled?

Have you tried these settings enabled?

namely, the top one

namely, the top one

Thanks for the tip, but that's not quite what I'm looking for. What I'm talking about is adding a "process window" attribute to zones, or something to that effect.
For example:
I have two layouts with two zones each; in each layout zone 1 is on the left and zone 2 is on the right.
I assign Chrome to zone 1 on layout 1, and zone 2 on layout 2.
The behavior I'm describing would snap my Chrome window to zone 1 when I switch to layout 1, and zone 2 when I switch to layout 2.

Thanks for the tip, but that's not quite what I'm looking for. What I'm talking about is adding a "process window" attribute to zones, or something to that effect.
For example:
I have two layouts with two zones each; in each layout zone 1 is on the left and zone 2 is on the right.
I assign Chrome to zone 1 on layout 1, and zone 2 on layout 2.
The behavior I'm describing would snap my Chrome window to zone 1 when I switch to layout 1, and zone 2 when I switch to layout 2.

Duplicate of https://github.com/microsoft/PowerToys/issues/4235 tracked in https://github.com/microsoft/PowerToys/issues/6611

Duplicate of https://github.com/microsoft/PowerToys/issues/4235 tracked in https://github.com/microsoft/PowerToys/issues/6611


Health check

Is a call to https://api.resend.com/ a health check and if it is, can this fetch request have a dedicated method in the Resend class.
It could make easy to create a status check for email service.

hey there, a call to our base path is not necessarily a base path and it will not necessarily respond with specific fields if say only part of the endpoints are down for some reason.
I would say that a more accurate and reliable way for you to understand the status of our systems is to subscribe to https://resend-status.com/, where we post incident statuses as they happen.
in the latest versions of the SDK we are now throwing an error if we can't reach the API:
{
  name: 'application_error',
  message: 'Unable to fetch data. The request could not be resolved.',
}

so feel free to check for this specific case if you need to in your code 🙏
thanks!


fix the bug will make the error  in the file dynamic-import-vars.ts

fix the bug will make the error object null is not iterable (cannot read property Symbol(Symbol.iterator)) in the file dynamic-import-vars.ts

  /*
  When use import(`@/${var}`) in the code, and this code's file path is the sub dir of the src,
  and make the src dir path name to be a alias name: @, like this:
  ├─┬ src
  │ ├─┬ router
  │ │ └── dyncRoute.js

  If the code 'import(`@/${var}`)' is used in dyncRoute.js, the variable glob will be the value `../*`
  */
  // this value will make the expression `glob.match(/(.*\w\/?)\*/)` in the old code to get null,
  // and expression `const [, importPath] = null` will occur an error with messaged
  //`object null is not iterable (cannot read property Symbol(Symbol.iterator))`, so I fixed the error as follow

  // const [, importPath] = glob.match(/(.*\w\/?)\*/)
  //                                    👇🏻
  const [, importPath] = glob.match(/(.*\/?)\*/)

https://github.com/caoxiemeihao/vite-plugins/blob/ac55271950c5c0e7977a153eb44c5ba9a59ba095/packages/dynamic-import/src/dynamic-import-vars.ts#L163

Can you release a new version to fix this bug?

Can you release a new version to fix this bug? @caoxiemeihao

Still have BUG,  has now been fixed | https://github.com/caoxiemeihao/vite-plugins/commit/166b4959820a5e2a2578033025e991f3de2775c5
Now, you can try to install<EMAIL_ADDRESS>


fix(apps): remove fig - sunset

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


Update Cognitive Services Scanner

Describe the feature
Update the Cognitive Services Scanner to differentiate between the different kind of Cognitive services.
Mostly to have the correct naming conventions following the CAF guides.
Kinds I've already identified:

OpenAI
Face
ComputerVision
ContentModerator
ContentSafety
CustomVision.Prediction
CustomVision.Training
FormRecognizer
HealthInsights
ImmersiveReader
TextAnalytics
SpeechService
TextTranslation

Link

You can assign it to me, I'll handle it one of the coming days!


Include Angular version in interpolate context

Including the Angular version would allow a localLocationPattern like "https://cdnjs.cloudflare.com/ajax/libs/angular.js/{{version}}/i18n/angular-locale_{{locale}}.js" for those who wish to use a CDN

If you would make it angularVersion, then that would be best. The token version is not clear if it is Angular version or this module version.

Done! :)

Merged. Thanks!

I will cut a new release with this change later this week


Update electron.rb

Symlink the .app
Symlink the binary into /usr/local/bin for use in shell
I have no idea why stage_only was added.

Thank you for the contribution. It needed some fixes, so they were made in the below referenced pull request. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

Thanks!

@vitorgalvao The binary is called Electron but I think the default binary name from the npm installation is (lowercase) electron. Is there any way to do that?

Yes.

Yay thanks!


Stock Linux distro support

Description
Added support for squirrel package manager of the Stock Linux distro
Features
squirrel support
Issues
TODO

I can't find anything about the Stock Linux distro or the squirrel package manager online (except for Squirrel.Windows). If the distro is published, can you tell me more about the Stock Linux distro?

Hello ! Thanks for your approval. The distro is effectively published. Stock Linux is my new source-based distro (with a future choice to build and download binaries). It is based on Linux From Scratch documentation and has a new package manager named squirrel. I haven't already made a website because I'm concentrated on building package. I successfully created the KDE packages :) This is the current github link of my distro: https://github.com/stock-linux
Thanks you :)

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!


feat(rpc): populate some getblocktemplate RPC block header fields using the state best chain tip

Motivation
[More to be added]
Populate the following fields:

capabilities
target
min time
cur time
bits
height
previous block hash

https://github.com/ZcashFoundation/zebra/issues/5455
Solution
[TBA]
Review
Reviewer Checklist

[x] Will the PR name make sense to users?

[x] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[x] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?
[ ] How do you know it works? Does it have tests?

Follow Up Work
Populate root/commitment fields

zebra_rpc_conflict failing https://github.com/ZcashFoundation/zebra/actions/runs/3493963607/jobs/5849620107#step:3:9825
Retrying.

@teor2345 you number of comments you made in your review is too much, i am still trying to figure out which of them are important ... I think only 2 of them are and the rest is optional. I want to discuss this with you as i think is not helping.

@teor2345 you number of comments you made in your review is too much, i am still trying to figure out which of them are important ... I think only 2 of them are and the rest is optional. I want to discuss this with you as i think is not helping.

Sorry about that, and thanks for the catch-up. I've resolved all the comments that can wait for later, and I made a suggestion or PR for some other comments.

By now, we don't have any tests for testnet specific code under minimum difficulty block mining.

By now, we don't have any tests for testnet specific code under minimum difficulty block mining.

This existing test might help?
https://github.com/ZcashFoundation/zebra/blob/c812f880cf8e63d518c88908210764fd21a0cc62/zebra-chain/src/work/difficulty/tests/vectors.rs#L393-L396

Please apply any of the missing stuff and merge if you can. Thank you.


Player locations are only accurate on last key press and release.

To get the accurate location, the program must simulate the Everybody Edits physics engine.

The game is going to shut down on December 31, 2020. There is no time to implement this.


Fix incorrect type for host

Fix host type which is a string :)

Test are running fine on my side, is this timeout failure an issue on circleci side?

This last commit fix a critical bug introduced by my previous pull request, default method interface is called instead of implementation because of the non matching signature (without Override annotation on registerWithTTL and registerWithIdAndTimeToLive method it did not trigger a compile error). This lead discovery service to do not register end point definition.


Update Dependencies

We were a little outdated on some deps and it was not only causing a couple vulnerabilities to slip into our Dev stack, but also made it so node:6 specs would not run in CI.
This PR updates those all deps (except for husky and nodemon, still waiting on some things to be fixed up in those packages).

Coverage remained the same at 100.0% when pulling d55c10a5e6bdf20fbf7dcc7971bcdfaa1961f57d on update-deps into 37542123e8f6d43ad5d020479fed3ac5cb758780 on master.

Coverage remained the same at 100.0% when pulling d55c10a5e6bdf20fbf7dcc7971bcdfaa1961f57d on update-deps into 37542123e8f6d43ad5d020479fed3ac5cb758780 on master.


Peek editor pointlessly navigates to destination as well as presenting preview

Issue Type: Bug
Reproduction
On a fresh install of VS Code:

Open a JS/JSX file which references an identifier with more than one potential definition (I chose React's Fragment component as that identifier, as it was the first external example that worked in an arbitrary file in my project)

Hold CTRL (on Windows) and hover — note the "Click to show 2 definitions" tooltip:

Click the underlined identifier

Expected behaviour
Either:

The window navigates to the sole definition of the clicked identifier

Or:

The peek editor appears, allowing the user to select which definition to navigate to

Actual behaviour

The window navigates away from the current context, and the peek editor appears, duplicating the view inline like so:

This is pointless, because the view sacrifices the code's visual flow, without the benefit of keeping your position in the file you are currently viewing.
I believe this is the source of frustration that led to issues like https://github.com/Microsoft/vscode/issues/34755 and https://github.com/Microsoft/vscode/issues/53164 being opened.
Edit: s/symbol/identifier/

VS Code version: Code - Insiders 1.26.0-insider (4fa62b246c4305b4ca00507551f77c58135ce928, 2018-08-03T12:32:37.823Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.88GB (15.88GB free)

Process Argv
C:\Program Files\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Extensions: none

I see a similar issue when using Typescript functions declared in the following fashion:
export const observeChangesAll = (collectionPath: string): Observable<firebase.firestore.DocumentSnapshot[]> => ...

This results in two definitions being found on the same line, see below.  The editor jumps to the definition but insists on displaying the peek window as well.

This is rather frustrating as it simply gets in the way and reduces the utility of Find All References

@tullo-x86 Sounds like #53341. Are you using an older version of TypeScript in your workspace?

@mjbvz: The dual definition for that identifier is not the problem; it is merely an easy way I found to reproduce the issue. The problem is that the IDE both navigates and opens the peek editor, in the same operation.

@chillitom: While that specific circumstance makes this issue present itself, fixing the dual reference in the codebase will only mask the real problem, which is that the IDE takes two actions where either alone would be an appropriate choice.

Mine is doing this too, on version 1.26.0.
I press f12 on the function in one file, a nodejs app (using .js files, of course), export style is to do a module.exports = {list of exported functions};

It goes to the method, then inserts the peek definition on the same line that i'm going to! Super annoying!

@mjbvz: What information can I offer for you to rescind the "needs more info" label on this issue? It seems pretty cut-and-dry:
if (definitions.length == 1)
  navigateTo(defintions[0]);
else
  showPeek(definitions);

Please share a complete code / project example


TabIndex of -1 not working for TextField and Select (possibly others)

[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Specifying a tabindex of -1 should make it so that the control does not receive focus when using the tab key.
Current Behavior
TextField and Select still receive focus when a tabindex of -1 is specified.  I haven't tested other controls but they may also be affected.
Steps to Reproduce (for bugs)
See here: https://codesandbox.io/s/l24mx1240q

Click on the compiled window.
Press the tab key
Both the TextField and Select receive focus.

For TextField, you have access to the input element via InputProps, which will be spread to the Input component.  Input has an inputProps property, which is used to spread properties onto the input element.
Here is an example:
<TextField InputProps={{ inputProps: { tabIndex: -1 } }} />

@kgregory It does feel like intuitively this should just work, i.e. tabIndex should be passed the underlying form component, without the user having to worry about the internals in this case.

It does feel like intuitively this should just work, i.e. tabIndex should be passed the underlying form component

@mbrookes How is this property related to an input, users could be targeting the root element.

@mbrookes A big part of the problem is that we offer some props that will make their way to input.  This gives the impression that any input attribute should be handed down, but TextField is an abstraction of several other components and the input element isn't directly exposed.
I think the InputProps, InputLabelProps, SelectProps, approach is fine, as long as the structure is clearly expressed to the user.  I know we got rid of inputProps for this component, but maybe it makes sense for an abstraction like this.
It would be a nice interface to offer control to the elements that are obvious to the user.  Eliminate the top level individual input element props, restore inputProps, and now we have specific control over Input, InputLabel, Select, input, and all other props are spread onto the FormControl.

restore inputProps

I would vote 👍  for this. It's kind of better documented now: https://material-ui-next.com/api/text-field/#advanced-configuration.

@oliviertassinari I saw that and thought it should be very helpful.  I'll see if I can put a PR together tonight.

Thanks for the help!  I was able to update the codesandbox example and now the TextField behaves as expected.  However, the Select is still having issues.  Any thoughts?  See here: https://codesandbox.io/s/l24mx1240q

@oliviertassinari

How is this property related to an input? Users could be targeting the root element.

tabIndex affects the focus order, so it intuitvely makes sense that it target the component that is focused when keyboard focus is applied. I can understand the argument for being consistent, so whatever's the priority works for me, I was just pointing out the obvious cause for confusion.

@SimplerSoftware It is possible with native Select:
      <Select
        value=""
        input={
          <Input
            name="age"
            id="age-helper"
            inputProps={{ tabIndex: -1 }}
          />
        }
        native
      />

However, it isn't possible with non-native Select, because it is being set to zero or null, depending on the value of disabled.

I saw that and thought it should be very helpful. I'll see if I can put a PR together tonight.

@kgregory If you don't mind. I have made a PR in the subway going back from work. I have seen quite some people asking about it after only a week. Let's revert. It won't cost us much.

@oliviertassinari I don’t mind at all.  I’ll find something else to work on tonight!


Window Walker not part of MSI build

Window Walker is not included currently in the MSI build

Window Walker needs this components:

WindowWalker.exe
WindowWalker.dll
MaterialDesignColors.dll
MaterialDesignThemes.Wpf.dll

The fix is now available in 0.16 https://github.com/microsoft/PowerToys/releases


cargo emptying log spam

Now that I know what I'm doing, kind of want to replace the cargo emptying routine with something that prioritizes using the wares in player storage somewhere before trying to sell them to the AI.

Little more detail, something seems to happen with Nostrop Oil in my test save where a ship that has already bought it and is on their way to sell it has the trade cancelled for some reason. An error shows up in the log about "NO CARGO SPACE". Then the ship runs the aiscript again and gets stuck trying to offload the cargo with no buyers available.

I'm wondering if the mule is trying to take nostrop oil to a factory that produces it, and a production cycle fills up the storage. Then the ship is stuck with a cargo full of a ware that's hard to sell a full hold to the AI, so the emptying routine then fails. Only ever seen this happen with food so far. Thinking I'll just dump cargo at the end of the emptying routine for now. Easiest fix for the spam that this produces.

fixed in https://github.com/Misunderstood-Wookiee/Mules-and-Warehouses-Extended/pull/54


Fix breaking zig change: deprecated float NaN constants

This commit has breaking changes: https://github.com/ziglang/zig/commit/387b0ac4f1c54cb2f83792299aa628a316e17d88

Thanks!


feature: Eviction notification to Prometheus server

Any plans to add eviction event notification to Prometheus Server.

Hi @muthusamymm
I will try to implement this feature in next release

Thank you for considering my request,
Thanks,
Muthu
On Wed, 1 Dec, 2021, 12:39 pm Maksim Paskal, @.***>
wrote:

Hi @muthusamymm https://github.com/muthusamymm
I will try to implement this feature in next release
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/maksim-paskal/aks-node-termination-handler/issues/12#issuecomment-983352903,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJT43NUNLAJQDSL5AOSXNC3UOXCZLANCNFSM5JDRPXDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@muthusamymm standart way to push metrics to Prometheus - is using https://github.com/prometheus/pushgateway
To resolve you issue you need to install pushgateway to your infrastucture and with next release you need to install aks-node-termination-handler with new arguments
-webhook.url=http://prometheus-pushgateway.prometheus.svc.cluster.local:9091/metrics/job/aks-node-termination-handler
-webhook.template='node_termination_event{node="{{ .Node }}"} 1'

where -webhook.url is url to pushgateway endpoint and -webhook.template is template for building a message - you can  use for templating {{ .Node }} or any information about event {{ .Event.EventId }} more information about event information here https://docs.microsoft.com/en-us/azure/virtual-machines/linux/scheduled-events
Is it OK to you?

hi Maksim Paskal,
This is really good and yes it will solve the problem . I will integrate
pushgateway and let me know once this new release is available .
thanks,
Muthu
On Wed, 1 Dec 2021 at 15:23, Maksim Paskal @.***> wrote:

@muthusamymm https://github.com/muthusamymm standart way to push
metrics to Prometheus - is using https://github.com/prometheus/pushgateway
To resolve you issue you need to install pushgateway to your infrastucture
and with next release you need to install aks-node-termination-handler
with new arguments
-webhook.url=http://prometheus-pushgateway.prometheus.svc.cluster.local:9091/metrics/job/aks-node-termination-handler
-webhook.template='node_termination_event{node="{{ .Node }}"} 1'
where -webhook.url is url to pushgateway endpoint and -webhook.template
is template for building a message - you can use for templating {{ .Node
}} or any information about event {{ .Event.EventId }} more information
about event information here
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/scheduled-events
Is it OK to you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/maksim-paskal/aks-node-termination-handler/issues/12#issuecomment-983471272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJT43NSP6D5ALR2QTX2BY5LUOXWABANCNFSM5JDRPXDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

this feature released
Update or install your current installation
helm repo add aks-node-termination-handler https://maksim-paskal.github.io/aks-node-termination-handler/
helm repo update

helm upgrade aks-node-termination-handler \
--install \
--create-namespace \
--namespace aks-node-termination-handler \
aks-node-termination-handler/aks-node-termination-handler \
--set args[0]=-webhook.url=https://hooks.slack.com/services/ID/ID/ID \
--set args[1]=-telegram.token=<telegram token> \
--set args[2]=-telegram.chatID=<telegram chatid> \
--set args[3]=-webhook.url=http://prometheus-pushgateway.prometheus.svc.cluster.local:9091/metrics/job/aks-node-termination-handler \
--set args[4]=-webhook.template='node_termination_event{node="{{ .Node }}"} 1'

Thank you Maksim Paskal for quick support.
Validated it, works good .
Thanks,
Muthu
On Wed, 1 Dec, 2021, 5:50 pm Maksim Paskal, @.***>
wrote:

this feature released
Update or install your current installation
helm repo add aks-node-termination-handler https://maksim-paskal.github.io/aks-node-termination-handler/
helm repo update
helm upgrade aks-node-termination-handler
--install
--create-namespace
--namespace aks-node-termination-handler
aks-node-termination-handler/aks-node-termination-handler
--set args[0]=-webhook.url=https://hooks.slack.com/services/ID/ID/ID
--set args[1]=-telegram.token=
--set args[2]=-telegram.chatID=
--set args[3]=-webhook.url=http://prometheus-pushgateway.prometheus.svc.cluster.local:9091/metrics/job/aks-node-termination-handler
--set args[4]=-webhook.template='node_termination_event{node="{{ .Node }}"} 1'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/maksim-paskal/aks-node-termination-handler/issues/12#issuecomment-983588312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJT43NUIL3XF5WXUM6P3US3UOYHJZANCNFSM5JDRPXDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


KubeIP sporadically fails to assign the address on GCP because of missing access config check during delete step

When on GCP, if e.g. a static IP address is manually detached from the node which then gets assigned another ephemeral IP address, kube IP will fail to attach the labeled static address as it will try to delete the new ephemeral address for the node as it is thinks the node lacks network access config, but the process to fetch might be bugged (I think I've listed the node's details and saw that it in fact has a network access config listed). If I rescale the cluster and thus create a new node, then the process works as expected.
{"error":"failed to delete current public IP address: failed to get instance network interface access config: instance network interface has no access configs","file":"/app/cmd/main.go:97","func":"main.assignAddress","level":"error","msg":"failed to assign static public IP address to node some node","time":"2023-12-14T09:24:23Z","version":"sha-ce43fbb"}
I can try to reproduce it again and list the node details to show that it does have the access config.

@ijozic How are you able to reproduce, I am getting the issue not on all nodes, but sometimes.

@ijozic @olivierboucher can you reproduce this? If so, could you provide clear steps on how to do so?

i have the same issue. In my case this happens when Kubernetes nodes upgrades. I have blue-green update enabled and have exactly the same amount of ip addresses as there are Kubernetes nodes. Let's say I have 5 nodes in my cluster I labeled 5 IPs to be used in that cluster. All my node pools are scaled to 1 and my batch node count is 1 and soak duration 3600.

@Pestilenciaa do you manage to consistently reproduce the error reported in the original post?
If so, could you share the YAML description of your node pools [1]?
[1] https://cloud.google.com/sdk/gcloud/reference/container/node-pools/describe

@eyalzek
It's been a while and I moved back to the previous version. If I remember correctly, the main issue was that the code didn't check for the network interface existence before deleting it. I could've opened a PR but lacked time at that moment.

@olivierboucher AFAICT the 2 places in the code where it interacts with the NIC (add/remove address), it's fetching the network interface or erroring out [1][2].
In any case, if you have the time to help with reproducing it'd be highly appreciated :)
[1] https://github.com/doitintl/kubeip/blob/master/internal/address/gcp.go#L150-L153
[2] https://github.com/doitintl/kubeip/blob/master/internal/address/gcp.go#L181-L184

@eyalzek
What I'm saying is that there should be a distinction between not being able to fetch the NIC and the instance having no NIC... therefore, the deletion should be skipped if there is not NIC to delete

@olivierboucher the function getNetworkInterface [1] should return an error if there is no NIC (or if it can't get it). And since this is the first check in both the functions I mentioned above, the deletion/adding of an address should be skipped if it fails.
Or did I misunderstand what you meant?
[1] https://github.com/doitintl/kubeip/blob/master/internal/address/gcp.go#L369-L371

Yes but the problem is that the whole function errors out [1] when it should just complete and return nil if there is no NIC
[1] https://github.com/doitintl/kubeip/blob/master/internal/address/gcp.go#L152

Well if it doesn't fail then the following logic in Assign/Unassign will, since it doesn't really make sense to proceed without the NIC. That's where the retry mechanism should do the trick.
Unless you're describing a scenario where KubeIP goes into a failing loop? If so, could you provide steps to reproduce?

This can happen when a new node is attached to a GKE cluster for whatever reason: node upgrade, scale-up, health restart, etc.
In this case, node info is updated with some delay and the KubeIP can fail to assign a static IP to the node.
To overcome this issue, the KubeIP retries to assign a static IP for (RETRY_INTERVAL X RETRY_ATTEMPTS): 1m x 60 == 1h (by default; and this parameters can be configured)
So, if node network interface details are not updated when the KubeIP runs the first time, the KubeIP will retry again and again until it succeed or max retry attempts achieved.
@olivierboucher @ijozic @Pestilenciaa @vini-intenseye can you try to reproduce the issue, wait a few minutes and see if all nodes got a static IP assigned? If not, please attach the KubeIP logs from problematic nodes.

#138 fixes this issue too
Tested with:

manual delete node - static address released and new node added taking released address
cluster resize - resize cluster (scale up and down); everything works as expected

Closing this issue for now. If you can reproduce it with the latest version, please reopen it and describe the process for reproducing it.


aquery.js

the project does not work, I suspect that the problem is with Gulp. The ready "aquery.js" file doesn't work either!

Fixed in v0.7.1!


Tutorial for AbstractFileOutputOperator

This is turning out to be quite long. Please review

@PramodSSImmaneni

Looks good from my end.  Let me know if I can merge it.

I'll look at it tomorrow

Any updates on this?  Can I merge it?

Reviewing it right now.

@amberarrow @sashadt @PramodSSImmaneni
Please merge

Still reviewing, will likely spill into the weekend.

Make a couple more suggestions; @sashadt can merge after they are addressed.
Rest looks good.

@sashadt @PramodSSImmaneni @amberarrow
Please merge. Any more additions/changes can be made to this document after this is merged.


Weird sound plays every 10 seconds

every 10 seconds a strange sound plays.
strange sound.zip
zips got a video of the sound

hm no idea

mod list pls


Inquiries about the POW feature in Cortex API?

I have a query about the POW function; what did you do with the EEG to get these values?
I need to understand what happened in the background.
Also, can you tell me what is the MAX and MIN Values for each value in the resulted array?

Hi @shathaalotaibi
The band power are calculated every 0.5 seconds, using the last 2 seconds of EEG data.
The steps are:

High pass filter
Remove DC offset (for each EEG channel, remove the mean value)
Hanning window
FFT
make the sum of the squares of the FFT values


Set custom favicon

Is your feature request related to a problem? Please describe.
I would like to customize the favicon file
Describe the solution you'd like
I would like to be able to set the favicon icon file
Describe alternatives you've considered
One option is to use the default favicon file that is auto generated by the site. However I would like to use a custom favicon file

You can change it here - #L159


Stop host kernels from executing Solo5 binaries

This is implemented for hvt, spt, muen and virtio bindings by:

Adding a PT_INTERP and .interp in binding-specific linker scripts.
Populating .interp via DECLARE_ELF_INTERP in binding-specific start.c.
The content of PT_INTERP is "/nonexistent/solo5/".

The Genode bindings have not been changed, as they seem to do their own
thing regarding PT_INTERP.
With this change if a Solo5 unikernel binary is run directly (e.g.
./test_hello.hvt) on a Linux/FreeBSD/OpenBSD host, the host kernel will
not attempt to execute the binary and will fail with a more or less
sensible error message rather than produce a segmentation fault or
similar.
This provides both for a nicer "naive user" experience, and is a
security improvement against accidentally exposing the host kernel to
unikernel binaries from untrusted sources.
TODO before merging: Manually test targets that are not tested in CI:

[ ] muen: Ensure that the Muen toolchain does not get confused by the presence of a PT_INTERP.
[ ] virtio with byhve on FreeBSD.
[ ] virtio using Google Compute Engine.

Fixes #442.

Updated to use /nonexistent/solo5/ (trailing slash added) as discussed in the linked issue.

@yomimono I believe you are a user of Solo5/MirageOS on Google Compute Engine. As I don't currently have a straightforward way to test there, would you mind building any random MirageOS unikernel using Solo5/virtio off this branch and testing that it boots correctly? Given that both qemu and bhyve have no issue with the changes here I'd expect it to be 99% yes, but it won't hurt to confirm.

Commit amended to note that OpenBSD has a different behaviour, which we can do nothing about.

@mato - sorry, just saw this notification today.  I'm not using GCE for running unikernels anymore, but I can help someone through the process if anyone's interested in trying it at the hack retreat perhaps?


How to pass arguments to app

Hi, I found that electron based apps (and chromium derivatives itself) need to be run with extra arguments to start in a wayland session. So for ungoogled chromium I need to run flatpak run io.github.ungoogled_software.ungoogled_chromium --ozone-platform-hint=auto to have proper scaling and such.
Is there a way to add this argument declarative?
Thanks!

Hi @Axolord . nix-flatpak does not provide a wrapper for flatpak run or any launcher scripts. You can customize application requirements with  services.flatkpatk.overrides (it wraps flatpak override, but not parametrize run.
For you use case, did you consider aliasing  ungoogled_chromium in your shell config? Or maybe having a script with the desired flatpak run command params.
Either way, you should be able to do it declaratively with Nix. The specific method is somewhat system-dependent; on NixOS, you could use shellAlias.


Add Count documents API in client

The API must be added on the collection level and accept filters. The syntax can be similar to the below,
func (c *Collection[T]) Count(ctx context.Context, filter filter.Filter) (int64, error)
The server-side support is added here: https://github.com/tigrisdata/tigris/pull/1068, which returns count in the response and honors the filters if passed to return the count for the matched documents.

Fixed by #273


Cannot access Windows network port from WSL2 using virtual IP

Environment
Windows build number: 10.0.19041.0
Your Distribution version: Ubuntu 10.04
Whether the issue is on WSL 2 and/or WSL 1: WSL2 (Linux version 4.19.104-microsoft-standard (oe-user@oe-host) (gcc version 8.2.0 (GCC)) #1 SMP Wed Feb 19 06:37:35 UTC 2020)

Steps to reproduce
I just installed a fresh Ubuntu 20.04 LTS  on WSL2 for creating this ticket (I experienced the same problem with my Debian on WSL2, so I downgraded that one back to WSL1 as I needed the functionality).
Make sure to have a webserver running on Windows, and you cannot access it with the IP from /etc/resolv.conf in WSL2, even through that should work according to the docs.
In this case, I created a simply NodeJS server on port 8080, which is listening on <IP_ADDRESS>. It is accessible from the entire LAN on <IP_ADDRESS> (local IP).
Now, if you want to access it from WSL2, using the nameserver (<IP_ADDRESS> here, or virtual IP) as suggested in the article, it unfortunately does not respond at all. Using localhost doesn't work either (as expected), but using the direct IP address does work.
Below is a screenshot of my test: top-left is the WSL2 Ubuntu, top-right is Powershell on Windows running the actual server. De two browser windows on the bottom are run on the host machine, demonstrating the webserver responds to both localhost and the external IP (external IP access was also tested from another machine on the LAN, that works fine).

Expected behavior
I would expect to either

see the virtual IP behave as described in the documentation
see the local IP in /etc/resolv.conf (but that breaks DNS resolving at this time)
see the local IP in some other file

The latter would be useful for scripting.

Actual behavior
Well, it isn't working as I would have hoped😄
P.S. While working on this, I also managed to break DNS resolving completely. This was probably related to me resetting the Windows firewall (for testing), as it was fixed after rebooting.

I also facing similar issues where as sometimes port open from wsl2 accessible from windows host and sometimes not.. Didn't able to get actual reason anywhere... I have all things at latest version on todays date....

Is there any update on this at all?
Currently i did exactly as described above

At first the DNS was broken and i couldnt access the internet at all, but after disabling swap and restarting a few times that finally started working
i re-enabled swap and configured "generateResolvConf = true" and the file gets generated as it should with the correct nameserver IP and now i can access the internet, but i cannot access the Windows host at all

Did you ever get this to work?

No, I did not get this to work, stil using WSL1 at the moment

I thought I can use this, but finally end up in dual booting my system as always!

try to use that ip in your windows host
C:\Users\Ubuntu> ipconfig.exe
Ethernet adapter vEthernet (WSL):

   Connection-specific DNS Suffix  .  :

In my case it's <IP_ADDRESS>
And then create a server in you host machine using this address
C:\Users\Ubuntu> py -m http.server -b <IP_ADDRESS>
Serving HTTP on <IP_ADDRESS> port 8000 (http://<IP_ADDRESS>:8000

and then in you WSL 2.
arthur@memyself:~$ curl <IP_ADDRESS>:8000
Hello World!

but if i use <IP_ADDRESS>, <IP_ADDRESS> or other things it didn't work
should report this to MS

You're probably being blocked by the Windows firewall as Ethernet is a "Public" network.
Create a new rule in windows firewall settings to allow inbound traffic
check out this answer on https://superuser.com/a/1496354

I an inbound rule for my test port (8000 in my case) that should apply for all interfaces and for all types of networks (so including public). Still, I can't access the service from within WSL2 using the IP in /etc/resolv.conf.
I also don't see dropped connections logged by the Windows firewall.

This is still an issue as to this day. I tried every possible workaround... The big problem is that this is preventing many people from migrating from Docker for Windows to Podman running inside Linux on WSL 2, as they cant test the services in the containers against an application server running on Windows (which you usually do when developing said server). Downgrading to WSL 1 is also not an option as this causes issues with Podmans File system drivers

You need to allow inbound connection in Windows Firewall for that program which wants to connect with WSL2 side.

Allowed inbound connection in firewall and no luck

still open after two years and no response from Microsoft ... what a shame
but i found a workaround for this on Windows 10 using Internet Connection Sharing:

right click your host network adapter -> properties -> sharing -> Allow other network users to ... -> choose WSL in dropdown list
you may get a dialog say some warning, click  Yes. after this the host can access internet normally but WSL can't
restartt, then WSL can access internet, and host can be access via host network adapter's IP in step 1

it's wired the chosen WSL disappear after restart in step 1's properties dialog, but seems have no impact.

well...I also faced this issue today...

 Create a new rule in windows firewall settings to allow inbound traffic
check out this answer on https://superuser.com/a/1496354

Tried that, still no luck. This issue still persists.

Hi. Can you please collect networking logs by following the instructions below?
https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#collect-wsl-logs-for-networking-issues

I was finally able to connect from a docker container on WSL2 to Windows 10 host. (Docker -> WSL2 -> Windows 10). Here is some insight about how I did it:
In my case, I was trying to connect to a gRPC server which was listening "localhost" in windows 10, from a client that was also targeting "localhost" and was inside a docker container on WSL2. So, the setup was looking something like this:
Docker(targets: "localhost") -> WSL2 -> Windows 10 (listens: "localhost")
I changed it to something like this:
Docker(targets: "the IP in /etc/resolv.conf on WSL2") -> WSL2 -> Windows 10 (listens: "<IP_ADDRESS>")
And it worked.
It might not applicable in your case but in the end I was able to receive connections from my WSL2 ubuntu instance towards Windows 10 host.
P.S.: I also put the docker container network mode to "host". Meaning it was not in its own sub "docker network".


NUnit3Settings does not provide an option to specify the configuration file to load

What You Are Seeing?
Nunit3 Console runner 3.8.0 has a new command line option for loading the app.config file from a different path. The NUnit3Settings class does not have an option to pass that argument through to the nunit console runner.
The new config file parameter is:
--configfile=app.config

Here is the Nunit issue that brought this new feature in
What is Expected?
That the NUnit3Settings object has a property to pass the argument to the console runner.
What version of Cake are you using?
0.30.0
Are you running on a 32 or 64 bit system?
64bit
What environment are you running on?  Windows? Linux? Mac?
Windows
I will be opening a PR to resolve this issue.

Closing as the PR that I opened to satisfy this issue has been merged. Thanks @devlead


Recommended way to adjust/finetune the scale parameters in config.json

After 80 epochs of training on my own dataset, the results aren't that good. At the end the average Recall ist 0.80 but some classes are better recognized than others
input_size: 290
Classes:     3
Dataset:    1300 pictures (about 400 each class and 100 negative samples without class)
The rest in config.json file is same as the one downloaded here.
I think finetuning the scale parameters is going to have most impact on training results?

What's the mAP for your 3 classes. You try to do detections on a few images to see what's the problem. If it failed to detect the objects in both 3 classes, increase obj_scale. If it does detect the objects but fails to classify, increase class_scale. If the size and the location of the predicted is not good, increase coord_scale.


Hitting "up" does not bring the whole multi-line entry from history.

Running latest reedline after cargo update (a44a590b25e2fb2593db24a17c06ed887633a5fc).
> def foo [] {
    "foo"
}

> }  # <-- appears after hitting 'up'

Interestingly, reedline by itself handles this fine. It must be something within engine-q.

Reedline is currently still not able to store the multiline entries correctly in a file. So as soon as the Reedline object is dropped the multiline entries become individual lines
https://github.com/nushell/reedline/issues/116

Closed by nushell/reedline#207 and nushell/reedline#210 and was pulled into engine-q with #562


Add pyproject.toml for Custom Node Registry

We are working with dr.lt.data and comfyanon to build a global registry for custom nodes (similar to PyPI). Eventually, the registry will be used as a backend for the UI-manager. All nodes go through a verification process before being published to users.
The main benefits are that authors can

publish nodes by version and users can safely update nodes knowing ahead of time if their workflows will break or not
automate testing against new commits in the comfy repo and existing workflows through our CI/CD dashboard

Action Required:

[ ] Go to the registry. Login and create a publisher id (everything after the @ sign on your registry profile).
[ ] Add the publisher id into the pyproject.toml file.
[ ] Merge the separate Github Actions PR, then merge this PR.

If you want to publish the node manually, install the cli by running pip install comfy-cli, then run comfy node publish
Check out our docs if you want to know more about the registry. Otherwise, feel free to message me on discord at haohao_81202 or join our server if you have any questions!

hey @ZaneA, wanted to send a small nudge and ask if you had any questions!

@ZaneA, wondering if there's any way we can support you on this PR!

Hey @haohaocreates thank you for following up and sorry for the wait. This is a great initiative for the Comfy ecosystem! Hopefully I've done what is required here for now, and I can see the node appearing in the registry :)


wrong log level for replace_sample(). capturesync_drop -- misclassification, spamming, etc.

The k4a_log_level_t of the below code is incorrectly set to error. This state in the code is a quite common natural state (up to 30 times per second) in which the underlying subsystem is managing its queue and keeping frames fresh. Nothing wrong, erroneous, dangerous, or warnful about having fresh frames.
Therefore, error is not appropriate and leads to spamming the console/message handler and making it exceedingly difficult to find the actual errors within the spam.
https://github.com/microsoft/Azure-Kinect-Sensor-SDK/blob/52cfb4a87639b3db9cb176b8b9ac814e2b5a4900/src/capturesync/capturesync.c#L137-L142
My preference is K4A_LOG_LEVEL_TRACE
At worst, it should be K4A_LOG_LEVEL_INFO
That it is not an error is also supported by @wes-b when they write "...There are other warnings generated..." at https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/1112#issuecomment-596688870
Fix
Trivial. Change LOG_ERROR to LOG_TRACE

Under normal operation this message will never fire. So this is an error message.
We talked about this, and I am leaning toward not fixing this. I do admit when it fires, it is spammy, however it does indicate a system issue.
The other issue, is that users tend to copy and past default logs into new github issues. If we lower the level of this message, then users by default will get less useful logs. In turn our triage process will be more drawn out by the fact that we have to ask for more detailed logs.
If the spammyness is causing issues, we can consider generating the log message only 1 time, and then disabling the message. But then the sensor will be running and it will appear that no data is arriving.

I woke yesterday and read your reply above. Then started on a reply with some data...but now...those messages are gone. I've tried a day to replicate those messages in the OP. Including intentionally writing bugs using the API. Now...they won't appear. That I can't reproduce them, suggests some failure state (independent of this issue) that existed during the day of the OP. On that day, they appeared consistently many times/second for hours when I used the API. So much that I wrote this issue and custom log handler with a regex. But now...silent.
I caution writing C code to satisfy an issue reporting copy/paste procedure. That would be like turning on Windows debugging by default so that debug logs would be always there. But that's not the right approach. Instead, I align with actionable logs targeted for the consumer of those logs with the ability to get more detail for debugging and different consumers of logs.
Line of Distinction
I think the line that exists in this issue's discussion is a line distinguishing:

logs to debug the SDK itself. Microsoft is the primary consumer of these. You express that when you write above "...our triage process will be more drawn out...".
logs to debug the user application. Microsoft is not the primary consumer of these. Instead, the primary consumer of these is the developer using the SDK, consultants, ISVs, etc. Naturally, you want to make an SDK that will generate logs that these users of the SDK can use for debugging their custom applications. But Microsoft is not as invested in these.

Is the state that generated the log in the OP a state that is only useful to distinguish for Microsoft (the SDK developer)? That makes it bucket 1 above. Which leads to the question, "Who is the consumer of these logs and what is there need?" For us using this SDK...we are bucket 2. We are not focused on debugging the SDK itself. Instead, we are writing custom applications.
In general, I don't want multiple log entries for bucket 1. Why? Because they are not actionable. I can't fix them. They are inside the SDK itself. Of course, I can report issues, and as more detailed info is needed I can provide it.  (I acknowledge this is not absolutely true...there is a grey area).
You already volunteered an approach of log once and then quiet. That's an interesting approach for bucket 1 SDK debugging log entries. This might mitigate some concerns of your triage efficiency, while also freeing logs of bucket 1 spam.
Example
While trying to reproduce those spammy OP logs entries, I generated other entries with log levels demonstrating ambiguities in the two buckets. For example, this is a series of log entries that appears if I open a specific Kinect, and then try to open that same Kinect a second time. I'll walk through the entries and apply my thinking from above on them.
[2020-07-08 15:33:53.160] [critical] [t=3876] D:\a\1\s\extern\Azure-Kinect-Sensor-SDK\src\usbcommand\usbcommand.c (366): find_libusb_device(). libusb device(s) are all unavalable. Is the device being used by another application?

[2020-07-08 15:33:53.160] [error] [t=3876] D:\a\1\s\extern\Azure-Kinect-Sensor-SDK\src\depth_mcu\depth_mcu.c (68): usb_cmd_create(USB_DEVICE_DEPTH_PROCESSOR, device_index, NULL, &depthmcu->usb_cmd) returned failure in depthmcu_create()

[2020-07-08 15:33:53.160] [error] [t=3876] D:\a\1\s\extern\Azure-Kinect-Sensor-SDK\src\sdk\k4a.c (133): depthmcu_create(index, &device->depthmcu) returned failure in k4a_device_open()

Exception thrown at 0x00007FFC5F703E49 in Max.exe: Microsoft C++ exception: k4a::error at memory location 0x00000000007DB8B0.

Entry 1 should not be critical. This is supported by both Windows and Linux. Critical is "your Kinect temp is 1000f and is on fire", "wake up the admin", "the apocalypse is here", . But as you can see...this is just because I called open() twice on the same ordinal of a Kinect. Look at this situation from a "user's" perspective. This is a file in-use/locked scenario. Like trying to open my Word doc and I'm not able to because someone else has it already open. No reasonable person wakes their sysadmin because of this. And therefore, this is not a critical condition. It is instead an error condition.  It is not actionable for bucket 2 users of this SDK.
Entry 2 looks also to be an SDK-only low-level bucket 1 condition. I agree it is error for a bucket 1 user, but it is spam to bucket 2 users because it is not actionable.
Entry 3 is good. It is actionable. I agree it is error and it indicated the user API that failed. I suggest moving the "Is the device being used by another application?" from entry 1 and put it on entry 3. Then this log entry is even more useful.
Entry 4 is the exception throw, vscode seeing it, and my code catches it.
Personally, I would make:
Entry 1 = trace
Entry 2 = trace
Entry 3 add suggested sentence
Entry 4 good as is

Surfacing log entries to downstream customers
Log levels are important not only to me the bucket 2 ISV...but they are potentially useful to my downstream customers to which I sell my custom Kinect application. I am 97% sure that they don't want to see the spam from the OP. Its not actionable by them. One approach that I might consider is to surface critical errors to these hands-on customers. Though that means that this SDK should only log things critical when it is on fire. Consider in bulk warehouses and monitoring solutions...I might want my Kinects to send status/logs to my centralized systems only when things are error or critical. Now compare that against the above example. Entries 1 and 2 are not in alignment.

You raise great points! We will work to only fire the error 1 time at the existing error level. Still considering if continuing to generate the trace at INFO level is useful, or if we should just remove additional messages,


Problem in Injecting Module ngSelect

After installing bower package succesfully and when i am trying to inject ngSelect module, its gives an error that,  the module you are trying to inject is not available .
So if anyone has solution then help me in solving this issue.

@suraj740 Have you include the script from bower package in your HTML?
For example
<script src="bower_components/ngSelect/ngSelect.min.js"></script>

ya i have installed bower package and included script also.


LSTnet throws Method Error in training

ERROR: MethodError: no method matching iterate(::Nothing) in Flux.train!.

This is somehow caused when updating Zygote v0.5.5 to v0.5.6. First RSL throws error, the other one works:
RSL = Chain(a -> dropdims(a, dims = (findall(size(a) .== 1)...,)), GRU(convlayersize,recurlayersize))
RSL = Chain(a -> a[1,1,:,:], GRU(convlayersize,recurlayersize))


ngPluralize setting when property via directives link function causes rules to not be detected

While in a directive, If you try to set ng-pluralize's when via an object defined in the link function it will throw ngPluralize: no rule defined for other. However if you use the controller it will work perfectly fine.
I've created a  Plunker Example  demoing this behavior.

This is expected behaviour (imo), since the controller is instantiated before the child-directive's post-linking, while link: fn (which is short for link: {post: fn}) is executed after the child-directive post-linking phase.
If (for whatever reason) you don't want to use a controller, you can use the parent directive's pre-linking phase:
return {
  ...
  link: {
    pre: function (scope) {
      ...
      scope.postItem = {...};
    }
  }
};

@gkalpak is right!


Add json output for lxc info

in issue #1789 and the merge request with commit hash cddaada(pr #1876) the --format=json was added for lxc list.
Similar should exist for subcommand: lxc info

ping @cmars who did an awesome job on #1789

Note that #1943 should take care of most of those, quite possibly with less effort.

Going to close this in favor of #1943 which should offer a better general solution to this problem.


Add back in the 1px border between filter groups on jobs results page

Was mistakenly deleted recently
Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1201
Screenshots of UI changes:
Before

After

It hasn't been deleted, it gets hidden by JS. I'd prefer an approach that hides the correct element instead, rather than adding extra CSS that actually already exists and is present

It hasn't been deleted, it gets hidden by JS. I'd prefer an approach that hides the correct element instead, rather than adding extra CSS that actually already exists and is present

You mention hiding, but I'm not sure what you think should be hidden. The border should always be visible.

It hasn't been deleted, it gets hidden by JS. I'd prefer an approach that hides the correct element instead, rather than adding extra CSS that actually already exists and is present

It was deleted accidentally by me in https://github.com/DFE-Digital/teacher-vacancy-service/pull/1889/files
What JS hides this border?

It hasn't been deleted, it gets hidden by JS. I'd prefer an approach that hides the correct element instead, rather than adding extra CSS that actually already exists and is present

It was deleted accidentally by me in https://github.com/DFE-Digital/teacher-vacancy-service/pull/1889/files
What JS hides this border?

The border as per the Figma exists when the filter is expanded, so if the .tv-checkbox__group element was hidden instead of the .filter-group__checkboxes element, then the border would still be there. If this approach matches the design then ok, but this looks like a double border to me

Good spot on the double border. Looks like a conflict between the ticket and the Figma. The Figma shows this 1px border in every instance, and does not show the thicker, lighter grey border in any instance. I will delete the thicker lighter border when I return to this ticket this afternoon.

Have dismissed Alex's earlier approval as this PR now contains about 10 times as many changes as it did before.


Add links to document to about panel

Closes #210
Closes #325

@subdavis, is this AExt thing I invented superfluous? Or perhaps there's an easier way to accomplish what must be a very common task?


No error message is shown when reaching min/max

When I run this:
const { buildLexoRank, NumeralSystem64 } = require("@wewatch/lexorank");
const LexoRank = buildLexoRank({
  NumeralSystem: NumeralSystem64,
  maxOrder: 8,
  initialMinDecimal: "1000",
  defaultGap: "1000",
});

const a = LexoRank.min();
const b = a.genPrev();
console.log(a+"", b+"");

I expect b to either return a lexo-rank that would sort before a, or throw an error. Instead I get a lexorank that is identical to a: both are 0|00000000:.
CodeSandbox demo: https://codesandbox.io/s/lexorank-issue-v9ld50?file=/src/index.js

Actually, it maybe makes sense that calling genPrev() does not error, even in that case; I'm not sure what the "correct" behavior would be.
I'll close it for now, as it doesn't seem to have an obvious answer.


Detect Coffee version and use correct compiler switch

For CoffeeScript >= 1.7.0 we should pass --compilers coffee:coffee-script/register. This pull request checks version the first time it's run and use correct compiler switch.
I haven't really tested CoffeeScript < 1.7.0 so if you could take a look that would be great!

I removed support for CoffeeScript because of issues in described in https://github.com/janko-m/vim-test/issues/52. For mocha you can just set the compiler in test/mocha.opts, which I think should be done anyway.


Projects won't load if you have net8.0 sdk installed

Projects won't load if you have net8.0 sdk installed. Even if you are working with net7.0 projects, they still won't load due to the existence of net8.0 sdk.
Environment data
C# Extension version: 2.0.357

installing C# Dev Kit I believe is a workaround

Installing C# Dev Kit doesn't help.

Oh strange. @hez2010 are there any errors in the C# output window that you can share that are giving errors?

Also what is your output of dotnet --list-sdks?

Output from "C#":
Dotnet path: /Users/***/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/7.0.10/dotnet
Activating C# + C# Dev Kit + C# IntelliCode...
[Info  - 4:03:23] [Program]Language server initialized
[Error - 4:03:23] [LanguageServerHost]System.InvalidOperationException: Was not able to get hello world response from remote - line 39
   at Roslyn.Utilities.Contract.Fail(String message, Int32 lineNumber) in /_/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/Contract.cs:line 154
   at Microsoft.CodeAnalysis.LanguageServer.BrokeredServices.Services.HelloWorld.RemoteHelloWorldProvider.SayHelloToRemoteServerAsync(CancellationToken cancellationToken) in /_/src/Features/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/BrokeredServices/Services/HelloWorld/RemoteHelloWorldProvider.cs:line 39
   at Microsoft.CodeAnalysis.LanguageServer.BrokeredServices.ServiceBrokerFactory.CreateAndConnectAsync(String brokeredServicePipeName) in /_/src/Features/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/BrokeredServices/ServiceBrokerFactory.cs:line 82
   at Microsoft.CommonLanguageServerProtocol.Framework.QueueItem`3.StartRequestAsync(TRequestContext context, CancellationToken cancellationToken) in /_/src/Features/LanguageServer/Microsoft.CommonLanguageServerProtocol.Framework/QueueItem.cs:line 136

From "C# Dev Kit":
Starting Open a solution...
Starting Open a solution with environment service...
Starting Clear environment...
Starting Spawn .NET server...
.NET server started and IPC established in 1536ms
Completed Spawn .NET server (1546ms)
Completed Clear environment (1711ms)
Completed Open a solution with environment service (1863ms)
Starting Restore solution...
Completed Open a solution (1873ms)
Completed Restore solution (1054ms)

From "Projects":
[info] Project system initialization finished. 0 project(s) are loaded, and 1 failed to load.

And dotnet --list-sdks:
7.0.400 [/usr/local/share/dotnet/sdk]
8.0.100-preview.7.23376.3 [/usr/local/share/dotnet/sdk]

BTW I'm on macOS.

I saw the same, for some reason vscode-csharp trying to use net7, I deleted net7 and vscode-csharp can't load a csproj with net8-preview:
Using dotnet configured on PATH
Dotnet path: C:\Program Files\dotnet\dotnet.exe
Activating C# standalone...
[Info  - 12:40:20 PM] [Program]Language server initialized
[Info  - 12:40:20 PM] [LanguageServerProjectSystem]Loaded MSBuild at C:\Program Files\dotnet\sdk\8.0.100-preview.7.23376.3
[Info  - 12:40:20 PM] [LanguageServerProjectSystem]Loading <Solution>.sln...
[Error - 12:40:21 PM] [LanguageServerProjectSystem]Failure while loading <PATH>src\Shared\Shared.csproj: The "CheckIfPackageReferenceShouldBeFrameworkReference" task could not be loaded from the assembly C:\Program Files\dotnet\sdk\8.0.100-preview.7.23376.3\Sdks\Microsoft.NET.Sdk\targets\..\tools\net8.0\Microsoft.NET.Build.Tasks.dll. Could not load type 'System.Runtime.CompilerServices.NullableContextAttribute' from assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.

dotnet info:
Host:
  Version:      8.0.0-preview.7.23375.6
  Architecture: x64
  Commit:       65b696cf5e
  RID:          win-x64

.NET SDKs installed:
  6.0.408 [C:\Program Files\dotnet\sdk]
  8.0.100-preview.6.23330.14 [C:\Program Files\dotnet\sdk]
  8.0.100-preview.7.23376.3 [C:\Program Files\dotnet\sdk]

The only workaround is to uninstall C# Dev Kit and check the Use OmniSharp in C# extension settings.

@jasonmalinowski it isn't fixed yet, at least without a roslyn update in the repo, isn't it?

@vchirikov We updated Roslyn in this repo with https://github.com/dotnet/vscode-csharp/pull/6285; we're currently working on getting a prerelease build posted with it.

@vchirikov 2.2.10 is now out if you want to give that a shot. It's pre-release only at the moment, but you can easily switch back and forth. Instructions are here.

I'm still affected by this problem. On Windows 10, with VSCode Insiders, C# extension v2.2.10: with .NET 8 SDK (8.0.100-rc.1.23463.5) all projects fail loading. Uninstalling .NET 8 SDK fixes the issue.
Output of "C#":
Using dotnet configured on PATH
Dotnet path: C:\Program Files\dotnet\dotnet.exe
Activating C# + C# Dev Kit + C# IntelliCode...
[Info  - 4:39:29 PM] [Program] Language server initialized

Output of "C# Dev Kit"
Starting Spawn .NET server...
Starting Open a solution...
Starting Open a solution with environment service...
Starting Clear environment...
.NET server started and IPC established in 1017ms
Completed Spawn .NET server (1160ms)
Completed Clear environment (1609ms)
Completed Open a solution with environment service (1653ms)
Starting Restore solution...
Completed Open a solution (1669ms)
Completed Restore solution (8550ms)

Output of "Projects"
2023-09-19 16:39:37.137 [info] Project system initialization finished. 0 project(s) are loaded, and 86 failed to load.

Output of "dotnet --list-sdks"
3.1.426 [C:\Program Files\dotnet\sdk]
5.0.404 [C:\Program Files\dotnet\sdk]
5.0.416 [C:\Program Files\dotnet\sdk]
6.0.101 [C:\Program Files\dotnet\sdk]
6.0.200 [C:\Program Files\dotnet\sdk]
7.0.304 [C:\Program Files\dotnet\sdk]
7.0.306 [C:\Program Files\dotnet\sdk]
7.0.401 [C:\Program Files\dotnet\sdk]
8.0.100-rc.1.23463.5 [C:\Program Files\dotnet\sdk]

can you tell us the version of C# Dev Kit you are using, and whether you have setting dotnet.preferRuntimeFromSDK enabled or not (we are phasing out this setting, but the earlier version CDK still uses it).
It was a problem that older version CDK is incompatible with new builds of NET 8 SDKs. You are more likely to have this problem by using the extension from its 'release' channel (as it was up to a month older than 'pre-release' one).
C# Dev Kit adds a "projects" output panel, which may provide some information on what might go wrong (and if it doesn't contain helpful information, it would be helpful to open the extension log folder, and find ServiceHub folder inside CDK logging data. There are at least two Composition.log files in the folder, which may give us some clue here.

BTW, if you expand projects failed to load, what is the error message string?

Nevermind, I was not running the latest version of CDK and neither of VSCode Insiders. Upgrading both fixed my issue. Thanks.


Meteor add packages

meteor add cordova-plugin-admobpro: exact version or tarball url is required ? How can I fix

meteor is based on cordova CLI, usually it's not required to add version number.
if it's really required, you can just add the latest version, and get the version number here:
https://www.npmjs.com/package/cordova-plugin-admobpro