Wrong generated XML Style/Rule from MSS

This could be a dup of issue #20, but
When I use carto to convert the following mss/mml to xml...
#amenity[tourism='motel'] {
  [zoom = 20] {
    point-file: url('/usr/share/icons/sjjb/png/accommodation/motel.glow.0092DA.24.png');
  }
  [zoom > 20] {
    point-file: url('/usr/share/icons/sjjb/png/accommodation/motel.glow.0092DA.32.png');
  }
}

#amenity[amenity='bench'] {
  [zoom = 20] {
    point-file: url('/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.24.png');
  }
  [zoom > 20] {
    point-file: url('/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.32.png');
  }
}

{
  "bounds": [
    -180,
    -85.05112877980659,
    180,
    85.05112877980659
  ],
  "center": [
    0,
    0,
    2
  ],
  "format": "png",
  "interactivity": false,
  "minzoom": 0,
  "maxzoom": 22,
  "srs": "+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0.0 +k=1.0 +units=m +nadgrids=@null +wktext +no_defs +over",
  "Stylesheet": [
    "style.mss"
  ],
  "Layer": [
    {
      "geometry": "point",
      "name": "amenity",
      "id": "amenity",
      "srs": "+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0.0 +k=1.0 +units=m +nadgrids=@null +wktext +no_defs +over",
      "class": "poi",
      "Datasource": {
        "table": "(\n  SELECT way, amenity, tourism\n  FROM planet_osm_point\n  WHERE (amenity IS NOT NULL OR tourism IS NOT NULL)\n) as amenitypoi",
        "key_field": "",
        "geometry_field": "way",
        "extent": "",
        "type": "postgis",
        "host": "localhost",
        "port": "5432",
        "user": "",
        "password": "",
        "dbname": ""
      }
    }
  ]
}

I get...
<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE Map[]>
<Map srs="+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0.0 +k=1.0 +units=m +nadgrids=@null +wktext +no_defs +over">

<Style name="amenity" filter-mode="first">
  <Rule>
    <MaxScaleDenominator>500</MaxScaleDenominator>
    <Filter>([amenity] = 'bench')</Filter>
    <PointSymbolizer file="/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.32.png" />
  </Rule>
  <Rule>
    <MaxScaleDenominator>750</MaxScaleDenominator>
    <MinScaleDenominator>500</MinScaleDenominator>
    <Filter>([amenity] = 'bench')</Filter>
    <PointSymbolizer file="/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.24.png" />
  </Rule>
  <Rule>
    <MaxScaleDenominator>750</MaxScaleDenominator>
    <MinScaleDenominator>500</MinScaleDenominator>
    <Filter>([amenity] = 'bench') and ([tourism] = 'motel')</Filter>
    <PointSymbolizer file="/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.24.png" />
  </Rule>
  <Rule>
    <MaxScaleDenominator>500</MaxScaleDenominator>
    <Filter>([amenity] = 'bench') and ([tourism] = 'motel')</Filter>
    <PointSymbolizer file="/usr/share/icons/sjjb/png/accommodation/motel.glow.0092DA.32.png" />
  </Rule>
  <Rule>
    <MaxScaleDenominator>500</MaxScaleDenominator>
    <Filter>([tourism] = 'motel')</Filter>
    <PointSymbolizer file="/usr/share/icons/sjjb/png/accommodation/motel.glow.0092DA.32.png" />
  </Rule>
  <Rule>
    <MaxScaleDenominator>750</MaxScaleDenominator>
    <MinScaleDenominator>500</MinScaleDenominator>
    <Filter>([tourism] = 'motel')</Filter>
    <PointSymbolizer file="/usr/share/icons/sjjb/png/accommodation/motel.glow.0092DA.24.png" />
  </Rule>
</Style>
<Layer name="amenity"
  srs="+proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0.0 +k=1.0 +units=m +nadgrids=@null +wktext +no_defs +over">
    <StyleName>amenity</StyleName>
    <Datasource>
       <Parameter name="table"><![CDATA[(
  SELECT way, amenity, tourism
  FROM planet_osm_point
  WHERE (amenity IS NOT NULL OR tourism IS NOT NULL)
) as amenitypoi]]></Parameter>
       <Parameter name="key_field"><![CDATA[]]></Parameter>
       <Parameter name="geometry_field"><![CDATA[way]]></Parameter>
       <Parameter name="extent"><![CDATA[]]></Parameter>
       <Parameter name="type"><![CDATA[postgis]]></Parameter>
       <Parameter name="host"><![CDATA[localhost]]></Parameter>
       <Parameter name="port"><![CDATA[5432]]></Parameter>
       <Parameter name="user"><![CDATA[]]></Parameter>
       <Parameter name="password"><![CDATA[]]></Parameter>
       <Parameter name="dbname"><![CDATA[]]></Parameter>
    </Datasource>
  </Layer>

</Map>

Notice the XML has rules for ([amenity] = 'bench') and ([tourism] = 'motel') which given my original mss I can't see how these rules should be present.

Carto doesn't know that the columns amenity and tourism are mutually exclusive, so it can't know that it doesn't make sense to generate these rules. Carto generates all possible combinations, which includes a row where amenity == 'bench' and toursim == 'motel' (even if that combination never appears in your database). This is because Carto inherits rules:
#amenity[tourism='motel'][zoom = 20] {
  point-file: url('/usr/share/icons/sjjb/png/accommodation/motel.glow.0092DA.24.png');
}

is what it starts out with and merges in
#amenity[amenity='bench'][zoom = 20] {
  point-file: url('/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.24.png');
}

This results in
#amenity[tourism='motel'][zoom = 20][amenity='bench'] {
  point-file: url('/usr/share/icons/sjjb/png/amenity/bench.glow.8E7409.24.png');
}

However, there's a slight inefficiency in carto as it wouldn't have to generate the second rule because the previous roule would already have rendered all those, so the ([amenity] = 'bench') and ([tourism] = 'motel') rule never matches. The behavior, however, is intended.

I'll use a different layer for each column then. ie. #amenity which just selects amenity, and #tourism which just selects tourism.
I realise now that this generated XML is exactly how I expected the rendering style to be applied. That is, if it finds tourism=motel AND amenity=bench then it will apply the style of the most recent one, which is what the rules generated do, I just didn't realise it needs to write this out to the XML file, as it just blows up the file size to a point that it can't be used. I thought Mapnik would implicitly decide this just from the ([amenity] = 'bench') rule and the ([tourism] = 'motel') rule without the need of another rule dealing with both.
Thanks anyway, at least I can fix it it my style sheet.

You in fact found a valid bug (it generates too many styles), so I'll keep this open. However, this bug shouldn't affect rendering at all; it's only about the efficiency.

I'm pretty sure this is a dupe of #20 - it's describing the same situation and the same end result.


Conflicting default values for  “query_maxretrywaittimeinseconds” and “query_maxretryattemptsonthrottledrequests”

As per this document the default vales for below parameters are
“query_maxretrywaittimeinseconds” = 1000
“query_maxretryattemptsonthrottledrequests” = 1000.
But according to https://github.com/Azure/azure-cosmosdb-spark/wiki/Configuration-references, the default values are
“query_maxretrywaittimeinseconds” = 30
“query_maxretryattemptsonthrottledrequests” = 9.
Can you please let me know which one is correct?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5c77b390-e483-d7d4-3ea8-813a296c9e89
Version Independent ID: 3b1e6865-894e-9958-3e6d-4b04bf40e892
Content: Connect Apache Spark to Azure Cosmos DB
Content Source: articles/cosmos-db/spark-connector.md
Service: cosmos-db
GitHub Login: @tknandu
Microsoft Alias: ramkris

@saravp123 The GitHub wiki isn't in sync with latest updates, what we have in public docs is the latest. These are the right values:
“query_maxretrywaittimeinseconds” = 1000
“query_maxretryattemptsonthrottledrequests” = 1000.
I updated the wiki with latest information.

#please-close

@saravp123 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Login page doesn't redirect if you are already logged in.

If you open the login page while already being logged in, you wont be redirected.

Good catch @wovasteenGova!
Yeah we should just make the /login page redirect to / if the user is already signed in. The 403 error is not very useful.


Aspect ratios of the input and output images are different

The output image used to have the same aspect ratio as the input image in the original version, since then a change has resulted in the output image being squished.
I believe the cause was this code removal:  https://github.com/pfnet/PaintsChainer/commit/963e2da553487a473aeb6abd89808323e8cbc2f0#diff-bfd7a84bc84a48c1888381141bfa873b
I'm not certain if it should be JavaScript that goes back in as the fix though. If it did, the webpage display would be correct but users downloading the output image would save one with the wrong aspect ratio.
Potentially better solution; the server side code does a final image resize back to the original aspect ratio when it's finished processing. This way if someone downloads the output image the aspect ratio will be same as the original (even if the image is now smaller).

It is needed to resize the input image to multiple of 16 otherwse, NN will crush...
so server side resize is better but its a little bit cpu cost


Add a test hub to check Pulumi's PR output

For some reason, adding diff: true to the Pulumi Github Action created nicer-looking info on the PR comment

Closing this because the point was to check Pulumi's comment the PR (we don't want to add this test hub to the list)


fix: correct language for gh stars count

✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Shows the language's equivalent formatting for the GitHub stars count on localized pages -- enhances the work from #1123.
The language detection code is implemented from previous work.

Screenshots
before

after

Is this better? @a4addel @Sboonny

Yes, but with a small note in Arabic we Use ","  as a seperator not "،"

As a programmer, I advise against it, because you will be asked to change the Markdown format from .1 and .2 to .۱ and .٢, and persion and urdu have different number system.
As a translator, I recommend using the word that describe numbers like how the newspaper articles' translators do.
aka 42,069 is 42 thousands, so it will be ٤٢ الف.

Persion, urdu and arabic translators in freeCodeCamp didn't care about this, so I advise that you shouldn't as well

We could use the compact notation instead, this implements the feedback from @Sboonny.
As a bonus, since the star count isn't realtime there is no need to be super specific with the number for the star count, so the compact version works for that case as well.
Examples below:
Arabic

English

French

Norwegian

Polish

Portuguese

Russian

Chinese


Data just contains 0's

Hi,
First time user of HidLibrary.
I followed the simple example. My OnReport method gets called continuously and report.Data is always and empty byte[8].
My device is a barcode scanner which works as a HID keyboard. When I have notepad open the scanner outputs to USB so I'm assuming it's working.
I should be replacing ReadReport(OnReport) with a call to Read after the scanner has scanned? Is it possible it just isn't supported?

I have the same issue. My report data is always empty byte[3] which is being converted to "AAAA" in string.
Did you find any solution for it?

No. I changed my scanner to serial mode.

This seems like a duplicate of #73


Can we implement AsyncWrite for openssh_sftp_client::File?

Hi, File provides most it's API through async fn like:
impl File
    pub async fn write(&mut self, buf: &[u8]) -> Result<usize, Error> {
        ...
    }
}

This make it's hard to implement AsyncWrite upon it: users will need to build their own state machine to make implement poll_write correctly.
Is it possible to add native AsyncWrite and AsyncRead support? (we already have AsyncSeek now).

cc @NobodyXu and @silver-ymz, thanks in advance.

There is TokioCompatFile which provides async operations.

There is TokioCompatFile which provides async operations.

Thanks, but is it possible to bypass the buffer logic? OpenDAL is going to have it's own general buffer in the future.

Oh, I see. You need to capture the buffer before building future.

It's exactly the same as completion based async I/O so it encountered similar problems.

Yeah, I got it.

Thank @silver-ymz for the quick answer. And thank @NobodyXu for the explaination. My issue is fully addressed now, see you next time :laughing:


Add support for repo setting use_squash_pr_title_as_default

Adds support for the repository setting use_squash_pr_title_as_default.
Related API docs:
https://docs.github.com/en/rest/repos/repos#get-a-repository
https://docs.github.com/en/rest/repos/repos#create-an-organization-repository
https://docs.github.com/en/rest/repos/repos#update-a-repository

@gmlewis thanks for reviewing, addressed your comments!


[v1.1][ISSUE-218] DLRM demo update with fixes on infer error

What changes were proposed in this pull request?

update dlrm demo
update SDA dlrm advisor with customize script path
update dlrm patch to fix infer fail issue

Why are the changes needed?
use unified workflow_prepare.sh
How was this patch tested?
existing CI/CD
manual notebook run

@zigzagcai , I found USE_NUMPY=1 is not working on SR602 as well. I tested with my local env, and force to use NUMPY fix this issue.

I find in the latest test results, sr602 still has the previous error.

I find in the latest test results, sr602 still has the previous error.

yes, I found this is very strange, using pip install or conda install, all shows numpy already installed, but torch continually report numpy is not found, which cause this issue.

I find in the latest test results, sr602 still has the previous error.

yes, I found this is very strange, using pip install or conda install, all shows numpy already installed, but torch continually report numpy is not found, which cause this issue.

@xuechendi , I have fixed this error in https://github.com/intel/e2eAIOK/pull/273. The root cause is version conflict between numpy and torch1.5


Feature request: Support Home Assistant notifications

This will make many android users happy I guess. I currently have an Iphone with the Home Assistant app and I receive push notifications trough that app. My GF has a Android phone and I need to install an extra app to support push notifications for her, would be better if this app can support it.

This project is a hobby, so I don't have enough free time to add all your requests. But, as this project is open source, anyone can add their own features.


Not able to clear a text box in protractor

Hi All,
I am using below Script to clear a Text Box in protractor. However the Text Box is not getting cleared. It gets appended with whatever i pass in SendKeys() function. Kindly help me on this.
Script used is as below
browser.element(by.xpath("")).clear();
browser.sleep(2000);
browser.element(by.xpath("")).sendKeys("100");
This Script does not works. Please help

It would be better to create a question on the stack overflow. What exactly doesn't work? What error do you get?

We are having the same issue. Our suite started failing (maybe it's connected to chromedriver 2.43), input.clear() method just stopped working, no error message

We are having the same issue. Our suite started failing (maybe it's connected to chromedriver 2.43), input.clear() method just stopped working, no error message

Check if you have a a onblur javascript execution. I've been having issues as well with clearing an input field which contains a validation function that runs onblur

Experiencing the same issue; clear method is just not working.

I'm also experiencing this issue with ChromeDriver 2.43, and downgrading to 2.42 fix the issue. :/

You should add explicit waiter for checking that this element is clickable before use clear.

@CrispusDH waiting does not solve the onblur problem

@kovacsbalu , you are right. In my case after updating to Chrome 70 I have started to get errors before clear method. But before clear method I have already had explicit waiter on visibility of element. When I change it to clickable condition the errors gone. So, just in case...

Still failing with ChromeDriver 2.44.

I've got the same issue, too. Rolling back to using ChromeDriver 2.42 works. But not working with ChromeDriver 2.43 +

We are having the same issue. Our suite started failing (maybe it's connected to chromedriver 2.43), input.clear() method just stopped working, no error message

Check if you have a a onblur javascript execution. I've been having issues as well with clearing an input field which contains a validation function that runs onblur

@sergioalen Maybe I have the same. How did you fix this?
.....sendKeys(protractor.Key.SHIFT, protractor.Key.END, protractor.Key.BACK_SPACE, <data>?

My issue was that before the update, I was clearing an input with clear() then entering a new value which with the new update clear() now focuses out the input field for some reason, triggering the unblur function. To stop this from happening I created a helper function to delete the values on the input field using backspace before entering the new value without focusing out the input field:
export function backspace(inputEl: ElementFinder, spaces?: number) {
  return inputEl.getAttribute('value').then((text) => {
    let spacesToGoBack = spaces || text.length;
    do {
      inputEl.sendKeys(Key.BACK_SPACE);
      spacesToGoBack--;
    } while (spacesToGoBack > 0);
  });
}

Then in the test I would use this like this:
backspace(inputEl).then(() => inputEl.sendKeys(price));

I'm not sure if that was the actual issue, but it worked for us.

I am facing the same issue, and in one specific case, even with chromedriver 2.42, when running the test in headless mode, the clear method does not work, but when running with headless mode Off, then it works.
It may be worth mentioning that I'm testing a React app, which uses react-final-form for the forms where I face this issue.

clear works only on the value attribute and not innerText , so we cannot use it to clear text field. Is there a work around ?

Try this to clear the field:
await elem.sendKeys( protractor.Key.chord(protractor.Key.CONTROL, "a"))
await elem.sendKeys( protractor.Key.chord(protractor.Key.DEL))

It works to delete text inside input element, if you use the following code:
element(by.id('Your_ID_Here')).sendKeys(protractor.Key.CONTROL,'a');
element(by.id('Your_ID_Here')).sendKeys(protractor.Key.BACK_SPACE;

If none of the above work, you can try this as well!
let ctrlA = protractor.Key.chord(protractor.Key.CONTROL, "a");  await input.sendKeys(ctrlA); await input.sendKeys(protractor.Key.BACK_SPACE);

I was having the same problem with some inputs and I solved doing this method:
clearAndWriteField(fieldElement, text) {     return fieldElement       .getAttribute('value')       .then((val) =>         fieldElement           .sendKeys(protractor.Key.ARROW_RIGHT.repeat(val.length))           .sendKeys(protractor.Key.BACK_SPACE.repeat(val.length))           .sendKeys(text)       );   }


updates PRNGKey to remove deprecation warning

In the readthedocs-built documentation for our upcoming workshop, we were getting a lot of jax deprecation warnings on initial nemos import. That's due to this change. I've updated nemos so we should no longer those deprecation warnings with most recent version of jax.
Specifically:

calls to jax.random.PRNGKey(x) where x is an int have been updated to jax.random.key(x)
type annotations have been updated from jax.random.PRNGKeyArray or KeyArray to jax.Array, which is apparently the correct type annotation.

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

Comparison is base (e693128) 94.75% compared to head (3c4b94c) 94.74%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
- Coverage   94.75%   94.74%   -0.01%
==========================================
  Files          11       11
  Lines         934      933       -1
==========================================
- Hits          885      884       -1
  Misses         49       49

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


endpoint: Skip waiting-to-regenerate -> waiting-for-identity state transitions

Regeneration logic fails if waiting-for-identity changes to ready state in a scenario like this:
builder: ready -> waiting-to-regenerate
..
label change etc: waiting-to-regenerate -> waiting-for-identity
..
labels resolved: waiting-for-identity -> ready
..
builder: (ready) -> regenerating (FAILS as this is not expected)
Resolve this by giving precedence to the waiting-to-regenerate state
over the waiting-for-identity state.  Compensate for possibly blocking
this state change in Cilium endpoint PATCH API.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>Fixed bug causing policy realization being skipped in some scenarios with endpoint identity churn.

test-me-please

test-me-please

test-1.21-4.9 hit by known flake #14959

test-1.21-4.9

test-me-please

test-1.21-4.9 hit by known flake #14959

test-1.21-4.9

test-1.16-netnext hit by known flake #13853

test-1.16-netnext

hello @jrajahalme 👋 could you rebase?

@michi-covalent why is it necessary to rebase? :thinking:

oh can i just push it? i saw "This branch is out-of-date with the base branch" and assumed it needs to be rebased.

ci-aks

The backport labels weren't correctly set after the backport commits were re-added - https://github.com/cilium/cilium/pull/16497#issuecomment-860810182. Updating the labels manually.


Refactoring of the ethutil package

Introduction
We're preparing to integrate the Celo blockchain within our keep-core and keep-ecdsa clients. That integration will require to import a new dependency (celo-org/celo-blockchain) which is actually a fork of ethereum/go-ethereum package. The next step will be adding a new chain implementation in our client code. This will be done by introducing a new package pkg/chain/celo similarily to the existing pkg/chain/ethereum. Unfortunatelly, that approach will require duplicating all ethereum tools placed in keep-common/pkg/chain/ethereum/ethutil package which directly depend on go-ethereum project. In this PR we're trying to address that problem.
Implementation details
The first part was to decouple ethutil tools from the go-ethereum dependency. This has been done by introducing some interfaces like BlockSource or NonceSource declared at the point of usage. Then, specific tools have been switched to use them instead of plain go-ethereum types. As a next step, those tools have been moved to a new ethlike package to indicate they are ready to use with all ETH-like chains.
An alternative approach could be copying go-ethereum interfaces, place them in the ethlike package and change all the go-ethereum references in the ethutil package to use the ethlike types. However, this approach is very problematic as we'll have to copy a lot of code from go-ethereum. We will also have to do a big and hard refactoring in our gen/contract bindings as they directly depend on gen/abi bindings. Hiding gen/abi behind interfaces seems to be unfeasible. Finally, the ethutil tools in most cases doesn't need to know the full Ethereum client interface so small interfaces like BlockSource or NonceSource seem to be a better choice.
To tie everything together, several adapters have been created in the ethereum/ethutil/adapter.go file. They implement appropriate interfaces declared in the ethlike package. Those adapters are intended to use by the client code in keep-core and keep-ecdsa while creating instances of ethlike tools. To handle the Celo chain, similar adapters can be created in the future celo/celoutil/adapter.go package.
It's also worth to note this PR doesn't port all of the tools from the ethutil package to the new ethlike. It just moves the tools which are generic enough to be compatible with ETH-like chains. The rest of the ethutil tools seem to be coupled with go-ethereum much stronger. This is why this PR doesn't touch them. They'll be revisited once the Celo integration will be more advanced and new knowledge will be gained.

I noticed that is this PR we'll have a celo package, meaning that we have ethereum, celo & ethlike. I wonder if ethlike serves as a partial or full interface for both Ethereum & Celo chains (that will be used by keep-* clients)?

I noticed that is this PR we'll have a celo package, meaning that we have ethereum, celo & ethlike. I wonder if ethlike serves as a partial or full interface for both Ethereum & Celo chains (that will be used by keep-* clients)?

@dimpar I'd rather say ethlike package is some kind of "commons".  It is intended to hold common tools to avoid duplicating them between ETH-like (ethereum and celo) chains. Of course, not all tools can be moved here and some level of duplication is unavoidable.

@pdyraga All comments addressed! Ready for another look.


Added functions, added checks for _writeRegister return value, replaced #define by enum, small text changes

Updated INA226 library:

Added waitConversionReady() & waitConversionReady(timeout)
Moved some defines from .h to .cpp
Replaced some #define by enum definitions
Added some enum definitions
Changed some uint types to enum
Added some checking for _writeRegister return value

All added & changed functions have been tested. waitConversionReady() has been tested very thorough.

@HenkHoldijk
I looked into this PR today and I extracted several things I wanted to include in the 0.5.2 version.
Goal was to have most parts included without breaking the interface.
That gives time to look at the breaking parts of the code a bit longer.
That said, your work is appreciated!

Reopen to see the remaining delta / proposals.

Reopen to see the remaining delta / proposals.

Reopen to see the remaining delta / proposals.

Reopen to see the remaining delta / proposals.


api call to "List the authenticated users flattrs" => response is empty array

hello
i have a little trouble receiving a list of users flattres: 'List the authenticated users flattrs'
If i make a call to the api endpoint '/user/flattrs' i get a 200 Ok response with an empty array.
The request header contains the option (Authorization: 'Bearer xxxxxx...') witch worked fine so far for flattering a podcast etc.
i know the documentation quotes:
"If the response header is 200 OK but the response is empty there is two possible reasons. One is that the user have choosen to hide what he/she flattrs. The other is because that user haven't flattred anything yet."
but with my test user, i flatterd a few things and can see them in the: "Flattr Dashboard" and i can't find an option to show/hide what i flattred ...
So is there something i am making wrong ?

Hi,
You aren't doing anything wrong. This documentation is deprecated and the API itself is mostly defunct. Which is why neither of them are publicly available on flattr.com anymore.

Hi,
thx, so how would it be possible to list users flattrs and unflattr users flattred items ?
are there functional endpoints available for that ?
regards
Wanja

Sorry, it's no longer possible though the API. :(

Thanks all right, thanks for the info :/


Need links in different colors on our homepage.


Fill selection field in analyses view correctly on reload

Given the following situation: we open two browser tabs, one for surfaces and one for analyses.
Both pages work on the same selection of topographies.
If you change than the selection in the "Surfaces" tab, save it and reload then the "Analyses" tab,
the analyses are updated correctly, but not the Selection input field, it stays like it is.

Seems to be fixed. Closing.


Prapare images to be usable in ADO build jobs

Description
Changes proposed in this pull request:

Prapare images to be usable in ADO build jobs

Related issue(s)

/hold

/unhold

/test pre-serverless-operator-build-ado


Add a legend to the visualization

Based on feedback from users we've determined that a legend for the visualization would be beneficial.

Finished, just merge the pull request


Hangs in .NET Framework 4.8

I am in the unhappy position of needing to use this in a legacy MVC 5 app. But when I call AuthenticationService.GetOAuthTokenAsync(), the task never completes.
Identical code works fine in .NET 8.
Any ideas?

Can you post a min repro?

There was an issue in my code that could cause a deadlock, and for some reason it didn't happen in mvc core, so I thought it was a compatibility issue.


Alignments error

Hello,
I ran Gviz successfully. But today when I ran Alignments function, I got the following error:
Error in h(simpleError(msg, call)) :
error in evaluating the argument 'x' in selecting a method for function 'sort': different row counts implied by arguments
here is the command:
AlignmentsTrack(thisrangs,cigar=thisrange.cigar,isPaired=FALSE,name=hic_name,type="sashimi", lwd.sashimiMax=3)

Hi @BioEpi,
could you share with me part of the data in order to reproduce the error?
Thanks, Robert

Closing no response.


[Spark dataset runner] Make sure PCollection views get only broadcasted once if reused

The Spark dataset runner currently materializes and broadcasts PCollections used as side-input per consuming ParDo.
If a PCollection is used as side-input for multiple ParDos, it would be evaluated, collected (on the driver) and finally be broadcasted repeatedly for every such ParDo (#24035).
To avoid this the broadcast variable is added to TranslationResult and will be reused if needed.
Additionally, on the executor side, memory was wasted because we kept both the broadcasted binary data as well as a cached representation of the deserialized values in memory.
This implementation uses Kryo to broadcast the binary data. During deserialization the binary shape is dropped and only deserialized values are kept. Caching is done per partition for multimap materialization (using a lru cache). iterable materialization is a no-op as serialization is was already done.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aromanenko-dev
R: @echauchot

@aromanenko-dev would you be available to review this one ?

Run Spark StructuredStreaming ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

Also, there were added a bunch of new classes/methods that are public. Do we really need them be public? if not then I'd suggest to check them and reduce the scope if possible.

Yes, in these cases public is necessary... different packages or implementing methods of an interface

Though, there's a few other places where visibility can be reduced ... I'll follow up with a separate PR @aromanenko-dev

Though, there's a few other places where visibility can be reduced ...

Sounds good, though, please, take into account the potential breaking changes.

Sounds good, though, please, take into account the potential breaking changes.

@aromanenko-dev Let's agree on the fact that this runner is currently not used at all, also there's still so many gaps that the runner isn't ready for production usage.  And last but not least, the runner is very noisy about being experimental, breaking changes are to be expected!
Let's not worry so much about breaking any interface for that runner, but focus on how to get the quality / issues fixed without making that effort unnecessary difficulty so that it can finally be used. Btw, we've decided to rename the runner ... that's going to be one massive breaking change. In my opinion any other (additional) breaking changes until then simply don't matter.

@mosche Agree. Actually, this runner misses @Experimental annotations which is probably still not too late to add.
On the other hand, I'd argue that is not ready at all to be used since 1) it's not broken and 2) it can execute the most of the Batch pipelines despite missing some features and having some perf issues.

Btw, we've decided to rename the runner ... that's going to be one massive breaking change. In my opinion any other (additional) breaking changes until then simply don't matter.

Formally, it can be done by deprecating the packages with old name and wrap all public API to use a package with a new name. Though, as you mentioned, it likely won't be needed because this runner is not much used now.

@aromanenko-dev There's still a couple of things massively broken ... current master is only working in local mode, just noticed this the last few days. That bug originates from some badly broken metrics interfaces. One of the next things to work on... :/

Good point on @Experimental, I'll add that!

There's still a couple of things massively broken ... current master is only working in local mode, just noticed this the last few days.

Ouch... It would be great to have any kind of regressions tests for this to detect it earlier (for RDD runner as well). Though it's a different topic for sure.

Ouch... It would be great to have any kind of regressions tests for this to detect it earlier (for RDD runner as well). Though it's a different topic for sure.

Yes :( Let's discuss this as one of the next things, just relying on the local runner is really bad.


Support customized junit report format

Clear and concise description of the problem
Currently, when using reporters: ["junit"], only testsuites is configurable, could we also make classNameTemplate andtitleTemplate comfigurable, so that I can use custom seperator between classname and title, or remove classname totally.
https://github.com/vitest-dev/vitest/blob/f91da4849ffafd52b0969c21927873a87c47222a/packages/vitest/src/node/reporters/junit.ts#L167-L169
Suggested solution
In jest-junit there are some configurable env values, like JEST_JUNIT_CLASSNAME
and we also have VITEST_JUNIT_SUITE_NAME in vitest, we could add more env variables to help config the junit output
Alternative
No response
Additional context
The configratuion for jest-junit documentation:
https://github.com/jest-community/jest-junit#configuration
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Instead of supporting environment variables we should open an API for providing reporter options. The coverage reporters already support this kind of API:
https://github.com/vitest-dev/vitest/blob/726a57e71c94b8b1a281e2079e3f2ce7d712d65a/docs/config/index.md?plain=1#L786-L800
This is something that 3rd party custom reporters could also utilize.

I created a simple PR to solve my own usecase (to remove the filepath in className), would it be OK to merge that PR first?
https://github.com/vitest-dev/vitest/pull/3808


[Feature Request] Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/tokenize' is not defined by "exports" in /var/www/html/my_proyect/public_html/notus-react-main/node_modules/postcss/package.json

What is your enhancement?
I'm trying to build, but this error appears<EMAIL_ADDRESS>build

react-scripts build && gulp licenses

node:internal/modules/cjs/loader:571
throw e;
^
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/tokenize' is not defined by "exports" in /var/www/html/lnavarroa.cl/public_html/notus-react-main/node_modules/postcss/package.json
at new NodeError (node:internal/errors:399:5)
at exportsNotFound (node:internal/modules/esm/resolve:361:10)
at packageExportsResolve (node:internal/modules/esm/resolve:697:9)
at resolveExports (node:internal/modules/cjs/loader:565:36)
at Module._findPath (node:internal/modules/cjs/loader:634:31)
at Module._resolveFilename (node:internal/modules/cjs/loader:1061:27)
at Module._load (node:internal/modules/cjs/loader:920:27)
at Module.require (node:internal/modules/cjs/loader:1141:19)
at require (node:internal/modules/cjs/helpers:110:18)
at Object. (/var/www/html/lnavarroa.cl/public_html/notus-react-main/node_modules/postcss-safe-parser/lib/safe-parser.js:1:17) {
code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}
Node.js v18.15.0

I got this fixed... it took some digging around, but here is how i did it.
Since I didn't try this with a fresh repo I did the first steps which was
npm run build:tailwind

Then I ran these commands
npm update
npm audit fix --force

Then I delete react-scripts in the node_modules folder and ran
npm install react-scripts

Then I realized that postcss had an update which fixed this issue, so I updated the postcss reference in the package.json.

"postcss": "8.2.12",

Then ran the command
npm update

I'm sure that you might be able to consolidate some of these steps like

adding the postcss version change
then npm install
npm update
npm audit fix --force

Here is a screen shot of me  running it after I did all of that

I got this fixed... it took some digging around, but here is how i did it.
Since I didn't try this with a fresh repo I did the first steps which was npm run build:tailwind

Then I ran these commands
npm update
npm audit fix --force
Then I delete react-scripts in the node_modules folder and ran
npm install react-scripts
Then I realized that postcss had an update which fixed this issue, so I updated the postcss reference in the package.json.
"postcss": "8.2.12",
Then ran the command
npm update

I'm sure that you might be able to consolidate some of these steps like

adding the postcss version change
then npm install
npm update
npm audit fix --force

Here is a screen shot of me running it after I did all of that

Fixed the same issue for me, thanks!

I got this fixed... it took some digging around, but here is how i did it.
Since I didn't try this with a fresh repo I did the first steps which was npm run build:tailwind

Then I ran these commands
npm update
npm audit fix --force
Then I delete react-scripts in the node_modules folder and ran
npm install react-scripts
Then I realized that postcss had an update which fixed this issue, so I updated the postcss reference in the package.json.
"postcss": "8.2.12",
Then ran the command
npm update

I'm sure that you might be able to consolidate some of these steps like

adding the postcss version change
then npm install
npm update
npm audit fix --force

Here is a screen shot of me running it after I did all of that

Thanks a lot. This solution works for me too.


I can't access multiplayer

When I tried using Hypixel with this it gave me an error: java.lang.nullpointerexception
how do i fix this?

Did you launch it with Official Minecraft Launcher?

How do I do that? I'm just doing it through IntelliJ.

please follow this guide @Akul2010

So I have to build whenever I want to test on multiplayer? Is there any other way or is this the only way?

yes this is the only way, srry for late comment

Oh, okay. Also I wanted to ask is there a tutorial on adding mods to it? Like Keystrokes maybe?

for those, you can use this Repo as a reference
https://github.com/Geuxy/pvp-mixin-base
if this repo helpful for you at some point, consider giving a star :)

Thank you, this is what I was looking for! Can you make some kind of mini-tutorial on adding mods?

i will make a tutorial video on these and ill let you know once it completed

Thank you, I appreciate it.


Distribute using F-Droid

It would be great if this app could be published on F-Droid, allowing new and privacy conscious users to use this app. However, the app currently uses proprietary dependencies, such as Firebase, which prevent it from being included in F-Droid. Could these be made optional and removed for an F-Droid release?

Hi
I've thought about publishing the NewQuiz on fdroid, but because of the proprietary libraries I haven't been able to.
NewQuiz uses crashlytics and firebase performance which can be removed easily from the fdroid version.
It also has a user system in which it saves the results of the games and that later the user can see the statistics and the leaderboard (not done yet). I'm saving this data in the cloud because in the future I want this app to be on multiple platforms, so the users can see the data in the other platforms, so that's why it has this firestore database dependency. To solve this I already thought of 2 alternatives:

Move to an open source database, I'm studding Supabase to see if it's a good alternative.
Save the data locally and then if the user wants save in the cloud.

The biggest problem is with firebase remote config, which I depend on to get app configuration values like categories. But I still haven't got good alternatives

It's great to hear that you're open to F-Droid inclusion.
As a user, I would prefer a combination of both alternatives, saving the data locally first, and only when I want to use certain only features sync it to the cloud.
Maybe it's possible to build your own alternative to Firebase remote config (e.g. using Supabase), it's just another way to get the data from the cloud.
Alternatively, if none of this works, you could write your own client-side code to load config values. F-Droid prohibits apps with non-free dependencies, which would no longer be the case (instead it would be flagged with the NonFreeNet anti-feature).

Currently login is optional and user can play the quizzes completely without login.
I'm currently updating the translation system that uses google's mlkit. I will try to make it optional on being in another apk, for the apk size reductions and now also f-droid inclusion.
But if I don't find an alternative for the firebase remote config I must create my own. I already have the app implemented for an easier change.


double free or corruption (fasttop)

🐛 Bug Report
I have an issue running jest in a test where I'm using a native module wrtc.
This error happens in Node v12 and v13.
> jest

 PASS  tests/index.test.js
  ✓ test (9ms)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        0.796s, estimated 1s
Ran all test suites matching /index/i.
double free or corruption (fasttop)
abort (core dumped)  npm test

To Reproduce
Steps to reproduce the behavior:

Install jest and wrtc.
Runs a test with:

test('test', (done) => {
  const wrtc = require('wrtc')
  done()
})

Expected behavior

Just pass the test without end up with the core dumped error signal.
Link to repl or repo (highly encouraged)

https://github.com/tinchoz49/core-dumped-jest
envinfo

  System:
    OS: Linux 5.4 Manjaro Linux
    CPU: (8) x64 Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz
  Binaries:
    Node: 12.13.1 - ~/.nvm/versions/node/v12.13.1/bin/node
    Yarn: 1.21.1 - ~/.nvm/versions/node/v12.13.1/bin/yarn
    npm: 6.12.1 - ~/.nvm/versions/node/v12.13.1/bin/npm
  npmPackages:
    jest: 24.8.0 => 24.8.0

Closed since seems to be an error of the native module node-webrtc.


Admin / non-Admin tests for Unix

This is a follow-up from PR #2280 and Windows counter-part #1486
We should

[ ] implement similar test mechanic and separation for Unix
[ ] document this design in our test docs

@andschwa does sudo: required in .travis.yml means that when you need sudo you should specify it explicitly in the commands?

I'm not sure what you mean... sudo: required requests an image from Travis CI that allows you to use passwordless sudo. Moreover, it is required to request the Trusty (Ubuntu 14.04) image. See here.

Ok, then as far as I see, we already run tests in Travis CI unelevated. Correct?

Correct. I do not see any Pester tests that call sudo, and we don't start the PowerShell process with sudo.

Sweet, then we just need to document windows side of the things.

Hm, I conveniently already covered it in https://github.com/PowerShell/PowerShell/blob/master/docs/testing-guidelines/WritingPesterTests.md#admin-privileges-in-tests


Feat/rpc client get transactions

What
Adds getTransactions to the rust crate - addresses https://github.com/stellar/soroban-rpc/issues/195
Why
getTransactions was added to the server, just adds the corresponding function to the rust code.
Known limitations
N/A

@BlaineHeffron feel free to move the PR to https://github.com/stellar/rs-stellar-rpc-client


Any idea how we can change new message color. It show as bold and i need change also color for clarify.

Is your feature request related to a problem? Please describe
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

You can override css using shadow DOM:
Shadow DOM
CSS classes cannot be simply overridden anymore. You can still use CSS variables, and if needed you can override CSS classes like this:

const style = document.createElement('style')
style.innerHTML = styles
this.$refs.chatWindow.shadowRoot.appendChild(style)

Closing this, please reopen if you still have an issue

Closing this, please reopen if you still have an issue

Hi Antoine, can you show me where to write the code to edit the css? I write it in Created(), then console log "can not read properties of undefined (reading 'shadowRoot')"

Have you added chatWindow ref to the chat window component?
Please refer to the demo code, everything is there: https://github.com/antoine92190/vue-advanced-chat/blob/master/demo/src/ChatContainer.vue#L33

Have you added chatWindow ref to the chat window component? Please refer to the demo code, everything is there: https://github.com/antoine92190/vue-advanced-chat/blob/master/demo/src/ChatContainer.vue#L33

It worked. Thank you very much

Here is my solution
```
:deep(.vac-message-new) {
animation: new-message 1s infinite;
font-size: 14px;
font-weight: 700;
  @keyframes new-message {
    50% {
      color: #f00;
    }
  }
}

It worked for me as well, but since I use webpack and typescript, I was getting Module object on scss import instead of CSS text.
Solved by altering webpack import behavior.
// eslint-disable-next-line import/no-webpack-loader-syntax,import/no-named-default
import { default as ChatCss } from '!css-loader!sass-loader!./chat-shadow-dom.scss';

// have to alter webpack behavior to make it load text, not Module object
// @see https://stackoverflow.com/a/66539355/1074834

const ChatCssString = ChatCss.toString();

export default ChatCssString;

then use ChatCssString to add
 const addChatShadowDomCss = async () => {
    const style = document.createElement('style');
    style.innerHTML = ChatCssString;
    chatWindow.value.shadowRoot.appendChild(style);
  };

better solution https://github.com/antoine92190/vue-advanced-chat/issues/453#issuecomment-1638289893

@reslear as to me it is not better, as I want my styles to be SCSS and loaded from a file

`

`


{« desc »:« default text from ResponseContentFilter »}"

Hi,
First thanks for your development that is helping a lot of people.
I think that something has changed with IOS 16 (but not sure about it as some people indicates the same issue with IOS 15). Anyway, I am facing an issue since one month max.
In the make_request function of FindMyPhone.php file, after the CURL call, I get this in $http_result :
" {« desc »:« default text from ResponseContentFilter »}"
Are you facing the same issue ?
Thanks and have a nice end of week end

Hi. Were you able to reproduce this issue ?

Same error for me

Same error for me, too


Bug/stream date format

Noticed that I'd made an error in the date formatting for SnapshotUTC. Used slashes, but should have used dashes

Thanks for fixing! It's out with v0.5.1. 😎


rancher monitoring ,i want to find the promethus yaml

how can i find the yaml like this :

job_name: 'node'
static_configs:

targets: ['<IP_ADDRESS>:9100']
params:
collect[]:

cpu
meminfo

can i update this file?


Control handling of escaped ampersands in URLs

Simple  example:
import htmlmin
def web_compress_html(content):
     return htmlmin.minify(
             content,
             remove_comments = True,
             remove_empty_space = True,
             remove_all_empty_space = False,
             reduce_empty_attributes = True,
             reduce_boolean_attributes = False,
             remove_optional_attribute_quotes = False,
             keep_pre = True,
             pre_tags = ('pre', 'textarea', 'nomin'),
             pre_attr = 'pre'
             )

in[0]: web_compress_html('<!DOCTYPE html>\n<title></title>')
out[0]: '<!DOCTYPE html>\n<title></title>'
in[1]: web_compress_html('<!DOCTYPE html>\n<title></title><nomin></nomin>')
out[1]: '<!DOCTYPE html>\n<title></title><nomin></nomin>'

Is there a way to control whether (or not) ampersands in URLs like in the above example are escaped? Thanks to w3c's validator, which is throwing tons of errors at me (error: “&” did not start a character reference), I'd like to have a switch for this behaviour. However, no matter what I do, htmlmin will turn & into &, even within (custom) pre-tags.
(Just googled through the standards and a number of related articles. Apparently, there is not consensus on how to use ampersands in URLs - not even in the HTML5 standardization community ... and I really do not want to discuss this nonsense. This is about best practices and reducing errors while debugging a website. Nevertheless, thanks a lot for this excellent tool.)

You can now prefix attributes with pre- in order to avoid changing them.


Add note on procuring hphpize to build instructions

This was in my original build instructions PR and I think it's still helpful for new users that might not have hphpize available.

Sorry - I missed this earlier. Merged now!


@EnableEncryptableProperties are breaking rabbitmq initialization using Spring Data Flow

Spring data flow is not working with jasypt-spring-boot while annotating @EnableEncryptableProperties , it' looking breaking rabbitmq initialization. if i comment the annotation (EnableEncryptableProperties) , RabbitMQ initialized but it will break the encrypted properties (they will no more be decrypted). do we have any work around to support  rabbitmq initialization and encryption/decryption at same time.

But @ulisesbocchio @rockyhopper  any insights why was the bean written using functional programming
@Bean(name = CONFIG_SINGLETON)
    public Singleton configProps(
            final EnvCopy envCopy) {
        return new Singleton<>(() -> JasyptEncryptorConfigurationProperties.bindConfigProps(envCopy.get()));
    }
because from spring boot 3, the spring cloud stream considers  functional programming as a stream event hence takes this one as a streaming event and produces events, can we not use functional programming because it requires additional step of adding function definition, and stream properties
https://docs.spring.io/spring-cloud-stream/reference/spring-cloud-stream/producing-and-consuming-messages.html
https://spring.io/projects/spring-cloud-function
Upgrading to springboot 3 , can’t use any more old  jasypt or anywork without much configuration  changes


Update README.md with changes to the waybar module

The original module uses an interval of 3 (seconds) to update the status of the WireGuard tunnel as up or down (well, the systemd service, anyway).
With my changes to the module, this constant polling is made unnecessary, as it updates everytime the script is run successfully (i.e. the systemd service for wg0 is toggled successfully). However, this is useful only if the interface used to toggle the WireGuard tunnel is the module itself; if someone were to "manually" toggle WireGuard on a terminal emulator, the module will not update itself until clicked on.
My use case doesn't require any other interface to put wg0 up or down other than the module itself, so I chose to contribute here in case anyone else thinks this might be a good idea.

Hi! thank you for your contribution. May I suggest adding a bit of text explaining how setting the interval to a certain number of seconds is useful for people who happen to toggle Wireguard manually?

Alright, I added back the original module and explained the benefits of either approach. Sorry if I'm kind of janky; this is my first pull request ever on GitHub :).

Looking good :+1:

Thank you for your contribution and please do not hesitate if you have further improvements to propose. :)

Thank you for your contribution and please do not hesitate if you have further improvements to propose. :)


What is the best way to customize returned errors?

In legacy API (which is now deprecated), I can define an global errorFormatter(). How can I achieve
the same behavior in the new version?

👍

Please see #393 - Unfortunately error formatters were out of the v4.0.0 release 😊

v4.1.0 published with the new .formatWith(formatter) method in the validation result API.
Please try it out and let me know of any problems.


Fee tx already received

Not sure if this is an error but it looks like Decredition keeps asking vspd about ticket every couple of minutes (sometimes even few seconds). Why not tell to client only once that ticket has been confirmed and stop spamming?

This message is logged when a client is making an invalid request - eg. requesting a fee address or attempting to pay a fee when a fee has already been paid. If the request is being spammed it is a client issue (decrediton or dcrwallet) and not a server issue.


fzf-tmux: A weird scrolling behavior when tmux pane is closed

Hello,
First of all, thanks for the awesome utility, fzf.
I have an issue while using it with tmux, and I believe that perhaps tmux, not fzf, is responsible for the problem; however, as I think this situation might happen to a couple of fzf users and similar issues have been never reported to the best of my knowledge, I am going to open a question or issue.
Please take a look at the following animated screenshot:

When inside a tmux session, fzf-tmux (or other hot-keys) will trigger a tmux pane is spawned, at the bottom of current window.
Once the fzf pane is closed, the previous tmux pane to which fzf's result is passed would have a weird scroll position, which is quite annoying; specifically,

The absolute position of cursor (that user can see) is changed.
The contents (scrollback) before the clear command is (unwantedly) shown. (this is known to take place even when resize happens)

However, it seems that this does not happen in your animated screenshot at README.
I can reproduce a similar behavior even when I do not use fzf, i.e. closing a splitted pane in tmux would always make the top pane messed. This might be an evidence that this is a tmux problem.
I can reproduce this in all of combinations I could have;

bash, zsh
TERM being screen-256color, xterm, xterm-256color, etc.
my tmux is currently 1.9a, but reproducible as well in 1.8, 2.0 (snapshot)
Terminal emulator: iTerm2 and the OS X default Terminal.app

How are you using fzf-tmux? I see no differences on dotfiles or tmux configurations between us.
Does it ever happen to anyone other than me?
Thanks!

Hi, thanks for the report. Actually I experience the same problem. Yeah, it looks like tmux pulls down the content of the above pane when the one below it is closed. (Note that fzf-tmux -l and fzf-tmux -r don't have the issue.) The reason the demo on the README page doesn't have the problem is that it was performed on a new tmux window with empty scrollback buffer. Like you said, I guess it's the way tmux operates, yet I haven't checked if it's a known issue or if there's a workaround to it.

Thanks for the immediate feedback.
I think I need to delve into the issue in company with tmux developer groups, or I will try to find any possible workaround. I will let you know once I come up with any progress.

Thanks.

Also have the same issue, which when opening and closing multiple panes regularly ends up to be kind of a bummer. @wookayin Did you end up finding any workaround? Thanks.

Hey @wookayin, did you ever find a workaround? I just started using fzf and this is a minor annoyance.

Unfortunately, I do not have any workaround as of now :disappointed:
To the best of my knowledge, after looking into the source code of tmux for a moment, I believe that tmux is not responsible for the issue. When closing a pane or resizing happens, tmux just adjusts the size and offset of each remaining pane while not being aware of scroll positions. The similar phenomenon happens even out of tmux; whenever a single terminal changed its size (e.g. by magnifying the window size of iTerm2 or PuTTY) its scrollback appears.
I suspect that to address this issue, one might utilize some control sequences in xterm related to cursor manipulation. However, my clumsy attempt was not successful.

You can work around this with
alias clear='clear; tmux clear-history'
alias reset='reset; tmux clear-history'

Thanks for that workaround @chaoren !

The issue should now be fixed in tmux: https://github.com/tmux/tmux/commit/2627ab322e0e8dffbf86b1c2eb969139a8062174. I think this can be closed.

@chaoren Just tried the latest build (brew reinstall tmux --HEAD) and it works just as expected. Great job!

@chaoren Thanks for letting us know! Then we just need a latest version of tmux :)


Split AbstractMultiProjectJvmApplicationInitIntegrationTest

This class seems to be quite slow because it contains 8 test cases,
and each test case can be as slow as 30s. Now let's further split them.

@bot-gradle test this

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

@bot-gradle test and merge

OK, I've already triggered a build for you.


Add aspect ratio to Bluesky image embeds

I've noticed that Bluesky (at least via the official clients) has recently stopped interpreting image aspect ratio for itself, relying instead on the information provided in the record:

Annoyingly I think the only way to address this will be to download the whole image into memory (or onto disk) and get the dimensions that way (we currently stream the image data). I'm happy to take this one, @snarfed do you have any opinions about this?

Yes! We've been discussing this in https://github.com/snarfed/bridgy-fed/issues/1571 . I haven't prioritized it due to the in-memory image processing, but if you want to add that - maybe with https://pypi.org/project/pillow/ ? - I'm all for it!
Also sadly Bridgy Fed uses different code to generate image blobs, https://github.com/snarfed/arroba/blob/c518924d31587dc9efa367c287c9d7d0d8654eca/arroba/datastore_storage.py#L361-L363 , but on the plus side, it should be straightforward to copy whatever we do here.

per the Bluesky docs images are max 1MB - imo we're fine to put that in memory. I think the only reliable way to ensure a max size of image downloaded is to iterate the chunks manually in the response and append them to some buffer? My understanding is we can't rely on Content-Length headers, and some image formats like JPEG don't reliably put their size information at the start of the file.

Thanks for checking! We do check Content-Length on text and JSON responses, https://github.com/snarfed/webutil/blob/6384196b72a3e4768652dc45ad55f0ff8ef9100e/util.py#L1763-L1777 , but not others. I'm not too worried about a DOS though, it's not an amplification attack, an attacker has to send us as much data as we read, and Bridgy's not a big target anyway. I'd say don't worry about it for now.

...oh actually you could use FileLimiter, it does the chunked limited stream reading you mentioned: https://github.com/snarfed/webutil/blob/6384196b72a3e4768652dc45ad55f0ff8ef9100e/util.py#L1571-L1596

@snarfed did this roll out to bridgy? My recent post didn't have the right aspect ratio unfortunately https://bsky.app/profile/joelotter.com/post/3ld45wjrxw52w

Here it is when run locally, assume it just hasn't rolled out yet :) https://bsky.app/profile/joelotter.com/post/3ld46btlkdp2w

Yes! Sorry about that, deploying now.


hnswlib has no Index attribute??

Declaring index
p = hnswlib.Index(space = 'cosine', dim = EMBEDDING_SIZE) # possible options are l2, cosine or ip
Anyone knows why? Import hnswlib works tho
AttributeError                            Traceback (most recent call last)
 in
1 # Declaring index
----> 2 p = hnswlib.Index(space = 'cosine', dim = EMBEDDING_SIZE) # possible options are l2, cosine or ip
AttributeError: module 'hnswlib' has no attribute 'Index'

Hi @MaxTran96,
What output are you getting when you run python setup.py test in the python_bindings folder?

Solved! It's just a different python environment issue

p = hnswlib.Index(space = 'cosine', dim = EMBEDDING_SIZE) # possible options are l2, cosine or ip

AttributeError                            Traceback (most recent call last)
 in
1 # Declaring index
----> 2 p = hnswlib.Index(space = 'cosine', dim = EMBEDDING_SIZE) # possible options are l2, cosine or ip
AttributeError: module 'hnswlib' has no attribute 'Index'
Hi @MaxTran96, I am getting the same error and haven't had any luck with trying different python environment.
Can you please tell me what solved your issue?


How to use copy-paste augmentation for YOLO5 object detection

Hey @conradry,
I'm curios to know how can i use the copy-paste augmentation in YOLO. Im using albumentations to do some augmentation on my custom dataset and I have added the copy-paste aug as you said in the readme file but im not able to make it work! im getting an error that needs the mask params... which I dont have it! any idea how can I use it in yolov5 detection??
transform = A.Compose([
       A.NoOp(p=0.4),
       A.RandomScale(p=0.2)
       CopyPaste(blend=True, sigma=1, pct_objects_paste=0.5, p=1)
], bbox_params=A.BboxParams(format="pascal_voc"))

Thanks!

Copy-paste requires masks, it's designed for use with instance segmentation not object detection.  I expect that would make object detection performance worse, but if you want to try it, it is possible.
You'll have to create masks from the bounding boxes that you have. Code like this would go into load_example in coco.py.
image = load_image(index) #some image (H, W, 3)
bboxes = load_bboxes(index) #some bounding boxes (N, 4)
bbox_classes = load_bbox_classes(index) #(N,) object classes

masks = []
copy_paste_bboxes = []
for ix, (bbox, bbox_class) in enumerate(zip(bboxes, bbox_classes)):
    mask = np.zeros(image.shape[:2], dtype=np.bool)
    ymin, xmin, ymax, xmax = bbox[:4] #this is pascal format, not coco
    mask[ymin:ymax, xmin:xmax] = True #a rectangle
    masks.append(mask)
    copy_paste_bboxes.append(bbox + (bbox_class, ix)) #assumes bbox is a tuple, not a list

#apply transforms, etc. (see coco.py)

@Auth0rM0rgan @conradry good news 😃! Your original issue may now be fixed ✅ in PR https://github.com/ultralytics/yolov5/pull/3845. This update implements copy-paste augmentation using the new copy_paste hyperparameter (set from 0-1 for 0-100% of labels copied and pasted). Note this is only available for segment labels, not for box labels.
To receive this update:

Git – git pull from within your yolov5/ directory or git clone https://github.com/ultralytics/yolov5 again
PyTorch Hub – Force-reload with model = torch.hub.load('ultralytics/yolov5', 'yolov5s', force_reload=True)
Notebooks – View updated notebooks
Docker – sudo docker pull ultralytics/yolov5:latest to update your image

Thank you for spotting this issue and informing us of the problem. Please let us know if this update resolves the issue for you, and feel free to inform us of any other issues you discover or feature requests that come to mind. Happy trainings with YOLOv5 🚀!

Hello,
I have an add-on to this question.
How do shadows affect the performance when using copy-paste augmentation?
Lets say I have a dataset consisting of 10 differently colored marbles in high definition. For training purposes, the marbles have been laid out on 10 differently colored backgrounds.
In the test set, all marbles can appear on all backgrounds. In the training set, each marble only appears on its own colored background. Unfortunately, the model appears to be learning the color-keyed backgrounds.
If I use copy-paste augmentation, but I do not bring the shadows produced by the marbles, will it work?

@Auth0rM0rgan @conradry good news 😃! Your original issue may now be fixed ✅ in PR ultralytics/yolov5#3845. This YOLOv5 update implements copy-paste augmentation using the new copy_paste hyperparameter (set from 0-1 for 0-100% of labels copied and pasted). Note this is only available for segment labels, not for box labels.
To receive this update:
* **[Git](https://github.com/ultralytics/yolov5)** – `git pull` from within your `yolov5/` directory or `git clone https://github.com/ultralytics/yolov5` again

* **[PyTorch Hub](https://pytorch.org/hub/ultralytics_yolov5/)** – Force-reload with `model = torch.hub.load('ultralytics/yolov5', 'yolov5s', force_reload=True)`

* **[Notebooks](https://github.com/ultralytics/yolov5/blob/master/tutorial.ipynb)** – View updated notebooks  [![Open In Colab](https://camo.githubusercontent.com/84f0493939e0c4de4e6dbe113251b4bfb5353e57134ffd9fcab6b8714514d4d1/68747470733a2f2f636f6c61622e72657365617263682e676f6f676c652e636f6d2f6173736574732f636f6c61622d62616467652e737667)](https://colab.research.google.com/github/ultralytics/yolov5/blob/master/tutorial.ipynb) [![Open In Kaggle](https://camo.githubusercontent.com/a08ca511178e691ace596a95d334f73cf4ce06e83a5c4a5169b8bb68cac27bef/68747470733a2f2f6b6167676c652e636f6d2f7374617469632f696d616765732f6f70656e2d696e2d6b6167676c652e737667)](https://www.kaggle.com/ultralytics/yolov5)

* **[Docker](https://hub.docker.com/r/ultralytics/yolov5)** – `sudo docker pull ultralytics/yolov5:latest` to update your image [![Docker Pulls](https://camo.githubusercontent.com/280faedaf431e4c0c24fdb30ec00a66d627404e5c4c498210d3f014dd58c2c7e/68747470733a2f2f696d672e736869656c64732e696f2f646f636b65722f70756c6c732f756c7472616c79746963732f796f6c6f76353f6c6f676f3d646f636b6572)](https://hub.docker.com/r/ultralytics/yolov5)

Thank you for spotting this issue and informing us of the problem. Please let us know if this update resolves the issue for you, and feel free to inform us of any other issues you discover or feature requests that come to mind. Happy trainings with YOLOv5 🚀!

What changes should be implemented in augmentations.py if i am using coco2017 dataset for training?


Improve Javadoc for ValidatePlugins

It was not clear from the Javadoc where to use the task. Moreover, the task name in implementing_gradle_plugins.adoc was wrong and pointed to the now removed task.

@bot-gradle test and merge

@bot-gradle test and merge

@bot-gradle test and merge


How to Listen to URI changes on Web?

Hey, I am not sure if this is an amateur question but is there any way to stream the URI changes for the web as we can do with Android and iOS. If not is there an alternative way to listen to those as I have looked throughout multiple posts and articles but none seem to cover it?
#93 states the following but I am unable to understand it thus I am asking.
As the URL cannot be changed in Web apps by an external process without restarting the app, attempting to use the stream APIs will result in an UnsupportedError.

I'm having this issue as well, were you able to work around it?

var url = window.location.href;


Smithy to OpenAPI conversion does not honor useJsonNames: false

The Smithy to OpenAPI plugin does not honor useJsonName: false. Instead it continues to generate field names using the mapping defined via @jsonName('new_name')
Example:
Given the following smithy-build.json, we expect any @jsonName mapping to be ignored and instead use the original property name when generating the corresponding OpenAPI specification.
smith-build.json
{
    "version": "1.0",
    "projections": {},
    "plugins": {
         "openapi": {
             "service": "com.some.project#MyService",
	     "protocol": "aws.protocols#restJson1",
	     "unionStrategy": "structure",
	     "useJsonName": false                     // <---- ignore jsonName mappings
        }
    }
}

Now assume that in the Smithy model, we have defined the following structure:
structure SomeStructure {
    @jsonName("new_property")
    property: MyProperty
}

With regard to expected behavior when generating the corresponding OpenAPI model from Smithy:

We expect useJsonName: true to output property with the name new_property.
We expect useJsonName: false to output property with the original name property.

In this case, useJsonName is false, hence the expected OpenApi output is:
    SomeStructure:
      type: object
      properties:
        property:      ## <------ Note that the original property name is retained as useJsonName is false
          $ref: '#/components/schemas/MyProperty'

Unfortunately, it continues to honor the @jsonName mapping and unexpectedly produces
    SomeStructure:
      type: object
      properties:
        new_property:            ## <---------- should be property
          $ref: '#/components/schemas/MyProperty'

In the documentation for useJsonName, it's noted that:

This property MAY be automatically set to true depending on the protocol being converted.

The restJson1 protocol does override this setting here, which is overriding where you have set it to false in your smithy-build.json.


Add new maintainer

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

cc @thewchan for awareness since you agreed to be added here in https://github.com/conda-forge/ghapi-feedstock/pull/8#issuecomment-1258169624.
Thanks!


Add clang 6 docker image

This PR adds a clang docker image for clang:6 using the binaries that Clang provides for Ubuntu 16.04. Clang is a compiler that we support in Drafter. I want to get this PR out as a start, but I will be following up with clang/4 and clang/5 afterwards as we want to test in multiple versions for Drafter.
We will need to create the apiaryio/clang docker repo on DockerHub as mentioned in the README for new images. As I do not have credentials I will need help from SRE to do this task beforehand.
Tasks

[x] Create apiaryio/clang repo on DockerHub

@kylef repo created.


doc: Adding video to "Getting Started" section

Reason/Context
Please try answering a few of those questions
I'm in the process of going through and learning as much as I can about AsyncAPI, through examples, spec, etc.
I just watched this video from @fmvilas https://www.youtube.com/watch?v=g2pqCIhXQ3k&t=2025s about Zero to AsyncAPI which was great, it has some good content in there for anyone that wants to learn.
Maybe we can embed a link to the video on the website somewhere? My suggestion maybe here?
https://www.asyncapi.com/docs/getting-started
I guess long term maybe we could make a shorter (this is 1.5 hours) version of this talk, and give a 10minute walkthrough of what AsyncAPI is. (I don't mind giving this ago once I learn more and am confident enough to do it).
Why
I think it will just help people that are visual/video learners vs tutorial-based learning.

One important thing to highlight is that should we add at some point a video to a page as important as the intro section on the Docs, I don't want us to simply add a video w/ white men. 🙂❤ (Sorry, this includes you too @derberg 😂😀)
In addition, the intro section of a Docs page isn't really where I would add a video.
That said, I think that this issue and some other items floating around in both your world and mine (Dave) are opening a new discussion on how we could design a learning platform. I am actually going to write more about this in another issue and when I do, I will make sure to tag you. 😀👍🏽
I love discussions on how to add interactivity to learn, but I am starting to think that instead of assuming we should add that component to the Docs, I believe it merits us discussing a learning platform. ✨✨🥳

Let me tell you about the Apollo (GraphQL) learning platform. Their Learning Paths are not on the Docs section. Rather, the Docs also direct the user to their separate learning platform, named Odyssey: https://odyssey.apollographql.com/

Architecture
Odyssey is a React application built with Gatsby and MDX and deployed with Netlify. These tools were an easy first choice for our team, because several other Apollo web properties (the documentation, blog, and developer hub) already use them. We know how they work, and we know they work well! .... To power our interactive coding challenges, we use Netlify Functions. The reader’s code is passed to a serverless function that executes tests against the code in a Node.js environment, then returns the results to the browser.

Source:
https://www.apollographql.com/blog/platform/how-we-built-odyssey/

I don't want us to simply add a video w/ only white dudes

This doesn't make much sense. Do you really mean that? Do you mean white dudes are bad at explaining technical topics? 😄
Dave proposed this video not to promote white dudes or postman. Dave proposed this video as it is the best we have so far that we can share with anyone as an alternative to text. Do you have a better video that we can put there with a woman in it?

In addition, the intro section of a Docs page isn't really where I would add a video

Then where would you add it? It is totally fine to mix docs with videos. It makes the learning experience more accessible to others. You mean you do not like that it is breaking the "read" experience, and should it be in some side pannel like "alternative materials" or something like that?
I also do not think that small improvements need to be blocked just because we have some long-term plans (you already know that in Open Source long-term plans take months to implement) to make it better. More important is that Dave came to create an issue as a person that was learning from docs and found that there were some materials that help to learn but are not promoted in the documentation.

This issue is almost 2 months old. So comment is not most accurate:

This video doesn't really align w/ our Docs, and in order to make a contribution to the Docs, we need to first create an issue that is tagged in the Docs Board here.

well, I know you talk about it at conferences, that you organize Docs work this way, but this is not part of the contribution guide, and we should not disqualify an issue just because somebody did not watch your presentation 😄 (reminding again, this is old issue). The way you organize work on docs is pretty unusual here in AsyncAPI so it should not affect contributors too much, it should rather help them contribute, and therefore I think it is your responsibility as a maintainer to triage issues from this repo and add/remove them from the board, or drive discussion.
We should be more liberal as maintainers. The current contribution guide describes ideal (this word is used) workflow. It is not always required to open an issue if the change is super small. It is ok to hold a discussion on the PR level too, sometimes it is even better, for example in the spec repo we encourage PR over the issue. We describe the ideal world in the Contributor guide to defend contributors, so in case they want to make a great PR with some functionality, that required lots of work but was not aligned initially - thus lots of time was wasted, and we suggest an issue first.

tl;dr we need more flexibility IMHO, especially when change is driven by feedback and can't be super picky on every single rule 😄

I'm closing this one, we do not have proper video atm. We will add more videos once we have them ready, work is currently in progress https://github.com/asyncapi/training/pulls


Pause on Android

Hi! Awesome fork but one question: Is it possible to pause the audio?
I can see references to it the Java files.
Thank you!

Congrats for the baby! I end it up just using regular HTML5 audio, perfect for my purpose.
For those we want a pause on this plugin, a simple dummy workaround is modifying the stop function to make it work like a pause, just removing the line that set the seek to 0 in the javascript files.

Thanks, @julianCast for the warm wishes!
Got your workaround about removing the line.


Graph view not displaying

When I try to show graph view, all I get is a white screen. It turns out there is a white error message on that white screen, which makes it difficult to see.

Moved this issue from dendron-site to here

Hi @cmadland is this error still happening?
If it is, could you run the command "Dendron:Dev: Open Logs" and then add the output here (or message to me at SeriousBug on the Dendron Discord)?

dendron.log


NOISSUE - Fix Bug On Passing Token During Listing Policies

What does this do?
Fixes a bug on listing polices. Token should be the second arguments and not the first argument
Which issue(s) does this PR fix/relate to?
No issue
List any changes that modify/break current functionality
No changes modify the current functionality
Have you included tests for your changes?
No
Did you document any new/modified functionality?
No
Notes
Steps to reproduce

Try the following command before and after to see the differencemainflux-cli policies list users $ADMIN_TOKEN

check DCO

Changed merged on https://github.com/mainflux/mainflux/pull/1831


OPAI outputs normally, but Perplexica cannot use Searxng to complete the search function。

I have tested the latest code deployment multiple times. I have two environments, one is a Mac with VPN that can search normally, and the other is deployed on a server with VPN configured, which can be requested both inside and outside the container https://www.google.com.hk/ However, the server-side Perplexica cannot search for content

@WanQuanXie   Expert, I would like to inquire about the issue with Searxing. I know it is caused by my network problem. When requesting, I checked both the backend and Searxing logs and found no error output. However, Perplexica cannot use the search function. Does using Searxing depend on the client's network environment,

Occasionally, this problem arises

It seems like something wrong with remote server VPN environment configuration, but I do not have any server to deploy for debugging, @ItzCrazyKns is there any idea?

@hanjinxin could u show us your server Perplexica deploy configuration? Deployed by Docker or directly, if Docker and what is the docker-compose.yml configuration? thx:)

Might be a VPN issue, I cannot say anything without more context. If you are able to access the UI on the server port and the backend works then it can be a docker networking issue since we access the SearXNG API over the docker network.

I uploaded the VPN information into Docker Compose, and my server and container can access google.com.Hk. However, for some reason, the front-end page can answer questions normally, but there is no search function. Later, I tried to modify the configuration file to put the back-end service and sealXNG in my local Mac environment and then search. The VPN was opened from my Mac   @ItzCrazyKns  @WanQuanXie

I have the same issue.


[BugFix] PartialUpdate rowset rewrite may not work with tablet clone

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #4880
Problem Summary(Required) ：

Tablet clone assumes rowset file is immutable, while during rowset apply for a partial update, rowset file may be changed. It may raise a bug.
Actually, this bug is only related to the PrimaryKey table. If the tablet is a PrimaryKey table, we should dump the snapshot meta-file first and then link rowset files to the snapshot directory
When doing a partial update, if dump the snapshot meta-file first, there are four conditions as below:

rowset status is committed in meta, rowset file is a partial rowset
rowset status is committed in meta, rowset file is a partial rowset when we link files, and rowset applies success after link files.
rowset status is committed in meta, rowset file is full rowset
rowset status is applied in meta, rowset file is full rowset

case1 and case4 are normal cases, we don't need to do an additional process
case2 is almost the same as case1. In normal cases, partial rowset files will be deleted after rowset apply. But we do a hard link of partial rowset files, so the partial rowset files will not be deleted until snapshot dir is deleted. So the src BE will download the partial rowset files.
case3 is a bit tricky. If the rowset status is committed in meta but the rowset file is full rowset. The src be will download the full rowset file and apply it again. But we handle this contingency in partial rowset apply, because if BE crash before updating meta, we also need to apply this rowset again after BE restart.

Tablet clone assumes rowset file is immutable, while during rowset apply for a partial update, rowset file may be changed. It may raise a bug.
Actually, this bug is only related to the PrimaryKey table. If the tablet is a PrimaryKey table, we should dump the snapshot meta-file first and then link rowset files to the snapshot directory
When doing a partial update, if dump the snapshot meta-file first, there are four conditions as below:
rowset status is committed in meta, rowset file is a partial rowset
rowset status is committed in meta, rowset file is a partial rowset when we link files, and rowset applies success after link files.
rowset status is committed in meta, rowset file is full rowset
rowset status is applied in meta, rowset file is full rowset
case1 and case4 are normal cases, we don't need to do an additional process
case2 is almost the same as case1. In normal cases, partial rowset files will be deleted after rowset apply. But we do a hard link of partial rowset files, so the partial rowset files will not be deleted until snapshot dir is deleted. So the src BE will download the partial rowset files.
case3 is a bit tricky. If the rowset status is committed in meta but the rowset file is full rowset. The src be will download the full rowset file and apply it again. But we handle this contingency in partial rowset apply, because if BE crash before updating meta, we also need to apply this rowset again after BE restart.


Template studio code should follow clean linter rules

It would be useful to clearly document and automatically test linter rules on template studio.
A good set of rules for code quality was open-sourced and made public at: https://www.npmjs.com/package/@clausehq/eslint-config

This issue has been moved to https://github.com/accordproject/template-studio-v2/issues/2


cockpit-plugin-reporting does not compile

The cockpit-plugin-reporting plugin cannot be compiled.
The exception is
[INFO] Compiling 4 source files to .\camunda-bpm-examples\cockpit-plugin-reporting\target\classes
[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] .\camunda-bpm-examples\cockpit-plugin-reporting\src\main\java\org\camunda\bpm\demo\cockpit\plugin\reporting\resources\ReportingPluginRootResource.java:[22,9] error: cannot find symbol
[ERROR] .\camunda-bpm-examples\cockpit-plugin-reporting\src\main\java\org\camunda\bpm\demo\cockpit\plugin\reporting\resources\ReportingPluginRootResource.java:[23,27] error: cannot find symbol
[INFO] 2 errors
[INFO] -------------------------------------------------------------

The underlaying problem is that the class DemoDataGeneratorResource is not contained in the example.

ah - already fixed some while ago :-) We have a jenkins build for it in the meantime. Just never looked at the issues here in the Github project - sorry...


generate xml of test summary

While this will "duplicate" the error reporting on Linux and OS X (the same way it is already on the Windows jobs) it helps to debug failing tests faster (e.g. if a test doesn't generate a result file the CTest log often contains more information).

With this change, I get much less output to console. Looks like some combination of stderr and/or stdout is hidden from the console (perhaps redirected to a file?). Is that the intent?

The new arguments should not affect the console output. Can you please describe what you ran and what the output was.

Turns out that I had an older version of this branch that contained a bug that was fixed in a subsequent commit. But there was no way for me to know, even though I was watching this PR, that I needed to pull to get the newer version, because the commit with the fix was squashed.

+1


implement headers with routeRules

It feels like you could implement setting various headers with routeRules - e.g. a default value imposed via:
export default defineNuxtConfig({
  routeRules: {
    '/**': {
      // default headers
    }
  }
})

That way they could also be overridden with other route rules.
Moreover, it would be good to configure cors in the same way, and for the same reason. As a heads-up, likely future cors enhancements are anticipated directly within nitro.

I will implement the route roules the way you recommended but for cors I will leave it for now as it was quite a recent feature :)
Later on I will reconfigure the cors to work similar way

Hi @Baroshem!
I wondering about the possibility to apply the headers rules per route (for example, contentSecurityPolicy).
For current nuxt-security v0.13.0 it seems not possible. But maybe you can advise on how to do that?

Hey @maxdzin
It is possible.
Take a look at this example file https://github.com/Baroshem/nuxt-security/blob/main/playground/nuxt.config.ts
export default defineNuxtConfig({
  modules: ['nuxt-security'],

  // Per route configuration
  routeRules: {
    'secret': {
      headers: {
        'X-XSS-Protection': '1' // Here you need to pass 'Content-Security-Policy': 'your values here'
      },
    },
  },
})

Hi @Baroshem!
OK, thank you! So it seems the headers are supported (I didn't yet check that under the hood).
Then at least its declaration needs to be updated, because there's a TS error:
Object literal may only specify known properties, and 'headers' does not exist in type

The route rules don't include headers:
interface NuxtSecurityRouteRules {
    requestSizeLimiter?: RequestSizeLimiter | false;
    rateLimiter?: RateLimiter | false;
    xssValidator?: XssValidator | false;
    corsHandler?: CorsOptions | false;
    allowedMethodsRestricter: AllowedHTTPMethods | false;
}

@maxdzin
Ah I see. You are correct. Could you please create a new issue with this type bug?
If you have some time and would like to contribute, the PR will be more than welcome :)

@Baroshem Ok, sure. I'll create a PR then.


Github Package Registry's authservice Alpha image should be public

Such that no auth is required to run docker pull docker.pkg.github.com/istio-ecosystem/authservice/authservice:0.1.0-snapshot.1
This is a follow-up from https://github.com/istio-ecosystem/authservice/issues/12

https://github.community/t5/GitHub-Actions/docker-pull-from-public-GitHub-Package-Registry-fail-with-quot/td-p/32782

Docker images are available in GHCR


Performance - reduce urljoin/urldefrag overhead

Based on the work in #182 (I cherry-picked a few commits) for issue #158. Also happens to fix the problem described in #201 at the same time.
Rebased on master, fixed tests and style checks.
Removed base_uri since it was not really being used now that there is a stack.
I hope this is a smaller chunk to review.

Awesome, definitely helps, I'll give a shot at looking this over.
I mentioned in the other ticket I would want to maintain backwards compatibility for APIs even if they're not used internally anymore, I haven't checked carefully but I expect that to be fairly easy to add back for this subset?

I mentioned in the other ticket I would want to maintain backwards compatibility for APIs even if they're not used internally anymore

Cool, I believe the only thing that was removed was RefResolver.in_scope(). It is easy enough to restore that with the stack implementation.

A benchmark, even a simplistic one, would also be lovely for anything performance related

I was using a very simple one myself to profile the changes, so I it should be no problem to include it.
I actually have another branch which uses a cache around some of the slower operations and is showing another significant performance increase ontop of these changes.  I'll see about getting all of these changes together into a single PR.

I've restored in_scope() which I believe is the only breaking API change, and added bench.py.
Using the example schemas in the bench.py docstring,
master: ~56ms
this branch: ~44ms
I'm going to experiment more with the caching branch.

With caching this is down to ~21ms. I'm going to get that branch cleaned up and replace this review I think.

A great thank you @dnephin  for getting up to this!

Definitely, appreciated.
Running your benchmark on a warmed up PyPy basically yields equivalent times on my machine, which means this doesn't really speed up anything there (probably expectedly). I'm generally OK with CPython-only speed improvements (which I assume is where you're seeing improvement) as long as it doesn't slow down things on PyPy or introduce other complexity, so seems like we're on the right track here. I'll play a bit more with this until you've got your caching branch as well, this is small enough for me to have a shot at reviewing right now :)

Fair point, I wouldn't expect to see much improvement in pypy.  I've opened #203 since I rebased over a bunch of these smaller commits.


Make the DC0 or whatever inline with matrix.org

https://github.com/matrix-org/matrix.org/blob/main/CONTRIBUTING.md#sign-off
I honestly don't think the wording is too different but it removes a bunch of jargon around name requirements.

I mean it was a carbon copy anyways, they just updated it lol


cachix tests fail - report too old Nix

Describe the bug
See e.g. https://hydra.nixos.org/build/121014543
To Reproduce
Running the build locally reproduces it, though I can't see any relation with the reported Nix version on my system.
Speculation: some dependency of the cachix build might have "cached" version of the nix daemon running on its builder (and succeeded)?  Maintainers will probably know much quicker than me.
Notify maintainers
@domenkozar

A clarification:
This appears to just be a test at parsing various Nix versions, not that Nix 2.0.1pre6053_444b921 is installed on whatever machine ran the build/test. This was discussed on @peti's livestream last Friday, where he filed this issue: https://github.com/cachix/cachix/issues/308.
(Here's the part of the stream where it occurred: https://www.twitch.tv/videos/635552787?t=00h28m33s.)