PSScriptAnalyzer Logo or Wallpaper

Hi,
your designs look really great. Would you be interested in creating an icon or a Wallpaper for PSScriptAnalyzer? There is no business need for it so don't feel obliged to do so but if enjoy doing your work, then I think some folks might also enjoy seeing it in other places. For PSScriptAnalyzer, we could put the logo on the main Readme of the repo or set the wallpaper as the 'social preview' image of the repo to make it more recognizable and stand out.

Hey @bergmeister, this sounds like a great idea! I'd love to. Let's aim for both logo and banner/preview. Can you provide a couple ideas for inspiration, and then I can get a draft going?

Thanks. I don't have anything specific in mind but maybe something similar ESLint for the logo
https://images.app.goo.gl/GUyMvLbxZ9Hfn5QJ7
Or maybe something with PoshChan similar to the GoLang linter?
https://images.app.goo.gl/B8V2y1taLfZkBp7A7

I'd say a magnifying glass incorporated into the logo would be cool.
https://www.sqlrepairtool.net/img/sql-log-analyzer.png
This is a database tool but just reference to see the idea.
If you have maybe the new PS logo, or modified version of it, with the magnifier on there somewhere.

These are both great directions, @wsmelton and @bergmeister! I'll work on some designs with this inspiration , and report back.

How about a lightbulb similar to what shows up in VSCode with a green squiggly line as the filament per @jwmoss

Hi Peeps!
Sorry I've been a bit preoccupied with career changes, and turning 40!
Will start work on this this week, and should have some designs up by this weekend.
Thanks for the additional suggestion @stevenjudd and @jwmoss !

Just put together a draft of @wsmelton's suggestion of a magnifying glass and PowerShell logo. Let me know what y'all think.

@adilio Looks really awesome, with lots of detail, I'd be very happy to take it. I like the way the logo is placed inside the magnifier, the non-central position makes it look so much better.
Are the 2 inner bits inside the magnifying glass on the top left shadows? Together with the PS logo, they look a bit confusing, maybe make them smaller or leave them out?

He @bergmeister, really enjoying the feedback here and on Twitter here: https://twitter.com/CBergmeister/status/1147794431149518848
I do appreciate Jakub @nohwnd's feedback about the rectangular shape more suited to a logo. I'm going to re-work the design so the magnifying glass fits on the PS Core logo. ALso, I will icorporate the PSSA in some way. Stay tuned!

Round 2. Incorporating some ideas thus far, including:

Rectangular (skewed) logo
PSSA branding
Green squiggly line

Please provide feedback 😁 .

my suggestion would be to increase the contrast between the magnifying glass and the PS logo behind it. the rim of the magnifying glass can blend into the background in a small version of the logo. there is probably nothing to change with the magnifying glass if the classic blue PS logo is used behind it.

Good point, @compwiz32! So, part of my motivation behind lining up the angle of the PS log and magnifying glass handle was to accent the same underlying shape. If that makes things more confusing or unclear, I'm open to tweaking it more.
I'll continue soliciting suggestions, and roll all feedback into one edit.

To me the new version is a step back. I really liked the way the magnifying glass inspects the powershell logo. 🙂 Now it does not feel like it anymore, because I can't see the powershell logo, and because I am right-handed the new position of the magnifying glass does not even feel like I am using it.
Realising what the green squiggly line means also took me way too long. To me it would work if the logo was just "PS Script Analyzer" with the squiggly line under "Analyzer", but in this logo it does not fit.

Fair enough, @nohwnd. I guess your suggestion of square logo is what i was going for, and also I'm left-handed so maybe that explains the orientation 🤣. Actually, I did that to make it fit the skew of the PS logo better (as it lines up with angle bracket symbol better).
Your comments are appreciated, though. I don;t want to try and incorporate too much, as the beauty is in the simplicity. Will have to think on this further.

Wouldn’t it be enough if it just magnified the logo underneath the magnifying glass, and just skip the letters. :thinking:

Hi @bergmeister and all, I've created a folder for this project here:  https://github.com/adilio/ps-design/tree/master/psscriptanalyzer
Based on feedback, I've revised the design to include:

magnification instead of acronym
original orientation of magnifying glass, as well as drop shadow to give dimension
sample of PSScriptAnalyzer with green squiggle underneath (this can be used in addition to simple logo, or not)

The evolution continues. Appreciate the feedback.

Awesome, this logo is the best of all previous logos, I like the magnifying effect and the asymmetric placements.
I still find the shadows inside the magnifier a bit disturbing. Personally, I am wondering if making the magnifier 10-20% bigger (size still smaller than Ps logo) would make it look even better

Looks good!

This is amazing work. It really looks good.

I am liking the iterations on this. Would it help to put “PSSA” on the magnifying glass handle, and the green squiggly line as well? I think the shadows helps give it depth, but maybe tone them down, so that they are present, but not obvious. Also, just spitballing, but if you wanted to drop the handle and go for a mechanical arm that comes in from the top left or right corner, it could line up with the “>” angle brackets again, or compliment the angles. I am impressed with what you have already done!

Also, in case anyone is curious about the rectangular thingy... it’s called a rhombus, helllooo!

P.S. It took way to long to learn how to put a gif in a comment...

The problem with making the shadow any less prominent, and the writing on the handle, is that it is a logo. Logo's get shrunk down real tiny, and you want to be ensure that subtle elements are still visible in smaller form. I have made the shadow a little less subtle, but any more and it will disappear on shrinking.
@bergmeister, based on your suggestion, I've removed the light reflection on magnifying glass lens. Also, I've increased the size of the magnifying glass. Ideally, I'd like to have move the magnifying glass up a little, but I'd have to redo the magnified image, and that's.... effort 🤣.
I do kind of like how the handle hangs down almost into the text.

Great work, this got much better proportions and looks like a real logo. 👍
The first version with lens "shadows" looks more refined to me, because it make the magnifying glass look more real. And I don't find it disturbing at all. Especially when I zoom out.
The subtle shadow on the second one is definitely better.
The text needs a bit more work:

the shadow on the text is imho unnecessary.
I would choose another font. Here is a list of some cool free fonts. https://www.logaster.com/blog/best-free-logo-fonts/ Qanelas, beaver, bebas, aileron, maxwell, andis, coves all look interesting. Here a mockup with MuseoSans that I quickly put together in PowerPoint (hence no squiggly line :D) the positioning is far from ideal but the thicker rounder font seems more flattering.

The squiggly line should have more height, and maybe repeat slightly more often. Example of squiggly line from vscode.

I may have over read the discussion on this but why have text on a logo that won't be readable (e.g. once taken down to a logo size on PS Gallery).
I look at it as once the logo is associated with the module, the text is a mute point.
Just another point of view on the green line. What about having it outline the inside of the magnifying glass? Just a play on words type thing instead of visually trying to fit a squiggle line (that just like words is not going to be very readable once shrunk down to size).

@wsmelton I would use that version of the logo with the text and the squiggly line in repo readme, but for the PSGallery I would use just the logo without any text.

Appreciate all the feedback.
@nohwnd is right, the logo without text serves a different purpose, whereas the one with text can be used for added recognition (like on stickers).
As we have refined the design a bit, it's coming down to more personal preference, and I'll do my best to cater to whatever the majority decides. But I do think I'm kind of done!
@bergmeister, here is my final-ish version, barring something glaring.
Couple of changes:

Font to Segoe UI Semi-Bold (MS/Windows font). I prefer to keep it generic. You never know when something is listed free in the internet, and later is not. Copyrights can be pesky.
Green squiggle given higher frequency, and made taller; also given 60 % opacity
Text shadow removed
Magnifying glass reflection brought back (I like it better, too 🤷♂)

And here is just the logo:

All versions are in: https://github.com/adilio/ps-design/tree/master/psscriptanalyzer
Have a good night all!

@adilio looks awesome, great job :)

Awesome work @adilio I agree the last one is the best of all worlds :-)
I will start various PRs the next day and speak to people. The plan is to add the logo to the repo and then have the IconUri point to it so that one sees it on the PSGallery.
There is also a feature on GitHub where one can modify the logo being displayed if one pastes e.g. a link in Twitter. I'll try to also add the logo as a custom emji to the PSSlack channel. Maybe a long shot but files named PSScriptAnalyzerSettings.psd1  could display that logo in VSCode as well (the icon repo for it was blocked for a while but got unblocked yesterday). Any other ideas on where to use the icon?

Sounds awesome, @bergmeister! I'll have to think about it more and let you know.
I will mention that the SVG file is the vector image, so that's the one you would use to scale up or down. If you need PNG's or other formats in different pixel or size dimensions, please let me know, and I'd be happy to prepare them.
Oh, and if you get a chance, please let the folks in Twitter know about your design and the ps-design repo, in case they want to collaborate on a design of their own! 😄
Cheers!

Thanks for your hard work, it looks awesome. I will speak to people and try to get things going the next days, updating the PSSA preview picture of the repo on links should be quite quick but it could be that we see the new logo only at the next release in the PSGallery as it's a property in the manifest, I still have to figure that out and play with it. Shall I close the issue now?

Awesome job, looks great. . .
I noticed on the twitter post it came out with a black background . . so the text is hard to see.

https://twitter.com/adilio/status/1148313230491521024

Thanks for the heads up, @brwilkinson!
I have created a "white text" version here:
https://github.com/adilio/ps-design/blob/master/psscriptanalyzer/pssa-final-white-text.png
I have also replied to @bergmeister's thread here:
https://twitter.com/adilio/status/1149063785761521664
Hope that helps!

haaa now it looks great on the normal twitter feed and clear on the pop up
Perhaps a black with a silver outline?  😁

Hehe, this would definitely be a wormhole to go down 🤣.
Looks fine on Twitter with Dark Mode, so I'm just gonna leave it 🤷♂. Appreciate the heads up, though!

@adilio haa yeah definitely....
looks great anyway.


Removing configlet lint from critical CI path.

Configlet lint was configured as a prerequisite for CI. It should instead just be an FYI, and checked manually on any PRs for changes to the config file.
It still runs as a separate Github Action "Configlet CI".
As this is fixing our CI pipeline, which will unblock other changes, if I can I will just approve and merge this myself once it passes.
Passes the current linting checks #422
Closes #420.

I can't merge this myself, so this will sit here until someone else can take a look.
This blocks pretty much any other change, as none of the tests will run without this change.

Works great locally, merging.


Check for return statements before mutex unlock

This check aims to catch errors where a mutex is locked and the function can return before unlocking.
var (
       cacheMu sync.RWMutex
       cache = make(map[string]string)
)

func retrieveItem(key string) string {
       cacheMu.Lock()
       if item, ok := cache[key]; ok {
              return item  // mutex not unlocked
       }
       cacheMu.Unlock()

       // ...retrieve item due to cache miss...
}

To avoid false positives where someone may be wrapping a mutex, this check does not warn when the function does not contain an Unlock call.
This is my first time working with the ast package. If there is a better way of implementing this check I'd be happy to rework the PR.

After looking at this a bit more, think the main logic would be more succinct as below, but I don't want to push a bunch more changes before you have a chance to look at it.
func CheckReturnBeforeMutexUnlock(f *lint.File) {
	var returnLocks []*ast.ReturnStmt
	var locks, unlocks int

	funcVistor := func(n ast.Node) bool {
		if r, ok := n.(*ast.ReturnStmt); ok {
			if locks > unlocks {
				returnLocks = append(returnLocks, r)
			}
			return true
		}

		sel, ok := n.(*ast.SelectorExpr)
		if !ok {
			return true
		}
		fn, ok := f.Pkg.TypesInfo.ObjectOf(sel.Sel).(*types.Func)
		if !ok {
			return true
		}
		sig := fn.Type().(*types.Signature)
		if sig.Params().Len() != 0 || sig.Results().Len() != 0 {
			return true
		}

		switch fn.Name() {
		case "Lock", "RLock":
			locks++
		case "Unlock", "RUnlock":
			unlocks++
		}

		return true
	}

	for _, decl := range f.File.Decls {
		if _, ok := decl.(*ast.FuncDecl); !ok {
			continue
		}

		locks, unlocks = 0, 0
		returnLocks = nil
		ast.Inspect(decl, funcVistor)
		if unlocks == 0 {
			continue
		}
		for _, r := range returnLocks {
			f.Errorf(r, "return before mutex unlock")
		}
	}
}

This is a very ambitious check, and unfortunately not one that's at all easy to implement. The primary issue with this way of implementing it is that there's a thousand ways of triggering false negatives, rendering the check a lot less effective.
I'm not sure yet if it also suffers from false positives, I'll do that analysis soon.

My aim was to to identify the common cases and avoid any false positives. It is indeed possible to create a false negative, but the examples I've come up with so far haven't been very realistic. That said, it's completely understandable if this approach ends up not being rigorous enough for this project, either way I learned a bit about the ast package :).

I think I'll have to reject this PR. The easiest way to trigger a false positive is this piece of code:
func foo(b bool) {
	var mu sync.Mutex
	if b {
		mu.Lock()
	} else {
		return
	}
	mu.Unlock()
	return
}

Admittedly, it's not idiomatic code, but it's realistic code. I'm going to close it, instead of suggesting fixes to your algorithm, because it's really a deep rabbit hole, and the hole leads to detecting feasible paths and other undecidable problems, and all real-world algorithms either permit a certain amount of false positives, or rely on code annotations, neither of which I want for staticcheck.
If you want to read more about this topic, research race and deadlock detection as well as lock pairing. http://web.stanford.edu/~engler/racerx-sosp03.pdf is an interesting paper on one particular implementation.


Show envido cards at the end of the round

And only update Envido-related score then (thanks @devblac)

There should be a new action: REVEAL_ENVIDO_CARDS, that is mandatory before CONFIRM_ROUND_ENDED only for a player who has won envido but hasn't revealed one of more cards of that score.
Envido-related points should be in "escrow" until the cards are revealed. This could happen naturally by revealing cards or at the time of this action. This is also a good opportunity to communicate this information to the UI. There should be a score int and an enMesa bool in the REVEAL_CARD action, as well as in REVEAL_ENVIDO_CARDS.

The action of showing cards is fixed by 7fc169e2568004a91138a0b72e0dce6587f50d57.
The points being in escrow is still pending improvement.

Fixed awarding envido points when envido cards are revealed on 9034e13.
The React UI has been updated to reflect the fixes.


Moving from the static library based solution for telemetry to a head…

…er based approach to reduce complexity.
The difference (on disk) size between the lib based approach and header based approach is 1 KB (header > lib). In favor of reducing the complexity, the decision has been made to go with a header based solution

/azp run

    TraceLoggingBool(true, "UTCReplace_AppSessionGuid")

do we want this can be opt out?

Refers to: dev/WindowsAppSDK_Insights/WindowsAppSDKInsights.h:62 in 4e7bd28. [](commit_id = 4e7bd288fb81e981a5e8b8afee21b8ecd3d27194, deletion_comment = False)

    TraceLoggingBool(true, "UTCReplace_AppSessionGuid")

This is needed. There were WDX wide discussions to make it part of common PartB.

In reply to: 901503109

Refers to: dev/WindowsAppSDK_Insights/WindowsAppSDKInsights.h:62 in 4e7bd28. [](commit_id = 4e7bd288fb81e981a5e8b8afee21b8ecd3d27194, deletion_comment = False)

/azp run

/azp run

/azp run

/azp run


Info: wfrest can build on ARM64 and AMD64 , either docker or podman

Tested docker build wfrest, on platform ubuntu arm64 and amd64, docker and podman both works.
make[2]: Leaving directory '/home/project/workflow/build.cmake'
Install the project...

-- The C compiler identification is GNU 9.3.0
-- The CXX compiler identification is GNU 9.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Build Type: Release
-- Found OpenSSL: /usr/lib/aarch64-linux-gnu/libcrypto.so (found version "1.1.1f")
-- Found ZLIB: /usr/lib/aarch64-linux-gnu/libz.so (found version "1.2.11")
-- workflow exists(version >= 0.9.9), not build
-- CXX_FLAGS = -g -std=c++11 -rdynamic -pthread -fno-exceptions -O2 -DNDEBUG
-- lib/cmake/wfrest
-- Configuring done
-- Generating done
-- Build files have been written to: /home/project/wfrest/build
Scanning dependencies of target wfrest
[  4%] Building CXX object wfrest/CMakeFiles/wfrest.dir/StrUtil.cc.o
[  8%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpMsg.cc.o
[ 13%] Building CXX object wfrest/CMakeFiles/wfrest.dir/Router.cc.o
[ 17%] Building CXX object wfrest/CMakeFiles/wfrest.dir/RouteTable.cc.o
[ 21%] Building CXX object wfrest/CMakeFiles/wfrest.dir/UriUtil.cc.o
[ 26%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpDef.cc.o
[ 30%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpFile.cc.o
[ 39%] Building C object wfrest/CMakeFiles/wfrest.dir/MultiPartParser.c.o
[ 43%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpContent.cc.o
[ 47%] Building CXX object wfrest/CMakeFiles/wfrest.dir/PathUtil.cc.o
[ 34%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpServerTask.cc.o
[ 52%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpServer.cc.o
[ 56%] Building CXX object wfrest/CMakeFiles/wfrest.dir/Timestamp.cc.o
[ 60%] Building CXX object wfrest/CMakeFiles/wfrest.dir/SysInfo.cc.o
[ 65%] Building CXX object wfrest/CMakeFiles/wfrest.dir/Compress.cc.o
[ 69%] Building CXX object wfrest/CMakeFiles/wfrest.dir/BluePrint.cc.o
[ 73%] Building CXX object wfrest/CMakeFiles/wfrest.dir/base64.cc.o
[ 78%] Building CXX object wfrest/CMakeFiles/wfrest.dir/ErrorCode.cc.o
[ 82%] Building CXX object wfrest/CMakeFiles/wfrest.dir/FileUtil.cc.o
[ 91%] Building CXX object wfrest/CMakeFiles/wfrest.dir/MysqlUtil.cc.o
[ 91%] Building CXX object wfrest/CMakeFiles/wfrest.dir/HttpCookie.cc.o
[ 95%] Building CXX object wfrest/CMakeFiles/wfrest.dir/Aspect.cc.o
[100%] Linking CXX static library ../lib/libwfrest.a
[100%] Built target wfrest
[100%] Built target wfrest
Install the project...
-- Install configuration: "Release"
-- Installing: /usr/local/lib/cmake/wfrest/wfrest-config-version.cmake
-- Installing: /usr/local/lib/cmake/wfrest/wfrest-config.cmake
-- Installing: /usr/local/lib/libwfrest.a
-- Installing: /usr/local/lib/cmake/wfrest/wfrest-targets.cmake
-- Installing: /usr/local/lib/cmake/wfrest/wfrest-targets-release.cmake
-- Installing: /usr/local/include/wfrest/AopUtil.h
-- Installing: /usr/local/include/wfrest/Aspect.h
-- Installing: /usr/local/include/wfrest/BluePrint.h
-- Installing: /usr/local/include/wfrest/Compress.h
-- Installing: /usr/local/include/wfrest/Copyable.h
-- Installing: /usr/local/include/wfrest/ErrorCode.h
-- Installing: /usr/local/include/wfrest/FileUtil.h
-- Installing: /usr/local/include/wfrest/HttpContent.h
-- Installing: /usr/local/include/wfrest/HttpCookie.h
-- Installing: /usr/local/include/wfrest/HttpDef.h
-- Installing: /usr/local/include/wfrest/HttpFile.h
-- Installing: /usr/local/include/wfrest/HttpMsg.h
-- Installing: /usr/local/include/wfrest/HttpServer.h
-- Installing: /usr/local/include/wfrest/HttpServerTask.h
-- Installing: /usr/local/include/wfrest/Macro.h
-- Installing: /usr/local/include/wfrest/MultiPartParser.h
-- Installing: /usr/local/include/wfrest/MysqlUtil.h
-- Installing: /usr/local/include/wfrest/Noncopyable.h
-- Installing: /usr/local/include/wfrest/PathUtil.h
-- Installing: /usr/local/include/wfrest/RouteTable.h
-- Installing: /usr/local/include/wfrest/Router.h
-- Installing: /usr/local/include/wfrest/StrUtil.h
-- Installing: /usr/local/include/wfrest/StringPiece.h
-- Installing: /usr/local/include/wfrest/SysInfo.h
-- Installing: /usr/local/include/wfrest/Timestamp.h
-- Installing: /usr/local/include/wfrest/UriUtil.h
-- Installing: /usr/local/include/wfrest/VerbHandler.h
-- Installing: /usr/local/include/wfrest/base64.h
-- Installing: /usr/local/include/wfrest/json.hpp
-- Installing: /usr/local/include/wfrest/json_fwd.hpp
-- Installing: /usr/local/include/wfrest/BluePrint.inl
-- Installing: /usr/local/lib/cmake/wfrest/FindBrotli.cmake
STEP 6: COMMIT wfrest
--> 21f71b2f0a8
Successfully tagged localhost/wfrest:latest
21f71b2f0a8bc5d1d8c3ca6dd4b2b15093eff3f409f9ebdc7ca87848ba6719ea

This issue tracking is just for information , not for bug. i will close this track  in tomorrow.

Thanks for your contribution!

My honor , sir


Roll engine c6f9fe7e00e2..4d9b9aacd076 (2 commits)

https://github.com/flutter/engine/compare/c6f9fe7e00e2...4d9b9aacd076
git log c6f9fe7e00e2648de1a7aa02673b7771a9b92c49..4d9b9aacd07602031ab9cee863a74ecf02686ed4 --no-merges --oneline
4d9b9aacd Roll src/third_party/skia e1f2a572cec0..143542fa28e9 (1 commits) (flutter/engine#8192)
c14fde841 Roll src/third_party/skia d95286de1dda..e1f2a572cec0 (1 commits) (flutter/engine#8191)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


[Question] Is there a way to force center the now-line?

Is there a way to force center the now-line? Currently I have set :watchRealTime="true" and I want to make it so that the now-line stays in the view, with scrolling enabled or disabled. Is there a way to achieve this using the vue-cal api?
Thanks!

Please make this easer to understand. You want scrolling, but you want the current time line to always be visible?
Pure usage-related questions are probably better to post on Stack Overflow than as an issue in this repo.
In any case, a minimal reproduction demonstrating the issue using CodePen, CodeSandbox or similar is needed to be able to help you.

You need to do this from outside of Vue Cal by auto-scrolling the view in a setInterval. It's a costly process that will not be part of the Vue Cal codebase.
Closing the issue now.


Add support for android support library 23.1.1

android.support.v7.internal.* was moved to android.support.v7.*
now it's necessary to explicit attrs for android:prompt and android:disableChildrenWhenDisabled
improve gradle build system with shared compile target and build tools values

There were no commits since months, is there a better, supported fork?

thanks that save my life.

@andrewpros  check
https://github.com/fernandodev/material/commit/bc14c21bef31aa734f300710ec08baf380f12fc3

@AppDev666 Yeah i know, but i want a gradle workflow, the author have not even time to marge pulls and make new release, then this project is really dead.

https://github.com/rey5137/material/releases/tag/1.2.2


leaves dont sway

Description
leaves don't sway in the wind with shaders on
To Reproduce
Steps to reproduce the behavior: put on any shaders and look at one of the trees and you can see it doesn't sway
Screenshots

Rendering Glitches
I'm using optifine (for the shaders)
Additional context
the leaves don't sway like if you put them down yourself they just sit there
Crashes
If you are crashing you must include a crash report. The aforementioned rules apply when attaching crash reports.
Mods List
Versions

Dynamic Trees: beta 24
Forge: 36.2.2
OS: windows 11

OptiFine doesn't allow for for us to do this. There's simply nothing we can do. You need to see how it's done for the shader pack you're using an add the leaves blocks from Dynamic Trees manually (same reg name as the vanilla ones but with namespace dynamictrees).


fix IndexError

This fix is not exactly correct.
The leftmost None page will be gone:

[10,11,12,13,14,15,None]

And in template it results in this:

previous 10 11 12 13 14 15 ... next

(missing ... on the left side)


Any idea about private register api?

Which client are you using?

[x ] app (instagram_private_api/)
[ ] web (instagram_web_api/)

Describe your issue
So I want to register new account with private instagram api
With the web version I know how to do it

  {
   "created": "2018-07-05T01:13:38.320Z",
   "request": {
    "hasPostData": true,
    "headers": {
     "Accept": "*/*",
     "Content-Type": "application/x-www-form-urlencoded",
     "Origin": "https://www.instagram.com",
     "Referer": "https://www.instagram.com/",
     "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.87 Safari/537.36",
     "X-CSRFToken": "txxxxxxxxxxxxxxxxxxxxxxxxxxxxE",
     "X-Instagram-AJAX": "xxxxxxxxxxxxxxxxxxxxxxxx",
     "X-Requested-With": "XMLHttpRequest"
    },
    "initialPriority": "High",
    "method": "POST",
    "mixedContentType": "none",
    "postData": "email=example%40yandex.com&password=1example&username=example&first_name=some+user&seamless_login_enabled=1&tos_version=row&opt_into_one_tap=false",
    "referrerPolicy": "no-referrer-when-downgrade",
    "url": "https://www.instagram.com/accounts/web_create_ajax/"
   }

But with the private api I don't know how ! any help ?

It's not supported and never will be. https://github.com/ping/instagram_private_api/issues/55#issuecomment-368289939

@notcammy Why not?

Try reading the contents of the link I gave you.


Change width for pagination dot current index

Hello! I use this example for pagination dots
https://github.com/dohooo/react-native-reanimated-carousel/blob/fae0fb7d8e8d6afb03247e55f12fe57dacc57859/exampleExpo/src/parallax/index.tsx#L56
but I don't know, how can I change size (width or height) slowly for my pagination dots with current index. This is a picture to understand my problem.

I found only this  solution:
const PaginationItem: FC<{
index: number;
length: number;
animValue: Animated.SharedValue;
}> = ({animValue, index, length}) => {
const width = 28;
const animatedWidth = useAnimatedStyle(() => {
return {
width: withTiming(Math.round(animValue.value) === index ? 28 : 14),
};
});
const animStyle = useAnimatedStyle(() => {
let inputRange = [index - 1, index, index + 1];
let outputRange = [-width, 0, width];
if (index === 0 && animValue?.value > length - 1) {
inputRange = [length - 1, length, length + 1];
outputRange = [-width, 0, width];
}
return {
transform: [
{
translateX: interpolate(
animValue?.value,
inputRange,
outputRange,
Extrapolate.CLAMP,
),
},
],
};
}, [animValue, index, length]);
return (
<Animated.View style={[styles.animDot, animatedWidth]}>
<Animated.View style={[styles.animView, animStyle]} />
</Animated.View>
);
};

@dohooo can you please implement this.

@NadeemKhanFh you can check my PR https://github.com/dohooo/react-native-reanimated-carousel/pull/164/files I don't know why Dohooo (owner) don't accept it to this lib

See this example for more details: https://reanimated-carousel.dev/usage


Override the default headers

I am hoping you have tried this and have some tips.
I am using the variations end point passing in a file which requires headers of  'Content-Type': 'multipart/form-data' (and not 'application/json')
The problem is that heads are added within the await.request.go() method and application/json is set instead of the multipart/form-data.
 response = await request.go();
How can I override the headers used?
  Future<String> createDifferentVariationsWithOpenAI({required File inputImage, int n = 5}) async {
    final Request requestImage = _openAIClient.images.variation(image: inputImage,n: n);

    return _getImagesFromRequest(request: requestImage);
  }

 Future<String> _getImagesFromRequest({required Request request}) async {
    final Images image;
    late final Response response;
    try {
      response = await request.go();

      image = response.get();
    } on StateError catch (_) {
      return response.json['error']['message'];
    } catch (e) {
      return "error occurred: $e";
    }

    return (image)
        .data
        .map((e) async => e.url)
        .toList()[0];
  }

Hey @dyardy, thanks for opening first of all.
I didn't think about this fallback, I would say that we should pass a custom parameter to the request.go() method to override the header.
What's your take on this?

I would suggest that you include tests for getting images and variations and we can go from there.

I didn't think about this fallback, I would say that we should pass a custom parameter to the request.go() method to override the header.

Hey @dyardy, I tried to implement this solution, and it should be working.
You could try it by installing the package from this brand-new branch in this way:
dependencies:
  openai_client:
    git:
      url: git@github.com:Azzeccagarbugli/openai_client.git
      ref: 3-override-the-default-headers

Let me know if it's okay!


Cannot plot data after filtering

Describe the bug
I cannot plot the data again after applying filtering.
Steps to reproduce
	data_highpass = data.copy().filter(l_freq=0.5, h_freq=None)

	data.plot(n_channels=61, scalings="auto") # this works as expected
	plt.show()

	data_highpass.plot(n_channels=61, scalings="auto") # here I get the error
	plt.show()

Expected results
It should be able to plot data after filtering
Actual results
ValueError: cannot convert float NaN to integer
Additional information
Platform:      Linux-5.3.0-45-generic-x86_64-with-glibc2.10
Python:        3.8.1 (default, Jan  8 2020, 22:29:32)  [GCC 7.3.0]
Executable:    /home/andrea/anaconda3/envs/bci_competition/bin/python
CPU:           x86_64: 8 cores
Memory:        7.6 GB
mne:           0.19.2
numpy:         1.18.1 {blas=mkl_rt, lapack=mkl_rt}
scipy:         1.4.1
matplotlib:    3.2.0 {backend=TkAgg}
sklearn:       0.22.2.post1
numba:         Not found
nibabel:       Not found
cupy:          Not found
pandas:        1.0.1
dipy:          Not found
mayavi:        Not found
pyvista:       Not found
vtk:           Not found

it seems filtering creates NaN

@Andrea-V we can't tell from the last line of the traceback where the error is actually happening. Can you paste the full traceback?

here is the full error:
`
Traceback (most recent call last):
File "preprocessing_upper_limb.py", line 54, in
data_highpass.plot(n_channels=61, scalings="auto")
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/io/base.py", line 1562, in plot
return plot_raw(self, events, duration, start, n_channels, bgcolor,
File "</home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/externals/decorator.py:decorator-gen-132>", line 2, in plot_raw
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/utils/_logging.py", line 90, in wrapper
return function(*args, **kwargs)
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/viz/raw.py", line 458, in plot_raw
callback_proj('none')
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/viz/utils.py", line 334, in _toggle_proj
params['plot_update_proj_callback'](params, bools)
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/viz/raw.py", line 43, in _plot_update_raw_proj
params'plot_fun'
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/viz/raw.py", line 827, in _plot_raw_traces
'%s %s ' % (_simplify_float(inv_norm), units),
File "/home/andrea/anaconda3/envs/bci_competition/lib/python3.8/site-packages/mne/viz/utils.py", line 523, in _simplify_float
if isinstance(label, float) and float(str(label)) != round(label):
ValueError: cannot convert float NaN to integer
`

Can you see if https://github.com/mne-tools/mne-python/pull/7583 at least does not die?
Then we can figure out why filtering the data has caused nan to show up everywhere.

@Andrea-V if you can share the data via DropBox or other link, it would also help

You can find a data sample here:
https://drive.google.com/drive/folders/1DKs5xPE_DfFbCaIToBN9F3UYnxf5s9Hf?usp=sharing
This is the code I use for loading the data
data = mne.io.read_raw_gdf(os.path.join(raw_path, file), verbose=True)
data.load_data()

>>> np.isfinite(raw.get_data()).all()
False

So it appears as though your data already have NaN (filtering will just spread these). The fix might just be to fix the auto-scaling to account for nan values, I'll look.

is there a way to deal with NaN directly into the Raw object? It would be nice to have methods similar to pandas .dropna() or fillna() methods

Or else, a way to create the Raw object from a pandas DataFrame, so that I couls deal first with NaN in pandas, load the data into Raw object, and then apply some filtering

You can create Raw objects from NumPy arrays
https://mne.tools/stable/auto_examples/io/plot_objects_from_arrays.html#sphx-glr-auto-examples-io-plot-objects-from-arrays-py

It looks like this file is particularly problematic, as there are channels that need to be resampled, and those channels have NaN or inf in them. I've added plotting fixes to #7583, but really we need to deal with EDF resampling better somehow (replace with linear interpolated values or something?)


⚠ Bofrost.at has degraded performance

In cbd23b9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1742 ms

Resolved: Bofrost.at performance has improved in 17c5edd.


Add back button to connect bank account message page

@sobitneupane @arosiclair PR is ready for review.
Details
Whenever login with phone number as primary account, and go to workspace connect bank account at that time Connect back account page shows message as  "To add a verified bank account please ensure your primary login is a valid email and try again. You can add your phone number as a secondary login." and this page does not have Back button.
So in this pr we solve this problem and now it will show back button on connect bank account message page.
Fixed Issues
$ https://github.com/Expensify/App/issues/19708
PROPOSAL: https://github.com/Expensify/App/issues/19708#issuecomment-1565229357
Tests
Preconditions: Use account with phone number as primary login.

Open the app
Use account with phone number as primary login.
Go to Settings > Workspaces
Select any Workspace
Tap "Bank Account"
It will show "Connect bank account" page with message as "To add a verified bank account please ensure your primary login is a valid email and try again. You can add your phone number as a secondary login.".
Verify that "Connect bank account" page shows back button (previously there was no back button).

[x] Verify that no errors appear in the JS console

Offline tests
Preconditions: Use account with phone number as primary login.

Open the app
Use account with phone number as primary login.
Go to Settings > Workspaces
Select any Workspace
Tap "Bank Account"
It will show "Connect bank account" page with message as "To add a verified bank account please ensure your primary login is a valid email and try again. You can add your phone number as a secondary login.".
Verify that "Connect bank account" page shows back button (previously there was no back button).

QA Steps
Preconditions: Use account with phone number as primary login.

Open the app
Use account with phone number as primary login.
Go to Settings > Workspaces
Select any Workspace
Tap "Bank Account"
It will show "Connect bank account" page with message as "To add a verified bank account please ensure your primary login is a valid email and try again. You can add your phone number as a secondary login.".
Verify that "Connect bank account" page shows back button (previously there was no back button).

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Chrome
https://github.com/Expensify/App/assets/7823358/ff2cdb3b-fb78-4670-922f-e6eefcc2a743
Safari
https://github.com/Expensify/App/assets/7823358/0662a61f-f1e6-4232-ac2e-8462e975438f

Mobile Web - Chrome
https://github.com/Expensify/App/assets/7823358/f08003f5-0d7d-4e03-acfd-41ea2744f4e6

Mobile Web - Safari
https://github.com/Expensify/App/assets/7823358/6cac97be-f81c-46d1-bac2-cbbc3d0a0939

Desktop
https://github.com/Expensify/App/assets/7823358/4cfc7f4d-8f2d-4e9b-9159-cf53a810cafc

iOS
https://github.com/Expensify/App/assets/7823358/69d015bf-1d5d-4795-b06a-55b8cff4dfaa

Android
https://github.com/Expensify/App/assets/7823358/1064bb21-c00a-4a1a-981f-2ba641a44cc1

@sobitneupane @arosiclair    yes on main branch onBackButtonPress added here  and for HeaderWithBackButton.js it has shouldShowBackButton default prop true here So it solve the issue and not reproducible now.

Ah okay looks like those were changes from the navigation refactor which was just merged yesterday. I also confirmed it is already working on main. Closing this out


Check if array param is not null

Dialect
PostgreSQL
Failing SQL
select b.resourceid,
       b.datecreatedutc as basecreated,
       b.datedeletedutc as basedeleted
from productbasetable b
where b.datedeletedutc is null
  and case
          when :resourceIdList is not null
              then b.resourceid in :resourceIdList
          else true
    end;

Description
The problem occurs when I'm trying to use it in SqlDelight: the generated code has a problem with when :resourceIdList is not null (encode fun expects a T type, not a List)

the same problem if i rewrite sql statement to
AND (:resourceIdList IS NULL OR b.resourceId IN :resourceIdList) instead of
and case
          when :resourceIdList is not null
              then b.resourceid in :resourceIdList
          else true
    end;

so, the problem is that the library does not know that :resourceIdList is a list, while :resourceIdList IS NULL executing.

I was also able to reproduce the problem with sqlite

We can probably make this crash more elegantly but this shouldn't work because the type expected is an array parameter, SQLDelight would be replacing :resourceIdList is not null with () IS NOT NULL which would end up returning true and probably not be what you want. I think the correct thing here is to use a different variable as the flag like :useResourceIdList = TRUE


json files and names

After an extensive review of the documentation today, I realized that distinguishing JSON filenames is particularly challenging. This commit introduces additional clarity to address this issue.

Resolves: #

Summary
TODO

[ ] ...

Checklist

[ ] Code is properly formatted
[ ] All commits are signed
[ ] Tests included/updated or not needed
[ ] Documentation (manuals or wiki) has been updated or is not required

@matsuro-hadouken Thanks a lot for the PR! Could you please fix the DCO error, as per the contribution guidelines so that we can merge it?

@matsuro-hadouken Merged. Thanks a lot for your contribution! :pray:


Update Catch2 library to latest version

The current version of the Catch2 testing framework in our repository is v2.9.2. Recommend we update to the latest version, v2.11.0 which offers a better reporter and bench-marking support.
Whoever wants to take this on just follow these simple directions:

[ ] Create a new branch git checkout -b update_catch2
[ ] Download the latest release here
[ ] Replace the file catch.hpp in the third_party/ folder with the latest version
[ ] Build and run to ensure all tests still pass
[ ] Commit changes and push to GitHub
[ ] On the GitHub repo page, open a new pull request.
[ ] Let the automated tests pass and allow time for review before merging.

fixed in pull request update_catch2


Term defintion update : N-acetyllactosaminide alpha-2,3-sialyltransferase activity

GO:0008118    N-acetyllactosaminide alpha-2,3-sialyltransferase activity

New proposed definition (genus-differentia pattern, plus additional information if needed)
Should we specify that it can also sialylate glycolipids?

Reference(s)
PubMed:10206952

Additional information
As stated in the EC number and shown in PubMed:10206952, it sialylates glycoproteins but also glycolipids. It is stated in the parent term GO:0008373 sialyltransferase activity :"Catalysis of the transfer of sialic acid to an acceptor molecule, typically the terminal portions of the sialylated glycolipids (gangliosides) or to the N- or O-linked sugar chains of glycoproteins."

I realised that lipid glycosylation GO:0030259 is not part of glycolipid biosynthetic process GO:0009247 as protein glycosylation is for glycoprotein biosynthetic process.

Hi @lbreuza
Thanks for pointing that out. I will update the reaction to the same as RHEA, that doesn't specify proteins or lipids, and add the a comment based on the one in EC "The enzyme recognizes the sequence beta-D-galactosyl-(1->4)-N-acetyl- D-glucosaminyl (known as type 2 histo-blood group precursor disaccharide) in non-reducing termini of glycan moieties in glycoproteins and glycolipids." >> The substrate for this reaction is a beta-D-galactosyl-(1->4)-N-acetyl- D-glucosaminyl (known as type 2 histo-blood group precursor disaccharide) in non-reducing termini of glycan moieties in either glycoproteins and glycolipids.
Does that work?

Also, add RHEA:52316 (was only present as def xref)


Build failure: visidata

Steps To Reproduce
Steps to reproduce the behavior:

Using this as my Nix starter template for macOS - uses home-manager and nix-darwin.
Run nix flake update
Run nix run ./#build-switch, which causes some kind of build failure with visidata.

(Previously visitdata seemed to be working correctly - however, I recently ran a nix flake update - and it seems to have been broken for the last week or so).
Build log
❯ nix run ./#build-switch
warning: Git tree '/Users/foobar/nixos-config' is dirty
Running build-switch for aarch64-darwin
Starting build...
warning: Git tree '/Users/foobar/nixos-config' is dirty
error: builder for '/nix/store/x63ii5nwcjsymn6lja476456ilnz06sz-visidata-3.0.2.drv' failed with exit code 1;
       last 10 log lines:
       > index 3a5decf..5c68fcc 100644
       > --- a/tests/golden/edit-type.tsv
       > +++ b/tests/golden/edit-type.tsv
       > @@ -4,5 +4,5 @@ Customer     0       0       17
       >  SKU        0       0       34
       >  Item 0       0       40
       >  Quantity     0       0       10      1       144     403     2       7.90    21.57
       > -Unit    0       0       28      0.0     2495.99 11974.530000000006      12.95   234.79  585.85
       > +Unit   0       0       28      0.0     2495.99 11974.53        12.95   234.79  585.85
       >  Paid   0       0       36      -2300.0 3602.1  10886.21        51.8    213.46  819.74
       For full logs, run 'nix log /nix/store/x63ii5nwcjsymn6lja476456ilnz06sz-visidata-3.0.2.drv'.
error: 1 dependencies of derivation '/nix/store/xvkdbn9frapnpkdqd5r5dvgvmawg4kid-home-manager-applications.drv' failed to build
error: 1 dependencies of derivation '/nix/store/f876nb0z7z11lyd59aa95c8blk3a9ci9-home-manager-fonts.drv' failed to build
error: 1 dependencies of derivation '/nix/store/8d79rdgzvd3jdkw70g04aqdjhhhrdwcj-home-manager-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/5i47v39n5kyz4y0sxcqbk96dhnaf8g6z-man-paths.drv' failed to build
error: 1 dependencies of derivation '/nix/store/82w9pvr6h87y5rlzmafwyqqpx54xkz6v-system-applications.drv' failed to build
error (ignored): error: cannot unlink '/private/tmp/nix-build-swift-5.8.drv-4/src/llvm-project/clang/test/CodeGenCXX': Directory not empty
error: 1 dependencies of derivation '/nix/store/hyb9d89pppijms5c773ikivlwrnp66qj-visidata-3.0.2-fish-completions.drv' failed to build
error: 1 dependencies of derivation '/nix/store/6kfi2yd4dnji5kvj3h19scrx425mnpza-darwin-system-24.11.20240708.655a58a+darwin4.cf297a8.drv' failed to build

I've uploaded the build logs (nix log /nix/store/x63ii5nwcjsymn6lja476456ilnz06sz-visidata-3.0.2.drv > /tmp/visidata_nix_build_logs) as a gist here:
https://gist.github.com/victorhooi/95531bfe36848d45ed54cadb41269fb1
Additional context
Add any other context about the problem here.
Notify maintainers
@markus1189 @pacien

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
I get this error when I try to run this command:
❯ nix-shell -p nix-info --run "nix-info -m"
error:
       … while calling the 'import' builtin
         at «string»:1:18:
            1| {...}@args: with import <nixpkgs> args; (pkgs.runCommandCC or pkgs.runCommand) "shell" { buildInputs = [ (nix-info) ]; } ""
             |                  ^

       … while realising the context of a path

       … while calling the 'findFile' builtin
         at «string»:1:25:
            1| {...}@args: with import <nixpkgs> args; (pkgs.runCommandCC or pkgs.runCommand) "shell" { buildInputs = [ (nix-info) ]; } ""
             |                         ^

       error: file 'nixpkgs' was not found in the Nix search path (add it using $NIX_PATH or -I)

nixos-config on  main [!]

Add a :+1: reaction to issues you find important.

Looking at the build log, that is a failing test from visidata itself, where something is wrong with the number having a different value 11974.530000000006 vs 11974.53
So that should hopefully be fixed upstream

Well, that's what it looked like at first to my naive eyes (a failing test).
However, it didn't make sense because it's Visidata 3.0.2 - which was released back in January 2024. Also, if I look at the Nix visidata package - there's been no changes in 6 months.
And I could have sworn visidata was working for me last week or so, before I ran nix flake update since then...although maybe I'm mistaken...

Hi @victorhooi!
Do you, by any chance have options.disp_float_fmt set to anything?

That gives me the info to address this!

This is fixed upstream.

Thanks @anjakefala and @pacien! =).
Quick question for a Nix newbie - if I'm currently using nixpkgs-unstable (default on the templates I'm using) - but Visidata hasn't cut a new release yet with this fix just yet - and nixpkgs is using 3.0.2 (visidata default.nix) - is there an easy way to have it build from the visidata develop branch instead?
Or is it just a case of waiting for the next visidata release, and then also for the nixpkgs to get updated?


Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value

I'm using AzureData to connect to Cosmos DB. My first attempt at adding a document failed. I used create or replace method on DocumentCollection
Bug Information
Fatal Error in -->public enum ResourceLocation line 72
case let .child(type, resource, id):                            return ResourceOracle.getAltLink(forResource: resource)! + id.path(in:"/" + type.rawValue)
Steps to Reproduce
-->in code snippet
Expected Behavior
-->create document
Actual Behavior
Code snippet
func createNewEventWith(completion: @escaping (JSON, NSError?) -> Void) {
self.collectionId = "events"
if(AzureData.isConfigured()){
self.collection!.createOrReplace(self) { (response) in
if(response.result.isSuccess){
completion(JSON(response.resource!), nil)
return
}
debugPrint(response.error as Any)
}
}
}
Screenshots
$›

Need to pass a Document object into createOrReplace(). It appears that DocumentCollection object is being passed in above code sample instead.  Method signature is: createOrReplace(document: T, callback: (Response<T>) -> ())

Closing issue.  Please comment again here if you need further assistance and I'll reopen the issue.


Improved JS

Tidies up the JS source code against this.

We can add jshint to the project :) https://www.npmjs.com/package/laravel-elixir-jshint

We can add jshint to the project :) https://www.npmjs.com/package/laravel-elixir-jshint

Yeah, we should do that!

Yeah, we should do that!


Forward solve fails with LinearAlgebra.Adjoint initial values

Here's a quick toy with 1D state and batch dimension.
using Flux
using DiffEqFlux
using MLDataUtils
using Distributions
using OrdinaryDiffEq

D = 1
N = 10000
BS = 100

train_x = rand(Uniform(0,5),(D,N))
train_y = train_x .^ 2
batches = batchview((train_x,train_y),BS)

nn = Chain(Dense(1,10,tanh),Dense(10,10,tanh),Dense(10,1))
tspan = (0.0f0,1.0f0)
model = x-> neural_ode(nn,x,tspan,Tsit5(),atol=1e-6,rtol=1e-6)[:,:,end]

I thought I could just create a single batch element by transposing a singleton array. However LinearAlgebra.Adjoint breaks the forward solve in neural_ode:
julia> x0 = rand(1,1)
1×1 Array{Float64,2}:
 0.49820519072539593

julia> size(x0)
(1, 1)

julia> model(x0)
Tracked 1×1 Array{Float64,2}:
 0.3671767104193268

julia> x1 = rand(1)'
1×1 LinearAlgebra.Adjoint{Float64,Array{Float64,1}}:
 0.2720790231482093

julia> size(x1)
(1, 1)

julia> model(x1)
ERROR: MethodError: Cannot `convert` an object of type Array{Float32,2} to an object of type LinearAlgebra.Adjoint{Float64,Array{Float64,1}}
Closest candidates are:
  convert(::Type{LinearAlgebra.Adjoint{T,S}}, ::LinearAlgebra.Adjoint) where {T, S} at /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.3/LinearAlgebra/src/adjtrans.jl:186
  convert(::Type{T}, ::T) where T<:AbstractArray at abstractarray.jl:14
  convert(::Type{T}, ::LinearAlgebra.Factorization)

If I collect first it's fine:
julia> x2 = rand(1)'|>collect
1×1 Array{Float64,2}:
 0.22926764437395208

julia> size(x2)
(1, 1)

julia> model(x2)
Tracked 1×1 Array{Float64,2}:
 0.07379717434344342

This is surprising because LinearAlgebra.Adjoint plays well with normal solves:
u0=randn(1,1)
tspan = (0.0,1.0)
prob = ODEProblem(f,u0,tspan)
sol = solve(prob,Tsit5(),reltol=1e-8,abstol=1e-8)[:,end]
# > 1×1 Array{Float64,2}:
 1.211417535470834

u0=randn(1)'
tspan = (0.0,1.0)
prob = ODEProblem(f,u0,tspan)
sol = solve(prob,Tsit5(),reltol=1e-8,abstol=1e-8)[:,end]
# > 1×1 LinearAlgebra.Adjoint{Float64,Array{Float64,1}}:
 -1.1428152078709033

rand(1,1)' works though. Hmm....

x1 = rand(1)'
nn(x1)

gives a Matrix instead of an Adjoint, and so it's not the same type. DiffEq assumes that you're using the same array type.
nn = Chain(Dense(1,10,tanh),Dense(10,10,tanh),Dense(10,1),x->vec(x)')

works

You're right, that is where the issue is. Thanks @ChrisRackauckas


Allow customization of the full names for groups

Groups have different schemes for naming its full group names. This is bad but we have to live with it. This PR avoids hardcoding this conversion or guessing by exposing the full names for users to adapt.

/hold
until #181 is merged.

/hold cancel

#181 is in
/hold cancel

/approve

/lgtm


There was a problem during powerdns installation

@lishige
after the installation did you try to turn on the Run at start up option? Did it work?

@lishige
after the installation did you try to turn on the Run at start up option? Did it work?

@lishige
after the installation did you try to turn on the Run at start up option? Did it work?

I haven't installed it yet. This error occurred when I used the installation package to install. I originally installed it on the C drive. After I installed it on the D drive, I no longer have this error.

@lishige
after the installation did you try to turn on the Run at start up option? Did it work?

I haven't installed it yet. This error occurred when I used the installation package to install. I originally installed it on the C drive. After I installed it on the D drive, I no longer have this error.

Closing this issue for now, if you run again into this problem, please reopen it. Thanks.


Direct Debit Pay In

Hi
I'm trying to do a Direct Debit Pay In for a SEPA payment (I have the Mandate submited already)
I couldn't find an example for this kind of payment (At the Mangopay Docs Webpage the Code is unavaliable) https://docs.mangopay.com/endpoints/v2.01/payins#e282_create-a-direct-debit-direct-payin
I get this error:
MangoPay\Libraries\Exception Object (     [message:protected] => Payment is not defined or it is not object type     [string:Exception:private] =>      [code:protected] => 0     [file:protected] => D:\home\site\wwwroot\vendor\mangopay\php-sdk-v2\MangoPay\ApiPayIns.php     [line:protected] => 59     [trace:Exception:private] => Array

Hey @Daneel-Olivaw, could you please show us the relevant bit of your code so we can help identify the problem?

Quite similar to the classic Payin process. Here is an example below :
$payIn = new PayIn();
$payIn->Tag = $tag;
$payIn->CreditedWalletId = $creditedWalletId;
$payIn->AuthorId = $authorId;

$payIn->DebitedFunds = new Money();
$payIn->DebitedFunds->Amount = $amount;
$payIn->DebitedFunds->Currency = CurrencyIso::EUR;

$payIn->Fees = new Money();
$payIn->Fees->Amount = $fees;
$payIn->Fees->Currency = CurrencyIso::EUR;

$payIn->PaymentDetails = new PayInPaymentDetailsDirectDebit();
$payIn->PaymentDetails->MandateId = $mandateId;
$payIn->PaymentType = PayInPaymentType::DirectDebit;

$payIn->ExecutionType = PayInExecutionType::Direct;
$payIn->ExecutionDetails = new PayInExecutionDetailsDirect();

Thanks for your example @iammichiel !


cleanup(sources): remove deprecated syntax 'source:' (singular source)

This PR removes the deprecated syntax source: (singular source).
Depends on #588
Associated documentation PR: https://github.com/updatecli/website/pull/264
Test
To test this pull request, you can run the following commands:
make test

Additional Information
Tradeoff

Potential improvement

I'm waiting for review from 2 maintainers on this one (to be really sure that I did not do something wrong).
Proposed milestone: 0.22.3

Sorry, had to rebase due to merge conflicts. I need at least another new approval (putting this PR in auto-merge)


#466 Cassandra error handling

Thanks a lot, I was thinking that we may catch some exception higher in the hierarchy, like QueryExecutionException. What do you think?

Released as 4.22.1


wrappedOpener is deprecated.

If you would like your pane item with class ShowTodoView to support modified behavior, please implement a ::onDidChangeModified() method. If not, ignore this message. ::on methods for items are no longer supported.
wrappedOpener (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace.js:523:18)
Workspace.openURIInPane (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace.js:433:22)
Workspace.open (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace.js:366:19)
Object.show (/Users/rainer/.atom/packages/todo-show/lib/show-todo.coffee:75:19)

Closed by 4cd460f28584ffc2ee7ba33413ed49989dfef387.


Illegal instruction when importing tensorflow

After compiling tensorflow from source I get an "illegal instruction" error.
I traced it back to this error with gdb:

Thread 1 "python" received signal SIGILL, Illegal instruction.
0x00007fffdc684d16 in google::protobuf::SimpleDescriptorDatabase::DescriptorIndex<std::pair<void const*, int> >::AddFile(google::protobuf::FileDescriptorProto const&, std::pair<void const*, int>) ()
from /usr/local/lib/python2.7/dist-packages/tensorflow/python/_pywrap_tensorflow_internal.so

I can successfully install and import the latest nightly build (and I have been able to compile and import tensorflow from source in the past on the same machine). I tried uninstalling all the dependencies and let pip install them again when I install the tensorflow wheel to no avail. I also tried to install the protobuf 3.1 wheel as suggested in the documentation, as well as the latest 3.2 wheel.
Any suggestion is welcome.
Here the details of my system:
Operating System: Linux Ubuntu 16.04
CUDA and CuDNN version

ls -l /usr/local/cuda/lib64/libcud*
-rw-r--r-- 1 root root   556000 Jan 27 00:48 /usr/local/cuda/lib64/libcudadevrt.a
lrwxrwxrwx 1 root root       16 Jan 27 00:51 /usr/local/cuda/lib64/libcudart.so -> libcudart.so.8.0
lrwxrwxrwx 1 root root       19 Jan 27 00:51 /usr/local/cuda/lib64/libcudart.so.8.0 -> libcudart.so.8.0.61
-rw-r--r-- 1 root root   415432 Jan 27 00:48 /usr/local/cuda/lib64/libcudart.so.8.0.61
-rw-r--r-- 1 root root   775162 Jan 27 00:48 /usr/local/cuda/lib64/libcudart_static.a
lrwxrwxrwx 1 root root       13 Sep 24  2016 /usr/local/cuda/lib64/libcudnn.so -> libcudnn.so.5
lrwxrwxrwx 1 root root       17 Sep 24  2016 /usr/local/cuda/lib64/libcudnn.so.5 -> libcudnn.so.5.1.5
-rwxr-xr-x 1 root root 79337624 Sep 24  2016 /usr/local/cuda/lib64/libcudnn.so.5.1.5
-rw-r--r-- 1 root root 69756172 Sep 24  2016 /usr/local/cuda/lib64/libcudnn_static.a

Tensorflow commit: a9b7946540a27cc53bbdc9db1564196979fe30ae
Bazel version

Build label: 0.4.5
Build target: bazel-out/local-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Thu Mar 16 12:19:38 2017<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

If you are still able to install and use the nightly installs, then it would suggest some funky interaction with your system setup and the build. Perhaps you can isolate to a clean environment in a docker container (you could even use the same docker images that are used for the nightly builds (using tensorflow/tools/ci_build/builds/pip.sh gpu --test_tag_filters=local,-benchmark-test --distinct_host_configuration=false --config=cuda -c opt

It indeed seems to be a problem with my environment. Thank you for your answer, I will investigate further.

@fvisin Did you ever determine the cause of this? I'm experiencing the same problem.

In the end it was my fault: I had a set of extra flags that I mistakenly thought were right for my architecture. I was able to compile with the default settings, so I figured out the error was in my optimization flags.

Ended up figuring it out last night, after installing the pre-built cuda package worked. It was the same problem for me; I had the flag for SSE enabled on a machine that didn't support it.

after Upgrading to TensorFlow 1.6, I have this issue too. Programs which worked before stopped with
"fish: 'python3 <scriptname.py>' terminated by signal SIGILL (Illegale Instruktion)"
test.py has just a single line, in which it imports tensorflow library.

@leonard7e  Is it an older machine?  Perhaps the newer Tensorflow is compiled with more of the (newish) cpu SSE/parallelization flags.

Hi hware.
My computer supports up to sse4_2.
$> cat /proc/cpuinfo
gives ...
...
model name	: Intel(R) Celeron(R) CPU G3900TE @ 2.30GHz
...
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf tsc_known_freq pni pclmulqdq dtes64 monitor ds_cpl vmx est tm2 ssse3 sdbg cx16 xtpr pdcm pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave rdrand lahf_lm abm 3dnowprefetch cpuid_fault invpcid_single pti retpoline intel_pt rsb_ctxsw tpr_shadow vnmi flexpriority ept vpid fsgsbase tsc_adjust erms invpcid rdseed smap clflushopt xsaveopt xsavec xgetbv1 xsaves dtherm arat pln pts hwp hwp_notify hwp_act_window hwp_epp


Implement Rotation for Keypoints

Keypoints currently hold x, y, and z coordinates. This is enough data for us to display a rectangle representing the Keypoint, but it is not enough for displaying a 2D image of a necklace.
If the user rotates their camera left/right or has their shoulders at different heights, we want to rotate the Keypoint's 2D image. For a full range of motion, add rotationX, rotationY, and rotationZ instance variables to Keypoint along with related getters and setters.

Resolved in commit 1f67cfba68ec087a3af5d0461fbfa743454c46a5.


Migrate Jersey REST resources to Spring Web

Body
Although we moved to Spring Boot as base technology for all StreamPipes services, the REST interfaces are implemented in JAX-RS using Jersey. This causes conflicts with Spring and adds overhead.
For simplification reasons, we can refactor the REST interfaces to use Spring Web instead.
StreamPipes Committer
I acknowledge that I am a maintainer/committer of the Apache StreamPipes project.

Closed with #2277


Change from raw json.ld to py module

Related: https://github.com/HTTP-APIs/hydra-flock-vocab/issues/1 and https://github.com/HTTP-APIs/hydra-python-core/pull/29
Simply changing the format so we import:
from hydra-flock-vocab.hydra_flock import flock_doc

Probably we'll need to change literals true, false, and null to strings as it is a python dict now.

Also I noticed an old typo on line 87.

Can I simply replace it with the dict in hydrus?  https://github.com/HTTP-APIs/hydrus/blob/develop/hydrus/samples/hydra_doc_sample.py
I think changing the file format might be a problem for the demo, so if Akshay says so I'll just close this issue and add one more file so we have it on both formats.

Can I simply replace it with the dict in hydrus? https://github.com/HTTP-APIs/hydrus/blob/develop/hydrus/samples/hydra_doc_sample.py

Yeah you can.

I think changing the file format might be a problem for the demo, so if Akshay says so I'll just close this issue and add one more file so we have it on both formats.

👍


Add Who funds SV2 development section

Current supporters supporting devs are known and can be added.

https://Spiral.xyz
https://www.galaxy.com
http://foundrydigital.com
http://summerofbitcoin.org

https://www.figma.com/file/H42PrGnniKFtbfqHYBGevj/Stratum?node-id=0%3A1

Added logos.

PR is in #30

Completed with #33


Auto-completing permissions not working

Hello,
recently auto-completing permission has stopped working.
I don't know why, but i've seen some people address auto-complete problems before like you can't auto-complete users.
I am on the latest 1.13.1 build, bungeecord with luckperms server side and proxy side.

I’ve had several users complain that somewhere between the 4.2 and 4.3 builds it stopped working on 1.13.1.

I checked, it started on 4.3. exactly when you added the brigadier which is the only thing that shows up now.

Duplicate of #1257


Allow move_to_joint_positions() to be aborted by calling a test function

Rebasing of PR #58

Great! Thanks for the rebase. It LGTM... merging :)


Reset search with question mark value to the empty string

Hi, I would like to suggest a feature here. Currently I'm using react-router-dom for one of my projects, which of course uses history dependency under the hood. Both push and replace methods have two possible overloads and may be executed whether with path as string or location descriptor as object. Using push method with path works as expected and suppresses difference between f.e. /about and /about?, whereas push called with location descriptor differentiate search values such as ""(empty string) and ? which causes redundant update issues in environment where you f.e. execute data fetching on search change, so that location.search equals to ? after such navigate action. Wouldn't it be applicable to reset next search from ? to "" --even if ? was explicitly set-- directly in history library to prevent such a case?
For more detailed example you might have a look on this example with react-router-dom. Just click the navigations links and observe how fetch counter is increased.

I have an alternative parsePath implementation (based on Regex) lying around, which coincedentially has the desired behavior:
function parsePath(path) {
  const parts = path.match(/(\/?[^?#]*)(\?[^#?]+)?(\#[^#?]+)?/);
  if (parts && parts.length === 4) {
    return {
      pathname: parts[1] || "/",
      search: parts[2] || "",
      hash: parts[3] || ""
    };
  }
  return {
    pathname: "/",
    search: "",
    hash: ""
  };
}

I'll submit a PR later.

I'll get my hands on it soonish


Calling client.subscribe with a zero length list causes TimeoutError

Calling client.subscribe([]) should fail/return immediately, but it gets passed on to the paho client, which happily subscribes to no topics. The client.subscribe function then sets up a callback to wait for paho to fire an on_subscribe callback, but it never does, because there were no topics subscribed to.
Recreate with:
import asyncio

from asyncio_mqtt import Client

async def main():
    client = Client("test.mosquitto.org")

    print("Connecting...")
    await client.connect()
    print("Connected!")

    print("Subscribing...")
    await client.subscribe()
    print("Subscribed!")

    print("Disconnecting...")
    await client.disconnect()
    print("Disconnected!")

if __name__ == "__main__":
    loop = asyncio.get_event_loop()
    try:
        loop.run_until_complete(main())
    finally:
        loop.close()
        loop.stop()

The client.subscribe function for reference:
    async def subscribe(self, *args, timeout=10, **kwargs):
        result, mid = self._client.subscribe(*args, **kwargs)
        # Early out on error
        if result != mqtt.MQTT_ERR_SUCCESS:
            raise MqttCodeError(result, 'Could not subscribe to topic')
        # Create future for when the on_subscribe callback is called
        cb_result = asyncio.Future()
        with self._pending_call(mid, cb_result):
            # Wait for cb_result
            return await self._wait_for(cb_result, timeout=timeout)

As a side note, does this callback mechanism support multiple subscriptions? Does paho call back with once per subscribed topic, or once per invocation of self._client.subscribe()?

Nice find. I never tested this case. Reminds me that we also need test cases. :))

As a side note, does this callback mechanism support multiple subscriptions? Does paho call back with once per subscribed topic, or once per invocation of self._client.subscribe()?

I'm not sure that I completely follow. :)
When we call self._client.subscribe(), paho sends a SUBSCRIBE packet.
When paho receives a SUBACK packet from the server, it calls the on_subscribe callback.
There is also this interesting sentence from the MQTTv5 spec:

When the Server receives a SUBSCRIBE packet from a Client, the Server MUST respond with a SUBACK packet.

So if everything goes as planned (e.g., no network errors), we will receive exactly one on_subscribe call for every self._client.subscribe call that we make.
Does that answer your question?

Maybe I over-complicated that, sorry. :))
Yes, it's completely valid to call self._client_subscribe multiple times. Even with overlapping topics strings. :+1:


Rework galaxy map, move to 2d system space

Description
Rework the galaxy map to let the player manually move the "cursor".
Moves the system space to 2d to make it easier to visualize and control, particularly in the galaxy map.
Alternate Designs
N/A
Benefits
Better control for the user.
Possible Drawbacks
N/A
Applicable Issues
#82

Codecov Report

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

@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   97.27%   97.27%
=======================================
  Files           3        3
  Lines         147      147
=======================================
  Hits          143      143
  Misses          4        4

Continue to review full report at Codecov.

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


CLI option to silence the listing of nameservers

Can we get an option to silence the output of the nameservers list when no nameservers are specified on the command line? I have a list of about 500 and it fills up the screen, obscuring whatever else I was doing.

This can be controlled by setting the log verbosity.
On Thu, Aug 25, 2016 at 8:19 AM brandonpoc<EMAIL_ADDRESS>wrote:

Can we get an option to silence the output of the nameservers list when no
nameservers are specified on the command line? I have a list of about 500
and it fills up the screen, obscuring whatever else I was doing.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zmap/zdns/issues/62, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMSUEHkx46B2uog_hi2lsoJ3qXxs7x4ks5qjbJjgaJpZM4JtL7K
.

What must verbosity be set to? I tried setting to the lowest and it still showed the nameservers;
~:# zdns MX -verbosity 1
INFO[0000] no name servers specified. will use: <IP_ADDRESS>:53, <IP_ADDRESS>:53
Same with every -verbosity setting 1 - 5.

Any idea? I set verbosity from 1 to 5 and got the same output. I also tried "0" (even though that is not listed as a valid verbosity setting) and got the same result.

Hello,
Have you been able to resolve the verbosity question?

There's nothing using the 'Verbosity' option at all, as there's no SetLevel() being assigend for the logrus 'log' class. I fixed this.
So, I did a simple fix for the DNS server "will use: " output and others. I have it interpreting the Verbosity flag and assigning the proper log level from the logrus package. For instance, verbosity 1 is log level ErrorLevel; verbosity 5 is log level DebugLevel (the higehst, most verbose).  In this way, now the log.InfoMsg () etc work as expected with the proper verbosity. Also default verbosity has been changed from 3 to 4 (InfoLevel).
You can merge it into the source tree, I am not familiar with git and "forking pushing pulling pull requests" all that fodder (I'm from the age of CVS for version control/changes and shared development over NFS and centralized development boxes with "screen" ;D) , but the diff -Nru patch is attached.
Thanks,
Brandon
zdns.patch.txt

This was merged in 085964c2280001ececafb38e9a8f7db0f6ff2771.


Added sequence diagrams with RFC 9101

What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
Implement RFC 9101 for securing the /authorise endpoint.
Which issue(s) this PR fixes:

Fixes #93
Special notes for reviewers:
Please review the diagrams and provide your comments.
Changelog input
 release-note
Update diagram to reflect RFC 9101

Additional documentation
This section can be blank.
docs

Hello @ECORMAC
Any update on this?
I've noted to wait from feedback from you before to move forward.

Hi Ludovic,
My apologies for the dealy. Checked the last flow done by Ming (simplified showing the aggregator etc.). I think it looks good now. Good-to-go from my side.
Thanks,
Cormac

Hi, I've checked with my collegues from Identity and there are a few issues we are yet not sure about:

The request object according to RFC 9101 is not yet in the CAMARA ICM profile. It will probably be but, for now, is not yet approved,
The max_age param does not make sense in this flow since network auth must occur for every authorization. See previous comment
The way the scopes and purposes are sent is not the one appoved for the current version of ICM in CAMARA.
In general, aggregation or federetion should be out of the CAMARA scope. It is something defined in Opengateway, in CAMARA the API consumer could be the BE of an App or an aggregator, but CAMARA is not involve in how the routing should be or if there is or not a Telco Finder, etc. It is an entity defined in Opengateway.

The main thing here is that, even being a Telefónica's proposal, the use of RFC9101 hasn't been defined or agreed at CAMARA level yet, so I think we should at least wait for the ICM resolution and then continue with this topic.

I'm not sure how to move forward on this... @mhfoo @fernandopradocabrillo @ECORMAC @AxelNennker  any suggestion?

I'm not sure how to move forward on this... @mhfoo @fernandopradocabrillo @ECORMAC @AxelNennker any suggestion?

@bigludo7 The intention is to raise it again for Spring25 meta-release. see ICM issue #128

I'm not sure how to move forward on this... @mhfoo @fernandopradocabrillo @ECORMAC @AxelNennker any suggestion?

@bigludo7 The intention is to raise it again for Spring25 meta-release. see ICM issue #128
I will create the issue in ICM again, before 30 Sept and add to the following: ICM 0.3.0 - preparing the scope for meta-release Spring25

@bigludo7: As Ming states think we agreed earlier to deal with this in the Spring release.


Fix indexing when game is not hosted on The Forge

Moulinette-Core 10.2.2 breaks indexing when not hosted on The Forge. It throws a "ReferenceError: ForgeVTT is not defined" in moulinette-file-utils.js:346.
This PR fixes it by adding a check that ForgeVTT is defined before accessing the object, as is done in other places.

Thanks for reporting and fixing this.
I can't believe I didn't test this locally.


ARTEMIS-3918 support FQQN + anycast + redistribution

When a message is sent to an anycast queue via FQQN on one node of a
cluster and then a consumer is created on that same anycast queue via
FQQN on another node in the cluster the message is not redistributed to
the node with the consumer.
This commit fixes this use-case primarily by including the FQQN info in
the notification messages sent to other nodes in the cluster.

What is the expected behavior with multiple FQQN anycast queues per address?

Messages will routed round-robin to the anycast queues on an address.

Messages will routed round-robin to the anycast queues on an address.

Hey @jbertram, does this mean that a routed message can go from one anycast FQQN queue to another with a different queue name? For example, let's assume an address "test" with anycast queues "test-1" and "test-2", could it occur that a message is placed on "test::test-1" and then routed to "test::test-2"?

...does this mean that a routed message can go from one anycast FQQN queue to another with a different queue name?

No. A message sent to an FQQN goes only to that FQQN.
I was answering that question in the context of a message sent to an address with multiple anycast queues (i.e. not using FQQN).

No. A message sent to an FQQN goes only to that FQQN.

I believe I'm currently having this problem. I'm using addresses to group queues from different applications, so the address name is the name of the application and all the application queues are inside it.
Before 2.25.0, the redistribution in my symmetric  cluster did not work, I've updated it to 2.25.0 and it works, but I'm having messages delivered to a FQQN queue and redistributed to another FQQN queue that is in the same address.
I've looked at the code, and the test case you've created only uses an address with one queue, is it possible that with an address with multiple queues it will fail?

@rtista, this PR is already merged so there's not much else to be done here. If you've got a way to reproduce the problem then please open a Jira explaining your use-case and attach your reproducer. Thanks!


devkitpro repo download link does not exist

----------------------------
fast install devkitpro BEGIN
----------------------------
>> you use 1)amd64 or 2)arm64 arch
1
--2022-10-19 08:43:28--  https://github.com/devkitPro/pacman/releases/download/v1.0.2/devkitpro-pacman.amd64.deb
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2022-10-19 08:43:28 ERROR 404: Not Found.

dpkg: error: cannot access archive 'devkitpro-pacman.amd64.deb': No such file or directory
./install-devkitpro.bash: line 45: syntax error near unexpected token `fi'
./install-devkitpro.bash: line 45: `    fi'

thank for report it, i will update it.


Aadityalr123/xfn 242 configure ssl for jupyterhub

@James

https://linear.app/indieagi/issue/XFN-242/configure-ssl-for-jupyterhub
live deployed at https://jupyter-dev.indieagi.org/
important note:

the docker volume is not mounted, this was the case before, would lead to data loss on the container's restart
aaditya was having issues at the dev IP due to his wifi's firewall.


Added controller. hint() to trigger UI hint

Fixes #32

Version bumped to 0.5.0

Usage
FlipCardController _controller = FlipCardController();

 _controller.hint(
   // optional
   duration: Duration(milliseconds: 400),
   // optional
   total: Duration(milliseconds: 900),
 );

 FlipCard(
   direction: FlipDirection.HORIZONTAL,
   controller: _controller,
   onFlip: () {
     //
   },
   back: Center(
     child: Container(
       child: ProjectFrontWidget(
         project: widget.project,
       ),
     ),
   ),
   front: Container(
     color: Colors.red,
     child: Center(
       child: ProjectBackWidget(
         project: widget.project,
       ),
     ),
   ),
 );

The duration will only run if not flipped yet and not already animating.
Animation runs for duration and reverses afterwards. The total duration will be used for this animation only and resets afterwards.

Actually, don't merge this just yet, I'll clean up the syntax and expose the controller as well to allow more control from outside.

Even if GlobalKeys are possible, Controllers are preferred. The controller that's introduced by this PR also provides the ability to access widget state if required.
This can be merged if you want, it's stable now.

@fedeoo Could you please merge this?

0.5.1 solves a use case where the card is aligned to the top (which can't be changed from outside this package)

Merged in this PR. https://github.com/fedeoo/flip_card/pull/39
I'm closing this one. Thanks for your work.
I hope @BrunoJurkovic  or you can help to maintain this repo as I am not working with flutter now.


Add troubleshooting to custom_osm.md

I found more ways that could lead to a broken OSM file. I added a quick guide on how to fix it.
I wanted to combine my road segments into one long road (to have less splines in the end) but I noticed that this also leads to a broken file, because one of those two road parts gets extended but the other one gets marked as "delete".
There are probably more actions that lead to deleted elements. We can document them here and add workarounds or options how to avoid them.

@H4rdB4se, hey!
Thank you for contributing!
Merged


Unable to handle data returned from smart contract due to BigUInt cast problem

What happened?
When calling a smart contract, in my case ERC20 with the allowance method, the data is returned as an __SwiftValue
Printing Type(of: result["0"]) outputs "__SwiftValue"
And when printing the actual result, the output is as I expect. The problem occurs when I attempt to perform logic on the value, e.g if result["0"] > 0, I begin to get type errors.
After following this post on StackOverflow, I did the following to cast the value to BigInt:
let value = result["0"] as! BigUInt
This causes an error at run time:
Could not cast value of type 'BigInt.BigUInt' (0x104da0790) to 'BigInt.BigUInt' (0x1036d8d48).
Which for me is ambiguous. I am not sure how to proceed, and having looked through my dependencies, can only find one instance of BigInt, but it seems that it is referring to another.
I cannot operate with the result as a __SwiftValue, are there any ways around this?
  `let path = Bundle.main.path(forResource: "erc20", ofType: "json")!
    let ABI = try String(contentsOfFile: path, encoding: String.Encoding.utf8)

    let contractAddress = EthereumAddress(tokenAddress)
    let contract = web3?.contract(ABI, at: contractAddress)

    var options = TransactionOptions.defaultOptions
    options.from = EthereumAddress(userAddress)
    options.gasLimit = .automatic
    options.gasPrice = .automatic

    let userAddressETH = EthereumAddress(userAddress)!
    let approvalAddressETH = EthereumAddress(approvalAddress)!

    let parameters = [userAddressETH, approvalAddressETH] as [AnyObject]

    let txn = contract?.method("allowance", parameters: parameters, transactionOptions: options)
    let result = try txn!.call()
    let value = result["0"] as! BigUInt //Error thrown here at runtime

    print(type(of: result["0"]!))`

What are the steps to reproduce?
Step 1: Call a smart contract and have a result
Step 2: Cast the result to BigUInt
What is the expected behavior?
The value is casted and I would be able to perform logic/operate on it.
What is the error thrown?
Could not cast value of type 'BigInt.BigUInt' (0x104da0790) to 'BigInt.BigUInt' (0x1036d8d48).
What's the stack trace said?
2022-12-02 09:57:45.759976+0000 atm-connect-ios[30760:2962543] Could not cast value of type 'BigInt.BigUInt' (0x104da0790) to 'BigInt.BigUInt' (0x1036d8d48).
OS version
macOS Monterey Version 12.6
Library version
2.6.5

Oh, it's hard to say, but all 2.*.* branch are deprecated. Now we're supporting only 3.*.* version even for bugfixin.
Anyway, at a first glance it looks like there's a two same named type are conflicting with each other. I have remember that something like this one appeared because of some of our dependency update that have been expanding its internal BigInt type into the public. I suggest you to dig in that way if you're stuck with 2.6.5 version. But believe me, 3.0.0 is a much-much better.

Updated the library to 3.0.0, will investigate from here.
Not sure why cocoapods was pulling 2.6.5, very annoying!
Thanks for the push in the right direction

@adeneels this is a wired one, the minimum available version on Cocoapods from within 3.*.* should be 3.0.4 or something like that, because of the Cocoapods dependency resolving problems we're released it several patches away. I mean there is no way that you've updated up to exact 3.0.0 version.
But in case you're updated to a current one and the problem reappears please reopen this issue -- we will investigate it.

Yeah, I realised the cocoapods issue after investigating more. Have now moved to SwiftPM and am using version 3.0.6 well.
If I encounter the same problem I will let you know but I am feeling positive!


lb: small features

In gitlab by @rdallman on Jul 13, 2017, 11:48

[ ] re-route requests that error to a different backend. this may require buffering bodies which isn't great, we need to take care that an attacker couldn't easily OOM us; but it shouldn't add much response latency [in normal path]. for now, probably only re-route guys that didn't run.
[ ] enable SSL / h2. this is just simple plumbing since go does this out of the box.
[x] investigate adding k8's backend [in addition to db's]
[ ] integrate with a real stats backend (statsd/prom), get charts from that
[ ] ping/version endpoints

i guess it would also be nice to list the [not missing] features somewhere, too. anyone is welcome to add, I think we may be able to finagle out of trying to integrate with vulcand/traefik and have something we can control that is pretty light (thanks to Go, mostly)

Still want this open?


Quickfix for interface method stubs

I find this a very useful feature, mostly for compiled languages like java or scala.
Let's say you have an interface:
interface IFoo {
  foo(): number
  bar(): string
}

Now when you start writing an implementation, and you have this:
class FooImpl implements IFoo {
}

It would be really helpful to have a quickfix, that is able to generate for you something like this:
class FooImpl implements IFoo {
  foo(): number {
    return undefined;
  }
  bar(): string {
    return undefined;
  }
}

Because otherwise you have to go to the interface, copy-paste the methods, ... I don't have to explain the process.
Do you consider implementing this? If not, would you consider accepting a PR if I were to implement it?

Would absolutely accept a PR.  Thank you.

@nycdotnet , was there any traction on this feature add request?

I haven't seen any, but would still love a PR for it.

I started looking at how this could be done, when I found this: https://github.com/TypeStrong/atom-typescript/blob/master/lib/main/lang/fixmyts/quickFixes/implementInterface.ts (which is I guess what this issue is about), but unfortunately I didn't go any further and forgot about it :cry:

Is there any plan to be implemented? As long as I see the code of quick fix, the quick fix code wouldn't do anything.  The quickfix just shows implement interface suggestion.
I the quickfix should be removed if the feature is not implemented. I thought that was bug first time but actually that is only not implemented.

I've implemented this today locally, using the TypeScript Language Service getCodeFixesAtPosition API, which seems to do exactly what this Quickfix is supposed to do.
Any thoughts as to why the LanguageService API wasn't used before or why it doesn't seem to be used for any of the other Quickfixes (afaics from a quick look through the code)
The only problem I have is that the formatting is off. Anyone knows how to fix the formatting? I would've thought the LangaugeService would return me the code fixes properly formatted, but that doesn't seem to be the case.
If I can get this formatting resolved, I can make a Pull Request out of this

Quickfixes were only recently added to the Typescript API while atom-typescript has had them for a while. It's a similar story with a few other features like glob support in tsconfig.json. Now that the compiler has these, it doesn't make sense to use the custom versions. There's a rewrite of the plugin to only use official APIs in #1166 and I intend to deprecate the current master once that beta gets some testing in the trenches.
One thing it is currently lacking is Codefix support and I would be extremely happy if some enterprising soul would add that.


[BUG]: Video does not fit the height and width of parent

Version
v6 (Beta)
What platforms are you having the problem on?
IOS and Android
Architecture
Old architecture
What happened?
The video does not scale to the height and width of the parent when the resolution is less than the parent's width/height.
Video is not stretching to fit the parent width.
When I'm streaming HLS video with different resolutions, selecting a resolution with a height of 180 causes the video to shrink and not take up the width of the parent. In the attached screenshot, the background loads the first frame and keeps flickering, while the actual video plays on only a quarter of the screen.

Reproduction
repository link
Reproduction
When streaming HLS/DASH video choose video height less than viewport height/width

Please give more details, the sample in this repo works fine

This is an emulator issue. I also reproduce it with exoplayer sample app.
Let's close this ticket


guide to install that sdk package through composer

I am not understand how to install it through composer. I have searched but not got understand. Please Can you give me the step by step instruction to install that package through composer to run in my xampp.

Maybe this can help: How to Install composer on Windows with XAMPP


about the spatial embedding file

hello，i want to know where i can get the spatial embedding file(suffix is npy)？parser.add_argument('--SE_file', default='**.npy',                     help='spatial emebdding file')

You can use graph embedding algorithms such as node2vec or LINE for the adjacency matrix in the dataset to get the spatial embedding file, the spatial embedding file format is $Nodes\times Features$.


2888 update walletconnect sign v2 sdk

For production release, need create WalletConnect project ID and put it into keys.secret with name walletConnectProjectId. @JamesSmartCell

For production release, need create WalletConnect project ID and put it into keys.secret with name walletConnectProjectId. @JamesSmartCell

Ok, send me the id (via session) and I'll add it in.

For production release, need create WalletConnect project ID and put it into keys.secret with name walletConnectProjectId. @JamesSmartCell

Ok, send me the id (via session) and I'll add it in.

I have one with my personal account. Can you register one with AW official account via https://cloud.walletconnect.com/sign-in


Rename to tilelive

We should rename this repo and remove the .js, which is no longer relevant. cc @yhahn

:+1:

:+1:

Tom did this


MixedRealityTextMeshPro Sprite shader

Describe the problem
TextMeshPro uses TextMeshPro/Sprite shader to render any custom sprites that you author. This shader does not support single pass instanced rendering nor clipping primitites. So it makes TMP sprites not usable with MRTK.
Describe the solution you'd like
Provide MRTK specific shader for use with TextMeshPro Sprite rendering.
Describe alternatives you've considered
Manually adjust the shader but it does not work and feels hacky. Proper MRTK solution would be better.
Additional context
Link to desired TMP functionality:
http://digitalnativestudios.com/textmeshpro/docs/rich-text/#sprite

Hello @mgrman, Thanks for bringing TMP Sprite Shader issue this to our attention! Can you give us a little bit more information about your Unity and MRTK setup? This will help give a starting point for us to reproduce this issue as you saw it. Did you see any warnings or errors in the Unity console when you were trying to initialize custom sprites?

Hi @elbuhofantasma,
the use case we have, is to use ClippingBox to clip text as you scroll.
For text we use TextMeshPro.
The problem is that even if you use ClippingBox and set both the TMP renderer and the sprite submesh renderer, the sprite is not clipped.

This sample was created using standard TMP and MRTK, the sprites are EmojiOne which are included in TMP.
If you need any more info to reproduce this, please let me know.

Hello @mgrman Thank you for the detailed information! We will investigate and update this issue when the investigation is finished.

The team and I looked at this issue earlier this morning, and determined that this is a Unity issue. So I marked it as an external bug and will follow up as we get more information from Unity. I am going to keep this issue open in the meantime.

@elbuhofantasma Unity bug? MRTK provides own variant of the main Text Mesh Pro shader for use with clipping box, but does not provide a variant of Text Mesh Pro Sprite shader, which makes sprites in Text Mesh Pro not usable with MRTK Clipping Box.
To me it seems on MRTK to provide a variant of the shader for use with it's own features. Since Unity cannot provide a shader for feature only MRTK has.

Thank you @mgrman for reporting this! I agree this looks like a new feature request/bug we should support. In the MRTK we should fork the TMP sprite shader and add the relevant single pass instanced rendering support and clipping primitive support.

Hello @mgrman, I have written the shader that you have requested in PR #10096. This PR fixes the issue of sprites not clipping with the ClippingBox and not running on devices when Single Pass Instanced Rendering is enabled. I will close this issue when the PR is merged. Thank you again for your feature request!
Here is this shader in action in the editor: