Review Field Boundary GNSS Quality Information

@crakerb-ship-it  and the Field Boundary GNSS Working group have prepared the attached.  We should review vs. the current state of Field Boundary (https://adaptstandard.org/docs/field-boundary/) and GNSSSource (https://adaptstandard.org/docs/gnss-source/)
Initial Field Boundary GNSS Attributes.pdf

The rest of the team can chime in too, but I think what you suggest with regard to ADAPT is accurate. The list the WG put together was based on what someone receiving a boundary would want to know to make an informed decision on if it was captured with a sufficient level of accuracy for their purposes. The idea being the needs of an autonomous vehicle are likely different than someone clipping a background soil type map. WGS84 makes sense to align with the rest of ADAPT, the reason we had for including was just to know if was different so it could be handled/converted properly. But in the context of a transfer format like ADAPT I agree it would make sense to specify what will be used and not leave it open to interpretation.
Some of the finer points about GNSS source will likely be the part to discuss in greater detail. We had a lot of discussion about knowing what correction source, down to the base station ID was used during definition. The goal being to ensure when a boundary goes from one system to another its true location on the ground stays the same regardless of correction source the machine is using, assuming they are of comparable accuracy.

Per discussion in the 22 May 2024 meeting, we will remove GNSSSource from the beta schema at this time with goal of coming back to it and implementing it more fully and correctly.

Just to ensure that you understand that WGS84 do have an accuracy cost, varying over the globe according to local gravity.
To my knowledge the accuracy cost is not important for operative agriculture.
Hälsningar
Andreas Oxenstierna
Dalen Hörbyvägen 53
243 94 Höör
0730-26 97 12
On 22 May 2024, 17:51 +0200, Kelly Nelson @.***>, wrote:

Per discussion in the 22 May 2024 meeting, we will remove GNSSSource from the beta schema at this time with goal of coming back to it and implementing it more fully and correctly.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@Andreasox Yes, we discussed the issues with using WGS84 globally.  It was also our collective experience that any accuracy issues are not substantive for ADAPT's use cases.


Email Notification SOP- "Built for you: New in Reportstream"

Problem statement
As a ReportStream organization, I want to establish a standard operation procedure on when and how these emails will be sent out.
What you need to know
This will be an exploratory ticket conducted by Experience Team Content.
PROPOSED RELEASE NOTES 2.0 PROCESS:

Engineers make two changes to existing release notes operations: 1) link PR with original ticket that has user outcome(s) described; 2) Write PR title as [original ticket number] + [title]. Detailed proposal here. TLDR one-pager here.
Product managers: create user-friendly content, day after every sprint
Content/Experience Team: ensures content from Product is in plain language and adheres to ReportStream content style guidelines
Operations send out the email after every Major release / pre-determined frequency

Example 1 and Example 2
CONSIDERATIONS:

[ ] Who will get these email notifications?
[ ] How will the contents be added to the email?
[ ] What will be considered Major release?
[ ] Or what will be the pre-determined frequency?
[ ] What will be the protocol and time-frames for creating the content/reviewing the content/sharing the contents with the O&O team, sending out the email and storing them?

Acceptance criteria
DRAFT OF CONTENT STRATEGY:

[x] Clear and easy to follow DRAFT SOP for sending out these email notifications
[x] Establishes roles and responsibilities for each task
[x] Establishes timeframes for each task
[x] Consults O&O (SME = Adrian B.) to consider current and ideal future state outreach process

Content update 5/12/2022: This ticket can be worked on simultaneously with Content activities in Issue # 5382. Pulling into Sprint Backlog for "Content" spec work that can be done this sprint.

Content update 5/23/2022: Content work should be done by EOD today. Can have ready for review for Design Sync tomorrow 5/24. cc: @a-chappelear @RakshindaAslam

Updated 5/23/2022: See DRAFT on SharePoint. SharePoint saves the format it in a wonky way. For best view, go to link and download the file to your desktop. cc: @a-chappelear @RakshindaAslam


chore: update repo

update version to 2.2.4-1
Issue: https://github.com/linuxdeepin/developer-center/issues/5731
Log: update repo

/approve


question: Is there a tool for adding more users to the local backend?

Describe the issue
As an administrator I wish to add more users in the local_backend users.json database.
My question is: Is there is a tool or web-page to do this (as an admin)?
I know of the /auth/register endpoint, but this seems to primarily be meant for users to register themselves, and then the administrator has to acknowledge the registration by moving the entries from the registration json file to the users json file manually using a text-editor, then restarting caddy for the changes to take effect. Is this correct?
The way to add new users I have found is to simply copy-paste the existing entry and change it.
https://authp.github.io/docs/authenticate/local/identity-store
Configuration
N/A
Version Information
Provide output of caddy list-modules -versions | grep auth below:
/app # caddy list-modules -versions | grep auth
http.authentication.hashes.bcrypt v2.4.5
http.authentication.hashes.scrypt v2.4.5
http.authentication.providers.http_basic v2.4.5
http.handlers.authentication v2.4.5
http.authentication.providers.authorize v1.3.19
http.handlers.authp v1.4.27
/app #

Expected behavior
N/A
Additional context
Building caddy manually with auth-portal and authorize plugins, inspired by the Dockerfile in caddy-auth-portal repo

My question is: Is there is a tool or web-page to do this (as an admin)?

@andoks , I am working on it. It should come in the next couple weeks.

@greenpau thank you, that will be great! And thank you for your work so far too, auth-portal+authorize was a huge improvement over caddy's built-in basic auth.

@andoks , please reopen in github.com/greenpau/caddy-security if this is still relevant for you. Please consider sponsoring the work on the project! 👍
With the migration to caddy-security, we are now much closer to achieve this.


Method to get marketvalue history

Hi, is there an option to get the marketvalue history for a player?
I could open the pages for every player and scrape the polypoints of the chart. Is there an easier option with your api?

Hey there, not yet. The endpoint for this is https://kevinskyba.github.io/kickbase-api-doc/index.html#request-player-player-stats. There you can get all market values for the past year for each day. If you want feel free to add this endpoint and model to the library yourself and do a pull request. :)

I think this is partly done in #1. So I extend that:
pull request code player_stat


Updated get-claim-summary query to check reference Id along with clai…

Issue #145 - Added code to parse reference ID to be used in query.  Returns an error if reference ID and claim ID do not match
Checklist

[x] Unit tests
[x] End-to-End tests
[x] Code coverage checked
[x] Coding standards
[x] Error Handling
[ ] Security/performance considered?
[ ] Deployment changes considered?
[ ] README updated

…m Id

Closing to sort out conflicts and update for new logic


Some code written in strict mode is invalid in non-strict mode

TypeScript Version:  2.4.0
Code
Consider the following snippet:
const EMPTY_ARRAY = [];

Typescript infers EMPTY_ARRAY to be any[] and doesn't complain even with noImplicitAny turned on. Things get weirder, when we start using this variable:
const EMPTY_ARRAY = []; // Variable 'EMPTY_ARRAY' implicitly has type 'any[]' in some locations where its type cannot be determined.

const foo = () => EMPTY_ARRAY; // Variable 'EMPTY_ARRAY' implicitly has an 'any[]' type.

We get no implicit any error in 2 places. A questionable attempt at fixing this might be to type EMPTY_ARRAY as never[] or ReadonlyArray<never> which results in no errors:
const EMPTY_ARRAY: ReadonlyArray<never> = [];

const foo = () => EMPTY_ARRAY;

Questionability of this 'fix' aside, one would expect that turning off strictNullChecks would result in the code continuing to compile, but it's not the case:
// --strictNullChecks false
const EMPTY_ARRAY: ReadonlyArray<never> = [];
/** Type 'undefined[]' is not assignable to type 'ReadonlyArray<never>'.
  Types of property 'concat' are incompatible.
    Type '{ (...items: undefined[][]): undefined[]; (...items: undefined[][]): undefined[]; }' is not assignable to type '{ <U extends ReadonlyArray<never>>(...items: U[]): never[]; (...items: never[][]): never[]; (...i...'.
      Type 'undefined[]' is not assignable to type 'never[]'.
        Type 'undefined' is not assignable to type 'never'. */

const foo = () => EMPTY_ARRAY;

Expected behavior:
Not sure how this should be fixed tbh. To me it seems like never[] is not a type that should be allowed in either mode, since mutating the array will certainly break this. ReadonlyArray<never> looks like a valid option. On top of this, I'd expect that making the compiler less strict will keep the strict code compiling.
Actual behavior:
Described above.

Seems like the ergonomics of using [] in several combinations are difficult. #16651 is related.

This is working as intended. See #11432 and #13779 for context. In particular, note that in --noImplicitAny mode we perform control flow analysis to determine the type of variables initialized with [].
I'm perfectly happy to consider alternate proposals, but the currently behavior is a careful balance of performing better analysis when we can in --noImplicitAny mode while not breaking code when not in that mode.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


[FLINK-15163][1.10][build] [FLINK-15163][build] Set japicmp.referenceVersion to 1.9.1

What is the purpose of the change
This sets japicmp.referenceVersion to 1.9.1
Brief change log

See commits

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

Wrong


No controller to clear the fields from outside the widget

Once the widget is created with default value, then the only way to change value is to open the dialog and select a new value. The value shall be able to be controlled in the place where it is used

We added the function and proceeded with the 2.0.4 update.


Fix extended master secret

Checklist

[x] CLA is signed

Description of change

Commit 94ed2c6 dropped a ! operator by mistake, which causes extended
master secret connections to fail. This puts in back.

Why isn't there any test that catches this?

Because we inter-operate with ourselves just fine. It negotiates EXTMS and then both client and server just fail to actually use it.

So there is no test that checks that it's being used?

There are tests to check it is being negotiated - which it is. I'm not sure how you could check if its actually being used without using a third party implementation.

Would you mind using external servers? In CPython's SSL test suite some test cases involve servers on the Internet. Those tests are disabled by default and can be toggled with CLI switches

@yan12125, possibly, although we also probably wouldn't want that on by default. Perhaps some kind of nightly job might be ok.
Anyway I've pushed this PR now. Closing.

@mattcaswell Shouldn't the BoringSSL tests catch this?

Ah, yes good point. They probably do, but we've not switched them on in Travis yet.

See https://github.com/openssl/openssl/pull/1888! :)


feat: registry tags in action input

Support passing in tags as an input to sdk-generation-action as discussed in this RFC
Here's an example of a few different tag input formats that we can simultaneously support. These are all tested and they work.

Most readable
Single Line
Advanced Usecases - this would encompass things like conditional tagging based on github variables. Unfortunately, github actions does not support the single line if statement format, that would be super ideal.

Overall, this approach provides a good amount of flexibility to the github action user. We will of course have support documentation on our recommended ways to setup different types of tagging.
Notes:

Right now this is just printing. I'm currently working on implementing tagging as an argument speakeasy run as we speak.
Let's have a bit of debate on how we feel about naming of registry-tags vs build-tags. I kind of like registry-tags in the action because its more directly aligned with our concept of speakeasy registry as a product. Certainly would like to hear other opinions.

👎 - roopak


hi so I just want to know does my router have to be portforwarded I simple yes or no would work dont close me request

please dont just close it

Yes, most if not all routers use NAT nowadays, so you need port forwarding.


Possible to use variables in bosh-init yml

In the past, we define variables at the top of yml (embraced with <%...%>) when deploying micro-bosh with bosh-cli and micro-plugin. But we found it can't work with bosh-init, is there a way to use variables in yml to deploy a micro bosh with bosh-init? Thanks.

Not currently possible with bosh-init. Not sure if it's a planned feature or anything.
As an alternative, you might be able to pre-parse it with something like erb bosh-init.yml.erb > bosh-init.yml before running your bosh-init deploy bosh-init.yml.

@mattcui it's not planned to add ERB rendering of manifests. we are planning to add {{..}} concourse/fly style interpolation soon.

did this get implemented?

@DonBower yes, please see this doc for the new BOSH CLI.

@dpb587-pivotal The Document you refer to is for Bosh CLI V2, not BOSH-INIT.

@DonBower bosh create-env replaces bosh-init create and has support for variables. This bosh-init repository is no longer maintained in favor of that.


add MCP server badge

This PR adds a badge for the Apple Shortcuts Server listing in Glama MCP server directory.
https://glama.ai/mcp/servers/15z6abk6p2
Glama performs regular codebase and documentation scans to:

Confirm that the MCP server is working as expected
Confirm that there are no obvious security issues with dependencies of the server
Extract server characteristics such as tools, resources, prompts, and required parameters.

This badge helps your users to quickly asses that the MCP server is safe, server capabilities, and instructions for installing the server.

Thank you!

Yes - I'm on discord! Thank you @punkpeye. User name is xiaoyi1968

Don't see you on the server, but once you join, ping me and I will add it.


Not able to use it ,Failed to resolve: com.github.DmytroShuba:DailyTags:1.0.0

i have added  implementation "com.github.DmytroShuba:DailyTags:1.0.0" in app level build.gradle
and
buildscript {     repositories {         maven { url 'https://jitpack.io' }     }     ext {         compose_version = '1.0.1'     } }// Top-level build file where you can add configuration options common to all sub-projects/modules.
this in project level build.gradle file

Hi, thank you for your request!
Looks like you added the dependencies and repository correctly. However, your repositories list should also have the default repositories such as google() and mavenCentral(). It should look like this:
    repositories {
        google()
        mavenCentral()
        maven { url 'https://jitpack.io' }
    }


Resize grapples not updated after undo/redo

Move a node inside a compound and then select that compound. Press undo and then redo. After redo operation grapples are not refreshed. See the image below.

This particular issue is solved, but the same problem is happening when moving a node in and out of a compound and undo/redo that. But this issue was already there I think (at least on the master branch of newt).
It is linked to a deeper problem. When restoring elements (after an undo/redo for example), no event is triggered. Add/remove events are part of how we detect parent change. But when restoring it gets impossible to react to that. Maybe we should trigger an 'add' event when restoring an element. Or maybe there should be a changeParent event triggered somewhere.
Because in the end, changing an element's parent isn't the same as just removing and adding a node.

@royludo Looks like not that deeper after all. Listening for undo/redo events and updating drawing is enough.

@royludo can you verify and close the issue? Thanks.

Ha, I didn't think about that. Thanks.

There is still some issue in fact.
Select a compound. Move one child outside of it. Undo. Then move that same child out again. Grapples are not updated.


Raise CommandError when failing to remove the sqlite database file

Proposed fix for issue #789: "reset_db: Raise CommandError when failing to delete sqlite database file".

Seems like a good idea...
Could you add an option to '--fail-silently', so people can have the old behavior if they really want/need too ?
Thanks


fixed example code in async/README.md

Looks there may have been some problems in the README.md example for lapin-async. I got it working with these changes. connection.rs in the examples folder also does not compile for me. This PR only fixes the readme.
Thanks for all the hard work put into Lapin. I think the async implementation is the most elegant AMQP library I've had the pleasure of using.

Coverage remained the same at 60.496% when pulling 4f99c83d0f721aeea82e32fe8b0b41a21c50400a on thavlik:master into aa2bbeb7f95f6de1ab7518d716b99ba4bcb3358c on sozu-proxy:master.

Hello,
Thanks ! The code in master has changed quite a lot though in the past months so it won't work anymore


Cannot make DFP Interstitial working

I'm getting an onAdFailLoad event when trying to display an interstitial from DFP: {"isTrusted":false,"adNetwork":"AdMob","adType":"interstitial","adEvent":"onAdFailLoad","error":3,"reason":"No fill"}
Here is how I initiate the call:
	console.log("initDfpAd " + adId);
	var _g = 0;
	var _g1 = ["onAdLoaded","onAdFailLoad","onAdPresent","onAdDismiss","onAdLeaveApp","onBannerFailedToReceive","onBannerReceive","onBannerPresent","onBannerLeaveApp","onBannerDismiss","onInterstitialFailedToReceive","onInterstitialReceive","onInterstitialPresent","onInterstitialLeaveApp","onInterstitialDismiss"];
	while(_g < _g1.length) {
		var e = [_g1[_g]];
		++_g;
		window.document.addEventListener(e[0],(function(e) {
			return function(data) {
				console.log("AdMob event " + e[0] + ": " + JSON.stringify(data));
			};
		})(e));
	}
	AdMob.prepareInterstitial({ adId : adId, autoShow : false},function() {
		onSuccess();
	},function() {
		onError({ errorCode : "APP", debug : { err : "AdMob.prepareInterstitial failed", data : adId}});
	});

In my case, adId == "/1626606/app_nm_new".
I've tried with isTesting set to true and it works but it's not the interstitial I want to display of course...

My fault, the Interstitial size was tablet only while I was testing on smartphone...


Cycle detection doesn't work well with fibers

Cycle detection is broken when using fibers, because it assumes that if the function is entered multiple times, it's a cycle. With fibers, the function might be entered multiple times in different fibers if one of them suspends somewhere within the function.
What are your thoughts to moving cycle detection to an interface? There could be a NullCycleDetector that doesn't detect cycles at all, a CountingCycleDetector and then a FiberLocal based implementation that keeps track of the depth for each fiber separately. I'm of course also fine with fibers being supported natively, but I don't think it's a good idea to put this logic into Logger itself in that case.

Given that concurrency isn't really the default model for PHP still, for now you'd need to call $logger->useLoggingLoopDetection(false)  (which was added for Swoole "support" https://github.com/Seldaek/monolog/commit/9c1566a971d42ff1acb4756708e8d9a5568bb194) - I'm not sure how the FiberLocal impl would look like, as I am not super familiar with fibers tbh. But I'm open to supporting that if possible as I am sure fibers will only see more usage with time.

Fibers each have their own call stack. Fiber locals are basically variables bound to a specific fiber, so a bit like static, but per fiber. I can work on a PR. I guess FiberLocal doesn't have to be used directly, as clearing isn't required here, so support can be built-in.


dashboard_image error

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Not a user config issue

[X] Confirm

Neovim version
NVIM v0.9.5
Operating system/version
Arch WSL2
Terminal name/version
WSL2
$TERM environment variable
No response
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
The following problem occurs when I use this  method  to change dashboard image

The Follow is my settings:

Expected behavior
only show my image
Actual behavior
No response
Additional information
No response

settings["dashboard_image"] = function()
	return {
		-- your ascii image
	}
end

ok，it works

settings["dashboard_image"] = function()
	return {
		-- your ascii image
	}
end


Define Picture Output Format (e.g. WebP)

Is your feature request related to a problem? Please describe.
Currently you cannot define output format and the origin format is used. If you pick a jpg file and crop it you have to convert to webp on your own
Describe the solution you'd like
On the library level it would be easy to convert e.g.
FileOutputStream out = new FileOutputStream(path);
bitmap.compress(Bitmap.CompressFormat.WEBP, 100, out);
out.close();

It's available on 2.3.17


Review: Manoa-Inventory-POST Landing

Overview
The focus for this code review will be centered around the Landing page.
Please pay attention too:

Javascript issues
React components

Review Branch
issue-15
Files to review

Landing.jsx
NavBar.jsx

Checklists

Architecture
Design
JS
React
Testing

Due date
January 30, 2023
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

Landing.jsx
DE-05: Adding comments would improve readability.
L22, 23: ES-01: Consider breaking line up rather than disabling ESLint.
NavBar.jsx
DE-05: Adding comments would improve readability.

Landing.jsx
L22,23: ES-01: There is an Eslint error of too many words in one line.
NavBar.jsx
Looks fine...
Not sure if the Esline error means a lot or if it has a proper way to fix it.

Landing.jsx
REACT-01: consider using components/subcomponents to break up the page
DESIGN-01: refactor to reduce duplication

Remove all stuff page or item

Landing.jsx:
Design: Ensure DRY
Navbar.jsx:
Design: Add in comments for readability

Landing.jsx
ES-01 (Line 22-23) Too many words in one line
NavBar.jsx
Nothing

** Landing.jsx
ES: Max length but we got to do what we got to do
Design: Add Comments
** NavBar.jsx
Design: Add Comments

Line 20, 37 etc. are long, consider break them into multiple lines to make it more readable


Integrate PDI in the spack default repository

In GitLab by @ksiero on Jul 19, 2019, 14:17
We should integrate PDI in the Spack repository.

This has now moved to https://github.com/pdidev/spack

In GitLab by @hasnaoui on May 27, 2020, 14:05
The following packages have been added:

paraconf
zpp
pdi
decl-hdf5
user-code
trace
mpi
pycall (under progress... will be committed soon)

Plugins remaining :

decl-sion
flowvr
fti
test

most of the tests don't work

closed


Rename the JSON for GCEPersistentDisk, fix examples

I know the internal JSON tags are slated for death.  Until then, make them
correct.  Also fix some examples of v1b3 conversions that did not rename the
JSON field.

@bgrant0607 @markturansky

LGTM.
volumes.md has an older v1beta1 GCE example.

LGTM. Will merge on green.

LGTM. Thanks.


fix(scroll): using pcall in case user has already closed the popup window

fix #728

@lewis6991 Please do consider merge this request, it fixes the bug that gitsigns reports error when users close the gitsigns window explicitly with <C-w>c and then scroll the window.

The change is failing CI.
And in general using pcall is just working around a different issue.

Thanks for fixing it finally.


Debugging Code in MSVC 2010/2017

Hi All,
How may I debug the in MSVC 2010 or 2017. All instruction are given through command line, so I guess, In windows we can debug in command prompt. If anyone help on the same it will very useful to all.
Or
Please guide me how to make build with FFmpeg  with version 4.X.
While making build with FFmeg getting error
  server-ffmpeg.obj : error LNK2019: unresolved external symbol _ffio_open_dyn_packet_buf referenced in function "int __cdecl ff_server_send_packet_1(char const *,void *,int,struct AVPacket *,__int64,struct timeval *)" (?ff_server_send_packet_1@@YAHPBDPAXHPAUAVPacket@@_JPAUtimeval@@@Z)
rtspserver.obj : error LNK2019: unresolved external symbol _ff_rtsp_parse_line referenced in function "void * __cdecl rtspserver(void *)" (?rtspserver@@YAPAXPAX@Z)
server-ffmpeg.dll : fatal error LNK1120: 1 unresolved externals

Thanks,
Dinesh Gupta

hi dineshGuptaa,
can you guide me through the installation steps because i got stuck in between the run GA server and client process
i'm doing this process for my thesis work, i would be helpful if you guide me with the installation steps

thanks for the update, but i'm trying to install it in Ubuntu because i'm trying to implement it with containers technology, do you know how to install it in ubuntu


[main| 1.8.2] Unable to delete accounts

Expected behavior

The account gets deleted
Current behavior

The sever returns a internal error
Steps to reproduce

Platform info
Dont know their platform
Other

top10wiigames
February 26, 2024 at 2:47:39 PM
btw when you try to delete accounts, it gives "Unexpected E:104 | Internal error!"

This is now fixed.


Pub get not working

[tmdb_movie_app_riverpod] flutter pub get
Running "flutter pub get" in tmdb_movie_app_riverpod...
The current Dart SDK version is 2.18.4.

Because riverpod_generator >=2.0.0 depends on riverpod_analyzer_utils ^0.1.0 which depends on custom_lint_core ^0.2.12, riverpod_generator >=2.0.0 requires custom_lint_core ^0.2.12.
So, because custom_lint_core >=0.2.11 requires SDK version >=2.19.0 <3.0.0 and tmdb_movie_app_riverpod depends on riverpod_generator ^2.0.0, version solving failed.
pub get failed (1; So, because custom_lint_core >=0.2.11 requires SDK version >=2.19.0 <3.0.0 and tmdb_movie_app_riverpod depends on riverpod_generator ^2.0.0, version solving failed.)
exit code 1

$ flutter --version
Flutter 3.3.8 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 52b3dc25f6 (4 months ago) • 2022-11-09 12:09:26 +0800
Engine • revision 857bd6b74c
Tools • Dart 2.18.4 • DevTools 2.15.0

Got dependencies!
Update your dart and flutter version @arqeco
Pub get is working
@bizz84 You can now close this


Improve Column Alignment

The width of the columns are to small, maybe it's possible to make it dynamic?

I looked into the code and you could change the code in stock.pyin the function ticker_tape. There is a line with  s = self.ticket.ljust(8). If you change thelength to for example 12 it should fit everything.
Resizing it dynamically would be no problem by searching for the longest entry of the watchlist and add a constant value (for example, 3 to it). I would create a pull request but I am not sure how @jkwill87 would like to pass the value to the stock class. Maybe a new formatter class would be a good idea?

I agree that this could probably be improved upon. Its not just the ticket field that's a problem here either-- there's nothing stopping any of the other fields from pushing into their neighbour's columns.
I would also prefer a more dynamic solution but this would be a bit more involved. Each Stock is printed without the context of its neighbours. A proper solution would be to decouple the Stock class' model from its view and leave the responsibility of formatting elsewhere, e.g. by StockStore or a helper class that's a member of StockStore as @MMartin09 suggests. I'd be happy to review a PR for that, otherwise I'll probably get around to implementing it myself 😛 .
In the short term I could probably do a combination of increasing column padding and truncating their contents.

Should be fixed in release 1.2.2.


SQLite real datatype maps to double

PR Info

Closes https://github.com/SeaQL/sea-orm/issues/637

Fixes

[x] SQLite: real, double, float datatype should maps to double in SeaQuery instead of decimal

Right


Companion ui

-Designed home page
-Redesigned installation page
-Redesigned documentation page

This code doesn't work. Submitting a new PR with working code.


Syslog warning: Fix to remove unused sock_ warning

For weak version of Syslog_facility

jenkins test please

jenkins test please


the launcher has some errer

open the app,will can't work,now

android.content.res.Resources$NotFoundException: Resource ID #0x7f040018


MSFT: 6982641 6982542 6982434 6982395 6982802 7033315

Fix a couple of fault injection failures. We should notify the host if
importing child module failure for some reason, and don't continue down the execution
pipeline.
We should also skip dtor at shutdown time and the allocator might have been gone.
assert dispose when in noscriptscope
don't do console output if console is not supported by the host.

@boingoing can you take a look?

LGTM

Looks good


[Builder] Fix last_vid_commitment

Closes #0000
This PR:
We shouldn't ignore genesis view when fetching last known commitment.
This PR does not:

Key places to review:

We'll still terminate at genesis, either by returning it's commitment or by returning None from checked_sub.
The loop shouldn't really iterate too much in most circumstances, but even in some degenerate case where we have a long line of uninterrupted view failures up to genesis and are even somehow missing commitment for genesis itself we're still bounded by checked_sub returning None on target_view 0 and thus terminating the loop.


Resync with instance file

Is there a way to re-retrive the file from the instance... 3 persons are working in the same file, how to resync with the file that is in the instance instead of just uploading

In the config panel, you can pull the existing content from an instance. (See example pic)

Next version of the plugin will be a full rewrite, but will include this functionality exposed as an actual command in the command palette, which means you could also create a key binding for it.
In the current version however, that is not exposed, and the functionality is only available from the config panel (default ctrl+alt+a)

Oops i didn't saw all fileds where filled. Thx. Also i will do another pull request, i just added support for  ui actions

@vr-fox
Just a quick FYI.
You can put a file named tableConfig.user.cson in the folder at ~/.atom/packages/servicenow-sync/config (Mac/Linux) or C:\Users\username\.atom\packages\servicenow-sync\config (Windows).
The servicenow-sync package will read that and merge its contents with the default tableConfig.cson.
It must be formatted in the same manner as the default tableConfig.cson is...
So there really isn't a huge need to keep submitting pr's for new table support. Although for default tables introduced in geneva/helsinki I think it is still a good idea to include them as default. (I wrote this package when the fuji was the current instance.)

Yeah i saw that file, but as you said, this are default tables, and for my team it is easier when they just install the package from atom and not me having to send them the user config. But thanks as i use it for not default tables


Can I translate the README.md file into Korean?

Hello, I am a university student from Korea and I am interested in deep learning.
So I'd like to translate README.md file of this project.
If you are interested in this, please comment below. Thank you!

@rang21c thank you for your interest! That sounds great. The only thing is that I wouldn't know how to expose it. What were you thinking? Another README_KR.md file? or something different?

@w4nderlust
Thank you for your comment!
I thought a lot about that part, and looked a lot of the case of other projects.
I think there are two main ways.

Creating README_KR.md file as you said
Attaching a Korean-translated GitHub link to the top of an existing README.md file
However, the second method seems to have some difficulties when other Koreans find errors and raise issues.

I will follow your decision and if you allow me to translate, I will complete the translation within a month.
Thank you!

@rang21c thank you! So I would say let's go the first route, and if in the meantime we decide differently we can always change. To me the best idea would be to have such a separate file and link to it from the main one, but let's see how it shapes up before making a final decision ok? Looking forward your contribution!

@w4nderlust
Thank you for your permission!
I will do my best in this translation work and try to make more Koreans to know the convenience of Ludwig.
I will reply when the translation is complete.

@w4nderlust
Translation work almost done.
Where can i link translated README file with original README file? (especially which line of original file)
Thank you

I would say you can add a "Translated in Korean" link at the top of the main readme ;)

Thank you 😄
Then, I will look over the work for the last time and make pull request in a few days.


(Geode) Noclip In Editor

what do you mean?

you have ignore damage in game this feature is here and its in vanilla geometry dash


Don't negotiate TLSv1.3 if our EC cert isn't TLSv1.3 capable

TLSv1.3 is more restrictive about the curve used. There must be a matching sig alg defined for that curve. Therefore if we are using some other curve in our certificate then we should not negotiate TLSv1.3.
Additionally we should not apply signature algorithm curve checks in TLSv1.2. RFC8446 says this:
    ECDSA signature schemes align with TLS 1.2's ECDSA hash/signature
    pairs.  However, the old semantics did not constrain the signing
    curve.  If TLS 1.2 is negotiated, implementations MUST be prepared
    to accept a signature that uses any curve that they advertised in
    the "supported_groups" extension.

Fixes #7435

Please add tests

I updated this to add a test and rebased it.
I also dropped the commit about sig alg curve checks in TLSv1.2. This turned out to be unnecessary (and wrong).

I wonder if we should also add some test that fails.

I wonder if we should also add some test that fails.

Can you suggest what such a test might look like?

Ping @kroeckx

Ping @kroeckx

I wonder if we should also add some test that fails.

Can you suggest what such a test might look like?

Could that be a cert with an invalid curve for TLSv1.3, perhaps?  I'm not savvy enough to know if that should cause an error or a downgrade to TLSv1.2, but yeah, something like that.  The way I understand this PR, you've only added a test for a cert that successfully enables TLSv1.3, is that correct?

Could that be a cert with an invalid curve for TLSv1.3, perhaps? I'm not savvy enough to know if that should cause an error or a downgrade to TLSv1.2, but yeah, something like that. The way I understand this PR, you've only added a test for a cert that successfully enables TLSv1.3, is that correct?

The tests do add a curve which is invalid for TLSv1.3. Prior to the fix in this PR libssl would have negotiated TLSv1.3 anyway and then subsequently failed. After the fix in this PR libssl realises that its cert is not TLSv1.3 compatibile and successfully negotiates TLSv1.2 instead.

@vdukhovni I added a commit to implement your suggestion for the key generation in the script. I also added a commit to add a test which checks for a failed connection if the minimum protocol is TLSv1.3 and brainpool is used as per @kroeckx suggestion.
Can you reconfirm?
@kroeckx are you ok with this now?

Yes, it's fine for me.


fix(tabs): recalculate right arrow condition when navs has changed

🤔 这个 PR 的性质是？

[x] 日常 bug 修复
[ ] 新特性提交
[ ] 文档改进
[ ] 演示代码改进
[ ] 组件样式/交互改进
[ ] CI/CD 改进
[ ] 重构
[ ] 代码风格优化
[ ] 测试用例
[ ] 分支合并
[ ] 其他

🔗 相关 Issue
resolve #658
💡 需求背景和解决方案

📝 更新日志

fix(tabs): 修正右箭头出现的时机

[ ] 本条 PR 不需要纳入 Changelog

☑ 请求合并前的自查清单
⚠ 请自检并全部勾选全部选项。⚠

[ ] 文档已补充或无须补充
[ ] 代码演示已提供或无须提供
[ ] TypeScript 定义已补充或无须补充
[ ] Changelog 已提供或无须提供

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@66fa54a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #700   +/-   ##
==========================================
  Coverage           ?   60.93%
==========================================
  Files              ?      402
  Lines              ?    13908
  Branches           ?     2610
==========================================
  Hits               ?     8475
  Misses             ?     5194
  Partials           ?      239

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 66fa54a...ef1d2ab. Read the comment docs.


Generate AST structs by using codegen

Most struct/enum definitions are now generated programmatically.
Context: #13, #17

bors r+


If the first describe is the same for different specs, which should be startet in own browser session -there are some difficulties with generating XML-Reports.

I have put the question on Stackoverflow, but perhaps that can be solved only through fix in the framework.
http://stackoverflow.com/questions/42591171/jasminereporter-junitxmlreporter-overwrites-the-results-of-the-run
My specs share the same first describes (the second is different) e.g.
spec1.js:
describe("topic1",()=> {
  describe("testcase1",()=> {
   ...
  });
});

spec2.js:
describe("topic1",()=> {
  describe("testcase2",()=> {
   ...
  });
});

which works fine if I run the both specs in the same browser like:
protractor --specs spec1.js,spec2.js config.js

But as I want to restart the browser between runs of spec1.js and spec2.js with either:
protractor --specs spec1.js config.js; protractor --specs spec2.js config.js

or setting in capabilites:
shardTestFiles: true

then the junitresults.xml resp. junitresults-topic1.xml (depending on consolidateAll option beingtrue or false) is generated containing only the results of the last spec (spec2.js) run.
How can I get all the runs in my XML results?

This issue seems related to both #175 and #149.


Feature request: Add option to remove image from word

It happened to me, that I chose an image for a word, and then I realized, that it would be very difficult to find an image that represented that particular word.
It would be great to be able to click on a selected image again and have that image separate from the word.

In branch issue_25_remove_images.  Deletion not done yet but selection is.

Fixed in develop: click the image in the term form to highlight it (red border), and then hit delete or backspace to delete it.

Released in 3.1.4.


Image Processing complete?

Alright, hopefully there will be no errors this time and I can merge onto fake master.
With this PR, image processing should be complete. Unless there are bugs
All unit tests written and pep8 errors caught, I think

Screw it, this is fake master. Something is causing travis to fail. I'm merging and we can fix before merging onto real master eventually.

make sure you pushed the files you needed up as well?


[release-1.9] backport test framework changes

Cherry picks #30232 #30092 #30272 to keep 1.9's test framework up to date.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.9
749dcb75a0b773cd799f9345ae44c23e2257c4e8
link
/test release-notes_istio_release-1.9

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

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.9
749dcb75a0b773cd799f9345ae44c23e2257c4e8
link
/test release-notes_istio_release-1.9

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

/retest

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio_release-1.9
749dcb75a0b773cd799f9345ae44c23e2257c4e8
link
/test integ-pilot-k8s-tests_istio_release-1.9

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

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio_release-1.9
749dcb75a0b773cd799f9345ae44c23e2257c4e8
link
/test integ-pilot-k8s-tests_istio_release-1.9

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

/retest

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio_release-1.9
749dcb75a0b773cd799f9345ae44c23e2257c4e8
link
/test integ-security-k8s-tests_istio_release-1.9

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

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio_release-1.9
749dcb75a0b773cd799f9345ae44c23e2257c4e8
link
/test integ-security-k8s-tests_istio_release-1.9

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

/retest

/retest


process.env.NODE_ENV undefined warning

if using process.env.NODE_ENV inside src/, leads to warning "'process' is not defined", while the string replacement does work. (Maybe replace could happen before linting?)

process.node.NODE_ENV ? did you mean process.env.NODE_ENV ?

yes apologies - typo'ed only here..

Thanks for reporting!

breaking the issue-to-commit record!

Wasn’t that hard 😄


Arch Linux: Cannot log in to Last.fm

OS: Arch Linux
Issue Descriptions: Cannot log in to Last.fm
Steps to Reproduce:

Launch program
Go to Desktop Settings from the sidebar or notification icon
Go to LASTFM tab
Click on Log In To Last.fm
Text changes from "Not Authorized" to "Authorization in progress" then back to "Not authorized". No windows pop up. No error output from the program.

If you're using the git build it doesn't come with Last.fm keys. You can either get some API keys yourself and build them into the application or download the stable release.

What @jostrander said :+1:


TIGER Data Layer Not Appearing

URL
No response
How to reproduce the issue?

Edit using ID editor
under Background Settings > Overlays, select TIGER 2017 to TIGER 2022.

TIGER overlay does not appear, even where it previously existed.
Screenshot(s) or anything else?
No response
Which deployed environments do you see the issue in?
Released version at openstreetmap.org/edit
What version numbers does this issue effect?
2.27.1
Which browsers are you seeing this problem on?
Chrome

As a heads up, I suspect it's related to this change and will require a minor change to iD and a new release to fix: https://community.openstreetmap.org/t/updating-the-url-for-tiger-road-overlay-layers/103880

Also, related discussion in the OpenStreetMap US Slack for members (or anyone wanting to join): https://osmus.slack.com/archives/C029HV951/p1695398195227329
TL;DR: Also happening in JOSM. The ticket for JOSM is here: https://josm.openstreetmap.de/ticket/23198#ticket though it may have been resolved now

will require a minor change to iD and a new release to fix:

Yes, iD needs to include the latest editor-layer-index where this is already fixed (https://github.com/osmlab/editor-layer-index/pull/2060) and publish a new release.

The recent releases appear to have addressed this issue. I can get the TIGER layers to load in iD 2.27.3

Looks like this was resolved in 01f00b9e6b9a6cfed557f86f01612ffa8ed709ad. Closing. Thank you!


Problem with listing blobs by using of ListBlobsHierarchy()

Bug Report

import path of package in question github.com/Azure/azure-sdk-for-go/sdk/storage/azblob
SDK version
github.com/Azure/azure-sdk-for-go/sdk/azidentity v0.12.0
github.com/Azure/azure-sdk-for-go/sdk/storage/azblob v0.2.0
github.com/Azure/azure-sdk-for-go/sdk/to v0.1.1 // indirect
github.com/aws/aws-sdk-go v1.40.45
Go version: go version go1.16.6 darwin/amd64
Azurite: mcr.microsoft.com/azure-storage/azurite:3.14.3

In my code I'm invoking ContainerClient.ListBlobsHierarchy() to iterate over a list of blobs in an Azure Blob container.
I'm using azure-sdk-for-go in combination with Azurite.
However, tests fail with error get response from Azure API: no Host in request URL.
I started troubleshooting and found out that containerClient.ListBlobHierarchySegment() on the next call of NextPage() is passing test-fixtures-TestAzBlobClient_ListObjects-2021-11-26T114346.107911+0100/test.csv.1 into runtime.NewRequest() as value of parameter endpoint.
I'm wondering is it problem of Azurite which doesn't compose NextMarker properly or it's problem of ListBlobHierarchySegment() which doesn't respect containerClient configuration ?
Here is a screenshot of debugger demonstrating the process of troubleshooting:

My code:
                 perPageConv := int32(2)

		listOptions := &azblob.ContainerListBlobHierarchySegmentOptions{
			Prefix: &prefix,
			Maxresults: &perPageConv,
		}
		azPager := azClient.ListBlobsHierarchy("/", listOptions)

                for {
			hasNext := azPager.NextPage(ctx)
                         if azErr := azPager.Err(); azErr != nil {
				fmt.Println("get response from Azure API: ", azErr.Error())
				break
                        }

			page := azPager.PageResponse()

			for i, commonPrefix := range page.Segment.BlobPrefixes {
                                 fmt.Println("Blob prefix ", commonPrefix.Name)
			}

			for i, keyObject := range page.Segment.BlobItems {
                                 fmt.Println("Blob ", keyObject.Name)
			}

			if !hasNext {
				break
			}

			pageNo++
		}

I have just checked on real Azure env and the problem is the same: after requesting the second page I get :
2021/11/26 13:48:25 list blobs:  operational: get response from Azure API: no Host in request URL

@kenegozi may I ask you to take a look to this issue? Labels in this issue are incorrect: it's not a question but a bug report and the issue is a serious one.

Currently looking.

Looks like the generated code is incorrect. Seems to be supplying the entire marker as the new URI. I'm going to shoot out a PR to override this incorrect behavior.

Thank you @adreed-msft for looking at this issue. I cannot test your PR as there are broken dependencies. Will test when your PR be merged.
$ go get github.com/Azure/azure-sdk-for-go/sdk/storage/azblob@adreed/fix-pagers
go: downloading github.com/Azure/azure-sdk-for-go/sdk/storage/azblob v0.1.1-0.20211201002034-a166c1381cd2
go: downloading github.com/Azure/azure-sdk-for-go v58.3.1-0.20211201002034-a166c1381cd2+incompatible
# github.com/Azure/azure-sdk-for-go/sdk/storage/azblob
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_append_blob_client.go:17:47: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_blob_client.go:17:9: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_blob_client.go:21:41: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_block_blob_client.go:31:46: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_connection_string.go:38:62: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_container_client.go:18:9: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_container_client.go:27:51: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_page_blob_client.go:23:45: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_service_client.go:30:9: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_service_client.go:40:47: undefined: azcore.Credential
../../../Library/Go/pkg/mod/github.com/!azure/azure-sdk-for-go/sdk/storage/azblob@v0.1.1-0.20211201002034-a166c1381cd2/zc_service_client.go:40:47: too many errors

Hey @an2deg ,
Can you try
go get github.com/Azure/azure-sdk-for-go/sdk/storage/dev-azblob

dev-azblob is our development branch and I've merged the changes there.

Hi Mohit,
I tried to do so but in my project and even in empty project I alway get
error message:
✔ ~/Projects/test
11:18 $ go get github.com/Azure/azure-sdk-for-go/sdk/storage/dev-azblob
go get: module @.*** found
(v60.0.0+incompatible), but does not contain package
github.com/Azure/azure-sdk-for-go/sdk/storage/dev-azblob
✘-1 ~/Projects/test
11:19 $
On Fri, Dec 3, 2021 at 7:05 AM Mohit Sharma @.***>
wrote:

Hey @an2deg https://github.com/an2deg ,
Can you try
go get github.com/Azure/azure-sdk-for-go/sdk/storage/dev-azblob
dev-azblob is our development branch and I've merged the changes there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-go/issues/16295#issuecomment-985241774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC47UWWSEFI42BF7MO2CR3UPBMZBANCNFSM5I2N22UQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Best regards,
Andrii Degtiarov

Hey @an2deg ,
What's the version of Golang you're using?
Can you try go get github.com/Azure/azure-sdk-for-go/sdk/storage/azblob@b76e8f674ad2125742c244cfc8faa417202525bf
or go get github.com/Azure/azure-sdk-for-go/sdk/storage/azblob@dev-azblob

Hi Mohit,
Ok, go get @.***
works.
However, I still have problems with the fix. I have the following function
to load fixtures to Azurite:
var containerClient azblob.ContainerClient
func setUp() {
if azblobHost == "" {
log.Fatalf("AZBLOB_HOST env variable is required to run
depository/azure tests\n")
}
if azblobPort == "" {
log.Fatalf("AZBLOB_PORT env variable is required to run
depository/azure tests")
}
if azblobKey == "" {
log.Fatalf("AZBLOB_KEY env variable is required to run
depository/azure tests")
}
if azblobAccount == "" {
log.Fatalf("AZBLOB_ACCOUNT env variable is required to run
depository/azure tests")
}
clientOpts = Opts{
Endpoint:    fmt.Sprintf("%s:%s", azblobHost, azblobPort),
AccountName: azblobAccount,
SecretKey:   azblobKey,
Container:   azblobContainer,
DisableSSL:  true,
}
creds, err := azblob.NewSharedKeyCredential(azblobAccount, azblobKey)
if err != nil {
log.Fatalf("Cannot initialize Azure Blob Storage client: %v\n",
err.Error())
}
srvClient, err :=
azblob.NewServiceClientWithSharedKey("http://"+azblobHost+":"+azblobPort+"/"+azblobAccount,
creds, nil)
if err != nil {
log.Fatalf("Cannot initialize Azure Blob Storage client: %v\n",
err.Error())
}
containerClient = srvClient.NewContainerClient(azblobContainer)
ctx := context.Background()
_, err = containerClient.Create(ctx, &azblob.CreateContainerOptions{})
if err != nil {
azErr := &azblob.StorageError{}
if errors.As(err, &azErr) && azErr.Response().Status == "409 The
specified container already exists." {
// Container is already exist, ignore
} else {
log.Fatalf("Cannot provision a container in Azure Blob
Storage client: %v\n", err.Error())
}
}
}
func loadFixturesForListing(prefix string) error {
ctx := context.Background()
const alreadyExistingObjectKey   = "test.csv"
// Create objects to test pagination
for i := 0; i < 14; i++ {
blobName := path.Join(prefix,
alreadyExistingObjectKey+fmt.Sprintf(".%d", i))
blockBlob := containerClient.NewBlockBlobClient(blobName)
reader := bytes.NewReader(testKeyContent)
if _, err := blockBlob.UploadStreamToBlockBlob(ctx,
newNopSeekCloser(reader),
azblob.UploadStreamToBlockBlobOptions{}); err != nil {
     return fmt.Errorf("upload blob %s into Azure Container: %w",

blobName, err)
}
}
// create a subfolder
blobName := path.Join(prefix, "subfolder", alreadyExistingObjectKey)
blockBlob := containerClient.NewBlockBlobClient(blobName)
reader := bytes.NewReader(testKeyContent)
if _, err := blockBlob.UploadStreamToBlockBlob(ctx, newNopSeekCloser(reader),
azblob.UploadStreamToBlockBlobOptions{}); err != nil {
  return fmt.Errorf("upload blob %s into Azure Container: %w",

blobName, err)
}
return nil
}
This code provisions 14 blobs with names "test.csv.0" - "test.csv.13"
under prefix like
"test-fixtures-TestAzBlobClient_ListObjects-2021-12-06T115616.24455+0100/".
And 1 blob test.csv under prefix
"test-fixtures-TestAzBlobClient_ListObjects-2021-12-06T115616.24455+0100/subfolder".
I expect that ListBlobsHierarchy() for prefix
"test-fixtures-TestAzBlobClient_ListObjects-2021-12-06T115616.24455+0100"
after all will returns 14 BlobItems items and 1 BlobPrefixes.
The I invoke azPager := azClient.ListBlobsHierarchy(delimiter,
listOptions) with the following params
[image: image.png]
and iterate over results by using azPager.NextPage(ctx).
Please note, even when I'm not specifying Maxresults, behavior is the same.
On the first NextPage call() I have the following results:
[image: image.png]
on a second call I get almost the same but MaxResults now is 5000 and
NextPage() returns false
[image: image.png]
Because NextPage() returns false, I exited from a loop at this point.
azPager.Err() is always nil .
However I executed a few more times from the debugger, but the
response is still the same.
On Mon, Dec 6, 2021 at 11:35 AM Mohit Sharma @.***>
wrote:

Hey @an2deg https://github.com/an2deg ,
What's the version of Golang you're using?
Can you try go get
@.***
or go get @.***
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-go/issues/16295#issuecomment-986648370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC47UWHBCEOME6Y227L4ETUPSGXPANCNFSM5I2N22UQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Best regards,
Andrii Degtiarov

I have faced the same issue when executing the sample code from the README.md
Pulling the dev release go get github.com/Azure/azure-sdk-for-go/sdk/storage/azblob@dev-azblobfixed the issue for me. Is there already a date fixed when this fix will be merged into the official release?

Fixed in sdk/storage/azblob/v0.3.0


Base Url should be a path parameter

With the latest URL templating changes, it seems the various request builders have the base URL e.g. https://graph.microsoft.com/v1.0/ is "hardcoded" in the URL templates.
Examples
Dotnet - https://github.com/microsoft/kiota-samples/blob/4f23266baf055c9e17bdc5bbef8efad6d7c79175/msgraph-mail/dotnet/Users/UsersRequestBuilder.cs#L31
Typescript - https://github.com/microsoft/kiota-samples/blob/4f23266baf055c9e17bdc5bbef8efad6d7c79175/msgraph-mail/typescript/src/users/usersRequestBuilder.ts#L19
This potentially could cause issues in the event we wish to override the base URL with a national cloud URL such as https://graph.microsoft.us. This wasn't an issue before given that the base URL would be passed forward as the URL would be built allowing the parameter at the GraphClient constructor to dictate this.
This could potentially be resolved if we set the base URL as a path parameter to allow it to be replaced/modified/overriden.

Thanks for bringing this up. I debated myself during the implementation. We could go all the way to saying "each SDK is generated for a specific endpoint, if you want to change endpoints, generate a new SDK" and ship service libraries for national clouds too.
This would make sense as national cloud only provide a subset of the APIs available in the public cloud.
Obviously this would not be practical for ISVs building solutions that span across clouds, or even for local debugging when people test client solutions against a locally running APIs (not necessarily talking about Microsoft graph for that last example).
The alternative would be make the base URL as a parameter as you're suggesting. Couple of notes here:
This parameter need to have a + sign to avoid url encoding (: and /) so it'd look something like that:
{+baseurl}/me/inferenceClassification{?select}
The Api client constructor needs to set the base url, and we need to provide a way for consumers to change that base URL so I'd suggest the following for this https://github.com/microsoft/kiota-samples/blob/4f23266baf055c9e17bdc5bbef8efad6d7c79175/msgraph-mail/dotnet/ApiClient.cs#L25
            _ = requestAdapter ?? throw new ArgumentNullException(nameof(requestAdapter));
            PathParameters = new Dictionary<string, object>();
            UrlTemplate = "{+baseurl}"; // this is changed
            requestAdapter.SetBaseUrl("https://graph.microsoft.com/v1.0"); // this is new
            RequestAdapter = requestAdapter;
            ApiClientBuilder.RegisterDefaultSerializer<JsonSerializationWriterFactory>();
            ApiClientBuilder.RegisterDefaultDeserializer<JsonParseNodeFactory>();

Obviously the request adapter interface would need two more methods Set & Get base URL (dotnet could use auto-properties obviously).
And at runtime, the request adapter implementation would add the parameter value to the request information before resolving the URL and performing the request.
What do you think about all of that? I could take this on as soon as I'm done with Go snippets generation, so probably later this week.

Hey @baywet,
I agree with this.
Your description covers how I was thinking about this with the request builder classes getting the baseUrl from one source (in this case the requestAdapter instance) and feeding it to the path parameters collection so that the URL could still be built in a configurable fashion as we could before with minimal changes.

This parameter need to have a + sign to avoid url encoding (: and /) so it'd look something like that:
{+baseurl}/me/inferenceClassification{?select}

I was unsure if you could use the path parameter to even replace the protocol section of the URL but other than that, this makes sense to me.

I support this approach.

@andrueastman I've started the work under #795 in case you want to take a look and provide feedback while I work on other languages.


Azure Form Recognizer pdf Train

Hi,
I am trying to use Azure Form Recognizer within my project. I am creating modules by uploading 5 pdf documents from the user via https://fott-preview.azurewebsites.net. I could not find a sample code where I can give the module creation process to the user. I will be glad if you can help.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.


Basic support for IPv4 static routes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

26 of 63 (41.27%)  changed or added relevant lines in 2 files are covered.
9 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 63.92%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sysrib/server.go
1
3
33.33%

sysrib/static.go
25
60
41.67%

Files with Coverage Reduction
New Missed Lines
%

sysrib/zapi.go
9
65.79%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
3398

Relevant Lines:
5316

💛  - Coveralls


changelog: Add note for ptrace events (#438)

r? @fiveop @posborne

@homu r+

:pushpin: Commit 3a07b5c has been approved by posborne

:hourglass: Testing commit 3a07b5c with merge 54e6197...

:sunny: Test successful - status


TikTok X-Gorgon, X-Khronos, X-SS-STUB, Device Register, Captcha Solver

https://github.com/reverser007/tiktok read for more info.

$2k? I have it for only a few hundreds.

$2000 lol

for anyone still searching for free xgorgon hmu https://telegram.me/AlexWardner


Adding custom toolbar button or editing default icons

How can I add custom toolbar button and edit the default icons/buttons?

Found that in the beta release but some important features are missing like:

Online Image Insert
Youtube video embedding and preview

Hope you will work on these.
Thanks

Hi @rakib205
Yes, custom toolbar buttons is in beta. Regarding adding the online images and embedding YouTube videos, I will add them in upcoming releases once I find sometime.

Most usable HTML editor for Flutter hands down. Custom icons will allow me to match the colors of the default icons to the custom icons in dark mode (buttons are black, custom buttons are white).

Most usable HTML editor for Flutter hands down. Custom icons will allow me to match the colors of the default icons to the custom icons in dark mode (buttons are black, custom buttons are white).
Will the update also include a 'color' option inside QuillHtmlEditor or a default ToolBarStyle color option for the text inside the editor? Thanks in advance

Hi @jstdk ,
Thank you for your feedback :)
I just deployed a new version 2.0.3, which includes the functionality to customise color and  size of the toolbar items. Please check it out.

@the-airbender, incredible, thanks for your feedback and quick update.


Adding SSL Context to TCP connector

@HonzaKral bumped ES version in setup.py

@HonzaKral PTAL tests and docs added.


Added support for Jamo Extended B

Also I have debugged many errors in the code.

It's all in the pull requests.


Says right code is wrong?

I'm following along a pluralsight course and copied this code straight out of the video, they are using promises as well to test out AJAX calls and I think the playground is awesome... But his same exact code snippet works while mine says 'echo' is undefined? I'm not a beginner but I'm definitely baffled by this.
`
function echo(message, delay) { };
var log = r => {
console.log(r);
return r;
}
echo('1st echo')
.then(log)
.then(log)
.then(log);
`

Not sure what's baffling here. The echo function doesn't have a .then method. So you get an error because echo#then is not defined. This happens when pasting to the DevTools in any browser as well.

I'm not sure I understand. All I'm doing is typing exactly what I see in a pluralsight training course. I'm not copying and pasting.
A trainer is using promisees and I'm following along, or I was. I just thought I would try and help out as I don't need to follow along but if it gets me used to using promisees then that's like a good thing for the site.
Would it help to explain the issue here if I provided screenshots?


Refactor cli args

[x] Docopt delegates to sub-parsers
[ ] Figure out -w, --with <key> <val> and [options]
[ ] Test the help strings
[ ] spag history show 0 -> spag history 0
[x] Rename show -> cat
[x] Rename list -> ls
[ ] Add an alias spag request == spag r

#80 in progress

Now we have

spag env ls and spag request ls
spag env cat and spag request cat <name>
spag request show-params is now  spag request inspect

I guess we could change spag request inspect to spag inspect but I didn't feel like doing that right now.

Yeah so docopt doesn't like, in the options section, multiple args per flag for either of the following
--with <key> <val>          Supply a request parameter
-w --with <key> <val>       Supply a request parameter

But it does pattern match the usage line:
spag request (-w <key> <val>|--with <key> <val>)

So I left the -w, --with out of the options section, and just made the thing work. I think adding an examples section to each of the usage strings will be more useful than that options section anyway.

Just one comment inline. We can file an issue and fix later if you'd rather merge this as is. It all seems to work.


updating members list

Could you please also update RPIP-36?

Superseded by https://github.com/rocket-pool/RPIPs/pull/132


Update CSV, HTTP, and DataFrames versions

Codecov Report

Merging #7 (8793736) into master (3a56932) will decrease coverage by 0.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   79.16%   78.72%   -0.45%
==========================================
  Files           3        2       -1
  Lines          48       47       -1
==========================================
- Hits           38       37       -1
  Misses         10       10

Impacted Files
Coverage Δ

src/QuandlAccess.jl

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 3a56932...8793736. Read the comment docs.


Pagination input doesn't display Return icon in border-box context

Describe the bug
Inside our product, we forcibly enable an border-box context mode, but this leads to problems displaying the pagination component: the Return icon that should appear when focusing on the page input field isn't displayed. Maybe there's a way not to rely on content-box mode in the component markup?
Code example
https://codesandbox.io/s/eloquent-bird-v6tcpl
To Reproduce
Steps to reproduce the behavior:

Click/focus on page input
Make sure that Return icon doesn't appear:

Expected behavior
The Return icon is displayed.

Hi! I'm not sure, that this is a bug.
This is happen, because in box-sizing: border-box; width is including paddings (in our case width is 16px and paddings by default are - 8px). So, 16-8*2 = 0 - and you don't have a place for the content (icon).
Let me ask you - why do you use box-sizing: border-box; for all elements?
Because by default in the CSS box model, the width and height you assign to an element is applied only to the element's content box.

But if you use our components in your app, your selector (because of id) will be always more specific then ours.
Could you maybe change your selector from id to class?
Another way - write something like that:
import React from "react";
import Pagination from "@semcore/ui/pagination";
import Return from "@semcore/ui/icon/Return/m";

const Demo = () => {
  const totalPages = 122360;
  return (
    <Pagination totalPages={totalPages}>
      <>
        {totalPages === 1 ? null : (
          <>
            <Pagination.FirstPage />
            <Pagination.PrevPage />
            <Pagination.NextPage />
          </>
        )}
        <Pagination.PageInput>
          <Pagination.PageInput.Value />
          <Pagination.PageInput.Addon
            tag={Return}
            interactive
            aria-label={"confirm"}
            style={{ boxSizing: "content-box" }} // <- set correct style here
          />
        </Pagination.PageInput>
        <Pagination.TotalPages />
      </>
    </Pagination>
  );
};

Okay, I have the possibility to replace the our selector from the id to the class to fix the specificity, but our application CSS are inserted later than the Uikit CSS, so it still have a higher priority. Anyway, manual overriding <Pagination.PageInput.Addon /> styles to content-box solves the problem, thank you!


[Service] Swagger Specs are valid, when only warnings are shown

If the official example  from here is used against the swagger Validator it generates a valid badge, but does also generate some warning level errors. (See valid badge and debug messages)
I adjusted the parsing of the endjust endpoint so warnings don't generate an invalid swagger Spec.
I also added a test for this scenario.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @Bl4d3s!

  Generated by :no_entry_sign: dangerJS against f93ea998626b08f565b1dce9ba715b3b4fdc8ec3

Unfortunatley I don't know why some tests are now failing, if someone can point me in the right direction, I'll gladly fix it. (with npm test locally all went well)

Unfortunatley I don't know why some tests are now failing, if someone can point me in the right direction, I'll gladly fix it. (with npm test locally all went well)
Edit: Now they have passed ?

Sorry I forgot to mention it earlier, but the original problem was the title of the PR (which started with [Service]. After I changed it to put the brackets around Swagger the tests passed. The way the CI process works in Shields for PRs is that the service tests for services listed inisde [] are executed. Since you had [Service] in the PR it was trying to run the tests for a service named Service which does not exist

If I'm following correctly, the objective was to make the resulting badge consistent with the Swagger validator by handling warning-only level "'errors".
As noted in the PR description, using the sample https://petstore3.swagger.io/api/v3/openapi.json returns valid with the Swagger validator, but invalid for Shields
http://validator.swagger.io/validator?url=https://petstore3.swagger.io/api/v3/openapi.json

Prod Shields
https://img.shields.io/swagger/valid/2.0/https/petstore3.swagger.io/api/v3/openapi

However, AFACIT the staging app for this PR is still saying the sample is invalid
https://shields-staging-pr-4294.herokuapp.com/swagger/valid/2.0/https/petstore3.swagger.io/api/v3/openapi.json

Did I get any of that wrong? If not, does that mean the original problem still exists?

If I'm following correctly, the objective was to make the resulting badge consistent with the Swagger validator by handling warning-only level "'errors".
As noted in the PR description, using the sample https://petstore3.swagger.io/api/v3/openapi.json returns valid with the Swagger validator, but invalid for Shields
http://validator.swagger.io/validator?url=https://petstore3.swagger.io/api/v3/openapi.json

Prod Shields
https://img.shields.io/swagger/valid/2.0/https/petstore3.swagger.io/api/v3/openapi

However, AFACIT the staging app for this PR is still saying the sample is invalid
https://shields-staging-pr-4294.herokuapp.com/swagger/valid/2.0/https/petstore3.swagger.io/api/v3/openapi.json

Did I get any of that wrong? If not, does that mean the original problem still exists?

You are right, I tested with my own specification, located here, where it works. Ill look into it and update the PR

I wonder if it's an issue with the file extension (when it is a json based spec) getting sucked in and dropped from being passed along to the validator service.
TBH we should probably move the spec location param from a route param to a query param, refs #3714 and #4178

I wonder if it's an issue with the file extension (when it is a json based spec) getting sucked in and dropped from being passed along to the validator service.
TBH we should probably move the spec location param from a route param to a query param, refs #3714 and #4178

It most definitly is.
See attached log of badge debug output

npm run badge -- swagger/valid/2.0/https/petstore3.swagger.io/api/v3/openapi.json

<EMAIL_ADDRESS>badge /home/michael/GitRepos/shields
cross-env NODE_CONFIG_ENV=test TRACE_SERVICES=true node scripts/badge-cli.js >"swagger/valid/2.0/https/petstore3.swagger.io/api/v3/openapi.json"

1103204445 FsTokenPersistence configured with /home/michael/GitRepos/shields/private/github->user-tokens.json
1103204445 Server is starting up: http://localhost:1111/
Service class  👩🍳
SwaggerValidatorService
Named params  🎫
{ scheme: 'https', url: 'petstore3.swagger.io/api/v3/openapi' }
Query params  🖍
{
color: undefined,
colorA: undefined,
colorB: undefined,
label: undefined,
labelColor: undefined,
link: undefined,
logo: undefined,
logoColor: undefined,
logoPosition: undefined,
logoWidth: undefined,
style: undefined
}
Request  🏹
http://validator.swagger.io/validator/debug
Response status code  🎯
200
Response JSON (before validation)  🎯
{
schemaValidationMessages: [
{
level: 'error',
message: "Can't read from file https://petstore3.swagger.io/api/v3/openapi"
}
]
}
Response after validation  🛁
{
schemaValidationMessages: [
{
level: 'error',
message: "Can't read from file https://petstore3.swagger.io/api/v3/openapi"
}
]
}
Service data  🛡
{ message: 'invalid', color: 'red' }
Rendered badge  🛡
{
label: 'swagger',
message: 'invalid',
color: 'red',
link: [],
name: 'swagger',
value: 'invalid'
}

I'm not entierly sure if this is the correct approach, as it will be a breaking change. Is this how you intended it or how should I do it? Locally now both the json and yaml spec show a valid spec, the tests Im currently fixing

It might also be helpful to add a NotFound check here, since presumably, everyone that's tried to get a badge for a json based spec would've received invalid regardless of whether the spec was valid. It looks like the Validator API will return an HTTP 200 response with a single error like the one below, and it would be great if we could detect that and convey that in the badge (for example ) instead of invalid
{
  "schemaValidationMessages":[
    {
      "level":"error",
      "message":"Can't read from file {{spec url}}"
    }
  ]
}

We could do that by detecting that condition as part of the transform function (single error level error with that message) and then throwing a Shields error:
  throw new NotFound({ prettyMessage: 'spec not found' })

I did implement the not found feature, and pushed the (currently not working) redirect, maybe you can see the issue better this way

I did implement the not found feature, and pushed the (currently not working) redirect, maybe you can see the issue better this way

Thanks that should help! My understanding is that a swagger spec will always have a .json or a .yml/.yaml extension, do you know if that is correct?

I did implement the not found feature, and pushed the (currently not working) redirect, maybe you can see the issue better this way

Thanks that should help! My understanding is that a swagger spec will always have a .json or a .yml/.yaml extension, do you know if that is correct?

Yes, quote from the website

API specifications can be written in YAML or JSON.

Okay I managed to get this working, though with one caveat that IMO we'll need to accept. I'd started making some local changes so the service class diverged just a bit from what you have here. Feel free to incorporate or toss, but something like this should get us to a point of having this solved.
swagger.service.js
'use strict'

const Joi = require('@hapi/joi')
const { optionalUrl } = require('../validators')
const { BaseJsonService, NotFound } = require('..')

const schema = Joi.object({
  schemaValidationMessages: Joi.array().items(
    Joi.object({
      level: Joi.string().required(),
      message: Joi.string().required(),
    }).required()
  ),
}).required()

const queryParamSchema = Joi.object({
  specUrl: optionalUrl.required(),
}).required()

module.exports = class SwaggerValidatorService extends BaseJsonService {
  static get category() {
    return 'other'
  }

  static get route() {
    return {
      base: 'swagger/valid',
      pattern: '2.0',
      queryParamSchema,
    }
  }

  static get examples() {
    return [
      {
        title: 'Swagger Validator',
        staticPreview: this.render({ status: 'valid' }),
        namedParams: {},
        queryParams: {
          specUrl:
            'https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v2.0/json/petstore-expanded.json',
        },
      },
    ]
  }

  static get defaultBadgeData() {
    return { label: 'swagger' }
  }

  static render({ status }) {
    if (status === 'valid') {
      return { message: status, color: 'brightgreen' }
    } else {
      return { message: status, color: 'red' }
    }
  }

  async fetch({ specUrl }) {
    return this._requestJson({
      url: 'http://validator.swagger.io/validator/debug',
      schema,
      options: { qs: { url: specUrl } },
    })
  }

  transform({ json, specUrl }) {
    const valMessages = json.schemaValidationMessages

    if (!valMessages || valMessages.length === 0) {
      return { status: 'valid' }
    }

    if (valMessages.length === 1) {
      const { message, level } = valMessages[0]
      if (level === 'error' && message === `Can't read from file ${specUrl}`) {
        throw new NotFound({ prettyMessage: 'spec not found or unreadable' })
      }
    }

    if (valMessages.every(msg => msg.level === 'warning')) {
      return { status: 'valid' }
    }

    return { status: 'invalid' }
  }

  async handle(_routeParams, { specUrl }) {
    const json = await this.fetch({ specUrl })
    const { status } = this.transform({ json, specUrl })
    return this.constructor.render({ status })
  }
}

swagger-redirect.service.js
'use strict'

const { redirector } = require('..')

module.exports = [
  redirector({
    category: 'other',
    name: 'SwaggerRedirect',
    route: {
      base: 'swagger/valid/2.0',
      pattern: ':scheme(https|http)/:url*',
    },
    transformPath: () => '/swagger/valid/2.0',
    transformQueryParams: ({ scheme, url }) => {
      const suffix = /(yaml|yml|json)$/.test(url) ? '' : '.json'
      return { specUrl: `${scheme}://${url}${suffix}` }
    },
    dateAdded: new Date('2019-11-03'),
  }),
]

Currently, no one is able to use the Swagger badge with the default/extensionless mode when using a json-based spec.
So this doesn't actually work, as the filename is misinterpreted as the requested .json format from Shields
https://img.shields.io/swagger/valid/2.0/https/raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v2.0/json/petstore-expanded.json
Badge doesn't work because it returns json --> , but the json response is also misleading (it's really a case of the spec not being found)
{"label":"swagger","message":"invalid","color":"red","link":[],"name":"swagger","value":"invalid"}

Shields users would have to explicitly include the .svg extension to get a working badge
https://img.shields.io/swagger/valid/2.0/https/raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v2.0/json/petstore-expanded.json.svg

Using the approach I've shared above, this edge case (json-based spec without explicit badge extension) is still providing a json response, but at least now it is the correct json response.
{"label":"swagger","message":"valid","color":"brightgreen","link":[],"name":"swagger","value":"valid"}

I do not see a way to magically correct that original edge case as part of the redirector, so I believe this the best we can do. We knew that there would be a handful of edge cases that would be problematic when we updated the badge formats to .svg by default (and dropped the need to always explicitly include .svg).
Everything works fine when explicitly including the extension with the legacy routes, and everything will work fine going forward (since the url of the spec has been moved to the query param)

Also, I'd love to actually get some live service tests for this. If there's no publicly accessible specs that are relatively consistent (ones that will be valid or invalid for an extended duration) then we can have the tests point to a live spec with an assertion that uses Joi.allow('valid', 'invalid') for the message.
We can also add some tests for the not found scenario now that we're properly detecting them.

Now json should be usable with json format by default, svg needs to be specified in addition just as you said.
I also added live tests and tests for the redirector.
One live test I'm not sure though: The invalid spec I just used the schema for a spec, but its not a spec as such. (but valid json)


Allow SelectView::with_all to accept StyledString

Changes the bound on the iterator items from Into<String> to Into<StyledString>.
This brings the API in line with the other item methods,
which all use the same bound.

This change is

Thanks for the PR!


Is there any way to get the css.colorDecorators to work with Less/Sass Variables recursively?

Is there any way to get the css.colorDecorators square icons that represent the hex code colors to work with @import statements across files?  I currently have a "_colors.scss" file that I import into my master.scss for compiling.  The _colors.scss files shows all the icon colors as expected next to the hex codes, but in the master.scss if I use a color variable the icon to represent the color is not shown.
Example in _colors.scss
$my-purple: (ICON SHOWS HERE) #332D7B;

Example in master.scss
@import 'colors';
#hd1 {background: (ICON IS MISSING) $my-purple;}

Am I missing something or is there no way to display the icons recursively across files with Less/Sass Variables?

I see the issue referenced here: https://github.com/Microsoft/vscode/issues/21705
Any updates?


Enable custom swift package generate-xcodeproj command

Add ability to customize swift package generate-xcodeproj following the existing naming for swift build and swift test custom files.

Hey @na-gupta, can you share some details on how this would be useful from a CI build/test perspective? Is this just to test that an xcode project can be generated? If so, that sounds more like a SwiftPM test? Maybe I am just missing something.

In a new repo for MySQL we need to add -Xlinker -L/usr/local/lib to swift build, swift test as well as swift package generate-xcodeproj on macOS to point the linker to the correct location.
Otherwise, the subsequent xcodebuild -project $PROJECT ... test command which uses the generated xcodeproj fails with the message:
ld: library not found for -lmysqlclient for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)


Not able add threads to the thread group and miscount number of threads

I have followed the below steps to add more number of threads. But the thread count is increasing for the very first time. If we inject the thread second time, it is not working.

Run the script with one thread
Add threads e.g. 5
Thread count is increased to 5. (it is also a bug, the count should be 6)
Add 5 threads again
Thread count is still showing 5 even after waiting.

Here is the recording:

Yeah.. Thanks for confirming this; it's related to issue #18.
I'm actively working on this, hopefully I'm able to come up with a fix tomorrow.

Closing because it's related to #18


Deadlock when storing additional information like settlement amount

Hi all,
First of all, thanks for adding the payment to the events this week. 🙏
I've implemented a listener which updates the local payment like posted below.
Sometimes it works, sometimes it throws an error. I can image this is because there are database transactions not committed yet, right? Can you think of another way to get this done, after the database transactions have been committed?
SQLSTATE[40001]: Serialization failure: 1213 Deadlock found when trying to get lock; try restarting transaction

UPDATE
  `orders`
SET
  `mollie_payment_id` = tr_2uncgFCWdh,
  `mollie_payment_status` = OPEN,
  `processed_at` = '2023-02-10 10:10:44',
  `orders`.`updated_at` = '2023-02-10 10:10:44'
WHERE
  `id` = 26092

<?php

namespace App\Listeners\Subscriptions;

use Laravel\Cashier\Events\FirstPaymentPaid;
use Laravel\Cashier\Events\OrderPaymentPaid;
use Laravel\Cashier\Payment;

class UpdatePaymentDetails
{
    /**
     * Handle the event.
     *
     * @param  FirstPaymentPaid|OrderPaymentPaid  $event
     * @return void
     */
    public function handle(FirstPaymentPaid|OrderPaymentPaid $event)
    {
        // Get payment details.
        $payment = $event->payment;

        // Store extra data at our local payment.
        Payment::where('mollie_payment_id', $payment->id)
            ->update([
                'settlement_amount_currency' => ! is_null($payment->settlementAmount) ? $payment->settlementAmount->currency : 'EUR',
                'settlement_amount' => ! is_null($payment->settlementAmount) ? $payment->settlementAmount->value * 100 : 0,
                'method' => $payment->method ?? null,
            ]);
    }
}

Hello @voicecode-bv,
I'm sorry to hear that you're facing another problem with the recently implemented feature request. Currently, we're triggering the OrderPaymentPaid event within a database transaction call, which, as you've noticed, is not an ideal approach since it can lead to deadlocks.
I'll investigate the matter by the end of this week and make the necessary changes to dispatch the OrderPaymentPaid event outside of the transaction. Once this is done, your code should function as expected.
Meanwhile, you can work around the issue by dispatching the update of your Payment to the queue, which will execute the update at a later time. This should serve as a temporary fix for the problem.

Hello @voicecode-bv,
the fix was just released in v2.6.1


[@xstate/test] Missing config for done.invoke.fetch

Bug or feature request?
Bug?
Description:
When model-testing a machine with a Promise service and providing the test model with all explicit events, the tests fail with the message Missing config for event "done.invoke.fetch". Providing a blank event implementation allows the test to continue, but they never get past the state that invokes that service.
(Bug) Expected result:
Tests don't fail.
(Bug) Actual result:
Tests fail.
(Bug) Potential fix:
I would anticipate that internal service invocation events aren't required to be supplied to the test model. There is no meaningful action I can think of that will map to that event.
Link to reproduction or proof-of-concept:
https://codesandbox.io/s/xstate-missing-config-repro-i4xls
You'll have to click the server rack icon on the left side and run the yarn test script to see this in action.

I've experienced exactly the same issue.
The repo is: https://github.com/metamn/react-best-practices/tree/master/src/components/State
The message is:
 FAIL  src/components/State/State.test.js
  StateWithMachine
    ✕ coverage (1ms)
    reaches state: "closed"
      ✓ via  (32ms)
    reaches state: "opening"
      ✓ via OPEN (17ms)
    reaches state: "closing"
      ✓ via OPEN → CLOSE (10ms)
      ✕ via OPEN → done.invoke.openMenu → CLOSE (53ms)
    reaches state: "open"
      ✕ via OPEN → done.invoke.openMenu (9ms)

  ● StateWithMachine › reaches state: "closing"  › via OPEN → done.invoke.openMenu → CLOSE

    Missing config for event "done.invoke.openMenu".
    Path:
    	State: "closed"
    	Event: {"type":"OPEN"}

    	State: "opening"
    	Event: {"type":"done.invoke.openMenu"}

    	State: "closing"

      at TestModel.Object.<anonymous>.TestModel.getEventExecutor (node_modules/@xstate/test/src/index.ts:259:13)
      at TestModel.<anonymous> (node_modules/@xstate/test/src/index.ts:270:27)
      at step (node_modules/@xstate/test/lib/index.js:43:23)
      at Object.next (node_modules/@xstate/test/lib/index.js:24:53)
      at node_modules/@xstate/test/lib/index.js:18:71
      at Object.<anonymous>.__awaiter (node_modules/@xstate/test/lib/index.js:14:12)
      at TestModel.Object.<anonymous>.TestModel.executeEvent (node_modules/@xstate/test/lib/index.js:342:16)
      at Object.exec (node_modules/@xstate/test/src/index.ts:108:39)
      at TestModel.<anonymous> (node_modules/@xstate/test/src/index.ts:157:33)
      at step (node_modules/@xstate/test/lib/index.js:43:23)
      at Object.next (node_modules/@xstate/test/lib/index.js:24:53)
      at fulfilled (node_modules/@xstate/test/lib/index.js:15:58)

  ● StateWithMachine › reaches state: "open"  › via OPEN → done.invoke.openMenu

    Missing config for event "done.invoke.openMenu".
    Path:
    	State: "closed"
    	Event: {"type":"OPEN"}

    	State: "opening"
    	Event: {"type":"done.invoke.openMenu"}

    	State: "open"

      at TestModel.Object.<anonymous>.TestModel.getEventExecutor (node_modules/@xstate/test/src/index.ts:259:13)
      at TestModel.<anonymous> (node_modules/@xstate/test/src/index.ts:270:27)
      at step (node_modules/@xstate/test/lib/index.js:43:23)
      at Object.next (node_modules/@xstate/test/lib/index.js:24:53)
      at node_modules/@xstate/test/lib/index.js:18:71
      at Object.<anonymous>.__awaiter (node_modules/@xstate/test/lib/index.js:14:12)
      at TestModel.Object.<anonymous>.TestModel.executeEvent (node_modules/@xstate/test/lib/index.js:342:16)
      at Object.exec (node_modules/@xstate/test/src/index.ts:108:39)
      at TestModel.<anonymous> (node_modules/@xstate/test/src/index.ts:157:33)
      at step (node_modules/@xstate/test/lib/index.js:43:23)
      at Object.next (node_modules/@xstate/test/lib/index.js:24:53)
      at fulfilled (node_modules/@xstate/test/lib/index.js:15:58)

  ● StateWithMachine › coverage

    Missing coverage for state nodes:
    	(machine).open

      123 |    */
      124 |   it("coverage", () => {
    > 125 |     testModel.testCoverage();
          |               ^
      126 |   });
      127 | });
      128 |

      at TestModel.Object.<anonymous>.TestModel.testCoverage (node_modules/@xstate/test/src/index.ts:300:13)
      at Object.testCoverage (src/components/State/State.test.js:125:15)

Test Suites: 1 failed, 1 total
Tests:       3 failed, 3 passed, 6 total
Snapshots:   0 total
Time:        1.762s, estimated 2s
Ran all test suites related to changed files.

Watch Usage: Press w to show more.

Thanks a lot!

Thanks for the repro, gonna look into this.

By the way, for testing, you probably do not need to invoke services since instead you should model the services being invoked as states, where an event that transitions to a "done" state would be the resolution of that imaginary "service" (e.g., polling to wait that a loading indicator is no longer showing), so this has workarounds.
But I think the best solution would be to not error when event configuration for built-in events is missing.

you probably do not need to invoke services since instead you should model the services being invoked as states

I'm having trouble wrapping my head around that. It seems like I'd have to change the machine definition to make that work, but only in test environments. Can you provide a small example of what you mean?

A "test machine" could use such a strategy to implement "event executor".
The problem here (as per @metamn repro) is slightly different though. It tests an "implementation machine" which invokes external~ service and from what I understand in such case you don't want to provide an executor for that done event. But rather test model should be able to await that service "as normal".
It fails here:
https://github.com/davidkpiano/xstate/blob/6f883a1f9d5fe001d244e804d9c8b46b0fab0f5e/packages/xstate-test/src/index.ts#L177
because it's just a proxy to executeEvent:
https://github.com/davidkpiano/xstate/blob/6f883a1f9d5fe001d244e804d9c8b46b0fab0f5e/packages/xstate-test/src/index.ts#L128
So I think a solution (which would have to be implemented) would be to recognize those special events and await for service to be completed - and that if needed could be mocked externally to avoid real API requests in tests etc.
@davidkpiano is my understanding correct?

It tests an "implementation machine"

Yes, this is the important line. By using services and (assumedly) the real machine, you're testing the machine that actually implements the app, not the abstract machine that represents the behavior as a user would experience it. Big difference.

Yes, that's correct. From what we've discussed testing "implementation" machines might be OK in certain situations, right? And thus a created test model could support that.
I'm not sure though how to properly await invoked service right now though. Seems like we would have to recognize which ID has been invoked and await it through children/activities property of the machine?

Hello, I've stumbled across this issue and wonder whether there emerged any best practice yet on how to test with services. Could someone point me to relevant resources, please?


Text for Multiple File Imput

The file input's text box only shows the first file selected rather than the whole set.
<form action="didlysquat.php">

  <input class="file-path validate" type="text"/>

    Upload Files
    <input type="file" name="filesToUpload[]" id="filesToUpload" multiple/>

</form>

My proposed fix:
Use a similar function to this:
var fileFormID = document.getElementById('fileFormID');
for (var i = 0; i < fileFormID.files.length; ++i) {
// append: fileFormID.files.item(i).name; into an array
}

Instead of document.getElementById('fileFormID').value

Sounds like a good fix, could you provide a pull request?

This was merged into master #1654 a bit ago. Its not in the release version but if you are up to date with master it is there.


why the latest dagobah disable the job_on_start action?

Hi, thank you for your dagobah. I'am using dagobah for scheduler in pyhton, but the latest version hasn't the job_on_start event, which makes my code written using old dagobah should be changed on new version. So tell me why you do that. Thank you!

I can't find any reference to the event you're talking about in the Git history, could you provide an example?

ohh, it is my fault! I am very sorry, this event was added by myself six months ago, but forgot it.


Add OtherModuleName and OtherModuleErrorf to mutators

TopDownMutators can visit dependencies, provide OtherModuleName and
OtherModuleErrorf so they can query and report errors on those
dependencies.

LGTM


implementing Painforged's counterattack double damage bonus

Implemented the counterattack double damage bonus via an entry in ModParser-3_0.lua and updating the matching entry in ModCache.lua.

Cancelling as #1699 supersedes this.


Genotype conversion memory footprint. Covert to stream conversion?

Admittedly, my use case here is unusual, so this issue is unlikely to affect more users, but:
I have a very large dataset that I am trying to convert from EIG to PLINK using trident genoconvert. Submitting the job to a scheduler revealed that the job fails because it runs out of memory after processing part of the dataset.
My dataset is 154,353,735 SNPs long and 4GB of memory will only run through ~9M of these SNPs. Providing ~ 68GB of memory will probably get things working, but it seems a very hefty amount of memory to require for such a conversion. Is there a reason genotype conversion is done in memory instead of as a stream that writes the new genotype format as it works its way through the input without overloading the memory?
I can see that creating a potentially truncated output file is not a good idea, but what about streaming to a temporary file that then gets renamed?

I can confirm @TCLamnidis' observation: There is a space leak in genoconvert. It already consumes about 300Mb for a 1240K dataset. Either we missed that in the past, or we introduced this issue recently. forge does not leak.

This has been fixed through #188. It's now in master and will end up in the next release.


Bug with 10 or more virtualbox network adapters

We found an interesting edge case today when using this script with a VirtualBox instance that has more than 10 hostonly interfaces.   VirtualBox names the interfaces vboxnet1, vboxnet2...etc all the way to vboxnet10, vboxnet11.
Since it does not pad it's interface numbers with 01, 02 this allows the following line to grab too many IPs.
VBoxManage list hostonlyifs | grep "${prop_network_id}" -A 3 | grep IPAddress | cut -d ':' -f2 | xargs
This will grab multiple IPs because the grep clause will match on vboxnet1, vboxnet10, vboxnet11.  The way we were able to fix this is by changing the grep to be this, grep "${prop_network_id}$" -A 3
I will work on getting this to you in a PR.

Pull request in #72


docs: Fix broken links to CONTRIBUTING.md

Links to CONTRIBUTING.md were pointing to the .github directory, which didn't contain the file.

There are no changes at all here... Probably happened when you merged dev into your branch since I'm pretty sure this was already fixed recently.