Make all argument results optional in type def

I think I overlooked this in my first PR - of course all flags are possibly missing to start with, therefore all the flags should be marked as optional in TypeScript.

Yeah LGTM :)

Release as 4.0.1. Thank you @blakeembrey :)


[XLA] Allow for disabling of some tests in the XLA CC test suite

Allow for 2 XLA CC tests to be disabled in a manifest file.
[some backends may not support features required for passing these tests]

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@tatatodd thanks for the review 👍


Upgrade to ALS 1.0.0 and adjust related configration and code

Done. https://github.com/ansible/ansible-language-server/pull/448


[Stable] Release 0.2.1

Summary
0.2.1 release with untyped form of qiskit_algorithms (not to break typed dependent packages).
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

170 of 182 (93.41%)  changed or added relevant lines in 45 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 90.516%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_algorithms/eigensolvers/numpy_eigensolver.py
5
6
83.33%

qiskit_algorithms/optimizers/adam_amsgrad.py
0
1
0.0%

qiskit_algorithms/optimizers/aqgd.py
1
2
50.0%

qiskit_algorithms/optimizers/gradient_descent.py
6
7
85.71%

qiskit_algorithms/optimizers/spsa.py
4
5
80.0%

qiskit_algorithms/optimizers/umda.py
2
3
66.67%

qiskit_algorithms/phase_estimators/phase_estimation_result.py
1
2
50.0%

qiskit_algorithms/amplitude_amplifiers/grover.py
3
5
60.0%

qiskit_algorithms/minimum_eigensolvers/diagonal_estimator.py
7
10
70.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
6452

Relevant Lines:
7128

💛  - Coveralls


Bump dependencies

This doesn't impact any generate code. I don't have a strong motivation
to do this (I was trying to fix something, no progress), but its good to
stay latest

/approve

/ok-to-test


Add OpenAPIHub to Development

[x] My submission is formatted according to the guidelines in the contributing guide
[ ] My addition is ordered alphabetically
[ ] My submission has a useful description
[ ] The description does not have more than 100 characters
[ ] The description does not end with punctuation
[ ] Each table column is padded with one space on either side
[ ] I have searched the repository for any relevant issues or pull requests
[ ] Any category I am creating has the minimum requirement of 3 items
[ ] All changes have been squashed into a single commit

Due to PR inactivity, this API was added to the list on commit: 7b6640dc9470aafeebc121c66e4f2d1d6e2836e5


[beta] Declaring Variables in Data Visualization

Challenge Name
http://beta.freecodecamp.com/en/challenges/data-visualization-with-d3/work-with-data-in-d3
Issue Description

Not sure if this is related to the local memory issues..
The tests fail if you leave the variable as dataset because it says that it has already been declared as a variable.  But if you change the variable it works fine.
Browser Information

PC Chrome

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

Data Visualization with D3: Work with Data in D3

I maked the test and it pass.
The issue is the local memory of your pc.

Hit the same thing. Also #14329 is exhibiting the same problem, so could be closed as a duplicate if we start a list of problematic challenges. Reloading the page works, while reseting doesn't fix the problem.

And I guess both are duplicates of #12762

Hi! Thanks for reporting and looking into this issue.
This is a known issue. We're working on resolving it through #12762. I'm closing this as a duplicate.
Happy coding!


Remove all old-style class declarations

More code cleanup.

Coverage decreased (-0.002%) to 94.767% when pulling 709cd270a66d1ffc30dcd6a9c7bc08f9e3d95fd6 on drdavella:remove-old-style-classes into 41200690ed3e3542794bdccb2a765d55f0d86433 on spacetelescope:master.


EF7 SQL Server Compact Create a Model from an existing database

Hi, I have the SQLCe provider working, creating new database and opening existing. I have tried to create a model from an existing database but have hit two problems. The first was when I tried to install EntityFramework.SqlServerCompact40.Design -Pre:
PM> install-package entityframework.sqlservercompact40.design -pre
Attempting to gather dependencies information for package 'entityframework.sqlservercompact40.design.7.0.0-beta7a' with respect to project 'EF7DataLayer', targeting '.NETFramework,Version=v4.6'
Attempting to resolve dependencies for package 'entityframework.sqlservercompact40.design.7.0.0-beta7a' with DependencyBehavior 'Lowest'
install-package : Unable to resolve dependencies. 'EntityFramework.Relational.Design 7.0.0-beta8-15723' is not compatible with 'EntityFramework.Commands 7.0.0-beta8-15723 constraint: EntityFramework.Relational.Design (≥ 7.0.0-beta8-15723)', 'EntityFramework.SqlServerCompact40.Design 7.0.0-beta7a
constraint: EntityFramework.Relational.Design (= 7.0.0-beta7)'.
At line:1 char:1

install-package entityframework.sqlservercompact40.design -pre

  + CategoryInfo          : NotSpecified: (:) [Install-Package], Exception
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackageCommand

I then installed the package anyway by ignoring the dependencies, just to see if it might work. I then got this error when I tried to run the scaffold:
PM> scaffold-dbcontext -connection "DataSource=C:\Users\Patrick\Documents\z.sdf" -provider entityframework.sqlservercompact40
scaffold-dbcontext : Exception calling "CreateInstance" with "8" argument(s): "Type 'System.Management.Automation.SwitchParameter' in assembly 'System.Management.Automation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' is not marked as serializable."
At line:1 char:1

scaffold-dbcontext -connection "DataSource=C:\Users\Patrick\Documents ...

  + CategoryInfo          : NotSpecified: (:) [Scaffold-DbContext], MethodInvocationException
  + FullyQualifiedErrorId : SerializationException,Scaffold-DbContext

Any ideas on  what I should try?
Thanks

Great you got it working - you should reference either only nuget.org or the myget feed, not both at the same time.

Hmm, I am using EF7 nightly builds, current version 7.0.0-beta8-15723. If I try to install EntityFramework.SqlServerCompact40 from nugget.org then I get:
Attempting to gather dependencies information for package 'EntityFramework.SqlServerCompact<IP_ADDRESS>-beta7a' with respect to project 'EF7DataLayer', targeting '.NETFramework,Version=v4.6'
Attempting to resolve dependencies for package 'EntityFramework.SqlServerCompact<IP_ADDRESS>-beta7a' with DependencyBehavior 'Lowest'
Unable to resolve dependencies. 'EntityFramework.Relational 7.0.0-beta8-15723' is not compatible with 'EntityFramework.Commands 7.0.0-beta8-15723 constraint: EntityFramework.Relational (≥ 7.0.0-beta8-15723)', 'EntityFramework.Relational.Design 7.0.0-beta8-15723 constraint: EntityFramework.Relational (≥ 7.0.0-beta8-15723)', 'EntityFramework.SqlServer 7.0.0-beta8-15723 constraint: EntityFramework.Relational (≥ 7.0.0-beta8-15723)', 'EntityFramework.SqlServerCompact40 7.0.0-beta7a constraint: EntityFramework.Relational (= 7.0.0-beta7)'.
Do I need to revert to an earlier nightly build?

Sounds like you are mixing beta7a and beta8 bits. Make sure to install only beta8 bits in your project, and that only beta8 bits are referenced by your code.

I think during the course of events I probably did have the beta7/8 bits mixed (my second comment was probably made at that time). I am quite sure I have that sorted now, but I still get an error:
Attempting to gather dependencies information for package 'EntityFramework.SqlServerCompact40.Design.7.0.0-beta8-328' with respect to project 'EF7DataLayer', targeting '.NETFramework,Version=v4.6'
Attempting to resolve dependencies for package 'EntityFramework.SqlServerCompact40.Design.7.0.0-beta8-328' with DependencyBehavior 'Lowest'
Unable to resolve dependencies. 'EntityFramework.Relational 7.0.0-beta8-15723' is not compatible with 'EntityFramework.Commands 7.0.0-beta8-15723 constraint: EntityFramework.Relational (≥ 7.0.0-beta8-15723)', 'EntityFramework.Relational.Design 7.0.0-beta8-15723 constraint: EntityFramework.Relational (≥ 7.0.0-beta8-15723)', 'EntityFramework.SqlServer 7.0.0-beta8-15723 constraint: EntityFramework.Relational (≥ 7.0.0-beta8-15723)', 'EntityFramework.SqlServerCompact40 7.0.0-beta8-328 constraint: EntityFramework.Relational (= 7.0.0-beta8-15719)'.
All the references are to beta8 packages now, but the last constraint seems to be for an exact build which I think may be the problem.

It would be useful if your could share your packages.config file, but I think the version of EF.Commands installed is too new - see the (just updated) instructions here: Install-Package EntityFramework.Commands -Pre -Version 7.0.0-beta8-15719
 Install-Package EntityFramework.Commands -Pre -Version 7.0.0-beta8-15719

I was going to mention in my previous post that I had tried installing EntityFramework.Commands15719 but I couldn't remember the error it gave me. I have tried again and I also had to move the EntityFramework.Relational back to 15719. I'm afraid I'm still then back to the following error:
PM> scaffold-dbcontext -connection "Data Source=C:\Users\Patrick\Documents\z.sdf" -provider EntityFramework.SqlServerCompact40
scaffold-dbcontext : Exception calling "CreateInstance" with "8" argument(s): "Type 'System.Management.Automation.SwitchParameter' in assembly 'System.Management.Automation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' is not marked as serializable."
At line:1 char:1

scaffold-dbcontext -connection "Data Source=C:\Users\Patrick\Document ...

  + CategoryInfo          : NotSpecified: (:) [Scaffold-DbContext], MethodInvocationException
  + FullyQualifiedErrorId : SerializationException,Scaffold-DbContext

packages.config below:

Could you indent packages.config - could be a typo / syntax issue with your command?

I'm afraid this amateur is having trouble! Has this worked?
image

Or maybe my docs are wrong!

I can see the packages.config now, your EF Core package is too new!

I think the command should be changed to:
scaffold-dbcontext "DataSource=C:\Users\Patrick\Documents\z.sdf"  entityframework.sqlservercompact40

Sorry, but I still get:
PM> scaffold-dbcontext "DataSource=C:\Users\Patrick\Documents\z.sdf"  entityframework.sqlservercompact40
scaffold-dbcontext : Exception calling "CreateInstance" with "8" argument(s): "Type 'System.Management.Automation.SwitchParameter' in assembly 'System.Management.Automation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' is not marked as serializable."
At line:1 char:1

scaffold-dbcontext "DataSource=C:\Users\Patrick\Documents\z.sdf"  ent ...

  + CategoryInfo          : NotSpecified: (:) [Scaffold-DbContext], MethodInvocationException
  + FullyQualifiedErrorId : SerializationException,Scaffold-DbContext

Packages.config:

I will investigate...

I am able to repro - I have logged an issue here: https://github.com/aspnet/EntityFramework/issues/3178

FYI - using just the beta 7 bits from NuGet Works fine - why do you need the daily builds?

Thanks for the support, I will hopefully get to try the beta7 later tonight. There are a couple of features in the latest builds that are currently WIP which are important to me and I am trying to keep up with, namely TPH and the GraphBehavior.

Progress... With beta7 I now get:
PM> scaffold-dbcontext -connection "Data Source=C:\Temp\z.sdf" -provider EntityFramework.SqlServerCompact40
The type initializer for 'System.Collections.Immutable.ImmutableSortedSet`1' threw an exception.
Packages:

Remove all references, delete your packages folder and start over - System.Collections.Immutable must be version 1.1.36

I was a bug in EntityFramework.Commands, and will be fixed in my next "daily"/refresh

@ChangedHourly I have released a new build that should contain the fix

OK, I think I still need some help regarding the packages though. I can see your new build 7.0.0-beta8-330. Am I correct in assuming this needs to be installed with EF beta8 bits (re don't mix beta7/8 bits)? If so, then what about System.Collections.Immutable, does that still have to be 1.1.36?

Start with a blank project, and follow the instructions here: https://github.com/ErikEJ/EntityFramework7.SqlServerCompact/wiki/Using-EF7-with-SQL-Server-Compact-in-Traditional-.NET-Applications-(daily-builds) - and of course I would appreciate any feedback you may have if the instructions are unclear!


Generate the natural language from a Clause

A Clause is defined by a link to a template and some data (JSON) for the values of the variables in the template.
It would be very useful to be able to generate the natural language for the Clause, such that the text could be passed to the parse method to round-trip the Clause to extract the data.

Implemented by https://github.com/accordproject/cicero/pull/192 which adds the TemplateInstance.generateText method.


Separate sentence about Elastic Plans

Could you in one of first sentences explicitly write that all plans for Premium Plans named "Elastic Premium Plans".
Currently, it's looks like some inconsistency.
For example in pricing - there are no word "elastic" https://azure.microsoft.com/en-us/pricing/details/functions/
In this article - it also was not explicitly explained. As a result, I spend 10 minutes to understand that "Elastic" it's not just yet another marketing term, but real name of SKU.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3847e16b-92ae-0202-c7e1-ca9a8474e65f
Version Independent ID: a25b6b5b-05ab-b3b2-616c-5d06eadb928d
Content: Azure Functions Premium plan
Content Source: articles/azure-functions/functions-premium-plan.md
Service: azure-functions
GitHub Login: @jeffhollan
Microsoft Alias: jehollan

Hi @SychevIgor Thanks for your feedback. We will review and update as appropriate.

@SychevIgor PR has been created for this which will address this. Thanks so much for providing the feedback that help us in improving the documentation. For now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Show file/stream in the preview window

Hello,
We have an archive page where we get files from an API that we show in NAV. Now we want to open those files in the same way as you preview a report.
Is it possible to open the preview page with our own files/streams or is this only accesible via running a report?

What do you mean by "open those files in the same way as you preview a report" ? Do you mean that you want to use something like a smaller window in which you can preview files?

Yes, by using this window:

I think it should be possible to create a pdf viewer control add-in. But I want to know if we could use this window?

@JesperSchulz what do you think?

This is only accessible via reports. You would need to use a custom control add-in for your scenario.


"Enter Fullscreen" does nothing

The option when I press and hold the background, and yes, I know I need to doubletap it

net::ERR_NAME_NOT_RESOLVED

?

Linux / Unix: Sort Specific Field or Column

Muyidee

Muyideen

Linux / Unix: Sort Specific Field or Column

Muyidee

Muyideen

?


Make README.md more friendly for a first-time user.

Hello!
First, I love that someone finally got around to creating integration for ruff. :)
This pull introduces minor touch-ups to the README.md:

Add badge with the current number of downloads
Fix typos and capitalization.
Move "Sponsors" to the bottom of the page to make room for more important information at first glance (e.g. features, screenshots).
Remove the "Settings" screenshot, as it adds nothing to the already existing information on the page.
Made "Links" more accurate.
Added link to the "Development" section of the documentation to the "Contributing" section in README.md to help people get started faster.

Some of those changes may be too opinionated, feel free to remove or disregard what you don't feel is right. :)

@lensvola
Thank you for creating the PR.
I agree with you.
We should improve the document to become a good open-source project :)
Feel free to give us feedback!!

Thank you for your support! :)


fix(tuta): application of accent color

🔧 What does this fix? 🔧
Tuta now uses their actual brand colors in their app. This fixes the application of our accent colors.
I also adjusted the application to icons: The selected icon in the navbar now also uses the accent color.
🗒 Checklist 🗒

[X] I have read and followed Catppuccin's contributing guidelines.
[X] I have updated the version appropriately in the ==UserStyle== header of the catppuccin.user.css file.

before
after


fix some styles

Hi @abdelhamiderrahmouni , I have added some changes in tailwind styles. I notice that there is no style in filament dark mode so I added a dark mode. Added also some other changes. Screenshots below. Let me know if there is anything else I can do.

Full screen:

❤🔥 Thnk you for your kind contribution @199ocero.

Thank youu @abdelhamiderrahmouni ? when will you release it?

It's released now👍🏻 thank you for your contribution.


Usage documentation with examples.

We have a few examples up and running now:  github.com/kudobuilder/frameworks


pagination and minor site refinements

resolves #1859 - implement pagination UI on /browse and user pages
resolves #1865 - editor patches search icon has incorrect dimensions
resolves #1640 and resolves #1639 - add/update favicons
also contains:

replace user v and link to /v in public graph cards with 360.vizor.io
add a subtle mark to ranked graphs
when logging-in, navigate to the user's own page
minor request context is now exposed to handlebars engine
restore broken /latest-graph route, now at /~latest-graph
bugfixes to Vizor.pageObjects
mongoose module updated to ~4.5 (also in shrinkwrap)

lgtm


Highlight links somehow

Currently links are only visible/noticeable when hovering over the link.
What do you think about making links blue or something to make them stand out more?

Oh, they already have a color:
a {
color: #1e6bb8;
}
But it's not shown in the footer or is it @Imipenem ?


test: add more E2E test case

Summary

rspack fixture set default value for context and builtins.define.
crossoriginloading case test public_path
add tailwindcss test case
add asset-emitted hook test

Test Plan

LGTM, just need to get CI to pass.


ArgoCD Knownhost SSH clone

We need a possiblity to declare ssh know hosts into the CRD

closed with https://github.com/caos/boom/releases/tag/v0.9.29


[#12] Fix Matrix display

Fix display of the matrix following TC update 2018-1

Looks well. Thanks for you contribution.


Issue 49328: Ensure RootMaterialRowId is set for all materials

Rationale
This addresses Issue 49328 by updating the upgrade script to repoint row's with invalid LSIDs, LSIDs of samples that do not exist in the system, to themselves for their RootMaterialRowId value. This can be done as an update to an existing script because upgrades will fail if there is NULL values for RootMaterialRowId which will mean this script will need to be re-run.
Related Pull Requests

#4844

Changes

Update upgrade script to account for NULL values in RootMaterialRowId

It is currently up in the air if we'll target 23.12 for this and create a point release or defer to develop/24.01.


Percent (%) in routes params break the application due to encoding/decoding issue

I have the following route:
    path: '/article/:title--:id.html',

This is my params
{ id: 10 , title: '-50% on nutella' }

It fail to encode the percent, or it fail to decode if I encode it myself the title with encoreURIComponent or encoreURI
Version

react-router 5.1.2
react-router-dom 5.1.2

Steps to reproduce
Use history.push(page, { id: 1, title: '-50%' })

Notice that it fail to encode

Use history.push(page, { id: 1, title: encodeURIComponent('-50%') })

Notice that it now work

Press F5, notice that it can't decode

Notice that it fail to encode

Expected Behavior
I expect RR to be able to encode and decode any characters
Actual Behavior
It fail to encode the %
 URIError: Pathname "/article/-50%--13.html" could not be decoded. This is likely caused by an invalid percent-encoding.
    at createLocation (history.js:87)

With encodeURIComponent(title), on refresh (F5)
But then a decode does: URIError: Failed to decode param '/article/-50%--14.html'
    at decodeURIComponent (<anonymous>)

Please see the discussion in ReactTraining/history#505

Thanks,  I have read it but I don't get  if this can now be fixed or not... do you have a clue ?


browser-window dependency not found

You're attempting to use a browser-window dependency not specified in package.json so this library cannot be used because of this dependency not being found.
Nice work though! I like how you abstracted signin for electron apps.

Leaving a comment to get notices when this is resolved. Would've loved to use this but can't for there is no browser-window.

@mciparelli , @black-snow, Thank you for the issue!
browser-window is a core Electron module, and not an npm package, so it should not be listed in package.json deps.
However, recent electron releases has changed way browser-window should be required (it's now a property of electron core module).
Please check out version 2.0 where I fixed this issue and also removed some deprecated argument in API...

@parro-it Where is version 2.0? I don't see it in any of the published releases (tags), nor a release branch for it.


Doesn't build with go 1.3 or 1.6 on Debian 8.3

I tried to build check_docker with go 1.3 standard Debian wheezy package installed and it failed. I downloaded and installed go 1.6 from golang.org and got the same result, as below.
$ go get github.com/newrelic/check_docker
# github.com/newrelic/check_docker
src/github.com/newrelic/check_docker/check_docker.go:68: cannot assign *docker.DockerInfo to cd.dockerInfoData (type *docker.Env) in multiple assignment

The line is part of the 5 Feb 2015 refactor and there are binary downloads after that, so clearly it (used to) build for others.

I am also receiving this error.

Im also having this error on go 1.5.1

I am also having this error on centos 6.8 with go 1.7


[17.06] vndr libnetwork to bring in fix for overlay network ip reuse

vndr libnetwork to bring in fixes:

docker/libnetwork#1901 Revert "Remove dynamic mac entry from fdb on endpoint deletion"
docker/libnetwork#1903 [backport 17.06] PeerDbDelete was passing the wrong field
docker/libnetwork#1912 [backport 17.06] Overlay fix for IP reuse

comparison of changes from the libnetwork repo: https://github.com/docker/libnetwork/compare/2102668...9397626

@andrewhsu
This change doesnt include :
docker/libnetwork#1901 Revert "Remove dynamic mac entry from fdb on endpoint deletion"

@mavenugo the diff does not show it because 17.06-CE did not vendor the libnetwork branch with the revert, so the final result is like that revert never happened

@fcrisciani thats correct. and hence, we should remove it from the description :)

I am completely confused.
Could you tell me whether docker-ce 17.06-1 include this fix or not?
I see the git commit from the released docker version is 874a737. Which is the children of commit 4770c7af51ec27af8cf8491a3a5358bad3b4c8b7 and 99a27323fec5e67d7708140814e7e60fa974abe2 . These commits looks like included this PR.

@athurg the patch is part of docker-ce 17.06.1.
The confusion comes from the fact that the issue overlay issue was fixed around 1 month ago, but during the testing of the recent RCs we found out that the specific patch was introducing a regression so we had to revert it for a week.
During this time docker-ce did not revendor libnetwork so that overlay issue never got introduced back to this repository. If you see the diff of libnetwork you can see the whole history where the patch is reverted: https://github.com/docker/libnetwork/commit/2d5f45223296a9b2df9f84668e114ff722c8ed74 and when it is brought back: https://github.com/docker/libnetwork/commit/9587764de8f44c6b0f1960fdca17cab4938ef5db with the addition of the fix of the regression: https://github.com/docker/libnetwork/commit/31133464286321c641611533f2bd6ee5756d1ec9
Hope this clarify the confusion, let me know if you have other questions

OK, I got it. Thanks for your explanation!


Fix job spec file name in job_spec.feature

Hi!
I have corrected the spec file name in job_spec.feature as I believe the Job spec file name is generally the product code file name with _spec at the end.
I noticed this when reading Relish.
If I have missed something, you can close this PullRequest.
Thank you.

Thank you!


✨ Use kairos-agent from packages

Also drops all uneeded targets, code and so on for the agent
Moves profile-build into its own dir
Moves go.mod files into theyr own, one for testing and one for the
profile build
Adjusts earthly targets to the new stuff
Signed-off-by: Itxaka<EMAIL_ADDRESS>What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

This also cuts the time it takes to build isos under CI because it no longer needs to build the whole agent+webui+docs by itself :eyes:


Adding Extra Data

The problem
As I first brought up in this reddit comment, I think this crate would really benefit from additional data being stored with each emoji.
Here I'll present a draft of what kind of data may be included, how to scrape that data, and methods of generation. If this idea is accepted, it can be work-shopped before implementation.
Source Files
Currently, this repo pulls from the emoji-test page. While this does well to give basic information, it lacks much useful information. I propose using the Unicode CLDR to gather data from. Not only is it how major projects typically build emoji libraries, but it has much more data.
Some data that can be scraped with this method is as follows:

Codepoint characters

Ex: 😂

Canonical name

Ex: face with tears of joy

Category name

Ex: Smileys & People

Subcategory name

Ex: face-positive

Keywords

Ex: face, face with tears of joy, joy, laugh, tear

Qualification

Ex: fully-qualified

Scraping Method
Gathering data can be done in a few steps:

Categorize emoji-specific codepoints
Parsing basic info
Cross referencing keywords

Gathering the emoji-specific codepoints and initial data is easy. It's found in cldr/tools/java/org/unicode/cldr/util/data/emoji/emoji-test.txt via the CLDR link above. I believe this is the same data as where this project currently pulls from, but this should be double checked and a link to latest should be found.
Parsing basic info is done directly from the above file. This gives codepoint, string representation, qualification, and canonical name.
Cross referencing is done by examining files within common/annotations/*.xml.
Packaging
Interpreting the scraped data and dumping into a rust crate should be done with great care. There is a lot of data here, and I think a lot of room for improvement over the current method. I propose the following method:

Use build.rs to download files and generate Rust code. This removes the dependency on javascript that this crate currently has, and would allow for a very small footprint -- all generation is done during build time.
After scraping data, use build.rs to dump pre-formatted rust code into OUT_DIR to be included directly in lib.rs.
In addition to having each codepoint chronicled, include a final metadata marker -- a compile time hashmap in lib.rs to help in searching and filtering emoji.

Localization
Good news is that CLDR gives annotations in a large number of languages. Bad news is this project should eventually account for that. I propose we stick with English for now and work that out later.
However, here is a rough idea of what I was thinking:

Use crate features for each localization. This will require semi-manual updating when new CLDR localizations come out, but I think it's worth it.
Each feature is the name of the annotations/*.xml file. For example, English localication would be enabled via the en feature.
en should be a default feature

Dependencies
I recommend some of the following crates while working on this project:

phf to create perfect compile-time hashtables
xml-rs to parse the annotations
quote to generate the rust library code
proc_use for separating large modules into different files (these files will get seriously huge if not seperated)

More will be needed obviously, but I've had positive experiences with the ones above.

All this really sounds perfect, don't let me stand in the way. You definitely have a much stronger vision for this than I.

dropping the dependency on javascript sounds great
I'll research more into the CLDR
focusing english sounds good for now!

Added to you have access to the repo!

Thanks for the access! I have a fork going right now and will finish work soon. First goal is to remove the JS dependency and generate an identical library to what's already here. I have a lot of free time coming up so I'll probably binge code and finish in a few days... Hopefully.


fix Avoid throw unknown UserProperty after FE rollback versio…

Proposed changes
pick #28325
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Simulation breaks after a period of time

Describe the bug

When the simulation is open for an extended period of time, at some point it goes blank, only showing the robot.
This has only been confirmed to happen for the Robotino plugin, but it might be an issue on other systems too.
The simulation works again after closing and reopening it.

To Reproduce
Steps to reproduce the behavior:

Go to robotino
Open simulation
Wait until you see an error

Expected behavior

The simulation should never disappear.

Screenshots

Added later when reproduced

Device information

OS: Linux 20.04
Chrome
Touchscreen monitor was attached

Additional context

This needs testing for other systems
There might be a difference in how frequently it happens when the simulation is idle or running.

I switched all energy savings options on my win10 laptop off and run simulations of robotino and ev3 programs for more than 1 hour. Everything was fine.

not reproducable


Game crashes on startup

It crashes the game before it can even load, I've tried having just this mod in the folder and it still crashes. I don't know what I'm doing wrong if anything at all.

It crashes the game before it can even load, I've tried having just this mod in the folder and it still crashes. I don't know what I'm doing wrong if anything at all.

Just found out it's optifine and architectury having compatibility issues, so I just gotta wait till either one of them updates it for the other


Carriage return (% / ^M) added to every entry

All my unencrypted entries in /run/secrets/... have a carriage return symbol added to the end of them making my passwords fail (I use them to authenticate my email), because a "^M" is added to the end of them, when neomutt/aerc is trying to authenticate with IMAP servers.
I get the passwords via the passwordCommand cat /run/secrets/.... When I just try the command in the terminal, a "%" symbol is added to the end of the password.
Is this an issue with sops/sops-nix, or is this an issue with my passwordCommand?

A fix is to run the output through dos2unix, even though nothing is ever input from Windows


SDK version needs an update

When scanning the expo QR in the Readme file the following screen appeared, stating that the project uses SDK 48.0 but the new version of Expo Go supports only SDKs 49.0 and 50.0.

Is this issue assigned?


architecture: log redeclared in this block

queue.go has a var log = logging.MustGetLogger("BEANSTALKG") and job.go imports github.com/labstack/gommon/log, which causes the conflict.
Kill the github.com/labstack/gommon/log import?

We do not currently import github.com/labstack/gommon/log anywhere right now. Could you explain?

My emacs buffer maintained the copy of job.go with with gommon/lock.  Apologies.


Device Discovery not working with a Tempest station

Home Assistant Version?
20221213.1
Add-On or Self Hosted?
Self managed Container
Weatherflow2MQTT version?
3.1.3
What happened?
I tried to setup your MQTT bridge as docker container.
As far as I can tell it is working in general, but I cannot see any devices in Homeassistant even after waiting a few hours.
As far as I can tell there a no /config event emitted. When I subscribe to homeassistant/# on my MQTT Broker I can see a few homeassistant/sensor/weatherflow2mqtt/weather/attributes messages but no /config
Am I doing something wrong or how can I trigger the /config events?
Relevant log output
No response

I guess it has something to do with my network configuration. As far as I understand is that the UDP signals are not arriving.
I am running the container on a NAS with multiple network cards. The primary connection is on the same VLAN as the WeatherFlow Bridge.  The trick was to really use -p <IP_ADDRESS>:50222:5222/udp
I used the IP Adress of the network card I wanted to enforce instead of the general one but that does not work as expected.
Now the UDP packages are coming in.


Instructions don't seem to be working

I've been using the technique documented here to integrate with Dynamics for 2 years. This however stopped working a couple of days ago, either because of a change on Dynamics or (more likely) Postman. Can you please verify with the latest version of Postman?
I'm getting a popup error with the following, when I try to get a token with OAUth 2.0:
Error
An error has occurred.Try this action again. If the problem continues, check the Microsoft Dynamics 365 Community for solutions or contact your organization's Microsoft Dynamics 365 Administrator. Finally, you can contact Microsoft Support.

Timestamp: 12/4/2020 7:51:59 PM UTC
Service Request Id: ddd7349f-0cbf-4552-a366-a9570948fb5a

in this URL: https://xxxxx.crm.dynamics.com/_common/error/errorhandler.aspx?BackUri=&ErrorCode=&Parm0=

Error Details%3A An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.&RequestUri=%2Fdefault.aspx%3Fresponse_type%3Dcode%26state%3D%26client_id%3D51f81489-12ee-4a9e-aaae-a2591f45987d%26scope%3D%26redirect_uri%3Dhttps%253A%252F%252Foauth.pstmn.io%252Fv1%252Fcallback%2Fdefault.aspx&user_lcid=1033
And the details part says:
Tenant Id: 72f988bf-xxxxxxx
Url: https://xxxxx.crm.dynamics.com
Organization Id: aeae68a5-xxxxxxxxxxx
Type: Secondary
Unique name: xxxxxx
Created on: 7/23/2014 5:36:49 PM
State: Enabled
Security Group: 00000000-0000-0000-0000-000000000000

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 80534df2-e9da-fd0a-80bf-4bfc0dbb2fcc
Version Independent ID: b8053ca7-e385-2f0b-06af-81b908f345a1
Content: Set up a Postman environment (Microsoft Dataverse for Apps) - Power Apps
Content Source: powerapps-docs/developer/common-data-service/webapi/setup-postman-environment.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: jdaly

Not long ago Postman shipped  version 7.35 which broke OAuth for everyone.
I know I had to completely uninstall the client, then install version 7.34
Details are here
But they seemed to have fixed it and I'm using version 7.36 now with no serious issue.
But your case may be different:
The error in your URL (after decoding) looks like this:
https://xxxxx.crm.dynamics.com/_common/error/errorhandler.aspx?BackUri=&ErrorCode=&Parm0=

Error Details: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.&RequestUri=/default.aspx?response_type=code&state=&client_id=51f81489-12ee-4a9e-aaae-a2591f45987d&scope=&redirect_uri=https://oauth.pstmn.io/v1/callback/default.aspx&user_lcid=1033

That redirect_uri doesn't look right, and the error is coming from the web application rather than from the API endpoint.

Make sure your environment variables look like this:

Note that the callback variable is 'https://callbackurl'.

Then check that the authorization tab uses those variables, Grant Type is Implicit.

Postman changed their UI so it is different from the topic. We can't always maintain this to keep up with every change PostMan does, but the core ideas are the same.
I've found that when creating a new tab, I have to reconfigure the Authorization each time. I don't think I needed to do that before..

@lokijota Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue.


Installs into site-packages/data

On windows pip install tensorflow puts imagenet_classes.py into site-packages/data.
This seems like a pretty generic package name to write into... Was this intended?

data/imagenet_classes.py contains imagenet label list which will be used in VGG, Inception examples.

Can you shift it to site-packages/tensorlayer/data instead of site-packages/data? At the moment if I try to import from my own data.py file the data package installed by tensorlayer is used instead.

I moved data folder into example folder.


Dart does not throw error when analyzer fails with "type_argument_not_matching_bounds" error for functions

Dart SDK Version: 2.0.0-dev.64.1
OS: Windows 10 (64 bit)
The following test example tries to create a Function object with incorrect type parameter. It produces type_argument_not_matching_bound in analyzer and passes with dart:
class A {}
typedef F<X extends A> = X Function();

class B {}
B tryme() {}

main() {
  F<B> f = tryme;
}

Sample output is:
$> dart --version
Dart VM version: 2.0.0-dev.64.1 (Thu Jun 21 08:47:55 2018 +0200) on "windows_x64"

$> dartanalyzer test.dart
Analyzing test.dart...
  hint - This function declares a return type of 'B', but doesn't end with a return statement at test.dart:5:1 - missing_return
  hint - The value of the local variable 'f' isn't used at test.dart:8:8 - unused_local_variable
2 hints found.

$> dart test.dart

$>

I believe dart should throw error here.
Please note that there is are similar issues about incorrect class parameters and mixins: #33585, #33624.

@eernstg

I agree that the VM should emit a compile-time error for that bounds violation; given that #33308 deals with that topic in one place I'll add a comment there that this issue contains additional examples for that topic.
But the analyzer also fails to emit a compile-time error, and it is now the purpose of this issue to get that fixed (relying on #33308 for fixes in the VM).

Analyzer now reports an error here:

error • 'B' doesn't extend 'A'. • a.dart:8:5 • type_argument_not_matching_bounds

Analyzer now reports an error here:

error • 'B' doesn't extend 'A'. • a.dart:8:5 • type_argument_not_matching_bounds


Will d455 have slam capabilities similar to t265 ?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ D455}

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
Raspberry Pi/ NVIDIA Jetson / Android

SDK Version
{ 2}

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Smartphone/VR/AR}

Issue Description
< question / feature request >
Since t265 is on the market for over a year now, and its a very unique and promising piece of a hardware, there is some demand in the area of mobile XR devices for more versatile device, like t265 with rgbd sensor. Right now to rely on realsense full spectrum of capabilities one have to strap two chunky sensors to a mobile device : t265 and any of the d400 (omitting d435i because two imu's on one device would be rather pointless) increasing the weight of the device even more.
Considering that d455 specs are almost perfect for slam there could be a version with intel movidius inside.

Hi @TheBricktop   Apologies for the delay in responding.  The RealSense D455's SLAM will be the same as the D435i model, and it will not perform SLAM in the way that the T265 does.
Intel test numerous different configurations of camera hardware. They do not comment on products that do not exist yet though.

So there will be slam, that's a good news, of only there was a way to use it outside Ros.

Yes, the D455 has the IMU that the D435i has.
I recalled a D435i compatible SLAM project by MIT called Kimera and found the link to the details.
https://support.intelrealsense.com/hc/en-us/community/posts/360043774354-Highlighting-a-tutorial-for-Metric-Semantic-SLAM-with-Kimera-and-RealSense-D435i-
At 11 minutes 25 seconds into the YouTube video at the above link, it says that Kimera can be installed without ROS, though it takes longer (30 minutes) and more skill than an easy ROS based installation (5 minutes to begin compilation, 20 minutes to complete).

Problem with Ros is that it's mostly good for laboratory and experimental projects, for ar vr  especially now when mobile headsets take over its not very viable, correct me if I'm wrong but I don't see any way to port Ros based installation to for example unity Android.

There is a free ROS wrapper for Unity created by Siemens, though I do not know of any cases where it has been used with RealSense.
https://assetstore.unity.com/packages/tools/physics/ros-107085
https://github.com/siemens/ros-sharp/wiki

@TheBricktop Yes please intel make a Depth + tracking camera with the capabilities of both the T265 and D400 series all in one device.
I am sure I am not the only one who is really wishing for such a product.


URL in description and README broken

The URL in the repository description and README are pointing to GH-Dev-Progress-2's copy of the game instead of yours.
Please fix both, so they point to your copy of the game at https://GH-Dev-Progress-2.github.io/github-games-kunalberlia.

fix issue#1


Fix org.junit as resolution:=optional

wicket-core and the wicket-util bundle import junit packages, those should be resolution:=optional
getting issue using wicket library as bundles
refer issue :
http://apache-wicket.1842946.n4.nabble.com/wicket-core-and-util-OSGI-wiring-problem-with-junit-framework-with-test-environment-td4680840.html

@SubrahmanyamSistha it seems you are trying to merge fix for wicket-9 to wicket-8 branch ...
I believe this PR need to be closed and corrected


How to connect to a separate db?

Hi all;
I have found a same question in gitter but it did not be answered. So I have to ask again here.
I have a rails application that makes a connection to the different databases.
The model is like this:
# Bar Service
class Billing
   connects_to database: { writing: :foo_service, reading: :foo_service}
   has_closure_tree
end

Fetching records in Bar Service works fine. However, when I try to create an object it fails with: ActiveRecord::StatementInvalid: PG::UndefinedTable: ERROR: relation "billing_hierarchies" does not exist.
I create billing_hierarchies in foo service, and I do not hope to create billing_hierarchies  in bar service.
So is it a clean way to connect the closure tree to a separate db?
Thank you.

This is intersting bug. I will work a fix in the day.
Basically the hierarchy table should inherit the connection from the base table.

Thanks for such a quick reply!

@seuros I have made some changes to the code for this issue. PTAL. Thank you. #384

@shawn-zhuxy , i will push a pre-release today.

@shawn-zhuxy , i will push a pre-release today.

Thank you!


Better README in english

Translated current README.md into english and make it more structured.
Although, there is missing docs such as

How to use hdnnp
etc...

I won`t use codes related to photons, so please add docs if needed in other Pull Requests.
Thank you.
NOTE: This can be just a helper or reference for someone who is trying to make doc from now.

日本人学生が使うことも多そうなので、末尾か別ページに日本語README残すってできますか？

承知しました
README.ja.mdで残しておきます。


display heading classes don't scale based on font-size-base

I've noticed that while customizing bootstrap themes for my sites that the display classes don't scale their size based on the $font-size-base value, which can lead to some strange situations and the need for additional customization. Example:

The fix for this seems to be relatively straightforward but I am not sure if it's intentional or if there's other side effects. Or, if there's other parts that should scale as well such as the navbar padding in the example above. I'd be happy to open a PR if you'd like.

Duplicate of #22104.

Ah, good deal, I didn't see the dupes when I searched. Cheers!


Create instance of Aspect::Base and apply to multiple targets(classes/instances)

Then aspect instance can be used like a regular class, have instance variable, methods and can be called from advice logic, e.g.
class TestAspect < Aspector::Base
  attr_accessor :call_count

  def initialize
    @call_count = 0
  end

  before aspect_arg: true do |aspect|
    aspect.call_count += 1
  end
end

a = TestAspect.new
a.apply A, method: 'do_a'
a.apply B, methods: %w(do_this do_that)

Closing - once travis is done I will merge this. It required some additional work not to break existing API to much and now instead of accessing aspect directly we will have to use interception_arg instead of aspect_arg and just |interception| interception.aspect - that way we can have access to both interception options and aspect options - so the "default" aspector options API won't be broken


FBX Import bug fixes with special character in mesh names

Bug fixes for FBX import.

If the model name uses ":" in their mesh names, it should import its blendshapes.

NSFW - https://sketchfab.com/3d-models/sci-fi-girl-v02-walkcycle-test-07ac452d75cc47969234b224422ea85d
The blendshape does not move.

Thanks!


"Error: could't find the site key" fix

This seems related to issue https://github.com/JimmyLaurent/cloudflare-scraper/issues/28
@KumpelStachu this fix is not working for me (can be reproduced using torrent-search-api with YGG provider).
Something has surely (but sadly) changed at Cloudflare's.

Hcaptcha bypass was removed from V2.


[SPIKE] Research why Epiphany nodes hang when memory is overcommited

Is your feature request related to a problem? Please describe.
It seems that when k8s resource limits are incorrectly defined and memory is overcommited by applications user loses all ssh access to Epiphany VMs. This indicates that operating system is not protected from memory and cpu overusage. Usually it's not possible to debug such machines, restart is required and cluster becomes unstable. Since OS is not protected there is also security risk of possible denial-of-service attacks. :scream:
I suppose it's likely systemd slices are not correctly configured on Epiphany VMs in 0.8 release (at least).
Describe the solution you'd like
What we need to do in this spike is decide what is the culprit here and research ways to prevent such situations in the future. We need confirmation also that this situation happens.
Describe alternatives you've considered
None.
Additional context
Logs found on affected nodes:
Killed process 13428 (java) total-vm:14218144kB, anon-rss:2096560kB, file-rss:3684kB, shmem-rss:0kB, UID:0 pgtables:25228kB oom_score_adj:998

kernel: [200403.323264] memory: usage 3145696kB, limit 3145728kB, failcnt 17191

dockerd[14005]: time="2020-11-25T21:40:40.609327299Z" level=warning msg="Your kernel does not support swap limit capabilities,or the cgroup is not mounted. Memory limited without swap."

 kernel: [ 0.289359] PM: Registered nosave memory: [mem 0x00000000-0x00000fff]

I was basically unable to reproduce the issue in a generic way. I tried Ubuntu and JVM versions exactly as on the faulty environment:
  storage_image_reference:
    publisher: Canonical
    offer: UbuntuServer
    sku: 18.04-LTS
    version: 18.04.202006101

openjdk version "11.0.5" 2019-10-15
OpenJDK Runtime Environment (build 11.0.5+10-alpine-r0)
OpenJDK 64-Bit Server VM (build 11.0.5+10-alpine-r0, mixed mode)

Epiphany's Kubernetes behaved well in every attempt to crash node VMs. I think the next step would be to work closely with the team that experienced the issue and carefuly observe what their software does. :thinking:
@to-bar thank you for very nice memory stress testing toolset! :+1::heart_eyes:

@sk4zuzu what is next step here in this task? Why is it blocked?

@sk4zuzu what is next step here in this task? Why is it blocked?

I got vpn access, I'm looking at the cluster when it all happened because "generic" tests showed no issues. I can move it to in progress now I think and continue examining the cluster.

I got vpn access, I'm looking at the cluster when it all happened because "generic" tests showed no issues. I can move it to in progress now I think and continue examining the cluster.

✔ epicli apply
✔ epicli upgrade from 0.6, 0.7 HA, 0.8
After patching it looks to be working properly.

✔ epicli apply
✔ epicli upgrade from 0.6, 0.7 HA, 0.8
After patching it looks to be working properly.


tortoise: update opinion on the terminated layers within sliding window when received new data

depends on https://github.com/spacemeshos/go-spacemesh/issues/3569
once layer is verified we do not revisit opinion on that layer unless rerun is triggered. it was assumed that syncer will trigger a rerun when it notices a good reason for it. opinion may change because of the following cases:

hare output changed within hdist
node downloaded atxs with additional ballots, previously counted ballots won't be sufficient to cross global threshold
ballots from higher layers voted differently from the previous layers and changed threshold for already terminated layer
counting delayed ballots changed opinion on already terminated layer

from that list above only 2nd case can be tracked successfully by syncer, in other cases it is either impossible (4th) or inefficient (1st and 3rd) to trigger full rerun.
implementation plan
revisit opinions on terminated layer if changes are within sliding window
epoch weight
it is already computed in current implementation, but there are some changes:
// float64 is not used in code
func expectedWeight(epochs map[uint32]uint64, target, last layer) float64 {
  weight := 0
  if target.epoch() == last.epoch() {
    weight = epochs[target.epoch()]
    return weight*(last.ordinal()-target.ordinal())/target.epoch().length()
  }
  weight += epochs[target.epoch()]*(target.epoch().length() - target.epoch().ordinal()) / target.epoch().length()
  for epoch := target.epoch() + 1 < last.epoch(); epoch++ {
    weight += epochs[epoch]
  }
  weight += epochs[last.epoch()]*(last.epoch().ordinal() / last.epoch().length())
  return weight
}

OnAtx handler
add OnAtx handler to the tortoise. every time when tortoise receives new atx update expected weight by adding full atx weight or a fraction, using same computation as above. OnAtx is expected to be called before OnBallot that references such atx.
related tortoise state
type state struct {
  // weight of individual atxs in the sliding window
  atxs map[id]uint64
  // referenceWeight is computed by dividing atx weight by an eligibility count when reference ballot is received
  // to get ballot weight - reference weight is multiplied by eligibilities attached to the ballot
  referenceWeight map[id]float64
  // weight of all epochs (only atxs that are were used within sliding window)
  epochs map[uint32]uint64
  // if missed weight is larger than 1/3 of epochs weight - trigger state reload (rerun from scratch)
  // this is specifically for atxs that weren't counted within sliding window
  // we rely on p2p layer and syncer to deduplicate this
  // but even if it doesn't - it just trigger state reloading prematurely - which is not a huge problem
  missed map[uint32]uint64
}

changes in the counting weight
count weight for layers past verified layer within whole sliding window. notify mesh to perform a revert if opinion changed.
full tortoise doesn't need any changes. but remember to profile implementation how long it takes to count votes from a single ballot for 2 000 - 10 000 previous layers.
verifying tortoise needs to be refactored avoid changes to the total good weight when layer is verified.
type verifying struct {
  // total weight of all good ballots
  total float64
  // uncounted good weight specifies how much weight from total should not be counted
  // for the keyed layer
  // intuitively 'total' includes weight from all layers between [0, 100]
  // but from that weight all layers below 51 can't vote for layer 50
  // so uncountedGoodWeight for layer 50 is a sum of all good ballots in layers [0, 50]
  uncountedGoodWeight map[uint32]float64
}

@countvonzero i think existing interface for rerun was based on misinformed design. we definitely can't have just that interface. and most likely that interface is not even needed, because we can't recover from partition that lasted longer than sliding window anyway

@countvonzero i think existing interface for rerun was based on misinformed design. we definitely can't have just that interface. and most likely that interface is not even needed, because we can't recover from partition that lasted longer than sliding window anyway

i am not sure which interface you are referring to. do you mean this one?
func rerun(ctx context.Context) error

?
in my prototype, i changed this to
func rerun(ctx context.Context, from types.LayerID) error

my plan is:
for every ballot, we do in the following order

OnAtx(ballot.atxid, ignore): tortoise keeps track of only those ignored ATXIDs
OnBallot(ballot): weight is set to 0 if its atx is ignored

and there are two scenarios that will trigger rerun()

when syncer sees that is hash differ at layer N from a heavier weighted peer, and determine it should rerun from layer N
when tortoise accumulated enough ignored ATX weight in memory, and determine it should rerun from beginning of that epoch

yes basically about rerun

func rerun(ctx context.Context, from types.LayerID) error

i don't understand how this interface can be supported using current implementation. rerun can start either from genesis, or within sliding window. and this issue describes what we need to change for it to work within sliding window.

and there are two scenarios that will trigger rerun()

those two are not sufficient. as i mentioned opinion may change for other reasons

when syncer sees that is hash differ at layer N from a heavier weighted peer, and determine it should rerun from layer N

how do you know the weight? it was just communicated over rpc? i don't think that we can use this

rerun can start either from genesis, or within sliding window

if rerun can start from genesis (which i assume is outside the sliding window), why can't it start from a random layer in this range (genesis, windowStart)?

those two are not sufficient. as i mentioned opinion may change for other reasons

in the scenarios you listed

node downloaded atxs with additional ballots, previously counted ballots won't be sufficient to cross global threshold

i think this is consistent with when tortoise accumulated enough ignored ATX.

ballots from higher layers voted differently from the previous layers and changed threshold for already terminated layer
counting delayed ballots changed opinion on already terminated layer

i don't understand how these could happen if known ATX weight remain constant. i thought the security assumption is, once a block cross the global threshold, it's final until more ATX weight are discovered.

when syncer sees that is hash differ at layer N from a heavier weighted peer, and determine it should rerun from layer N

how do you know the weight? it was just communicated over rpc? i don't think that we can use this, anybody can trigger rerun by communicating high weight and a different hash

so i'm adding fetch API (direct P2P request) to support hash resolution. when syncer finds that it has a different accumulated hash from a peer X, it requests the list of ATXs from X. the node needs to determine how much new ATXs weight X introduced and whether the weight is enough for it to change opinions. whatever weight X claims it has, it needs to back it up by the actual ATXs.
note that syncer currently download ATXs in two ways

during the catch up sync, it download ATXs before everything else (ballots/blocks) to the current epoch.
at the first layer of every epoch, it download ATXs from the last epoch.

if rerun can start from genesis (which i assume is outside the sliding window), why can't it start from a random layer in this range (genesis, windowStart)?

tortoise votes encoding is recursive. there might be a way to bootstrap tortoise from arbitrary layer, but there are many assumptions that needs to be handled. currently such interface is not supported

i don't understand how these could happen if known ATX weight remain constant. i thought the security assumption is, once a block cross the global threshold, it's final until more ATX weight are discovered.

there is no such assumption. there is an assumption that if global threshold is crossed and adversarial weight is cancelled - node still crosses local threshold. but otherwise crossing global threshold after counting votes from one layer doesn't guarantee any finality
every ballot votes on all previous layers, therefore opinion may change because of voting even if crossed global threshold. moreover global threshold total value depends on the number of layers that can vote on a layer that supposed to be verified

tortoise votes encoding is recursive. there might be a way to bootstrap tortoise from arbitrary layer, but there are many assumptions that needs to be handled. currently such interface is not supported

thanks for the explanation. do you think it can be simplified and made possible after mutable ballot is made possible?

every ballot votes on all previous layers, therefore opinion may change because of voting even if crossed global threshold. moreover global threshold total value depends on the number of layers that can vote on a layer that supposed to be verified

right. my mental model keeps forgetting that global threshold is a moving target as layers grow.


How do you get an array of Models from a Collection?

How do you get an array of Models from a Collection, so that you can do stuff with the models?
For example, I'm trying to update a bunch of models in a transaction, but something like this does not work (Thing is a Model, Promise is from bluebird):
Bookshelf.transaction(function (t) {
    return Thing.fetchAll({transacting:t}).then(function (things) {
        return Promise.map(things, function (thing) {
            return thing.save(..., {transacting:t});
        });
    });
}).tap(function () {
    console.log("update complete");
});

Because things can't be passed to Promise.map.
So how would I do that?
My end goal is to update a bunch of models, in a transaction. Is there a way to do that?

All right, I found a solution, at least.
First step is to make a function that computes the value and saves it be a member of the bookshelf model. So, for the example from my post, I'd define the following function in Thing when extending the model:
... = bookshelf.Model.extend({

    ...

    updateSomeValue: function (options) {
        return this.save({
            value: computeSomeValueSync(this)
        }, options);
    }

});

Where options is the options to pass to save, which we can use to pass the transaction through. Easy enough. Then, we can do the equivalent of Promise.map with Collection#invokeThen, like this:
Bookshelf.transaction(function (t) {
    return Thing.fetchAll({transacting:t}).then(function (things) {
        return things.invokeThen("updateSomeValue", {transacting:t});
    });
}).tap(function () {
    console.log("update complete");
});

There, invokeThen essentially does what I intended to do with Promise.map -- returns a promise that becomes fulfilled once all the promises returned by Thing#updateSomeValue are fulfilled.
It's only mildly inconvenient in that I have to add the model method, but it does make a bit of sense at least. The interface is a little weird because the docs are tough to piece together. But, at least it's possible.
Still open to other ideas.

@JC3 I know this is closed, but another option is to use collection.slice() with no parameters, which returns a shallow copy of the underlying model array that backs the collection.


Missing URL tables

Once deployed and inspecting in Azure Storage Explorer, only able to see $Metrics tables but no URL tables.  Did I miss a step?

I was able to find answer in discussions post.  For anyone with same questions, tables are created if they don't yet exist.

This is correct. Tables are created at the first call when they don't exist.
[ ] Update docs to specify this. I Good place could be here

Updated

Once the deployment of AzUrlShortener is finished, Validate the deployment
This will create the table UrlsDetails. It will be empty but it will be present.
Then you can create your first Short URL by following the instruction here Create a new Short URL

The table with the clicks will be automatically created when a link will be used.

Thank you. Great project.


Share events bug

Issue Description
When we tap on share event and share via iMessage or sms it tries to attach the entire app in the message. (Specifically the cross culture block party event)
When sharing other events via iMessage or sms the url is duplicated
Steps to Reproduce / Code Snippets

go to an event (cross culture block party event)
share it via sms or iMessage
wait for the message to send and try to open it on the other device

Expected Results
When we share events it should only share the title and the link to the event itself
Additional Information
V6.8+

Was unable to reproduce the bug on android on a number of different messaging platforms including sms, Facebook messenger, slack and copying to the clip board. Sharing seemed to work fine.
Assigning this to you @g9singh to check if you can reproduce this on iOS.

This issue is being closed due to platform migration and may be revisited at a later date.


ERR_UNSUPPORTED_ESM_URL_SCHEME error

Hi azu!
First, thank you for the lib - really cool thing.
After upgrade to 5.0.1 (from 4.0.2) I've got such error on simple create-validator-ts run:
$> create-validator-ts "types.ts"
Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
    at new NodeError (node:internal/errors:393:5)
    at throwIfUnsupportedURLScheme (node:internal/modules/esm/resolve:1026:11)
    at defaultResolve (node:internal/modules/esm/resolve:1106:3)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:841:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ESMLoader.import (node:internal/modules/esm/loader:525:22)
    at importModuleDynamically (node:internal/modules/esm/translators:110:35)
    at importModuleDynamicallyCallback (node:internal/process/esm_loader:35:14)
    at createValidator (file:///C:/Users/user/code/project/node_modules/create-validator-ts/module/index.js:93:69) {
  code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
}

As I can see the line create-validator-ts/module/index.js:93  corresponds with create-validator.ts:123
Googling pointed out the problem is using the await import with path.resolve: https://stackoverflow.com/a/70057245
Editing the compiled file, the line 93 with adding 'file://' + ... removed the problem:
const { generator, generatorOptions = {} } = (await import('file://' + path.resolve(options.cwd, options.codeGeneratorScript)));

I'm not completely sure that this is the right fix so I'm not making a PR here. Howerer, I guess it would help you figure out how the problem may be fixed:)
Environment: windows 10, node 18

Ah, Probably, It is related to Windows file path.
We need to use pathToFileUrl.
https://github.com/nodejs/node/issues/31710

fixed in https://github.com/azu/create-validator-ts/releases/tag/v5.0.2

So cool!
Thank you really much!


⚠ sandbox-api.piste.gouv.fr healthcheck has degraded performance

In 33545b3, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 3454 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in f7325df.


Conflicting variable declaration on quickstart.js

The default function parameter conflicts with the variable declaration on line 35.
Steps to reproduce:
Uncomment line 35 on quickstart.js, change name to dbaah-devlrel-neos
Run node quickstart.js
Throws the following error
const bucketName = 'dbaah-devrel-neos';
        ^^^^^^^^^^

SyntaxError: Identifier 'bucketName' has already been declared
    at Object.compileFunction (node:vm:355:18)
    at wrapSafe (node:internal/modules/cjs/loader:1039:15)
    at Module._compile (node:internal/modules/cjs/loader:1073:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1138:10)
    at Module.load (node:internal/modules/cjs/loader:989:32)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12)
    at node:internal/main/run_main_module:17:47

If we remove the default value on line 17, it works.
Or, we can remove the const from the variable declaration and just assign the value: bucketName = 'your-unique-bucket-name'

Hi @danielduhh thank you for bringing this up! This is actually the intended design. If you look at where the sample region tags begin and end you will see they do not include the main function. If a user were to copy the sample from the documentation on the web there would not be a redundant declaration. If a user runs the sample locally from the repo the second declaration by default is commented out so that arguments can be passed from the command line into the main function.

Going to close this as it is the intended sample design. If you feel we should investigate a different approach feel free to reopen.


Stream.appendSelf(Function<Stream,Stream> mapper)

I believe we're a little closer to scala now :)

Wow, really nice. Never heard of this, yet.
I ask myself if a similar/slightly different functionality could be used to calculate primes with the sieve of eratosthenes?

I'm not sure, but I think the realization of this algorithm in a functional style requires a lot of memory.
But with addSelf() method searching for prime numbers may become easier
val primes: Stream[Int] = 2 #:: Stream.from(3, 2).filter(i =>
    primes.takeWhile(j => j * j <= i).forall(k => i % k > 0)
)

We can make refactoring PrimeNumbers.java because I believe it's completely non-obvious

Sound good. Would be great if you can do it!

#370 done


Esp crashs after delete pClient if there's a characteristic

I'm trying to resolve a memory leak in my code, so I started to use delete pClient;, but if there's a remote characteristic envolved, like in the code below, the system crashes.
 pRemoteCharacteristic = pRemoteService->getCharacteristic(charUUID);
    if (pRemoteCharacteristic == nullptr) {
      Serial.print("Failed to find our characteristic UUID: ");
      Serial.println(charUUID.toString().c_str());
      pClient->disconnect();
      delete pClient;
      return false;
    }

That's the error message:
 - Created client
 - Connected to server
 - Found our service
Failed to find our characteristic UUID: beb5483e-36e1-4688-b7f5-ea07361b26a8
CORRUPT HEAP: Bad head at 0x3ffe06a4. Expected 0xabba1234 got 0x3ffe0768
assertion "head != NULL" failed: file "/home/runner/work/esp32-arduino-lib-builder/esp32-arduino-lib-builder/esp-idf/components/heap/multi_heap_poisoning.c", line 214, function: multi_heap_free
abort() was called at PC 0x400e72f7 on core 1

Backtrace: 0x40091448:0x3ffc9440 0x40091679:0x3ffc9460 0x400e72f7:0x3ffc9480 0x400910d5:0x3ffc94b0 0x40084b26:0x3ffc94d0 0x400850e1:0x3ffc94f0 0x4000bec7:0x3ffc9510 0x4008dfa7:0x3ffc9530 0x400d6c23:0x3ffc9550 0x400d49ff:0x3ffc9570 0x400d50c5:0x3ffc9590 0x400d5466:0x3ffc95b0 0x400d5485:0x3ffc95d0 0x400d3816:0x3ffc95f0 0x400d1de9:0x3ffc9620 0x400d1e6e:0x3ffc9670 0x400d7621:0x3ffc96e0 0x4008e089:0x3ffc9700

Rebooting...

And that's my code, just a simple alteration in the BLE Client example:
/**
 * A BLE client example that is rich in capabilities.
 * There is a lot new capabilities implemented.
 * author unknown
 * updated by chegewara
 */

#include "BLEDevice.h"
//#include "BLEScan.h"

// The remote service we wish to connect to.
static BLEUUID serviceUUID("4fafc201-1fb5-459e-8fcc-c5c9c331914b");
// The characteristic of the remote service we are interested in.
static BLEUUID    charUUID("beb5483e-36e1-4688-b7f5-ea07361b26a8");

static boolean doConnect = false;
static boolean connected = false;
static boolean doScan = false;
static BLERemoteCharacteristic* pRemoteCharacteristic;
static BLEAdvertisedDevice* myDevice;

static void notifyCallback(
  BLERemoteCharacteristic* pBLERemoteCharacteristic,
  uint8_t* pData,
  size_t length,
  bool isNotify) {
    Serial.print("Notify callback for characteristic ");
    Serial.print(pBLERemoteCharacteristic->getUUID().toString().c_str());
    Serial.print(" of data length ");
    Serial.println(length);
    Serial.print("data: ");
    Serial.println((char*)pData);
}

class MyClientCallback : public BLEClientCallbacks {
  void onConnect(BLEClient* pclient) {
  }

  void onDisconnect(BLEClient* pclient) {
    connected = false;
    Serial.println("onDisconnect");
  }
};

bool connectToServer() {
    Serial.print("Forming a connection to ");
    Serial.println(myDevice->getAddress().toString().c_str());

    BLEClient*  pClient  = BLEDevice::createClient();
    Serial.println(" - Created client");

    pClient->setClientCallbacks(new MyClientCallback());

    // Connect to the remove BLE Server.
    pClient->connect(myDevice);  // if you pass BLEAdvertisedDevice instead of address, it will be recognized type of peer device address (public or private)
    Serial.println(" - Connected to server");

    // Obtain a reference to the service we are after in the remote BLE server.
    BLERemoteService* pRemoteService = pClient->getService(serviceUUID);
    if (pRemoteService == nullptr) {
      Serial.print("Failed to find our service UUID: ");
      Serial.println(serviceUUID.toString().c_str());
      pClient->disconnect();
      return false;
    }
    Serial.println(" - Found our service");

    // Obtain a reference to the characteristic in the service of the remote BLE server.
    pRemoteCharacteristic = pRemoteService->getCharacteristic(charUUID);
    if (pRemoteCharacteristic == nullptr) {
      Serial.print("Failed to find our characteristic UUID: ");
      Serial.println(charUUID.toString().c_str());
      pClient->disconnect();
      delete pClient;
      return false;
    }
    Serial.println(" - Found our characteristic");

    // Read the value of the characteristic.
    if(pRemoteCharacteristic->canRead()) {
      std::string value = pRemoteCharacteristic->readValue();
      Serial.print("The characteristic value was: ");
      Serial.println(value.c_str());
    }

    if(pRemoteCharacteristic->canNotify())
      pRemoteCharacteristic->registerForNotify(notifyCallback);

    connected = true;
}
/**
 * Scan for BLE servers and find the first one that advertises the service we are looking for.
 */
class MyAdvertisedDeviceCallbacks: public BLEAdvertisedDeviceCallbacks {
 /**
   * Called for each advertising BLE server.
   */
  void onResult(BLEAdvertisedDevice advertisedDevice) {
    Serial.print("BLE Advertised Device found: ");
    Serial.println(advertisedDevice.toString().c_str());

    // We have found a device, let us now see if it contains the service we are looking for.
    if (advertisedDevice.haveServiceUUID() && advertisedDevice.isAdvertisingService(serviceUUID)) {

      BLEDevice::getScan()->stop();
      myDevice = new BLEAdvertisedDevice(advertisedDevice);
      doConnect = true;
      doScan = true;

    } // Found our server
  } // onResult
}; // MyAdvertisedDeviceCallbacks

void setup() {
  Serial.begin(115200);
  Serial.println("Starting Arduino BLE Client application...");
  BLEDevice::init("");

  // Retrieve a Scanner and set the callback we want to use to be informed when we
  // have detected a new device.  Specify that we want active scanning and start the
  // scan to run for 5 seconds.
  BLEScan* pBLEScan = BLEDevice::getScan();
  pBLEScan->setAdvertisedDeviceCallbacks(new MyAdvertisedDeviceCallbacks());
  pBLEScan->setInterval(1349);
  pBLEScan->setWindow(449);
  pBLEScan->setActiveScan(true);
  pBLEScan->start(5, false);
} // End of setup.

// This is the Arduino main loop function.
void loop() {
  if (doConnect == true) {
    if (connectToServer()) {
      Serial.println("We are now connected to the BLE Server.");
    } else {
      Serial.println("We have failed to connect to the server; there is nothin more we will do.");
    }
    doConnect = false;
  }

  // If we are connected to a peer BLE Server, update the characteristic each time we are reached
  // with the current time since boot.
  if (connected) {
    String newValue = "Time since boot: " + String(millis()/1000);
    Serial.println("Setting new characteristic value to \"" + newValue + "\"");

    // Set the characteristic's value to be the array of bytes that is actually a string.
    pRemoteCharacteristic->writeValue(newValue.c_str(), newValue.length());
  }else if(doScan){
    BLEDevice::getScan()->start(0);  // this is just eample to start scan after disconnect, most likely there is better way to do it in arduino
  }

  delay(1000); // Delay a second between loops.
} // End of loop

Did you just try to disconnect? Library should delete pClient after that and probably this is the case.

Did you just try to disconnect? Library should delete pClient after that and probably this is the case.

First of all, thank you for the quick reply. And yes, i tried to disconnect, but i kept having memory leaks every time I connect with another device, so i tried to delete the pClient's

First of all, thank you for the quick reply. And yes, i tried to disconnect, but i kept having memory leaks every time I connect with another device, so i tried to delete the pClient's

another important thing, I can delete the pClient before obtain the characteristic, after this the system crashes

another important thing, I can delete the pClient before obtain the characteristic, after this the system crashes

There is many options where the leak can be. Recently i answered on similar issue and the suggestion was to create just 1 instance of callbacks and pass only pointers.
Im not saying there is no leak in library, it is possible there is one or more. The thing is this library is no longer developed/maintained.

There is many options where the leak can be. Recently i answered on similar issue and the suggestion was to create just 1 instance of callbacks and pass only pointers.
Im not saying there is no leak in library, it is possible there is one or more. The thing is this library is no longer developed/maintained.

Even when i don't use callbacks the leaking continues, after the fourth conexion more or less, the system stops.  Do you know another library or something?

Even when i don't use callbacks the leaking continues, after the fourth conexion more or less, the system stops.  Do you know another library or something?

Yes, Nimble library. Its using less flash and memory and is great replacement.
https://github.com/h2zero/NimBLE-Arduino

Yes, Nimble library. Its using less flash and memory and is great replacement.
https://github.com/h2zero/NimBLE-Arduino

Thank you so much @chegewara, you helped me a lot

Thank you so much @chegewara, you helped me a lot


Building to a .wasm file

Hi,
I'm trying to compile a C file to a .wasm file which my rust code then loads as a library in a wasmtime instance. I noticed that cc generally outputs a .a file. Is there a way to have it produce a .wasm file?
Thanks

Currently, no, this library only invokes gcc and ar, it doesn't produce final linked artifacts. For that reason there's no built-in way to build a *.wasm file with this library.


improve error messages for type mismatches

By mistake I passed a non-rarefied qza to qiime diversity beta-phylogenetic and got the following error, which I think it's not informative for a naive user. I think adding something like, you need to rarefy your table, will be easier ...
  File "/Users/antoniog/miniconda3/envs/qiime2-2017.4/lib/python3.5/site-packages/q2cli/commands.py", line 218, in __call__
    results = action(**arguments)
  File "<decorator-gen-151>", line 2, in beta_phylogenetic
  File "/Users/antoniog/miniconda3/envs/qiime2-2017.4/lib/python3.5/site-packages/qiime2/sdk/action.py", line 151, in callable_wrapper
    self.signature.check_types(**user_input)
  File "/Users/antoniog/miniconda3/envs/qiime2-2017.4/lib/python3.5/site-packages/qiime2/core/type/signature.py", line 282, in check_types
    " %r." % (name, spec.qiime_type))
TypeError: Argument to input 'table' is not a subtype of FeatureTable[Frequency] % Properties(['uniform-sampling']).

I agree that it would be really good to get better error messages in place for these types of situtaions. We should solve this in general for type mismatches.
What we need is some layer that interprets type mismatches based on the expected and the observed types (in this case, FeatureTable[Frequency] was observed/provided but FeatureTable[Frequency] % Properties(['uniform-sampling'] was expected) and maps that to an error message that is relevant for that type mismatch (in this case, something like You provided an unrarefied feature table, but a rarefied feature table is required.).


Add doctype

Specifying doctype is required by HTML specs
cheers!

i dont follow the html spec i follow the tadiweb spec

but without doctype it shows me just a blank page :shrug:


Difficulty with filenames including captions (emojis in particular?)

Describe the bug
I want the filename to be along the lines of: caption-date-username, but I am having difficulty saving profiles when I include caption in the filename. I think it's an issue with emojis. I'm running Windows 8.1.
I encounter same issue with both Windows executable and Python installation.
To Reproduce
Steps to reproduce the behavior:

instaloader --login * --filename-pattern={caption} profile cristiano
breaks upon the first post, which is https://www.instagram.com/p/BubxcOFANRv/, as of writing this. If I remove the caption from filename it works as expected.
It does not occur for all posts containing emojis, e.g. : https://www.instagram.com/p/BuXJorflk92/ works as expected.
Maybe it's only newer emojis or something?

Error messages and tracebacks
Above command gives following error:
Retrieving posts from profile cristiano.
[  1/2485] cristiano/Portugal. É de fibra que somos feitos!!!
💪🇵🇹⚽🏍 🧢
#afibradosportugueses.jpg Traceback (most recent call last):
File "main.py", line 442, in
File "main.py", line 438, in __main
File "main.py", line 413, in main
File "main.py", line 189, in _main
File "instaloader\instaloader.py", line 985, in download_profiles
File "instaloader\instaloader.py", line 447, in download_post
File "instaloader\instaloader.py", line 65, in call
File "instaloader\instaloader.py", line 227, in download_pic
File "instaloader\instaloadercontext.py", line 523, in get_and_write_raw
File "instaloader\instaloadercontext.py", line 489, in write_raw
OSError: [Errno 22] Invalid argument: 'cristiano/Portugal. É de fibra que somos
feitos!!!\n💪🇵🇹⚽🏍 🧢\n#afibradosportugueses.jpg'
[9444] Failed to execute script main
I should add that this command:
instaloader --login * --filename-pattern={caption} profile leomessi
fails at https://www.instagram.com/p/Bt3nFrJokI9/
which is currently the 5th post on his profile, with this error (different from the above one):
[  5/421] leomessi/Fui con todo y me sumé al reto por la última lata de @pepsi.
/// Going all in for the love of it as we compete for the Last Can Standing.  #F
ORTHELOVEOFIT.jpg Traceback (most recent call last):
File "main.py", line 442, in
File "main.py", line 438, in __main
File "main.py", line 413, in main
File "main.py", line 189, in _main
File "instaloader\instaloader.py", line 985, in download_profiles
File "instaloader\instaloader.py", line 447, in download_post
File "instaloader\instaloader.py", line 65, in call
File "instaloader\instaloader.py", line 227, in download_pic
File "instaloader\instaloadercontext.py", line 523, in get_and_write_raw
File "instaloader\instaloadercontext.py", line 489, in write_raw
FileNotFoundError: [Errno 2] No such file or directory: 'leomessi/Fui con todo y
me sumé al reto por la última lata de @pepsi. /// Going all in for the love of
it as we compete for the Last Can Standing.  #FORTHELOVEOFIT.jpg'
[4916] Failed to execute script main
It downloads properly when I remove caption from the filename.
So there may also be an issue with videos that contain illegal characters (e.g. '/') when caption is part of the filename?
Instaloader version
4.2.3
Additional context
I've fiddled with this a bit, and there seems to be a problem with certain emojis, or maybe more broadly illegal characters in Windows filenames such as / and *. It's strange though, as those emojis are not illegal characters, I can create filenames that contain those successfully. I don't know, it seems the caption parameter is failing in many cases.

I think the problem with the first one might be the newline character, \n. Can't confirm on my non-windows machines, but you could try adding .replace('\n', ' ') to line 110 in instaloader.py.
The second one seems to be a problem with /. On linux Fui con todo... /// Going all in... resolves to the directory Fui con todo... plus the file(s) Going all in.... I'm guessing windows treats the Fui...///Going... as two folders with no name at all (which is probably illegal), which results in the error. But the result isn't really what one would expect on either platform. @aandergr: / can't really be blacklisted, if one want's to create subdirectories, but it should be possible for _PostPathFormatter to sanitize "{caption}" (which ought to be the only problematic one) prior to the vformat.

I think it's a similar problem as in issue #197 having emojis on highlight titles or issue #199 where the colon is an invalid character (which is btw. closed but not really solved for Windows users)

I had a similar problem with illegal chars in stories, inside instaloader.py --> download_storyitem() , fixed with:
...
filename = dirname + '/' + self.format_filename(item, target=target)
filename = re.sub('[!@#$: ]', '', filename)  #FIX illegal chars go here
if (filename[-1]='.')            # FIX last char as . gives problems
filename = filename+'X';

For downloading tagged, also an illegal char is the folder :/tagged that gets created as Windows does not allow this folder. So:
    if target is None:
        target = profile.username + '/tagged' # FIX /: --> /

Using captions for filename patterns is not officially supported.
As @e5150 pointed out, the problems you are facing originate from the fact that the captions contain slashes ('/') and newlines ('\n') which are not allowed in filenames on some/most systems. The emojis should not be a problem, even on Windows. In order to replace the problematic characters you could override get_value() inside _PostPathFormatter like the following:
    def get_value(self, key, args, kwargs):
        """Replaces '/' with similar looking fullwidth solidus and on windows newline with space"""
        ret = super().get_value(key, args, kwargs)
        if not isinstance(ret, str):
            return ret
        ret = ret.replace('/', '\uff0f')
        if platform.system() == 'Windows':
            ret = ret.replace('\n', ' ')
        return ret

After doing so, there still persists a problem with the length of the caption strings which may exceed the maximal file length on some systems.
 We
therefore discourage to use captions in filenames.


Translate whitepapers page

Hi, I added Chinese translation for Rust Whitepapers. Please help me check my translation. Thanks!

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @steveklabnik (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

r?

r?


Peaceofmind123search forum complete

clicking on category on a thread results in error, indicating that we have a gap in our test suite (already tracked by: https://github.com/restarone/violet_rails/issues/111)


feat: wrap GitHub errors

Adds a wrapAsync helper to the github module that catches Octokit's RequestError and allows us to specify a error handler callback. The fall-through error handler wraps the caught exception in our own Error type so that we control the interface of the errors throw.
Adds a handler for creating releases that throws a DuplicateReleaseError if the release failed because the release tag already exists
Force updates some octokit dependencies to ensure we get certain error response fields set

Fixes #932
Fixes #945

The alternative is to try to parse the octokit errors in the calling program, but that's exposing the types of a transitive dependency and error prone if for whatever reason the underlying implementation changes.


refactor(sbb-title): extract base class

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

:exclamation: No coverage uploaded for pull request base (main@ffbcef7). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2594   +/-   ##
=======================================
  Coverage        ?   93.17%
=======================================
  Files           ?      315
  Lines           ?    25331
  Branches        ?     2057
=======================================
  Hits            ?    23602
  Misses          ?     1696
  Partials        ?       33

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


Unit Test Debugging with Syntax Error Requires VS Code Restart

Environment data
dotnet --info output:
.NET Core SDK (reflecting any global.json):
Version:   2.1.301
Commit:    59524873d6
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.301\
Host (useful for support):
Version: 2.1.1
Commit:  6985b9f684
.NET Core SDKs installed:
2.1.3 [C:\Program Files\dotnet\sdk]
2.1.4 [C:\Program Files\dotnet\sdk]
2.1.101 [C:\Program Files\dotnet\sdk]
2.1.103 [C:\Program Files\dotnet\sdk]
2.1.200 [C:\Program Files\dotnet\sdk]
2.1.201 [C:\Program Files\dotnet\sdk]
2.1.300 [C:\Program Files\dotnet\sdk]
2.1.301 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download
VS Code version: 1.24.1
C# Extension version: .
vscode\extensions\ms-vscode.csharp-1.15.2.omnisharp\1.30.1\OmniSharp.exe
Steps to reproduce
Create a fresh xunit application using dotnet new xunit
, then open the unit test and create a syntax error (this example missing semicolon after assert.true) and click debug:
    public class UnitTest1
    {
        [Fact]
        public void Test1()
        {
            Assert.True(true)
        }
    }

Expected  behavior
Syntax error returned, fix error and debug again.
Actual behavior
Syntax error is returned but clicking debug shows two debuggers running and then an error that the debugger is already started. Clicking debug again shows 3 debuggers running and same error, then so on so forth.
.NET Test Log Output
Debugging method 'test.UnitTest1.Test1'...

Microsoft (R) Build Engine version <IP_ADDRESS>72 for .NET Core
Microsoft (R) Build Engine version <IP_ADDRESS>72 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
Copyright (C) Microsoft Corporation. All rights reserved.

  test -> c:\Temp\test\bin\Debug\netcoreapp2.1\test.dll
  test -> c:\Temp\test\bin\Debug\netcoreapp2.1\test.dll

Build succeeded.
Build succeeded.
    0 Warning(s)
    0 Warning(s)
    0 Error(s)
    0 Error(s)

Time Elapsed 00:00:00.67
Time Elapsed 00:00:00.67

Error Message:
Failed to start debugger: "System.InvalidOperationException: Debug session already started.\r\n   at OmniSharp.DotNetTest.DebugSessionManager.VerifySession(Boolean isStarted)\r\n   at OmniSharp.DotNetTest.DebugSessionManager.StartSession(TestManager testManager)\r\n   at OmniSharp.DotNetTest.Services.DebugTestService.Handle(DebugTestGetStartInfoRequest request)\r\n   at OmniSharp.Endpoint.EndpointHandler2.d__18.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at OmniSharp.Endpoint.EndpointHandler2.<Process>d__16.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at OmniSharp.Stdio.Host.<HandleRequest>d__15.MoveNext()"

@xcellsoft The double , triple times diagnostics is coming because we are not disposing the event listeners properly in the extension, and the problem with the debugger already started is coming because the exceptions are not being handled properly. I am working on it and a PR will be out soon. Thank you for taking time to file the issue.

This will save allot of time, can’t wait. Thanks

What is the appropriate way to get this fix?
I see that setting "omnisharp.path": "latest" pulls down recent builds, looks like today it's 1.31.2-beta.98, how does this correlate with commits to master and/or what is the best way to get this patch to test it out?

@xcellsoft  Just setting "omnisharp.path": "latest" should work for you.

I would recommend opening a new issue rather than commenting on a closed issue. "omnisharp.path": "latest" would no longer be necessary or recomended, so I would restore that back to the default if you haven't already. Also, I am not sure if this is your problem, but the LiveShare extension cause a similar sounding issue. So if you have that installed, I would recommend trying without it.

LiveShare is not installed. my VSCODE version is Version: 1.28.2. I restored back to default. didn't help

Please open a new issue so that folks can try and help investigate.


update aslprep

Hi,
I'm checking that there is a new version of aslprep (0.4.0)
Is possible to include this in neurodesk?
Thanks a lot!
best,
Patricio

Dear @PatricioSolis-Urra,
yes, we can update this :) I just did that and once build and tested it will be there.
Thank you
Steffen

Dear @PatricioSolis-Urra ,
The new version for ASLprep is there. Can you test if it's all working as expected for you?
Depending on how you use Neurodesk you might have to click on the update button in the menu before the new entry gets added to the Application menu.
Kind Regards
Steffen

Thanks @stebo85 , it works.
best,
Patricio

Thank you for testing :) would you be keen to contribute an aslprep tutorial or example notebook using open data (https://www.neurodesk.org/tutorials-examples/) so that we can test if it works and neurodesk users can see how to use it?

Hi @stebo85,
Sounds great! I would love to contribute to this. I'll take some time to review everything and then begin working on the example over the next few weeks.
best,
Patricio


Add AND, NOT and paren expressions

Extend implementation to support AND, NOT and paren expressions, according to the JMESPath specification.
Left to do is to verify correct precedence of AND vs OR and other expressions by adding another test case. Also, I'm not sure if the binding power for lparen should be adjusted, or if parsing paren expressions with right binding power 0 is the correct thing to do. Any ideas for additional test cases regarding these things are welcome.

I opened PRs and issues in jmespath.rb the other day about the incorrect precedence of || and the lack of support for parentheses. I also fixed a bug in OR expressions (https://github.com/jmespath/jmespath.rb/pull/15).
I also tried to get this repo in sync with upstream, but there's been some changes that don't merge easily. We should get this in sync so that we can contribute features upstream, and get fixes too. There are some fixes and tests that are in upstream that we don't have.

This seems to be available in newer versions of JMESPath. Closing.


fix(chat-latest-article): Add threadId in the method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.778%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
262

Relevant Lines:
360

💛  - Coveralls


libxkbcommon: fix build on darwin with upstream patch

Motivation for this change
https://github.com/xkbcommon/libxkbcommon/pull/101 got accepted upstream.
Made more sense to just fix the portability issue in the test program.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I think this should go to staging to avoid mass rebuild on darwin

up to 1000 packages is a lot but not that bad. Darwin bootstrapping is currently broken on staging-next and staging. Considering this isn't a critical fix it can just wait in staging :)

Thanks @FRidh


change from oauth2client

Hi there,
I am new to oauth2 and changing oauth2client-related code to google_auth_oauthlib since the former is deprecated now.
Our private website compares a user's google+ id with our stored google+ ids to decide whether to give access.
However id_token of the two are different types with different info and I don't find a way to access google+ id (gplus_id) using your Credentials object. The two versions of codes are basically as follows. Could you please help with this?
oauth2client version
http://oauth2client.readthedocs.io/en/latest/_modules/oauth2client/client.html#OAuth2Credentials
from oauth2client.client import flow_from_clientsecrets
@app.route('/connect', methods=['POST'])
def connect():
    code= request.data
    oauth_flow = flow_from_clientsecrets("client_secrets.json", scope='')
    oauth_flow.redirect_uri = 'postmessage'
    credentials = oauth_flow.step2_exchange(code)
    gplus_id = credentials.id_token['sub']

google_auth_oauthlib trial version
http://google-auth.readthedocs.io/en/latest/_modules/google/oauth2/credentials.html#Credentials
import google_auth_oauthlib.flow
@app.route('/connect', methods=['POST'])
def connect():
    code = request.data
    flow = google_auth_oauthlib.flow.Flow.from_client_secrets_file('client_secrets.json', scopes='')
    flow.redirect_uri = 'postmessage'
    flow.fetch_token(code=code)
    credentials = flow.credentials

Thank you.

Glad you got it figured out!


[Script]: Domain Name Availabilty Checker

Aim
The python script uses GoDaddy's API to get all the domain names which are available according to the keyword given by user.
Details
All the results are stored in a .txt file, named after the keyword.
Setup

A virtual environment.
pip install -r requirements.txt
Generate your GoDaddy API KEY and Secret KEY from here
Paste the token in a .env file

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

GO Ahead @sabhisharma-ise