Documentation not clear on monitor socket limitations

Referring to get_monitor_socket(events = [], addr=None) which is documented here
Problem
I recently ran into an issue whilst creating an automated test. I wanted to simulate the Majordomo Protocol being used with a pool of about 25 clients and 50 workers.
Most of the test runs failed, the error coming from socket.get_monitor_socket(). The error was that the "Address was in use".
Setup
I had created a context class to manage handling the setup of a client and worker, it contained boiler plate code for connecting zmq sockets to the broker.
I use monitor sockets to ensure that the sockets I connect to the broker have finished their handshake and are ready to start sending/receiving data. I simply wait for the 'connected' event and then return from an asyncio method, allowing the calling client to begin requesting.
The automated test uses asyncio tasks, one for each client and one for each worker. Each client and worker gets one monitor socket as a result so the respective client/worker can ensure the socket connection to the broker is a success before proceeding further.
Method to resolve the problem
The method get_monitor_sockets(events = [], addr=None) is in zmq.sugar.socket.py , note lines 845 to 847
        if addr is None:
            # create endpoint name from internal fd
            addr = "inproc://monitor.s-%d" % self.FD

I noticed that self.FD was used to distinguish the endpoints for each monitor socket instance. I guessed that this might refer to File Descriptors, and presumed that I might had run out of file descriptors (although there were only really 75 ish sockets needed for this test, plus monitor sockets, one for each client/worker socket).
So I made a small change to my code, instead of leaving the addr argument blank, I populated it using uuid.uuid4() to ensure each endpoint was unique.
This suggests something a bit odd though. If I was truly running out of file descriptors, then I would expect the test to still fail, but it does not. So I may be wrong about the meaning of self.FD. I did a test to examine how many fd were indeed open. The numbers closely, but not exactly, matched those of self.FD when printed to console.
Impact on pyzmq consumers
In any case, the documentation gives no hint that you can run out of monitor sockets if you don't specify the addr argument. I admit that my use case is a bit extreme, but, I am building something i hope will be extreme and have to test it. Profiling performance will be a big part of tuning the system and making sure its fit for purpose. Therefore being able to simulate lots of clients and workers with just one physical system is important for development.
And that's where I feel this issue really becomes relevant, it impacts people trying to do ambitious projects.
Requested changes (and I am happy to fork your documentation source and issue a PR if you like)
I think it would be good to add a note to the current documentation  to alert devs to the fact that there is an upper limit on the number of sockets available if no addr argument is given.

Yes, absolutely! It is appropriate to add to the docstring that if addr is unspecified, a unique per-socket address will be used (that's what the FD is used for - it's a unique integer across currently open sockets, which is all we really need), meaning that it can be called at most once per socket without specifying addr. If you'd like to make a PR clarifying that, it would be most welcome.
It is quite unusual to connect multiple monitors to one socket, which should be the only way this error can occur. Is there a reason you are using multiple simultaneous monitors on one socket? If you're not doing that, can you share an example to illustrate the problem?

Hi min, I am almost certain that these are separate instances of the pyzmq socket class.
I have a class that I create for each client which then does;
async def connect(self):

        logger.info("Creating a zmq_socket to connect to broker")

        self.zmq_socket = self.z_cont.socket(self.SOCK_TYPE)

        logger.info("Creating a monitor zmq_socket to listen for connect event")
        #monitor for connect
        mon_sock = self.zmq_socket.get_monitor_socket()

Looks like it's a failure to close the monitor sockets, which are not closed implicitly if you close the socket itself (because then your monitor couldn't get close events!)
To fully close a monitored socket and its monitors, you must do:
socket.disable_monitor()
mon_socket.close()
socket.close()

This is absolutely not obvious (disable_monitor must be before mon_socket.close()), and should be clearly documented.
Here's a complete example that can create thousands of sockets, where it would fail at 30-50 before:
import asyncio

import tqdm
import zmq
import zmq.asyncio

class Thing:
    def __init__(self):
        self.z_cont = zmq.asyncio.Context.instance()
        self.SOCK_TYPE = zmq.PUB

    async def connect(self):

        self.zmq_socket = self.z_cont.socket(self.SOCK_TYPE)

        #monitor for connect
        self.mon_sock = self.zmq_socket.get_monitor_socket()

    def close(self):
        self.zmq_socket.disable_monitor()
        self.mon_sock.close()
        self.zmq_socket.close()

async def main():
    for i in tqdm.tqdm(range(5000)):
        t = Thing()
        await t.connect()
        t.close()
        # close is async, so it takes a finite amount of time to free FDs
        await asyncio.sleep(1e-3)

if __name__ == "__main__":
    asyncio.run(main())

Daaaammnnn!! Ok that makes sense, and yes, let's definitely update the docs then. I mean, I was pretty sloppy to be honest, I know I wasn't living up to good housekeeping with explicitly closing sockets. So this has reinforced that lesson. But it would certainly not be obvious at all for others too.
And your example is spookily on point for the failure point. I was at about 25 on the client side, where it failed. I'm going to implement what you just showed me. Heck I should have done this in the async tear down func for python unit tests that I am using.


How to save grasp?

Hi Jennifer,
I'd like to save grasp from EigeinGrasp Planners. Could you give me some suggestions?
I added:
FILE *f = fopen("grasp.txt","w");
for (int i=0; i<mPlanner->getListSize(); i++)
      mPlanner->getGrasp(i)->writeToFile(f);
fclose(f);

to egPlannerDlg.cpp and visualized Tx Ty Tz as point(x,y,z) as following:

I though the red points should be placed on object surface as touching points, but they do not. How to save touching points?

Hi,
This question probably belongs to the graspit repo, I presume you weren't using my graspit-pkgs packages?
Generally, when you save a grasp, it would save the joint (denavit Hartenberg) parameters, and wrist pose, AFAIK. But it's been a while I dealt with the graspit core code in detail so I could remember this wrong...
I can take a look at the code and I'll be able to tell you exactly, but I won't have the time before next week unfortunately... Ping me if you haven't found the answer before then :)

Hi Jennifer,
Yes I should ask the question in the graspit repo, but they are not likely be active. Thank you for replying all my questions, really really helpful.
For saving grasps, I solved it. Just need modify egPlannerDlg.cpp a little bit.
I am having a new question. EigenGrasp Planner works for object and robot in the scene. But if I add an obstacle to the scene, the planner doesn't functional anymore. Is there any way to make the planner work with object with obstacles?

Yes I've used obstacles (see tutorials in wiki) so it should work. However if your object lies on the table with suitable grasp points badly reachable by the gripper, the planner will fail because it finds collisions with the table for the suitable grasps. Make sure you have enough contact points on the object. Maybe increasing number of iterations helps, or re-planning a few times

It works! Thank you!


Failed to load the tool-bar package after 1.1.5 update

Atom Version
1.23.3
OS
Mac
Tool-bar package version
1.1.5 (latest)
Error description
Failed to load the tool-bar package
variable @button-text-color-selected is undefined in /Users/$$$$/.atom/packages/tool-bar/styles/tool-bar.less:37:13

ditto

Oh it seems to be an issue when using atom-material-ui theme. I'll fix it asap

I don't use atom-material-ui but I still have the issue. I'm using the Atom light UI (the default one).

I just manually removed the changes from PR #177 and everything works fine again.

I'm very sorry for introducing this bug! I just released 1.1.6 which should resolve it!

works fine again, thx! :) 👍

Yep, thanks!


Reference system overhaul

To continue discussions on https://github.com/processing/p5.js/issues/2857 and https://github.com/processing/p5.js/pull/2875 this issue should be more focused on the overhaul as I've made some progress on it now.
Without resorting to minifying opentype.js or using git submodules as discussed on the two threads above, I managed to just get the reference pages to render like any other pages on the website. Mainly because of the backbone.js system it is actually much simpler than I expected since we don't need anything yuidoc generate except for data.json. I have a working fork available to test here.
With the fork above, instead of copying the whole generated reference folder over to this repo, it just needs the assets folder (containing reference specific js and files for examples) and the data.json+data.min.json files. It will build the reference page with the templates in src/templates/pages/reference/index.hbs just like any other pages. This means the translation of sidebar, header and footer will be consistent across the whole site.
Ultimately, I want the backbone.js part (the actual reference without sidebar, header and footer) to be relatively standalone, that way this standalone version can be used both for previewing the reference on the main p5.js repo (the sidebar are useless anyway there) and also to be packaged up as the offline reference (which already removes the sidebar). Then on the site it will just wrap the standalone bit with other things. That way we don't have to maintain too much of the website stuff over on the main p5.js repo.
I still have more to do this is just to share some progress but if this seems like a good way to go, I'll merge this in along with the Chinese translation stuff as soon as it is ready to merge.

hey @limzykenneth sounds great! can you clarify one thing -- so all reference related files (templates, css, js, etc) live in the website repo, correct? will there still be a way to generate a preview from within the p5.js repo?

For now nothing has changed in terms of workflow in the p5.js repo and the plan is not to change it but just simplify the rendering. I have copied the templates over to this repo so the templating and assembly can all be done in this repo, I haven't moved the JS files yet (ie. the require.js stuff that generate reference.js), css are integrated into all.css bundle (which are just two rules only), and the assets still remain on the p5.js repo.
To update the reference, only three things need to be copied over from the p5.js repo, all in the generated docs/reference folder: assets folder, data.json and data.min.json. The rest from what I can tell are not needed by both repos which we can remove with the build step to clean things up a bit as well.

I see. I guess I'm just worrying about the template file in p5.js repo and the template file in p5.js-website repo deviating. Or am I misunderstanding? Is there a way to copy the template from the p5.js-website repo into the p5.js one automatically? Maybe that's too loopy.
To be clear, I'm very in favor of this! Just want to make sure we've accounted for all the processes around them.

I'm still working out the exact setup that would fulfill certain criteria for the whole system so some bits may still be unclear or unaccounted for. To get a bit more technical I'll explain in more detail what I'm thinking of.
For the general templating, ie the handlebars stuff, it will be mainly on the website repo to keep the sidebars and stuff consistent, the reference itself isn't much handlebars templating just come containers and JS for i18n; the p5.js repo will have a minimal version equivalent to the reference handlebars (ie without the sidebar and tuff) just to be able to run yuidoc and preview the reference.
For the reference templating and rendering, ie the backbone stuff including underscore templates, they are going to be in the p5.js repo to preserve the ability to preview, they will eventually all compile into reference.js which is then copied over to the website repo. So in a sense it can be thought of as having two rendering steps: the handlebars one are all in the website repo then the backbone one are all in p5.js repo.
As mentioned the JS would still be in p5.js repo and to be copied over along with the assets, whether it make sense to move it or not I'll still need to think about it.


Getting 500 error code  and feature enhancement plan to gray out or hide on files not having permission to view

Hi Author,
I'm getting 500 error code when trying to browse files from nested folder and also is there a feature enhancement plan to gray out or hide on files not having permission to view.

Sounds like a duplicate of https://github.com/IBM/jupyterlab-s3-browser/issues/15


Progressive-enhancement bugs

I'll collect progressive enhancement bugs we can tackle here. In general, as we've done our best to extend native HTML elements for our functionality we degrade relatively well in modern browsers. I'll be doing research into how that translates for older versions of IE as part of this issue too.
Some good quick PE tips: https://www.gov.uk/service-manual/making-software/progressive-enhancement.html

Make any new page or feature work with HTML alone – no images, no CSS, no JavaScript, nothing but HTML. Interactive elements are only those capable of being implemented with forms and server-side processing. This gives a baseline experience, which will work in practically every browser. It allows your site to work for as many people and devices as possible, including older legacy browsers and devices.

Closing in favour of what we're tackling in #674


Playback Broken in G+

All videos in G+ will no longer play if YTC loads on the page. Had to exclude G+ entirely as of today. Hopefully somebody can correct this. Probably best to simply disable G+ support entirely.

Same with Hangouts and Hangouts On Air (that feeds into YouTube).


missing update_mask option

I was bitten 😅

@googlebot I signed it!

Huh. I went to all that effort to make an update mask, and then just left it there.
Thanks for spotting that and submitting a PR!


Expected a template that would generate exactly one statement to replace one statement, but generated 2

Hi,
I'm trying to write some Recipes for an API migration (non public API). I'm using version 7.24.0
The class names between the old and the new APIs stayed the same, but the packages have changed and some of the method return types, too.
Let's say before that change, there was the following class:
package oldPackage.A;
class A
{
    double getValue() { .. }
}

and now there is:
package newPackage.A;
class A
{
    Double getValue() { .. }
}

So I want to convert calls from a.getValue() to a.getValue().doubleValue()
So my Java vistor looks like that:
            public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, ExecutionContext ctx)
            {
                J.MethodInvocation methodInvocation = super.visitMethodInvocation(method, ctx);

                if (methodInvocation != null)
                {
                    Method methodType = methodInvocation.getMethodType();
                    if (methodType != null)
                    {
                        if (methodType.getDeclaringType().getPackageName().startsWith("oldPackage"))
                        {
                            @Nullable
                            Primitive primitiveReturnType = TypeUtils.asPrimitive(methodType.getReturnType());
                            if (primitiveReturnType == Double)
                            {
                                JavaTemplate.Builder value = JavaTemplate.builder(this::getCursor, "#{any(java.lang.Double)}.doubleValue()");
                                String declaringType = methodType.getDeclaringType().toString();
                                String newDeclaringType = oldToNewMap.get(declaringType); // returns: "newPackage.A"
                                doAfterVisit(new ChangeType(declaringType, newDeclaringType, true));
                                return methodInvocation.withTemplate(value.build(),
                                methodInvocation.getCoordinates().replace(),
                                methodInvocation.withDeclaringType(ShallowClass.build(newDeclaringType))); /* new type is on classpath */
                            }
                        }
                    }
                }
                return methodInvocation;
            }

While this seems to work fine for the Java code, there is one Javadoc in the code which states:
import oldPackage.A;
class MyClass
{
    /**
     * @return {@link A#getValue()}
     */
    public double foo()
    {
         return a.getValue();
    }
}

which results in the following IllegalArgumentException:
Expected a template that would generate exactly one statement to replace one statement, but generated 2. Template: __P__.<java.lang.Double>/*__p0__*/p().doubleValue()
Is there something in addition I have to do for the Javadoc? Or is my general approach wrong how I want to replace the method calls?

Hi @ZzetT, thanks for reporting the issue! After a few tests, I think there is a bug in JavaTemplate in regards to JavaDocs. I suspect the issue is related to slight differences between source code and JavaDocs like special characters (#).
We'll take a look!

Hi @traceyyoshima
I think I found another example where the method invocation replacement fails with exactly the same error message:
import oldPackage.A;
class MyClass
{
    public void foo()
    {
         double value = 0;
         if ((value = a.getValue()) != 0) {}
    }
}

I debugged it a bit and saw that the resulting stub looks like that:
class __P__ {  static native <T> T p();  static native <T> T[] arrp();  static native boolean booleanp();  static native byte bytep();  static native char charp();  static native double doublep();  static native int intp();  static native long longp();  static native short shortp();  static native float floatp();}class __M__ {  static native Object any(Object o);  static native <T> Object anyT();}class A{
public void foo(){
double value =0;
boolean __b9__ =value =
/*__TEMPLATE__*/__P__.<java.lang.Double>/*__p0__*/p().doubleValue()
;;}}

As you can see, for some reason it thinks that (value = a.getValue()) is a boolean.

I think I found another example where the method invocation replacement fails with exactly the same error message:

@ZzetT Thanks for reporting this issue too! I created a new issue based on what you reported here.

@ZzetT I am closing this one since the related issue https://github.com/openrewrite/rewrite/issues/2090 has been completed.
Please re-open if the bug persists.

@pway99 just making sure you see @ZzetT 's comment. Was the JavaDoc issues resolved as well?

Thanks for the clarification, @ZzetT. I'm unaware of a fix for the JavaDoc issue and will re-open it.

@ZzetT I'm going to edit the issue to clarify this is to track an issue in updating method invocations in JavaDocs.


ci: run on gpu

Description
This is a test PR to see if gpu dependent cuda tests will pass on a machine with an nvidia gpu.
Related links
Parent Issues:

https://github.com/autowarefoundation/autoware.universe/issues/7668
https://github.com/autowarefoundation/autoware.universe/issues/7724

How was this PR tested?
It has passed:

https://github.com/autowarefoundation/autoware.universe/actions/runs/9693666201/job/26749627081

Notes for reviewers
This is not meant to be merged.
Interface changes
None.

Effects on system behavior
None.

When run on a machine with gpu with --gpus all flag, it passes.

Closing since this was only for reference.


angular-material namespace collating with polymer.

Is there anyway to scope polymer layouts statements like flex** to use something instead like polymerl-flex so that does not interfere with other layouts engines?

I'm gonna ask something maybe silly? but it's not one of the scopes of the web components to isolate styles to avoid conflicts with another web components. I see iron-flex and other components are mixing styles with another frameworks which makes not possible to guarantee delivery of those web components. I realized this problem does not go playing together with angular-material but also with another libraries and the application itself.

It's unfortunately angular-material decide to use the same naming conventions as Polymer. However, not that the layout attributes have moved to classes so flex is not class="flex". On their site, it looks like they use attributes so there should be fewer conflicts that before.
Until Shadow DOM is native in browsers, we're stuck with Shady DOM. Unfortunately, it does not have the same style scoping that the native API has.


meta: AAP roadmap 2023 edition

This is a roadmap for the entire Android Audio Plugin Framework in 2023. It mostly takes over Roadmap 2022: https://github.com/atsushieno/android-audio-plugin-framework/issues/96 . It will be occasionally updated as new tasks show up.

Core features

[x] fix aapbarebonepluginsample that still has some problem that prevent migration to implicit ports
[ ] port-config extension #69
[ ] dynamic manifest #44

MidiDeviceService improvements

[x] default activity as settings for MidiDeviceService #120
[x] Presets integration.
[x] strict MIDI message dispatcher implementation based on atomic ops (currently it isn't).

Apps

[x] build simpler aap-juce test hosting app
[x] create realtime application example (can be based on samples-host-engine) #76
[ ] get augene-ng (or other tracktion_engine apps) working
[x] get helio-workstation (aap-juce-helio) working

GUI integration #34

[x] design patch protocol over MIDI2 ports we don't need it. See GUI.md
[x] plugin in-process editor - JUCE PluginEditor integration
[ ] plugin out-process editor - get atsushieno/aqua working.
[ ] plugin out-process editor - juce_emscripten integration? (for now I don't expect the official JUCE development to go further on their own wasm support)

Testing

[ ] test instrument + effect connection, within a plugin connection (graph)
[ ] test runner app/orchestrator that can process multiple plugins from other apps #40

Community / Publication

[ ] move to organization
[x] contribution guidelines
[ ] MidiDeviceService apps that can be published to Play Store.
[x] Some PoC-ish effect plugins and effector app: can change presets by MIDI2 program change.

Removed from roadmap 2022: active sensing (we have good-ish status check now on logcat), MIDI 2.0 support in aap-core (can wait until Android MIDI API supports it on MidiDeviceService), MIDI 2.0 apps to aap-lv2 (LV2 itself does not support MIDI2 yet), aap-juce (can wait until JUCE AudioProcessor supports MIDI2 like AU already does)

Transition to 2024 edition.


Cannot fully switch to I18n

reproduced on docker
git clone https://github.com/apache/incubator-superset/
cd incubator-superset
# you can run this command everytime you need to start superset now:
docker-compose up

language resource redirect to 302.


New release

Hi,
There are a few CVEs open against this repository, which we are using as part of our distribution.
There are fixes that have been merged since the last release, such as: https://nvd.nist.gov/vuln/detail/CVE-2018-10536, to mitigate some CVEs.
When will the next release occur so we can update and mitigate from these vulnerabilities?

I have been working on some new features that I would like to add (like accepting AIFF files and ID3v2.2 tags), but have been very busy with paying work lately and so haven't had any time, and I am not sure when that will change.
However, I will try to produce a 5.1.1 bugifx release in the next month or so because there have been a lot of simple fixes and improvements since the last release that would be great to get in.
Thanks for bringing this up!

Hi @dbry,
understandable and thanks for the response, we will keep track of this release and upgrade when it is ready :)

WavPack 5.2.0 Release
Sorry for the delay on this, and I should have mentioned this before, but I would like to say now that the reason that I was not overly concerned about a prompt release is that all of the outstanding CVEs only involve the use of the WavPack command-line program and malformed source files (not malformed WavPack files). There were absolutely no exploitable bugs in the library or the command-line decoder, and only one or two of the CVEs were actually possible to exploit, and again would have required manual execution by the user. In any event, there's a new release now with all the fixes in. Thanks!


compiler: support type aliasing

Closes #39

Codecov Report

Merging #40 (ccb59e6) into main (81edf40) will decrease coverage by 0.07%.
The diff coverage is 40.62%.

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   46.49%   46.42%   -0.08%
==========================================
  Files          66       66
  Lines        5654     5702      +48
==========================================
+ Hits         2629     2647      +18
- Misses       2891     2919      +28
- Partials      134      136       +2


fix(parser): parse negative numbers

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?

Summarize your change (mandatory)
Partial fix for #4344 . Able to parse negative numbers e.g. -1, without wrapping in unary_op Minus.

Describe any limitations of the current code (optional)

Cannot deal with nested literal expressions -(-(-1)). However, I think these types of expressions are rare, can be fixed another time. Have mentioned this in original issue description.
If we wish to handle these types of expressions, we need to capture and update the sign of Value::Number or Expr::TypedString.
Since both of these store inner numbers as String, it is not efficient or elegant to update them. So refactoring will need to be done to use a more efficient representation for inner numbers, which can capture sign too.

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
#4344

Maybe better to just do this in binder... :thinking: rewrite_positive is done there as well.


add git integrity checks

Came across this and thought it was worth taking a closer look at:

Everyone (afaik) should consider adding the following snippet to their ~/.gitconfig:
# for $deity's sake, check that anything we're getting is complete and sane on a regular basis
[transfer]
        fsckobjects = true
[fetch]
        fsckobjects = true
[receive]
        fsckObjects = true

ref: https://groups.google.com/forum/#!topic/binary-transparency/f-BI4o8HZW0

At the end of the thread it's implied that this is enough:
[transfer]
  fsckobjects = true


Remove the old menu implementation

This removes the old menubar implementation to allow macOS and the native menubar experience to take advantage of the new menubar registration. Please test the new branch and review.

@sbatten thanks for the PR. Before I test the new branch could you clarify something:

Why is there still a file menus.ts, isn't the idea to get rid of this file and the old registration?
There are some conflicts with master, can you please resolve those
Since now the native menu takes advantage of the new menu bar registration - is it possible to dynamicly enable / disable actions in the native bar (as we can currently do in custom menu)?

I can test this in great detail, but for the review I only did a birds eye perspective review (comments lnline). @bpasero would be better for the more detailed review (if needed next week once he comes back from vacation)

In response to your initial comment: I will remove the menus.ts and possibly keyboard.ts as they are no longer used. Yes, the native menus will get the same dynamic enablement features as the custom menus with this change. @bpasero had discussed that this change was desired ASAP to get some coverage in insiders and recommended you as the reviewer.

@sbatten great, just ping me once all those files are removed and I will test this out on mac.
In my view we should create a test plan item complexity 4 for this on all three platforms. In the endgame week we can decide if we want to ship this or not (in case of too many issues)

Tried it out on mac, here are the issues

Error on startup

Preferences menu is top level (also in Code-Insiders as it should so it is duplicated)

Apart from that seems to work fine. Let me know if you want me to try it out on other platforms.

@isidorn fixed issues, please merge if ok, thanks for the review

Looks great, nice work! I have resolved some conflicts and am merging.


Do not apply HU window in Backend

Fixes #211
Proposed Changes

Remove code that applied HU window on DICOM images.
In future we will add such feature to the Frontend (#178).

Please rebase to master and fix conficts.


[QUESTION] Where were photos downloaded by download_photos_by_hashtag.py saved?

Before submitting an issue, make sure you have:

[x ] Updated to the lastest version
[ x] Read the README
[ x] Searched the bugtracker for similar issues including closed ones
[x ] Reviewed the sample code in tests and examples

Which example file are you using?
download_photos_by_hashtag.py

Describe your Question/Issue:
I started the script download_photos_by_hashtag.py and it seems to me that  everything is working fine, but i cannot find the downloaded pictures. Under the folder photos I can only finde the default example photos

Paste the output of python -V here: Python 3.8.2
Error/Debug Log:
2020-03-12 13:48:10,456 - INFO - Received 86 medias.
2020-03-12 13:48:10,456 - INFO - After filtration 4 medias left.
2020-03-12 13:48:10,459 - INFO - Going to download 4 medias.
100%|████████████████████████████████████████████████████████████████████████████████████| 4/4 [00:42<00:00, 10.57s/it]
2020-03-12 13:48:57,769 - INFO - Received 85 medias.
2020-03-12 13:48:57,769 - INFO - After filtration 0 medias left.
2020-03-12 13:48:57,770 - INFO - Nothing to downloads.
2020-03-12 13:49:02,821 - INFO - Received 82 medias.
2020-03-12 13:49:02,822 - INFO - After filtration 5 medias left.
2020-03-12 13:49:02,823 - INFO - Going to download 5 medias.
100%|████████████████████████████████████████████████████████████████████████████████████| 5/5 [01:06<00:00, 13.34s/it]
2020-03-12 13:50:13,819 - INFO - Received 83 medias.
2020-03-12 13:50:13,819 - INFO - After filtration 2 medias left.
2020-03-12 13:50:13,820 - INFO - Going to download 2 medias.
100%|████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:28<00:00, 14.21s/it]
2020-03-12 13:50:42,245 - INFO - Total requests: 90

Same question~

Same question :(


CI fix: Update release.yml GITHUB_TOKEN permissions

merge #11

doing merge this time, because carrying the whole history with you is not a good option


How to use angular-translate

How can I use angular-translate in conjunction with this library?
Single:
<select class="chosen-select"
		chosen
		allow-single-deselect="true"
		data-placeholder="WE NEED TRANSLATION"
		no-results-text="WE NEED TRANSLATION"
		ng-model="someModel"
		ng-options="value for value in array">
</select>

Multiple:
<select class="chosen-select"
		chosen
		multiple
		allow-single-deselect="true"
		placeholder-text-multiple="WE NEED TRANSLATION"
		no-results-text="WE NEED TRANSLATION"
		data-ng-model="someModel"
		data-ng-options="value for value in array">
</select>

+1

+1

+1
For single, I have tried following method
<select class="chosen-select"
        chosen
        allow-single-deselect="true"
        data-placeholder="WE NEED TRANSLATION"
        no-results-text="WE NEED TRANSLATION"
        ng-model="someModel"
        ng-options="value | translate for value in array">
</select>

but item does not translate after change language using $translate.use()

It's possible to translate it using the translate filter.
<select chosen
        data-placeholder="{{ 'my.translation.placeholder' | translate }}"
        no-results-text="{{ 'my.translation.noResults' | translate }}" >
</select>

However, attributes are not watched so if the translations are changed/loaded after the directive has been initialized you are stuck with the first or no translation.

It's possible to translate it using the translate filter. Make sure you remember to wrap it around with ' so it's a string.
<select chosen
        placeholder-single-text="'{{ 'my.translation.placeholder' | translate }}'"
        no-results-text="'{{ 'my.translation.noResults' | translate }}'" >
</select>

You cannot use data-placeholder as this attribute is not watched. Use one of these whitelisted attributes:
https://github.com/localytics/angular-chosen/blob/master/src/chosen.coffee#L9-L23

I always use  data-placeholder="{{ 'translation_key' | translate }}" and it works?

They are expression attributes, so you need to use a variable or quote, like so:

no-results-text="'No results match'"
placeholder-text-multiple="'Select Some Options''"
placeholder-text-single="'Select an Option''"

Here's an example in Brazilian Portuguese: http://plnkr.co/edit/P3bWW2?p=preview

@IpsBetty your method works - but there is a bug then when the dropdown is repopulated after page load the interpolation doesnt fire anymore.

@stenmuchow there was an angular-chosen change that caused that this doesn't work anymore..
I now use this:
placeholder-text-single="'{{::'translation_key' | translate}}'"

Aha - thanks for that - I missed the single quotes immediately inside the doubles. So many quotes....

Yeah I also think that is a strange syntax ;)


simple-hub: remove xstate-wallet dependency

Fixes https://github.com/statechannels/monorepo/issues/1177

I assume the code is copy-pasted?

Yes. I started by copy pasting the functions from xstate-wallet used by the hub. Then I copy-pasted functions called by that code (+ removing export statements).


random-password-generator

Description
Initial commit to generate random passwords
Checklist

[ x] I read the extension guidelines
[ x] I read the documentation about publishing
[ x] I ran npm run build and tested this distribution build in Raycast
[ x] I checked that files in the assets folder are used by the extension itself
[ x] I checked that assets used by the README are placed in the metadata folder

hi @textnav 👋 Nice work! 💪 CI seems to be failing because the images don't conform to the guidelines. Mind updating them? 😸

sure

Hi @textnav 👋
Thanks for the corrections and your extension! 🔥
Your metadata images are not following the guidelines, can you look into this? If you need help, let us know 😄
Request a new review when you are ready, feel free to contact me here or at Slack if you have any questions.

🆗 ✅

🎉


Index conf disabled

This PR changes how IndexConfig are enabled/disabled. Instead of using enabled, we will use disabled.
We discussed about this change in https://github.com/apache/pinot/pull/10584.
This changes breaks compatibility. Specifically if someone used enabled: false to disable an index, this PR will ignore that configuration, as shown in the following table:

when enabled was
prev this PR
after this PR
if we use disabled

not defined
sometimes disables the index
enables the index
enables the index

null
sometimes disables the index
enables the index
enables the index

true
enables the index
enables the index
enables the index

false
disables the index
disables the index
enables the index

We assume that given that this new way to configure indexes was merged less than three weeks ago and no release was made, customer won't be actually using enabled: false

I'll let jackie review/approve.
I can offer 1 data point since I initially raised the issue; I'm ok with the behavior switch. The only reason we have "enabled": false at the moment is from the initial bug and not because it was desired.

Codecov Report

Merging #10585 (83d77bf) into master (440a938) will decrease coverage by 0.02%.
The diff coverage is 94.44%.

@@             Coverage Diff              @@
##             master   #10585      +/-   ##
============================================
- Coverage     70.38%   70.36%   -0.02%
- Complexity     5616     6481     +865
============================================
  Files          2106     2106
  Lines        112890   112891       +1
  Branches      17001    17000       -1
============================================
- Hits          79456    79435      -21
- Misses        27872    27888      +16
- Partials       5562     5568       +6

Flag
Coverage Δ

integration1
24.49% <0.00%> (+0.02%)
:arrow_up:

integration2
24.09% <0.00%> (+<0.01%)
:arrow_up:

unittests1
67.90% <94.44%> (-0.02%)
:arrow_down:

unittests2
13.89% <0.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...pinot/segment/spi/index/creator/H3IndexConfig.java
70.58% <75.00%> (ø)

...pinot/segment/spi/index/DictionaryIndexConfig.java
68.18% <83.33%> (ø)

...he/pinot/segment/spi/index/ForwardIndexConfig.java
62.00% <100.00%> (ø)

...apache/pinot/segment/spi/index/FstIndexConfig.java
66.66% <100.00%> (ø)

...ache/pinot/segment/spi/index/RangeIndexConfig.java
100.00% <100.00%> (ø)

...pache/pinot/segment/spi/index/TextIndexConfig.java
83.78% <100.00%> (ø)

...ache/pinot/spi/config/table/BloomFilterConfig.java
68.00% <100.00%> (ø)

...org/apache/pinot/spi/config/table/IndexConfig.java
100.00% <100.00%> (ø)

...apache/pinot/spi/config/table/JsonIndexConfig.java
91.89% <100.00%> (ø)

... and 24 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


IObit Advanced SystemCare Pro <IP_ADDRESS>2 Crack + Portable Download

IObit Advanced SystemCare Pro <IP_ADDRESS>2 Crack + Portable Download
IObit Advanced SystemCare is a powerful optimization tool designed to enhance PC performance and stability. Its one-click optimization feature simplifies system maintenance by cleaning junk files, optimizing settings, and improving startup speed
✅▶▶DOWNLOAD IObit Advanced SystemCare Pro
✅▶▶MIRROR IObit Advanced SystemCare Crack
Software Specifications
The software includes deep registry cleaning, real-time monitoring, and privacy protection to safeguard user data. Additionally, it optimizes internet settings for faster browsing and scans for malware to enhance security. Users can schedule automatic maintenance, ensuring their systems remain in top shape. With an intuitive interface and lightweight design, IObit Advanced SystemCare makes PC optimization accessible for both beginners and advanced users alike.
License Key:

9DO0H-L8SKB-TWLVF-UFDM8-H94ZH
T0QSZ-AIYSD-OSCYP-I11ZC-90SYW
QXIKA-3H25T-URNTO-POVVU-OO73H
POJC2-B0PC7-9GNDM-9NJAN-0UFM7

Free Download Here⬇

✅▶▶DOWNLOAD IObit Advanced SystemCare Pro
✅▶▶MIRROR IObit Advanced SystemCare Crack


[BUG] could not initiate connection with system probe - connection refused

Agent Environment

Describe what happened:
Cannot load system probe despite being enabled in env variables.
Describe what you expected:
System Probe loaded successfully, to see NPM metrics
Steps to reproduce the issue:
Installing the agent directly produces the same results
Additional environment details (Operating System, Cloud provider, etc):
Ubuntu 20.04.5 LTS
5.19.7-xanmod1
Bare-metal co-lo in DC

2022-11-06 11:21:45 UTC | PROCESS | INFO | (pkg/workloadmeta/store.go:337 in startCandidates) | workloadmeta collector "cloudfoundry-container" could not start. error: component workloadmeta-cloudfoundry-container is disabled: Agent is not running on CloudFoundry
2022-11-06 11:21:45 UTC | PROCESS | INFO | (pkg/workloadmeta/store.go:337 in startCandidates) | workloadmeta collector "kubelet" could not start. error: component workloadmeta-kubelet is disabled: Agent is not running on Kubernetes
2022-11-06 11:21:45 UTC | PROCESS | INFO | (pkg/workloadmeta/store.go:154 in Start) | workloadmeta store initialized successfully
2022-11-06 11:21:45 UTC | PROCESS | INFO | (pkg/tagger/collectors/workloadmeta_main.go:115 in stream) | workloadmeta tagger collector started
system-probe exited with code 1, signal 0, restarting in 2 seconds
2022-11-06 11:21:45 UTC | PROCESS | INFO | (api/server.go:40 in StartServer) | API server listening on localhost:6162
2022-11-06 11:21:46 UTC | SECURITY | INFO | (app/app.go:401 in StopAgent) | See ya!
security-agent exited with code 0, disabling
2022-11-06 11:21:46 UTC | PROCESS | INFO | (pkg/process/checks/process.go:76 in Init) | no network ID detected: could not detect network ID
2022-11-06 11:21:46 UTC | PROCESS | WARN | (pkg/process/checks/net.go:70 in Init) | could not initiate connection with system probe: temporary failure in system-probe-util, will retry later: Get "http://unix/debug/stats": dial unix /opt/datadog-agent/run/sysprobe.sock: connect: connection refused
2022-11-06 11:21:47 UTC | CORE | INFO | (pkg/collector/scheduler/scheduler.go:92 in Enter) | Scheduling check ntp with an interval of 15m0s
2022-11-06 11:21:47 UTC | CORE | INFO | (pkg/collector/scheduler/scheduler.go:92 in Enter) | Scheduling check uptime with an interval of 15s
2022-11-06 11:21:47 UTC | CORE | INFO | (pkg/autodiscovery/autoconfig.go:200 in LoadAndRun) | Started config provider "file"
2022-11-06 11:21:47 UTC | CORE | INFO | (pkg/autodiscovery/autoconfig.go:200 in LoadAndRun) | Started config provider "kubernetes-container-allinone"
starting system-probe
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (pkg/util/log/log.go:588 in func1) | runtime: final GOMAXPROCS value is: 20
2022-11-06 11:21:47 UTC | SYS-PROBE | WARN | (pkg/util/log/log.go:616 in func1) | Error loading config: open /etc/datadog-agent/system-probe.yaml: no such file or directory
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (pkg/util/log/log.go:588 in func1) | Features detected from environment: docker
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (pkg/util/log/log.go:583 in func1) | process_config.enabled is deprecated, use process_config.container_collection.enabled and process_config.process_collection.enabled instead, see https://docs.datadoghq.com/infrastructure/process#installation for more information
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (pkg/util/log/log.go:588 in func1) | no config exists at /etc/datadog-agent/system-probe.yaml, ignoring...
2022-11-06 11:21:47 UTC | SYS-PROBE | WARN | (pkg/util/log/log.go:616 in func1) | Agent configuration relax permissions constraint on the secret backend cmd, Group can read and exec
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (pkg/util/log/log.go:583 in func1) | network_config not found, but system-probe was enabled, enabling network module by default
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (cmd/system-probe/app/run.go:150 in StartSystemProbe) | running system-probe with version: Agent 7.40.0 - Commit: 86fc8167bc - Serialization version: v5.0.32 - Go version: go1.18.7
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (pkg/network/tracer/utils_linux.go:44 in IsTracerSupportedByOS) | running on platform: ubuntu
2022-11-06 11:21:47 UTC | SYS-PROBE | ERROR | (cmd/system-probe/api/module/loader.go:60 in Register) | new module network_tracer error: system-probe unsupported: Kernel unsupported (5.19.7) - required functions missing: bpf_perf_event_output, bpf_perf_event_read
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (cmd/system-probe/api/module/loader.go:50 in Register) | tcp_queue_length_tracer module disabled
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (cmd/system-probe/api/module/loader.go:50 in Register) | oom_kill_probe module disabled
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (cmd/system-probe/api/module/loader.go:50 in Register) | security_runtime module disabled
2022-11-06 11:21:47 UTC | SYS-PROBE | INFO | (cmd/system-probe/api/module/loader.go:50 in Register) | process module disabled
2022-11-06 11:21:47 UTC | SYS-PROBE | CRITICAL | (cmd/system-probe/app/run.go:207 in StartSystemProbe) | Error while starting api server, exiting: failed to create system probe: no module could be loaded
Error: Error while starting api server, exiting: failed to create system probe: no module could be loaded
system-probe exited with code 1, signal 0, restarting in 2 seconds
❯ seccomp
zsh: command not found: seccomp
❯ grep CONFIG_SECCOMP= /boot/config-$(uname -r)
CONFIG_SECCOMP=y
❯ grep Seccomp /proc/$$/status
Seccomp:	0
Seccomp_filters:	0

  datadoghq:
    container_name: dd-agent
    volumes:
      - '/var/run/docker.sock:/var/run/docker.sock:ro'
      - '/proc/:/host/proc/:ro'
      - '/sys/fs/cgroup/:/host/sys/fs/cgroup:ro'
      - '/sys/kernel/debug:/sys/kernel/debug'
      - '/etc/passwd:/etc/passwd:ro'
    environment:
      - DD_API_KEY=XXXXXXXXXXXXXXXXXXXXXXX
      - DD_PROCESS_AGENT_ENABLED=true
      - DD_SYSTEM_PROBE_ENABLED=true
    image: 'datadog/agent:latest'
    cap_add:
    - SYS_ADMIN
    - SYS_RESOURCE
    - SYS_PTRACE
    - NET_ADMIN
    - NET_BROADCAST
    - NET_RAW
    - IPC_LOCK
    - CHOWN
    security_opt:
    - apparmor:unconfined

Installed bpftrace and libbpf-dev since the required functions bpf_perf_event_output, bpf_perf_event_read seem to be missing. The same output remains in the logs.

Looks like CONFIG_FTRACE is disabled by design on Xanmod. So I'll roll back to the stock kernel.
https://www.kernelconfig.io/config_bpf_events?q=&kernelversion=6.0.7&arch=x86
https://github.com/xanmod/linux/issues/192#issuecomment-912102238


Missing Synapse folder in DP000

Lab Name: Analyze Data in a Data lake with SQL
Task: Query data in files
Step: 06
Issue Type: non-blocking
Description of issue: Drilling down, the DP000-xxxxx resource group contains a synapse workspace with a Data-->Linked-->files container with one Sales folder. It doesn't seem to contain a Synapse container. However, checking the DP500-xxxx resource group's synapse workspace-->Data-->Linked-->files container there is a Synapse folder, but no Sales folder.
Repro steps (please include screenshot if applicable):
From the DP000-xxxx resource group:

From the DP500-xxxx resource group:

Lab profile: 111889


Fix scaling on smaller terminal window sizes

Super small windows just loose the clock entirely

Pushed a fix which mostly addresses this except for very small (now doesn't disappear but goes to the top of the window) or weird layouts such as thin rectangles (only happens in some terminals such as cool-retro-term) (in normal terminals the text goes to the top of the window)

Closing this issue right now as I don't think it's possible to account for that in another way than sending a message to resize your window which is worse in my opinion.

So it seems like Kitty, Alacritty and Black Box work well, it's just cool-retro-term which moves the text off-center for some reason

The first three only go off-center when the scale is an odd number, but as far as I know there is no fix for that.


Save emails using Sheets by Spatie

About
Currently the logic operating with Filesystem is quite cumbersome, and configuring the mailbox path and disk is not straight forward.
Solution
Instead we should utilize spatie/sheets to simplify Mailman, and leave the nitty gritty filesystem logic to Sheets, as it's very well tested and highly configurable.

@niladam I've finished the inital implementation, and it really cleans up things considerably!
I'll run it through some tests and release. 👍


Support Datetime function in select: year, semester, quarter and month

If a column is of type DATE, we should be able to use the following date functions: semester, quarter, year and month.
Example:
var query = Query.from(this.storeName)
          .select(List.of("YEAR(date_sales)"), List.of(CountMeasure.INSTANCE))
          .build();
this.executor.execute(query);

Currently, it throws:
io.squashql.query.exception.FieldNotFoundException: Cannot find field with name Year(date_sales)

	at io.squashql.query.database.AQueryEngine.lambda$createFieldSupplier$1(AQueryEngine.java:70)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197)
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625)
	at java.base/java.util.stream.Streams$ConcatSpliterator.forEachRemaining(Streams.java:735)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682)
	at io.squashql.query.QueryExecutor.createQueryScope(QueryExecutor.java:268)
	at io.squashql.query.QueryExecutor.execute(QueryExecutor.java:150)
	at io.squashql.query.QueryExecutor.execute(QueryExecutor.java:129)

Because select only supports column table names.
To reproduce, use this test in ATestPeriodComparison
  @Test
  void test() {
    var query = Query.from(this.storeName)
            .select(List.of("YEAR(date_sales)"), List.of(CountMeasure.INSTANCE))
            .build();
    Table finalTable = this.executor.execute(query);
    finalTable.show();
  }

Maybe we should not let the user write the string himself but provide a function to do this. That function would generate a string with special characters that could be identified by SquashQL to understand the user wants to use an SQL function and we could adapt it to the underlying database if needed. For instance:
  static String year(String column) {
    return "__YEAR__(column)";
  }

...

select(List.of(year("date_sales")), List.of(CountMeasure.INSTANCE))

By doing so, the type of the column can be known (integer in case of year, semester, quarter, month).
We are already using something similar to identify grouping functions. See SqlUtils and  static final Pattern GROUPING_PATTERN = Pattern.compile("___grouping___(.*)___");

Fixed by #182


Added Jenkins Access logging

As per https://wiki.jenkins.io/display/JENKINS/Access+Logging

Configuration added and will be enabled on next restart


Naming inconsistency in PayloadFactory mediator

Description:
When adding a PayloadFactory mediator, in the mediators list it is indicated Payload. But after it get adds to the design pane it is indicated as PayloadFactory as indicated below.

Suggested Labels:

Suggested Assignees:

Affected Product Version:
v0.9.24072507
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

The latest release v1.0.5 has resolved the issue. Please reopen if the problem persists.


lombok 1.16.20 installer.  Unable to set eclipse location

lombok 1.16.20 installer, unable to select eclipse locations.  Resolution is 3000 x 2000 (Microsoft Surface book 2).
I'm unable to select the eclipse installation dir.  Please see attached screenshots.

@rspilker Thanks for the response. I'm not sure of the cause,  but this seems to be java version related.  If I use java 9, the UI is displayed properly.   However, if I just java 8, I get the behavior that I initially described.
Apologies for not seeing this earlier, but I just forked the repo and had to install java 9 to get the build to work and then when I tried the jar I built it worked, but if I switch back to java 8 with the same jar I get the bad layout.

For comparison, I ran 'java -jar lombok.jar' with both java 8 and java 9.  The upper windows is java 8 and the lower window is java 9.

Just found this change in java 9.  JEP 263: HiDPI Graphics on Windows and Linux
This might explain why the installer is sized properly in java 9 and not java 8.

Can someone who encounters this problem try java -Dsun.java2d.uiScale=2.5 -jar lombok.jar?

Just tried java -Dsun.java2d.uiScale=2.5 -jar lombok.jar. No change in behavior.

However, I found this link which has an answer about changing the Compatability settings of the java.exe and this did work.

The more I've read, this just might not be fixable with java 8 and current swing based installer.

Thanks for the research!

Hi just ran this:
$ java -jar lombok-1.18.6.jar
Got this.
Segmentation fault

Wow, cool. That sounds like a JRE bug to me. What JRE did you use?

Actually, I was running the command in the git bash shell and probably did set it up correctly. I ran the same command using PowerShell (windows 10). I didn't get the Segmentation fault error. But instead, the UI shows up, then disappears quickly. I tried both 1.18.6 and 1.18 version of the jar files. Any suggestion?
My java version is:
java version "1.8.0_121"

Well, I solved the problem. I have to install the latest JDK. Now STS is working with Lombok. Thanks.


Wildcard query on store for autocompletion

I need to store between 7k and 8k tags in the database, which the user can access through autocompletion from the frontend.
Is there a straightforward way through the api in which wildcard queries can be achieved?
Example: 'orb%'

Looks like something could be done using a substring, as shown in the docs:
const all = db.query((doc) => doc.followers >= 500)

Closing but I'm going to leave a reference to https://github.com/orbitdb/orbitdb/issues/438.
Perhaps this can be re-opened or addressed in a later version of OrbitDB using a new issue.


Release 0.2.0

@gradle-clojure/developers After a long delay, I think we're ready to push out 0.2.0 (see release notes for the rc). This is focused on some refactoring to match up with current Gradle conventions.
This release will also be published to JCenter (needed due to having a separate JAR that's used for tasks but not on the plugin classpath). I'm waiting on the final approval from Bintray that we've met their publishing standards.
I'll plan to push this out Thursday, if no one has any objections before then.

Here's the full diff for this release.

Thanks for all the hard work on this, and I'm sorry I haven't had time to participate more. I'll see if I can get some time this week to try this on the Cursive build.

No worries, @cursive-ide. Glad to have any participation people can make.
Forgot to come back to this last week, like originally planned, but I'll cut 0.2.0 now.


Document new "Pay for service" system

This issue was created on gitlab and then migrated here. Only the original post was migrated, not the comments. Please take a look at the discussions on the original Gitlab issue before commenting here: https://repo.getmonero.org/monero-project/monero-site/-/issues/1014

The new "Pay for service"(RPC pay) system is one of the most interesting features of 0.15, i think would be useful to have some documentation about it. Could be a dedicated dev guide, and addition to the existent dev guides or something else (ence the generic 'docs' label), but IMHO, the feature should be well documented.
More info: https://github.com/monero-project/monero/pull/5357/commits/2899379791b7542e4eb920b5d9d58cf232806937
Feel free to propose how to better add it.

This could be added as a dev guide. At the moment we only have daemon and wallet RPC.

This could be added as a dev guide. At the moment we only have daemon and wallet RPC.


Cuda 11 fix

Cuda 11 compile patch as suggested in issue #250 using sm_53 instead of sm_30 when Cuda 10 and up is detected.

Supports 2.0~ on CUDA 9
Supports 3.0~ on CUDA 10
Supports 5.2~ on CUDA 11
https://arnon.dk/matching-sm-architectures-arch-and-gencode-for-various-nvidia-cards/

 cmake ..
-- Found git, set version to: v5.3.4 (master-57520b2)
-- System is: Linux (Linux)
-- Could NOT find CUDA (missing: CUDA_INCLUDE_DIRS CUDA_CUDART_LIBRARY) (found version "11.4")
-- CUDA not found. disabled.
--
-- Config summary:
-- 	OpenCV: 4.5.2
-- 	OpenCL: 2.2
-- 	CUDA: Not found
-- 	Unicode: TRUE
-- 	Installing models to: /usr/local/share/waifu2x-converter-cpp
-- 	Not building test binaries
-- 	Building for: Unix Makefiles-x86_64
--
-- Configuring done

Fedora 34 / nvidia gtx 1650
nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Sun_Aug_15_21:14:11_PDT_2021
Cuda compilation tools, release 11.4, V11.4.120
Build cuda_11.4.r11.4/compiler.30300941_0


feat: useClickOutside

I'm not sure is this best way to do it but seems fine for me.
Alternatively we can provide a useClickOutsideFn utility for be able to use it with @click="...". For example:
const clickOutside = useClickOutsideFn(handler)

<div @click="clickOutside">test</div>

Thanks a lot! Well done and nice demo 👍

Thanks a lot! Well done and nice demo 👍

Thank you too. Learned a lot 😊


Fix typo

The default use merge constant was mis-spelled

@mrambacher  - Please update the history file as well. Thanks

Thanks for that, but this type is being fixed as part of a related PR.


Issue with delete state

Hi I was getting an error when I try to delete the state after I retrieved it, Here is the error message I got:
Connected to pydev debugger (build 173.4674.37)
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
{'subject': 'World', 'agenda_group': 'pre_app_change', 'domain_id': '1', 'sid': 0}
{'sid': '0', 'result': '1'}
<IP_ADDRESS> - - [24/Mar/2018 00:03:17] "POST /rule_entry HTTP/1.1" 200 -
<IP_ADDRESS> - - [24/Mar/2018 00:03:18] "POST /rule_entry HTTP/1.1" 200 -
{'subject': 'World', 'agenda_group': 'pre_app_change', 'domain_id': '13', 'sid': 21}
{'sid': '21', 'result': '2'}
{'subject': 'World', 'agenda_group': 'pre_app_change', 'domain_id': '1', 'sid': 0}
[2018-03-24 00:03:22,088] ERROR in app: Exception on /rule_entry [POST]
Traceback (most recent call last):
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\flask\app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\flask\app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\flask\app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\flask\_compat.py", line 33, in reraise
    raise value
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\flask\app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\flask\app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "C:/Users/allen/PycharmProjects/backend/app.py", line 17, in rule_entry
    result = host.get_state(message["agenda_group"], message["sid"])
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\durable\engine.py", line 971, in get_state
    return self.get_ruleset(ruleset_name).get_state(sid)
  File "C:\Users\allen\PycharmProjects\backend\venv\lib\site-packages\durable\engine.py", line 505, in get_state
    return json.loads(rules.get_state(self._handle, sid))
rules.error: Could not get state, error code: 305
<IP_ADDRESS> - - [24/Mar/2018 00:03:22] "POST /rule_entry HTTP/1.1" 500 -
start action base exception type <class 'TypeError'>, value the JSON object must be str, bytes or bytearray, not 'NoneType', traceback ['  File "C:\\*\\PycharmProjects\\backend\\venv\\lib\\site-packages\\durable\\engine.py", line 573, in dispatch\n    state = json.loads(result[0])\n', '  File "C:\\Python\\Python36\\lib\\json\\__init__.py", line 348, in loads\n    \'not {!r}\'.format(s.__class__.__name__))\n']
Exiting the JSON object must be str, bytes or bytearray, not 'NoneType'

Process finished with exit code 1

and I have to restart redis to enable the server
Here is the test code I've written that will reproduce this problem
trigger
@app.route('/rule_entry', methods=["POST", ])
def rule_entry():
    message = json.loads(request.stream.read())
    print(message)
    host.post(message["agenda_group"], message)
    # load data
    result = host.get_state(message["agenda_group"], message["sid"])
    print(result)
    host.delete_state(message["agenda_group"], message["sid"])
    return make_response(json.dumps({'outcome': str(result)}))

defination
with ruleset("pre_app_change"):
    @when_all(m.subject != "")
    def say_hello(c):
        # consequent
        print('Hello {0}'.format(c.m.subject))
        c.s.result = c.m.domain_id

and btw I was running durable_rule on flask
and here is the code I write to post the json
import json

import requests

json_test = {
    'subject': 'World',
    'agenda_group': "pre_app_change",
    'domain_id': '1',
    'sid': 0
}

json_test2 = {
    'subject': 'World',
    'agenda_group': "pre_app_change",
    'domain_id': '13',
    "sid": 21
}

if __name__ == '__main__':
    result = requests.post('http://localhost:5000/rule_entry', json=json_test)
    print(json.loads(result.text))

    result1 = requests.post('http://localhost:5000/rule_entry', json=json_test2)
    print(json.loads(result1.text))

Hi, thanks for reporting the issue. From the error message (and the 305 code), it seems the get_state call is being executed without the state for the "SID" being created. Out of curiosity, what is the outcome of the post call before the failed get_state?
print(host.post(message["agenda_group"], message))

Thanks for your quick reply, the result I get before getstate is 0, int type,

Thanks a lot for reporting the issue. I fixed the bug an uploaded a new version (the engine did not remove the local cache entry after deleting the state in redis). Please use version 0.33.124.

Thanks, I will test it when I get home, I will close this issue after I tried it.

Thanks, it works fine now


Create Data Exploration notebook

The first step in the XBT is to run some analysis to understand the makeup of the data set so we can make some appropriate choices when it comes to classification algorithms and verification metrics.

First draft of notebook created. Next step is to show some plot of the numbers of different types in each year.

I have now put all the plots and analysis into the notebook that I intend to at this stage, so I am closing the notebook.


Terraform changes to add more node size and resolve out of IP errors

As a result of adding more CDK pods per namespace, we are facing  auto scaling issues with node pools running out of IPs. As a result, there is a need to update terraform to fix autoscaling issues so that it can handle additional workloads from the increase in CDK pods per namespace

Code changes completed and works
https://github.com/ForgeCloud/sbat-infra/pull/237


[sqlite3] Fix typo for the Windows build

Describe the pull request

What does your PR fix?
Fixes a small (but critical?) typo in sqlite build script

Which triplets are supported/not supported?
Yes

Please modify the value of port-version in vcpkg.json as 2.

Please modify the value of port-version in vcpkg.json as 2.

Done

How is the macro used? Would it make sense to burn it into the installed header?

How is the macro used? Would it make sense to burn it into the installed header?

I don't know, just here to fix the typo


configuring multiple gateway objects and virtualservice objects in different namespaces not working

i am configuring multiple gateway, virutalservice objects with different hostnames in different namespaces. but the requests are getting routed only to first one.
namespace: k8s
hosts: k8s.example.com
gateway: k8s-gateway
virtualservice: k8s-virtualservice(referring to k8s-gateway)
namespace: k8s1
hosts: k8s1.example.com
gateway: k8s1-gateway
virtualservice: k8s1-virtualservice(referring to k8s1-gateway)
when i invoke url http://k8s1.example.com/appname the request are getting routed only k8s.example.com instead of k8s1.example.com gateway and virtualservice
we have only one istio-ingressgateway pod and istio-ingressgateway service exposed.
i had referred istio documentation but didnt help. do i need to create multiple ingressgateway pod and multiple ingressgateway services under istio-system namespace.  Please some on this erquirement

Can you please provide  the full config you have, not just a tiny snippet, so we can attempt to reproduce? and a config dump from ingress: https://github.com/istio/istio/wiki/Troubleshooting-Istio

its issue with host name based routing i saw a thread where it was mentioned it was fixed in istio 1.6.  for 1.5.2 version we have a workaround sending request host header from browser using browser plugin with hostname then the requests are getting routed properly appropriate gateway for multiple namespace.  unfortunately when i use 1.6 version external services arent working.  i had raised request #24575.

Hi @howardjohn
The issue i am referring in this thread is the same issue where other people are facing and same is discussed in #8807.  In #8807 thread you had mentioned that the issue is fixed in istio 1.6.  can you please confirm whether its fixed in 1.6.x of istio.  also you had mentioned that its not fully fixed.  so its confusing.  i am not able test hostname based routing feature using 1.6 with our application as i am facing external service issue mentioned in #24575.

Hi @howardjohn
External service issue we had experienced in 1.5.2 version also when we enable mTLS option.  we use external service to connect to database residing outside the mesh and cluster and when we enable mtls  get “java.sql.SQLException: Cannot create PoolableConnectionFactory”  error.  When we disable mTLS option the applications are able to connect to the db.

We cannot reproduce this unless you give the full configuration as requested.

Closing as we do not have enough information to proceed


Create REST API endpoint to get recently items in JSON format

I want create a PR for this

@CharlyJazz PR is greatly welcome


ModuleNotFoundError: No module named 'gradio' Error when running Docker with AutoTrain

I am getting ModuleNotFoundError: No module named 'gradio' Error when running Docker with AutoTrain error when ruuning the example from https://huggingface.co/blog/Llama2-for-non-engineers.
WARNING: The NVIDIA Driver was not detected.  GPU functionality will not be available.
Use the NVIDIA Container Toolkit to start this container with GPU support; see
https://docs.nvidia.com/datacenter/cloud-native/ .

** DEPRECATION NOTICE! **

THIS IMAGE IS DEPRECATED and is scheduled for DELETION.
https://gitlab.com/nvidia/container-images/cuda/blob/master/doc/support-policy.md

INFO    Installing latest xformers
INFO    Successfully installed latest xformers
Traceback (most recent call last):
File "/app/env/bin/autotrain", line 33, in
sys.exit(load_entry_point('autotrain-advanced', 'console_scripts', 'autotrain')())
File "/app/src/autotrain/cli/autotrain.py", line 48, in main
command.run()
File "/app/src/autotrain/cli/run_app.py", line 57, in run
from ..apps.main import main
File "/app/src/autotrain/apps/main.py", line 1, in
import gradio as gr
ModuleNotFoundError: No module named 'gradio'
ERROR conda.cli.main_run:execute(49): conda run /bin/bash -c autotrain setup && autotrain app --port 7860 failed. (See above for error)

please factory reset. this issue is resolved now

thanks, all good. Issue resolved.


Phonon reverb can get cut off

Repro:
Built lovr d2a9e23
Ran showcase/Break sample from https://github.com/mcclure/lovr-docs/tree/break-phonon-test
Phonon spatializer Oculus driver+headset
Expected:
This test is Break but 1. In a small setGeometry box slightly larger than the play field 2. All sounds are replaced with solid saw waves (no attack no decay tails).
The sounds that play when the puck hits the walls, bricks or paddles are spatial (have all effects enabled). The sounds that play when you lose or when a level is starting are not spatial (no effects).
Play by tilting right handset side to side.
Expected behavior was that there would be spatialization and some nice audible echo on the five notes that play when the puck bounces off walls or game pieces.
Observed:

Spatialization is weak, but maybe it's physically accurate actually so that's okay.
No tails on the echo.
The end of level "BRNUH-UH" and "BEEP BEEP BEEP" sounds sound echoey. But they don't have reverb effect applied on them

Current status: "I know how to fix this but ugh I have to write a linked list"


refs: #4

Fields "unhandled" and "downtime" were wrong

New pull request with these corrections and new corrections is open
I close this pull request


unsupported type interface {}

Hello! 👋
We have data of the form:
type NotificationData struct {
	Metadata map[string]interface{}
}

// Notification model
type Notification struct {
	Data NotificationData
}

And it breaks with:
unsupported type interface {}

Is this something you would consider supporting?
Can try and get a repeatable example if you need!

Hi, unfortunately no, this library is by design not supporting untyped data types, as it doesn't embed type information in.
You can either write a custom marshaler for this (i.e implement BinaryMarshal interface) or use some other package.

@kelindar Thank you for getting back to me! :)


Add release version to manpage

What's in this pull request?
This PR passes a value to the --release flag of pod2man. This value is shown on the bottom left of the manpage, and it is common to put project name and version there.
Passing a value also resolves an incompatibility with newer versions of pod2man, shipped on Arch Linux and Fedora for example.
Resolved bug number: (SR-2099)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Can you add a test that pod2man works. I do not care about the output. Just that it produces reasonable output (i.e. a smoke test).

Where should I add this test? If it is of any help, pod2man will already exit with a non-zero status and make the build fail when it encounters errors.

Basically here is why I brought this up: I do not think that most people ever build the manpages unless they are trying to package something. That means that issues like this lie unfixed if we people on new OSes and then people (like you) run into them (which is unfortunate).
Really, the incompatibility should have been caught immediately. Perhaps by cmake. That being said, I think this is a small enough change that it is probably not interesting.
On the other hand, from reading the manpage isn't this supposed to be --release=""?

Good catch with the quals. Both seem to work, but I'll update it with the documented way.
The manpages are built by default, unless the user explicitly disables them (setting SWIFT_INCLUDE_DOCS to false), so I think this will be caught easily. I have known about this since I first built swift on Arch Linux, only found the time to track down the corresponding issue and create a PR for it just now.

Ok. SGTM.

@swift-ci Please smoke test and merge


Implementation of FurnaceConsumFuelEvent

Actually i can't test this because furnace.getData throw an abstract method error,  so in the state is more like a WIP don't hesitate to provide advice and require modifications.

You may want to format your code a bit. As it stands, it is extremely difficult to read.

I have checked your guidelines here, https://docs.spongepowered.org/en/contributing/guidelines.html and all seem okay. Maybe you want more vertical space inside my method ? If not please hilight an issue.

The spacing is inconsistent. You have an extra space at line 96, and are omitting spaces around operators and curly brackets everywhere. I'm not sure if I'm missing something here, but why are you casting to Object before casting to the correct class in lines 84 and 85?

Ok, I see. Will edit that later.
The (Object) is here for compilation purpose because I cast a net.micraft ItemStack to Sponge ItemStack or TileEntityFurnace to Furnace. Without I can't compile.

Try using the auto formatter from your Ide.
In the extra folder (from the API) there are formatter files you can import and use.

This event isn't correct anymore due to new events system.


A puzzle about features returned by EfficientNet in efficientnet.py

According to function EfficientNet defined in efficientnet.py, the features returned by it has five elements corresponding to five different stages of efficientnet respectively. Why is the first element  not the output of the first convolution operation followed by BN and swish?

I have no idea. It's consistent with the official implementation of EfficientNet, you may ask them.

I have no idea. It's consistent with the official implementation of EfficientNet, you may ask them.

Ok. Thanks!


TypeError: Input 'y' of 'Mul' Op has type float32 that does not match type int32 of argument 'x'

i was passes embeddings (enrollment, verification) checked similarities between these.
throws this issue.
Traceback (most recent call last):
   File "just_see.py", line 178, in speaker_verification
    similarity_matrix = similarity(embedded=verif_embed, w=1, b=0, N=N, M=M, P=proj, center=enroll_embed)
  File "just_see.py", line 105, in similarity
    S = tf.abs(w)*S+b   # rescaling
  File "/env/lib/python3.6/site-packages/tensorflow/python/ops/math_ops.py", line 866, in binary_op_wrapper
    return func(x, y, name=name)
  File "/env/lib/python3.6/site-packages/tensorflow/python/ops/math_ops.py", line 1131, in _mul_dispatch
    return gen_math_ops.mul(x, y, name=name)
  File "/env/lib/python3.6/site-packages/tensorflow/python/ops/gen_math_ops.py", line 5042, in mul
    "Mul", x=x, y=y, name=name)
  File "/env/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 546, in _apply_op_helper
    inferred_from[input_arg.type_attr]))
TypeError: Input 'y' of 'Mul' Op has type float32 that does not match type int32 of argument 'x'.

@MuruganR96
Try w=1.0, b=1.0 should resolve the problem.

@xtluo-ai  sir Thanks. it is my mistake.resolved


add info to readme about scraping and github actions

write more details about scraping and github actions

reference
https://github.com/owid/covid-19-data/blob/master/scripts/docs/vaccinations/README.md


Client: Optimize default retry strategy

Is your feature request related to a problem? Please describe.
Describe the solution you'd like

make sure all the retry cases in client is reasonable
decide whether to enable the retry policy by default(add an option to control?)

Describe alternatives you've considered
compatibility needs to be considered (the influence to the existing user)
Additional context
https://datatracker.ietf.org/doc/html/rfc7231#section-4.2.2

I want to try this.

Well done!

Thank you for your trust. I will sort out the plan as soon as possible

Hertz client has default retry logic with low threshold. And user doesn't know this normally. So I turn off the default retry logic and user needs to open it manually.

What does retry "cases” mean? In other words, which part of the code should I focus on. I read "pkg/protocol/http1/client go::Do()" .  I don't know if the direction is right

Yes, and and all scenarios where doNonNilReqResp(https://github.com/cloudwego/hertz/blob/develop/pkg/protocol/http1/client.go#L522) returns true/false.
However, the default logic is changed by #102 ,  so here we only need to care about the retry scenarios.

Sort out all possible retry scenarios.
Comb through the configurable options currently available to users.
Determine whether the retries currently provided by the framework are reasonable
Are there any retry scenarios that cannot be covered currently?
etc...

Hi~ I am wondering how it's going? Do you encounter any difficulties? You may join our Feishu Chat Group to discuss more details about this task.


How to add multiple ftp servers

How to add multiple ftp servers in .ftpconfig ? I didn't found anything in the documentation.

If you enable multiple hosts in the settings of the package you can have multiple projects in the same window, which each have their own .ftpconfig file

Oh, i was thinking that the client could upload the same project on multiple servers while adding them in the config.
Would be nice to see

That's currently not a feature of the plugin

Why do not add it ? :+1:

It would probably be alot of work to make sure there are no conflicts. You'd also run into the issue of which remote tree you would display to the user. Or would you have 2 trees at that point?
But currently there are more pressing amends to make to the package

Of course i'm not saying that it's needed right now bot of course, it will be very cool to have.
For the three problem, you could just tab the panel like "Host 1" and "Host 2". (I know it's not only "just", it's a lot of works, i'm a dev too.)
Most of the devs have two different servers, dev and production so it would be cool to have a double upload on file save without having to sync all the files between both.

Nik99,
you can achieve this by commenting out different settings.
For example:
`{
/*  ------------------
REMOTE ON
------------------
*/
"protocol": "ftp",
"host": "",
"port": 21,
"user": "",
"pass": "",
"promptForPass": false,
"remote": "/",
"secure": false,
"secureOptions": null,
"connTimeout": 10000,
"pasvTimeout": 10000,
"keepalive": 10000,
"watch": [""],
"watchTimeout": 500
/*  ------------------
DEV OFF
"protocol": "ftp",
"host": "",
"port": 21,
"user": "",
"pass": "",
"promptForPass": false,
"remote": "/",
"secure": false,
"secureOptions": null,
"connTimeout": 10000,
"pasvTimeout": 10000,
"keepalive": 10000,
"watch": [""],
"watchTimeout": 500
------------------

*/
}`
I do agree with Nik99 that a multiple host connection solution would be a big milestone.
Something for the 'remote' future probably.

Duplicate to #122


integrations: Add webhook code, API endpoint, and tests for GoSquared.

Automated message from Dropbox CLA bot
@540KJ, it looks like you've already signed the Dropbox CLA. Thanks!

@540KJ, the image doesn't match with the rest. Also, shouldn't we also add the HTML which is shown when we click on the image?

Ah, don't mind this

Also, shouldn't we also add the HTML which is shown when we click on the image?

I just saw the task description.

I added a few inline comments.  Do you we know how often GoSquared is going to make webhook calls here?  Also, do we want to send any stats in the message besides just the number of users online?

How often GoSquared is going to make webhook calls depends solely on the user, for they when they receive notifications. e.g. when their site receives an amount of visitors above a certain threshold, the user chooses the threshold.

This is not quite ready to merge yet.  @540KJ I made some comments that may have gotten overlooked when you re-pushed, but they still apply.  Can you look at them?

Merged, thanks, @540KJ !


Inside notebook t81_558_class_05_4_generators. A link to be changed?

There is a link to ImageDatagenerator  in the notebook https://github.com/jeffheaton/app_deep_learning/blob/main/t81_558_class_05_4_generators.ipynb, which still points to a Keras page. However, in the subsequent lines of code, you make use of Pytorch transforms from torchvision for image augmentation examples.

Thanks!


feat(net): Throughput metering metrics

Resolves #140, intended to build on top of #707

@onbjerg would appreciate a quick glance at my current approach here to address #140!
Should I also add a BandwidthMeter a la #707 so that we can gauge / count outbound bandwidth from the sender?

Codecov Report

Merging #726 (cefa445) into main (2da8284) will decrease coverage by 0.14%.
The diff coverage is 34.37%.

@@            Coverage Diff             @@
##             main     #726      +/-   ##
==========================================
- Coverage   73.18%   73.04%   -0.15%
==========================================
  Files         269      270       +1
  Lines       27907    27928      +21
==========================================
- Hits        20425    20400      -25
- Misses       7482     7528      +46

Flag
Coverage Δ

unit-tests
73.04% <34.37%> (-0.15%)
:arrow_down:

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

Impacted Files
Coverage Δ

crates/net/common/src/lib.rs
100.00% <ø> (ø)

crates/net/common/src/metered_sender.rs
0.00% <0.00%> (ø)

crates/net/common/src/bandwidth_meter.rs
94.70% <100.00%> (ø)

crates/transaction-pool/src/test_utils/mock.rs
53.11% <0.00%> (-6.60%)
:arrow_down:

crates/transaction-pool/src/pool/txpool.rs
58.52% <0.00%> (-0.52%)
:arrow_down:

crates/net/network/src/peers/manager.rs
81.70% <0.00%> (-0.20%)
:arrow_down:

crates/net/discv4/src/lib.rs
60.20% <0.00%> (-0.08%)
:arrow_down:

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

Wrapping around the type and tracking makes sense, do we intend to integrate this anywhere in this PR?

Am happy to! Would appreciate any pointers on where this would be the highest signal, maybe in the Senders used in the discv4 crate?

Marked this as blocked on #785

Note to self: add docs to metrics.md after #785 is merged

Unblocked @akirillo we got #785 in now!

making use of dynamic scope now - any suggestions on where to integrate this metric? which senders do we care the most about? @gakonst


[QUERY] Looking for similar issue "TimeoutException continuously happened"

Query/Question
We have Eventhub upgrade and TmeoutException occurred during sending event to Eventhub.
TimeoutException continued until restarting the client IIS and this issue was mitigated after restarting.
This issue happened frequently from August 7th.
Here is code snippet
try
{
    FooTrace.Info("IM00002", $"sendToEventHub {eventData.PartitionKey}");
    eventHubInfo.Client.Send(eventData);
}
catch (MessagingException me)
{
    if (me.IsTransient == false)
    {
        FooTrace.Error("EM00001", me, "sendToEventHub error. " + eventHubInfo.Client.Path);
        throw;
    }
    FooTrace.Info("IM00001", me, "sendToEventHub busy. " + eventHubInfo.Client.Path);
    throw new LdcIfException(LdcStatusCode.ServiceUnavailable, "eventhub is busy.");
}
catch (TimeoutException te)
{
    FooTrace.Info("IM00001", te, "sendToEventHub timeout. " + eventHubInfo.Client.Path);
    // operation timeout default 1 min
    throw new LdcIfException(LdcStatusCode.ServiceUnavailable, "eventhub timeout.");
}

Stack Trace
System.TimeoutException: The operation has timed out. Reference:af2766695b4145db817366c851cbcf92_G5, TrackingId:N/A, Timestamp:8/22/2021 3:55:34 AM ---> System.TimeoutException: The operation has timed out.
   at Microsoft.ServiceBus.Messaging.SingletonManager`1.EndGetInstance(IAsyncResult asyncResult)
   at Microsoft.ServiceBus.Messaging.Amqp.AmqpMessageSender.SendAsyncResult.<>c.<GetAsyncSteps>b__11_1(SendAsyncResult thisPtr, IAsyncResult r)
   at Microsoft.ServiceBus.Messaging.IteratorAsyncResult`1.StepCallback(IAsyncResult result)
   --- End of inner exception stack trace ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.ServiceBus.Common.AsyncResult.End[TAsyncResult](IAsyncResult result)
   at Microsoft.ServiceBus.Messaging.Amqp.AmqpMessageSender.OnEndSendEventData(IAsyncResult result)
   at Microsoft.ServiceBus.Messaging.IteratorAsyncResult`1.StepCallback(IAsyncResult result)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.ServiceBus.Common.AsyncResult.End[TAsyncResult](IAsyncResult result)
   at Microsoft.ServiceBus.Messaging.EventHubClient.Send(EventData data)
   at Panasonic.Pcpf.Ldc.LdcIf.Common.M2MHandler.SendToEventHub(LdcPacket packet, String senderId, String serviceType, String senderType, EventHubClientInfo eventHubInfo)

We use the very old client library and plan to upgrade it . Actually , we are  very aware of that we need to upgrade the SDK version. But we are looking for the confirmation the issue was solved in the latest version before the upgrading.
If you know the similar issue which TimeoutException continuously happen until app restarted and it resolved, please let me know.
Environment:

Name and version of the Library package used: WindowsAzure.ServiceBus 4.1.6
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows Server 2016、 .NET Framework 4.6.2
IDE and version : Visual Studio Professional 2017 Version 15.9.26

//fyi:  @serkantkaraca,  this inquiry is about the WindowsAzure.ServiceBus package.

@kekatahi, that exception you provided looks like a client-side timeout. In fact, issues being recovered after restarting the clients also supports the point. Given that this is a client-side timeout there can be various root-causes here. You should look these things at bare minimum:

High CPU
Low available memory
High thread count
Thread pool deprivation
Network issues

See if the issue reproduces ever again and try to collect a process dump for further investigation. And also, check the VM's health where client is running.

@serkantkaraca Thank you for checking.
We've already done VM level investigation and we think this is SDK issue.
We really hope to upgrade the SDK. Before that,
We just wanted to know that are there any similar issue knowledge in SDK team not the trouble shooting step.

It is always recommended to stay up to date with the latest version of an SDK. There are numerous fixes and improvements in the client code past 3 years and you well may be hitting one of those. Please upgrade to the latest release at https://www.nuget.org/packages/WindowsAzure.ServiceBus/6.2.0

Thank you for checking . We will upgrade the SDK.


[OpenAI] Regression after service side renames "prompt_annotation" to "prompt_filter_results"

For Completions Endpoint,
After the service side renames "prompt_annotation" to "prompt_filter_results".
For Completion API,  we don't have prompt filter results value anymore.
// completions.getPromptFilterResults().get(0).getContentFilterResults()
Also, we could have "length" as the finish reason in the streaming completion API.

No longer needed. This PR already found another way to get around it: https://github.com/Azure/azure-sdk-for-java/pull/36908


Add the Grafana dashboard

What problem does the new feature solve?
We should add and maintain the Granfana dashboard in the community for better observability. By using the dashboard, we can know the health status of greptimedb(standalone or cluster). According to the metrics, we also can use it to create the alert rules.
What does the feature do?
In my opinion, the dashboard should contain:

The greptimedb overview, including version/cluster size/mode etc.
The important metrics of different services(frontend/meta/datanode)
Maintain the dashboard as code
Add the CI to submit to Grafana Dashboard, and the greptimedb user can just import the dashboard by dashboard ID;

Implementation challenges
No response

Close for duplicated issue: https://github.com/GreptimeTeam/greptimedb/issues/3262.