After refreshing the model list, the order of the list is reversed

Describe the bug

After refreshing the model list, the order of the list is reversed.  This situation can also occur when paging
Steps to reproduce
https://github.com/user-attachments/assets/0159728e-dda8-420f-9a52-affce14365c4
Expected behavior

After refreshing the list, the most recently created models should be displayed at the front
Environment

GPUStack version: main 74d5e7c

Additional context
Add any other context about the problem here.

Verified on main e4a7bf3


CTRL is in a incorrect location when remapped

Describe the bug:
When you run setxkbmap -option "ctrl:swapcaps" to swap CTRL and CAPS LOCK keys then osu! does not respect this and all shortcuts are mapped to CAPS LOCK + key combo instead.
osu!lazer version: 2021.212.0
Platform: Linux

This will be essentially the same thing as https://github.com/ppy/osu/issues/11143. If the behaviour that seems to be expected here is to be achieved, then default keybindings would have to be unmapped against the user's current keyboard layout (so that the physical Ctrl is always used by the game, regardless of the layout).
I'm not even honestly sure if it's correct to do so - I think the logical key position should be the one that matters (i.e. the one indicated by the keyboard layout), not the physical one. In other words, I think the current game behaviour is more correct.

At least on windows and macOS, my assumption is that any remapped keys would follow their remapped variants (and they do).

This will be essentially the same thing as https://github.com/ppy/osu/issues/11143. If the behaviour that seems to be expected here is to be achieved, then default keybindings would have to be unmapped against the user's current keyboard layout (so that the physical Ctrl is always used by the game, regardless of the layout).

Looks like the same issue to me (just the key in the question isn't ctrl)

I'm not even honestly sure if it's correct to do so - I think the logical key position should be the one that matters (i.e. the one indicated by the keyboard layout), not the physical one. In other words, I think the current game behaviour is more correct.

I am little confused here. The one reported by the mapping should be always used instead of scancodes in my opinion (keycodes > scancodes) so yes, you are right there. The issue is that osu seems to prefer scancodes over keycodes in this case (it thinks that ctrl is in a place of caps because it assumes that ctrl is in left bottom of a keyboard which is wrong). In what way is the current behavior more correct? If there is any misunderstanding please tell me. I might made mistake and wrote something in a way it could confused you.

I've tested with setxkbmap and I'm pretty sure that after swapping I was getting a caps lock scancode after pressing physical Ctrl, and vice versa. Which seems correct, again, unless I'm misunderstanding, which I might be as this is very confusing.

@DavidHusicka can you confirm this behaviour? Maybe with screenshots attaching what you expect the case to be?
To my eyes, correct behaviour would be that osu/osu-framework sees the post-rebound key, as @bdach has implied. So in your osu! settings you would see 'caps lock' for the key which works as 'caps lock' everywhere else on your system.

Is this what you want to see?
https://youtu.be/ZblF0Uw0tgE
The overlay is screenkey

That... doesn't help much? How are we to know that the overlay isn't showing scan codes rather than key codes, or vice versa?
The least confusing thing that you could provide in my opinion is an actual real world video of you pushing the actual real world keys. Alternatively a keyboard overlay that shows the layout, as long as it reflects reality.

The overlay shows the keys the way these keys are mapped so when I press physical caps lock which I has mapped to ctrl then the overlay shows ctrl because these are my mappings and this is how the presses should be recognized.
There is the video of me pressing the keys. I am sorry for the mess on my table.
https://youtu.be/gj2gF9ZkTko

Note: I tried to make the led for caps lock visible so you can clearly see that my physical ctrl is in fact a caps lock.

That video you linked does confirm the behaviour that's been questioned about, osu!lazer looks to receive Key.ControlLeft instead of Key.CapsLock when pressing Ctrl physically, on your end. (assuming you didn't confuse/modify the game key binding configuration into doing that)

Might in the end be a duplicate of https://github.com/ppy/osu/issues/11143 (wrong key received, from scancode instead of keycode, linux-only)

I think this most likely is a duplicate of the issue linked above. It's the same behavior, just with different keys.


Implement Delete in the oci store content implementation

Signed-off-by: viovanov<EMAIL_ADDRESS>

Thanks @shizhMSFT!
Do you know when this will make it into a v1 release?

Thanks @shizhMSFT! Do you know when this will make it into a v1 release?

I think it will happen very soon. We just need a ceremony :D

anything we can do to help for the ceremony @shizhMSFT?
maybe send some cake over? :)


Can't make the theme work...

Summary
So I just started using jet brains suite (IntelliJ, PyCharm), saw that this theme existed. So I install Material-UI 2.8.3. Material-UI works flawless. Than I tried adding this theme.
It gives me this error (Same error with PyCharm) :
Plugin 'io.acari.DDLCTheme' failed to initialize and will be disabled.  Please restart IntelliJ IDEA.
Screenshot : http://prntscr.com/kz6ifa
Do I need another plugin, what I am missing?
Thanks for your time.
Expected behavior
The IDE should not stop working.
Actual behavior
The IDE should work.
Environment
IntelliJ IDEA 2018.2.2 (Community Edition)
Build #IC-182.4129.33, built on August 21, 2018
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0

Nervermind, just disable the Material-UI. Forget to look for closed issue, my bad...
Still, you should probably put this in your readme.

I will put it in the README, considering this has happened quite a few times now :)


unable to connect presto and hive through presto jdbc interpreter

What is this PR for?
when executed a sql query through the zeppelin presto jdbc interpreter in notebook，couldn't get a correct result, there was always sqlException printed in the log even though the completely correct configuration of this inerpreter, hope some others have good iedas and can resolve this bug of the presto jdbc interpreter
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-4601
How should this be tested?
Pass CI
and functional testing
Screenshots (if appropriate)
Questions:

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

@fred521 Could you explain more about how this fix the issue ?

Hi @zjffdu
I think zeppelin jdbc interpreter default 'url' in the properties, but is invalid in the presto jdbc driver
see the [PrestoDriverUri.java] (https://github.com/prestodb/presto/blob/303e640f5da27a8c3785aae2f4129fa2293d062f/presto-jdbc/src/main/java/com/facebook/presto/jdbc/PrestoDriverUri.java#L324)
and ConnectionProperties.java
the same as the hive in the jdbc interpreter.
see the jira exception https://issues.apache.org/jira/browse/ZEPPELIN-4601


Does not work properly with input type="radio"

I just tried to implement initial-value for radio buttons, which unfortunately does not work the way I understand it. My approach was the following: http://codepen.io/anon/pen/OyqXGw
The problem is that using checked="checked" and initial-value attributes together works, but only for the one radio button having them assigned, and only if it is the only radio button having it.
This works, for the first one only:
<input
  initial-value
  checked="checked"
  type="radio"
  name="foo"
  value="1"
  ng-model="name"  /> one

<input
  type="radio"
 name="foo"
 value="2"
 ng-model="name"  /> two

But as e.g. the second radio button should be used as initial-value if it is checked, I thought I just continue and also add "initial-value" to the second radio:
<input
  initial-value
  checked="checked"
  type="radio"
  name="foo"
  value="1"
  ng-model="name"  /> one

<input
  initial-value
  type="radio"
 name="foo"
 value="2"
 ng-model="name"  /> two

This breaks the whole intial-value directive and no initial value is set for the model

Found the problem on https://github.com/glaucocustodio/angular-initial-value/blob/master/src/angular-initial-value.js#L50
On L47 you check if there is an ngModel assigned, and then on L50, you assign either values (for the select) or val for other things. This however won't work, as this assigns undefined for radios that should not have a value set (which is why it only works on the last radio button; there, the correct value is set. If the second should be set, the third overrides it with undefined).
IMHO, there shouldn't either be variables val and values as I can imagine the unclear naming being a possible source of the problem.
Replacing L47 with if($attrs.ngModel && (val || (values != undefined && values.length))){  solves the problem, but I deem this not as a clean solution as it smells (code repetition).

Sorry for delay. Why do you need to have two radio buttons (in the same group) selected?

I don't have to have two selected at the same time. I need to have the one checked which has initial-value set, but that does not work for radio buttons. As described above: If I have two radio buttons and the second one is checked, the initial-value fails.

I got it. Thanks for the feedback, I've put your fix for while. See https://github.com/glaucocustodio/angular-initial-value/commit/b92b4c8501781cb1db16d56843dfb52b8cf8db08.


Nextjs sst deploy fails (0.0.131) due to favicon format.

After fixing the same error as #22 by excluding sst.config.ts in tsconfig, deploy resulted in a different error. I've upgraded to 0.0.131.

simple static site, next@14
sst dev works locally
nextjs build/linting works
But sst deploy --stage=test fails.

When I directly run npx --yes<EMAIL_ADDRESS>build, the output is:
Bundling middleware function...
Bundling static assets...
Bundling cache assets...
node:fs:456
    return binding.readFileUtf8(path, stringToFlags(options.flag));
                   ^

Error: ENOENT: no such file or directory, open '/Users/bebbi/code/landing/.next/standalone/.next/server/app/icon/route.js.nft.json'
    at readFileSync (node:fs:456:20)
    at computeCopyFilesForPage (file:///Users/bebbi/.npm/_npx/243dd04f789c84f7/node_modules/open-next/dist/build/copyTracedFiles.js:39:42)

Hi @bebbi, thanks for digging into this. It seems to be an issue with OpenNext. Can you open an issue here https://github.com/sst/open-next/issues?

Maybe this open-next issue


Weapons counting druid only stats in green calculation

This is actually because we try to compensate for the weapon DPS by adding 14 AP to the EP calculation for each weapon DPS. It's a bit hacky, but WAI


XAML on .net core

Hi,
Thanks a lot on the huge work done in .Net Core 1.0, and its features.
It would be great to have a XAML to rule them all (to rule the desktop of all OSs), that benefits from any GPU/CPU on that system, and to be a the defecto desktop UI development, that is more powerful than both old .Net XAML, and Xamarin XAML, and that draws its own shapes, not a layer above current UI frameworks in each platform.

It would be good to have really xplat XAML based UI for .NET Core, but Xamarin is definitely the way to go in xplat UI development for .NET.
However, all MS's UI xplat products for now are Javascript based: Visual Studio Code, Skype for Linux etc.
Maybe they don't believe in XAML?

There are a couple projects to look at:

Xamarin.Forms, also raised above.
AvaloniaUI

Maybe they don't believe in XAML?

I don't think that this is quite right. There is no xplat Xaml UI stack for those teams to use. As one of the projects above supports both mobile and desktop, then that may change.
Closing, since the .NET Core team doesn't have any plans for a x-plat XAML UI stack. Best to look at those other projects.


Update module github.com/urfave/cli to v2

This PR contains the following updates:

Package
Type
Update
Change

github.com/urfave/cli
require
major
v1.22.15 -> v2.27.5

Release Notes

urfave/cli (github.com/urfave/cli)
v2.27.5
Compare Source
What's Changed

docs(flag): add UseShortOptionHandling description by @BlackHole1 in https://github.com/urfave/cli/pull/1956
[Backport] Fix: Use $0 env var to correctly retrieve the current active shell by @asahasrabuddhe in https://github.com/urfave/cli/pull/1970
Update dependencies in v2-maint by @meatballhat in https://github.com/urfave/cli/pull/1980

Full Changelog: https://github.com/urfave/cli/compare/v2.27.4...v2.27.5
v2.27.4
Compare Source
What's Changed

Fix:(issue_1962) Fix tests failing on 32 bit architectures by @dearchap in https://github.com/urfave/cli/pull/1963

Full Changelog: https://github.com/urfave/cli/compare/v2.27.3...v2.27.4
v2.27.3
Compare Source
What's Changed

v2 Docs: Mention value from env as default value by @sj14 in https://github.com/urfave/cli/pull/1910
Bump github.com/xrash/smetrics dependency by @elezar in https://github.com/urfave/cli/pull/1911
fix: disable bash completion if double dash is included in arguments (v2) by @suzuki-shunsuke in https://github.com/urfave/cli/pull/1938
Fix improper whitespace formatting in usageTemplate, AppHelpTemplate … by @caeret in https://github.com/urfave/cli/pull/1947

New Contributors

@sj14 made their first contribution in https://github.com/urfave/cli/pull/1910
@elezar made their first contribution in https://github.com/urfave/cli/pull/1911
@caeret made their first contribution in https://github.com/urfave/cli/pull/1947

Full Changelog: https://github.com/urfave/cli/compare/v2.27.2...v2.27.3
v2.27.2
Compare Source
What's Changed

Cleanup: Remove unnecessary intermediate variables by @1ambd4 in https://github.com/urfave/cli/pull/1857
Docs:(issue_1866) Fix documentation on filepath vs env preference by @dearchap in https://github.com/urfave/cli/pull/1867
Fix:(issue_1860) Remove hidden flags from flag categories by @dearchap in https://github.com/urfave/cli/pull/1868
Fix repeated [arguments...] in usage template in v2 by @edeustua in https://github.com/urfave/cli/pull/1872
Update dependencies, actions steps, and usage for v2-maint by @meatballhat in https://github.com/urfave/cli/pull/1888

New Contributors

@1ambd4 made their first contribution in https://github.com/urfave/cli/pull/1857
@edeustua made their first contribution in https://github.com/urfave/cli/pull/1872

Full Changelog: https://github.com/urfave/cli/compare/v2.27.1...v2.27.2
v2.27.1
Compare Source
What's Changed

v2: Add build tag urfave_cli_no_suggest by @dolmen in https://github.com/urfave/cli/pull/1847

Full Changelog: https://github.com/urfave/cli/compare/v2.27.0...v2.27.1
v2.27.0
Compare Source
What's Changed

v2 Add integration with golangci-lint by @skelouse in https://github.com/urfave/cli/pull/1830
v2: GitHub Actions: upgrade Go, upgrade actions by @dolmen in https://github.com/urfave/cli/pull/1848
Feat:(issue_1797) Add Args for app/cmd/subcmd to avoid argument... be… by @dearchap in https://github.com/urfave/cli/pull/1829
Fix:(issue_1850) Add RunAction for uint/uint64 slice flags by @dearchap in https://github.com/urfave/cli/pull/1851

Full Changelog: https://github.com/urfave/cli/compare/v2.26.0...v2.27.0
v2.26.0
Compare Source
What's Changed

Bash completion nits by @meatballhat in https://github.com/urfave/cli/pull/1762
Chore: Rename mkdocs requirements file name by @dearchap in https://github.com/urfave/cli/pull/1776
Fix:(issue_1787) Add fix for commands not listed when hide help comma… by @dearchap in https://github.com/urfave/cli/pull/1788
Fix nil HelpFlag panic (v2) by @wxiaoguang in https://github.com/urfave/cli/pull/1795
Always get 0 for a nested int64 value in v2.25.7 by @stephenfire in https://github.com/urfave/cli/pull/1799
Helper messages for documenting build process by @abitrolly in https://github.com/urfave/cli/pull/1800
fix: check duplicated sub command name and alias by @linrl3 in https://github.com/urfave/cli/pull/1805
Fix:(issue_1689) Have consistent behavior for default text in man and… by @dearchap in https://github.com/urfave/cli/pull/1825
Fix linting issues by @skelouse in https://github.com/urfave/cli/pull/1696

New Contributors

@stephenfire made their first contribution in https://github.com/urfave/cli/pull/1799
@linrl3 made their first contribution in https://github.com/urfave/cli/pull/1805

Full Changelog: https://github.com/urfave/cli/compare/v2.25.7...v2.26.0
v2.25.7
Compare Source
What's Changed

Fix: fix v2 broken tests by @dearchap in https://github.com/urfave/cli/pull/1757
Fix:(issue_1755) Ensure that timestamp flag destination is set correctly by @dearchap in https://github.com/urfave/cli/pull/1756

Full Changelog: https://github.com/urfave/cli/compare/v2.25.6...v2.25.7
v2.25.6
Compare Source
What's Changed

Fix:(issue_1668) Add test case for sub command of sub command completion by @dearchap in https://github.com/urfave/cli/pull/1747
Update dependencies for v2 by @meatballhat in https://github.com/urfave/cli/pull/1749
Document slice flags as part of examples (v2) by @carhartl in https://github.com/urfave/cli/pull/1751

Full Changelog: https://github.com/urfave/cli/compare/v2.25.5...v2.25.6
v2.25.5
Compare Source
What's Changed

Fix:(issue_1737) Set bool count by taking care of num of aliases by @dearchap in https://github.com/urfave/cli/pull/1740

Full Changelog: https://github.com/urfave/cli/compare/v2.25.4...v2.25.5
v2.25.4
Compare Source
What's Changed

Bug/fix issue 1703 by @jojje in https://github.com/urfave/cli/pull/1728
Fix:(issue_1734) Show categories for subcommands by @dearchap in https://github.com/urfave/cli/pull/1735
Fix:(issue_1610). Keep RunAsSubcommand behaviour as before by @dearchap in https://github.com/urfave/cli/pull/1736
Fix:(issue_1731) Add fix for checking if aliases are set by @dearchap in https://github.com/urfave/cli/pull/1732
Fix func name referenced in doc comment by @meatballhat in https://github.com/urfave/cli/pull/1738

New Contributors

@jojje made their first contribution in https://github.com/urfave/cli/pull/1728

Full Changelog: https://github.com/urfave/cli/compare/v2.25.3...v2.25.4
v2.25.3
Compare Source
What's Changed

Fix incorrectTypeForFlagError for unknowns by @danhunsaker in https://github.com/urfave/cli/pull/1708

New Contributors

@danhunsaker made their first contribution in https://github.com/urfave/cli/pull/1708

Full Changelog: https://github.com/urfave/cli/compare/v2.25.2...v2.25.3
v2.25.2
Compare Source
What's Changed

Fix missing required flag error uses flag name and not alias by @nirhaas in https://github.com/urfave/cli/pull/1709
Remove redundant variable declarations by @huiyifyj in https://github.com/urfave/cli/pull/1714
Fix:(issue 1689) Match markdown output with help output by @dearchap in https://github.com/urfave/cli/pull/1723

New Contributors

@nirhaas made their first contribution in https://github.com/urfave/cli/pull/1709
@huiyifyj made their first contribution in https://github.com/urfave/cli/pull/1714

Full Changelog: https://github.com/urfave/cli/compare/v2.25.1...v2.25.2
v2.25.1
Compare Source
What's Changed

Shift tested Go versions in v2-maint by @meatballhat in https://github.com/urfave/cli/pull/1692
Preserve separator spec on subcommands by @thschmitt in https://github.com/urfave/cli/pull/1710

New Contributors

@thschmitt made their first contribution in https://github.com/urfave/cli/pull/1710

Full Changelog: https://github.com/urfave/cli/compare/v2.25.0...v2.25.1
v2.25.0
Compare Source
What's Changed

Drop support for Go versions before 1.18 by @meatballhat in https://github.com/urfave/cli/pull/1667
add Integer type casting when loading yaml file by @kjeom in https://github.com/urfave/cli/pull/1669

New Contributors

@kjeom made their first contribution in https://github.com/urfave/cli/pull/1669

Full Changelog: https://github.com/urfave/cli/compare/v2.24.4...v2.25.0
v2.24.4
Compare Source
What's Changed

Deprecation of package ioutil in Go 1.16 by @ArangoGutierrez in https://github.com/urfave/cli/pull/1678
Backport [v2]:  Fix some issues in bash autocompletion by @MrNaif2018 in https://github.com/urfave/cli/pull/1676

New Contributors

@ArangoGutierrez made their first contribution in https://github.com/urfave/cli/pull/1678

Full Changelog: https://github.com/urfave/cli/compare/v2.24.3...v2.24.4
v2.24.3
Compare Source
What's Changed

Make trim space optional by @palsivertsen in https://github.com/urfave/cli/pull/1675

Full Changelog: https://github.com/urfave/cli/compare/v2.24.2...v2.24.3
v2.24.2
Compare Source
What's Changed

Update README badges for v2 by @meatballhat in https://github.com/urfave/cli/pull/1664
Target two most recent Go versions in v2 by @meatballhat in https://github.com/urfave/cli/pull/1666
flag: remove dependencies on shared variables by @zllovesuki in https://github.com/urfave/cli/pull/1671
Show non categorized flags with categorized on help by @skelouse in https://github.com/urfave/cli/pull/1673

New Contributors

@zllovesuki made their first contribution in https://github.com/urfave/cli/pull/1671

Full Changelog: https://github.com/urfave/cli/compare/v2.24.1...v2.24.2
v2.24.1
Compare Source
What's Changed

Fix v2 docs by @dearchap in https://github.com/urfave/cli/pull/1655

Full Changelog: https://github.com/urfave/cli/compare/v2.24.0...v2.24.1
v2.24.0
Compare Source
What's Changed

Fix:(issue_1592) Add support for float64slice, uint, int, int64 for a… by @dearchap in https://github.com/urfave/cli/pull/1647

Full Changelog: https://github.com/urfave/cli/compare/v2.23.10...v2.24.0
v2.23.10
Compare Source
What's Changed

Doc:(issue_1593) Add flag category topic in docs (v2-maint) by @ovcharenko-di in https://github.com/urfave/cli/pull/1653

Full Changelog: https://github.com/urfave/cli/compare/v2.23.9...v2.23.10
v2.23.9
Compare Source
What's Changed

Fixes #1648 by @palsivertsen in https://github.com/urfave/cli/pull/1649

New Contributors

@palsivertsen made their first contribution in https://github.com/urfave/cli/pull/1649

Full Changelog: https://github.com/urfave/cli/compare/v2.23.8...v2.23.9
v2.23.8
Compare Source
What's Changed

Fix:(issue_1277) Remove default text for version/help flags by @dearchap in https://github.com/urfave/cli/pull/1633

Full Changelog: https://github.com/urfave/cli/compare/v2.23.7...v2.23.8
v2.23.7
Compare Source
What's Changed

Fix:(issue_1617) Fix Bash completion for subcommands by @dearchap in https://github.com/urfave/cli/pull/1618

Full Changelog: https://github.com/urfave/cli/compare/v2.23.6...v2.23.7
v2.23.6
Compare Source
What's Changed

Disable slice flag separator by @feedmeapples in https://github.com/urfave/cli/pull/1588
Fix:(issue_1591) Use AppHelpTemplate instead of subcommand help template by @dearchap in https://github.com/urfave/cli/pull/1608

Full Changelog: https://github.com/urfave/cli/compare/v2.23.5...v2.23.6
v2.23.5
Compare Source
What's Changed

Update x/text to 0.3.8 by @dirkmueller in https://github.com/urfave/cli/pull/1571
Update github actions events for v2-maint branch by @meatballhat in https://github.com/urfave/cli/pull/1574
Update dependencies in v2 series by @meatballhat in https://github.com/urfave/cli/pull/1573

New Contributors

@dirkmueller made their first contribution in https://github.com/urfave/cli/pull/1571

Full Changelog: https://github.com/urfave/cli/compare/v2.23.4...v2.23.5
v2.23.4
Compare Source
What's Changed

Chore: Add altsrc flag definition generation by @dearchap in https://github.com/urfave/cli/pull/1553

Full Changelog: https://github.com/urfave/cli/compare/v2.23.3...v2.23.4
v2.23.3
Compare Source
What's Changed

Fix: Avoid duplication of help commands by @remiposo in https://github.com/urfave/cli/pull/1565
Feature:(issue_1550):Add support Int64Slice by read toml config file by @Edelweiss-Snow in https://github.com/urfave/cli/pull/1551

New Contributors

@remiposo made their first contribution in https://github.com/urfave/cli/pull/1565
@Edelweiss-Snow made their first contribution in https://github.com/urfave/cli/pull/1551

Full Changelog: https://github.com/urfave/cli/compare/v2.23.2...v2.23.3
Note. This is considered a minor release even though it has a new "feature" i.e support for int64slice for alstrc flags. The int64slice is verbatim copy of existing code and doesnt include any new behaviour compared to other altsrc flags.
v2.23.2
Compare Source
What's Changed

Fix:(issue_1114) Add SkipFlagParsing to app to allow -- by @dearchap in https://github.com/urfave/cli/pull/1562

Full Changelog: https://github.com/urfave/cli/compare/v2.23.1...v2.23.2
v2.23.1
Compare Source
What's Changed

FIx: Allow ext flags to be opt-out by default rather than opt-in by @dearchap in https://github.com/urfave/cli/pull/1561

Full Changelog: https://github.com/urfave/cli/compare/v2.23.0...v2.23.1
v2.23.0
Compare Source
What's Changed

Feature:(issue_269) Allow external package flag definitions by @dearchap in https://github.com/urfave/cli/pull/1540

Full Changelog: https://github.com/urfave/cli/compare/v2.22.0...v2.23.0
v2.22.0
Compare Source
What's Changed

Feature:(issue_1090): Add unwrap for ExitCoder by @dearchap in https://github.com/urfave/cli/pull/1545

Full Changelog: https://github.com/urfave/cli/compare/v2.21.0...v2.22.0
v2.21.0
Compare Source
What's Changed

Feature: (issue_1451) customized slice flag separator by @FGYFFFF in https://github.com/urfave/cli/pull/1546

Full Changelog: https://github.com/urfave/cli/compare/v2.20.5...v2.21.0
v2.20.5
Compare Source
What's Changed

Fix:(issue_1548) Check root before run default cmd by @smalnote in https://github.com/urfave/cli/pull/1549

New Contributors

@smalnote made their first contribution in https://github.com/urfave/cli/pull/1549

Full Changelog: https://github.com/urfave/cli/compare/v2.20.4...v2.20.5
v2.20.4
Compare Source
What's Changed

Spelling by @jsoref in https://github.com/urfave/cli/pull/1543
Documentation:(issue_786) Add docs for flag validation by @dearchap in https://github.com/urfave/cli/pull/1544

New Contributors

@jsoref made their first contribution in https://github.com/urfave/cli/pull/1543

Full Changelog: https://github.com/urfave/cli/compare/v2.20.3...v2.20.4
v2.20.3
Compare Source
What's Changed

Docs:(issue_1276) Make destination/value/default text clear by @dearchap in https://github.com/urfave/cli/pull/1539
Fix:(issue_1263) FlagNames should return names set via env as well by @dearchap in https://github.com/urfave/cli/pull/1537
Fix:(issue_1288). Add windows specific env format by @dearchap in https://github.com/urfave/cli/pull/1538

Full Changelog: https://github.com/urfave/cli/compare/v2.20.2...v2.20.3
v2.20.2
Compare Source
What's Changed

Fix:(issue_1206) Default value shouldnt depend on env variable or val… by @dearchap in https://github.com/urfave/cli/pull/1528

Full Changelog: https://github.com/urfave/cli/compare/v2.20.1...v2.20.2
v2.20.1
Compare Source
What's Changed

Docs: Fix docs from merged PR1498 by @dearchap in https://github.com/urfave/cli/pull/1532
Cleanup: Fix app.go formatting by @dearchap in https://github.com/urfave/cli/pull/1533

Full Changelog: https://github.com/urfave/cli/compare/v2.20.0...v2.20.1
v2.20.0
Compare Source
What's Changed

Cleanup redundant code in App/Command by @dearchap in https://github.com/urfave/cli/pull/1498

Full Changelog: https://github.com/urfave/cli/compare/v2.19.3...v2.20.0
v2.19.3
Compare Source
What's Changed

Build and run urfave-cli-genflags via its Makefile by @meatballhat in https://github.com/urfave/cli/pull/1526

Full Changelog: https://github.com/urfave/cli/compare/v2.19.2...v2.19.3
v2.19.2
Compare Source
What's Changed

fix: stop automatic sorting for --help by @FGYFFFF in https://github.com/urfave/cli/pull/1430

New Contributors

@FGYFFFF made their first contribution in https://github.com/urfave/cli/pull/1430

Full Changelog: https://github.com/urfave/cli/compare/v2.19.1...v2.19.2
v2.19.1
Compare Source
What's Changed

Fix:(issue_1500). Fix slice flag value duplication issue by @dearchap in https://github.com/urfave/cli/pull/1502

Full Changelog: https://github.com/urfave/cli/compare/v2.19.0...v2.19.1
v2.19.0
Compare Source
What's Changed

Fix:(issue_1505) Fix flag alignment in help by @dearchap in https://github.com/urfave/cli/pull/1506

Full Changelog: https://github.com/urfave/cli/compare/v2.18.2...v2.19.0
v2.18.2
Compare Source
What's Changed

Configure GenericFlag's Destination type as struct not pointer by @nkuba in https://github.com/urfave/cli/pull/1442

New Contributors

@nkuba made their first contribution in https://github.com/urfave/cli/pull/1442

Full Changelog: https://github.com/urfave/cli/compare/v2.18.1...v2.18.2
v2.18.1
Compare Source
What's Changed

Ensure "generate" step runs in CI prior to diff check by @meatballhat in https://github.com/urfave/cli/pull/1504

Full Changelog: https://github.com/urfave/cli/compare/v2.18.0...v2.18.1
v2.18.0
Compare Source
What's Changed

Call FlagStringer in String() method of slice flags by @fjl in https://github.com/urfave/cli/pull/1508

New Contributors

@fjl made their first contribution in https://github.com/urfave/cli/pull/1508

Full Changelog: https://github.com/urfave/cli/compare/v2.17.2...v2.18.0
v2.17.2
Compare Source
What's Changed

Remove nonexistent phony targets by @meatballhat in https://github.com/urfave/cli/pull/1503
wrap: Avoid trailing whitespace for empty lines by @abitrolly in https://github.com/urfave/cli/pull/1513

New Contributors

@abitrolly made their first contribution in https://github.com/urfave/cli/pull/1513

Full Changelog: https://github.com/urfave/cli/compare/v2.17.1...v2.17.2
v2.17.1
Compare Source
What's Changed

Fix help results inconsistency by @dearchap in https://github.com/urfave/cli/pull/1499

Full Changelog: https://github.com/urfave/cli/compare/v2.17.0...v2.17.1
v2.17.0
Compare Source
What's Changed

Flag-level Action by @xwjdsh in https://github.com/urfave/cli/pull/1337

New Contributors

@xwjdsh made their first contribution in https://github.com/urfave/cli/pull/1337

Full Changelog: https://github.com/urfave/cli/compare/v2.16.6...v2.17.0
v2.16.6
Compare Source
What's Changed

fix: Context.Set no such flag by @Torwang1 in https://github.com/urfave/cli/pull/1497

New Contributors

@Torwang1 made their first contribution in https://github.com/urfave/cli/pull/1497

Full Changelog: https://github.com/urfave/cli/compare/v2.16.5...v2.16.6
v2.16.5
Compare Source
What's Changed

Fix:(issue_1197) Set destination field from altsrc for slice flags by @dearchap in https://github.com/urfave/cli/pull/1495

Full Changelog: https://github.com/urfave/cli/compare/v2.16.4...v2.16.5
v2.16.4
Compare Source
What's Changed

Accept the MKDOCS_REMOTE_GITHUB_TOKEN var as intended by @meatballhat in https://github.com/urfave/cli/pull/1493

Full Changelog: https://github.com/urfave/cli/compare/v2.16.3...v2.16.4
v2.16.3
Compare Source
What's Changed

Move more functionality into internal/build/build.go by @meatballhat in https://github.com/urfave/cli/pull/1490

Full Changelog: https://github.com/urfave/cli/compare/v2.16.2...v2.16.3
v2.16.2
Compare Source
What's Changed

Fix: Help name consistency among app/commands and subcommands by @dearchap in https://github.com/urfave/cli/pull/1489

Full Changelog: https://github.com/urfave/cli/compare/v2.16.1...v2.16.2
v2.16.1
Compare Source
What's Changed

Set codecov status thresholds to 5% by @meatballhat in https://github.com/urfave/cli/pull/1486
Fix: dont generate pointer for dest for Generic flag by @dearchap in https://github.com/urfave/cli/pull/1488

Full Changelog: https://github.com/urfave/cli/compare/v2.16.0...v2.16.1
v2.16.0
Compare Source
What's Changed

Feature:(Issue 1334) Add support for uint64slices by @dearchap in https://github.com/urfave/cli/pull/1378

Full Changelog: https://github.com/urfave/cli/compare/v2.15.0...v2.16.0
v2.15.0
Compare Source
What's Changed

Add count option for bool flags by @dearchap in https://github.com/urfave/cli/pull/1257

Full Changelog: https://github.com/urfave/cli/compare/v2.14.2...v2.15.0
v2.14.2
Compare Source
What's Changed

Update version of codecov action by @meatballhat in https://github.com/urfave/cli/pull/1480
Add a few notes to release docs by @meatballhat in https://github.com/urfave/cli/pull/1479

Full Changelog: https://github.com/urfave/cli/compare/v2.14.1...v2.14.2
v2.14.1
Compare Source
What's Changed

Fix:(issue_1272) Generic flag not set from env by @dearchap in https://github.com/urfave/cli/pull/1458
GitHub Workflows security hardening by @sashashura in https://github.com/urfave/cli/pull/1469

New Contributors

@sashashura made their first contribution in https://github.com/urfave/cli/pull/1469

Full Changelog: https://github.com/urfave/cli/compare/v2.14.0...v2.14.1
v2.14.0
Compare Source
What's Changed

Update v2 dead link by @abousselmi in https://github.com/urfave/cli/pull/1466
Fix:(issue_1293) Wrap usage text for commands by @dearchap in https://github.com/urfave/cli/pull/1460
Add App.InvalidFlagAccessHandler by @icholy in https://github.com/urfave/cli/pull/1446

New Contributors

@abousselmi made their first contribution in https://github.com/urfave/cli/pull/1466
@icholy made their first contribution in https://github.com/urfave/cli/pull/1446

Full Changelog: https://github.com/urfave/cli/compare/v2.13.0...v2.14.0
v2.13.0
Compare Source
What's Changed

Fix:(issue_1094) Dont execute Before/After handlers during shell comp… by @dearchap in https://github.com/urfave/cli/pull/1457
Fix:(issue_1455) Allow bool flags from input altsrc by @dearchap in https://github.com/urfave/cli/pull/1456
Add configurable Base to int, uint and uint64 flags by @ccremer in https://github.com/urfave/cli/pull/1464
fix: allow required flag with one character by @Gerrard-YNWA in https://github.com/urfave/cli/pull/1449
Add support for alias in YAMLs by @james-prysm in https://github.com/urfave/cli/pull/1448

New Contributors

@ccremer made their first contribution in https://github.com/urfave/cli/pull/1464
@Gerrard-YNWA made their first contribution in https://github.com/urfave/cli/pull/1449
@james-prysm made their first contribution in https://github.com/urfave/cli/pull/1448

Full Changelog: https://github.com/urfave/cli/compare/v2.12.3...v2.13.0
v2.12.3
Compare Source
What's Changed

issue_62: Make slice options more posix like by @dearchap in https://github.com/urfave/cli/pull/1454
Move genflags tool to cmd/ and pin to previous release by @meatballhat in https://github.com/urfave/cli/pull/1451
Shift supported go versions by @meatballhat in https://github.com/urfave/cli/pull/1452
docs: split documentation into individual pages by @hay-kot in https://github.com/urfave/cli/pull/1453

New Contributors

@hay-kot made their first contribution in https://github.com/urfave/cli/pull/1453

Full Changelog: https://github.com/urfave/cli/compare/v2.12.1...v2.12.3
v2.12.2
Compare Source
What's Changed

issue_62: Make slice options more posix like by @dearchap in https://github.com/urfave/cli/pull/1454
Move genflags tool to cmd/ and pin to previous release by @meatballhat in https://github.com/urfave/cli/pull/1451
Shift supported go versions by @meatballhat in https://github.com/urfave/cli/pull/1452
docs: split documentation into individual pages by @hay-kot in https://github.com/urfave/cli/pull/1453

New Contributors

@hay-kot made their first contribution in https://github.com/urfave/cli/pull/1453

Full Changelog: https://github.com/urfave/cli/compare/v2.12.1...v2.12.2
v2.12.1
Compare Source
Full Changelog: https://github.com/urfave/cli/compare/v2.11.5...v2.12.1
v2.12.0
Compare Source
Full Changelog: https://github.com/urfave/cli/compare/v2.11.2...v2.12.0
v2.11.5
Compare Source
Full Changelog: https://github.com/urfave/cli/compare/v2.11.2...v2.11.5
v2.11.4
Compare Source
Full Changelog: https://github.com/urfave/cli/compare/v2.11.2...v2.11.4
v2.11.3
Compare Source
Full Changelog: https://github.com/urfave/cli/compare/v2.11.2...v2.11.3
v2.11.2
Compare Source
What's Changed

Fix HideHelp by @Dokiys in https://github.com/urfave/cli/pull/1445
Fix After not run by @Dokiys in https://github.com/urfave/cli/pull/1444

New Contributors

@Dokiys made their first contribution in https://github.com/urfave/cli/pull/1445

Full Changelog: https://github.com/urfave/cli/compare/v2.11.1...v2.11.2
v2.11.1
Compare Source
What's Changed

Fix timestamp flag error when using help by @hmiyado in https://github.com/urfave/cli/pull/1439

New Contributors

@hmiyado made their first contribution in https://github.com/urfave/cli/pull/1439

Full Changelog: https://github.com/urfave/cli/compare/v2.11.0...v2.11.1
v2.11.0
Compare Source
What's Changed

feature: add DefaultCommand field to App by @jalavosus in https://github.com/urfave/cli/pull/1388
accept timezone for timestamps by @julian7 in https://github.com/urfave/cli/pull/1432
Approve v2 addition of App.DefaultCommand by @meatballhat in https://github.com/urfave/cli/pull/1426

New Contributors

@jalavosus made their first contribution in https://github.com/urfave/cli/pull/1388
@julian7 made their first contribution in https://github.com/urfave/cli/pull/1432

Full Changelog: https://github.com/urfave/cli/compare/v2.10.3...v2.11.0
v2.10.3
Compare Source
What's Changed

Clean up the v2 manual a bit by @meatballhat in https://github.com/urfave/cli/pull/1421
Fix list formatting in v2 manual by @meatballhat in https://github.com/urfave/cli/pull/1423

Full Changelog: https://github.com/urfave/cli/compare/v2.10.2...v2.10.3
v2.10.2
Compare Source
What's Changed

Update dependencies to latest by @meatballhat in https://github.com/urfave/cli/pull/1417
Remove temporary go mod workaround by @meatballhat in https://github.com/urfave/cli/pull/1418

Full Changelog: https://github.com/urfave/cli/compare/v2.10.1...v2.10.2
v2.10.1
Compare Source
What's Changed

Upgrade gopkg.in/yaml to v3 by @jszwedko in https://github.com/urfave/cli/pull/1415

Full Changelog: https://github.com/urfave/cli/compare/v2.10.0...v2.10.1
v2.10.0
Compare Source
What's Changed

Add word-wrap support, with wrap length provided by the user by @mostynb in https://github.com/urfave/cli/pull/1119

Full Changelog: https://github.com/urfave/cli/compare/v2.9.0...v2.10.0
v2.9.0
Compare Source
What's Changed

Fix 'repetetive' typo by @mostynb in https://github.com/urfave/cli/pull/1402
Add some missing go.sum entries needed with 1.16.x by @meatballhat in https://github.com/urfave/cli/pull/1407
Add SliceFlag wrapper and fix bugs in existing implementations by @joeycumines in https://github.com/urfave/cli/pull/1409
Update the v2 docs since merging #1409 by @meatballhat in https://github.com/urfave/cli/pull/1412

New Contributors

@joeycumines made their first contribution in https://github.com/urfave/cli/pull/1409

Full Changelog: https://github.com/urfave/cli/compare/v2.8.1...v2.9.0
v2.8.1
Compare Source
What's Changed

Run docs tests against current work tree by @meatballhat in https://github.com/urfave/cli/pull/1403
Remove GPL2 dependency introduced in v2.7.0 by @AkihiroSuda in https://github.com/urfave/cli/pull/1405

Full Changelog: https://github.com/urfave/cli/compare/v2.8.0...v2.8.1
v2.8.0
Compare Source
What's Changed

Add flag category support (#796) by @michaeljs1990 / @meatballhat in https://github.com/urfave/cli/pull/1368
Introduce override hooks for suggestions by @meatballhat in https://github.com/urfave/cli/pull/1396
Point to the docs tree and site from top-level README by @meatballhat in https://github.com/urfave/cli/pull/1399

Full Changelog: https://github.com/urfave/cli/compare/v2.7.2...v2.8.0
v2.7.2
Compare Source
What's Changed

Auto-detect zsh via SHELL (#1218) by @z0rc / @meatballhat in [https://github.com/urfave/cli/pull/1350](https://redirect.github.com/urfave/cli/pull/1350

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/cpuguy83/go-md2man/v2
v2.0.4 -> v2.0.5


Permit MMIO exits to bypass the emulation.

It appears to be impossible for a program to handle a mmio exit in userspace rather than round tripping to kernel every time.  If the registers are reloaded after HAX_EXIT_FAST_MMIO they will likely then be corrupted in em_emulate_insn.  I also tried to use HAX_EPT_PERM_NONE but it seems to not support small memory regions.

I guess what you want is something like the macOS Hypervisor.framework API, which allows user space to access most VMCS registers, so the QEMU HVF accelerator (which runs in user space) can fetch the MMIO instruction, emulate it, and then resume the guest from the next instruction.
The HAXM API provides a higher-level abstraction by design, which is why HAX_EXIT_FAST_MMIO isn't suitable for your purpose (it assumes most of the instruction emulation work is done in the kernel).
HAX_EXIT_PAGEFAULT (assuming the MMIO region can be protected with HAX_RAM_PERM_NONE) might be a better alternative. However, right now the GPA protection mechanism is limited to non-MMIO regions, and even if it worked on MMIO regions, you would still need to make additional "round trips" to the kernel:

One for reading the vCPU register state to get the current instruction pointer (RIP), before you can fetch the MMIO instruction.
One for writing the vCPU register state to make RIP point to the next instruction, after you emulate the MMIO instruction.

With the HAXM API, you can't read/write the guest RIP alone, but have to sync a larger set of vCPU registers (again that's by design), so these round trips will be very costly.
In fact, if the instruction only accesses one MMIO address (which is the most common case), I think the HAX_EXIT_FAST_MMIO approach already guarantees the minimum number of kernel/user context switches (kernel => user => kernel), with pretty low overhead.

To be clear I'm trying to emulate VGA which requires an MMIO region 64K in size.  HAX_EXIT_FAST_MMIO requires a round trip for every vram read and write and there'll be thousands while a frame is drawn then none for a long period.  Try a vga program that uses unchained mode such as doom in qemu with haxm and you'll see the performance is horrid, far slower than emulation so I was trying to do it like dosemu which exit kvm at the first vram access and emulates until it appears the program is finshed.  As you say HAX_EXIT_PAGEFAULT would likely be the best solution but it appears that it has a 2MB granularity which is far to large for the 64k vga vram window and will put the entire vm address space within 2MB when running in real mode.

[...] so I was trying to do it like dosemu which exit kvm at the first vram access and emulates until it appears the program is finshed.

I see, so you don't need to sync vCPU state very often. Is "kvm" a typo? I wonder if dosemu uses KVM at all, since it predates KVM. And I'm curious whether KVM API (KVM_EXIT_MMIO, etc.) meets your requirements, since it's similar to HAXM.

As you say HAX_EXIT_PAGEFAULT would likely be the best solution but it appears that it has a 2MB granularity which is far to large for the 64k vga vram window and will put the entire vm address space within 2MB when running in real mode.

True. The 2MB granularity stems from the fact that HAXM divides each RAM block (host buffer backing guest memory) into 2MB chunks:
https://github.com/intel/haxm/blob/0d3922d8a64da41487ecab7ae19150ce838d6085/core/include/memory.h#L37-L38
Theoretically, it's possible to switch to 4KB chunks, but last time we tried that, we ran into some stability issue. Maybe you could try to make it work?

Is "kvm" a typo? I wonder if dosemu uses KVM at all, since it predates KVM. And I'm curious whether KVM API (KVM_EXIT_MMIO, etc.) meets your requirements, since it's similar to HAXM.

Dosemu2 supports KVM but the original dosemu presumably had the same problem but with v86 mode instead.  I think dosemu2 uses v86 mode in KVM for compatibility with the original dosemu and then they use regular page faults to trap vga access.  That makes it basically impossible to support any programs which need their own paging.

Maybe you could try to make it work?

Well, I can take a stab at it I suppose.

@cracyc I wanted to port/update dosemu support for NetBSD some time ago but I've faced rather very legacy code that needed modernization in regards of used interfaces (e.g. switch to mcontext from sigcontext). We have also dropped v86 support from the NetBSD kernel and dosemu was using it.
Was this situation changed? Is dosemu(2) switching to HAXM now?
@raphaelning I got a report from @polprog that OpenBSD in HAXM is terribly slow.. is this VGA related?

Not that I know of, what I'm working on is a bit different.

@raphaelning I got a report from @polprog that OpenBSD in HAXM is terribly slow.. is this VGA related?

I have no idea. Other desktop OSes seem to run smoothly. Since VGA rendering is probably done with MMIO, you may be able to identify the bottleneck by observing exits to QEMU of type HAX_EXIT_FAST_MMIO.

@raphaelning NetBSD with X Window is also unusable due to slowness... I will file a dedicated PR for it in future once all the booting issues will be solved.

If it's using basic vga mode (4 plane/16 color) it's (at least from my experimentation) going to be very slow.  It appears the svga cirrus emulation in linear framebuffer mode doesn't mmio exit for vram writes which would be far faster.

From what I have noticed, the apparent VGA slowness is because of general emulation slowness in  case of having extremely verbose logging enabled. On NetBSD, caused by setting the following debug level variable to 0.
int default_hax_log_level = 0; in platforms/netbsd/hax_wrapper.c L47

From what I have noticed, the apparent VGA slowness is because of general emulation slowness in case of having extremely verbose logging enabled. On NetBSD, caused by setting the following debug level variable to 0 (That is not the default value!).
int default_hax_log_level = 0; in platforms/netbsd/hax_wrapper.c L47

Are you sure that performance bottlenecks disappear if we set default_hax_log_level to 0? I don't see many messages in dmesg(8) myself..

OK, @polprog explained off-list that if we add extra logging performance is reduced.. but reverting to HEAD it's back to the current state. Unfortunately this doesn't solve our primary issue with MMIO bottleneck.

ad "off-list explanation":
Setting the log level to 0 makes it print every message, including hax_debug() calls - those printf-s are blocking so they stall the whole VM for the time they print the message, and there's several of them per VM exit, so they slow things down.
To add a little context - when I was debugging, I would change that value back and forth as I needed to see more or less info in dmesg(8) and the VM would run either slow or fast.
My test box is not the fastest, and with loglevel set to zero (most verbose) I could literally see QEMU's BIOS print the messages line-by-line as if it was a 9600bps terminal! And that also caused longer kernel load times - just as if the emulated CPU clock was a couple times slower. - This is not a bug, since implementing those debug messages in a non-blocking way would be just plain over-engineering and they are suspressed in most cases anyway.
This is a proof that it's not MMIO related, I just feel that it should be mentioned :)

Do you mean that your report of slowness of OpenBSD was caused by debug level and not MMIO bottleneck?

Yes, exactly.

I'm facing the same problem, one MMIO exit for every write operation in the VGA area is incredibly slow, so in fact it's unusable. dosemu approach of writing a whole emulator just for VGA emulation seems to be a ridiculous amount of work and way too complicated.
No, it's not loglevel related, the roundtrips ARE costly, there are thousands of them per second!
Now what I have tried and at least improves the situation a bit is when I attach the VGA area to a page, check if it changed on every haxm exit (mark already read bytes) and write changed values to the emulated VGA. At least the performance is somewhere near usable, but there still is the problem of detecting changes (I tried GetWriteWatch() API, but it doesn't fire when memory area gets modified by HAXM Guest more than once, I can provide a simple test application as proof).
KVM has a module called "coalesced IO", which collects all MMIO writes into a ringbuffer that can be read and flushed by userspace. Still, I'm not convinced by this concept, as it still has 1 exit per write which is still costly when there are thousands of them per second.
Now I had the following idea, maybe someone can tell me if it would work and if it can be implemented in HAXM that way:

VirtualAlloc a PAGE_READWRITE memory page in userspace (Host)
Attach the page to the VGA area at Seg A000, BUT mark it WRITE-ONLY in guest EPT (currently I cannot tell HAXM to mark a page write-only, why?). Add that page to a linked list of pages that need to be checked in 3)
On every VM exit, check the dirty-bit of the EPT pages attached to VGA. If it is marked dirty, make an Exit to userspace and notify host that it should check VGA area for modifications and sync it to emulated adapter. After call to userspace, clear Dirty-Bit in pagetable.
If the VM attempts a read operation (fortunately, this is less likely than write), an exception would be generated which can in turn be translated into an HAX_EXIT_FASTMMIO call, so reads work like with normal MMIO area. There isn't much we can do here, they cannot be coalesced like the writes.

Would that work and is someone here knowledgeable enough to enahance HAXM functionality this way?

I tried to implement coalesced MMIO writes to HAXM for evaluation. I got a preformance gain by up to 50% (of course depending on the tested aplpication), but as expected, the video performance is still unacceptable. Should anybody be interested in the experimental coalesced MMIO implementation, please tell me and I will commit it to my repository accordingly. Coalesced MMIO can be turned on via a flag in my patch, so it shouldn't break compatibility.
I will try to comparison approach next (where I currently have acceptable results, but MMIO read needs to be implemented as mentioned above), and the concept has the flaw that you have a 1/255 chance per byte to miss a write on comparison.

@AlexAltea maybe can help you with this.

@leecher1337 Sorry for the late reply, I missed these GitHub notifications until @frymezim pinged me.
I'd be happy to look over your patch. Coalesced MMIO sounds an awesome feature, and I'd be happy to add support for it in QEMU's HAXM backend.

Just for comparison purposes, I also tried to run QEMU with HAXM (and installing MS-DOS into it), and it suffers from exactly the same performance problem.
I also read about Intel vGPU, but documentation states: "Legacy VGA is not supported in the vGPU device model", so this is also a dead end.
So I'm running out of ideas now.
I will create a branch with the coalesced MMIO support for your review, shall I create a PULL request for it too or just drop a link here for review?

@leecher1337 Feel free to drop a link for now! Whether you want to submit a pull request is up to you (and up to the Intel team, if that could be merged).

Hi,
Here is a first draft. I tried to port it to the new HAXM release, as my current working copy is 6 months old, so I hope I merged the code correctly. Feeld free to comment on it:
https://github.com/leecher1337/haxm/commit/5d6a603438f9c1b571794b3b6a93b3fe20191bb7
/* We have to following assumption on coalesced writes:
*

The usermode application is responsible for checking of REP prefix on
HAX_EXIT_FAST_MMIO and handle it accordingly. The result of such an
instruction will NOT be added to coalesced MMIO queue, as usermode
application can handle it faster by knowing counter in ECX and ESI->EDI
directly.
Usermode application must update ESI, EDI and ECX accordingly when
handling it!

When consuming the coalesced queue, caller has to reset count to 0!
*/

Here is an example for handling from NTVDMx64 HAXM implementation:
		case HAX_EXIT_FAST_MMIO:
			hax_handle_fastmmio((struct hax_fastmmio *)iobuf);
			break;
		case HAX_EXIT_COALESCED_MMIO:
			hax_handle_fastmmio_coalesced(*(struct hax_coalesced_mmio **)iobuf);
			break;

void hax_handle_fastmmio_coalesced(struct hax_coalesced_mmio *coal)
{
	uint32_t i;

	for (i=0; i<coal->size; i++)
		hax_handle_fastmmio_op(&coal->mmio[i]);
	coal->size = 0;
}

void hax_handle_fastmmio_op(struct hax_fastmmio *hft)
{
	switch (hft->direction)
	{
	case 0: /* Read */
		switch (hft->size)
		{
		case 1: *((PBYTE)&hft->value) = sas_PR8((DWORD)hft->gpa); break;
		case 2: *((PWORD)&hft->value) = sas_PR16((DWORD)hft->gpa); break;
		case 4: *((PDWORD)&hft->value) = sas_PR32((DWORD)hft->gpa); break;
		default: haxmvm_panic("Seems hax_handle_fastmmio also gets size %d", hft->size);
		}
		break;
	case 1: /* Write */
		switch (hft->size)
		{
		case 1: sas_PW8((DWORD)hft->gpa, (BYTE)hft->value); break;
		case 2: sas_PW16((DWORD)hft->gpa, (WORD)hft->value); break;
		case 4: sas_PW32((DWORD)hft->gpa, (DWORD)hft->value); break;
		default: haxmvm_panic("Seems hax_handle_fastmmio also writes size %d", hft->size);
		}
		break;
	case 2: /* gpa -> gpa2 memcpy */
		sas_PRWS((DWORD)hft->gpa, (DWORD)hft->gpa2, hft->size);
		break;
	}
}

void hax_handle_fastmmio(struct hax_fastmmio *hft)
{
	UCHAR *pCmd = ((UCHAR *)Sim32GetVDMPointer(
				   (state._cs.selector << 16) | state._eip,
				   1, ISPESET));

	if (*pCmd == 0xF3)
	{
		BOOL bHandled = FALSE;
		DWORD bytes;

		switch (hft->direction)
		{
		case 0: /* Read */
			switch (pCmd[1])
			{
			case 0xA4:	/* Move (E)CX bytes from DS:[(E)SI] to ES:[(E)DI].*/
				if (getDF()) break; // Not implemented yet
				sas_PRWS((DWORD)hft->gpa, RMSEGOFFTOLIN(getES(), getEDI()), getECX());
				setEDI(getEDI() + getECX());
				setESI(getESI() + getECX());
				setECX(0);
				bHandled = TRUE;
				break;

			case 0xA5:	/* Move (E)CX words from DS:[(E)SI] to ES:[(E)DI].*/
				if (getDF()) break; // Not implemented yet
				sas_PRWS((DWORD)hft->gpa, RMSEGOFFTOLIN(getES(), getEDI()), getECX() * hft->size);
				setEDI(getEDI() + getECX() * hft->size);
				setESI(getESI() + getECX() * hft->size);
				setECX(0);
				bHandled = TRUE;
				break;
			}
			break;
		case 1: /* Write */
			switch (pCmd[1])
			{
			case 0xA4:	/* Move (E)CX bytes from DS:[(E)SI] to ES:[(E)DI].*/
			case 0xA5:	/* Move (E)CX words from DS:[(E)SI] to ES:[(E)DI].*/
				if (getDF()) break; // Not implemented yet
				sas_PRWS((DWORD)RMSEGOFFTOLIN(getDS(), getESI()), (DWORD)hft->gpa, getECX() * hft->size);
				setEDI(getEDI() + getECX() * hft->size);
				setESI(getESI() + getECX() * hft->size);
				setECX(0);
				bHandled = TRUE;
				break;
			case 0xAA:  /* Fill (E)CX bytes at ES:[(E)DI] with AL. */
				sas_fills((DWORD)hft->gpa, getAL(), getECX());
				setECX(0);
				bHandled = TRUE;
				break;
			case 0xAB:	/* Fill (E)CX words at ES:[(E)DI] with AX. */
				switch (hft->size)
				{
				case 2:
					sas_fillsw((DWORD)hft->gpa, getAX(), getECX());
					setECX(0);
					bHandled = TRUE;
					break;
				case 4:
					sas_fillsdw((DWORD)hft->gpa, getEAX(), getECX());
					setECX(0);
					bHandled = TRUE;
					break;
				}
				break;
			}
			break;
		}
		if (bHandled)
		{
			if (!DeviceIoControl(hVCPU, HAX_VCPU_SET_REGS, &state, sizeof(state), NULL, 0, &bytes, NULL));
			return;
		}
	}

	hax_handle_fastmmio_op(hft);
}

I hope that explains its usage.


Min/MaxLength attributes relay added

Hi guys,
I have implemented MinAndMaxLenghtAttributeRelay (I created that based on StringLengthAttributeRelay). I have decided to create one relay for both Min and Max attributes to be able to handle a situation when both of them are applied to a single property at once.
All remarks will be highly appreciated!
Closes #678

Hi @zvirja, will you find some time to review the changes?

@damian-krychowski Thanks a lot for the follow-up! I've added a few minor tweaks to the code.
@moodmosaic Do you want to take a look as well? 😉

@moodmosaic Thanks for the quick check 😉
@damian-krychowski I've merged the PR. Thanks for all your efforts and sorry for the belonged replies 🙏 🍷


Displaying content in Scene

Hi,
I am trying to display a few different components in the scene view but it comes up empty.
There is remotely sourced image, come text and a webview with remotely sourced content.
I have verified that remote content fetching is working. but nothing displayed.
Here is the code.
<ScrollView style={styles.scroll_container}>
      <Image
        style={styles.image}
        source={{uri: "https://********/assets/img/product_images/web/" + this.state.product.image}}
      />
      <View>
        <Text style={styles.name}>{this.state.product.name}</Text>
        <WebView source={this.state.product.desc} scrollEnabled={false}/>
      </View>
</ScrollView>

If I replace all the content inside that with just a simple text view, it works. but this doesn't.

I realised that the components are loaded then the is fetched. I think this might be the issue. But I thought a change in state would trigger a re-render. At least thats what happens in my other components like list. I am also grabbing the contents of the lists remotely.

I tried loading the product info from a local file instead of remotely. Other components work except webview.

Looks like there is an issue with webview. I ended up using react-native-autoheight-webview


 Post-processing(Disabled), Basic Rotation, Keys for debugging

Post-processing(Disabled) for bloom, enabled with sprite2D, change the define on top of file to 1
Basic Rotation of entities -> need to overhaul transform system
Keys for debugging (A&D for scaling and Q&E for rotation)

#18 also resolved by this


Upgrading com.google.guava:guava from 30.1.1 to 32.0.1 to stay up to …

Upgrading com.google.guava:guava from 30.1.1 to 32.0.1 to stay up to date and leverage the most recent improvements
Reviewers, please add label "dependencies".

@Jackie-Jiang FYI

Codecov Report

Merging #11115 (dc15f06) into master (abf3959) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #11115     +/-   ##
=========================================
  Coverage    0.11%    0.11%
=========================================
  Files        2203     2148     -55
  Lines      118196   115672   -2524
  Branches    17887    17580    -307
=========================================
  Hits          137      137
+ Misses     118039   115515   -2524
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration1temurin20
?

integration2temurin17
?

integration2temurin20
?

unittests1temurin11
?

unittests1temurin17
?

unittests1temurin20
?

unittests2temurin11
?

unittests2temurin17
?

unittests2temurin20
0.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 57 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


There is no available state for destination for the user

raise RiverException(ErrorCode.NO_AVAILABLE_NEXT_STATE_FOR_USER, "There is no available state for destination for the user.")
RiverException: There is no available state for destination for the user.
Hi ,
Need to know why this exception raises, and the details please. Development of the project was done as per the documentation.
The issue is only with Postgres, as sqlite3 is working fine
Please help

The error means there is no next state that the user has permission to progress to. I agree the error could be more clearly written. Are you positive that this only came up in PG, not in sqlite? I suspect other differences between the two environments were at play.

hi Scott,
The issue has been resolved !!
We where changing the States in postgres manually, which actually affected workflow of the system. Currently, while updating new fields to the postgres, leave the statefield blank, it will work fine.
Thanks for the response.


feat: Make editor zoomable using ctrl++/- #45

added scrolling with ctrl + +/-
extracted resusable function to avoid repetition

@kartik-madhak

added ctrl + = to zoom
and changed increment to isIncrement


Only newly opened tabs are displayed

I'm on build 4099 on a Mac. The issue is that the extension ignores any tabs that were opened when ST started up, and only recognises newly opened tabs.
This includes tabs opened after the extension was installed, they are ignored if I quit and then open ST.

simply saving the file should update it

This isn't really too much of a problem and should update once you save/type in a file, hence the "Just Launched" and "Idle" texts.


Update future  to version 1.18.3

future version 0.18.3

[X] Check the upstream
https://github.com/PythonCharmers/python-future/tree/v0.18.3

[X] Check the pinnings

[X] Check the changelogs
https://github.com/PythonCharmers/python-future/blob/v0.18.3/docs/whatsnew.rst

[X] Additional research
https://github.com/conda-forge/future-feedstock/issues
There are currently no open issues mentioned at the time of the review in conda forge

[X] Verify the dev_url
https://github.com/PythonCharmers/python-future

[X] Verify the doc_url
http://python-future.org

The doc_url seems to have issues an eye must be kept on the package until a new doc_url is specified

[X] License is spdx compliant
MIT

[X] License family is present
MIT

[X] Verify that the build_number is correct

[X] Verify if the package needs setuptools

[X] Verify if the package needs wheel
Wheel does not seem to be mentioned

[X] pip in the test section
pip

[X] Veriy the test section

[X] Verify if the package is architecture specific or Noarch

[X] Verify that private modules are not mentioned on the recipe For example: (_private_module)

Results:
Based on the research findings and the results we can conclude
that it is safe to update future to version 0.18.3

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_19lcgoj70h/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_19lcgoj70h/clone/recipe/meta.yaml:51: http_url: http://python-future.org is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0bdfuk3by1/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_0bdfuk3by1/clone/recipe/meta.yaml:47: http_url: http://python-future.org is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_a4xidu8kh1/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_a4xidu8kh1/clone/recipe/meta.yaml:49: http_url: http://python-future.org is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b86_edippe/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
Errors were found


postgresql cluster creation hangs in creating null_resource.postgresql_master_attach_volume[0]

postgresql cluster creation hangs in creating
null_resource.postgresql_hotstandby1_attach_volume[0]: Still creating... [1h34m31s elapsed] null_resource.postgresql_master_attach_volume[0]: Still creating... [1h33m31s elapsed] null_resource.postgresql_hotstandby1_attach_volume[0]: Still creating... [1h34m41s elapsed]

@ttscoff @gabrielgrigorie @mlinxfeld  can anyone help me with it pls ???


react-dom/server bundled in client bundle

We found on our private project that react-dom/server was bundled to out application. After some investigation we found that this is cased because of react-lazy-svg, as server and none server code is included in the same bundle/ file.
We use Webpack and Razzle to build our application and run a production build.
As soon as we integrate react-lazy-svg it bundles the browser compatible react-dom/server dependency as well with ~20 KB of unused JS.
Most packages support a browser build in the package.json so that the bundler does not need to optimise this.
Would it be possible to do the same here?

thanks for the report, that's definitely not how it's intended.
I think the problem is that the ssr functions have to be excluded and provided in a different bundle.
These are the 2 functions to extract
export const createSpriteSheetString = async (knownIcons: IconsCache) => {
  const arr = await Promise.all(Array.from(knownIcons.values()));

  return renderToStaticMarkup(
    <SpriteSheet icons={arr.filter((a): a is IconData => a != null)} />,
  );
};

export const renderSpriteSheetToString = async (
  markupString: string,
  knownIcons: IconsCache,
  spriteSheetId = internalSpriteSheetId,
) => {
  const spriteSheet = await createSpriteSheetString(knownIcons);

  const ssrEmptySpriteSheet = `<svg id="${spriteSheetId}" style="display:none"></svg>`;
  return markupString.replace(ssrEmptySpriteSheet, spriteSheet);
};

To build a separate entry file for the ssr bundle the tsdx/rollup config needs to be extended/modified.
I'm not sure if I can get to implementing these changes in the next days, so if you have some spare resources, contributions are very welcome

@garthenweb I chose to solve this issue with an explicit second bundle file in the package to contain the imports to react-dom/server. As this is a breaking change due to a new import path for the 2 server side rendering relevant functions you need to upgrade to v2.0.0

@kaoDev Thanks a lot! Because of the breaking change I was not able to come back and test this for the moment. Will try to have a look at this soon.


[FEATURE REQUEST] Add an inverse modifier to oiiotool's --ociodisplay option

Problem Description
The oiiotool --ociodisplay option is very useful for applying an OCIO DisplayViewTransform.  This takes an image in a "src" color space and then applies a "view" transform for a specific "display".  For example, one might take a source image in ACEScg space and apply the "ACES 1.0 - SDR Video" view for the "sRGB - Display", using either of the following:
% oiiotool acescg.exr --ociodisplay:from="ACEScg" "sRGB - Display" "ACES 1.0 - SDR Video" -o srgb.jpg
% oiiotool acescg.exr --iscolorspace "ACEScg" --ociodisplay "sRGB - Display" "ACES 1.0 - SDR Video" -o srgb.jpg

However, it is often useful to be able to apply the DisplayViewTransform in the inverse direction.  For example, to convert sRGB back to ACEScg, inverting a specific view transform.  The ocioconvert tool (one of the OCIO command-line tools) is already able to do this, as follows:
% ocioconvert --invertview srgb.exr "sRGB - Display" "ACES 1.0 - SDR Video" acescg.exr "ACEScg"

Inverting a DisplayViewTransform is easily possible via the OCIO v2 API.  Simply set the direction of the DisplayViewTransform to TRANSFORM_DIR_INVERSE.
Proposed solution
The --ociofiletransform option in oiiotool already has an "inverse" modifier.  So the proposed solution is to add an "inverse" modifier to --ociodisplay.

Proposed fix in #3650
I'm hoping that maybe somebody could apply this patch on their end and test it by hand, verifying that it appears to work to make the inverse display transform (and also doesn't break the forward transform in any way).
To be honest, I don't really have a test for this in this PR. I noticed recently (to my embarrassment) that the current OIIO testsuite does not contain a test that directly exercises --ociodisplay (though it does for --ociolook and --colorconvert). I haven't had a chance to fix that yet, so of course it also doesn't test the inverse. At my facility, oiiotool --ociodisplay is used in production, so I am not actually worried that it's horribly broken. But I would prefer to have it tested directly in the OIIO testsuite and include in the CI, rather than just assume that it's used enough that I'd get yelled at if it stopped working.
The main impediment to my adding these tests is that the OCIO config that I ship in the testsuite itself, which is the nuke-default, doesn't actually contain any "displays." I think that the solution is going to be to shift the tests to using the new OCIO 2.2 "built-in" configs, which do contain displays. That gives me more testing fodder, as well as knowing that the testsuite doesn't need to come with its own sample config (and, for that matter, allowing oiiotool to rely on the built-in configs in cases where the user doesn't have their own full config). But that's a separate task.

To that end, @doug-walker, I was just poking around the ocio docs and... is there anyplace that directly addresses how to use the built-in configs? Are there any new API calls? Or do you just ask for a config as always, and use the name of the built-ins? I had trouble finding anyplace that really spelled it out in the docs, but it's possible I just wasn't looking in the right place.

@lgritz , I have some documentation in progress for the new features in OCIO 2.2.  I'm still writing it, but here is what I have for the built-in configs section so far:
Built-in Configs
For users
The OCIO Configs Working Group has put a lot of effort building new ACES configs that take advantage of the features of OCIO v2.  These configs are available separately from the project's GitHub repo:
https://github.com/AcademySoftwareFoundation/OpenColorIO-Config-ACES/releases
To make them more accessible, they are built in to the library itself and may be accessed directly from within applications that incorporate the OCIO 2.2 library.  Wherever you are able to provide a file path to a config, you may now provide a URI-type string to instead use one of the built-in configs.  For example you may use these strings for the OCIO environment variable.
To use the ACES CG config, use this string for the config path:
ocio://cg-config-v1.0.0_aces-v1.3_ocio-v2.1
To use the ACES Studio config, use this string for the config path:
ocio://studio-config-v1.0.0_aces-v1.3_ocio-v2.1
This string will give you the current default config, which is currently the CG config:
ocio://default
In future releases, it is expected that updated versions of these configs will be added, each with a unique name string.  However, the previous strings will continue to be supported for backwards compatibility.
For developers
The new URI-type strings may be passed into the existing APIs such as CreateFromEnv and CreateFromFile.  This is also a new CreateFromBuiltinConfig.
If you'd like to offer the built-in configs from your user interface, there is an API to access them through the new BuiltinConfigRegistry.  Here is basic Python code to access the registry and print the strings to present in a UI:
    registry = ocio.BuiltinConfigRegistry().getBuiltinConfigs()
    for item in registry:
        # The short_name is the URI-style name.
        # The ui_name is the name to use in a user interface.
        short_name, ui_name, isRecommended, isDefault = item

        # Don't present built-in configs to users if they are no longer recommended.
        if isRecommended:
            print(ui_name)

Prints:
Academy Color Encoding System - CG Config [COLORSPACES v1.0.0] [ACES v1.3] [OCIO v2.1]
Academy Color Encoding System - Studio Config [COLORSPACES v1.0.0] [ACES v1.3] [OCIO v2.1]
If your application saves the path to a config and the user enters "ocio://default", it is recommended that you don't save that.  Instead, call getDefaultBuiltinConfigName to get the name of the current default.  This is guaranteed to give the same results, whereas the default config could change somewhat between releases.  Prepend "ocio://" in order to save it as a valid config path.  Here's the Python code:
    "ocio://" + ocio.BuiltinConfigRegistry().getDefaultBuiltinConfigName()

Oh, thanks, this is very helpful.


Can not get the FTP set by others

Looks like it's not in the PowerController instance anymore.  Strangely for self-activities it IS on the Activity instance which was previously deprecated.  Not sure what they are up to.  Maybe it's an upstream regression.  I'm going to wait at least a week before diving back into this.  Sometimes they make mistakes and roll back.  I find it hard to believe that this was related to a feature.

Patched up FTP detection as best as I can see how to with e83f7a084c2141afee1bb0f5466ce56ed916f14b


Feature request: Replace exfat fs

Hello:
The exfat filesystem should be replaced with only one full sized / ext4 filesystem. These are the reasons:

The second card slot does not work with the actual Arkos.
Much faster and better.  Native filesystem for Linux.
POSIX-like filesystem. You could finally enable the second card with symlinks thanks to the native filesystem. The second slot could be used then (and fixed this little bug)
For Windows users there are (very) good options like sftp through a file browser like Filezilla, which is also free software. One example, this was the way many years ago to copy files to the modded xbox1. It is just to install an sftp software.
Also, many people is actually used to Linux. A Linux user can still plug the SD cards on linux boxes.
Finally, you could unload the exfat support in Arkos since it has no sense and it's not needed in Linux.

I want to apologize if this has been asked and for my words (this is not my native language)
Thank you very much for this fantastic software Christian.

No way.  The reason arkos exists is because the original OS it was forked from (TheRA) did not support an easy solution for Windows and Mac users to load their roms.  That’s why exfat support was added,  The second sd slot works fine if you go to Options>Advanced>Switch to second SD slot.  Then it will use the second sd slot for games.  The card in the second sd slot can be ext4, exfat, fat32, or ntfs.  The FAQ for the 351V(https://github.com/christianhaitian/arkos/wiki/Frequently-Asked-Questions---RG351V#q-how-do-i-make-use-of-the-second-sd-slot-for-roms-and-games) and 351MP(https://github.com/christianhaitian/arkos/wiki/Frequently-Asked-Questions---rg351mp#q-how-do-i-make-use-of-the-second-sd-slot-for-roms-and-games) explains how to setup and use the second sd slot.


Update Date and Time Imputation Vignette with preserve and ignore_seconds examples

The functions showcased in this vignette have been updated to have preserve and ignore seconds arguments.  #589 #592   This issue will be done when examples showcasing those updated arguments are presented.

Adding extra requests to this issue from #727 :

We also need to add an example to the date and time imputation vignette where date_imputation=”06-15” is used.
Include an example using new slice_derivation function, e.g. when AE onset partial date/time before first dose, it will impute last day of year or month. when AE onset partial date/time after first dose, it will impute first day of year or month.

The third item is covered by #701

great - thanks @bundfussr! one less job


claimable_balance only takes one parameter in the go sdk

What version are you using?
<EMAIL_ADDRESS>What did you do?
I was trying to get a list of claimable balances of a single asset on a single wallet.
balances, err := client.ClaimableBalances(horizonclient.ClaimableBalanceRequest{
	Claimant: "GDJCDWRAAB3LA6TT32OBOXV2UYVJEVPKMQXDLUWYH7FQ73MCTH3UNPT7",
	//Sponsor: "GDFCYDQOVJ2OEWPLEGIRQVAM3VTOQ6JDNLJTDZP5S5OGTEHM5CIWMYBH",
	Asset: "AQUA:GBNZILSTVQZ4R7IKQDGHYGY2QXL5QOFJYQMXPKWRRM5PAV7Y4M67AQUA",
})

What did you expect to see?
I expected to see the same results I would see from this url:
https://horizon.stellar.org/claimable_balances/?claimant=GDJCDWRAAB3LA6TT32OBOXV2UYVJEVPKMQXDLUWYH7FQ73MCTH3UNPT7&asset=AQUA:GBNZILSTVQZ4R7IKQDGHYGY2QXL5QOFJYQMXPKWRRM5PAV7Y4M67AQUA&limit=200
What did you see instead?
I got the error: "invalid request: too many parameters"
I think I can fix this in claimable_balance_request.go; I can make a PR, but want to make sure this isn't already being fixed already.

PR: https://github.com/stellar/go/pull/4248


Refactor for naming consistency

meets TIKI's stds. (i.e. files in sub-packages should carry their parents prefix)


Openresty infront of mailcow

I am running a rig with a local api and a remote mailserver (based on mailcow)
I have crowdsec installed on the remote server and hooked up to my api with a firewall bouncer, but I would like to use captcha on the remote server. I have installed the docker version of crowdsecurity/openresty, but I am struggelig as to where and how implement this in the process.
Is it possible to run openresty infront of another nginx both in separate docker images?
//Thomas

Yes you should be able to put this infront of a Nginx you would have to mount a custom default server configuration into openresty so all requests sent to the server are proxy pass to the other nginx.

So in my own words: Traffic should hit openresty nginx, and then be proxy-passed to mailcow?

So in my own words: Traffic should hit openresty nginx, and then be proxy-passed to mailcow?
I am trying to wrap my head around this information (mailcow behind proxy), and this documentation on your docker image - but I am not able to understand where to do what.
Care to share some insight?

Not to sound completely rude but we cant walk you through how to configure a reverse proxy as that is not our goal.
Maybe you can try joining our discord and the community over there might be able to help.

Fair point.


incorrect conversion from ESRI WKT to Proj.4 for NAD83 UTM

First I would like to say this lib helps me a lot. I am using it in conjunction with PyProj lib in several projects already.
Issue:
I have a ESRI WKT string, and in this particular case I know its corresponding EPSG code is 26916 because the ESRI WKT string represents NAD83 UTM Zone 16.
I was trying to re-project a point from above projection into EPSG:3857 using PyProj for a web mapping app, but the result looked incorrect. The following code shows the different results when PyProj objs are init'd by proj.4 string produced from PyCRS and EPSG code.
import pycrs
import pyproj
esri_prj_str_nad83_utm_zone16 = 'PROJCS["NAD_1983_UTM_Zone_16N",GEOGCS["GCS_North_American_1983",DATUM["D_NORTH_AMERICAN_1983",SPHEROID["GRS_1980",6378137,298.257222101]],PRIMEM["Greenwich",0],UNIT["Degree",0.017453292519943295]],PROJECTION["Transverse_Mercator"],PARAMETER["latitude_of_origin",0],PARAMETER["central_meridian",-87],PARAMETER["scale_factor",0.9996],PARAMETER["false_easting",500000],PARAMETER["false_northing",0],UNIT["Meter",1]]'
fromcrs = pycrs.parser.from_esri_wkt(esri_prj_str_nad83_utm_zone16)
proj4_str = fromcrs.to_proj4()
prj_obj_from_proj4 = pyproj.Proj(proj4_str)
prj_obj_26916_from_epsg = pyproj.Proj("+init=EPSG:26916")
prj_obj_3857_from_epsg = pyproj.Proj("+init=EPSG:3857")
(-9748715.328601453, 3929480.5837150216)
print pyproj.transform(prj_obj_from_proj4, prj_obj_3857_from_epsg, 446495, 3681410)
(-9748750.584671738, 3931248.690782473)
print pyproj.transform(prj_obj_26916_from_epsg, prj_obj_3857_from_epsg, 446495, 3681410)
Thanks

I'm impressed that it is that close.  This comes down to the definition of the UTM projection has changed in PROJ.4.
So PROJ.4 (which pyproj is built atop) takes this line of code:
prj_obj_26916_from_epsg = pyproj.Proj("+init=EPSG:26916")
It has a lookup in epsg and creates finds this proj4 string:
# NAD83 / UTM zone 16N
<26916> +proj=utm +zone=16 +datum=NAD83 +units=m +no_defs  <>

The proj4_str that you show above from pycrs results in the proj4_string:
+proj=tmerc +datum=NAD83 +ellps=GRS80 +a=6378137.0 +f=298.257222101 +pm=0.0  +lat_0=0.0 +lon_0=-87.0 +x_0=500000.0 +y_0=0.0 +units=m +axis=enu +no_defs
I replaced +proj=tmerc with etmerc, I get the first result that show.  This is because PROJ.4 version 4.9.3 changed UTM projection algorithms from tmerc to etmerc (https://github.com/OSGeo/proj.4/issues/316).   At some point, PROJ.4 may change the definition of tmerc to be the same as etmerc, but I wouldn't hold my breath.

Glad to hear you find it useful @zhiyuli. I'm just going to have to concur with @micahcochran, and props for keeping up with the latest on proj4.
So if I understand correctly, PyCRS uses the old way of referring to utm (tmerc)? How recent was this change and do you recommend that we change to the newer etmerc or stick with the old?

@karimbahgat I found out about this when I was creating the GIGS testing framework for Proj4.
That is correct. etmerc and tmerc are algorithms for Transverse Mercator projections.
Proj4 version 4.9.3 was released 2016-08-22, which made UTM use the etmerc version.  The issue was raised in 2015-10-06.  Their change was based on a report by the US National Geospatial Inteligence Agency on 2014-03-25.  etmerc algorithm was added to Proj4 release 4.8.0 (source), which was released 2012-03-07.
The WKT itself doesn't not specify the method/algorithm used to reproject data.  The WKT only specifies that the coordinates stored in Transverse Mercator projection.  tmerc and etmerc both give valid results.
I don't know if there is enough information in the WKT to say that this is a +proj=utm.  Another problem is that other spatial reference systems use the tmerc algorithm.  I'm not sure if it is fair to those countries geospatial authorities to do an algorithm shift.  Even the utm algorithm switch has already changed that algorithm around the world.
I haven't really thought through the right coarse of action.

@micahcochrain, thanks for the details. So since this is essentially a very recent legacy/incompatability issue, there is no clear solution of switching to the new etmerc method or sticking with the old tmerc, as this might lead to unexpected changes for projection definitions written before or not agreeing with the recent USGS recommendation.
Perhaps the best solution in this scenario is to let the user set a function arg or library constant deciding whether to use the old or new algo, defaulting to the old..
To be clear though, algorithms aside, setting the proj4 +init=utm is simply a shorthand to allow you to specify more easily w the +zone arg, but is functionally equivalent to using +etmerc with all the additional args (as was produced by pyproj)? Is this all that differentiates the utm projection from ordinary trans merc? This makes me wonder too, i cant remember taking into account utm zones when i wrote pycrs...

@karimbahgat  Everything you have stated is correct.
What you propose seems like a good solution.


Add reference to an SW intro article

closes #4441

The build failure seems like an unrelated, temporary fluke.


Add faq to debug when chaos not working

What problem does this PR solve?
Add faq to debug when chaos not working
What is changed and how does it work?
Add faq to debug when chaos not working
Check List
Tests

No code

Code changes

No code change

Side effects

None

Related changes

None

Codecov Report

Merging #194 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #194   +/-   ##
=======================================
  Coverage   39.47%   39.47%
=======================================
  Files          17       17
  Lines         608      608
=======================================
  Hits          240      240
  Misses        337      337
  Partials       31       31

Continue to review full report at Codecov.

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

@cwen0 @YangKeao PTAL again

@dcalvin PTAL

/test

/test


Remove cumulative alteration work

Change behaviour so that only the alteration_work from the active revaluation is counted

Codecov Report

Merging #331 (12ce5ed) into develop (1659008) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #331      +/-   ##
===========================================
- Coverage    93.35%   93.34%   -0.01%
===========================================
  Files          129      129
  Lines         4830     4826       -4
  Branches       446      446
===========================================
- Hits          4509     4505       -4
  Misses         233      233
  Partials        88       88

Impacted Files
Coverage Δ

application_form/pdf/haso.py
100.00% <ø> (ø)

cost_index/utils.py
85.36% <ø> (-0.69%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update debian9.rst

added notice with reference to official composer Download website.
If using the current command this would end into a "Installer corrupt" state, due to new hash of the newer composer-setup release.

Many thanks for your contribution!


feat: tauri desktop app

The desktop app is soon switching from electron to tauri, and I want to make sure the app can still build properly
There are some things that I need to do beforehand, such as making a new channel for the tauri builds to separate them from the edge channel
I was thinking of doing something similar to what Ubuntu did with their app centre, where they had a channel called edge/preview for the builds of the now released app centre rewrite

[ ] Investigate how to make new channels with custom names
[ ] Set up tauri builds for the new client
[ ] Make new GitHub action that automatically builds the app and publishes it to edge/preview

It seems like the client is located on revoltchat/frontend, as opposed to being inside the desktop repo.


Constellix Provider: Fails to create new zone and add requested records in one run

If I add a new zone and records into my config, and the zone does not already exist at Constellix, my expectation would be that when I execute octodns-sync, in that run, a new zone would be created and records added as well.  What actually happens is, the new zone is indeed created, but record creation fails, raising ConstellixClientNotFound().  If I subsequently run octodns-sync immediately thereafter, with no change in my config, the records are then created with no error.  The Constellix API requires providing a zone ID typically when performing a record operation, and I believe the provider is just missing the logic to make itself aware of the zone it just created, before moving on to record creation.  I can tweak the ConstellixClient class' domains method to make this work but I don't think I'm doing it the best way.
I am using octodns version 0.9.9 and python 2.7.17

I seem to remember that being expected when I created it, but it can be made better for sure. I think you're on the right path though.
I believe within ConstellixClient's domain_create method, after the POST, you can just have something like
# Add newly created zone to domain cache
self._domains['{}.'.format(name)] = resp[0]['id']

Thank you.  I figured there was a better way.  I just tried your recommendation, the only tweak I had to make before it worked was to decode the json in the response:
self._domains['{}.'.format(name)] = resp.json()[0]['id']
I'll test a bit more thoroughly as well and advise.

@hounster PR was merged, some reworking but should create zone + add all the records in the same run.
Thanks for opening the issue