Potential memory leak in stream retries

Please see a screenshot of our node.js heap usage on the same diagram as the number of log entries with Issue with streaming: falling back to polling while the SDK attempts to reconnect

I was trying to track down the cause of this and found the following comment:
    // TODO - the first attempt of each strategy is fired off at the same time.
    //  This will need improved so each strategy only activated after the previous is spent

via https://github.com/harness-apps/eventsource/blob/master/lib/eventsource.js#L66-L67
I can also see that not all of your changes to your eventsource fork are published to github. The github version only goes as far as version 2.1.0 but on npm we have 2.1.3.  2.1.3 still has the same comment and some changes to retries but still kicking of all retry strategies at the same time:
https://www.npmjs.com/package/@harnessio/eventsource
the relevant retry code https://unpkg.com/<EMAIL_ADDRESS>Could you please tave a look at this?

I have now confirmed this locally with tweaking the eventsource examples.
A single SSE client will establish 3 new connections (and receive 3 times as many events) after the first retry.
In case of an intermittent errors, further retries then resulted in even more connections.
After running my small local example for a few minutes I had 100+ connections and greatly increased message volumes on the clients.

Please find some sample code here as well: https://github.com/csabapalfi/harness-eventsource-bug

Looks like #68 resolves this.


Add workaround for pkcs11

This is a workaround for an issue that happens when you include the pkcs11 module. The compiler will complain about three missing headers even in amalgamation builds.

Interesting... Do you have any idea as to why this issue occurs? This smells like something that would potentially require an upstream fix instead (i.e. a fix in Botan itself).
I'll be looking into this. Could you tell me which version(s) of Botan you encountered this issue with?

https://github.com/randombit/botan/issues/1447
https://github.com/randombit/botan/issues/976
There have been discussions about this in the past, but it seems nothing has been done about the issue since. I'd give it a shot of fixing it but I'm afraid I'm not qualified to tackle that with my current skill.
I was building the most recent version at the time which was 3.1.1.
My cmake generation section looks like this:
find_package(
        Botan 3.1.1
        REQUIRED
)

botan_generate(
        botan_pkcs11
            pkcs11
)

Building on Windows with MSVC.

Thank you for clarifying this!
This certainly seems like an interesting solution. Personally, I'd agree that this could be considered a "bug" in Botan. However, given the purpose/goal/mission of this CMake integration I also think that it's pretty reasonable to include your fix.
I will sleep one more night over it and most likely merge it.

Sorry about that, no idea how those whitespaces got changed.

Thank you for your contribution!


NmeaFileDevice does not stop when EOF

Thank you for this component.
While using the following syntax to read out a text file content, after it reaches end of file it starts again from beginning.
How do we stop once its EOF?
  var device = new NmeaFileDevice(filename, 1000);
  device.MessageReceived += Device_MessageReceived;

  Console.WriteLine($"\nOpening file {filename} and sending NMEA strings");
  await device.OpenAsync();

This is actually by design since it's meant for simulation (real datasources doesn't suddenly stop). I'd consider adding a property for stopping at the end, or at least an event that you can subscribe to and stop.

I agree about looping it for usual device cases other than NmeaFileDevice.
I am using this library to send data to for testing simulation and unable to send just few lines from the file now.
Is this where the code needs to be changed? I took a quick look
https://github.com/dotMorten/NmeaParser/blob/bbc7cb3589ebc06ebbf69d116c0aefc9dee613ba/src/NmeaParser/BufferedStreamDevice.cs#L128

Yeah perhaps just raise an event here, and check if the stream is still open after the event. That allows you to just cancel out in the event.

Thank you for the changes @dotMorten

Can you publish this to NuGet?

2.1 is not quite ready yet. You can build yourself or get the artifacts from the action

Tried building but I think because of all the dependencies, it fails.

Ok No worries. I edited the project file and got it working for now.


style: add semantic styling to dropdown

Closes #215
Screenshot:

Sorry for jumping in late. Since invenio-app already defined that configuration (see [here]). Would it be more sensitive to use it instead of duplicate it? (or at least use the same syntax - i.e. semantic-ui) to avoid user/developer confusion mainly :)


There are no plans to develop a desktop version of this app.

WELL, HOW CAN YOU "BE A GREAT SYSTEM", WHEN THE SYSTEM CANT BE
UP-GRADED....???, MY HELIUM MINER-MENTED, IS NOT,DOING WHAT, IT
PROMISES,ME.......
On Sun, Jan 15, 2023 at 2:21 PM Kent Williams @.***>
wrote:

There are no plans to develop a desktop version of this app.
—
Reply to this email directly, view it on GitHub
https://github.com/kent-williams/hotspot-utility/issues/19#issuecomment-1383230226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5G524DVSXH6ABJKRJOFXPDWSRE2LANCNFSM6AAAAAAT4AN2WU
.
You are receiving this because you authored the thread.Message ID:
@.***>

This is not an official app from Helium.

SIR,CAN YOU,HELP,WITH FIXING THE SYSTEM,SO I CAN BE MAKING HELIUM......?????
On Sun, Jan 15, 2023 at 3:15 PM Kent Williams @.***>
wrote:

This is not an official app from Helium.
—
Reply to this email directly, view it on GitHub
https://github.com/kent-williams/hotspot-utility/issues/19#issuecomment-1383241219,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5G524HVCBTECMQO7RSJFGLWSRLHDANCNFSM6AAAAAAT4AN2WU
.
You are receiving this because you authored the thread.Message ID:
@.***>


upgrade nsq-ruby to ~> 2.2

Reason for Change

2.2 includes a fix for memory leakage (https://github.com/wistia/nsq-ruby/pull/42)

A simple bundle update nsq-ruby would work in most cases but this will ensure the version w/o memory leaks is used in the future.
List of Changes

Update Gemspec

Risks

None.

Recommended Reviewers
@fastly/internal-engineering

nobody likes memory leaks

10k buffer limit is interesting though. We don't actually care about this because we override the queue when we construct listeners.

Yeah seems like it should be configurable. Maybe I'll do a quick PR to make that an option even though we don't currently need it.

We don't actually care about this because we override the queue when we construct listeners.

This 10k limit looks like it only for the write queue which wouldn't affect the listeners. either way, we're good


0.97.4 Breaks Meteor

Uncaught ReferenceError: require is not defined

It is probably because https://github.com/Dogfalo/materialize/blob/master/js/initial.js
// Check for jQuery.
if (typeof(jQuery) === 'undefined') {
  var jQuery = $ = require('jQuery');
}

which makes that wrong assumption about its environment.

+1

+1 @Dogfalo

+1
I explicitly install jquery as a dependency when using 0.97.1, now after I upgrade to 0.97.4, I got this error
ERROR in ./~/materialize-css/dist/js/materialize.js
Module not found: Error: Cannot resolve module 'jQuery' in /home/travis/build/PinkyJie/angular1-webpack-starter/node_modules/materialize-css/dist/js
 @ ./~/materialize-css/dist/js/materialize.js 8:19-36

I think it's also related to the newly added jQuery check logic,
// Check for jQuery.
if (typeof(jQuery) === 'undefined') {
  var jQuery = $ = require('jQuery');
}

Is that require(jQuery) should be require(jquery)? Because npm package jquery using the lowercase string.

Sorry guys, fixed in 18e012a982f626e46cd5b1f7ecb4d487e0abb886
And i've pushed a hotfix release

Is this also caused by the upgrade?
WARNING in ./~/materialize-css/~/jQuery/dist/jquery.js
There is another module with an equal name when case is ignored.
This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.
Rename module if multiple modules are expected or use equal casing if one module is expected.

WARNING in ./~/materialize-css/~/jquery/dist/jquery.js
There is another module with an equal name when case is ignored.
This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.
Rename module if multiple modules are expected or use equal casing if one module is expected.

It's weird, because I didn't see there is a jQuery folder under materialize-css/node_modules.

I get this warning twice when I add materialize in my webpack project.
I included jquery the right way, what is wrong!
WARNING in ./~/jquery/dist/jquery.js
There is another module with an equal name when case is ignored.
This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.
Rename module if multiple modules are expected or use equal casing if one module is expected.

WARNING in ./~/jQuery/dist/jquery.js
There is another module with an equal name when case is ignored.
This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.
Rename module if multiple modules are expected or use equal casing if one module is expected.

@MurhafSousli I also getting the same issue, some "solutions" can be found #2690
But my solution is roll back materialize to<EMAIL_ADDRESS>and change the file on /node_modules/materialize-css/package.json, add new line below:
 "version": "0.97.3",
  // other config..
  "main": "bin/materialize", // <== this
 // other config..

it works for me, hope it can help you :)


change the term "concept" to "observation"

In the bulk editor box, change the term "concept" to "observation" for consistency

Fixed


Full Detail View - Editing Copyrights inside the File Edit is confusing

The current workflow when updating a file's Copyright is to:

Click on a copyright
Add or edit a current copyright inside a Popover
Click "Done" to save the changes made inside the Popover
Contribute the updates made on the definition

If you're in the Modal, click "Ok" and then "Contribute"
If you're in the Full Detail Page, click "Contribute"

In step 3, if you don't click "Done" and click outside the Popover, the changes will stay in the popover but the Full Detail View won't know you made any changes.
As you can see in the gif, the green "Contribute" button is disabled, until you press "Done"

Possible solutions are:

Keep the Popover open when editing, so the user MUST click on "Done". Making sure to allow the close the Popover when there are no changes (read-only)
Ideally, when the Popover is closed, as the user clicks outside, those edits should be visible to the Full Detail View

@gianpaj let's go with 1. "Keep the popover open when editing" as it seems to be the most obvious solution to the user (clicking on Done will close the popover) and will prevent from inadvertently clicking outside while editing.
Let's keep in mind that click-outside-to-close should be still allowed in Copyrights read-only mode.


training performance

Dear Dr. Koes,
I have implemented the training on the refmodel3 using the train.py script and obtained the same results as reported in your publication. With one GPU it takes about 11s for 40 iterations.
Again, I want to test how the framework performs on DUD-E target PUR2. I use docked (with Smina) structure for each ligand and generated .gninatypes file for each ligand. Then I did the similar thing: the ligands are put in a single folder, and then split into traing and testing randomly (0.8:0.2), I can see the following output:
...........

I1104 11:14:08.481962 13687 solver.cpp:352] Iteration 0, Testing net (#1)
I1104 11:14:20.100967 13687 sgd_solver.cpp:99] Gradient clipping: scaling down gradients (L2 norm 10.878 > 10) by scale factor 0.919283

Iteration 40
Train time: 11.914718
Eval test time: 1.151939
Test AUC: 0.961171
Test loss: 0.934658
Eval train time: 4.759745
Train AUC: 0.962875
Train loss: 0.743320
Loop time: 17.827261 (1:13:58.988101 left)
Memory usage: 0.695gb (746708992)
Best test AUC/RMSD: 0.961171 inf   Best train loss: 0.743320

Iteration 80
Train time: 11.716022
Eval test time: 1.129195
Test AUC: 0.998048
Test loss: 0.108145
Eval train time: 4.739851
Train AUC: 0.991603
Train loss: 0.091265
Loop time: 17.586117 (1:13:11.258961 left)
Memory usage: 0.697gb (748298240)
Best test AUC/RMSD: 0.998048 inf   Best train loss: 0.091265

Iteration 120
Train time: 11.722106
Eval test time: 1.123328
Test AUC: 1.000000
Test loss: 0.046416
Eval train time: 4.852293
Train AUC: 0.997313
Train loss: 0.038289
Loop time: 17.698747 (1:12:52.898324 left)
Memory usage: 0.697gb (748834816)
Best test AUC/RMSD: 1.000000 inf   Best train loss: 0.038289

..................

Do you think everything is going in good order? I am currently testing other DUD-E :
I am wondering that when generating .gninatypes files for the ligands, which model is used? I can see from the docked ligand file, there are 10 models, and when gninatyper process finished, there seems to be only one model is used. Is the top ranked one is used (e.g., model 1), or gninatyper will select the exactly right model? Attach you will see an example for DUDE target PUR2.
PUR2.zip

Training structure-based machine learning models on DUDE is a bad idea.  See:
https://chemrxiv.org/articles/Hidden_Bias_in_the_DUD-E_Dataset_Leads_to_Misleading_Performance_of_Deep_Learning_in_Structure-Based_Virtual_Screening/7886165
The structures are of unknown quality and there is a lot of ligand-only signal in DUDe.
gninatyper should generate files for all poses in an sdf - not sure why you are seeing only one.

Thank you very much. I will choose other high quality datasets to perform a training and virtual screening.
I have also generated multiple gninatypes files from sdf files. Not sure why cannot do so from pdbqt files, but I don’t think it will be a problem.

Training structure-based machine learning models on DUDE is a bad idea. See:
https://chemrxiv.org/articles/Hidden_Bias_in_the_DUD-E_Dataset_Leads_to_Misleading_Performance_of_Deep_Learning_in_Structure-Based_Virtual_Screening/7886165
The structures are of unknown quality and there is a lot of ligand-only signal in DUDe.
gninatyper should generate files for all poses in an sdf - not sure why you are seeing only one.


How to set config parameters to get depth map for face anti-spoofing

Required Info

Camera Model
D435

Firmware Version
2.22.0

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17)

Platform
PC

SDK Version
{2.0 }

Language
{C++/python }

Segment
{face recognition and face anti-spoofing }

Issue Description

0 - Jet
1 - Classic
2 - WhiteToBlack
3 - BlackToWhite
4 - Bio
5 - Cold
6 - Warm
7 - Quantized
8 - Pattern

What is the difference between these different rendering methods, and what kind of colorizer should I use in which scene?
Is there documentation on this aspect?
I require to get depth map with relatively high depth accuracy within 0.3-1.5m and at the same time make the depth value and the color value(not change dynamically) one to one correspondence.
Here is my config:
rs2::colorizer _colorizer;
# set near mode
_colorizer.set_option(RS2_OPTION_VISUAL_PRESET, 2.f);
_colorizer.set_option(RS2_OPTION_COLOR_SCHEME, 0.f);
_colorizer.set_option(RS2_OPTION_HISTOGRAM_EQUALIZATION_ENABLED, 0);
_colorizer.set_option(RS2_OPTION_MIN_DISTANCE, 0.3f);
_colorizer.set_option(RS2_OPTION_MAX_DISTANCE, 2.f);

Actually, I also try to use WhiteToBlack randering method, but I found pixel values are difficult to reflect relative depth. Please help me ?

#https://github.com/IntelRealSense/librealsense/issues/2133#issuecomment-407519929

https://github.com/IntelRealSense/librealsense/issues/2133#issuecomment-407519929

@xiao-keeplearning The rendering method is for depth visualization in preview. It will not affect the depth value or accuracy. If you want to high depth accuracy, you can try preset "high accuracy".

@xiao-keeplearning Any other questions for this? Looking forward to your update. Thanks!

@xiao-keeplearning The rendering method is for depth visualization in preview. It will not affect the depth value or accuracy. If you want to high depth accuracy, you can try preset "high accuracy".
Recently I tried “high accuracy” mode, I found some problem.

The first image is from the default mode，and the second image is from high accuracy mode. Why the second image has more the Invalid Depth Band?

@xiao-keeplearning This is what is expected with high accuracy preset. High accuracy preset is for high confidence threshold value of depth and lower fill factor. For some usage such as collision avoidance, the high confidence valid depth is more important than good fill factor. You can select the preset according to your usage. For more information about preset, please refer to https://github.com/IntelRealSense/librealsense/wiki/D400-Series-Visual-Presets

@xiao-keeplearning Any other questions about this? Looking forward to your update. Thanks!

@xiao-keeplearning Any update from your side? If we don’t hear from you in 7 days, this issue will be closed. Thanks!

Thanks for your help，I have solved the problem temporarily.

@xiao-keeplearning Glad to hear the issue resolved. Thanks for the update!


export analytics for Meteor 1.3

Hi,
I'm using Meteor 1.3 so I need to be able to import analytics in order to track custom events on client side.
Thanks,
Mickael

+1

Merging this would be nice :+1:

@mlecoq thank you for this. I would love to get it merged. I'm embarrassed that we haven't replied yet. But looking at it now, there are a couple of things that jump out at me right away, such as the whitespace being changed. We will also need to update the example apps and the README.
Would you be able to resubmit it without the whitespace changes at least?
Thanks.

@mlecoq @pauldowman I cleared up the whitespaces. I reverted a49d76a, then created a new commit with just the differences, but marked @mlecoq as the author. Sorry for not getting to it sooner. I ran into issues testing it on Friday, but it turned out they were all on my end! I've released it as 1.0.7

Thanks @rgould !

Can someone push v1.0.7 to atmosphere, please? Thanks! 😁


Cannot find name 'BackgroundMode'. with ionic

I have added backroundMode module in module.ts and using in component.ts to get it in the whole app.
But when I  run
ionic cordova build android
I'm seeing the following error

importing it in app.module.ts should be enough, I think this way works until angular 10, is your project with angular?


Upgrade to sbt-houserules 0.3.5

To validate this pull request against other sbt modules, please visit this link.


Can't Install saleor repostiory

What I'm trying to achieve
I am trying to install salor repostiory by cloning "git clone git@github.com:mirumee/saleor-storefront.git" in to a terminal and I get: "Cloning into 'saleor-storefront'...
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository."
Steps to reproduce the problem

Go to MacOS Mojave 10.14.5 terminal
enter "git clone git@github.com:mirumee/saleor-storefront.git"

What I expected to happen
Get acces to the storefront
Screenshots

System information
Operating system: MacOS Mojave 10.14.5
Browser: Safari

Hi
You are using the wrong ssh key for GitHub.
There are two solutions for you:

Cloning through HTTP instead of SSH
Creating a new SSH key in ~/.ssh/github_rsa and adding to ~/.ssh/config:

Host github.com
IdentityFile ~/.ssh/github_rsa


Bug fix/disable gdb coredumping

when the process is gone we can't get any more stats from it - continue processing anyways since i.e. crash analysis may happen
disable coredumping for gdb
move 'bt full' to the end so we get as much information as possible before we enter the danger zone.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/8857/

tests blue


Increasing memory consumption when running on MSEED files

I'm running the model on my MSEED files that consist of 1 month of data, 33 stations. Using mseed_predictor with CPU, around station 8, the process gets killed. During the run, I watched the system resources and the memory consumption just keeps increasing and when it's full, the process crashes.
Running on Ubuntu 20.04

to run mseed_predictor on miniseeds of 1 month long you need at least 16 GM of RAM. Otherwise either break your mseed files to smaller chunks (e.g. 10 days or less) or run the hdf_maker and predictor if you have enough storage for pre-processed data.

I have 32GB of RAM and 32GB swap space. I ended up breaking 33 stations into 8 chunks. If I iterate over these chunks in python by calling mseed_predictor for each chunk, it still consumes all the memory and crashes. I had to iterate over these chunks with a bash script, only then the memory was released after each chunk.
So it seems like there is a memory leak in mseed_predictor. It is possible to work around it as I did or as you suggested but if there's an easy fix, it would make things much easier.

That is kind of strang. What is the sampling rate of your data? one place that the code could take too much memory is in the resampling part (especially when the data has very low -40 or 20 Hz- or very high -1000 Hz sampling rate). In this case, resampling your data in advance might help.
The code itself does not keep too many things in memory. What I am suspicious about the most is the TensorFlow itself. Some versions of it have this issue that does not release the cache and you need to manually release the memory or restart the terminal. You might learn more about this is TensorFlow forums. I don't know if they have fixed it in the newer versions. So maybe upgrading or downgrading to a specific version of TF helps.

Sampling rate is 100Hz. I'll look into TF pages. Thanks!

Hello,
I have the same issue with you ,did you fix it up?
looking forward to your reply
sincerely


Update apache-spark-3-runtime.md

These changes are in line with latest release

@abhishegupta : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@SnehaGunda: I'm sorry - only the author of this article, @midesa, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Quotes rendering incorrectly in quotation element

I've since removed these and it's going in for review soon--but flagging as something that needs fixing. It only happens with curly quotes, “” as opposed to straight quotes "".
Front-end:

Back-end:

Sorry this one is still not resolved. We've been in touch with the font foundry about it, there's a mistake with the codepoints and the quote marks are the wrong way round in this weight. They should hopefully provide a fixed version soon. Will update as soon as we have more information.

Update: We now have the bolder weight fixed, just waiting on the corresponding italics.

This should be fixed now as we've switched to a different stylistic set. Please reopen if not.


DO NOT MERGE - Try to reproduce issue #90 visual acceptance test

This project uses semver, please check the scope of this pr:

[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Try to reproduce issue #90 visual acceptance test

Visual Acceptance tests:
No changes

Finished the investigation. I did comment the outcomes in ember-cli-visual-acceptance issue#30.


cgicc link error in VS2015 - LNK2019

Hi,
I am trying to build the examples here. The installation of cgicc by vcpkg was fine without errors.
I just copied the example named "Simple URL Example: Get Method" in a new c++ project in VS2015 and during the linking I got lots of errors like:
Error	LNK2019	 unresolved external symbol "__declspec(dllimport) public: class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > __thiscall cgicc::FormEntry::operator*(void)const " (__imp_??DFormEntry@cgicc@@QBE?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@XZ) referenced in function _main
Searching for the generated cgicc.lib, it is pointed correctly by the MACRO variable:
$(VcpkgRoot)debug\lib located at Project Properties -> Linker -> General -> Additional Library Directories
What could be wrong? The compilation of cgicc was succesful without errors nor warnings.

same in vs2017. details:
1>c:\program files (x86)\microsoft visual studio\2017\professional\vc\tools\msvc\14.16.27023\include\xstring(4373): note: see declaration of 'std::basic_string<char,std::char_traits,std::allocator>'
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: class std::basic_string<char,struct std::char_traits,class std::allocator > __thiscall cgicc::FormEntry::operator*(void)const " (_imp??DFormEntry@cgicc@@QBE?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@XZ) referenced in function _main
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: bool __thiscall cgicc::FormEntry::isEmpty(void)const " (_imp?isEmpty@FormEntry@cgicc@@QBE_NXZ) referenced in function _main
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: class std::_Vector_const_iterator<class std::_Vector_val<struct std::_Simple_types > > __thiscall std::vector<class cgicc::FormEntry,class std::allocator >::end(void)const " (_imp?end@?$vector@VFormEntry@cgicc@@V?$allocator@VFormEntry@cgicc@@@std@@@std@@QBE?AV?$_Vector_const_iterator@V?$_Vector_val@U?$_Simple_types@VFormEntry@cgicc@@@std@@@std@@@2@XZ) referenced in function _main
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: __thiscall cgicc::Cgicc::Cgicc(class cgicc::CgiInput )" (_imp??0Cgicc@cgicc@@QAE@PAVCgiInput@1@@Z) referenced in function _main
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: __thiscall cgicc::Cgicc::~Cgicc(void)" (_imp??1Cgicc@cgicc@@QAE@XZ) referenced in function _main
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: class std::_Vector_iterator<class std::_Vector_val<struct std::_Simple_types > > __thiscall cgicc::Cgicc::getElement(class std::basic_string<char,struct std::char_traits,class std::allocator > const &)" (_imp?getElement@Cgicc@cgicc@@QAE?AV?$_Vector_iterator@V?$_Vector_val@U?$_Simple_types@VFormEntry@cgicc@@@std@@@std@@@std@@ABV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@4@@Z) referenced in function _main
1>ConsoleApplication1.obj : error LNK2019: unresolved external symbol "__declspec(dllimport) public: class std::vector<class cgicc::FormEntry,class std::allocator > const & __thiscall cgicc::Cgicc::operator(void)const " (_imp??DCgicc@cgicc@@QBEABV?$vector@VFormEntry@cgicc@@V?$allocator@VFormEntry@cgicc@@@std@@@std@@XZ) referenced in function _main
1>c:\users\vzay\source\repos\ConsoleApplication1\Debug\ConsoleApplication1.exe : fatal error LNK1120: 7 unresolved externals

Hi @edras, thanks for reporting this issue!
I've fixed it, please update vcpkg and rebuild cgicc.
Thanks.


It's a shame to see so many Chinese characters

English is the general language for Github. And the author of this repository is NOT Chinese, I don't get why so many Chinese getting so high here.
I am Chinese, and I feel embarrassed about this.

hi,you are too serious

要不要这么自卑啊兄弟，动不动就shame

能明白你的心情，就像中国清朝，一直还以为和古代一样，文化、发明、经济等都领先全世界的时候，对外国也是shame。
Can understand your feelings, just like China's Qing Dynasty, has always thought that as in ancient times, culture, inventions, economy, etc. all lead the world, it is also a shame to foreign countries.

2333333

agreed, चिंकी चूतिये है

It's rude to talk in non-general languages because it is essentially preventing foreign speaker from joining the conversation.

Реальное имя,Opozita reale e sistemit të emrit

No shame buddy! Google translation is getting more intelligent with machine learning. We know what you said.  Writing sh*te is another. Shows their sad mentality

@DanteAndroid  Why the Chinese people can use English talking on here , but you don't use Chinese talking to them , do you realy have no idea?

shame 你妹啊， 说明广大码农属国人最闲

xin chào, tao là Việt Nam và tao cũng không thích điều này

@connectkushal Well, peace is nice, but I think some people's behavior has become some trouble to others, so just a kind reminder.

@blackcatpolice Nice question. I just want to talk about this problem with others, so with English. And I don't think they are unable to understand it because they came to this repository (which is written in English).

English is the general language for Github. And the author of this repository is NOT Chinese, I don't get why so many Chinese getting so high here.
I am Chinese, and I feel embarrassed about this.

太搞笑了，说母语真就自己开始shame起来了呗？一个搞笑项目在这儿上纲上线，跪外国爹跪得还舒服吗？
英语是行规推荐用语，但没任何条目限制其他语言的出现，真正和平友好的氛围应该尊重任何语言的发言，至于需不需要花时间理解是你自己的选择，而不是在这儿阴阳怪气自己的母语。
可别你是国人了，我都嫌丢人，居然还能看到这样的言论。

It's a technical website, not a forum. So language matters. Of course, every language should be respected, but so do people. You see, there's no other language except English and Chinese. Why other people choose English instead of their own one? What impression will Chinese leave to others?

It's a technical website, not a forum. So language matters. Of course, every language should be respected, but so do people. You see, there's no other language except English and Chinese. Why other people choose English instead of their own one? What impression will Chinese leave to others?

你自己也说了这个是技术网站，那你为什么你不用英语来diss这纯娱乐项目呢？大家也都纯娱乐而已，而不是让你借此机会在这shame过去shame过来。所以我开头就讲，这是个搞笑项目，没必要上纲上线。
退一步就算不是娱乐项目，我也提了英语是行规用语，能用英文大家无障碍交流最好，但即便出现了其他语言也不应该成为你diss的理由。要是有个德语issue下面有人说"嘿，我也是德国人，在GitHub就用英语，我就不明白你为什么要在这儿逼逼赖赖，我看到这些德文都感到害臊"，那可得笑死。
你觉得这是在为国人遮羞吗？不，别人只会从你的言论感到深深的劣根性。


Activity Feed: mention user/team link 404

If there is a mention

when clicking on top of the @name, we get a 404 GET https://sandbox-beta.open-metadata.org/user/pmbrull 404 (Not Found)
cc @vivekratnavel @Sachin-chaurasiya

@Sachin-chaurasiya the same is happening with teams as well.

@Sachin-chaurasiya the same is happening with teams as well.

@parthp2107 , for the team, patch is included in this PR #3074


Mark dapptools as london ready

What was wrong?
Dapptools was marked as WIP.
How was it fixed?
Dapptools is now marked as London ready.
Cute Animal Picture

Codecov Report

Merging #298 (1b24b5e) into master (ab60d58) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #298   +/-   ##
=======================================
  Coverage   93.55%   93.55%
=======================================
  Files          28       28
  Lines        1676     1676
=======================================
  Hits         1568     1568
  Misses        108      108

Flag
Coverage Δ

unittests
93.55% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ab60d58...1b24b5e. Read the comment docs.


Remoting from a non Windows Server.

We need to do Remoting (https://docs.microsoft.com/en-us/windows/mixed-reality/add-holographic-remoting)  to the Hololens for Medical Resarch project, but its seems that the server side is limited to ensive Microsoft Desktop / Windows 10 ?
Whatever the video codecs used are we can then match then. Same goes for the Event stream from the headset. We are using Rust and vulcan for the rendering on a Visualization Cluster we built ourselves and it allows us to Render very large MRI / fMRI images as well as overlay any other Dicom medical data images.
Please let me know because we need to decide on best MR headset soon for the project.

I was talking to another developer here and we think maybe we can do all our rendering in our rust code and then send the frames to the c++/c# code that does the HolographicStreamerHelpers stuff.
The question of course will be the FFI. DO you do its locally or do we transport the frames over a 10G network nic. Any advice is really useful please to advice us architecturally.


git pull or git clone actions return 'https://github.com/joan2937/pigpio/': SSL certificate problem: certificate is not yet valid

While trying to perform a git pull action the returned out is
fatal: unable to access 'https://github.com/joan2937/pigpio/': SSL certificate problem: certificate is not yet valid
I found one solution was to set the http.sslVerify flag to false.  I would like to keep the this flag set to true if at all possible.
This is the SBC info from screenfetch
OS: Arch Linux
Kernel: armv7l Linux 5.10.78-2-raspberrypi-ARCH
Uptime: 4d 16h 35m
Packages: 267
Shell: bash 5.1.16
Disk: 7.9G / 30G (28%)
CPU: ARMv7 rev 4 (v7l) @ 4x 1.2GHz
RAM: 492MiB / 930MiB

This is the output from git config -l --show-origin
file:/home/alarm/.gitconfig     http.sslbackend=openssl
file:/home/alarm/.gitconfig     http.sslverify=true
file:.git/config        core.repositoryformatversion=0
file:.git/config        core.filemode=true
file:.git/config        core.bare=false
file:.git/config        core.logallrefupdates=true
file:.git/config        remote.origin.url=https://github.com/joan2937/pigpio
file:.git/config        remote.origin.fetch=+refs/heads/*:refs/remotes/origin/*
file:.git/config        branch.master.remote=origin
file:.git/config        branch.master.merge=refs/heads/master

Originally the http.sslbackend flag was not configured, but was set during trying some resolution was searching the net.   With the http.sslbackend set to openssl is when I see the "certificate is not yet valid".  If the http.sslbackend flag is set to "schannel" then the output is
fatal: Unsupported SSL backend 'schannel'. Supported SSL backends:
        openssl

Found out that my date/time was not setup correctly on the RasPi.  After installing the NTP package, then starting and enabling the service the system was rebooted.  The correct time was not set and was able to perform a get pull without any issue.


Change the thread pool size to minimumIdle on blocked initialization

Environment
HikariCP version: 3.3.1
JDK version     : 1.8.0_111
Database        : PostgreSQL|MySQL|...
Driver version  : x.x.x

In the following code
if (Boolean.getBoolean("com.zaxxer.hikari.blockUntilFilled") && config.getInitializationFailTimeout() > 1) {
         addConnectionExecutor.setCorePoolSize(Runtime.getRuntime().availableProcessors());
addConnectionExecutor.setMaximumPoolSize(Runtime.getRuntime().availableProcessors());

         final long startTime = currentTime();
         while (elapsedMillis(startTime) < config.getInitializationFailTimeout() && getTotalConnections() < config.getMinimumIdle()) {
            quietlySleep(MILLISECONDS.toMillis(100));
         }

         addConnectionExecutor.setCorePoolSize(1);
         addConnectionExecutor.setMaximumPoolSize(1);
}

The thread pool size of the addConnectionExecutor should be set as  the value of minimumIdle. There are two reasons:  first, minimumIdle is the number of connections we need, no matter it is smaller or bigger than the available processors. Second, creating connection is not CPU intensive, most works happen at the network and remote database server and the wait time is much bigger than the compute time. The number of threads should not depend on the number of available CPUs. Brian Goetz in his famous book "Java Concurrency in Practice" recommends the following formula:
Number of threads = Number of Available Cores * CPU utilization percentage *  (1 + Wait time / compute time)
With the current code, if the minimumIdle is smaller than the available CPUs, we create some threads not used. If it is bigger than the available CPUs, it takes longer to create all the required connections because the initialization thread pool is too small.

@YingLiu4203 I find no point of disagreement.  Feel free to submit a Pull Request.


persist space sizes in session

What changes are proposed in this pull request?
Currently, if a space is resized in the app, the updated sizes of spaces is not persisted in session. As a result, if app reloads, the resizing is lost. PR adds changes to persist sizes of all spaces in the app. In addition, it also adds the ability to update the sizes of spaces from active Python session
How is this patch tested? If it is not, please explain why.

resizing in the app and refreshing the app
setting sizes of a space from python shell
unit test

Preview
https://github.com/voxel51/fiftyone/assets/25350704/9084d71a-a06a-49b2-b010-571a414fa0f7
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[x] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

Persist sizes of spaces in app session
What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Updated implementation to use percent instead of pixel to be responsive to different window sizes.
Thanks @benjaminpkane for the suggestion and callout! 🙌

Good idea! Updated to use floating point instead


[europa] Add new extractor

This resolves #6528.

Except that this completely doesn't work. Even with the patch rebased and applied, I get the same error:
matej@mitmanek: youtube-dl (ec.europa.eu)$ ./youtube-dl 'http://ec.europa.eu/avservices/video/player.cfm?ref=I107758'
[generic] player: Requesting header
WARNING: Falling back on generic information extractor.
[generic] player: Downloading webpage
[generic] player: Extracting information
ERROR: Unsupported URL: http://ec.europa.eu/avservices/video/player.cfm?ref=I107758
matej@mitmanek: youtube-dl (ec.europa.eu)$

(code is in the PR #7038 )

WARNING: Falling back on generic information extractor.

You're not using my extractor.
Try running python -m youtube_dl 'http://ec.europa.eu/avservices/video/player.cfm?ref=I107758' in the root of your git checkout.

Cherry-picked in 3bb3f0410822d3d21c6199bb8915b598990628e6, thanks.


docs: scripts: generate the packages table

I created code and updated the integrations/platforms/index.mdx file.
Code generates this file with the current partner package information.
Right now, this script is started manually.
I didn't add the generation script to the Makefile: generate-files since the current documentation generation is in refactoring.
NOTE: All packages should be listed in the packages.yaml file. See the Discovery protocol. Right now, only the packages are presented from the repositories in the LangChain-ai and from googleapis organizations. See result here.

Out of curiosity, how it it currently discovering external repo partners like [cohere, elastic] and not others [google]?
@efriis Those are discovered only locally. They will be gone when the script runs in the build or in the current partners/ directory.

I like it! Can't merge this until either

some sort of discovery protocol is up to pull those in
@efriis Let me know if you want me to draft the Discovery protocol. It will take a couple of days.

@efriis Your recommendations have been addressed.

@efriis Please let me know if you want other changes? I've added profiles for the packages:

in the 'langchain` repo
in the langchain-ai organization but outside of the longchain repo
intrgration packages ouside of the langchain-ai organization (now I aware only about the Google packages. It could be something else)

I've added the correspondent instructions. They are also could be organized as a separate "Integration Package specification" doc.

Still sorry for the delay on this one. I'm planning on modifying to make the above suggested changes, as well as:

make the generation script happen during the docs build
split the generated package listing by type (base ones, shared repo ones, monorepo ones, then third party ones)
try to standardize the paths that the packages point to (e.g. the googleapis ones point to a different relative parent than the langchain ones)

Re: split the generated package listing by type (base ones, shared repo ones, monorepo ones, then third party ones) - it is also here. See the slug parameter in the packages.yaml It is not explicit but it is f(slug)

I think e.g. all the google third party ones actually point to the wrong directory. All the langchain ones point to the package root (that has pyproject). All the google ones point to the src (code) root (that has root init.py)

I think e.g. all the google third party ones actually point to the wrong directory. All the langchain ones point to the package root (that has pyproject). All the google ones point to the src (code) root (that has root init.py)

I'm not sure I get you. We are talking about the path parameter in the packages.yaml file, right?
For example, the langchain-google-firestore-python package, packages.yaml: L156:160 The path: path: src/langchain_google_firestore is OK. It is the source code that IS the python package, copied here site-packages/langchain_google_firestore in the python environment when the package is installed.
Using packages/path allows us to parse the source directly from GitHub without installing the package. It is not required. It is used freely in the METADATA package file without any standard.
It helps us to find (and parse) the package source code without installing the package. But we can just remove this path parameter and add a requirement to install the package to allow parsing its files (I'm not fun of it because many packages are incompatible in dependencies and we should deal with it, with the install-parse-uninstall cycle?). The path parameter is also helpful when we have multiple packages in one repo (all packages under partner/). LangChain repo has a standard on the source code placement but other vendors have different standards (like Google with src/ prefix), so the path helps everybody with simple navigation in the repo code.

closing because these are now populated in a slightly different way in FeatureTables
on the package front still open to a yaml file outlining them, but don't think it has value until it powers a feature

@efriis

on the package front still open to a yaml file outlining them, but don't think it has value until it powers a feature
IMO, having a separate file with a list of packages is much cleaner than ingesting a package list inside code. It is better is support.  New packages go to this file and don't touch the code.


qdrant: 1.2.2 -> 1.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for qdrant is: Vector Search Engine for the next generation of AI applications
meta.homepage for qdrant is: https://github.com/qdrant/qdrant
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.0 with grep in /nix/store/k0zrkw8151yxpjngqcdw385m17xmnd3z-qdrant-1.3.0
found 1.3.0 in filename of file in /nix/store/k0zrkw8151yxpjngqcdw385m17xmnd3z-qdrant-1.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
qdrant

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k0zrkw8151yxpjngqcdw385m17xmnd3z-qdrant-1.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A qdrant https://github.com/r-ryantm/nixpkgs/archive/99db658cb603e624d0df3e73cea140600f716bb5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k0zrkw8151yxpjngqcdw385m17xmnd3z-qdrant-1.3.0
ls -la /nix/store/k0zrkw8151yxpjngqcdw385m17xmnd3z-qdrant-1.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

qdrant

Maintainer pings
cc @dit7ya for testing.

@ofborg build qdrant qdrant.passthru.tests

Result of nixpkgs-review pr 240047 run on aarch64-darwin 1

1 package built:

qdrant


The named parameter 'phone_number' isn't defined.

Hi. I am getting this error : The named parameter 'phone_number' isn't defined.

Isn't it fromNumberId instead ?

thanks for report, package is updated.


Build failure nixpkgs 23.05

Description of changes
Fix https://github.com/NixOS/nixpkgs/issues/235179
Update Microchip uboot version to linux4microchip+fpga-2023.06
Things done

[OK ] Tested the changes in your own NixOS Configuration
[OK] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

Will raise again in single commit


Optionally map relationships to nested attributes

For "unusual" non Ruby on Rails usage mapping relationships to nested attributes should be optional.

@markosankovic Thanks! This certainly could be useful for more people. Would you mind adding a unit test for this method?
It would help us to make sure this works as intended in the feature if any changes are made.

I made two issues for the remaining things :)
https://github.com/hyperoslo/NSManagedObject-HYPPropertyMapper/issues/79


[dif_kmac] Functions for SCA

#7757 adds an initial implementation for SHA3/KMAC SCA program sha3_serial.c. I had to perform several writes to KMAC registers to unblock that PR but most of those should be replaced when they are implemented in dif_kmac library:

[ ] kmac_block_until_idle(): We can remove this if we implement a function like dif_kmac_get_status() that returns the status bits in the KMAC_STATUS register.
[ ] kmac_reset(): We can remove this when dif_kmac_reset() is implemented (currently only declared in dif_kmac.h)
[ ] kmac_absorb_end(): I think we can wait until we finalize the way we capture traces (see #7841).

I've done a PR here #8003 to facilitate SCA measurements. With these changes, software can exercise KMAC using the following sequence:

Configure mode and key etc.
Write data to the message FIFO.
Append output length to message FIFO.
Send START and PROCESS commands, got to sleep and wait for event_absorbed interrupt. After a configurable delay. KMAC will first do the absorb and then the squeeze.
Read the digest to verify correct operation.
Start over.

I've implemented a simple self-checking test to verify this on my branch here: https://github.com/vogelpi/opentitan/commits/kmac-sca-dev

Update: #8240 changes the way how we capture traces. sha3_serial now uses the FPGA SCA mode of KMAC. As for AES, Ibex configures the timer, starts the processing (with a delay), and then enters sleep (wfi).


onnx conversion for custom trained trocr base stage1

Feature request
I have trained the base stage1 trocr on my custom dataset having multiline images. The trained model gives good results while using the default torch format for loading the model. But while converting the model to onnx, the model detects only first line or part of it in first line. I have used this https://github.com/huggingface/transformers/issues/19811#issuecomment-1303072202
for converting the model to onnx. Can you kindly provide the insights about what i should do differently, in order to get the desired multiline output from the onnx converted model.
Motivation
How to update the onnx conversion of trocr in order to support multiline trocr trained model
Your contribution
Trained a trocr base stage1 model for multiline dataset.

Hi @Mir-Umar, thank you for the report. Could you provide a reproducible example of the issue? Thank you!


Fixed Header columns do not adjust when animating the datatable width

Hi sir, I do encountered an issue these past few days when using fixed header extendsion in datatable. The column's width are automatically adjust when resizing the browser but in my case I do have a different situation. In my case, the container of the datatable is being animated specifically the width so for example when the sidebar pops out at the left side the content will consumes the remaining space in the screen so the datatable width is animated to a smaller one but the header cell's width is not the same as the content cell's width, It will be adjusted only after when I clicked the header cell for sorting. Pls help me :(

Hi - my apologies if the information in the "New Issue" template text and "Contributing" document aren't clear enough (any suggestions on how to make them more clear are welcome).
Please post support questions in the DataTables forums with a link to a test page showing the issue.


Gelf writer

Fixes #589
Have fun. 😉

The current build problems doesn't seem to relate to the new writer. Could you please look at them?

The build is failing because mutation testing is a bit low (30%). The default is 100% (which you are not expected to achieve). Have a look at other modules to see how to decrease that threshold.

Note that you can sort the pom.xml with mvn sortpom:sort (I know, it is a pain, but it makes diffs so much more readable when things are in a predictable order, and thus so much easier to code review).

@gehel I think, I fixed all your requested changes and the build is successful.

This is looking really good! I have a few more comments (I am a pain in the ..., yes I know). Feel free to let me know if I'm too much of a pain.
Mutation coverage is still on the low side. But most of the nominal paths are correctly asserted, so I'm fine with merging as is.

@gehel I will look into your suggestions tomorrow, please wait with merging. Thanks for all the good tips and suggestions.

Done.

The booleanAsNumber and settings parameters on GelfWriterFactory are still unused.

Damn. Corrected that as well.

Looks good! Merging!
Thanks a lot for the efforts!

You're welcome. Added wiki documentation as well. Can you tell me, when this will be available in an official release?

@gehel The GELF-Writer doesn't seem to be in 266-SNAPSHOT. Any clues why?

The jar itself seems to be published (https://oss.sonatype.org/content/repositories/snapshots/org/jmxtrans/jmxtrans-output-gelf/266-SNAPSHOT/). I have not checked the main ueber jar, but I suspect just some delay with sonatype. If that's not the case, let me know, I'll investigate some more.
I can make a release tonight...

Release in progress: https://travis-ci.org/jmxtrans/jmxtrans/builds/255795067 (I'll check on it tomorrow, but it should just complete with no problem).

I just downloaded the latest nightly all.jar from https://oss.sonatype.org/content/repositories/snapshots/org/jmxtrans/jmxtrans/266-SNAPSHOT/ and it was missing there. Thanks.


JSON namespacing

Is there any way to configure the output of eventrouter so that it namespaces events, e.g wrap every output in {'eventrouter': {<all fields in here>}}?
We manage our elasticsearch indexes using a company-wide index template and having it properly namespaced (preferably in a configurable way, i.e we'd be able to pass in what it wraps the json object in) would help to ensure the event related fields don't get mistakenly used for other purposes.

FWIW i let the eventrouter log to stdout (which is captured by fluentd) and then i can namespace on the runtime environment.

That’s interesting, we have the same setup so guess I can get fluentd to do some magic there.

having discussed this internally we decided the least hacky way to handle this was to attempt to add it ourselves - don't want to change the global fluentd + other options involved copious bash hackery.


There are two publication entries for Iwami (1985), both with different number of characters/taxa, but the citation info is for the same paper

Feedback from web UI.

DOI/URI for the publication is: https://ci.nii.ac.jp/naid/110000009175/en/


Cannot specify GPU when carrying out inference with mask DINO

When I set cfg.MODEL.DEVICE = 'cuda:1' ,it says illegal memory access. The problem does not exist when i set cfg.MODEL.DEVICE = 'cuda:0', how to solve this?

I am still trying to figure out this problem. If you wanna use cuda:1 and run on this single GPU, you can add 'CUDA_VISIBLE_DEVICES=1' before your command, for example
CUDA_VISIBLE_DEVICES=1  &&  python train_net.py --resume --num-gpus 1 --config-file \
    configs/coco/instance-segmentation/maskdino_R50_bs16_50ep_3s_dowsample1_2048.yaml

This will only use the cuda:1 device.

Hi, thanks for your reply.
We would like to implement multiple maskdino models on multiple gpus, it would be great if we can do so.
Is it because the .cuda() defaults using cuda:0 in the code? Could it be .to('cuda:1') where the 'cuda:1' reads from the cfg.MODEL.DEVICE? Just my guess

Maybe you guess if right, but I never used your way of specifying cuda devices. You can implement multiple maskdino models on multiple gpus with my command, which works well for me.

We are implementing the models for inference on our project, so probably cannot just use the command above. I want to see if the codes can be modified that it can read from the cfg.MODEL.DEVICE.
https://github.com/IDEA-Research/MaskDINO/blob/95cf05ccfd1d0496bc92980f29a18536c92b450f/maskdino/modeling/transformer_decoder/maskdino_decoder.py#L268
As I see the codes here use 'cuda' only without specifying the gpus, is there a way to pass the gpu no. as specified by cfg.MODEL.DEVICE into those codes?


switch to BIP157 instead of current address subscription client sync model

Goal: to replace subscribing to addresses on a wallet server.
Today, to figure out which transactions the client is missing, the client has to send the server a list of tx hashes that it already has. For an address with 100k transactions, that's a lot of information to send back and forth.
Instead of subscribing, the wallet server will add bloom filters to the headers for each block. When a new block comes in, the client will check the bloom filter for addresses it cares about. If the check is positive, the client will request the whole block (as per BIP157), or even more efficiently, just the txns for the addresses it cares about.
Note: you still have to subscribe to block headers in this model.

Neutrino is an implementation of BIP157: https://blog.lightning.engineering/posts/2018/10/17/neutrino.html

bip158 (prereq for 157) support was merged into bitcoin core: https://github.com/bitcoin/bitcoin/pull/14121


e2e: extract APIExport/APIBinding helpers

Summary
Extract helpers for APIResourceSchema, APIExport, APIBinding from the cross cluster conformance test so they can be more easily reused.
Related issue(s)
Fixes #1194

/assign @shawn-hurley
/cc @shawn-hurley
I know you asked for Cowboys, but will Sheriffs work?


Scaling down fails due to missing permission

It seems that Kubernetes is failing to scale down an autoscaling group. The permission I could see that it seems to be missing is autoscaling:DescribeLaunchConfigurations. Not sure if it needs more than just that one.

Thanks for raising this.
I added autoscaling:DescribeLaunchConfigurations to the master IAM role. But I think that should not be needed in all cases. But it seems there were some things missing in the RBAC rights on Kubernetes level. I fixed that and now it seems to work for me. Please give it a try.

Thanks for that, going to give it a shot.


main: added structured log option [Fixes #39]

Description

Added option for structured log
Usage is displayed in case config file is not provided as an option

Fixes #ISSUE-39
Type of change

New feature (non-breaking change which adds functionality)

Checklist

[ ] Unit tests updated: NA
[+ ] Documentation updated

@faizan82 thanks for the PR. It looks good to me, I just want another set of eyes on it.
Apologies for moving so slow, unfortunately we're short staffed due to the pandemic.


The pm:update event is not fired after calling disable on a layer for the version 2.11

I've updated my version from 2.9.0 to 2.11. The pm:update event is not fired after calling disable on a layer. I downgrade to 2.9 and it works.

in addition to that in version 2.9 when the Edit mode is activated for polygons, the corner points are not visible.

pm:update is only fired if the layer was edited. So it is not fired everytime. I found that pm:update is not fired only for markers, can you confirm that?

yes, I confirm that. It actually works well for polygons but I still can't see the corner points for polygons when edit mode is activated.

can't see the corner points for polygons

In the newest version?

Yes in the newest version 2.11.3

can't see the corner points for polygons

Can you please create an sample and confirm that this also happens here: https://jsfiddle.net/o1dwu2vg/
The corners looks good:

@adamsjoe96 ?

sorry for the delay. in the sample, it works correctly. I don't know why it still doesn't work for me. I'll test it again Tuesday.


Any new release plan?

The last release is v2.0.0 on Oct 2, 2018. Is there any new release plan?

Not much have happened since then :)
I will release a 2.1.0 once https://github.com/emilk/loguru/pull/108 and https://github.com/emilk/loguru/pull/118 lands

Loguru 2.1.0 released: https://github.com/emilk/loguru/releases/tag/v2.1.0


Whole screen goes white when activating TextExtractor

Microsoft PowerToys version
0.75.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
TextExtractor
Steps to reproduce

Activate TextExtractor either by keyboard shortcut or system tray shortcut.
Screen becomes blank (white).
Text grabbing impossible.

Bug report and screenshot attached.
[PowerToysReport_2023-10-31-08-19-29.zip](https://github.com/microsoft/PowerToys/files/13213300/PowerToysReport_2023-10-31-08-19-29.z

ip)
✔ Expected Behavior
To see the app windows and be able to grab text from them.
❌ Actual Behavior
When I run Text Extractor, the whole screen becomes white. No windows are visible, therefore marking any text to be grabbed is impossible.
This happens both with the keyboard shortcut and the PowerToys app shortcut in the system tray.
I have 2 monitors and the both become fullscreen white when I run TextExtractor.
On top of the screen area I can see the OCR language selector drop down menu, the selector for single line, select as a table and the settings icon.
Other Software
No response

This is frequently caused by the app Mica For Everyone. Please see #28217 for more details on how to resolve this incompatibility. Does this help? /needinfo

Yes, that helped!


Deprecated CRD API version

When installing the Helm chart on Kubernetes 1.16+ (for example Kubernetes 1.22) the following warning is issued:
W0906 17:35:07.323205    6568 warnings.go:70] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
This issue was fixed in KubeMod's manifests: see https://github.com/kubemod/kubemod/issues/65
The same fix should be applied to the manifests carried with the Helm chart.

+1 on this.  The helm chart does not work on recent versions of k3s, which has deprecated APIs removed.


Remove duplicated CollectExpression.of

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6af39e9). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #5915   +/-   ##
=========================================
  Coverage          ?   82.94%
  Complexity        ?    12144
=========================================
  Files             ?     1427
  Lines             ?    46461
  Branches          ?     5731
=========================================
  Hits              ?    38539
  Misses            ?     5241
  Partials          ?     2681

Impacted Files
Coverage Δ
Complexity Δ

...ort/task/elasticsearch/GetResponseRefResolver.java
90.47% <100%> (ø)
15 <1> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6af39e9...f38242b. Read the comment docs.


CASSANDRA-9143 change needs dtest change

CASSANDRA-9143 introduced strict check for not allowing incremental subrange repair.
dtest needs to pass -full to repair command to invoke subrange repair.
The change works for older version of cassandra as well.

+1


Cloudflare R2 Storage

It is recommended to add the Cloudflare R2 storage plugin.

https://developers.cloudflare.com/r2/

Hi, @Liamn0244
Have you tested configuring R2 using the storage-s3 plugin? I think it should work fine.

Sorry, does anyone have configuration best practices for Cloudflare R2?
I can't find the AccessToken, I wrote my Account ID and API Token to AccessKeyID and AccessKeySecret, is the AccessToken required?

storage-s3 can be used for Cloudflare R2


allow Pythonic None as checksum

fixes #11 cc @FrancescAlted @FrancescElies

a bot hacky, but O.K.

Hi @esc
I had a look to the failing test and this PR did not introduce any new failed tests.

Merge Travis CI output
Previous comit

The traceback
Traceback (most recent call last):
  File "/Users/cesc/anaconda/envs/py3/lib/python3.5/site-packages/nose/case.py", line 19
8, in runTest
    self.test(*self.arg)
  File "/Users/cesc/src/bloscpack/test/test_headers.py", line 170, in test_decode_blosc_
header
    nt.assert_equal(expected, header)
AssertionError: {'blocksize': 88, 'typesize': 8, 'version[58 chars]: 92} != OrderedDict(
[('version', 2), ('versionlz'[86 chars]08)])

I executed them locally, the only parameter not matching are the flags

expected flag: 3 (0x3)
header flag: 19 (0x13)

The test sets blosc_args.shuffle = True therefore according to header spec if I understand it correctly the expected flag should be 0x1X.
About X (bit 4 to 7) I'm not sure anymore, we have 0x3 in the header which means is using zlib, right? but blosc_args value is Out[8]: BloscArgs(typesize=8, clevel=7, shuffle=True, cname='blosclz')
Is this a bug, or shall the expected flag in the test be 19 (0x13)?

It works fine with older versions, 1.2.7 for example. Not exactly sure, but perhaps the header format for blosc has changed? cc @FrancescAlted @aalted

If I see correctly write_compression_header has changed after that version, now it  sets dont_split and sets BLOSC_DOSHUFFLE and BLOSC_MEMCPYED. In the links you can see the lines that are actually setting the flag to 0x13.

O.K. let's move the discussion to #62 -- if python-blosc is broken, we should a) fix bloscpack and b) limit the minimum version of python-blosc via the requirements.txt file.


Configurable keybindings

I love lazydocker. Good job, Jesse!
Is your feature request related to a problem? Please describe.
Default keybindings are hard coded. Problem is, not every keyboard has [ and ] keys.
Describe the solution you'd like
Make the keybindings configurable.
Describe alternatives you've considered
Move the previous/next tab keybindings from [ and ] to something else.

A way to solve this is to use the config file and this little library

very cool library! I wonder how best to go about this: we currently have most (though not all) of our keybindings defined in one place, so for those which are stored there, it would not be hard to merge in what we have defined in the user config. In fact even in the other places it wouldn't be hard just to refer to the user config.
in config.yml we can have:
keybindings:
  stopContainer: 's'
  nextTab: '['
  previousTab: ']'

then in keybindings.go we can have:
		{
			ViewName:    "containers",
			Key:         keybinding.MustParse(gui.Config.UserConfig.Keybindings.StopContainer),
			Modifier:    gocui.ModNone,
			Handler:     gui.handleContainerStop,
			Description: gui.Tr.Stop,
		},

How does that approach sound? Only real downside with that approach is that you don't get to specify if you want to use the gocui.ModAlt modifier, but we aren't currently using that anywhere anyway.

That approach sounds good, we could indeed look into the modAlt, but that is an issue for the library i guess. I could spend this weekend on it?

I agree @glvr182 , and if you want to spend time on this that would be awesome :)

Also worth noting that if we set a key to be the empty string, that will enable the user to disable certain keys, which satisfies https://github.com/jesseduffield/lazydocker/issues/27

also worth noting we'd need to support arrays of keys which luckily is already covered by that library

@jesseduffield Do you think we should use a map or a struct for the config?

I think a struct works better, that way we have strong typing. I made the decision to switch the i18n stuff to a struct and it's already proven really helpful in telling me when things are out of sync :)

Any news on this one?

This is totally do-able: we can just copy the approach that lazygit uses: https://github.com/jesseduffield/lazygit/blob/master/pkg/config/user_config.go
Any takers?


iOS 7 support

What about iOS 7 support ?

Installation won't be possible via Cocoapods/as a dynamic library, but dragging and dropping files should work. Haven't actually tested on iOS 7 though—what errors are you seeing?
Thanks!

Closing, feel free to reopen if there's more info to provide :)


feat: integrate postgresql datasource

@thucpn i think your problem comes from not calling vectorStore.checkSchema(db), but I also think that the user should not have to call this, let me refactor PGVectorStore and I'll re-assign this PR to you


Move lingva ML to unstable as and some other not tested translators

Sometimes pipeline with tests fails for lingva translator: https://github.com/translate-tools/core/actions/runs/5609336894/jobs/10262901755
The same thing with TartuNLPTranslator, also tests are disabled for the LibreTranslateTranslator.
We have to mark all those translators are unstable, because we can't ensure its quality and availability

We have to add translator LingvaTranslate


50mm fan cage is too short

This is probably a pretty simple error, but I don't have time to fix ATM and do not want to forget again.

Fixed in the beta branch.


FormRequestServiceProvider tries to copy non-JSON request as JSON

Laravel Version: 8.83.7
PHP Version: 8.0.16-r1

Description:
Summary
During bootstrapping, the FormRequestServiceProvider makes a deep copy of the Request object. While doing so, any Request is treated as if it contained JSON payload which leads to the side-effect that json_last_error is set for non-JSON requests. As other parts of Laravel check for json_last_error being clear later on, this has undesired effects.
Details
At some point during bootstrapping, the backtrace looks basically like this
Request.php:384, Illuminate\Http\Request->json()
Request.php:435, Illuminate\Http\Request::createFrom()
...
/Providers/FormRequestServiceProvider.php:33-37}()
...
Kernel.php:111, Illuminate\Foundation\Http\Kernel->handle()
index.php:74, {main}()

and the Request object makes a deep copy of itself in Illuminate\Http\Request::createFrom().
/**
 * Create a new request instance from the given Laravel request.
 *
 * @param  \Illuminate\Http\Request  $from
 * @param  \Illuminate\Http\Request|null  $to
 * @return static
 */
public static function createFrom(self $from, $to = null)
{
  $request = $to ?: new static;

  $files = $from->files->all();

  $files = is_array($files) ? array_filter($files) : $files;

  $request->initialize(
    $from->query->all(),
    $from->request->all(),
    $from->attributes->all(),
    $from->cookies->all(),
    $files,
    $from->server->all(),
    $from->getContent()
  );

  $request->headers->replace($from->headers->all());

  $request->setJson($from->json());

  if ($session = $from->getSession()) {
    $request->setLaravelSession($session);
  }

  $request->setUserResolver($from->getUserResolver());

  $request->setRouteResolver($from->getRouteResolver());

  return $request;
}

The problematic line is $request->setJson($from->json()) and the invocation of $from->json(). This getter method is not side-effect free.
/**
 * Get the JSON payload for the request.
 *
 * @param  string|null  $key
 * @param  mixed  $default
 * @return \Symfony\Component\HttpFoundation\ParameterBag|mixed
 */
public function json($key = null, $default = null)
{
  if (! isset($this->json)) {
    $this->json = new ParameterBag((array) json_decode($this->getContent(), true));
  }

  if (is_null($key)) {
    return $this->json;
  }

  return data_get($this->json->all(), $key, $default);
}

For non-JSON requests, this method calls json_decode on content which is not JSON. As a smaller cosmetic defect, it also modifies the original request $from by setting $this->json to an new, empty ParameterBag.
Proposed solution
Remove $request->setJson($from->json()) from Illuminate\Http\Request::createFrom().
Calling this method is not necessary at all for making a deep copy. Everything which is needed is already copied by $request->initialize(...) some line earlier, especially the content of the request. If a JSON interpretation of the content is required later  in the application, then json() can (and will) still be called on the deep copy which leads to the exact same result. Bonus points:

no unconditional call to json_decode for non-JSON requests
no side-effect on json_get_last_error
slightly higher efficiency by skipping an unnecessary step
no modification of the original Request object

Can you give a clear way to reproduce this in a fresh Laravel app?

Create a new Laravel App
Create a route and a controller method for the root
Make a AJAX request to the route which is non-JSON, for example POST some data encoded as multipart/form-data
During the middleware Laravel makes a deep-copy of the Request object and also tries to copy the content as JSON. This silently fails, but leaves a stale error code in json_last_error

I'll need a clear explanation of what actually breaks. There's still no bug here. Having json_last_error set isn't a bug imo.

I don't think there's a big issue here tbh. You're free to make a proposal through a PR if you want. Thanks

It is a bug in combination with #42107, because this is one of the code lines which set json_last_error and then later break setData (see #42107). Of course one could argue that this is an error in setData.
However, and this are the real bugs, the copy-method

is not side-effect free,
modifies the original object, and
tries to unconditionally interpret some content as JSON which is not JSON

Luckily, json_decode stops early after it has encountered the first parsing error. Simply assume you upload a huge file (e.g. a media file) and then you let json_decode process it. Surely, json_decode will simply stop after the first few bytes, but it is still something you should not do.


Layout redesign phase 2 - refactor

Phase 2 of #1609 INCOMPLETE
Phase 2 - Refactor

[ ] Group containers -> containers/
[ ] Refactor sidebar/left-bar -> containers/left-bar
[ ] Refactor sidebar/right-bar -> containers/right-bar
[ ] Pull sidebar content -> widgets/

e.g. widgets/command

[ ] Pull filter-list / selector -> ui/selector
[ ] Pull loader out of status
[ ] Merge status into bottom bar

Closing as interest in this has stalled. Happy to reopen if someone wants to pick it up


Hydrated docking

I'm getting an error with this exact process. What could be the reason?
Return message: Command line parse error: unknown option maps
https://github.com/ccsb-scripps/AutoDock-Vina/blob/d13f9e93e253573b01ffbb28cfd1e2dd26ff9002/docs/source/docking_hydrated.rst?plain=1#L131

Hi @mirjr, could you check:
vina --version

If you possibly have multiple versions of vina like i do, --maps should work for the latest version 1.2.5, and you can get it from here. I got the same error when i tried to run the command using AutoDock Vina 1.1.2.

Hi @mirjr, could you check:
vina --version

If you possibly have multiple versions of vina like i do, --maps should work for the latest version 1.2.5, and you can get it from here. I got the same error when i tried to run the command using AutoDock Vina 1.1.2.

Thanks a lot, it worked!


Linear regression tutorial was not reproducible

I copied the code  but did not get the correct results:
julia> describe(chain)
2-element Array{ChainDataFrame,1}

Summary Statistics
        parameters     mean     std  naive_se    mcse        ess   r_hat
  ────────────────  ───────  ──────  ────────  ──────  ─────────  ──────
   coefficients[1]   1.0000  0.0000    0.0000  0.0000  2016.6363  0.9998
   coefficients[2]   0.0000  0.0000    0.0000  0.0000  2089.0093  0.9997
   coefficients[3]   0.0000  0.0000    0.0000  0.0000  1703.9638  1.0013
   coefficients[4]  -0.0000  0.0000    0.0000  0.0000  2006.2867  1.0006
   coefficients[5]   0.0000  0.0000    0.0000  0.0000  1575.2603  0.9998
   coefficients[6]   0.0000  0.0000    0.0000  0.0000  1448.6048  1.0002
   coefficients[7]  -0.0000  0.0000    0.0000  0.0000  1925.6609  1.0006
   coefficients[8]  -0.0000  0.0000    0.0000  0.0000  2077.7749  1.0000
   coefficients[9]  -0.0000  0.0000    0.0000  0.0000  2040.8867  1.0010
  coefficients[10]   0.0000  0.0000    0.0000  0.0000  1595.8217  0.9999
  coefficients[11]  -0.0000  0.0000    0.0000  0.0000  1270.4007  1.0005
         intercept  -0.0000  0.0000    0.0000  0.0000  1761.9700  0.9999
                σ₂   0.0000  0.0000    0.0000  0.0000     8.0321  1.6490

Quantiles
        parameters     2.5%    25.0%    50.0%   75.0%   97.5%
  ────────────────  ───────  ───────  ───────  ──────  ──────
   coefficients[1]   1.0000   1.0000   1.0000  1.0000  1.0000
   coefficients[2]  -0.0000  -0.0000  -0.0000  0.0000  0.0000
   coefficients[3]  -0.0000  -0.0000   0.0000  0.0000  0.0000
   coefficients[4]  -0.0000  -0.0000  -0.0000  0.0000  0.0000
   coefficients[5]  -0.0000  -0.0000  -0.0000  0.0000  0.0000
   coefficients[6]  -0.0000  -0.0000   0.0000  0.0000  0.0000
   coefficients[7]  -0.0000  -0.0000   0.0000  0.0000  0.0000
   coefficients[8]  -0.0000  -0.0000  -0.0000  0.0000  0.0000
   coefficients[9]  -0.0000  -0.0000  -0.0000  0.0000  0.0000
  coefficients[10]  -0.0000  -0.0000   0.0000  0.0000  0.0000
  coefficients[11]  -0.0000  -0.0000  -0.0000  0.0000  0.0000
         intercept  -0.0000  -0.0000  -0.0000  0.0000  0.0000
                σ₂   0.0000   0.0000   0.0000  0.0000  0.0000

Julia: 1.4.2
Turing: 0.13.0

I can confirm this is the case (Julia 1.4.1 and Turing 0.13.0 on OS X).

Thanks! There are some other problems with the linear regression tutorial currently as well. Among other things, the standardization is incorrect (see https://github.com/TuringLang/TuringTutorials/issues/73), it contains a data leakage, and the model (or its documentation) are incorrect. I'm currently updating the tutorial, and hopefully that will resolve the problems that you observed as well.

Tested it with the updated tutorial code:
using Turing, Distributions
using RDatasets
using MCMCChains, Plots, StatsPlots
using MLDataUtils: shuffleobs, splitobs, rescale!
using Distances
using Random
Random.seed!(0)

Turing.turnprogress(false)

data = RDatasets.dataset("datasets", "mtcars")
select!(data, Not(:Model))
trainset, testset = splitobs(shuffleobs(data), 0.7)

target = :MPG
train = Matrix(select(trainset, Not(target)))
test = Matrix(select(testset, Not(target)))
train_target = trainset[:, target]
test_target = testset[:, target]

μ, σ = rescale!(train; obsdim = 1)
rescale!(test, μ, σ; obsdim = 1)
μtarget, σtarget = rescale!(train_target; obsdim = 1)
rescale!(test_target, μtarget, σtarget; obsdim = 1)

# Bayesian linear regression.
@model function linear_regression(x, y)
    σ₂ ~ truncated(Normal(0, 100), 0, Inf) # Set variance prior.
    intercept ~ Normal(0, sqrt(3)) # Set intercept prior.
    nfeatures = size(x, 2)
    coefficients ~ MvNormal(nfeatures, sqrt(10)) # Set the priors on our coefficients.
    mu = intercept .+ x * coefficients # Calculate all the mu terms.
    y ~ MvNormal(mu, sqrt(σ₂))
end

model = linear_regression(train, train_target)
chain = sample(model, NUTS(0.65), 3_000)

plot(chain)
describe(chain)

Looks good but I get two warnings.
[ Info: [Turing]: progress logging is disabled globally
┌ Info: Found initial step size
└   ϵ = 1.6
┌ Warning: The current proposal will be rejected due to numerical error(s).
│   isfinite.((θ, r, ℓπ, ℓκ)) = (true, false, false, false)
└ @ AdvancedHMC C:\Users\kaisv.TUGRAZ\.julia\packages\AdvancedHMC\P9wqk\src\hamiltonian.jl:47
┌ Warning: The current proposal will be rejected due to numerical error(s).
│   isfinite.((θ, r, ℓπ, ℓκ)) = (true, false, false, false)
└ @ AdvancedHMC C:\Users\kaisv.TUGRAZ\.julia\packages\AdvancedHMC\P9wqk\src\hamiltonian.jl:47
2-element Array{ChainDataFrame,1}

Summary Statistics
        parameters     mean     std  naive_se    mcse       ess   r_hat
  ────────────────  ───────  ──────  ────────  ──────  ────────  ──────
   coefficients[1]  -0.0413  0.5648    0.0126  0.0389  265.1907  1.0010
   coefficients[2]   0.2770  0.6994    0.0156  0.0401  375.2777  1.0067
   coefficients[3]  -0.4116  0.3850    0.0086  0.0160  695.3990  1.0032
   coefficients[4]   0.1805  0.2948    0.0066  0.0126  479.9290  1.0010
   coefficients[5]  -0.2669  0.7168    0.0160  0.0316  373.0291  1.0009
   coefficients[6]   0.0256  0.3461    0.0077  0.0119  571.0954  1.0028
   coefficients[7]   0.0277  0.3899    0.0087  0.0174  637.1596  1.0007
   coefficients[8]   0.1535  0.3050    0.0068  0.0117  579.1998  1.0032
   coefficients[9]   0.1223  0.2839    0.0063  0.0105  587.6752  0.9995
  coefficients[10]  -0.2839  0.3975    0.0089  0.0195  360.9612  1.0019
         intercept   0.0058  0.1179    0.0026  0.0044  580.0222  0.9995
                σ₂   0.3017  0.1955    0.0044  0.0132  227.2322  1.0005

Quantiles
        parameters     2.5%    25.0%    50.0%    75.0%   97.5%
  ────────────────  ───────  ───────  ───────  ───────  ──────
   coefficients[1]  -1.0991  -0.4265  -0.0199   0.3244  1.1093
   coefficients[2]  -1.1369  -0.1523   0.2854   0.7154  1.6488
   coefficients[3]  -1.1957  -0.6272  -0.3986  -0.1800  0.3587
   coefficients[4]  -0.3896  -0.0155   0.1663   0.3593  0.7818
   coefficients[5]  -1.6858  -0.6835  -0.2683   0.1378  1.1995
   coefficients[6]  -0.6865  -0.1672   0.0325   0.2214  0.7251
   coefficients[7]  -0.7644  -0.1976   0.0090   0.2835  0.8185
   coefficients[8]  -0.4980  -0.0194   0.1451   0.3428  0.7685
   coefficients[9]  -0.4643  -0.0294   0.1237   0.2807  0.7218
  coefficients[10]  -1.0898  -0.5091  -0.2846  -0.0413  0.5163
         intercept  -0.2240  -0.0671   0.0083   0.0746  0.2364
                σ₂   0.1043   0.1860   0.2525   0.3530  0.8490

I'm pretty sure you can ignore the errors, it just tells you that two proposals were rejected since the gradient and some other values that I don't remember (maybe a more descriptive warning would be better? I saw https://github.com/TuringLang/AdvancedHMC.jl/issues/110, but IMO the variable names θ, r, ℓπ, ℓκ are a bit cryptic) were non-finite (e.g., NaN or Inf). I would assume that might be related to the truncated normal distribution in this example, but I guess @xukai92 knows a lot better what could possibly go wrong here.

They can be safely ignored. During the initial phase of the sampling, large step sizes were tried which might lead to numerical errors, which can be ignored as long as there is no consistent errors during sampling.

Hi @devmotion,
I just tried to run the tutorial and it stops at line
μ, σ = rescale!(train; obsdim = 1)
with ERROR: MethodError: no method matching /(::String, ::Int64)
rhe rescale! function cannot cope with strings.
Hence I changed the code as follows
strExclude = :Model
train = Matrix(select(trainset, Not([target, strExclude])))
test = Matrix(select(testset, Not([target, strExclude])))
The model then runs and prints the coefficients.
However, for the tutorial it would be great to pass the names of the covariates to the coefficients.
I'm pretty sure that is possible with turing.jl, but at the moment, I don't know how to do that.
Could you please elaborate the code.
Thanks in advance

I just tried to run the tutorial and it stops at line

I assume you forgot to run the line select!(data, Not(:Model)), which removes the Model column from the data set. The fix should not be needed (and the code ran fine when the tutorial was generated). Can you check if you forgot to ran this line before creating the training and test data sets?

However, for the tutorial it would be great to pass the names of the covariates to the coefficients.

Hmm I'm not sure if I understand you correctly. Would you like to name the coefficients in a way that relates them to the covariates in the model? Or would you mainly like to use the covariates in the trace plots instead of coefficients[1] etc.? Turing does not work with data of type DataFrame but only with arrays (and hopefully at some points whatever samples are generated by the distributions you specify). Therefore we convert the data frame to a matrix in the beginning. However, this conversion removes all headers and hence one has to relate the columns of the matrix and the samples of coefficients to the covariates manually. It is possible, e.g., to change the parameter names of the sampled chain by
newchain = let names=names(data, Not(target))
	set_names(chain, Dict("coefficients[$i]" => name for (i, name) in enumerate(names)))
end

with MCMCChains 3.0.* or
newchain = let names=names(data, Not(target))
	replacenames(chain, ("coefficients[$i]" => name for (i, name) in enumerate(names))...)
end

for MCMCChains 4.0.*.

Hi @devmotion,
thanks for your fast response.
In fact, I somehow missed the select!-line.
Now it runs as expected.
In terms of getting meaningful names to the coefficients I tried the proposed solution for MCMCChains 3.0* and get this error
julia> newchain = let names=names(data, Not(target))
set_names(chain, Dict("coefficients[$i]" => name for (i, name) in enumerate(names)))
end
ERROR: MethodError: no method matching names(::DataFrame, ::InvertedIndex{Symbol})
names(data, Not(target)) seems not to work.
Then I tried this
julia> newchain = let names=names(select(data, Not(target)))
set_names(chain, Dict("coefficients[$i]" => name for (i, name) in enumerate(names)))
end
ERROR: MethodError: no method matching Chains(::Array{Float64,3}, ::Array{Any,1}, ::Dict{Any,Any}; info=NamedTuple(), evidence=missing, sorted=true)
While names(select(data, Not(target))) produces a 10-element array it is not processed by the loop.
How to get it to work?
Better install MCMChains 4.0.*?

@devmotion
Update:
I found the solution
newchain = let names=String.(names(select(data, Not(target))))
set_names(chain, Dict("coefficients[$i]" => name for (i, name) in enumerate(names)))
end
Cheers

Great, that it works! I'm sorry, I didn't test the suggested renaming and just assumed that names(data, Not(target)) would work due to the DataFrames documentation.
I'll close this issue since it seems your issue is resolved.


Add project name to project preferences

Put the project name in the project_preferences (optional attribute?) response object to avoid multiple api lookup calls just to show a project preference name that makes sense.
Currently when showing the email un/subscribe preferences it requests the project preferences then the projects by id and links them up via some lookup mech (incorrectly as it turns out). Instead of running the extra project calls just put the project name in the project prefs so this can be 1 call.

this could be used on the user stats wagon wheel to avoid the extra project cards lookup

The wheel also links to the projects, so we'd need the URL (at least) as well.


Could you please update the documentation?

I'm doing my best but the information provided in the Quick Start section is very confusing, especially with the current Arduino IDE version 2.

This discussion to me was essential to get examples with TFT usage working; perhaps incorporate (a tested and edited version of) it? https://github.com/Xinyuan-LilyGO/LILYGO-T-display-RP2040/issues/4


How to add support for custom markdown syntax?

Markwon version: 4.5.0

I want to add support for Reddit's spoiler syntax, which is >!some spoilers!<
Using this syntax will make the spoiler text has a black background color so that users will not see the text at first. They can click on the spoiler text to remove the background color to see the spoiler.
Currently plugin.addExtension doesn't support delimiters like >! and !<.
Should I create a SpoilerNode to use in builder.setFactory to append some spans?

Ok. I have figured out a way to show a background color on spoiler text. But since > is used to render blockquote, it will simply override the spoiler. How to make spoiler rendering have higher priority than blockquote?
The method I use to hide spoiler text is to override afterSetText in AbstractMarkwonPlugin. I take out the text displayed in TextView and use regex to match !spoiler!<. I cannot match >!spoiler!< because blockquote removes the > at the beginning. After that, I use a BackgroundColorSpan and ClickableSpan on the spannable text to hide the spoiler.
I don't think this is a good implementation so could anyone help with this?

Hello @Docile-Alligator ,
I think that you did it the parsing the right way (searching for >! !< occurrences in text). I've added a sample to confirm this. So, in order to not render blockquote - it is sufficient to replace >! in your markdown with >!. Parser would turn this into >! anyway but blockquote won't be detected

This works! Thank you so much!!!!

Hey @noties , the spoilers tag doesnt work in tables. It shows the spoiler as it is.  What can I do to render spoilers in table ?

Hello @damionx7 ,
yes, tables have limitation and accept only static content. This is due to tables being drawn to canvas directly (you can take a look at TableRowSpan source code). Right now the only workaround is to use markwon-recycler-table artifact which renders markdown in a RecyclerView and table in a TableLayout where each cell is a standalone TextView

How I can do that @noties ? Can you show some code or link to sample ?

Thanks @noties


External deposits always go to the user's entity

External deposits now always go to the user's entity.
Removed externalDepositToEntity()
Should we remove internalTransfer() (rename internalTransferFromEntity() to internalTransfer())?

Thanks for the simplification 👍
Just need word on what to do with internalTransfer()?
🌞


delete disk_size metrics when collection deleted

3341
What problem are we solving?
3341
How are we solving the problem?
3341
Checks

[ok] I have added unit tests if possible.
[ok] I will add related wiki document changes and link to this PR after merging.

Maybe then we will remove all the metrics for the remote collection?

@askeipx It seems to be worth adding the function of deleting metrics for UnmountVolume in the same place as s.DeletedVolumesChan


Replace Guava with java 8 utils

LIKE @RobiNino!
Just one comment - let's add unit tests for the CommonUtils methods.
Thanks!


fix: Update scanoss dependencies

Pull Request
Description
Update the dependencies for the scanoss. Since an error occurs during the operation of the scanoss
Fixes #489
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Test Configuration:

Java: v17
LPVS Release: v1.4.2

Checklist:

[x] My code follows the style guidelines of this project
[x] My code meets the required code coverage for lines (90% and above)
[x] My code meets the required code coverage for branches (80% and above)
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.25%. Comparing base (bc9df2a) to head (19db0f7).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #490   +/-   ##
=========================================
  Coverage     91.25%   91.25%
  Complexity      508      508
=========================================
  Files            48       48
  Lines          1806     1806
  Branches        219      219
=========================================
  Hits           1648     1648
  Misses           91       91
  Partials         67       67

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@tdrozdovsky
1.553 ERROR: THESE PACKAGES DO NOT MATCH THE HASHES FROM THE REQUIREMENTS FILE. If you have updated the package versions, please update the hashes. Otherwise, examine the package contents carefully; someone may have tampered with them.
1.553     crc32c==2.3.post0 from https://files.pythonhosted.org/packages/56/70/9bc4e4b676472af3c052d81a3e8b83fe72bf75f37d4d3a1fa226588521a7/crc32c-2.3.post0-cp39-cp39-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.whl#sha256=a2f6105a430f4f1f393cc660bf8ac3a4860396fa9b5ac8bf0c7ba1de044a3cbc (from -r requirements.txt (line 4)):
1.553         Expected sha256 e0925a7a8fcc216744e86c21f1749c22f950f9bdad512cb5c80ee85017625bd5
1.553              Got        a2f6105a430f4f1f393cc660bf8ac3a4860396fa9b5ac8bf0c7ba1de044a3cbc


Unclear hint: Step 52 equation solver

Describe the Issue
Campers are getting told to return a different string when there are very minor issues in their code like a repeated dash. It would be wise to improve the hints
Affected Page
https://www.freecodecamp.org/learn/scientific-computing-with-python/learn-interfaces-by-building-an-equation-solver/step-52
Your code
output_string = f'\n{equation.type:-^24}'
string =f'\n\n{equation!s:-^24}\n\n'
output_string +=string

Expected behavior
The camper should be told they shouldn't have dashes on the new line and get a better hint.
Screenshots
No response
System

Device: Desktop
OS: Windows 10
Browser: Firefox
Version: 130.0

Additional context
Noticed in this thread: https://forum.freecodecamp.org/t/learn-interfaces-by-building-an-equation-solver-step-52/709626/2

Hello, I was able to reproduce the bug as you have described. Will it be possible for me to work on resolving this issue?

Hello, I was able to reproduce the bug as you have described. Will it be possible for me to work on resolving this issue?

Hi there,
Thanks for your interest in contributing.
This issue has not been triaged yet, and a solution has not been agreed upon. Once the issue is open for contribution, you are welcome to create a pull request to reflect the issue consensus. Please also note that we typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue.

Honestly, if the test says "Your function should return a different string", we should be testing that the returned value is NOT the original string. We shouldn't care what gets returned.

Honestly, if the test says "Your function should return a different string", we should be testing that the returned value is NOT the original string. We shouldn't care what gets returned.

That hint means that the returned string is different from the expected one. I would reword it to make it clear and add an example of the expected output in the description.

I would add something like this at the end of the instructions:
solver(lin_eq) should return:

----Linear Equation-----

       2x +3 = 0

I don't know how to reword the hint honestly.

We should add an example of the expected output:
----Linear Equation-----

       2x +3 = 0

We should add a test to see if the correct amount of new lines are added.
We should add a test to see if the new !s is used .
We should add a test to see if the !s is used on the equations object and not the equations.type
The existing test text should be changed to: Your solver function should output the correct string.

Hi @a2937, I got a chance to explore the unclear hint at step 52 for the equation solver. I tried the Equation Solver Python project as well, and in my opinion, I didn't come across any unclear hints that left me stuck until Step 52. As you have mentioned, the hint they provide with your error is "The solver function should return a different string." However, what makes this error unclear is when I solved this step myself is I returned the same string and eventually was able to do it correctly. This is how I solved Step 52:
Step 52 Corrected:
def solver(equation):
    if not isinstance(equation, Equation):
        raise TypeError("Argument must be an Equation object")
    output_string = f'\n{equation.type:-^24}'

    equation_repr = f'\n\n{str(equation):^24}\n\n'
    output_string += equation_repr

    return output_string

I included two additional lines of code which are centering the string representation of the equation that would contain a width of 24 characters. Afterwards I concatenated the output string (or the header containing the equation type) and the equation representation. In the console, this is how it should appear:
Console:
----Linear Equation-----

       2x +3 = 0

I agree with the previous comments that having a sample output as simple as the one above will be helpful. I believe there are other code learning platforms that provide sample outputs if you're stuck on a problem, as this provides a clear goal of what should be achieved.
Also, the hints given by freeCodeCamp seem very general and don't address the specific problem of what may be incorrect or missing. For instance while trying this, I did not include the newline characters in my equation represntation and the course gave the same hint which did not make sense. Instead, they could have mentioned "newline character missing", but not specifically how to implemented to still create a challenge, but keep the user on the right track. Hopefully this helps. Please let me know if you have any questions


Implemented feature flags for wasmcloud binary

This PR allows you to run the wasmcloud binary with either the wasm3 or wasmtime engine. The default is wasm3, and in order to run with wasmtime you must use the following command:
cargo run --features wasmtime --no-default-features

In order to enforce a default, as @ChrisRx noted in #56, we have to explicitly remove the default feature in order to use the other engine type. I've adjusted the default log verbosity for the wasmtime-provider as in info mode you will get a lot of output as wasmtime JITs the actor module, in debug mode you get an unusable amount of wasm debug output.
This PR also uncomments our integration tests.

@brooksmtownsend are we happy with the default wasm3 being packaged into our deb/rpm packages? If a customer wants to use wasmtime I assume they would have to compile and package themselves?

@billyoung Yeah that's correct, everything will be packaged as wasm3 (which is what we were doing before). We could certainly just adjust the default, or even just the release action to release wasmtime, but I don't think it's necessary to publish both engines as a binary (at least for now when both engines are essentially the same). ADR time for default wasm engine for published binary? @autodidaddict

We seem to go back and forth on this, and while wasm3 seems to be better for the low cold start penalty it does end up slower than wasmtime in execution in long running wasmcloud hosts. Perhaps now is the time to make our released wasmcloud binary use wasmtime instead.

"slower" than wasmtime is a very subjective number. wasm3 per-function-call latency via wapc is a matter of micros.

if we're going to change how we release then we essentially add the engine to the matrix, so that we build for OS+CPU+engine and release wasmcloud-wasm3 and wasmcloud-wasmtime, etc. Or we continue just releasing the default wasm3 which is the better general purpose engine and let those who want wasmtime choose it as a feature-flag when they cargo install.

Apparently by merging the wasmcloud-host version it dismissed your review @autodidaddict


[FEATURE] Add proper types to clients

Description of the problem being solved
Please add types to the client.  Seems like all but a very few of the SlackAPIClient methods are untyped.  This makes the DX very poor.
Alternative solutions
Generate your own types based on the docs, but this can be inaccurate
Requirements
Have accurate types for all endpoints of the APIClient

This is a hard problem for Slack. While argument types are possible (though they take a lot of work to maintain; see e.g. the node PR that put them into place for the @slack/web-api package), response types are difficult due to the context of execution impacting responses. For example, certain APIs payload will differ depending on the Slack paid plan you are on. Other fields are also client-context-specific: some events will have different payload shapes depending on what object within Slack is being operated on.
Ideally, we would re-use the node @slack/web-api library instead of depending on this library, but last I tried to use the node-to-deno compatibility layer with the @slack/web-api npm library, support was not good.


refactor: data scratch file references

Fixes #1759
Test Plan

How do we know the code works?

Code is refactored according to the description in #1759
File reference has been replaced and is acting as expected.

Checklist

[ ] Unit tested

@flank-it

@flank-it