Feature: Blog Icon

In external sites, we want to be able to display an icon for each blog. This is effectively the same problem as in issue #4869 - we need a static URL at which an external service such as an RSS feed or ghost.org can access to fetch a representative URL.
This icon should be used as the favicon for the blog. It should also be used when any external service looks for an icon. Browsers always look for a /favicon.ico file, at the root of the current domain. This means the behaviour of this file will always be slightly different for blogs which use a subdirectory. When rendering HTML, browsers use
Ideally, we want to serve a .png file. I'm not 100% on what we can or can't do here, and Googling didn't turn up much. Most browsers support a /favicon.png , and it'll be fine to use this for browsers & ghost.org, but to ensure we also capture the RSS issue, it may be worth 301-redirecting favicon.ico -> favicon.png for blogs which have an icon uploaded. This is an unknown, and will require a little bit of experimenting with. Any solution which works in feedburner should be considered acceptable for this task.
Image Requirements

Only permit PNG images (client + server?)
Enforce that the image must be square (server side only?)
Dimensions between 32px and 1000px (server side only?)
Size not larger than 100kb (server side only?)

Solution
Part A: Getting the icon uploaded:
The icon will be a 3rd image on the general settings screen in Ghost. The main difficulty here is implementing the image validation, to ensure the image meets our requirements. At least for now, this can most easily be done server side, I believe.

Add a new general setting in Ghost for blog icon
Add an additional image upload UI to the /settings screen
- The icon should only allow uploads, not urls - so the url feature needs to be disabled
Ensure that when uploaded the image is validated:

First check the filesize, and reject anything over 100kb, this means that further processing should at least be quick & easy!
Use our image-size utility to check that the size is square and within the right dimensions
If any check fails then fully detailed error messages should be returned explaining to the user exactly why an image is not suitable

The client will need to display any validation errors returned from the server.
This image will likely be small, we will need to ensure that the image upload modals in general/settings do show the trash icon!

Part B: Using the icon
Any blog that has an uploaded icon should use this instead of any theme favicons.
Note : the existing favicon implementation works as described in the asset helper documentation . It is odd & fiddly and needs overhauling. The existing implementation requires users to output their own meta tag, as Ghost 1.0 will need to output the meta tag itself, this will constitute a breaking change to the theme API.
Ghost will need an internal concept of what its "current favicon" is. This is possibly the 2nd MOST! tricky part of this whole implementation. In the case that a favicon has been uploaded, it's super easy, but otherwise, it requires some work to detect favicons in the active theme (and this must be refreshed when a theme is activated or overridden).
The ideal order of precedence for the favicon for a blog, is:

uploaded favicon
activetheme/assets/favicon.png
activetheme/assets/favicon.ico
activetheme/favicon.png
activetheme/favicon.ico
Ghost's favicon.png file (this is a new default, that needs to be added)
Ghost's favicon.ico file (not sure if we need this? maybe for older browsers)

Outputting the favicon in themes:
The {{ghost_head}} helper should be updated to output a new meta tag.
In the case that the "current favicon" is a .png file:
<link rel="shortcut icon" href="favicon.png" type="image/png" />
In the case that the "current favicon" is a .ico file:
<link rel="shortcut icon" href="favicon.ico" type="image/x-icon" />
Note : It would be good to experiment here with what will happen for themes that already have this output above {{ghost_head}} i.e. Casper.
**Note: ** I don't know if the {{amp_head}} should output something different?
As well as outputting the meta tag with {{ghost_head}} , we should also make sure that the icon url (e.g. favicon.png) is available as<EMAIL_ADDRESS>.
Outputting the icon on a URI
In order for the icon to work for external sites, we need to render the icon whenever it is accessed via a URI, e.g. /favicon.png
The existing favicon implementation uses a very odd piece of logic to serve any requests for /favicon.ico using our shared file helper , which is totally wrong for this usecase - this code should basically be deleted and forgotten about, and a new piece of middleware for favicons should be written.
The new middleware should live in the exact same place in the stack (e.g. after the two themeHandler middlewares) and the logic should be as follows:

The middleware should be aware of what the "current favicon" is.
If the "current favicon" is a .png file, ensure it is served from /favicon.png . Also, 301 redirect any request for /favicon.ico to /favicon.png (this may need testing with RSS readers to ensure it works).
If the "current favicon" is a .ico file (i.e. No icon was uploaded), ensure it is served from /favicon.ico . In order for this case to work for external sites, we also need to either 301 redirect /favicon.png to /favicon.ico OR just serve the Ghost default icon.

In terms of implementation, this middleware should be small and super fast, it's going to need to read the contents of the icon file, and then cache those contents.
Update: The existing favicon implementation assumes that it's ok to read from the file system, totally ignoring our storage adapter layer. The new solution needs a read method added to the storage adapters (see #7687) so that if a file has been uploaded, we are able to access it. If no file has been uploaded, then for now it is ok to read from the file system, as currently we only support local file system storage for themes & the shared files.
We then need to manage ensuring that the cache is cleared at the right times, so that if a user uploads a new logo, or deletes a logo, we correctly display the right one without requiring a restart of Ghost. Handling the caching is the 1st MOST! tricky part of this.
The rules are, to the best of my knowledge, as follows:

on first load, we need to figure out the "current favicon" and store a reference to it. That reference will need to include:

the path to the favicon file = where the file itself can be opened and read
a reference to the type, either upload, theme or default (default is when we serve Ghost's favicon)

For uploaded icons - the cache needs to be cleared when a new icon is uploaded, or the icon is deleted
For theme icons - the cache needs to be cleared whenever the assetHash is cleared
For default icons - the cache never needs to be cleared

Useful links
Some places to use for inspiration for how to write the file reading and caching logic are:

Ghost's shared file middleware
The main express serve favicon middleware (note this only supports favicon.ico)
Ghost's logic for refreshing the theme cache
Ghost's weird theme settings cache
Ghost's theme handling middleware which deals with the assetHash and express's undocumented view cache

There was a discussion about this in slack ages ago, but just updating requirements for the record:
The ideal order of precedence for the favicon for a blog, is:

Uploaded icon (png or ico)
activetheme/assets/favicon.png
activetheme/assets/favicon.ico
activetheme/favicon.png
activetheme/favicon.ico
Ghost's favicon.png file (this is a new default, that needs to be added)
Ghost's favicon.ico file (this is a new default, that needs to be added)

Ah good point - the theme ones were there because I thought this feature was going to land in LTS and I wanted to keep backwards compat 😁
Have added a note to the theme API changelog 👍

Left over TODOs for this issue:

[ ] clean up ghost_head_spec (we're still assuming that theme:icon is set to core/shared
[ ] make either the asset url for favicons absolute or the in ghost_head produced tag relative

[ ] test the browser behaviour with the currently two favicon links
[ ] test rss feed favicon behaviour
[ ] update theme docs
[ ] remove favicon tag from casper theme
[ ] optimize/investigate cache behaviour

I don't think this issue should be closed just yet? There are still a bunch of todos in the previous comment and I think we need to also add an item to update Casper ready for Ghost 1.0?
cc @AileenCGN I'm particularly interested in the outcome of the tests as I am not sure how much of a breaking change this is.

@JohnONolan (or maybe @AileenCGN already knows the answer to this):
With the addition of this new feature, what should we serve as the favicon for the admin panel? Should we serve the default Ghost icon always, or should this now also use the custom one if available?

Should use default Ghost icon always for admin
Also this issue should def not be closed till UX testing is complete. In fact any new features w/ UX testing needed probably best to assign to me for final pass design review rather than close?

@JohnONolan Roger on the icon. Also really like the idea of UX review 👍  cc @kirrg001 & @kevinansfield on that.

Yeah, had the UX feedback still open as a todo in the admin PR I made. But lost track of it, because it got merged already. Sorry about that. So, yes @JohnONolan: happy about feedback. You'll find it on master already and in general settings.
I also recommend to everyone to try it and test the browser and our cache behaviour. It worked fine for me and @kirrg001, but @kevinansfield had some caching issues in the browser.
Also, I never worked with RSS, so any testing/help/instructions are highly appreciated!

If the favicon for the admin interface is always going to be served as the Ghost icon (no issue with this btw) can it be served from the adminURL if this is set?
This would prevent the current issues on Ghost(Pro) where you visit your Ghost interface on ghost.io and your browser requests the favicon returning get a 301 -> http://your.domain if you've got a custom domain setup. This causes the page be marked as having SSL errors in firefox.

@jloh yar very good point - I'm revisiting this in conjunction with #6922, #7958, #6864 and #7491 etc, trying to iron out the assets story across themes vs admin.

@JohnONolan: This is how the upload for the blog icon looks like atm:

Error, when trying to upload wrong file extension:

Error, when file to small:

Error, when too large:

Finally, error when not square:

@AileenCGN

Admin Favicon should always be ghost, never custom
There should be a single error message for all cases of "Blog icon must be a square .ico or .png file between 32px –1,000px
The description of the field should be "A square icon for your publication"

Update about what's left to do to finish up this feature and also working towards the clean split of admin and server, which includes the usage of the {{asset}} helper:
Ghost Admin
The favicon in Ghost Admin needs to be the default Ghost favicon
Currently we’re requesting the favicon in Admin in two different places:

The favicon itself in general (which ends up in the browser tab as the symbol). We're currently loading the favicon from Ghost, but this can be a custom uploaded one.

Admin Favicon should always be ghost, never custom

It should definitely not change to a custom favicon. Safest and easiest way to do so, is to upload our default favicon in /assets/img for the Admin (see comments here as well).
There is a similar image in the Admin asset folder already: /assets/img/ghosticon.jpg
 but reg. #8221 the usage of it will be removed. I suggest to replace it with Ghosts new favicon.png.

The favicon which we’re requesting as a background image next to the user name here:

This, on the other hand could be either Ghosts default blog icon, or the custom uploaded one. The style (@JohnONolan)  needs an overhaul, as it's always showing a black background, which doesn't not look appealing when a logo like this is uploaded:

Testing and optimising all the things
There is definitely some weird caching behaviour (at least for me). Once an icon is uploaded, it refreshes the favicon in the browser tab, but it will only refresh the icon inside Admin only after requesting http://blog.com/favicon.ico and png and making sure that the right favicon gets rendered (sometimes needs a few refreshes) and then finally go back to Admin and refresh it. Voila. I guess this should be optimised, but I'm not sure if it's us or the browser.
We also need to test RSS feed behaviour (using ngrok) with our favicon URI as described here:

If the "current favicon" is a .png file, ensure it is served from /favicon.png . Also, 301 redirect any request for /favicon.ico to /favicon.png (this may need testing with RSS readers to ensure it works).

Ghost
One simple error message for blog icon upload

There should be a single error message for all cases of "Blog icon must be a square .ico or .png file between 32px –1,000px, under 100kb. "

Currently there are four different error messages implemented, which can be unified as one error that rules them all.
my.ghost.org
Usage of publication icon in my.ghost.org
That is the bigger part, which is left to do. To do this, we can refer to the URI of each blog (http://blog.ghost.io/favicon.ico), which should always return the favicon. Assuming this works fine, this feature shouldn’t really take long.

tl;dr TODOs:
Ghost-Admin

[ ] Serve and use favicon in Ghost Admin from /assets/img/favicon.png
[ ] Style adjustment for blog icon in Admin
[ ] Test favicon with RSS reader
[ ] investigate caching behaviour after blog icon upload

Ghost

[ ] Simplify error message (only one instead of four)

my.ghost.org

[ ] Implement publication icons

Little update on the test with RSS reader:
I used ngrok to add my development blog as content. After reading through the RSS board specs, I added an image property (see #8289) to it, but it would still not load the initial image. I tried it as well with other - definitely accessible - images with the same result.

Feedly seems to load the favicon from Google cache server... Just a thought, but maybe it's not possible to test it properly in dev environment?
cc @ErisDS

Yeah just do transparent bg in admin

Update on RSS reader behaviour:
Feedly uses the - what I found - deprecated but still working and - of course - not documented Google Service to get a converted (into png) version of a favicon from any website: http://www.google.com/s2/favicons?domain=website.com
This service is afaik pretty quick, so it would even serve the favicon for my localhost, which I ngrok'ed. Changing the favicon didn't changed it in the Google Service, tho. This might be related to the cache headers, I guess.
Newsblur and Feeder worked perfectly, except changing favicons, but again... cache.
Digg reader seems to go straight for the root favicon.ico but still wouldn't fetch the right favicon.

There are still some issues with the implementation here that I can see:
First is the image size requirement inconsistency that I mentioned here: https://github.com/TryGhost/Ghost/pull/8260#issuecomment-292722049
I'm not sure if that's me getting confused between icon and logo, but something doesn't seem to add up!
Second, is that after uploading a custom favicon it gets output by ghost head like this:
<link rel="shortcut icon" href="/content/images/2017/04/4448260-2.png" type="png">
The expectation (outlined in the original issue) is this:
<link rel="shortcut icon" href="favicon.png" type="image/png" />
Third, this line will only ever say that an icon is type x-icon if the file is called favicon.ico, rather than for instance if I upload a file called myicon.ico.
Finally, there seems to be a couple of places where logic is being done to determine a) what the icon url should be and b) what type it is (as in the previous item), when these should really be packaged up into a util.
I also think the code in serve-favicon.js can be simplified & the tests improved.

@ErisDS I fixed the issue in serve-favicon and improved the output in ghost_head with #8298.

Finally, there seems to be a couple of places where logic is being done to determine a) what the icon url should be and b) what type it is (as in the previous item), when these should really be packaged up into a util.

I also think the code in serve-favicon.js can be simplified & the tests improved.

I will do these things in a follow up PR now.

This feature has only one subtask left: Implement publication icons in my.ghost.org.
I will open a separate issue on my.ghost.org for it, so we can close this issue.


Remove WebhookConfig.Headers in favour of WebhookConfig.HTTP.HTTPHeaders

WebhookConfig was recently introduced with a top-level headers property. However it also contains an http property - a standard ffrestry.HTTPConfig object which already provides the facility for defining HTTP headers.
This PR removes the top-level headers property, and updates the web hook action to use HTTP.HTTPHeaders on the request instead.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5b53a57) 99.98% compared to head (1422330) 99.98%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage   99.98%   99.98%
=======================================
  Files          78       78
  Lines        6437     6439    +2
=======================================
+ Hits         6436     6438    +2
  Misses          1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Password filed in botbuilder

Hello Team,
I want to user with their credentials, for that i ask user to enter username and password. Is there any way to hide password / accepted user password in **** / ooooo similar formats. Please help me out on the same.

Hi @chetanmekha, there's no way to hide a password in chat. We discourage bot developers from requesting this information directly within the chat window.
One sample that shows how to log a user in is AuthBot: https://github.com/MicrosoftDX/AuthBot. Please take a look at the sample and ensure it meets your security requirements before using the code.

@chetanmekha @dandriscoll We are using Azure Multi Factor Authentication to secure a channel and verify the user with known data (in this case mobile number from Active Directory). Its works best when the user is trying to chat with a bot on mobile devices, the authentication experience is quite seamless.
You can read more about it here: http://www.acuvate.com/solutions/bot/

@dandriscoll thanks for the valuable guidance. Will implement the same.


Integrate fsharp aug 11 2019

31.0.0

Integrate dotnet/fsharp from 5a8f454a1 to 05c558a61
Notable changes include:

Removal of the Microsoft.FSharp.Compiler.SourceCodeServices namespace
A new API for determining if an identifier needs to be quoted is available: FSharp.Compiler.LexHelp.Keywords.DoesIdentifierNeedQuotation
Enhancements to the correctness of PDBs, especially with anonymous records
Better string formatting of records and values
More stack overflow fixes in the compiler
Inline IL parsing error handling
LeafExpressionConverter handles struct tuples
FSharpChecker now can parse a file without caching: ParseFileNoCache

Spurious build failures on weekly mono, merging despite them.


Language miscalculated

Not sure why, but https://github.com/AaronBratcher/ALBNoSQLDB is being shown as an Objective-C project when it only has 1 tiny .h file and several .swift files.

Actually, Linguist fails and recognizes the sqlite3.h file as Objective-C too. You might want to mark it as vendored.

I'm also having .h files reported as Objective-C (and some reported as C++).  It seems to be the case where headers are nested in the repo?
ex: include/foo.h registers as C, but include/data_structures/bar.h registers as Objective-C.
It's worth noting I have a .gitattributes file that contains: *.h linguist-language=C.

EDIT: Apparently this is an ongoing issue... #1626

Classification for such languages will never be perfect (Linguist rely on probabilistic methods as a last resort). You can override classifications using Linguist overrides.


Volume buttons on keyboard does not work

Before opening the issue, have you...?

[X] Searched for existing issues
[X] Looked through the wiki
[X] Updated MonitorControl to the latest version (if applicable)

Describe the bug
Nothing happens when using the volume keys on the keyboard or app slider. But the brightness slider works fine.
Recently, I updated my mac os. Before the update, everything was working fine.
Steps to reproduce

go to app
use the volume slider (nothing happens)
use the brightness slider (works fine)

Expected behavior
Volume to respond
Anything else?

macOS version: 15.2
Mac model: 2019 intel i7 macbook pro 16"
MonitorControl version: 4.3.3
Monitor(s): BenQ2480
Apple Silicon/M1 (yes or no): no

Environment Information (please complete the following information)
- macOS version:
- Mac model:
- MonitorControl version:
- Monitor(s):
- Apple Silicon/M1 (yes or no): no

Hi there, I am not sure what could be the issue. Does brightness control work between the 50-100% range as well (where DDC control is being used by default)? Is the audio output set to the display?


Definition list strange output

The first definition is not split in paragraphs if Text exists.
https://markdown-it.github.io/#md3={"source"%3A"Term 1\n%3A Par1.\n\n    Par2.\n\nTerm 2\n%3A Par1.\n\n\nText\n"%2C"defaults"%3A{"html"%3Afalse%2C"xhtmlOut"%3Afalse%2C"breaks"%3Afalse%2C"langPrefix"%3A"language-"%2C"linkify"%3Atrue%2C"typographer"%3Atrue%2C"_highlight"%3Atrue%2C"_strict"%3Afalse%2C"_view"%3A"html"}}

Migrated to another repo https://github.com/markdown-it/markdown-it-deflist/issues/2


Random seed of 0 accepted, yields same results as random seed 1

Summary:
According to help-all a random seed of 0 is invalid. Not only is it accepted, it seems to be equivalent to specifying a random seed of 1.
Reproducible Steps:
examples/bernoulli/bernoulli sample random seed=0 data file=examples/bernoulli/bernoulli.data.R output file=output-seed-0.csv
examples/bernoulli/bernoulli sample random seed=1 data file=examples/bernoulli/bernoulli.data.R output file=output-seed-1.csv
diff output-seed-0.csv output-seed-1.csv

Current Version:
v2.18.0

Happy to submit a PR if someone tells me what the desired behavior is.

I see the fix that's needed - the cmdstan argument parser allows any int value.
the fix is to add function "is_valid" to file 'cmdstan/arguments/arg_seed.hpp'
will do ASAP.

fixed by https://github.com/stan-dev/cmdstan/pull/695


inference_api.py not found error when using custom deployment template

Describe the bug
When deploying the custom deployment template defined here the deployment complains inference_api.py not found.
I think this is because the work directory is set to WORKDIR /workspace in Dockerfile: https://github.com/kaito-project/kaito/blob/e0f28f00776c349bae10f7cac18c21a628194e84/docker/presets/models/tfs/Dockerfile#L8
Meanwhile the template is expecting the file to be in the work directory:
https://github.com/kaito-project/kaito/blob/e0f28f00776c349bae10f7cac18c21a628194e84/docs/custom-model-integration/custom-deployment-template.yaml#L25
But the file is not in /workspace/inference_api.py but it is in /workspace/tfs/inference_api.py and /workspace/vllm/inference_api.py:
https://github.com/kaito-project/kaito/blob/e0f28f00776c349bae10f7cac18c21a628194e84/docker/presets/models/tfs/Dockerfile#L18
https://github.com/kaito-project/kaito/blob/e0f28f00776c349bae10f7cac18c21a628194e84/docker/presets/models/tfs/Dockerfile#L27
Steps To Reproduce
Follow the documentation for option 2: https://github.com/kaito-project/kaito/blob/main/docs/custom-model-integration/custom-model-integration-guide.md#option-2-build-a-custom-docker-image-with-model-weights
Expected behavior
Either provide a comment here for users to modify:
https://github.com/kaito-project/kaito/blob/e0f28f00776c349bae10f7cac18c21a628194e84/docs/custom-model-integration/custom-deployment-template.yaml#L25
Or provide some parameterization so that the "correct" inference_api.py is used.

Thanks for finding this! Yes we should be using tfs/inference_api.py as this file is using accelerate/transformers library. This file didn't account for new vllm release and is now updated (https://github.com/kaito-project/kaito/pull/757) thanks!


Can't upload videos longer than 30 seconds.

Please excuse me if this is not the correct place to post this question.
Using:
media = api.media_upload(path)
It often gives me the error that the video is longer than 30000 (ms) and that this is not allowed, even tough videos longer than 30 seconds are allowed on twitter.

This is not actionable, as there is no information here at all.
You need to provide what exactly you're trying and encountering.

Excuse me, first time posting an issue to another repo.
I made this simple script to reproduce the error:
`import tweepy
from fileReader import FileReader
import sys
reader = FileReader("./accounts/"+sys.argv[1])
account_data = reader.read()
auth = tweepy.OAuthHandler(
account_data['consumer_key'],
account_data['consumer_secret']
)
auth.set_access_token(
account_data['access_token'],
account_data['access_token_secret']
)
api =tweepy.API(auth)
media = api.media_upload("test.mp4")
tweetinfo = api.update_status(status="test", media_ids=[media.media_id])
`
The filereader class is just to import the relevant api keys and tokens in to the script. After that I try to upload a file called 'test.mp4'. Which is a file I made 5 minutes ago. It is a video with a still picture that is 41 seconds long and 1 mb in size. I can tweet this video without any issues manually, but when I try to do it using this script it returns this error:
tweepy.errors.BadRequest: 400 Bad Request 324 - Duration too long, maximum:30000, actual:41174 (MediaId: snf:1616970598772604930)

You need to pass the media_category parameter for API.media_upload as "tweet_video".
For code block usage, see https://docs.github.com/en/github/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks.


cowboy_static won't serve file with + in the path

Steps to reproduce:
Use the static_world example, add a+/index.html in priv/ and then try to get http://localhost:8080/a+/index.html
-> 404

I think it's the way the URL is parsed in cowlib and maybe '+' is considered as special. Although it seems it shouldn't in a fragment, according to https://www.ietf.org/rfc/rfc3986.txt
I'll check cowlib.

It's possible that the wrong urldecode function is used. One urldecode function has + become space, one doesn't. I don't remember offhand which is which.

Looks like both cow_qs:urldecode and cow_qs:parse_qs_name change $+ to space. I think it should be done for a query part fragment only, not for a path segment.

I believe you are correct.

This has been fixed a couple days ago and it's now tested. Thanks!


feat(icon): support new eslint config system

The new flat ESLint config system will only support eslint.config.js: https://eslint.org/docs/latest/user-guide/configuring/configuration-files-new#configuration-file

Fixes #IssueNumber
N/A
Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Thanks for the PR. There’s a format issue. Let’s fix it before merging.


Use full attitude range in gimbal controller

This convert -PI to PI attitude range to -1 to 1 for gimbal output.

Cherry-picked, thanks!


[ Pooling2D ] Pooling 2D Layer : Draft

This PR provides skeleton of pooling 2d layer
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #158. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


documentation for unobtrusive methods

This blog post was all I could find for official api documentation without wasting an hour reading the 400 line unobtrusive.js file in the projects. This example makes now sense without this link (perhaps the information should be reproduced here in case it's taken down).

Let me toss a few more out for consideration over the external link, since we control, validate, and maintain these ...

https://docs.microsoft.com/aspnet/mvc/overview/older-versions/creating-a-mvc-3-application-with-razor-and-unobtrusive-javascript#enabling-client-side-validation
https://docs.microsoft.com/aspnet/core/mvc/models/validation?view=aspnetcore-2.1#client-side-validation
https://docs.microsoft.com/aspnet/web-pages/overview/ui-layouts-and-themes/validating-user-input-in-aspnet-web-pages-sites#adding-client-side-validation
https://docs.microsoft.com/aspnet/mvc/overview/older-versions/hands-on-labs/aspnet-mvc-4-helpers-forms-and-validation#exercise-7-using-unobtrusive-jquery-at-client-side
https://docs.microsoft.com/aspnet/core/tutorials/first-mvc-app/validation?view=aspnetcore-2.1

... lol ... that was the easy part. 🏃 😅
Quick look ... number 2 looks good.

Brad Wilson was the lead developer for ASP.NET MVC when he wrote that blog. I think his blog would be hard to beat.

@user135711  Thanks for this very valuable contribution.


transparent background

It would be nice to be able to set transparent background or, if already possible, documented how to.
Or, maybe the background shouldn't be overridden with the default black, if that would be a way?

Hey, thanks for creating this issue! 🐻
You can do:
systeroid-tui --bg-color reset

Documented in 48e5ad6cc62fe498a73f8a7ece9580d0b68aeee2


Control character length of individual random values

Description
Cannot control length of randomly-generated value, which causes database insert errors where field length matters.
Proposed solution
If I want a randJobTitle() for example - it'd be nice if I could pass an option to limit the results by character length. I'm not talking about a simple substring, but rather, only return values that are of that length or less.
randJobTitle({ size: 50 }) - returns a random job title that is of 50 characters or less. The value "Senior Operational Administrative Assistant - Marketing" would not be selected but "Developer" would be. I'm making these values up of course, to illustrate. This would prevent an insert error if the field was varchar(50).
randJobTitle({ size: 50, length: 10 }) - would do the same, but would be an array of those w/ 50 characters or less.
Alternatives considered
No response
Do you want to create a pull request?
No

I would like to work on this issue !! if u can, can I do this?

Sure

But it's already have a PR - https://github.com/ngneat/falso/pull/316

Oh!
I thought only email can be sized because the example of email is written!
I misunderstood.
That's great!🚀

Merged in #316


HDFS-17515. Erasure Coding: ErasureCodingWork is not effectively limi…

Description of PR
https://issues.apache.org/jira/browse/HDFS-17515
How was this patch tested?
unit test
For code changes:

Add pendingECBlockReplicationWithoutTargets for DatanodeDescriptor
When construct ErasureCodingWork, increase pendingECBlockReplicationWithoutTargets. When set targets, decrease pendingECBlockReplicationWithoutTargets.
Because we can not decide the real source datanode when scheduleReconstruction, so update pendingECBlockReplicationWithoutTargets for all source datanode. So when we calculate getNumberOfBlocksToBeReplicated, use a factor to appropriately lower the value.

Not ready for review! Wait for HDFS-17516!


Create a README.md file

Describe what the repo is all about.
Add the tools used

Done the description and listed the tools


Use std::span as interface containers + some minor improvements

See each commit for details :)
Next steps:

Remove vector usage for command history
Remove map usage for descriptor print

Might implement: auto deduction of arg size based on commands so the buffer is always the exact size it needs to be. :white_check_mark: Implemented in: 575850d (#17)

@mads-bn can you re-review? I've updated the example and c++20'ed the tests as well.
Also added auto argument buffer size deduction based on provided commands.


Manage boost by tools/install-dependencies script

Make sure we have same boost version across macOS / Linux / Android / iOS, Wasm could be different for now; This should also solve the issue when you use Android Studio + M1 MacBook

@Milerius does this issue still make sense?

Not so much benefits without caching - closing for now.

current approach has inconsistent boost versions, any idea to solve this?

Current approach has inconsistent boost versions, any idea to solve this later? I realize we don't need cache boost, only use pure headers

We can always specify the version in apt-get or brew, my long term goal is to get ride of boost dependencies by using only the standard.

sounds good, we're using boost/multiprecision, hex and base64, not that much


RuntimeError: output with shape [1, 128, 64] doesn't match the broadcast shape [3, 128, 64]

I tried to run deep_sort/deep/feature_extractor.py, but there was a problem.
Traceback (most recent call last):
File "D:/目标检测各模型/deep_sort_pytorch-master/deep_sort/deep/feature_extractor.py", line 56, in
feature = extr(img)
File "D:/目标检测各模型/deep_sort_pytorch-master/deep_sort/deep/feature_extractor.py", line 44, in call
im_batch = self._preprocess(im_crops)
File "D:/目标检测各模型/deep_sort_pytorch-master/deep_sort/deep/feature_extractor.py", line 39, in _preprocess
im_batch = torch.cat([self.norm(resize(im, self.size)).unsqueeze(0) for im in im_crops], dim=0).float()
File "D:/目标检测各模型/deep_sort_pytorch-master/deep_sort/deep/feature_extractor.py", line 39, in
im_batch = torch.cat([self.norm(resize(im, self.size)).unsqueeze(0) for im in im_crops], dim=0).float()
File "F:\softmanage\Anaconda3\lib\site-packages\torchvision\transforms\transforms.py", line 61, in call
img = t(img)
File "F:\softmanage\Anaconda3\lib\site-packages\torchvision\transforms\transforms.py", line 166, in call
return F.normalize(tensor, self.mean, self.std, self.inplace)
File "F:\softmanage\Anaconda3\lib\site-packages\torchvision\transforms\functional.py", line 208, in normalize
tensor.sub(mean).div(std)
RuntimeError: output with shape [1, 128, 64] doesn't match the broadcast shape [3, 128, 64]
can you help me ? thanks!!!

Hi!
It is because the Extractor expects list of images, however it is called with a single image
Change this:
feature = extr(img)
to this:
feature = extr([img])
Note that this does not affects the behaviour of the whole algorithm since running feature_extractor.py only is just for testing.


enhance unetr trainer

Signed-off-by: ahatamizadeh<EMAIL_ADDRESS>This PR enhances UNETR trainer and addresses #17.

Hi @wyli
I would appreciate your review of this PR. It basically improves the trainer by changing the way best accuracy is kept tracked off. I have already tested this.


deps: bump SDK to v0.47.1

Description

Waiting on this PR in interchaintest to be merged.
closes: #XXXX
Commit Message / Changelog Entry
deps: bump SDK to v0.47.1

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

I am a bit stuck here... I bump interchaintest to the latest (v7.0.0-20230417164426-9b8f7ae9bf1d) but I still get this error when I do go mod tidy:
github.com/cosmos/ibc-go/e2e/testconfig imports
        github.com/cometbft/cometbft/types imports
        github.com/cometbft/cometbft/crypto/secp256k1 imports
        github.com/btcsuite/btcd/btcec/v2/ecdsa tested by
        github.com/btcsuite/btcd/btcec/v2/ecdsa.test imports
        github.com/btcsuite/btcd/chaincfg/chainhash: ambiguous import: found package github.com/btcsuite/btcd/chaincfg/chainhash in multiple modules:
        github.com/btcsuite/btcd v0.22.0-beta<EMAIL_ADDRESS>        github.com/btcsuite/btcd/chaincfg/chainhash v1.0.1<EMAIL_ADDRESS>
interchaintest was having the same issue, but in the latest commit that was fixed and it works now. But I am lost why we have this error in ibc-go...

I think the problem is fixed! Thank to @misko9 for the solution to the puzzle!

looks like tests are still failing

looks like tests are still failing

I know :) This is now a different problem.

Looking a bit more into the failures, I attempted running the tests using a simd from main, but with interchain test pointing at sdk 0.47.1 and ran into the same account sequence mismatch error. Looking at the SDK changes I do suspect it has something to do with either CLI args (used by interchain test) or the clientContext which is also used in interchaintest is the BroadcastTx function which calls the sdk BroadcastTx.

After some more debugging with @charleenfei , we managed to figure it out. The problem was because of the changes in this PR.
Specifically
	initNum, initSeq := fc.accountNumber, fc.sequence
-	if initNum == 0 || initSeq == 0 {
+     if initNum == 0 && initSeq == 0 {
		num, seq, err := fc.accountRetriever.GetAccountNumberSequence(clientCtx, from)
		if err != nil {
			return fc, err
		}

-		if initNum == 0 {
-			fc = fc.WithAccountNumber(num)
-		}
+ fc = fc.WithAccountNumber(num)
+ fc = fc.WithSequence(seq)
-
-		if initSeq == 0 {
-			fc = fc.WithSequence(seq)
-		}
	}

In interchain test, we were accidentally not explicitly passing the account sequence when creating the tx Factory. As a result, the || conditional in the above code made it so that we correctly got the sequence.
I will create a follow up PR in interchain test to fix this issue there.
I believe there should be a check in the sdk that ensures if exactly one of the account / account sequence is zero, it should error out.

Fantastic work y'all!


Updated Swedish translation

Hello,
I have added new labels and improved some of the translations...
BR
/YraFyra

Thanks for the update, tack för ditt bidrag!


Upgrade for Laravel 9.x

Add support for Laravel 9.x & remove support for older Laravel versions.
I recommend to tag this as a major release.

Thanks


fix: Protocol should use a space-efficient serialization format

What is the bug?
Farcaster v2 currently uses FlatBuffers in most protocol messages and RPC. While FlatBuffers is extremely fast to decode, its wire format is indeed bigger than alternatives such as Protobuf. Since hub's storage is a scarce resource in Farcaster, the serialization format chosen for persisted data (such as signed protocol messages) should be as space-efficient as possible.
How can it be reproduced? (optional)
I wrote a benchmarking project for Protobuf and FlatBuffers on Go (because Node is too slow for a meaningful benchmark, and Go is a popular choice for implementing decentralized protocols). The test data is a Farcaster's Message.
It shows that the differences in encoding performance is tiny, but the wire format size is substantially larger (+40%).
https://github.com/kcchu/buffer-benchmarks

Test
Protobuf
FlatBuffers
Capnp

Encode (ns/op)
882.7
859.0
2541

Decode (ns/op)
1165
18.36
1726

Wire format size (bytes)
299
432
344

Wire format size, gzipped (bytes)
323
406
368

Additional context (optional)
The bigger wire format of FlatBuffers is mostly due to its use of vtable, field alignments and constant size integers.
Besides space-efficiency, FlatBuffers have other drawbacks:

Security - FlatBuffers's use of vtable could easily lead to security vulnerabilities on memory unsafe languages if the incoming messages were not verified carefully. Some FlatBuffers libraries can verify the vtable, but this would eliminates any advantages on decoding speed. And even if verification is done, DoS may still be possible with maliciously crafted messages containing circular pointers. (Reference)
Fewer support for gRPC+FlatBuffers: While gRPC+FlatBuffers is possible by gRPC specification, there is very scarce open source support for this. With gRPC+Protobuf, server stubs and clients can easily be generated from proto file for many programming languages. With gRPC+FlatBuffers, server stub and client have to be written manually.
Tedious message construction - Building of FlatBuffers messages is difficult. As a result, extra hand-written code is maintained to build the messages.

Thanks for doing the benchmarking! I'll certainly take a look at that. It's worth noting though that the flatbuffers decode performance was a large part of why we chose it, because it means that hubs can validate and resolve conflicts between encoded messages much faster than they could with a framework that required deserialization.

These are all very good points. We've done a lot more benchmarking over the last couple of days that confirm the benchmarking you already did and are evaluating what it would take to make the switch to protobufs.

For the sake of planning, here's a list of tasks we'd need to do in order to migrate to protobufs.
Messages and contract events migration

[ ] Add @farcaster/protobufs package with schemas and static code
[ ] Port message schemas to protobufs (message.fbs)
[ ] Port contract event schemas to protobufs (id_registry_event.fbs and name_registry_event.fbs)
[ ] Add factories for protobufs messages and events in @farcaster/utils
[ ] Add verifications for protobufs in @farcaster/utils
[ ] Add signers that use protobufs SignatureScheme rather than flatbuffers to @farcaster/utils

Storage engine migration

[ ] Update stores to use protobuf messages and events
[ ] Port job-related schemas to protobufs (job.fbs)
[ ] Update jobs to use protobufs
[ ] Update storage engine to use protobuf messages and events

gRPC migration

[ ] Port gRPC schemas to protobufs (rpc.fbs)
[ ] Update gRPC server implementations
[ ] Update gRPC client implementations

Eth migration

[ ] Update Eth events provider to use protobufs

Gossip and sync migration

[ ] Port gossip schemas to protobufs (gossip.fbs)
[ ] Update p2p classes and gossip factories to use protobufs
[ ] Update sync engine to use protobufs
[ ] Update sync console to use protobufs

Hub script migration

[ ] Port hub state schemas to protobufs (hub_state.fbs)
[ ] Update hub executable and cli to use protobufs

If I can help doing smaller tasks in this epic, I‘m happy to send some PRs.

Closing due to https://github.com/farcasterxyz/hubble/pull/480


_change Not Triggered When Using Date/Time Pickers

Describe the bug

_change event is not triggered for date inputs when using the pickers

What is the level of criticality

Describe the level of impact that this bug has on your service.

To Reproduce (include pictures or video if helpful)
Steps to reproduce the behaviour:

Click on the date or time input trailing icon
Select a date or time

Expected behaviour

Picking a date or time using the picker should trigger a _change event

System configuration
Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox (91.6 ESR), Brave (1.38.76), Chrome (100)

Duplicate of: https://github.com/GovAlta/ui-components/issues/492

:tada: This issue has been resolved in version 3.4.0-alpha.60 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0-alpha.54 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0-alpha.8 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


added Devpost Scraper

Description

The Devpost scraper will scrape all the projects and return the latest projects along with their decription, like and commment count, image and member details.

Resolves: [#580]
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

@nikhil25803 please review this PR


Revert "Revert "fix: insert hostedUIProviderCreds empty array on hostedUI""

Reverts aws-amplify/amplify-cli#6682. Reintroducing this PR to fix issue with HostedUIProviderCreds not found and skipping prompting the users for providing the Provider Secrets on amplify pull and amplify env checkout

Closing this in favor of https://github.com/aws-amplify/amplify-cli/pull/6739


[typescript] TS4023:Exported variable '...' has or is using name 'StyledComponentProps' from external module "..." but cannot be named.

When creating declaration files ("declaration": true in tsconfig.json) for a component which uses withStyles the typescript compiler complains with :
TS4023:Exported variable 'MyComponent' has or is using name 'StyledComponentProps' from external module "./node_modules/material-ui/index" but cannot be named.
See https://github.com/Microsoft/TypeScript/issues/9944 for more context on the issue

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behaviour
Type inference to handle the issue.
Current Behavior
The compiler issues and error. A work around is to import StyledComponentProps
Steps to Reproduce (for bugs)
import {withStyles} from 'material-ui/styles';

const decorator = withStyles(({ spacing }) => ({
  root: {
    padding: spacing.unit * 2
  }
}));

const Example = decorator<{}>(({classes}) => (

));

export default Example;

and set "declaration": true in tsconfig.json
To work around add import {StyledComponentProps} from 'material-ui'; to the top of the file
Context
I am creating a node package of shared MUI components for internal use and exporting them (hence using declaration files)
Your Environment

Tech
Version

Material-UI
1.0.0.beta-17

Typescript
2.4.2

Per the link you gave, this is a limitation/quirk of TypeScript. AFAIK there's nothing that can be changed about material-ui that would ameliorate it, is there?

Let me know what you think of my dodgy solution :neutral_face:

Carrying on the discussion from https://github.com/callemall/material-ui/pull/8771#issuecomment-338310349
I agree that since the main use case of material-ui is not for use in libraries it would be adding complexity to material-ui to solve this is not a good idea.
I'm building libraries and this problem is an issue for almost anything I create or use. If anyone else is having the same issues then my solution is to wrap the material-ui components you have to use often, redefine the types with reference to the originals and in such a way that it has no need for external dependencies. This keeps the complexity in one place and can easily be removed in the future.


Improve dockerfile to leverage container caching

This improves the dockerfile by moving stuff around.
By adding the dependencies before the application, we do not need to run npm install every time when dockerizing the app, since package.json has not been updated then. To be honest, we probably only need to run the last three commands now, since the remainder can be cached perfectly
@alexnederlof ready for review!

:+1:


Fix doc commit script 2

The fix in #1034 got a little further in the document commit job, but there was another mistake: the documentation lives in the top-level directory not inside docs. This PR should fix it. Also fix for first-time creation of the directories if they do not exist.

The script almost runs to completion. It needs permission to push to the repo:
+ git push -u origin gh-pages
ERROR: The key you are authenticating with has been marked as read only.
fatal: Could not read from remote repository.

How does the pytorch/pytorch job get permission to write?

Answering my own question: it is here. Someone with admin status needs to make a GITHUB_PYTORCHBOT_TOKEN for the repo, then this PR will need extending again.

@seemethere @soumith Could you provide a guidance here regarding the push permission?


Problems identified @ by webstorm package manager when selecting Ni

I use the package manager Ni in webstorm. When there is @ in the directory folder, I can't recognize it, as shown in the following figure

My current solution is to copy all the files under @ antfu / Ni to the Ni folder so that they can be used

But this is only a temporary solution. It is not convenient for me to manage it (such as future upgrade and other operations)

I can't determine whether this is a webstorm problem or a Ni problem, but what I want is if the released package doesn't bring @. That is, the released package is not private. Can this problem be solved?

webstorm Node Package Manager  There is no example of @ in the package manager in the document


Error PostgreSQL::ColumnDefinition

This solve the error in rails 5.1:
Bundler::GemRequireError: There was an error while trying to load the gem 'activerecord-jdbcpostgresql-adapter'.
Gem Load Error is: uninitialized constant ActiveRecord::ConnectionAdapters::PostgreSQL::ColumnDefinition

sorry but this breaks the intention of the code (AR40 means >= 4.0) and breaks existing (AR < 5) support
please have a look at rails-5 branch. thanks for your effort ...


App made fully customisable through styles.xml

@creativepsyco I think I have made everything customisable through style sheet. Please review and tell me if I have missed out anything. Thanks.

@creativepsyco please review and merge.

You have a separate commit for some other issue as well, can u separate it and file a PR for it?

Sure.

@creativepsyco Done. Please review now.

Hi @mananwason
The styles don't follow the correct inheritance guidelines for Android. Please read the [Android Guidelines][1] on themes and styles, and how you should structure them, using proper scopes for inheritance purposes.
Some quick guidelines:

Dont use . without understanding what it does.
Use the parent attribute when the child style name is completely different, e.g. Custom.Text should not need a parent it already has one, i.e. Custom is the parent here, and Custom.Text is the style you are defining.
Use proper meaningful names, dont have stuff like MyToolbarStyle, makes it look it, its been copied from somewhere
Don't use view class names as Style parents or style names.
Also check that parent styles are already defined.

Fix it up and I will check again.
Cheers,
[1]: http://developer.android.com/guide/topics/ui/themes.html

okay @creativepsyco I'll fix everything and get back to you


[PSET] Input / Output

A standalone pset_input.go into pset package must be created collecting the methods for creating, validating and (de)serializing psbt inputs.
The same should be done for partial outputs, by creating pset_output.go.

Reference implementaion https://github.com/btcsuite/btcutil/blob/master/psbt/partial_input.go.


No JSON object could be decoded

I have put acme-dns into a Docker container with exposed ports 53 and 443. I get response for requests on 53/UDP but I get no response on 443/TCP. All I get is "No JSON object could be decoded"
I have the following (DEFAULT) settings in the api section of config.cfg
 [api]
 # domain name to listen requests for, mandatory if using tls = "letsencrypt"
 api_domain = ""
 # listen port, eg. 443 for default HTTPS
 port = "443"
 # possible values: "letsencrypt", "cert", "none"
 tls = "none"

Any hint, where to look into?

Unfortunately I'm unavailable to reproduce the problem. Could you post the HTTP request you are sending in?
Does this work:
curl -X GET http://your-container-ip:443/register
how about:
curl -X POST http://your-container-ip:443/register --data '{"allowfrom": ["<IP_ADDRESS>/32"]}'

I have tried a couple of POST /GET but no success
`root@docker ~]# curl -s -X POST https://eknori.blogsite.org:443/register |python -m json.tool
No JSON object could be decoded
[root@docker ~]# curl -s -X POST https://eknori.blogsite.org/register |python -m json.tool
No JSON object could be decoded
[root@docker ~]# curl -s -X POST https://<IP_ADDRESS>/register |python -m json.tool
No JSON object could be decoded
[root@docker ~]# curl -s -X POST https://<IP_ADDRESS>:443/register |python -m json.tool
No JSON object could be decoded
[root@docker ~]# curl -s -X POST https://<IP_ADDRESS>:443/register  --data '{"allowfrom": ["<IP_ADDRESS>/32"]}' |python -m json.tool
No JSON object could be decoded
[root@docker ~]# curl -s -X POST https://<IP_ADDRESS>:443/register  --data '{"allowfrom": ["<IP_ADDRESS>/32"]}'
[root@docker ~]# curl -X POST https://<IP_ADDRESS>:443/register  --data '{"allowfrom": ["<IP_ADDRESS>/32"]}'
curl: (35) SSL received a record that exceeded the maximum permissible length.
[root@docker ~]# curl -X GET https://<IP_ADDRESS>:443/register  --data '{"allowfrom": ["<IP_ADDRESS>/32"]}'
curl: (35) SSL received a record that exceeded the maximum permissible length.
[root@docker ~]# curl -X POST https://eknori.blogsite.org/register |python -m json.tool
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   167  100   167    0     0    826      0 --:--:-- --:--:-- --:--:--   830
No JSON object could be decoded
[root@docker ~]# curl -X GET https://eknori.blogsite.org/register |python -m json.tool
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   159  100   159    0     0   1121      0 --:--:-- --:--:-- --:--:--  1127
No JSON object could be decoded
[root@docker ~]# `

Ah, I see. The "No JSON object could be decoded" - message is printed by Python, indicating that it recieved anything but a JSON object. To help with debugging, try leaving the |python -m json.tool out, all it does is pretty printing after all.
What comes to the underlying error, I guess curl tries to communicate with acme-dns using HTTPS, but acme-dns configuration value means that it speaks plain HTTP:
 # possible values: "letsencrypt", "cert", "none"
 tls = "none"

I suggest testing it out with:
curl -X POST http://eknori.blogsite.org:443/register
note http substitution in the protocol part of the URL.

sorry, does not work either. I have also tried with port 80 exposed, but it gives me
curl: (7) Failed connect to <IP_ADDRESS>:80; Connection refused
So, I think, that I'l have to create a self signed cert and retry with that.
what I got from my latest tests is
`[root@docker ~]# curl -X POST https://eknori.blogsite.org/register
Unable to Process RequestHttp Status Code: 404Reason: Unable to process request, resource not found[root
@docker ~]# curl -X POST https://eknori.blogsite.org/register
Unable to Process RequestHttp Status Code: 404Reason: Unable to process request, resource not found[root
@docker ~]# curl -X GET https://eknori.blogsite.org/register
Unable to Process RequestHttp Status Code: 404Reason: File not found or unable to read file[root@docker
~]# curl -X POST http://eknori.blogsite.org:443/register
curl: (52) Empty reply from server`
Looks like acme-dns is looking for the files in /etc/tls but cannot find any.

Unfortunately this looks like some other service answering the requests, you might be able to get more information from the HTTP headers by adding -v (verbose) parameter to curl command line.
acme-dns doesn't give HTTP answers (.... part there), so it must be some other server getting the requests in the endpoint. The connection refused when trying port 80 suggests that there's some kind of firewall in between or port hasn't been exposed from Docker host.

Bummer. Yes. Need to use the IP because blogsite.org is pointing to another server
Now I get
`curl: (35) SSL received a record that exceeded the maximum permissible length.
[root@docker dns]# curl -v -X POST  https://<IP_ADDRESS>:443/register

About to connect() to <IP_ADDRESS> port 443 (#0)
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 443 (#0)
Initializing NSS with certpath: sql:/etc/pki/nssdb
CAfile: /etc/pki/tls/certs/ca-bundle.crt
CApath: none
NSS error -12263 (SSL_ERROR_RX_RECORD_TOO_LONG)
SSL received a record that exceeded the maximum permissible length.
Closing connection 0
curl: (35) SSL received a record that exceeded the maximum permissible length.`

Check my comment https://github.com/joohoi/acme-dns/issues/13#issuecomment-328542535 the issue is that you have configured acme-dns to talk plain HTTP but curl assumes HTTPS, because of https:// - part in the URL. Should work fine with http://yourip:443/register
Naturally either enabling TLS or moving to another port would be a bit less confusing :)

AHHHHH. Got it
`curl -v -X POST  http://<IP_ADDRESS>:443/register

About to connect() to <IP_ADDRESS> port 443 (#0)
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 443 (#0)

POST /register HTTP/1.1
User-Agent: curl/7.29.0
Host: <IP_ADDRESS>:443
Accept: /

< HTTP/1.1 201 Created
< Content-Type: application/json; charset=UTF-8
< Vary: Origin
< Date: Mon, 11 Sep 2017 16:00:32 GMT
< Content-Length: 243
<

Connection #0 to host <IP_ADDRESS> left intact
{"allowfrom":[],"fulldomain":"935a8902-cf96-4a65-90b6-132076192af0.eknori.blogsite.org","password":"zWEhjPvYHC0rZffY3NFqZYuAugF9DWwpnfMT2_Sh","subdomain":"935a8902-cf96-4a6
5-90b6-132076192af0","username":"5c50d03f-71c5-49a5-bba5-c48ba5a1a9ad"}[root@docker dns]# `

Closing the issue as resolved. If you run into other problems while installing, feel free to open new issue!


Revert Landing Page back to normal

@iterative/websites We need to revert the landing page to how it was before.  Question though.  Would it be possible to keep the modal if it can auto feed what's in the blog?  If it's something we have to keep changing, then nevermind, just revert.  But if it can be auto populated with what's the latest blog post, it may be worth keeping.

@iterative/websites We need to revert the landing page to how it was before. Question though. Would it be possible to keep the modal if it can auto feed what's in the blog? If it's something we have to keep changing, then nevermind, just revert. But if it can be auto populated with what's the latest blog post, it may be worth keeping.

I'll revert the video! As for the the model, it should be doable. I'll look into it!


How to pull Euro denominated data for multiple firms?

Just wondering how to obtain Euro denominated data when I want to retrieve data drom multiple firms and for multiple items, e.g. for example for the request below:
res = DWE.fetch(['@AAPL','U:MMM'], fields=['P','MV','VO','PH'], date_from='2000-05-03')
print res['MV'].unstack(level=0)

I tried to shoe in ~~EUR , but to no avail ...

Hello Martien,
I would suggest to contact TR service desk and ask how the request should look like in this case - and then I could have a look how to do this via fetch in the pydatastream.
Best
Vladimir

                                                                                Will do, I am currently running a trial of Eikons, they happily support me.                                                                                Sent from my BlackBerry Classic smartphone | M: +64 21 134 7500 | M: +44 774 558 4224 | H: +44 1524 888 914 | M: +316 8155 8570 | BBRY PIN 2BFBFAF3 |Zoom: 848 342 9576 | No Need to Hit the 'Send' Key. Just Talk to Me.                                                                                From: Vladimir FilimonovSent: Sunday, 16 July 2017 07:33To: vfilimonov/pydatastreamReply To: vfilimonov/pydatastreamCc: Martien Lubberink; AuthorSubject: Re: [vfilimonov/pydatastream] How to pull Euro denominated data for multiple firms? (#13)Hello Martien,

I would suggest to contact TR service desk and ask how the request should look like in this case - and then I could have a look how to do this via fetch in the pydatastream.
Best
Vladimir
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/vfilimonov/pydatastream","title":"vfilimonov/pydatastream","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/vfilimonov/pydatastream"}},"updates":{"snippets":[{"icon":"PERSON","message":"@vfilimonov in #13: Hello Martien,\r\n\r\nI would suggest to contact TR service desk and ask how the request should look like in this case - and then I could have a look how to do this via fetch in the pydatastream. \r\n\r\nBest\r\nVladimir"}],"action":{"name":"View Issue","url":"https://github.com/vfilimonov/pydatastream/issues/13#issuecomment-315556824"}}}

Hello Martien, were there any response from TR?

Is there any solution for this issue?

I suspect that not, and possibly will not be - it looks like Thomson Reuters is moving to the new REST API anyway...
As it was mentioned by Andrew Han in https://github.com/vfilimonov/pydatastream/issues/17#issuecomment-410558525:

I just got off the phone with some of their support folk. They said that they are phasing out access to Datastream via DataWorks Enterprise, and that we should use Datastream Web Services (DSWS) now.


Make sound commands cancelable.

Don't use a subshell to tokenize the commands. Using the subshell breaks
terminate() and kill() - they would only stop the subshell.
See #316
The downside of this approach is that once we call terminate(), we'll leave defunct subprocs:
[beep] <defunct>

Those go away, when the parent terminates. Once thing I also noticed that it is not a good idea to kill() a beep, this will beep forever (how?) until you play another short beep.

Given that there are many applications where beeping is done constantly, I'd be hesitant to allow processes to hang around. What we really should be doing is beeping through proper APIs instead of hacking a textual command together :wink:

@WasabiFan the processes only hang around if you cancel the beep.

@WasabiFan the processes only hang around if you cancel the beep.

If we're going to enable this functionality, people are going to use it.

Could we try the solution from here? http://stackoverflow.com/questions/12354586/python-what-are-the-nearest-linux-and-osx-equivalents-of-winsound-beep

@WasabiFan it is not a problem of the beep command, but how we handle it. I have an idea how to avoid the zombie issue. Let me try ...

I'm not specifically focused on this issue of processes left behind; I would much rather replace the hacks with proper calls than put lipstick on a pig.

Beep is the right tool for the job, we also use the same technique for playing wavs and the speech synth. There are no python solution for this available (pyaudio is terrible, trust me). I've fixed the zombie process issue - if you to cmd.terminate(), you need to also call cmd.wait() to get the return value and then the zombie is gone.

Ok, that's better; what do you think about returning a wrapper object which simplifies usage? That would make this easier to document in a useful way.

What do you mean by Wrapper object? All those methods return Popen objects. Those have a decent interface for being an async operation.
Also I would rather separate changing the interface and fixing unexpected behavior.

What do you mean by Wrapper object?

Maybe this is just a result of my background in more OOP-focused languages:
class SoundPlayResult:
    def __init__(self, popen_obj):
        self.popen_obj = popen_obj

    def cancel():
        self.popen_obj.terminate()
        self.popen_obj.wait()

Then we could add additional methods/properties for future additions and whatever other useful attributes the Popen objects have. If we later replaced the Popen calls with something more direct, we could change the internals of this class and the interface would remain unchanged.

Also I would rather separate changing the interface and fixing unexpected behavior.

Sure, we should get a review from @ddemidov. I had you as a captive audience so I figured I'd try :wink:

I see Popen with shell=True is also used in set_volune. It is different in the sense that it waits for the command completion, but may be we should switch to shlex splitter here as well for consistency?
Otherwise, the changes look good. I am on a fence regarding Popen wrappers suggested by @WasabiFan. Wrapping the returned Popen objects so that wait() is called automatically is nice, but Popen objects probably have other functionality that we don't want to loose. Of course, we could solve this by making the wrapper a subclass of Popen, but it seems cleaner to just return the result as Popen instance.

Also changed set_volume().


PyramidFlow训练demo会开放吗

同问

Yes, we plan to release an improved training code in another repository in the future, which will allow for reproducing the paper results. This repository will remain user-friendly and easy to use. It is a collection of models (like timm) but does not include other stuff (like training code). Furthermore, we intend to maintain this repository with other SOTA normalization flow models to increase the influence of this methodology.

Yes, we plan to release an improved training code in another repository in the future, which will allow for reproducing the paper results. While the autoFlow repository will remain user-friendly and easy to use. It is a collection of models (like timm) but does not include other stuff (like training code). Furthermore, we intend to maintain this repository with other SOTA normalization flow models to increase the influence of this methodology.

具体什么时候开放呀

在六月中旬的main conference前。


Generic Aries WASM - Externalize start up opts

https://github.com/trustbloc/edge-agent/blob/054adb2dd98791c6fee9697c7c9c9a38fb993a19/cmd/user-agent/web/src/main.js#L24-L34

Updated as part of https://github.com/trustbloc/edge-agent/pull/72.


provide bot.start() without run_forever()

I have other things I do in the main task, and I rather not create another task just for the bot.  Is there anyway to have a way to run_forever kwargs that I can skip that command?
For example, in signalbot/bot.py
class SignalBot:
    ...

    def start(run_forever=True):
         ...
         if run_fovever:
             self._event_loop.run_forever()

Hi! That's a good point. I might introduce an alternative method instead for this use case instead of adding an argument. I may refactor the bot API a bit and then take this into account.


How to assign a key to the created topic

Hi
actually all topic created are without any key and this making me some problem when i want consume this data do any db
how can i set the key for each table?

Hello @Sano123456 ,
There is a setting called
{topic}.pk.fields

Please refer to Configuration section for details.
Kindly let me know if this doesn't help.

Hello @sbcd90
this is my configuration
{     "name": "sap",     "config": {         "connector.class": "com.sap.kafka.connect.source.hana.HANASourceConnector",         "tasks.max": "10",         "topics": "sap_OHEM,sap_OHST",         "connection.url": "jdbc:sap://<IP_ADDRESS>:30015/",         "connection.user": "user",         "connection.password": "pass",         "mode":"bulk",         "poll.interval.ms":"86400000",         "sap_OHEM.table.name":                          "\"public\".\"OHEM\"",         "sap_OHEM.poll.interval.ms":                    "86400000",         "sap_OHST.table.name":                          "\"public\".\"OHST\"",         "sap_OHST.poll.interval.ms":                    "86400000",     } }
OHEM has over 300K rows and OHST has about 500.
My problem is that the connector copy just 100 rows of each of them and push them to the topic over adn over again without duing an incremental step. It doesn't copy all the table but just 100 rows of it and than copy it agin and again.
I tried with batch.max.rows putting 100K as limit, this solved the problem with small tables but with big tables because it broke JVM memory
I hope i described better the problem

Hi @sbcd90
this means that i need to define for each table? it doesn't read it automatically ?
Actually i am using about 500-1000 tabls

{topic}.pk.fields - This setting can be used to specify a comma-separated list of primary key fields when {topic}.pk.mode is set to record_key
which means if you specifiy the record_value instead of key should go automatically.

I am wondering why we are not extracting the primary keys into the key part by default.  Shouldn't we be at least provide an option to extract the keys? I suppose that was what @Sano123456 asked.

The comments from @sbcd90 and @igorcosta are not helpful because they reference sink connector settings when the question is about the source connector.
Just as a workaround to achieve setting a key: one can use general Kafka Connect SMTs (single message transforms) to achieve this with any connector. See here for details: https://rmoff.net/2020/12/09/twelve-days-of-smt-day-2-valuetokey-and-extractfield/
Note: this will only work if the full key is contained in one field, it cannot concatenate a primary key which uses multiple fields.


MouseEvent.path() only supported in Chrome

in v.1.0.10, https://github.com/Baremetrics/calendar/blob/master/dev/js/Calendar.js#L189 uses MouseEvent.path(), which appears to be only supported by Chrome.  Ran into problems in our test suite with phantom.
Thanks for the calendar picker, love it.

Ah whoops! Will take a look and fix asap. Or feel free to submit a PR fix


WIP Collapse multiple endpoint flaps into one

When endpoint has more than one flap, initially it will be collapsed and shows its current status

When you click it, it expands and show all flap related to the endpoint

There is definitely room for improvement, but we currently ride on DaisyUI's collapse component but we're reaching the limits of what it could do. Next step would be to implement the collapse ourselves

Before continuing on this, we need to validate that it's what the NOC wants because the work to do this properly is non-trivial


Provider foreign key in encounters table

Hello. I recently dowloaded the 1K sample synthetic patient records CSV (8 mb, Synthea_sampledata_csv_jan2020). When I join the encounters table to the provider table (encounter provider to provider ID), there are no matches. Is there an update to the provider or encounter CSV with matching providers?

This has been fixed in the project, see https://github.com/synthetichealth/synthea/issues/547#issuecomment-595466592.
I've been too lazy to post the updates. Fixing it now...

Fixed.


Implement org.ehcache.integrations.shiro.EhcacheShiro#keys by wrapping over the cache

Assuming we can make that return collection unmodifiable - which seems to be the case in current implementation of the ehcache 2.x module, it would be better to have a delegation model to the cache, thus making sure we do not have to materialise the whole key set on heap.

Merged - Pull Request#11


Get-AdfFromService $DataFactoryName vs $FactoryName Internal Param

Describe the bug
When used by with example and parent cmdlet $DataFactoryName is declared so no issue. When used in isolation internal param mapping is invalid.
Version of PowerShell module of:

azure.datafactory.tools [e.g.0.17]

Thank you, @mrpaulandrew


Failure to open the draft fasta file

Hello,
Hello, I am trying to use Nanopolish v.11.0 on the Flux Linux-based operating environment at the University of Michigan to polish assembled contigs from Canu v1.8 of demultiplexed amplicons
generated in Minibar v0.21 from fungal genomic DNA using a Nanopore MinION.
Below is an example of an attempted workflow (using bwa v.0.7.15, samtools 1.9, and minimap2 v2.16), ultimately resulting in failure to read the draft fasta file (contigs.fasta from Canu, including only tig00000002, with a length of 9173):
/home/chytrids/nanopolish/nanopolish index \
-d /scratch/tyjames_flux/chytrids/nanopore/CZEUM03_NoBaseCall/20190325_1730_MN29144_FAK52702_d2a7a2ad/fast5/ \
-s /scratch/tyjames_flux/chytrids/nanopore/CZEUM03_NoBaseCall/20190325_1730_MN29144_FAK52702_d2a7a2ad/fastq/sequencing_summary.txt \
../CZEUM03.fastq

[readdb] indexing /scratch/tyjames_flux/chytrids/nanopore/CZEUM03_NoBaseCall/20190325_1730_MN29144_FAK52702_d2a7a2ad/fast5/
[readdb] num reads: 221187, num reads with path to fast5: 221187

bwa index TYJ_large.contigs.fasta

/home/chytrids/programs/minimap2-2.16_x64-linux/minimap2 -ax map-ont -t 8 TYJ_large.contigs.fasta ../CZEUM03.fastq \
| samtools sort -o TYJ_large.contigs.sorted.bam -T reads.tmp

[M::mm_idx_gen::0.004*1.89] collected minimizers
[M::mm_idx_gen::0.005*2.81] sorted minimizers
[M::main::0.005*2.81] loaded/built the index for 1 target sequence(s)
[M::mm_mapopt_update::0.005*2.72] mid_occ = 8
[M::mm_idx_stat] kmer size: 15; skip: 10; is_hpc: 0; #seq: 1
[M::mm_idx_stat::0.006*2.67] distinct minimizers: 1391 (77.93% are singletons); average occurrences: 1.224; average spacing: 5.386
[M::worker_pipeline::43.459*1.24] mapped 155055 sequences
[M::worker_pipeline::47.969*1.16] mapped 66132 sequences
[M::main] Version: 2.16-r922
[M::main] CMD: /home/chytrids/programs/minimap2-2.16_x64-linux/minimap2 -ax map-ont -t 8 TYJ_large.contigs.fasta ../CZEUM03.fastq
[M::main] Real time: 48.116 sec; CPU: 55.584 sec; Peak RSS: 1.559 GB
[bam_sort_core] merging from 2 files...

samtools index TYJ_large.contigs.sorted.bam
samtools view TYJ_large.contigs.sorted.bam | head

/home/chytrids/nanopolish/nanopolish variants --consensus \
-o TYJ_large.contigs.polished.vcf -w "tig00000002:1-9173" \
-r ../CZEUM03.fastq -b TYJ_large.contigs.sorted.bam -g TYJ_large.contigs.fasta

[fai_load] build FASTA index.
[fai_build] fail to open the FASTA file TYJ_large.contigs.fasta

Would you please let me know what I'm doing wrong, or is there some step that I am missing?
Thank you for your attention and assistance,
Rabern.

Hi,
That is very strange. Nanopolish is only trying to build an .fai file for your draft genome using an API call to htslib (the same function that samtools faidx uses). I can't see why this would fail if the file exists and is readable.

Maybe try running strace <nanopolish command> and strace samtools faidx TYJ_large.contigs.fasta and send me the result of each.

Thank you for your assistance. Below are the requested outputs:
strace_samtools.txt
strace_nanopolish.txt

It looks like samtools faidx reported the same error (file not found), so this isn't a nanopolish issue. You'll have to figure out why these programs can't see the file (can you open it in a text editor, or run head on the file?)

Yes, it opens fine in other applications that I've attempted. It is the Canu output contigs.fasta file. Below is the .txt of it.
TYJ_large.contigs.fasta.txt

Sorry for the lack of response, I just downloaded your file though and samtools faidx TYJ_large.contigs.fasta worked without a problem, so I'm really not sure why it doesn't work for you.


 If you dont provide any reproduction, I will be moving your content to discussions. Because issues are for bugs, and bugs require reproductions. You totally ignored the issue template, filled in only the very first part, and now you came with this "follow up"? Dont expect answers for low quality questions.
Why on earth should I help you now, given how "nice" you behave?


[Android] Fix some crashes when running MacBacon specs on Android platform

This PR should fix some crashes like #194 when running MacBacon specs on Android platform.
On Android platform, even constant is NOT defined, defined? returns "constants", so expression like:
if defined?(UIDevice) && !UIDevice.currentDevice.model.match(/simulator/i)

throws NoMethodError and causes crashes.
Since evaluating undefined constant is false on Android platform,
defined?(Foo) #=> "constant"
Foo           #=> false

by adding extra condition like defined?(UIDevice) && UIDevice && ..., you can check if constant is really defined or not.
I think that defined? should return nil when testing undefined constant. But I would like add this temporaly fix to run specs.
Thank you for your consideration.

Hello!
This problem is due to a regression in the current version of Rubymotion that makes undefined constants and variables return false instead of nil.
http://hipbyte.myjetbrains.com/youtrack/issue/RM-801
It has been fixed and will be available in the next version of RM, which will be out pretty soon.
Sorry if this caused you any inconvenience.
Mark.

@MarkVillacampa
I think that the issue is Object#defined? returning "constant" instead of nil for undefined constant. Is this issue fixed, too?
Thank you.

You're totally right, this is a different issue, I have opened a youtrack here: http://hipbyte.myjetbrains.com/youtrack/issue/RM-826
I'll make sure this gets fixed for the next release so we don't need this pull request.
Thanks!


Design Home Page

Add home page with similar format to DocSearch. Replace branding & text with Celo info.

Not adding home page at this time.


Missing dependencies

I installed this repo on a project using venv, and it complained about scikit-image and scikit-learn
I can´t make a PR right now, but whenever I get the time I´ll do it, just a heads up!

I look forward to receiving your PR
I'm very busy right now 🤗


feat(CStarAlgebra): vectors with entries in a C⋆-algebra

This PR defines the type CstarVec n A as a copy of n → A meant for the case where A is a C⋆-algebra. This type is then given a Hilbert C⋆-module structure, namely an A-valued inner product and the norm ‖v‖ = √‖⟪v, v⟫‖.

[ ] depends on: #15244

@dupuisf is there anything else that we should snag from this PR that I missed? Or should we close it? Or do you think we should still have CStarVec as an abbrev for C⋆ᵐᵒᵈ (n → A) (which would be weird because it's actually longer given the notation!)?

No, I think we can just close this one.


tmp: [NPM-WIN] #1893 and #1900

Reason for Change:

Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

/azp run


User status doesn't reflect my actual status and permissions

When I'm logged in, my account has admin privileges, but the account status on the top right says 'Viewer', not 'Admin'.

https://github.com/zooniverse/alice/blob/2f287eda190b96827806076e38acefc27b4fc706/src/store/TranscriptionsStore.spec.js#L366-L369
This test doesn't add a current transcription to the mocked store. So it's really testing that the locked document warning can appear while the store is empty. I don't know if that's directly related to this issue but the test passes, which points to a bug in store.transcriptions.current. Perhaps checkIfLocked runs while the page is loading?
Found while working on #333.

I'm not sure if the 'status doesn't reflect actual status/permissions' diagnosis is the only thing happening here. Mine seems to actually reflect the wrong permissions. I'm listed as a Collaborator on project 4973 (Anti-Slavery Manuscripts) and am listed in ALICE as a Viewer (see screenshot below):

If I had actual Admin privileges and was just incorrectly listed, I'd expect to see a lot of buttons at the top of the page, including the option to Unapprove the subject, as well as the 'More' dropdown, which lets you download Approved subject data. Here's what an Approved subject looks like on a project where I'm an Admin:

Might be something else going on?

That’s interesting. I didn’t see those buttons either, but API requests were made with admin privileges.

I’m trying to remember what I meant when I wrote up this issue. It was a long time ago, but I think I meant that I could see projects even when I’m not on the team.
Something weird is going on with roles, though.
According to the git history, 'admin' used to be called 'researcher'. I’m a collaborator on a project, which should give me the ‘researcher’ role, but still show up as a Viewer, not Admin, for that project in Alice.

Here's a screenshot from browser dev tools, for a project where I'm a collaborator. The API request for project roles is coming back empty from Panoptes. It was sent with the correct Authorization header, and this is for a single project, so pagination isn't a problem.

Nevermind. If I make sure that Collaborator and/or Researcher are checked in the project builder, then project roles work for my account.


Click on a scrollbar unselects selected rows

If you click on a scrollbar (Or if you click inside the canvas but not actually on a cell/header/row) the selected rows getting unselected.
This was no issue in 5.2.1.
You can reproduce this behavior it the storybook:
https://quicktype.github.io/glide-data-grid/?path=/story/glide-data-grid-dataeditor-demos--multi-select-columns
I would say it should only be possible to unselect a row via the rowMarkers!?
Would be great if this can be fixed. ;)
Thank you, for the awesome GlideTables!

Yes will fix

Fixed in 5.3.1


Raise warning if reactive.Effect function returns a value

Closes #425.
If a user function passed to reactive.Effect returns a value, that is most likely a mistake. This PR causes Shiny to raise a warning in that case. For example:
@reactive.Effect
def _():
    return 1

Note that the warning message should be improved to print the line number in the user code where the return happened. This implementation simply causes Python to print out the source where the warning is raised, which always in the same place in _reactives.py.
await reactive.flush()
/Users/winston/py-shiny/shiny/reactive/_reactives.py:516: ReactiveWarning: reactive.Effect function returned a value, but should return None; reactive.Effects should only be used for side effects. Did you mean to use reactive.Calc instead?
  await self._run()

@jcheng5 @wch is this good to merge?


CI: test installation guides

Moved static tests to CI setup script and added a new CI test to execute
all install guides if any one changes.
Fixes #132.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Blocked on https://github.com/kata-containers/tests/pull/384.

@chavafg, @grahamwhaley, @egernst, @gnawux - this is a particularly interesting PR which I'd really like to get landed :)
Note the blocking PR and my cunning plan:

https://github.com/kata-containers/documentation/pull/135#discussion_r193356040

lgtm

lgtm


feat(linter): Implement jsdoc/check-access

Just as a trial. 🙈
Some preparation PRs may be needed in advance.

@Boshen These are some concerns noticed when I tried it. What do you think?
Is it OK to use run_once?
I got a review in a previous PR: If run_once is implemented, but run is still executed.
Is this unavoidable?
struct JSDocTag
Currently, there is a single struct JSDocTag and distinguished by its .kind.
Should there be a separate struct for each tag?
Then, for example, something like has_valid_level() could be implemented in JSDocAccessTag.
But it seems that the current string based implementation is fine.
(There is no specification of how users actually describe JSDoc. 🙄 )
Depends on how deeply Oxc wants to support JSDoc?
Failed CI
(I'm not sure I can fix this...!)
This is the failed comments from antd.mjs.
/**
 * flat tree data on expanded state
 *
 * @export
 * @template T
 * @param {*} data : table data
 * @param {string} childrenColumnName : 指定树形结构的列名
 * @param {Set<Key>} expandedKeys : 展开的行对应的keys
 * @param {GetRowKey<T>} getRowKey  : 获取当前rowKey的方法
 * @returns flattened data
 */

/**
 * @param {Set<Key>} expandedKeys : 展开的行对应的keys
 * @param {GetRowKey<T>} getRowKey  : 获取当前rowKey的方法
 */

This also fails. Something wrong in @param parsing.

Is it OK to use run_once?

This is the work flow:
https://github.com/oxc-project/oxc/blob/53a8e7ffbebb19cdd92bd2bcef6c308e0897bf05/crates/oxc_linter/src/lib.rs#L107-L129
We try to avoid visiting the AST twice.

Should there be a separate struct for each tag?

Do this instead:
impl<'a> JSDocTag<'a> {
    fn kind(&self) -> JSDocTagKind<'a> {
        self.kind
    }
}

impl<'a> JSDocTagKind<'a> {
    fn is_deprecated(&self) -> bool {
        matches!(self, Self::Deprecated)
    }

    fn parameter(&self) -> Option<&Param<'a>> {
        if let Self::Parameter(param) = self {
            Some(param)
        } else {
            None
        }
    }
}

Many of plugin-jsdoc rules will use run_once() only.
In these cases,
for node in semantic.nodes().iter() {
   for (rule_name, rule) in &self.rules {
       ctx.with_rule_name(rule_name);
       rule.run(node, &ctx); // this run() fn is almost empty 👀
   }
}

Does this mean negligibly cheep?

Write to dump thoughts...

plugin-jsdoc has tagNamePreference settings
By this settings, user can create alias for tag name

e.g. @prm {string} x instead of @param {string} x, etc..

This means that we cannot determine parsing logic from the tag name(kind)
More example:

@foo x as @param x(x = type name) and @foo x as @access x(x = comment) cannot be distinguished

How to solve this?

Give up to support tagNamePreference?
Parse tag body lazily on demand?

tag.parse_as_type_comment(), tag.parse_as_comment(), etc...
How many syntax patterns there??

Thank you! Sorry I missed the PR.


Require of filter function in get-battle API

https://github.com/fetus-hina/stat.ink/blob/master/doc/api-2/get-battle.md
I'm an iOS developer and prepare to build an iOS app for stat.ink.
I read the public API and It seems that the get-battle API doesn't accept any filter options. Are there any more functional APIs for me to build an app?

Unfortunately "no", as of now.
I am busy with work now and current top priority task is Salmon Run.
But filters are necessary, so we will create them later.


Avoid name-clash with future java.lang.Record

Motivation:
Recent Intellij versions are starting to anticipate future versions of Java that include a java.lang.Record class, and the Intellij compiler gets confused by the Record class in our ResorceLeakDetector.
Modification:
Rename our Record class to TracerRecord. This matches what the class is doing, while avoiding any future name clashes.
Result:
Intellij can now compile the project again, even when configured to use a future (snapshot or early access) version of Java.

@netty-bot test this please

@chrisvest I guess we also want to add this to 4.1 ?

@normanmaurer sure, we can do that.


Add length option for random string provider

#18

I added the "length" option that accepts constant and range.
Range must be specified like that : "3..20"
Constant must be specified like that : "8"
I'm not completely satisfied by this solution because the constant must be between double quotes. If an integer is specified the function column["length"].as_str() returns None.
@vianneybacoup do you know a better solution to be able to specify an integer or a string for the constant length?
Thank you for your help.

Thank you @vianneybacoup for your help, with pattern matching I was able to solve my issue. Length option can now be specified into the yaml without double quotes.
Agree with you about "Having a dedicated logic for parameters", I let this point for a specific issue.
I also modified tests/tests.rs to manage unix OS.


[SPARK-48047][SQL] Reduce memory pressure of empty TreeNode tags

What changes were proposed in this pull request?

Changed the tags variable of the TreeNode class to initialize lazily. This will reduce unnecessary driver memory pressure.

Why are the changes needed?

Plans with large expression or operator trees are known to cause driver memory pressure; this is one step in alleviating that issue.

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UT covers behavior. Outwards facing behavior does not change.
Was this patch authored or co-authored using generative AI tooling?
No

test failures look unrelated.

Merged to master for Apache Spark 4.0.0.


image security

I am certain that the state of the art in scanning images, signing them, and publishing SBOMs has evolved since any of those things were last touched. We should revisit those things.
Wondering if @34fathombelow would be willing to bring his expertise to bear on this...

#557 addressed this.


Models: Invalid pointer redirect

Problem statement
Generated code includes incorrect pointer operation.
// GroupPreference group preference
// swagger:model groupPreference
type GroupPreference map[string]float32

// Validate validates this group preference
func (m GroupPreference) Validate(formats strfmt.Registry) error {
	var res []error
	for k := range m {
		if swag.IsZero(m[k]) { // not required
			continue
		}
		if err := validate.Minimum(k, "body", float64(*m[k]), -5, false); err != nil {
			return err
		}
		if err := validate.Maximum(k, "body", float64(*m[k]), 5, false); err != nil {
			return err
		}
	}
	if len(res) > 0 {
		return errors.CompositeValidationError(res...)
	}
	return nil
}

*m[k] causes compiler error invalid indirect of m[k] (type float32). It should be m[k]
Swagger specification
The swagger definition is :-
  groupPreference:
    type: object
    additionalProperties:
      type: number
      format: float
      minimum: -5
      maximum: 5
    example:
      default: 0
      CN: -0.2
      GB: 0.5
      NL: -5

format: float or format: double, both have the same issue.
Steps to reproduce
swagger generate client -f spec_containing_definition.yaml -t $GOPATH/github.com/user/repo
Environment
swagger version: 0.14.0
go version: go version go1.10 linux/amd64
OS:  Linux 4.13.0-41-generic #46-Ubuntu SMP

OK from reading comment on another issue, adding x-nullable: false is a workaround for now.

I am currently working on a fix for that.
The problem is that the map element is independently elected as nullable and rendered as a pointer (because minimum < zero value of number < maximum) and the aliased map currently ignores this.
See also #1561


support for latex expression ?

I have tries the plugin gatsby-remark-mathjax. But it doesn't work. Is any problem compatible ? Thank you.

@greglobinski , I have succed using the plugin gatsby remark katex.
Just add one line in the file "src/components/Main/Articles.js"
import "katex/dist/katex.min.css";


Determine initial hierarchical structure

First thoughts from the workshop
fortran-index
├── Fortran standards
├── Compilers (+ flag differences?)
├── IDEs
├── Debuggers
├── Test frameworks
├── Linters
└── Training material

We should also add a "references" section pointing to other lists.

In the unconference session on "working with legacy code" at RSECon we discussed adding a section on common old-fashioned coding patterns and recommended refactoring techniques.