Run SIFT with KdTree slow for large dataset

Hi matlabe,
I am using your find object to recognite with my test dataset (~ 25000 images, contains duplicate images)
Beginning train

After some train

I train with SIFT descriptor and KDTree as nearest neighbor strategy.
Find object train my dataset seem slowly when more objects added. This is very bad because my real dataset will have more than 25000 images.
I searched and find out that because flann kdtree in opencv speed is not good
https://github.com/opencv/opencv/issues/10325
They suggest use nanoflann can improve speed.
https://github.com/jlblancoc/nanoflann
Can you help me? Thanks

If the latency happens always after 6,600,000 words, it could be that your computer is lacking of RAM, so that memory on the SWAP (hard drive) is used. The update still seem happening after 2000 words and not 16384, is this latest find-object built from source (it seems okay on my version when I increase General/vocabularyUpdateMinWords)?
Another approach would be to use a fixed vocabulary. 6 000 000 words is quite a huge vocabulary. You may stop there (or even smaller), set General/vocabularyFixed to true, then export the vocabulary (File->Save vocabulary...) in binary format to avoid large ascii vocabulary file (binary format is only available with this latest commit).

Hi matlabe,
Thanks for your suggest, but now I have another problem.
After add my dataset to find_object and create session, find_object can not detect image from scene when request.
Steps to reproduce:

Run find_object as console to train my dataset with SIFT detector/descriptor (find_object UI not work because too many images)
./find_object --console --objects <myDatasetPath> --session_new my_session.bin

After run ~ 25,000 image above, stop find_object (ctr-c), find_object automatically save my_session.bin

Reload find_object with session
./find_object --console --debug --session my_session.bin

Run tcpRequest from find_object bin directory

Result: find_object will stuck in Do Nearest Neighbor and then throw error

I try to debug a little bit and find out that knnSearch() method in Vocabulary:461  make program stuck.
flannIndex_.knnSearch(descriptors, results, dists, k,
		cv::flann::SearchParams(
		Settings::getNearestNeighbor_search_checks(),
		Settings::getNearestNeighbor_search_eps(),
		Settings::getNearestNeighbor_search_sorted()));

Can you help me? Thanks

I revised how arguments are handled when starting find-object. One problem is that changing parameters in a loaded session was not possible, so we could change to a fixed vocabulary with a session created from a not fixed vocabulary. Update code to latest version and now it is possible to do something like this:
./find_object --console --session_new session.bin --objects ~/objs --General/vocabularyIncremental true
We are creating a new session with default parameters (+ custom incremental vocabulary) using some objects.
./find_object --console --session session.bin --objects ~/objs2 --General/vocabularyFixed true
We reload the session but we fix the vocabulary, and we add more objects.
./find_object --console --session session.bin --scene ~/scene.jpg --json results.json
We reload the full session and feed a scene to detect objects.
Can you try with the latest version?

It is like the flann version included in OpenCV has already been updated over the past years:
https://github.com/opencv/opencv/commits/master/modules/flann/include/opencv2/flann
while the official flann project has too:
https://github.com/mariusmuja/flann/commits/master
Not sure if some fixes of the official library has been already pushed back to OpenCV version. The version of rtabmap is older than both those versions. Not sure it would fix your problems. To better debug the flann error you got in your previous post, build find-object in debug mode (cmake -DCMAKE_BUILD_TYPE=Debug) as well as OpenCV. Then launch find-object in gdb:
gdb --args find_object --console ...

type "bt" when it crashes to get the call stack.
Also, if you are able to reproduce the problem with a small subset of images, you may share this dataset so it will be easier for us to debug and reproduce the problem.

Matlabe, do you have email? I will share you small subset of images if I can
Here's the stacktrace:
#0  0x00007f671c0a1277 in __GI_raise (sig=sig@entry=6)
    at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
#1  0x00007f671c0a2968 in __GI_abort () at abort.c:90
#2  0x00007f671c09a096 in __assert_fail_base (fmt=0x7f671c1f5580 "%s%s%s:%u: %s%sAssertion `%s' failed.\n%n", assertion=assertion@entry=0x7f6722a34af4 "result.full()", file=file@entry=0x7f6722a34e38 "/home/dinhnn/git/opencv/modules/flann/include/opencv2/flann/kdtree_index.h", line=line@entry=463, function=function@entry=0x7f6722a36c00 <cvflann::KDTreeIndex<cvflann::L2<float> >::getNeighbors(cvflann::ResultSet<float>&, float const*, int, float)::__PRETTY_FUNCTION__> "void cvflann::KDTreeIndex<Distance>::getNeighbors(cvflann::ResultSet<typename Distance::ResultType>&, const ElementType*, int, float) [with Distance = cvflann::L2<float>; typename Distance::ResultType"...) at assert.c:92
#3  0x00007f671c09a142 in __GI___assert_fail (assertion=0x7f6722a34af4 "result.full()", file=0x7f6722a34e38 "/home/dinhnn/git/opencv/modules/flann/include/opencv2/flann/kdtree_index.h", line=463, function=0x7f6722a36c00 <cvflann::KDTreeIndex<cvflann::L2<float> >::getNeighbors(cvflann::ResultSet<float>&, float const*, int, float)::__PRETTY_FUNCTION__> "void cvflann::KDTreeIndex<Distance>::getNeighbors(cvflann::ResultSet<typename Distance::ResultType>&, const ElementType*, int, float) [with Distance = cvflann::L2<float>; typename Distance::ResultType"...)
    at assert.c:101
#4  0x00007f6722a11080 in cvflann::KDTreeIndex<cvflann::L2<float> >::getNeighbors(cvflann::ResultSet<float>&, float const*, int, float) (this=0x21aa4e0, result=..., vec=0x221090000, maxCheck=32, epsError=1)
---Type <return> to continue, or q <return> to quit---
    at /home/dinhnn/git/opencv/modules/flann/include/opencv2/flann/kdtree_index.h:463
#5  0x00007f6722a02e07 in cvflann::KDTreeIndex<cvflann::L2<float> >::findNeighbors(cvflann::ResultSet<float>&, float const*, cvflann::SearchParams const&) (this=0x21aa4e0, result=..., vec=0x221090000, searchParams=...)
    at /home/dinhnn/git/opencv/modules/flann/include/opencv2/flann/kdtree_index.h:213
#6  0x00007f67229ede51 in cvflann::NNIndex<cvflann::L2<float> >::knnSearch(cvflann::Matrix<float> const&, cvflann::Matrix<int>&, cvflann::Matrix<float>&, int, cvflann::SearchParams const&) (this=0x21aa4e0, queries=..., indices=..., dists=..., knn=2, params=...)
    at /home/dinhnn/git/opencv/modules/flann/include/opencv2/flann/nn_index.h:86
#7  0x00007f67229ead09 in cvflann::Index<cvflann::L2<float> >::knnSearch(cvflann::Matrix<float> const&, cvflann::Matrix<int>&, cvflann::Matrix<float>&, int, cvflann::SearchParams const&) (this=0x22f9310, queries=..., indices=..., dists=..., knn=2, params=...)
    at /home/dinhnn/git/opencv/modules/flann/include/opencv2/flann/flann_base.hpp:218
#8  0x00007f67229e66b8 in cv::flann::runKnnSearch_<cvflann::L2<float>, cvflann::Index<cvflann::L2<float> > >(void*, cv::Mat const&, cv::Mat&, cv::Mat&, int, cv::flann::SearchParams const&) (index=0x22f9310, query=..., indices=..., dists=..., knn=2, params=...)
---Type <return> to continue, or q <return> to quit---
    at /home/dinhnn/git/opencv/modules/flann/src/miniflann.cpp:495
#9  0x00007f67229e3574 in cv::flann::runKnnSearch<cvflann::L2<float> >(void*, cv::Mat const&, cv::Mat&, cv::Mat&, int, cv::flann::SearchParams const&) (index=0x22f9310, query=..., indices=..., dists=..., knn=2, params=...)
    at /home/dinhnn/git/opencv/modules/flann/src/miniflann.cpp:503
#10 0x00007f67229e0914 in cv::flann::Index::knnSearch(cv::_InputArray const&, cv::_OutputArray const&, cv::_OutputArray const&, int, cv::flann::SearchParams const&) (this=0x1f47f08, _query=..., _indices=..., _dists=..., knn=2, params=...)
    at /home/dinhnn/git/opencv/modules/flann/src/miniflann.cpp:586
#11 0x00007f675cca94ae in find_object::Vocabulary::search(cv::Mat const&, cv::Mat&, cv::Mat&, int) (this=0x1f47f00, descriptorsIn=..., results=..., dists=..., k=2) at /home/dinhnn/git/find-object/src/Vocabulary.cpp:465
#12 0x00007f675cc8cc2c in find_object::FindObject::detect(cv::Mat const&, find_object::DetectionInfo&) const (this=0x2306870, image=..., info=...)
    at /home/dinhnn/git/find-object/src/FindObject.cpp:1502
#13 0x000000000041b56e in FindObjectWorker::detect(cv::Mat const&) (this=0x230e390, image=...)
    at /home/dinhnn/git/find-object/build/app/../../app/TcpServerPool.h:43
#14 0x000000000041af81 in FindObjectWorker::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) (_o=0x230e390, _c=QMetaObject::InvokeMetaMethod, _id=1, _a=0x7f66e6f983d0)
    at /home/dinhnn/git/find-object/build/app/moc_TcpServerPool.cpp:93
#15 0x00007f675b9b8237 in QMetaObject::activate(QObject*, int, int, void**) ()
---Type <return> to continue, or q <return> to quit---
    at /lib64/libQt5Core.so.5
#16 0x00007f675ccdd559 in find_object::TcpServer::detectObject(cv::Mat const&) (this=0x21ed55800, _t1=...)
    at /home/dinhnn/git/find-object/build/src/__/include/find_object/moc_TcpServer.cpp:197
#17 0x00007f675cca5b7f in find_object::TcpServer::readReceivedData() (this=0x21ed55800) at /home/dinhnn/git/find-object/src/TcpServer.cpp:167
#18 0x00007f675ccdd241 in find_object::TcpServer::qt_static_metacall(QObject*, QMetaObject::Call, int, void**) (_o=0x21ed55800, _c=QMetaObject::InvokeMetaMethod, _id=5, _a=0x7f66e6f98670)
    at /home/dinhnn/git/find-object/build/src/__/include/find_object/moc_TcpServer.cpp:111
#19 0x00007f675b9b8237 in QMetaObject::activate(QObject*, int, int, void**) ()
    at /lib64/libQt5Core.so.5
#20 0x00007f675d0b5503 in QAbstractSocketPrivate::emitReadyRead(int) ()
    at /lib64/libQt5Network.so.5
#21 0x00007f675d0b5598 in QAbstractSocketPrivate::canReadNotification() ()
    at /lib64/libQt5Network.so.5
#22 0x00007f675d0c65b1 in QReadNotifier::event(QEvent*) ()
    at /lib64/libQt5Network.so.5
#23 0x00007f675b990205 in doNotify(QObject*, QEvent*) ()
    at /lib64/libQt5Core.so.5
#24 0x00007f675b9902e6 in QCoreApplication::notifyInternal2(QObject*, QEvent*) (---Type <return> to continue, or q <return> to quit---
) at /lib64/libQt5Core.so.5
#25 0x00007f675b9df590 in socketNotifierSourceDispatch(_GSource*, int (*)(void*), void*) () at /lib64/libQt5Core.so.5
#26 0x00007f671857b969 in g_main_context_dispatch () at /lib64/libglib-2.0.so.0
#27 0x00007f671857bcc8 in g_main_context_iterate.isra.22 ()
    at /lib64/libglib-2.0.so.0
#28 0x00007f671857bd7c in g_main_context_iteration ()
    at /lib64/libglib-2.0.so.0
#29 0x00007f675b9deabc in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () at /lib64/libQt5Core.so.5
#30 0x00007f675b98edeb in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () at /lib64/libQt5Core.so.5
#31 0x00007f675b7e26c8 in QThread::exec() () at /lib64/libQt5Core.so.5
#32 0x00007f675b7e6b71 in QThreadPrivate::start(void*) ()
    at /lib64/libQt5Core.so.5
#33 0x00007f671be56e25 in start_thread (arg=0x7f66e6f99700)
    at pthread_create.c:308
#34 0x00007f671c169bad in clone ()
    at ../sysdeps/unix/sysv/linux/x86_64/clone.S:113

Now I have strange result when load session to detect scene.
If I use find_object GUI to add object then detect it from scene, everything is fine.

If I load session, then detect it from terminal, the result is no object detected.
Step to reproduce: (File download below)

Add object to find_object use terminal, save it to session
./find_object --console --config /config/config.ini --object /object/219996.jpg --session_new test.bin --General/vocabularyIncremental true
Use saved session to detect scene
./find_object --console --session test.bin --Homography/minimumInliers 5 --scene /scene/20180720_004.jpg --json results.json

Result: No object detected
** Note: If use tcpRequest from find_object/bin, the result is the same (no object detected)
session_bug_reproduce.zip

I fixed the problem of your session_bug_reproduce.zip example in this commit. The feature detector was not updated, thus the features in the scene were not the same type than those of the object in the saved session.
For the error above, if you have also a subset of images that can reproduce the problem it could be nice. My gmail is the same than my username here if you don't want to share images publicly.
cheers,
Mathieu


Hovering some x-value on the graph should show the y-values at that position

Testing #107623
I know that the goal is to keep this graph lightweight by not using any libs, but a thing I am missing is the ability to look at  historical values on the graph easily with hover.

i now see that if you have the bottom items compact that this does show the hovered text, but that is not intuitive. I don't understand why they would behave differently.


Stats are not showing up correctly (-a option)

I have just realized that my 889563a867152f8d50ff275644b377b1ed9d6c97 commit has broken the statistic report in PuppetResources.hs (when using the -a option)
I will have a look but maybe you'll have an immediate piecing debug lense ;-)

Wicked ! The order matters here ...

The joys of imperative code !


No notifications for replies, mentions or messages

As the title says, but from looking at the settings there appear to be no notification options so maybe this just isn't implemented yet?

for implementing this, I need 3rd party backend that handles APNS and related features. Sorry.


Twitter tab do not close and sattelizer do not authenticate in OS X fullscreen mode

This happens to me only with OS X (Yosemite and later) in fullscreen mode (in Chrome and Safari both). Instead of popup another tab is opened and I'm redirected to main page and that's all. On other systems (Windows, Linux) everything works fine with popup (not a tab) like it should be. If I start authentication on OS X not in fullscreen mode - authentication also goes fine with popup). Also I noticed some isuues:

If I jump fast to the tab I started with, not waiting till second opened tab will be loaded - authentication also works
If I set html5mode to false - authentication works fine. But that is partial solve since I'd like to see urls without hashbangs.

Thank you for any help in solving this problem.

Google OAuth popup window do not closes after redirectUrl was changed


Custom font not work

in this window can work but in this window....

i use linux, I not sure this bug about windows

what spotify version are you on

could you send your comfy config (the export at the bottom of our settings menu) and anything else you have installed to do with spotify / spicetify

this folder i couldn't found any about comfy's config.... i don't know where is the config folder

So the issue is linux has an outdated spotify version, i cant support custom fonts on both 1.2.42 and >1.2.42 because spotify completely changed how their fonts work - please either downgrade the theme installed locally or wait for a spotify update!

one second im going to try one thing that might remedy this, please try the latest commit and if you are using marketplace make sure your cache is cleared (this could take a full spotify restart or you can inspect element and refresh with the cache setting off in the network tab)
if this doesnt work im afraid there isnt much i can do :/

will close due to inactivity soon

in currently version, it's works good,i can use custom font in spotify,thx


Tweak GitHub labels

Description of the new feature / enhancement
No priority, but want to ask if it's possible to tweak a few desciptions for the labels. Some don't fit well when searching this list:

[ ] bits like "Refers to the idea of [...]" and "Issue pertaining to [...]" and "Things that relate with [...]" are not important, what comes after is more interesting
[ ] Product-Awake has "Espresso" in the description.
[ ] nice to have a bit of guidance to choose between different priorities (are priorities only for bugs or for all issues?)
[ ] DCR is hardly ever used, but I'm not sure why.
[ ] add a label for issues that specifically deal with Admin rights
[ ] add label for Hardware-compatibility ? (think VCM or FZ)
[ ] add label Hotkeys that can be combined with many other labels/modules
[ ] Remove FZ-xxx, and use "stand-alone" labels (like App-compat, hotkey etc.) which can be used with any other product
[ ] a few labels deal with PT-Run plugin things, but where lies the diference between them, if you don't know?

Also improve contrast (between color and text) for some labels.
Scenario when this would be used?
Searching for appropriate labels :)

DCR is hardly ever used, but I'm not sure why.

I think this label can get removed.

Maybe add a label for issues that specifically deal with #Admin rights?

For this we use Area-Enterprise.
cc: @crutkas @dedavis6797

Let’s setup a triage team sync. A bunch of these are very targeted for … “reasons”
I want to reduce labels personally as of now personally.

Also enterprise deals with enterprise customer scenarios (big companies, Citrix scenarios, ….)
I deal with prioritize since that dictates what the team should focus on. If a community member wants to pick something up that is a p3 go nuts but we need to focus on p0 and 1 issues.
Espresso was the original name of awake.  Just like launcher was for PT Run, sonar was find my mouse. We will always cross mix terms. I still refer to halo 5 half the time by its code name since that is what i had to call it for years before it was disclosed.

For this we use Area-Enterprise.

Ah yes, found it:

Jay, can you be more targeted at which you want tweaked?

Jay, can you be more targeted at which you want tweaked?

PowerToys GitHub labels.xlsx
Yes I can :)

Jay, can you be more targeted at which you want tweaked?

PowerToys GitHub labels.xlsx
Yes I can :)

Good job. But I think every label should have a description

Is text color/contrast automatically done by the website or can it be set manually?

Is text color/contrast automatically done by the website or can it be set manually?

Can be set manually

I looked at the sheet and wondered, who are labels for. Like, it is ok for not everyone to understand what a label means as long as the core team or a subset knows what it means
can we add another layer here and go what are the ones you really want to focus on as of now, I’m just seeing verbiage tweaks. Most of these were based [stolen] from terminal too
DCR == design change request. This tends to mean a deviation from the core idea/ plan and will increase engineering costs due to a shift. This will tend to be either a me or Jaime usage tag.

Is part of the root you want a shorter description?

I just wanted to polish the labels a bit, make them just a bit easier. I wasn't prepared for the depth of these follow-up questions 😆

Let’s be tactical here. What are the top few offenders and fix those. If there is a pattern, let’s fix the pattern
most of these labels have been borrowed from other MSFT team repos so nice thing is we all now what they mean.

I want to note here that in my opinion the fancyzones- labels are a good idea but unnecessary.

To be honest, after the points up in the original comment, I don't really know what to add.

So I think we should make sure all FZ labels map back to FZ product and rename the editor to product-FZ editor.
past that deal one by one

Sorry, but frankly I don't see any difference. There's still 106 of them. I'd be happy do do this - if only I could.

Labels are way finders. Let’s have targeted move labels XYZ to A label. I have an email to the accessibility team for reducing their list
vagueness of the issue makes this issue hard to ever say “we are done”


releases request

Can you please publish compiled version, so anyone can simply download single js file?

To use it with node.js you can install metasync using package manager npm.
Repository URL: https://www.npmjs.com/package/metasync
Just a single command:
npm install metasync

I believe it could work in browser but it is not tested with browsers and build web systems. Most of build systems also supports npm packages.


Current Version Display for Questions and Response Sets

This PR adds a display of the current version and version currently selected on the show page for both response sets and questions. Please review how it looks and let me know any stylistic changes that should be made.
Make sure you have checked off the following before you issue your Pull Request:

[N/A] Added unit tests for new functionality
[x] Passed all unit tests using rails test with 90%+ coverage
[N/A] Added cucumber tests for any new functionality
[x] Passed all cucumber tests using bundle exec cucumber
[x] Passed overcommit hooks, including running all code through Rubocop
[N/A] If any database changes were made, run rake generate_erd to update the README.md

I think this will work, but in my opinion, there is too much logic going on in the views. A lot of it seems to be there to accommodate how the other_versions method on ResponseSet/Question works. I wrote that method to support the UI as is, but as our last sprint review showed, that UI doesn't show all of the information necessary to support the user. My recommendation would be to refactor that method into something like "all_versions". It should be easy (remove the where.not clause). The views can then have a simple loop that just compares ov.version to @response_set.version to write out current version into the list.
I'd also recommend adding methods to the ResponseSet/Question instances to see if they are the most recent version and to return the most recent version number. We want the views to ask the models for the information that they need. We don't want the views to have to figure that sort of thing out.

I moved the logic to the models - thanks for the suggestion @eedrummer, it was much easier to do it the way you were suggesting / with the suggested methods. I rebased and retested everything. Let me know if you see any other issues.

This looks much more clean. The only issue that I see is somehow the authentications table made it into schema changes. I don't think this PR requires any changes to the DB. Could you just drop the changes to schema.rb from the PR?

@eedrummer schema.rb changes are no longer part of the PR


Update release & adding new services

Version update
New services providers (lcd, rpc, explorers and grpc)

Not sure about repeating the info at fields "versions" and "binaries" - "recommended_version"

Can we also leave the old version in the version array?

Is not compatible 😅
El jue, 20 abr 2023 23:38, JeremyParish69 @.***>
escribió:

Can we also leave the old version in the version array?
—
Reply to this email directly, view it on GitHub
https://github.com/cosmos/chain-registry/pull/1932#issuecomment-1516977540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4T75V6A4SY4OC6GOXUCBLXCGUGFANCNFSM6AAAAAAXFDM2CA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Is not compatible 😅 El jue, 20 abr 2023 23:38, JeremyParish69 @.> escribió:
…
Can we also leave the old version in the version array? — Reply to this email directly, view it on GitHub <#1932 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4T75V6A4SY4OC6GOXUCBLXCGUGFANCNFSM6AAAAAAXFDM2CA . You are receiving this because you authored the thread.Message ID: @.>

It's not meant to be compatible, just building a record of historical versions. Only version listed in the 'compatible_versions' array would actually be compatible.

Got it!
I will modify on Monday, thanks!
El vie, 21 abr 2023 19:37, JeremyParish69 @.***>
escribió:

Is not compatible 😅 El jue, 20 abr 2023 23:38, JeremyParish69 @.
> escribió: … <#m_-7627621107504785991_> Can we also leave the old
version in the version array? — Reply to this email directly, view it on
GitHub <#1932 (comment)
https://github.com/cosmos/chain-registry/pull/1932#issuecomment-1516977540>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4T75V6A4SY4OC6GOXUCBLXCGUGFANCNFSM6AAAAAAXFDM2CA
https://github.com/notifications/unsubscribe-auth/AA4T75V6A4SY4OC6GOXUCBLXCGUGFANCNFSM6AAAAAAXFDM2CA
. You are receiving this because you authored the thread.Message ID: @.>
It's not meant to be compatible, just building a record of historical
versions. Only version listed in the 'compatible_versions' array would
actually be compatible.
—
Reply to this email directly, view it on GitHub
https://github.com/cosmos/chain-registry/pull/1932#issuecomment-1518142013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4T75WRAL6WLV64E2ABKMLXCLAU3ANCNFSM6AAAAAAXFDM2CA
.
You are receiving this because you authored the thread.Message ID:
@.***>


WIP: Long-running form submit spinners

First pass at addressing https://github.com/torchbox/wagtail/issues/305

Still to apply this to more forms

Looks good!

@joshbarr do you have time to review this?

[ ] media embedding modal badly needs this spinner. The round trip to embed.ly seems to take forever.

@tomdyson  sure thing, I can take a look at the media embedding modal. Are there any others you can think of while I'm there?

Hey guys,
I was wondering whether we need to use setTimeout to disable the button? I think we can get the parent form from the current node and trigger submit from the click handler. Something like:
// Trigger form submission if it's a submit button
if (this.form && this.getAttribute('type') === 'submit') {
    $(this.form).trigger('submit');
}

// Then carry on and disable the button + bootup the timeout to clear the spinner...

Also, I've found an issue with trying to embed multiple media assets into the rich text field (in Chrome at least). If the first element is an embed (no other text nodes at the start of the block), trying to insert a second embed generates an Uncaught HierarchyRequestError: Failed to execute 'insertBefore' on 'Node': Only one element on document allowed. in rangy-core.js:29. I'll take a look at at one next.

Cheers for reviewing, @JoshBarr. As far as submitting the form goes, I don't have any issue with that. I was trying to avoid having to refer to forms at all purely to keep it all within the scope of the button, but if it's more reliable that's fine. I was hoping to avoid having to test the button type too, allowing this to be used for ajax forms, synchronous ones, and ones where true "submit" buttons aren't used.

Ready for merge, I believe.

There's a problem with the form submit logic from https://github.com/torchbox/wagtail/pull/1075#issuecomment-89169810 - the form submission won't include the name/value of the submit button, as it would when the submit action is triggered directly from the button. We use this to differentiate between the save/publish/submit actions on the page editor - consequently, with this patch applied, the 'publish' button fails to publish.

Ah yes, that was the main reason I didn't use form.submit() before, now I remember.

Reverted to version that supports passing of button names.


More layer and unit details

Missing layer information in the viewer:

activation at output
regularization (L1/L2, kernel/bias/activity)
dropout at input
batch normalization at input

Missing unit information in the viewer:

bias/intercept

Depends on #22


add vim mode

Adds a vim mode option to the rescript playground.

I did this locally and really liked it. Putting this here as a suggestion. Feel free to close this out if it's no good or isn't something anyone wants.

messed up the rebase. just going to close it


Additional instructions needed for running local tests.

What happened (or feature request):

Request for additional instructions and/or documentation.

What you expected to happen:
When attempting to run make test on ddev master, I was unsuccessful. The documentation here https://github.com/drud/ddev/blob/master/docs/developers/building-contributing.md seems to imply there are no dependencies, and yet I'm getting a considerable amount of warnings stating that certain packages cannot be found. I'll paste my output in a comment.
How to reproduce this:

From a new machine (or at least my machine), git clone ddev and run make test

Related source links or issues:
See comment here https://github.com/drud/ddev/issues/480#issuecomment-333567138

Quick update. It looks like I simply didn't have my $GOPATH setup properly. I'll update and I should be good to Go! (get it?).

Here are the instructions that assisted me. Add the following to .zshrc
mkdir -p ~/go/src/github.com/drud/ddev
export GOPATH=~/go

We should probably update the documentation (linked above in the issue summary) to Go's documentation regarding how to properly set up $GOPATH and then perhaps the snippet above as the quick and dirty solution.

You wouldn't want to add the mkdir to your .zshrc; you only create that directory once. And it needs to be populated with the checkout. So one time only
mkdir -p ~/go/src/github.com/drud/ddev
cd ~/go/src/github.com/drud/ddev
git clone git@github.com:drud/ddev.git .

Then after that, in that directory, with GOPATH set to ~/go, make test

Sorry I was moving too fast! Yes, I wouldn't need to make the directory each time, but rather set the variable.

I think we were moving just too fast in general. If we're going to get you running local tests we need to get your environment set up before anything else, and probably get you a hello-world golang program. Here's the official starter, https://golang.org/doc/code.html - It's a lot better thing to do than ddev. Try the hello-world there after doing the environment setup. Then the tricky thing is to remember you always have to be in the golang workspace/GOPATH.  First programming languages I've ever seen that had arcane setup weirdness like this. You are not the first.

Going to move this to hibernate for a bit. The motivating issue was closed and we can revisit if/when outside collaborators need assistance getting up to speed.


Add ability to extract tor to temp file

ResourceLoader.For.Static for JVM and Android unit tests will need the capability to extract the resource to a temp file in order to load the library. Need to add the option to the Platform DSL.

Actually this is not necessary as can simply specify extraction directory using UUID


Improve pin spec voltage options for power supply pins

The Pin spec has a lot of options for IO in terms of voltage: vil, vih, AbsMax, etc.
It has poor support for voltage pin. Instead of Vmax, maybe we should have VoltageOperatingRange (or something)

VoltageOperatingRange seems useful. Should be clear this is for supplies.

I can add this as part of the pin specifications update. Replace "vmax" with "voltageOperatingRange"


Possible to add left and right button

Possible to add left and right button

Hi @sridharan31 , it is possible to add navigation buttons by applying drag-scroll-nav on your carousel container.

#34 enables the functionality to add custom navigation buttons.
For example
@Component({
  selector: 'sample',
  template:`
  <div drag-scroll #nav>
    Big text goes here...
  </div>
  <button (click)="moveLeft()">Left</button>
  <button (click)="moveRight()">Right</button>
  `
})
class Sample {
  @ViewChild('nav', {read: DragScroll}) ds: DragScroll;

  moveLeft() {
    this.ds.moveLeft();
  }

  moveRight() {
    this.ds.moveRight();
  }
}

Let me know if you have more question.


Claim or consider asaman on PyPI

Hello! Thanks for making this.
I'm about to try it out, but I notice the package/distribution is called amrasaman while the binary/executable is called asaman -- I see asaman is unclaimed on PyPI so perhaps it's worth at least making that redirect to amrasaman (via a package with trivial dependency) if you intend to keep the "true" distribution name as it is here. Users will undoubtedly try pip install asaman at some point.
And thanks again, looks like another useful addition to the packaging ecosystem.

Changed the full project name itself, otherwise it was creating a lot of confusion. Thank you for the issue. I am closing now.
You can now do:
python3 -m pip install asaman

Great!

What do the names mean?

FYI @kushaldas there is still one mention of amrasaman in the README:
https://github.com/kushaldas/asaman/blob/fc9b67b3f18e67198f278c90655a4471e3cb790e/README.md#L120-L123


Use eval instead of JSON.parse

This handles:

trailing commas on the last line of array/objects
no need to use quotes around object keys
makes it possible to use "new Date()" in a query to have it return the current time

Using eval on server side for this type of logic is overkiller.
It leads to exploits, allowing use not only new Date(), but file i/o and SSRF.
Here's an example.


v3.4.15 release

Before releasing v3.4.15:

[x] Changelog entry for https://github.com/mapnik/node-mapnik/pull/581 /cc @ingalls
[ ] Rebuild binaries (determine latest mapnik version to use /cc @artemp)
[ ] Pull in new tag of mapnik-vt (v0.14.3)

@springmeyer CHANGELOG entry has been added - https://github.com/mapnik/node-mapnik/commit/4aeff70638a74e2e24dd2ad8e85a33f836402b8d

done: v3.4.15 is published.


Add common code for bundler.d support

This can be tricky, and perhaps impossible, because the bundler.d code needs to execute before gems are pulled down, but this is living in a gem.

It may be possible to leverage bundler plugins: https://bundler.io/v1.17/guides/bundler_plugins.html

Closed via #42


Add pulse_power support for OPP

OPP firmware version <IP_ADDRESS> or newer support PWM for solenoid pulses. This PR adds support for that.
OPP pulse PWM is a 32bit bitmask where each bit represents 500us. The bitmasks were selected to reduce harmonics and minimize noise. The firmware added the same bitmasks to the hold PWM, but that change does not affect the MPF interface to OPP.

Since this adds new commands at startup, the OPP unit test is failing. I am currently struggling to edit the test to accept the new commands.
FAIL: testOpp (test_OPP.TestOPPFirmware2)
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/mpf/tests/MpfTestCase.py", line 560, in setUp
self._initialise_machine()
File "/usr/local/lib/python3.6/dist-packages/mpf/tests/MpfTestCase.py", line 581, in _initialise_machine
self._wait_for_start(init, 20)
File "/usr/local/lib/python3.6/dist-packages/mpf/tests/MpfTestCase.py", line 593, in _wait_for_start
init.result()
asyncio.base_futures.InvalidStateError: Result is not set.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/mpf/tests/test_OPP.py", line 361, in setUp
super().setUp()
File "/usr/local/lib/python3.6/dist-packages/mpf/tests/MpfTestCase.py", line 577, in setUp
raise self.startup_error
File "/usr/lib/python3.6/asyncio/events.py", line 145, in _run
self._callback(*self._args)
File "/usr/local/lib/python3.6/dist-packages/serial_asyncio/init.py", line 260, in _write_ready
n = self._serial.write(data)
File "/usr/local/lib/python3.6/dist-packages/mpf/tests/test_OPP.py", line 68, in write
raise AssertionError("Unknown command. " + "".join("\x%02x" % b for b in msg))
AssertionError: Unknown command. \x21\x1b\x1f\x0c\xf7\xff\x21\x14\x0c\x00\x0a\x01\x6d\xff\x23\x1b\x1f\x00\xff\xff\x23\x14\x00\x02\x2a\x00\xa8\xff\x20\x1b\x1f\x00\xc5\xff\x20\x14\x00\x02\x17\x00\xc3\xff\x20\x1b\x1f\x01\xc2\xff\x20\x14\x01\x04\x17\x00\xa8\xff\x20\x1b\x1f\x02\xcb\xff\x20\x14\x02\x04\x0a\x00\x2c\xff\x20\x1b\x1f\x03\xcc\xff\x20\x14\x03\x00\x0a\x06\x83\xff

Ah...I was not adding the new command to the COMMAND_LENGTH list in the test. It now passes the unit tests.


Change the keys to rotate

Hi,
What about changes key to rotate blocks?
Up arrow to rotate and space to hard drop?

I personally find using z and x for rotation much more convenient, but I still think, for most of the players, using up-arrow for hard drop doesn't come natural to them.
maybe we can have these controls on top of the current controls we have.

key
Action

left_arrow
move left

right_arrow
move right

z or up-arrow
rotate clockwise

x  or  ctrl
rotate counter clockwise

~up_arrow~  space
hard drop

down_arrow
soft drop

p
pause game

In most tetris version which I play they use space as hard-drop and up-arrow as clockwise.
So for the player it's easier to start play with well-know combination of keys.


Future certs being treated as valid in 2.5.1

Setup: Roll back your system clock to a paste date such that the cert for an API you're using is invalid, because it's not valid until some future date. I used 1960, for consistency.
Devices tested on include various iOS 8 and iOS 7 iPads, both in simulation and on physical hardware.
With AFNetworking 2.5.0 installed, the validation will fail.
With AFNetworking 2.5.1 installed, the validation will succeed, despite future certs technically not being valid.
I looked through the changelog and commit log, and didn't find anything about this.

The problem is not related to date, but to validation of certificates in general in v2.5.1, see issue https://github.com/AFNetworking/AFNetworking/issues/2560

Noted; thank you.


Allow JRuby failures on TravisCI

We're having some global failures on TravisCI unrelated to RuboCop itself. Perhaps we should allow failures for now as a temporary measure.

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Used the same coding conventions as the rest of the project.
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[X] All tests(rake spec) are passing.
[X] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[X] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

@Drenmi I got the failures passing in #4703. Want a PR superseding it to move on?

I think that this PR is an independent one focusing on JRuby <IP_ADDRESS> error in the current matrix of Travis CI. And this looks like a global Travis CI error.

Right, the error I'm talking about is for <IP_ADDRESS>. However, since #4703 is fully removing <IP_ADDRESS> and I got tests passing I thought we might want to skip this PR.

Yep. I think that this PR is temporarily a workaround to avoid this build error.
In essence, JRuby <IP_ADDRESS> build errors in Travis CI should be resolved, or JRuby <IP_ADDRESS> issues should be resolved in #4703. I think that this PR will expect that in any case will be removed from allow_failures in the future.

I think that this PR will expect that in any case will be removed from allow_failures in the future.

Correct. I will update the PR description to explain this. This is a temporary measure so we can clearly see which PRs are actually passing. 🙂

I understood what this PR is for.
I just say the given that both PRs get tests green, but one of them also supersedes the other, it does not make a lot of sense to me to merge and review this one. Anyways, it's no big deal :)

I don't mind closing this if #4703 gets merged first. 🙂 Looks like it's still red though. 🤔  We need to do quite a bit of fixes for 0.50.1, so getting a green CI as soon as possible would be nice.

That's why I was offering a PR in https://github.com/bbatsov/rubocop/pull/4748#issuecomment-329987035! :smile:

@deivid-rodriguez A new PR is welcome!

Now the green build is back, I think it's fine to wait for @rrosenblum to wrap #4703 up! :)


chore: Update Rubocop and pull in recommended plugins

This PR
While working on some specification work, I noticed that the version of Rubocop was far behind. Further, Rubocop recommended two extensions when running locally that we should consider including.

Updates rubocop and pulls in rubocop-rake and rubocop-rspec. I pessimistically locked to the minor version, as Rubocop is generally safe to have a looser versioning constraint in favor of benefiting from new rules sooner.
Corrected Rubocop violations, either through configuration or minor code changes

Notes
I'll do my best to annotate this PR with the reason behind configuration or code changes; I'm open to discussing any fix here.
One thought I had while working on this is if the project would entertain moving to Standardrb instead of using stock Rubocop. I can add an issue for more discussion, but I am curious about any initial thoughts. Not maintaining an RSpec configuration is nice.
Follow-up Tasks

Created #71

How to test
Clean Rubocop CI run

@toddbaert  hmm, it is the same error as https://github.com/open-feature/ruby-sdk/pull/64#issuecomment-1717837383 when trying to report, even with https://github.com/open-feature/ruby-sdk/pull/67
debug - 2023-09-14 00:53:24,761 -- Starting create commit process --- {"commit_sha": "c24ea2244b3606b23931171c81d93bbe9137744c", "parent_sha": null, "pr": "72", "branch": "mveldink/update-rubocop", "slug": "open-feature/ruby-sdk", "token": null, "service": "github", "enterprise_url": null}
Error: Codecov token not found. Please provide Codecov token with -t flag.
Error: Codecov: Failed to properly create commit: The process '/home/runner/work/_actions/codecov/codecov-action/845c445181131d954f0198d3d0f26242acc0376e/dist/codecov' failed with exit code 1

@technicalpickles Great, I added #74 and will take a stab at switching over to see how much work/reformatting would be involved.

@technicalpickles the codecov thing ended up being a big can of worms: https://github.com/codecov/codecov-action/issues/1087.
I'm reverting to an older version for now: https://github.com/open-feature/ruby-sdk/pull/75

Closing this since #76 was accepted


hugo: 0.76.5 -> 0.77.0

Motivation for this change
https://github.com/gohugoio/hugo/releases/tag/v0.77.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.


Can you safely copy the file while it’s being written to?

Somewhat of a follow-up to this question:
https://github.com/cwida/duckdb/issues/1330
if you have one process continually writing to a DB, can you copy that file and be sure that it won’t be in a corrupt or somewhat indeterminate state?
(obviously, it would be hard to predict which writes made it in or not. I’m copying the dB file while it’s being written might lead it to be in a corrupt state. Basically not sure if writes are atomic, etc)

The file on disk should always be in a consistent state. The latest data that is being written may still live in transaction buffers or the write-ahead log though, so you might lose that.

The file on disk should always be in a consistent state. The latest data that is being written may still live in transaction buffers or the write-ahead log though, so you might lose that.


Uses requests

The library uses requests, in what world is it asynchronous?

It is just practice, I'd use aiohttp, but keep in mind this is open-source and can be edited by users


BUG: About button in nav bar is not working properly.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When we clicking on the About button on nav bar it doesn't work.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@himanshi-0908 There is no about us page and this issue has already been raised


Move async API for 3.7 compatibility

Fixes #6.
The big block of _async functions feel clunky, but I'm not aware of a more concise representation.

Looking good, thanks! Since I'm no longer using this module myself, would you like to get the commit bit?

The rights to manage this repository yourself.

Absolutely.


Plugins not loaded

I'm running rtm 1.5.1 retrieved from a pre-built binary. I can't get plugins to load. I have been trying both rtm-cli-plugin-example and rtm-plugin-export. I tried manually setting the plugin directory path in the configuration file to no avail. Is this an issue with pre-built binaries or what?

I just tested the export plugin with a pre-built binary, so that shouldn't be an issue.  What OS are you using?
This is the configuration file I have:
{
  "dateformat": "ddd mmm d",
  "hideDue": 14,
  "aliases": [
    {
      "name": "today",
      "description": "Display tasks with a priority, due or completed today, or overdue",
      "command": "lsp",
      "args": "(not priority:none and status:incomplete) or completed:today or (dueBefore:tomorrow and status:incomplete)"
    }
  ],
  "plugins": [
    "~/.rtm.plugins/"
  ]
}

Then, the ~/.rtm.plugins/ directory contains:
.
└── plugin-export
    ├── LICENSE
    ├── README.md
    ├── export.js
    ├── index.js
    ├── node_modules
    │   ├── rtm-api
    │   └── rtm-cli
    └── package.json

I'm having a similar problem on Mac OS.  My plugin-export directory doesn't contain the 'node_modules' directory.  Is that something that's created at runtime?  Or did I miss a node config step?

In regards to plugin-export, it requires rtm-cli and rtm-api as peer dependencies. Therefore, I think in order for it to work as it is, it needs to be placed into the rtm-cli directory. Otherwise, those require statements will cause the package to fail and the plugin will not be loaded. I could be wrong but that's the only way I could get it to work.

.
└── plugin-export
    ├── LICENSE
    ├── README.md
    ├── export.js
    ├── index.js
    ├── node_modules
    │   ├── rtm-api
    │   └── rtm-cli
    └── package.json

This doesn't make sense with how that package is set up because the packages are listed as peer dependencies rather than pure dependencies, so even if you npm install, you won't get those packages in this plugin directory.

Ubuntu 20.10 :
I fixed this by running sudo npm install rtm-cliin the export sub directory of ~.rtm.plugins/
I suspect this may have been a sledgehammer approach !

Ubuntu 20.10 :
I fixed this by running sudo npm install rtm-cliin the export sub directory of ~.rtm.plugins/
I suspect this may have been a sledgehammer approach !


setup the comments model views for the comments endpoint of the plugin

As a user, I want to be able to comment on the songs in the music room so that i can share my observations about them

I created the Views for the comments section of playlist endpoint of the plugin.
Live links to test are below
http://music.zuri.chat/comments
http://music.zuri.chat/comments/1


The ability to detect os family.

I am very impressed with goss and want to move my serverspec tests to goss. One thing that I can't find how to do in goss is detecting os family. For example:
if os[:family] == 'redhat'
    packages = ["git", "unzip", "epel-release", "nginx"]
elsif ['debian', 'ubuntu'].include?(os[:family])
    packages = ["git", "unzip", "uwsgi", "nginx"]
end

Is this feature already in goss or can it be added ?

So, I was personally thinking that this feature (the ability to probe system for "facts") should be out of scope for goss. That said, I do think goss should have some sort of support for variables. see: https://github.com/aelsabbahy/goss/issues/136#issuecomment-242773611
Do you think the approach posted in that comment would sufficiently cover your usecase?

yeah that looks okay to me.

Any updates on this ?

No, unfortunately I haven't had time to put into goss recently.

Released in v0.3.0


fix(explore): Tag component overlap

SUMMARY
Fixes #14607
Reintroduces the max-width that got lost during refactoring. For reference PR #12774
BEFORE

AFTER

TEST PLAN

Open a deck.gl Screen Grid chart
Make sure the "LONGITUDE AND LATITUDE" Tag component does not overlap with the "WEIGHT"

ADDITIONAL INFORMATION

[x] Has associated issue: #14607
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@junlincc @rusackas


Icons

@baimingze
We will need an icon for:
- Transcriptomics
- Multiomics (Probably here instead of using only one color, you can use multicolor someting like the
almetric byu with our colors.
- I will explore the idea of adding meta-genomics then have an icon ready.

@baimingze are these icos already created.

Transcriptomics and Multiomics have been done, except the meta-genomics

Transcriptomics and Multiomics have been done, except the meta-genomics

+1


How do we set starting playback at a specific timecode in vimeo

Be sure to search for your issue before opening a new one.
Current Behavior
<ReactPlayer
className="react-player"
width="100%"
height="100%"
onPlay={() => setPlaying(true)}
onPause={() => setPlaying(false)}
playing={playing}
url={${link}#t=1000s}
controls={true}
onProgress={handleTime}
onEnded={handleVideoEnd}
/>
When I inspected the page, it show like this my url
https://player.vimeo.com/video/53****679?title=0&byline=0&portrait=0&playsinline=0&autopause=0&app_id=122****3
Expected Behavior
Actually what I expect it like this
https://player.vimeo.com/video/53****679**#=1000s**?title=0&byline=0&portrait=0&playsinline=0&autopause=0&app_id=122****3
There should be a way to add #t=1000s end of the video id.
Steps to Reproduce

This is the vimeo resource I found to do this. Link

onDuration={() => playerState.seekTo(<seconds>, 'seconds')}


Delete redundant commands & Clean up the codebase

closes #129 & closes #138

@simonteozw Hi Simon
So basically in this PR, I did the following things:

For Contact: Removed List & Clear commands
For Lesson: Removed List & View commands
For Module: Removed List & View commands
For Task: Removed View command
And I refactored all the common messages between commands to Messages.java + Standardised their names

Thank you for reviewing it!


feat(admin): set charts based on websocket data

Set charts with teams and laps based on the incoming websocket data.
Added fastapi thingy to serve static files
moved script tags to the js file

Looks Good!


[Bug]There are two instances of milkdown when changing defaultValue dynamically

Describe the bug
There are two instances of milkdown when changing defaultValue dynamically
MilkdownEditor.tsx
import type { FC } from "react";
import { Editor, defaultValueCtx, rootCtx } from "@milkdown/core";
import { ReactEditor, useEditor } from "@milkdown/react";
import { listener, listenerCtx } from "@milkdown/plugin-listener";
import { milkdownPlugins } from "@dolan-x/milkdown-plugins";

import "prism-themes/themes/prism-nord.css";

interface MilkdownEditorProps {
  disabled?: boolean
  value?: string
  onChange?: (s: string) => void
}
const MilkdownEditor: FC<MilkdownEditorProps> = ({ value, onChange, disabled }) => {
  const { editor } = useEditor((root) => {
    const e = Editor.make()
      .config((ctx) => {
        ctx.set(rootCtx, root);
        value && ctx.set(defaultValueCtx, value);
        if (onChange && !disabled) {
          ctx.get(listenerCtx).markdownUpdated((_ctx, markdown, _prevMarkdown) => {
            onChange(markdown);
          });
        }
      })
      .use(listener)
      .use(milkdownPlugins);

    return e;
  }
  , [value],
  );

  return <ReactEditor editor={editor} />;
};

export default MilkdownEditor;

Page.tsx
const EditPost: FC = () => {
  const [content, setContent] = useState("");

  function onFetch () {
    setContent('abcd')
  }
  useEffect(onFetch, []);

  const Milkdown = <MilkdownEditor value={content} onChange={setContent} />;

  return (
    <div>
      {Milkdown}
    </div>
  );
};

export default EditPost;

Expected behavior
One instance only
Screenshots

Milkdown version
6.2.0
Desktop (please complete the following information):

OS: Windows 11
Browser: Edge
Version: 103.0.1264.44

However, once Vite is hot-updated or the content of the editor is manually modified, redundant instances will disappear.

Why're you recreating a new editor everytime the content is changed?

Not quite sure what I should do, could you please give some advice?

They're not same, in my code theonChange won't cause the md state change. But in your code, the onChange will cause the content change, which means everytime user input anything, the onChange will update the content and a new editor will be created.
You should treat milkdown editor as an uncontrolled component

Thanks!

But...I still don't quite know what I should do...

Sorry, it works now

Sorry, it works now

hi，how did you solve this problem before？I want to wrap milkdown-editor as a form-item...

hi，how did you solve this problem before？I want to wrap milkdown-editor as a form-item...

The issue is a little bit stale and I forgot what I have done :(
but you can checkout https://github.com/dolan-x/dolan-admin/blob/main/src/components

ok，thanks a lot！

hi，how did you solve this problem before？I want to wrap milkdown-editor as a form-item...

The issue is a little bit stale and I forgot what I have done :(
but you can checkout https://github.com/dolan-x/dolan-admin/blob/main/src/components/MilkdownEditor.tsx. It uses an older version of milkdown so it might not work in the new version.
I'm using milkdown in vue now so I no longer get this issue. In the link I may not use React properly. Hope this will help.

ok，thanks a lot！


Cannot update commands while using the libaray with java

I can't see the function updateCommands() in the CommandRegistryBuilder class. It just doesn't exist

You need to .build() the CommandRegistry before updating your commands, as shown in the example in the README.


Added Icons / Make massFlowRate_dp_WallFriction easier for tool to differentiate

Can I assume you tested all this Reynold's number stuff to make sure you get a continuous mass flow rate as a function of the Re number?  Also, any impact on differentiability?

The change is just a reordering of the same algorithm, so the output is as before. It is to aid differentiability in our compiler, though I do not have other compilers to test it against.


/x does not render text as bold if followed by a hard line-break

It's entirely possible this bug is upstream.
Repro
The markup
- **Breaking: Changed how clients are instantiated with an API (#0)**\
  The API object must now be passed in as an api property on an object.

How GitHub renders it

Breaking: Changed how clients are instantiated with an API (#0)
The API object must now be passed in as an api property on an object.

And on deno.land/x

Interestingly it seems to only affect lines that end with a non-alphanumeric character, but I haven't tested further.

@hashrock does remark handle this case better?

Yes it does

@hashrock, I think this can be closed. When I go to https://deno.land/x/jamf_school@0.2.0/README.tpl.md?source=#version-020, I see the problematic text now rendered correctly.

Agreed. It’s been a while since I had this problem, to be honest I forgot this issue was still open in the first place!


exercise extension - bad path

The exercise extension contains an incorrect path:
load_css('/nbextensions/testing/exercise/main.css');

should be:
load_css('/nbextensions/exercise/main.css');

I can't work out how to use this extension? Is it still in testing/incomplete? Looks like it could be really useful...

So  - progress. The exercise extension requires the rubberband extension, and seems to work partially:

the main.css file in the extension is not picked up - it seems as if the extension handler is looking to load from localhost/nbextensions/usability/rubberband/main.css and then 404ing
I can only set exercises where the first cell in the notebook is the first one looped by the rubberband. If I try to loop cells lower down the notebook, nothing happens when I click the exercise Hide solution button.

I have made a fix for the invalid path. Thanks for reporting.
The extension works for me, but the logic is a little brittle. It should probably have stayed in testing.
I intend to work on it in September, but feel free to play around with it.
Basically, the extension works by selecting a number of cells with the rubber band and then hiding all but the first cell. Clicking on the plus should show all cells again.

I have a demo in am Exercises notebook posted here.
One thing I found was that things seem a little brittle when trying to create  an exercise where the first cel is not the first cell in the notebook?

This was (probably?) fixed by the updated exercises in #383

I like this extension but I have a problem. when I use twice (or more) Exercise 2 button, pressing the first button created, it hides all the buttons below. Is there any solution for this or am I doing something wrong?

If I understand well, this may be a consequence of what follows.
Currently, the extension hides the series of consecutive cells that are marked (in their metadata) with solution2="hidden". If one creates two consecutive exercices, then they are "merged" by this mechanism - there is currently no test for detecting if a cell is the beginning of a new exercise. This should be implemented in some future. For now, you may simply separate the exercises by a non-exercise cell. Typically, I use an empty raw-cell.

This was exactly the problem. Thank you jfbercher.


[CLOSED] Update administrative appeals tribunal decisions sub-categories values

 Issue by robmckinnon
Thursday 03 March 2016 at 15:33 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/672

Update administrative appeals tribunal decisions sub-categories values.
Fix numeric sorting errors.
Add a few new sub-categories.

robmckinnon included the following code: https://github.com/alphagov/specialist-publisher/pull/672/commits

 Comment by h-lame
Thursday 17 March 2016 at 16:06 GMT

@robmckinnon - this seems clear enough - but it needs a rebase before we can merge.


Added Some content related to facts about pizza

Description
I have added some content related to facts about pizza.

What type of PR is this? (check all applicable)

[ ☑] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

#36 Add pizza fact or trivia
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@CBID2 please review this PR😀.

Hi @kumar007ambi,
We've merged a PR before yours. So please resolve the conflicts and fix the suggestion from @CBID2 before we can merge your PR. :)
Also be aware that we're now using unordered list with dash (-) instead of ordered list (number).
Thank you :)

@CBID2 and @adiati98 I have edited my content according to suggestion.

@CBID2 I have edited my content according to suggestion.


[new-tag] certificate-management

Originally posted by @BiNZGi in https://github.com/endoflife-date/endoflife.date/pull/4256#discussion_r1423823961

certificate-management is a new tag. We already have configuration-management. WDYT of using this instead?

Originally posted by @ajayk in https://github.com/endoflife-date/endoflife.date/pull/4256#discussion_r1424322819

cert-manager is specifically handling certificates , probably I can remove the tag all together for now if it is too much of hassle to create a new one

Should we add a new tag certificate-management?

hashicorp vault and cert-manager would probably the ones on the cert management to start with

If hashicorp vault and cert-manager are the only product having this tag I don't think it would add any value.
FYI I am currently considering removing all existing tags used on only one or two products that are not company names nor categories (those are the exception to the rule when we discussed adding tags in #2756).

FYI I am currently considering removing all existing tags used on only one or two products that are not company names nor categories (those are the exception to the rule when we discussed adding tags in https://github.com/endoflife-date/endoflife.date/pull/2756).

Would it possible to put this rule into CI ? That would help contributors auto-moderate the PRs : what do yo uthink abou that ?

Would it possible to put this rule into CI ? That would help contributors auto-moderate the PRs : what do yo uthink abou that ?

Unfortunately not, as we have an exception for company names (categories is OK as they are well-known).


Update Composer dependencies (2021-02-05-00-10)

Visual regression test passed!
View the visual regression test report


[Release-1.28] - Support for containerd 1.5+ config_path: On-the-fly container registry configuration

Backport fix for Support for containerd 1.5+ config_path: On-the-fly container registry configuration

#5568

Validated on release-1.28 branch with version v1.28.6+k3s2
Environment Details

Infrastructure

[x] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.2 LTS"

$ uname -m
x86_64

Cluster Configuration:

HA: 3 server/ 1 agent

Config.yaml:

system-default-registry: test-private-registry.com
debug: true
token: xxxx
cluster-init: true
write-kubeconfig-mode: "0644"
node-external-ip: <IP_ADDRESS>
node-label:
- k3s-upgrade=server

Registries.yaml

$ cat /etc/rancher/rke2/registries.yaml
mirrors:
  test-private-registry.com:
    endpoint:
      - https://test-private-registry.com
  docker.io:
    endpoint:
      - https://test-private-registry.com
  k8s.gcr.io:
    endpoint:
      - https://test-private-registry.com
configs:
  test-private-registry.com:
    auth:
      username: user
      password: password
    tls:
      ca_file: /home/ubuntu/ca.pem

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Copy registries.yaml to /etc/rancher/k3s/registries.yaml
copy ca.pem to the user home directory (as per path provided in the registries.yaml file).
2. Install k3s
curl -sfL https://get.k3s.io | sudo INSTALL_K3S_VERSION='v1.28.6+k3s2' sh -s - server

Verify content of config.toml file for config_path:

$ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/config.toml

Verify content of config_path directory and the corresponding host.toml files:

 $ sudo ls /var/lib/rancher/k3s/agent/etc/containerd/certs.d
 $ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/certs.d/test-private-registry.com/hosts.toml
 $ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/certs.d/docker.io/hosts.toml
 $ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/certs.d/k8s.gcr.io/hosts.toml

Validation Results:

k3s version used for validation:

$ k3s -v
k3s version v1.28.6+k3s2 (c9f49a3b)
go version go1.20.13
'''
Cluster Status:
'''
$ kubectl get nodes
NAME               STATUS   ROLES                       AGE     VERSION
ip-172-31-24-154   Ready    control-plane,etcd,master   4m6s    v1.28.6+k3s2
ip-172-31-24-217   Ready    control-plane,etcd,master   3m21s   v1.28.6+k3s2
ip-172-31-24-94    Ready    <none>                      2m41s   v1.28.6+k3s2
ip-172-31-29-236   Ready    control-plane,etcd,master   4m55s   v1.28.6+k3s2

$ kubectl get pods -A
NAMESPACE        NAME                                      READY   STATUS      RESTARTS   AGE
auto-clusterip   test-clusterip-576485fd68-tfps5           1/1     Running     0          55s
auto-clusterip   test-clusterip-576485fd68-vkrvl           1/1     Running     0          55s
auto-daemonset   test-daemonset-2dqph                      1/1     Running     0          55s
auto-daemonset   test-daemonset-bnbf4                      1/1     Running     0          55s
auto-daemonset   test-daemonset-rc4ch                      1/1     Running     0          55s
auto-daemonset   test-daemonset-vm2gc                      1/1     Running     0          55s
auto-dns         dnsutils                                  1/1     Running     0          55s
auto-ingress     test-ingress-6xfhf                        1/1     Running     0          55s
auto-ingress     test-ingress-c5hxv                        1/1     Running     0          55s
auto-nodeport    test-nodeport-7445df8f5b-2ft4c            1/1     Running     0          55s
auto-nodeport    test-nodeport-7445df8f5b-7sld5            1/1     Running     0          55s
default          clusterip-pod-demo                        1/1     Running     0          58s
default          clusterip-pod-demo-2                      1/1     Running     0          58s
default          clusterip-pod-demo-3                      1/1     Running     0          58s
kube-system      coredns-c489f67f7-4sjhg                   1/1     Running     0          4m17s
kube-system      helm-install-traefik-crd-4gz6d            0/1     Completed   0          4m17s
kube-system      helm-install-traefik-n8pbx                0/1     Completed   2          4m17s
kube-system      local-path-provisioner-7cc69c5cb7-nv4hk   1/1     Running     0          4m17s
kube-system      metrics-server-df8f6756d-dwzwq            1/1     Running     0          4m17s
kube-system      svclb-traefik-ed9c4027-cj86c              2/2     Running     0          3m22s
kube-system      svclb-traefik-ed9c4027-db94g              2/2     Running     0          2m42s
kube-system      svclb-traefik-ed9c4027-hkmdx              2/2     Running     0          3m46s
kube-system      svclb-traefik-ed9c4027-pd7q9              2/2     Running     0          3m46s
kube-system      traefik-7fd84897bb-8kl5z                  1/1     Running     0          3m46s
pvt-reg-test     pvt-reg-test-d8b6bcc4d-8l7wl              1/1     Running     0          42s
pvt-reg-test     pvt-reg-test-d8b6bcc4d-p5hsl              1/1     Running     0          42s

config.toml content:
$ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/config.toml
# File generated by k3s. DO NOT EDIT. Use config.toml.tmpl instead.
version = 2

.
.
.

[plugins."io.containerd.grpc.v1.cri".registry]
  config_path = "/var/lib/rancher/k3s/agent/etc/containerd/certs.d"

[plugins."io.containerd.grpc.v1.cri".registry.configs."test-private-registry.com".auth]
  username = "user"
  password = "password"

certs.d directory content:
 $ sudo ls /var/lib/rancher/k3s/agent/etc/containerd/certs.d
docker.io
test-private-registry.com
k8s.gcr.io

hosts.toml file contents:
 $ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/certs.d/test-private-registry.com/hosts.toml
# File generated by k3s. DO NOT EDIT.

[host."https://test-private-registry.com/v2"]
  capabilities = ["pull", "resolve"]
  ca = ["/home/ubuntu/ca.pem"]

$ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/certs.d/docker.io/hosts.toml
# File generated by k3s. DO NOT EDIT.
server = "https://registry-1.docker.io/v2"

[host."https://test-private-registry.com/v2"]
  capabilities = ["pull", "resolve"]
  ca = ["/home/ubuntu/ca.pem"]

 $ sudo cat /var/lib/rancher/k3s/agent/etc/containerd/certs.d/k8s.gcr.io/hosts.toml
# File generated by k3s. DO NOT EDIT.
server = "https://k8s.gcr.io/v2"

[host."https://test-private-registry.com/v2"]
  capabilities = ["pull", "resolve"]
  ca = ["/home/ubuntu/ca.pem"]


a problem

there is single Parameters in line 141 in deepcpg/data/annotations.py. So if we use deepcpg using pip, there seems to be error when run index.sh by selecting File -> Download as -> Bash (.sh).
please check this problem.

Thanks for pointing out! I removed the line.


Rendering Issue

Issue description: Client crash with a rendering issue
What happens: Client crashes
What you expected to happen: Client not to crash
Steps to reproduce:

I attempted to convert a copper block to ingots in Grid's crafting grid.
Crashed

...
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>5
Refined Storage: 1.2.14

Does this issue occur on a server? [yes/no] Yes
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]
http://pastebin.com/bRSXZBjC

It seems like minetweaker is chocking on some tooltip. You might want report it to them. Or check your minetweaker scripts or mod pack issue tracker.


Fix TraceLogSink not releasing StringBuilder back to StringBuilderCache

What does the pull request do?
One of the Format methods in TraceLogSink was acquiring a StringBuilder from StringBuilderCache, but not releasing the StringBuilder back to the cache at the end of the method. Now it will properly call StringBuilderCache.GetStringAndRelease, matching the behavior of the other Format method in TraceLogSink.
What is the current behavior?
The StringBuilder is not released back to StringBuilderCache.
What is the updated/expected behavior with this PR?
The StringBuilder is released back to StringBuilderCache.
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0028935-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0028976-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Does not bring in highlight.js

I followed your install instructions for npm, but this isn't bringing in the highlight.js.  I think this used to be its own plugin, but I cannot find it.
Can I bring the two .js files into my project and follow the dependency instructions from there?

Sorry for the really late response. As mentioned in the slides, the highlight.js library used should not be the Reveal plugin, but rather an actual copy of just the highlight.js library. Once it's loaded this plugin should be able to work.

I did find that in the slides. I was following them in order and stopped
when I got stuck, before I saw that info.
On Sun, Apr 30, 2017, 10:35 PM Benjamin Tan<EMAIL_ADDRESS>wrote:

Sorry for the really late response. As mentioned in the slides
https://demoneaux.github.io/reveal-code-focus/#/2/1, the highlight.js
library used should not be the Reveal plugin, but rather an actual copy of
just the highlight.js library. Once it's loaded this plugin should be able
to work.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/demoneaux/reveal-code-focus/issues/11#issuecomment-298276461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEceGJy8dM37-6NmedXMsnWkawYo8t3Dks5r1UTlgaJpZM4Lg9mk
.


Nested binding with ambiguities in syntactic categories

K is having problems with nested binding when there is more than one possible match for a particular syntactic category. In the following example, our binder is sub ID with PROCESS in { BODY }. Nil is the only process, and two processes can be written side-by-side as Process | Process.
If processes (Proc) and the body (Body) are members of multiple syntactic categories, given below as MoreProc and MoreBody, the substitution rule will return an assertion error on the program below.
sub x with Nil in {
        sub y with Nil in { x | y }
  }

require "substitution.k"

module BIND-SYNTAX
imports SUBSTITUTION
imports DOMAINS

syntax Proc ::=
	  "Nil"
	| Proc "|" Proc [left]

syntax Body ::=
	  Proc
	| "sub" Id "with" Proc "in" "{" Body "}" [binder(1 -> 3)]
	| Id
	| Body "|" Body [left]

syntax MoreProc ::=
	  Proc

syntax MoreBody ::=
	  MoreProc
	| "sub" Id "with" MoreProc "in" "{" MoreBody "}" [binder(1 -> 3)]
	| MoreBody "|" MoreBody [left]

syntax KVariable ::= Id

endmodule

module BIND
imports BIND-SYNTAX

rule sub X:Id with P:Proc in { B:Body } => B[ P / X ]

endmodule

The assertion error can be fixed minimally by removing the substitution, i.e. replacing B[ P / X ] in the rule by B, or by deleting "sub" Id "with" MoreProc "in" "{" MoreBody "}" [binder(1 -> 3)] from MoreBody.
This is on the latest version of K5.

Please refer to the attached PR which fixes your issue. Note that K no longer supports specifying the targets being bound and bound to in the binder attribute; ie, binder(1 -> 3) is equivalent to just binder which binds the first child within the last child, the same behavior I assume you wanted here. If you have other binders in your definition which do not bind the first child within the last child, I would suggest doing the following replacement:
syntax Foo ::= someTerm(A, B) [binder(2 -> 1)]
To
syntax Foo ::= someTerm(A, B) syntax Foo ::= someAlias(B, A) [binder] rule someTerm(A, B) => someAlias(B, A) [alias]

Thank you for the PR!

On Aug 23, 2018, at 6:30 PM, dwightguth<EMAIL_ADDRESS>wrote:
Please refer to the attached PR which fixes your issue. Note that K no longer supports specifying the targets being bound and bound to in the binder attribute; ie, binder(1 -> 3) is equivalent to just binder which binds the first child within the last child, the same behavior I assume you wanted here. If you have other binders in your definition which do not bind the first child within the last child, I would suggest doing the following replacement:
syntax Foo ::= someTerm(A, B) [binder(2 -> 1)]
To
syntax Foo ::= someTerm(A, B) syntax Foo ::= someAlias(B, A) [binder] rule someTerm(A, B) => someAlias(B, A) [alias]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/kframework/k/issues/139#issuecomment-415614084, or mute the thread https://github.com/notifications/unsubscribe-auth/AkKOytUVeh-oY05DhhUVE5LW-A1VJycfks5uT0kpgaJpZM4WKAsg.

I just implemented an example with the rule tagged [alias]. Using your syntax, if I write:
someAlias(x,
	someTerm(x, A)
)

then K does not know to shadow x using someTerm( ) and will bind/substitute x, rendering incorrect syntax.
For the full code/example, see:
Code
require "substitution.k"

module BIND-SYNTAX
imports SUBSTITUTION
imports DOMAINS

syntax Body ::=
	  Proc
	// Alias
	| "sub" Proc "into" Id "in" "{" Body "}"
	// Binder
	| "sub" Id "with" Proc "in" "{" Body "}" [binder]
	| Id
	| Body "|" Body [left]

syntax Proc ::=
	  "Nil"
	| Proc "|" Proc [left]

syntax MoreProc ::=
	  Proc

syntax MoreBody ::=
	  MoreProc
		// Alias
		| "sub" MoreProc "into" Id "in" "{" MoreBody "}"
		// Binder
		| "sub" Id "with" MoreProc "in" "{" MoreBody "}" [binder]
		| MoreBody "|" MoreBody [left]

syntax KVariable ::= Id

endmodule

module BIND
imports BIND-SYNTAX

rule sub P:Proc into X:Id in { B:Body } => sub X with P in { B } [alias]

rule sub X:Id with P:Proc in { B:Body } => B[ P / X ]

endmodule

Example
sub y with Nil in {
        sub Nil | Nil into y in { y }
  }

The example should return Nil | Nil, since the inner binder shadows y. Instead, the outer binder binds y and K returns sub Nil | Nil into Nil in { Nil }.

I cannot reproduce the behavior you report: I tested with that example and it works exactly as expected and yields the correct result.

Must be something on my end then, this is very good. Thank you!

You might try deleting the bind-kompiled directory and see if that helps, sometimes there are issues with caching  when you change only the attributes of a sentence.

That didn't do it, but it's been a couple weeks since I updated K. Maybe that's the problem.
I actually have a question for you re the aliases. Right now I have a construct that comes in this form:
f(A1, B1 ; A2, B2 ; ...  ; A_n, B_n){ C }

where we bind free vars in the A_i, receive input from the B_i and substitute the B_i into the A_i contained in C. I would ideally like to write an alias that rewrites
f(A1, B1 ; A2, B2 ; ...  ; A_n, B_n){ C } => f( {A1 ; A2 ; ... ; A_n} , {B1 ; B2 ; ... ; B_n} ){ C }

in one go (I have some intermediary steps but they are giving me a bit of trouble). Is it (a) possible to compose aliases, and (b) can you think of some way to write a rule of that form—like some for all feature?

You can essentially view an alias as a rewrite rule without a side condition that applies statically before rewriting begins and applies statically in reverse after rewriting ends.
I haven't tested it, but something like this should work:
rule f(A, B ; Rest) { C }

You can essentially view an alias as a rewrite rule without a side condition that applies statically before rewriting begins and applies statically in reverse after rewriting ends.
I haven't tested it, but something like this should work:
rule A, B; ({ As }, { Bs }) => { A; As }, {B; Bs}
rule .ABs => { .As }, { .Bs }

Is there/will there be a nightly build that incorporates this PR?


Support Zoom

Current progress supporting zoomable graphs:
https://i.gyazo.com/421834f17db89ef01d11b637de9d5391.mp4

I think this will do for now. I could still improve the movement of the nodes but at least it is usable.
https://i.gyazo.com/a019dda57bf115517bd2b36e700001dd.mp4


Make it easier to find/use extensions

Right now, when you load extensions, it loads VERY large tiles that you have to scroll through to get to anything useful.  Worse, there are extensions for all platforms, so you don't even know which will work in your particular toolbox.  Also, if your friend made an extension, it's hard to figure out how to add it.

Ideally, recently used extensions (including ones that were included in tutorials) would be presented near the top of the list, whether or not they were MakeCode extensions
Ideally, there would be a dropdown menu/filter for which extensions to display, with the current platform as default

We implemented this after new extension design.


Bug in the code

in NLPS -> course 2 -> week 4 -> Assignment -> Ex.4 -> we should separate the l1 computation into l1_relu and l1 variables. l1_relu should be used for gradient computations of grad_W1 and grad_W2, while the original l1 should be used for grad_b1 and grad_b2. Look into this quicker. thanks!

Could you please provide the code for this?


Very slow to move to the next unread feed when connected to Inoreader

Newsboat version (copy the output of newsboat -v or the first line of git show):
newsboat 2.17.1
System: Linux 4.19.88-1-MANJARO (x86_64)
Compiler: g++ 9.1.0
ncurses: ncurses 6.1.20180127 (compiled with 6.1)
libcurl: libcurl/7.67.0 OpenSSL/1.1.1d zlib/1.2.11 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.2.0) libssh2/1.9.0 nghttp2/1.39.2 (compiled with 7.66.0)
SQLite: 3.30.1 (compiled with 3.29.0)
libxml2: compiled with 2.9.9
Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
show-read-feeds no

urls-source "inoreader"
inoreader-login<EMAIL_ADDRESS>inoreader-password "ZZZZZZZZZ"

bind-key j down
bind-key k up
bind-key j next articlelist
bind-key k prev articlelist
bind-key J next-feed articlelist
bind-key K prev-feed articlelist
bind-key G end
bind-key g home
bind-key d pagedown
bind-key u pageup
bind-key l open
bind-key h quit
bind-key a toggle-article-read
bind-key n next-unread
bind-key N prev-unread
bind-key D pb-download
bind-key U show-urls
bind-key x pb-delete

color listnormal cyan default
color listfocus black yellow standout bold
color listnormal_unread blue default
color listfocus_unread yellow default bold
color info red black bold
color article cyan default

macro v set browser "mpv %u&!"; open-in-browser ; set browser "brave %u"

So, I gave Inoreader a shot and then connected it to Newsboat instead of using a local urls file. The problem is that it got incredibly slow, to the point where moving toward the next unread feed takes seconds (I can go through about four feeds/minute tops). That is surprising since I would not think that, once synchronized, Newsboat would have to connect to Inoreader to simply move from one feed to the next. Am I missing something? Just wondering if anybody had the same experience.
Steps to reproduce the issue:

wait until sync is over

open one feed

press n to get to the next feed - it takes a few seconds (can get through a mas of four feeds/minute)

Other info you think is relevant:

Hi! Sorry for the delay—took a bit of time off the project for the New Years :)
Does Newsboat consume one of your CPU cores after you press n? If so, try setting show-read-articles to no. Newsboat has to format the whole list before displaying it, so if you have a lot of feeds or article, the program takes a while to display them all.

(Not sure if this is a bug or just a question, so tagging with both for now)

+1
Probably it's caused by the delay of updating read status with inoreader.com. Sadly, I became a paid user of inoreader just because newsboat has support for it.
Hope this delay got fixed someday.
Thank you for the nice work!

Borrowed some code from ttrssapi.cpp. Still compiling..
diff --git a/src/inoreaderapi.cpp b/src/inoreaderapi.cpp
index a27f66a3..0bdd69d4 100644
--- a/src/inoreaderapi.cpp
+++ b/src/inoreaderapi.cpp
@@ -4,6 +4,7 @@
 #include <curl/curl.h>
 #include <json.h>
 #include <vector>
+#include <thread>

 #include "config.h"
 #include "strprintf.h"
@@ -214,8 +215,18 @@ bool InoreaderApi::mark_all_read(const std::string& feedurl)

 bool InoreaderApi::mark_article_read(const std::string& guid, bool read)
 {
-	std::string token = get_new_token();
-	return mark_article_read_with_token(guid, read, token);
+	// Do this in a thread, as we don't care about the result enough to wait
+	// for it. // borrowed from ttrssapi.cpp
+	std::thread t{[=]()
+	{
+		LOG(Level::DEBUG,
+			"InoreaderApi::mark_article_read: inside thread, marking "
+			"thread as read...");
+		std::string token = get_new_token();
+		mark_article_read_with_token(guid, read, token);
+	}};
+	t.detach();
+	return true;
 }

 bool InoreaderApi::mark_article_read_with_token(const std::string& guid,

@scateu, after you've tested it, can you submit it as a pull request? Thanks for working on it either way!

Yay. I'm so lucky that it works. Pull request right on the way.

I have the same problem with nextcloud. Could the fix be similarly straightforward?

@mredaelli Yes. Since nextCloud share the same API with ownCloud, I believe it's here: https://github.com/newsboat/newsboat/blob/master/src/ocnewsapi.cpp#L157
However I don't have an instance of nextCloud/ownCloud, maybe you can patch it and file a pull request.


Add functionality to aten::index converter to handle numpy indices

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

Describe the solution you'd like

Describe alternatives you've considered

Additional context

#2396


Regular Socket Implementation Server Reader

Hi, I have many socket's client with a specific protocol connection, generally is:
/*
sc = new Socket( HOST , PORT );
out= new DataOutputStream(sc.getOutputStream());
out.write("Hello".getBytes());
sc.close();
*/
This connection is not possible to change to the new websocket protocol, because the clients are hardware device with specific connection by tcp.
Is possible to add a reader for regular tcp in the server?
Thank you.

Not gonna implement this. Please use a seperat socket server and let this server communicate with your websocket server.
Greetings
marci4


Adding standard files

Hi @hyandell ,
Thank you very much for the pull request. We will conduct the change to our internal branch accordingly.
Thanks,
Liusu


HBASE-24479: Deflake TestCompaction#testStopStartCompaction

Polling of active compaction count is racy. Tightened the asserts
to be more reliable.

I have branch-1 checked out locally, so I submitted a PR for branch-1 now, will forward port this to other branches after this is merged. This applies to all branches AFAICT.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
7m  3s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
9m 53s
branch-1 passed

+1 :green_heart:
compile
0m 40s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
compile
0m 44s
branch-1 passed with JDK v1.7.0_262

+1 :green_heart:
checkstyle
1m 39s
branch-1 passed

+1 :green_heart:
shadedjars
3m  4s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 51s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 40s
branch-1 passed with JDK v1.7.0_262

+0 :ok:
spotbugs
3m  1s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m  0s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 57s
the patch passed

+1 :green_heart:
compile
0m 39s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javac
0m 39s
the patch passed

+1 :green_heart:
compile
0m 47s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
javac
0m 47s
the patch passed

-1 :x:
checkstyle
1m 32s
hbase-server: The patch generated 1 new + 12 unchanged - 0 fixed = 13 total (was 12)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
2m 47s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 40s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 31s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
0m 44s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
findbugs
2m 46s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
123m 17s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

171m 30s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1820/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1820

JIRA Issue
HBASE-24479

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux b17f2a4dc55b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1820/out/precommit/personality/provided.sh

git revision
branch-1 / 3c13884

Default Java
1.7.0_262

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_262

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1820/1/artifact/out/diff-checkstyle-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1820/1/testReport/

Max. process+thread count
4113 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1820/1/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Springload 2019 logo update

Updating Springload logo and favicon to 2019 update.

Coverage remained the same at 89.14% when pulling f0760d36006b2021adb01de6e1a04e68a53f33b5 on feature/logo-update into<PHONE_NUMBER>8f616b1d99590b14af29a5085b4a10 on master.

Wicked, cheers @janzenz.


Fix authentication

Apparently the cookie's name for session auth changed, it's now "_gabsocial_session".

Interestingly, it seems like maybe both are required? When I log into Gab, I see both _gabsocial_session and _session_id being set:

Is there an error you're running into? I suppose we could update this check to verify both are present.

Hi @milesmcc, I quickly talked with @lxcode about this, it looks like on my end only _gabsocial_session is used, on his, he was only seeing _session_id indeed.
I guess it's more one or the other than both at the same time. We could add a check to see if one of the two is present?

That approach makes sense to me! Want to update this PR? I can also just make the push.


Tweaks to docker compose file for mainnet deploy

@tizoc This deploy_monitoring is deprecated now. I only left in so I can continue reworking the deploy part, as I already extracted the monitoring part. Please edit the compose in /tezedge/apps/node_monitoring :)

@adonagy ok good, I will update that one tomorrow then.


Map view layout feedback in Manager

Describe requirements
With the new map view layout,  we would like to make the following changes

[ ] If related table is present in the map, disable the layer because we can't really navigate to related records by selecting records from map.
[ ] Tools to show in the toolbar first, if overflows then go to More menu
[ ] Add Clear selection button

we can select features from map but there is no way to clear the selection unless we change the layout, could we please add Clear selection button in the More menu?

Add Clear selection button in the More menu
If no feature is selected it will be disabled just like the split and table view
It will be enabled when there is selection.

[ ] Change initial icon and instruction in the Information panel to map icon and the text would be "Select a feature from map to get started. Click while pressing the Cntrl key to select multiple features".

cc @sarahmcdonald7

@azizaparveen

If related table is present in the map, disable the layer because we can't really navigate to related records by selecting records from map.

If related tables are present in the map do we want to disable them from the layer picker shown in that mapview?

Could you please confirm if our understanding is correct if not please add more details on this - which layer to be disabled when we have related table in the map

Could you please confirm if our understanding is correct if not please add more details on this - which layer to be disabled when we have related table in the map

Yes, you are correct, the layer should be disabled in layer picker

@azizaparveen, @sumitzarkar, just to clarify. I think we would want all tables to be disabled, whether they are related tables or not. It is just not possible to select a table record in the Map View, so it makes no sense to make it the active layer/table.

Solution

Disabled the table from map-layer-picker when in map view layout
Tools should shown in the toolbar first, if it gets overflows then more dropdown button will shown and it the tools should shown in the more dropdown button
Added clear selection tool in the map view layout's tool bar to clear the selection if feature is selected

If no feature is selected then clear selection tool should shown as disabled
Enable the clear selectionn tool when feature is selected

Dev Checklist

Items
Is required
Status
Comments

Required NLS changes?
Yes

Required Backward compatibility
No

Required 508
Yes

Need Sanitize?
No

Impact Analysis Report

Impacted Areas
Comments

Have Configuration changes?
No

Have Runtime changes?
Yes

Impact on RTL
Yes

Impacted components
1. map card  2. layer-table  3. crowdsource-manager

Impact on map view layout
- Table should shown as disabled from map-layer-picker

- Tools should shown in the toolbar first, if it gets overflows then more dropdown button will shown and it the tools should shown in the more dropdown button

- Clear selection tool should shown in the map view layout's tool bar to clear the selection if feature is selected   - If no feature is selected then clear selection tool should shown as disabled   - Enable the clear selectionn tool when feature is selected

This looks good in devext


"Blank" lines with no days related to the current month displayed on certain months

I'm Submitting a ...

[x] Bug report
[] Feature request
[] Support request

In DaysView, sometimes there is a full line of days belonging to the past month, which shouldn't be displayed at all since the whole week belongs to the previous month and not the current month and the "empty week" simply doesn't anyhow relate. Same applies for a full line of days belonging to the next month, which sometimes appear on Februarys ending with last day of week.
I see in the code you're using hard-coded constant 42 to build the calendar table (I get it is 4x7 weeks for minimal length of the month + 2x7 for the first and the last line) but I'd expect you to drop the "blank" lines completely. You should be checking whether the previous month ended with the last day of week (to drop the first line) or current month ended with the last day of week (drop the last line).
Steps to Reproduce

Open the example page
Set date to October 2000 - the first line only contains days from previous month - nothing related to current month
Set date to February 1999 - the last line only contains days from previous month - nothing related to current month
Set date to February 1998 - both the first are not related

Expected Results
The lines containing no days related to current month should be dropped. Of course I get you'll lose the "fixed grid" of 7x6 to 7x5 or even 7x4 but the real question here is whether having it fixed is a necessary feature. Checking other date pickers on the market (mainly via flight companies' sites) not showing unrelated lines of days is an expected behaviour.

Hi @goodnightcat. I see what you mean and sure, maybe this is sub-optimal.  If you want to submit a PR then I'll gladly review it.


Problem with MenuButton hovering popup menu

Godot version
v4.2.dev4.mono.official [549fcce5f]
System information
MacOS 10.15
Issue description
I'm using the MenuButton component, which contains a popup menu. I activated the hover functionality switch to ensure that when the mouse is moved it always hovers and follows the new button.
However, when I switch back and forth between full screen and windowed, or change the window size, the hover functionality is hit or miss.
Steps to reproduce
Open the Minimal reproduction project and switch between full screen and windowed, or resize the window.
The position of the Popup menu will be wrong.
Minimal reproduction project
Archive.zip

I'm experiencing the same issue, but with a standard Button. One thing I've found is that I see this incorrect hover behavior when manually resizing the window size. But if I set the window size through GDScript e.g:
get_window().size = Vector2i(1920, 1080)
The Button behaves as normal.


Move firmware to TinyGo

Currently can't move the firmware to TinyGo due to the rp2040 not having PWM support, this is being worked on. See: https://github.com/tinygo-org/tinygo/issues/1993

Now being worked on


BVAL-668 Addressing Guillaume's review remarks

https://hibernate.atlassian.net/browse/BVAL-668

Merged, thanks!