static library build

I'm trying to build this as a static lib that I can link to, but must admit I'm out of my depth.  Is it possible?  I've added the usual cmake commands but am getting various errors.  If I try to build the archive manually, I get that it's empty.  If I compile src/json.hpp to an object file, it's only 200B in file size, which seems wrong, objdump doesn't print any symbols from it, and ar complains that the would be archive would be wrong.  What am I missing here?  I recognize the library is meant to be included as a header, and static linking doesn't save me from having to import the forward declarations anyways.

@nickdesaulniers You can not compile the json.hpp because it's just header file. You have to include it in you source files and then build your own library by using ar if you wish. It happens because header files contain only definition, templates or inline functions (if we talk about class definition), templates and so on which will be placed where you want to use them in your code.
So, for example you have header foo.h and source file foo.cpp:
// foo.h

void foo();

// foo.cpp
#include <iostream>

void foo() {
    std::cout << "Hello from foo!" << std::endl
}

So, you want to use foo() function in you project. To do that you need to include foo.h header where definition of your function is placed. After that you add foo.cpp to your project and it will be compiled to object file and then linked. But if you make some changes in foo.cpp it must be recompiled again. One day you decide that your function void foo() is ideal and you do not want to make any changes or you want to hide implementation from others. In that case you could build object file and archive it with ar for making library which you could statically linked to your project. Then you will need only header file for understanding which functions you are able to use from you library.
I hope the process more or less clear.

ah, so that's why inline is everywhere.  Thanks for explanation Vladamir, I appreciate it!

Thanks @vpetrigo for the explanation. In fact, the class makes heavy use of templates to be applicable in many scenarios where we know little about the values we want to store to JSON, but can rely on the fact that they are some kind of STL containers.


fix(metadata): correct V2 parsing err response

PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/master/.github/CONTRIBUTING.md.
What is the current behavior?

Issue Number:
Fix https://github.com/edgexfoundry/edgex-go/issues/2769
What is the new behavior?
V2 add request JSON parsing isn't handeled correctly.  Make them return
the response format meet the swagger definition.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

recheck

recheck

recheck

Codecov Report

Merging #2779 into master will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2779      +/-   ##
==========================================
+ Coverage   38.61%   38.73%   +0.12%
==========================================
  Files         159      159
  Lines       12467    12482      +15
==========================================
+ Hits         4814     4835      +21
+ Misses       7372     7367       -5
+ Partials      281      280       -1

Impacted Files
Coverage Δ

internal/core/data/v2/controller/http/event.go
86.66% <100.00%> (+1.21%)
:arrow_up:

.../core/metadata/v2/controller/http/deviceprofile.go
100.00% <100.00%> (ø)

.../core/metadata/v2/controller/http/deviceservice.go
100.00% <100.00%> (ø)

internal/core/data/v2/application/event.go
83.01% <0.00%> (+11.32%)
:arrow_up:

Continue to review full report at Codecov.

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

@cloudxxx8 Please rebase this PR.  Thanks!

recheck


fix(redirect): add select handler to ensure matching queries get redirected

If you click on an item, that when that response returns will have a redirect, then you are also redirected

We're not going this approach, as there's no indicator of the redirect behaviour.


Rename list.md to README.md

This will allow you to view the file without opening it on github. Read more here.

And note: You can do this. Just hide very long lists of builds under spoiler (<details>) tag. Enjoy, @SS13-Source-Archive.


Double linespaces after method and class

PSR2 checking / fixing with phpcbf, flags up that the final method in a generated class has two clear linespaces after it, and that there are two clear linespaces after the class (at the end of the file).

Possibly the first self::LINE_FEED here is the cause of the linespaces after the final method, and the second in combination with this one gives the double lines at the end of the file?

That's kinda in the "won't fix" scenario. Zend\Code produces artifacts, not
well-formatted code (that's really secondary)
On 24 Feb 2017 14:59, "Gary Jones"<EMAIL_ADDRESS>wrote:

PSR2 checking / fixing with phpcbf, flags up that the final method in a
generated class has two clear linespaces after it, and that there are two
clear linespaces after the class (at the end of the file).
[image: screenshot 2017-02-24 13 46 24]
https://cloud.githubusercontent.com/assets/88371/23305858/a9902e62-fa98-11e6-8ff0-74cf92836ea3.png
Possibly the first self::LINE_FEED here
https://github.com/zendframework/zend-code/blob/2899c17f83a7207f2d7f53ec2f421204d3beea27/src/Generator/ClassGenerator.php#L1084
is the cause of the linespaces after the final method, and the second in
combination with this one
https://github.com/zendframework/zend-code/blob/2899c17f83a7207f2d7f53ec2f421204d3beea27/src/Generator/FileGenerator.php#L539
gives the double lines at the end of the file?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-code/issues/103, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAJakA6fSWEhN_TmG2s-G3gXV_1wHVuiks5rfuJCgaJpZM4MLO8Y
.

Is there a negative to having artefacts that are well-formatted to PSR-2, since most of it already seems to be?

No, it's just not worth tracking as bug/issue.
On 25 Feb 2017 10:23 a.m., "Gary Jones"<EMAIL_ADDRESS>wrote:

Is there a negative to having artefacts that are well-formatted to PSR-2,
since most of it already seems to be?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-code/issues/103#issuecomment-282472211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakJC9RscCyu0AN6qsfN6dI1Gc_CP0ks5rf_MegaJpZM4MLO8Y
.

If you're want to send a patch, feel free to do it. Meanwhile closing here.


SubtitleEditor

Includes all previous PRs for the subtitle editor.

This PR aims at integrating the different views of the subtitle editor with the parsed subtitles from Opencast.

The add subtitles button in the select view now has functionality
The subtitle select view only shows buttons for languages that already exist in Opencast or have been newly created
Displays error message for subtitles that could not be parsed (TODO: Make this more user friendly)

Feedback after using the Subtitleeditor the first time:

The curser looks different and cannot be moved, which is pretty annoying the first time.
 VS
If you don't want the cursor to be set to a specific position, it could have the same shape without the part indicating you can grab it.
Timeline: You cannot scroll horizontally (at least it did not work on firefox/fedora 36)
Timeline in focus: You cannot add a segment with keyboard shortcuts
Input fields do not support dark mode

I'll write it down here as well as on the backend (saving subtitles) pull request since it touches both parts:
I did only briefly skim through the code, but if I interpret the code correctly, subtitles are loaded from the internal publication, but then saved to the asset manager. This means that you don't necessary get what you save the next time you open the editor. It also means that you may overwrite completely different subtitle files and you might also miss subtitle files which are only in the asset manager.
My suggestion would be to always get the subtitles from the asset manager?

Where are the flags coming from? Why is Generic Canadian? (…or Canada generic?)

Another interesting question would be if we don't want to show all captions/* flavored tracks instead of hiding those not specifically configured. If they exist already, I would tend to show them. Hiding them could be confusing.

Where are the flags coming from? Why is Generic Canadian? (…or Canada generic?)

The code for that is at /src/main/SubtitleSelect.tsx line 64-66 (can't figure out how to link to it). The flags are just kind of guessed at from the specified subflavor, as that usually includes the language code. Arguably that's a terrible approach, but I really thought the flags added to the visual.

Another interesting question would be if we don't want to show all captions/* flavored tracks instead of hiding those not specifically configured. If they exist already, I would tend to show them. Hiding them could be confusing.

Iirc that behaviour was requested, though I do not remember the reasoning behind it.

Can we put this on the next Technical Meeting agenda?

I noticed some elements in darkmode are either very dark and hard to see or they don't support darkmode.
The big buttons are missing the boxShadow and background-color, since there is no more finishMenuButtonStyle. The new tileButtonStyle is missing these two attributes. This also affects the big buttons on the finish page. In darkmode the buttons are very hard to see.

Same here for the 'Pick a language' Button.
Additionally the text on the dropdown-menus is hard to read in darkmode.

The dropdown-menus are also missing darkmode colors.

Maybe give these buttons also a lighter box-shadow for darkmode? Also the inputfields are very bright in darkmode, maybe give them the same look, as the textbox and inputfields on metadata-page so the design is consistent?

What is the purpose of the Mini Timeline?
At first I was confused why there is a white thick bar under the waveform (I am in darkmode) until I inspected it in the browser and saw it's a timeline.
What is this timeline for? Do we need it? And if so maybe you could make it look more like a timeline?

Feedback after using the Subtitleeditor the first time:

Hey, thanks for the feedback and sorry for the late reply. Let me comment on some of the problems you faced

* The curser looks different and cannot be moved, which is pretty annoying the first time.

Agreed, having a similar look and feel to the cutting view only to work differently is confusing. Will go with your suggestion on that.

* Timeline: You cannot scroll horizontally (at least it did not work on firefox/fedora 36)

That is strange, it works for me on firefox/fedora 35. Do you perhaps have more details on that?
Ways to scroll horizontally:

Clicking somewhere on the little grey bar at the very bottom of the page
Drag and dropping on the colored part of the timeline, below the subtitle segments (or any part of the timeline that is not a subtitle segment, really. Drag and dropping on a subtitle segment will try to move the subtitle segment instead).

* Timeline in focus: You cannot add a segment with keyboard shortcuts

That is currently arguably intended, as the keyboard shortcuts are specifically for "Adding a segment above/below (the current segment)". If you focus somewhere else, the editor would have to guess at where you want your segment to be added (at the end, at the beginning, at the segment that focused on last, relative to the segment currently hovered by the indicator on the timeline (which can technically be multiple segments)). Not sure what the intuitive behaviour would be in this case.

* Input fields do not support dark mode

PR so old it's older than dark mode. I'll try and get to that.

What is the purpose of the Mini Timeline?

Since the "actual" timeline is strongly zoomed in, the mini timeline let's you get to other parts of the subtitle in a fast manner. I would argue this is useful, as the second fastest thing you could to achieve the same effect is scrolling the list view and then clicking on a cue. If you have a (naming) suggestion on how to make the mini timeline more intuitive, I'm all ears :D

Maybe call it "overview"? It gives you an overview of the whole timeline.

the mini timeline let's you get to other parts of the subtitle in a fast manner

Ah yes that makes sense :D didn't thought of that.

If you have a (naming) suggestion on how to make the mini timeline more intuitive, I'm all ears :D

'Mini Timeline' is good, maybe the left side of the marker could be colored? Like the timeline in Studio or the timeline on Youtube videos.

Maybe this could be improved in the following way:
The mini timeline could be made a little bit higher (20px instead of 15px) and the waveform could also be shown in the mini timeline. Additionally, you can show a lightly coloured "window" of the zoom displayed in the timeline above. Now, when you move the zoomed-in view it clearly shows where in the overall timeline you are.
I created a mockup.

Instead of overlaying the subtitles, moving a subtitle over another one could change the duration accordingly. Likes it's solved in the subtitle editor of a giant video platform.

The first subtitle element gets "cut" after moving the blue one over it.

This idea only makes sense if we do not want subtitle parts to overlay. I don't know if there is a use case for that.

Instead of overlaying the subtitles, moving a subtitle over another one could change the duration accordingly. Likes it's solved in the subtitle editor of a giant video platform.

Agreed, that would be nicer. I also do not know if there is use case for overlapping segments. However, since the WebVTT spec allows it and the editor potentially has to deal with subtitle files from many different sources which may or may not use overlapping segments, I decided it was best to err on the side of caution and respect the spec, instead of forcing non-overlapping segments.

Regarding the suggestions for the mini timeline, I went with Nadines first idea of making it look more like a player bar for now. Mainly because the time I've been given to get this PR ready for approval is somewhat limited and Daniels approach would likely require some work to work well. For example, from the looks of it I would expect the highlight box to support drag and drop, but for long videos the area would get very small, so clicking should also still be supported. Also I'm assuming the center of the box denotes the current point in time? Then you might get some interesting behaviour when dragging the box to the very start or end of the video. In short, while it is not perfect, I hope the current MiniTimeline is good enough for approval ^^

I'll create a new Issue for my proposal for the Mini Timeline.

If we allow overlapping cue timings, it would be nice to have them as automatically stacking timeline elements, so it's clearly visible where the elements start and end.
Diggin around I also found resize handles to resize a single cue element on its left side. These resize handles should be more clearly visible. I propose to change them either to three vertical dots, as that's what I mostly see used when something is resizable. Or only show them as two chevrons, when hovering over the left edge of a cue element.
Both can be seen in the screenshot.

I think something went wrong with the last commit. https://test.editor.opencast.org/638/2022-09-16_12-42-28/ does not contain the subtitle tab.

Oh, that was intentional, then maybe the config for the test deployments should be different?

Are you gonna squash those commits before merging? :)

Are you gonna squash those commits before merging? :)

Not planning to. While it's a hell of a lot of commits, most of them actually do relevant stuff, and squashing them all together feels like grossly disregarding the development history.

Sorry Arne, I found one more bug in the subtitle editor. But I hope this one is easy to fix:

HTML seems to be escaped, saved and then escaped when you load it again.

HTML seems to be escaped, saved and then escaped when you load it again.

Do you have more details on that? I could not reproduce that.

Although I could get the div tags to disappear after saving and reloading


APP-320: [Objective-C] Update DataCollection demo

http://jira.kaaproject.org/browse/APP-320

@vilimets review please.


code of conduct.md : is not available

why we need this 👍
A code of conduct defines standards for how to engage in a community. It signals an inclusive environment that respects all contributions.

@SparkYuan assign me this issue can I try

@SparkYuan assign me this issue can I try

Hi @Madhurchandran feel free to take a crack at it. We recommend creating a fork of the repo and submit the PR for review.

@SparkYuan what is this CLA where to sign I made pull request I got rejected.

@Madhurchandran You can sign the CLA by just posting a Pull Request Comment same as "I have read the CLA Document and I hereby sign the CLA"

@Madhurchandran If you have any further questions, we can continue our discussion in your PR.

@SparkYuan again pull request fails

@SparkYuan my pull request got closed should I create new pull request.


GSLUX-634: Update catalog on login logout

JIRA issue
https://jira.camptocamp.com/browse/GSLUX-XXX
Description
Prevent undefined value for current selected theme on login or logout => may not exists when user connect or disconnect according to his profile (eg. theme "Professionel" is private and available at connection for user c2c).

Don't forget to rebase, so the lib gets build into a lux.dist.js as well.

yes good point, rebasing done!


Tree element not found error when creating nested folders in single root workspace

From https://github.com/microsoft/vscode/issues/78153#issuecomment-522271070
Steps to Reproduce:

Open single folder in vscode containing just one folder FolderA
Select  FolderA, create child folder f
Click New Folder Action and cancel
Click New Folder Action again, type a/s/d and accept
Click New Folder Action and cancel
Click New Folder Action again, type q/w/e and accept

Great catch and a wonderful fix, thanks a lot!


[2.0.1] New posts in a single-account install don't pull author name from settings

More info needed?

Checking it right now. I am not sure if the author from the settings is a good idea. If there are more than one authors, then there is no way to check if the current user is the same as the author from the settings or not, so it is unclear what to use in those cases. I think it is better to check the current user and use the first and last name from the user profile.
So I will change it the following way:

In the system settings I will rename the "author" to "website owner". I think he is only used for the copyright line.
In the meta for each page I will try to get the first and last name of the current user and use that for the author line.

fixed and published/pushed


dedicated turning lane policy is only applied superficially

I activated dedicated turning lane policy in game and this happened:

The lane arrows are pointing to the right direction, but the routing manager doesn't seem to have been updated.

I tried force updating the segment using Network detective (NetManger.UpdateNode/Segment()) and even that didn't help.
If I restart the game it works fine.
if I select the segment and press delete to reset lane arrows, that will also fix the problem (even though I had not forced any lane arrows).

Info 74.6320267: UpdateAllDefaults(dedicatedTurningLanes:{dedicatedTurningLanes}) was called.

I do have this in the log but obviously updating does not fix the issue.

ai.UpdateLanes(segmentId, ref netSegment, true);
NetManager.instance.UpdateSegmentRenderer(segmentId, true);

This is not enough to update routing manager. maybe I should force routing manager to change when this is called?
at the very least updating the segment should update routing manager ... right?


Bootstrapping cargo - instructions unclear

Hi,
the readme says:

If, however, you have neither rustc nor cargo previously installed you can run:
python -B src/etc/install-deps.py
./configure --local-rust-root="$PWD"/rustc
make
make install

This seems to be outdated.
The src/etc/install-deps.py doesn't seem to exist anymore.
Also, running configure says:
configure: CFG_CARGO            :=
configure: error: needed, but unable to find any of: CFG_CARGO cargo

I'm wondering now how does one compile cargo on a system without cargo.
Thanks!

Oh this is actually just a case where we forgot to update the docs! Cargo now requires a previous installation of Cargo/rustc to be present to build Cargo.

What to do if I do not want to run cargo binaries not built by myself?

Cargo requires a Rust compiler to build, and all Rust compilers should be coupled with Cargo, so you can run that Cargo you've got on hand.

@starius There's @dhuseby/cargo-bootstrap

@vojtechkral Thanks!
I wrote a script installing Rust and Cargo locally:
https://github.com/starius/config/blob/master/.bin/install-rust

@alexcrichton

Cargo requires a Rust compiler to build, and all Rust compilers should be coupled with Cargo, so you can run that Cargo you've got on hand.

Which directory does cargo get installed into after ./configureing and make installing rustc?

@carlorosati right now make install in the rust repo doesn't install Cargo (but it will eventually!)

@alexcrichton Temporarily I will be using the cargo release tar.
As of now, I cannot find these release tars documented anywhere on the rust website(s). Is there any plans on documenting these or are they meant sheerly for developers.

They're mostly just meant for developers currenlty


Add first content on redux middleware

Add a description of what redux middleware is, some examples of what it can be used for, and how to add it to your app.

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @ccolin84 for the article and congratulations on your first PR merge to this repo. 🎉


[Content]: Inflection Destination documentation has wrong details

What article on segment.com/docs is affected?
https://segment.com/catalog/integrations/inflection/
What part(s) of the article would you like to see updated?
The page mentions that Inflection supports page, screen, identify and track calls, but Inflection only supports Identify, group and track calls.
Additional information
No response

made pr to address: https://github.com/segmentio/segment-docs/pull/5525

Closed with the merge of #5525. thanks @tcgilbert!

Thank you!


Gradle 7.0 problems regarding "missing an input or output annotation"

Some problems were found with the configuration of task ':foo-web:startFooWeb' (type 'JavaExecFork').

  - Type 'JavaExecFork' property 'environment' is missing an input or output annotation.
    Reason: A property without annotation isn't considered during up-to-date checking.

    Possible solutions:

      1. Add an input or output annotation.
      2. Mark it as @Internal.

    Please refer to https://docs.gradle.org/release-nightly/userguide/validation_problems.html#missing_annotation for more details about this problem.

  - Type 'JavaExecFork' property 'executable' is missing an input or output annotation.

    Reason: A property without annotation isn't considered during up-to-date checking.

    Possible solutions:
      1. Add an input or output annotation.
      2. Mark it as @Internal.

    Please refer to https://docs.gradle.org/release-nightly/userguide/validation_problems.html#missing_annotation for more details about this problem.

  - Type 'JavaExecFork' property 'killDescendants' of type boolean shouldn't be annotated with @Optional.
    Reason: Properties of primitive type cannot be optional.

    Possible solutions:
      1. Remove the @Optional annotation.
      2. Use the java.lang.Boolean type instead.

    Please refer to https://docs.gradle.org/release-nightly/userguide/validation_problems.html#missing_annotation for more details about this problem.

  - Type 'JavaExecFork' property 'process' is missing an input or output annotation.
    Reason: A property without annotation isn't considered during up-to-date checking.

    Possible solutions:
      1. Add an input or output annotation.
      2. Mark it as @Internal.

    Please refer to https://docs.gradle.org/release-nightly/userguide/validation_problems.html#missing_annotation for more details about this problem.

  - Type 'JavaExecFork' property 'workingDir' is missing an input or output annotation.
    Reason: A property without annotation isn't considered during up-to-date checking.

    Possible solutions:
      1. Add an input or output annotation.
      2. Mark it as @Internal.

    Please refer to https://docs.gradle.org/release-nightly/userguide/validation_problems.html#missing_annotation for more details about this problem.

@melix Thanks!

@psxpaul We are running a patched version of gradle-execfork-plugin with https://github.com/psxpaul/gradle-execfork-plugin/pull/38 and https://github.com/psxpaul/gradle-execfork-plugin/pull/37 now, and I can confirm that it runs great in a Gradle 7 project (currently running 7.0-milestone-3).


binanceApiService.getServerTime().equal(new BinanceApiCallbackAdapter<>(callback));

BinanceApiAsyncRestClientImpl.java
line 52 write a wrong method,equeue  wrong to ‘equal()’
 @Override   public void getServerTime(BinanceApiCallback<ServerTime> callback) {     binanceApiService.getServerTime().enqueue(new BinanceApiCallbackAdapter<>(callback));   }

Hi @donaldyang, could you please update your repository? This was fixed a few days ago.


Missing ability to count tokens before sending a query

Is it possible to add a functionality to this library that counts tokens prior to sending a query to ChatGPT? It would be beneficial to have such a feature, as it could allow the correct model - either gpt-3.5-turbo or gpt-3.5-turbo-16k - to be set before submitting the request. This can be achieved using the JTokkit library. An example of this implementation can be found here.

Hey @piotrkuczko ,
Thanx for the suggestion. The lib looks solid. Will add it to my list.

PR -> https://github.com/cequence-io/openai-scala-client/pull/43


Formatter Updates Deprecated Specialization Declaration Parameters

This PR implements issue #1202, i.e., added an update rule that appends deprecated specialization declaration with a ... parameter.
To do so, I first modified the syntax tree definition of SpecializationGenerator.Provider. Previously, all the parameters are regarded as a Terminal without a detailed structure. Therefore, updating and formatting rules cannot be applied. For example, the extra space in controlled(q, ...) cannot be removed. Instead, I define Provider.parameters as specializationParameter tuple, which solves the problem.
Based on the previous definition, I implemented the rule. It updates the parameters based on the number and the text of existing parameters.
The code example used in the issue is added to Examples.fs as a test case.

/azp run


Webhook configure on a provider level only

Is it possible to have a webhook configured on provider level only? Such that when any of the consumer contract changed that involves the provider, the webhook will be triggered? Through this way also, new consumer contracts automatically trigger the webhook configured on provider (thus not needing a new webhook for that contract).

This idea makes a lot of sense. I'm happy to accept a PR if you're interested in developing it.

Most of the code will exist already.

Thanks. Will do that later

Done.


Fixes title in activator properties and strings

Sorry for the duplicated work, could you review? Thanks

LGTM


Error downloading 5.9.0

When I was notified that #199 was fixed, I noticed that 5.9.0 was released and updated to that instead, so I switched to that, but now I'm getting this error:
Failed to find a valid digest in the 'integrity' attribute for resource 'https://cdn.amplitude.com/libs/amplitude-5.9.0-min.gz.js' with computed SHA-256 integrity 'WQe+NZuuojWTX5h+FwRIEemfwqBvrMZRsZVvRiz8IHw='. The resource has been blocked.

@daveisfera when you update the version, you'll also need to update the hash. Can you update it to the correct value based on what's here: https://github.com/amplitude/Amplitude-JavaScript/blob/master/src/amplitude-snippet.js#L5

👍 thanks


[Enhancement] ChatImage Support

Checks

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting.
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly.

Description
I've done some experimenting, and I suspect the bare minimum for this feature should be rather easy to implement.
Using a mod called ChatImage, it is possible to show images in Minecraft chat with nothing more than a url that links to an image, it even works with some GIFs.
I was able to send an image through Discord to Minecraft using MCDC and ChatImage, but I think it'd be really cool if MCDC was able to add the necessary text to the beginning and end of an image url sent in Discord automatically, such that ChatImage would recognize it.
I believe it would also be possible to show images sent in Minecraft using ChatImage in Discord, but currently it is not easily done because of the way ChatImage requires you to format your message.
I have yet to reach out to the developer of ChatImage about this. The mod does support other Discord bridge mods, but they are far less clean or user friendly than MCDC in my opinion.
Will add screenshots soon.

I found multiple Minecraft plugins/mods called ChatImage. Is https://github.com/kitUIN/ChatImage the one you are referring to?

Yes that's it.

I was able to send an image through Discord to Minecraft using MCDC and ChatImage, but I think it'd be really cool if MCDC was able to add the necessary text to the beginning and end of an image url sent in Discord automatically, such that ChatImage would recognize it.

It does not seem sensible for MCDC to fulfil another mod's specification ([[CICode,url=<url>,name=<name>,nsfw=<nsfw>]]).
This is indeed more of a feature request for automatically decorating recognized image URLs with default parameters, and I suggest it be submitted on the ChatImage side.

I believe it would also be possible to show images sent in Minecraft using ChatImage in Discord, but currently it is not easily done because of the way ChatImage requires you to format your message.

In fact, the Discord client has the ability to parse image links directly. While this may result in losing functionality for the nsfw and name parameters, providing the mentioned option would help improve ChatImage's compatibility with most mods.
The compatibility of local file:/// images with Discord is questionable.
Additionally, the format of ChatImageCode causes image URLs and hyperlinks to be incorrectly recognized and truncated, which is another issue that has to be addressed.
You can reference this issue for them when you submit an issue on their side.

This is indeed more of a feature request for automatically decorating recognized image URLs with default parameters, and I suggest it be submitted on the ChatImage side.

The creator of ChatImage said they will not make it recognize image URLs as it would be too resource intensive. (I don't remember where they said that, and I really don't believe it but that's what they said.

It does not seem sensible for MCDC to fulfil another mod's specification ([[CICode,url=<url>,name=<name>,nsfw=<nsfw>]]).

I was thinking more along the lines of adding a custom message option for specifically images or links sent through Discord, so you could add the CIC yourself through MCDC.

Thanks for checking it out, and sorry I never added screenshots.

It does not seem sensible for MCDC to fulfil another mod's specification ([[CICode,url=<url>,name=<name>,nsfw=<nsfw>]]).

I was thinking more along the lines of adding a custom message option for specifically images/links/embeds sent through Discord, so you could add the CIC yourself through MCDC.

This is indeed more of a feature request for automatically decorating recognized image URLs with default parameters, and I suggest it be submitted on the ChatImage side.

The creator of ChatImage said they will not make it recognize image URLs as it would be too resource intensive. (I don't remember where they said that, and I really don't believe it but that's what they said.

In fact, the Discord client has the ability to parse image links directly. While this may result in losing functionality for the nsfw and name parameters, providing the mentioned option would help improve ChatImage's compatibility with most mods.

This is what I meant when I said it wasn't really that easy due to the formatting, never expected this to be implemented, but I figured it may or may not be possible to recognize a CIC message and turn it into something that Discord can parse (an image link, maybe marked as spoiler if nsfw).

I was thinking more along the lines of adding a custom message option for specifically images/links/embeds sent through Discord, so you could add the CIC yourself through MCDC.

Good idea! This should resolve Discord to Minecraft messages compatibility. ╰(*°▽°*)╯
As for Minecraft to Discord messages, I thought I would submit an issue on ChatImage side first to try and see if I can get better opinions from its developer.

Adding some context for interaction between ChatImage and DMCC.
Below is how images are normally formatted when sent through Discord:

While this is when the formatChatMessages setting is off:


Correct recipient type for email builder cc and bcc

The builder has cc and bcc methods that take a recipient.  The new recipient created from the one passed in has the RecipientType set to 'TO'.  I would expect it to be set to RecipientType.CC, or RecipientType.BCC.

Thank you, I will release a new minor version to maven central soon.

You are welcome. Thanks for the library. Very nice to use.

On Aug 11, 2015, at 9:59 PM, Benny Bottema<EMAIL_ADDRESS>wrote:
Thank you, I will release a new minor version to maven central soon.
—
Reply to this email directly or view it on GitHub.


Add function-based patterns and improve code organization

This PR adds support for function-based patterns while improving the overall organization of the code. While I apologize for the large diff, these changes help support the new pattern functionality.

Added function-based patterns in mappings

Functions should return a table containing one or more captures, or nil for no match
Added docs and tests for function-based patterns

Refactored state management to be more explicit
Added type annotations
Improved error handling around option access
Added fallback values for configuration
Better organization of utility functions

All existing functionality should remain unchanged and all tests are passing, except for the livewire test that I've commented out. That test was failing before any of my modifications, and I wasn't able to figure out exactly why.
Fixes #58

Wow, that's quite the PR—thanks a ton for putting in the effort! 🙌 I'll run a few tests, and as long as everything checks out, I'll go ahead and merge it. 😊

Thanks again, really great pr - very much appreciated! Really nice refactoring and implementation! 🚀

Since your Lua skills clearly surpass mine, I’ve sent you an invite to collaborate on the repo. 😄

Awesome, thanks for merging! I will keep my eye on the issues to see if the changes broke anyone’s workflow and try to fix them if so.


Specify tox environments in format from official documentation.

Closes #87
The official tox
documentation (https://tox.wiki/en/latest/config.html#tox-environments)
shows python versions in a short format (py37, py38, py39, py310).
The previous commit to this block (d2342f3) changed it from the
preferred format to python3.x. This does not work in Linux
environments. With version names in the (non-preferred) format, when
running tox, the program cannot find the python versions and
defaults to the system python for every version. When using the
preferred (short) names, tox works as expected.

@malandis Tested on my mac after whiping out .tox dir was able to loop through all the versions properly w/ new tox.ini format.

Thanks Ellery. Tyler was also able to reproduce this, so this is good.


use non-deprecated getters for WitnessCS

There is a deprecated warning in the result of the check-lurk-compiles job in this pending pr due to a change that was made in bellpepper see: https://github.com/lurk-lab/bellpepper/pull/48
This change uses the new public getters for WitnessCS to resolve the deprecated warnings.

Tyvm! This was under my target but I hadn't written an issue about it yet.
Btw, there's a similar use (or maybe multiple uses, I don't remember) of this deprecated API in https://github.com/lurk-lab/neptune/


NOTE TO BE MERGED: Move test code out of extension

https://github.com/microsoft/vscode-python/issues/8053
I'll test this and get it working later (i.e. finish this later).
Basically created a delegate that gets updated in test environment (when running tests).

This is also used when opening the index.html so that the view actually has data in it. That's what it was originally used for.

Aah got it.

Closing as this has been resolved by @rchiodo


Specs for renewal dependent add

Regarding #publish test failures
https://github.com/dchbx/gluedb/blob/release-4.2/app/models/enrollment_action/renewal_dependent_add.rb#L15
#added_dependents makes that call on L15 which utilizes RenewalComparisonHelper. This is included into the class so you need to mock the request at that level.
https://github.com/dchbx/gluedb/pull/73/files#diff-78fc4216cf711aac3433c87c4ee48b1cR135 In this example the subject is the class itself so I only had to mock the subject request.
In your case you need to allow subject.class to receive a message ':same_carrier_renewal_candidates' with an action as an attribute. It should respond with an array of policies (I think) that should have enrollees which is an array of elements that have a m_id attribute.
Basically it's saying my current action includes members [1,2,3] does this policy have enrollees with m_ids that represent an extra id from what we started with.


Incorrect BlockActionPayload definition

The Action class has a field initialOption of type Option, however, it is not correct,.
It works alright when the block action is submitted through a button click, however it does not work with static inputs.

Thanks for flagging this! I've created a PR to fix this.


suggestion

What problem does this feature solve?
Can you provide the starter of the locked version, similar to the SpringBoot in Java? In other words, many components are easy to use in vue 2 version and may not be easy to use in vue 3 version. If I use the official starter, then I will no longer worry about the version of each component. Of course, this will increase the official work and is just a suggestion.
What does the proposed API look like?
no

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


[SPARK-33269][INFRA] Ignore ".bsp/" directory in Git

What changes were proposed in this pull request?

After SBT upgrade into 1.4.0 and above. there is always a ".bsp" directory after sbt starts:
https://github.com/sbt/sbt/releases/tag/v1.4.0
This PR is to put the directory in to .gitignore.
Why are the changes needed?

The ".bsp" directory is an untracked file for git during development. This is annoying.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Manual local test

Test build #130366 has started for PR 30171 at commit 3bb31f3.

It doesn't affect anything in the build. Let me just merge it in.

Merged to master.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34969/
Test FAILed.

Test build #130366 has finished for PR 30171 at commit 3bb31f3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130366/
Test PASSed.

Thank you, @gengliangwang and @HyukjinKwon .


Gitignore : add enforce list of files

On some project, we might need to add a ddb.local.yml file in order to override a specific set of settings.
We could add it into gitignore files but it would be great to automatically be added on configure command execution.
We need to add the following setting :

key : enforce
type : array of string (full name or regex)

Regex for all variations of ddb.local.yml : .?ddb.local.ya?ml


As a logged-in user, I could invite my friends or randomly match other users to play Who is undercover.

Acceptance Criteria:

[x] The game requires at least three players, with  maximum limit 8.
[x] Each player randomly gets the same word, only one player gets another word.
[x] Each player in turn describes the word they get, and when a round ends, everyone has to vote for who they think is undercover.
[x] The player who gets the most vote are out, the game continue until there is only two players or undercover is spotted
[x] If undercover is still in the game at the end, then undercover wins and gets two points, otherwise everyone except undercover gets one point.

Development Tasks:

[x] sopra-fs23-group-26/client#30
[x] sopra-fs23-group-26/client#31
[x] sopra-fs23-group-26/client#32
[x] sopra-fs23-group-26/client#33
[x] sopra-fs23-group-26/client#34
[x] sopra-fs23-group-26/client#35
[x] sopra-fs23-group-26/server#80
[x] sopra-fs23-group-26/server#81
[x] sopra-fs23-group-26/server#82
[x] sopra-fs23-group-26/server#83
[x] sopra-fs23-group-26/server#84
[x] sopra-fs23-group-26/server#85
[x] sopra-fs23-group-26/server#86
[x] sopra-fs23-group-26/client#67
[x] sopra-fs23-group-26/client#68

28e581801b50306b797e112d535bb4f8c6d82bb3


Documentation for Database Ownership and Setup is incomplete and potentially confusing

The instructions for the Helm Charts indicate that the installer can choose whether they want to create a new database instance via the Helm Chart, or use an existing database instance.
What is missing is an explicit statement that if they choose to use an existing database instance, that database must be configured in advance by installing the schemas for the Spotfire configuration, either manually by running the install scripts supplied in the install files, or by using the config.sh create-db command.
So really, there are 4 options:

Create a database instance using Helm charts - everything is taken of by the Helm Charts
Use an existing Spotfire database instance - no action required
Use an existing Spotfire database Instance of an older version - requires the upgrade script to be run
Use an existing database instance that does not contain a Spotfire schema - requires the install steps to be performed

Trying to do either of the last two without performing the required steps on the database will result in an error.  One example of this is as follows:
Successfully created the bootstrap configuration file /opt/tibco/tss/tomcat/webapps/spotfire/WEB-INF/bootstrap.xml

Error during server validation : Error reading server configuration from the database: No configuration found

In this case, most likely the user tried option 4) but did not setup the database instance with a valid Spotfire Configuration.

Hi Dave,
If you set the values in for the helm charts "correctly" depending on your requirements, then the charts will take care of the all 4 our the scenarios you've listed. Of course errors can happen, but the charts are build to be able to cope with those 4 scenarios.
In scenarios 1, 3 and 4 you should set "database.create-db.enabled" and "database.upgrade" to true(and also set the values for the the required "database." values like "database.create-db.databaseUrl" etc.) and the deployment will take care of creating needed schemas and updating the database during deployment.
The error you mention could also happen in scenario #1 if you set "configuration.useExisting" to true, as the charts then expect a configuration to exist in the database.

I think we can close this issue.


Add sitemap generator

Simple metalsmith plugin to generate a sitemap with sitemap.js
https://deploy-preview-337--govuk-design-system-preview.netlify.com/sitemap.xml
Required options:
hostname

Optional options:
changefreq
lastmod
priority
filename -> default `sitemap.xml`
destination -> default `deploy/public/design-system`
pattern: -> default `**/*.html`

It will loop through all metalsmith files and check for two frontmatter properties:
ignore_in_sitemap (Boolean) : if set to true it will ignore the file
canonical (String) : url to be used instead of the canonical data set by metalsmith-canonical plugin.
Url is relative to the hostname
This is related to https://trello.com/c/31ttWqd3/1047-add-site-map-to-the-design-system but not required to complete it

You can preview this change here:
Built with commit 99908803f83950fc01acc2b2f9760965f5714dd5
https://deploy-preview-337--govuk-design-system-preview.netlify.com

update to work with the canonical plugin we use

excluded all the example pages


[ProgramTest] Transaction not processed when signed with a custom keypair

Summary
I am trying to test a program using  ProgramTest but I need to sign the test transaction with a different keypair than the payer provided by the helper. Unfortunately, this seems to cause the transaction to timeout. The reason I am using another keypair is that I need the public key of the keypair to use as an owner for some token accounts used in the test. I apologize if I am misunderstanding something terribly, but I have created a simple reproduction of the issue as a PR here: https://github.com/Konosh93/rusty-sol/pull/1
Goal

Demonstrate how to use another keypair to sign transactions when running e2e tests with ProgramTest.

Expected

Start transaction processing.

Actual

fail to process the transaction

running 1 test
[2021-09-20T03:27:47.673069000Z INFO  solana_program_test] "another-payer" program loaded as native code
test test_escrow has been running for over 60 seconds
test test_escrow ... FAILED

failures:

---- test_escrow stdout ----
thread 'test_escrow' panicked at 'called `Result::unwrap()` on an `Err` value: IoError(Custom { kind: TimedOut, error: "Client dropped expired request." })', tests/functional.rs:56:57
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    test_escrow

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 60.33s

error: test failed, to rerun pass '--test functional'

Hi there, apologies for the late response, but we don't typically use GitHub for tech support, and invite you to join us on Discord if you haven't already made your way there: https://discord.gg/JJkWhPjb
To quickly answer your question, you need to fund the other keypair that you're signing with, so you can just do a transfer from the payer to the other keypair.


More effient graph-edge storage, some docs

Hey @b-ma , for your information. Docs are still a work in progress!

Hey, looks pretty good and comments are really helpful to understand better the graph logic. I ran through the whole graph.rs file this time and I have 1 related question / remarks, why is this necessary to loop through all the nodes:
// visit all registered nodes, depth first search
self.nodes.keys().for_each(|&i| {
  self.visit(
    i,
    &mut marked,
    &mut marked_temp,
    &mut ordered,
    &mut in_cycle,
  );
});

At first sight, I wonder why this is necessary to loop through all self.nodes because as the graph is guaranteed to have one single output (.e.g. listener) and visit is recursive, something like (without the loop) should do the trick no?
// visit all registered nodes, depth first search
self.visit(
  ListenerIndex,
  &mut marked,
  &mut marked_temp,
  &mut ordered,
  &mut in_cycle,
);

I would imagine that this could gives you all the nodes that are currently connected, and would more be deterministic as it wouldn't rely anymore on keys and as the connections are sorted in SmallVecs?

Why is this necessary to loop through all the nodes there?

Great question. I think the main thing to realize is that there may be pieces of the audio graph that are not connected to the destination node.
For example, you may want to use the web-audio-api to create a visual spectogram of a sound input. In this case, you don't want to play the audio on the speakers, but you want to use an AnalyzerNode and write the frequency values to a html canvas. This means we will never reach the audio input node if we would start from the destination node and visit the connected nodes from there.
My implementation is much inspired by https://webaudio.github.io/web-audio-api/#rendering-loop . Definitely worth reading.
Sidenote, the recursive implementation is a bit strange in rust because of all the mutable borrows I need to carry around. I may rewrite it into a sequential (non recursive) variant. Let me check if that makes it more readable.

Ok good point, really didn't think of these cases (that would make a nice comment in the code :). For the recursive implementation I think it's quite readable actually, there are some Rust weirdness for sure... but it's also kind of pedagogical for newbies such as me.

note: Maybe it would be interesting to try to see with the spec people if defining a kind of Output or Sink type (which can be only a few nodes I guess), just as there are well defined Sources wouldn't worth the effort?

Interesting thought, but I think this will be hard in practise. For example, you could set up a microphone input and connect it to a large delay node. Then after a minute or so, you could connect the delay node to the output node. In this case, the mic and delay should really render every frame, even though they are not (yet) connected to the output. Basically any non-pure Node (means: having side effects such as caching, moving data to user side, etc) should be considered a Sink in this case, which could be quite many of them. This would decrease the usefulness in my opinion


handleToolStart no longer passes the tool name

As of langchain version 0.0.95,
// used as part of agent executor .call()
BaseCallbackHandler.fromMethods({
          handleToolStart: (tool, input): Promise<void> | void => {
             console.log(tool)
          },
        }),

gives
{
  lc: 1,
  type: 'not_implemented',
  id: [ 'langchain', 'tools', 'SearchTool' ]
}

but two versions ago, it used to have a name property of the tool being used.
I use the property to show that a particular tool is being used by the agent.
Edit: it looks like 0.0.93 introduced the breaking change:
https://github.com/hwchase17/langchainjs/compare/0.0.92...0.0.93#diff-55008d5075c76fcd778bb75e24c8775de2c23b6a6a6d02c8e30630fe8922e685R22

@theogravity did you manage to fix this?

You mean find a workaround? Nope.

It's relevant


Metric charts

It would be great to support different charts for availability and gauges metrics just like Console does. For availability it uses a bar chart and for gauges it uses a line chart.
This will require some determination to find out the type of a metric from the Inventory side. Metrics response looks like this and Metric response looks like this for gauges. As far as I can see there are no proper fields in models to determine the type of a metric.
Of course it is possible hard-code type association, i. e status.duration.type is gauges and status.code.type is availability, but Hawkular is not limited to these types.

Partially affected by the upstream issue.
https://issues.jboss.org/browse/HAWKULAR-452

Another upstream issue.
https://issues.jboss.org/browse/HAWKULAR-453

just a heads up, it was done on the inventory side 2 weeks ago
here is how it looks like:
curl -X GET -H 'Content-Type: application/json' 'http://jdoe:password@<IP_ADDRESS>:8080/hawkular/inventory/metricTypes/status.code.type' | less

{
  "path" : "/mt;status.code.type",
  "unit" : "NONE",
  "type" : "GAUGE",
  "tenantId" : "28026b36-8fe4-4332-84c8-524e173a68bf",
  "id" : "status.code.type"
}

side note:
btw. you may want to present the traits collected by the pinger component for the 'URLtyped' resources:
curl -X GET -H 'Content-Type: application/json' 'http://jdoe:password@<IP_ADDRESS>:8080/hawkular/inventory/resourceTypes/URL/resources' | less

[ {
  "path" : "/e;test/r;ed646a3334ca891fd3467db131372140",
  "type" : {
    "path" : "/rt;URL",
    "tenantId" : "28026b36-8fe4-4332-84c8-524e173a68bf",
    "id" : "URL"
  },
  "properties" : {
    "trait-collected-on" :<PHONE_NUMBER>117,
    "trait-powered-by" : "gws",
    "trait-remote-address" : "<IP_ADDRESS>",
    "url" : "http://www.google.com"
  },
  "environmentId" : "test",
  "tenantId" : "28026b36-8fe4-4332-84c8-524e173a68bf",
  "id" : "ed646a3334ca891fd3467db131372140"
} ]

Sent as #42.

Merged.


Usage of 'next' and 'last' with weekdays

Would love to see the ability to get dates for "next monday", "last monday" and other similar usages.

Hi @mrj1m0thy!
There is an open PR that tries to implement it but it's not finished: https://github.com/scrapinghub/dateparser/pull/638
I also would love to have it, so I will try to push to add it in the next release.
Cheers!
closed as duplicated of this: https://github.com/scrapinghub/dateparser/issues/573


publications: "Categorize by" rather than "Filter by"

This is a small improvement proposal for the publications list. I don't think there is much need for a "Filter by Publication_type/Year" option (people don't want your publications from a specific year, but rather want to be clear when you wrote every publication) as much as a Categorization/Order by Publication_type or Year. (The only "Filter" I think would be useful for such a list is a Filter only publications as first author - see #323 ).
I would typically see it as a button (similar to the projects' tags selector on home/projects) allowing to choose the categorization:
Order by [ Date | Publication type ]

if Date:
--------------------- 2017 ---------------------
- JournalArticle 2017 [2017-10-01]
- ConferencePaper 2017 [2017-08-01]
- JournalArticle 2017 [2017-05-01]
--------------------- 2016 ---------------------
- JournalArticle 2016
--------------------- 2015 ---------------------
- ConferencePaper 2015

if PubType:
--------------------- Journal articles ---------------------
- JournalArticle 2017 [2017-10-01]
- JournalArticle 2017 [2017-05-01]
- JournalArticle 2016
--------------------- Conference papers ---------------------
- ConferencePaper 2017 [2017-08-01]
- ConferencePaper 2015
--------------------- Book chapters ---------------------
etc.

Exemple of display/style (from http://srv.uib.es/david-gessner/ ):

I like the idea to separate publications by year using a line which provide a much clean and nice visualization. But "filter by Publication_type" still is a good feature as readers will be interested to see only journal or conference publication.

@byzheng I think @sbibauw  also means to have the feature "categorized by Publication_type".
I would like this feature, if it would be implemented

@byzheng Yes, as @jagd said, the idea is to have BOTH (year/type) options to order the present the publication type. The main idea is to have a visual categorization/ordering of either year or publication_type, rather than a filter which hides the other types (which I don't think is needed, since people don't want to hide some publications, they just want to see them in a specific order or presentation).

@sbibauw Agree with you about categorisation by year is really useful feature. But I think filter by publication type is still useful in some cases. For example

some researchers have hundreds of publications (journal, conference, book chapter). Filter by types would provide a simple way to reduce publication number in one page. A long page to view if all publications in one page.
Readers will be only interested to my journal publications as they think they are peer reviewed. Although categorisation can provide the same feature, filter can quickly show the information reader needs. Don't have to skip a few pages of conference papers to find journal papers.
The link by publication type can be easily to share with others for a specified type.

@byzheng I see. I still think that dividing the publications by type (and possibly have a quick list of types at the top, with anchor links to each category, so that you can jump to a specific category) would serve this purpose. But if some people prefer the Filter by option, it could be left to the user to choose, with an option in config.toml

Also, given we are able to add projects to publications, it would be nice if we can look at publications by project.
Depending on the categorization topic chosen, 'Year/Decade', 'Publication type', 'Project', ..., it would be nice to have all possible options up front and then we can go to the section of interest by selecting and then use Top to navigate back up: a non-Hugo example: http://mpf.biol.vt.edu/lab_website/publications.php#
For year, for newbies (like me), it might make sense to specify years. For senior professors, decades might make more sense. A simple way to customize that might be great too!
thanks for considering. 👍

@gcushen Is this definitely discarded? I really believe it is more important to put some order with categories in (potentially very long) publication lists, rather than allowing filtering them.

@sbibauw I closed the issue as it's stale. No one has contributed a PR and everyone who has commented appear to have slightly different preferences.
If we look at how the academic social networks list publications, academia.edu groups by publication type (which is not useful when there are large number of publications) and filter by publication type (which we already have). On the other hand, ResearchGate doesn't appear to offer group by or filtering, but have a search box instead. I would expect they have carried out some kind of research in terms of optimizing the use of group-by/filters to determine the best approach, but perhaps I'm too optimistic.
Although quite a few labs just group publications by date, I don't really see the benefit of this since it's already ordered by date anyway and if you are looking for a specific publication, it's easier just to search for it using the search widget.


fix: ensure only pages with allowed extensions are required

What kind of change does this PR introduce?
Bug fix
What is the current behaviour?
Pages with extensions different the pageExtensions option might still be loaded.
What is the new behaviour?
Make sure to only load pages with extensions defined by config file.
Does this PR introduce a breaking change?
No.
What changes might users need to make in their application due to this PR?
Other information:
Please check if the PR fulfills these requirements:

[X] Tests for the changes have been added
[ ] Docs have been added / updated

Coverage remained the same at 100.0% when pulling 7fb26d01c914c8fc6a1db2de70df1badfa162cbe on page-extensions into 3fcfd3eedede027f887c6bdb9c56b617216c61b2 on master.


New release

Hello
When you decide to release a new stable version of bandit? The old version on PYPI works correctly only on python 3.7 (know several issues about it). I want to use newer python (at least 3.8) and use all new features.
Thanks

This looks to me to be a duplicate of #611.
(ps. I am also interested in a new release but the maintainers should perhaps not be bothered by a lot of duplicate issues. ds.).

Ok, close ticket because of duplicate


Make Rule Manager and filters internal

Since we are still determining whether transactions can work with rule operations, we should leave rule manager and the filters internal for Preview 2.

Closed with https://github.com/Azure/azure-sdk-for-net/pull/11719


Allow React 16 as peerDependency, as well as update compatible packages

thanks @hwrod !
Approved. Please be free to give feedback / PR and so on.

Cheers! Please also npm publish to the npm repository: https://www.npmjs.com/package/react-d3-treemap

done!


Add Cobalt.io Connector

This adds a connector/task for Cobalt.io (full disclosure, I'm a software engineer at Cobalt.io). It works by pulling in findings and asset information from the Cobalt.io public API. I based the task implementation and corresponding README off of Snyk's task.
Some notes on the task that may require additional attention:

Findings marked as "wont_fix" and "valid_fix" in Cobalt are imported and flagged as "risk_accepted" and "resolved" respectively. Alternatively these could not be imported at all.
Findings marked as "duplicate", "out_of_scope", or "invalid" in Cobalt are not imported at all.
It currently imports all findings for all assets in Cobalt.io. I implemented it this way because that's how Snyk's task seems to work. A reasonable alternative would be to import findings from one asset or one pentest. Note that in that case there will always be only one Kenna Security asset in the imported dataset.

One open question from my side is whether to import findings as vulns or as findings. The difference is not quite clear to me, except that when I import findings from Cobalt as findings I cannot find them in the Kenna Dashboard.

Might want to add it into the main README.md as well.

I'm not sure what you're referring to @jcran, I did add it in the README.md at the root of the project in the "Toolkit Capabilities (TASKS)" section? I didn't see any other list of tasks in that README.md, and wouldn't know what other README.md is the main one.

Ah sorry, missed it at first glance. Great work @ericcornelissen!

@ericcornelissen I added a number of comments. We can schedule a call to review if you want. There are essentially 2 models where cwe/app type findings can go, under the VM model used for network assets (servers and such) or under the newer Findings model. It is mostly the same data but the findings model has more the statuses you are looking for. Under the VM model, there are only 2, open and closed. If you want to test with the findings model, you would call the add asset finding method and I would need to turn on that model in your instance as it is currently under development. If you look at Contrast as an example you will see that they made VM vs Findings an option via param. I can create you an additional instance if you want to use findings. A kenna instance can support one or the other for files and URLs.

Thank you for the review @lindambrown. I think a review call would be helpful. Thank you for the explanation, it does explain why I couldn't see the imported findings in our instance. I would indeed agree that Kenna's findings model better suits Cobalt's data. bit since that is still under development it would make sense to provide support for both findings and vulns. From that perspective, having two test instances would be ideal for us.


Add a FIFO evictor

Add a FIFO evictor.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3194/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5357/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5360/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5361/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5362/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9446/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder-jdk11/49/
Test FAILed.

Can one of the admins verify this patch?


Fix #127

Fixes #127.
Should we provide some kind of automatic migration process?

I'd say that if we were changing the config and log folders after have a stable release, then it might be wise to have a migration step. Otherwise I think people running alpha releases should be content with having to manually rename the folders, or copy over the files, or just have to start with a clean slate. My 2 cents, anyway


Error: qdrant initialization failed

Describe the bug
I had compiled and installed bloop on ubuntu 18.04.5,
i executed these commands
cargo build -p bleep --release
cargo run -p bleep --release -- --source-dir /home/eric/

but i can't start bleep. when i execute
./target/release/bleep
i got the error
Error: qdrant initialization failed

Caused by:
    Qdrant initialization failed. Is Qdrant running on `qdrant-url`?

Expected behavior
i want start the application
To Reproduce
How can we reproduce the bug? Helpful information could include:

Details about your system
ubuntu 18.04.5

The data that you had indexed

The query that you ran

Screenshots or output
If applicable, add screenshots to help illustrate the bug.
Additional context
Add any other context about the problem here.

It looks like you need to run qdrant separately as docker container:
https://qdrant.tech/documentation/quickstart/ it will pick it up


UiConstants should not call EmsEvent class methods

Currently, UiConstants cache EmsEvent configuration values.
Unfortunately, it gets loaded early in our process and loads the database models.

memoize 2 EmsEvent config variables in EmsEvent rather than in UIConstant.
leverage .present? instead of !x.nil? || x == "", and .presence which is basically x.blank? ? nil : x
reduce duplication when composing the event_set.

/cc @lfu this should fix the migrations spec for #1527
/cc @gmcculloug Let me know if this is too intrusive. While a few were obvious, and a few were more tame that we could have gone, it still is a bit of change
/cc @gmcculloug tl_filter_options pushes an array when a :critical or :detail value. so it will be [a,b,[c,d]] - which doesn't quite look right to me, but I didn't want to change.
/cc @Fryguy do you like the memoization?
/cc @dclarizio This changes UIConstants. Think @chessbyte would like to reduce that class a bit.

@dclarizio @h-kataria @martinpovolny Please review.


Update bower.json

version does not fit release

You are right, I don't know why that happened. Thanks for pointing it out.


Switch to pbr

To remove some incorrect information from setup.py, and to also stop
duplication of requirements in two places, switch to using pbr. This
also buys us an AUTHORS and ChangeLog generation for free, add them.

My only problem with this is that you dont get python setup.py install to install dependencies automatically, while python setup.py test does install everything, which is something I never understood (sure, openstack "we dont support this method" is the reason but its inconsistent IMHO)
Rest looks awesome, less code to maintain the better!

My only problem with this is that you dont get python setup.py install to install dependencies

So do we need to adapt REAME's installation instruction?

So do we need to adapt REAME's installation instruction?

No as it works and in the readme we explicitely say to use pip install -r requirements.txt but people have a tendency of not reading readme's and just do a python setup.py install and expect dependencies to be installed along.
In any case, this will be sent prepackaged to customers right? So we should not really care much about it.

True, we package it. But some users  might want to get it from here...

The README instructions in HEAD will work as is.

I think I've addressed everyone's concerns, aside from @Itxaka 's setup.py install, which sadly is an upstream decision.


Don't overwrite loop variable

Description
Don't overwrite loop variable
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[ ] Konflux configuration change

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

Project coverage is 74.86%. Comparing base (7280586) to head (754f312).
Report is 28 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2062      +/-   ##
==========================================
- Coverage   74.89%   74.86%   -0.04%
==========================================
  Files          70       70
  Lines        2892     2892
==========================================
- Hits         2166     2165       -1
- Misses        726      727       +1

Files with missing lines
Coverage Δ

ols/utils/token_handler.py
63.07% <100.00%> (ø)

... and 1 file with indirect coverage changes

/lgtm

/approve
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"


java.lang.IllegalArgumentException while running

Thanks for building this promising piece of software! Looking forward to start scrobbling, but ran into following error message when running audiostreamscrobbler (configured with lastfm):
Starting 'BluOS' detector...
Starting LSDP discovery thread...
Detected player 'BluOS/NODE 2i' is managed by the 'Player Group' group.
All players of type 'BluOS' in group 'Player Group' have been found. Stop looking for other players of this type.
Starting monitor for player 'NODE 2i (Standard: BluOS, model: N125, version 3.6.12, IP address: <IP_ADDRESS>, MAC address: XX:XX:XX:XX:XX:XX)'...
Starting player polling thread for player 'NODE 2i (Standard: BluOS, model: N125, version 3.6.12, IP address: <IP_ADDRESS>, MAC address: XX:XX:XX:XX:XX:XX)...
Stopping 'BluOS' detector...
New song detected: '"Ancient Skin" (Jack Penate, "After You"), played 206 seconds of 232 seconds'
Could not update Playing Now for service 'lastfm': java.lang.IllegalArgumentException: Comparison method violates its general contract!
Stopping LSDP discovery thread...
New song detected: '"Swept to the Sky" (Jack Penate, "After You"), played 0 seconds of 282 seconds'
Could not update Playing Now for service 'lastfm': java.lang.IllegalArgumentException: Comparison method violates its general contract!

Some information on my environment:
java -version
openjdk version "1.8.0_232"
OpenJDK Runtime Environment (Zulu<IP_ADDRESS>-CA-linux_aarch32hf) (build 1.8.0_232-b195)
OpenJDK Client VM (Zulu<IP_ADDRESS>-CA-linux_aarch32hf) (build 25.232-b195, mixed mode, Evaluation)

gradle --version

------------------------------------------------------------
Gradle 4.10.3
------------------------------------------------------------

Build time:   2018-12-05 00:50:54 UTC
Revision:     e76905e3a1034e6f724566aeb985621347ff43bc

Kotlin DSL:   1.0-rc-6
Kotlin:       1.2.61
Groovy:       2.4.15
Ant:          Apache Ant(TM) version 1.9.11 compiled on March 23 2018
JVM:          1.8.0_232 (Azul Systems, Inc. 25.232-b195)
OS:           Linux 4.19.66-v7+ arm

I am running a Raspberry Pi 3 Model B.  Any idea what could be causing the issue?
p.s. I did retrieve the sessionKey from lastfm and added it to the config.json file

Fixed the issue by running:
java -Djava.util.Arrays.useLegacyMergeSort=true -jar audiostreamerscrobbler.jar

I am a happy scrobbler now!


Add marketo email form

We are swapping from Sendgrid to Marketo for email communications. This PR adds the Marketo email form.

cc @Peter-Metz @MattHJensen

Form is centered up now:


[Badge] Remove unused code

Reverts the parts of #24253 that affected the Badge. We cannot allow code in the codebase which we don't understand. If we do understand it, we should do everything we can to explain it. Code that is not understood or its purpose is kept a secret has no place in open source.

All style related functions (the styles, overrides resolver and the creation of the utility classes should use the styleProps as source of truth for the props). The implementation previously was wrong. What do you mean we don’t understand, or how come we understand the previous implementation and not this one?

All style related functions (the styles, overrides resolver and the creation of the utility classes should use the styleProps as source of truth for the props). The implementation previously was wrong. What do you mean we don’t understand, or how come we understand the previous implementation and not this one?

What do you mean we don’t understand, or how come we understand the previous implementation and not this one?

If you are not able to explain what code does, you do not understand it.

All style related functions (the styles, overrides resolver and the creation of the utility classes should use the styleProps as source of truth for the props).

Where is this documented? We should create an API that either catches mistakes or makes them impossible. Not create an API that is only understood by its creator.

All the components are meant to follow the same template.

Where is this documented?

What do you mean we don’t understand, or how come we understand the previous implementation and not this one?

If you are not able to explain what code does, you do not understand it.

All style related functions (the styles, overrides resolver and the creation of the utility classes should use the styleProps as source of truth for the props).

Where is this documented? We should create an API that either catches mistakes or makes them impossible. Not create an API that is only understood by its creator.

All the components are meant to follow the same template.

Where is this documented?

Also could you explain to me when we're allowed to use templates and when we need to create abstractions? So far it seems to me that this just depends on the person making the change not what we're actually doing.

Also could you explain to me when we're allowed to use templates and when we need to create abstractions? So far it seems to me that this just depends on the person making the change not what we're actually doing.

All style related functions (the styles, overrides resolver and the creation of the utility classes should use the styleProps as source of truth for the props).

Where is this documented? We should create an API that either catches mistakes or makes them impossible. Not create an API that is only understood by its creator.

My first thought was that we can pass only stylesProps on the overridesResolver and then we should be good to go, but it may be confusing when people use this for their own custom components and they don't have the convention of adding all styling props under one prop as we do. Let me know if you think otherwise on this one.
@eps1lon we still haven't created a universal RFC documentation to follow, as we are still dog fooding the implementation. This week I plan to create the RFC and track all converted components to emotion. Would that RFC be a good place for documentation things like this, or did you have something else in mind?

All style related functions (the styles, overrides resolver and the creation of the utility classes should use the styleProps as source of truth for the props).

Where is this documented? We should create an API that either catches mistakes or makes them impossible. Not create an API that is only understood by its creator.

My first thought was that we can pass only stylesProps on the overridesResolver and then we should be good to go, but it may be confusing when people use this for their own custom components and they don't have the convention of adding all styling props under one prop as we do. Let me know if you think otherwise on this one.
@eps1lon we still haven't created a universal RFC documentation to follow, as we are still dog fooding the implementation. This week I plan to create the RFC and track all converted components to emotion. Would that RFC be a good place for documentation things like this, or did you have something else in mind?

Can we close this one now that we have https://github.com/mui-org/material-ui/issues/24405 and the migration guide https://gist.github.com/mnajdova/b0562785b7cf1b9d9aeae382d578594b or should we extend the guide somehow?

Can we close this one now that we have https://github.com/mui-org/material-ui/issues/24405 and the migration guide https://gist.github.com/mnajdova/b0562785b7cf1b9d9aeae382d578594b or should we extend the guide somehow?


Create EDC installation guide

Discussed in https://github.com/eclipse-tractusx/tractusx-edc/discussions/270

Originally posted by tmberthold April 21, 2023
WHAT
Create a documentation on how to deploy a Catena-X EDC via helm charts from scratch (e.g. to azure). This documentation should be used as installation guide for companies willing to deploy EDC. Outcome should be aligned with Catena-X standards and components. Documentation deployed in Tractus-X.
WHY
Various documentations at different locations exists, which makes it difficult to deploy an EDC Connector for Catena-X.
SOLUTION PROPOSAL

Step-by-step installation guide
Highlight basic requirements for installation (e.g. Java, Kubernetes, maven…)
Document prerequisites or external sources
Document required configurations, properties etc.

FURTHER NOTES
Continuous improvement of documentation planned. Starting with first minimal viable documentation which will be enhanced over time.
--
If such a guide already exist, please let us know and link it :) Otherwise we are happy to make our contribution and start with such a guide.
Also refers to https://github.com/eclipse-tractusx/tractusx-edc/issues/170
@DominikPinsel @stephanbcbauer @sebk46

@bcronin90 isnt this done (or at least planned) with the new operation view in the new structure of the connector Kit?

@stephanbcbauer Imo this documentation already exists and yes, the new operation view will link to it once it is merged.


Adding lambda layers to functions broke my amplify stack

Describe the bug
I thought I'd try and share the helper JS files across my lambda functions but am now faced with a long list of errors:
Embedded stack arn:aws:cloudformation:us-east-1:xxxxxxx:stack/amplify-xxxxxxxx-develop-200029-functiongetScreenshot-xxxxxx/aaadfdd0-83ff-11ea-a49e-0ad4194c8c9d was not successfully updated. Currently in UPDATE_ROLLBACK_IN_PROGRESS with reason: No export named apixxxxxxGraphQLAPIIdOutput:GetAtt:ImageTable:Nam

For reference, I've removed the changes that I made that caused this issue and am now stuck in this failed stack state...
UPDATE_FAILED      AmplifyResourcesPolicy AWS::IAM::Policy      Fri Jul 24 2020 17:39:18 GMT+0100 (British Summer Time) The policy failed legacy parsing (Service: AmazonIdentityManagement; Status Code: 400; Error Code: MalformedPolicyDocument; Request ID: xxxxxx-430f-a58c-e66e565ee4c1)

I get around 10 of these..
Amplify CLI Version
4.25.0
To Reproduce
Add a lambda layer to your functions. Add some js files to your opt/ folder.
Expected behavior
Adding a lambda layer shouldn't break the stack.
Desktop (please complete the following information):

OS: Ubuntu 18.04
Node: 12

Additional context
This may be something to do with enabling analytics permissions in one of my functions ... (I've seen others in the issues having similar problems). Not sure where to go now...updating the stack takes a while and without removing the lamber layers from each function one by one, I don't think I'll get to the bottom of the issue.

Looks like it could be something to do with the modifications that were made to the backend config...
-            "dependsOn": [
-                {
-                    "category": "auth",
-                    "resourceName": "xxxxxappa2ad5d26",
-                    "attributes": [
-                        "UserPoolId"
-                    ]
-                },
-                {
-                    "category": "api",
-                    "resourceName": "xxxxxxx",
-                    "attributes": [
-                        "GraphQLAPIIdOutput",
-                        "GraphQLAPIEndpointOutput"
-                    ]
-                },
-                {
-                    "category": "storage",
-                    "resourceName": "userAssets",
-                    "attributes": [
-                        "BucketName"
-                    ]
-                }
-            ]
+            "dependsOn": []

When doing a git diff all of my dependencies for functions have been removed.

Related: https://github.com/aws-amplify/amplify-cli/issues/4852

A fix was merged and deployed in Amplify CLI version 4.26.0.


Replace uses of path.Segments() with path.Scanner()

Commit 43d3c75bf9675b3ce65e0ca650ed154818df2e52 added a new path scanner which functionally replaced the older 'Segments' method. (As suggested in Issue #181)
This PR replaces the uses of Segments in the renderers of gio, htmlcanvas and tex with the new scanner.
I tested the examples of all three and they still function just like before.

Excellent, thank you!


[New Feature] Autocomplete/Combobox

Summary
A select input that allows typing, or searching of select items on input. As well as async select, to load the items only on dropdown/input change.
Basic example
Examples are like MUI Autocomplete component, Svelte Skeleton Autocomplete
Motivation
Currently if there's too many items in a Select list, it is difficult to find the item. Typing the value to narrow down the Select would help with that.

You can use TableSearch.

Will be great to have this function natively!

Since there is no component in flowbite.com, this will be a community effort and will be under EXTEND menu.

This feature request is implemented as [Dropdown with Search]
https://github.com/themesberg/flowbite-svelte/blob/main/src/routes/docs/components/dropdown.md#dropdown-with-search

Dropdown with Search is quite a different look'n'feel from the MultiSelect though, both in behavior within forms and data binding for whats selected options - they do not feel or behave the same


Grunt eslint modules

@nhubbard You're mister listing in this project. ;) Feel free to send me a PR.

Hi @MichMich and @nhubbard
When I puted  into the .eslintignore file was not ignore by grunt eslint. Was so stranger, can do the test @nhubbard ?


Support for Prometheus/Grafana

Request details
To my knowledge, both Prometheus and Grafana are very commonly used analytics/statics platforms.
As shown to me by Dzikoysk in the Discord server, Javalin has functionality to track web requests built-in. This just needs to be implemented somehow. (I'm hoping through the statistics plugin?)

If the plugin interface provides enough hooks (or access to internal statistics) this could be realized via a plugin. I recommend that actually for prometheus, as it requires another endpoint to be added.

Prometheus plugin will be available in the next release. Currently it mostly exposes Jetty related stats, but I guess we could extend in the future. Preview:

https://reposilite.com/plugin/prometheus


logger widget driven by log file watcher

The current setup of the qt logger display is touchy. In particular it will mess up the debug process because it steps through the qt_logger.py code each log statement, which is annoying. It also seems to be logging DEBUG statements as well.
An alternate setup may be to run things through a QFileWatcher and have it update with new text whenever it gets added to. This may also be a good general way to setup a program because I would like to be able to change behavior real time at run time by editing a text file...it might be a good way to rapidly iterate on some ideas without having to cobble together an entire GUI just yet.

Just sticking with the previous solution to create a custom handler for the logger. Logger did not save file with each statement (wouldn't want it to) which makes this not feasible


Add species modifier reference to species used in reaction rate laws that are neither reactants or products

SBML needs all species referred in a reaction or a kinetic law to have an associated species reference object. I had fixed this issue in bioscrape/sbmlutil.py, so would just need to port those changes over to biocrnpyler so that valid SBML models are created. Currently, the SBML models are not valid whenever hill propensities are used as shown here:

COPASI could not import biocrnpyler generated model due to this error. I will fix this  but creating this issue to track it.

PR #131 solved this


manifest: Update mcuboot and tf-m

This change brings mcuboot and tf-m to the latest revisions in their upstream respos.  This is necessary to do together as there has been an API change in the mcuboot interface.

Do you really need to pull in the latest TF-M? As asked by @nordicjm earlier I opened a TF-M cherry-pick PR with only the needed fixes: https://github.com/zephyrproject-rtos/trusted-firmware-m/pull/112

Will use #77783 instead to just update to latest mcuboot, not a whole new tf-m merge.


[Bug]: Filtering and sorting doesn't work for nested fields

Browser
Chrome
Package version
3.0.235
React version
18.0.2
Description
I have a table with PropertyFilter and my table data looks like:
const data = [
  {
    name: "Item 1",
    alt: "First",
    Customers: { Description: "Main Customer" }
  },
  {
    name: "Item 2",
    alt: "Second",
    Customers: { Description: "Second Customer" }
  },
  {
    name: "Item 3",
    alt: "Third",
    Customers: { Description: "Test Customer" }
  }
];

The problem is the Customer field as you can see its objects with the Description key.
So I can easily define table columns like this: Customers.Description but this syntax doesn't work for filtering and also my table can't do sorting by Customer column too
It seems like something happened while it preparing FilteringOptions

Source code
No response
Reproduction
https://codesandbox.io/s/bold-shamir-fx5t73?file=/src/App.js
You can reproduce it in my codesandbox:

try to do sorting by Customer field: it doesn't sort
try to use the filter for CUstomer: id doesn't find any values

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems

Hello,
We are working on supporting nested keys in Property Filter (see https://github.com/cloudscape-design/collection-hooks/issues/30) but at the moment this will not work the way you wrote it. If this is acceptable for you, I would suggest flattening your data items, something like this:
const data = [
  {
    name: "Item 1",
    alt: "First",
    Customers: "Main Customer"
  },
  {
    name: "Item 2",
    alt: "Second",
    Customers: "Second Customer"
  },
  {
    name: "Item 3",
    alt: "Third",
    Customers: "Test Customer"
  }
];

Hello,
We are working on supporting nested keys in Property Filter (see cloudscape-design/collection-hooks#30) but at the moment this will not work the way you wrote it. If this is acceptable for you, I would suggest flattening your data items, something like this:
const data = [
  {
    name: "Item 1",
    alt: "First",
    Customers: "Main Customer"
  },
  {
    name: "Item 2",
    alt: "Second",
    Customers: "Second Customer"
  },
  {
    name: "Item 3",
    alt: "Third",
    Customers: "Test Customer"
  }
];

Hello
Thanks for answer
I'll be waiting for supporting nested keys and now can move with your suggestion - flattening data

Glad to hear that you have a way forward. I will close this issue for the time being. Feel free to reach out to us again if you need any help.


Fix node-zeromq ArrayBuffer type to Buffer

I noticed (the hard way) that node-zeromq expects a Node.js Buffer, not an ArrayBuffer.
The function is defined here:
https://github.com/JustinTulloss/zeromq.node/blob/master/lib/index.js#L444

@panuhorsmalahti thanks mate!


[it] Update dev-it with main

Describe your changes
[it] Update dev-it with main
Related issue number or link (ex: resolves #issue-number)
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Hi @jihoon-seo ,
can you please check if this Pull is correct now?
Thank you,
Francesco


Level 80 Spell Ranks

Describe the bug
The sim is currently using level 80 spell ranks. This causes the sims to be inaccurate. Certain spells might be doing more damage than they normally would in a rotation causing result numbers, stat weights etc to be faulty.
To Reproduce
Steps to reproduce the behavior:
Sim with any class
Go to Results
Hover over a spell to see rank
Expected behavior
It should use the max rank of the spell available at level 70
Sim Links and Screenshots

Learn at level 77:
https://www.wowhead.com/wotlk/spell=47813/corruption
Extra Notes
I assume this would be a lot of work to fix but it is a pretty major issue

That's just the UI, haven't gotten to that yet

Fixed


docs: move to documentation website

Summary:
This PR removes most of the documentation from README in order to switch to the documentation page: https://callstack.github.io/react-native-visionos-docs/

Nice and clean!


Add yetus-excludes for sdk/

Signed-off-by: Avi Deitcher<EMAIL_ADDRESS>As suggested by @aw-zededa

As soon as you review @aw-zededa and CI passes, I will merge in, then rebase the other.


Log in - Footer Login can hover and click on empty space to open link

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Login page
Scroll down
Move the mouse to the right of the affiliate links, Notice we can hover and click to open the link from the space

Expected Result:
Can only hover and click open link from text
Actual Result:
Can hover and click to open the link from the space
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.35-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/95b16c90-0439-4520-9ac3-c26994ec795f
https://github.com/Expensify/App/assets/93399543/44e44fbd-f8d8-4332-8fc8-0a4b3242dc81
Expensify/Expensify Issue URL:
Issue reported by: @namhihi237
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688231565265009
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
In this case, the child element takes up the full width of the parent element here
What changes do you think we should make in order to solve the problem?
We need to update the child element only to equal the length of the text
<View style={[styles.footerRow,styles.alignItemsStart]}>
 {_.map(column.rows, (row) => (
 ...

What alternative solutions did you explore? (Optional)
N/A

This is dupe of https://github.com/Expensify/App/issues/15783 , i think wevcan close it

Thanks @gadhiyamanan, you're right - this is a dupe and expected behavior.


[main] Update community files

Cron -knative-prow-robot
Produced by: knative-sandbox/knobots/actions/update-community
Details:

/override kind (v1.27.x, istio)
/lgtm
/approve

/retest
/lgtm


Update free-courses-en.md

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Add Flutter courses website
Why is this valuable (or not)?
As these websites are free
How do we know it's really free?
It shows the popular websites
For book lists, is it a book? For course lists, is it a course? etc.
yes, It is a course
Flutter Tutorial: Udacity
Flutter Tutorial For Beginners : Udemy
Flutter Crash course : Flutter crash course
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

i have made the changes kindly review
On Sat, Oct 17, 2020 at 3:21 PM Seth<EMAIL_ADDRESS>wrote:

@SethiPandi commented on this pull request.
You may want to read the contribution guidelines (CONTRIBUTING.md) for
guidance on formatting!
In free-courses-en.md
https://github.com/EbookFoundation/free-programming-books/pull/4157#discussion_r506915917
:

@@ -15,6 +15,7 @@

Databases
Deep Learning
Docker
+* Flutter

The link didn't have to be capital. Just #flutter is fine.
Not a big deal, big figured I'd raise is since I had another comment
anyway.
In free-courses-en.md
https://github.com/EbookFoundation/free-programming-books/pull/4157#discussion_r506916111
:

@@ -187,7 +188,10 @@

Docker Tutorial for Beginners - FreeCodeCamp
Learn Docker, Container Runtimes, Builders and Registries - Katacoda

Please add back this line break. There should be two line breaks above a
new section.
In free-courses-en.md
https://github.com/EbookFoundation/free-programming-books/pull/4157#discussion_r506916218
:

@@ -187,7 +188,10 @@

Docker Tutorial for Beginners - FreeCodeCamp
Learn Docker, Container Runtimes, Builders and Registries - Katacoda

+### Flutter
+* Flutter Tutorial - Udacity
+* Flutter Tutorial for Beginners - Udemy
+* Flutter Crash Course - Flutter CrashCourse
Game Development
There should also be 2 line breaks above this section.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/pull/4157#pullrequestreview-510973541,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANCAXEHTCIVA5EPQ6B3BSN3SLFSLFANCNFSM4SSJKLYQ
.


Add null byte testcases for object API.

Add negative unit tests for fpgaHandleGetObject, fpgaTokenGetObject, and
fpgaObjectGetObject. These tests pass in a malformed name string that contains
a null byte.

Coverage remained the same at 80.035% when pulling 319f8f06156d177cae7a08ace1dce3815f13503e on zhang/sysobj-null-bytes-tests into c54fc476164156c5a5507645879abdc1da19252d on master.


Layout loading doesn't stop

For some reason nulls are being sent back to the server for the layout table and the server is then erroring.
We need to replicate this in the dev server so we can replicate it locally, we also need to work out why nulls are appearing in the layout table (I suspect clicking on links or in blank spaces is adding it).

I can't seem to replicate this on the dev server but I have updated the code so it removes null and undefined from the list of selected blocks


Kill count command

It would be nice to have the "!kc" command in the RuneLite as well. I don't know how hard that would be, but it is doable I think.

As discussed on discord, the reason why this probably won't be done is that it is too hard to prove when you have gotten a kill. Probably will have to wait for Jagex to release an API for this.

Added in #4067


The template website link of java11-vert-x is wrong

The template web link of java11-vert-x is wrong

Thanks

Thanks


Minor CMake and benchmark fixes

Well if this gets merged first my rebase makes the benchmark fix for my arm improvement get eliminated entirely, hah.

I thought I had seen that fix somewhere else. But when I didn't see anything change after rebasing I figured it still needed to be fixed.

Still LGTM


Plan collision engine

Allow one collider per entity.
Figure out how to send/receive events between systems/entities.
Colliders should always be voxel-based (there won't be support for sphere colliders, capsule colliders, etc).