'follow' crash

I wanted to follow a series and got a crash:
cum follow http://bato.to/comic/_/comics/rozen-maiden-ii-r2656
Batoto username: JonnyRobbie
Batoto password:
Traceback (most recent call last):
  File "/usr/bin/cum", line 11, in <module>
    load_entry_point('cum==0.8', 'console_scripts', 'cum')()
  File "/usr/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/cum/cum.py", line 15, in wrapper
    return f(*args, **kwargs)
  File "/usr/lib/python3.6/site-packages/cum/cum.py", line 209, in follow
    series = utility.series_by_url(url)
  File "/usr/lib/python3.6/site-packages/cum/utility.py", line 71, in series_by_url
    return Series(url)
  File "/usr/lib/python3.6/site-packages/cum/scrapers/batoto.py", line 18, in __init__
    self.chapters = self.get_chapters()
  File "/usr/lib/python3.6/site-packages/cum/scrapers/batoto.py", line 40, in get_chapters
    name = columns[0].img.next_sibling.strip()
AttributeError: 'NoneType' object has no attribute 'strip'

$ cum --version
cum version 0.8 "Miyamo Chio"

Duplicate of #46

yeah, I just noticed the duplicity...sorry about that

No problem.


Add springdoc to backend and move java files to different folder

Add springdoc to backend and move java files to different folder.

Codecov Report
Base: 73.88% // Head: 73.88% // No change to project coverage :thumbsup:

Coverage data is based on head (36eebf2) compared to base (4e96780).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##               main       #7   +/-   ##
=========================================
  Coverage     73.88%   73.88%
  Complexity     1431     1431
=========================================
  Files           154      154
  Lines          5889     5889
  Branches        670      670
=========================================
  Hits           4351     4351
  Misses         1257     1257
  Partials        281      281

Flag
Coverage Δ

unittests
73.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Check ALL link responds 404

Maybe it's moved to somewhere else. I tried to find where it is now and found this It would be good to confirm if it is same extension originally included in this list.

That does appear to be the same one.
I've updated the list with your new link. Thanks!


[WIP][HUDI-1295] Metadata Index - Bloom filter and Column stats metadata to speed up index lookups

What is the purpose of the pull request

Today, base files have bloom filter at their footers and index lookups
have to load the base file to perform any bloom lookups. Though we have
interval tree based file purging, we still end up in significant amount
of base file read for the bloom filter for the end index lookups for the
keys. This index lookup operation can be made more performant by having
all the bloom filters in a new metadata partition and doing pointed
lookups based on keys.

This PR adds the basic infrastructure for initializing the new bloom filter
metadata partition, stubs for write and read code paths

Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

This PR is taken over by https://github.com/apache/hudi/pull/4352


Add required JSON annotation in H3IndexResolution

This is a bugfix.
H3IndexConfig objects were not able to be deserialized with Jackson, which was incompatible with the new index configuration syntax.
A couple of tests were added to verify that the new code is working and prevent that error in the future

Codecov Report

Merging #10792 (a106b9e) into master (3851f33) will decrease coverage by 36.95%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10792       +/-   ##
=============================================
- Coverage     68.51%   31.57%   -36.95%
+ Complexity     6479      453     -6026
=============================================
  Files          2159     2164        +5
  Lines        116044   116325      +281
  Branches      17569    17592       +23
=============================================
- Hits          79510    36725    -42785
- Misses        30899    76282    +45383
+ Partials       5635     3318     -2317

Flag
Coverage Δ

integration1
24.10% <0.00%> (+0.10%)
:arrow_up:

unittests1
?

unittests2
13.66% <0.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ot/segment/spi/index/reader/H3IndexResolution.java
0.00% <0.00%> (-73.92%)
:arrow_down:

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


apply file.exclude rules to reference search and go to declaration

As discusses here #5740, we should consider to honour the exclude settings for files in the reference search window, in go to declaration, and likely other places

@alexandrudima this could help to avoid confusion with the addition of monaco-editor.d.ts

Would excluding the said patterns be the responsibility of the language server or the core VS Code product?
As in will the extensions do the excluding before returning data to VS Code or will VS Code do the excluding after getting the data from the extensions?

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Python3 Installation Error

Ref. https://forsetisecurity.org/docs/latest/setup/install.html
v2.16.0
Step 3. python3 install/gcp_installer.py
Generates error
Traceback (most recent call last):
File "install/gcp_installer.py", line 23, in
gcp_installer.run()
File "/home/<...>/forseti-security/install/gcp/gcp_installer.py", line 83, in run
from .installer.forseti_server_installer import ForsetiServerInstaller
File "/home/<...>/forseti-security/install/gcp/installer/forseti_server_installer.py", line 23, in
from .forseti_installer import ForsetiInstaller
File "/home/<...>/forseti-security/install/gcp/installer/forseti_installer.py", line 24, in
from future.utils import with_metaclass
ImportError: No module named 'future'
I resolved this by running following in my Cloud Shell VM
pip3 install future
May wish to update the documentation or add it to the requirements.txt ?

Thanks @Red-Five for reporting this error! Submitted #2893 to update the documentation to incorporate this.


Support generic collection type for ClumpSource get multiple

@fwbrasil

How does this interact with Clump.sourceZip? Obviously it doesn't make sense to have sets there.

How does this interact with Clump.sourceZip? Obviously it doesn't make sense to have sets there.

It will work fine with Clump.sourceZip because the zip aspect is something we use to wrangle the underlying repository into a form we can build key-value pairs out of. This generic collection is at a much higher abstraction and just effectively represents a convenience way to a developer being able to choose a collection type they wish to use to lookup multiple keys in the underlying Map[K,Future[V]]. ClumpFetcher will still use a list to call the repository under the hood and then use the zip function to match the result with the input list it used. Hope that made sense

So clumpSource.get(Set(1,2)) would effectively call (say) clumpSource.get(2) then  clumpSource.get(1). Then under the hood ClumpFetcher would call the underlying function fetch(List(2,1)) then the results of that fetch would be zipped back with the input list, a bit like futureResult.map(listResult => listInputs.zip(listResult)

Also the cool thing is it will still honour call order, so if you have:
def fetch(keys: List[Key]): Future[List[Value]]
val source = Clump.sourceZip(fetch _)
val result = source.get(Seq(1,2,3))

then it will honour the call order under the hood, calling fetch(List(1,2,3))

That's pretty cool! Thanks for the explanation.

:+1:


Collection.link crashes Meteor

I'm having an issue:
During some test, I have a FilesCollection collection instance with some missing GridFS files. I wanted to simulate some data loss and realized that, when this is the case, and the file is missing, the entire Meteor app crashes when I try to download the file :
const imageLink = fooCollection.link(image);

return (<img src={ imageLink } />);

I'm not sure if this can actually be resolved itn this module, but I'd like so fail safe to prevent Meteor from crashing it this is the case.

Hello @yanickrochon ,
Please update this issue according to our issue template:
 - Give an expressive description
 - Version of `Meteor-Files` you're experiencing this issue
 - Version of `Meteor` you're experiencing this issue
 - Is it *Client* or *Server* issue?
 - Post *Client* and/or *Server* logs with enabled `debug` option, you can enable "debug" mode in [*Constructor*](https://github.com/VeliovGroup/Meteor-Files/wiki/Constructor)

Closed due to silence at issue owner end.
Feel free to reopen it in case if the issue still persists on your end.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1277 ms

Resolved: Bofrost.lu performance has improved in 2bc15e4 after 9 minutes.


travis-ci: split openssl into 1.0.2, 1.1.0 matrix

Changes proposed in this pull request:

introducing openssl-1.1.0 in travis-ci builds

Your great patch is much appreciated. We are considering to apply your patch into the SoftEther VPN main tree.
SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch
You have two options which are described on the above policy.
Could you please choose either option 1 or 2, and specify it clearly on the reply?

one

@moatazelmasry2 , @davidebeatrici - I'd like to get review from you

ok, that makes sense

There's a problem though.
The 1.1.0f builds have the system's OpenSSL libraries as dependencies, according to ldd:
$ ldd bin/vpnserver/vpnserver
	linux-vdso.so.1 =>  (0x00007ffd1cefe000)
	librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007f1f6f4e5000)
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f1f6f2c7000)
	libssl.so.1.0.0 => /lib/x86_64-linux-gnu/libssl.so.1.0.0 (0x00007f1f6f068000)
	libcrypto.so.1.0.0 => /lib/x86_64-linux-gnu/libcrypto.so.1.0.0 (0x00007f1f6ec8c000)
	libreadline.so.6 => /lib/x86_64-linux-gnu/libreadline.so.6 (0x00007f1f6ea46000)
	libncurses.so.5 => /lib/x86_64-linux-gnu/libncurses.so.5 (0x00007f1f6e823000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007f1f6e60a000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f1f6e241000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f1f6f6ed000)
	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f1f6e03d000)
	libtinfo.so.5 => /lib/x86_64-linux-gnu/libtinfo.so.5 (0x00007f1f6de14000)

@davidebeatrici , thank you for your attention.
I sure I eye witnessed (on my laptop) proper ldd
we definetly need better detection of "is softethervpn linked to the proper openssl lib" and we need better build system (something like autoconf or cmake)

@chipitsine I agree.

Nothing to add here. Looks great and I was hoping for a long time, that someone will finally create a simple matrix for travis-ci. This will definitely make our life easier.
I also totally agree that we need a a functioning build system, one that is capable of cross compiling obviousely build windows binaries among others. I'm considering cmake with wingw or so.
Anyways I'm accepting your merge. Many thanks for your excellent work.

One question though, how did you do multiple commits and they appear here as one commit?

It isn't, but you have to push forcibly: git push --force or git push -f.
It's kind of dangerous because you have to be careful if someone else is working on the same branch, otherwise you risk overwriting his commits if your local branch is not up-to-date.
This can be avoided by using git push --force-with-lease, which checks that your local branch is up-to-date before pushing.
There's a great article explaining in detail: https://evilmartians.com/chronicles/git-push---force-and-how-to-deal-with-it

@davidebeatrici thanks

It is indeed a bit dangerous. I'll just stick to squashing and rebasing commits when accepting a PR :)

@davidebeatrici thanks

@moatazelmasry2 You're welcome.

It is indeed a bit dangerous. I'll just stick to squashing and rebasing commits when accepting a PR :)

I think that's the best thing to do, as the previous pull requests have been merged this way.
Starting to squash and rebase the commits directly in the branch and then merging it via a merge commit would probably cause confusion in the master branch's commits list.

@moatazelmasry2 , do you mean Settings --> Merge button --> ... ? Or do you manually squash those commits during merge ?
well, I also hoped that I'll do simple travis-ci and someone will see it and develop into real matrix. seems, I need to do it myself


option is not binding when select2 ajax is used.

Hi,
I used 4.0.3 version early at that time this function worked properly after updated to 4.0.13 has some issue.
Issue is:

Ajax success its not calling templateResult. When I search for result I get the searching in templateResult but after success its not entering.
After ajax success early option from db is adding but now its not adding any result. When I debug I can see the result is returning.

Adding the code I using:
function changeunderlyingCommon(){
$(".select2").select2({
minimumInputLength: 2,
placeholder: 'Select a option',
ajax: {
url: 'com/PricingToolQuote.cfc?method=searchUnderlyingAssets',
type: 'post',
data: function(key,page){
return {
query:key.term,
type:key._type
}
},
dataType: 'json',
processResults: function (data) {
return {
results: data
};
}
},
templateResult: appendFullName
});
}
//templateResult
function appendFullName (data) {
if (!data.id) {
return data.text;
}
var $data='';
if(data.element.dataset.name !=''){
$data = $('' + data.text + ' | '+ data.element.dataset.name +'');
}else{
$data = $('' + data.text + '');
}
return $data;
};

response from the ajax call is:
[
{
"element":{
"dataset":{
"requestedIssuerID":"1234",
"registrationType":"6",
"actionname":"REQUESTQUOTEREFRESH",
"issuershortcode":"x",
"optlock":3,
"tradeDate":"June, 22 2021 00:00:00",
"requestID":"223334",
"issuerName":"xxxxx",
"issuerid":708963
},
"metadata":{
}
},
"keyName":"REQUEST_QUOTE_REFRESH",
"text":"Request Refresh",
"title":"",
"id":445,
"class":""
}
}

Hard to read, why don't use insert code like this:
{
     "test1" : 1,
     "test2" : 2,
}

Also make a example on some page for code like https://jsfiddle.net/

@PaolaRuby Got issue its something related to response got from backend. In backend "element":  is adding so it conflict with select2 some conditions.
Thanks @PaolaRuby  for responding. Closing this issue.


refactor: cleanup simp types and pre and post method semantics

Before this commit, Simprocs were defined as Expr -> SimpM (Option Step), where Step is inductively defined as follows:
inductive Step where
  | visit : Result → Step
  | done  : Result → Step

Here, Result is a structure containing the resulting expression and a proof demonstrating its equality to the input. Notably, the proof is optional; in its absence, simp assumes reflexivity.
A simproc can:

Fail by returning none, indicating its inapplicability. In this case, the next suitable simproc is attempted, along with other simp extensions.
Succeed and invoke further simplifications using the .visit
constructor. This action returns control to the beginning of the
simplification loop.
Succeed and indicate that the result should not undergo further
simplifications. However, I find the current approach unsatisfactory, as it does not align with the methodology employed in Transform.lean, where we have the type:

inductive TransformStep where
  /-- Return expression without visiting any subexpressions. -/
  | done (e : Expr)
  /--
  Visit expression (which should be different from current expression) instead.
  The new expression `e` is passed to `pre` again.
  -/
  | visit (e : Expr)
  /--
  Continue transformation with the given expression (defaults to current expression).
  For `pre`, this means visiting the children of the expression.
  For `post`, this is equivalent to returning `done`. -/
  | continue (e? : Option Expr := none)

This type makes it clearer what is going on. The new Simp.Step type is similar but use Result instead of Expr because we need a proof.
Other modifications:

The pre and post methods have now type Simproc.
simprocs field at Methods was deleted.
Added andThen combinator for Simproc.
Cleanup simpLoop semantics.

Todo: cleanup reduceStep.

The register_simp_attr <id> command now also creates a simproc set.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/key-protect-all-inclusive/ibm (source)
module
patch
4.8.4 -> 4.8.5

terraform-ibm-modules/secrets-manager/ibm (source)
module
minor
1.1.6 -> 1.2.1

Release Notes

terraform-ibm-modules/terraform-ibm-kms-all-inclusive (terraform-ibm-modules/key-protect-all-inclusive/ibm)
v4.8.5
Compare Source
Bug Fixes

pull key protect module from hashicorp instead of github
gave key_protect_instance_name variable a default value and removed the associated validation logic (#430) (6ab4529)

terraform-ibm-modules/terraform-ibm-secrets-manager (terraform-ibm-modules/secrets-manager/ibm)
v1.2.1
Compare Source
Bug Fixes

Add missing variable required for ibm catalog onboarding (#61) (c8cf2f5)

v1.2.0
Compare Source
Features

Added Secrets Manager DA to IBM catalog (#54) (98c1073)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

This should resolve automatically under renovate at the weekend, after https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/pull/139 is merged.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.10.5 :tada:
The release is available on:

GitHub release
v1.10.5

Your semantic-release bot :package::rocket:


Add a CLI option for always returning exit code 0

🚀 Feature Proposal
Please add a CLI option --alwaysReturnExitCodeZero that can be used to make Jest always return exit code 0, even if some tests fail. This option should especially be honoured in CI mode and when the environment variable CI ist set.
Motivation
In my company we are using Jenkins for CI.  In our F#, C# and C++ projects the test runners in CI mode always return exit code 0, even when some tests fail (at least they can be configured to do so).
The repoting stage further down the build pipeline then uses the test results by mean of Jenkins Junit plugin or Jenkins Corbertura plugin to determine whether the overall build is a failure or a success.
The advantage is that Jenkins can still build reports AFTER the test stage, even if some tests failed.
With Jest, however, the whole Jenkins build pipeline is stopped, if just one single test fails, because Jest then returns exit code 1. Hence, the developer does not get any reports and summaries in the Jenkins job UI, because the reporting stage, that comes after the test stages, is never reached.
In this case, the only source for error investigation is the console output which might not be as nicely presented as the test results from the Jenkins Junit Plugin.
Example
Please provide an example for how this feature would be used:
set CI=true
npm run test --ci --alwaysReturnExitCodeZero --coverage --reporters=default --reporters=jest-junit

We won't add this. Feel free to utilize shell scripts to always return 0 from your test runs, like jest || exit 0

Thanks @thymikee for clarifying and for the shell script suggestion. I was not aware of the EXIT command on Windows. :+1:


feat: allow marking non-SOV DIDs as public

In the continuity of the work with non-indy DIDs: remove the restriction for did:sov when marking a DID as public at the wallet level.
These wallet implementations don't include any indy-related logic. All the logic and restrictions are done in the wallet/set-public-did route.
This change allows using the wallet to mark DIDs from other methods as being public as opposed to having to re-implement all the logic in an ad-hoc plugin.

@chumbert — can you please update the branch so we can merge it?  Thanks!

Can someone with the permissions restart the integration tests ? It looks like the failing test is flaky and not related to these changes.

@swcurran if you go here, do you see the failed tests? If you go here, for example, do you see the "rerun jobs" button at the top?

Thanks @ryjones — I was able to find that button. Cool stuff - didn’t know it was possible.
That will be a big a help.  Of course if we made the tests a little more stable, this wouldn’t happen… :-)


Fixes documentation for swift driver "container" parameter

No longer claims that an additional segments container is used

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fixSwiftDocs" git@github.com:BrianBland/distribution.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGMT


Error: Unable to resolve action GeekyEggo/delete-artifact@2, unable to find version 2

Example:
      - uses: geekyeggo/delete-artifact@2
        with:
          name: "**"

The “v” prefix is missing before the 2; try geekyeggo/delete-artifact@v2.

Ahh snap!  Thank you

I copied it from https://github.com/marketplace/actions/delete-artifact#specify-multiple-names
steps:
    - uses: geekyeggo/delete-artifact@2
      with:
          name: |
              artifact-*
              binary-file
              output

The only one without a “v” 😂

Haha, what an unlucky coincidence, sorry about that! 🙈 Appreciate the PR.


Update for release/89 api changes.

The arrays in the first loop are different from the arrays in the second loop. So the keys should not be mismatched.
The hash should be the same as what you are getting right now. In your current code the first loop does the check:
https://github.com/Ensembl/ensembl-webcode/blob/release/87/modules/EnsEMBL/Web/Object/Transcript.pm#L1104
and the second doesn't:
https://github.com/Ensembl/ensembl-webcode/blob/release/87/modules/EnsEMBL/Web/Object/Transcript.pm#L1130
The code I provided should be building the same hash.

OK, no worries - just wanted to be sure it wasn't an error!


Add enumeration by ref values

Sorry for the mess ... it's my first pull request ...

X being faster than y should be backed up by benchmarks mate.

Is "Update" the corresponding function to compare with?


3324/5701

Sensor Units
R
Single
Optional
Float

Measurement Units Definition e.g. “Cel” for Temperature in Celsius

Type should be String, or?

Seems to wrong in many definitions (e.g. 3316).
May be, a general clean up would be not too bad.

You are correct, sensor units are string.
Thanks for finding the bug. I will fixit asap.

It should be fixed now.
BTW I have been looking for help adding a better validation to travis #22.
It'd make it easier to catch this type of bugs.


[BEAM-7656] Add sdk-worker-parallelism arg to flink job server shadow…

… jar
R: @angoenka
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).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Portable_Python PreCommit

LGTM
Thanks!


feat: support storybook composition

Hi, thanks for the great addon!
I wonder if displaying the badges in a storybook composition setup is possible - probably not because the host just queries the metadata of the components and stories of the other storybook instances right?

Hi @RobbyRabbitman!
For sure, tagBadges addons defined on the composed Storybooks would not load (https://storybook.js.org/docs/sharing/storybook-composition says somewhere that composed addons do not get loaded).
I don't know if addons defined on the Storybook instance that loads external stories run everywhere. I also don't know if the sidebar entries go through the same code, and that's something I'd need to try out.
If you have a public repo set up with composition already configured, feel free to share it. It'll save me some time to repro and test.

Thanks for the reply, I will try it out in the next couple of days but i cannot share it 😄

Thanks! I'm assigning this to you so I know I'm waiting to hear back from you :)

Observation: the badges are shown when the config is also set on the host storybook instance - however I wonder why the output is different. Left=host and Right=Child. Both instances use the framework @storybook/web-components-vite and set exactly the same config, because its in a shared package. I haven't got the time to debug it yet.

Could you please check lock files for both projects to check they use the
same version? Just to eliminate the matter of a recent update I did that
changes display conditions for stories badges.


Fix wheel data section to use normalised package name, and test

Resolves: python-poetry/poetry#7117

[x] Added tests for changed code.
[x] Updated documentation for changed code.

@robtaylor I rebased your branch for merging and missed that you had merged the main branch before, so the author information is now incorrect. Can you please re-commit with your account?

I rebased your branch for merging and missed that you had merged the main branch before, so the author information is now incorrect.

It seems the author information is still correct. I was just confused by github only displaying my avatar. The reason is probably that you committed with an e-mail address that is not connected to your account. If I squash your PR, another address will be used.

Ah perfect, I was surprised it didn’t have the right author info.
I’m happy for it to be squashed if necessary.
On Fri, 23 Dec 2022 at 14:49, Randy Döring @.***> wrote:

I rebased your branch for merging and missed that you had merged the main
branch before, so the author information is now incorrect.
It seems the author information is still correct. I was just confused by
github only displaying my avatar. The reason is probably that you committed
with an e-mail address that is not connected to your account. If I squash
your PR, another address will be used.
—
Reply to this email directly, view it on GitHub
https://github.com/python-poetry/poetry-core/pull/532#issuecomment-1364012348,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB4O4ASOW4ZAFSSJPKWGDTWOW3XPANCNFSM6AAAAAASN267DI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Different font-size on local chromium.

Steps to reproduce
Tell us about your environment:

Puppeteer version:
Platform / OS version:
URLs (if applicable):
Node.js version:

What steps will reproduce the problem?
Please include code that reproduces the issue.

What is the expected result?
What happens instead?

I set my body font-size to 14px. In local chromium it works, but, in Heroku when the pdf is created, its default font-size it's bigger.


Question about the train data and test data

I have noticed that  we train the model with the input of the Surrounding Camera Images and Global Neural Map Prior. How can we get the Neural Map Prior of the train data and test data? I guess that we run all the scenes by the base models such as BevFormer to get it and then use it to train the BevFormer+NMP using the same data. Do I understand correctly?

Sorry for the late reply. The neural map prior is initially empty. Once features are stored into it, these features can serve as a prior. The initial input consists only of camera images, and gradually, each location has its neural map prior.

你好，你的邮件我已收到，会尽快给你回复。


Notification is not received when the android app is killed

This is a very common scenario. Most of the apps like what's app , Skype , LinkedIn and all receiving notification when app is closed .
And notification doesn't rely on app being open or close .
But when it comes to pushy notification doesn't arrive when app is closed . Is there anything we can do to receive notification when app is killed ?

HI @santhosh-mobile-engineer,
Thanks for reaching out. We'd be glad to assist.

What is the device manufacturer name and model you are experiencing this issue on?
Are most of your end users located in mainland China and using Chinese manufacturer devices?

Some Chinese Android manufacturers make use of custom power-saving restrictions that inhibit sending push notifications to your users while your app is in the background, swiped away from recent apps, or after a device restart.
Popular apps such as Skype, Facebook, LinkedIn, Facebook Messenger, WhatsApp, and more are whitelisted by default from these power optimizations by the Chinese OEM manufacturers. These OEMs want users to have a great notification experience on popular apps, while sacrificing user experience on non-popular apps in exchange for longer battery life.
Most notably, Chinese market devices from the following manufacturers include custom power-saving restrictions:

Xiaomi
Huawei
OnePlus
Samsung
Oppo
Vivo
Asus
Sony
Honor

In order to receive notifications in the background on these devices, there are two options:
a) If the affected users are not within mainland China, we recommend implementing FCM high priority fallback delivery. This is proven to work around several of these manufacturers' custom power saving mechanisms (Xiaomi, OnePlus, and more), by having Pushy deliver your notifications through MQTT and Firebase Cloud Messaging's high-priority channel simultaneously, while only waking up your app once.
b) If the affected users are located within mainland China, consider displaying a custom dialog asking those users to whitelist your app from the respective manufacturer's power saving mechanism(s), within the device settings. Unfortunately, the steps for achieving this are different for each manufacturer, but are closely documented on DontKillMyApp. Users using these phones are aware that by default all non-popular apps are killed when swiped away / in background, and have most likely already added a few apps to the battery optimization whitelist.
We apologize for the lack of an easy, straightforward solution -- there is no easy workaround for this, except for having your users actually whitelisting your app. Our SDK includes a few built-in mechanisms to attempt to keep the connection active specifically on Chinese devices, but these mechanisms may sometimes still be throttled by the OS.
Please let us know if you have any questions as per the above and if anything is unclear.

Thanks for reverting back . We will try this and let you know

Hi @santhosh-mobile-engineer,
Sounds good. Please let us know if it ended up working, and if you still need assistance with this issue.


New theme ideas: CottonCandyLight and CottonCandyDark

Both themes with light pink and light blue accents (will be very specific hex color codes ;-) ), but mainly with a white fill for the light one and black for the dark one.
Because this would show up before "Default", I need to manually put that before the rest of the themes in the list.

The themes have been added and they look pretty good. Only thing is there isn't perfect contrast in some places, so they aren't as good as I'd like. Good enough for now, though.

Forgot to add screenshots, as per tradition:
CottonCandyLight:

CottonCandyDark:

Of course, it's available in PortableThemeEngine as well, if you'd like to apply it there. It's CottonCandyLight and CottonCandyDark, loadable via libportablethemeengine.ThemeEngine.SelectTheme() I think.


Crash when new commits were pushed while running CI (before running Danger)

Report
What did you do?
We're running Danger on Jenkins as a post-build action. Builds are started by the GitHub pull request builder plugin when we type phrases like "please test X" into GitHub Enterprise comments. I started a build with a comment and then, while the job was not yet done (and Danger had not yet run), I pushed a new commit.
What did you expect to happen?
I expected Danger to update the commit status on the commit that was built in the Jenkins job.
What happened instead?
Danger crashed:
+ bundle exec danger --new-comment
fatal: --unshallow on a complete repository does not make sense
bundler: failed to load command: danger (/Users/jenkins/.rvm/gems/ruby-2.3.1/bin/danger)
RuntimeError: Commit b6feb139 doesn't exist. Are you running `danger local/pr` against the correct repository? Also this usually happens when you rebase/reset and force-pushed.
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/scm_source/git_repo.rb:58:in `raise_if_we_cannot_find_the_commit'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/scm_source/git_repo.rb:43:in `ensure_commitish_exists!'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/request_sources/github.rb:77:in `setup_danger_branches'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/danger_core/environment_manager.rb:57:in `ensure_danger_branches_are_setup'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/danger_core/dangerfile.rb:258:in `setup_for_running'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/danger_core/dangerfile.rb:269:in `run'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/danger_core/executor.rb:27:in `run'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/lib/danger/commands/runner.rb:66:in `run'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/claide-1.0.1/lib/claide/command.rb:334:in `run'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/gems/danger-4.0.5/bin/danger:5:in `<top (required)>'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/bin/danger:23:in `load'
  /Users/jenkins/.rvm/gems/ruby-2.3.1/bin/danger:23:in `<top (required)>'

b6feb139 was the newly pushed commit that apparently doesn't exist yet in our checkout.
Your Environment

We're running Danger on Jenkins CI
This happened with Danger 4.0.5 but the diff from 4.0.5 to 4.1.0 doesn't look like it would affect this.

I wonder if this is the same as https://github.com/danger/danger/issues/660 - any chance you can try that?

I tried it with #662. Now there is no crash (:tada:), but Danger didn't run on the commit that the CI job was started with but on the newest commit. Jenkins sets the status to green on the first commit and Danger sets it to green on the newer commit. Can I specify which commit Danger should look at?

Yep, you can pass in a base, and head to danger via the CLI ( check with --help for syntax I have the JS version installed globally ATM, so hard to check )

OK, thanks! I still do need #662 though, I tried it again with the current release and got this:
+ bundle exec danger --new-comment --head=0b71e64d207768becee06a6619c90e640bc5debd
fatal: --unshallow on a complete repository does not make sense
bundler: failed to load command: danger (/Users/jenkins/.rvm/gems/ruby-2.3.1/bin/danger)
RuntimeError: Commit 3a894fca doesn't exist. Are you running `danger local/pr` against the correct repository? Also this usually happens when you rebase/reset and force-pushed.
...

3a894fca is the newer commit again. Should we just git pull again before running Danger for now?

That does sound like it, not really sure what a good answer is outside of that?
Depending on how set up you are in the Ruby version, you can use the JS version which cannot exhibit this problem ( it only uses the GH API to get this kind of metadata )

Fixed with #662 👍


Logging update

Maybe we should also change the logging level of the failure to decode a message from RLPxConnectionHandler to debug? Or maybe we could at least hide the exception message, so that the frequent Tx with invalid signature doesn't appear on the console.


[5.3] base64_encode serialized objects before passing them to json_encode

In Queue::createPayload(), it is unsafe to json_encode(['command' => serialize(clone $job)]). Instead of throwing exception when json_encode is failed, we can do json_encode(['command64' => base_64(serialize(clone $job))]).
Related PR: #15284
@taylorotwell @themsaid Do you consider it as a breaking change? I tried to handle old jobs already in the queue by changing the array key from 'command' to 'command64', and handling both keys in CallQueuedHandler.

Can fix #15235

Breaking change surely?

People should be able to base64 their binary data themselves before putting it on the queue.

People usually find out about these sorts of things after many hours of
debugging if that happen at all ;)


TBI - Update/sync border color for input, combobox, dropdown with latest vscode variables

Value
Styles should be up to date with VSCode themes
We noticed that standard buttons has border color --vscode-editorWidget-border while in latest VSCode theme there is variable --vscode-input-border.
We need to appy latest variable + additionally check also border for dropdowns/comboboxes, checkboxes if they aligned
Notes
Tasks

[ ] Input
[ ] Combobox
[ ] Dropdown
[ ] Checkbox

Checkboxes have their own set of variables.

And we add focus-border as well to this mix.

Error states for inputs:

normal: inputValidation-errorBorder
hover: charts-red


BCMC logo too large and wrong file type

A merchant found out a small discrepancy on the magento module. The image of bcmc seems to be too large. Also it is an svg file instead of png. Can we change it to a 100x60 or 100x70 png logo?

Image has been converted to 100x70 png and deployed on test server

When will this be live?

@SAPontwerp we do not production server, it is just a plugin. Master branch is live environment itself

Let me ask the question differently, how can I get the correct image, on the master branch it's still the old size

@SAPontwerp , checking with our dev @agolota; will reply asap

@SAPontwerp please check it now, should be on Master


My life

Issue Type: ug
when i open vISUAL sTUDIO cODE Ⓡ my life seems worthless. please fix this pⓇoblem OwO
;w;
thanks ownie chan ;;;;;w;;;;;
VS Code version: Code 1.28.1 (3368db6750222d219c851f6l90eb619d886e28f5, 1966-09-11T25:00:-1.139Z)
OS version: Arch linux  ia32 6.1.7602

sowwy ;w;

thanks i solved the problem :)


Array: Merge intervals in C++

Given an array of intervals where intervals[i] = [starti, endi], merge all overlapping intervals, and return an array of the non-overlapping intervals that cover all the intervals in the input.
Example 1:

Input: intervals = [[1,3],[2,6],[8,10],[15,18]]
Output: [[1,6],[8,10],[15,18]]
Explanation: Since intervals [1,3] and [2,6] overlap, merge them into [1,6].

assign this to me

assign this issue to me @akgmage


Add BSONBinaryHandler.

Probably an oversight?
Cheers!

Not sure why the build fail, but seems like a Travis issue.

Hi @aloiscochard , would you mind rebasing against master (which includes a fix for CI)?

Rebased in #359, thanks @aloiscochard


dune: 2.3.1 -> 2.4.0

Motivation for this change
https://github.com/ocaml/dune/releases/tag/2.4.0
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @marsam @vbgl

@GrahamcOfBorg build ocaml-ng.ocamlPackages_4_08.phylogenetics

@GrahamcOfBorg build ocamlPackages.eigen

So what’s the plan generally in cases like this? Wait for a next version of either that makes them work together?

This might be reported upstream.
You can also look for a workaround or mark the package as broken.

eigen is also failing on darwin in master, it should be fixed by https://github.com/NixOS/nixpkgs/pull/82299

Apologies for wrongly blaming this PR.


Fixed validation.py function validate()

used to check for first letter being numeric and returned false.
now checks to see first char is alphabet and return false otherwse.
Closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

From Lab but just changed a logic slightly so that validate() takes care of all name string that doesn't start with a-zA-Z


Error retrieving Notebook status: Invalid incoming value for property 'interactive' of class 'osparc.data.model.NodeStatus':

What version of this service are you using?
3.0.2
Long story short
First time seeing this error i think:

Steps to reproduce
Just start the service, but only happened once afaik, runs before worked.
Your environment
osparc-master.speag.com
studyid: c694d3f4-fcad-11ee-9b0a-02420a00f1ae

@pcrespov FYI


Split Settings into subcategories

Split settings into subcategories and create a basic automatic conversion for old configs.
Closes #70

Build failed?

Unsure why.

as said in build logs:
Error: /home/runner/work/AdGoBye/AdGoBye/AdGoBye/Blocklist.cs(254,34): error CS1061: 'Settings.SettingsOptionsV2' does not contain a definition for 'SendUnmatchedObjectsToDevs' and no accessible extension method 'SendUnmatchedObjectsToDevs' accepting a first argument of type 'Settings.SettingsOptionsV2' could be found (are you missing a using directive or an assembly reference?) [/home/runner/work/AdGoBye/AdGoBye/AdGoBye/AdGoBye.csproj]

Okay, I think I'd be good with signing this off in this state.

Once again, thank you very much for the help.


fix(ScreenRotationAngle): Pan gesture broken on "VR" gyro mode with landscape mode on iOS

Issue
#179

Coverage increased (+0.2%) to 91.755% when pulling 546a5f7bcae8db653f8f76da60b0cef0dbe06e40 on ScreenRotationAngle#179 into 1189ffd8428adf87cc26769e7049c688549bd0be on 3.0.0-rc.

LGTM!


Conversion to Swift 5 is available warning

When building a project inclusing this pod with Xcode 13.4.1, I get a Conversion to Swift 5 is available warning.
Since the conversion assistant tells that there is No source changes necessary, maybe you can increase the Swift version in the .podspec file ?

I updated this in v.1.1.7. Let me know if you still see the issue!

Great! Thanks!


Inaccurate transformed document caching

The caching part of the code for transformed documents in the legacy runtime is totally inaccurate. It doesn't prevent the call of applyRequestTransforms but instead of it overwrites its result.
Source: packages/legacy/runtime/src/useSubschema.ts
The code below is copied from the master branch ("version": "0.99.7")
const transformationContext: Record<string, any> = {};
const transformedRequest = applyRequestTransforms(
  originalRequest,
  delegationContext,
  transformationContext,
  subschema.transforms,
);
const cachedTransfomedDocumentNode: DocumentNode = transformedDocumentNodeCache.get(
  originalRequest.document,
);
if (cachedTransfomedDocumentNode) {
  transformedRequest.document = cachedTransfomedDocumentNode;
} else {
  transformedDocumentNodeCache.set(originalRequest.document, transformedRequest.document);
}

As you can see the applyRequestTransforms is called every time without checking the cache for match. After that the transformed document will be overwritten by the cache result.
Also the caching logic doesn't consider transformers that produce different transformed documents based on the originalRequest.variables (example: @graphprotocol/client-auto-pagination)

Would you create a reproduction?

Reproduction repository
See the race1.ts and race2.ts in the repo. Run them separately and see the difference.
They are executing the same query with different variables (A -> B | B -> A).
You could debug the installed   "@graphql-mesh/runtime": "0.99.7" in the node_modules and see the missfunctionality of the cache I described above.
Also fixed it locally and based on that I have created the PR.

See https://github.com/ardatan/graphql-mesh/pull/7039#issuecomment-2329180770


jsii-config produces non working configuration

Describe the bug
I am getting started with jsii. I facing an issue when running npm run build. Below is my jsii configuration as created by jsii-config
  "...",
  "keywords": [],
  "author": {
    "name": "Ganesh Kumar",
    "email"<EMAIL_ADDRESS>  },
  "repository": {
    "url": "https://github.com/ganesh/jsii-getting-started.git"
  },
  "license": "ISC",
  "stability": "stable",
  "types": "index.d.ts",
  "jsii": {
    "versionFormat": "full",
    "targets": {
      "java": {
        "package": "com.ganesh.learn.jsii",
        "maven": {
          "groupId": "com.ganesh.learn",
          "artifactId": "jsii"
        }
      }
    },
    "tsc": {
      "outDir": "dist"
    }
  },
  "devDependencies": {
    "jsii": "^5.5.9",
    "jsii-pacmak": "^1.104.0"
  }
}

The error I am getting is
[2024-11-10T12:04:33.432] [ERROR] jsii/compiler - Type model errors prevented the JSII assembly from being created
error JSII4: Could not find "main" file: C:\Workspace\internal\index.ts
The project is in C:\Workspace\internal\jsii-getting-started. It seems jsii is looking at the parent directory rather than the project directory.
Regression Issue

[ ] Select this option if this issue appears to be a regression.

Expected Behavior
jsii should generate Java files
Current Behavior
I am getting this error
[2024-11-10T12:04:33.432] [ERROR] jsii/compiler - Type model errors prevented the JSII assembly from being created
error JSII4: Could not find "main" file: C:\Workspace\internal\index.ts
Reproduction Steps
Follow the steps listed in the documentation to create a sample project. Run the command npm run build. You will get the error in Windows 11 enterprise
Possible Solution
None
Additional Information/Context
No response
SDK version used
jsii: 5.5.9 & jsii-pacmak: 1.104.0
Environment details (OS name and version, etc.)
Windows 11 Enterprise

I was also tripped up by the missing outdir required on jsii configuration. In addition to adding this to the sample, I would suggest we get it into the documentation as well. There will be some inevitable confusion on the case sensitive difference with tsc config's outDir having an uppercase D.


Figure out where to take Meltano

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2004
Originally created by @DouweM on 2020-04-30 17:40:12

View 2 previous comments from the original issue on GitLab


allow indexing

Sorry for merge without approval, but I already change dns and don't want to mess up our seo


Update: add ecmaVersion 2021 and es2021 environment to demo

This PR adds 2021 to the ECMAVersion list and es2021 to the list of environments in ESLint's online demo.
Created as a draft since it needs ESLint v7.8.0.

Thanks for doing this!


Causing problems with duplicate xml entries.

https://gist.github.com/9f8b04a5116559dc3216706401849be8 heres my log.

Basically it seems this mod is not checking that an item is already minifyable before adding the tag.


Remove ConfigMap

ConfigMap is useful for enduser to customize their own properties which can fetch them via ConfigMapContext.getInstance().
But this feature should not belong to ShardingSphere, user do it by themselves is better if they need.
Right now, ConfigMap is a key input argument from Sharding-JDBC's API via ShardingDataSourceFactory & MasterSlaveDataSourceFactory, and yaml config. It is over-design for most scenario.
After removed ConfigMap from ShardingSphere, user can defined it themselves, ShardingSphere should keep clear and lightweight.

so will official provide a ss-config-center ?

ShardingSphere’s config center is already released from version 2.0.0 more than one year.
Source codes are in:
https://github.com/apache/incubator-shardingsphere/tree/dev/sharding-orchestration/sharding-orchestration-reg

copying config center from ss to duowen.


Add URL to HTTP Basic Auth Pop Up Window

This is a suggestion for a new feature
Hostname in Window Title 2.0 (hostname-in-window-title) is presently not able to add the URL to HTTP Basic Auth pop up. With large amount of HTTP Basic Auth pop ups it's hard and slow to scroll through a long list to find the correct entry.
Steps to reproduce

Go to https://www.httpwatch.com/httpgallery/authentication/#showExample10
Click on "DISPLAY IMAGE" button
The HTTP Basic Auth pop up window will open. Notice that the URL was not added to the HTTP Basic Auth pop up title. This is the challenge.

The suggested new feature expected result is a URL would be added to the HTTP Basic Auth pop up title.
Authentication Required - https://<URL.HERE>
Instead of Authentication Required

keepass-helper add-on is able to do add URL to HTTP Basic Auth pop up. Maybe some of its open source code could be recycled for url-in-title-keepass?
Any volunteer for a patch? I would be happy to contribute testing and or documentation.
Using:

Hostname in Window Title 2.0
Firefox 60.5.0esr at 64 bit
Debian 8 Jessie

According to  M-Gregoire

WebExtension API does not allow to change the title of the Basic Auth window https://github.com/M-Gregoire/Firefox-AddUrlToWindowsTitle/issues/1#issuecomment-395957565

The Basic Auth window/login prompt/dialog is a response built into the user-agent/browser and cannot be changed by the server https://github.com/M-Gregoire/Firefox-AddUrlToWindowsTitle/issues/1#issuecomment-395967794

By the way thanks @Aaron-P for your extension :) I love it. Today I contributed a 5 stars review at https://addons.mozilla.org/en-US/firefox/addon/hostname-in-window-title/reviews/1284035/

The title of the basic auth modal cannot be changed by a webextensions api extension.
I have done some exploring into injecting scripts into the chrome dom using userChrome.css but I don't know much about the internal structure of the chrome dom.  I have spent a few hours trying to find some way to get the context of the basic auth modal window but haven't had any luck so far.  There is very little useful documentation on this, so I'm really not sure how to go about finding it or if it's even possible.

I have found a way to kind of do it, though not as well as I would prefer.  You need to use https://github.com/nuchi/firefox-quantum-userchromejs to inject this script through userChrome.js.
(function () {
    var observers = {
        "common-dialog-loaded": {
            aTopic: "common-dialog-loaded",
            observe: function (aSubject, aTopic, aData) {

                var promptType = aSubject.args.promptType;
                if (["promptUserAndPass", "promptPassword"].indexOf(promptType) === -1)
                    return;

                var openerUrlMatch = aSubject.args.text.match(/^https?:\/\/[^ ]+/);
                if (openerUrlMatch.length === 0)
                    return;

                try
                {
                    var url = new URL(openerUrlMatch[0]);
                    aSubject.document.title += " - " + url.hostname;
                } catch (e) {}

                //TODO: It would be nice to get the url from the opener.gBrowser.currentTab but I'm not sure how to do that on initial load.
                // console.log(aSubject)
                // console.log(aSubject.opener);
                // console.log(aSubject.opener.gBrowser);
            },
            reg: function () {
                Services.obs.addObserver(observers[this.aTopic], this.aTopic, false);
            },
            unreg: function () {
                Services.obs.removeObserver(observers[this.aTopic], this.aTopic);
            }
        }
    };

    observers["common-dialog-loaded"].reg();
}());

This is the product:

Thanks for your replies & efforts Aaron-P :) The present Webextensions API seems challenging to work with. Others mentioned the same challenge. In the future, it might be easier.
Congrats on finding this workaround. I would love to contribute testing this, but I'm not a software engineer. I'm able to test Firefox extensions though. If I understand correctly the suggestion workaround, it would require users to manually customize the userChrome.css file(s).
I agree that this work around is risky to be unstable. Speaking for myself I prefer using stable Firefox extensions. Or when possible use extensions that are not dependent on external script(s) which needs to be manually updates.

This is a suggestion about another resolution. Which is inspired from your comments above. If not already done, how about opening a ticket with Firefox? Suggesting parent & child titles? I mean for Firefox to automatically pass on the "parent" page title to it's "child" HTTP Basic Auth Pop Up Window?
In this suggested mockup below, the number 2 shows the location of the "parent" title. The number 3 shows the location of the "child" title. Which as you know is the HTTP Basic Auth Pop Up Window.

In other words, if Firefox add this suggested new feature about parent & child titles, it might be easier for you from there, to potentially customize both the parent and child titles. Without being dependent on external scripts. Also it would be much easier for users. They would simply install your extension, then optionally, using your extension Templates, customized the child title :)

Closing this unless there are changes to the Firefox api.  I have added the sample code for the userChrome.js workaround to the repo for those interested https://github.com/Aaron-P/HostnameTitleBar/tree/master/BasicAuthDialog.

At some point Mozilla changed the way the password dialogs work to make them in-window modals instead of system modals. These in-window modals don't have a title bar, so this workaround no longer works.
Since the modals are now in-window, keepass can read off the title of the window the modal is in, partially alleviating the original issue, however, it is important to note that it will be the title state of the window that started the navigation, not the destination of the navigation. The potential impact of this is that if you go to "yourbank.com" and click a link to "evilserver.com" which uses basic authentication, then use keepass autofill, the window title that it will see will be "yourbank.com" and you can potentially send credentials to the wrong site.
Thus, it is still important to read the password dialog box to make sure you are authenticating to the domain you want, and if you are navigating from one domain to a different domain which uses basic auth, shouldn't use keepass autofill.

At some point Mozilla changed the way the password dialogs work to make them in-window modals instead of system modals. These in-window modals don't have a title bar, so this workaround no longer works.

Hopefully Mozilla changed this to better match the end-users' needs. Time will tell.

Since the modals are now in-window, keepass can read off the title of the window the modal is in, partially alleviating the original issue, however, it is important to note that it will be the title state of the window that started the navigation, not the destination of the navigation. The potential impact of this is that if you go to "yourbank.com" and click a link to "evilserver.com" which uses basic authentication, then use keepass autofill, the window title that it will see will be "yourbank.com" and you can potentially send credentials to the wrong site.
Thus, it is still important to read the password dialog box to make sure you are authenticating to the domain you want, and if you are navigating from one domain to a different domain which uses basic auth, shouldn't use keepass autofill.

Thanks for flagging this risk. Good catch.

Suggested Collaboration Between HostnameTitleBar & KeePassXC
@Aaron-P and all interested :) For those using KeePassXC, schmittlauch suggested a new feature for KeePassXC. Which would automatically match the HTTP Basic Auth (HBA) in-window modal with its window title dialogue's parent window. Details https://github.com/keepassxreboot/keepassxc/issues/412
In other words, this suggested resolution could potentially allow both HostnameTitleBar & KeePassXC addons to collaborate toward resolving this chalellenge.
Scenario

Using Firefox 78+, user go to this test page at https://www.httpwatch.com/httpgallery/authentication/#showExample10
HostnameTitleBar automatically customize this above page window title. HostnameTitleBar is already able to do this. Including automatically injecting a window title into pages that don't have one. So that all pages can both be unique & easily targeted by KeePassXC.
User click on the "DISPLAY IMAGE" button. Which triggers the opening of a HTTP Basic Auth (HBA) in-window modals.
The challenge is that Firefox changed the way HBA work to make them in-window modals instead of system modals. Notice that this HBA in-window modals that just opened above does not have a title bar.
To resolve this challenge, KeePassXC would check for the parent window title. In this example, https://www.httpwatch.com/httpgallery/authentication/#showExample10
Then KeePassXC would use this as an additional identifier. Which in turn would allow KeePassXC to automatically match each HBA in-window modal with its parent page. In other words, this suggested resolution would use the parent window info by default to distinguish between windows either without name or with the same title.

Can KeePassXC discern an in-window modal from the window itself somehow? I'm not sure how they could do that unless they are reading the process memory or something like that.
I have some potential ideas that may alleviate some of this by catching authentication responses using browser.webRequest.onAuthRequired.  The issues I currently see with that idea are properly keeping track of state when tabs are switched, because you can now switch tabs while the auth dialog is open, and potential issues that may arise if some other extension cancels the request later since onAuthRequired is one of the methods that requests can be canceled in.

Can KeePassXC discern an in-window modal from the window itself somehow? I'm not sure how they could do that unless they are reading the process memory or something like that.

Not presently with Firefox new in-window modals. This is a suggested new feature for KeePassXC at https://github.com/keepassxreboot/keepassxc/issues/412. Thus the suggested collaboration in https://github.com/Aaron-P/HostnameTitleBar/issues/2#issuecomment-743491812 above.

I have some potential ideas that may alleviate some of this by catching authentication responses using browser.webRequest.onAuthRequired. The issues I currently see with that idea are properly keeping track of state when tabs are switched, because you can now switch tabs while the auth dialog is open, and potential issues that may arise if some other extension cancels the request later since onAuthRequired is one of the methods that requests can be canceled in.

Interesting. If your Discovery turns into a potential resolution. I would be happy to contribute testing & documentation if needed.

Suggested Resolution: HostnameTitleBar Open The Page
@Aaron-P :) This is another suggested resolution. How about HostnameTitleBar addon would open the page? Would this allow HostnameTitleBar more permissions? In turn, would it be able to inject a title into the modal?
In other words, how about the parent page would be open by HostnameTitleBar addon instead of be open by Firefox or by the user.
Scenario

Using Firefox, user go to this page about:addons. For a one time configuration of HostnameTitleBar Preferences.  Using a new checkbox & fieldset. The number 1 in this mockup below shows those. By default the checkbox would be unchecked and the fieldset would be empty. So that only interested users could activate it. To view the fieldset tin line you might need to click on this mockup to zoom in. So that you see all that the user would see.

Title
Inject title into in-window modal on this page(s)1
Help text
One URL per line
Foot note
1This optional feature adds title to in-window modal(s). Such as but not limited to HTTP Basic Authentication (HBA) modal(s). Note that when you visit any URL(s) listed below, Hostname in Window Title will automatically reload the page. So that it has the appropriate permissions to be able to insert a title into the modal. Example of a valid page URL is: https://www.exemple.com:8080/path?search=1#hash

Using this new fieldset above, the user enter one URL per line.
The number 1 in this mockup below shows this. With two URLs.

Two URLs:

https://www.httpwatch.com/httpgallery/authentication/#showExample10
http://www.pagetutor.com/keeper/mystash/secretstuff.html

Using Firefox, the user go to http://www.pagetutor.com/keeper/mystash/secretstuff.html

In the back-end, HostnameTitleBar somehow automatically detect this preset URL, intercept it, then reload this URL into the same window. In this example at http://www.pagetutor.com/keeper/mystash/secretstuff.html
Again, this steps is into the back-end. Most users would not notice this. Maybe a short flash when the page reload.

Because HostnameTitleBar opened the page, it somehow now has both additional permissions and an history of the parent's page URL.

HostnameTitleBar inject a title into this in-window modal. Using this script at https://github.com/Aaron-P/HostnameTitleBar/issues/2#issuecomment-462048771 or any other script to your liking.
The number 3 in this mockup show this. Except that in this example, the modal title would read
[#] Authentication Required - http://www.pagetutor.com/keeper/mystash/secretstuff.html [#]

Password managers, such as KeePassXC would somehow be able to target this modal above.

End-users are so happy that he/she jump in joy jump and click heels together, LOL ;) Because this automation saved billions of hours ;)
https://youtu.be/C2f8mfqO43M

Attribution
This suggestion is inspired from maximbaz and other contributors to browserpass-extension at https://github.com/browserpass/browserpass-extension#modal-http-authentication or at https://archive.md/0G46N

That wouldn't work  It's not an issue of window permissions its an issue that it's not an actual system window anymore, it's something that's rendered entirely within the chrome of the browser.  Because of this it doesn't matter if we can access the modal itself, it has no 'title bar' for the system to see, and thus nothing for keepass to find when it looks at window titles.

I'm pretty sure my idea will work, I just need to figure out how to properly keep track of state.

That wouldn't work. It's not an issue of window permissions its an issue that it's not an actual system window anymore, it's something that's rendered entirely within the chrome of the browser. Because of this it doesn't matter if we can access the modal itself, it has no 'title bar' for the system to see, and thus nothing for keepass to find when it looks at window titles.

Interesting. I learned something new. I understand now that the new type of window/modal is entirely within the the browser. Thanks for considering this suggestion @Aaron-P :)

I'm pretty sure my idea will work, I just need to figure out how to properly keep track of state.

I look forward to it. As you know if your Discovery-Adventure turns into a potential resolution. I would be happy to contribute testing & documentation if needed. I'm able to test Firefox on both Debian and Windows.


Display problems with rviz_visual_tools

All of the markers are not always displayed.

Does it happen only in fanuc_grinding or also in bezier_application ?

It seems to never happen the first time we publish

Make sure we are doing things right

Report the bug in rviz_visual_tools if we are

@VictorLamoine I've made series of test using bezier_application and different meshes and everything looks fine. It Seems also working in fanuc_grinding on my side.

I can't reproduce either so I'm closing and we'll re-open if this happens again.
Tested with rviz_visual_tools fe2dd66b9d0248207f89356f354c0780826a2d76


Add references to Windows 20H2 test images.

Hmm.. looks like there's some flakiness in other tests (at least, doesn't look related to this PR);
--- FAIL: TestSnapshotterSuite (0.67s)
    --- FAIL: TestSnapshotterSuite/DevMapperUsage (0.66s)
        snapshotter_test.go:141: assertion failed: expression is false: layer2Usage.Size < sizeBytes+256*dmsetup.SectorSize: 1179648 < 1179648
FAIL
FAIL	github.com/containerd/containerd/snapshots/devmapper	39.953s
ok  	github.com/containerd/containerd/snapshots/devmapper/dmsetup	0.639s
ok  	github.com/containerd/containerd/snapshots/native	7.604s
ok  	github.com/containerd/containerd/snapshots/overlay	15.367s
FAIL
Makefile:177: recipe for target 'root-test' failed
make: *** [root-test] Error 1
Error: Process completed with exit code 2.

/retest

/ok-to-test

/retest

/retest

How do I trigger that test to run again?

/retest

/retest

@jeremyje "Re-run jobs" can be used for restarting GHA https://docs.github.com/en/actions/managing-workflow-runs/re-running-a-workflow
(I already clicked the button)

@AkihiroSuda I think that's only available for users with write access; at least, button doesn't show for me;

🤦♂
Then only available option would be to amend the commit message slightly and push...

Let me try if closing/re-opening still works as a workaround.

yup, that looks to work

LGTM


Could I use the hcaptcha test site you built?

Hi, I used the hcaptcha test site you built in the hcaptcha-challenger project I created recently.
In addition, I noticed that the performance issues were mentioned in the previous discussion of this extension.

https://github.com/maximedrn/hcaptcha-solver-python-selenium/issues/1
https://github.com/maximedrn/hcaptcha-solver-python-selenium/issues/3

Coincidentally, all these problems are solved in hcaptcha-challenger.🤷♀

The recognition task is done independently and not with the help of extensions, so the browser can be started headless.
The recognition module uses a self-trained yolov5n6(onnx) to perform two challenging rounds of the recognition task in one second in extreme cases.This model has excellent penetration of the hcaptcha challenge, is less than 15MB in size, and can run on almost any embedded device.

In the following demo, I am using yolov5s6(onnx) with a larger parameter size to start the solution. if you specify yolov5n6(onnx) to start, the detection speed can be twice as fast...🤦♂

To answer the title Can I use the hcaptcha test site you built, yes you can :)
This repository is no longer maintained and, I know, is very slow compared to image recognition (Yolo v5s6).


network interface (create, update, destroy)

Adds the ability to:

create (with options)
update (machine attachment, device index attachment, security groups)
destroy

Not included:

Elastic IP association

@jkeiser @tyler-ball ready for review ( after chefconf, of course :wink: )

Some minor comments, it needs a rebase, and it needs some tests written.  Otherwise, LGTM!  @patrick-wright what is the testing branch you are waiting on?

@tyler-ball here's the test branch I was referring to: https://github.com/chef/chef-provisioning-aws/pull/152

If this included eip association it would be beautiful. We would be able to build and associate NAT instances w/ route tables end to end.


safely convert arithmetic types

@lisitsyn nyeh... not really... safe_convert of input type bool is crying :) will look into it.


feat(pci.failover-ip): add ipfo order

MSB-90

CDS Report public-cloud#2549.0 ■

Stage 1

Build CA ✔
Build EU ✔
Build US ■

CDS Report freefax#2549.0 ■

Stage 1

Build CA ■
Build EU ✔
Build US ✔

@marie-j Is there any internal ticket reference related to this pull request?
Thanks!

@marie-j Is there any internal ticket reference related to this pull request?
Thanks!

@antleblanc Description has been updated


Make API/Fetch Request

I can't seem to make any form of API calls with this extension
Keep getting this error
Error: Network Error
at createError (eval at  (localhost:3000/js/todoapp.bundle.js:3567), :16:15)
at XMLHttpRequest.handleError (eval at  (localhost:3000/js/todoapp.bundle.js:3555), :88:14)
Any help would be really really appreciated

Based on the evals in the error message, it might have been due to Chrome disabling eval by default.


Jest 12.0.1

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Please rename Lib directory to lib in the NuGet package

Description
I had the following problem when installing Foq on Mono: it has successfully been downloaded by NuGet:
Added file 'Foq.dll' to folder '/home/travis/build/ForNeVeR/1969/packages/Foq.1.7.2/Lib/net40'.
Added file 'Foq.XML' to folder '/home/travis/build/ForNeVeR/1969/packages/Foq.1.7.2/Lib/net40'.
Added file 'Foq.dll' to folder '/home/travis/build/ForNeVeR/1969/packages/Foq.1.7.2/Lib/net45'.
Added file 'Foq.XML' to folder '/home/travis/build/ForNeVeR/1969/packages/Foq.1.7.2/Lib/net45'.
Added file 'install.ps1' to folder '/home/travis/build/ForNeVeR/1969/packages/Foq.1.7.2/tools'.

But the compiler still complains it can't found Foq.dll:
			Target ResolveAssemblyReferences:
/usr/lib/mono/xbuild/14.0/bin/Microsoft.Common.targets:  warning : Reference 'Foq' not resolved
				For searchpath {CandidateAssemblyFiles}
				Warning: {CandidateAssemblyFiles} not supported currently
				For searchpath {HintPathFromItem}
				Considered ../packages/Foq.1.7.2/lib/net45/Foq.dll, but it does not exist.

Repro steps

Create new F# console application.
Install Foq from NuGet.
Try to build the project on Mono.

Expected behavior
Project builds without problems.
Actual behavior
Project fails with the message above.
Known workarounds
I've solved the problem with this commit, simply changing lib to Lib in my fsproj.
Related information

Operating system: Windows + Visual Studio when installing the package; Linux + Mono when trying to build the program.
Branch: Foq 1.7.2 from NuGet
.NET Runtime, CoreCLR or Mono Version: .NET 4.6.2, Mono <IP_ADDRESS>

@ForNeVeR thanks for reporting the issue, I'll fix it for the next release

Foq 1.8.0 has lower-case Lib folders :+1: This can be closed

Confirmed. Thanks!


Add link to the GitHub repo

I would like to have a link back to the GitHub repo from the docs.
Jazzy automatically shows a link in the top-right corner of the docs site if you specify the repo URL in their config: https://github.com/sindresorhus/KeyboardShortcuts/blob/498fcd6cc1562313b5e3aa97b35430cfb2c91292/.jazzy.yml#L2

Live example: https://sindresorhus.com/KeyboardShortcuts/

This is definitely something I'd like to support. Jazzy uses a github_url setting, but I'd like to keep the amount of configuration to a minimum (however much of an uphill / losing battle that may be).
My current plan is to read the remotes from git to determine where the code is hosted publicly, and implement logic that route source location to URLs for the most popular websites (GitHub, GitLab, BitBucket, etc.) The advantage here is that we can use other information from git like the commit reference to provide permalinks, which won't break if methods move around between versions.

I think there was a discussion here somewhere about configs to allow swift-doc to become more configurable, but I don't remember PackageConfig mentioned anywhere. So here's the link for posterity 🙂


[Hexagon] Include Utils.cmake for tvm_file_glob used in HexagonSDK.cmake

Don't include it from HexagonSDK.cmake because the Hexagon SDK file can be included from different directories.

Close/reopen to restart CI.

cc: @tkonolige


 If I ask the Decadac to output 1000 V on one channel, it is highly unlikely that I actually meant "go to the end of your range ASAP", and rather, I should be informed that I am trying to do something wrong.
This PR protects the Decadac against "out of range attacks". I hope that I found the right place in the driver to implement the safety precaution.
Changes proposed in this pull request:

Make driver raise a ValueError if asked to set an out-of-range voltage.

@jenshnielsen @spauka

👍  is there still any chance for a +- 1 over/underflow that we need to catch?

Hmm, good question. I guess it is always better to be on the safe side.

I would like to merge this but can we add the checks back to ensure that off by one does not give an overflow?

@jenshnielsen Yes, I'll do that today.

There we go. This issue is really a good case for having a simulated instrument so that we may have a test for such safety measures.

If I understand #904 correctly, the code we just merged is wrong!

No more than before I would say?

No, not more than before, but still dangerous.


Unable to collect Gunicorn metrics using DogStatsD in K8s

Note: If you have a feature request, you should contact support so the request can be properly tracked.
Additional environment details (Operating System, Cloud provider, etc):
EKS
Describe the results you received:
I setup a K8s cluster and one of the apps is a Gunicorn app and I'm trying to collect stats via DogStatsD. For DogStatsD, I set nonLocalTraffic to true in the datadog-values.yaml for the Helm chart. I then set the hostIP as an environment variable in the flask deployment:
env:
   - name: DD_AGENT_HOST
      valueFrom:
      fieldRef:
      fieldPath: status.hostIP

When I deploy, Gunicorn has statsd host set to statsd_host: ('<IP_ADDRESS>', 8125) by referencing the env variable I defined above. And in the datadog pod logs it's listening for statsd requests:
2019-08-16 19:02:28 UTC | CORE | INFO | (pkg/dogstatsd/listeners/udp.go:79 in Listen) | dogstatsd-udp: starting to listen on [::]:8125
Describe the results you expected:
I expected the statsd requests to come through and log to datadog.
Additional information you deem important (e.g. issue happens only occasionally):
I'm not sure how to debug this, so any help would be greatly appreciated.

@mlaythe, we'll need some additional information to help troubleshoot. Can you open a support ticket so we can proceed? Thanks.
https://www.datadoghq.com/support/


Recreate links when setting new item in layout

Fix #192
I'll add test later

Codecov Report

Merging #193 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #193      +/-   ##
==========================================
+ Coverage   90.06%   90.08%   +0.02%
==========================================
  Files          30       30
  Lines        4872     4873       +1
==========================================
+ Hits         4388     4390       +2
+ Misses        484      483       -1

Impacted Files
Coverage Δ

panel/layout.py
96.05% <100%> (+0.01%)
:arrow_up:

panel/holoviews.py
83.06% <0%> (+0.54%)
:arrow_up:

Continue to review full report at Codecov.

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


[Snyk] Security upgrade firebase from 8.10.0 to 10.2.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

791/1000  Why? Currently trending on Twitter, Proof of Concept exploit, Has a fix available, CVSS 8.6
Prototype Pollution SNYK-JS-PROTOBUFJS-5756498
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

🐈


Docker: add Dockerfile and docker-compose

Types of Changes

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

Description

This pull-request adds Docker support for the application which allows for easier deployments.
You can now docker build -t geoguess . to build an image that you can easily use.
Everything is configurable with environment variables, just like on the baremetal install.
How Has This Been Tested?

Tested by deploying it both locally and then on a production server using the provided example of docker-compose.
Signed off by: Jae Lo Presti<EMAIL_ADDRESS>

The final Docker image is around 40MB which is an ideal size to deploy easily.

@all-contributors please add @jae1911 for code


Version
3.0.0-beta.14
What is expected?
.foo[data-v-asdfgasg] {}
What is actually happening?
.foo {}

Can you provider an repo?

https://github.com/allowing/tem-vue-next

tracking in vue-loader repo already


fix: repositoryUrl listed as required but build fails when not included

Changes proposed

Removing repositoryUrl from the theme options causes the build to fail, this option is listed as not being required so the theme should handle this.
This fix should allow repositoryUrl to be omitted from the theme options if desired.

Additional context

Thanks for the PR @colmcgiv. I'll be taking a look soon at it 💜

Great news! Thanks @jpedroschmitz 🎉

I really appreciated your PR!
2.3.1 was released on NPM with the fix!

Excellent! Thanks for the great theme


WIP: Tagged Unions

Implemented tagged unions + pattern matching over algebraic types. Cleaning up the code before merging.

TODO: remove the hacks to simplify the matched type in pattern.rs


Consolidate FileIO

Feature Request / Improvement
Can we consolidate and standardize FileIO to the PyArrow implementation?
There are currently two different FileIO implementations, ARROW_FILE_IO and FSSPEC_FILE_IO. ARROW_FILE_IO uses Apache Arrow's Filesystem Interface while FSSPEC_FILE_IO uses the fsspec library.
Here are a few reasons for consolidating:

PyArrow is already preferred over FsSpec for various FS implementations.
https://github.com/apache/iceberg-python/blob/cd7fb502900a717d6b902a398b267eb10e4faa9b/pyiceberg/io/__init__.py#L273-L282

PyIceberg is becoming more coupled with PyArrow, to_arrow() and pa.Table are widely used for reading and writing, including the new feature #305

Easier to keep the 2 FileIO's behavior in sync. For example, FsSpec defaults the path with no scheme (/tmp/warehouse) to the file scheme, but PyArrow does not. See #301

The two FileIO implementations are not that different from one another. FsSpec can use its underlying FS implementations, including LocalFileSystem, S3FileSystem, GCSFileSystem, and AzureBlobFileSystem.
While PyArrow uses its FS implementations including LocalFileSystem, S3FileSystem, HadoopFileSystem, and GcsFileSystem.
PyArrow is currently missing the HadoopFileSystem implementation but it has support for HDFS.

Fsspec and PyArrow can be used directionally
PyArrow can use fsspec-based filesystem.
FsSpec can wrap PyArrow filesystem.

What would be your proposal? The FileIO is an abstraction layer to use different implementations for your needs. For example, fsspec is lightweight compared to Arrow and might be preferred if you are inside of a lambda/cloud function or in an orchestration engine like Apache Airflow. As you mentioned, Arrow is more equipped to read tables. Next to that, PyIceberg is designed to be used as a library as part of a query engine. If that query engine prefers a different implementation to fetch the data from an object store, the FileIO abstraction layer allows for that.


Use oc instead of skopeo

Promotion jobs were recently failed due to different reasons:
dial tcp [2606:4700:4400::ac40:963a]:443: connect: permission denied
Error: unable to pull quay.io/containers/skopeo:v1.11: unable to pull image: Error parsing image configuration

when trying to pull skopeo image; and
ChildProcessError: Error running ['podman', 'run', '--rm', '--privileged', '[quay.io/containers/skopeo:v1.11](http://quay.io/containers/skopeo:v1.11)', '--override-os=linux', 'inspect', '--no-tags', '--format={{.Digest}}', '--', 'docker://quay.io/openshift-release-dev/ocp-release:4.11.35-multi']: exit_code=125, stdout=, stderr=Error: could not get runtime: mkdir /run/user/1000: permission denied

even after manually uploading the desired skopeo image into buildvm, and loading it with podman.
oc provides the same exact capabilities even with multiarch payloads: --show-multiarch and --filter-by-os options can be used for this purpose.
A first test run with only get_image_digest() updated worked until it broke on get_image_info() invocation.
The data returned by skopeo looks like this:
{
  "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
  "manifests": [
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 1583,
      "digest": "sha256:5fe0dc83567ec7ce6f89c5b7961d312e4a21cccf3ef5aa8b86b0a43c0f96d33b",
      "platform": {
        "architecture": "amd64",
        "os": "linux"
      },
    },
    [...]
 }

while oc data looks like this:
[
  {
    "name": "quay.io/openshift-release-dev/ocp-release-nightly@sha256:d7ae6e3495d104b489c977805c4188a3d2552b905dd4e4a08ac401ed4af2d0b5",
    "digest": "sha256:a054de5a6770c4d8d1276bcb22da7035001c6e18bf9d4e9f8332e33601a3a253",
   [...],
   "config": { ...}
  }
]

Instead of changing all usages of the returned info, it seemed much easier and less error prone to convert the new format to the one we already expect from skopeo. This way, the rest of the code can stay unchanged.

I ran the 4.10 promote in hack space https://saml.buildvm.hosts.prod.psi.bos.redhat.com:8888/job/hack/job/dpaolell/job/promote-assembly/4/console - but I don't see the oc invocations run so I guess it didn't get tested

I ran the 4.10 promote in hack space https://saml.buildvm.hosts.prod.psi.bos.redhat.com:8888/job/hack/job/dpaolell/job/promote-assembly/4/console - but I don't see the oc invocations run so I guess it didn't get tested

@thegreyd you can see the --show-multiarch usage in 4.11.35 promotion logs

@thegreyd also get_image_digest() got tested:
2023-04-06 06:41:20,265 pyartcd.exectools:INFO Executing:cmd_gather_async ['oc', 'image', 'info', 'quay.io/openshift-release-dev/ocp-release:4.11.35-multi', '--filter-by-os', 'linux/amd64', '-o', 'json'


add a timeout attribute for Api class

adds a timeout attribute to the Api class to be passed to the requests library
This prevents code from hanging excessively and is considered good practice by the maintainers of the requests lib
it was implemented as a kwarg of init with default value as 0, so behavior shouldn't change for most users
was unsure on how to add this to the README so I didn't

thank you very much @Abello966 for the contribution!

I just added a new issue to track the documentation on the new attribute, feel free to send a contribution if you have the time

I just added a new issue to track the documentation on the new attribute, feel free to send a contribution if you have the time

I'd gladly document it, where should I do it? Anyplace else besides the README?

I just added a new issue to track the documentation on the new attribute, feel free to send a contribution if you have the time

I'd gladly document it, where should I do it? Anyplace else besides the README?

Sure, README is fine


ble_gap_adv_start Returns with Error BLE_ERR_INV_HCI_CMD_PARMS on ESP32S3 (IDFGH-8914)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I have an application that utilizes a custom BLE GATTS interface for provisioning a device.
This exact application works on an ESP32 but when porting the code to an ESP32S3 the same application always reports this error.
What I have done to troubleshoot:

Compare bleperph example code to my current implementation to notice any blaring differences
Upgrade IDF version from 4.4.2 to 5.0
Validated SDKConfigs are the same between the ESP32 app and the ESP32S3 app
Ensured I am not sending extended advertising data

I can attempt to show some of my code but will require approval first.
If anyone else has experienced this please let me know of any workarounds.

The error code BLE_ERR_INV_HCI_CMD_PARMS  indicates the the parameter invalid, so you can modify it based on the ESP-IDF BLE_50 demo, or you can provide your current code for review.

`void BLEGAPAdvertiseConfig(uint8_t *advData, uint32_t advDataLen, uint8_t *scanRespData, uint32_t scanRespDataLen)
{
esp_err_t ret;
ret = ble_gap_adv_set_data(advData, advDataLen);
if (ret != ESP_OK)
{
    LogInterfaceLogInfo(BLE_GAP_TAG, "Cannot set advertising data err: %d", ret);
}

ret = ble_gap_adv_rsp_set_data(scanRespData, scanRespDataLen);
if (ret != ESP_OK)
{
    LogInterfaceLogInfo(BLE_GAP_TAG, "Cannot set scan response data err: %d", ret);
}

}
void BLEGAPAdvertiseStart(bleGAPAdvertiseParameters_t advParameters)
{
esp_err_t ret;
struct ble_gap_adv_params advStartCnf;
if (!BLEReady())
{
    BLE_NOT_READY;
}

advStartCnf.channel_map = advParameters.channelSel;
advStartCnf.conn_mode = advParameters.connectionMode;
advStartCnf.disc_mode = advParameters.discoverableMode;
advStartCnf.filter_policy = advParameters.filterPolicy;
advStartCnf.high_duty_cycle = advParameters.highDutyOn;
advStartCnf.itvl_max = advParameters.intervalMax;
advStartCnf.itvl_min = advParameters.intervalMin;
ret = ble_gap_adv_start(advParameters.bdaAddressType, (const ble_addr_t *)advParameters.bdaDirect, advParameters.duration,
                        &advStartCnf, gapEventHandler, NULL);
if (ret != ESP_OK)
{
    LogInterfaceLogInfo(BLE_GAP_TAG, "Cannot start advertising err: %d", ret);
}

if (bleGAPCb != NULL)
{
    bleGAPCb(BLE_GAP_EVENT_ADV_START, (bleGAPEventInfo_t)
    {
        0
    });
}

}
`
Here is a snippet of our code.
BLEGAPAdvertise is called with advData of advDataLen at 31 bytes and scanRespData of scanRespDataLen of 31 bytes.
Both  ble_gap_adv_set_data and ble_gap_adv_rsp_set_data return 0 meaning the parameters are ok.
When calling BLEGAPAdvertiseStart the following is configured:

ble_gap_adv_params

.conn_mode = BLE_GAP_CONN_MODE_UND
.disc_mode = BLE_GAP_DISC_MODE_GEN
.itvl_min = 0x20-0xF
.itvl_max = 0x20+0xF
.channel_map = ADV_CHNL_ALL
.filter_policy = BLE_SCAN_FILTER_ALLOW_ALL
.high_duty_cycle = false

own_addr_type = BLE_ADDR_TYPE_PUBLIC
direct_addr = NULL
duration_ms = INT32_MAX
cb = gapEventHandler
cb_arg = NULL
We are returning 0x212 from ble_gap_adv_start.

Hope this provides enough information...

Hi @matt001k ,
.itvl_min = 0x20-0xF .itvl_max = 0x20+0xF
Here,  itvl_min gets set to 0x11 and itvl_max gets set to 0x2F .
Can you please update the itvl_min to atleast 0x0020 and try ?  I think, it is out of the range of the values supported in spec .

@matt001k   ,  Thanks for reporting, would you please help share if any further updates for the issue?

Closing this issue. Feel free to reopen in case of further comments


Go implementation of encrypt/decrypt and ECDH to work together with the circuits

Porting the implementation of poseidon based encryption and decryption in zkp/js/lib/utils, based on https://drive.google.com/file/d/1EVrP3DzoGbmzkRmYnyEDcIQcXVU7GlOd/view.
Also added GenerateECDHSharedSecret() as a convenient method, to be used by the receiver of encrypted transaction outputs.
Currently dependent on an enhancement to go-iden3-crypto, which has been submitted as https://github.com/iden3/go-iden3-crypto/pull/66. But until that's merged, we are using the fork in kaleido-io/go-iden3-crypto as a go dependency replacement

Looks great to me. Will leave the approval and merge to @EnriqueL8


build: Stop creating builtins.json

Individual jsons are being generated for all modules,
even the builtin ones.
Signed-off-by: Anselmo L. S. Melo<EMAIL_ADDRESS>

@glima, @dorileo

v2: #753


Translation : Write all the texts of the app in the I18n file

What should be done ?
Basically, we want to migrate all hardcoded texts of the platform to src/locales/content/default.json which is the default locale in english.
Why is this useful ?
If we ever want to add languages support to disco, this is really important, once it is done, one could copy default.json in say french.json and write translation for the whole platform.
Labelled for later because this is quite tedious but it's a good practice that we should keep.

Will be an ongoing process, better to have less issues.


Update 1-cicd-background.md

Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects

Closing because it has been fixed in another PR. Please reopen if you think this was closed by mistake!


Muted problem

As soon as I give someone a rank on my server (play.galaxyservers.org) it says 'You are muted!'
is there anything I can do to prevent this.

Yes, sounds like a misconfiguration of your permissions.
Check with the verbose command to see what the issue is.
https://github.com/lucko/LuckPerms/wiki/Verbose


Allow SOCKS5/SOCKS4a to resolve DNS on the proxy

Both protocols support sending the domain in the CONNECT packet.

This is currently supported in SOCKS5/SOCKS4a. You can use the following code to parse DNS from the server side.
let mut client = SocksClientBuilder::new(proxyip, proxyport).socks4a().build_tcp_client(); let mut stream = client.connect_hostname("www.baidu.com", 80).await?;
You can refer to a SOCKS testing tool I wrote. sockstest

It should be noted that the Dante SOCKS server does not support SOCKS4a


build progress not updating in terminal

As you can see in the screenshot below, build is completed successfully but the progress bars are not updated.
I want to know if I'm doing anything wrong.
I'm fetching data from cms in bootstrap hook and adding it to data object as documented in "Fetching External Data" section. In each route.js, I'm reading the externalData object from {data} to build the page.
Thank you!

@s-kris I believe you’re seeing a race condition.
The issue is that it spins up child processes across all cores and only starts updating the bars after short delay once that is done.
You’re building faster than that total delay is my bet.

Oh my :)

Also, can I know why the bootstrap hook is being run multiple times? The Idea is to fetch data once in that hook and re-use it.
Btw, I just quickly tried to fetch external data and set it on global object before starting the build to read it in bootstrap hook instead of network call.
Even that function is called multiple times.
const { build } = require('@elderjs/elderjs');
const { preflightCheck } = require('./utils/preflight');

(async () => {
  await preflightCheck(); // fetch external data via network call and set it on global object here.
  await build();
})();

@s-kris Not sure 100% why your preflight would be called multiple times but bootstrap is called once for each worker that is spun up.

I will check about the preflight, thanks!
So, I think it's really helpful to have a hook beforBuildStart, so that we can fetch external data here and re-use it in bootstrap instead of multiple network calls.

I will check about the preflight, thanks!

@nickreese
okay, something is definitely weird.

I cloned the sample elderjs repo with npx degit Elderjs/template elderjs-app.
I added a log statement.
I built it with yarn build
It's being called multiple times!
There's that progress bar issue/race as well.

const { build } = require('@elderjs/elderjs');

(async () => {
  console.log("this should not appear twice");
  await build();
})();

@s-kris Set your workers to 1 by adding this to your elder.config.js to see if you still see it twice.
build: {
    numberOfWorkers: 1, // This controls the number of worker processes spun up during build. It accepts negative numbers to represent the number of cpus minus the given number. Or the total number of processes to spin up.
  }

My bet is that the console.log is getting called twice due to the cluster implementation... but I'd need to set aside more time to dig in to be sure.
I'm open to a PR including a beforeBuildStart hook, but there be dragons here. The main issue is that serialization of data across processes is a bit complex and there are pitfalls. That said, the existing build process is pretty messy and could use a refactor if you're up for it.

Sure @nickreese  Setting numberOfWorkers to 1 didn't fix it.
I'm not very confident on handling the multi-process data-serialization, but I will anyways need to implement beforeBuildStart hack for my project. I'll go through elderjs code base and come back to you. I am already behind a PR (for stemap) :). Sorry, I'm a solo-founder. So..

@s-kris As a prior solo founder, I'd just tell you to do any non-parallelizable work before you invoke Elder.js in your CI/CD process. Even if that is writing all of the API data to a json file that you toss out later. I went down the route of trying to implement a beforeBuildStart and after hitting edge-case after edge-case when it came to data parallelization... ultimately I just decided to preprocess things. 👍

I'm going to close this but feel free to use https://github.com/Elderjs/elderjs/discussions if you'd like to chat about your journey in building with Elder.js

@nickreese I thought of the json file way as a last measure. But after hearing you, I assume it'd be unwise for me to do otherwise :)  Thank you!