Enhancement of the hyint recipe to include etccdi indices

This version of hyint now includes the recipe_hyint_extreme_events with the hyint diagnostics in pipe with the extreme_events diagnostics. See #1063. This capability was already in the previous version of hyint but not called, so only a few refinements were applied.

Tasks

[x] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)

[x] Give this pull request a descriptive title that can be used as a one line summary in a changelog

[x] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.

[x] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.

[x] Please use yamllint to check that your YAML files do not contain mistakes
New recipe/diagnostic

[ ] Add documentation for the recipe to the doc/sphinx/source/recipes folder and add a new entry to index.rst

[x] Add provenance information

Closes #1063

Hi @earnone, is this something you are still working on, or is it ready for merging from your point of view?

@bouweandela it needs a little update to the provenance piped from extreme_events.

@earnone Can you finalize this pull request? Then we can test it when you're done.

@earnone We recently started using the styler package to style all our R code. Of course this introduced some conflicts with the changes implemented here, which I just tried to fix. Can you please check if your changes are still as expected (and use the styler package from now on to style your code?).

@earnone is this still relevant?
If yes, please finalize, otherwise close it and remove the branch.

Sorry I kept this on stand by because of the cdo issue but this is now
solved. Yes, will finalize quickly as it was actually already working.
Thanks @bouweandela https://github.com/bouweandela for the styler, yes
will check everything is working.
On Thu, 19 Dec 2019 at 10:58, Mattia Righi<EMAIL_ADDRESS>wrote:

@earnone https://github.com/earnone is this still relevant?
If yes, please finalize, otherwise close it and remove the branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/pull/1133?email_source=notifications&email_token=AG4WXMN3AOYNTT5KAZBWT2LQZNATTA5CNFSM4HTWUUH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHJCMQA#issuecomment-567420480,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG4WXMICWGNMSI4SPWJQDVTQZNATTANCNFSM4HTWUUHQ
.

I am closing this and opening a new pull request from scratch starting from master.

Please also delete the branch, if not used.

@bouweandela @mattiarighi I have reopened this to finalize the extreme_events option of hyint. No need for a new pull request. Solving conflicts with latest version and I need to deal with the provenance in cascade from extreme_events, could you remind me how to do that? I.e., how to pass the extreme_events (first diagnostic called) provenance to hyint (second diagnostic called). Thanks

To avoid losing the improvements in hyint in master, needed changes from branch version2_hyint_extrmes were re-applied starting from hyint in master. @bouweandela I am afraid I was still not able to pass the provenance from extreme_events to hyint: I do not know how to read extreme_events provenance and included it in that of hyint. Any idea?

@bouweandela ci/circleci: documentation fails because of a missing perfmetrics figure. I do not understand why the test is failing, can you help with this?

@bouweandela ci/circleci: documentation fails because of a missing perfmetrics figure. I do not understand why the test is failing, can you help with this? The recipe works smoothly on our machines.

This pull request deletes that file, resulting in the failure. Could you undo the commit where you deleted it? Or re-add it if that is difficult?

how to read extreme_events provenance and included it in that of hyint. Any idea?

You can specify a list of ancestors when saving the provenance of each figure or netcdf file. For the first diagnostic script, this will be files from the preproc directory. For the diagnostic script that uses the results of extreme events, this will be files from the work_dir of that diagnostic script.
See also here: https://esmvaltool.readthedocs.io/projects/esmvalcore/en/latest/interfaces.html#information-provided-by-the-diagnostic-script-to-esmvalcore

You can specify a list of ancestors when saving the provenance of each figure or netcdf file. For the first diagnostic script, this will be files from the preproc directory. For the diagnostic script that uses the results of extreme events, this will be files from the work_dir of that diagnostic script.
OK. I thought hyint (second diagnostic) should include the original model files used by extreme_events (first diagnostic) in preproc to produce the results read by hyint (second diagnostics). OK, much easier, thanks. Only, the link to the original files will be then missing?

Only, the link to the original files will be then missing?

No, this link should work fine: the preprocessor links to the original files, the first diagnostic links to the preprocessed files, and the second diagnostic links to the files produced by the first diagnostic. Of course that only works if both diagnostic scripts create a diagnostic_provenance.yml file.

@bouweandela could you have a look at the provenance file obtained with the inclusion of etccdi?
diagnostic_provenance.yml.txt
Or have a quick test? It seems ok to me but the provenance for timeseries nc files and figures do not load info from the preprocessed files.

Hi Enrico,
The ancestors in the provenance records should correspond to the ancestors as they are listed in the recipe. So for the second diagnostic script, that can be files from the work_dir of the first diagnostic and files for pr in the preproc dir. Not any other files.

ok, testing it.

All tests were successful. @bouweandela could you test it and review this? Thanks.

@bouweandela The current recipe_hyint.rst in master already mentions the possibility of including additional indices from extreme_events. In fact, the recipe_hyint_extreme_events.yml is just as recipe_hyint.yml but with the further call to the extreme_events diagnostic and inclusion of its results in hyint.
I would like to avoid to cause confusion, would it be better to point to a joint documentation? Or do we want to duplicate everything and add the further specs for the extreme_events part?

In that case you could maybe update recipe_hyint.rst so it includes this recipe? E.g. mention it here: https://github.com/ESMValGroup/ESMValTool/blob/master/doc/sphinx/source/recipes/recipe_hyint.rst#available-recipes-and-diagnostics and add some explanation on what the difference between the two recipes is?

Yes, I'd prefer to keep all in one place. How does it work with the file recipe_hyint_extreme_events.rst? We simply don't create it?

The recipe_hyint.rst file was updated to cover also the options for recipe_hyint_extreme_events.yml (without creating a separate recipe_hyint_extreme_events.rst). This should complete the PR.

extreme_events.R stops because of a missing library:
Error in library(ncdf4.helpers) :
  there is no package called ‘ncdf4.helpers’
Execution halted

extreme_events.R stops because of a missing library:
Error in library(ncdf4.helpers) :Yes, this is the issue affectingextreme_events.R(lack ofncdf4.helperslibrary on CRAN and need to install it from archive) and the reason for keeping thishyintcapability in stand-by. I understand we decided to proceed anyway while waiting for the library to be upgraded orextreme_events` rewritten without the missing library?

Sorry, I missed the discussion.
Should I test without extreme_events.R for now and leave this to another PR?

Sorry, I missed the discussion.
Should I test without extreme_events.R for now and leave this to another PR?

Actually this improvement of hyint is solely related to ingesting the results from extreme_events, so there is nothing to test without that part (everything else is untouched). As you prefer, we can also keep this aside and not include it in master for now. @bouweandela?

@mattiarighi The problem is reported here #1443. I would suggest going ahead with testing and merging this pull request, because the figures are part of the paper and the problem is with an existing recipe, not so much with what is added in this pull request.
We can see how to proceed with the dependency problems later, either by re-implementing the indices or by helping the authors of the ncdf4.helpers package to get their package back on CRAN.

Dumb question: where can I get the `ncdf4.helpers' package to test this PR?

Dumb question: where can I get the `ncdf4.helpers' package to test this PR?

Here
And here is a suggestion on how to install from url:
# Download package tarball from CRAN archive
url <- "https://cran.r-project.org/src/contrib/Archive/ncdf4.helpers/ncdf4.helpers_0.3-3.tar.gz"
pkgFile <- "ncdf4.helpers_0.3-3.tar.gz"
download.file(url = url, destfile = pkgFile)
# Install package
install.packages(pkgs=pkgFile, type="source", repos=NULL)
# Delete package tarball
unlink(pkgFile)

Thanks, I have tested both recipes successfully.
Maybe you can add the above snippet to the recipe documentation until we solve #1443.

@mattiarighi Could you have a look? Do we have a standard way to include known issues or warnings in the doc? I can replicate that in recipe_extreme_events too.

Thanks, it looks good to me.
I don't think we have a standard way of documenting this. Another option would be to add this to the recipe description section, it's more likely that users would look at that.
@bouweandela any opinion?

I think it's fine like this: we do not have a standard for documenting workarounds, because we try to just fix bugs instead. However, in this case that is taking some time, unfortunately.


Add [xau.cl xau.cl] #exchange #issuer GBCB4W…ACLXAU

Address: GBCB4WO6J4ET55RWK2SVX76LUQ4PQ7TCDHG2YFILQML7D6XR3HACLXAU
Tags: exchange issuer
Title: xau.cl  Domain: xau.cl
Requested by @exchangecl

@exchangecl These have already been flagged as you pointed them out a while ago.

Hi CryptoHobist,
I'm creating private assets for myself and friends with my own .CL domain names, trying to follow guidelines to be listed on Stellar Network and working on it. TOML files created and all verifications passed.
It's really weird that we were listed as SCAM. Definition of SCAM means illegal activities and fraud related topics and that's not the intention.
Please let me know how to remove the Scam flag

@exchangecl do you have any information to help support these claimes in your TOML file?
name="GOLD: Gold from Chile (XAU)"
desc="GOLD is backed by 24k Chilean Gold"
is_asset_anchored=true
anchor_asset_type="stock"

Yes, got all invoices of gold purchases and our tokens are backed on liquidity pool also. Invoices information of purchases contains sensible data, so i can send it to you to a private email.

GOLD token has 0 transactions, XAU (xau.cl) token already have approximately 500USD on value all backed on liquidity pool, Gold coins as collateral. Let me know contact information to send you all information required.

@exchangecl Private purchase of gold assets do not collatorize the token unless there is some kind of escrow or 3rd party storage audit.
What kind of stock are you claiming to be backed by. Is this shares of a colatorlaized storage audit.
Is there way that holders can exchange their tokens for the physical gold?

@exchangecl Can you supply the link to the pools you are referring to?

Ok, so it's not the case because gold purchases are personal. But all tokens purchased by third parties are stored on liquidity pool shares.
https://stellar.expert/explorer/public/liquidity-pool/3d5b824f55ba5a3ed025a10ae126fbc1399fc512901a0af136ce7bcd380808f9

@exchangecl Can you please help to clearify how the value/equity of the gold assets are avaliable throught the pool? The top holders in the pool are:
Rank	Account	Pool Share
1	GDTS3JKIAJNWQK7IEBRM6PVGVGOUKVI5YAWR2NSO4AJAWPMQ6VWQGOLD	84.4%
2	GA63IPURUGJQJ3UQ7C7T3FQNKCD2HDMRX2APU54JU3TD5ZM4WFBZCHXR	7.786%
3	GD3DPR4O4TM7DQ3VVOKXLPVM7PUUDFC47TRZPKGI3QWBMT6ET53GCVWN	7.786%
And the top holders of the token are:
Account	Account balance
GDTS3JKIAJNWQK7IEBRM6PVGVGOUKVI5YAWR2NSO4AJAWPMQ6VWQGOLD	1,263,535,669.329668 XAU (19.36%)
GA63IPURUGJQJ3UQ7C7T3FQNKCD2HDMRX2APU54JU3TD5ZM4WFBZCHXR	1,260,150,400 XAU (19.31%)
GBVIQ55I4WK4G2FUHCUWH54KVMQ32RJTOSLNYJZU76LKC2KNCLLF4JI3	332,999,331.9973279 XAU (5.10%)
GDRCCPE2EPVDHS6N7SQITCEWC64WQKZHQWE4QQFXWFT4PMJ5Q5OTRKYE	299,999,999.7118624 XAU (4.60%)
GDUGEP7V3WUYB64ZAQD72HI3L5BMVSJTGFM2QJNZNYL5VKBVRQKDVXCG	50,850,000 XAU (0.78%)
GD3DPR4O4TM7DQ3VVOKXLPVM7PUUDFC47TRZPKGI3QWBMT6ET53GCVWN	2,000,000 XAU (0.03%)
GD2O4DNBUBC26VNJQNWOYHO6X76YRKJKY3QJGQ6V5BJBAJY4OPZGV7GV	1,058,014.4497293 XAU (0.02%)
GCVDNKAWF7FYIBAHT3NKLI4KLPYXZJACAVCZJISEAWUA5E374YT222Q3	1,000,000 XAU (0.02%)

Sure,
GDTS3JKIAJNWQK7IEBRM6PVGVGOUKVI5YAWR2NSO4AJAWPMQ6VWQGOLD (Distributor Account)
GA63IPURUGJQJ3UQ7C7T3FQNKCD2HDMRX2APU54JU3TD5ZM4WFBZCHXR (My main account, personal)
GD3DPR4O4TM7DQ3VVOKXLPVM7PUUDFC47TRZPKGI3QWBMT6ET53GCVWN (My secondary account, personal)

[[CURRENCIES]]
code="XAU"
status="live"
issuer="GBCB4WO6J4ET55RWK2SVX76LUQ4PQ7TCDHG2YFILQML7D6XR3HACLXAU"
display_decimals=2
name="XAU: XAU is a private token not related with Physical Gold"
desc="XAU is a private token related to the XAU.CL domain. This token is not backed by physical gold, only backed by it's liquidity pool and intrinsic value."
conditions=""
image="https://www.xau.cl/logo.png"
is_unlimited=false
is_asset_anchored=true
anchor_asset_type="stock"
anchor_asset="XAU - Long term investing in XAU.cl operations on Stellar Network"

@orbitlens They seem to have modified the TOML file to be not deceptive but on stellar.expert these changes don't show. I checked last night and just now. Does stellar.expert take snapshots that have to be refreshed after original minting, or is something else the issue?
https://stellar.expert/explorer/public/account/GBCB4WO6J4ET55RWK2SVX76LUQ4PQ7TCDHG2YFILQML7D6XR3HACLXAU
I still am not able to edit the flagging, but that is a different issue. I suggest removing the flag on the name and issues and just leaving the domain.
I also would just add to the "Additional information" section.
They don't have a website but that really isn't an issue unless it gives the old narrative.
@exchangecl

TOML code on stellar.expert is updated with .well-known/stellar.toml information on domain.

I see that the tokens now correctly show the TOML information that you changed a few days ago. It didn't show up correctly, at least on my machine until today.
Thank you for helping to keep the Stellar Blockchain safer for all users. Good luck on your project.
I'll keep this open until we can get the warning on the issuing account removed. Still working on it though.
Rob TheCryptoHobbyist

How long it takes to be delisted?
Is it necessary to resubmit?

@orbitlens can you please help with removing the scam label from their issuing address? They have edited their their TOML file to not be misleading. The tokens are not flagged but the issuing address still has the SCAM label/flag. I tried to change the name to be "xau.cl" and tag as "Issuer"
GBCB4WO6J4ET55RWK2SVX76LUQ4PQ7TCDHG2YFILQML7D6XR3HACLXAU
I still am not able to modify/edit such things. I get "Error: Unhandled error"

Any updates?

I am sorry for the long delay. I just sent a reminder to @orbitlens via discord.

Thank you! Now it's labeled as #issuer [XAU CL]

Good luck with your project and be honest in all you do.


default dither method?

I'm wondering if FloydSteinberg() a good default method so that we can expose the following two interfaces to the users without the need to remember the algorithm names.
dither(img)
dither(img, n)

Closed in 477c98e.


Anti-crash for loading missing songs in week selector

What feature do you want to get added on the base engine?
read the title
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor and 6K+ support.

because Story Mode song loading is more complex, this will be more difficult to implement.

however, i will consider it

because Story Mode song loading is more complex, this will be more difficult to implement.

pretty sure 0.7/1.0 added this

because Story Mode song loading is more complex, this will be more difficult to implement.

pretty sure 0.7/1.0 added this

ive checked and im pretty sure it only prevents a crash if the first song in the week is missing

because Story Mode song loading is more complex, this will be more difficult to implement.

pretty sure 0.7/1.0 added this

ive checked and im pretty sure it only prevents a crash if the first song in the week is missing

No, it only happens if you have a "missing" week file

technically added in recent commit


Secrets

See https://github.com/PistonDevelopers/dyon/issues/266
This PR adds optional "secret" information in bool and f64 that is automatically derived from any composition of ∀/all and ∃/any with min/max loops.
What is a "secret"?
A secret is when a bool or f64 contains additional information that explains the reason why it has the current value. It allows you to write shorter algorithms for with the same expressiveness as a longer algorithm:
For example, this is normally how you would solve something:
fn main() {
    // Find where 3 is in 2D array
    data := [[1, 2], [3, 4]]
    hit := []
    x := any i, j {
        // remember the indices for later
        hit = clone([i, j])
        data[i][j] == 3
    }
    println(x) // prints `true`
    if x {
        println(hit) // prints `[1, 0]`
    }
}

When using a secret, you only need to look for the data and then unlock the secret with why:
fn main() {
    // Find where 3 is in 2D array
    data := [[1, 2], [3, 4]]
    x := any i, j { data[i][j] == 3 }
    println(x) // prints `true`
    if x {
        println(why(x)) // prints `[1, 0]`
    }
}

How to use secrets
Dyon will not tell you a secret unless you ask. There are two questions you can ask:

why(bool) - why has this expression a true value?
where(f64) - where does this number come from?

The secret contains the answer to any meaningful question you can ask within the Dyon syntax. There are some questions that does not have meaning, but an error message will help if it is wrong.
You can add information to a secret:

explain_why(bool, any)
explain_where(f64, any)

For example, we want to find "the top soccer player who scored most goals in a season where no player scored more than 50 goals":
ffn main() {
    seasons := [
        season(players: ["Ronaldo", "Hughes", "Solskjear"], goals: [20, 30, 40]),
        season(players: ["Ronaldo", "Solskjear", "Beckham"], goals: [30, 5, 60])
    ]
    top_players := []
    loop {
        top := any j {
            // Skip seasons we already have.
            if any i {top_players[i].season == j} {continue}
            max k {
                // Add name of player to secret.
                explain_where(seasons[j].goals[k], seasons[j].players[k])
            } <= 50
        }
        if top {
            player := why(top) // `[season, player_index, player_name]`
            season := player[0]
            name := player[2]
            push(mut top_players, {name: name, season: season})
        } else {
            break
        }
    }

    top_all_time := max i {
        explain_where(sum j, k {
            if seasons[j].players[k] != top_players[i].name {continue}
            seasons[j].goals[k]
        }, top_players[i].name)
    }
    println(top_all_time)
    player := where(top_all_time) // `[top_player_index, name]`
    name := player[1]
    println(link {
        "***** TOP PLAYER OF ALL TIME *****\n"
        "(in a season where no one scored more than 50 goals)\n"
        "name: "name"\n"
        "total goals: "top_all_time
    })
}

fn season_players_goals(players: [Player str], goals: [Goal f64]) -> Season {} {
    return {players: clone(players), goals: clone(goals)}
}

For detail information on usage, see https://github.com/PistonDevelopers/dyon/issues/266.
Performance
About the same performance as before.
Secrets use unused memory on the stack for bool and f64, which just adds noise in the CPU for old design.


Docs: Remove inaccurate info

Details
The documentation for upgrading app-template from 1xx to 2xx contains an inaccurate dot point when explaining changes:

The sentence says

ingress.main.enabled can be removed, since items are considered enabled by default (they can still be disabled by adding enabled: false).

This is incorrect as the app-template still has enabled: false as the default, so anyone removing that line from their values would have their ingress removed.
This line is found on this page:
https://bjw-s.github.io/helm-charts/docs/app-template/#from-1xx-to-20x
Please let me know if you require any further details or if I can help in any way. :-)

Thanks for the heads-up! I'll update the docs on this. In general the remark is correct as Ingress items will default to being enabled, but for the main one that's not the case due to the "overridden" value in values.yaml. This is something I plan to remedy in an upcoming v3 bump.

Ah, I get you - I noticed the same thing with the default "config" persistence item, but then I saw that your example in the docs was for a persistence item called "media" and kind of realised that any other persistence item was correctly "true" by default.
Exciting stuff, thanks! :-)


升级了flutter2.5.2后，ios环境运行demo报错architecture arm64

您好，我在升级了flutter2.5.2，运行demo报错architecture arm64， clang: error: linker command failed with exit code，即使在pods的agora_rtc_engine下的exclude architectures增加arm64，仍然报此错误，很困惑

尝试更改Validated workspace为true

清理一下，真机运行正常。
应该是agora_rtc_engine在flutter2.5.2生成pods时，影响模拟器arm64的支持。
感谢

Any solution for simulator?

@adfdev do this then run it with Xcode, not flutter run

Pls try to set Validate Workspace to Yes

@adfdev do this then run it with Xcode, not flutter run

Pls try to set Validate Workspace to Yes

Even running with xcode I always get the same error, the following:
building for iOS Simulator, but linking in dylib built for iOS, file '/Users/mac/Desktop/test-app/ios/Pods/AgoraRtcEngine_iOS/AgoraJNDExtension.framework/AgoraJNDExtension' for architecture arm64

Could you pls share the screenshot about the Validate Workspace. It works for me.

My solution work for me:
This guide after that run:

delete pods folder
delete podfile.lock
run flutter clean
run flutter pub get

Now I obtain this following error:
/Users/mac/Desktop/app/ios/Runner/GeneratedPluginRegistrant.m:12:9: fatal error: module 'agora_rtc_engine' not found
    @import agora_rtc_engine;

post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)
    target.build_configurations.each do |config|
      config.build_settings.delete 'IPHONEOS_DEPLOYMENT_TARGET'
      config.build_settings['EXCLUDED_ARCHS[sdk=iphonesimulator*]'] = 'i386 arm64'
    end
  end
end


dodo line manipulation

Currently dodo has 2 unimplemented functions parse_line and eval_line
A decision needs to be made on whether or not dodo should support line operations.
Any such operations cannot consume more of the file than needed to process any line instructions as dodo must always be able to operate efficiently on large files (a 16G postgres dump is my current example).
The implementation isn't too complex; when dodo is given the instruction l10 to goto line 10 dodo can keep stepping through the file (in chunks to reduce the overhead of frequent reads) counting \n characters until it has seen 9 and then step 1 character past (so at to be at the start of line 10).
If dodo never finds a line 10 then it is an error (similar to a failed expect).

My thinking here is that I do want to be able to say 'goto line 10' (l10) and have dodo behave correctly for this.
A 'small' (1024?) buffer with an fread and counting \n should be sufficient as a naive first attempt.

I will be too busy for a while.

I have this tentatively written (locally), I'll do some more testing and tidying before submitting.


Touchpad support

It seems that Pooltool doesn't work with touchpad

Can you describe the behavior you're observing? Pooltool works at the level of mouse position, and does not care what hardware was used to move the mouse.

So, simulator doesn't react on any touchpad events. No matter, is it aim mode or no, it doesn't even move screen with 'V' key.
upd:
After few minutes I tried with trackback with trackball ( red dot in the middle of keyboard of ThinkPad), and it works as intended.
I am on Ubuntu 22.04.3
p.s. do you have any plans on sounds?
P.s.p.s. big respect for this truly amazing work!

That is unfortunate :\
I'm not sure how I can help with that. If the touchpad controls the mouse, it should control the game, with a few caveats.
First, since you're on Linux, make sure you run
pip uninstall panda3d -y
pip install --pre --extra-index-url https://archive.panda3d.org/ panda3d

If you can only move just a little in any direction before the cursor goes out of the game screen, you need to run the above step.
Second, any unlikely scenario can occur where the game window is active, and the mouse position is frozen, but lies outside the game window. When this happens, no mouse movement controls control the game. However, this is independent of the mouse hardware. If you try the trackball and the trackpad right after one another, does the trackball work and the trackpad doesn't?

p.s. do you have any plans on sounds?

Yes! Let me know if you want to collaborate on that.

P.s.p.s. big respect for this truly amazing work!

Thank you very much.
p.s. How did you hear about pooltool?

hmm
Installing collected packages: panda3d ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts. pooltool-billiards 0.2.1 requires panda3d==1.10.13, but you have panda3d 1.11.0.dev3306 which is incompatible.
No, trackpad doesn't work after I play with trackball
How can I collaborate on introducing sounds into this beautiful simlator?
p.s. I asked stupid gpt what billiard games are available in the world and surprise surprise - pooltool was the first one in the list

Installing collected packages: panda3d ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts. pooltool-billiards 0.2.1 requires panda3d==1.10.13, but you have panda3d 1.11.0.dev3306 which is incompatible.

This is an expected warning, since we are overriding pooltool's panda3d requirement. In the pooltool python env, can you run
python

Then:
print(__import__("panda3d").__version__)

To exit,
exit()

What version is printed out? It should be >1.10.13
You should give it another go and see if your experience feels better.

p.s. I asked stupid gpt what billiard games are available in the world and surprise surprise - pooltool was the first one in the list

Wow, that's cool GPT has pooltool on the radar.

How can I collaborate on introducing sounds into this beautiful simlator?

Are you a sound engineer, or do you have a good mic and access to a pool table? We basically need a collection of sound bytes that get mapped to different events, like ball-ball, stick-ball, and ball-cushion collisions. Some background "bar ambience" would also be good.

No, unfortunately I am no sound engineer.... neither I have the pool table. But maybe for first iteration just sounds from legendary pool games from internet would be enough? Like with your first table that had no 3d )
p.s. But I would be happy to help with coding... maybe!!! If you have some small tasks

I'm going to close this issue since there is no clear action to take.
Since pooltool installation instructions and build dependencies have been updated since this issue, it's possible this no longer an issue.
Feel free to reopen if this issue is still experienced using the current installation instructions


Kube cluster upgrade

Once nodes get cordoned, perform graceful Pod move (with minimum number of failovers) from cordoned nodes to the new ones.

Coverage decreased (-0.3%) to 8.55% when pulling 5f28830c5dfaaf461e860b6641ad942e8ffe417e on kube-cluster-upgrade into 8aa11ecee2f5b073e3f8b53240ec1555ca1663c4 on master.

Coverage decreased (-0.3%) to 8.55% when pulling 9eac8534de7300494e164da8f52651d1902303d0 on kube-cluster-upgrade into c5d97585b86e920a7ddc39fb5206c6f0786058e3 on master.

Coverage decreased (-0.3%) to 8.556% when pulling bf0f15333c9e632b838a48d279fc1a0b7ed6eecf on kube-cluster-upgrade into c5d97585b86e920a7ddc39fb5206c6f0786058e3 on master.

Coverage decreased (-0.3%) to 8.545% when pulling faa1a70919d97a7ca4410eda744ae43cf223d6a7 on kube-cluster-upgrade into c5d97585b86e920a7ddc39fb5206c6f0786058e3 on master.

Coverage decreased (-0.5%) to 8.428% when pulling 34d840ce321a43440fb6e8a07b9627cc3076264b on kube-cluster-upgrade into 969a06f521d2cda53bfa9f840f0558e385c2215d on master.

Coverage decreased (-0.5%) to 8.428% when pulling 34d840ce321a43440fb6e8a07b9627cc3076264b on kube-cluster-upgrade into 969a06f521d2cda53bfa9f840f0558e385c2215d on master.

Coverage decreased (-0.5%) to 8.428% when pulling 95ab6f98c5ebf0a65b90be1bd6b8fc9e74ec71ab on kube-cluster-upgrade into 969a06f521d2cda53bfa9f840f0558e385c2215d on master.

Coverage decreased (-0.4%) to 8.492% when pulling 13d705b2d39bbd8cab6b2b3705aa837b60824883 on kube-cluster-upgrade into 969a06f521d2cda53bfa9f840f0558e385c2215d on master.

Coverage decreased (-0.4%) to 8.463% when pulling 18d3fb5d15eee4ac4a39184be796023ecf578cf9 on kube-cluster-upgrade into 7667847bfe8751c2b54658afed551b32037cb709 on master.

Coverage decreased (-0.4%) to 8.457% when pulling 08f13c30d124bee0c6ec79da276335fa305ab52a on kube-cluster-upgrade into 7667847bfe8751c2b54658afed551b32037cb709 on master.

Coverage decreased (-0.4%) to 8.462% when pulling ef003c4f225026485f872a0a0abc903acde25def on kube-cluster-upgrade into 48a43c5188c2585c2797c5540c4e3c3416ebbe40 on master.

Coverage decreased (-0.4%) to 8.46% when pulling ef003c4f225026485f872a0a0abc903acde25def on kube-cluster-upgrade into 48a43c5188c2585c2797c5540c4e3c3416ebbe40 on master.

Please also create a single pod disruption budget per cluster.

Coverage decreased (-0.7%) to 8.213% when pulling d20b2511e15bdf797ad171f1c64310d0e899b6e5 on kube-cluster-upgrade into ead11dd69b0d61b44ac455c2a6fb0a5580107871 on master.

Coverage decreased (-0.7%) to 8.213% when pulling 0da2154a8e410ebb363185f0e43c569c1049c3ad on kube-cluster-upgrade into 5cfdabb63e64e3fe8f2049d19c35fc24c2c85e1c on master.

Coverage decreased (-0.7%) to 8.213% when pulling 4da0d04499d01cc474dd0a22c4091e3f3d2e0e3a on kube-cluster-upgrade into 5cfdabb63e64e3fe8f2049d19c35fc24c2c85e1c on master.

Coverage decreased (-0.6%) to 8.077% when pulling ea535e4d323618f18e8781304a789cb9a3d59eae on kube-cluster-upgrade into 48ec6b35b9876b05621402ffce038aba85ffa4be on master.

Changes Unknown when pulling 6fba0c9d081cc60cee3e573980b11c0df37db5a3 on kube-cluster-upgrade into ** on master**.

Changes Unknown when pulling 6e2e9b24a4bd0de8c0d19ef76e98d95c95491580 on kube-cluster-upgrade into ** on master**.

Changes Unknown when pulling fda81f6d4ead7a9103d08d4beb49797bc75924fa on kube-cluster-upgrade into ** on master**.

Coverage decreased (-0.6%) to 8.072% when pulling be396a9ac7bfbf2198bbe112fa17e82289499eff on kube-cluster-upgrade into 83c8d6c419c6da6eb88e76637bbe9e5d6aeba8b3 on master.

Coverage decreased (-0.6%) to 8.041% when pulling c176260bccd5e7155341ff33de12f7068ae31977 on kube-cluster-upgrade into 83c8d6c419c6da6eb88e76637bbe9e5d6aeba8b3 on master.

Coverage decreased (-0.6%) to 8.041% when pulling 77805a45dfb18052539acdb1022ac7d19c17d37e on kube-cluster-upgrade into 702d901bd9a92208b86e7504bb0ee1a133e98d54 on master.

Coverage decreased (-0.6%) to 7.988% when pulling e4f32bb6811919d42e528debf11b3a6191c5e989 on kube-cluster-upgrade into 959b483d11c86fb2e9576d59ea849e5ed0d3861c on master.

Coverage decreased (-0.6%) to 7.988% when pulling 3e64d1e9be049aac87aaa00f538ffa6f83b9e132 on kube-cluster-upgrade into 5b29576a8e7425d3b08f6d46d1580ee28c64814a on master.

Coverage decreased (-0.8%) to 7.928% when pulling 20817d7e3900480473b4e798c5b2d60069c506bf on kube-cluster-upgrade into 6c4cb4e9dad3c77bdb3846a2861ddf8f58af9a22 on master.


Remove linked services when app removed?

I just removed an app, but I still see a Postgres service running for it. I suggest we remove corresponding services when removing an app.

@michiels I assumed it did remove those links as well. But probably it does not actually kill the container, which might be a bug in Dokku. Will look further into this.

Agreed. I also see the app container still running. Maybe we can change this issue into: "clean everything up on app removal" or something?

:+1:


Strong typing for indices

This would be helpful because the implementation uses job, task and machine indices a lot which are often combined (machine and task indices, specifically). The recent PR (#146) streamlines this but having stronger typing may help mitigate these issues.

Not going to do this any time soon.


Add handling of off-server users to user command

This addresses point 5 in https://github.com/python-discord/bot/issues/654
How it looks like currently:

How it will look like:

I wonder if it stands out enough. A moderator might miss the fact that they're not on the server.

Just to make sure, this doesn't close the issue. I will be working on point 1 after a future PR from @scragly is merged

This won't be too effective since the user convert only looks at the cache. Chances are that if the user is not in the guild, they won't be in the cache. Maybe briefly after they get banned or leave, but this won't be the case once the bot restarts. There is a custom FetchedUser converter which addresses that problem.

I used it on a user that was never on the server and it seemed to work without a problem

After checking out and running it locally, I can confirm it does not work:

Thanks, should be fixed


How to run from console_scripts / entrypoint?

Hello!
I'm trying to build an async cli utility which I can install using pip and which creates an entrypoint script.
See https://packaging.python.org/specifications/entry-points/ or https://python-poetry.org/docs/pyproject/#scripts for details about this feature.
How should my code look like?

import asyncclick as click
@click.command()
async def run():
    print("foo")

[tool.poetry.scripts]
foobar = "foobar.cli:run"

kind of works. But it's running with asyncio instead of trio.
I don't see any way to pass _anyio_backend="trio" without creating another synchronous function.

That's right. You need to create a (simple) sync function, or you can do this:
from functools import partial
run = partial(run, _anyio_backend="trio")

That being said, adding a decorator that does this is probably the cleanest solution. I'll do that in the next version.

Fixed in current main, there's now an @asyncclick.async_backend decorator.


CAS/MCAS missing?

Shouldn't *.cas.ms, *.mcas.ms and *.mcas-gov.us be added to the list?
As per this link

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

ID: d8336877-da93-ecd6-829a-0cc068caaf29
Version Independent ID: 1e70f094-f220-92c6-7e5b-cfaf071101d5
Content: Allow the Azure portal URLs on your firewall or proxy server - Azure portal
Content Source: articles/azure-portal/azure-portal-safelist-urls.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

Thanks for brining this to our attention. We will investigate it further and update you shortly.

@andrePKI - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Reduce the size of the Reference list tooltip

The tooltip font size is huge, please reduce it to be the same font size as the reference list component
Fix the tooltip alignment. It should be aligned with the component

Hi @nthangeniphumudzo, with the current implementation the tooltip still looks big. The tooltip width should adjust dynamically based on the amount of information it contains


The continue button is disabled all the time

Regardless of the location, the "continue" button is always disabled. Even it I change the style and remove the disabled class, clicking on the button doesn't take me to the next tab.

I tried different browsers but the result is all the same.

Yes, you need to search for an address or city name first and move the pin to the spot where the business is.

Why you need to search for and address if you grant Access to your location? One pinpointed the contributor should be able to create the POI with his data. Or at least that's the way that sounds more quick and logic from the business contributor standpoint

The point is to start with address. Attempt to geocode it to find the
likely location. Then allow the user to move the pointer to the physical
location of the building.
This process make sure we get the official address for the business. It
also helps OSM to build a more realistic set of data. Making it easier for
geocoders and human map make subsequent guesses at nearby locations.
On Fri, May 3, 2024, 5:48 AM Juan Jose Iglesias @.***>
wrote:

Why you need to search for and address if you grant Access to your
location? One pinpointed the contributor should be able to create the POI
with his data. Or at least that's the way that sounds more quick and logic
from the business contributor standpoint
—
Reply to this email directly, view it on GitHub
https://github.com/osmlab/onosm.org/issues/139#issuecomment-2092669054,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIGQRDPWGZO7Y6BKQAYVRDZANMN5AVCNFSM6AAAAABDIGDZOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSGY3DSMBVGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I think it is better to let the user move the pointer regardless of whether they enter an address. The address can help the user to move the pointer faster (if needed).


cmd: add a shard proxy server

e2e: framework: expose artifact and data dirs to tests
When we're running many tests in parallel, random directories are chosen
for every invocation. In order to ensure that test processes are placing
artifacts and data in the correct place, we need to pass that
information down.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
e2e: framework: isolate the accessory process code
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
e2e: rework how artifacts are deferred
Now, you can call server.Artifact() from anywhere and it should
correctly be run before the test is finished and kcp has been
terminated.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>
cmd: add a shard proxy server
Today, this server knows how to find all of the organizational
workspaces, the per-organization workspaces, and map them to the shards
they are placed on through history.
Signed-off-by: Steve Kuznetsov<EMAIL_ADDRESS>

Depends on https://github.com/kcp-dev/kubernetes/pull/32

Note to reviewers - yes, there is a lot of space here to write unit tests etc for the actual mapping of user-facing RV to etcd-local RV but as of now we don't even have the right types or process in place to migrate data, so I would love to hold off on locking into an impl until the basics are in and we can do something more intelligent while actually landing a real migration workflow.

Comments to be addressed in follow-ups.


[Docs]  List supported operations for the Meshery adapter for Octarine

Current State:
The operations supported by the adapter for Linkerd are not listed in its documentation.
Desired State:
Under the heading Sample Applications and include a list of the supported operations, with a small explanation for each. The operations can be found listed here; (lines 34-54). Look out for the following line: opType: meshes.OpCategory_SAMPLE_APPLICATION. This means that the operation listed is a sample application.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

@chandrashritii can I use this issue as a reference for the tutorial? The one I spoke about on Slack? I think its fairly simple and easy to understand.

@KarenEfereyan Thank you for taking up the initiative to make a tutorial 🎈
Considering that the tutorial is majorly on using Git (CLI and desktop app), and demonstrating how to sign off on your commits, the issue referred can be a relatively simple one. Regardless, I would suggest you take up something a bit more UI related because that'll be easier to showcase and will be more interesting than adding a few dry lines of documentation. It'll also be another chance to showcase the meshery, layer5 or the smp sites. I'll make sure to point you to a few good issues on Slack.

@chandrashritii I want to take up this issue

@chandrashritii @Ruth-ikegah I want to take up this issue

@chandrashritii @Ruth-ikegah I want to take up this issue

@was-coder you can go ahead. Let us know if you have issues. Be sure to check out the contributions guide


ADD: FileControllerTests

what is the difference to #105 ? this should be probably done in on PR an Master. Maybe this on can be merged into #105


no refetch with suspense since 0.3.17

I've updated my dependencies and the fetching behaviour is different. In my app, whenever I now navigate to a page I was previously at, there is no refetch if I use suspense.
The same happens in the suspense codesandbox examples, please notice the network requests as I click the button:
0.3.21 without suspense:

0.3.21 with suspense:

0.3.16 with suspense:

Yep, looks like a bug. I'll dig into this later today.

This should be fixed with 0.3.23


r/adbcsqlite: int type conversion to R

Again, not sure which package/library is to blame for this (if we agree that the behavior is undesirable). I'm currently mostly trying to keep track of things I run into.
An integer in SQLite is returned as a numeric in R.
With DBI/RSQLite we have
str(
  DBI::dbGetQuery(
    DBI::dbConnect(RSQLite::SQLite(), ":memory:"),
    "SELECT 1 AS a"
  )
)
#> 'data.frame':	1 obs. of  1 variable:
#>  $ a: int 1

whereas using adbcsqlite I "only" get a numeric value back:
library(adbcdrivermanager)

db <- adbc_database_init(adbcsqlite::adbcsqlite(), uri = ":memory:")
con <- adbc_connection_init(db)

str(as.data.frame(read_adbc(con, "SELECT 1 AS a")))
#> 'data.frame':	1 obs. of  1 variable:
#>  $ a: num 1

@paleolimbot

@paleolimbot I'm happy to tackle the problem of providing the user with options (and implementing them) for how to handle the int64 type in R on the adbi side. This is backend agnostic and from a distance I don't see any problems as the required hooks are already there.
There's an (SQLite-specific) sub-problem here though: An int of any size is represented as an int64
library(adbcdrivermanager)

con <- adbc_connection_init(
  adbc_database_init(adbcsqlite::adbcsqlite(), uri = ":memory:")
)

res <- read_adbc(con, "SELECT 1 AS a,<PHONE_NUMBER>123456789 as b")
res$get_schema()$children
#> $a
#> <nanoarrow_schema int64>
#>  $ format    : chr "l"
#>  $ name      : chr "a"
#>  $ metadata  : list()
#>  $ flags     : int 2
#>  $ children  : list()
#>  $ dictionary: NULL
#>
#> $b
#> <nanoarrow_schema int64>
#>  $ format    : chr "l"
#>  $ name      : chr "b"
#>  $ metadata  : list()
#>  $ flags     : int 2
#>  $ children  : list()
#>  $ dictionary: NULL

Most intuitive from a user's perspective I believe (I might be biased though) however is something like
str(
  DBI::dbGetQuery(
    DBI::dbConnect(RSQLite::SQLite(), ":memory:"),
    "SELECT 1 AS a,<PHONE_NUMBER>123456789 as b"
  )
)
#> 'data.frame':	1 obs. of  2 variables:
#>  $ a: int 1
#>  $ b:integer64<PHONE_NUMBER>123456789

i.e. int types that fit into the R integer range are returned as such and only for those which do not, we resort to any of the sub-optimal options like bit64::integer64, double, etc.
I could of course look up max/min values and decide what to do based on that.
@krlmlr I assume this is what RSQLite does, right?
The not-so-nice thing is that this now is a backend-specific behavior. I might want to do this for SQLite, but probably not for PostgreSQL. An int64 in PostgreSQL should always go the bit64::integer64, double, etc. route irrespective of value range.
@paleolimbot Do you any ideas for (cheaply) getting information on value ranges (only when needed) before materializing the data as R object?

The bigint argument to dbConnect() decides how integers are returned.
https://rsqlite.r-dbi.org/reference/sqlite


[ZEPPELIN-5071] Appears when building zeppelin-web：'npm run build:dist' failed.

What is this PR for?
When you use root user to build zepeelin on Linux host, because Some componentsdoes not support root user execution, an exception will be thrown：
[ERROR] npm ERR! code ELIFECYCLE
[ERROR] npm ERR! errno 3
[ERROR] npm ERR<EMAIL_ADDRESS>build:dist: `npm-run-all prebuild && grunt pre-webpack-dist && webpack && grunt post-webpack-dist`
[ERROR] npm ERR! Exit status 3
[ERROR] npm ERR!
[ERROR] npm ERR! Failed at the<EMAIL_ADDRESS>build:dist script.
[ERROR] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[ERROR]
[ERROR] npm ERR! A complete log of this run can be found in:
[ERROR] npm ERR!     /root/.npm/_logs/2020-09-25T18_56_09_331Z-debug.log
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:22 min
[INFO] Finished at: 2020-09-26T02:56:09+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.6:npm (npm build) on project zeppelin-web: Failed to run task: 'npm run build:dist' failed. org.apache.commons.exec.ExecuteException: Process exited with an error: 3 (Exit value: 3) -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.6:npm (npm build) on project zeppelin-web: Failed to run task

What type of PR is it?
Improvement | Feature |
Todos
You need to modify the following code in zeppelin-web/package.json
"build:dist": "npm-run-all prebuild && grunt pre-webpack-dist && webpack && grunt post-webpack-dist",
"build:ci": "npm-run-all prebuild && grunt pre-webpack-ci && webpack && grunt post-webpack-dist",

change to：
"build:dist": "npm-run-all prebuild && bower install --silent --allow-root && grunt pre-webpack-dist && webpack && grunt post-webpack-dist",
"build:ci": "npm-run-all prebuild && bower install --silent --allow-root && grunt pre-webpack-ci && webpack && grunt post-webpack-dist","lint:watch": "esw --watch src",

Of course, you can also add a configuration file~/.bowerrc to make bower have root privileges{"allow_root": true }

What is the Jira issue?
ZEPPELIN-5071
How should this be tested?
Environment:
MavenVersion：Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f)
JavaVersion：1.8.0_141
OSVersion：Centos7.3

npmVersion：3.10.10
nodeVersion：v6.17.1

(1)
[root@xinxing]# cd zeppelin (Project root directory)
[root@xinxing zeppelin-web]# mvn -X clean package -pl 'zeppelin-web' -DskipTests

(2)
[root@xinxing]# cd zeppelin-web
[root@xinxing zeppelin-web]# npm run build:dist

Screenshots

Questions:

Does the licenses files need update?
Is there breaking changes for older versions?
Does this needs documentation?

In my opinion we should not activate the root build by default. This should be an additional step for the user, as security is being overridden.
In general, no root rights are required to build Zeppelin.

For the security of Zeppelin, root should not be granted by default. However, if the user builds the root user into Zeppelin , he will receive an exception, which is not a good experience for the new user. This question will be provided to future friends for reference, thank you.

I think a note on how_to_build.html should be sufficient for new users.


Alternative packages for R?

It is Awesome. Do you guys know if R has packages that can do the same things? Thank you.

Hi guys,
do you find alternative packages ??
Please,let me know.
Thanks


Merge repositories

This is simply copying the contents of https://www.github.com/harche/{kata-operator-payload,kata-operator-daemon} into the kata-operator repository. We've created separate repositories in the beginning but it's not necessary anymore.

/lgtm


chore: migrate qdrant client to use new API

Description
We have deprecation warnings when building core since we updated qdrant_client from 1.9.x to 1.10+.
Main changes:

QdrantClient -> Qdrant
QdrantClientConfig -> QdrantConfig
client methods now use a "builder" pattern, instead of positional args + ...Default()
They removed all the xxx_blocking methods, and there is now a .wait() method to replace them. By default, qdrant is eventually consistent but with wait you can wait for changes to be actually propagated (but AFAICT we never use that)
Payload::new_from_hashmap is deprecated and replaced with ::from

Risk
Somewhat risky as it touches qdrant client
Deploy Plan
deploy core

I tested creating a data source, adding a doc in it, querying the data source, deleting the data source. All of these looking good

collection creation tested locally, looking good


Create comilation error

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

Sure, here is a Java code snippet with at least five compilation errors:
public class CompilationErrors {
public static void main(String[] args) {
// 1. Missing semicolon
System.out.println("Hello, World")
    // 2. Undefined variable
    int result = a + 5;

    // 3. Method not defined
    printMessage("This will cause an error");

    // 4. Incorrect method signature
    public void incorrectMethod() {
        System.out.println("This method signature is incorrect");
    }

    // 5. Mismatched data types
    String number = 12345;
}

// 3. Method not defined - Correct definition for reference
// public static void printMessage(String message) {
//     System.out.println(message);
// }

}
Here are the errors in this code:

Missing semicolon after  System.out.println("Hello, World") .
The variable  a  is not defined before it's used in  int result = a + 5; .
The method  printMessage  is called but not defined in the code.
The method  incorrectMethod  is defined inside another method, which is not allowed in Java.
The variable  number  is declared as a  String  but assigned an  int  value.

To fix these errors, you would need to:

Add a semicolon after  System.out.println("Hello, World") .
Define the variable  a  before using it.
Define the method  printMessage .
Move the method  incorrectMethod  outside of the  main  method.
Assign a  String  value to the  number  variable, or change its type to  int .

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).


[3.x] Synchronous shaders in editor

The old style synchronous shaders allow the project manager and editor to load faster.
Fixes #62244
This is just a draft to get some discussion going on this.
Notes
This is just a draft for now, as there are a number of ways of addressing this. Based on discussions with @akien-mga there seem to be some pros and cons for defaulting asynchronous shaders in the editor which occurred in #61995 :

pros: enables finding async shader compilation bugs in the editor faster
cons: on some systems, seems to significantly slow down the editor / project manager, even with caching. Additionally, if there are problems compiling async shaders with some drivers, it could mean the editor not working at all (e.g. #62203 )

In particular it doesn't seem as though compiling all the shaders in a project ahead of time is advantageous in the editor. You may only be editing certain scenes, and the editor startup time is crucial to keep as low as possible for devs.
Some options (not exhaustive):

Do nothing (deal with the decreased editor performance)
Fix the performance to be as good as before
Disable async in the editor / project manager (easy for users, but means no feedback on async shader compilation issues)
Add a separate editor setting to enable / disable async in the editor, rather than using the project setting (if this defaulted to async, we would still get feedback, but most people's editor would run slower)

I'm also not sure that editor_hint is currently set when in the project manager (it seems to be turned on in one place, and turned off in another), so we might need a little modification to get this working in the project manager.

Given the evidence, I'm in favor of having separate runtime and editor settings. For some GPUs and projects, async is indeed bad for agility in the editor.

Something like this might be better implementation wise:
diff --git a/servers/visual_server.cpp b/servers/visual_server.cpp
index 5baf0ebaff..685e46dc05 100644
--- a/servers/visual_server.cpp
+++ b/servers/visual_server.cpp
@@ -2738,6 +2738,7 @@ VisualServer::VisualServer() {
 	GLOBAL_DEF("rendering/gles3/shaders/shader_compilation_mode", 2);
 	ProjectSettings::get_singleton()->set_custom_property_info("rendering/gles3/shaders/shader_compilation_mode", PropertyInfo(Variant::INT, "rendering/gles3/shaders/shader_compilation_mode", PROPERTY_HINT_ENUM, "Synchronous,Asynchronous,Asynchronous + Cache"));
 	GLOBAL_DEF("rendering/gles3/shaders/shader_compilation_mode.mobile", 0);
+	GLOBAL_DEF("rendering/gles3/shaders/shader_compilation_mode.editor", 0);
 	GLOBAL_DEF("rendering/gles3/shaders/max_simultaneous_compiles", 2);
 	ProjectSettings::get_singleton()->set_custom_property_info("rendering/gles3/shaders/max_simultaneous_compiles", PropertyInfo(Variant::INT, "rendering/gles3/shaders/max_simultaneous_compiles", PROPERTY_HINT_RANGE, "1,8,1"));
 	GLOBAL_DEF("rendering/gles3/shaders/max_simultaneous_compiles.mobile", 1);

Is there any in-built way of detecting whether we are in the project manager? Or could we simply overwrite this project setting in the ProjectManager constructor or something like that? :thinking:

There's Main::is_project_manager() that you can use for this.
BTW I checked and the project setting override I proposed would actually always be active for TOOLS_ENABLED builds, so it would also affect running the project from the editor, which might not be wanted. So your approach might be better.

I've just tested and it turns out that is_editor_hint() is still set in the project manager at the time the setting is queried, so it actually works already as is.
I have however added:
if (!Engine::get_singleton()->is_editor_hint() || !Main::is_project_manager()) {

As this is clearer and more future proof, as editor_hint is set to false later by the ProjectManager.
The only thing is the is_project_manager() check adds a dependency to main.h, although imo clarity of intention is probably more important here, unless we should be avoiding this dependency for chaining reasons.

The only thing is the is_project_manager() check adds a dependency to main.h, although imo clarity of intention is probably more important here, unless we should be avoiding this dependency for chaining reasons.

Yeah that's annoying but it's a pre-existing design issue. We should refactor this to keep better track of what kind of Godot instance we're running and have this easy to query in Engine (e.g. Engine::get_run_mode() that would return an enum value that can be PROJECT_MANAGER, EDITOR, RUNTIME).

Thanks!


linkCustomizationName prop on 'react-plaid-link'

unable to pass in linkCustomizationName prop into  to get customized plaid link
import PlaidLink from 'react-plaid-link';
<PlaidLink
clientName="plaid"
env="sandbox"
publicKey="ADD_PUBLIC_KEY_HERE"
product={['assets']}
onSuccess={this.handleOnSuccess}
linkCustomizationName="my_customized_plaid"

Connect bank account

this prop does not exist in 'react-plaid-link'
is there another way around this or do i have to wait for a new version of 'react-plaid-link' to have that prop?

please email me at<EMAIL_ADDRESS>

#81 should solve this for you shortly!


registry: Adding registry profile to be able to build independently

"-P registry" allows to build the registry without the other semantics
modules
If not profile is given, the "default" profile is built, which
contains all other modules (behavior as before the change)
@fmisir Can you please review?
Matthias

Yes, this is an option. Problem in my case: I'm creating Dockerfiles which only contains the registry an not the other subidrs of semantics/
I want to keep it like that to increase build performance and reduce size of the Docker layers.
The proposed change, should keep the default behavior as ist was before the change, but make it possible to build the registry without having the other subdirs available.
Output with -pl <module>
#15 10.59 [ERROR] [ERROR] Some problems were encountered while processing the POMs:
#15 10.60 [ERROR] Child module /build/framework of /build/pom.xml does not exist @
#15 10.60 [ERROR] Child module /build/framework/dsc of /build/pom.xml does not exist @
#15 10.60 [ERROR] Child module /build/framework/edc of /build/pom.xml does not exist @
#15 10.60 [ERROR] Child module /build/adapter of /build/pom.xml does not exist @
#15 10.60 [ERROR] Child module /build/semantic-hub of /build/pom.xml does not exist @
#15 10.60  @
#15 10.61 [ERROR] The build could not read 1 project -> [Help 1]
#15 10.61 [ERROR]
#15 10.61 [ERROR]   The project net.catenax:semantics:1.3.0-SNAPSHOT (/build/pom.xml) has 5 errors
#15 10.61 [ERROR]     Child module /build/framework of /build/pom.xml does not exist
#15 10.61 [ERROR]     Child module /build/framework/dsc of /build/pom.xml does not exist
#15 10.61 [ERROR]     Child module /build/framework/edc of /build/pom.xml does not exist
#15 10.61 [ERROR]     Child module /build/adapter of /build/pom.xml does not exist
#15 10.61 [ERROR]     Child module /build/semantic-hub of /build/pom.xml does not exist


UserOptions.AllowedUserNameCharacters?

I am wondering what value was added here.  Previously we set a regex for this option.  It appears now we need to assign an entire character map via string.  I much preferred the regex method as our validation method was standardized, it allowed us to force a particular username format and in most cases required far fewer characters to define.

We have been moving away from regular expressions throughout asp.net as part of our security effort. Some folks tend to write complicated regular expression based validators that leave themselves open to DoS, and that's very hard to mitigate against.
You could, if you wished to continue with regular expressions, implement your own IUserValidator implementation and register that with the IdentityBuilder.AddUserValidator method.

Fair enough.

@blowdart , sorry for this question on a old topic but it will be great if you could guide me a bit. If you use AllowedUserNameCharacters which is presumably safer, how would you, for instance, go about allowing a user to enter a UserName  in ASP.NET Identity in the form myuser.name but disallow for dots at the edge of the string (myusername. or .myusername)? I am concerned on both client-side and server-side validation and recalling that UserName is part of the Asp.Net Identity EF.
Thanks

You can always write your own derived UserValidator that adds additional validation like excluding dots from the ends of the user name.

Hi @HaoK , if understand well UserValidator is server side and you would do it with regex or linq, right? If so, on the client side you would use RegularExpression? cheers

Up to you, but if you are just trying to block starting with ".", you can simply check for starts/EndsWith

Yep, I was however unclear on whether UserValidator remains server side only, I think it does.

Ah yes its server side only

The client side validation you would need to do in your view (i.e with annotations on your model)

OK thanks a lot


Can´t modify correctly geojson data

Hi. And sorry. I´m new with Mapbox and code... and all. But I really what to make a map similar to nyc-attractions. Someone know why if I change de data of art_galleries.geojson file and tried to run the complete map... The art galleries continue appear in the same place although I deleted them. And the points of my geojson doesn't appear. :cry: I tried to modify another of the files and now... all the points (even those I not modified) disappeared :disappointed:
I think it might be related to the "Running locally" and "Building" but... I couldn´t understand these requirements.
:sweat_smile:

Did you figure this out?  If you make changes you need to build again so that it grabs the changes in bundle.js.  I discovered that it does not work with the mapbox id beforehand.  It solely needs to be npm run build.


gradle.properties is getting updated and Versions.kt stops updating when running buildSrcVersions

Checklist

[x] I verified that the task ./gradlew :dependencyUpdates works correctly. If that's not the case, it's better to submit a bug-report to https://github.com/ben-manes/gradle-versions-plugin
[ ] I tried to reproduce the bug within sample-groovy, sample-kotlin or sample-versionsOnlyMode
[x] I have run with $ ./gradlew --scan buildSrcVersions or $./gradlew --scan refreshVersions to provide a Build Scan URL

Describe the bug
I've just upgraded to 0.6.0 and ran ./gradlew buildSrcVersions. It produced an updated Versions.kt as expected. I updated Versions.kt with newer versions and updated Gradle from 5.5.1 to 5.6.2. I ran ./gradlew buildSrcVersions again and it looks like Versions.kt did not update again.
Note the console output for dependencyUpdates is correct.
The gradle.properties file also keeps getting updated every time I run ./gradlew buildSrcVersions. My understanding is this should only happen when running ./gradlew refreshVersions?
To Reproduce
Build scan URL: https://scans.gradle.com/s/lvzdafpbzky7y/console-log
see above
Expected behavior
I expected Versions.kt to get updated after I manually updated versions.
I expected Gradle runningVersion in Versions.kt to get updated from 5.5.1 to 5.6.2 after I upgraded gradle wrapper via gradle wrapper --gradle-version 5.6.2.
Additional context

Thanks for the report, I will look into it next week
Le sam. 28 sept. 2019 à 17:46, Shane Witbeck<EMAIL_ADDRESS>a
écrit :

Checklist

I verified that the task ./gradlew :dependencyUpdates works
correctly. If that's not the case, it's better to submit a bug-report to
https://github.com/ben-manes/gradle-versions-plugin
I tried to reproduce the bug within sample-groovy, sample-kotlin or
sample-versionsOnlyMode
I have run with $ ./gradlew --scan buildSrcVersions or $./gradlew
--scan refreshVersions to provide a Build Scan URL
https://dev.to/jmfayard/the-one-gradle-trick-that-supersedes-all-the-others-5bpg

Describe the bug
I've just upgraded to 0.6.0 and ran ./gradlew buildSrcVersions. It
produced an updated Versions.kt as expected. I updated Versions.kt with
newer versions and updated Gradle from 5.5.1 to 5.6.2. I ran ./gradlew
buildSrcVersions again and it looks like Versions.kt did not update again.
Note the console output for dependencyUpdates is correct.
The gradle.properties file also keeps getting updated every time I run ./gradlew
buildSrcVersions. My understanding is this should only happen when
running ./gradlew refreshVersions?
To Reproduce
Build scan URL: https://scans.gradle.com/s/lvzdafpbzky7y/console-log
see above
Expected behavior
I expected Versions.kt to get updated after I manually updated versions.
I expected Gradle runningVersion in Versions.kt to get updated from 5.5.1
to 5.6.2 after I upgraded gradle wrapper via gradle wrapper
--gradle-version 5.6.2.
Additional context
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jmfayard/buildSrcVersions/issues/92?email_source=notifications&email_token=AADQFSDFS4F3YECESFBBJ3TQL533VA5CNFSM4I3OUFAKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HOKE2OA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADQFSB6HXKT4XLDWOW6JV3QL533VANCNFSM4I3OUFAA
.

Hello @digitalsanctum  I can't reproduce what you describe.
I tried this:
git clone git@github.com:jmfayard/buildSrcVersions.git
cd buildSrcVersions/sample-groovy/
./gradlew buildSrcVersions
# gradle.properties is not modified
./gradlew refreshVersions
# gradle.properties is modified

Sorry. I forgot to mention that I ran refreshVersions at some point and I think the problem started after that. I just reproduced with the following steps:
git clone git@github.com:jmfayard/buildSrcVersions.git
cd buildSrcVersions/sample-groovy/
./gradlew buildSrcVersions
# gradle.properties is not modified
./gradlew refreshVersions
# gradle.properties is modified
# update build.gradle and change guava version to 14.0 from 15.0
./gradlew buildSrcVersions
# Versions.kt did not update

@digitalsanctum I was surprised, but you were totally right.
Is it supposed to be fixed in version 0.6.2.
Can you confirm that this is the case?

@jmfayard Sorry to say I'm having a different issue now. I upgraded to 0.6.2 and ran ./gradlew buildSrcVersions. This resulted in both my Libs and Versions files getting corrupted. All the constants got prepended with the string: from-editorconfig-file. Here's an example:
from-editorconfig-fileconst val clikt: String = "com.github.ajalt:clikt:2.1.0"
I also tried running ./gradlew refreshVersions && ./gradlew buildSrcVersions which updated gradle.properties but still corrupted Libs

Sorry to hear that, I will have a look at this today

@digitalsanctum I think/I hope I have fixed my mistake in v0.6.3 that I just released


Feature setup

Set up linters
Add devise for authentication
Add rack-cors to communicate with frontend

Thanks for the review @tamana-Balkhi. I will fix it.

Please you can check it now, thanks.


Labels sorting by colour

Gitea version (or commit ref):
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
...
We are using same colours for same "type" of labels so it would be great to be able to sort the labels by colour. :)
Screenshots

@Marcsiq2 Any further explanation?

Interesting idea, there is talk about attaching a weight to labels for prioritizing the issues here https://github.com/go-gitea/gitea/issues/2616 may be slightly related?

To do this I actually prefixed my labels with kind/ or type/ (like they do here) etc as they are sorted alphabetically, however that makes selecting them with the keyboard from a long list almost imposable.

Color is not sortable.
Now Gitea supports scoped labels, and in the future the prioritized labels would also help.


Move metric logging from utils to training

Context
What is the purpose of this PR? Is it to

[ ] add a new feature
[ ] fix a bug
[ ] update tests and/or documentation
[ ] other (please add here)

Addresses #1440.
Changelog
What are the changes made in this PR?
Test plan
Please make sure to do each of the following if applicable to your PR. (If you're not sure about any one of these just ask and we will happily help. We also have a contributing page for some guidance on contributing.)

[ ] run pre-commit hooks and linters (make sure you've first installed via pre-commit install)
[ ] add unit tests for any new functionality
[ ] update docstrings for any new or updated methods or classes
[ ] run unit tests via pytest tests
[ ] run recipe tests via pytest tests -m integration_test
[ ] manually run any new or modified recipes with sufficient proof of correctness
[ ] include relevant commands and any other artifacts in this summary (pastes of loss curves, eval results, etc.)

UX
If your function changed a public API, please add a dummy example of what the user experience will look like when calling it.
Example of docstring: https://github.com/pytorch/torchtune/blob/6a7951f1cdd0b56a9746ef5935106989415f50e3/torchtune/modules/vision_transformer.py#L285
Example in our docs: https://pytorch.org/torchtune/main/tutorials/qat_finetune.html#applying-qat-to-llama3-models

[ ] I did not change any public API;
[ ] I have added an example to docs or docstrings;

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

Project coverage is 71.97%. Comparing base (0bdd308) to head (7e52db3).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1459      +/-   ##
==========================================
+ Coverage   69.80%   71.97%   +2.17%
==========================================
  Files         272      272
  Lines       13053    13040      -13
==========================================
+ Hits         9111     9386     +275
+ Misses       3942     3654     -288

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


[Refactoring] Add noise parameter and allow multi-qubit gates in noisy circuit

This pull request includes the following modifications :
I.  Allow multi-qubit gate in noisy circuit by using tensor method
1. Bug Fix:

[x] The tensor method has been modified to correct the bug mentioned in issue #210
[x] It is now possible to call tensor for control gates, as well as for the special definition of SWAP and CSWAP gates, following the logic of the BlockToTensor method from qadence. (in fact I will not use it -> can be erased because not really well implementing @jpmoutinho)

2.Modification of the tensor Method for operator_product:

[x] Remove the promote_operator method because the tensor method now automatically handles the promotion by accepting the n_qubits parameter as needed.
For example, if a gate is defined on 2 qubits (control and target) and it needs to be defined for a 4-qubit system, setting n_qubits to 4 is now sufficient.
[x] Consequently, the operator_product function has been modified to no longer depend on promote_operator and we remove the target attribut. It now assumes that it receives tensors with the appropriate size, eliminating the need for further modifications #221.

3.Modifications link to the forward passes of digital gates:

[x] Create the apply_density_mat to directly compute $O \rho O^{\dagger}$
[x] Modify the Noise and Primitive forward methods according all the modification bellow.

4.Add tests:

[x] Add test for the new function apply_density_mat
[x] Modify test for the new function test_operator_product

II. Add noise as a parameter of gates.

[x] Create a class Noisy_protocols as the Noise class from qadence.
[x]  Add a noise attribut to the primitive class. This attribut will be an instance of Noisy_protocols.
[x]  Modify operator_product to directly compute $U \rho U^{\dagger}$ instead of $U \rho$.
[x] Modify the Primitive forward function to call the Noise one.
[x] Modify the Noise forward function
[x] Modify the noise gates unit tests: test_flip_gates,  test_damping_gates, test_noise_circ.
[x]  Add test on the primitive gates noise parameter
[ ] Modify the noise part of the documentation.

Closes #210
Closes #221
Closes #200.

To be merged after #220.

Hey @EthanObadia, closing this MR since your work is already merged to main :)
Thanks again for your contributions!

Hey @EthanObadia, closing this MR since your work is already merged to main :)
Thanks again for your contributions!

Hi @jpmoutinho, I ended up taking a few unexpected days off, so I’ve just now had the chance to look at this branch. Thank you so much for your help with my work.
I noticed all the reorganization that has been done, and I find that the organization of pyq within different files and folders is much clearer now. I also find the syntax of the two forward functions much clearer this way, as well as the parent class QuantumOperation.


Error: the string "Header: Type 52695 must be 1 or 2" was thrown, throw an Error :)

The sample file: BIRD.WMF.zip
The sample code:
var jsEnv = require("browser-or-node");
var canvas = require("canvas");
// eslint-disable-next-line no-undef
global.ImageData = canvas.createImageData;

var data = fs.readFileSync(path.join(__dirname, 'test-data', 'BIRD.WMF'));
var WMF;
if (jsEnv.isNode) {
    WMF = require('wmf/dist/wmf.node');
} else {
    WMF = require('wmf/dist/wmf');
}
console.log(typeof data);
var size = WMF.image_size(data);
console.log(size);

im facing this issue as wellim facing this issue as well

Parsing your sample WMF, even with META_PLACEABLE record support, will fail. There is a META_ESCAPE_ENHANCED_METAFILE record with non-standard commentIdentifier: MGXI: created by Micrografx, Inc. Mgxgre v1.0.


Installer does not work on fish shell

Describe the bug
All I know for sure is that after following the install instructions, I cannot execute okta-aws. I've been debugging and I already notice one huge bug, which is that the installer added
export PATH="$HOME/bin:$PATH"

to my config.fish. That syntax is valid in bash, but in fish it should probably be
set -x PATH $HOME/bin $PATH

Furthermore, it didn't even create a /bin folder in my $HOME directory.
To Reproduce
Steps to reproduce the behavior:

Install the fish shell
Execute

curl 'https://raw.githubusercontent.com/oktadeveloper/okta-aws-cli-assume-role/master/bin/install.sh' | bash

Check your config.fish

Good catch. This is an easy fix. The line can be removed. It doesn't make sense anymore since PREFIX  defaults to /usr/local.
withokta lands in /usr/local/bin and okta-aws is created as a shell function.

I don't think it's as simple as that. I cleaned everything up, ran the installer again, removed that line, and tried to execute okta-aws:
okta-aws test sts get-caller-identity
Error: Could not find or load main class com.okta.tools.WithOkta


Dont show diff if no base-lcov-file

In the readme, there this description for the base-lcov-file argument :
Lcov.info file that would be used to compare code coverage. The comparison will be disabled if the file is not passed
But if I don't pass and lcov file, I have the line : Your code coverage diff: NaN% ▴ but it shouldn't be here according to the description

Hello @TanguyRMalou!
Thanks for reporting it, you're right - it shouldn't be there in the first place. Feel free to create a PR once you have a chance.
Meantime, can you post your configuration of action here as well?


set Dropwizard dependency to scope provided

this should improve compatibility with multiple Dropwizard versions. see
also #1.

@yunspace: could you please review this solution?

@tkrille I think this is a good idea, as long as there are no conflicts.
However by adding the provided means anyone without commons-lang 2.6 in their class path will break when using your LevenshteinComparator. Since org.apache.commons.lang.StringUtils doesn't exist in 3.3.
You may need to explicitly declare the commons-lang dependency. So that the correct version of commons-lang that you require will be pulled down, regardless of DW 0.7.1 or 0.8.0.

@yunspace Thanks for the hint about StringUtils. Actually it's not my LevenshteinComparator. In fact, I just copied the whole configuration mechanism from Dropwizard source and augmented it with the environment variable replacement at the right spot.
So afaic, removing the StringUtils, Levenshtein and suggestion stuff would be sufficient too. I don't know if anybody is relying on suggestions for misspelled configuration keys. Do you?

I removed the suggestion stuff. There shouldn't be any problematic dependencies left now.
@yunspace Is this Ok with you?

Actually the suggestion is kinda useful :) I've relied on it a few times when I mistyped a config item out of a long list of items.
Instead of removing functionalities from the Dropwizard's ConfigurationException, it could be re-used simply by moving the EnvironmentConfigurationFactory into io.dropwizard.configuration package.
This way it frees you up from copying and pasting the logic into EnvironmentConfigurationException, and less dependencies to worry about. I've reflected this in #1,

Ok, I will leave the suggestion stuff as it is. But moving my class to a foreign package seems to be not a good idea. After all it's my class and therefore should belong to my own namespace, despite the fact that I just copied about 90% of it from Dropwizard's original implementation :)
So I think, at this point I would rather stick to either depend on commons-lang 2.x or repackage it with the maven-shade-plugin. I really try to not introduce any transitive dependencies, as this plugin should be as unobtrusive as possible, especially freeing you from any worries about dependency hell.
I will update my PR during the next hours. Is there anything you did in #1 that you would like to see here, too? Then please, either update your PR, create another one, or tell me and I will incorporate it here.
BTW: I am currently implementing the successor of this project: https://github.com/tkrille/dropwizard-template-config. It takes the whole idea a step further by allowing you to write a Freemarker template to replace environment variables and system properties (also anywhere inline a value) and do other cool stuff that Freemarker supports. Would be nice to get some feedback, like ideas for more features, stuff that's wrong, etc., from a real user :) Also if you ever switch to the new thing, please let me know.

Ok, I'm done. I've added the dependency to commons-lang 2.6 and also shaded and repackaged it, so it won't show up in the dependency resolution and should not conflict with any commons-lang dependency of one's project.
@yunspace could you please test this? if it's ok, I will do some cleanup and then make a fresh release out of this.

sure thing. A bit busy these couple of days. Will test this out and also have a look at your other project.

@tkrille I've tested this. Please release

thank you

@yunspace version 1.1 has been released and should be available on the central by now.


%taskinfo auto update

It would be nice for %taskinfo to update its output automatically. The technical problem here is that %taskinfo requests output in HTML format from command sos status, so the command has to be executed independent of regular sos status runs.

Too troublesome to implement.


Fixed issues for IPv4 and IPv6 compat for prometheus and applications stats metrics scraping

Fixed issues for IPv4 and IPv6 compat for prometheus and applications stats metrics scraping
Get following error when enable pilot-agent stats metrics scraping:
2022-06-07T03:43:01.067302Z     error   failed scraping envoy metrics: error scraping http://localhost:15090/stats/prometheus: Get "http://localhost:15090/stats/prometheus": dial tcp [::1]:15090: connect: connection refused\
2022-06-07T03:43:14.939616Z     error   failed scraping envoy metrics: error scraping http://localhost:15090/stats/prometheus: Get "http://localhost:15090/stats/prometheus": dial tcp [::1]:15090: connect: connection refused\
2022-06-07T03:43:26.783580Z     info    Agent draining Proxy\
2022-06-07T03:43:26.783582Z     info    Status server has successfully terminated\

This error causes the ingress gateway pod crashed in IPv6-only and dual-stack k8s cluster

/test integ-ipv6_istio

@zhlsunshine: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ipv6_istio
cbe94ee957137019872b89002feb5a6b6a41f678
link
true
/test integ-ipv6_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ipv6_istio
cbe94ee957137019872b89002feb5a6b6a41f678
link
true
/test integ-ipv6_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ipv6_istio
cbe94ee957137019872b89002feb5a6b6a41f678
link
true
/test integ-ipv6_istio

unit-tests_istio
53c6ab12a085437d1619565d6d80de7ae13ed1dc
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
53c6ab12a085437d1619565d6d80de7ae13ed1dc
link
true
/test unit-tests_istio

integ-ipv6_istio
53c6ab12a085437d1619565d6d80de7ae13ed1dc
link
true
/test integ-ipv6_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhlsunshine: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ipv6_istio
995b5b4b7b4aa3519e893fede46bd75323c08fb5
link
true
/test integ-ipv6_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-ipv6_istio


Integration Test Cleanup:

This ticket will track some improvements we can make to the integration tests.
https://github.com/kubernetes-sigs/jobset/pull/246#discussion_r1475205027

it would be great to change updates to steps and also change the struct from update to step.

and we should add a map of jobs to updates so we don't assume order. ReplicatedJobStatus order is enforced by sorting by name but that is a bit janky.

Hey @kannon92, Is this issue still up for grabs? I would love to work on this.

@prashantrewar the issue is still open as far as I am aware. Feel free to claim it.


web ui not working for me

please execute python -m SimpleHTTPServer from web directory, then open http://localhost:8000

If you are on latest commit on master branch, running python -m SimpleHTTPServer is no longer necessary. Simply open http://localhost:8000 after the bot has started.

@widnyana @wchill Bot Path and Showing Trainer just works on localhost. If you use your own local ip like 192.168. ... it doesn't work anymore. Do you have any suggestions?
JS Error:
main.js:149 Uncaught TypeError: Cannot read property 'setOptions' of undefined

Same error even on lokalhost here
updating web did help with the trainer stuff...but it got a lot of other issues. you cant move the map or see anything there


Add option to load high-res (5 minutes) data

Add option to load high-res (5 minutes) data
Add option to smooth total and total_increase sensors if their average value is below sensor precision
Add build instructions and requirements.txt for python dependencies

I like the direction and do appreciate you sorting some of the quality of life stuff I never bothered with so far.
I'll have a look see just to make sure I understand what the code is doing throughout, but I'm generally positive. That said, I have a one year old, so it may be a day or two before I get to it properly.
Two questions right off the bat:

Out of curiosity, what's your use case for five minute data?
Is the smoothing detection based on some standardised algoritm or is it a DIY thing?

I have a one year old

Hehe, two children here... ;-)

Out of curiosity, what's your use case for five minute data?

You can see much more details when using 5 minutes data. For example at this mains frequency heatmap you can see hourly frequency corrections and when analyzing my home's cunsumption you can also see much more details.

Is the smoothing detection based on some standardised algoritm or is it a DIY thing?

A DIY thing :-)

Have been a bit pressed for time, still. Hoping to remedy that RSN. Sorry about that.
Since the smoothing is DIY, would you possibly have a small raw dataset that I could check in? I'm getting to the point where I'd actually need to add some tests, would like to make sure that I understand the impact of any future changes.

Todo list for self:

Figure out what to do with Max/Min values. If you set f.x a Max based on your main fuse, things get weird if you change from one hour resolution to a 5 minute resolution. Could either document or auto scale this somehow.
Consider making the time period a bit more selectable, opening up for more options than 5 mins / 1 hr going forward. Should be an easy fix.
Consider de-repeating date_table_headers() a bit, it may be cleaner to generate the HTML after all.
Move UI elements around, set resolution next to days. Can warn if days > the recorder settings.
Some more thinking around smoothing. May make sense to support different strategies (even if there's only one present) from the get-go. Would entail a dropdown in the UI rather than a checkbox.
Figure out what do with the "Round to the nearest 0.01 to match sensor resolution" assumption for smoothing.

As evident from the history, I pulled it into a branch to do some edits edits for the sake of future extensibility and some minor UX stuff. PR much appreciated, I think it's a good addition. Thank you :-)


Error: stream ended unexpectedly

I am getting this error i don't know from where i am getting this error  and why

Hm, weird. I would love to take a look and figure it out. Can you help provide instruction on how to reproduce so I can take a look? Provide the following, for example:

Version of Node.js
Version of this and all other used modules
Complete server code I can copy and paste and run to see the issue
Instructions on what request (include a copy of the request or code that creates the request) to make to produce the given error.

Thanks!

Same here:
pakages.json
"connect-multiparty": "^2.1.0",
    "express": "^4.16.3",

app.js
const express = require('express')
const app = express()
const multipart = require('connect-multiparty')
const multipartMiddleware = multipart()

app.post('/', multipartMiddleware, (req, res) => {
  console.debug(req.body, req.files)
})

raises the following:
Listening on port 3000

Error: stream ended unexpectedly
    at Form.<anonymous> (/Users/vinyll/Projects/eurordis/receipt-sender/node_modules/multiparty/index.js:754:24)
    at Form.emit (events.js:165:20)
    at finishMaybe (_stream_writable.js:633:14)
    at endWritable (_stream_writable.js:641:3)
    at Form.Writable.end (_stream_writable.js:583:5)
    at IncomingMessage.onend (_stream_readable.js:602:10)
    at Object.onceWrapper (events.js:255:19)
    at IncomingMessage.emit (events.js:165:20)
    at endReadableNT (_stream_readable.js:1101:12)
    at process._tickCallback (internal/process/next_tick.js:152:19)

Seems like it is when re-submitting the _post_ed form refreshing the page.
Works fine when browsing back and reclicking on the submit button.

Hi @vinyll I can definitely take a look. I ran your app, and that error doesn't actually appear. After I copy the app and run it, what are the steps to see the error?

I did try that in Chrome and there wasn't an error. I'm hoping that you can possible some more specific instructions if possible. Maybe the code for how to do the post is different between us? Or the web browser?

Good point. I tried with Chromium and Firefox 56 and I had no error risen; it occurs on Firefox 60.
What I do more precisely to reproduce:

run
submit a file
the server loops as there's no response. Ctrl+C to stop the server
restart the server
refresh the page accepting to post again

At this stage I'm wondering if it isn't a bug with FF… :/

Ah, gotcha. I will work to get Firefox installed soon and try it out 👍

Hi @Akash0333 so I downloaded & installed Firefox 30 and setup my form to do the same thing you described: I have a <form> that makes a POST and then displays "success". I then click on the Reload button and Firefox asks if I want to send the data again. I accept and "success" is displayed again and the form uploaded without an error. Not sure what the difference is between our setups are, though.
Is it possible you can update your above app.js to send an HTML page with a <form> that represents your form? Ideally the less I have to try and make up on the fly, the closer our implementations will get to each other to reproduce the issue.
Also, you're always welcome to submit a fix as a PR as well 👍I want to get this fixed, but I don't (yet) have a way to see the issue to understand what to change.

Hey @dougwilson! Why are you talking about Firefox 30?
I can't see a reference from @Akash0333 about it, but maybe I'm missing some message here.

Sorry that was a typo. I meant Firefox 60 (the current version).

I have the same issue if I don't send any body from post man to any service. What is the wrong ?? and how to fix it ??

Open a new issue with complete code and steps to reproduce the issue so we can figure it out.


[BUG] elementAddDelay should be elementStartDelay in the readme

Duplicate issue

[X] I've searched for any related issues and avoided creating a duplicate issue.

Component
Simulator
Description
Just a minor one, but it was difficult to spot.
The readme documents worker startup config elementAddDelay, but the field is actually called elementStartDelay.
This field is useful to add a delay between each charger connecting, so I don't DoS my ocpp server :)
Version
latest
Node.js version
not relevant
System
not relevant
Expected result
No response
Actual result
No response
Steps to reproduce
No response
Attachments
No response

I do not understand: the tunable was renamed to elementAddDelay with a deprecation warning on elementStartDelay. And the code properly handle elementAddDelay as elementStartDelay.
What is actually not working with the elementAddDelay tunable?

Will be closed soon unless the reporter gives more information on the issue.


Histogram Series overlap and hide smaller values (regression in 3.5.0)

Lightweight Charts Version: 3.5.0
It worked in 3.4.0
Steps/code to reproduce:
Worked in https://codesandbox.io/s/lightweightchartsdemo-forked-5xq2p?file=/src/index.js
Does not work (same code) in https://codesandbox.io/s/lightweightchartsdemo-forked-qlg1u
Actual behavior:
The largest bar series on a given value overlaps the others, and covers up all smaller ones
Expected behavior:
The bars are rendered from largest in the back to smallest in the front
Screenshots:
CodeSandbox/JSFiddle/etc link:
Worked in https://codesandbox.io/s/lightweightchartsdemo-forked-5xq2p?file=/src/index.js
Does not work (same code) in https://codesandbox.io/s/lightweightchartsdemo-forked-qlg1u

@timocov I think this is related to the 3.5.0 release - it works as expected on the one right before it

Duplicate of #812

This is intended behaviour, you need to change the order of creating series to make it work as you want.
Just updated the release notes to avoid further confusing https://github.com/tradingview/lightweight-charts/releases/tag/v3.5.0.

What if the bars alternate in tallest height?
For instance, in series 1 the values go 0 10 5 and in series two they go 0 5 10, and we render series one first.
Series two will completely overlap series 5 in that case.
There needs to be specific logic on each tick render to render them in order, by value, unless I'm misunderstanding that

Actually, what is the difference between the two versions you provided? It seems that they are the same?

Ha, that's because I added a hist.reverse() to the not working link to confirm that it would work :)
I just reset it - it should be back to "not working" - which is, as you said, just the order the series were added

Ha, that's because I added a hist.reverse() to the not working link to confirm that it would work :)

So, what the issue now is? 🙂

OK I just updated to a minimum viable example
https://codesandbox.io/s/lightweightchartsdemo-forked-qlg1u?file=/src/index.js:1877-1913
If you add two series that aren't strictly increasing, one of them will overlap
You can see that in the third datapoint, the blue line overlaps the red line.
The order in which series are added is not guaranteed to be the ordering of values on the series. Histograms will overlap if the tallest one alternates


Improve log handling on server

The current configuration on the server for the logs does not take into account log-rotation:

add logrotate config to paramiko log
convert current crontab-timestamp ...>> /home/ubuntu/transfer-date +%Y%m%d%H%M%S.log 2>&1 in logs to a log to file and adding logrotate config on server-level.

Logs will be pushed to cloudwatch. @TheJenne18 will make sure access rights for @peterdesmet  are provided to the cloudwatch logs and define the retention time to keep the logs.

Fixed by https://github.com/enram/vptstools/pull/45


ign_ros2_control: 0.7.5-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository ign_ros2_control to 0.7.5-1:

upstream repository: https://github.com/ros-controls/gz_ros2_control
release repository: https://github.com/ros2-gbp/ign_ros2_control-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.7.4-1

ign_ros2_control
* Load the URDF to the resource_manager before parsing it to CM (#222 <https://github.com/ros-controls/gz_ros2_control/issues/222>) (#225 <https://github.com/ros-controls/gz_ros2_control/issues/225>)
  Co-authored-by: Alejandro Hernández Cordero<EMAIL_ADDRESS>  (cherry picked from commit 5a948050563881dba20effec3ccb678e4f375529)
  Co-authored-by: Sai Kishor Kothakota<EMAIL_ADDRESS>* Contributors: mergify[bot]

Holding for Humble sync


Resample filter in Volume renderer not handling double-precision coordinates 100%.

There have been different reports of the Splatting volume renderer not making a picture. I've replicated the problem (I think) using this procedure. 1. Export noise.silo's hardyglobal variable to a VTK file.2. Edit the VTK file, changing the coordinates to "double" instead of "float".3. Open the VTK file4. Make a Volume plot of "hardyglobal".5. Draw The Volume plot renderer will not show any output. There will only be an empty bounding box. If you turn off "Sample data onto regular grid" then the plot will appear. This suggests that the resampling code is not handling double precision coordinates 100% or there is a problem in the Splatting renderer.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1126
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Resample filter in Volume renderer not handling double-precision coordinates 100%.
Assigned to: Kathleen Biagas
Category: -
Target version: 2.5.2
Author: Brad Whitlock
Start: 07/09/2012
Due date:
% Done: 0%
Estimated time:
Created: 07/09/2012 12:40 pm
Updated: 07/13/2012 01:20 pm
Likelihood: 3 - Occasional
Severity: 3 - Major Irritation
Found in version: 2.5.1
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
There have been different reports of the Splatting volume renderer not making a picture. I've replicated the problem (I think) using this procedure. 1. Export noise.silo's hardyglobal variable to a VTK file.2. Edit the VTK file, changing the coordinates to "double" instead of "float".3. Open the VTK file4. Make a Volume plot of "hardyglobal".5. Draw The Volume plot renderer will not show any output. There will only be an empty bounding box. If you turn off "Sample data onto regular grid" then the plot will appear. This suggests that the resampling code is not handling double precision coordinates 100% or there is a problem in the Splatting renderer.
Comments:
avtMassVoxelExtractor was assuming float coordinates. Updated to handle double-precision (as well at other data types).SVN Revision 18733 (2.5RC) 18735 (trunk)