ctx.body overwrites status set by ctx.assert

Sometimes you may want to wrap your middleware in try - catch statement.
If one uses ctx.assert -method inside a try statement, which sets the ctx.status, one might come across with  counter intuitive behavior.
For example:
(ctx, next) => {
  try {
    ctx.assert(false, '400', 'this is bad')
  }
  catch (error) {
    console.log(ctx.status) // 400
    ctx.body = 'whoops, the status will be 200 after setting me'
    console.log(ctx.status) // 200
  }
}

However, this one works as expected:
(ctx, next) => {
  try {
    ctx.assert(false, '400', 'this is bad')
  }
  catch (error) {
    ctx.status = ctx.status // This sets the _explicitStatus to true
    console.log(ctx.status) // 400
    ctx.body = 'now works'
    console.log(ctx.status) // 400 as expected
  }
}

I guess there is nothing wrong in this behavior. ctx.assert does not set ctx.status. It throws Error object with status key set to the value you've specified. After that it's as per the user to decide what to do with it. So error throwing and status handling are kind of decoupled.
Here's koa context default error handler source code:
https://github.com/koajs/koa/blob/841844ee2fd0cf91478b5ed8d2f401e8b97eb4a2/lib/context.js#L143-L144
Then it uses error status to set context status:
https://github.com/koajs/koa/blob/841844ee2fd0cf91478b5ed8d2f401e8b97eb4a2/lib/context.js#L149
Btw assert test checks error object, not context.status:
https://github.com/koajs/koa/blob/841844ee2fd0cf91478b5ed8d2f401e8b97eb4a2/test/context/assert.js#L15
If you raise the error without handling it in try/catch you'll get the correct response code.
Here's a source example of 3 cases:

koa default error handling
try/catch without setting context status
try/catch with setting context status

const
  CODE = 400,
  MSG  = "this is bad";

const handlerDefault = (ctx) => {
  ctx.assert(false, CODE, MSG);
};

const handlerCustom = (ctx) => {
  try
  {
    ctx.assert(false, CODE, MSG);
  }
  catch (err)
  {
    console.log(`Hit custom error handler, err.status: ${err.status}`);
  }
};

const handlerCustomCode = (ctx) => {
  try
  {
    ctx.assert(false, CODE, MSG);
  }
  catch (err)
  {
    ctx.status = err.status || 500;
    console.log(`Hit custom error handler, ctx.status set to err.status: ${ctx.status}`);
  }
};

You'll get:

400 as koa propagates it in default error handler
404 as a default HTTP response code (or 200 if body is set)
400 as context status was set explicitly

Full example source code here.
It's also worth checking documentation.

@garcia556, thanks for your extensive answer!


I am trying to make paging in flutter in which i have customscrollview and scrollcontroller paging is working perfect in all emulators and devices but not in android 8.1 devices

i am calling the get more data function as
 if (scrollController.offset >= scrollController.position.maxScrollExtent &&
        !scrollController.position.outOfRange) {
      getMoreData();
    }

works perfectly in other devices rather than android 8.1 tested device redmi 6 pro. In this device only first page is shown and no more scrolling happens

Hi @cimplesid
Are you still experiencing this issue?
Thank you

Closing for now,
@cimplesid please comment if the issue still persist and I'll reopen it
Thank you

no, I am over it thank you it is ok to be closed.


long prompts with commas

This works well for simple prompts. But if i give it a long prompt it dies bad (Windows 10). If i give it a prompt with commas it seems to choke pretty bad on the save file also.
I tried moving the path to the c:\tmp
and even chopping down the length but its not having it.
I actually have a good 30+ prompts already made that are quite long with many commas, was hoping to feed it. It would be nice to get the accurate folder name but not imperative, just cant get the save to work.
Any suggestions thanks?
console log - very long prompt no commas
C:\Users\Chris\Documents\apps\Bimg-0.1.5>node scriptdebug.js
Prompt Data:
[
'Black and white pencil sketch of annoyed English bus driver with moustache sitting in the bus driver seat. one hand on his forehead. camera view from left passenger side door entrance. reminiscent of a pencil sketch with a touch of cartoonish charm.The theme is dark and eerie. with strong shadows and crosshatching. '
]
Sending request...
https://www.bing.com/images/create?
Redirected to https://www.bing.com/images/create?
Waiting for results...
. { end: '', flush: true }
. { end: '', flush: true }
. { end: '', flush: true }
. { end: '', flush: true }
Downloading images...
An error occurred: Error: ENOENT: no such file or directory, mkdir '/tmp/Black and white pencil sketch of annoyed English bus driver with moustache sitting in the bus driver seat. one hand on his forehead. camera view from left passenger side door entrance. reminiscent of a pencil sketch with a touch of cartoonish charm.The theme is dark and eerie. with strong shadows and crosshatching. '
at Object.mkdirSync (node:fs:1398:3)
at saveImages (file:///C:/Users/Chris/Documents/apps/Bimg-0.1.5/node_modules/bimg/dist/imgen.js:104:12)
at generateImageFiles (file:///C:/Users/Chris/Documents/apps/Bimg-0.1.5/node_modules/bimg/dist/imgen.js:153:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async createImages (file:///C:/Users/Chris/Documents/apps/Bimg-0.1.5/scriptdebug.js:28:32) {
errno: -4058,
syscall: 'mkdir',
code: 'ENOENT',
path: '/tmp/. '
Console log - with COMMAS
Prompt Data:
[
'Black and white pencil , \r'
]
Sending request...
https://www.bing.com/images/create?
Redirected to https://www.bing.com/images/create?
Waiting for results...
. { end: '', flush: true }
. { end: '', flush: true }
. { end: '', flush: true }
. { end: '', flush: true }
Downloading images...
An error occurred: Error: ENOENT: no such file or directory, mkdir '/tmp/Black and white pencil sketch of annoyed English bus driver wi'h moustache sitting in the bus driver seat, one hand on his forehead. camera view from left passenger side door entrance,
at Object.mkdirSync (node:fs:1398:3)
at saveImages (file:///C:/Users/Chris/Documents/apps/Bimg-0.1.5/node_modules/bimg/dist/imgen.js:104:12)
at generateImageFiles (file:///C:/Users/Chris/Documents/apps/Bimg-0.1.5/node_modules/bimg/dist/imgen.js:153:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async createImages (file:///C:/Users/Chris/Documents/apps/Bimg-0.1.5/scriptdebug.js:28:32) {
errno: -4058,
syscall: 'mkdir',
code: 'ENOENT',
path: '/tmp/Black and white pencil  \r'
}

Windows probably doesn't want to create folders that end in periods or commas. I run my prompts through a sanitizer script to remove any punctuation and make sure it ends on a letter not a space. I limit the length to 225 characters but even that can be too long for some cloud servers like Gdrive I've found.

If there's a way to get a second parameter that is just for the output path , the prompt can still be anything, but the file path would be a variation (e.g first 220 characters and sanitized).  I couldn't get that second parameter working though.
I did get the script to sanitize and parse new lines with multiple prompts from the text input file but many prompts are very long and can go up to maybe 500 chars on some of them. If that output path parameter gets added in would gladly donate.


Add option to select multiple Pivot Row

Please file a jira and add this PR to it and Add that Jira to the description

Please file a jira and add this PR to it and Add that Jira to the description

Please make sure CLA is fixed

Please make sure CLA is fixed


proposal: errors: Is(err, target) should allow err to decide even when target is nil

Currently Is looks like this:
func Is(err, target error) bool {
	if target == nil {
		return err == target     // <-------------
	}
	isComparable := reflectlite.TypeOf(target).Comparable()
	for {
		if isComparable && err == target {
			return true
		}
		if x, ok := err.(interface{ Is(error) bool }); ok && x.Is(target) {
			return true
		}
		if err = Unwrap(err); err == nil {
			return false
		}
	}
}

See:
https://golang.org/src/errors/wrap.go?s=1170:1201#L29 and also
https://github.com/golang/xerrors/blob/master/wrap.go#L50

[See: https://github.com/rocketlaunchr/dataframe-go/blob/master/error_collection.go#L11]
Sometimes error objects are designed as containers of for multiple errors.
type ErrorCollection struct {
	sync.Mutex
	errors []error
}

The proposal suggests that errors.Is(err, target) should be adjusted such that when target is nil and err is the ErrorCollection, the ErrorCollection should be able to decide whether it is nil or not via implementing Is(err error) bool {.
See: https://github.com/rocketlaunchr/dataframe-go/blob/master/error_collection.go#L62
It is feasible that when the error collection holds no errors, it would like to allow the user to write:
	ec := NewErrorCollection()
	isNil := errors.Is(ec, nil)

where isNil returns true.
But due to the implementation of errors.Is, this will in fact return false.

The documetation currently states:

An error is considered to match a target if it is equal to that target or if
it implements a method Is(error) bool such that Is(target) returns true.

Due to the phrasing of the documentation, the current proposal is still consistent with the documentation and arguably should not be deemed to breach Go1 backward compatibility guarantees.

The predeclared type error is an interface type.  It is only nil if it is completely unset.  Giving a type like ErrorCollection a way to decide whether it is nil is going to run straight into https://golang.org/doc/faq#nil_error.  I think it would be a mistake to make the handling of a nil interface any more complex.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

this feels like a duplicate of #40673?

The essence of this behaviour can be achieved by defining an empty error type/value that the error collection can compare itself to in the Is call. Alternatively (and probably preferably), the error collection can be created with a NewErrorCollection(errs []error) error function that returns nil if len(errs) == 0 at the end of the region that generates the collection of errors.

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

No change in consensus, so declined.
— rsc for the proposal review group


Pair With Given Difference | C++

Title -Pair With Given Difference
what will change - Add algorithm in C++ to the given problem.
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding!

@div-bargali please assign this issue to me. Thanks.

Please assign the issue to me.

Sorry, @akshaychaudhari. The one who raised the Issue will have the highest priority. Please come up with a new issue for any new implementations that do not yet exist in this repository so that I can assign you there.
Thanks!


Component has extra height

I am using svg-icon 10.0.0 for loading a 24 * 24 svg
but when it loads the svg element has the same height and width but svg-icon component has a height of 29 px, and this makes my alignments go wrong.
Relevant screenshot is attached

It is looks like the font-size or possibly line-height used the back button is interfering.  To confirm this, see if the 'extra height' happens if the svg-icon is used outside of your button mark-up.
You can explicitly set the height of the SVG in the svg-icon inside the button by doing something like the following:
<svg-icon src="assets/images/left-arrow.svg" [svgStyle]="{ 'height.px':'24' }"></svg-icon>

Yes, it is the font-size. The svgStyle isn't needed. Since the SVG is a 24x24 if you hover over the svg. The svg-icon is getting its height from the line-height set by the font.

Thanks for the quick reply
I have previously replaced svg-icon with pure svg , and now added it back , this time very different result  , width changed to 32 and height remained 29

I applied line-height to 1 but for both button and svg-icon  , but things are no different . Also setting  [svgStyle]="{'height.px':'24'}" has no effect

The size of svg-icon is determined by the button's font-size. The size of svg-icon is not the size of the svg it loads. To confirm this hover over the svg element inside the svg-icon.
If you want to explicitly set the height of the svg-icon to be 24px, you will need to adjust its style. For example, this might work:
<svg-icon src="assets/images/left-arrow.svg" style="font-size:21.5px;"></svg-icon>

That said, the alignment isn't being caused by svg-icon, it is being caused by the font-size being used on the back-button style.


Support for Gree YAW1F

Hello,
Thanks so much for this project. I have a gree aircon which uses remote YAW1F. I cloned the repo and tried to use the gree protocol as specified in the file, but unfortunately it is not compatible with the protocol specified by remote YAW1F.
I found this other project which seems to have a specification for my remote, in the files:
https://github.com/crankyoldgit/IRremoteESP8266/blob/master/src/ir_Gree.h
https://github.com/crankyoldgit/IRremoteESP8266/blob/master/src/ir_Gree.cpp
I am trying to reverse engineer their specification, but the fact that it's written in C++ definitely does not help. Are there any easier solutions to add YAW1F remote functionality to the project? I would very much do it myself but I cannot find any website documenting how these protocols work. If there is any source pointing me to how YAW1F assembles the state then I can try myself and maybe even open a pull request afterwards!
Thanks,
Fab

I have the YT1FF remote.  They might be similar?

I don't know. In any case I ended up using a tasmota-ir firmware in the end, and that worked for me!


Removing source causes crash

I have a style that works fine with 3 sources (custom data source + mapbox.mapbox-terrain-v1 + mapbox.mapbox-streets-v5).  When I remove the first source, Studio crashes and then crashes on start. Manually re-adding the first source in project.yml makes everything work again.

Here's the error code from console:
2/12/15 9:16:15.428 AM com.apple.xpc.launchd[1]: (com.mapbox.mapbox-studio.132944[5218]) Service exited with abnormal code: 8

Sounds like another case of possible corruption of the ~/.mapbox-studio/app.db file. Can you try replicating again and then:

move ~/.mapbox-studio/app.db aside and see if restarting then works again
if so, can you post your app.db here (it's json) after stripping anything sensitive?

per chat with @michaelsteffen - he's going to try to reduce a testcase. He found that removing app.db did allow Studio to start, but of course Studio would still crash once he accessed the same project. The problem appears to be that editing the layers puts the style out of sync in a crashy way when it simply needs to provide some modal notice about what in the style needs removed (that might reference the removed layer). We found that shell.log and app.log contained no errors indicating why the crash happened.

Doesn't seem to be the stylesheet.  I've whittled the offending project down to a single stylesheet that contains just the following:
// Background //
Map {
    background-color: #fff;
}

The crash is still reliably reproducible by deleting one source.

o_0

Indeed

Could the problem have to do with the fact that the offending source belongs to someone else's Mapbox account? Adding sources to this project works fine, as does deleting either of the other two sources currently in the project.

I have no idea. Thanks get getting us this far. I presume @samanpwbb might know or have ideas about the issue.

I tried to reproduce this and was not able to.
@michaelsteffen where you making these changes with the web UI or in the project yml file? If you still remember, it would be super helpful for you to detail the steps you took before hitting the error.

Same as https://github.com/mapbox/mapbox-studio/issues/1093

see also #1194

@springmeyer i think #1194 is different. This is fixed by https://github.com/mapbox/mapbox-studio/pull/1195


[FLINK-33457][tests] FlinkImageBuilder checks for Java 21

What is the purpose of the change
The PR allows to pass FlinkImageBuilder to proceed with e2e tests
Verifying this change
This change is a trivial rework
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no )
The runtime per-record code paths (performance sensitive): (no )
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no )
The S3 file system connector: (no )

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for taking a look


Couple things with 105.0.5167.0

With the last release, my system sees Thorium as "thorium-browser-unstable" even though I'm executing the "thorium-browser" script. Not sure if this is on purpose or not. With the past couple releases, my system would see it as just "thorium-browser".
And this might be nitpicking a little, but the flag "--disable-features=EventPath" is passed twice, once before "--flag-switches-begin" and another time after.

@wdym it "sees" it as thorium-browser-unstable? Could you post a screenshot(s) of everything you are describing?
And the eventpath thing is an upstream chromium thing.

I use the mouse gesture program Easystroke and it would recognize the last few versions as "thorium-browser" so the same gestures would work on a new release, but with the last release the gestures stopped working and when selecting Thorium again within Easystroke, I saw it now recognized it as "thorium-browser-unstable".

@Alex313031
Some observations with this build from me. First of all thanks a lot for getting Google Sync working, I now can use Thorium as my new default browser.
On the other hand, I faced some trouble with the user-provided PKGBUILD for Arch Linux as that does no longer work with this release out-of-the box.
There are several reasons, the first is due to the different package name in the source section which points to the released DEB-file on Github, a consistent naming scheme over all releases would fix that. While that was easy to fix in the PKGBUILD, this is also avoidable.
For reference, the first entry is the command used for the new version, the one after the comment symbol was the one used before.
source_x86_64=("https://github.com/Alex313031/Thorium/releases/download/M${pkgver}/thorium-browser_${pkgver}_amd64.deb"
#"https://github.com/Alex313031/Thorium/releases/download/M${pkgver}/thorium-browser_${pkgver}-${pkgrel}_amd64.deb"
)

I am a bit puzzled about the other two problems, as I also had to comment out the following lines:
#chmod 4755 "$pkgdir/opt/chromium.org/thorium-unstable/chrome-sandbox"
and the whole install icons section
# install icons
    #for i in 16x16 24x24 32x32 48x48 64x64 128x128 256x256; do
    #    install -Dm644 "${pkgdir}"/opt/chromium.org/thorium-unstable/product_logo_${i/x*}.png \
    #                   "${pkgdir}"/usr/share/icons/hicolor/${i}/apps/thorium-browser-unstable.png
    #done

I don't know if they are no longer needed or if my solution was just a dirty hack to get the new package to install, but for anyone interested, here is the now working PKGBUILD:
PKGBUILD.txt

As the icons went missing this way, and I noticed that the directory name has changed as well, hence some adjustements to the file path in the PKGBUILD were needed to get it working, here is the now fixed
PKGBUILD.txt
.

@raisedintheusa This should fix your initial issue https://github.com/Alex313031/Thorium/commit/947c28a91dbab41e4931747389a19fb1db717279

Issue fixed with release 38.


Optimize license header check output

Add failure output to help developers self-solving issues.
Optimized the output to make info clear.

test run: https://github.com/YanxuanLiu/spark-rapids-jni/actions/runs/11965688951/job/33360240539

please close https://github.com/NVIDIA/spark-rapids-jni/issues/2614 after merge and send out a notice to developers thanks


WIP: test scheduling with a scheduleonmetric only policy

This change highlights and fixes an issue where error handling of the filter method in TAS can cause the scheduleonmetric strategy to fail. The first commit adds a test to show the failures pointed to in this TODO https://github.com/intel/platform-aware-scheduling/blob/48c2d0fe5b6e90e318da1cceeb03ae627e4b9b5c/.github/e2e/e2e_test.go#L288
Additional commits will be added to show the solution to the issue. There is a decision on error handling that needs to be made in order to enable this behaviour while keeping the other features of TAS as expected.
This issue interacts with the handling of the error in #55 so no change should be merged until that change is merged. One issue is around the handling of adding policies and the registered strategy map which can create a perceivable delay in adding strategies to the cache. The solution in #55 of removing access to the RegisteredStrategies store could be part of the solution to this issue - but it will have to be thought of in combination with that issue.

Fixed in #78


policy: add additional benchmarks for different types of rules

This PR refactors code in resolve_test.go to allow for different benchmarks to be ran for different types of rules (e.g., L3-only ingress, L3-only egress, CIDR-only, etc).

This change is

Looks like I broke something, investigating...

test-me-please

Coverage decreased (-0.02%) to 44.193% when pulling f95541aace70b4bb529293690ad3a9798899fad2 on pr/ianvernon/improve-policy-benchmark into 028a814267c61ef64722426536836bc7ed1425e0 on master.


Terragrunt doesn't de-dupe external dependencies

When you run one of the xxx-all commands in a folder foo, Terragrunt walks the dependency tree, and one of the first things it does is identify all dependencies outside of the foo subtree, and confirm whether you want to run xxx-all in those subfolders (e.g., to ensure we don't apply or destroy in a dependency you don't want to affect).
It seems that this confirmation process is not de-duping the external dependencies, so it asks to confirm the same ones over and over again multiple times.

I'd like to revive this issue if you don't mind @brikis98
We're getting our kubernetes provider configuration from our EKS module, which itself depends on our VPC module for inputs. This results in really long wait times if you have application states that rely on each other.
I think we'd need some kind of deduplication on dependencies here but haven't taken a look at the code yet.

Hi, are you using dependency blocks, or dependencies? If you are using the former, can you create a different issue and update the title to terragrunt doesn't reuse outputs of dependencies? This issue is referring to a different problem related to dependencies blocks.


Store modified file locally

Currently, any unsaved file modifications are forgotten and removed as soon as the user opens another file

resolved in b3e4956


Increase KUTTL timeout

The current 180s which is used in other operators is not enough and tests fail during Prometheus pod initialization.

/lgtm


Upgrade to Erlang 24.2

@mp may want to provide slightly more than a cursory glance, as this was a little more extensive than most other fixes and less straightforward, and the path forward wasn't exactly known (see commit "replacing pg2 with pg").  Having said that, it seems to work.
mac:stats_hero lbaker$ ~/chef-server/src/bookshelf/rebar3 compile
===> Verifying dependencies...
===> Analyzing applications...
===> Compiling stats_hero
mac:stats_hero lbaker$

https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/4745
https://buildkite.com/chef/chef-umbrella-main-chef-server-full/builds/222
builds and tests with stats_hero changes^^


Inquiry About Customising Tab Behaviour in SunEditor

Version
"suneditor-react": "2.16.5"
Additional context
I am unable to customize tab behavior. I want tab not to add 4   spaces but one   so that I can remove the tab in one backspace button press. But I am seeing that   is converted into plain text '    ' (four white spaces). I tried it with \u2003 as well but in vein.
I set tabDisable: true and added customization for tab.
    const handleKeyDown = (e: KeyboardEvent) => {
       if (e.key === 'Tab' || e.keyCode === 9) {
            e.preventDefault();

            const editor = editorRef.current?.editor;
            if (editor) {
                editor.insertHTML('\u2003', true)
            }
        }
    };

    return (
        <SunEditor
            defaultValue={html}
            setContents={html}
            onChange={onChange}
            showToolbar={showToolbar}
            ref={editorRef}
            disable={readOnly}
            setOptions={{
                buttonList: buttonsList,
                customPlugins: [header, templateEditor, level1, level2, level3, level4, level5, level6, sectionSign],
                minHeight: "300px",
                resizingBar: showResizingBar,
                showPathLabel: false,
                tabDisable: true,
                textTags: {
                    underline: 'u',
                    bold: 'b',
                    italic: 'em',
                    strike: 's'
                },
                addTagsWhitelist: 'L1|L2|L3|L4|L5|L6|HEADING|T|t| |\u2003',
                pasteTagsBlacklist: 'div|p|span',
            }}
            onKeyDown={handleKeyDown}
        />
    )

   CC: @JiHong88

I made a work around

Hello @muhammad971,
Could you please share the details of how you accomplished this?


b/29618677: limit max concurrent requests

@alainjobart @enisoc
There was previously no limit on how many concurrent requests
VTTablet could accept. This change allows you to set such a limit
to prevent too many requests from OOMing the process.

This change is

LGTM


Roll engine 6280d4266983..e5305f4223af (3 commits)

https://github.com/flutter/engine/compare/6280d4266983...e5305f4223af
git log 6280d4266983ff8eff24f26164d10172e3824385..e5305f4223afc5ee5648dcf273aa7135e35f5c92 --no-merges --oneline
e5305f422 Roll src/third_party/skia 6c8f5b31ac49..6db27c28c61e (4 commits) (flutter/engine#8802)
ed7d128d7 Reformat dart dependencies in DEPS. (flutter/engine#8800)
ed912a01d Roll src/third_party/dart 5fbf7720f8..b6997deb3e (26 commits)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (bmparr@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/4908726510157824

Commit queue failed; closing this roll.


Aditya harsh patch 1

I am unable to understand  the use of 6 commits for a single upload of code

I am unable to understand the use of 6 commits for a single upload of code

Due to some confusion from my sides, there are 6 commits
All the code files are same.

Make a new PR because on merging there will be 6 files with the same name.


can't open the google photo to inset image

can't open the google photo to inset image
when i uesd the top buttom to insert image from google. the popwindow show me:
Google
403. That’s an error.
We're sorry, but you do not have access to this page. That’s all we know.

I'm getting this too. I dug a bit deeper, but I'm not sure where to go from here. I'd be happy to help
fix it because this is a showstopper for my use-case (namely, I have to have some form of drag-drop
or click to upload photo). I just don't know where to go from here.
My app is configured with Google OAuth2 ClientID, Secret, and API key. I enabled the following APIs in the
project:

Google Drive API
People API (needed to login to StackEdit)
Google Picker API
Photos Library API

I looked through the rest of the available APIs and didn't see anything relevant.
Next I looked at the traffic and noticed that the Google Drive iframe uses the same picker API, just a different nav
parameter.
Here's a request with the picker for Google Drive, which works just fine
Google Picker to open file from Drive
GET /picker?protocol=gadgets&origin=https%3A%2F%2F<MYSTACKEDIT-DOMAIN>&oauth_token=<MY-TOKEN>&std=true&title&multiselectEnabled=true&hostId=<MYSTACKEDIT-DOMAIN>&parent=https%3A%2F%2F<MYSTACKEDIT-DOMAIN>%2Ffavicon.ico&nav=((%22all%22%2Cnull%2C%7B%22mimeTypes%22%3A%22text%2Fplain%2Ctext%2Fx-markdown%2Capplication%2Foctet-stream%22%7D)%2C(%22all%22%2Cnull%2C%7B%22mimeTypes%22%3A%22text%2Fplain%2Ctext%2Fx-markdown%2Capplication%2Foctet-stream%22%2C%22td%22%3Atrue%7D))&rpcService=<MY-RPCSERVICE-ID>&rpctoken=<MY-RPCSERVICE-TOKEN>&thirdParty=true HTTP/1.1
Host: docs.google.com
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:96.0) Gecko/20100101 Firefox/96.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,*/*;q=0.8
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
DNT: 1
Alt-Used: docs.google.com
Connection: keep-alive
Referer: https://<MYSTACKEDIT-DOMAIN>/
Cookie: <MY-COOKIES>
Upgrade-Insecure-Requests: 1
Sec-Fetch-Dest: iframe
Sec-Fetch-Mode: navigate
Sec-Fetch-Site: cross-site
Pragma: no-cache
Cache-Control: no-cache

Google Picker to "Open from Google Photos"
GET /picker?protocol=gadgets&origin=https%3A%2F%2F<MYSTACKEDIT-DOMAIN>&oauth_token=<MY-TOKEN>&std=true&title&hostId=<MYSTACKEDIT-DOMAIN>&parent=https%3A%2F%2F<MYSTACKEDIT-DOMAIN>%2Ffavicon.ico&nav=((%22photos%22%2Cnull%2C%7B%22type%22%3A%22highlights%22%7D)%2C(%22photo-upload%22))&rpcService=<MY-RPCSERVICE-ID>&rpctoken=<MY-RPCSERVICE-TOKEN>&thirdParty=true HTTP/1.1
Host: docs.google.com
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:96.0) Gecko/20100101 Firefox/96.0
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,*/*;q=0.8
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
DNT: 1
Alt-Used: docs.google.com
Connection: keep-alive
Referer: https://<MYSTACKEDIT-DOMAIN>/
Cookie: <MY-COOKIES>
Upgrade-Insecure-Requests: 1
Sec-Fetch-Dest: iframe
Sec-Fetch-Mode: navigate
Sec-Fetch-Site: cross-site
Pragma: no-cache
Cache-Control: no-cache

Checked my local tokens, and it appears the photo and drive scopes are both present.
{
  "id": "tokens",
  "type": "data",
  "data": {
    "google": {
      "<TOKEN-ID>": {
        "scopes": [
          "https://www.googleapis.com/auth/photos",
          "https://www.googleapis.com/auth/drive.appdata",
          "https://www.googleapis.com/auth/drive",
          "https://www.googleapis.com/auth/drive.install"
        ],
        "accessToken": "<MY-ACCESS_TOKEN>",
        "expiresOn":<PHONE_NUMBER>920,
        "idToken": "<MY-ID-TOKEN>",
        "sub": "<MY-SUB-WHATEVERTHATIS>",
        "name": "dcode",
        "isLogin": true,
        "isSponsor": false,
        "isDrive": true,
        "isBlogger": false,
        "isPhotos": true,
        "driveFullAccess": true
      }
    }
  },
  "hash": -1
}


support Phoenix.HTML 4.0

This library needs to be updated for Phoenix.HTML 4.0. (edit: opened #93 )

Hello! Thank you so much for the library!
I see that the work to support Phoenix HTML 4.0 was done (thank you!)! Do you have any news on when/if it is going to be released?

Hi @mathieuprog, possible to merge the linked PR?


Feature Proposal: Improved Array Literal Formatting in Typstyle

generated by genai from unstructured chat log
Issue Description
Currently, the formatting of array literals in Typstyle can be improved to enhance readability and compactness. This proposal suggests several enhancements to the way array elements are formatted, particularly focusing on multi-line arrays and import sorting.
Proposed Features

Multi-line Array Literals:

Allow array elements to be placed on the same line when possible, rather than forcing each element onto a new line. For example:[1, 2, 3, 4,
 5, 6, 7, 8]

Instead of:[1,
 2,
 3,
 4]

Implement a heuristic to determine when to break lines based on element length and column width. For instance:

Calculate the length variance of elements.
Ensure no single element exceeds a certain proportion of the column width (e.g., half or one-quarter).
If the longest element exceeds the threshold, place each element on a new line.

cc @QuadnucYard


Invalid Argument: Output satoshis is not a natural number

Start merging your [CAT] tokens ...
Insufficient satoshis balance!
mint failed!
Invalid Argument: Output satoshis is not a natural number
Done in 1.14s.

the same error, how to fix it?

solved with rebooting machine

the same error, how to fix it?

reboot machine.
stop docker tracker
docker stop tracker
Start stracker
docker start tracker

I will rent a new machine, reconfigure the nodes, and use a new wallet，it show the same error

have you try rebooting prev. machine? it worked for me
rebooting and stop/start tracker not work ,get same message

Insufficient satoshis balance!
mint failed!
Invalid Argument: Output satoshis is not a natural number

Reboot and restart docker doesn't work for me. I tried to reproduce  by doing curl -X POST http://<IP_ADDRESS>:8332/wallet/cat-910b90fe
-H "Content-Type: application/json"
-H "Authorization: Basic Yml0Y29pbjpvcGNhdEF3ZXNvbWU="
-d '{"jsonrpc":"2.0","id":"cat-cli","method":"listunspent","params":[0,9999999,["My wallet Address"]}' and it returned

out of sync tracker

#22


Errors raised twice on the client when catched

In order to display error messages on the client side we use a app-level error hook raising an event, which is then catched by the error display. The problem is that if we try/catch a specific operation in order to perform a specific processing two errors are raised on the client (the one from initially the failing operation, then the one from the catch block).

Should be closed by https://github.com/kalisio/kdk/commit/9fc1153f7304eab19af10bade926b96d1b87dac2.


Split searchindex.json and searchservice.json models and operations into separate namespaces

Models and operations in searchindex.json shoud go into namespaces like so (adjusted for your language):

Azure.Search.Documents (SearchClient, SearchClientOptions)
Azure.Search.Documents.Models

Models and operations in searchservice.json should go into namespaces like so (adjusted for your language):

Azure.Search.Documents.Indexes (SearchIndexClient, SearchIndexerClient)
Azure.Search.Documents.Indexes.Models

@xiangyan99 just noting that this is the current file system split:

searchindex.json -> azure.search.documents._index
searchservice.json  -> azure.search.documents._service

and then currently everything is brought up to the azure.search.documents level. This task should be a fairly straightforward rename of azure.search.documents._service to azure.search.documents.indexes

Fixed in #11584


[Event Request] Table 7332 - OnBeforeOpenItemTrackingLines, OnOpenItemTrackingLinesOnBeforeSetSource

Add new integration events OnBeforeOpenItemTrackingLines, OnOpenItemTrackingLinesOnBeforeSetSource in Table 7332.
Function OpenItemTrackingLines:
    BEGIN
+     OnBeforeOpenItemTrackingLines(Rec);
      TESTFIELD("Item No.");
      TESTFIELD("Qty. (Base)");
      TempWhseWorksheetLine.InitLineWithItem(
        TempWhseWorksheetLine."Whse. Document Type"::"Internal Put-away","No.","Line No.",
        "Location Code","Item No.","Variant Code",
        "Qty. (Base)","Qty. (Base)" - "Qty. Put Away (Base)" - "Put-away Qty. (Base)","Qty. per Unit of Measure");

+     OnOpenItemTrackingLinesOnBeforeSetSource(Rec, TempWhseWorksheetLine);
      WhseItemTrackingForm.SetSource(TempWhseWorksheetLine,DATABASE::"Whse. Internal Put-away Line");
      WhseItemTrackingForm.RUNMODAL;
    END;

New publisher functions:

[Integration]
LOCAL PROCEDURE OnBeforeOpenItemTrackingLines@100000010(VAR WhseInternalPutawayLine@100000000 : Record 7332);
BEGIN
END;

[Integration]
LOCAL PROCEDURE OnOpenItemTrackingLinesOnBeforeSetSource@100000011(VAR WhseInternalPutawayLine@100000001 : Record 7332;VAR TempWhseWorksheetLine@100000000 : TEMPORARY Record 7326);
BEGIN
END;

@JesperSchulz, @jobulsin   What is the actual status of this issue?


Survey about the style guide, Style Council, content community backlog

Quick survey for people on the Style Council mailing list and Slack channels.
To be reviewed alongside Mo's qualitative UR.

Trying to add Mo Abudu as assignee and incorrectly closed issue. Still in progress.


Add reference architecture pages

Fixes: https://github.com/pulumi/pulumi-hugo/issues/643
This PR updates the solutions page and adds some reference architecture pages under the solutions URL.

@susanev I filed a follow up issue for the "How Pulumi helps" section: https://github.com/pulumi/pulumi-hugo/issues/665


RIP node-sass

It breaks on every new release of nodejs (at least for windows users) and every SCSS & SASS feature can be reproduced with PostCSS.
It makes sense to use PostCSS as it also provides more flexibility.
Feel free to tell me PR's are always welcome. 😃

Can you please describe what you mean by "It breaks on every new release of nodejs"?


V0.5 Milestone

Coverage decreased (-0.05%) to 89.497% when pulling 57688de5842d2446c47f8b044404b653fd807d55 on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.05%) to 89.497% when pulling 7db41944651bf0d1a9fbe88892d3863013f341bd on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.05%) to 89.497% when pulling 5e034edeea3160c27430a480d4a256dfea46b8bd on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.06%) to 89.494% when pulling 318794a4986ba9819b5cc9c7ba61c8840e438362 on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.06%) to 89.494% when pulling 318794a4986ba9819b5cc9c7ba61c8840e438362 on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.1%) to 89.454% when pulling 9e53e0efd9af05979dad03bb7f9b20a47d85241c on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.1%) to 89.454% when pulling 5073a16b954f5f6e286c4e110481823a4118bd03 on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.1%) to 89.454% when pulling 5073a16b954f5f6e286c4e110481823a4118bd03 on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage decreased (-0.1%) to 89.454% when pulling 6c93f89e794f4a5c3c8deaa0159a71a010a709ca on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage increased (+0.3%) to 89.83% when pulling 3c7a72c2da1172540e67dbfe8b280a49f403d7e5 on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage increased (+1.2%) to 90.778% when pulling 9ca954e0d443882b1c330b1995d0fe8a9977d40f on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.

Coverage increased (+1.2%) to 90.778% when pulling 9ca954e0d443882b1c330b1995d0fe8a9977d40f on V0.5 into 16e47ac3e35c1037a17b4ef9ac56a521c5f991ee on master.


409 Already Registered doesn't have a reason property

I don't know enough Python to fix this effectively, but when I ran the script on my server (CentOS running Python 2.6.6), the HTTPError object that I got back when checking if my account had been registered didn't have a reason property. As a result, it crashed on line 62 (https://github.com/diafygi/acme-tiny/blob/master/acme_tiny.py#L62) when it tries to return the result.
I got around this by just replacing the second call to getattr() with a string literal, but since the result isn't actually used anywhere in the script that I can see, other than for debugging logs (everything else branches off the status code), maybe it could be eliminated? Or the e.reason.__str__ default could be wrapped in another call to getattr() with a sensible default?

Hmmm, HTTPError should have a .reason in python 2.6. Is it reproducible? Can you please post a full traceback?

Sure, here we go:
  File "acme_tiny.py", line 198, in <module>
    main(sys.argv[1:])
  File "acme_tiny.py", line 194, in main
    signed_crt = get_crt(args.account_key, args.csr, args.acme_dir, log=LOGGER, CA=
args.ca)
  File "acme_tiny.py", line 85, in get_crt
    "agreement": "https://letsencrypt.org/documents/LE-SA-v1.0.1-July-27-2015.pdf",
  File "acme_tiny.py", line 62, in _send_signed_request
    return getattr(e, "code", None), getattr(e, "read", e.reason.__str__)()
AttributeError: 'HTTPError' object has no attribute 'reason'

I get the same error with Python 2.6. Using 3.3 from IUS repo appears to fix the problem.

It was d3c6451a4f5d260a2845117d4b3ff22924bf9f4e which caused the error.  Not entirely sure why d3c6451a4f5d260a2845117d4b3ff22924bf9f4e or its follow up 69a457269a6392ac31b629b4e103e8ea7dd282c9 was needed though.  I reverted to 7f2325e49fc57b304d203a651224c5264c7bfd64 and it works fine with Python 2.6.6 on CentOS 6.6.

Fell back to just including the string representation of the error itself.

doesnt work with python 2.7.5 centos 7

@deanet can you please post a full output and error traceback?

@diafygi
here..
+ python acme-tiny/acme_tiny.py --account-key /root/abc.dgprasetya.com//abc.dgprasetya.com_account.key --csr /root/abc.dgprasetya.com//abc.dgprasetya.com.csr --acme-dir /home/admin/web/abc.dgprasetya.com/public_html//.well-known/acme-challenge/
Parsing account key...
Parsing CSR...
Registering account...
Registered!
Verifying abc.dgprasetya.com...
Traceback (most recent call last):
  File "acme-tiny/acme_tiny.py", line 198, in <module>
    main(sys.argv[1:])
  File "acme-tiny/acme_tiny.py", line 194, in main
    signed_crt = get_crt(args.account_key, args.csr, args.acme_dir, log=LOGGER, CA=args.ca)
  File "acme-tiny/acme_tiny.py", line 123, in get_crt
    wellknown_path, wellknown_url))
ValueError: Wrote file to /home/admin/web/abc.dgprasetya.com/public_html//.well-known/acme-challenge/JbnwE_fKfKRyBxZAeRKO8ODMT2pkmmA3enB9u1mBHNQ, but couldn't download http://abc.dgprasetya.com/.well-known/acme-challenge/JbnwE_fKfKRyBxZAeRKO8ODMT2pkmmA3enB9u1mBHNQ

works well with python 2.7.3 at debian 7 :confused:
root@djaja:~# bash -x gen-le-ssl.sh
+ '[' -d ./acme-tiny ']'
+ mkdir ./acme-tiny
+ curl https://raw.githubusercontent.com/diafygi/acme-tiny/master/acme_tiny.py
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  9150  100  9150    0     0  37979      0 --:--:-- --:--:-- --:--:-- 47409
+ echo -n 'Enter DN or FQDN: '
Enter DN or FQDN: + read domain
www.abc.com
+ echo -n 'Enter Full Path www: '
Enter Full Path www: + read fullpath
/srv/st
+ echo www.abc.com
www.abc.com
+ echo /srv/st
/srv/st
+ rm -rf www.abc.com
+ echo 'mkdir working directory...'
mkdir working directory...
+ mkdir www.abc.com
++ pwd
+ workdir=/root/www.abc.com/
+ echo 'generate account key for www.abc.com ..'
generate account key for www.abc.com ..
+ openssl genrsa 4096
Generating RSA private key, 4096 bit long modulus
.........................................................................................................................................................................................++
........................................................................................++
e is 65537 (0x10001)
+ echo 'generate domain private key....'
generate domain private key....
+ openssl genrsa 4096
Generating RSA private key, 4096 bit long modulus
................................................++
...........................++
e is 65537 (0x10001)
+ echo 'generate csr..'
generate csr..
+ openssl req -new -sha256 -key /root/www.abc.com//www.abc.com.key -subj /CN=www.abc.com
+ echo 'create directory acmi at /srv/st..'
create directory acmi at /srv/st..
+ mkdir -p /srv/st/.well-known/acme-challenge
+ echo 'create file verification at /srv/st/.well-known/acme-challenge/..'
create file verification at /srv/st/.well-known/acme-challenge/..
+ echo 'Get a signed certificate..'
Get a signed certificate..
+ python acme-tiny/acme_tiny.py --account-key /root/www.abc.com//www.abc.com_account.key --csr /root/www.abc.com//www.abc.com.csr --acme-dir /srv/st/.well-known/acme-challenge/
Parsing account key...
Parsing CSR...
Registering account...
Registered!
Verifying www.abc.com...
www.abc.com verified!
Signing certificate...
Certificate signed!
+ wget -O - https://letsencrypt.org/certs/lets-encrypt-x1-cross-signed.pem
--2016-01-06 03:07:05--  https://letsencrypt.org/certs/lets-encrypt-x1-cross-signed.pem
Resolving letsencrypt.org (letsencrypt.org)... <IP_ADDRESS>, 2a02:26f0:b7:188::2a1f, 2a02:26f0:b7:187::2a1f
Connecting to letsencrypt.org (letsencrypt.org)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1675 (1.6K) [application/x-x509-ca-cert]
Saving to: `STDOUT'

100%[==============================================================================================================================>] 1,675       --.-K/s   in 0s

2016-01-06 03:07:08 (19.1 MB/s) - written to stdout [1675/1675]

+ cat /root/www.abc.com//signed.crt /root/www.abc.com//intermediate.pem
+ echo 'File www.abc.com/chained.crt = Signed + Intermediate Cert'
File www.abc.com/chained.crt = Signed + Intermediate Cert
+ echo 'File www.abc.com/signed.crt = Signed Cert'
File www.abc.com/signed.crt = Signed Cert
+ echo 'File www.abc.com/www.abc.com.key = Private key'
File www.abc.com/www.abc.com.key = Private key
root@djaja:~#

solved with commented out https://github.com/diafygi/acme-tiny/issues/11#issuecomment-162134072 .


Unsupported Engine

When doing an npm install, I am receiving an unsupported engine from react-dfp. Is there any reason that the node version in the react-dfp engines definition cannot be updated?  Thanks.
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '<=6.0.0' },
npm WARN EBADENGINE   current: { node: 'v12.21.0', npm: '7.6.0' }
npm WARN EBADENGINE }

Oh good catch, I don't see any reasons on why that property can not updated. I'll try to make a fix in the following days. Thank you!

Great, thanks!

@jlafosse I just pushed a fix attempt. For some weird reason these warnings were not showing up in my local. Please give it a try and let me know in case u still find issues. Thanks.

That did the trick, thanks!


Backend extra data

Hello,
call.ts;

I am doing admin control on this part. How can I show this in app.tsx?
//
or is there a commit I can follow?

oh problem fix.
call.html add field.


syscall: Getrlimit test failures in docker on armv7 / aarch64

What version of Go are you using (go version)?

$ go version
go version go1.13.10 linux/arm64

Does this issue reproduce with the latest release?
Yes, 1.14.2 still failes
What operating system and processor architecture are you using (go env)?
Alpine Linux in docker on Alpine Linux on armv7 / aarch64
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/home/buildozer/.cache/go-build"
GOENV="/home/buildozer/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/buildozer/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_arm64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build704765024=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Build golang and run the test suite in a docker container on either armv7 or aarch64
See our APKBUILD for details
What did you expect to see?
Test suite pass
What did you see instead?

 --- FAIL: TestRlimit (0.00s)
    syscall_unix_test.go:332: Getrlimit: save failed: operation not permitted
FAIL
FAIL	syscall	0.029s

Build log
This doesn't happen when we build / run the test suite in a privileged container.

It runs fine on other arches (x86, x86_64, ppc64le, s390x)

Does the following program print the resource limits or also fail with an error?
#include <errno.h>
#include <stdio.h>
#include <string.h>
#include <sys/resource.h>

int main() {
  struct rlimit lim;
  if (getrlimit(RLIMIT_NOFILE, &lim) == 0) {
    printf(
        "limit = %llu\n"
        "hard limit = %llu\n",
        lim.rlim_cur, lim.rlim_max);
  } else {
    fprintf(stderr, "%s\n", strerror(errno));
  }
  return 0;
}

That works, yes:
~ $ cat test_getrlimit.c
#include <errno.h>
#include <stdio.h>
#include <string.h>
#include <sys/resource.h>

int main() {
  struct rlimit lim;
  if (getrlimit(RLIMIT_NOFILE, &lim) == 0) {
    printf(
        "limit = %llu\n"
        "hard limit = %llu\n",
        lim.rlim_cur, lim.rlim_max);
  } else {
    fprintf(stderr, "%s\n", strerror(errno));
  }
  return 0;
}

~ $ gcc test_getrlimit.c -o test_getrlimit
~ $ ./test_getrlimit
limit = 1048576
hard limit = 1048576

@ianlancetaylor any thoughts?

Can you run both the C program and the Go program under strace -f, and see exactly which system calls they are using for getrlimit?  Thanks.

This is the golang test that is failing (I show the last part of the output of the process that is calling getrlimit)

~ $ PATH=$PWD/bin:$PATH strace -f -s520 -o testrlimit.strace  go test -v syscall -run TestRlimit

4822  write(1, "=== RUN   TestRlimit\n", 21
4822  <... write resumed>)              = 21
4822  futex(0x40000aa148, FUTEX_WAKE_PRIVATE, 1
4822  <... futex resumed>)              = 0
4822  getrlimit(RLIMIT_NOFILE,
4822  <... getrlimit resumed>0x400007ced8) = -1 EPERM (Operation not permitted)
4822  write(1, "--- FAIL: TestRlimit (0.00s)\n", 29
4822  <... write resumed>)              = 29
4822  write(1, "    syscall_unix_test.go:332: Getrlimit: save failed: operation not permitted\n", 78
4822  <... write resumed>)              = 78
4822  futex(0x40000aa148, FUTEX_WAKE_PRIVATE, 1
4822  <... futex resumed>)              = 1
4822  write(1, "FAIL\n", 5
4822  <... write resumed>)              = 5
4822  write(3, "# test log\n", 11
4822  <... write resumed>)              = 11
4822  close(3
4822  <... close resumed>)              = 0
4822  exit_group(1
4822  <... exit_group resumed>)         = ?
4822  +++ exited with 1 +++

This is the test program provided by @andybons

~ $  strace -f -s 512 -o test_getrlimit.strace ./test_getrlimit

4843  execve("./test_getrlimit", ["./test_getrlimit"], 0xffffe8cd4d28 /* 7 vars */) = 0
4843  set_tid_address(0xffffb99220c4)   = 4843
4843  mprotect(0xaaaabafcb000, 4096, PROT_READ) = 0
4843  prlimit64(0, RLIMIT_NOFILE, NULL, {rlim_cur=1024*1024, rlim_max=1024*1024}) = 0
4843  ioctl(1, TIOCGWINSZ, {ws_row=64, ws_col=239, ws_xpixel=0, ws_ypixel=0}) = 0
4843  writev(1, [{iov_base="limit = 1048576", iov_len=15}, {iov_base="\n", iov_len=1}], 2) = 16
4843  writev(1, [{iov_base="hard limit = 1048576", iov_len=20}, {iov_base="\n", iov_len=1}], 2) = 21
4843  exit_group(0)                     = ?
4843  +++ exited with 0 +++

So golang is calling getrlimit, which gets permission denied, while the test program calls prlimit64, which succeeds.

Thanks for the info.  Can you see if https://golang.org/cl/230339 fixes the problem for you?

@ianlancetaylor Yes, I can confirm this patch fixes the issue, all tests succeed now.

For the curious, the root cause of this denial is a bug in libseccomp where it wrongly presumed getrlimit didn't exist on these ARM arches: https://github.com/seccomp/libseccomp/issues/234 (apparently fixed in libseccomp 2.4.4).


Updating ose-jenkins-agent-nodejs-12 builder & base images to be consistent with ART

Updating ose-jenkins-agent-nodejs-12 builder & base images to be consistent with ART
Reconciling with https://github.com/openshift/ocp-build-data/tree/f66c03011773dc3755ad874fc691be612914d65f/images/ose-jenkins-agent-nodejs-12.yml
If you have any questions about this pull request, please reach out to @art-team in the #aos-art coreos slack channel.
Depends on https://github.com/openshift/jenkins/pull/1165 . Allow it to merge and then run /test all on this PR.

/ok-to-test

/retest

/lgtm


No validation on TI Group Edit Screen

Pressing the add button on the TI Group edit screen results in an error when you leave the address field empty.
It's on this page: https://HOSTNAME/admin/tiGroups/1/edit

Repro steps are:

Log in as a CiviForm admin
Go to intermediaries tab
Go to existing trusted intermediaries (or create one if there are none)
Click edit members
Click add then see error

We could even disable the add button until there is anything there. We should also add validation that its an email.


Fix commented LS code action test

Purpose

Fix commented LS code action test

Fixes #21888
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #22399   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 24eec40...04efdd7. Read the comment docs.


Acces account keystore file

Hello
can someone help me?
Is it possible to enter the wallet with keystore files?
I have all the words to enter the account but do not let me. He says there is no account for these words.
I found the keystore.json file
Is it possible to log in with him?

That's not possible. the keystore file is encrypted. you need your 12 words in the correct order to access your account. If there's no account for the 12 words you entered, it means those words are invalid and you've got the wrong words.


Implement floating anchors

This is a follow-up to #618.
Is your feature request related to a problem? Please describe.
When handling globally distributed 3d-objects in a single overlay, one of the problems is that a single anchor-point isn't enough to maintain acceptable precision in all locations.
Describe the solution you'd like
To solve this, we could add a feature I would call "floating anchor". With this, users would provide the geo-reference of an object in a property object.userData.geoPosition (see three.js documentation) and the overlay would automatically calculate the anchored world-space position for those geo-coordinates.
The anchor-point could then be automatically updated from map-movement. With every update, the scene is traversed and world-space coordinates are recomputed from the specified geo-reference.

I go through the same issue, any updates ?

@anorakdepluie @ArthurdeLophem we want to move forward with this and wanted to ask if you could maybe add some more details on what issues you are currently having that could be solved by implementing floating anchors.
That way we can make sure that the solution we end up with actually helps to solve those problems.

Hi,
This solution would solve some problems of mine aswell. I am building an app that allows users to add objects anywhere on the world map.
Currently the objects are blurry / distorted if the objects are placed too far from the current anchor. (If the objects are real world scale such as cars/buildings etc)


feat: Footer Component #142

<Footer />
[merge of feature branch] Resolves #142
This PRs adds the USWDS 2.0Footer component with slim, medium, and big footer types . Also adds other sub-components used in the stories.  New components:

Address
Footer [main]
FooterNav
Logo
SocialLinks

To test:
Compare markup and behavior from   USWDS - Footer documentaiton.

Warnings

:warning:
Please add a changelog entry for your changes.

  Generated by :no_entry_sign: dangerJS against d12bf3bb61e75610a58a3ca6a7fbb1f29fea0c8d

@suzubara uhhhh, yeah forgot again about mobile.  I've created a branch where I'm doing that work now hw-footer-mobile.
This back to wip until then. Will open a PR from that branch so we can discuss mobile stuff there.


Computing embedding for train is extremely slow

Is it normal to cost more than 1 hour to computing embedding?

It depends on several things:
e.g.

how many sentences
running on what kind of machine


1.2.7 breaks migrations when updating 1.2.5

Playground link doesn't make any sense, as I'd need to use multiple versions at once.
Say you have the following model:
type User struct {
	BaseModel

	// The user's email address
	Email                    string        `gorm:"not null;uniqueIndex:idx_user_email" json:"email"`
}

Migrating with:
tx.AutoMigrate(&domain.User{})
Causes:
2024/03/10 22:09:04  ERROR: constraint "uni_users_email" of relation "users" does not exist (SQLSTATE 42704)
[0.697ms] [rows:0] ALTER TABLE "users" DROP CONSTRAINT "uni_users_email"

I'd appreciate, if releases would be properly tested beforehand. I'll be pinning 1.2.5 for now.
link for not being marked as stale: https://github.com/go-gorm/playground/issues/0

v1.25.7-0.20240204074919-46816ad31dde works

Have the same issue while using  v1.25.9 and PostgreSQL. Downgrading to v1.25.5 resolves the issue.

we are also seeing this
2024/04/09 05:52:14  pq: constraint "uni_image_statistics_image_id" of relation "image_statistics" does not exist
[0.955ms] [rows:0] ALTER TABLE "image_statistics" DROP CONSTRAINT "uni_image_statistics_image_id"

,"error":"pq: constraint \"uni_image_statistics_image_id\" of relation \"image_statistics\" does not exist"

I can confirm that - this error happens because in https://github.com/go-gorm/gorm/blob/v1.25.9/migrator/migrator.go#L569 field.Unique returns false when UniqueIndex is used on the models - simply swapping it  to unique;index fixes the problem for now, but field.Unique should be set to true if uniqueIndex is used

I wish this library:

used proper versioning - semver
had a changelog
test their changes
big asks fsho... I don't expect any of these to be implemented though. Who versions their libraries anyways... changelog? c'mon...

+1 on this issue. Also noticed this upon upgrading from 1.25.5 to 1.25.9.
This may be helpful for debugging this issue -
I noticed that creating tables on 1.25.5 and 1.25.9 results in different table metadata, which is likely related.
Looking at the \d+ output of a table:

1.25.5 creates an idx_<table>_<column> index, as a UNIQUE CONSTRAINT.
1.25.9 creates an idx_<column> index, but only as UNIQUE; not as a constraint.

Not stale. Doesn't look fixed to me. @jinzhu

If all it wants is a playground link...  I added the playground link to the first message ...

 I added the playground link to the first message ...

What it needs is a link to a pull request in the playground repo. See example how it is here: https://github.com/go-gorm/gorm/issues/6963 - it links to a playground PR where the PR code is a reproducer for the bug

@varfrog that ticket is marked as stale as well. I don't understand how and why I'd even want to open a PR to anywhere for this topic.
As I'm migrating off of this library, I'll also unsubscribe to this PR. Gorm is not something I'm interested in using anymore.

@ChristianSch do you know some good alternatives? I'm also planning to migrate to some other tool

@klipach pgx + scany is what I'm using

I am also encountering this issue after upgrading from v1.25.5 to v1.25.9. The (abbreviated) definition of my model is:
type Service struct {
	gorm.Model
	Name string      `json:"name"`
	Nickname string  `gorm:"default:NULL;uniqueIndex;check:((nickname IS NOT NULL AND NOT nickname = '' AND nickname ~ '^[a-z0-9_-]+$') OR quick_quote_config_id IS NULL)"`
	ConfigID *uint   `json:"-" gorm:"default:null;check:chk_services_config,(config_id IS NOT NULL OR nickname IS NULL)"`
	Config   *Config `json:"-"``
}

Pinning to v1.25.5 solved my issue.

Requesting to reopen this @jinzhu - this is a breaking change that is impacting multiple people

we are migrating away from gorm, there has been too many issues like this. Our weapon of choice is go-migrate and sqlx.

The versioning in this library is so cursed. Just got hit by what seemed like a patched version. (1.25.2 -> 1.25.10)


Debug Native Custom Widget

I am a newbie in React Native!
I am following the following links:
https://docs.mendix.com/howto/mobile/getting-started-with-native-mobile
and
https://docs.mendix.com/howto/extensibility/pluggable-widgets
So now I can create my custom native widget and can see it on my physical IPhone (not Simulator)
However, if i want to debug my script, it is not straightforward how to do it.
Could you add a section to clarify the way to debug?
(If possible, how to see my app on Simulator install on my PC also, it is not really convenient to use my phone all the time)

We are outlining some widget debug documentation now. Thank you for your comment.

Hi @vietduc1441 ,
I'm sorry for the long delay on this. We have some more documentation and tech to help fix your issue. Specifically, we have an element inspector which helps speed up native widget debugging. You can find the details in the [Debugging Your Styling]([https://docs.mendix.com/howto/mobile/native-debug#3-debugging-your-styling] section of Debug Native Apps.
If you wish to use an emulator for Android mobile testing, install a product such as Bluestacks or Genymotion (your emulator must have Google Play services supported).
I hope this speeds up your debugging. Can I help you with anything else?
Connor


Website cut off

The top and bottom of the website are cut off on my screen. I'm using Google Chrome Version 46.0.2490.86 m

Thanks for letting us know about the top and bottom of the website getting cut off in Chrome, @MelissaSims.  We're investigating, and we'll let you know when it's fixed!

Hi @MelissaSims.  I think I initially misunderstood your bug report -- just in case you're referring to the cut off logotype ("A List Apart") and footer text ("*For People Who Make Websites"), those are design choices, rather than layout bugs.  If you're referring to something else, please let me know.
I did notice incorrect whitespace below the footer text while reviewing this issue, so we'll update the site to remove that.
Please let us know if you spot any other issues!


Sstc description hasn't been merged well

In this chapter:

Chapter 16. "Sstc" Extension for Supervisor-mode Timer Interrupts, Version 1.0.0

I don't think this text should be there anymore

To make it easy to understand the deltas from the current Priv 1.11/1.12 specs, this is written as the
actual exact changes to be made to existing paragraphs of Priv spec text (or additional paragraphs
within the existing text).

and in these chapters:

3.1.18. Machine Environment Configuration (menvcfg) Register
18.2.5. Hypervisor Environment Configuration Register (henvcfg)

there is this text:

The definition of the STCE field will be furnished by the forthcoming Sstc extension. Its allocation
within [mh]envcfg may change prior to the....

Also the stce bit of Xenvcfg (which I was actually looking for when I found this) isn't mentioned in the Sstc chapter at all, so the definition of that field seems to be missing.

I believe @gfavor is aware that one of the many, many things on his to-do list is to better integrate the Sstc chapter.  (Sstc is certainly not the only offender in this regard.)


Contributed tree's indent guide overlaps with icon

Repro:

Checkout a PR
Click Files
My file icon is Ayu

Version: 1.37.0-insider
Commit: 46cc0a1670b7047e610aaef1bf84f1190802e9ec
Date: 2019-07-11T05:37:24.194Z
Electron: 4.2.5
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.6.0

How can I get that view? I can't seem to get it in the Github Pull Request Extension.

Oops I forgot to post a reply. Simply checkout a PR:


Update static files cache TTL

Update default cache control header for static files to a year.
https://flask.palletsprojects.com/en/1.1.x/config/#SEND_FILE_MAX_AGE_DEFAULT

Toto, what are the motivations for this change?

@jkfran static content can be cached for ever, we have a hash that would show if there is an update. This will avoid requesting too often the same assets from the same browser.
eg. https://charmhub.io/static/js/dist/base.js?v=4dd35c4

@jkfran static content can be cached for ever, we have a hash that would show if there is an update. This will avoid requesting too often the same assets from the same browser.
eg. https://charmhub.io/static/js/dist/base.js?v=4dd35c4

It's what I was thinking! Cool! I'm wondering if caching all the static files and their versions for a long time could cause a space problem for IS


Reserved word attributes: documentation for add_attribute

FactoryGirl errors in a number of ways if you have an attribute that is a reserved word.

factory :payment do
method 'paypal'
end

> .rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/factory_girl-4.5.0/lib/factory_girl/definition_proxy.rb:122:in `sequence': wrong number of arguments (0 for 1+) (ArgumentError)

>```
factory :dna do
  sequence { "GATTACA" }
end

factories.rb:23:in `method': wrong number of arguments (0 for 1) (ArgumentError)

You can use sequence to override as mentioned in this closed ticket. This works for both examples.
factory :dna do
  sequence(:sequence) { "GATTACA" }
end

This method is not in the documentation anywhere and as @drapergeek mentions in the ticket is "not the prettiest solution." I agree. It seems there is still an add_attribute method in FactoryGirl::DefinitionProxy that seems to work, but is not mentioned anywhere in the wiki.
factory :dna do
  add_attribute(:sequence) { "GATTACA" }
end

I think some documentation is all that is needed.  If you agree I'd be happy to work up a PR for the wiki.

@andynu Great point. I think the GETTING_STARTED doc might be a better place to put this, in a section titled Reserved Method Names or something similar. Fascinating that we don't have public docs on add_attribute - that seems like a think we should include!

It took a while to find this, leaving a note for future googlers:
`color': wrong number of arguments (0 for 2) (ArgumentError)


ExperienceReplayBuffer stores second-last transition twice

Hi Maxim,
first of all, thank you so much for the book! It helps me a lot for my thesis!
Second, I think that the ExperienceReplayBuffer stores the second-last transition twice, which could bias the training if an environment only has a few steps (like mine).
Maybe I have overlooked something, but this is my minimal example showing the described behaviour:
import ptan
import torch
import torch.nn as nn
import torch.nn.functional as F
import numpy as np
import gym

EPSILON_START = 1.0
GAMMA = 1.0
REWARD_STEPS = 1
LEARNING_RATE = 0.001
MAX_STEPS = 20
REPLAY_SIZE = 10
MAX_STEPS_PER_EPISODE = 3
device = torch.device("cpu")

class Environment(gym.Env):
    def __init__(self):
        self.state = 0
        self.observation_space = gym.spaces.Discrete(5)
        self.action_space = gym.spaces.Discrete(2)

    def reset(self):
        self.state = 0
        return self.state

    def step(self, action):
        self.state += 1
        if self.state == 4:
            done = True
        else:
            done = False
        reward = self.state
        return self.state, reward, done, None

class DQN(nn.Module):
    def __init__(self, input_shape, n_actions):
        super(DQN, self).__init__()
        self.n_actions = n_actions

    def forward(self, x):
        return torch.rand(1,self.n_actions)

env = Environment()
net = DQN(env.observation_space.shape, env.action_space.n).to(device)
selector = ptan.actions.EpsilonGreedyActionSelector(EPSILON_START)
agent = ptan.agent.DQNAgent(net, selector, device=device)
exp_source = ptan.experience.ExperienceSourceFirstLast(env, agent, GAMMA, steps_count=REWARD_STEPS)
buffer = ptan.experience.ExperienceReplayBuffer(exp_source, REPLAY_SIZE)

step_idx = 0

while step_idx < MAX_STEPS:
    step_idx += 1
    buffer.populate(1)
    new_rewards = exp_source.pop_rewards_steps()

    if new_rewards:
        print("episode over: step {}: (total_reward, steps) = {}".format(step_idx, new_rewards[0]))

print()
print(*buffer.buffer, sep='\n')

The output is:
episode over: step 6: (total_reward, steps) = (10.0, 4)
episode over: step 11: (total_reward, steps) = (10.0, 4)
episode over: step 16: (total_reward, steps) = (10.0, 4)

ExperienceFirstLast(state=0, action=0, reward=1.0, last_state=1)
ExperienceFirstLast(state=1, action=0, reward=2.0, last_state=2)
ExperienceFirstLast(state=2, action=0, reward=3.0, last_state=3)
ExperienceFirstLast(state=2, action=0, reward=3.0, last_state=3)
ExperienceFirstLast(state=3, action=0, reward=4.0, last_state=None)
ExperienceFirstLast(state=0, action=0, reward=1.0, last_state=1)
ExperienceFirstLast(state=1, action=0, reward=2.0, last_state=2)
ExperienceFirstLast(state=2, action=0, reward=3.0, last_state=3)
ExperienceFirstLast(state=2, action=0, reward=3.0, last_state=3)
ExperienceFirstLast(state=3, action=1, reward=4.0, last_state=None)

Notice how the transition from state 2 to 3 is stored twice each time. I used the ptan version that you can get via pip today. Could you have a look into this?
Best regards!

Hi!
Thanks for reporting! Will take a look on this.

That's indeed a bug, fixed in my dev branch https://github.com/Shmuma/ptan/tree/torch-1.0.1

Hi,
I tried the new version (again with the minimum working example) and the error still persists unfortunately :-(


Pretty date

Allow pretty date to be set via options

There's a wider question over whether req.form[key + '-formatted'] should even exist. I created it in order to more closely mimic HOF's date-controller. (But the functionality could probably be moved into HOF if it is unwanted in HMPO tech.)
Possibly something nicer could be done using the formatters?
Perhaps it just shouldn't exist and code that wants nicely formatted dates should simple do their own formatting rather than expecting the date controller to have done it for them?
What do people think?
@gavboulton, @JoeChapman, @easternbloc, @joefitter, @oli-weighell2: Ping!

Yeah, I agree we shouldn't be storing the formatted date this way, at least at the base controller level. IMO it should be down to the individual app to format the date

I'd say date formatting is a views thing. Template mixins has a date method which allows a custom format and defaults to 'D MMMM YYYY', https://github.com/UKHomeOffice/passports-template-mixins/blob/master/lib/template-mixins.js#L325-L331

We could add a mustache lambda for displaying the formatted date, Ideally this would live in hof in a a shared partials folder or something

ha - what he said ^

@gavboulton: Very nice. Sounds like we should be using that instead!

closing this as we can format the output using the mixins.


fix: field config type accuracy when calling extendTypes

Improve inferring types for the resolver parameters when adding GraphQL fields using extendTypes.
Before
extendTypes({
  Order: {
    formattedStatus: defineField({
      args: { extra: 'String' },

      resolver: ({ source, args }) => {
        return `Order status: ${source?.status}${args.extra ?? ''}`
      },
      returnType: 'String!',
    }),
  },
})

Now
extendTypes({
  Order: {
    formattedStatus: {
      args: { extra: 'String' },

      resolver: ({ source, args }) => {
        return `Order status: ${source.status}${args.extra ?? ''}`
      },
      returnType: 'String!',
    },
  },
})

So using defineField is not necessary anymore (is now marked as @deprecated). Also noticed that source?.status is now source.status which is the biggest improvement: the source parameter is now inferred from the parent type so the resolver is now aware that the source should be of type Order. For Query and Mutation, the source will be undefined.
I also added integration tests for those fields added using extendTypes. Moreover, to make sure we keep the accuracy of the inferred types, a types:check script was added to the dev-playground package.

:tada: This PR is included in version graphql-gene-v1.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Prepare Pizza to bake

Instructions
When the oven is ready, dust a pizza peel with cornmeal or semolina flour. (If you don’t have a pizza peel, you can use a rimless baking sheet or the back of a rimmed baking sheet. But a pizza peel is well worth the investment!) Stretch the dough into a circle; see How to Stretch Pizza Dough for instructions. Then gently place the dough onto the pizza peel.

Ensure oven is 500f.

prep done


feat: fix contract membership check

Please provide a paragraph or two giving a summary of the change, including relevant motivation and context.
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Benchmark results
Metrics with a significant change:

circuit_simulation_time_in_ms (private-kernel-init): 645 (+682%)
circuit_simulation_time_in_ms (private-kernel-ordering): 107 (+112%)
circuit_simulation_time_in_ms (private-kernel-inner): 599 (+924%)
note_history_successful_decrypting_time_in_ms (5): 1,920 (-25%)
note_history_successful_decrypting_time_in_ms (10): 3,747 (-25%)
note_successful_decrypting_time_in_ms (8): 284 (-17%)
note_successful_decrypting_time_in_ms (32): 819 (-22%)
note_successful_decrypting_time_in_ms (128): 2,953 (-24%)
note_trial_decrypting_time_in_ms (8): 85.0 (+93%)
note_trial_decrypting_time_in_ms (32): 30.0 (-74%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 388a47ba and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
223,008
868,196
3,447,980

l1_rollup_execution_gas
842,095
3,595,304
22,203,349

l2_block_processing_time_in_ms
2,295 (+2%)
8,713 (+1%)
34,299

note_successful_decrypting_time_in_ms
:warning: 284 (-17%)
:warning: 819 (-22%)
:warning: 2,953 (-24%)

note_trial_decrypting_time_in_ms
:warning: 85.0 (+93%)
:warning: 30.0 (-74%)
135 (-1%)

l2_block_building_time_in_ms
14,589
58,005
231,334

l2_block_rollup_simulation_time_in_ms
13,275
52,752
210,541

l2_block_public_tx_process_time_in_ms
1,277 (-5%)
5,123 (-3%)
20,343 (-4%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
24,916
48,655

note_history_successful_decrypting_time_in_ms
:warning: 1,920 (-25%)
:warning: 3,747 (-25%)

note_history_trial_decrypting_time_in_ms
139 (+15%)
144 (-1%)

node_database_size_in_bytes
1,631,251
1,101,852

pxe_database_size_in_bytes
27,188
54,187

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
:warning: 645 (+682%)
61,697
18,905

private-kernel-ordering
:warning: 107 (+112%)
24,297
8,153 (+1%)

base-rollup
2,086
656,311
814

root-rollup
91.9 (+3%)
4,072
1,097

private-kernel-inner
:warning: 599 (+924%)
81,568
18,905

public-kernel-private-input
46.6
41,519
18,841

public-kernel-non-first-iteration
28.6
41,497
18,841

merge-rollup
0.906 (+11%)
2,592
873

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
8,787 (+1%)
27,304 (-7%)


Setting to align scrolled item (e.g to bottom)

It would be good to have a setting to specify how scrollable item is aligned in container.
I have read this issue https://github.com/flesler/jquery.scrollTo/issues/172 but it is not quite it since this talks about the case where item is scrolled always to bottom but I think this thing needs also "auto" setting where item is scrolled to nearest edge, i.e to start when it is behind top/left edge and to end when it is behind bottom/right edge. It should respect also the case not to scroll when the target is already visible.
E.g align

start - scrolls to starting edge (top or left depending on axis), default
center
end
nearest (or "auto"?)
{ x: start/center/end/nearest, y: start/center/end/nearest }

if I'm understanding correctly, I think you can handle most of that with offset and over. Have you checked those out?

Yes, I am aware of offset. But I cant see how "over" helps me.
But even if they can help me positioning target item, there is an issue of how to not scroll if item is already visible?

@kosmodisk I understand the bit about not scrolling, my personal philosophy (with this plugin) was a build a good building block that does its bit well and can be combined. NOT scrolling happens before and the logic people could need for not scrolling varies greatly. So my decision for that, was that people should decide if it scrolls or not from "the outside". That is, there are incredible jQuery plugins to check if stuff is "above the fold", "below the fold", on screen, etc.
I'd recommend, on that regard, to combine it whatever suits your specific needs.
As for offset + over, I haven't experimented but seems to me like this can be achieved with those. I'm not currently shipping my own code to this plugin, I (try to) reply to GH issues and review & accept PRs. If you want to provide a PR for that setting, I'll review it gladly


Failed to load: Error: NameProperty data type for 'IndustryNameArray' is not implemented

Describe the bug
Opened a brand new save into Railroad Studio. Nothing previously built in the world.
To Reproduce
Steps to reproduce the behavior:

Create new save game in RRO (No tracks built)
Open save in Railroad Studio
Error given

Expected behavior
Save opens
Desktop (please complete the following information):

OS: Windows
Browser: Opera GX

Thank you for the report @OrionsStarr. I have added support for the NameProperty data type, as well as two others that were missing. I'm going to resolve this issue and use https://github.com/scottanderson/railroad.studio/issues/63 to track other issues related to loading saves with the new update.
Please feel free to reopen this issue or create a new one if you see another data type is not implemented error. Please include a save file with your report to make it easier to reproduce the issue.


[Feature request] Sessions token Auth for API and secure file downloading.

The api should be able to interact with the loader much like how keyauth functions where session tokens are used. And if you were to add secure file downloading, if there was no session created, meaning they forced a success response illegitimately, they wouldn't be allowed access and the user could customize it to ban them or whatever. If these 2 things were added, I'd switch from KeyAuth and maybe make some pull requests with more cool things.
[https://github.com/KeyAuth/KeyAuth-Source-Code](The KeyAuth source code)
[https://github.com/KeyAuth/keyauth-cpp-library](The KeyAuth client api source code)

Hi,
Thanks for your suggestion.
Okay a few things:
The suggestion for ban customization is a very good idea.
Could you also explain what API interactions/functions etc would be important to you?
Because just shitting the github link of keyauth in the issue doesn't help us.
I can tell you that the API will deny access if the user authentication fails.
Also, the panel also uses session tokens but not for the API.
The API has been neglected because most people are only interested in user-pass auth, username, ban status, and hwid.
But I can't imagine that the API will be extended/changed much.
Simply for the reason that we lose the desire to work on the panel because most people just paste it and create issues because they are too stupid to follow the setup or have broken it by pasting and then want us to fix it.
Also, it gets very low attention.

If you added sessions to the authentication aswell as file downloads, I'd make a toturial and make these retards quit opening issues. Also it should just have sessions idk how else to explain which is why I linked KeyAuth. Literally just look at client side library init function and login function and just big skid. I could even make my own c++ loader implementing everything if you made it serverside. Also a major keyauth flaw aswell as almost any auth is the success message being spoofed or set to true. This could be fixed by making the success message dynamic using a randomly generated key which only works for that session ig.

Hi,
I am working on another feature in combination with the last major update for an unspecified time.
I will implement such a system with several custom messages (probably a very basic version without variables).
If in the future I make another major update I will certainly come back to your suggestions.
Thx ;)

Hi,
I am working on another feature in combination with the last major update for an unspecified time.
I will implement such a system with several custom messages (probably a very basic version without variables).
If in the future I make another major update I will certainly come back to your suggestions.
Thx ;)

Nice to here. But if you would add the basis for a great auth with sessions and enckeys etc just like keyauth, this can be a really great auth. I will go through the effort of making a c++ example for the panel when you add this.


Update design of problems & solutions card

Edited css in chrome to get to a suitable design. Couldn't figure out how to edit bootstraps' scss files lest I spoil some other component of the website.
Attaching screenshots for the parameters that I edited to get to the desired design.

designed the same as attached in the screenshot


[FEATURE] Rebuild Volume Performance Enhancement

Is your feature request related to a problem? Please describe.
After reboot the Harvester node, the node disk manager will need to rebuild its volumes and validate the integrity of the disk volumes. This will take an extra bit of time and consume both the disk I/O and memory usage, so a large-scale VM cluster will be slow during the rebuild period of time, consider making an enhancement on this.

Describe the solution you'd like
Depends on https://github.com/longhorn/longhorn/issues/2460
Describe alternatives you've considered
N/A
Additional context

should be addressed via https://github.com/longhorn/longhorn/issues/2460, will need to verify via reboot the node with multiple VMs and check the concurrent rebuild on the Longhorn UI.

Verified using master-8542e305-head - passed

deploy a few VMs(e.g 5) on the same node and reboot the node manually
the volumes show unconnected on the Longhorn's volumes UI

wait for the host to be restored and check those attached volumes
All volume is re-attached and only maximum of 3 volumes is under concurrent rebuild


Release 0.21.2

Checklist

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

@conda-forge-admin, please rerender

At @jchodera when you get a chance can you look at this and approve it if it looks good?
Once this gets merged @ijpulidos we can get the CLI PR merged in, but I think we should do a re-build of the perses conda-forge package and set the minimum version for openmmtools to this version so that the CLI will work consistently. Another option is I could just build the docker image with the right openmmtools version, but it would be really annoying for someone to download perses and then try to test the CLI to then get hit with an error. Thoughts?

I agree with you. I guess the easier way is to get this released and then pin a minimum openmmtools version for the upcoming perses release with the new CLI. The other option seems like it might get confusing for users, in my opinion. I know there are still some things to think about form the perses side.


Error raised when calling Rails.cache.clear

When I try to call Rails.cache.clear I have this error message
...active_support/cache/litecache.rb:51:in `clear': wrong number of arguments (given 1, expected 0) (ArgumentError)

I think the signature of method should be
-def clear()
+def clear(options = nil)
  @cache.clear
end

Hi there! Wanted to comment on this older issue since it still seems to occur with v0.4.2. I checked the master branch of this project and saw that the change in #29 doesn't seem to be present in the litecache.rb file here.


feat!: optimize uploading of data

Parallelized lots of processes that did not need to be done sequentially when uploading data. The main increase in efficiency is from uploading and updating each test in parallel, but other processes are in parallel as well. (See diagram here).
Another optimization came from storing & maintaining in each test case a list of the 15 most recent runs of the test case. This means that there no longer needs to be 15 reads per test case. It turns out, the performance benefits of this were minimal since writing takes much more time. However, this reduces the cost of an upload to 1/2 to 1/3 what it previously was.
Previously, it took 25 seconds to upload a single build with 50 test cases. Now, it averages 27 seconds to upload a build with 3000 test cases. The time varies linearly with number of test cases, so it may have taken >10 minutes to upload all 3000 tests before.
See results: Google Sheets Graph
Breaking Change: Now only stores test names under a build
Breaking Change: A build's flaky test count now refers to the number of tests which are currently flaky and failed on that build
TODO: Test to see if using Firestore batch feature speeds it up furthur
I have yet to experience this issue with the new upload method, which previously randomly came up when calling addBuild many times at once. I do not know if it is fixed, or just less likely to occur now.
Resolved Connection Issues: Many of the connection issues were fixed by queuing updates to be done 50 at a time. The remaining connection issues were likely due to a combination of bad internet on development computer. On a better computer, ~20 builds with 3000 tests were able to be updated currently.
Resolved Concurrency Issues: All new recent dates are added using Arrayunion, and all old dates are removed using Arrayremove when mantaining a list of the 15 most recent dates on which a particular test was run.
 Error: 4 DEADLINE_EXCEEDED: Deadline exceeded
    at Object.callErrorFromStatus (/home/stuartrucker/flaky-dev/packages/api/node_modules/@grpc/grpc-js/build/src/call.js:30:
```26)

given this is a breaking schema change, I believe we will want to coordinate updating our collection name with the work when we deploy.


Merges Master to Live. See commits for details.

Docs Build status updates of commit d3c8e9d:
:clock10: Full build: the average full build time is 13 min(s) 22 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit d3c8e9d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit e503e75:
:clock10: Full build: the average full build time is 13 min(s) 23 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit e503e75:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Implement curved spacetime hydrodynamics

The special relativistic 5-moment equations currently implemented in Gkeyll permit a straightforward extension to curved spacetimes (i.e. to fully general relativistic hydrodynamics in the test fluid limit) by means of the fully-conservative, strongly-hyperbolic "Valencia" formulation of Banyuls, Font, Ibáñez, Martí and Miralles. The only differences between these curved spacetime hydrodynamics equations and the flat spacetime case are a gauge dependence (i.e. an appearance of the lapse function and the shift vector) within the flux vector, and the appearance of source terms proportional to certain derivatives of the Christoffel symbols. Both of these differences can be absorbed into a small modification of the finite-volume boundary extrapolation step, such that the equations that one is solving at the cell centers remain purely special relativistic.
Subsonic, steady-state accretion of a stiff relativistic fluid (i.e. an ultrarelativistic fluid with Gamma = 2, i.e. where P = rho) onto a Schwarzschild or Kerr black hole, quite surprisingly, admits an analytic solution due to Petrich, Shapiro and Teukolsky, against which one can therefore make direct numerical comparisons for the sake of unit and regression testing. Having a robust and well-validated general relativistic hydrodynamics model (at least in the test fluid limit) implemented within Gkeyll is a crucial initial step towards being able to do full GRMHD and even kinetic simulations in curved (static) spacetimes. With this in mind, the implementation steps that I currently have in mind are:

Increase the robustness of the current Gkeyll implementation of the Eulderink-Mellema iterative scheme for doing conservative -> primitive variable reconstruction in SRHD, especially in the presence of strongly relativistic shocks, low densities and low pressures. This can be achieved by first modifying the formula for extracting the Lorentz factor W by pulling the (C_0 * xi) term outside of the brackets, since this term becomes arbitrarily large for strongly relativistic shocks, and the term inside the brackets should be kept at order unity if possible. From there, one can enforce a minimum value of the fluid density, and then recalculate the pressure from the (potentially modified value of the) specific relativistic enthalpy. This scheme can be validated against both the mildly and strongly relativistic blast wave tests of Del Zanna and Bucciatini, since the latter problem in particular is an extremely effective and demanding test for assessing the robustness of the primitive reconstruction scheme. Riemann problems for the one-dimensional relativistic hydrodynamics equations admit an analytic solution due to Pons, Martí and Müller, against which we can write appropriate unit and regression test comparisons (along with the 2D quadrants test, which currently fails). Though not strictly a Riemann problem, we can also perform comparisons against the perturbed density test, since this will be crucial for assessing the extent to which we are able to resolve fine, oscillatory features.
Implement an abstract context struct for storing relevant spacetime data (for the case of black hole spacetimes, this would likely consist of parameters for mass, spin, position of the black hole, and possibly linear momentum of black hole if we intend to support non-comoving frames), which is then passed to all wv_gr_euler functions.
Add helper functions for computing relevant spacetime quantities on the basis of this context struct; in particular, we will need to add functions for calculating the components of the spatial and spacetime metrics (plus their determinants, and their inverses, for raising and lowering indices), the lapse function (and its derivatives), the shift vector (and its derivatives), and the components of the extrinsic curvature tensor. For the case of black hole spacetimes in Kerr-Schild coordinates, these quantities all admit known analytic solutions, which we can implement directly. Ideally, we should aim to implement this in a manner that is sufficiently general as to admit most or all standard coordinatization schemes for black hole spacetimes (e.g. Boyer-Lindquist/oblate spheroidal coordinates, general horizon-adapted coordinates, Cartesian Kerr-Schild coordinates, spherical Kerr-Schild coordinates, etc.)
Implement excision boundary conditions at the event horizon, in the case that one chooses not to use horizon-adapted coordinates. This can be achieved either by using the existing (provisional) multi-block infrastructure to excise a given region of the domain directly in computational coordinates, and then impose physical boundary conditions at the excision point, or by introducing a fictitious scalar field that delineates the interior and exterior spacetime regions (and exploiting the sign change in this field to allow the solver to detect the event horizon efficiently, similar to a level set method). I am currently undecided regarding which of these two approaches is preferable.
Implement appropriate modifications of the Roe, HLL and HLLC solvers for the curved spacetime Euler case (incorporating gauge and metric dependence in the boundary extrapolation step for the Riemann solver), along with wave speed estimates based on the known eigendecomposition of the full GRHD system due to Banyuls et al.
Perform comparisons against analytic spherical Bondi-Hoyle and non-spherical Hoyle-Lyttleton accretion problems onto both Schwarzschild and Kerr black holes, comparing against the analytic solutions of Petrich et al. in the stiff relativistic case, and the numerical solutions of Font, Ibáñez and Papadopoulos in the more general ultrarelativistic case in axisymmetry.

This looks like an excellent plan, specially given the work we will pursue this summer on the astro front. If we have the hyrdo, Maxwell and implicit sources (Lorentz + currents) completed this summer then we will be set to do some really cool, and totally unique, simulations of the plasma around compact objects!


Version21

Amnon, no metric name changes in 2.1?

This is taken from Master, It looked fine.
On Tue, Dec 19, 2017 at 12:50 PM, Tzach Livyatan<EMAIL_ADDRESS>wrote:

Amnon, no metric name changes in 2.1?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scylladb/scylla-grafana-monitoring/pull/259#issuecomment-352709694,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACBRv3UXGfDBX8MDPtR2MG5UEFxKP7V2ks5tB5T0gaJpZM4RGsKC
.

ok
On Tue, Dec 19, 2017 at 12:51 PM, Tzach Livyatan<EMAIL_ADDRESS>wrote:

@tzach commented on this pull request.
In versions.sh
https://github.com/scylladb/scylla-grafana-monitoring/pull/259#discussion_r157721322
:

@@ -1,3 +1,3 @@
-LATEST="2.0"
-ALL="1.5,1.6,1.7,2.0,master"
-DEFAULT_VERSION="1.7,2.0"
+LATEST="2.1"
+ALL="1.5,1.6,1.7,2.0,2.1,master"
+DEFAULT_VERSION="2.0,2.1"

@amnonh https://github.com/amnonh with the default update, I can not
merge till 2.1 is out
If you remove this change, we can merge now, and update teh default once
2.1 is out
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scylladb/scylla-grafana-monitoring/pull/259#pullrequestreview-84409465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACBRv3sZScmIq9HNgqiJthcB6d_F2Invks5tB5U4gaJpZM4RGsKC
.

On Tue, Dec 19, 2017 at 1:06 PM, Tzach Livyatan<EMAIL_ADDRESS>wrote:

@tzach commented on this pull request.
In grafana/scylla-dash-io-per-server.master.template.json
https://github.com/scylladb/scylla-grafana-monitoring/pull/259#discussion_r157724543
:

@@ -74,7 +74,10 @@
"transparent": false
},
{

                   "class": "dashlist"

                   "class": "dashlist",

                   "tags": [

                   	"master"

                ]

@amnonh https://github.com/amnonh why do we need to update the types in
the template?
I was assuming we will not have to change the template with each release

This is a tag that specify what version this is, so for master it will be
master,
for 2.1 it will be 2.1.
So we must change this tag each release

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scylladb/scylla-grafana-monitoring/pull/259#pullrequestreview-84413043,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACBRvzgb-zpxsNgvnInmVDxY9OrxaRgXks5tB5iugaJpZM4RGsKC
.


chore(main): release 1.28.0

:robot: I have created a release beep boop
1.28.0 (2024-12-20)
Features

✨ Support pairing and connecting to devices via QR code (14306b2)
🚀 Supports connecting historical devices via device list (e0687e8)

Bug Fixes

🐛 Solve the problem of Windows disabling video forwarding and crashing (10b902a)

Performance Improvements

♻ Update mirror icon (c214e5a)
✅ A new method is adopted to solve the problem of IP type configuration storage (5488a1e)
🚀 Optimize device connection experience (0690f2d)
🚀 Supports display of offline devices (dcf817f)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.28.0
:sunflower: