Tag counts are misleading

Screenshots

Steps to Reproduce

Create a post and tag it with tag from 2.2.2. Publish it
Create second post and tag it with same tag: tag from 2.2.2. Puslish it (this post ideally has to be created by another user with author permissions, but maybe will work for the same author)
Go to the public blog entry of any of the above posts and check the section with tags
The link to posts with current tags says 1 post → but links to a page with 2 entries

Technical details

Casper Version:
Ghost Version: 2.2.3
Browser Version: Google Chrome 69
OS Version: Ubuntu 18.04

Code effecting this bug
The filter for posts explicitly excludes post that is being displayed:
tags:{{primary_tag.slug}}+id:-{{id}}

But later on the wording explicitly says:
plural='See all % posts'

This could be a tricky one to fix because we can't have nested {{get}} helpers. Ideally the {{primary_tag}} data that Ghost exposes in the post context would include {{primary_tag.count.posts}} that way it would be possible to change the relevant line to:
{{#../primary_tag}}{{plural count.posts empty='No posts' singular='% post' plural='See all % posts'}} →{{/../primary_tag}}

A "quick fix" could be to change the wording to indicate more explicitly that it's "x related posts" rather than implying that it's "x total posts"

„See all x posts in the series” (or something like that) would be a better sentence


ZIDENTIFIER == LocalAccount issue & two convenience patches

In the HTML output strikethrough style is translated to <del>. This tag means that the text is marked as deleted, and it's not necessarily styled with a strikethrough. According to the MDN. IMO the striketrough font style in Notes may not always mean that, so a better fit could be <s> here, which just means striketrough.Patch: [1]

In the JSON output, it'd help processing if we could include a conforms_to (type) value. It would make it unnecessary for external processors to maintain a parallel list of possible types with the same set of properties. A new conforms_to property with image, audio, video, document, inline values would solve this. While looking at it, I've found an old/duplicate elsif branch for conforms_to_document, which can be deleted.Patch: [2] (there is probably a more elegant way)

FInally the critical, but fuzzy one: On some systems, the on-disk (--mac <path>) database (a one-to-one copy of the ~/Library/Group Containers/group.com.apple.notes/ tree, Notes version 15), says that the local Note attachments are in LocalAccount (via ZICCLOUDSYNCINGOBJECT → ZIDENTIFIER). apple_cloud_notes_parser interprets this as any other iCloud UUID and looks for the files inside <path>/Accounts/LocalAccount/{Media,Previews,Thumbnails}, but, there is no such directory on disk. Instead, these are present in bare <path>/{Media,Previews,Thumbnails}. There is no error while running, but the output is different and of course the files output directory is not populated. My solution is to symlink these three directories to where the script expect them to be, and it is also necessary to strip Accounts/LocalAccount/ from paths in the generated output (notably JSON). I'm not sure how to special-case this. The most naïve option is to omit Accounts/ if @note.account.identifier equals to LocalAccount. There are two other records with non-UUID identifiers: TrashFolder-CloudKit and TrashFolder-LocalAccount, though these are not touched by the export.

[1]:
diff --git a/lib/AppleNotesEmbeddedObject.rb b/lib/AppleNotesEmbeddedObject.rb
index 85ed073..8a87ae9 100644
--- a/lib/AppleNotesEmbeddedObject.rb
+++ b/lib/AppleNotesEmbeddedObject.rb
@@ -46,6 +46,22 @@ class AppleNotesEmbeddedObject < AppleCloudKitRecord
       add_cryptographic_settings
     end

+    # Fill conforms_to for convenience
+    tmp_uti = AppleUniformTypeIdentifier.new(@type)
+    if tmp_uti.conforms_to_inline_attachment
+      @conforms_to = "inline"
+    elsif tmp_uti.conforms_to_image
+      @conforms_to = "image"
+    elsif tmp_uti.conforms_to_audiovisual
+      @conforms_to = "video"
+    elsif tmp_uti.conforms_to_audio
+      @conforms_to = "audio"
+    elsif tmp_uti.conforms_to_document
+      @conforms_to = "document"
+    else
+      @conforms_to = "other"
+    end
+
     @logger.debug("Note<EMAIL_ADDRESS>Created a new Embedded Object of type #{@type}")

     # Create an Array to hold Thumbnails and add them
@@ -333,12 +349,6 @@ class AppleNotesEmbeddedObject < AppleCloudKitRecord
                                                             row["ZTYPEUTI"],
                                                             note,
                                                             backup)
-          elsif tmp_uti.conforms_to_document
-            tmp_embedded_object = AppleNotesEmbeddedPDF.new(row["Z_PK"],
-                                                            row["ZIDENTIFIER"],
-                                                            row["ZTYPEUTI"],
-                                                            note,
-                                                            backup)
           elsif tmp_uti.uti == "public.url"
             tmp_embedded_object = AppleNotesEmbeddedPublicURL.new(row["Z_PK"],
                                                                   row["ZIDENTIFIER"],
@@ -475,6 +485,7 @@ class AppleNotesEmbeddedObject < AppleCloudKitRecord
     to_return[:note_id] = @note.note_id
     to_return[:uuid] = @uuid
     to_return[:type] = @type
+    to_return[:conforms_to] = @conforms_to
     to_return[:filename] = @filename if (@filename != "")
     to_return[:filepath] = @filepath if (@filepath != "")
     to_return[:backup_location] = @backup_location if @backup_location

[2]:
diff --git a/lib/ProtoPatches.rb b/lib/ProtoPatches.rb
index 837c7cb..e59c1e8 100644
--- a/lib/ProtoPatches.rb
+++ b/lib/ProtoPatches.rb
@@ -227,7 +227,7 @@ class AttributeRun

     # Add in strikethrough
     if strikethrough == 1
-      html += "<del>" if (initial_run or previous_run.strikethrough != 1)
+      html += "<s>" if (initial_run or previous_run.strikethrough != 1)
     end

     # Add in superscript
@@ -315,7 +315,7 @@ class AttributeRun

     # Add in strikethrough
     if strikethrough == 1
-      html += "</del>" if (final_run or next_run.underlined != 1)
+      html += "</s>" if (final_run or next_run.underlined != 1)
     end

     # Add in underlined

Thanks for flagging this. I've always just used <del> on my sites, but I'm fine with changing it to better conform to the spec. (d59288fec6184bea2592cd3c14a6a81c2a8c6790)

I put this logic into AppleUniformTypeIdentifier so that it is in the same place as the conforms_to methods. I also added a few more and made other types explicit even if they're only being added in AppleNotesEmbeddedObject due to being handled separately. Let me know what you think of the bins. (e869efe6fac0927eb9a7c5327c67415765b3a6ec)

I'm going to need to play on my test system to reproduce this. Can you narrow down at all which "some systems" this occurs on? Does it have to do with the MacOS version? I am thinking of a fairly hacky elegant solution, but want to make sure I can effectively test it before I try.

@threeplanetssoftware Thanks so much for your updates, they look good indeed and worked perfectly.
This LocalAccount is present on macOS Monterey. This installation has been carried over from the times when Notes.app appeared in OS X Mountain Lion, so one suspect is that this has been added by an earlier Notes version and remained there ever since, for the "On My Mac" notes. The group.com.apple.notes.plist Preferences files has two properties that seems related: DidChooseToMigrateLocalAccount = true and DidMigrateLocalAccount = true. Overall, other than being old and used throughout OS versions and multiple machines, I can't report anything particular.

Thanks, I'm looking into this and believe I have a backup from Monterey to check. Can I ask if there is any iCloud account enabled on that version, or is it all purely the LocalAccount?

Yes, there is also an iCloud account enabled. That one has an UUID and its objects at the expected locations.

The good news is I have a MacOS computer set up the same as yours and I think I know how I would solve this if it was worth the time investment. The bad news is I can't recreate this issue even with a testbed very close to your setup. My Library/Preferences/group.com.apple.notes.plist has the same settings checked as yours:
{"CloudKitAccountStatus"=>1,
 "ThrottlingPolicyStartTime"=>692587188.601988,
 "DidMigrateLocalAccount"=>true,
 "DidChooseToMigrateLocalAccount"=>true,
 "ThrottlingPolicyCurrentBatchCount"=>0,
 "searchIndexingVersion"=>5,
 "CloudConfigurationPath"=>
  "/System/Library/[snip_long_path]/CloudConfigurations/Normal.plist",
 "DefaultNotesAccount"=>"DeviceLocalAccount",
 "ThrottlingPolicyCurrentLevelIndex"=>0,
 "ClientStateData"=>"[cut]"}

I added a few notes with images and embedded files, made sure that ZICCLOUDSYNCINGOBJECT.ZIDENTIFIER is LocalAccount for the local folder and... a 'LocalAccount' folder showed up in the Accounts folder right where it is supposed to be. It has the expected Previews, Media and FallbackImages folders within it:
[notta@cuppa LocalAccount]$ pwd && ls
/Users/notta/Library/Group Containers/group.com.apple.notes/Accounts/LocalAccount
FallbackImages  Media  Paper  Previews

Admittedly my MacOS version does not go back as far as yours, so this could be an issue faced by people with older installations carrying things forward. I don't like using "Won't Fix," but I'm also not sure how prevalent this issue is. Your symlinks might be the easiest answer, what do you think?

Thank your for taking the time going after this.
As for symlinks: Using them means that the backup (or live) folder needs to be written into. This is fragile, messes with timestamps and would fail on a read-only folder. They also cannot be left there, to avoid risking another tool (or Notes.app itself) getting confused by them. For these downsides, I'd happier with a different solution. If you haven't seen another system like this, it must be fairly rare indeed, but they certainly do exist. Here is a screenshot from a 2020 article that resembles the "bare" one: https://osxdaily.com/2020/01/15/where-notes-stored-locally-mac/ (This one lacks an Accounts folder altogether.)
Peeking into the Notes binary, there are some hints about "modern" and "legacy" LocalAccounts, though without reversing the code, that did not lead anywhere.
What do you think of a solution that, when encountering a LocalAccount UUID, first checks if there is such a directory under Accounts, and if there isn't one, resets the account root to the backup root? This seems fairly safe to do, at least for --mac inputs. Or, a command-line option to explicitly set the LocalAccount root. I can contribute a patch for these.
[ Found two more, possibly related settings via a defaults read dump: LocalNoteMigrationCompleted = 1, MailAccountMigrationCompleted = 1. Now I remember that Notes used to live inside the Mail.app before ML. ]

The legacy refers to when Notes were stored in HTML, whereas modern is the format currently seen. The migration settings you see are also present in NoteStore.sqlite files, but I don't think they are the key here. I suspect it is more order of operations where you had an account that was local before it was iCloud enabled, but I could be wrong. That is likely an edge case that will get more and more rare as time goes on, but given the additional URL you've provided, I'll support this case.
No need to submit a patch, I know how I'll implement it, just need a bit of spare time to do so. My hesitancy was that I'll end up touching (all?) of the embedded objects and will need to do a bit of testing to make sure I don't inject any regression bugs.
I'll leave it in a feature branch and let you know when it is ready so that you can affirm it works in your case, since I can't recreate the test.
Thanks for digging into this.

Sounds great and thank you for supporting this. Agreed on the circumstances of it, sadly I have no recollection on the order of these past events, but what you wrote is plausible. Possibly Apple wanted to avoid moving around a whole bunch of interconnected files throughout a past migration step.
I'm happy to make tests (or provide any more info), and looking forward to your updates.

Can you please try the 59-local-account-issue feature branch? I think it should work. The way I implemented this is, upon AppleNotesAccount creation, the AppleNoteStore checks to see if the account's folder exists. If not, it defaults to the folders being in the root of the app. This is not incredibly defensive programming, I don't check to make sure at least one of those folders DOES exist, but if the folders weren't needed, they'd be empty anyhow in the Accounts folder.
Please let me know the results of your tests.

I just run my tests and can confirm that your update does work correctly on this "odd" Notes folder (after deleting the local symlink hack and the /Accounts/LocalAccount stripping from the external JSON parser). Also thanks for the explanation, this scenario could very well be what actually happened here. Agreed that the extra directory check would be too defensive, depending on how Apple are creating them (on demand, for example.)
TL;DR: All good from my end!
Thanks again for your work on this.

Fixed in fde24c6d2d58b9096c2bd9dc01e6c97b7e8e5165.


Information related to AngularJS compressed in only one section.

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
Improve Repo
For resources
Description
After several times reviewing the two sections that we had in the document for AngularJS (Spanish), I decided that was better to have it all compressed in only one section.
Why is this valuable (or not)?
This will add value to the documentation because readers will be able to access the information more easily, avoiding moving through the document every time they want to review the documentation from section one to section two and vice versa.
How do we know it's really free?
N/A
For book lists, is it a book? For course lists, is it a course? etc.
N/A
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Please use the same categorizations as used in other lists. This change, creating a yet another categorization will be confusing for users. You can create an issue for discussion.


Failed validation: #/definitions/QuoteSummaryResult

Validation Error
Minimal Reproduction

Symbol(s) that it happened for

AAPl
F

Error Message

The following result did not validate with schema: #/definitions/QuoteSummaryResult
[
  {
    keyword: 'yahooFinanceType',
    message: "Got {}->null for 'number', did you want 'number | null' ?",
    params: { schema: 'number', data: {} },
    instancePath: '/incomeStatementHistory/incomeStatementHistory/0/totalOtherIncomeExpenseNet',
    schemaPath: '#/definitions/IncomeStatementHistoryElement/properties/totalOtherIncomeExpenseNet/yahooFinanceType',
    data: {}
  },
  {
    keyword: 'yahooFinanceType',
    message: "Got {}->null for 'number', did you want 'number | null' ?",
    params: { schema: 'number', data: {} },
    instancePath: '/incomeStatementHistory/incomeStatementHistory/1/totalOtherIncomeExpenseNet',
    schemaPath: '#/definitions/IncomeStatementHistoryElement/properties/totalOtherIncomeExpenseNet/yahooFinanceType',
    data: {}
  },
  {
    keyword: 'yahooFinanceType',
    message: "Got {}->null for 'number', did you want 'number | null' ?",
    params: { schema: 'number', data: {} },
    instancePath: '/incomeStatementHistory/incomeStatementHistory/2/totalOtherIncomeExpenseNet',
    schemaPath: '#/definitions/IncomeStatementHistoryElement/properties/totalOtherIncomeExpenseNet/yahooFinanceType',
    data: {}
  },
  {
    keyword: 'yahooFinanceType',
    message: "Got {}->null for 'number', did you want 'number | null' ?",
    params: { schema: 'number', data: {} },
    instancePath: '/incomeStatementHistory/incomeStatementHistory/3/totalOtherIncomeExpenseNet',
    schemaPath: '#/definitions/IncomeStatementHistoryElement/properties/totalOtherIncomeExpenseNet/yahooFinanceType',
    data: {}
  }
]

Environment
Browser or Node:
Node version (if applicable):
Npm version:
Browser version (if applicable):
Library version (e.g. 1.10.1):
Additional Context
yahooFinance.quoteSummary('AAPL' , { modules: [ 'incomeStatementHistory' ] })

Hi @currenpatel, this looks like #734 which was fixed in v2.9.1 and released on Feb 5th.  Can you confirm what version you're using, and the issue persists with the latest release?  Thanks.

Thanks that fixed it. I see that financial statement data still does not work though. Is there any update on that? Many Thanks

Thanks for reporting back, and glad the original issue is working.
Re financial statements, are you talking about https://github.com/gadicc/node-yahoo-finance2/issues/733?  Let's track there.  I posted a quick update there but in short, no working code yet.


Adding sections in between

I changed a partition coded as a phrase to a section and it didn't "push" the subsequent sections out. Meaning, what is now section 5 has the code of old section 4. It's still called "composition" and it should be "Improvisation". Which we can confirm because the phrase (coded a few days ago), still correctly says barhat (a type of improvisation).
Happy to describe if the above makes no sense. One way to test it is to go to one of your transcriptions, add and name/code 3-4 sections with some phrases. Then add a section in between existing sections (because you transcribed it later or because you realize you missed it before), or change a phrase to a section. See if the subsequent section names/codes carries forward.
If this turns out to be a reproducible glitch and not my user error, BUT it requires a complicated solve, I'm fine just disciplining myself to add sections sequentially and teleologically, from beginning to end.

I understand what you are describing, and am seeing the same thing on my end. I will look into it, hopefully not too complicated.


fspy: fix nix-env version parsing

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I don't know why this is necessary, but it doesn't seem to change anything. So, if it helps some thing, it's fine with me.

nix-env can't properly parse fspy-vX.X.X but fspy-X.X.X.


Provide feature to start sync validation of the form

Are you submitting a bug report or a feature request?
I need some action to make syncValidation of any form I want.
What is the current behavior?
Some code example to better understand the problem is there -https://paste.ofcode.org/QXxAiDtWbTZQZbf9wGavrP
The situation is I need each form have a unique name property. If some form has the same name like any another I should display error for this form.
Issue behavior:

User goes to the page.
Type hello form A on name field for form A.
2.1 It Fires onChange for form A and changes name
2.2 It Fires validation for form A and everything is OK
2.3 It Redraws forms
Type hello form A on name field B for form B.
3.1 It Fires onChange for form B and changes name
3.2 It Fires validation for form B and display error cause form A have the same name property.
3.3 It Redraws forms where form B has error
User changes name for form A to hello form A1
4.1 It Fires onChange for form A and changes name
4.2 It Fires validation for form A and everything is OK
4.3 It Redraws forms but form B still has error because it didn't fire validate method to check the name.

So the question is how can I fire validate method for other forms not only for form user changes. Submit is not OK because it marks fields as touched and it's not needed on my case.
What is the expected behavior?
I expect for some method like startSyncValidation for each form and let them validate names again after name on other form changed.

An action creator to start sync validation is a long requested feature, I'm pretty sure that we have some issues for this around here.
However, you could probably do what you want by providing the values of the other forms,
so that you can re-render forms unrelated to the field you just changed. This, combined with the shouldValidate setting of reduxForm.
I created an example for this idea: https://codesandbox.io/s/M8vmZYWPA
Can you check whether it helps you?

@gustavohenke oh man you made my day! The issue I worry about is lags because I can have more that 30 forms on the same page but it works!! Thank you so much!


Example visualization issue

Hi,
I have started having a look to streetscape but I have an issue with the viewer.
I have followed all the steps in the main readme to start the example, but after I started it and the web page pops up, it displays the interface for few seconds to then become all blank.
I inspected the web page and it says that there is nothing in the body after the interface has disappeared.
I have tried to change browser and I have tried to change OS (Ubuntu 18.04 and Ubuntu 20.04).
Here the requested dependencies versions:
-yarn 1.22.5
-node.js 10.23.1

The problem was running Ubuntu in the virtual machine. It works running yarn start-local instead of yarn start

The problem was running Ubuntu in the virtual machine. It works running yarn start-local instead of yarn start


AssertionError on adding a valid schema

Fails to add this schema to parser:
XMLSchema.xsd
An AssertionError dumps to console.
P.S. Thank you for the library and idea. Never found time to implement it myself.

I am having the same issue, according to this link, complexType can have id, name, abstract, mixed, block, final... but you are only checking name and abstract: https://github.com/peerlibrary/node-xml4js/blob/master/lib/xsd.js#L311

On which value does it fail? I think it would be easy to ignore id. But for abstract, block, final, and any I am unsure how exactly to map it. Not sure also if they do influence conversion at all. Remember, I am not making full schema parser, but just enough to know when to do an array and when not to, and to know to which type to convert a value at the leaves.

it fails on literally any value on any property other than id, name, mixed, because you asserting complexType.$ to be empty. if you only care about types then why not ignore other properties that don't contribute to resulting type?

Because I have not went through the whole list of other properties which can be there and think if they do or do not contribute to the resulting type.

Hi. I have problems using parser.addSchema.
I get
events.js:160
throw er; // Unhandled 'error' event
^
AssertionError: [ 'http://package.structure.to.my.schema/myxsdFilename', 'myxsdFilename.xsd', '' ]
At ...\node_modules\xml4js\lib\xsd.js line 517: assert(schemaLocation.length === 2, schemaLocation);
I guess I am missusing the method call? Thiss is how I call it:
var schema = fs.readFileSync('myxsdFilename.xsd', {encoding: 'utf-8'});
parser.addSchema('http://package.structure.to.my.schema/myxsdFilename', schema, function (err, importsAndIncludes) {....}
Could you please give me a hint?
Thanks in advance.

Hi. I have problems using parser.addSchema.
I get
events.js:160
throw er; // Unhandled 'error' event
^
AssertionError: [ '/C:/1_Entwicklung/folder1/folder2/schemaname',
'schemaname.xsd',
' ' ]
followed by
at .....\xml4js\lib\assert.js:10:5)
at......xml4js\lib\xsd.js:517:9
I guess I am missusing the method call? Thiss is how I call it:
var schema = fs.readFileSync('schemaname.xsd', {encoding: 'utf-8'});
parser.addSchema('/C:/1_Entwicklung/folder1/folder2/schemaname',  schema, function (err, importsAndIncludes) {....}
Could you please give me a hint?
Thanks in advance.


WFLY-5542 Use apache upstream JSTL instead of jboss fork...

... of java.net / apache.org one
It passes TCK (we ran the web/servlet/jsp/jstl/... related parts)
@wolfc @dpospisil Do you guys have any additional tests we could run to make sure this fixes all our issues including the known CVE?

It seems that the xalan in WF does not contain fix for CVE-2014-0107. That would explain why this can pass TCK. I'll have a look at it.

It is indeed the case. The wildfly xalan branch does not contain commit 10985d971a556d919498653fc51e0b82e88676bb. The BZ1081315 does not seem to be adressed in upstream at all.
That said there is nothing wrong with this PR. Just if we apply CVE-2014-0107 fix upstream, the TCK failure will pop up and st like this will be required https://github.com/jboss/jboss-jstl-api_spec/pull/10

how can we get that version of xalan to upstream? looking at the repo, 2.7.1.jbossorg-2 is the last one we have.
@n1hility what should we about xalan? even outside context of this PR.

we could probably go with http://mvnrepository.com/artifact/xalan/xalan/2.7.2 as it should work fine in modular environment in JDK8 with jaxp 1.7


Reintroduce Python::run and Python::eval

Should we also change code to a &CStr for both of these?

I think a &CStr makes a lot of sense here, yes. The current code will error with a ValueError on internal nut, which is pretty surprising to the user! 😬


Some WKT strings not working when loaded through Strategy.Fixed

If I load a single WKT MULTIPOLYGON through a direct call to Format.WKT.read, it works fine.  If I load the exact same string using Protocol.HTTP and Strategy.Fixed, it doesn't work.  If I wrap the MULTIPOLYGON in a GEOMETRYCOLLECTION, then it can be loaded through HTTP and Fixed.
In the Strategy.Fixed.merge function, I removed the && features.length > 0 part of the first "if" check, and this made my tests work fine.  It appears that GEOMETRYCOLLECTIONs are represented differently than MULTIPOLYGONs in the return from Format.WKT.read.  Through this change made my tests work, I don't know whether there was a good reason the "if" check was written the way it was or what the implications elsewhere might be.
Here is a code snippet.  The "multipolygon" file contains the same string defined in the JavaScript as "wktStr".  The "geometrycollection" file contains that same string but prepended with "GEOMETRYCOLLECTION (" and appended with ")".
<!DOCTYPE html>

  <meta http-equiv="content-type" content="text/html;charset=utf-8" />

I am experiencing a similar problem. Any potential fix for this?


sbt install issue

Hi,
I am having trouble installing the dependency.
Details:

Scala: 3.2.1
sbt: 1.8.2
organize-imports: 0.6.0

When I import the build into metals, I see this error:
2023.04.29 20:05:29 INFO  [error]   not found: /home/gregbrown/.ivy2/localcom.github.liancheng/organize-imports_3/0.6.0/ivys/ivy.xml
2023.04.29 20:05:29 INFO  [error]   not found: https://repo1.maven.org/maven2/com/github/liancheng/organize-imports_3/0.6.0/organize-imports_3-0.6.0.pom))), directCause=None)), directCause=None)), directCause=None)), directCause=None)

Likewise, when I run scalafix in the sbt console, I see the same error:
[error] stack trace is suppressed; run last update for the full output
[error] (update) sbt.librarymanagement.ResolveException: Error downloading com.github.liancheng:organize-imports_3:0.6.0
[error]   Not found
[error]   Not found
[error]   not found: /home/gregbrown/.ivy2/localcom.github.liancheng/organize-imports_3/0.6.0/ivys/ivy.xml
[error]   not found: https://repo1.maven.org/maven2/com/github/liancheng/organize-imports_3/0.6.0/organize-imports_3-0.6.0.pom
[error] Total time: 5 s, completed Apr 29, 2023, 8:09:13 PM

I may be doing something wrong whilst installing the dependency, as my build.sbt differs somewhat from the README:
ThisBuild / version := "0.1.0-SNAPSHOT"

lazy val scala3Version = "3.2.1"

lazy val server = (project in file("."))
  .settings(
    name         := "myproject",
    scalaVersion := scala3Version ,
    organization := "com.myproject",
    libraryDependencies ++= Seq(
      // other dependencies
      "com.github.liancheng" %% "organize-imports" % "0.6.0"
    )
  )

You should have been using scalafixDependencies, not libraryDependencies. In any case, OrganizeImports is now a built-in rule as of Scalafix 0.11.0 so this should not be required anymore.


Make Namespace core-compatible again

This resolves an issue where the core-compatible Namespace segment
allocation would erroneously use the non-compatible implementation
instead.
Resolves #52

Superseded by #54


Bump pulp versions to 3.11

Pulp-core goes to 3.11 and dependencies get updated to current versions
Bump various python modules to match
Update Makefile and readme to simplify requirements.txt update process (make update-requirements)

Works.
sauer@GRODD:/tmp> curl --noproxy '*' -Is https://xxx/repo_for_demo/Packages/t/testrpm-9-0.1-1.x86_64.rpm | grep location
location: https://other_xxx/pulp3-media/artifact/3d/337f03f5dfbf01124e05d87638aca193926def8c102f91e51e75262a654427?response-content-disposition=attachment%3Bfilename%3Dtestrpm-9-0.1-1.x86_64.rpm


Determine extension currency from the wallet

Context
The WM extension must determine the currency that it displays based on the currency denomination of the connected wallet.
TODOs

The extension uses a default currency symbol for the US dollar $
When a user connects their wallet to the extension, the extension must obtain & use the currency of the connected wallet.
This applies to the following form fields:

On the 'connect wallet' Settings page, the currency of the Amount
On the 'disconnect wallet' Settings page, the Remaining balance
On the page when monetization is active, the value that a user can enter to make a once-off payment/tip

Will mark this as cancelled - it's a duplicate of #113


use ignore_run_exports_from

run_exports specifies which package pins should be ignored (i.e. ignore dependency on python, exported from anything)
run_exports_from specifies which package's exported pins should be ignored (ignore dependencies on any package, exported by python)

avoids direct python-abi dependency in libdolfinx, since python exports both python and python-abi
transitive dependency still exists in adios until it splits (#15)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Going to merge if ppc fails again, since the whole ppc infrastructure has been frustratingly flaky and wasting a lot of time.


Change according to registry rework

Many documents still use RegistryObject and ForgeRegistries. These should be updated.

Duplicate of #49


Add trPr to CT_Row class

Add table row properties element (trPr) to CT_Row class.

That one still needs some attention, though! :)

Added in release v0.8.5, pushed on Feb 21, 2015.

Thanks, Steve!


cosmetic improvements

Nothing much to see here--just cosmetic/style fixes, plus a much simpler travis-ci config. The latter also fixes the test failures, which were due to out-of-date dependencies.

What a dramatic branch name for "not much to see here" ;)

They're the first non-trivial commits I've made in > 2 years!

Woah, scary. :)

ooh lala! Are these warnings ok?

Are we getting a new web interface / something else too? :popcorn:
:ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream:
:ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream:
:ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream:
:ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream:
:ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream: :ice_cream:
ice cream matrix!

Probably? Not sure.
There are no plans to further develop this package--it will be deprecated in favor of NiMARE at some point (though NiMARE is not ready for use yet). As for a web interface, that depends on what the NIH council says when they meet in two weeks. If all goes well, there will be a new and dramatically improved (and image-based) web-based engine for annotation and analysis.


Clarify license

The code is currently unlicensed, which makes it unclear what is allowed and what is not concerning distribution and source code.

Good point, I will do that at the soonest opportune moment.

Good point, I will do that at the soonest opportune moment.

@Googe14 Hey, I just wanted to suggest using either the Unlicense or the MIT License so that others can freely make their own improvements.

Added MIT license


Mannu292001

PR Description 📜
This PR is about Space Invaders Game
Fixes  #1108

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Mannu292001 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Bro what is the change here he requesting you. Even i am facing the similar issue. All my checks are passed and  i updated main readme file also but still it is sjhowing me 1 change requested

Bro what is the change here he requesting you. Even i am facing the similar issue. All my checks are passed and i updated main readme file also but still it is sjhowing me 1 change requested

bro i will also not ale to understand how to change readme.md file and mentor are also not replying here

Hello @Mannu292001 @U-DAY-333, Sorry for some inconvenience caused.
You have to add entry of your game in main README.md file that is : README file

Hey @Mannu292001 you are not reading documentation properly read it  properly and then make a PR

Thank you  @Mannu292001 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Can not transfer EDG in a local setup network

Can not transfer EDG in a local setup network
Description
I followed the tutorial to create a local 4 validators network. All four nodes were connected. I then connected metamask to node1 and did a transfer. But the transfer tx failed with error.
(It showed my balance currectlly though.)
Your environment

macos m1
docker image: 0xpolygon/polygon-edge:0.4.1

Steps to reproduce
Expected behaviour
Transfer can succeed
Actual behaviour
Transfer failed
Logs
edge-test-node-1-1  | 2022-07-27T03:44:39.029Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_call err="invalid signature"
edge-test-node-1-1  | 2022-07-27T03:54:19.301Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="contract creation without data provided"
edge-test-node-1-1  | 2022-07-27T03:54:26.015Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="unable to apply transaction even for the highest gas limit 5242880: invalid signature"
edge-test-node-1-1  | 2022-07-27T03:54:26.779Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="unable to apply transaction even for the highest gas limit 5242880: invalid signature"
edge-test-node-1-1  | 2022-07-27T03:54:27.370Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="unable to apply transaction even for the highest gas limit 5242880: invalid signature"

Proposed solution

Can not transfer EDG in a local setup network
Description
I followed the tutorial to create a local 4 validators network. All four nodes were connected. I then connected metamask to node1 and did a transfer. But the transfer tx failed with error.
(It showed my balance currectlly though.)
Your environment

macos m1
docker image: 0xpolygon/polygon-edge:0.4.1

Steps to reproduce
Expected behaviour
Transfer can succeed
Actual behaviour
Transfer failed
Logs
edge-test-node-1-1  | 2022-07-27T03:44:39.029Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_call err="invalid signature"
edge-test-node-1-1  | 2022-07-27T03:54:19.301Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="contract creation without data provided"
edge-test-node-1-1  | 2022-07-27T03:54:26.015Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="unable to apply transaction even for the highest gas limit 5242880: invalid signature"
edge-test-node-1-1  | 2022-07-27T03:54:26.779Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="unable to apply transaction even for the highest gas limit 5242880: invalid signature"
edge-test-node-1-1  | 2022-07-27T03:54:27.370Z [ERROR] polygon.dispatcher: failed to dispatch: method=eth_estimateGas err="unable to apply transaction even for the highest gas limit 5242880: invalid signature"

Proposed solution

Contact the support center to chat with a live agent on a live chat to report this for further assistance via LIVE CHAT

Hello @ianhe8x @Sanitwil please make sure that the transactions you are sending are signed on the clientside.
Thank you

Hi @lazartravica I'm confused about your comment, the tx was sent from metamask, how comes it not signed?

Hello, I have the same error: https://forum.polygon.technology/t/docker-deployment-invalid-signature/10082/55


Remove rename notices from component docs

This removes it from the React and the SCSS component docs.
The React ones can be removed because the migration is complete.
I never ended up actually starting the SCSS name changes. I propose that we leave the old classes as-is and revisit the rename migration for that in the future. It's high effort and low impact IMO.

@tomgenoni — Missed TextInput. Just updated.


CONFIGURATION-773: User's Guide > Properties files > Saving - small documentation bugs

Hi,
I found a couple of small bugs with missing brackets and quites in the code sample provided in the User's Guide > Properties files > Saving for the current version (2.6) as well as v1.10, please see the patch attached.
Thank you,
Dan

https://commons.apache.org/proper/commons-configuration/userguide/howto_properties.html#Saving
https://commons.apache.org/proper/commons-configuration/userguide_v1.10/howto_properties.html#Saving

@dandragut
Thank you. Merged.
Gary


index out of range error

I get the following error when I run a list of URL's:
goroutine 1 [running]:
main.getInjectedUrls(0xc4208f4280, 0xc42008e180, 0x51, 0xc420120e40, 0x2, 0x2, 0x0, 0x0, 0x0, 0x0, ...)
        /go/src/github.com/ameenmaali/qsfuzz/utils.go:191 +0xeb9
main.main()
        /go/src/github.com/ameenmaali/qsfuzz/main.go:151 +0x4e9

Hey @YouGina thanks for sending that over. Can you please pull a new version and retry? I fixed a bug yesterday with this and you may have a previous version. Thanks!

I am still seeing it with the updated binary. Let me know how I can help with debugging

@ShlomieLiberow, if you can provide problem URLs or the wordlist you are using that would be greatly appreciated (I know this may not be possible if the target is private, however). Unfortunately I am not able to reproduce as of now.

Yes, for me it seems fixed after updating.

Closing as it appears to be fixed. If you still experience the error, please let me know!

Was not able to repo as my data was a live steam but seems to be OK now. Cheers!


Refactor/quality of life

Added some very quick integration tests (no Circle CI set up, just with make files running locally), and then found a bug in the package.
Added tests for Snowflake, and found that to make the tests pass I needed to remove a safe_cast call -- this didn't break the Redshift tests so I'm pretty happy with it.
Without this change, there were errors like so when running on Redshift:
SQL compilation error: Function TRY_CAST cannot be used with arguments of types TIMESTAMP_NTZ(9) and TIMESTAMP_NTZ(9)

This is because we are trying to cast a field that's already a timestamp to a timestamp, and Snowflake doesn't like that -- check out this snippet that results in the same error:
  with sample_table as (
      select
          '2019-01-01'::timestamp as my_timestamp
  )
  select
      try_cast(my_timestamp as timestamp)
  from sample_table

Closes #14

I tested it with a project of mine and it works perfectly!


Fixes inconsistency in printed version format

Hello! Thank you very much for your contribution and interest in helping improve the Quartz community.
After a period of dormancy, the Quartz project is back under steady maintenance by multiple volunteers, who are working to once again handle contributions such as yours.
We notice that your contribution was made without use of the DCO feature (the sign-off feature on commits via the -s option). Can you please update your PR with commits that use the -s option, agreeing to assign copyright ownership and other terms as described at the contributor agreement referenced here: https://github.com/quartz-scheduler/contributing/blob/main/CONTRIBUTING.md
You can easily add signoff to your previous commits by running (on your PR branch):
git commit --amend --signoff --no-edit
git push -f

Or (for multiple commits):
# change 5 to the number of commits
git rebase HEAD~5 --signoff
git push -f

We notice that your contribution was made without use of the DCO feature (the sign-off feature on commits via the -s option). Can you please update your PR with commits that use the -s option ..

Done


I can't connect with Confluence use the Tools, it show:"Error attempting to check ConvertService (error downloading file to be converted)"

my confluence is 8.5.2
I use docker 19 to run OnlyOffice 8.0 ,it seems OK.
But When I use  onlyoffice-confluence  plugin 5.0 . I get a error:

Thank U very much

By The Way , my confluence and Onlyoffice  server  Installed on the same server.  and my  offline server have no Adress ,only IP


move all negatives to one side of the array

Hi,
I am a GSSOC'21 participant.
Given an array of positives and negatives, we have to move all negatives to left side of array.
There are two approach to solve this problem:

Partition approach of QuickSort
Two Pointer approach
Both the solution take linear time.
My code will be in C++
I will add proper test cases explaining every approach to the problem
Please assign me this issue @nakul-19 @plazzy99 @rudrakshi99

Hello
I am GSSOC21 PARTICIPANT
And I can code this issue in C++, C along with line comments and best time complexity.
Please assign me this issue.

I am a GSSOC '21 participant.
I wish to contribute a JAVA code for this issue:
My code will have comment lines to make it explanatory and understandable to everyone.
I will be very happy if I am assigned this issue and allowed to work on this.

I would like to contribute implementation in Java with proper comments using two pointer approach.
Please assign me this issue as a part of GSSOC 2021.

Hey, I am Kanchan Jeswani, a GSSOC'21 participant.
I am a Competitive Programmer and C++ enthusiastic.
I would love to solve this problem using CPP with well-explained code and along with its time and space complexity.
Please assign me this issue.
Hope to hear from you.
Discord username  kanchan19#3152
@anubhavitis @dheerajkotwani @tarun26091999 @todi-2000 @geekquad @plazzy99 @rudrakshi99

Hey, I'm gssoc'21 participant, I would like to give my hand in this project. Please assign this issue to me.

@vedant-jain03 do it in C++ only.


Watcher crash loop generates bursts of ethereum calls

Overview
When a watcher is misconfigured or a bug occurs, the watcher could enter a crash loop and restarts all the processes within. This in turn causes a burst of Ethereum calls to start all the processes. And because the crash is not self-recoverable, the burst could happen every few seconds (5 seconds?). This impacts integrators who relies on Ethereum node as a service e.g. Infura, and causes them to exhaust the quota rapidly.
Actual behaviour
When the watcher is misconfigured or a bug occurs, it brings the watcher into an irrecoverable crash loop and produces a constant burst of Ethereum calls every few seconds.
Expected behaviour
When the watcher is misconfigured or a bug occurs, the watcher should have an incremental backoff period for retries.

This is quite a concerning issues. Did this ever occur? Is there a way to replicate this? Have you seen it in the wild and can provide logs?


Event logging switch mechanism

Refactor the code so that there's a boolean that controls if we log any events to Firebase. By default in GitHub we have the comment so that when we deploy to test it is not logging.
But when we deploy to production we remove the comment so logging is turned on.

We need a mechanism to turn event logging on and off dynamically in production and in test.


Bump spartan and navstar

High Level Description

Overlay could be enable/disable via "dcos_overlay_enable" in config.yaml. Currently, this flag is read by mesos overlay module but not by navstar overlay module. It will be good if navstar overlay module could also read this flag and enable/disable accordingly.

Several performance improvements for spartan dns server.

Spartan HTTP API (in there release only for debugging purposes)

Add VIPs Debug HTTP API

Currently it is possible for spartan to start it's own epmd if dcos-epmd is not yet started or has died for some reason. The systemd services for each of the Erlang based components should be reviewed and modified so that each ensures dcos-epmd is already running before starting.

Related Issues

DCOS_OSS-1134 Read dcos_overlay_enable to start/stop navstar overlay module
DCOS_OSS-697 Spartan CPU demands are Too High at High Scale
DCOS-15247 Spartan repeatedly crashing on some nodes
DCOS-15840 common test to assert unique rules in ip rules table
DCOS_OSS-1128 Ensure DC/OS Erlang based apps use dcos-epmd.service
DCOS-15240 Spartan HTTP API
DCOS-15357 API to display state of the lashup

Checklist for all PR's

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: [link to CI job test results for component]
[x] Code Coverage (if available): [link to code coverage report]

https://github.com/dcos/erl-dns/pull/1
https://github.com/dcos/erl-dns/pull/2
https://github.com/dcos/spartan/pull/40
https://github.com/dcos/spartan/pull/41
https://github.com/dcos/spartan/pull/42
https://github.com/dcos/spartan/pull/43
https://github.com/dcos/spartan/pull/44
https://github.com/dcos/navstar/pull/77
https://github.com/dcos/navstar/pull/78
https://github.com/dcos/navstar/pull/79
https://github.com/dcos/navstar/pull/80
https://github.com/dcos/navstar/pull/81

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label [Ready For Review]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@GoelDeepak @drewkerrigan @nlsun please review

@orsenthil @mellenburg could you please review it?


Right click in Chrome prevents future mouse events.

Steps to reproduce:

Open http://phaser.io/examples/v2/input/drag in Chrome.
Right click anywhere on the canvas.
Attempt to drag sprite.

The sprite will follow the mouse forever, as no inputUp is generated.
This demonstrates the issue with enableDragDrop, but the issue appears with raw event handling as well. You will receive a single mouse down event after the right-click's onInputUp, but no further inputUp or inputDown events anywhere in the canvas.
I've tried to repro this in a few other frameworks and couldn't reproduce it, but if it turns out to be a general Chrome issue, I'd be glad to submit it there.

The following code will demonstrate this issue (can't figure out how to save/share Phaser Sandbox projects):
var lastGlobalUpdate = 'No global input yet.';
var lastSpriteUpdate = 'No sprite input yet.';

function preload() {
  game.load.baseURL = 'http://examples.phaser.io/assets/';
  game.load.crossOrigin = 'anonymous';
  game.load.image('logo', 'sprites/phaser-dude.png');
}

function create() {
  game.input.onDown.add(function(e, p) {
    lastGlobalUpdate = 'Global Input Down: ' + p.button;
  });
  game.input.onUp.add(function(e, p) {
    lastGlobalUpdate = 'Global Input Up: ' + p.button;
  });

  var sprite = game.add.sprite(300, 300, 'logo');
  sprite.inputEnabled = true;
  sprite.anchor.set(0.5);
  sprite.events.onInputDown.add(function(e, p) {
    lastSpriteUpdate = 'Sprite Input Down: ' + p.button;
  });
  sprite.events.onInputUp.add(function(e, p) {
    lastSpriteUpdate = 'Sprite Input Up: ' + p.button;
  });
}

function render() {
  game.debug.inputInfo(32, 32);
  game.debug.text(lastGlobalUpdate, 32, 120);
  game.debug.text(lastSpriteUpdate, 32, 150);
}

You can left-click the sprite and see up/down event, but once you right-click the canvas, a single mouseDown will register, then no further sprite events will be generated (though global events still work).

Looks like this doesn't affect Phaser 2.2: http://output.jsbin.com/vufiqanari

Not able to reproduce it, neither in the phaser example or your jsbin. Hmm, chrome version?

47.0.2526.106 (64-bit) on OS X.
It might be Mac-specific? It's not just me, I've gotten hundreds of bugs reports. If it helps, I just found out that the issue "repairs" itself once the browser loses focus. In other words, after right-clicking, the sprite will stick to your mouse and no mouse events are handled. If you then Cmd-Tab to another application and come back, mouse events work normally.

Oh, and yes, the JSBin is supposed to work, it uses some ancient version of Phaser. But if you replace it with 2.4.2 or higher, the JSBin stops working.

Weird stuff, especially if it's mac specific. I should note that whenever I do i right click I will get the chrome dropdown menu (save picture as... etc ) But it does register the click still. Tried all version of phaser from 2.0.0 to 2.4.4 and I have not noticed it even once.

Yeah, I just checked on a Ubuntu Chrome, and it worked fine, must be Mac
specific then. Couldn't find anything in Chrome's bug tracker... no real
pressure, I've downgraded to 2.4.1 and everything seems to be working.
On Thu, Jan 7, 2016 at 10:20 AM, nexiuhm<EMAIL_ADDRESS>wrote:

Weird stuff, especially if it's mac specific. I should note that whenever
I do i right click I will get the chrome dropdown menu (save picture as...
etc ) But it does register the click still. Tried all version of phaser
from 2.0.0 to 2.4.4 and I have not noticed it even once.
—
Reply to this email directly or view it on GitHub
https://github.com/photonstorm/phaser/issues/2286#issuecomment-169738774
.

I can confirm this bug with Mac osx 10.11.2 and Chrome Version 47.0.2526.111 (64-bit)
All versions of phaser after 2.4.1

Confirmed this is OS X only.

I used phaser 2.4.0 on my latest game (a solitaire) because of this

This was a super-nasty one to fix, but fix it I have. Chrome OS X doesn't fire any mouseup events if the context-menu is open. In fact the only event it fires is 'mouseout' on the window object. So I now hijack that and use it to do what's needed here. Seems to be working fine now for me on OS X and is pushed to dev branch.

The problem is back with the newest update!


Update changelog with 1.5.0 changed

Hi,
Could you update the changelog? Please,
Thanks!!

Done. https://github.com/chapter-three/next-drupal/blob/main/CHANGELOG.md#150
Thank you.


[Feature Request] Support for Prompt Lookup Decoding to speed up autoregressive decoding in LLMs

🚀 Feature
Hello, thank you for all the great work, I truly like this project! ✨
It would be great to incorporate Prompt Lookup Decoding to speed up autoregressive decoding in LLMs. The project is here: https://github.com/apoorvumang/prompt-lookup-decoding.
Motivation
For input-grounded tasks such as code infilling/editing, summarization, doc-qa, multi-turn chat etc., there is a high n-gram overlap between LLM input (prompt) and LLM output. This could be entity names, phrases, or code chunks that the LLM directly copies from the input while generating the output. Prompt lookup exploits this pattern to speed up autoregressive decoding in LLMs.
This would be a great addition to mlc-llm and the whole algo is quite simple as shown in https://github.com/apoorvumang/prompt-lookup-decoding.
Alternatives
Speculative decoding using a smaller draft model. Even this can be a great feature addition but it introduces overhead of using another smaller model.
Additional context
I use mlc-llm to locally host a fine-tuned starcoder model to assist me with code infilling tasks in vs code. However, currently only starcoder-1B suits the latency requirements on Mac M1 pro. It would be great to be able to use codellama 7B model with prompt lookup decoding for better performance and meeting the latency requirements.

Thank you @pacman100! Lookup decoding and speculative decoding are important methods and we are already aware of them. We plan to bring them to our serving project as a first step (under active development in branch serving). We will then followup with the algorithm support in the current chat app.

we have incorproated spec decode into MLC Engine


2nd yield call to http API in saga returns an undefined object

In my application, I need to make several API calls after the successful login to retrieve additional data for the application. The 1st call to login works and returns the necessary token, which is then put into the next Action for the subsequent API call. However, the next API call returns an undefined object even though I see from the API monitor that the call is successfully made and returned to the application. Have been struggling with this for a while now with no headway.
export default (api) => {
// attempts to login
    function * attemptLogin (login_params) {
        //make call to API
        const response = yield call(api.login, queryString.stringify(login_params)) // This call works
        if (response.ok) {
            yield put(Actions.loginSuccess(response.data)) //loginSuccess is logged
     }
    }

    function * getUserProfile(authObject) {
          const response = authObject
          const userParams =
          {
            "token": response.token,
          }
          const userProfile = yield call(api.listUsers, queryString.stringify(userParams)) //successfully get here
          console.log("after yield"+JSON.stringify(userProfile)) // this log entry returns undefined

    }

    // a daemonized version which waits for LOGIN_ATTEMPT signals
    function * attemptLoginWatcher () {
      while (true) {
        const action = yield take(Types.LOGIN_ATTEMPT)
        const {login_params} = action
        yield call(attemptLogin, login_params)

      }
    }

    function * loginSuccessWatcher () {
      while (true) {
        const authObject = yield take(Types.LOGIN_SUCCESS)
        yield call(getUserProfile, {authObject})
      }
    }
     return {
    attemptLogin,
    attemptLoginWatcher,
    getUserProfile,
    loginSuccessWatcher
  }
}

The code blocks look the same, yet why am I getting an undefined back in the 2nd function? In the API monitor, the call is successfully made and returned.
redux-saga version: 0.11.0

What does the API return / What does api.listUsers look like?

From the Monitor output, this is what is logged in the console.log
09-03 15:48:03.326 5572-9443/? I/ReactNativeJS: Monitor {"duration":163,"problem":null,"ok":true,"status":200,"headers":{"etag":"W/\"1ee-5EYE40mcgF46T9GFDuRS3w\"","access-control-allow-origin":"*","x-powered-by":"Express","vary":"Accept-Encoding","connection":"keep-alive","transfer-encoding":"chunked","content-type":"application/json; charset=utf-8","access-control-allow-headers":"X-Requested-With","date":"Sat, 03 Sep 2016 15:48:03 GMT","server":"nginx"},"config":{"transformRequest":[null],"transformResponse":[null],"headers":{"Accept":"application/json, text/plain, */*","Content-Type":"application/x-www-form-urlencoded","Cache-Control":"no-cache","Connection":"keep-alive"},"timeout":1000,"xsrfCookieName":"XSRF-TOKEN","xsrfHeaderName":"X-XSRF-TOKEN","maxContentLength":-1,"baseURL":"https://xxx.yyy.zz/api","method":"post","url":"https://xxx.yyy.zz/api/users/list","data":"set%5B%5D=1&token=a"},"data":[{"id":2,"uuid":"K7yoxMHDf4V4g7ACx3y4t7B_wMYr","short_name":"Support","full_name":"Support","status":"active","user_type":"normal","user_type_main_id":"mcr",,"employee_no":"","department_code":"28","department_name":"Support","subdepartment_code":"-1","subdepartment_name":"","designation_code":"91","designation_name":"Engineer","mobile":"99999999","email":"","permissions":[]}]}

The listUser function looks like:
  const listUsers = (params) => {
    console.log("params"+JSON.stringify(params))
    try {
    api.post(
    '/users/list',
    params,
      {
      headers: {
        "Content-Type": "application/x-www-form-urlencoded"
      }
    }
    )
    console.log("post submit")
  }
    catch (e) {
      console.log ("API error"+e+JSON.stringify(e))
    }
  }

Update: I think there is some problem handling the particular request or the response that I am passing in. I tried with another API just modifying request params and the api call and it works

You make an Api calk from this listUsers but you are not yielding nor
returning its response. Its just wrapped in try/catch block and nothomg else
On Sun, 4 Sep 2016 at 09:05, gavin-gmlab<EMAIL_ADDRESS>wrote:

Update: I think there is some problem handling the particular request or
the response that I am passing in. I tried with another API just modifying
request params and the api call and it works
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yelouafi/redux-saga/issues/526#issuecomment-244587752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJWMkneFw6Itla89W041ErTThpTN5s25ks5qmm3WgaJpZM4J0U-o
.

Ahh. Going through the code so much that the obvious doesn't stand out.
Thanks.


Adds NSException catcher

Any UICollectionView might throw NSException but I haven't found a way to catch exceptions produced by RxDataSources framework and posted the issue. That is why I decided to implement a mechanism that catches NSException and imbed this mechanism into RxDataSources.
The result of this NSExceptionCatcher is shown on a screenshot below.

P.S. I'm not sure that this change should be posted in Changelog. That is why I assume that this PR is #trivial

I would massively 👍 the idea but not necessarily the execution. Rather than fatalError I think the better thing to do would be to just reloadData()? Or potentially we can change the public API of RxDataSources to have an Observable that triggers whenever we encounter an exception while performing updates?
This is all very similar to what IGListKit does in this situation and I would love to see something similar here! I'd be happy to contribute to this as well.

Hi,
we can't catch anything and not crash because it is not clear what would be the consequences of that. Table view could be in a state that is not updatable, there could be memory leaks, etc... and this situation shouldn't never happen anyway.
IGListKit also rethrows the exception.
https://github.com/Instagram/IGListKit/blob/784f1beb9f8db5d6c156a92ce6a93b122d3f856b/Source/IGListAdapterUpdater.m#L230

we can't catch anything and not crash

@kzaher current PR is with catch and crash
Any suggestions on the PR? Is there something I can change to make the PR being merged?

Hi @iwheelbuy ,
I wouldn't want to pull in any code we don't need. It seems to me that you can catch these exceptions by providing your UI{Table,Collection}View subclass and override commit animated changes method.
I'm not sure what is the benefit of catching those exceptions, logging them separately and then crashing.

I agree with you!


Add Type#optinal?

This method returns a boolean indicating if type is Sum and left is nil

@GustavoCaso thanks man, we're almost there :)

After making the pull request I thought about that. Thanks @flash-gordon


Code structuring and formatting suggestions -- Review for ReScience

A lot of what I'm writing here is about personal preferences and such, so keep that in mind, but I'm trying to think of this from the sake that a wide range of users might be interested in reproducing the work, so having stuff be generally accessible is what I'm trying to target.
I really like that stuff is pulled out into different scripts to help organize the codebase
I think there are some ways to refine this just a bit more.
In particular, I would really isolate the code that a user is expected to run at the console vs. what the console is sourcing.
For example, you tell users in the README to just source the scripts, but that's actually not how those scripts are really set up.
There's a lot of stuff commented in the code that's important to know, and some of the stuff runs and some of it fails. Indeed, when I follow your README I get the following
> source('code/01_reformat_budworm_data.R')
> source('code/02_fit_candy_1991_models.R')
Parsed with column specification:
cols(
  ddeg = col_double(),
  total = col_double(),
  stage = col_double(),
  count = col_double()
)
initial  value 94150.793409
iter  10 value 187.831975
iter  20 value 92.623925
final  value 92.584687
converged
initial  value<PHONE_NUMBER>33
iter  10 value 234.717054
iter  20 value 97.619139
iter  30 value 96.373309
iter  30 value 96.373309
final  value 96.323276
converged
initial  value 1375.973037
iter  10 value 81.150500
iter  20 value 81.149096
final  value 81.149089
converged
There were 25 warnings (use warnings() to see them)

> source('code/03_fit_dennis_et_al_model.R')

Attaching package: ‘kableExtra’

The following object is masked from ‘package:dplyr’:

    group_rows

Parsed with column specification:
cols(
  ddeg = col_double(),
  total = col_double(),
  stage = col_double(),
  count = col_double()
)
initial  value 500.857239
iter  10 value 408.602852
iter  20 value 407.440856
iter  30 value 407.380269
final  value 407.380044
converged
There were 50 or more warnings (use warnings() to see the first 50)

Loading required package: stats4
Loading required package: splines
Parsed with column specification:
cols(
  ddeg = col_double(),
  total = col_double(),
  stage1 = col_double(),
  stage2 = col_double(),
  stage3 = col_double(),
  stage4 = col_double(),
  stage5 = col_double(),
  stage6 = col_double(),
  stage7 = col_double(),
  ddeg_cent = col_double()
)
Error in checkwz(wz, M, trace = trace, wzepsilon = control$wzepsilon) :
  NAs found in the working weights variable 'wz'
In addition: Warning message:
In Deviance.categorical.data.vgam(mu = mu, y = y, w = w, residuals = residuals,  :
  fitted values close to 0 or 1
VGLM    linear loop  1 :  deviance = 385.77698
Error in checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  NAs found in the working weights variable 'wz'
In addition: Warning messages:
1: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  5 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
2: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  7 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
3: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  10 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
4: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  12 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
5: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  12 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
6: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  12 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
7: In checkwz(wz, M = M, trace = trace, wzepsilon = control$wzepsilon) :
  12 diagonal elements of the working weights variable 'wz' have been replaced by 1.819e-12
8: In checkwz(wz, M, trace = trace, wzepsilon = control$wzepsilon) :
  46 diagonal elements of the working weights variable 'wz' have been replaced by 0.5
9: In tfun(mu = mu, y = y, w = w, res = FALSE, eta = eta, extra = extra) :
  fitted values close to 0 or 1

> source('code/05_fit_ordinal_models.R')

Attaching package: ‘ordinal’

The following objects are masked from ‘package:VGAM’:

    dgumbel, dlgamma, pgumbel, plgamma, qgumbel, rgumbel, wine

The following object is masked from ‘package:dplyr’:

    slice

Parsed with column specification:
cols(
  ddeg = col_double(),
  total = col_double(),
  stage = col_double(),
  ddeg_cent = col_double()
)

> source('code/06_collate_parameter_tables.R')

Which is to say that all by 04 ran without error, but 02, and 03 throw warnings and 02, 03, and 05 print messages, but not related to what a user would want
But 04 did not run because of the errors, which means that a user necessarily has to do more than what you say in the README and go find out why.
Something to improve the usability here would be to again quiet the extensive messaging that readr throws out. The less the user sees at the console of things they dont need to see, the better.
I'm all for having relevant info presented to the user, but I think a lot of what the user sees isn't meaningful.
I'm not a big fan of telling the user to just not worry about warnings that come to the console, so as I mentioned previously, I'd suggest either addressing those issues so the warnings go away or silence them specifically.
So yeah, the idea would be, rather than rely on the README for the user to follow for everything including code running, create a simple top-level script that

loads the environment
sources the internal function
walks through the analyses

This would remove the special Rprofile nature of the installation and align everything a user would want to read in the code for a first run through
This gets to another part of the concern I had, which is that it forces users to bounce between the source (which gives very little info and only via file names) in the README to the actual files, which have more details, but perhaps are a bit overwhelming
To address some of that, I'd suggest:

Any comment lines should have a space after the # character to help with reading

In general, take advantage of white space as much as you can to help new users read your code

Split function creation from function usage

Presently, the functions and the code using them are together in the .R files, which makes it challenging to navigate.
A simplifying format/structure would be to condense the functions into their own script (perhaps sectioned off by topic or if desired with a script for each topic) and then source those at the beginning of the top-level script (see 2. in previous list)

This allows the user to get the gist of what you're doing, and then dive into the function details if they want
The top level script that walks through the analyses (3. in previous list) can have a little table of contents and explanation in comments, which is why it's useful to have it separated from the README

This will circumvent the need for repeated calls to loading libraries in each sourced script
This also addresses an issue regarding the things that are printed to the screen within the scripts, but don't get echoed out when you source them, like
> logit_cm_candy_nll$par
[1]  5.53478517  9.48212482 12.35402857 15.85841654 21.46339190 27.51646597 -0.04613333

If that's something the user should be seeing, make sure they're actually seeing it!
What is the justification for saving things as .RDS files? If you don't care that much one way or another, I would suggest saving this kind of content in text type files, be they actual text, json/yml/whatever, it's just much more reliable and transferable (i shouldn't need R to open them).

Revisions checklist:

[ ]  clarify that only intermediate outputs are stored as rds to minimize typecasting and rounding errors
[ ]  improve readability of comment lines
[ ]  split function creation from function use
[ ]  minimize uninformative messages
[ ]  explicitly encourage stepping through scripts to follow the reproduction
[ ]  clarify library namespace conflicts


Remove race for creating rtc room

Fix https://github.com/livekit/livekit-server/issues/483

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


Improved user UI for rest demo

This is great! Showcases a few common Angular concepts this way too! Input etc


feat: Add support for self-managed CoreDNS addon and example of serverless cluster using Fargate Profiles

What does this PR do?

Adds support for self-managed CoreDNS addon
Adds example blueprint for Fargate Serverless cluster (data plane) using the self-managed CoreDNS addon; demonstrating how to modify the EKS provided CoreDNS deployment, etc.
Fixes pre-commit checks from prior PR

Motivation

Closes #549
Provides users with an example path on how to provision serverless EKS data planes using Fargate Profiles
Provides the start of how we might be able to manage the option to use either EKS managed addon or self-managed addon; this has come up before with requests like #523

More

[x] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)
[x] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)

⚠ No - I can add it to that repo but I think we need to test and validate that the GitOps route works first due to the tricky nature of how this is managed currently. Open to thoughts, feedback on how best to handle this

[x] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

my only problem with this example is that it assumes creds are available to the null-exec.
but nor does it not take a profile flag, nor it accepts cross-account assume roles.
I've already spent days trying to work that out without much luck :(

my only problem with this example is that it assumes creds are available to the null-exec. but nor does it not take a profile flag, nor it accepts cross-account assume roles. I've already spent days trying to work that out without much luck :(

It uses an ephemeral kubeconfig which is contructed at apply time using an auth token. This means that all authorization/authentication is left with the provider which is left up to the users discretion. The current provider implementation uses the provider recommended exec which does accept --profile


Fix: solve rotation error & upgrade style

Codecov Report
Base: 22.22% // Head: 22.22% // No change to project coverage :thumbsup:

Coverage data is based on head (6b60e0c) compared to base (ea9ad80).
 Consider uploading reports for the commit 4d9a370 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           dev/0.9      #90   +/-   ##
========================================
  Coverage    22.22%   22.22%
========================================
  Files           11       11
  Lines          477      477
  Branches        71       71
========================================
  Hits           106      106
  Misses         366      366
  Partials         5        5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Autoloop games

Describe the solution you'd like
A clear and concise description of what you want to happen.
i want the bot to start the games with auto loop
Version
What version of the lol-bot are you running?
Latest
Additional context
Add any other context or screenshots about the feature request here.
i want the bot to start the games with auto loop

This should be fixed with the latest release.


Change bind ip to work in docker container

Hey! :)
I am a total newby when it comes to docker and actix but still this little change could save some time for future users of this repository. It turns out that you cannot connect to the server running in docker container unless it's bound to <IP_ADDRESS> (instead of <IP_ADDRESS>).

Thanks!


Architecture documentation

Based on some of the discussion last time, I have created the following diagrams.
Multi-registry test
First a multi-registry test diagram, we talked that the fastes path to test the idea of multiple registry interaction using an orchestrator to mange the flow, was to do it all in the same process, by instantiating more processors and event-stores.
Orchestrator
Two proposals we need to decide on A or B, the difference are subtle, but could have larger impact.
Proposal A
A was what we discussed, the command orchestrator is blind to the content of a command, in merely consist of lines it should execute to specific registies, and does so in a serial faction. This enables a large degree of  freedom, since a command can consist of any number of command-lines. But has the negative impact that there is no enforcement of the structure of a command.
Proposal B
Here I proposal similar two what we also discussed but with some changes. We will create context aware commands, each containing the event(s) and proof(s) needed to execute the command. These commands would be serialized and put in a abstract command envelope before being sent from the client.
Internally I further see two possible solutions:
B1
The abstract command orchestrator would use a specific orchestrator implemented in the electricity library, to control the flow.
pros

more advance flow in a command is possible

cons

requires implementing custom orchestrators, creating possible more code.

B2
The context aware command can be deserialized and the electricity library contains functionality to output a list of command-lines from the command, the flow is returned to the abstract orchestrator.
pros

"only" requires commands to be able to translated to command-lines

cons

"rollback" can be more complex to implement, since it must be done in an abstract manner

On the choice of A, B, B1 and B2.
Just a thought - would it make sense to use an Abstract Factory pattern where the context is only domain aware in the specific implementation of the Factory ?


Warn on missing header in BrainVision files

Fixes #9999.

LGTM

All green on my first try!

All green on my first try!

Amazing! 🥇


Adds the Presentations page, navbar link, and metrics

This PR does the infrastructure work for a separate presentations page.
Work to be done:

replace presentation URLs with links to slide or poster pdfs (in VAST google drive or on GitHub?)

@clyne this should be good to go. It builds great for me and I've added my most recent talks.

This looks great, @jukent. Thanks! A couple of things:

Can you update the contributor's guide to include presentations (currently only has info for presentations)
It doesn't seem like the metrics on the home page are being calculated correctly. It has 0 for presentations.

Okay I'll do that tomorrow.
I just set the metrics to 0 to create the field, but will be updated when the GH Actions runs (which it isn't for your local build).

got it. thanks!

Okay contributor's guide is updated!


add method to check for typeof FileMaker === 'object'

This could be useful if you want to detect if the code is running in a webviewer or not and dynamically mock window.FileMaker if not.

something like this would work, but actually i think it's beyond the scope of this library.
const checkForFileMaker = async (timeoutInms) => {
return new Promise((resolve, reject) => {
let interval;
let timeout;
try {
interval = setInterval(() => {
if (typeof window.FileMaker === 'object') {
clearTimeout(timeout);
resolve(true);
}
}, 10);
timeout = setTimeout(() => {
clearInterval(interval);
resolve(false);
}, timeoutInms);
} catch (error) {
clearInterval(interval);
clearTimeout(timeout);
reject(error);
}
});
};


Polish #4674 & #4470

To resovle issues: #4674 & #4470

Codecov Report

Merging #5033 into master will decrease coverage by 0.01%.
The diff coverage is 93.33%.

@@             Coverage Diff              @@
##             master    #5033      +/-   ##
============================================
- Coverage     63.97%   63.96%   -0.02%
  Complexity      452      452
============================================
  Files           769      769
  Lines         33182    33181       -1
  Branches       5232     5230       -2
============================================
- Hits          21229    21224       -5
- Misses         9532     9534       +2
- Partials       2421     2423       +2

Impacted Files
Coverage Δ
Complexity Δ

...notation/ReferenceAnnotationBeanPostProcessor.java
87.61% <93.33%> (-0.12%)
0 <0> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
80.48% <0%> (-4.88%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.81%)
0% <0%> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
78.54% <0%> (-1.54%)
0% <0%> (ø)

.../dubbo/remoting/transport/netty4/NettyChannel.java
63.63% <0%> (-1.14%)
0% <0%> (ø)

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
89.71% <0%> (-0.94%)
16% <0%> (-1%)

.../rpc/protocol/dubbo/LazyConnectExchangeClient.java
58.82% <0%> (+2.35%)
0% <0%> (ø)
:arrow_down:

...e/dubbo/remoting/transport/netty/NettyChannel.java
60.22% <0%> (+4.54%)
21% <0%> (+1%)
:arrow_up:

.../apache/dubbo/qos/protocol/QosProtocolWrapper.java
70.73% <0%> (+4.87%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Do not throw all calculated Printf argument highlights if some of them are erroneous

It does not fix this https://github.com/fsprojects/VisualFSharpPowerTools/issues/1274 and other possible AST forms on which the feature does not work, but it prevents loosing all Printf highlight tags to disappear if a single such error occurs.

:+1:


excluded superclasses that have MONDO-XXXXXXX-obsoleted source

address #6004
@sabrinatoro

I will slowly work on cleaning up these superclasses that were rewired
If I disagree with the subclass, I made it an excluded subclass with my ORCID
if I agree with the superclass assertion, I added my ORCID
I cleaned up a few definitions too

We can close this, we'll address this with the action items in the ticket (https://github.com/monarch-initiative/mondo/issues/6004#issuecomment-2038336961).


[Feature] Tags View

Migrate Tags View feature (allow right-click action) from vue-element-admin

这个我来做

Done! @yin-shu


Traceback & Multiple errors

i always get these errors what should i do?

This is a duplicate of #40. There are issues running under Windows at the moment.


Identity Server security support

Along with the OpenID and OAuth2 that we currently offer, Identity Server should be the next support path we take for single sign-on, identity management, authorization, and API security.

Since Identity Server 4 operates as an OpenID Connect and OAuth 2.0 framework, is the intention that the SSO configuration of the Steeltoe app remains similar but Identity Server would replace UAA Server or Pivotal Single Sign-on service?

I think the idea here is making sure there's an opinionated way of establishing the connection with Identity Server, ideally creating an interchangeable setup where (similar to other areas of Steeltoe) you can switch your SSO server with just configuration or minimal code change. There may also be some need for a Kubernetes recipe here as well

Ok, that makes sense. I can start on identity server for SSO later this evening.

The solution I'm starting to go down is as follows:
Mutual TLS auth
CloudFoundry config
public void ConfigureServices(IServiceCollection services)
{
    services.AddCloudFoundryCertificateAuth(Configuration);
    services.AddControllers();
}

becomes
public void ConfigureServices(IServiceCollection services)
{
    services.AddIdentityServerCertificateAuth(Configuration);
    services.AddControllers();
}

The configuration that will be required is a config setting for the certificate and privateKey used for the authentication. This approach will basically adhere to this process: http://docs.identityserver.io/en/latest/topics/mtls.html.
I'm going to add additional comments talking about the other approaches for SSO/Jwt.

The solution I'm starting to go down is as follows:
SSO
CloudFoundry config
services.AddAuthentication((options) =>
            {
                options.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
                options.DefaultChallengeScheme = CloudFoundryDefaults.AuthenticationScheme;
            })
            .AddCookie((options) =>
            {
                options.AccessDeniedPath = new PathString("/Home/AccessDenied");
            })
            .AddCloudFoundryOpenIdConnect(Configuration);

becomes
services.AddAuthentication((options) =>
            {
                options.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
                options.DefaultChallengeScheme = IdentityServerDefaults.AuthenticationScheme;
            })
            .AddCookie((options) =>
            {
                options.AccessDeniedPath = new PathString("/Home/AccessDenied");
            })
            .AddIdentityServerOpenIdConnect(Configuration);

The additional configuration options that will need to be added will be the details typically provided in the p-identity auth mySSOService after it's been configured (clientId, clientSecret, identityServerUri, scopes, etc...).

@tscrypter have you made much progress or found anything interesting here?
I don't want to derail any progress you may have made, but I was talking with @macsux about this issue and since Identity Server's purpose really boils down to "easily customized Identity Provider that's standards compliant", a valuable solution here might need to be based on a non-trivial ID Server template since MSFT's libraries are theoretically sufficient for the standard scenarios.
Specifically it might be interesting to do all this with a deeper Kubernetes integration, such that Kubernetes is the data store and some app credential setup actions could happen automatically when apps are deployed to the cluster... We can talk more about this if you aren't already far down a different path
Thoughts?

@TimHess I'm not far down an implementation, am more than happy to pivot. Most of my progress has been on trying to identify options for the end user to swap identity providers with minimal code changes. The scenarios I mention in the comments above are the simpler cases (mostly looking at the security samples and making sure to have parody), and thinking through them did allow me to realize that their implementation's don't really save the end user much code compared to using MSFT's libraries.
I think having an opinionated approach to easily wire up ID Server via some Kubernetes configuration would be more interesting and likely add more value.

@tscrypter I chatted with @macsux about this today and got a little more in depth with the above... At this point I think this conversation is better suited for our Kubernetes incubator project and have started logging some issues over there.
If we all agree this idea is best suited for that initiative then I think we can just close this issue

Closing due to lack of interest. If you would like to see this supported, please respond to this issue


Possibility to scrape Articles without installing image libs

Heyho,
thanks for the great url article scraping tool.
But it would be nice to add the image downloading functionality as optional, so we can use your piece of sweet (code) cake without installing all the image libs (PIL, Pillow, libpng, ...).
Background:
it would be nice if i could use your lib on amazons aws lambda without building a custom package with all the needed packages, because you are not able to install system packages like libpng there.
Thanks!

You can disable downloading of images via fetch_images. E.g.:
article = Article(url='http://cnn.com', fetch_images=False)

More here. I'm able to use newspaper that way in a docker container without installing any image libs so hopefully that works for you too.

i already set fetch_images to false.
Unable to import module 'newspaper3k': cannot import name '_imaging'

okay i got it, now newspaper is found and but newspaper seems to create a data directory .newspaper_scraper which is not on lambda:
START RequestId: 4fc82e64-b3de-11e7-baae-7b689a397efc Version: $LATEST
module initialization error: [Errno 2] No such file or directory: '/home/sbx_user1062/.newspaper_scraper'

@codelucas maybe it is possible to add an ENV-Variable to set base .news-scraper path?


A collection of viability assessments

Finding published viability assessments is extremely difficult. Would providing an index of viability assessments, along with important information, be of value to local authorities and interested members of the community?

Maybe worth asking the 'commissioned out' market leaders - they could agree to provide a list of every assessment they've done that their clients agreed to being published, and keep it up to date. Guess BNP are one, also the software package suppliers they use do quite a few themselves it seems.


Configure for Docker

This configures the application to be used with Docker, so that dependencies can be isolated, and dev setup is simplified.
Also updates Ruby from 2.5.3 to 2.5.5 - there are a couple of security fixes there.
Notes:

It will probably need some changes before it is used in Heroku.
It would be great if someone with an existing setup could check this to make sure the changes to db.yml don't break existing setups

To do:

[ ] App startup waits for Postgres
[ ] Where to run rake fetch:fetchall

Not sure why the build is failing - could it be because of the upgrade to Ruby 2.5.5 @Br3nda ?

@Br3nda I can't debug the Travis failure - can you please look into why it's failing? I think it's probably missing the Ruby 2.5.5 image

Travis might need the .ruby-version file to match

I rebased this branch


[REQUEST] Add a basic Color object.

Describe the feature.
People may want to be able to easily call upon colors. Since it should be at least an accepted concept, I am wanting to add a basic Color object that includes the official branding colors. Please note that other colors that are custom-defined are ideally frowned on for this idea since the core library is about making implementations strict to what Discord offer, which we can argue debatably with the branding colors.
Code of Conduct

[X] I agree to follow the contribution requirements.

Closing issue as the associated PR has been merged (and not autoclosed).


InfoCard Expanded across wider screen

I dealt with this issue, but certainly am not offended or hurt if it's not merged, but you can take a look at it if you want. Oh, and I misspelled edit so the branch name is weird. Sorry!

@BryceEvans are you finished with this PR? It's still in draft mode. As it is, some elements overlap in the widths in between the breakpoints. The code currently on development does not. If you have changes that fix that, push them and I will merge this one.

@jessharrison83 I'm not sure what you mean above. It looked great on my computer but I only tested one browser. This PR really only expands the InfoCard on Desktop mode to take up more of the screen. I know you guys said not to mess with it (I was so engrossed I didn't see those messages until after I had spent 2 hours on it). Since I didn't catch what your referring too I'm okay if it's deleted but I wanted to save the work somewhere in case I ever wanted/needed to return to it.
If what we have in development does not overlap with some elements and this one does I'll go ahead and delete this branch. We all seem happy with what's in development anyway, right?


Inline Graphic Backend Not Working

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
The Inline Graphic Backend isn't ploting my graph in the Plots tab. I have tried with Automatic, and it works.
What steps reproduce the problem?

I got a x input numpy array with size (42372, 1034) and y output numpy array with size (42372,):
The following code reproduces the problem:

import pandas as pd
import numpy as np
import matplotlib.pyplot as plt
from sklearn.preprocessing import StandardScaler
from sklearn.model_selection import TimeSeriesSplit
cmap_data = plt.cm.Paired
cmap_cv = plt.cm.coolwarm

def plot_cv_indices(cv, x, y, ax, n_splits, lw=10):
    """Create a sample plot for indices of a cross-validation object."""

    # Generate the training/testing visualizations for each CV split
    for ii, (tr, tt) in enumerate(cv.split(X=x, y=y)):
        # Fill in indices with the training/test groups
        indices = np.array([np.nan] * len(x))
        indices[tt] = 1
        indices[tr] = 0

        # Visualize the results
        ax.scatter(
            range(len(indices)),
            [ii + 0.5] * len(indices),
            c=indices,
            marker="_",
            lw=lw,
            cmap=cmap_cv,
            vmin=-0.2,
            vmax=1.2,
        )

    # Plot the data classes and groups at the end
    ax.scatter(
        range(len(x)), [ii + 1.5] * len(x), c=y, marker="_", lw=lw, cmap=cmap_data
    )

    # Formatting
    yticklabels = list(range(n_splits)) + ["class"]
    ax.set(
        yticks=np.arange(n_splits + 1) + 0.5,
        yticklabels=yticklabels,
        xlabel="Sample index",
        ylabel="CV iteration",
        ylim=[n_splits + 1.2, -0.2],
        xlim=[0, 42372],
    )
    ax.set_title("{}".format(type(cv).__name__), fontsize=15)
    return ax

n_splits = 4
tscv = TimeSeriesSplit(n_splits = n_splits, max_train_size = 9593, test_size = 1000)
fig, ax = plt.subplots()
plot_cv_indices(tscv, x, y, ax, n_splits)

What is the expected output? What do you see instead?
Expected output (from Automatic Backend):

What i see (With Inline Backend in the Plots tab):

Versions

Spyder version: 5.2.0
Python version: 3.8.0
Qt version: 5.2.1
PyQt version: 1.11.3
Operating System name/version: Windows 10

Dependencies

# Mandatory:
atomicwrites >=1.2.0          :  1.4.0 (OK)
chardet >=2.0.0               :  4.0.0 (OK)
cloudpickle >=0.5.0           :  2.0.0 (OK)
cookiecutter >=1.6.0          :  1.7.3 (OK)
diff_match_patch >=20181111   :  20200713 (OK)
intervaltree >=3.0.2          :  3.1.0 (OK)
IPython >=7.6.0               :  7.30.1 (OK)
jedi >=0.17.2;<0.19.0         :  0.18.1 (OK)
jellyfish >=0.7               :  0.8.9 (OK)
jsonschema >=3.2.0            :  4.2.1 (OK)
keyring >=17.0.0              :  23.4.0 (OK)
nbconvert >=4.0               :  6.3.0 (OK)
numpydoc >=0.6.0              :  1.1.0 (OK)
paramiko >=2.4.0              :  2.8.1 (OK)
parso >=0.7.0;<0.9.0          :  0.8.3 (OK)
pexpect >=4.4.0               :  4.8.0 (OK)
pickleshare >=0.4             :  0.7.5 (OK)
psutil >=5.3                  :  5.8.0 (OK)
pygments >=2.0                :  2.10.0 (OK)
pylint >=2.5.0                :  2.12.2 (OK)
pyls_spyder >=0.4.0           :  0.4.0 (OK)
pylsp >=1.3.1;<1.4.0          :  1.3.2 (OK)
pylsp_black >=1.0.0           :  None (OK)
qdarkstyle =3.0.2             :  3.0.2 (OK)
qstylizer >=0.1.10            :  0.2.1 (OK)
qtawesome >=1.0.2             :  1.1.1 (OK)
qtconsole >=5.2.1;<5.3.0      :  5.2.1 (OK)
qtpy >=1.5.0                  :  1.11.3 (OK)
rtree >=0.9.7                 :  0.9.7 (OK)
setuptools >=49.6.0           :  58.0.4 (OK)
sphinx >=0.6.6                :  4.3.1 (OK)
spyder_kernels >=2.2.0;<2.3.0 :  2.2.0 (OK)
textdistance >=4.2.0          :  4.2.2 (OK)
three_merge >=0.1.1           :  0.1.1 (OK)
watchdog >=0.10.3             :  2.1.6 (OK)
zmq >=17                      :  22.3.0 (OK)

# Optional:
cython >=0.21                 :  None (OK)
matplotlib >=2.0.0            :  3.5.0 (OK)
numpy >=1.7                   :  1.21.4 (OK)
pandas >=1.1.1                :  1.3.4 (OK)
scipy >=0.17.0                :  1.7.3 (OK)
sympy >=0.7.3                 :  None (OK)

Hi @muriloasouza,
Thanks for giving me a complete case of your issue, when I'm testing this on my end I find two things:

With the options in Preferences > IPython Console > Graphics > Backend in inline, your code outputs me both the plot in the console and in the plots pane,

Please note that I have the option in the hamburguer menu of the plots pane entitled Mute inline plotting unchecked.

When I change the option to automatic, your code outputs me a new window that has the plot, and it doesn't show anything in the plots pane,

Please let me know if this helps with your issue, or if you are using another combination of the settings for me to try it out as well.

Hey @steff456 ,
Unchecking mute inline plotting solved. Thank you for your help!


Question about IIIF extensions

From the docs:
Request information beyond the scope of the existing parameters could be passed to an image server as query parameters.  (https://iiif.io/api/image/3.0/#49-extensions)
I don't think it is currently possible to add query parameters. How can I extend the IIIFTileSource?

My use case is rendering a pdf in separate layers

Current workaround
`
/** @ts-ignore */
extend(IIIFTileSource.prototype, {
_originalGetTileUrl: IIIFTileSource.prototype.getTileUrl,
getTileUrl(level: number, x: number, y: number): string {
return this._originalGetTileUrl(level, x, y) + "?layer=1";
},
});
`

Current workaround

This seems like a fine approach to me. Are you still having problems with it?


Lines with cacheAsBitmap=true rendered incorrectly

May be related to #4941 (the slime offset in image) although happens also for Canvas renderer.

Expected Behavior
Cached graphics, especially lines should render the same as non-cached versions.
Current Behavior
Cached lines don't display correctly. For example, lines deviating from diagonal get strange offset. The saved geometry (coordinates) is correct though. The more straight line is (both vertical and horizontal) the more is this apparent. There also seems to be issue with antialiasing (kinda_straight line in the example) or the lines are losing width.
Possible Solution
Definitely was not present in Pixi.js 3 but that's many version ago. Because issue is most pronounced with straight lines and lines are losing width, I'd suspect an error with bounding box.
Steps to Reproduce
Example: https://jsfiddle.net/vqrcsbf0/2/
Red square marks correct position where lines should end. Blue line is not using caching. Green lines do use caching. Perfectly diagonal line seems to be ok. Perfectly straight line shows largest error.
Environment

pixi.js version: 4.8.2
Browser: Chrome 70, Firefox 62
Device: Desktop

Still broken

Closing as fix merged in


Release Candidate

Why is this PR needed?
Get all we have in master and release it as version 5.0.0-RC
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.

LGTM :+1:


Option for a comment stream to specify limit size

I want to stream comments, yet the default 25 comments per call of
https://www.reddit.com/r/redditdev/comments.json
is not enough. Reddit does have an option to extend the size fo the return by doing
https://www.reddit.com/r/redditdev/comments.json?limit=100,
but I am not aware that graw implements this yet.

I see that limit=100 is already set, yet it doesn't act that way. I'm going to look into this further :+1: