"coqide -help" hangs when installed with the Coq platform, etc.

"coqide -help" hangs when installed with the Coq platform (1/22/22 version), whereas "coqide" by itself works.  I suspect this is from incorrect handling of command line arguments when CoqIDE is installed--IIRC this option works in the development environment [which I can't try at the moment].
CoqIDE comes up with coqide -I _build_ci/rewriter/src/Rewriter/Util/plugins.
"coqide --version" doesn't hang, but doesn't give any output, either.
@ejgallego Any thoughts?
For comparison, "coqtop -help" shows the usual
$ coqtop -help
Usage: coqtop <options>

Coq options are:
  -I dir                 look for ML files in dir
  :

Triage note: retest with current CoqIDE.


Error TS2339: Property 'replace' does not exist on type 'string | URL'.

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.

Is this a client library issue or a product issue?
This is the client library for . We will only be able to assist with issues that pertain to the behaviors of this library. If the issue you're experiencing is due to the behavior of the product itself, please visit the  Support page to reach the most relevant engineers.

Did someone already solve this?

Search the issues already opened: https://github.com/googleapis/gcp-metadata/issues
Search the issues on our "catch-all" repository: https://github.com/googleapis/google-cloud-node
Search or ask on StackOverflow (engineers monitor these tags): http://stackoverflow.com/questions/tagged/google-cloud-platform+node.js

Do you have a support contract?
Please create an issue in the support console to ensure a timely response.

If the support paths suggested above still do not result in a resolution, please provide the following details.
Environment details

OS: Red Hat Enterprise Linux
Node.js version: v20.15.0
npm version: v10.7.0
gcp-metadata version: v6.1.0

Steps to reproduce

run npm install
Getting error message:
src/index.ts:184:23 - error TS2339: Property 'replace' does not exist on type 'string | URL'.
Property 'replace' does not exist on type 'URL'.

184     url: options.url!.replace(getBaseUrl(), getBaseUrl(SECONDARY_HOST_ADDRESS)),
~~~~~~~
Found 1 error in src/index.ts:184
Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

The main branch has the correct code to handle this situation for replace function i.e. its converting the url to string first.
url: options.url
?.toString()
.replace(getBaseUrl(), getBaseUrl(SECONDARY_HOST_ADDRESS)),
Line no 185 in gcp-metadata/src/index.ts (main)

We will cut a release that will pull in this fix shortly:

https://github.com/googleapis/gcp-metadata/pull/633


Nullaway does not recognize @NullableDecl from the checker framework.

Recently, guava has moved from jsr 305 to the annotations of the checker framework. And it uses @NullableDecl extensively. Do you have any plan to support this?
Thank you.

#27

Sure, we can support this.  I'll look into it.

@JakeWharton just supporting @NullableDecl is probably a lot less work than fixing #27 (which would require integrating parsing of Checker Framework stub files, etc.).  Is there a strong need for fixing #27 as well?

I assumed this would be a subset, as more an more of checker is supported.
I guess that issue is solely for their JDK stuff though.
On Thu, Dec 14, 2017 at 3:03 PM Manu Sridharan<EMAIL_ADDRESS>wrote:

@JakeWharton https://github.com/jakewharton just supporting
@NullableDecl is probably a lot less work than fixing #27
https://github.com/uber/NullAway/issues/27 (which would require
integrating parsing of Checker Framework stub files, etc.). Is there a
strong need for fixing #27 https://github.com/uber/NullAway/issues/27
as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uber/NullAway/issues/82#issuecomment-351820574, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEEVvDyDIKuJGPi_5nYb5pPXW85KlNks5tAX79gaJpZM4RChNn
.

I assumed this would be a subset, as more an more of checker is supported. I guess that issue is solely for their JDK stuff though.

Yup, this could kind of be viewed as a subset, assuming their JDK stuff is also using @NullableDecl.

Thanks, Manu.

@chengniansun would it be helpful for us to cut a new release with this fix?

Thank you. I have no preference as I have switched to the SNAPSHOT this morning.


Deal with the occasional runaway FITS process

How can we stop the occasional runaway FITS process. This has been observed on one particular WAV file so far, which has been added to the CDR test corpora.

After setting a pre-emptive MALLOC OOM killer config and setting a Java -Xmx256mb setting, the java processes seem to just hang around, rather than expand to fill swap. This is an improvement, but java processes still accumulate, up to the number of CDR services threads that are running. In catch up mode this number seems to be 8 currently..

The java memory cap caused a java out of memory exception, with a stack trace reported back to the cdr services layer. This trace tells us what tool was running when java ran out of memory. It was an audio info tool called hclaps. We have disabled that tool in FITS and are re-running the FITS on DEV. So far some runs have taken a long time, but they have all finished successfully.

The last progress made on this was isolation of the DROID tool as responsible for runaway processing of corrupt WAV audio files. When we run FITS without DROID, JHOVE tells us that the file is corrupt. When you run FITS with DROID or even the latest DROID standalone, the java process doesn't stop.
So we've disabled DROID in production for all .wav or .WAV files.


redox: add openpty, login_tty, TIOCSCTTY, and organize functions

The added items are used by terminal programs.  I have organized some functions that were not sorted by header name. Sorting by header name makes it easier to keep this in sync with the Redox OS libc, relibc.

Thanks! @bors r+

:pushpin: Commit a6191c5f6f4e1a737eff83728173e2939211a6d2 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit a6191c5f6f4e1a737eff83728173e2939211a6d2 with merge ee1551a671f58a6f1ea2b1cac7559781ce4490f1...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing ee1551a671f58a6f1ea2b1cac7559781ce4490f1 to main...


Lean: fix bugs revealed by break the system competition

Fix axiom regex to correctly identify allowed axioms as abused here.
Disable global notation restriction again because it can easily be outsmarted as done here

The only way I am aware to solve the global notation cheat is to

Define the notation already in defs.lean
Import defs.lean in check.lean BEFORE submission.lean so that the elaborator will fail in case of a notation cheat - see this Zulip discussion.

(as much as I want to "break the system" again next month, I think I've annoyed you too much already so I'll mention it now)
I think
axiom «quot.sound : a» : false
theorem soundness_bug : false := «quot.sound : a»

still works even with this patch?

And now I am very confused, when I feed that input in even leanchecker looks like it gets confused and spits out just quot.sound : false?
If that wasn't the case maybe ^axiom (.*?(?= :)) : [^:]*$ would work?

Uff, yes you are right Alex - the leanchecker only spits out quot.sound : false in this case, which makes it impossible to detect the cheat at this stage. I posted this for an open discussion on Zulip

Fixed this issue (:crossed_fingers:) by matching the comparing the whole line containing an axiom. See here. Alternatively, one could use a different checker like trepplein, but I had issues setting it up at first and it requires more dependencies like sbt. I am in favour in keeping the setup simple.

I also updated mathlib. @maxhaslbeck  do not forget to merge before the competition.


Wrong position navigation/pagination in the version 0.10.0?

after first init call i see:

wrong position navigation:

navigationPrevLabel ist active, even first element
navigationNextLabel ist inactive, even first element and there are other elements

wrong position pagination:

last pagination dot ist active, even first element

In Version 0.9.0 everything worked correctly.
      <carousel
        :per-page-custom="[[540, 2], [720, 2], [960, 3], [1140, 3]]"
        :navigation-enabled="true"
        pagination-active-color="#c60039">
        <slide
          v-for="document in documentData"
          :key="document.id">

{{ document.documentDate | date }}

{{ document.title }} ({{ document.pages }} S.)

        </slide>
      </carousel>

Hey @jkwasniak, I'm not seeing this happening on my end. Care to send a JS fiddle or repo I can clone and test? Thanks - will flag accordingly now.

I experienced the same issue with version 0.10.0 on vue 2.5.2. Downgraded to 0.9.0 as suggested and it's working.
I'm using the global setup:
import VueCarousel from 'vue-carousel'
Vue.use(VueCarousel)
And my test was just a basic carousel with 2 slides, one per page.

Has this been resolved with the latest release? I'm unable to reproduce


SVG Export and CSS Formatting

Hello, I'm currently the gulp structure below (also attached) for my SVG icon system and have it exporting out to a php file to include in the head of Wordpress so I can use SVG's on the page.
For the most part everything works great however when I try to apply a hover: class to change the SVG fill, I'm not able to change the fill. I can change the fill if I use but not on hover. I assuming it's because of the path after the #ID.
Is there a better way I should be exporting my SVG or have my GULP file setup? The goal is to use symbol for my SVG's.
CSS
svg #pin path {
	fill:deepskyblue;
}

PHP

<svg><use xlink:href="#pin"/></use></svg>

Gulp
// SVG:
// Basic configuration example for SVG Sprite
var config                  = {
    shape               : {
        dimension       : {
            maxWidth    : 32,
            maxHeight   : 32,
            precision   : 2,
            attributes  : true,
        },
        spacing         : {
            padding     : 0,
            box         : 'content'
        },
        dest            : 'intermediate-svg'
    },
    svg                     : {
        xmlDeclaration      : false,
        doctypeDeclaration  : false,
        namespaceIDs        : true,
        dimensionAttributes : true,
        rootAttributes: {
            width: 0,
            height: 0,
            style: 'position:absolute; background:none !important'
        },
    },
    mode                : {
        view            : {
            bust        : false,
            render      : {
                scss    : true
            }
        },
        css             : {
            example     : true
        },
        inline              : true,
        symbol          : true
    }
};

// Build SVG sprites and move to build folder
gulp.task('svgSprite', function() {
	return gulp.src('assets/svg/*.svg')
		.pipe(svgSprite(config))
		.pipe(gulp.dest('assets/build/svg'))
		.pipe(notify({ message: 'SVG task complete'}));
});

// Images: Removes images not in src folder
gulp.task('cleanSVG', function(){
  del(['assets/build/svg/**/*', '!assets/svg', '!assets/svg/**/*'])
});

// Rename and create symbols php file
gulp.task('rename', function() {
	return gulp.src('assets/build/svg/symbol/svg/*.svg')
		.pipe(rename({
			basename: "svg",
			suffix: "-symbols",
			extname: '.php'
	}))
	.pipe(gulp.dest('parts'))
	.pipe(browserSync.stream())
	.pipe(notify({ message: 'Rename task complete'}))
});

Gulp-SVG.txt

Hi @jackfearing, if I understand your issue correctly this is not a problem that's related to svg-sprite / gulp-svg-sprite or the construction of the sprite. It's rather a limitation of current SVG support in browsers that you cannot apply arbitrary CSS on SVG elements that are <use>d. I'm pretty sure your hover code would work if your SVG was truly inlined (without the use of <use>).
Under certain circumstances there might be a solution for your special problem though, using currentColor in your CSS. Please see here for a description of the technique. Good luck!

Thanks for the quick response. Just a follow-up question. If I wanted to use an inline svg icon based the export gulp code. What would be the markup if I don't use the <use> tags?
Example:
How would I get this to display?
<svg class="notification"><use xlink:href="#notification"/></svg>


More robust against debris in the repository

Pull Request Description
Today Enso4Igv VSCode extension crashed for me. The reason is removal of runtime-with-polyglot & co. projects that leaves some debris (.enso_source* files) behind. The fix recognizes that and returns output folder instead and prints a warning.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] All code follows the
Java,
style guides.
All code has been tested:

[x] Manually tested

CCing @ola-lis

The VSCode Extension is ready for testing.

@ola-lis, please release the VSIX extension at the bottom of the referenced page. Thank you.


MWPW-162943-Fix button text not visible issue on tab

Fix button text not visible issue on tab

Resolves: MWPW-162943
Test URLs:
Before: https://stage--cc--adobecom.hlx.page/products/photoshop/ai?unitylibs=stage
After: https://stage--cc--adobecom.hlx.page/products/photoshop/ai?unitylibs=MWPW-162943

The code looks good, but, using the before URL, the CTA text does not disappear on focus 🤔 It gets the color: var(--color-black); from here.
My question is if this is still an issue? so that we don't push a fix for a problem that does not exist.

Hi @robert-bogos , I am still able to reproduce the error

issue tested with give branch and now tabbing focused on button CTA and can see the CTA test as well. PFA

@Ruchika4
I am able to reproduce the error now. I was losing the ?unitylibs=stage query parameter before, because of the microsoft login redirect.


Feature/ClaimRewards

damn why is this merged. I dont recall doing so.  I'll make sure nothing unfinished is visible to the user.


[android] fix reload jsservice bug

First of all, thank you for your contribution!
Please makes sure that these checkboxes are checked before submitting your PR, thank you!

CheckList:
if isBugFix :

[ ] Make sure that you add at least one unit test for the bug which you had fixed.

elif isNewFeature :

[ ] Update API docs for the component.
[ ] Update/Add demo to demonstrate new feature.
[ ] Update test scripts for the feature.
[ ] Add unit tests for the feature.

I will close this PR, feel free to reopen if you have problem.


Fix typo in lindera-cli/README

decopose ->
decompose

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: tracking of dev from main

dev should follow main.

pool token naming change
oracle owner to tx-sender
59760 token addition

Also include the fix #142. I tried to cherry-pick as many as possible in #157, but might have missed a few things.

done


Set, Bag and Mix operators not included or referenced in Perl6 Operator page

The problem
Even though there is a page describing operators for sets, bags, and mixes, there is no reference for that page on the Perl6 Operators page. Without knowing what to look for, a new user would be lost without help from someone more knowledgeable about the layout/
Suggestions
Add a section to the Operator page linking to setbagmix.pod6. Ideally, this would include references to all of those operators so that someone new would know they exist.

Work in progress.
Draft look available, here:
https://github.com/Xliff/doc/blob/op-setbagmix-changes/doc/Language/operators.pod6#L2958

Looks good. Thanks!

This overlaps with #2603. Please check it out and/or close it when you're done with this one.

Hmm.... just tried zef install Pod::To::BigPage and it said I was on the latest version. Will try installing direct from the repo.

Hum, that's weird. Anyway, I can transfer the issue there.
El dom., 5 may. 2019 a las 15:34, Xliff<EMAIL_ADDRESS>escribió:

Hmm.... just tried zef install Pod::To::BigPage and it said I was on the
latest version. Will try installing direct from the repo.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/perl6/doc/issues/2761#issuecomment-489427005, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAD5CABEBTFMXLMC2LH7TPT3O4TANCNFSM4HKZZ5MA
.

--
JJ


DOCUMENTATION

We should all sit down and over a few days figure out the current state of things... I sure have lost scope of cvTile since I haven't touched it in so long.
To cement the aforementioned discussion, we should all write some documentation, If @interputed    hasn't already. Just so we can make sure everyone knows what's going on, and we can on-board people more readily.
Just a thought.

We should also add @interputed as a collaborator so we can assign things to him cough cough

You should 100% rewrite the GPU code to use thrust algorithms - 10/10 better life for everyone. I use thrust at work and it's nice.
Sent from my iPhone

On Aug 11, 2016, at 2:47 PM, Tyler Nivin<EMAIL_ADDRESS>wrote:
We should also add @interputed as a collaborator so we can assign things to him cough cough
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Support Marathon 1.0.0-RC1+ JSON definition for 422 error responses

The error structure for 422 HTTP error codes has changed again in version 1.0.0-rc1, adapting the same format as the 400 HTTP error response.
This is an "unwanted successor" to #133.
Fixes #164.

@gambol99 could you please take a look?
Apologies for the somewhat hard-to-read code but the upstream changes are really subtle and require a bit of trickery to distinguish cleanly. Suggestions for alternatives are welcome of course.
We should also consider removing support for older formats at some point.

other than the one picky comment :-) .. looks good to me ..

Thanks 👍
Waiting for CI to go green, will merge afterwards.


How do you write underline text?

I would like to write underline text but I'm not sure if that is something I need to define with an external package like https://github.com/golang/freetype or something I have to manually calculate and DrawLine() for.
Looking at the source more, it seems functions for calculating the width and hight of each line of text already exist. These could be used to draw lines from x,y to x+w,y for each fontsize * lineheight.

	var maxTextWidth float64 = 120

	dc.DrawStringWrapped(myText, P, H/2, 0, 0.5, maxTextWidth, 2, gg.AlignLeft)

	var lineStrokeWidth float64 = 1
	var lineSpacing float64 = 2

	// How many lines does the text take-up?
	lines := dc.WordWrap(myText, maxTextWidth)
	textWidth, textHeight := dc.MeasureMultilineString(strings.Join(lines, "\n"), lineSpacing)
	fmt.Printf("%q\n", lines)
	fmt.Println(textWidth, textHeight)

	lineHeight := textHeight / float64(len(lines)-1)
	fmt.Printf("LineHeight: %f\n", lineHeight)
	// dc.DrawLine(P, H/2-(textHeight/2), P+textWidth, H/2-(textHeight/2))

	dc.SetLineWidth(lineStrokeWidth)
	for i := 1; i <= len(lines); i++ {
		textY := float64(i) * lineHeight
		fmt.Printf("\t%f of %f\n", textY, textHeight)

		if i > 1 {
		textY += lineHeight/lineSpacing - (float64(i) * lineStrokeWidth)
		}

		dc.DrawLine(P, H/2-(textHeight/2)+textY, P+textWidth, H/2-(textHeight/2)+textY)
	}
	dc.Stroke()

I don't think the underlying freetype package supports underlining or strikethrough.

Correct, the underlying freetype package does not support underlining or strikethrough.
As for golang.org/x/image/font/sfnt, the intention was to replace github.com/golang/freetype, including rasterizing glyphs, but it is sadly unfinished. As of a few years ago, I'm no longer paid to work on it, and I haven't had the spare time to finish it.


Update schedule

@subhransu can help bump gatsby to "gatsby": "2.30.1",
ref, resolve cors issue: https://github.com/gatsbyjs/gatsby/issues/28877


SCP-3168 Usability enhancements for marlowe-cli

This pull request addresses all of the comments in https://github.com/input-output-hk/marlowe-cardano/pull/30, and add usability features and commands to the tool:

implemented deposit, choose, and notify
disabled usage on mainnet
switched to untyped small validator
optional output to stdout
computing execution statistics for transactions
support for submitting transaction when it is built
waits for confirmations on submit
added "compute" command, which accesses computeTransaction.
grouped commands in logical sets.
added instruction for CARDANO_NODE_SOCKET_PATH
added example paths to tutorials and scripts
made xargs usage safer in test script
updated read-me, man pages, documentation, and examples
retested example scripts and CI test script
value parser now handles arbitrary combinations of native tokens
rewrote parsers to be more monadic or to use regular expressions
relocated several IO function to Language.Marlowe.CLI.IO.
--version now reports git revision.

Pre-submit checklist:

Branch

[x] Tests are provided (if possible)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

This is superseded by https://github.com/input-output-hk/marlowe-cardano/pull/45.


Add sphinx-intl

sphinx-intl is a utility tool that provides several features that make it easy to translate and to apply translation to Sphinx_ generated document. Optional: support the Transifex service for translation with Sphinx.

Hi @tkoyama010  Thank you so much for your time and effort here, as well as professional approach! ✨ Merged 🎉


Should allow primitive at root to comply RFC

Current jsmn disallows only a primitive at root.  But RFC7159  allows that like followings:

JSON-text = ws value ws
(snip)
value = false / null / true / object / array / number / string

So I think it should allow this even in JSMN_STRICT case.

at some point it will be fixed

I rebased the test to https://github.com/zserge/jsmn , it resulted the following error
https://travis-ci.org/zserge/jsmn/builds/525143871
I noticed the latest doesn't allow primitives at root regardless JSMN_STRICT or not.

Don't worry, I'm slowly building a list of things that need to be added/changed/improved in the parses without breaking it. If all goes well and I have enough time I might push fix for that in at most a month or so.

although it should work without strict I believe... hmmm


b3e7c2 appears to create compatibility issue with github.com/golang/protobuf/descriptor

https://github.com/golang/protobuf/blob/master/descriptor/descriptor.go imports google.golang.org/genproto/protobuf which as of b3e7c2fb04031add52c4817f53f43757ccbf9c18 contains no buildable go files (and further, removes expected types).
$ go install github.com/golang/protobuf/descriptor
../../github.com/golang/protobuf/descriptor/descriptor.go:46:2: no buildable Go source files in ${GOPATH}/src/google.golang.org/genproto/protobuf

Should this be filed against github.com/golang/protobuf instead?

@tmc Yes, the new home for the descriptor package is github.com/golang/protobuf/protoc-gen-go/descriptor. It was in two places in the past. I posted an issue on golang/protobuf for you.


Is it possible to combine this work with one-stage detector such as YOLOv5 or SSD?

Hi Klemen and Roozbeh, I like your work very much and wanna try but I am not sure the proposed structure work with static 2D images .

Hi Shirley. Thanks for your interest. Interactron is fundamentally a method for embodied object detection and is thus designed to work in interactive environments where it is possible to move the agent around to obtain multiple views. I am not entirely sure what you mean by working with 2D images. If you would like to chat about possible extensions and future work feel free to email me at<EMAIL_ADDRESS>


updates to get touchforms working on staging

see https://github.com/dimagi/touchforms/commit/f5957b989545b8d1e3b72fc62999222bdad24a37
@benrudolph

wow, nice. i was hurting friday afternoon when i was seeing those errors and changing the port didn't Just Work


Communication strategy edits

Added a few information. Rough Draft.

Are the changes to the strategy document intended, or added by accident?

I only pushed the communication. It was said it would be a reference for future purposes.


[$250] Users gets logged out after posting a list in this  FullStory-https://fsty.io/v/Gj3y2abz

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @danielrvidal
Slack conversation (hyperlinked to channel name): ts_external_expensify_convert
Action Performed:

<EMAIL_ADDRESS>is writing a note for herself (screen 1)
Then get’s logged out immediately when posting
Then when she comes back, the list is not there (image 2).

Expected Result:
User remains logged in after posting a list in fullstory
Actual Result:
User gets logged out after posting a list in fullstory
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021859430214689131084
Upwork Job ID:<PHONE_NUMBER>689131084
Last Price Increase: 2024-11-21

Issue OwnerCurrent Issue Owner: @ahmedGaber93

Hmm ... I can't reproduce this

Removing Help Wanted until we can reproduce. @ahmedGaber93 - do you mind giving reproduction a shot? Maybe I'm missing something. 🤔

Honestly, this is the first time I hear about FullStory. Also, there is no clear reproduction steps, is there is any guide about how to integrate and using FullStory with expensify?

Hmm. I'm going to close as I still can't repro, but please reopen if you disagree.


mock mode expects a schema property to exist when generating mockResponse

Hi,
I'm implementing restful API and after successful entity creation I'm returning 201 code with empty response.
The mock mode expects the schema property to exist when returning 201 code and in general when calling the getMockValue function (in my case, undefined is passed) so an error is thrown:
Error: Callback was already called.
    at /node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:30:31
    at /node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:251:21
    at /node_modules/a127-magic/node_modules/swagger-tools/middleware/helpers.js:576:13
    at /node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:249:17
    at /node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:125:13
    at Array.forEach (native)
    at _each (/node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:46:24)
    at async.each (/node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:124:9)
    at _asyncMap (/node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:248:13)
    at Object.map (/node_modules/a127-magic/node_modules/swagger-tools/node_modules/async/lib/async.js:219:23)

I guess that the schema property doesn't have to exist.
Thanks,
Boris

In my case it requires duplication of schema existence. First in the assignment and then when calling the getMockValue function

I just added support for the void response so I must had missed mock mode support when implementing it.  Thanks for the report.


An small error in ReadMe.md?

When I run setup and follow the readme.md,  pip install -e '.[dev,predictors,visualizers]'  ,an error occurs:
ERROR: '.[dev,predictors,visualizers]' is not a valid editable requirement. It should either be a path to a local project or a VCS URL (beginning with bzr+http, bzr+https, bzr+ssh, bzr+sftp, bzr+ftp, bzr+lp, bzr+file, git+http, git+https, git+ssh, git+git, git+file, hg+file, hg+http, hg+https, hg+ssh, hg+static-http, svn+ssh, svn+http, svn+https, svn+svn, svn+file).
After changing to python install -e .[dev,predictors,visualizers], no errors will be reported. I'm not sure if this is a error in ReadMe.md .

hi @hhy150 ! the quotes seem like needed for install with zsh, not for bash. i can clarify.
when you say python install, do you mean pip install?

Yes, I mean pip install.  Thank you for your reply.

no worries; thx for the issue!


invalid "Transformation file URL" should throw an error

I don't like the behavior when having a typo in the url.
Expected Behaviour
It should show an error: "Tranformation file not found"
Actual Behaviour
An import it run with the default config, and overrides any existing word documents.

No import file is a valid use case ("default import") and you can easily see in the console log that the file cannot be found (red 404 error).
Maybe simply showing a message like "No transformation file found, using the default one" to inform the user could be a good compromise.


More robust test of schema halting after time limit.

The previous test could fail due to a race condition.

I did wonder about that test. :-) We might want to consider increasing evalDelay given the sorts of timings we normally see out of travis, but let's see how this goes first.


refactor: remove unuse mtab function

Ⅰ. Describe what this PR did
remove unuse function that modify /etc/mtab in container.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
Signed-off-by: Rudy Zhang<EMAIL_ADDRESS>

Codecov Report

Merging #2060 into master will increase coverage by 4.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2060      +/-   ##
==========================================
+ Coverage   58.89%   63.75%   +4.85%
==========================================
  Files         202      202
  Lines       15535    15498      -37
==========================================
+ Hits         9149     9880     +731
+ Misses       5283     4385     -898
- Partials     1103     1233     +130

Flag
Coverage Δ

#criv1alpha1test
33.46% <ø> (?)

#criv1alpha2test
34.01% <ø> (-0.18%)
:arrow_down:

#integrationtest
38.21% <ø> (-0.26%)
:arrow_down:

#unittest
22.75% <ø> (+0.06%)
:arrow_up:

Impacted Files
Coverage Δ

daemon/mgr/spec_hook.go
23.72% <ø> (-16.9%)
:arrow_down:

pkg/system/device.go
0% <0%> (-100%)
:arrow_down:

storage/quota/prjquota.go
0% <0%> (-8.54%)
:arrow_down:

ctrd/image.go
76.84% <0%> (-4.44%)
:arrow_down:

daemon/mgr/system.go
76.92% <0%> (-1.71%)
:arrow_down:

daemon/logger/jsonfile/utils.go
71.54% <0%> (-1.63%)
:arrow_down:

storage/quota/quota.go
16.4% <0%> (-1.57%)
:arrow_down:

ctrd/container.go
49.48% <0%> (+0.68%)
:arrow_up:

ctrd/watch.go
75.75% <0%> (+3.03%)
:arrow_up:

cri/stream/httpstream/spdy/upgrade.go
60% <0%> (+5.71%)
:arrow_up:

... and 8 more


transformerMetaWordHighlight highlight is not working when word includes symbols

Describe the bug
Please first take a look at rehype-pretty-code word highlighting section
I want the same behavior with Shikiji but it seems transformerMetaWordHighlight only works with words without symbols
Thanks for your work ❤
Reproduction
https://stackblitz.com/edit/vite-rfm8dh
System Info
System:
    OS: Windows 11 10.0.22000
    CPU: (16) x64 Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz
    Memory: 4.66 GB / 15.83 GB
  Binaries:
    Node: 18.18.2 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.21 - C:\Program Files\nodejs\yarn.CMD
    npm: 9.8.1 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.14.1 - C:\Program Files\nodejs\pnpm.CMD
  Browsers:
    Edge: Spartan (44.22000.120.0), Chromium (120.0.2210.144)
    Internet Explorer: 11.0.22000.120

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Perhaps we can include other common symbols like " | . - ,. We will not support edge cases where the word is separated into multiple tokens

Still some refactor to do to support multiple token highlight

I am willing to submit a fix

Great, here is a reference in twoslash integration: https://github.com/antfu/shikiji/blob/4b61e33cd26a917dbc0e8764c4b4c17b619c77e2/packages/shikiji-twoslash/src/core.ts#L202-L228

I think vuejs/vitepress#3503 is also related to this. There, the text that's being matched against the regex is not the whole line instead individual tokens 👀 The comments like // [!code error] don't work if they are not entirely in the same text node.

That is caused by the limitation of design rather than the transformer itself, adding comments after attributes is not valid in Vue. Therefore, you'll see a weird highlighting here:
<template>
  <div
    class="container max-w-250 mx-a my-10 min-h-250 vp-doc p4" // something
    of-hidden
  >
    <ShikijiMiniPlayground />
  </div>
</template>

Writing this in real code even gives you syntax errors, which is not supposed to work.

@brc-dd Feel free to open an issue on https://github.com/shikijs/shiki and we could discuss what would be the solution for that :)


Provider with a self-signed SSL certificate

I use the Gem openid_connect to build a RP.  And, the Provider (an existing one) is using a self-signed SSL certificate.
In such scenario, the Gem openid_connect (used in the RP) always gives the SSL verification error.  Only when the Provider is equipped with a trusted SSL certificate, the error no longer occurs.
However, I cannot force all Providers to have trusted SSL certificates.
How can I make the Gem trust all SSL certificates?  (For example, in Excon, the statement "Excon.defaults[:ssl_verify_peer] = false" makes all SSL certificates be trusted, although it will then be less secure.)

I'm having the same problem here. The problem is, that the httpclient lib uses their own certificates, that get shipped with the gem. They to provide a method "add_trust_ca" to add additional trusted CA's but this method is not called by openid_connect. This problem is also "documented" in their code here https://github.com/nahi/httpclient/blob/master/lib/httpclient/ssl_config.rb

Represents SSL configuration for HTTPClient instance.
The implementation depends on OpenSSL.

== Trust Anchor Control

SSLConfig loads 'httpclient/cacert.pem' as a trust anchor
(trusted certificate(s)) with add_trust_ca in initialization time.
This means that HTTPClient instance trusts some CA certificates by default,
like Web browsers.  'httpclient/cacert.pem' is downloaded from curl web
site by the author and included in released package.

On JRuby, HTTPClient uses Java runtime's trusted CA certificates, not
cacert.pem by default. You can load cacert.pem by calling
SSLConfig#load_trust_ca manually like:

HTTPClient.new { self.ssl_config.load_trust_ca }.get("https://...")

You may want to change trust anchor by yourself.  Call clear_cert_store
then add_trust_ca for that purpose.

Could you please add an opertunity to add custom trusted CA's?

you can do so like this.
https://github.com/nov/openid_connect/blob/master/spec/openid_connect_spec.rb#L45


Remove requirement for 2 node default

This change implements similar logic to what is defined in the other EDPM gen script:
https://github.com/openstack-k8s-operators/install_yamls/commit/1b377bc39f91957e55e06a725fd3a4c0b95046c5

/override rdoproject.org/github-check
as it depends on https://github.com/openstack-k8s-operators/openstack-operator/pull/456

/override rdoproject.org/github-check

@fao89: Overrode contexts on behalf of fao89: rdoproject.org/github-check

Why're we overriding and force merging everything? We can surely test https://github.com/openstack-k8s-operators/openstack-operator/pull/456 with Depends-On :/

we were merging yesterday and I went with the flow, but you are right


Could not find or use auto-linked framework 'WebRTC'

I'm using Flutter on top of the native iOS application. According to this documentation, I'm inserting it like "Embed frameworks in Xcode".
Next error xCode throws: https://prnt.sc/zu7pbq
Only happens when AppDelegate.swift file has next code: GeneratedPluginRegistrant.register(with: self.flutterEngine);
https://prnt.sc/zu7vhh
Also I'm able to solve it with WebRTC from Cocoapods dependency, but this looks like not good way.
Android is fine.
Also iOS works fine until it's not integrated to native code (another app)

I have to finish the next steps:

Add then lines to generated Flutter.podspec

s.dependency 'GoogleWebRTC'  s.dependency 'Libyuv', '1703'

Xcode
[Target] -> Build Settings -> Enable Bitcode -> No


Publish 8.0-20230111003011+0000

Publish 8.0-20230111003011+0000

@bot-gradle test and merge

Your PR is queued. See the queue page for details.


Shared access authentication for google cloud storage?

Hi,
Is there a way to do shared access using gcloud? I'm coming from azure and s3 and we used to use access key and secret key to construct a token or sas signature for shared access. I can't seem to find any guides on this topic. I see from most tutorials, I need to reference a json file which stores my private key credentials, is it possible to use access key and secret key concept to connect to google cloud storage?
Can anyone please help?

Hi @raintears,
If I understood correctly you might be interested in signed URLs (see official documentation here). When using gcloud-java you can create a signed URL by using Storage.signUrl.

This remained open for a while with no follow up. Feel free to reopen this if you have further questions!


System-tests: create test for REST API - GET /v1/schema

tracker https://github.com/EnMasseProject/enmasse/issues/936

https://github.com/EnMasseProject/enmasse/pull/1045


feat: check for updates in the android sdk repo and update our dependencies file if there is

Summary by CodeRabbit

New Features

Introduced a new GitHub Actions workflow to check for updates to the AdGem SDK daily and manually.
Added functionality to compare the current AdGem SDK version with the latest available on GitHub and update the configuration file accordingly.
New script added to package.json for checking the AdGem SDK version.

Bug Fixes

Updated the .gitignore file to prevent unnecessary files from being tracked by version control.

Chores

Added new dependencies in package.json for improved functionality.

Will the Android SDK version bump also be reflected in the changelog when merging a corresponding PR created through this new workflow? I'm curious about how this will affect the changelog in scenarios where Unity dependencies are updated multiple times within a single release cycle. I assume there will be a separate entry there per update?

Will the Android SDK version bump also be reflected in the changelog when merging a corresponding PR created through this new workflow? I'm curious about how this will affect the changelog in scenarios where Unity dependencies are updated multiple times within a single release cycle. I assume there will be a separate entry there per update?

Since the Android SDK Dependency will be updated via a PR, I do think that the Changelog will get updated too, via the semantic release workflow

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Refreshes on same clicked link

First, thank you for creative this awesome library.
I hope you take a look whats wrong on this sample that I did. Cause whenever I click the same link the page refreshes.
https://codepen.io/miksv/project/full/DkPbev/

Hello @meiskv ,
that's intended.
You should check manually if the user is trying to click on the current page and eventually stop the event propagation.


Small fixes - convert param values to string, set auto failover to true if cluster enabled

what

Small fixes
Convert param values to string
Set auto failover to true if cluster enabled
Add descriptions to subnet and param groups

why

Param values will fail if they are not strings
InvalidParameterValue: invalid parameter value, allowed values are:yes,no

Auto failover needs to be set to true if cluster mode is enabled or it will fail with an error
InvalidParameterValue: Redis with cluster mode enabled cannot be created with auto failover turned off.

references

Thanks to @simoferr98 this closes https://github.com/cloudposse/terraform-aws-elasticache-redis/pull/140

/test all


Models with more than one input

Question
Hello everyone, I am starting to use DJL. After looking at all the examples provided I was wondering if it would be possible to use DJL to make inference with computer vision models that require more than one input (for example cyclegans or Mask R-CNN).
Thank you for your time,
Carlos

Yes, DJL support multiple input and multiple output.
You can refer to this document: http://docs.djl.ai/jupyter/mxnet/load_your_own_mxnet_bert.html#construct-translator
The Translator API allows you convert your java object into NDList. The NDList class can hold multiple NDArrays.

Great, thanks.

Yes, DJL support multiple input and multiple output. You can refer to this document: http://docs.djl.ai/jupyter/mxnet/load_your_own_mxnet_bert.html#construct-translator The Translator API allows you convert your java object into NDList. The NDList class can hold multiple NDArrays.

link is broken :(
could you provide the other one?

I updated the link: https://docs.djl.ai/docs/demos/jupyter/load_mxnet_model.html#step-3-create-a-translator


E315 when deleting a fold with zd

Describe the bug
While trying to remove a fold, the error E315 is raised and some text is wrongly removed in addition to the fold marker.
To Reproduce
Here's a single shell command reproducing the error:
$ vim -Nu NONE --cmd 'set fdm=marker' +"put! =['fu! Func() {{{1', 'endfu']" +'sil %y' +'wincmd w' +'exe "norm! ix\eVpzd"' -o /tmp/.a.vim /tmp/.b.vim

Expected behavior
The first line of the fold was originally:
fu! Func() {{{1

After pressing zd, I would expect:
fu! Func()

But in fact, I get:
fu!

Func() has been wrongly removed.
Screenshots
Here's a gif reproducing the issue in an interactive usage:

Environment

Vim version: 8.1  Included patches: 1-1888
OS: Ubuntu 16.04.6 LTS
Terminal: xterm (patch 322)

I'm not sure I did it correctly, but I tried to get a backtrace: backtrace.txt

I can reproduce it. The problem is, when the folds will be updated after the put, the buffer still has 1 additional line. nv_do_put() does already handle this condition (see here), but does not call changed_lines() and therefore the folds are not correctly updated. Later on deleting the fold markers, Vim therefore tries to access this additional line from the folds.
I tried to fix this by calling deleted_lines(curbuf->b_ml.ml_line_count, 1); after ml_delete() in nv_do_put (right after the above mentioned comment). However This will call foldUpdate() with the arguments (top=3, bot=2) which is wrong. This happens because deleted_lines will call changed_lines(lnum, 0, lnum + count, -count); and later in changed_common() it will call
foldUpdate(wp, lnum, lnume + xtra - 1);, meaning bot will always be one less than top. That looks really wrong. I wonder if this should be changed to foldUpdate(wp, lnum, lnume + abs(xtra) - 1);.
Here is a patch, that works around this problem:
diff --git a/src/change.c b/src/change.c
index f83587d83..7fc385e8f 100644
--- a/src/change.c
+++ b/src/change.c
@@ -518,7 +518,7 @@ changed_common(
 #ifdef FEAT_FOLDING
            // Update the folds for this window.  Can't postpone this, because
            // a following operator might work on the whole fold: ">>dd".
-           foldUpdate(wp, lnum, lnume + xtra - 1);
+           foldUpdate(wp, lnum, lnume + abs(xtra) - 1);

            // The change may cause lines above or below the change to become
            // included in a fold.  Set lnum/lnume to the first/last line that
diff --git a/src/fold.c b/src/fold.c
index 4cc7a477d..c84609989 100644
--- a/src/fold.c
+++ b/src/fold.c
@@ -1813,8 +1813,8 @@ deleteFoldMarkers(
 /*
  * Delete marker "marker[markerlen]" at the end of line "lnum".
  * Delete 'commentstring' if it matches.
- * If the marker is not found, there is no error message.  Could a missing
- * close-marker.
+ * If the marker is not found, there is no error message.
+ * Could be a missing close-marker.
  */
     static void
 foldDelMarker(linenr_T lnum, char_u *marker, int markerlen)
@@ -1826,6 +1826,9 @@ foldDelMarker(linenr_T lnum, char_u *marker, int markerlen)
     char_u     *cms = curbuf->b_p_cms;
     char_u     *cms2;

+    // safety check
+    if (lnum > curbuf->b_ml.ml_line_count)
+       return;
     line = ml_get(lnum);
     for (p = line; *p != NUL; ++p)
        if (STRNCMP(p, marker, markerlen) == 0)
@@ -2741,8 +2744,8 @@ foldUpdateIEMSRecurse(
        fold_changed = TRUE;
     }

-    /* Delete contained folds from the end of the last one found until where
-     * we stopped looking. */
+    // Delete contained folds from the end of the last one found until where
+    // we stopped looking.
     foldRemove(&fp->fd_nested, startlnum2 - fp->fd_top,
                                                  flp->lnum - 1 - fp->fd_top);

diff --git a/src/normal.c b/src/normal.c
index 283ba7c4a..89996ce2f 100644
--- a/src/normal.c
+++ b/src/normal.c
@@ -9425,6 +9425,7 @@ nv_put_opt(cmdarg_T *cap, int fix_indent)
        if (empty && *ml_get(curbuf->b_ml.ml_line_count) == NUL)
        {
            ml_delete(curbuf->b_ml.ml_line_count, TRUE);
+           deleted_lines(curbuf->b_ml.ml_line_count, 1);

            /* If the cursor was in that line, move it to the end of the last
             * line. */
diff --git a/src/testdir/test_fold.vim b/src/testdir/test_fold.vim
index c0641f74e..5d2298143 100644
--- a/src/testdir/test_fold.vim
+++ b/src/testdir/test_fold.vim
@@ -764,3 +764,18 @@ func Test_address_relative_fold()

   bw!
 endfunc
+
+func Test_delete_foldmarker()
+  new
+  let expected=['fu! Func() ', 'endfu']
+  call setline(1, ['fu! Func() '.. '{{{1', 'endfu'])
+  setl fen fdm=marker
+  :%y
+  new
+  setl fen fdm=marker
+  norm! Vpzd
+  call assert_equal(expected, getline(1, '$'))
+  sil %d _
+  bw!
+  bw!
+endfunc

I can confirm that the patch fixes the issue. Thank you very much for the detailed explanation.

I can reproduce it. The problem is, when the folds will be updated
after the put, the buffer still has 1 additional line. nv_do_put()
does already handle this condition (see
here),
but does not call changed_lines() and therefore the folds are not
correctly updated. Later on deleting the fold markers, Vim therefore
tries to access this additional line from the folds.

Thanks for looking into this.

I tried to fix this by calling
deleted_lines(curbuf->b_ml.ml_line_count, 1); after ml_delete() in
nv_do_put (right after the above mentioned comment). However This
will call foldUpdate() with the arguments (top=3, bot=2) which is
wrong. This happens because deleted_lines will call
changed_lines(lnum, 0, lnum + count, -count); and later in
changed_common() it will call
foldUpdate(wp, lnum, lnume + xtra - 1);, meaning bot will always
be one less than top. That looks really wrong. I wonder if this should
be changed to foldUpdate(wp, lnum, lnume + abs(xtra) - 1);.

No, I think it was correct.  The problem is that when adjusting folds
and it runs into the end of the buffer, it doesn't adjust the length of
the last fold.
--
Where do you want to crash today?
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Fix periscope database migrations script default location in source code

@doktoric please review

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1959/


Update compatible builds to docker version 73

Change overview

Update compatible builds to latest docker version 73
Telink update Docker image PR https://github.com/project-chip/connectedhomeip/pull/34980

Builds required old docker version 54:

build_linux_python_lib: name: Build on Linux (python_lib)
build_linux_python_lighting_device: name: Build on Linux (python lighting-app)
infineon: name: Infineon examples building
qemu-tizen: name: Tizen

K32W image update will be performed under https://github.com/project-chip/connectedhomeip/pull/34926

Blocked by https://github.com/project-chip/connectedhomeip/pull/34980
Will test in background which images we could update to latest by using 72 temporary, till version 73 is not published

Blocked by #34980 @andy31415 I will test in background which images we could update to latest by using 72 temporary, till version 73 is not published
Builds required old docker version 54:

build_linux_python_lib: name: Build on Linux (python_lib)
build_linux_python_lighting_device: name: Build on Linux (python lighting-app)
infineon: name: Infineon examples building
qemu-tizen: name: Tizen

K32W image update will be performed under #34926

I am building 73 now, however also 74 is soon following in #35011 .

Blocked by #34980 @andy31415 I will test in background which images we could update to latest by using 72 temporary, till version 73 is not published
Builds required old docker version 54:

build_linux_python_lib: name: Build on Linux (python_lib)
build_linux_python_lighting_device: name: Build on Linux (python lighting-app)
infineon: name: Infineon examples building
qemu-tizen: name: Tizen

K32W image update will be performed under #34926

I am building 73 now, however also 74 is soon following in #35011 .

No problem I can change in this PR into 74, when you will finish it
Also will be good to merge https://github.com/project-chip/connectedhomeip/pull/34926 to have almost all imges with latest docker version
See PR discribtion (change overview)

@andy31415
EFR32 require additional changes to use image version 74 https://github.com/project-chip/connectedhomeip/pull/35011
https://github.com/project-chip/connectedhomeip/actions/runs/10415011158/job/28844869810?pr=35009
I leave EFR32 in version 73 and add this to PR description


GCL segfaults on errors

Describe the bug
gcl segfaults
Steps To Reproduce
Steps to reproduce the behavior:

nix run nixpkgs#gcl / nix run nixpkgs#gcl_2_6_13_pre
(require 'foo)
Segfault

Expected behavior
Entry into the Common Lisp debugger
Notify maintainers
@7c6f434c

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.81, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - channels(kpg): `"nix-cl, nixpkgs-21.11.tar.gz"`
 - channels(root): `""`
 - nixpkgs: `/home/kpg/.nix-defexpr/channels/nixpkgs`

I got it to work with hardeningDisable = [ "all" ];

Hm, how easily bisectable it is to figure out which hardening flag is relevant?


Return multiple arrays instead of zipping arrays into a list in StellarGraph

Some StellarGraph methods zip multiple numpy arrays and return them as one list, these could optimized to return seperate array.
These method are:

edges
_transform_edges
_adjacency_types (maybe?)
_unique_type_triples (maybe?)

Relates to #718.


Intersect of the same polygons returns rotated vertices

Expected behavior and actual behavior.
In a previous version of shapely(1.6.4.post2) we were using intersecting a pair of polygons.
In case polygon was intersected over same polygon/itself it returned exactly the same geometry.
With update to 1.8.0 we have encountered weird behavior where every intersect rotates vertices by one and polygon is not exactly the same anymore because of this change.
I'd expect that intersect of itself should provide the same geometry or at least return predictable output.
Steps to reproduce the problem.
from shapely.geometry import Polygon

extent = Polygon([
    (10.55498600006104, 42.94454893489702),
    (10.56210994720459, 42.93694613058707),
    (10.54811954498291, 42.93672620078932),
    (10.55498600006104, 42.94454893489702)
])

intersect = extent.intersection(extent)
if extent != intersect:
    print('Not the same')
intersect2 = intersect.intersection(intersect)
if intersect2 != intersect:
    print('Still not the same')
intersect3 = intersect2.intersection(intersect2)
if intersect3 != intersect2:
    print('Still not the same')
print(extent.wkt)
# POLYGON ((10.55498600006104 42.94454893489702, 10.56210994720459 42.93694613058707, 10.54811954498291 42.93672620078932, 10.55498600006104 42.94454893489702))
print(intersect.wkt)
# POLYGON ((10.56210994720459 42.93694613058707, 10.54811954498291 42.93672620078932, 10.55498600006104 42.94454893489702, 10.56210994720459 42.93694613058707))
print(intersect2.wkt)
# POLYGON ((10.54811954498291 42.93672620078932, 10.55498600006104 42.94454893489702, 10.56210994720459 42.93694613058707, 10.54811954498291 42.93672620078932))
print(intersect3.wkt)
# POLYGON ((10.55498600006104 42.94454893489702, 10.56210994720459 42.93694613058707, 10.54811954498291 42.93672620078932, 10.55498600006104 42.94454893489702))
# We got bact to original extent.
assert intersect3 == extent

Operating system
Ubuntu 20.1
Shapely version and provenance
1.8.0 from pip, Python 3.8.10

This is perhaps a strange quirk of OverlayNG in GEOS 3.9 (@dr-jts ?), which was bundled with binary wheels for shapely 1.8.
As a more reliable workaround, a new function with shapely 1.8 is .normalize():
assert intersect.normalize() == intersect2.normalize() == intersect3.normalize()

This is perhaps a strange quirk of OverlayNG in GEOS 3.9 (@dr-jts ?)

Yes, this is a quirk of the OverlayNG codebase.  Strictly speaking OverlayNG does not provide any guarantees about output coordinate order.  But this is sufficiently odd that perhaps it worth looking into to see if this can be remediated somehow.  And yes, using normalize is always a good idea if the coordinate order in rings needs to be relied upon.

We have also encountered this when working on sectionproperties.. It made unit test writing a bit troublesome.

I marked this as an upstream issue and will close.


how to exec long sql file.

it is 660000 rows. copy to sql editor,dbeaver will die.

Unfortunately, it is not supported yet.
Thanks for suggestion

This issue is a duplicate if #4738, so I'm closing it.
You can vote for the referenced issue.


Add final rbi test

I realized that we don't ever test rbis with final. I thought there might be some issues with redeclaring things marked as final. Although the tests in here pass as I think they should, I think the following test (not included in this PR) should fail:
In r.rbi:
# typed: true
class C
  sig(:final) {void}
  def foo; end
end

In r.rb:
# typed: true
class C
  sig {void}
  def foo; end
end

On command line:
sorbet r.rbi r.rb

Expected output: some error about sigs not matching up.
Actual current output: no errors.
I'm not actually sure if this example should give an error or not, but in any case the tests in this PR do pass as I expect they should.
Motivation
To test more.
Test plan
See included automated tests.

@jez could you clarify if the test I put in the PR desc should fail?

@jez could you clarify if the test I put in the PR desc should fail?

I'd say it should for now be consistent with whatever happens for the other attributes, like abstract / implementation.
My guess is that it would be nice to make it an error, but it's going to be hard to do in the presence of fast path checks.

Arguably even this should be an error:
# typed: true
extend T::Sig

sig {params(x: Integer).void}
def foo(x); end

sig {params(x: Strin).void}
def foo(x); end

https://sorbet.run/#%23 typed%3A true
extend T%3A%3ASig

sig {params(x%3A Integer).void}
def foo(x)%3B end

sig {params(x%3A String).void}
def foo(x)%3B end


As a user, I want to search and browse content across all types so that I can find any results in full volumes as well as excerpts.

dev notes
This should work with no additional development needed after #395 is completed. Adding it here to make sure we test it explicitly.

Searched across archive using "free verse" -- all content types appeared in results as expected. Looks good!


hello 0.1.0-51557176+autotest

Created via alr publish with alr 2.0-dev+beta2

Closed using alr 2.0-dev+beta2 with reason: 'automated testing'


Adding missing parameter

Closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/7454

Also closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/2568

@Kateyanne Please merge this PR. Thanks

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

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add ESMAP / World Bank stations

Energy Sector Management Assistance Program (ESMAP) is a partnership led by the World Bank, and has historically funded solar radiation monitoring campaigns in developing countries.
An overview of the solar radiation monitoring stations can be found here (see also screen-shot below): https://energydata.info/dataset/world-solar-stations-inventory-2017
It seems some of these stations are the same as those listed on the Global Solar Atlas site: https://globalsolaratlas.info/solar-measurement

@AdamRJensen some information on the instruments used in the ESMAP stations can be found here.
Table 3 has a nice summary of the used instruments.


Create Contributor Docs

Create a set of contributor docs that tell contributors unfamiliar about IML, how to contribute to the project, and it's overall architecture. At a minimum we should include:

[x] High level overview of repos and how they fit together
[x] How to contribute a patch to both a small repo and the intel-manager-for-lustre repo.
[x] High level architecture of IML
[x] Review docs for correct content
[x] Review docs for grammar
[x] Make Corrections from Reviews
[x] Add contributor docs to IML Help Docs
[x] Add corrections to make docs consistent with Help Docs

Moved Docs to:
https://github.com/intel-hpdd/Online-Help/pull/21


playLayout: .game__meta__infos refreshes in certain situations

When the game ends, but maybe also in other situations, when merely requesting a draw or gifting time to the opponent
The operations done in the beginning must be applied on redraw as well

fixed in v2.3.70


New icon: KACE

Add an icon for the Quest KACE appliances

[x] I have carefully read and followed CONTRIBUTING document.

URL of the site: https://www.quest.com/kace/

Proof of the Aegis-compatible 2FA:
https://support.quest.com/technical-documents/kace-systems-management-appliance/11.1 common documents/administrator-guide/73#TOPIC-1610972

Logo / icon info:
Logo was taken from wikimedia on https://upload.wikimedia.org/wikipedia/commons/2/20/KACE_logo.svg
I used Inkscape to make the logo from the template file.
SVG was then run through SVGOMG with the "Prettify markup" option as instructed.

Background color info:
The color was taken from the original wikimedia logo and is the standard Quest strong orange hue. #f05223

The Quest KACE System Management Appliance (SMA) has the option for 2FA. I recently moved to Aegis and saw there wasn't any KACE related icons in the icon pack, so instead of just making one for my personal use, I thought of contributing to the repo.

Sorry, but I don't fully understand what this product does, so you have to answer this.
Do you need to login to use the system via Quest's "sign in" button or is this hostable software?
If it's former, then this submission isn't acceptable because KACE would use Quest account system, therefore Quest icon is needed rather then KACE.

It's a hostable software for endpoint management, it's not used to log in to Quest's website.
I'll be addressing the .svg formatting shortly, sorry for not reading the documentation fully.

I edited another thing, just FYI: circles are part of the logo, please include them. Make a hole with "K" to the circle.

Ok, so the logo should be in one single .
Just clarify, the K and the circle were taken from the original logo. I don't understand what you want me to make a hole into.
Should I break the guides and bring the inner circle closer to the border as original?
Should I cut the outer circle with the inner and make a transparent hole encircling the K?
Could you be more specific?
Original on the right, modified on the left.

Like this:


Hlchunk error messge WinScrolled autocommand "*"..User

Describe the bug
After i added hlchunk everytime i scroll down a page or type something i get an error related to hlchunk. When im scrolling down a page im getting the error: "Error when executing : "WinScrolled autocommand ""..User for WinscrolledY""
and when i type i get this error: "Error when executing: "TextChangedI autocommand for "" ""
To Reproduce
Steps to reproduce the behavior:

Open nvim
open any file
scroll down or type something
error

Expected behavior
No errors and everything works, i can type and scroll
Screenshots
Error message when typing:

Error message when scrolling:

hlchunk config:

Additional context
Im using Packer as a plugin manager. Nvim version 0.10.2_1

I am getting the same issue here.


Summarize methylation statistics from bisulfite DNA-seq libraries

Some recent BS-seq programs store methylation information in a SAM tag. One example is Bismark. A flag for --methylation would activate routines for summarizing:

[X] Cycle-specific CpG cytosine methylation
[x] Cycle-specific cytosine conversion
[ ] Methyl-C content percent per read

Since there are only a few data formats for representing methylation data from bisulfite sequencing, flags for --bismark, --bsmap etc.. might be useful.


[ICON] Air conditioner

Icon Request

Icon name: Air conditioner
Use case: Apartment Feature List for Rentals
Screenshots of similar icons:

Added 9368512


PoS-based Poset consensus

It is a separately draft in src/posposet package.
Next steps if OK:

interface coherence;
replace src/poset with it;

Strengths and goals:

genesis with stakes at  instead of fixed peer list;
more clean and readable code/tests: 1526/1025 vs 6272/5449 lines, no protobuf structs, dedicated types for hash, address, time etc;
one Store implementation over any key-value DB;
sync event processing for test and async processing for prod, no timeouts from now on;
no multiple checks for the same event;
ASCII-scheme parser and events naming for tests;
human-readable hashes in logs so easy debug;

Huge PR, would be nice to see smaller ones.

You are right generally, but the most of changes is a totally new package src/posposet. It doesn't sense to split it.

We have some kind of inconsistency:

Lachesis paper put flag table into node;
this implementation put flag table into node
prototype from researchers put flag table into event and do not use frame as an standalone data structure, it is an integer field in event structure. Though they haven't shown atropos selection implementation with that data structure.

Original poset has support for debug build tag. If this version of poset is a candidate to replace it it should have support for debug tag too, equivalent to poset.PrintStat() should be enough, this function print the list of all known events, https://github.com/Fantom-foundation/go-lachesis/blob/842c9ef9fe8ba2c2737b721647d4950dcacf4c1b/src/poset/debug.go#L11

@devintegral Also what's your reasoning for removing protocol buffer structs? - Are you replacing them with something?
We current use gRPC, which uses protobuf, which is why I got my team to implement protobuf internally also.

what's your reasoning for removing protocol buffer structs?

I think it is bad idea to describe difficult internal logic with primitive protobuf structs. We will get quite unreadable code. For example: string type for hashes, or message&body pattern. Golang is richer than protobuf generator.
gRPC is good for transfer data between processes/hosts, but it is not Poset's mission. Let's leave it for node.

Are you replacing them with something?

Serialization for storing is implemented with src/rlp.

what's your reasoning for removing protocol buffer structs?

I think it is bad idea to describe difficult internal logic with primitive protobuf structs. We will get quite unreadable code. For example: string type for hashes, or message&body pattern. Golang is richer than protobuf generator.
gRPC is good for transfer data between processes/hosts, but it is not Poset's mission. Let's leave it for node.

Are you replacing them with something?

Serialization for storing is implemented with src/rlp.

RLP is inferior to protobuf in supporting various data types, https://github.com/ethereum/wiki/wiki/Design-Rationale#rlp So the the argument about primitiveness of protobuf structs is invalid.
RLP was brought as copy past into go-lachesis source tree, why not just use https://github.com/ethereum/go-ethereum/tree/master/rlp ?
Protobuf has been selected as the fastest serialisation library available for vast majority of languages, so it would be easier to implement clients in other languages. It looks like RLP isn't supported in as many languages as protobuf. Have you compared performance of protobuf against RLP?
We would need to use protobuf to communicate over network, so why we need the second serialisation library ? - having two of the same in the project doesn't look as good engineering.

Agree with Sam and Maxime2. Can devintegral split the PR into smaller ones? Who is
devintegral? Any documentation about the potential design changes/effects? Please email for further discussions if need be.

Agreed, please split this into smaller PRs, work with @dev10 on the ASCII side, work with the rest of us on the rest.
I'll close this PR for now, as it's way too large to work with. The reviews should stay active, so keep those in mind.
Thanks

@Maxime2, you did not catch my mind again.

So the the argument about primitiveness of protobuf structs is invalid.

Just compare src/poset.Event and src/posposet.Event (size and readable), no any arguments are needed.

why not just use https://github.com/ethereum/go-ethereum/tree/master/rlp ?

You are right, we could. Let's leave this issue for PR #136 .

We would need to use protobuf to communicate over network...
It looks like RLP isn't supported in as many languages as protobuf.

Poset will not communicate over network with other languages, this role is for node.
However https://github.com/ethereum/wiki/wiki/RLP#implementations

Protobuf has been selected as the fastest serialisation library available for vast majority of languages...
Have you compared performance of protobuf against RLP?

https://github.com/ethereum/wiki/wiki/Design-Rationale#rlp

why we need the second serialisation library ?

Anyway, we got RLP with Merkle Trie implementation from ethereum (see src/trie) and use for PoS-balance storage. Do you suggest implement it again with protobuf?

@devintegral If RLP is better than Protobuf, profile it and show us the graph. Otherwise we'll need a good reason to drop Protobuf, because we've done a lot of development to get it working in the codebase.

So the the argument about primitiveness of protobuf structs is invalid.

Just compare src/poset.Event and src/posposet.Event (size and readable), no any arguments are needed.

It is as laughable as comparing bicycle and F1 car. Obviously we need fast, robust, safe, elegant and readable code, but I would not sacrifice the first three for the sake of later two.

why not just use https://github.com/ethereum/go-ethereum/tree/master/rlp ?

You are right, we could. Let's leave this issue for PR #136 .

I have opened a new issue https://github.com/Fantom-foundation/go-lachesis/issues/157
In general, copying code which can be imported is a bad idea, let not do that anymore.

We would need to use protobuf to communicate over network...
It looks like RLP isn't supported in as many languages as protobuf.

Poset will not communicate over network with other languages, this role is for node.

Do you have a picture how lachesis works in whole at this moment? The node sends the very same events it get out of poset and writes to poset the very same events it receives from network, so would you approach be having two serialisators for the same things or creating another things for the node when passing data from poset to node and back, or something else? Anyway it doesn't look like a good engineering as whole.

Protobuf has been selected as the fastest serialisation library available for vast majority of languages...
Have you compared performance of protobuf against RLP?

https://github.com/ethereum/wiki/wiki/Design-Rationale#rlp

It says nothing about performance comparison. If it's the only of your analysis it is very shallow.
I have opened #158 to make throughout  comparison.

why we need the second serialisation library ?

Anyway, we got RLP with Merkle Trie implementation from ethereum (see src/trie) and use for PoS-balance storage. Do you suggest implement it again with protobuf?

Obviously yes, if protobuf would perform faster, or insignificantly minor as its more universality would overweight minor loss in speed.
Am i correct assuming that RLP was brought in only for the sake of easy code borrowing from go-ethereum?

If RLP is better than Protobuf, profile it and show us the graph. Otherwise we'll need a good reason to drop Protobuf ...

@SamuelMarks, RLP is not faster than Protobuf, see issue #158. Anyway I didn't suggest to drop Protobuf totally.

The node sends the very same events it get out of poset and writes to poset the very same events it receives from network, so ...

Yes, formally events are same, but actually poset processing needs the different struct than network needs: map-index instead of array, caching field for  calculated hash and so on.

Am i correct assuming that RLP was brought in only for the sake of easy code borrowing from go-ethereum?

@Maxime2, you are correct. The main aim of src/posposet package is to get a poset consensus-with-PoS proof and implement Lachesis algorithms.
So we focused on elegant and readable code. As you said: premature optimisation is the root of all evil.
However, I agree to replace RLP with ProtoBuf to get you approve of PR.


Toradex updates for sumo

@mirzak ping

@mirzak I redid this against the same commits you were using.  Note as discussed previously that these changes are for colibri-imx7 and not colibri-imx7-emmc.


Question: Where to add the function in an Ionic project

Hi guys!
I am new to Cordova and Ionic and I would love to know to include the usage function into my Ionic project? I assumed I have to add it to app.component.ts file but I get all kind of errors if I do so.
Would be awesome if someone could guide me a bit in the right direction.
Thank you!

Hi guys!
I made it work to integrate the plugin and I can successfully print the intent to the console and set it to local storage. Great plugin!
However, I am struggling to call an outside method from the inner actionHandler function (in this case a router method to redirect the user in case an intent is found). I am quite new to javascript and I am not sure how to achieve this, but understand that my current "this" implementation makes no sense (see below).
Thanks so much for helping out!
`document.addEventListener('deviceready', setupOpenwith, false);
function setupOpenwith() {
// Increase verbosity if you need more logs
//cordova.openwith.setVerbosity(cordova.openwith.DEBUG);
// Initialize the plugin
cordova.openwith.init(initSuccess, initError);
function initSuccess()  { console.log('init success!'); }
function initError(err) { console.log('init failed: ' + err); }
// Define your file handler
cordova.openwith.addHandler(myHandler);
function myHandler(intent) {
for (var i = 0; i < intent.items.length; ++i) {
var item = intent.items[i];
//this part works
localstorage.setitem('test', item.utis)
//this part does not work
this.router.navigate[['/profile']];
}

}`

This is not an issue but more a question thats why I am closing this.
But I can try to help you. You define your Handler as a function. Functions in javascript change the meaning of this. Try to use an arrow function (research it a bit you will understand after some practice).
So lets change function myHandler(intent) { to const myHandler = (intent) => { That should work and lets you use the this as your class.
I was confused with this while starting, too.

Thank you @niklas. This was the trick, thanks for pointing this out!


Media player BraviaTv : Try to connect only if tv is not in off state.

Description:
When HA is restart with the TV in off state there was error log every 10s until the TV is set ON.
This patch only to try to connect when TV is ON (the check of TV is ON do not log message)
No documentation change.
No new depencies.
No new files
Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

🐬


MemoryError and SystemError

Description
I generated features with DatasetFeatures() function for MIL. The number of tiles in TFRecords is over 10 million.
The following error occurred:

Number of tiles in TFRecords: 12825545
Annotation columns:
Index(['Unnamed: 0', 'slide', 'patient', 'grade', 'category'], dtype='object')
[01:57:18] INFO     Calculating activations for 1247 tfrecords (layers=['postconv'])
INFO     Generating from <TorchImagenetLayerExtractor model=resnet50 layers=['postconv'] n_features=2048>
SystemError: deallocated bytearray object has exported buffers
Exception in thread Thread-130:
Traceback (most recent call last):
File "/home/ochi/anaconda3/envs/slideflow2/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/home/ochi/anaconda3/envs/slideflow2/lib/python3.8/threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "/home/ochi/anaconda3/envs/slideflow2/lib/python3.8/site-packages/slideflow/io/torch.py", line 1021, in interleaver
record = next(sampler_iter)
File "/home/ochi/anaconda3/envs/slideflow2/lib/python3.8/site-packages/slideflow/tfrecord/iterator_utils.py", line 56, in iter
yield next(iterators[choice])
File "/home/ochi/anaconda3/envs/slideflow2/lib/python3.8/site-packages/slideflow/tfrecord/reader.py", line 149, in iter
yield from read_records(start_byte, end_byte)
File "/home/ochi/anaconda3/envs/slideflow2/lib/python3.8/site-packages/slideflow/tfrecord/reader.py", line 103, in read_records
self.datum_bytes = self.datum_bytes.zfill(_fill)
MemoryError
SystemError: deallocated bytearray object has exported buffers

To Reproduce
Codes to  reproduce the behavior:
features = sf.DatasetFeatures(resnet50, dataset)
features.to_torch('/bag/')
Environment:

Slideflow Version (e.g., 1.0): 2.0.3
OS (e.g., Ubuntu): Ubuntu
How you installed Slideflow (pip, source): pip
Python version: 3.8
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

It sounds like your system ran out of memory when attempting to generate these features.
The DatasetFeatures interface keeps image features for all tiles in memory before exporting. If you have a lot of image tiles (10 million), this is likely to exceed your system memory.
If you're generating features for MIL, a better solution would be to save features to disk as they are being calculated, instead of keeping this data in memory. This would eliminate the problem of excessive memory usage, but would prevent users from doing other downstream tasks that require all data to be in memory (like UMAPs, mosaic maps, etc).
We are planning to add a function in version 2.1 that allows users to generate features and save them to disk as they are being calculated, for downstream MIL, to reduce memory consumption. But it's not ready quite yet.
In the meantime, you can get around the high memory utilization by splitting the full dataset into many smaller parts, and calculating/exporting features for each part separately.
For example:
import slideflow as sf

slides = dataset.slides()
for batch in sf.util.batch(slides, 8):
    ftrs = sf.DatasetFeatures(..., dataset.filter({"slide": batch}))
    ftrs.to_torch(...)

This would calculate and explore features 8 slides at a time.
I'll keep this issue open and update it when we have the permanent solution for 2.1.

Your example codes enabled us to deal with entire tiles in all TFrecords without any errors, but I am waiting for releasing version 2.1
Thank you!


[question] Does it work with HDRP?

No. You need to change some builtin cg functions and variables into hlsl.


feat(tangle-dapp): Unstaking/Undelegating for Restaking

Summary of changes
Provide a detailed description of proposed changes.

[x] Implemented the UI and logic for scheduling an unstake request to unstake assets from an operator.
[x] Implemented querying and displaying of scheduled unstake requests in the unstake request table.
[x] Implemented the UI and logic for canceling an unstake request from the unstake request table.
[x] Implemented the UI and logic for executing an unstake request from the unstake request table.

Proposed area of change
Put an x in the boxes that apply.

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[ ] apps/zk-explorer
[x] libs/webb-ui-components

Reference issue to close (if applicable)
Specify any issues that can be closed from these changes (e.g. Closes #233).

Closes https://github.com/webb-tools/webb-dapp/issues/2407

Screen Recording
If possible provide a screen recording of proposed change.
https://github.com/user-attachments/assets/ecaceabb-95fa-4ac7-82e2-72df14053f6f

@monaiuu I can see that the UI here is very similar to the one on liquid staking, however since we have differing logic, I think it nay be difficult or at least messy to simply have a single component work for both. So I guess what I can do is make it look the same.
Thoughts on this? @AtelyPham

@monaiuu I can see that the UI here is very similar to the one on liquid staking, however since we have differing logic, I think it may be difficult or at least messy to simply have a single component work for both. So I guess what I can do is make it look the same.
Thoughts on this? @AtelyPham

If the UI is similar, we can create a UI component that contains only the UI code, such as styles, JSX, etc., without any logic code. This component can then accept props to customize its behavior and rendering.

@AtelyPham the column labels should not have a checkbox, we also need a label on top of the unstake requests table detailing what it is we're looking at. @monaiuu are there no labels in the design?

@AtelyPham the column labels should not have a checkbox, we also need a label on top of the unstake requests table detailing what it is we're looking at. @monaiuu are there no labels in the design?

The identity column in the design has not been implemented yet because the previous version of the multi-asset-pallet lacked that data.
This is the table in design.

Some conflicts here @AtelyPham


ReferenceError: can't access lexical declaration `offlinePeerCount' before initialization

Revision 888d3cf (zip loaded via about:debugging in Firefox 58) produces error on load if node is initially offline:

Unable to reproduce with the same .zip file under Firefox 58 in Linux (error was originally reported  under MS Windows).


TypeError from PyCLibrary

I'm not sure if I have mentioned this before, but sometimes I get the following error. I assume it's a race condition because its occurrence is random. It happens for maybe 1 in >100 pipeline executions, and I have only seen it some pipelines (though I haven't run all pipelines >100 times).
This error is from unit test "test_two_sequences" on feature/multi-sequence-support.
Traceback (most recent call last):
  File "/usr/lib64/python2.7/threading.py", line 551, in __bootstrap_inner
    self.run()
  File "/usr/lib64/python2.7/threading.py", line 504, in run
    self.__target(*self.__args, **self.__kwargs)
  File "build/bdist.linux-x86_64/egg/bifrost/block.py", line 243, in _main
    self.main()
  File "build/bdist.linux-x86_64/egg/bifrost/block.py", line 380, in main
    self.write('out_sum')):
  File "build/bdist.linux-x86_64/egg/bifrost/block.py", line 259, in izip
    next_set = [iterator.next() for iterator in iterators]
  File "build/bdist.linux-x86_64/egg/bifrost/block.py", line 285, in read
    for ring_name, sequence in self.izip(args, sequences)]):
  File "build/bdist.linux-x86_64/egg/bifrost/block.py", line 259, in izip
    next_set = [iterator.next() for iterator in iterators]
  File "build/bdist.linux-x86_64/egg/bifrost/ring.py", line 231, in read
    with self.acquire(offset, span_size) as ispan:
  File "build/bdist.linux-x86_64/egg/bifrost/ring.py", line 225, in acquire
    return ReadSpan(self, offset, size)
  File "build/bdist.linux-x86_64/egg/bifrost/ring.py", line 328, in __init__
    offset=offset, size=size), retarg=0)
  File "/home/mcranmer/usr/lib/python2.7/site-packages/pyclibrary-0.1.4-py2.7.egg/pyclibrary/c_library.py", line 498, in __call__
    res = self.func(*arg_list)
ArgumentError: argument 2: <type 'exceptions.TypeError'>: expected LP_s instance instead of LP_s

I should note that the stated occurrence of 1 in 100 has not been reproduced in other build systems, such as travis-ci, as in I have never seen this issue outside of my own machine (or inside a docker container on my own machine).
My guess is therefore that this issue is due to something about my build system.

Closing this issue until I am able to reproduce it in another build environment.


static extension methods won't work, but the analyzer won't complain about declaring one

so, this is my dart version string:
Dart SDK version: 2.10.0 (stable) (Mon Sep 28 09:21:23 2020 +0200) on "linux_x64"
I've been able to reproduce the same as well in a dartpad though...
So the problem is: I was trying to add a static method to a class via extensions, something like this:
class A {
  int a = 1;
}

extension B on A {
  static A of(String c){ //no error here
    if(c == "a"){
      final a = A();
        a.a=2;
      return a;
    }else{
      return A();
    }
  }
}

int i = A.of("a"); //error here

this fails to compile because the of method isn't found:
Error compiling to JavaScript:
Warning: Interpreting this as package URI, 'package:dartpad_sample/main.dart'.
lib/main.dart:17:9:
Error: Method not found: 'A.of'.
A i = A.of("a");
        ^^
Error: Compilation failed.

So it looks like dart doesn't support static extension methods, that's fine, but in that case I would expect to get a compiler/analyzer error already when declaring the method.

A static method on an extension can be called directly, e.g.,:
  var a = B.of("a");

so the declaration is valid / usable.

A static method on an extension can be called directly, e.g.,:
  var a = B.of("a");

so the declaration is valid / usable.

I'll mark this as a lint request for consideration.  FYI - @lrhn @leafpetersen

I'll mark this as a lint request for consideration.  FYI - @lrhn @leafpetersen

A static method on an extension can be called directly, e.g.,:
  var a = B.of("a");

so the declaration is valid / usable.

ah, I see.
Thanks for clearing that up

A static method on an extension can be called directly, e.g.,:
  var a = B.of("a");

so the declaration is valid / usable.

ah, I see.
Thanks for clearing that up

@vsmenon what would the lint rule report? Any static declaration on an extension?

@vsmenon what would the lint rule report? Any static declaration on an extension?

Apologies, stale.


chore: npm-check-updates && yarn upgrade

Ran npm-check-updates and yarn upgrade to keep the yarn.lock file up-to-date.

superceded #2843


Update chatbox design and add send icon SVG file

This pull request includes several commits that update the design of the chatbox and add a new send icon SVG file.
The commits also fix the position of the "TOP" button (for scrolling up) to the left side to avoid the open chatbox display over it, as the chatbox is now at the right; following the more conventional styling
It adjusts the position and CSS styles of the chatbox header and footer. Additionally, the commits refactor the chatbox UI to improve the user experience by increasing the height of the chatbox and ensuring that the textarea resizes properly, closing #455

Summary by CodeRabbit

New Features

Introduced a close button for easier dismissal of the chatbox.
Added predefined messages for enhanced initial user engagement.
Implemented dynamic textarea height adjustment for improved usability.

Enhancements

Updated chatbox design with a modern background and improved button styles for better interactivity.
Enhanced visual hierarchy and organization of chatbox components, including refined typography and message timestamps.
Added a footer section crediting "Blogthedata" for improved branding.

Bug Fixes

Improved textarea functionality to prevent overflow and ensure consistent height adjustments.

@jsolly awaiting your review

Looking pretty good @freedompraise
Here are some thoughts:

You might want to take a look at the formatting of the blog post title + date for blog post cards for inspiration

Instead of having a button background color on hover, consider adjusting the color of the icon itself. E.g if the user hovers over the 'x' or the send message button, the svg can change color, instead of the whole background changing color.
I do this on my website when you hover over the social icons:

See https://johnsolly.dev/about/

See above comment for changes.
If I recall correctly, it is not possible to change an SVG color if it is inside /static and referenced via static in an img tag.
You might need to put the SVGs directly in the HTML in order to manipulate them dynamically.

Alright, noted

@jsolly for the close button having a bg color, I thought about that too.
I discovered the background is coming from a global styling on all buttons.
What solution do you suggest?

@jsolly I've effected the changes

Okay

@jsolly awaiting your review


[Help Needed] - Want to change current login page with Keycloack

I am new to Keycloack and my client product is using KeyCloack for identity management. Right now the frontend is using custom Form field with react-hook-form and sending the request to the keylock URL to authenticate which is deployed on the server. Now they don't want to use custom fields, they wanna use the keylock login page but they want to keep the current theming and UI and only wanna change the custom fields with keycloack.
This is the current Login page:

Now what we want is to keep the page as it is as there are other things happening on this login page and client only wants to change the login form that is circled in red with keycloack login form.
So in the end, users will not see any difference in the login page and the only thing that will be different is the login form will be from keycloack instead of the custom one that we already have.
I was playing with keycloackify and I understand that I can customize the whole page, but what I don't understand is how can I use this in my current login page.
Is it even possible to do that?
Frontend Specs:

React 18
Vite
MUI 5

Is it even possible to do that?

No, it's not. Maybe with an iframe but this is hacky as hell. It goes against the core principle of OAuth, that imply that the whole authentication process happens outside the application.

Right now the frontend is using custom Form field with react-hook-form and sending the request to the keylock URL to authenticate which is deployed on the server.

What you company was doing it generally considered an antipatern. See this post.
That beeing said, looking at your screenshot, it seems like you're using MUI. You can easyly replicate the same exact page as a Keycloak theme, the user won't even notice it's being redirected away.

There is a tutorial video for creating Keycloakify theme with MUI.
https://youtu.be/PhNE-3EwwP8?si=A4B82k_i1w-qHG9N
In the tutorial, the approach implemented is to start from the default styles and incrementally customize them.
In your case since you already have the all page ready to go, I'd just make sure to set the correct input names, urls and accomodate for the error messages but mainly start from scatch.
Make sure to se "doUseDefaultCss" to false for the login.ftl

Okay, so that means I have to create this whole page from scratch using keycloackify?
But how can I add extra stuff that I already have in my current login page which is not a part of keycloak login flow?

Be more specific

I mean the login.ftl file will only have the login related components

So, how can I add other components that I have in my current login page?

Keycloakify theme are just Vite/React (or Vite/Angular) web apps!
You can do whatever you want with it. Keycloakify provide you the keycloak context so that you know where to submit the forms.
It's not a restrictive framwork. It just comes by default with the react components that matches the aspect of the default keycloak theme but you're not forced to use thoses components.

Hi @muhammadjasim78,
I'm not sure if this will help you, but in this repo, I am adding my own components. I am using ShadCN and Tailwind.
For example, something like the scroll area is not part of Keycloakify by default.

You can check out the live demo of all the added components here:
View Storybook
If you'd like to learn more about this repo, you can check out this video as well:

Watch this video on YouTube

Hey @nima70,
Thanks for getting your help and offering support.
I think he's using MUI, the best strat here for him is probably just to put the main frame of he's page in template and have a simple form using MUI inputs component on the login page.
Tailwind and ShadeCN is another stack.

Hi @garronej
Oh, I see. Noted.

But don't get me wrong, I'm super happy if you can help pepole here. There's a ton of pepole that uses tailwind/shadecnUI.

Hi @garronej
No worry at all. It is all good.😉

Thanks for your help guys @garronej @nima70
It simplified a lot of things for me.
Thanks for the quick support.
Closing this issue.

@muhammadjasim78. You are welcome. It was our pleasure.

Hi @garronej
I understand the theming concept now.
Now I have another question related to redirects. I tried finding it on keycloack but there is no good answer over there.
So right now my current login page is at https://smartlab.ladecloud-dev.de/login
And now the main requirement for now is to create https://smartlab.ladecloud-dev.de/login-v2 where the keycloack login page should be placed which will be the exact copy of https://smartlab.ladecloud-dev.de/login but it will be designed with keycloakify theming system and pushed to keycloak as a theme.
I was playing with keycloack admin and there for every client that I create I get this kind of URL
https://my-keycloack-domain/auth/realms/my-keycloack/protocol/openid-connect/auth?client_id=my-client-id&response_type=code&redirect_uri=https%3A%2f%2fsmartlab.ladecloud-dev.de
Can I add proxy to this URL somehow as in keycloack issues they said, it is not possible: https://github.com/keycloak/keycloak/discussions/17690
Do you have any insights on that? we really don't wanna use iframes and according to the requirements if I will go to https://smartlab.ladecloud-dev.de/login-v2 I should land on my keycloack themed login page
Thanks

Hello @muhammadjasim78,
No it's impossible. It's not how OAuth works.  You can however redirect to the Keycloak login pages when they navigate to https://smartlab.ladecloud-dev.de/login with a library like oidc-spa (or whatever oidc client library you are implementing).
Do not use iframe, do not try to bend the standards to your requirements.
In this url:
https://my-keycloack-domain/auth/realms/my-keycloack/protocol/openid-connect/auth?client_id=my-client-id&response_type=code&redirect_uri=https%3A%2f%2fsmartlab.ladecloud-dev.de
This part can be customized: https://my-keycloack-domain/auth
This part cannot: realms/my-keycloack/protocol/openid-connect/auth?client_id=my-client-id&response_type=code&redirect_uri=https%3A%2f%2fsmartlab.ladecloud-dev.de
The users cannot directly navigate to the login pages, the redirectioin have to be initiated by your app.
You have two options here:

Change your requirements.
Roll your own auth


Provide implementation for Core Options newer format

Implement RETRO_ENVIRONMENT_SET_CORE_OPTIONS and RETRO_ENVIRONMENT_SET_CORE_OPTIONS_INTL by
figuring out how to read and convert the retro_core_option_definition struct to C#.

Turns out, cores that requests RETRO_ENVIRONMENT_SET_CORE_OPTIONS do work...
The issue comes from the RETRO_ENVIRONMENT_SET_CORE_OPTIONS_INTL environment request, where the pointer to the 'us' member appears to by invalid.
I re-added the previous parsing so cores using RETRO_API_VERSION == 1 can work again.
Cores using RETRO_API_VERSION == 1 that request RETRO_ENVIRONMENT_SET_CORE_OPTIONS_INTL can be forced to use RETRO_API_VERSION == 0 by adding them manually to "StreamingAssets/libretro~/cores_using_options_intl.json" (default location).
Example:
{
    "Cores": [
        "vecx"
    ]
}

PS: It may or may not, depending on the core, disable some of the core's internal features. But at least there's something to work with...


Ln437 in peft_model : hub_file_exists  will cause unhandled exception when TRANSFORMERS_OFFLINE=1

System Info
peft "0.4.0.dev0"
Who can help?
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder
[ ] My own task or dataset (give details below)

Reproduction
I use generate.py by https://github.com/tloen/alpaca-lora
Expected behavior
hub_file_exists in hub_utils
except :
return False
or if_offline_mode_is_enabled ,return False

Thanks for reporting. Yes, having an unhandled exception here is not nice. However, I think if we reach that code path, we have to raise some kind of error, since it means the weights couldn't be found locally and should not be downloaded from HF Hub. So would your expected solution just be to raise a better exception?