Include additional names of events for debug output.

Current style looks easier to read than the macro to me, and it's greppable. Why do we want this?

Less typing wins readiness and greppability? I'm not so sure about that. And it obviously requires more brain resource to see the text-level patterns, and it would require us to understand the macro when we add new ones and changing the names. Why don't you apply the patterns to the exceptions?

Based on the reasons I mentioned I personally find the use of macros here make it more concise and readable to me.
It's correct that token concatenation prevents finding all use of an identifier using text search.  I'm not sure that's such an overriding concern.  I would hope some of us have better browsing tools (although I don't).  But we have the option of a middle-ground approach, macros to avoid some of the repetition, but no use of token concatenation.

But we have the option of a middle-ground approach, macros to avoid some of the repetition, but no use of token concatenation.

Yeh, that would be a compromise. I'm still not sure I can accept it, but sounds like more acceptable than current one.
Another idea would be using something @SolidWallOfCode made recently. I don't remember which PR it was, but I thought we can use it for DebugNames, and I found something I don't like very much, IIRC.

For #4222, the code size difference depends on how many times it is used. For the switch approach, you need a non-trivial amount of code for each enumeration. With Lexicon, you have one implementation and when used it's just a variable initialization. Given the number of different enumerations we have, Lexicon would be by far less overall code. It does require repeating the enumeration values, but the syntax is very C++ standard.
For the YAML schema work, the code is generated so repeating enumerations is a negligible cost.

[approve ci autest]

I think perhaps the naysayers should contrast the feeling that the code as it is not hard to maintain, with the reality that it's been allowed to sit and rot (not maintained).

[approve ci autest]

I can't reproduce the autest failure in my MacOS docker container.

[approve ci autest]

What part was not maintained? I'd say it haven't been modified because nobody thought it needs to be modified.
Frankly speaking, adding / modifying two lines is much easier than understanding the macro and following the enforced rules to me. The macro might be great if everybody who maintain this function was familiar with the macro, like Debug() macro. But I'm not going to memorize a macro just for this.
Because the names would not be greppable, less people would notice that they actually need to maintain this function.

@maskit See lines 216 - 278 in HttpDebugNames.cc.  "unknown event" was being returned for all of these events.  grep cannot find missing text.
The macros are all defined (and undefined) within the switch statement where they are used.
Whenever you have to manually repeat the same information when coding, and an error will not be caught by the compiler, there is a risk of an error in the delivered code.  So a strategy that avoids a lot of repetition of information will almost certainly reduce errors in the delivered code.

I think @maskit's point is that if he wants to update this, he has to understand the macros and convert from the actual event name he has to the "shortened" form based on the macros. Given how rarely this will be done, I think the least error prone way is cut and paste the full explicit names.
If we were serious about this, we'd have a single YAML file with all this data in it and generate the actual C++ code.

See lines 216 - 278 in HttpDebugNames.cc. "unknown event" was being returned for all of these events. grep cannot find missing text.

At least I haven't needed texts for those and I would only add what I need. grep cannot find missing text, but that part is not addressed even with your macro. You just added everything we have now.
@SolidWallOfCode have already clarified my point with an example of adding case, but it's not the only case. If I had to rename or delete an event, I'd grep the event name to find places I need to edit, and I may do the edit with sed command if there are many places. But this doesn't work if the event name was splitted.

This is now dependent on https://github.com/apache/trafficserver/pull/4596

[approve ci autest]

[approve ci autest]


Add a right click option in the listing to "run to here" while debugging

Is your feature request related to a problem? Please describe.
It can happen that on some remote targets where unwinding the stack becomes difficult, step over in GDB can be erratic. It might not properly detect that you are on a function call and decides to step into instead. While I found some ways to combat this (such as making a thing to allow to import symbols from ghidra to gdb), it's not perfect and it can lead to inconveniences when trying to step over function calls. I almost always end up having to manually add a breakpoint under the call, run, let it break and remove the breakpoint after. This works, but it gets very inconvenient.
Describe the solution you'd like
Simply add a right click option that says "run to here" which would do these steps automatically. Many existing debuggers have that features, but here, it could be used to mitigate the limitations certain targets has that makes debugging inconvenient. It would simply add a temporary breakpoint that would get removed once hit. I think there might even be a way to tell GDB to do this via a command, but I am not sure.
Describe alternatives you've considered
Well I already tried them, but my idea was to tell GDB symbols like functions delimiters because I found that this helps. However I realised 2 problems with this: it's cumbersome to constantly sync the symbols bmanually and it's also not perfect; GDB still can sometime not step over properly. My realisation is that all I want is the convenience of "go through this function" which could be solved by a right click menu option.
Additional context
The target I have this particular issue is a GameBoy Advance game running on mgba, a GBA emulator. It turns out even this emulator itself cannot accurately unwind the stack at a precise moment in time without some aggressive tracking while the game runs so this is just an inherent limitation of the target I am working with.

@aldelaro5  Will definitely consider this.  In general, we haven’t included a lot of breakpoint/step options in the GUI, which are very debugger-implementation specific, because they tend to clutter the GUI and tend to be easier to express as console commands.
This might be an exception as it clearly adds functionality that benefits from the static/dynamic interaction.  Will have to figure out how to clarify step-over-to and step-into-to options for those debuggers that support them.

We have an idea like this sort of planned. I had implemented a method wrapper for GDB's advance command, which if I understand correctly, does what you ask. The idea would be that methods which take a single address parameter could be invoked by right-clicking an address and selecting the method/command from the menu.
In the meantime, I might suggest two alternatives. 1) Not so great, but set a breakpoint at the desired address, continue, and then delete or disable the breakpoint. 2) Copy the address and use the Interpreter window to issue the advance [address] command manually.


Fix typos in the names of InterpolationMode APIs in headers

@hughbe,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Block actions with ChannelName executors

Description
Changes proposed in this pull request:

Block actions with ChannelName executors

Testing

Define a executor with ChannelName RBAC policy:
executors:
  k8s-default-tools:
    botkube/helm:
      enabled: true
      config:
        defaultNamespace: "default"
        helmDriver: "secret"
        helmConfigDir: "/tmp/helm/"
        helmCacheDir: "/tmp/helm/.cache"
      context: &default-plugin-context
        defaultNamespace: "default"
        rbac:
          group:
            type: ChannelName

actions:
  'describe-created-resource':
    enabled: true
    displayName: "Describe created resource"
    command: "kubectl describe {{ .Event.Kind | lower }}{{ if .Event.Namespace }} -n {{ .Event.Namespace }}{{ end }} {{ .Event.Name }}"
    bindings:
      sources:
        - k8s-create-events
      executors:
        - k8s-default-tools

Such action config should be blocked by config validator.
Related issue(s)
https://github.com/kubeshop/botkube/issues/1045

Example faulty actions:
2023/05/15 14:53:51 while merging app configuration: found critical validation errors: 4 errors occurred:
	* Key: 'Config.Actions[describe-created-resource].Bindings.k8s-default-tools' Plugin botkube/kubectl has 'ChannelName' RBAC policy. This is not supported for actions.
	* Key: 'Config.Actions[describe-created-resource].Bindings.k8s-default-tools' Plugin botkube/helm has 'ChannelName' RBAC policy. This is not supported for actions.
	* Key: 'Config.Actions[show-logs-on-error].Bindings.k8s-default-tools' Plugin botkube/kubectl has 'ChannelName' RBAC policy. This is not supported for actions.
	* Key: 'Config.Actions[show-logs-on-error].Bindings.k8s-default-tools' Plugin botkube/helm has 'ChannelName' RBAC policy. This is not supported for actions.

As a general comment, so user just saw that message after they use actions with executors with channel RBAC, then what to do next? If they duplicate that without channel RBAC to use with actions, it will be a security concern for them. If we will introduce a support for actions then better to state that maybe? Any valuable link to our existing actions page would be valuable, of course we need to have a scenario for this.


Split getEvolution in UnscentedProcess

The predicted states cannot be accessed. This causes in Orekit the process noise to be evaluated with the wrong prior states.

@mgrutten if you want tackle this, I can give you a hand

Solved by Mark via MR


How can I limit the date to 3 months?

I would like to limit the calendar to 3 months from the current day. How I can do that please? I don't want to show the whole years.
Best regards,

There are dataSource methods
- (NSDate *)minimumDateForCalendar:(FSCalendar *)calendar;
- (NSDate *)maximumDateForCalendar:(FSCalendar *)calendar;

I'm sorry I'm confused I'm doing it in Swift. How I can call the method in the viewController.Swift or am I going to change it from your FSCalender.m?


json_pointer.contains() exception is incorrectly raised

When using contains() with a _json_pointer, if the json object being tested has an array in the path, the _json_pointer is incorrectly being restricted to reference an array by the contents of the object, instead of verifying that the _json_pointer is contained in the provided json object.
The operation of contains() demonstrates the following bug:

contains() must validate if a _json_pointer exists in a json object.
contains() must not validate if a _json_pointer may not exist in a json object.
contains() must never raise an exception if a _json_pointer can not be supported by a json object.

To reproduce:
json test = { { "a", "Ok" }, { "b", { "bad", "exception", "case" } } };
cout << "/a: " << test.contains("/a"_json_pointer) << endl;
cout << "/b/0: " << test.contains("/b/0"_json_pointer) << endl;
cout << "/b/x: " << test.contains("/b/x"_json_pointer) << endl;

Current Behavior:
/a: 1
/b/0: 1
/b/x: **(exception)**

Expected Behavior:
/a: 1
/b/0: 1
/b/x: 0

Compiler: MSVC 2017, GCC
This bug is present in v3.7.3 / master.
Originally submitted as a question, but there has been no discussion for a month: #1942
I believe that RFC 6901 confirms this to be a bug - in that for the contains() operation, the _json_pointer is NOT being EVALUATED at each depth in the path, but the path existence is being tested at each depth. The operation performed is to verify if the pointer is CONTAINED in the object under test. Therefore, contains() pointer path processing should simply stop in this case, and no exception is to be raised.

Background: RFC 6901 on _json_pointer
Section 4:

Implementations will evaluate each reference token against the document's contents and will raise an error condition if it fails to resolve a concrete value for any of the JSON pointer's reference tokens.
For example, if an array is referenced with a non-numeric token, an error condition will be raised.  See Section 7 for details.

Section 7:

In the event of an error condition, evaluation of the JSON Pointer fails to complete.
Error conditions include, but are not limited to:
o  Invalid pointer syntax
o  A pointer that references a nonexistent value
This specification does not define how errors are handled.  An application of JSON Pointer SHOULD specify the impact and handling of each type of error.
For example, some applications might stop pointer processing upon an error, while others may attempt to recover from missing values by inserting default ones.

@RobBeyer Is your example json document correct? I'm not seeing an array there.

Yes. The _json_pointer "/b" references an array, as per the definition in the specification.
JSON as first-class data type
Reproduce Code Sample:
Screen Capture: Example C++ Code in MSVC 2017 Debugger

This specification does not define how errors are handled. An application of JSON Pointer SHOULD specify the impact and handling of each type of error.

The specification does not define how errors are handled.
I think what you want is that the behavior should be same whatever the path is /b/x or others. They should return false if there is an error condition, right?

Well, if you want to try and explain the issue "backwards", then yes?
At the moment, the contains() operation is dependent on the contents of the object under test, instead of verifying that the path MAY be contained in the object.
When /b is an array, then /b/0 is contained in the object, and /b/x is not. The bug is that /b/x should not raise an exception just because /b is an array.
I do not want that invalid _json_pointer paths return false - any invalid path specifications should raise exceptions as/if required.
I want the contains() function to NOT evaluate the path at each depth, as the function should ONLY be checking for existence (contains) not evaluating and raising exceptions if the path can not be achieved with the provided object.

At the moment, the contains() operation is dependent on the contents of the object under test, instead of verifying that the path MAY be contained in the object.

As you said, the contains() should return true if the path can be achieved, otherwise return false. Also, any invalid path specifications should raise exceptions as/if required. But the contains() should catch the exceptions and return false.
Right?

No. That's not it at all.
I do NOT want a try/catch around the contains() function. Illegal (by syntax) json path specifications must still raise exceptions, as expected.
The exception is being raised ONLY because the json object being tested for the contains() path can not evaluate in the object, for the path being tested.
Even more explicit example: (showing json followed by contains())
{ "a" : "hello" }
contains("/a") -> true  (a exists)
contains("/a/0") -> false   (a is not an array)
contains("/a/x") -> false   (a is not an object)
All of these are operating as expected. They return false BECAUSE the path is not CONTAINED in the object.
{ "a" : ["hello", "there"] }
contains("/a") -> true  (path exists)
contains("/a/0") -> true   (a is an array, and the zero'th item exists)
contains("/a/x") -> false   (a is an array, and only BECAUSE a is an array, contains() raises an exception)
My point is that the json object being tested MUST NOT DETERMINE the operation of contains() - i.e. it should not evaluate the path against the object at any point - only test if the path is CONTAINED in the object.
Basically, contains() must only determine IF (and only if) the given path IS CONTAINED in the json object.
That the path can not be (legally) evaluated by the input json object, is not what contains() is meant to do.

@dota17 - I believe that adding tests similar to the patch I proposed in #1942 would be much more appropriate than catching exception cases after attempting a path evaluation. Your proposed commit is (again) only testing the object against the path, instead of testing if the path is contained. If I wanted this type of an "exception handled" case, I could just eval the path against the object, and catch the exception myself and therefore assume that the path is not contained. Exceptions should be exceptional, not used for something expected.
The #1942 proposed patch does not perform any further path checking, if the next element in the path (already having been identified as an array) can not possibly be contained in the object under test.
A fix using this methodology would be not only more efficient than trying to evaluate, and then catch exceptions, but it can ensure that the path is "not contained" in the object.
For example, in the current implementation of the contains() function - after identifying the current path element as an array, the array size() is checked to ensure the array index in the path is within the array bounds - and then returns false ... without exceptions.

@RobBeyer
Oh, i see.
But we still get an exception for contains("/a/11x") or contains("/001") just to chang that line to:
if (JSON_HEDLEY_UNLIKELY((reference_token < "0") || (reference_token > "9"))).
And we should check each char of the reference_token, because "999" > "9".
If we add more tests to check, it will be a lot of added code. Thus, i push the work to arrayindex() and std::stoi and handle the exceptions.

Fine, I will update and add complete check to deal with reference_token lately.

@dota17
Yeah ... thank you! That's exactly the concern, and you're absolutely right - my "hack" suggestion/change isn't correct, which is why I said "something like" and wanted to discuss this. I realize now that the issue of contains is actually a bit more complicated and involved - contains("/a/11x") is rather evil, while entirely legal.
In those cases std::stoi would need to verify a numeric index is provided as path element before testing, because the JSON spec also states that a /0... path for an array is invalid, but is not invalid when the path is a string. Complicated edge cases. :(

@RobBeyer

the issue of contains is actually a bit more complicated and involved

I had update my code and all testcases passed. Just consider the ABNF in RFC 6901 Sect 4.


FoldersFragment:

Fixed choosing as favorite again crash
Fixed back click pick button not updating

Requires further testing, since I am not very familiar with the methods in this flow.

Looks good

There are unresolved comments, so leaving this PR open

@melvin4u445 thank you, Melvin


validates whole partial

This is a working proof of concept on validating the whole partial, that would need a few tweaks and real world testing.
Some context, tried to run a headless chrome instance to actually validate the partials with a real browser. The differences between es6/require and idk what else made me give up. Also, Im not so convinced that headless chrome will run on all the enviroments this is actually used, and could cause some problems. So been thinking the behaviour of innerHTML must be specified, and possibly jsdom (an already know dependency) could maybe do the work. After some tests, I suspect thats the case.
Looking at an actual issue https://github.com/solidjs/solid/issues/2279
running the following in chrome
Document.parseHTMLUnsafe(`

`).body.innerHTML

yields

running the same through jsdom

So I think it's safe to assume that jsdom may be used for validating complete partials, however, I'm not confident about how this will behave in the wild, it will need real world testing.
Considerations:

main idea is to validate the nesting of markup, for this reason, html comments(possible to try to include them in the future if the approach works, because these are relevant to some rendering stuff), and text content is removed from the validation.
the throw path.buildCodeFrameError it's showing an unrelated path, Im not sure if thats because of the test environment. I tried to show the relevant path but Im not that confident of the best location, so left it there for people that are more familiar.
had to remove all the closing tags for the comparation, it would be better probably if we can have the string of the template with the closing tags, maybe as template.templateWithClosingTags, so the comparation would be more faithful, and also at the same time we could be checking that omitting closing tags won't bring unknown problems.
Im not sure if the location for the validation is the best possibly there's a better location for validating?
the current validation of checking for parent->child, its actually nice, so we dont have to run this code in case of an error
would be nice to provide a diff with the error message

Remaining validation related stuff that I could remember:

textnode cannot be child of TR https://github.com/solidjs/solid/issues/2183
it doesnt validate across partials, but I doubt this could be an issue because appending should kind of work? 🤔

Improvements:

dom-expressions comments are now also validated
template.templateWithClosingTags will accumulate template.template but using the closing tags, so the validation is more faithful and the error message easier to understand
added diffs to the error message

The error diff its supposed to be something like this

I have added this validation to my transform, running it against my full docs site found two things to handle:

text content outside tags wasn't handled, somehow I have a partial that its just text with a random  tag in the middle, the difference between doesn't vs doesn't was messing it up. (now fixed)
tables, this one was kind of expected, a partial
..
 will yield by a browser

..

. I think it will be safe to assume the user will use the
..
 inside a table/tbody, so we can special case the validation by adding the missing bits. I can't think of another tag that behaves like this. In any case I guess we can also special case it.

Improvement:

came to my mind that instead of removing the text nodes we could replace them by an underscore(or anything else), that way we will detect if the browser "moves" text nodes of location, handling the issue "text node cannot be child of TR"

Testing the code from the linked issue:
Document.parseHTMLUnsafe(`

_

`).body.innerHTML

yields
_

This is now handled by the previous commit

The following thread has a weird use case of trying to do something ilegal.
https://discord.com/channels/722131463138705510/1286323202871525456
They are trying to stop the event from bubbling to the other form.
<form>
  <Portal>
    <form>
      <button type="submit" /> submit events
    </form>
  </Portal>
</form>

Nesting forms is disallowed, you can get nested forms by using the form attribute on the input elements.
https://playground.solidjs.com/anonymous/a61c7d21-5eb9-4996-93b5-3deb0c64aefd
Anyway, what caught my attention about this, is that it will create 2 partials, and skip our validation. A possible solution, is to traverse the component ahead of time and accumulate the supposed JSX tags, to do something similar to template.template +=.. but without separating the component in a new partial. This way we can get validation working at least at a component level. It will be some progress over no validation at all, and separates the validation from the transformation. It won't catch mixing identifiers into the JSX, nor validating what Portal returns, but still could be an improvement.

I think that being split into multipl partials is out of your hands. If we can validate even on full partials is a huge improvement. So yeah don't worry about it.

  <TableSomething>

      <button type="submit" /> submit events

  </TableSomething>

This doesn't break anything currently or after this PR right? td is the top of the partial and has no parent so we can't validate?
Otherwise @titoBouzout I think I'm read to merge if you are. I'm assuming that analysis is limited to the one partial and I'm good with that.

td is the top of the partial and has no parent so we can't validate?

The td is the top of the partial yes, but we still validate it, because the validation code handles top level tr/td by adding

 as needed (without modifying the original partial).
--
@ryansolid this pr doesn't modify/change anything, just validates the accumulated string at the end and shows a warning if it doesn't validate. So nothing should break, at least not intentionally.
Summary: validates whole partials, positions of text nodes, tag nesting, positions of comments, validates content of top level tr/td.
It currently shows a warning and doesn't error out, I think that's better for now till we get some feedback about how it behaves. We can quickly tweak if the community finds some edge cases. We can error out at a later point.

Awesome.. ok getting ready to move forward with 1.9 stuff then. Thank you.


[Umbrella] Improve the docs site by simplifying it.

This is a Feature Request
What would you like to be added
Well ... nothing to be added.
We may want to remove some useless gadgets or decorators or components from the screen.
Why is this needed
I'm calling this an umbrella issue because ...

Comments
I'm not enumerating the other "warnings", "notes" we keep adding to a page, but we may want to be careful when adding new components to pages. We may instead strive to simplify the layouts.

/area web-development

Adding some context on the feedback form, we collect some data, but it's not reliable/useable. There's an open issue about this, https://github.com/kubernetes/website/issues/37201. To my knowledge it's not being looked at all.


Notice of Copyright Violation

This is an official notice of Copyright violation against the PIVX developers in which this project has been forked from. The MIT copyright license, and subsequent copyright headers in source files have been modified to explicitly exclude the PIVX developers from all copyrights.
You are hereby notified of an intention to pursue further action if this issue is not acknowledged and formally addressed with a resolution timeframe commitment wherein. Failure to acknowledge and correct the violation will result in a formal DMCA takedown notice filed with GitHub Proper after no less than five business days from the date of this notice.
Explicit acknowledgement and referencing progress to maintain compliance MUST be noted in this issue's comments or referred to in any subsequent pull requests to this repository
File in question:
  penguincore/LICENSE

Hi , we did as your info , we are sorry for any inconvinience caused.
Regards,
CWF


Metadata injection at LogConvert conversion time

Original report by Brian Calder (Bitbucket: brian_r_calder, ).

Rather than waiting until the WIBL file is created from another source, and then re-reading it to edit in metadata (and UUID, ship name, etc.) with “wibl editwibl”, it would be much more efficient (particularly with large numbers of files) to edit in the metadata as the WIBL file is being created.  Therefore, consider adding the ability to ingest the JSON metadata file at conversion time.

Original comment by Brian Calder (Bitbucket: brian_r_calder, ).

Resolved in commit 5108cda


missing cmx on install?

I got an optimising compile through by copying extend_main.cmx into the lib dir.
so I think that's missing from the makefile.

This is no longer a problem with the dune based build.


Feature: Add rewind (or restart) button to Stepper component

Describe the feature in detail (code, mocks, or screenshots encouraged)
Stepper Component has a great functionality, but missing a Rewind or Restart functionality to start everything from the scratch.

The button should be locked on first step and unlockeв at further steps.

On clik it should fire up corresponding  on:rewind  event
May be it would need a own visibility flag to hide/unhide button.
What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response

I could work on a PR if maintainers consider this feature interesting.

There's a couple of Issues for the stepper currently, Chris might want to collect them into one.
I'd also like it if the stepper could be assigned an active step programmatically while it's rendered and not just during initial render.

As Sarenor mentioned, folks have been asking for several changes lately. Let me evaluate how all these will work together and then we'll move forward.

As Sarenor mentioned, folks have been asking for several changes lately. Let me evaluate how all these will work together and then we'll move forward.

Sure, thing. Will wait then.

This issue should now be considered consolidated here: https://github.com/skeletonlabs/skeleton/issues/1780


The static method 'downloadFile' can't be accessed through an instance.

The static method 'downloadFile' can't be accessed through an instance.
Try using the class 'FileDownloader' to access the method

if i use the suggested way

The app crashes by showing fatal exception

Developing for android 12 though

Thank you for the feedback,

Did you try to download the file via the example?
Can you provide me with the file url that you are trying to download?
I am trying to reproduce this bug
Thank you in advance.

the example works just fine
i think it's the problem with threads , their memory might be full
let me try by embedding the example in my code and get back to you

Working on this bug, will keep you updated,
Thank you for reporting the bug.

@abdallah-odeh could you try to reproduce it ?
i think this might be a problem with threads

I'm trying to reproduce it, currently creating an AVD with android 12 to see if it's OS issue

I'm trying to reproduce it, currently creating an AVD with android 12 to see if it's OS issue

the fact that your example worked completely fine in the same app means it might not be an os issue ..
it most probably is an extra workload issue

The issue is with download file instance, when it's finished downloading, the instance that supposed to return the result is null,
It might be a Threads issue!
if you can provide me with a small project that does you "downloading files" logic it will be highly appreciated, in the mean time, I'm trying to reproduce it in my own projects

The issue is temporarily solved, could not reproduce it but found the bug
upgrade the library version to 1.0.12 will solve your issue
but if you provide me with a small project that does the same bug I will grateful
if the issue is solved close the bug or tell me to close it, if it persists please let me know

The issue is temporarily solved, could not reproduce it but found the bug upgrade the library version to 1.0.12 will solve your issue but if you provide me with a small project that does the same bug I will grateful also, can you please provide me with the result of the command flutter --version
if the issue is solved close the bug or tell me to close it, if it persists please let me know

i will try to reproduce it in another app so i can share the code with you ...
meanwhile what was the error that was causing this ? any Idea ?

The onDownloadCompleted call back was called on null,
I really don't know how is that possible as it is assigned once the download task is created,
It actually might be because one the libraries you use in your app, can you please copy and paste the libraries list you are importing in your pubspec.yaml?
and the output of flutter --version will be appreciated.

libraries used :
cupertino_icons: ^1.0.2
get: ^4.6.1
flex_color_scheme: ^5.0.0
firebase_core: ^1.12.0
firebase_auth: ^3.3.7
firebase_storage: ^10.2.8
firebase_messaging: ^11.2.7
firebase_analytics: ^9.1.1
firebase_crashlytics: ^2.6.3
animated_text_kit: ^4.2.1
get_storage: ^2.0.3
cloud_firestore: ^3.1.13
intl: ^0.17.0
cached_network_image: ^3.2.0
simple_animations: ^4.0.1
visibility_detector: ^0.2.2
flashy_tab_bar2: ^0.0.4
dropdown_button2: ^1.4.0
image_downloader: ^0.31.0
share_plus: ^4.0.4
settings_ui: ^2.0.2
url_launcher: ^6.1.0
awesome_notifications: ^0.6.21
flutter_native_splash: ^2.1.6
loading_animation_widget: ^1.1.0+3
video_player: ^2.4.0
inview_notifier_list: ^3.0.0
flutter_downloader: ^1.7.3
path_provider: ^2.0.9
permission_handler: ^9.2.0
new_version: ^0.3.0
wakelock: ^0.6.1+2
flutter_file_downloader: ^1.0.12
stacktrace of error using crashlytics are :
Fatal Exception: java.lang.RuntimeException: Error receiving broadcast Intent { act=android.intent.action.DOWNLOAD_COMPLETE flg=0x10 pkg=com.otft.chomu (has extras) } in com.example.flutter_file_downloader.core.DownloadCompleterBroadcast@96a6b86
at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1689)
at android.app.LoadedApk$ReceiverDispatcher$Args$$ExternalSyntheticLambda0.run(:2)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7870)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)
and
Caused by java.lang.NullPointerException: Attempt to invoke interface method 'void io.flutter.plugin.common.MethodChannel$Result.success(java.lang.Object)' on a null object reference
at com.example.flutter_file_downloader.core.DownloadCompleterBroadcast.onReceive(DownloadCompleterBroadcast.java:44)
at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1679)
at android.app.LoadedApk$ReceiverDispatcher$Args$$ExternalSyntheticLambda0.run(:2)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7870)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)

Is the bug still showing? if so please try to

kill the app
flutter clean
run the app
if it still showing please let me know

could it be caused by   flutter_downloader: ^1.7.3
or   image_downloader: ^0.31.0
i will run the app to with
 flutter_file_downloader: ^1.0.13
to see if it works

so good news , the app doesn't crash now but i'm still getting
I/flutter (31589): Downloading meme I/flutter (31589): https://i.redd.it/tc7hwdbhun091.jpg D/CompatibilityChangeReporter(31589): Compat change id reported: 78294732; UID 10291; state: DISABLED W/System.err(31589): java.lang.NullPointerException: Attempt to invoke interface method 'void io.flutter.plugin.common.MethodChannel$Result.success(java.lang.Object)' on a null object reference W/System.err(31589):    at com.example.flutter_file_downloader.core.DownloadCompleterBroadcast.onReceive(DownloadCompleterBroadcast.java:45) W/System.err(31589):    at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1679) W/System.err(31589):    at android.app.LoadedApk$ReceiverDispatcher$Args$$ExternalSyntheticLambda0.run(Unknown Source:2) W/System.err(31589):    at android.os.Handler.handleCallback(Handler.java:938) W/System.err(31589):    at android.os.Handler.dispatchMessage(Handler.java:99) W/System.err(31589):    at android.os.Looper.loopOnce(Looper.java:201) W/System.err(31589):    at android.os.Looper.loop(Looper.java:288) W/System.err(31589):    at android.app.ActivityThread.main(ActivityThread.java:7870) W/System.err(31589):    at java.lang.reflect.Method.invoke(Native Method) W/System.err(31589):    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548) W/System.err(31589):    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003) I/TransportRuntime.CctTransportBackend(31589): Making request to: https://firebaselogging-pa.googleapis.com/v1/firelog/legacy/batchlog I/TransportRuntime.CctTransportBackend(31589): Status Code: 200
is it normal ?

Yes, you should see the file downloaded to your downloads directory, and the path will be returned to your flutter code, that is the temporary solution for now until if find the real cause of the bug

Tried to import all the libraries you sent and it still not showing the error,
is you project's native code Java or Kotlin?
is your MainActivity empty? if not can you please provide me with code inside it

i think i'm using custom providers and they might be the ones causing the error
`

`

edit: i removed every customisation there was for any type of povider or every other downloading plugin , but it still shows the following problem , though the file is being downloaded in downloads
I/flutter ( 4696): https://i.redd.it/m4s0l7tl8s091.jpg D/CompatibilityChangeReporter( 4696): Compat change id reported: 150939131; UID 10291; state: ENABLED W/System.err( 4696): java.lang.NullPointerException: Attempt to invoke interface method 'void io.flutter.plugin.common.MethodChannel$Result.success(java.lang.Object)' on a null object reference W/System.err( 4696):    at com.example.flutter_file_downloader.core.DownloadCompleterBroadcast.onReceive(DownloadCompleterBroadcast.java:45) W/System.err( 4696):    at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1679) W/System.err( 4696):    at android.app.LoadedApk$ReceiverDispatcher$Args$$ExternalSyntheticLambda0.run(Unknown Source:2) W/System.err( 4696):    at android.os.Handler.handleCallback(Handler.java:938) W/System.err( 4696):    at android.os.Handler.dispatchMessage(Handler.java:99) W/System.err( 4696):    at android.os.Looper.loopOnce(Looper.java:201) W/System.err( 4696):    at android.os.Looper.loop(Looper.java:288) W/System.err( 4696):    at android.app.ActivityThread.main(ActivityThread.java:7870) W/System.err( 4696):    at java.lang.reflect.Method.invoke(Native Method) W/System.err( 4696):    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548) W/System.err( 4696):    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)

Can you please try now on version 1.0.14
I updated some code

Can you please try now on version 1.0.14 I updated some code
sure

### hey good news
It worked the version 1.0.14 worked successfully and now it downloads just fine . 💯 🥇

Great, thank you for contributing
I really appreciate your help in making this library better

Great, thank you for contributing I really appreciate your help in making this library better

My pleasure


Widgets freeze after some time.

If it helps, Silverfish asks for a permission on my phone (Stock 5.1.1), for creating Widgets and accessing their data.

It's not a permissions issue (I'm using an older method to bind widgets that does not require android.permission.BIND_APPWIDGET), otherwise widgets wouldn't be working at all.
To be honest it's an issue that I've seen with apps in general that host widgets.  I'll keep my eye out for ways to improve how widgets are handled, but I'm not sure how to fix this at this time.
Do you have any examples of widgets that freeze?

Ok, but the thing is that Silverfish for example creates the widget, it works, and only after a while asks for the permission. Taskbar acts just like this but instead of asking for the permission tge widgets stop working.
Besides, it's not actually freeze, the widgets seem to not be connected to their app. You can press buttons, you have visual feedback, but nothing happens.
It happens to every widget after like a day (no restart) no matter if Android's internal apps (calendar / power managment) or any other one (e.g. vlc (F-Droid version) or Red Moon).

Apps only need to ask for permission to bind widgets if they did not go through the system widget picker (which is what Taskbar does).  Are you sure that the apps providing the widgets aren't just being killed due to memory constraints?

Maybe, but I actualy never had problems with it - I have always at least 25% free.
Diese Nachricht wurde von meinem Android-Gerät mit K-9/p≡p gesendet.


STM32F411 - problem with serial example

Hello,
I'm trying embassy to evaluate differences with the standard use of stm32hal.
I have difficulties using the serial example.
first :
use embassy_stm32::usb::{Driver, Instance};
usb is not in the root, but usb_otg is, so I've replaced it with usb_otg
second :
bind_interrupts!(struct Irqs {
OTG_FS => usb::InterruptHandlerperipherals::USB_OTG_FS;
});
the interrupt binding is not working and provide the following error : unsafe impl for safe trait and failed to resolve use of undeclared crate or module.
even if I replace with usb_otg, I get the same error.
I've tried different solutions, but can't make the code build, any ideas ?

Depending on your chip, there might be USB_OTG_HS (high speed) instead.

I found one part of the problem.
Mainly because I was working with the crate, and not with repo.
After cloning the repo, it "works" better.
Now my problem is about finding the right configuration to make the USB work, for now I can't pass the configuration, I think it's because of the clock settings. Even if I choose the right Hse, I still can't make it work, but I'll look after that.
if someone has already found a way to make it work with different clock speed than the example.


Review the public API

We've never had the time to go through systematically and review the public API. (Could be done while improving the reference manual.)

I've gone through the list using dir on a basic parameterized object together with Philipp and here are our thoughts.  Of course, this is just expressing only our opinions as relatively veteran param users and this is just for reference for whenever a proper clean-up happens. :-)
Here are the things we don't like the look of and never use ourselves:
set_param, set_default,  print_param_defaults, print_param_values,  initialized, defaults.
The print ones seem particularly nasty to me, initialized is just a boolean (why doesn't it have an underscore in front of it?).  We see how set_param, set_default and defaults  might be useful in some cases but couldn't these be functions available in param instead of methods?
There are two more things that we know are useful but are also rarely used:  inspect_value, force_new_dynamic_value. Again, could these things be functions instead?
I would like to see script_repr renamed to pprint. We understand the need for everything else listed by dir and have ignored everything introduced with an underscore.

I would be happy to make set_param, set_default, print_param_defaults, print_param_values, and defaults into separate functions, if they are ever used, or to delete them if they are not, and to change initialized to _initialized.
I'm not irritated by inspect_value and force_new_dynamic_value, but if they can be made into external functions, then sure.
I would be happier with pprint than script_repr.

See also:

#154
#8

Also see #233

Covered by #233.


EIP-3074: AUTH and AUTHCALL opcodes

Hello!
I was wondering if it's possible to ask for support for EIP-3074 (Account Abstraction) in rEVM. This would be very helpful for testing, seeing as Foundry uses rEVM for their EVM (and Hardhat has plans to migrate afaict), and a big part of 3074 is creating invoker contracts that leverage the opcodes introduced by the EIP.
Is there a way the 3074 opcodes could be added as experimental opcodes?
I'm happy to talk more about why I think this is a worthwhile feature, who's looking for this, etc.
Thanks for reading!

Hi @wschwab,
I have the same stance as in #154 I am okay to spend time including this but only when it gets certain it is going to be included in the Ethereum mainnet.
It is not about if it is worthwhile or who is looking for it, I just don't want to have the maintenance/integration cost of including this, if somebody implements it, it can stand as a standalone branch as proof of concept. Especially as it is unknown when it is going to be included in the mainnet.

I don't know of any concrete plans for Ethereum mainnet. I'm at Polygon, and there is serious talk there, and I do think that if Polygon adds 3074 support and it's successful that in greatly increases the chances of inclusion on Eth mainnet, but that's a few ifs (if Polygon ends up including, we're still experimenting with it, and if that and/or anything else get it included on Eth mainnet).
I feel like we may have had this conversation elsewhere, but what would a standalone branch look like? This may be a question out of your scope, but the primary reason we would want to see this in revm is for Foundry tooling. If we have 3074 in a branch, is there even a way to get Foundry to use a branch of revm?

Hey, can you send me link to a discussion that polygon is having, I am not following polygon dev scene in that detail.
Yeah, inclusion in polygon does not have a lot to do with eth. It doesn't even mean that EIP is going to be the same when included in Eth.
To be perfectly honest, the standalone git branch is just a branch and something I don't need to worry or think about, you can imagine but I don't need more code to maintain that is not essential and this just becomes baggage for me. You can always fork revm and patch foundry and contribute like that, this would help with development when EIP gets included.

I don't have a link per se, but can tell you that we've stood up an initial internal devnet, and have developed an invoker (based on a well-known implementation) along with some scripts (link). We're discussing next steps now.

@wschwab @ZeroEkkusu I'd be open to reviewing the code and helping you guys get it in a workable stage. I think we'd be more reluctant to merging/maintaining that code in Revm main branch as we'd like to keep it on things that are integrated/have a path to being integrated on mainnet.

stale. Will reopen it when time comes

@rakita @wschwab time to reopen this with 3074 passing?


Scrolling to position doesn't always work

I believe this to be a limitation of the layout manager, but when I call "scrollToPosition" on the recycler with any value other than 0, it doesn't seem to want to go to the position.

I don't see how this library could interfere with that at all.
Remember, default LinearLayoutManager (and probably others) just scroll to get the item onto the screen, they do not scroll until the requested position is at the top. For that you'll need to extend the LayoutManager and set the smooth scroller to a custom one. Search for TopSnappedSmoothScroller and you'll probably find a few examples.
scrollToPositionWithOffset might work for you as well.

Closing for now.
Let me know if you have any other information that might point to this being an issue with the library.


Adding ethos dataset clean

I addressed the comments on the PR1318

Thanks !

Thanks as well for your hard work 😊!!

merging since the CI is fixed on master


Fix loading ngTagsInput css and js

Deleted loading redundantly from CDN.
Include ngTagsInput css and js to gulp files building.

I have already did the same loll.

That part is remain unchanged........ https://github.com/kriswme2/teamform-seed/blob/master/src/index.html

  <!-- ngTagsInput -->
  <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/ng-tags-input/3.1.1/ng-tags-input.min.css" />
  <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/ng-tags-input/3.1.1/ng-tags-input.bootstrap.min.css"
/>
...........

  <!-- ngTagsInput -->
  <script src="components/ng-tags-input/ng-tags-input.min.js"></script>
<link rel="stylesheet" type="text/css" href="components/ng-tags-input/ng-tags-input.min.css">

Oh Sorry, you're right

But you didn't use the "--save" flags when bower install

ng-tags-input had been bower installed with --save in this pull request:
https://github.com/kriswme2/teamform-seed/pull/31/files

I just placed the lines in the wrong place before so I did a fix with this pull request. @@


sfdx force:source:push work on Sandbox

Since the last update source:push work on sandbox as deploy.
This is an issue because this command should work only on scratch org

The source:push command will work on any source tracked org. I'm guessing you are working on a developer sandbox? See here. Feel free to open with more info if that isn't the case.


Consider moving to a PEP665 compliant build and packaging tool

Is your feature request related to a problem? Please describe.
We've been using poetry as our package management tool which works well for all our needs. Initially I had wondered what other major projects used for package management (moving from setuptools in the prestans days).
Looking under the hood of projects like fastapi I found they all use hatch.
@brettcannon started the issue Interested in supporting PEP 665? on the repository and it would seem that there are major design variations which would it difficult to make poetry compatible with pep665 (note that there's mention of the work being sponsored)
The python packaging site refers to hatch quite strongly.
Describe the solution you'd like
We should thoroughly read through the packaging guide to ensure we have the right structures and flows.
We should identify if poetry is still a suitable tool for the requirements, and if not what is and why?
We should then produce:

Detailed instructions of the newly chosen tool
Github actions for automating packaging
Updated Dockerfile for building the packages for development and production

Describe alternatives you've considered
NA
Additional context
See the Python packaging guide.

It might be easier for us to prototype this against a smaller projects like our gallagher client.

I believe Poetry is working towards support in Poetry 2.0.
Depending on your needs, Hatch and PDM are probably the next most popular workflow tools.

See Switching to Hatch by @oliverandrich

I found this issue linked via hatch and figured I'd drop in with a shameless plug  (hope you don't mind 😃) for my hatch plugin, hatch-pip-compile.
It enables lockfile support for hatch via pip-compile or uv and has allowed hatch to be a complete one-stop-shop for all my Python project management needs. If you're considering hatch but uncertain about its lack of lockfile support hatch-pip-compile might just be what you're waiting for. Cheers!

I found this issue linked via hatch and figured I'd drop in with a shameless plug (hope you don't mind 😃) for my hatch plugin, hatch-pip-compile.
It enables lockfile support for hatch via pip-compile or uv and has allowed hatch to be a complete one-stop-shop for all my Python project management needs. If you're considering hatch but uncertain about its lack of lockfile support hatch-pip-compile might just be what you're waiting for. Cheers!

Hey @juftin thanks for the shameless plug 😄
I started doing an initial migration to hatch on another project to judge the viability and indeed what hatch-pip-compile made me shelve the idea for a while.
Found your blog post and will study the tool and hopefully it tips us over to move to hatch.
Thanks for taking the time to comment on this issue.


processIsCloseTagExists

If it needs to initialize just set 0
let index = 0; ⭕

let index = startPosition ❌

Add start position to the stream.
const reader = fs.createReadStream(this.filename, { highWaterMark: this.highWaterMark, encoding: this.encoding, start: startPosition  }); ⭕

const reader = fs.createReadStream(this.filename, { highWaterMark: this.highWaterMark, encoding: this.encoding }); ❌

no need for new version


Implement the Before's and After's in SystemUnderTest

Needs to be easy to swap in and out on SystemUnderTest.

Closing with https://github.com/JasperFx/alba/commit/3a19d8aad6f87dce088e5b7c5307553cb9b1e85a


Performance deterioration after upgrading to 0.14.0

Describe the bug
Performance of the KnowledgeBaseClient deteriorated after updating to 0.14.0
To Reproduce

Run the code below.

from griptape.memory.tool import TextToolMemory, BlobToolMemory
from griptape.memory.structure import ConversationMemory, BufferConversationMemory
from griptape.tools import Calculator, WebScraper, FileManager, ToolOutputProcessor, KnowledgeBaseClient
from griptape.structures import Agent
from griptape.rules import Rule, Ruleset
from griptape.loaders import TextLoader, PdfLoader
from griptape.engines import VectorQueryEngine
from griptape.drivers import OpenAiPromptDriver

    prompt_driver = OpenAiPromptDriver(
        temperature=0.1
    )

    text_memory = TextToolMemory()

    memory_browser = ToolOutputProcessor(
        input_memory=[text_memory]
    )

    engine = VectorQueryEngine()
    namespace = "memo"
    artifacts = TextLoader().load(
          "Problem:
  Layer-packer Layer head 4-axis drive is stopping due to an overcurrent error on the drive itself. The head is always stopping as it tries to pick up the product from array plate. It stops right on-top of the product, every time.

  Observations:
  In the faulted/stopped position, the product on the extremities of the layer-packer head is fixed tightly against it, possibly causing a collision. The product in the center of the layer-packer head is loose and not held against head tightly.

  Solution:
  Vacuum cups were moved relative to the bottom of the acetal, causing them to protrude by 2mm more. The layer packer pick height was then adjusted by 1.5mm away from the vials. Overall, the clearance between the head and the vials has increased by 1.5mm, and the contact between the suction cups and head has been increased by 0.5mm"
    )
    engine.vector_store_driver.upsert_text_artifacts(
        {namespace: artifacts}
    )

    kbCLient = KnowledgeBaseClient(
        description="Contains problem, obervations, and solution related to the layer-packer."
        "Use it to answer any related questions.",
        query_engine=engine,
        namespace=namespace
    )

    agent = Agent(
        prompt_driver=prompt_driver,
        memory=ConversationMemory(),
        tools=[memory_browser, kbCLient],
    )

    response = agent.run("what is the problem on layer-packer?")

Unable to retrieve the answer and griptape doesn't try to call on the KnowledgeBaseClient.

Exact same code used to work perfectly fine on 0.13.0 (e.g., able to successfully state the problem, solution, and observations)

Expected behavior
State the problem outlined in the text fed to theTextLoader().
Screenshots

and this also started happening when I migrated to 0.14.0
As seen below, output is empty when KnowledgeBaseClient is used.

I noticed there were some changes to how memory was handled with the ToolOutputProcessor class, and suspected this might be the cause, so I reverted the changes to the previous implementation but it still wasn't able to find the problem on layer-packer.  Reverting griptape and griptape-tools to the previous versions worked. I'll continue to poke around at this and see what else I can find.

@jin167, thanks for reporting those. Currently working on debugging. Already found one actual bug. Additionally, tweaking prompts around memory. Will push an update today.

Awesome, thank you guys! Been building a demo to sell griptape to my colleagues as i've been really impressed with what i was able to do with this lib out of the box haha.

@jin167, I just pushed updates to griptape and griptape-tools with the bug fixe and multiple tweaks to the base prompt. To streamline things even more, you can also do the following:

Don't use TextToolMemory in this example, since the knowledge base tool will return a short answer, which won't be an issue for the prompt size. Also, Griptape now supports automatic memory for some tool activities, so you don't have to explicitly define it.
I would make the knowledge base description a bit more detailed. May be something like This knowledge base contains information about the problem, observations, and solution related to the 'layer-packer.' Always use it to answer all related questions

Let me know if you experience any further issues and thanks for the kind words about Griptape :)

I tried using the latest version of the griptape and it looks like the problem with KnowledgeBaseClient getting bypassed has been fixed but this happened when i removed the memory from the tool:
agent = Agent(
prompt_driver=prompt_driver,
memory=ConversationMemory(),
tools=[kbCLient],
)

and with the memory, i get the following result:
memory_browser = ToolOutputProcessor()
agent = Agent(
prompt_driver=prompt_driver,
memory=ConversationMemory(),
tools=[memory_browser, kbCLient],
)

Perhaps I am doing something wrong with the new implementation of the griptape?

https://github.com/griptape-ai/griptape/pull/118

@jin167 the above pull request was merged and fixes your problem you shouldn't have to use gpt-4.

Hey mgcollie, thanks for looking into the issue!
I had a chance to update the griptape this afternoon and i ran into the following issues:
with the tool memory, it says it will return the summary but never does:
memory_browser = ToolOutputProcessor()
agent = Agent(
prompt_driver=prompt_driver,
memory=ConversationMemory(),
tools=[memory_browser, kbCLient],
)

Also, without memory following happened. It seems to ignore the content of the KnowledgeBaseClient:
agent = Agent(
prompt_driver=prompt_driver,
memory=ConversationMemory(),
tools=[kbCLient],
)

Hey @jin167, do you have access to gpt-4? We just updated the OpenAI prompt driver to use it by default, so that Griptape agents can function correctly 100% of the time. gpt-3.5 issues are ongoing and we are still working on improving Griptape performance with those. One way to temporarily fix it is to disable tool memory in your agent with:
Agent(
    tool_memory=None
)

Let me know if any of those resolutions work for you!

Hi @vasinov, sorry for late response I was away over the weekend.
with the agent tool memory set to none, it started working again:

Agent(
tool_memory=None
)
Although it looks like the issue with agent losing memory is still present:

Yep, it's unfortunately still happening with gpt-3.5-turbo, https://github.com/griptape-ai/griptape/pull/149 should address it once it's complete.


add gemini pro experimental

the option is missing in https://www.chatbotui.com/

#1863
This is solved in my PR


add assign/unassign commands

Make it possible to manage split assignments via the CLI. Current thinking:

assign split_name --variant variant creates or updates an assignment in ~/.testtrack/assignments
unassign split_name removes the split from ~/.testtrack/assignments
unassign --all resets the ~/.testtrack/assignments file (deletes or empties? no strong feels if we're good at auto-recreating it)

/cc @samandmoore


System Information: Memory bugs + Development questions

Hi, im using 3.0.1902 (3fd1ee8b) on Windows 8.1 x64 and there seem to be some issues with missing data via System Information > Memory for Paged/Non-Paged Pool 'Limit', Physical Memory 'Kernel WS' and Memory lists 'Modified no-write'. I have the Windows SDK installed with Process Hacker configured to use those symbols correctly.

Are there any plans to add monitoring for APC/DPC procedure calls and IRP/NMI interrupt metrics under the System process at any point? That would be really handy.
Also are there any plugins available / under development which would allow graphing of hardware sensor data from external applications via WMI or Shared Memory (I was thinking AIDA64).
Thanks for all the hard work. I use the program everyday :)

The name for these variables changed. I can't change that code without breaking Win7 support so we need to wait until more people stop using Windows 7 (which is an issue since approx 50% of downloads are still for Windows 7).

Thanks for the information. I guess it's not possible to IFDEF this as a workaround?

I have some patches but haven't had the time to fix some other issues, clean them up and commit them.

Great news. I hope those patches are merged to master eventually once the bugs are ironed out. If you want a beta tester I'm here.

We have a plugin for performance counters (hundreds of them on Windows) but its not included with the main repository. The plugin atricturecture is easy enough to make you own and even easier if you duplicate one of the existing plugins.

I found the PerfMonPlugin in the plugins-extra repo (https://github.com/processhacker/plugins-extra/tree/master/PerfMonPlugin).
Thanks again.

I guess it's not possible to IFDEF this as a workaround?

It's a major rewrite which lots of changes every few months. We have to wait until the structure is more stable.

I found the PerfMonPlugin in the plugins-extra repo

👍


NTR: choline trimethylamine-lyase activity

Please provide as much information as you can:

Suggested term label:
choline trimethylamine-lyase activity

Definition (free text)
Catalysis of the reaction: choline = acetaldehyde + trimethylamine. [RHEA:35095]

Reference, in format PMID:#######
[RHEA:35095]

Gene product name and ID to be annotated to this term
Q30W70

Parent term(s)
GO:0016840 carbon-nitrogen lyase activity

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references
EC:<IP_ADDRESS>
RHEA:35095
MetaCyc:RXN-13946
KEGG_REACTION:R10285

Any other information
choline degradation III (MetaCyc:PWY-7167) is a single step pathway corresponding to EC:<IP_ADDRESS>, so add that as relatedMatch xref

We already have this related term:
id: GO:0140111
name: [choline trimethylamine-lyase]-activating enzyme activity
namespace: molecular_function
def: "Catalyzes the activation of choline trimethylamine-lyase by generation of an organic free radical on a glycine residue via a homolytic cleavage of S-adenosyl-L-methionine (SAM)." [PMID:24854437]
xref: MetaCyc:RXN-16470
is_a: GO:0043364 ! glycyl-radical enzyme activating activity
property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/14190" xsd:anyURI
created_by: pg
creation_date: 2017-10-20T13:57:38Z

+[Term]
+id: GO:0120525
+name: choline trimethylamine lyase activity
+namespace: molecular_function
+def: "Catalysis of the reaction: choline = acetaldehyde + trimethylamine." [RHEA:35095]
+comment: Note that this term has a MetaCyc pathway reference as the pathway only has a single step.
+xref: EC:<IP_ADDRESS> {source="skos:exactMatch"}
+xref: KEGG_REACTION:R10285 {source="skos:exactMatch"}
+xref: MetaCyc:PWY-7167 {source="skos:relatedMatch"}
+xref: MetaCyc:RXN-13946 {source="skos:exactMatch"}
+xref: RHEA:35095 {source="skos:exactMatch"}
+is_a: GO:0016840 ! carbon-nitrogen lyase activity
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/28808" xsd:anyURI
+created_by: sjm
+creation_date: 2024-08-30T13:13:55Z


Async logging file handler

Description:
Logging will block async Loop on slow HDD/SD.

@balloob I'm done. I don't know why some pytests Fails on pytest_catchlog since I implement all interface to wrapper...

Looks great!

done


L515 not detected by realsense viewer.

librealsense
2.53.1 RELEASE

OS
Windows

Please provide a description of the problem

Hi @ayusefi  Could you first try unplugging the micro-sized end of the USB cable from the base of the L515 camera, reversing the connector's orientation and plugging it back into the camera, please (USB-C cables are two-way insertion at the micro-sized end).
Does doing so make any difference?

I have a D415 stereo camera it works fine. But the L515 is not being detected. The PC detects the USB but the realsense_viewer
does not. I use Windows 11.

Can you confirm if you have tried reversing the L515's USB connector, please?  This reversal solution is for L515 only and does not affect 400 Series cameras such as L515.

Hi @ayusefi  Do you require further assistance with this case, please? Thanks!

Case closed due to no further comments received.


Double click

I want to ask how can I handle a double click event on mat table row?

Hi,
You can use @ondbclick either on MatTableRow or specific td elements
@context.ClientName

I consider this issue closed with the proposed solution.

Hello, sorry I'm new with blazor and matblazor, Is possible to have an example how to do it, I'm trying this.
I'm getting this error Unrecognized attribute '@ondblclick' on child content element 'MatTableRow'
But If I do the same in and simple html  works fine..
Greetings,

hum... Actually after verfication it seems I was wrong
@ondblclick has not been implemented on MatTableRow so the error you get is normal.
Well you can bind @ondbclick on "each" td or I guess make the following evolves in MatBlazor:
TableRow.razor
@namespace MatBlazor
@inherits BaseTableRow

@using Microsoft.AspNetCore.Components

<CascadingValue Value="@this">
    <tr class="@ClassMapper.AsString()"
        style="@StyleMapper.AsString()"
        @ref="Ref" @key="ChildContent"
        @onclick="OnClickHandler"
        @attributes="Attributes"
        Id="@Id"
        @ondblclick="OnDbClickHandler">
        @ChildContent
    </tr>
</CascadingValue>

BaseTableRow.cs
        [Parameter]
        public EventCallback OnDbClick { get; set; }

        protected async void OnDbClickHandler(MouseEventArgs e)
        {
            if (OnDbClick.HasDelegate)
            {
                await OnDbClick.InvokeAsync(null);
            }
        }

then you'd have OnDbClick parameter at your disposal.
I will test it and create a Pull Request
@lindespang could you please reopen the issue?

:С

Well it was a bit different from what I had expected.
One can't specify parameter to MatTableRow so it should be manage at MatTable level.
BaseMatTable.cs
        /// <summary>
        /// Action to execute on row item
        /// </summary>
        [Parameter]
        public EventCallback<object> OnRowDbClick { get; set; }

...

        #region events

        protected void OnRowDbClickHandler(object item)
        {
            if (OnRowDbClick.HasDelegate)
            {
                OnRowDbClick.InvokeAsync(item);
            }
        }

        #endregion events

MatTable.razor
                    <TableRow Class="@RowClass"
                              AllowSelection="@AllowSelection"
                              @ondblclick="() => OnRowDbClickHandler(item)"
                              RowItem="item">
                        @MatTableRow(item)
                    </TableRow>

Demo
<MatTable Items="people"
                  class="mat-elevation-z5"
                  OnRowDbClick="OnPersonDbClicked"
                  AllowSelection="true">
            <MatTableRow>
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>            </MatTableRow>
        </MatTable>
...
            void OnPersonDbClicked(object item)
            {
                var person = item as Person;
                if (person == null)
                {
                    _currentSelectedPerson = '""noone'"";
                    return;
                }

                _currentSelectedPerson = $'""{person.Firstname} {person.Lastname}'"";
            }

Thanks you very much @sebestyn168, I've seen there is an PR open for that!


etl: added generator

Description

Created a generator for the etl exercise. The generated test is functionally identical to the original, and the formatting of the hashes is as close as I could get it.
Motivation and Context

Part of Epic #396
How Has This Been Tested?

Generator has been ran and test file was successfully generated.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
~New feature (non-breaking change which adds functionality)~
~Breaking change (fix or feature that would cause existing functionality to change)~

References and Closures
References #396

Checklist:

~My change requires a change to the documentation.~
~My change relies on a pending issue/pull request~
~I have updated the documentation accordingly.~
[x] I have read the CONTRIBUTING document.
~I have added tests to cover my changes.~
[x] All new and existing tests passed.

bin/local-status-check returns no failures
rubocop -D returns no failures
Admin edit: Using strikethrough rather than empty checkboxes, so it doesn't look like an incomplete todo list.

@tommyschaefer Can you pull this locally, and run a bundle exec generate -f etl and see if exercises/etl/etl_test.rb changes in the test_bookkeeping method as far as version goes?
This seems to be unique here, as it does not seem to do the same in Hamming, for example.  It freezes like it should.

@kotp @tommyschaefer; sorry to disappear for so long, it's been a busy month. I operated on the hash using each_with_object as @kotp suggested, and it made life a lot easier.

@ajwann It looks like your x-common is now out  of date.  The generator will not build this on the current canonical data.  Can you confirm and fix?

@hilary Can you review this, and let us know what needs to be done here?

@kotp will do!

Minimal required work before merging:

first and most important, this branch needs to be rebased onto master.
Remove the EtlCases loop from lib/etl_cases.rb

PR carried forward in https://github.com/exercism/xruby/pull/575

Closing this PR now that we've carried the work forward. Let's get this generator in :)
@ajwann I've been working on reducing the effort to add a generator. When you recover from this one and get a chance to take a swing at another, I would love to hear whether you think the changes have actually reduced the effort or not.

Thank you @hilary! I'll be starting on another generator soon, I'll let you know how it goes.

@ajwann I think the best way to see what is needed to write a generator is to look at the xruby README, which we're keeping up to date.
If you'd like to see what we're planning, get involved in the discussion, etc., we have an issue for the generator changes: exercism/xruby#485


Adding PDF icon support to iconFieldName

I realize that it may not be the cleanest approach and would not be shocked if it is not accepted, but here is a workaround that I propose to deal with PDF icons which don't exist in the "image" flavour of FileTypeIcon. I find it annoying that PDF files have the empty icons.
Ideally, I could have access to FileTypeIcon's private _getFileExtension method for a more robust assessment of the PDF extension.

Q
A

Bug fix?
[ ]

New feature?
[X ]

New sample?
[ ]

Related issues?
related to #17

What's in this Pull Request?
PDF files will show an emulated PDF icon instead of the generic file icon.

Office UI Fabric now has a PDF icon. Will add a CDN fallback for this.


Bootstrap failure

Explain problem to solve
Have a explicite status of oscmachine when bootstrap fail (cloud-init script) or succeed, because oscmachine don't known if cloudinit failed or succeded.
Describe the solution you would like
Options 1:
After a timeout (to be defined), mark oscmachine as failed.
Additional context
https://github.com/kubernetes-sigs/cluster-api/issues/2554)
Environment
- Kubernetes version: (use `kubectl version`): v1.22.5-rancher1-1
- OS (e.g. from `/etc/os-release`): CentOS Linux 7 (Core)
- Kernel (e.g. `uname -a`): 3.10.0-1160.15.2.el7.x86_64
- cluster-api-provider-outscale version: v1.1.3
- cluster-api version: current
- Install tools: calico, ingress-nginx, osc-ccm
- Kubernetes Distribution: rke
- Kubernetes Distribution version: v1.3.4

We do not have currently a solution to execute command on existing vm like linux extension azure (https://github.com/kubernetes-sigs/cluster-api-provider-azure/issues/603) at 3DS outscale.
When we will have a solution like this, we will able to implement this feature.


adds email model, adds outlook as data source

Email contributes to carbon emissions, ideally people should only send and receive the emails they absolutely need and unsubscribe from what they don't need.
This pull request adds support for an email model and integrate with Outlook (Office 365 & consumer).
In addition, a lot of sites/apps do not provide APIs to integrate activities easily but send emails instead. Parsing those emails is a good way to extract additional activities automatically. This PR lays the infrastructure to do so (with an example with IKEA)

To create the app registration:

You'll need a live/outlook/office 365 account to perform the following setps.

Go to the azure portal
Click on Azure Active Directory
Click on App Registrations
Click on New registration
Enter a name for the application
Select the Multi-tenant and personal accounts option
Click Register
Go to API Permissions
Click Add Permission
Select Microsoft Graph and Delegated permission
Search for Mail.Read
Select the permission and click Add permission

This is very impressive - @corradio I envision potential privacy issue, can you advise?

This is very impressive - @corradio I envision potential privacy issue, can you advise?

I've looked briefly at the code and I don't see any privacy issues introduced.
I would however not recommend creating one activity per email (as it would create a lot of bubbles with a negligible footprint) and instead create activities of type ACTIVITY_TYPE_PURCHASE when a purchase has been detected from an email.
This will also enable the user to edit the activity without us having to do any more UI work.

so you would create one purchase a day/week with one item and the quantity being the number of emails basically?
Is it possible to purchase something at zero dollars today in the UI/model?
The reason why I started a digital category is because I also wanted to add online storage (onedrive, box etc...) and online video watching (youtube...) as sub-items of those categories.

so you would create one purchase a day/week with one item and the quantity being the number of emails basically?
Is it possible to purchase something at zero dollars today in the UI/model?
The reason why I started a digital category is because I also wanted to add online storage (onedrive, box etc...) and online video watching (youtube...) as sub-items of those categories.

The footprint would be so small (even for a week) that I would refrain against doing anything with the number of emails themselves (anything <1kg will clutter the UI up without contributing significantly).
The second issue is that we will need to change the UI (to enable editing) and that is not something we will be able to prioritise in the near future.
However, I think the vision of adding a vertical called "digital" makes sense. We're just not ready yet on our side ;-) We might work on it in Q3/Q4 by adding the footprint of Azure/AWS/GCP.
Finally, it makes totally sense to start working on email parsers that can extract activities from emails (such as purchases). That is something we can do with 0 changes to the UI.
Hence my suggestion: I would not create digital activities with the number of emails, but instead I would create activities extracted from emails (e.g. purchases) and encourage others to start writing more advanced parsers!
Does that make sense?

It does, couple of questions:
Would you prefer me to leave the code in place for email activities but disable it (not call it) or remove it all together at this point?
What about other digital activities such as watching online videos I wanted to add?

I suggest commenting out / disabling the code that creates an activity per email (but keep the code that extracts purchase activities)
Regarding the footprint of online videos, you could read this thread that we started a while back: https://github.com/tmrowco/northapp-contrib/issues/46
For now, it seems like the footprint is too small and will impact performance of the app (if we create a bubble for each video watched).
But maybe it's worth making some calculations to make sure it's worthwhile.

I have disabled the generation of activities per email but kept the parsing part.
I'm going to take a look at the video part whenever I get the time.
For the bubble issue: there's a simple solution -> aggregate any activity of the same type for the month that is under 100 g CO2 eq before you generate the bubble, and add an indicator that the bubble is aggregated (behold my paint skills).

No worries, thanks for your help. Replied to some of your comments with questions. I'll make the changes once I have the answers to those.

Thanks team! I can't wait to see people implementing more email providers! I'll try to get started on GMail soon.

Thanks team! I can't wait to see people implementing more email providers! I'll try to get started on GMail soon.

Ofcourse! I think GMail requires a paid access though..

@corradio juste checking in here. How close are we to see that in the app? The new design looks great by the way!

@corradio juste checking in here. How close are we to see that in the app? The new design looks great by the way!

Hey @baywet, we had to do some changes to make it work (#427) and some internal updates as well, but that has been done and it's now being QA tested internally - it will be included in next release within a few days, so I'll let you know when it's out :)


Disable xdebug in travis tests

This appears to be much faster on the PHP engines.
However phpenv is killing the HHVM build, which is actually 12X slower most of the time anyhow :-/

@ciaranmcnulty changing it to phpenv config-rm xdebug.ini || true will keep the HHVM build up and running :tada:

I suggest doing this simple change:
before_install:
    # Remove xdebug as per https://getcomposer.org/doc/articles/troubleshooting.md#xdebug-impact-on-composer
    - if ["$TRAVIS_PHP_VERSION" = 'hhvm']; then phpenv config-rm xdebug.ini; fi


Bumped whichcraft to 0.3.1

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

Merging [#682][cc-pull] into [master][cc-base-branch] will increase coverage by +n/a%

@@           master   #682   diff @@
====================================
  Files          13     13
  Lines         577    577
  Methods         0      0
  Branches        0      0
====================================
  Hits          577    577
  Misses          0      0
  Partials        0      0

[][cc-pull]

Powered by Codecov. Last updated by 731c727
[cc-base-branch]: https://codecov.io/gh/audreyr/cookiecutter/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/audreyr/cookiecutter/pull/682?src=pr

:+1:


Deactivating and activating AL Languages for workspace

Describe the bug
Options "Deactivate in Workspace" and "Activate in Workspace" does not work for AL Languages
To Reproduce
1.) Two different projects, one with an 2018 AL Language, one with a BC AL Language
2.) Switching between them always needs to uninstall the wrong language and reinstall the correct AL Language via .vsix
Expected behavior
Having workspaces with different AL Languages is a must-have if you have two or more customers with different AL Versions
Versions:
All versions of AL and VS Code

Hi! We will investigate this issue. One workaround for this would be to use the insider build of VSCode with one extension installed and the mainstream version of VSCode with the other extension installed.

Hi atoader, thanks but that is not really a suitable workaround for me as i am currently working on 5 projects with 5 different versions of AL. I think reinstalling the AL Language every time is pretty much faster. :-)

You should need at most 2 versions of the extension. Starting with the 2018 Spring release of Business Central you can always use the latest version of the AL extension independent of the server version. You still need to use the version of the AL language extension that comes on the DVD for NAV 2018. You can read more here https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-developing-for-multiple-platform-versions

@MarkusWinBI I can confirm that the latest extension can be disabled per workspace. You can try it out by using the latest developer preview: https://freddysblog.com/2019/07/31/preview-of-dynamics-365-business-central-2019-release-wave-2/


Remove track without snapshot_id

According the spotify API, snapshot_id is not mandatory. It seems to be in this interface, though. Is there a reason? If so, how do you retrieve a snapshot_id for a playlist?

@rsiegel827 The snapshot_id is not mandatory in that endpoint. What you can do is provide an empty object as the options parameter when calling removeTracksFromPlaylist.


Merged test PR Workflow sanity check back to regular condition

Current dependencies on/for this PR:

main

PR #1106

PR #1107

PR #1108

PR #1113   👈

PR #1114

This comment was auto-generated by Graphite.


Sync/async Sentry client convergence

Currently two different Sentry client implementation are present:

async Sentry client within the RPC server
sync Sentry client within the header downloader

The two different implementation should be merged into one unique suitable for both scenarios.

Done in #983


Android R8 Desugaring removes putIfAbsent method on DNSCache

We are using jmdns as a library within our own Android library.
Within our Android libraries, we would like to use Java 8 features, which in short means that we need to enable R8 desugaring (https://developer.android.com/studio/write/java8-support).
With desugaring enabled, we are facing a crash inside the jmdns library, specifically inside DNSCache, where the putIfAbsent method is being removed. Why R8 does this, we don't know.
This results in runtime crashes with logging similar to this:
java.lang.NoSuchMethodError: No static method $default$putIfAbsent(Ljava/util/concurrent/ConcurrentMap;Ljava/lang/Object;Ljava/lang/Object;)Ljava/lang/Object; in class Lj$/util/concurrent/ConcurrentMap$-CC; or its super classes (declaration of 'j$.util.concurrent.ConcurrentMap$-CC' appears in base.apk!classes2.dex)
        at javax.jmdns.impl.DNSCache.putIfAbsent(Unknown Source:0)
        at javax.jmdns.impl.DNSCache.addDNSEntry(SourceFile:2)
        at javax.jmdns.impl.JmDNSImpl.handleRecord(SourceFile:34)
        at javax.jmdns.impl.JmDNSImpl.handleResponse(SourceFile:5)
        at javax.jmdns.impl.SocketListener.run(SourceFile:15)

After some additional research (including checking out the jmdns code and modifying it) we found a possible fix, where instead of letting DNSCache inherit from a ConcurrentHashMap, we are giving it a private instance of it.
I also improved the synchronization around that code, since we can now synchronize on the map instead of the local entryList.

Hi @MWingbermuhle , we also use Jmdns and started facing this crash recently in our project after enabling desugaring support. Thanks for the issue and the fix.

Fixed with https://github.com/jmdns/jmdns/pull/208.


Add a Covid-19 analyzer

First basic implementation of an analyzer for covid-19 data, see #1091.
A first requirement would be not to split at hyphens (covid-19, sars-cov-2, etc.).
It can be improved iteratively.

hi @tteofili - just a follow up on this: I think the best course of action is to wait for TREC-COVID round 1 results so that we have relevance judgments to measure the effects of this.


[TEST] Selenium Tests 01

only test - selenium
merge - https://github.com/apache/zeppelin/pull/1048 , https://github.com/apache/zeppelin/pull/1053
Please, DO NOT MERGE

pass
more retry ci test.


Implement avahi/mdns discovery

Change-type: major
Implement avahi/mdns discovery, this removes the nmap dependency and fixes an issue where every so often nmap would fail to find dependent devices causing them to be marked as offline

@mccollam this is meant to be used on the hotspot network running on the gateway device. We could have an env var to let the user fall back to nmap if they needed to for whatever reason?

Seems like it is a "nice to have" rather than strictly necessary.  I suspect we'll hit Avahi being blocked in the same environment that you're working on this fix for :)  But we can certainly try this out and see.

@mccollam avahi won't be blocked as the hotspot network is running the rpi... or am I missing something here?
I will add the nmap scan back in as an option.

Ah, good point... you're right.  Not sure nmap is needed then tbh.


Port cvui to Python

OpenCV already has Python bindings, so it would be a natural thing to port cvui to Python as well. Maybe it is possible to use the C++ code and just create the bindings, or (not ideally) re-write the whole thing in Python.

I've used pybind wrapper in Intel Realsense and it have been "okay" to use, their documentation is not great with the wrappers, but there are just one gigantic file https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/python.cpp which have all definitions so when you need a function described in their cpp version it is pretty much same call in python to do the same.

That's a great insight, thanks! Regarding Inte Realsense, I've just checked their docs for the python wrapper and there are several steps to build and use it. That's a bit of a pain. I thought it would be easier.
I am more convinced now that a pure Python implementation of cvui will be much better. Just drop cvui.py into your project and your are good to go (the same simplicity of a header-only C++ lib).

Hi,
Long time no talk :-)
I happened to have worked with pybind also. I have had to hard-code the conversion between cv::Mat and numpy.ndarray (images are stored as ndarray in python with opencv).
See this gist : this is the conversion routines that I use
https://gist.github.com/pthom/ec2a2395003a2ea004b861d3686ef3fb
cheers,
Pascal

@Dovyski Great that it could help with some insight, in the end I would definitely prefer just to be able to import cvui.py into a project, that way the autocompletion will work wonders :+1:

@pthom Always a pleasure to see you around ;) Thanks for the gist contribution! Your feedback highlights even more how painful it is to mix C++ and Python code. I will definitely go with a pure Python port of cvui.
@TheMikeyR Yes, me too! And it is an excellent excuse for me to start using Python 😄

pure Python port of cvui :)

Guys , any idea on when the python port will be ready? I need to use cvui with python for the project. Is there any chance that you can do a temporary binding for python for people like me to use till the pure python port is ready. Thanks and I will do what I can to help.

Unfortunately, it is moving way too slow at the moment.
Regarding a temporary binding, I had previously checked this solution and this one as possible options for cvui bindings. I haven't tried them myself. Maybe you could give them a go and see if they work for your project?

I'm thrilled to say the python port of cvui is finally complete! :tada: All existing C++ examples were also ported to python. I've added some dummy/wrapper functions in cvui.py to improve code autocompletion in IDEs. Hopefully it will work as intended.
If you happen to be using python in your opencv project, please give the cvui port a spin and help me test it. Just drop cvui.py in your project and you are good to go.

Python support for cvui has been offically added in version v2.7.0. You can install it using pip:
pip install cvui


fails to properly read compilation database for system headers

I have a simple objective-c test file main.m.
// main.m
#include <Foundation/Foundation.h>

void myfunc() { NSLog(@"From my func"); }

int main(int argc, const char *argv[]) {
  NSLog(@"Hello world");
  myfunc();
  return 0;
}

The functionality of jumping to definitions, declarations, etc is fine for functions and objects that I define myself, however;  NSLog information is incomplete and inaccurate. For instance, jumping to definition on the "Hello world" call jumps to the call in myFunc(). Expected behavior is that it would jump to definition in the Foundation header.
Running the command:
clang -MJ main.json  -framework Foundation -isysroot $(xcrun --show-sdk-path) main.m -o app
Logs
System information
Output of clangd --version:
clangd version 14.0.6
Features: mac
Platform: aarch64-apple-darwin23.5.0

Editor/LSP plugin: vim-lsp
Operating system: macOS

Please attach clangd logs.
It's also worth trying with a newer clangd version (they can be downloaded from https://github.com/clangd/clangd/releases), 14.0 is a couple of years old by now.

logs attached:
log.txt

logs attached: log.txt

This is not a clangd log. This looks like something output by clang, not clangd.
Clangd logs are what clangd prints to its stderr while you are using it (e.g. while you are opening the file main.m). They are typically exposed by the editor (the exact method varies from one editor to another).

I'm just using ctags. If someone could post an example command for this minimal example, that would help others.


Unit tests fail on go-tip

I've seen this fail twice, with the same error. It could indicate a real problem.
https://github.com/jaegertracing/jaeger/actions/runs/11789532893/job/32838429890
--- FAIL: TestCreateCollectorProxy (0.00s)
    --- FAIL: TestCreateCollectorProxy/#02 (0.00s)
panic: Reuse of exported var name: gRPCTarget
	 [recovered]
	panic: Reuse of exported var name: gRPCTarget

goroutine 139 [running]:
testing.tRunner.func1.2({0x13a3400, 0xc000220f10})
	/home/runner/sdk/gotip/src/testing/testing.go:1706 +0x3eb
testing.tRunner.func1()
	/home/runner/sdk/gotip/src/testing/testing.go:1709 +0x696
panic({0x13a3400?, 0xc000220f10?})
	/home/runner/sdk/gotip/src/runtime/panic.go:787 +0x132
log.Panicln({0xc00001d258, 0x2, 0x2})
	/home/runner/sdk/gotip/src/log/log.go:446 +0x8e
expvar.Publish({0x152e8a4, 0xa}, {0x16cdc60, 0xc0001f25b0})
	/home/runner/sdk/gotip/src/expvar/expvar.go:311 +0x152
expvar.NewString(...)
	/home/runner/sdk/gotip/src/expvar/expvar.go:347
github.com/jaegertracing/jaeger/cmd/agent/app/reporter.NewConnectMetrics({0x16d8d80, 0xc000222720})
	/home/runner/work/jaeger/jaeger/cmd/agent/app/reporter/connect_metrics.go:32 +0x189
github.com/jaegertracing/jaeger/cmd/agent/app/reporter/grpc.(*ConnBuilder).CreateConnection(0xc0001f6360, {0x16d7ab8, 0xc0003c8000}, 0xc000382180, {0x16d8d80, 0xc0002225a0})
	/home/runner/work/jaeger/jaeger/cmd/agent/app/reporter/grpc/builder.go:99 +0x1025
github.com/jaegertracing/jaeger/cmd/agent/app/reporter/grpc.NewCollectorProxy({0x16d7ab8, 0xc0003c8000}, 0xc0001f6360, 0x0, {0x16d8d80, 0xc0002225a0}, 0xc000382180)
	/home/runner/work/jaeger/jaeger/cmd/agent/app/reporter/grpc/collector_proxy.go:30 +0x9a
github.com/jaegertracing/jaeger/cmd/agent/app.TestCreateCollectorProxy.func1.GRPCCollectorProxyBuilder.1({0x16d7ab8, 0xc0003c8000}, {{{0x1537aeb, 0x4}, 0x0}, 0xc000382180, {0x16d8d80, 0xc0002225a0}})
	/home/runner/work/jaeger/jaeger/cmd/agent/app/proxy_builders.go:15 +0xa5
github.com/jaegertracing/jaeger/cmd/agent/app.CreateCollectorProxy({0x16d7ab8, 0xc0003c8000}, {{{0x1537aeb, 0x4}, 0x0}, 0xc000382180, {0x16d8d80, 0xc0002225a0}}, 0xc00014fd08)
	/home/runner/work/jaeger/jaeger/cmd/agent/app/builder.go:239 +0x1b3
github.com/jaegertracing/jaeger/cmd/agent/app.TestCreateCollectorProxy.func1(0xc00021f500)
	/home/runner/work/jaeger/jaeger/cmd/agent/app/builder_test.go:[249](https://github.com/jaegertracing/jaeger/actions/runs/11789532893/job/32838429890#step:7:250) +0xa45
testing.tRunner(0xc00021f500, 0xc000034fa0)
	/home/runner/sdk/gotip/src/testing/testing.go:1764 +0x226
created by testing.(*T).Run in goroutine 136
	/home/runner/sdk/gotip/src/testing/testing.go:1823 +0x8f3
FAIL	github.com/jaegertracing/jaeger/cmd/agent/app	0.045s

@yurishkuro This seems to be due to parallel execution of CollectorTestProxy which conflicts with global state variable grpcTarget , to keep parallelization we would need some abstraction over to prevent target name collision .

@yurishkuro This seems to be due to parallel execution of TestCollectorProxy which conflicts with global state variable gRPCTarget. To keep parallelization, we would need an abstraction to prevent metric name collisions.

+1
Test failed when I try to re-run it after multiple itterations because of the concurrent nature as we are using t.Parallel here in cmd/agent/app/builder_test.go in TestCreateCollectorProxy.
Should we considering removing it or using sync.Once or Mutex so that expvar.NewString("gRPCTarget") only happens once?

expvar.NewString("gRPCTarget")

where is this called?

expvar.NewString("gRPCTarget")

where is this called?

Inside NewConnectMetrics function in /cmd/agent/app/reporter/connect_metrics.go
Here is that code block:
if r := expvar.Get("gRPCTarget"); r == nil {
	cm.target = expvar.NewString("gRPCTarget")
} else {
	cm.target = r.(*expvar.String)
}

ok, good catch. Yes, we should initialize this via Once


Platform Support for Dynamic Memory Allocation

This exposes the interface to fix #714. Once merged, I will implement the functions for Windows kernel mode in my driver.

I need to guarantee cleanup now that memory is dynamically allocating. I believe I need to add a delete aInstance to otInstanceFinalize now, but I am unsure how delete is supposed to work when you have your own new operator.

Current coverage is 72.24% (diff: 100%)

Merging #767 into master will decrease coverage by 0.05%

@@             master       #767   diff @@
==========================================
  Files           106        106
  Lines         15852      15852
  Methods        2163       2163
  Messages          0          0
  Branches       1885       1885
==========================================
- Hits          11461      11453     -8
- Misses         3768       3773     +5
- Partials        623        626     +3

Powered by Codecov. Last update 47920e8...fd487b3

When I build my driver using this, it gives me a link error for delete. I guess I have to implement the delete function. I will add one more commit for this.

Any issues with this PR preventing it from being merged? I have been using it in my Windows driver for a few days worth of testing now, and it is working great.


insert(model)

Hello,
The latest version of the stubs makes mypy log an error on insert(SAModel).from_select(...):
Argument 1 to "Insert" has incompatible type "Type[SAModel]"; expected "Union[str, Selectable]"

while this is something that works with SQLAlchemy.

I ran into this problem today as well, I have the SQLAlchemy plugin for mypy. I'm using a declarative base class with the @as_declarative decorater.

I've come across a related problem, but with update.
I have a database model create like:
from sqlalchemy.ext.automap import automap_base
ModelBase: Any = automap_base()

class MyTable(ModelBase):
"""A table"""
id = Column(Integer, primary_key=True)
name = Column(String)

Then attempting to create an SQL statement:
from sqlalchemy import update
statement = update(MyTable).where(id == some_value).value(**some_values)

The result from mypy is:
Argument 1 to "Update" has incompatible type "Type[MyTable]"; expected "Union[str, Selectable]"

SQLAlchemy == 1.4.15
sqlalchemy-stubs == 0.4
mypy == 0.800
mypy-extensions == 0.4.3

Getting the same thing as @rjaduthie:
table
class MyTable(Base):
    id = Column(Integer, primary_key=True)
    fk = Column(UUID(as_uuid=True), ForeignKey(ForeignTable.id), nullable=False)
    property = Column(Text, nullable=False)
    created_at = Column(TIMESTAMP(timezone=True), server_default=func.now(), nullable=False)
    updated_at = Column(TIMESTAMP(timezone=True), onupdate=func.now())

statement
statement = (
    update(MyTable)
    .where(MyTable.fk == fk)
     .values(property="property")
)
    await db_session.execute(statement)

mypy error
Argument 1 to "Update" has incompatible type "Type[MyTable]"; expected "Union[str, Selectable]"

I'm also running in the same error 😞

any updates?

Same error. Had to # type: ignore the issue.


PFFile profile image name won't save correctly, Missing PFFiles after migration. Parse Dashboard Naming bug

I save a user with a profile picture like so
`
let imageData: NSData = UIImagePNGRepresentation(profilePictureImageView.image!)!
        let profilePicture: PFFile = PFFile(name: "profile.png", data: imageData)!
        self.user.profilePicture = profilePicture

        self.user.saveInBackground().continueWithBlock({
            (task: BFTask!) -> AnyObject! in
            if task.completed {

`
But in the parse dashboard it shows up like the below screenshot, as you can see 2 user profile_pictures are named 'ile.png' INCORRECT.... The last user_name and profile_picture was from before I migrated to parse server. Strangely all the users created before the migration to parse server, their profile_pictures are missing. And the new profile pictures aren't named correctly.

Also on my model when I get the image from newly created users post the parse server migration the below does work. So maybe theres a PFFile naming bug on the new parse dashboard.
`
      var profilePicture: PFFile {
        get {
          return objectForKey("profile_picture") as! PFFile
        }

        set {
          setObject(newValue, forKey: "profile_picture")
        }
      }

`

Can you check what the actual response is when using the REST API? If the response contains everything you'd expect, it would likely point to an issue with the parse-dashboard, which has its own separate repository and issue tracker.
If the issue is not in Parse Dashboard, you'll want to determine if this is an issue in the iOS SDK or in Parse Server.
Feel free to reopen the issue once you've dug deeper into this. Make sure to refer to the new issue template which has all the fields necessary to open a bug report against Parse Server.


Clear text causes java.lang.IndexOutOfBoundsException

I want to clear field when incorrect digit typed (two-digit month field for credit card)
When invoking  monthCard.getText().clear(); there are such error:

FATAL EXCEPTION: main
Process: ru.perekrestok.app, PID: 28172
java.lang.IndexOutOfBoundsException: setSpan (1 ... 1) ends beyond length 0
at android.text.SpannableStringBuilder.checkRange(SpannableStringBuilder.java:1309)
at android.text.SpannableStringBuilder.setSpan(SpannableStringBuilder.java:680)
at android.text.SpannableStringBuilder.setSpan(SpannableStringBuilder.java:672)
at android.text.Selection.setSelection(Selection.java:76)
at android.text.Selection.setSelection(Selection.java:87)
at android.widget.EditText.setSelection(EditText.java:127)
at br.com.sapereaude.maskedEditText.MaskedEditText.afterTextChanged(MaskedEditText.java:303)

Anything new about this one?
Happens to me as well, only workaround that I found was to call .setText("")
For a numberPassword EditText field

Hello, yes, will need to investigate. If you can help me with that - it would be nice, and I'll provide you all the info about how to build the source code, upload changes, test/debug and so on! Please, reach me out, if you're interested!