extract transorformer layer feature

SpeechT5 , how can i extract the fixed transorformer layer feature;
in the hubert paper:

so,in the SpeechT5,how can i do,Is there any relevant code ？

for the second iteration

Sorry, we did not provide the scripts but you can modify from here and get the last layer features by passing "feature_only" parameter during forward which will return the feature in here.


Add unit tests for pkg/agent/openflow

Signed-off-by: Hongliang Liu<EMAIL_ADDRESS>

Codecov Report

Merging #87 (892e1df) into main (8427b4a) will increase coverage by 24.17%.
The diff coverage is 79.72%.

Additional details and impacted files

@@             Coverage Diff             @@
##             main      #87       +/-   ##
===========================================
+ Coverage   34.61%   58.78%   +24.17%
===========================================
  Files         144      376      +232
  Lines       21800    55319    +33519
===========================================
+ Hits         7545    32522    +24977
- Misses      13602    20546     +6944
- Partials      653     2251     +1598

Flag
Coverage Δ

*Carryforward flag

integration-tests
34.61% <50.00%> (ø)

Carriedforward from 8427b4a

unit-tests
54.89% <75.71%> (?)

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/multicast/mcast_controller.go
46.19% <ø> (+46.19%)
:arrow_up:

pkg/agent/openflow/cookie/allocator.go
73.33% <ø> (+35.55%)
:arrow_up:

pkg/agent/openflow/pipeline.go
81.45% <33.33%> (+23.59%)
:arrow_up:

pkg/agent/openflow/client.go
78.18% <51.85%> (+37.47%)
:arrow_up:

pkg/agent/openflow/multicast.go
79.21% <100.00%> (+79.21%)
:arrow_up:

pkg/agent/openflow/pipeline_other.go
96.49% <100.00%> (+0.33%)
:arrow_up:

pkg/agent/openflow/pod_connectivity.go
96.29% <100.00%> (+1.91%)
:arrow_up:

pkg/ovs/openflow/ofctrl_bridge.go
73.80% <100.00%> (-2.66%)
:arrow_down:

pkg/controller/networkpolicy/mutate.go
0.00% <0.00%> (ø)

... and 339 more


Traceback when using certain namespace prefixes

Hi!
After update of xmldiff to 2.6, we have started to experience tracebacks  in our tests (tracked in https://github.com/ComplianceAsCode/content/issues/10417) that are coming from xmldiff.
The problem is that when the XML content that we pass to xmldiff contains namespace prefixes matching ns\d+, eg. ns0, ns1, xmldiff tracebacks.
The traceback stops happening if we replace these prefixes by some other but not generic prefixes.
My understanding is that the ns\d+ prefixes are used by default when generating XML content using xml.etree.ElementTree when register_namespaces isn't called. Therefore, I think that such inputs should be supported by xmldiff. Is that  expectation correct? Or is the intention that xmldiff inputs will never contain these specific prefixes?
Version:
xmldiff 2.6, installed using pip3
Operating system:
Fedora 37
Steps to reproduce:
Create a Python script in which you will diff 2 XML files that use namespace prefixes matching ns\d+, eg. ns0. I have this small reproducer script xmldiff_bug.py:
import xmldiff.main

if __name__ == "__main__":
    left = '<ns0:title xmlns:ns0="http://checklists.nist.gov/xccdf/1.2">foo</ns0:title>'
    right = '<ns0:title xmlns:ns0="http://checklists.nist.gov/xccdf/1.2">bar</ns0:title>'
    diff = xmldiff.main.diff_texts(left, right)
    print(diff)

Actual results:
[jcerny@thinkpad ~]$ python3 xmldiff_bug.py
Traceback (most recent call last):
  File "/home/jcerny/xmldiff_bug.py", line 6, in <module>
    diff = xmldiff.main.diff_texts(left, right)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jcerny/.local/lib/python3.11/site-packages/xmldiff/main.py", line 44, in diff_texts
    return _diff(
           ^^^^^^
  File "/home/jcerny/.local/lib/python3.11/site-packages/xmldiff/main.py", line 37, in _diff
    return diff_trees(
           ^^^^^^^^^^^
  File "/home/jcerny/.local/lib/python3.11/site-packages/xmldiff/main.py", line 27, in diff_trees
    return list(diffs)
           ^^^^^^^^^^^
  File "/home/jcerny/.local/lib/python3.11/site-packages/xmldiff/diff.py", line 435, in diff
    etree.register_namespace(k, v)
  File "src/lxml/etree.pyx", line 197, in lxml.etree.register_namespace
ValueError: Prefix format reserved for internal use

Expected results:
I would expect the output that I got with xmldiff 2.5:
[jcerny@thinkpad ~]$ python3 xmldiff_bug.py
[UpdateTextIn(node='/ns0:title[1]', text='bar')]

lxml does indeed generate prefixes like that, and does reserve it for internal use, which seems to mean that it's fine with parsing XML with such namespaces, but it's not fine with registering it.
That on the other hand might just mean that we don't have to register them, so if the namespace matches that format, we might skip the registering, which would be an easy fix. That needs investigating.

2.6.1 released

fixed by https://github.com/Shoobx/xmldiff/pull/109


clg doesn't have autocomplete

Overall it is a great extension, I love it!
One thing that bothers me a bit is that "clg" won't let you use autocomplete when replacing "(object)"  with an existing variable name. It usually does when you manually type "console.log()" so I wonder if that could easily be fixed?
Thank you very much.
Best,
Komo

@romokebi there is a coo snippet for those :)


Datacite submission & Orcid authority

The first commit makes the new datacite submission the default, it updates the dissemination crosswalk in order to properly exposed all the metadata. It also adds new metadata fields, so run:
/dspace/bin/dspace registry-loader -metadata \
/dspace/config/registries/datacite.xml

The second commit adds a new implementation of orcid authority, one that does not use solr and keeps the authority keys in the authority column). It changes the configs introduced in #570 and exposes the orcid in datacite dissemination crosswalk.

@vidiecan
I think it could be merged into dtq-dev without submission-forms.xml and item-submission.xml


upstream-community-operators knative-operator (0.22.1)

Signed-off-by: Vincent Hou<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@mvalarh I have rebased and update this PR.

/hold cancel

@aliok It is quite strange that it could not choose you as reviewer. I am seeing error
Warning: Reviews may only be requested from collaborators. One or more of the users or teams you specified is not a collaborator of the operator-framework/community-operators repository.

From contributor list i see that you are contributor to this project. But probably it is not enough. I see in git blame that you are not listed as contributor there,
Would you be so kind and make PR and modify upstream-community-operators/knative-operator/ci.yaml file (just add empty line). I would like to test if this is causing problem. This would help us a lot. Thanks

@aliok please ignore my previous comment.


Short text for link-out collections

On OV3 we have "collections" which aren't actually sets of documents. I would like to have a collection page for each which then link to the outside resource.

Less special-casing in the code.
Less confusing context-switching for user.

@caseyedavis12 : can you provide a paragraph or two for

AAPB
Stock Sales
10 O'Clock News

and then assign back to me? (You might have more than one link for these "collections", perhaps to particularly interesting content they contain.)
... or if you think they really-really need to link directly, not impossible.

Stock Sales:
WGBH Stock Sales is the clip licensing division of the WGBH Media Library and Archives. Stock Sales licenses originally shot WGBH footage and web content to television producers, publishers and professional media organizations. The Stock Sales website features over 10,000 clips and 12 WGBH collections that users can search, browse, and license.
Search and browse thousands of WGBH clips available to license!
Click on the thumbnails below to screen examples:
BTVNDL:
The Boston TV News Digital Library, funded by the Institute of Museum and Library Services and the Council on Library and Information Resources, is a collaboration between the Boston Public Library, Cambridge Community Television, Northeast Historic Film and WGBH Educational Foundation. The website aims to bring to life local news stories produced in and about Boston from the early 1960’s to 2000.
Nearly 2,000 audiovisual news reports are available for viewing online, including content from Boston Public Library’s WHDH film collection (1960- mid-1970s), Cambridge Community Television (1988 to 1999), Northeast Historic Film’s WCVB film collection (1970-1979), and WGBH-produced The Reporters (1970-1973), Evening Compass (1973-1975) and the Ten O’Clock News.
Click on the thumbnails below to screen examples from our collection:
AAPB:
The American Archive of Public Broadcasting (AAPB) contains approximately 40,000 hours of digitized television and radio programs selected by more than 100 public broadcasting stations throughout the nation. Nearly 12,000 programs are available for viewing and listening in the AAPB Online Reading Room. Dating from the 1940s to the 21st century, the AAPB collection includes national, local and regional programming. A collaboration between the Library of Congress and WGBH, the AAPB seeks to preserve and make accessible significant historical content created by public television and radio, and to preserve at-risk public media before its content is lost to posterity.
Browse and discover the WGBH collection in the AAPB!
Click on the thumbnails below to screen examples:

@caseyedavis12 : just make the edits you need in the MD files: I don't need to be involved.

@caseyedavis12 : text looks good. I'll leave this open for getting the sample images on the page: I'll put some stubs there, and you can change it to fit your needs.

We're hoping that there don't need to be collection-based searches with the current approach, obviating all of this. Closing.


Upgrade Crossplane to v1.18

Description of your changes

I have:

[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Run make ceph-chainsaw to validate these changes against Ceph. This step is not always necessary. However, for changes related to S3 calls it is sensible to validate against an actual Ceph cluster. Localstack is used in our CI Chainsaw suite for convenience and there can be disparity in S3 behaviours betwee it and Ceph. See docs/TESTING.md for information on how to run tests against a Ceph cluster.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Handle StackOverflowError in macros

Description
In a similar fix to #1489, catch StackOverflowErrors in ScalacWorker so that buggy macros don't crash the persistent worker resulting in unhelpful logs.

Motivation

Fix a similar class of issue as #1488

Requesting a merge on this PR from any eligible committers


Remove Spectrum's gemstone tags from your gems

You tag your citrine and moonstone as Spectrum's gemstones of the same type, despite this being incredibly progression breaking in the case of the latter, and generally strange. Can this tag addition be removed?

As far as I remember, I made the moonstone tag long before spectrum moonstone was even released "for production", so that tag can definitely be removed. Both citrine are basic ingredients that are widely available so I don't find the tag for citrine obtrusive or strange.

,,,except that both citrines are acquired in different ways, and having both interchangeable is against Spectrum's lore and general expectations.

Then daFuqs shouldn't have a tag for it, if it matters that much

There's a tag for it because it is used for more than just the shards. It includes everything pertaining to the raw gemstone - buds, "raw" geode blocks, clusters, etc.

I see. that can still be accomplished without a tag, which would remove the concern of "leaking" progression.
My compromise is to make the tag, and a bunch of other random stuff "optional", with a simple kill switch of "integration/[thing]" in the AI config list for this purpose

Hi, spectrum dev here. While integration is nice, I want to point out that those are not integration tags. What you are adding items into are tags used for the mod's progression which should not be touched by anyone but pack makers.
While citrine is not as egregious as moonstone, in principle compat should not be touching those tags. AI and Spectrum are different mods with different progressions and different narratives, and gaining citrine in one does not set the proper tone nor lay down the proper knowledge for each other's progression here.

Then really you should refactor that to not be a tag, or specifically put into a progression/ subfolder. I already have a solution for resolving it as explained above.

new versions uploaded.


Fix regex env removal vs. non-utf8 env entries.

Previously the scie-jump boot would crash when a manifest requested
regex removals and the ambient environment contained non-utf8 entries.
Fixes #105

I'm going to push this along shortly to get out a release and update scie-pants to it and release it in-turn.


Optional support for OptionHelper class.

Issue Checklist

[X] I have checked for similar issues on the Issue tracker
[X] I have updated to the latest version of JDA-Chewtils.
[X] I have checked the branches or the maintainer's PRs for upcoming features and updates.

Affected Modules
Command
Description
Add java.util.Optional return type for each opt*** methods. For example;
    /**
     * Guarantees a String option value by providing a default value.
     *
     * @param event        The slash command event to get options from
     * @param option       The option we want
     * @param defaultValue The fallback option in case of the absence of the option value
     * @return The provided option, or the default value if the option is not present
     */
    @Nullable
    @Contract("_, _, !null -> !null")
    public static String optString(@NotNull SlashCommandEvent event, @NotNull String option, @Nullable String defaultValue) {
        List<OptionMapping> options = event.getOptionsByName(option);

        return options.isEmpty() ? defaultValue : options.get(0).getAsString();
    }

    /**
     * Guarantees a String option value by providing a default value.
     *
     * @param event        The slash command event to get options from
     * @param option       The option we want
     * @param defaultValue The fallback option in case of the absence of the option value
     * @return The provided option, or the default value if the option is not present
     */
    @NotNull
    public static Optional<String> optString(@NotNull SlashCommandEvent event, @NotNull String option, @Nullable String defaultValue) {
        return Optional.ofNullable(optString(event, option, defaultValue));
    }

Kinda pointless to have imo

it's a utility class so idk. but the purpose of this type of classes is this actually :d

A simple return OptionMapping == null ? defValue : OptionMapping#asWhatever() is sufficient enough.
Creating an Optional just for this is a waste of time and resources.

if you think on this class, Optional.ofNullable(event.getOption("optionName")) is also enough for each opt**** methods. my point is this class called helper class which can have helper methods for every option mapping related methods.

To my knowledge, you can't have the same method name + same parameters with a different return type.
As has been stated, I don't really see the point of this. There's a whole debate of whether Optionals are good or not, I'm on the side of don't really care but optX already does everything I assume you'd want (you're guaranteed an option) which was the ultimate point of the thing.


Themeing support

Added ability to add themes by pulling in git submodules to a themes/ folder as per discussion on #73. The themes can be configured using config/site.toml. Added documentation as well but would definitely need to be proofread.
Any thoughts/suggestions would be appreciated.

Besides a sign-off, could you please also sign your commits, please?
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits
Thanks!

Quick question: why are the static files from the docs directory moved to docs/themes/?
I would expect that, if we refactor the current templates into an actual theme, that would be a separate PR.

In general, we want to change as few files as necessary as part of a PR.
Thanks!

This is super cool!

I was also wondering which section the documentation will follow in the existing documentation so that I can add the proper links to the site to maintain the chaining of secitons.

@vdice I will resolve the conflict and push a new commit. I had one question as to where the docs for themes must be listed.
The current order of sections is

Introduction
Quickstart
Templates
Configuration
Scripting
Markdown guide

I am not sure where the "themes" section fits in that order.

@karthik2804 good question.  No strong opinion here; I think the current ordering in this PR of 4. Configuration 5. Themes 6. Scripting (and so on) makes sense.  We'll want to update the end sections accordingly, eg update https://github.com/fermyon/bartholomew/blob/main/docs/content/configuration.md?plain=1#L40 to navigate to the new Themes section and then at the end of the Themes doc we can link to the Scripting page.


Prepare EurOPDX data and instructions

Description
As a developer, I want to provide a comprehensive package for EurOPDX data release on EurOPDX Portal.
Acceptance criteria

[x] extract EurOPDX data from the repo
[x] extract EurOPDX mappings from the repo in JSON
[ ] write instructions on how to deploy the application
[ ] write instructions on how to do the upgrade of software stack required for the new version
[ ] do a test load of EuroPDX data to make sure it's working and deployable
[ ] data and instructions sent to EurOPDX

EuroPDX data and corresponding mappings were exported from repo.

moved to the next sprint

My part is done, document:
https://docs.google.com/document/d/1_B9Y4a-QT03sKhq6f1m-Az-kRPt7ZWivANzyVXO0laI/edit#
It's all yours now, @zperova


Read dependencies from requirement files for setup

Summary:

setup.py now has install_requires which reads from requirements.txt
extra requires for dev specific dependencies coming from dev-requirements.txt (moved pytest to it)
pytorch and torch* dependencies still need to be installed manually since we depend on nightly builds and there seems to be no principled way of getting them in setup.py

Test plan:

In a fresh conda env, followed Readme and tried to import flava. Checked no dev dependencies were installed in the env
In another env, followed updated contribution guide and ran pytest. Verified mypy, pytest and precommit were installed

We should consider adding version numbers in requirements file at least for packages where reproducible behaviors are important.

this is pretty much reproducing the manual readme right now so idk which ones we want to pin

this is pretty much reproducing the manual readme right now so idk which ones we want to pin

Versions are added in requirements files. All packages should have versions to guarantee reproducibility across developers and (especially) in CI.
But I've also seen repos not being strict with versions (i.e., only package names in requirements). That should be fine until we become more sensitive to versions.
For now I'm just referring to the linter/formatter (see my comments earlier) as those are known issues.


Suggestion: Tweek Inspector

Allow querying for key result for a specific request, provide context values hit Go and get the computed rule result.
I've had multiple cases where this would have been really helpful, for example: we opened a feature and got some reports about lagging in the system, and assumed its the experts we opened the feature to but couldn't really verify that.
This will mainly help debugging stuff.

@talarari is that the same as:
https://github.com/Soluto/tweek/issues/239


Use Link from react router in a cell

Describe the bug
I am using react router and want to use a column formatter so that the user can click on the link within the cell and be guided to a detail page.
To Reproduce
Here is the column formatter i wanted to use:
import { Link } from "react-router-dom";

const linkFormatter = (cell, row) =>
    _(
      <Link to={`/jobs/${row.cells[1].data}`} className="font-bold">
        {cell}
      </Link>
    );

I get the following error:
Error: Invariant failed: You should not use <Link> outside a <Router>
Expected behavior
Is there a way to make  work inside gridjs? Or maybe a workaround? I've tried using a regular <a href="..." /> tag, which works, but I don't want to re-render the entire page. Many thanks!

Is there any way of reopening this, as I am having exactly the same issue.

Hi @arnigunnar, just for you to know, I've switched to using React Table (https://react-table.tanstack.com/). It's a bit more work to set it up but it works great and allowes for more customization.

Send the history as a prop to CellLink
import { useHistory } from "react-router-dom";

const CellLink = ({ item, history }) => {
  function handleClick() {
    history.push(`/jobs/${item.jobId}`);
  }
  return (
    <button onClick={handleClick} className="font-bold">
      View
    </button>
  );
};

...

const WrapperComponent = () => {
  const history = useHistory();
  return (
    <Grid
      data={data.map((item) => ({
        ...item,
        button: _(<CellLink item={item} history={history} />),
      }))}
      columns={columns}
    />
  )
}

https://gridjs.io/docs/examples/cell-attributes
const grid = new Grid({
columns: [
{
name: 'Name',
attributes: (cell) => {
// add these attributes to the td elements only
if (cell) {
return {
'data-cell-content': cell,
'onclick': () => alert(cell),
'style': 'cursor: pointer',
};
}
}
},
'Email',
],
data: Array(5).fill().map(x => [
faker.name.findName(),
faker.internet.email(),
])
});
[ react ]
import { useHistory } from 'react-router-dom';
const history = useHistory();
...
onclick : () => history.push('/link')


[BUG] d2fps v1.0.2 & Project Diablo 2 - Multiple casts occur in quick succession when casting a "quick-cast" skill

Hi! Love the work. d2fps is nothing short of a miracle!
I play Project Diablo 2 single player and have incorporated d2fps into PD2's default d2gl installation.
Now, I have "quick cast" enabled in PD2, which mimic D2R's capabilities.

I have noticed that when I cast a quick cast skill using my keyboard, the skill will cast multiple times in quick succession.
When not in motion, it sometimes works correctly (i.e. cast a skill while stationary).
Not only do skills cast multiple times, skill internal cooldowns are ignored (i.e. PD2's Valkyrie 1 second cooldown).
The only way to "cancel" multiple casts is to click my left/right mouse button skills.
Skills assigned to left/right mouse buttons do not suffer the same issues. This is solely a quick cast issue.

I have verified that it still happens with motion smoothing disabled in d2fps.ini.

https://github.com/Jarcho/d2dx/issues/23
look this issues.

Already a known compat issue with the frame limiter. You can manually set the frame rate higher and rely on vsync to limit it.


Cumulative Layout Shift (CLS) problem again

I don't know why but console is shown this (see pic) How is it possible that console is marking all that entire block as CLS contribution? It should be shown as unique small parts
That is the url to try:
https: //www . b e b e m o v i l . com/inglesina-quid-2-silla-de-paseo-id575  (without spaces)
Image link: https://i.stack.imgur.com/she3k.png
and...
this also happens... there are no cls changes, however CLS value is so bad... I don't understand it (it is with this url https: //www.b e b e m o v i l.com/inglesina-aptica-quattro-carrito-bebe-trio-3-piezas-id567 )
Image link: https://i.stack.imgur.com/B0txx.png
Please i need help @philipwalton , @rviscomi
Thanks so much in advance

A test using the Chrome Performance report indicates there are a bunch of shifts within that element. I suspect CSS and font loading and causing multiple small shifts, all adding up to a CLS.
I moved all your CSS resources to the head section so that they become a high priority. And I preloaded a font:

No more layout shifts.

 I suspect CSS and font loading and causing multiple small shifts, all adding up to a CLS.
I moved all your CSS resources to the head section so that they become a high priority. And I preloaded a font:
No more layout shifts.

Great answer dear @Tiggerito
I will try it asap and will tell u about it.
How can a font to be preloaded?
Should i  preload icon font too?

The editor removed my example of the preload tag.
<link rel="preload" as="font" href="https://fonts.gstatic.com/s/montserrat/v15/JTURjIg1_i6t8kCHKm45_cJD3gnD_g.woff2" type="font/woff2" crossorigin>
One issue with this is that the font URL may change in the future, making the preload a waste of time. Or the fonts used change. Some host the font locally to take control.

 I suspect CSS and font loading and causing multiple small shifts, all adding up to a CLS.
I moved all your CSS resources to the head section so that they become a high priority. And I preloaded a font:
No more layout shifts.

GREAT MY FRIEND.
IT HAS WORKED.
CLS is now in green range.
Thank you very much @Tiggerito


Including bin files in

I've tried so many times, but I still can't compile the right file(maybe something wrong with the network). Could someone
share the bin file? thanks. (esp8266 st7789)

``.text' will not fit in region iram1_0_seg'
This happends all the time when I make. How can I solve it.
Thanks.


List only Cloud Stacks under Cloud section

List only Cloud Stacks under Cloud section
Fixes BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1337072

/cc @martinpovolny @bzwei @gmcculloug

@martinpovolny @bzwei could you click it through with an ansible tower provider? Or send me a creds to some, please?
@martinpovolny that was more changes needed than I expected, not sure I haven't missed something

@Ladas I sent you the creds of a testing ansible tower provider to your mailbox.

@Ladas is https://github.com/ManageIQ/manageiq/blob/master/product/views/OrchestrationStack.yaml still in use?

@dclarizio @h-kataria talking to @bzwei I did the blocker bug fix with https://github.com/ManageIQ/manageiq/pull/8975 , then instead of this, we will need to create orchestration_stack_cloud, orchestration_stack_infra and orchestration_stack_configuration controllers. As a proper fix

@miq-bot add_label wip

Removing the blocker and Darga flags as the blocker was fixed in #8975. This is going to be cleanup/systematic fix.

this was already solved by @lgalis, ansible jobs have their own list now


Memory issues with ImageProcessing.Web 4.4

James,
I posted a question about this on the umbraco forums and I am adding clarification specific to ImageProcessing.Web. To isolate the problem I created an empty website and only configured ImageProcessing and added a few thousand images. (IIS 7.5, .net 4.5)
I used a tool to request the images and the memory grows steadily. After all the requests I created a memory dump and there are 8347 System.Web.HttpResponseUnmanagedBufferElement objects on the heap. It looks like it is not the MemCache but the output cache that is the problem.
I'm not sure about this but I think AddFileDependencies in ImageProcessingModule.SetHeaders may be what is triggering asp.net to add the image to the output cache. I wasn't sure what AddFileDependencies did, but I found this https://msdn.microsoft.com/en-us/library/67z4z916(v=vs.110).aspx.
This makes me think the output cache is holding onto the image in unmanaged memory due to a combination of the length the expiration is being set to and the file dependency that is being setup.
Attached is an example of the windbg session for the memory dump.
mem-debug-session.txt
Thanks,
David

Hey, thanks for that. Very, very much appreciated!
I'll respond both here and the forums for clarity though i'll be linking to here from there.
If it's the output cache that is the issue we may have a workaround.
I think it's correct that I am utilizing the output cache plus file dependencies. As far as I understand the mechanism It's allowing much reduced io requests and increased scalability, plus it ensures that if an image changes, the request changes. I think it should be possible for me to cache in the browser though only which might well be better for larger sites.
I'm not the only person to utilize output caching in this manner though. Imageresizer does so also. https://github.com/imazen/resizer/blob/d987fc27fe2c020cde309620d36158ea3988a963/Core/Caching/ResponseHeaders.cs#L62
If output caching it required for decent performance there are some configuration changes that can be made that should mitigate growth.
First off it's possible to limit the size of the output cache.
http://www.iis.net/configreference/system.webserver/caching
That is only settable at application level though. According to documentation I have read it defaults to half of available physical memory or available virtual memory whichever is less.
We can also reduce the number of file watchers. There are a lot of DirMonCompletion objects created since I create a lot of folders. Changing the FCN configuration should help.
http://shazwazza.com/post/all-about-aspnet-file-change-notification-fcn/
The largest number of items with the greatest size in your output though is ApplicationImpersonationContext instances. I wonder? Are you using an UNC path in your site? That causes instances to be generated.
http://referencesource.microsoft.com/#System.Web/HttpContext.cs,1113
Cheers
James

James,
Thanks for the information.
About the ApplicationImpersonationContext instances. The dump is from a site I created quickly using the local filesystem. My load test script was created for a different site that includes credentials and I forgot to remove them when I reused the script. I think that may be where that is coming from.
Thanks,
David

Ah right... No worries,
I'm gonna change the config transforms for the nuget package on the next release to set FCN to "Single". On my test website in the solution that reduced the number of watchers looking out for web.config files from 800+
Would it be possible for you to do the same in your test website and report back?

I just made the FCN change and that dropped the number of DirMonCompletion objects to 5.

Ace. That sounds right to me. That should dramatically reduce your memory and process usage.


@sweir27 => Auction blocks responsive styles

Adds responsive/mobile layout to auction-blocks component
Fixes some key/propTypes errors

👍 many thanks!!


Fix 'steps to reproduce' code block

Fix for #728.
With the introduction of CSS line numbers, the JS trying to strip out real line numbers lead to the removal of numbers in the logs themselves.

My mistake, I have fixed the numbers issue but missed the visual feedback on copying that was also raised in the issue. I will add this to the PR.


undefined forEach method

there seems to be a bad reference to the forEach method in velocity-component.js, which causes issues when clearing the velocity cache, which seems to occur when certain react components re-render.
I've made a simple change in this pull-request, and would appreciate some feedback on it!
https://github.com/twitter-fabric/velocity-react/pull/151

Thanks for the fix!


Better modifiers

I find the modifier selection page a bit cumbersome.
Maybe it could be improved to something more compact, where all duplicated modifiers are stacked, and the number available / added would be displayed on mouseover:

I added some ugly numbers to illustrate. Wdyt  @RaphaelPage0110  ?

The reason why we did not do this initially was because there is no way to know from the name of the image file if it's a duplicate modifier.
So your idea is feasible for sure...but you would have to manually edit the database and enter which card is a duplicate of which. Which would take a looooooooong time


Templated deployment

Feature Request
Description of Problem:
It's desirable to be able to automatically generate deployment manifests for various deployment targets (such as Kubernetes) from CALM-defined architectures
Potential Solutions:

https://github.com/finos/architecture-as-code/tree/templating-spike initial spike based on Handlebars templates, with hard-coded JSONpath to extract values from CALM document
Inspecting nodes based on their node-type, interfaces and associated relationships to populate a handlebars template intelligently

Closed as duplicate of #31


Do i18n for every string in Tania.

I want to do internationalisation. So, we can accommodate users who can't understand English.

Just for reminder, we're using xliff format: https://en.wikipedia.org/wiki/XLIFF

This commit https://github.com/Tanibox/tania/commit/bbece71c9b203770f4568cec72f83705cf1db871 is completing this enhancement. We can close this issue.


Scala Stream Collector: support any {api-vendor}/{api-version} for GET and POST

So that different vendors (Snowplow, Sendgrid etc) can send events in their own payload format to the same collector, just by adjusting the URI to the collector.
In other words, currently we have:
collector-endpoint.com/i

We should move to:
collector-endpoint/com.snowplowanalytics/i

/cc @yalisassoon

Hey @fblundun - I think this has been done, please close if so!

I think this is currently in the branch for adding post support. The code is here: https://github.com/snowplow/snowplow/blob/feature/ssc-post-support/2-collectors/scala-stream-collector/src/main/scala/com.snowplowanalytics.snowplow.collectors.scalastream/CollectorService.scala
Is that what you had in mind @alexanderdean ?

Aha - we need to update this to be more permissive - see the Clojure Collector:
https://github.com/snowplow/snowplow/blob/master/2-collectors/clojure-collector/java-servlet/src/snowplow/clojure_collector/core.clj#L44
Note how a tracker or webhook can post or get to any {api-vendor}/{api-version} ?

Understood! I'll replicate this behaviour with Spray.

Thanks Fred!


bpf xdf not works through docker bridge

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Does bpf_redirect work in docker?
I have 3 dockers, client, elb and e2e-cluster-control-plane.

In docker elb , it has 2 nics, eth0 and eth1.
In docker e2e-cluster-control-plane, it has 1 nic, eth0
In docker client, it has 1 nic, eth0.

I have two docker networks, innet and kind,  as followed:

eth0 of client and eth1 of elb is on network innet.
eth0 of e2e-cluster-control-plane and eth0 of elb is on network kind.

I write a xdp program which is not works, any hints?

Attach xdp program to eth0 and eth1 of elb.
eth0 of client send package to eth1 of elb. OK.
eth1 of elb receive the package from eth0 of client.  OK.
elb modify the package, change the source ip, source port and source mac to eth0 of elb info. OK.
elb modify the package, change the dest ip, dest port and dest mac to eth0 of e2e-cluster-control-plane info. OK.
elb send the package by bpf_redirect(eth0 of elb), it return 4 which is XDP_REDIRECT, Not sure if it is OK. Maybe this is not send  successfully even bpf_redirect return good?
e2e-cluster-control-plane didn’t receive any package. Not OK. No seen any package loss by tcpdump.
I try to curl from eth0 of elb to eth0 of e2e-cluster-control-plane, and it is ok.

I don’t know why bpf_redirect didn’t send package successfully from  eth0 of elb to eth0 of e2e-cluster-control-plane?
Any one has any hints?
Cilium Version
none
Kernel Version
5.16.0-051600-generic
Kubernetes Version
kindest/node:v1.21.1
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please see https://github.com/cilium/cilium/blob/master/test/l4lb/test.sh#L14. Closing this issue, as it's not related to Cilium.


Added tests for BytesLib.sol

Use truffle test <file> to run the test.
e.g.
truffle test test/v5/solidity/TestBytesLib1.sol
truffle test test/v5/solidity/TestBytesLib2.sol
Prerequisites for this test to run is Migrations.sol and AssertBytes.sol.

Closing due to in favor of using abi.[encode/decode] method.


IPv6 with swarm manage doesn't work

docker-version:
Client:
 Version:      1.11.0-dev
 API version:  1.23
 Go version:   go1.5.3
 Git commit:   79edcc5
 Built:        Sat Feb 13 00:12:47 2016
 OS/Arch:      linux/amd64
 Experimental: true

Server:
 Version:      1.11.0-dev
 API version:  1.23
 Go version:   go1.5.3
 Git commit:   79edcc5
 Built:        Sat Feb 13 00:12:47 2016
 OS/Arch:      linux/amd64
 Experimental: true

docker info:
Containers: 5
 Running: 0
 Paused: 0
 Stopped: 5
Images: 1
Server Version: 1.11.0-dev
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 18
 Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 3.13.0-83-generic
Operating System: Ubuntu 14.04.4 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 994 MiB
Name: vagrant-ubuntu-trusty-64
ID: Q5RK:ESPZ:NMKC:KGRL:WLHV:PLQ2:7CIU:YSUX:42AV:WF6N:P6SP:NFPJ
Debug mode (client): false
Debug mode (server): false
WARNING: No swap limit support
Experimental: true

Additional environment details (AWS, VirtualBox, physical, etc.):
-Vagrant with Virtualbox.
-etcd node = 2001:db8:cafe:14::a
-manager node = 2001:db8:cafe:14::b
-swarm node = 2001:db8:cafe:14::c
-All three are reachable over IPv6.
Steps to reproduce the issue:

Run etcd on etcd host and make sure IPv6 works to that host:

MY_IPv6="[2001:db8:cafe:14::a]"

./etcd \
-name etcd-01 \
-initial-advertise-peer-urls http://$MY_IPv6:2380 \
-listen-peer-urls="http://<IP_ADDRESS>:2380,http://<IP_ADDRESS>:7001" \
-listen-client-urls="http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001" \
-advertise-client-urls="http://$MY_IPv6:2379" \
-initial-cluster-token etcd-01 \
-initial-cluster="default=http://$MY_IPv6:2380" \
-initial-cluster-state new -debug

# PUT/GET to etcd:
curl -L -g http://[2001:db8:cafe:14::a]:4001/v2/keys/message -XPUT -d value="IPv6 test"
{"action":"set","node":{"key":"/message","value":"**IPv6 test**","modifiedIndex":20,"createdIndex":20},"prevNode":{"key":"/message","value":"this works","modifiedIndex":17,"createdIndex":17}}

curl -L -g http://[2001:db8:cafe:14::a]:4001/v2/keys/message
{"action":"get","node":{"key":"/message","value":"**IPv6 test**","modifiedIndex":20,"createdIndex":20}}```

2. Run 'swarm join' from swarm node:
```docker run swarm:1.2.0-rc1 join --advertise=[2001:db8:cafe:14::c]:2375 etcd://[2001:db8:cafe:14::a]:2379
time="2016-03-30T22:26:04Z" level=info msg="Initializing discovery without TLS"
time="2016-03-30T22:26:04Z" level=info msg="Registering on the discovery service every 1m0s..." addr="[2001:db8:cafe:14::c]:2375" discovery="etcd://[2001:db8:cafe:14::a]:2379"
time="2016-03-30T22:26:07Z" level=error msg="client: etcd cluster is unavailable or misconfigured"```

3. Run 'swarm manage' from manager node:
```docker run -p 4000:2375 swarm:1.2.0-rc1 manage etcd://[2001:db8:cafe:14::a]:2379
time="2016-03-30T22:25:53Z" level=info msg="Initializing discovery without TLS"
time="2016-03-30T22:25:53Z" level=info msg="Listening for HTTP" addr=":2375" proto=tcp
time="2016-03-30T22:25:56Z" level=error msg="Discovery error: client: etcd cluster is unavailable or misconfigured"
time="2016-03-30T22:25:59Z" level=error msg="Discovery error: client: etcd cluster is unavailable or misconfigured"
time="2016-03-30T22:26:02Z" level=error msg="Discovery error: Unexpected watch error""```

**Describe the results you received:**
If I replicate this exactly only with IPv4 addresses it works perfectly.  I know etcd is working on IPv6. When I do a 'tcpdump' to watch to see if the node or manager try to use IPv6, I see no traffic at all.

**Describe the results you expected:**
I expect to see swarm nodes listed on the etcd console as well as a "Registered Engine" show up on the manager.

**Additional information you deem important (e.g. issue happens only occasionally):**

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@shmcfarl can you reopen this issue in the Swarm issue tracker, as requested here https://github.com/docker/docker/pull/20842#issuecomment-203160843 ?
I'll close this issue here, because this is something to be looked into in the swarm issue tracker, but feel free to comment here after I've closed

Absolutely. My bad. :-(

no problem, thanks!


1.9: Addressed safety issues up to 2023-11-05

Rolls back PR #511 into 1.9.2.

coverage: 33.997%. first build
when pulling 4cff0fd9e41ebef0c4cbdd33dc20220146a9cda4 on 1.9/andy/safety-2023-11-05
into 7b8d5ed6958b457c7ca18feb188920290d750203 on stable_1.9.


[UNDERTOW-2092] Update GitHub CI configuration to JDK11+

Created just as a test to check that the CI will actually work...

Closing in favor of #2


Allow Alert component to receive color

Motivation
Currently the color is decided only by the input status.
This issue is to allow the component to receive other input color, which will override status color.
Usage example
No response
Possible implementations
No response

I would like to work on this issue

of course @Nazeeh21


Added Xemnas

Xemnas transcription by VaultLink. + fixed a line by TX

@Vladabdf, what you think about "Audio fails to extract. File uses dummy codec."? IMO that's nothing Xemnas would say and it would be clearer why the line is empty. :-)

In virtually every other file where a value is unknown for some reason or another,  if it's plaintext,  we've used "?" thus far, so I kindly ask we keep it that way for consistency.

Sorry, my bad! Think I understood it wrongly at first. In case you can change it yourself, feel free to do so. Otherwise I'll change it later on. 👍


On stopping the screen share, Frames captured using ExternalVideoTrackSource is frozen at remote end.

Describe the bug
I'm using the following code in webrtc 2.0.2 to share my screen (UWP app) to a remote user (using Web)
using ExternalVideoTrackSource

_videoSrc = ExternalVideoTrackSource.CreateFromArgb32Callback(Screenshare.Instance.WebRTCFrameCallback);
_localVideoTrack = LocalVideoTrack.CreateFromSource(_videoSrc , new LocalVideoTrackInitConfig() { trackName = "screenshare" });
_videoTransceiver = _Connection.AddTransceiver(MediaKind.Video, new TransceiverInitSettings() { InitialDesiredDirection = Transceiver.Direction.SendReceive, StreamIDs = new List() { "local_av_stream" } });
_videoTransceiver.LocalVideoTrack = _localVideoTrack;

To stop my screenshare, I'm setting localvideotrack attached to the videoTransceiver to null

_videoTransceiver.LocalVideoTrack = null;
_localVideoTrack = null;

The above code is stopping the screen capture in my end as expected.
But, the

WebRTCFrameCallback(in FrameRequest frameRequest)

is getting called but I've no frames to send as the screen share is stopped. Is there a way to let the remote end know that frames will be stopped from sending? At the remote end, the captured screen is frozen. Ideally, the expectation is to clear the screen in the remote. I checked the behavior from android/iOS apps (that uses webrtc) to the same client that the remote user is using, but it works as expected (ie:- clearing the captured screen at remote call on stopping screen capture). Only for screens shared from UWP using MR webrtc 2.0.2, I'm facing this issue. Any thoughts? I'm I missing something

Duplicate of #681 no? See also discussion with @AtosNicoS in #139. Are you continuing to get frames that are frozen (callback called with the same frame data), or is the rendering callback not called anymore? You can work around by setting a timer, and if the callback is not called for, say, 300ms, then assume stream stopped and clear to black the UI.

It's actually the other way around. I'm sharing my screen (in UWP) to remote web client and

WebRTCFrameCallback(in FrameRequest frameRequest)

is getting called and requesting me to send a screen captured to the remote end which I'm doing it by calling

frameRequest.CompleteRequest

by fetching the latest frame from my frame pool.
The moment when I stop my screen share, the following callback from mixed reality library is still called

WebRTCFrameCallback(in FrameRequest frameRequest)

but I've no frames to send. This is resulting in a frozen state in the remote end. I was just wondering if there is way to gracefully let the remote end know that I've stopped my screenshare.

To stop my screenshare, I'm setting localvideotrack attached to the videoTransceiver to null

You're not stopping the video track source, you're only detaching the source from the transceiver. So the source is still live and continues to produce frames (and therefore the callback is called). Those frames are then discarded because there's no track connected to the source. You need to destroy the ExternalVideoTrackSource once the track is detached, and recreate it next time you need it to restart screen share. You cannot "pause" a video track source; the Google internal design doesn't allow that.

Note that on the remote side the issue after detaching the track is exactly what's #681 is about.

@KarthikRichie
Hey,
would it be possible for you to give me a little information on how you realised your screen capturing? Because I am currently facing exactly this problem. I'm using GraphicsCapturePicker  for screen capturing and what I get is a Direct3D11CaptureFrame.
I don't know how to get the connection between my Direct3D11CaptureFrame and those DeviceVideoTrackSource and I420AVideoFrame|Argb32VideoFrame. But I found your post and see that you used ExternalVideoTrackSource for that. I also created a post about that.

@arsi0001 , I've added the solution here.


[Azure Search] Add Get Documents API (a.k.a. Lookup) to data plane API spec

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-go
Nothing to generate for azure-sdk-for-go

Automation for azure-sdk-for-js
Nothing to generate for azure-sdk-for-js

Automation for azure-sdk-for-python
Nothing to generate for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node
Nothing to generate for azure-sdk-for-node

Automation for azure-sdk-for-java
Nothing to generate for azure-sdk-for-java

Big thanks to @mhko who did the majority of the work on this.

Can one of the admins verify this patch?

@jhendrixMSFT Some context on this change: This is not a new API, it's just that until now we've never generated code for it. This is the first of many steps we're taking to make this particular Swagger spec accurately reflect our data plane API.

@brjohnstmsft do you do any local C# codegen to verify this looks ok?

@jhendrixMSFT Yes, I'm working on a .NET SDK PR that I will send out once this is merged. I wrote the translation rules in tandem with changing the .NET SDK.


grpc-backend running out of memory

Tracking issue for the GRPC out of memory error. We likely need to bump the memory given the recent large OSV records we are now serving.

Manually bumped the memory to 8Gb from 4Gb, and will continue monitoring

This seems to have stopped OOM errors now, though there is a lot of 20 second queries now at 99%> . Theory is that this is caused by the paging limit correctly kicking in and preventing a timeout, and now that there is more memory these requests are not killed by OOM.

@another-rex is this okay to close now?


(WIP) Fix #928 - Modularize software bus routing

Describe the contribution
Fix #928 - refactor SB and move routing to a module
Still in progress...
Also

Deletes unused code CFE_SB_FindGlobalMsgIdCnt
Note writes to file not protected by lock (before or after refactor)
CFE_SB_SendPrevSubs isn't really protected by the lock... might be worth removing the locks completely (they don't really solve anything, also looks like a double lock potential when sending event
Whitespace/alignment is very difficult to work with
Excessive comments (in my opinion)
violates declare variables at the start of function coding standard, some fixed

Likely spawn follow-on PRs to clean up.
Testing performed
So far just core compiles, working unit tests and CI issues (likely documentation)
Expected behavior changes
None expected, but supports alternative implementations (hash table, searches)
System(s) tested on

Hardware: cFS Dev Server
OS: Ubuntu 18.04
Versions: bundle + this commit

Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

Ready for proof of concept discussion...

CCB 2020-10-14: Delayed discussion, will splinter or discuss next week.

Worth noting this is a step in the direction of implementing just the direct lookup (no longer has the extra abstraction layers for other implementations).  The hash implementation will have the extra hash layer, but no need to maintain it in the direct lookup implementation.  This results in simplification of the direct lookup implementation.

Trade removing the individual unsubscribe events when deleting a pipe to avoid having to unlock which introduces a possible race (old implementation unlocked so could race).  Would be messy to store all the messages up to send at the end after the unlock, other suggestions welcome.

Implementation/testing update - add a debug event for collisions (on subscribe), added hash implementation.
Enabled hash and there were no collisions with the nominal routing table size.  Reduced table size to 32 and there were only 4 single level collisions so the factor of 4 on the hash table seems very reasonable for handing a full routing table (and the hash performed well).  The messages past 32 (there's more like 40 or so in the framework) got rejected as expected.
Performing minor cleanup on the test cases (fails for size 32 routing table).  Some event counts could theoretically change due to the new debug message but none seen in practice.  Could switch subscribe test event count asserts to be >= to handle it, but as long as the hash function isn't changed and the routing table isn't incredibly small (16 or less) they will work fine.

Brute force ForEachMsgId is unreasonable for any significantly large MsgId space.  Search would also be resource intensive for larger routing tables... only behavior difference from ForEachRouteId is order, propose removing ForEachMsgId (only used for message map reporting, could be sorted offline if needed).

tagging self

Refactored commits for review.  Ready.

Squashed cppcheck warning.

CCB 2020-10-21
Conversation about collisions. System can be sized appropriately to avoid collisions. Users can deterministically predict the bound by appropriately choosing the MsgIDs.  There's a debug event upon subscription which alerts about collisions.
Open issue to add documentation about ideal usage.

Issues submitted, see #961 and #962.

The routing module has a common file and APIs, then it has optional (hash or direct) modules. Is the intention to allow the replacement of everything with a different implementation?

Yes.  That's the real difference from the core, the module can be completely replaced with a custom implementation.

Or would the common files (in unsorted.c) always be used?

Not if someone implements a sorted routing table with a search instead of a message map.

In the end, I think we need a big reorganization of the code to reflect the evolving nature of how things go together.

Agreed! opened #972 to discuss.  I like the msg/sbr modules and core being peers, and eventually all the cfe elements could become modules (someone could entirely replace table services or time services for example).  It's been an as-needed process so far though.


Other search engines

Currently not supported :

[ ] http://lemoteur.orange.fr/?kw=test
[x] https://fr.search.yahoo.com/?p=test
[x] https://ca.search.yahoo.com/?p=test
[x] https://search.yahoo.co.jp/search?p=test
[ ] https://lite.qwant.com/?q=test

The yahoo host weren't detected without keywords. That was fixed in https://github.com/piwik/piwik/commit/714f7c13849267707cab7c72539c33437858a11c
I'll have a look at the other two.


extension doesn't pick up nvim path

Hi! I was trying to set up the extension on linux (elementaryOS 5.1) but it doesn't pick up the neovim path that I set. I tried multiple versions, first the one from snap store (which is v0.5.0), then I tried downloading the nvim.appimage (which is v0.4.3) and setting that path, both didn't work.
steps to reproduce:

download nvim.appimage into ~/downloads/
in the vscode extension settings, set neovim path to ~/downloads/nvim.appimage
restart vscode
extension doesn't launch

Any idea what the problem could be?

Maybe try using the absolute path? The ~ bit usually doesn't work outside of a shell

Ahhh yep, that was actually it, thank you!


Add ability to share elements with 'unique' URL

Desired Feature:
It would be great to have a way to share single books/chapters/pages with a single party without requiring them to have an account. I realize it is possible to just make an element available to the public but this is not a great solution if I want to share content with multiple parties. Let‘s say I want to share book A with person X and book B with person Y. I don‘t think there is a way to do this without sharing books A and B with X and Y by making both books publicly available.
What I am thinking about is an obscured link with a hash like Google Docs does it.
I haven't been using BookStack for too long, so maybe I'm missing a clever way to achieve this with the existing permission system?

@wutfan Thanks for this request. I agree this would be a great feature and there's not currently a way to do this.

I would also love to see this functionality!

https://github.com/koios-sh/BookStack
Hijack the authenticate and bypass the url prefix '/books/', '/search/', '/uploads/images/', set the permission of one book to public, close global 'public access', and I have this feature with minimum code changes.
Hope for the official implements with QRCode url.

Any update? :)

This would be a great feature

This feature really need to be implemented.

yes please...

Agreed, I would utilize this feature a lot.

+1 for this feature ❤

Would be great 👍

+1

I totally agree, having the ability to share things using a unique URL would be an excellent feature to have!

+1 !

+1
That would be missing piece to establish Bookstack within my department, since we don't want the hassle of username/password authentication with the end-users.

+1

+1

+1

This would be really helpful - kind of like the sharable links that Wallabag has.

+1

Been trying to think this through so just jotting down my thoughts so they're not stuck in my head.
Proposal

Share menu on content (books, chapters, pages, probably not shelves) to create share links.
Share links lead to their own routes/controllers/views.

Super minimal UI & feature-set, with just content and page/context navigation.
Safer than punching through auth an adding extra visibility/access control.
Book/Chapter could complicate routing a tad with navigation within

plus would ideally need to handle internal cross-reference links.

Role permission needed for creating share link, disabled by default.
Overview of created links in settings/admin area.
Probably would need an indication/view of existing share links in entity permission controls view.
Share link would be entry in DB

Linked to entity
Stored creator & created date details.
Actual share unique ID a random gen 32 char (URL safe) value

Keep thinking this should have some added level of protection but we'd really want this to be accessible/reversible (to show and reuse as active share links) and any extra level of protection seems redundant.

Permission Complications

Not confident on what permissions these links should have. There's lots of options/assumptions that could be considered here. But I'm not keen on complicating matters here for potential needs and scenarios.

I'm leaning toward full view access for simplicity (in implementation and understanding/UX)

But then there's extra risk of over exposure which we'd probably need significant warnings at appropriate levels (for the role permission, when creating a link, when managing permissions) which spreads quite a bit of UI mess.

Permissions will complicate image & attachment access (which we'd really need to provide).

ich würde mich über diese Funktion auch sehr freuen

This feature is a must for me, I often make small little docs to share but have to export them at the moment as there's no way to share the individual doc with only that person without allowing all public to view all docs (this is without creating an account etc)

+1

It's the only feature that this awesome application is lacking of 😀

Just search this up and ran across this thread actually, definitely a +1 from me as well.


looking for appinsights-rs maintainer

@dmolokanov is looking for a new maintainer for https://github.com/dmolokanov/appinsights-rs .
My preference is to keep it as a separate repo, similar to how https://github.com/Microsoft/ApplicationInsights-dotnet is separate. Thoughts?

Hi @cataggar,
I have some time for contributions, so I'm happy to help here.


Find-GRIDJob

Returns a list of jobs or a specific job or something like that. No idea what sort of object it should return.

Include in this a get-gridjobspec, which will return in English print format the job.


Add missing name property when using timeSlice option

fixes #210

Coverage decreased (-0.5%) to 79.347% when pulling 32debc7bab03589f7017621bb8e5439c751a5a23 on file-name-210 into 60af1ff4b7b2c87f49855f8feef5490d5be7ba77 on master.

I'm not happy with these changes because I'm not able to reproduce the problem (#210), also not using the unit tests. So closing this. @SergeyKozlov did this PR help you at all?

@thijstriemstra Thanks, my problem #210 is solved.


Upgrade go to 1.17.8

What this PR does / why we need it:
Update go to 1.17.8 to solve some CVE vulnerabilities that affects previous versions.

https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-23772

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #7306
Special notes for your reviewer:
Release note:

Upgraded go version to 1.17.8

/hold
/cc @iholder-redhat @acardace
Need the builder to be pushed in kubevirt repository

/retest

Pushed correct builder image

/retest

/retest

/retest

/retest

/approve


fix: fix docs & feedback links

Fixes GDT-168

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#301  👈
#300
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


fix(ts_indexer): bug related to overriden functions.

This commit has 2 changes:

Properly emit overrides edges for functions when class/interface being extended/implemented is exported from another file.
Emit extends edges from class/interface to all classes/intefaces in its heritage clause.

Tested:
tests

This fix is related to internal bug: http://b/146173865

Why does this fix the bug
In the situation of imported interface:

import {IFace} from './interface';

class Implementation implements IFace {}

When we call getSymbolForLocation on IFace in implements clause - it gives us location of the IFace symbol in the import statement, not the IFace symbol in interface.ts. But getting type works as type contains the main symbol from interface.ts.


socket.error: [Errno 98] Address already in use

Hi!
I'm trying to get the script running but after getting gr-gsm running, when starting the IMSI-catcher I get the following error:
root@kali:~/IMSI-catcher# python simple_IMSI-catcher.py
Saving to SQLite database in imsi-catcher.sqlite
Nb IMSI ; TMSI-1     ; TMSI-2     ; IMSI              ; country      ; brand      ; operator              ; MCC  ; MNC   ; LAC    ; CellId
Traceback (most recent call last):
File "simple_IMSI-catcher.py", line 534, in
udpserver(port=options.port, prn=find_imsi)
File "simple_IMSI-catcher.py", line 487, in udpserver
sock.bind(server_address)
File "/usr/lib/python2.7/socket.py", line 228, in meth
return getattr(self._sock,name)(*args)
socket.error: [Errno 98] Address already in use
root@kali:~/IMSI-catcher#
Does any one have an idea for me or know of this issue? Ive scoured muliple forums but can't seem to find something that works.
Thanks so much in advance.

Use python simple_IMSI-catcher.py --sniff

thank you so much @Oros42 ! Highly appreciated. problem solved.


reexport path while using derive

imagine you creating new crate what depends on serde and reexport it
pub  use serde::*;
In your crate you creating a new proc_macro. In proc macro you creating something like
let pub_struct_something = quote::quote!{
    #[derive(your_new_crate::Serialize)]
    pub struct Something {
        my_field: bool
    }
};

quote::quote!{
    #pub_struct_something
    #some_other_logic
}.into()

The problem is - generated code is still trying to import modules from serde, not from your_new_crate.
I suggest wrap "expand_derive_serialize"(or something) in
pub fn derive_serialize(input: TokenStream) -> TokenStream {
    let mut input = parse_macro_input!(input as DeriveInput);
    ser::expand_derive_serialize(&mut input)
        .unwrap_or_else(syn::Error::into_compile_error)
        .into()
}

into new public function, something like
"expand_derive_serialize_WITH_PATH_TO_CRATE"
with additional(maybe few) parameters what change all serde occcurences to parameter value.
Of course defalut would be just serde.
For example new implementation would be
pub fn derive_serialize(input: TokenStream) -> TokenStream {
    let mut input = parse_macro_input!(input as DeriveInput);
    ser::expand_derive_serialize_WITH_PATH_TO_CRATE(&mut input, "serde")
        .unwrap_or_else(syn::Error::into_compile_error)
        .into()
}

and you can write something like
let generated_serialize_token_stream_for_something_struct = serde::expand_derive_serialize_WITH_PATH_TO_CRATE(&mut possible_something_struct_token_stream, "your_crate");
Well i can write implemenation mannualy in my macro but in time i think it will be outdated i just wont know about some vulnerability or new optimization.

I would prefer not to work around this in the way you are asking.
https://github.com/rust-lang/rust/issues/54363#issuecomment-1237544924 has a writeup of my preferred approach.


Version Packages (alpha)

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
⚠⚠⚠⚠⚠⚠
main is currently in pre mode so this branch has prereleases rather than normal releases. If you want to exit prereleases, run changeset pre exit on main.
⚠⚠⚠⚠⚠⚠
Releases
<EMAIL_ADDRESS>Patch Changes

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies []:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#3224 5971d528 Thanks @acao! - try removing some packages from pre.json

#3149 d9e5089f Thanks @mskelton! - Fix hover docs being off by one position.

<EMAIL_ADDRESS>Patch Changes

#3224 5971d528 Thanks @acao! - try removing some packages from pre.json

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS><EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#3224 5971d528 Thanks @acao! - try removing some packages from pre.json

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [5971d528, d9e5089f]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#3224 5971d528 Thanks @acao! - try removing some packages from pre.json

Updated dependencies [5971d528]:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>Patch Changes

#3224 5971d528 Thanks @acao! - try removing some packages from pre.json

<EMAIL_ADDRESS>Patch Changes

#3224 5971d528 Thanks @acao! - try removing some packages from pre.json

@B2o5T I'm pretty sure that the vscode pipeline will fail now because vsce doesn't support alpha tags

@B2o5T how we can prevent this issue in the future:

Warning: If you decide to do prereleases from the main branch of your repository, without having a branch for your last stable release without the prerelease changes, you will block other changes until you are ready to exit prerelease mode. We thoroughly recommend only running prereleases from a branch other than the main branch.

from changeset prerelease docs

@B2o5T for the record though, this is fine to publish as is, I will do a manual publish of the extension later in the week. gives me some time to do some manual regression testing too


static method hiding produces false positives when original method is forbidden

Consider the following code (contrived example, but simplest I could find in a hurry that anyone can try using only JDK methods w/o needing third-party libs) ...
import java.util.BitSet;
public class Main {
    public static final long[] data = new long[] {1, 2, 3, 4};

    public static void main(String[] args) {
        System.out.println(X.valueOf(data).toString());  // Line 6
        System.out.println(Y.valueOf(data).toString());  // Line 7
        System.out.println((new Z()).go());
    }

    public static class X extends BitSet { }

    public static class Y extends X {
        public static BitSet valueOf(long[] longs) {
            return new BitSet();
        }
    }

    public static class Z extends Y {
        public String go() {
            return valueOf(data).toString();            // Line 21
        }
    }
}

Let's see what happens when we tell forbidden-apis we don't want our code to use Bitset.valueOf(...) ...
hossman@slate:~/tmp/fapi-bug$ cat sigs.txt
java.util.BitSet#valueOf(**)
hossman@slate:~/tmp/fapi-bug$ java -jar forbiddenapis-3.6.jar -d . -f sigs.txt
Scanning for classes to check...
Reading API signatures: /home/hossman/tmp/fapi-bug/sigs.txt
Loading classes to check...
Scanning classes for violations...
ERROR: Forbidden method invocation: java.util.BitSet#valueOf(**)
ERROR:   in Main$Z (Main.java:21)
ERROR: Forbidden method invocation: java.util.BitSet#valueOf(**)
ERROR:   in Main (Main.java:6)
ERROR: Forbidden method invocation: java.util.BitSet#valueOf(**)
ERROR:   in Main (Main.java:7)
ERROR: Scanned 4 class file(s) for forbidden API invocations (in 0.06s), 3 error(s).
ERROR: Check for forbidden API calls failed, see log.

It correctly identified that line 6 is "bad" for it's "Forbidden method invocation", but the errors reported for Lines 6 & 21 are false positives -- those lines do not invoke Bitset.valueOf(...), they invoke Y.valueOf(...), which hides the static method with the same name provided by Y's ancestor class...
hossman@slate:~/tmp/fapi-bug$ java Main
{0, 65, 128, 129, 194}
{}
{}

Practically speaking: This came up in the context of LuceneTestCase extending Assert but some Solr devs really wanting to forbid the deprecated methods in Assert -- but using static method hiding to re-define the deprecated methods in SolrTestCase still triggered forbidden api failures, so that approach was abandoned...
https://github.com/apache/solr/pull/947#issuecomment-1279651282
https://lists.apache.org/thread/nycfk6b2bqkqsbq29snow51w31qo1dd3

Hi @hossman,
sorry I was on a business trip when you opened the issue. Actually you are right the code handles static and virtual methods the same. This bug is known, but at beginning it did not seem important.
After some checking: Only the Java Compiler searches parent classes for static methods (if not found in the class referenced), but bytecode always has the exact class name. If you move a method from a class to its superclass and do not recompile the code, it fails to link the code at runtime.
So basically, I will add an if statment to the method resolver to not check parent classes if the method is static.

Hi, correction: Static methods are inherited by subclasses, so the behaviour is correct. The problem is the viewpoint of forbiddenapis. In contrast to some code calling a method, forbiddenapis has to make sure that also hits are found when you call a virtual method in a subclass and this one calls super.xyz(). This is the problem of real "virtual" methods. The method called is decided at runtime (therefor virtual).
For static methods, we may stop going up to superclasses, but I have to check this one more time: I am not sure if you can use super() to call superclass's static method It makes no sense but it is unclear if the java language allows this.
If you cannot call a superclass static methods with super, I agree to remove the superclass check.

I have to read https://docs.oracle.com/javase/specs/jvms/se21/html/jvms-6.html#jvms-6.5.invokestatic and compare withhttps://docs.oracle.com/javase/specs/jvms/se21/html/jvms-6.html#jvms-6.5.invokevirtual
May take some time.

Static methods are not virtual, Uwe. They can be shadowed but the invokestatic call always uses an explicit class reference (because of that you can call SuperClass.staticMethod from Subclass.staticMethod... in fact, you can call SuperClass.staticMethod and Subclass.staticMethod from anywhere you like).

Here is the way how methods are looked up: https://docs.oracle.com/javase/specs/jvms/se21/html/jvms-5.html#jvms-5.4.3
It makes no difference between static and non-static. The difference is only implemented in the compiler. Basically you can refer to a subclass' static method, if it does not exist it find the superclass' method. As forbiddenapis does not know if another subclass hides this method, it has to go to a superclass (because it needs to ensure there are no other possibilities how to call this method). Forbiddenapis problem also exists for virtual methods: As it doe snot know it the virtual method calls super.name() it has to disallow calls to that method on all subclasses. Basically there's no difference with static methods regarding lookup. If you have code that calls in its bytecode the static method Subclass#method() and this method no longer exists (e.g., because of refactopring) the JDK will per spec search for Superclass#method() [see above].
@dweiss : I explicitely said they are not virtual, I just compared the spec. When you have a virtual call you have the argument with the class on stack, but it then does exactly the same: it looks up the method as described above.
In short: actually in JVM runtime theres no difference with method lookups. The difference is only with the argument on stack which is the class for the virtual call. But lookup of method is identical, it just does not use the class on stack but the one in bytecode.
So basically that's all forbiddenapis can do. It has its limitations. We can add a specialcase for static methods to only look at the exact descriptor and not dive into superclasses, but this would have the problem of undetected calls.

Ok. I think I can remove the static call. The problem with virtual calls is different. On bytecode you do not know which method is actually called, so you have to check all possible superclass es (from perspective of forbiddenapis, as first argument is unknown).
In case of static methods you can stop in forbiddenapis when method was found.
I will provide a PR.

I'm not sure I agree with you here (with regard to virtual vs. static method calls)... but anyway. I think considering odd cases like swapping out classes without proper recompilation (which would emit different bytecode) is not worth it. 99.9% of the cases are legitimate valid code and javac would see the difference and link it up properly.

I'm not sure I agree with you here (with regard to virtual vs. static method calls)... but anyway. I think considering odd cases like swapping out classes without proper recompilation (which would emit different bytecode) is not worth it. 99.9% of the cases are legitimate valid code and javac would see the difference and link it up properly.

See my response.

Sorry for the confusion. The problem of forbiddenapis is that it needs to look somehow without runtime types. For virtual methods it is hard, because you have no information which class actually receives the call.
For static methods you can just follow the spec and stop on first found method.

Btw., the same apples for fields.

I did see it. But I still think maybe forbiddenapis shouldn't try to play JVM here - if a static method doesn't exist at its target class, something is wrong - classes are stale and this should perhaps produce a warning (javac should never produce such bytecode because it always resolves static call to a concrete class).

I did see it. But I still think maybe forbiddenapis shouldn't try to play JVM here - if a static method doesn't exist at its target class, something is wrong - classes are stale and this should perhaps produce a warning (javac should never produce such bytecode because it always resolves static call to a concrete class).

Yes, if you scan your own compiled classes the output of javac should be "uptodate". Elasticserach uses forbiddenapis also to scan JAR files on unknown source which might be very outdated.
Anyways, I think the simplest / only thing to change is: If method call is static (invokestatic or MethodHandle with tag H_INVOKESTATIC) it should stop at first found method.

Hi, I think simplest is your first suggestion: for static lookup do not go up the chain.
BTW: I fixed the code to assume static also for invokespecial (and its new variant) bytecode, because it is used for ctors and super calls which are always handled like static (argument is ignored).

Here is a PR: #238
I still need to add a test.

I rewrote the PR to now also look into superclasses until a match was found, It was doing this for fields already, but it was not correct according to static or virtual. Now both methods and fields are handled in the same way. #238
I still need a test. I will also run a check on Lucene, Solr, Opensearch as integration test.
... and sleep a night over it.

Field handling was correct. For methods it now differentiates only between virtual/interface invocations (collect all superclasses) and static/special (only until found). Although for "special" it should not go to superclasses at all (used for constructors and super calls).

I still need to add a test.

FWIW: I was hoping to provide a (commitable) test case when i created this issue, but even as someone who has a lot of experience with java, and using forbidden-apis: I couldn't' make heads or tails of how to write a new test case for this type of situation based on the existing tests i found.
It would be great if there was a "HelloWorldIntegreationTest.java" that showed some basic rule checking of code compiled as part of the test class that could serve as an example to folks who want to submit future issues/PRs demonstrating problems they encountered.

The test needs to be done like the many antunit tests.
The actual tests need to be example class files and then you can assert on the results with antunit.
I will take your example code above.

Hi @hossman,
Test was added to PR: https://github.com/policeman-tools/forbidden-apis/pull/238/commits/fbb29ef9c6f254d8204d1652850a9bfda00ba8bb
I will also add a fields check and I want to extend the test to check virtual methods, too.

FYI, to setup a test like this: Add a class with Java7* as prefix (this will be compiled on ant generate-test-classes and saved in repo as "test file" (the Java8* prefix is for lambda checks only). Then write an antunit test that checks the output (like error messages and/or line numbers and number of errors).

I added a few more tests to check static and virtual methods:

virtual methods are checked on all levels so every access is a violation (no method hiding)
static methods are checked only until they are found (a subclass can hide the superclass' method)
all fields (static and instance) are checked until they are found (a subclass can hide them)
I think it's ready.

ahhhh, ok.  yeah. test looks good.
The part i wasn't understanding originally is that you keep all the "integration tests" in the anttest directory -- I assumed from the name that it was just for using anttest to check the behavior of the <forbiddenapis> ant plugin ... and i didn't look very closely so i didn't realize that's the way you do "end to end" testing (I kept expecting to find JUnit tests that would call new Checker(...))
Out of curiosity: why do you commit all the "test input" class files?  I understand some of them are because they have to be compiled using old JDKs, or using oracle specific JDKs, but couldn't most test just be re-compiled on every run and pulled from the test-classpath?

The part i wasn't understanding originally is that you keep all the "integration tests" in the anttest directory -- I assumed from the name that it was just for using anttest to check the behavior of the <forbiddenapis> ant plugin ... and i didn't look very closely so i didn't realize that's the way you do "end to end" testing (I kept expecting to find JUnit tests that would call new Checker(...))

This was easiest at beginning. There is still an issue open to have the Checker API return a list of structured "violation" instances. By this is would be easier to write "standard tests". Antunit was quite nice to configure input files/signature files/pass signatures. The only thing that is harder is to check is the returned violations. But with some logic you can grep on the log output and catch the cases you like.

Out of curiosity: why do you commit all the "test input" class files? I understand some of them are because they have to be compiled using old JDKs, or using oracle specific JDKs, but couldn't most test just be re-compiled on every run and pulled from the test-classpath?

That's also historical: At beginning forbiddenapis was java5+, so files for Java 7 and Java 8 and Java 9 needed to be compiled separately. In addition some tests were checking for undocumented features (the ones we would like to prevent, like sun.misc.Unsafe). At this time it was unclear if those classes survive. If you can't compile violating classes with all JDKs out there, it is hard to check (P.S.: In Lucene you can't run Gradle with IBM Semeru Runtime, because jfr module is missing there!). So back at that time I decided to add all class files as "test resources" and just added the option to regenerate those.
But yes, I am planning to switch to Java 8 as minimum requirement, so the classes can be compiled out of box (for all checks). That's something for a lonely weekend...

There are more generation tasks in forbiddenapis: The deprecated signatures files can be generate, too, but you need to run with exactly that JDK...

hi @hossman,
do you need a new version for Solr urgently? I would then release a new version soon (after doing more tests with the usual candidates Lucene/Solr/TIKA/Opensearch).

no no, this is a nice to have.
no urgency needed.

v9.7 was released a minute ago.


feat : HCX-323 - Create a script to update the existing audit events in elastic search

@AbhiGaddi - Create a foder with name scripts in hcx-platform and add the data-migration-script module to the scripts folder.


Fix default export

The proper way to import polly seems to be import polly from 'polly-js', however using export = polly will throw a syntax error on that import. This fixes that.

Current coverage is 97.40% (diff: 100%)

Merging #30 into master will not change coverage

@@             master        #30   diff @@
==========================================
  Files             1          1
  Lines            77         77
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             75         75
  Misses            2          2
  Partials          0          0

Powered by Codecov. Last update f2ef298...38c3e40

Not sure where the export = polly came from.
Thanks for the fix.


IGNITE-14143 Document metric for processed keys when rebuilding indexes.

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

@tkalkirill Please take a look at the PR.

I've applied all the suggested changes.

@Mmuzaf Hi Maxim! Could you please cherry-pick this improvement to the 2.10 documentation?

@Nikita-tech-writer
Can you also update the IndexRebuildKeyProcessed and IsIndexRebuildInProgress description at the new-metrics.adoc page?

@Nikita-tech-writer
Can you also update the IndexRebuildKeyProcessed and IsIndexRebuildInProgress description at the new-metrics.adoc page?

@Mmuzaf Done:
[1] https://issues.apache.org/jira/browse/IGNITE-14271
[2] https://github.com/apache/ignite/pull/8848/


Redistribute styles

some of the styles are only relevant for specific components, consider moving them there.

code state https://github.com/ardc-fair-checklist/ssg/tree/0b86ac1748a47e9af19528d0c6d5a449a4405cb2 has styles mostly included with and scoped to individual components. Closing.


Sync stoped at 469818

Hi,
the syncronisation stopped at 469818 blocks. do you know how can I fix it?
here are the logs.
app_logs.log
binary_logs.log
Thanks for your help

Hi @defialex , did you try Reindex option in Settings?

Hi @defialex , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.

Hi @defialex , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.

It worked. Sorry for not responding for so long!
Thanks for your help.

It worked. Sorry for not responding for so long!
Thanks for your help.


Helm chart clean up

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #892
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/ok-to-test

/hold cancel

🤔 deleting the E2E cluster failed....
/retest


Add additional 2022 Santa Fe FW

You could use Force Car Recognition to force recognize your vehicle.
The FW version should be PR'ed to upstream.
Thanks for the PR!


✨feat: Enhance support for containers

This PR aims to enhance LXC support for Proxmox. Here is the documentation of LXC(Containers) API.
https://pve.proxmox.com/pve-docs/api-viewer/index.html#/nodes/{node}/lxc

I'm not sure if this PR satisfies #118.

@alperencelik this pretty much what most people need to get going with containers. Closes #118 imho
If you think we can drop that type and/or know of a fix for the rrddata func for vms then feel free to drop another PR. going to get this merged and cut a release tonight as this is a ton of new funcs added that people may way to use

Thank you so much for reviewing @luthermonson.
I would like to hear your thoughts as well so for /lxc/status I'm offering to convert the strings to tasks since the /status returns a UPID but for /rrdData I guess I need some time to test it.


about Cosy Voice

In the generation process of Cosy Voice a flow matching module is employed to convert Speech Tokens to Mel Spectrum

my question is why is flow model employed instead of diffusion model？

well maybe @ZhihaoDU can answer this question, have you done some ablation study?

In the generation process of Cosy Voice a flow matching module is employed to convert Speech Tokens to Mel Spectrum  my question is why is flow model employed instead of diffusion model？

In general, flow matching models require much less iterations (5-10 steps) than denoising diffusion models to obtain a satisfactory Mel spectrum. We choose flow matching rather than denoising diffusion models for efficiency reasons.

In the generation process of Cosy Voice a flow matching module is employed to convert Speech Tokens to Mel Spectrum  my question is why is flow model employed instead of diffusion model？

In general, flow matching models require much less iterations (5-10 steps) than denoising diffusion models to obtain a satisfactory Mel spectrum. We choose flow matching rather than denoising diffusion models for efficiency reasons.

Thanks for answering


Running test in a single folder

All tests will now run on a single folder, to prevent generating too many RDFs or NonRDFs in the root folder.

@sprater Please merge this PR if it works on your side.


Verify flaky CI Setup workflow

Sometimes, it seems to lose the cache or something like that:

https://github.com/FuelLabs/fuels-ts/actions/runs/7787432043/job/21234957254

So the cache is invalidated because apps/docs has it's turbo inputs changed as we don't save typegen generated files to git. I believed this was resolved in #1694 with a generated file cache, potentially I have missed something. Another potential route I considered prior to #1694 was to completely remove the inputs from the docs turbo config, treating it the same as we do the other packages. A cache invalidation would then only be triggered when the dist hash doesn't match, not the src, so we don't have to worry about auto generated files. This is how this is not a problem with the GQL generated code in providers.

Another potential route I considered prior to https://github.com/FuelLabs/fuels-ts/pull/1694 was to completely remove the inputs from the docs turbo config, treating it the same as we do the other packages.

So removing inputs was invalid as then turbo responds to any changed file. I trialled this in #1719 but closed and will rethink. The other 2 possible approaches are to:

Ignore the `src/api' directory in the turbo input glob. This will therefore ignore the auto generated content from the cache hash, so will not bust the cache for missing typedoc files.
Move the typedoc generated files outside of src. My only consideration for this is potentially a breaking change with the docs hub? cc @FuelLabs/devrel @sarahschwartz

Actually I have now noticed an issue with the generated CI cache implemented in #1694. Should be resolved in #1721. We should not be saving the cache entry before attempting a build, other wise the values we are caching will not contain the generated files.


⚠ Bofrost.lu has degraded performance

In 05c2de1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2169 ms

Resolved: Bofrost.lu performance has improved in 9fc2804 after 37 minutes.


Should I use ID instead of certificate S/N?

The documentation is described as:

From the list of X.509 keys, copy the serial number (S/N) of the X.509 key you'd like to use. In this example, the serial number is a2dfa7e8c9c4d1616f1009c988bb70f:

However, it's incorrect.
When I did that, I could not sign a commit.
When I used ID (thumbprint) instead of S/N, I could sign successfully.
Environments:

smimesign --version
0.0.6
git --version
git version 2.19.1.windows.1

Thanks for opening this. This is a mistake in our documentation. You should be using the ID. I'll open an internal issue about getting this fixed, but will leave this open until that happens.

The documentation should be fixed now.


bubbleStats not working

for whatever reason the bubbleStats are not working properly for me.

Go to /admin/advanced/actions/setupTheme.php to setup your theme.
Make sure bluestats as write permissions to /assets

funny as it is, but I can't access that site, neither can I change my theme.
Had no problem installing BlueStats.

Go to /admin to login into the admin panel first

Did that, but it returns a 500, site not working

I will need some errors to help you any further. Give me the last few lines from /var/log/apache/error_log

[Mon Apr 18 09:51:58 2016] [error] [client <IP_ADDRESS>] File does not exist: /var/www/vhosts/random/stats/robots.txt [Mon Apr 18 17:10:46 2016] [error] [client <IP_ADDRESS>] File does not exist: /var/www/vhosts/random/stats/assets/style.css, referer: random
that pretty much sums up the latest errors.

Those are not the errors that are causing the 500 http error. If you want a fix for the 500 error, please send me atleast the last 3 entries in the error log.
A quick fix for the missing css file is to copy /themes/bigBlue/assets/style.css to /assets/style.css

http://pastebin.com/GeUC2VdG
hope that helps a bit more

Clear the /cache folder and copy the theme assets into the assets folder


Shelly type mismatch: SPEM-003CEBEU

Shelly type mismatch: SPEM-003CEBEU
The new Shelly Pro 3EM are not recognised, but the output is OK.

Ok I will add this signature.
Is it compatible to the EM3?

@FrodoVDR I assume that you configured it as gen 2 device Relay? Or?

@FrodoVDR I extended the nodes, but I can not test it as I do not have a 3EM Pro model here.
Can you support me with testing?

closed due to no response from reporter


[Redesign] Fix registration's link to the README

Background
For packages with an embedded readme, registration's readmeUrl links to the display package page using the #show-readme-container URL fragment. For example: https://api.nuget.org/v3/registration5-gz-semver2/knapcode.torsharp/index.json:

Example registration JSON for package with embedded readme...
{
  ...

  "items": [
    {
      ...

      "items": [
        {
          ...

          "@id": "https://api.nuget.org/v3/registration5-gz-semver2/knapcode.torsharp/2.7.0.json",
          "catalogEntry": {
            ...
            "licenseUrl": "https://www.nuget.org/packages/Knapcode.TorSharp/2.7.0/license",
            "readmeUrl": "https://www.nuget.org/packages/Knapcode.TorSharp/2.7.0#show-readme-container",
            ...
            "version": "2.7.0"
          },
          ...
        },

        ...
      ],

      ...
    }
  ]

  ...
}

Fix
In the current design, this is implemented using Bootstrap 3's collapse component. The redesign broke this feature as it uses Bootstrap 3's tab component instead, which does not support opening using fragments. This change re-adds support for the #show-readme-container fragment link using Javascript.
I also fixed some ARIA attributes to improve accessibility.
Addresses https://github.com/NuGet/NuGetGallery/issues/8698

I think the expander section works generically with:
https://github.com/NuGet/NuGetGallery/blob/dab825905a5844e999408a2413d0b6b3417d672a/src/NuGetGallery/Scripts/gallery/common.js#L204-L234
Would it make sense to do something similar for tabs?

@joelverhagen Nice catch, I didn't realize we had implemented that URL functionality.

Would it make sense to do something similar for tabs?

This would be useful, but this particular fix wouldn't use that general/shared logic. The URL fragment to open the README, show-readme-container, is specific to the naming scheme we use for collapsible sections (and changing this URL fragment would require rebuilding registration). Tabs use a different naming scheme: readme-tab, dependencies-tab, etc... If we add generic "open tab using URL fragment" logic, ideally it would work using this new naming scheme with a URL fragment like #readme-tab. So if we add generic logic, we would still need this "backwards compatibility" hack to support the existing registration API.
I'd suggest we punt the generic logic until we have a scenarios that would use this logic. Let me know if you feel otherwise @joelverhagen .


Fix table alignment when seperator contains spaces

Currently table alignment is correct for seperator like ----:|:----:|:----, but incorrect for ----: | :----: | :----, which is commonly used.

@painyeph good catch. Thanks. It appears that our test output was wrong and now that you fixed the bug, the test needs to be updated. If you could include the fix in your PR, I'll merge it in. You can update the test by running the following command.:
./run-tests.py update tests/extensions/extra/tables.txt

That will update the output file tests/extensions/extra/tables.html. Just push that change.

It would appear that you did not use the tool to update the test. Due to an annoying minor difference in (otherwise insignificant) whitepsace, the test is still failing. You can either delete the newline at the end of the file or use the tool to generate the file.

Excellent. Thanks.


Show extendedName and unlockLevelName in i18n-thang editor only if it needs to show

i.e. no unlock level name field for Captain (codecombat.com/i18n/thang/captain)

Nice!


Restructure the Javascript code

So we are minifying the javascript with uglify-js
As a result the Javascript is getting smaller (and faster to download).
Good.
As we go the code is getting complex in ONE uniq file.
So now is the time to create kind of separate js files instead of one big file.
To do this we will split into multiple JS files and then regenerate the uniq JS file with a build.sh file.
At the same time the build.sh will take care of the translation (other issues)

Let's close this one!!