Add Developer Advocacy

https://github.com/dmitryvinn/awesome-dev-advocacy#README
This lists include awesome resources necessary for anyone starting, planning to get involved with or wants to improve skills in the field of Developer Advocacy.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions.
You have read and understood the instructions for creating a list.
This pull request has a title in the format Add Name of List.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

Your entry here should include a short description about the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

Your entry should be added at the bottom of the appropriate category.
The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it.
Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
The default branch should be named main, not master.
Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

Non-generated Markdown file in a GitHub repo.
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate. Please search for existing submissions.
Only has awesome items. Awesome lists are curations of the best, not everything.
Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't use hard-wrapping.
Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.
Doesn't include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Go to the top and read it again.

Thanks for making an Awesome list! 🙌
It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome.

It looks like you didn't use awesome-lint. Here are the errors I get:
awesome-lint https://github.com/dmitryvinn/awesome-dev-advocacy
✖ Linting

  README.md:1:1
  ✖    1:1   Missing newline character at end of file                     remark-lint:final-newline
  ✖    1:1   Invalid URL: http:                                           remark-lint:double-link
  ✖    1:1   Git repository must be at least 30 days old                  remark-lint:awesome-git-repo-age
  ✖    1:1   The repository should have "awesome" as a GitHub topic       remark-lint:awesome-github
  ✖    1:1   The repository should have "awesome-list" as a GitHub topic  remark-lint:awesome-github
  ✖    1:1   License was not detected by GitHub                           remark-lint:awesome-github
  ✖    1:1   Missing or invalid Table of Contents                         remark-lint:awesome-toc
  ✖    1:31  Invalid badge source                                         remark-lint:awesome-badge
  ✖   14:1   Don’t add a trailing ? to headings                           remark-lint:no-heading-punctuation
  ✖   16:91  Text "NodeJS" should be written as "Node.js"                 remark-lint:awesome-spell-check
  ✖   20:1   Don’t use multiple top level headings (1:1)                  remark-lint:no-multiple-toplevel-headings
  ✖   20:1   List can only have one heading                               remark-lint:awesome-heading
  ✖   22:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   23:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   24:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   25:7   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   26:7   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   27:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   28:7   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   29:7   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   30:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   31:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   32:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   33:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖   34:3   Invalid list item link URL                                   remark-lint:awesome-list-item
  ✖  117:4   Text "StackOverflow" should be written as "Stack Overflow"   remark-lint:awesome-spell-check

  26 errors

Also, you have not unicorn and reviewed two PRs. I suggest you read the guidelines carefully.

Links don't follow the format of the awesome list of [Link] - Description.
Also, missing descriptions on the majority of the links.
Note this section from the contribution guidelines "Doesn't include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough."
Don't forget to review and comment on at least two other PRs.
You may need to reread the PR rules, you are missing a critical action that needs to be taken on your own PR.

Just checking in on this PR. The number of errors when run through the linter has increase to 164.
README.md:22:4
✖    1:1   Invalid URL: http:                                           remark-lint:double-link
✖    1:1   The repository should have "awesome" as a GitHub topic       remark-lint:awesome-github
✖    1:1   The repository should have "awesome-list" as a GitHub topic  remark-lint:awesome-github
✖    1:1   License was not detected by GitHub                           remark-lint:awesome-github
✖    1:1   Missing or invalid Table of Contents                         remark-lint:awesome-toc
✖    1:31  Invalid badge source                                         remark-lint:awesome-badge
✖   14:1   Don’t add a trailing ? to headings                           remark-lint:no-heading-punctuation
✖   16:91  Text "NodeJS" should be written as "Node.js"                 remark-lint:awesome-spell-check
✖   20:1   Don’t use multiple top level headings (1:1)                  remark-lint:no-multiple-toplevel-headings
✖   20:1   List can only have one heading                               remark-lint:awesome-heading
✖   22:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   22:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   23:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   23:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   24:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   24:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   25:8   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   25:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   26:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   26:8   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   27:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   27:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   28:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   28:8   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   29:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   29:8   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   30:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   30:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   31:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   31:8   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   32:8   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   32:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   33:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   33:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   34:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   34:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   35:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   35:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   36:4   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   36:4   Invalid list item link URL                                   remark-lint:awesome-list-item
✖   41:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   42:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   43:6   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   46:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   47:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   48:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   51:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   52:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   53:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   56:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   57:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   58:5   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   90:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   93:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   96:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖   99:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  102:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  105:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  108:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  110:13  "’" is used without matching "‘"                             remark-lint:match-punctuation
✖  111:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  114:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  117:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  120:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  123:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  126:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  129:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  132:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  135:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  138:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  141:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  144:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  147:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  150:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  152:9   "’" is used without matching "‘"                             remark-lint:match-punctuation
✖  153:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  156:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  159:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  162:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  165:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  168:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  171:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  174:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  177:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  180:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  183:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  186:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  189:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  192:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  195:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  198:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  201:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  204:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  207:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  210:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  213:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  216:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  219:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  222:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  225:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  228:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  231:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  233:26  Should not repeat "."                                        remark-lint:no-repeat-punctuation
✖  234:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  236:3   Invalid list item link URL                                   remark-lint:awesome-list-item
✖  237:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  240:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  243:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  246:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  249:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  252:8   Incorrect list-item indent: remove 1 space                   remark-lint:list-item-indent
✖  254:36  Should not repeat "."                                        remark-lint:no-repeat-punctuation
✖  254:37  Should not repeat "."


Cannot find namespace 'mitt'.Vetur(2503)

just copy code from readme to vscode
import mitt from 'mitt' const emitter: mitt.Emitter = mitt()
then I got Cannot find namespace 'mitt'.Vetur(2503)
how should i do

Same issue with TS: Cannot find namespace 'mitt'.ts(2503)
const emitter: mitt.Emitter = mitt() @

this works for me.
import mitt from "mitt"; const emitter = mitt(); export default emitter;

@jogunism yeah using the library works fine. It's the type definition we have an issue with 👍🏼

@RamirezET @Rosefalk
Same issue here,
but I tried this:
import mitt, { Emitter } from 'mitt';

const emitter: Emitter = mitt();

export default emitter;

it works for me.

@RamirezET @Rosefalk
Same issue here,
but I tried this:
import mitt, { Emitter } from 'mitt';

const emitter: Emitter = mitt();

export default emitter;

it works for me.

I tried that before, but must've made a mistake - it seems to work and compiler has no issue with it.
Thank you.
Another thing I found working was using mitt itself as the type ( emitter = typeof mitt).

The docs have been updated to recommend the above approach.


GH-39064: [C++][Parquet] Support row group filtering for nested paths for struct fields

Rationale for this change
Currently when filtering with a nested field reference, we were taking the corresponding parquet SchemaField for just the first index of the nested path, i.e. the parent node in the Parquet schema. But logically, filtering on statistics only works for a primitive leaf node.
This PR changes that logic to iterate over all indices of the FieldPath, if nested, to ensure we use the actual corresponding child leaf node of the ParquetSchema to get the statistics from.
Are there any user-facing changes?
No, only improving performance by doing the filtering at the row group stage, instead of afterwards on the read data

Closes: #39064

I'll take a quick look now

@github-actions crossbow submit wheel-macos-*


[test][IRGen] Fix check on local_extern test.

Windows IR seems to include a dso_local annotation, which the check
didn't expect and caused a failure in #33813. Added a regex check to
ensure the test passes with or without the annotation.

@swift-ci please smoke test and merge

CC: @xedin

@swift-ci please test Windows platform

@3405691582 seems that its not enough?

@3405691582 I think we need to revert; this is breaking Foundation builds on Windows too as it no longer correctly is processing typedefs.

Nope, false alarm there.

I suspect this is another rigid match problem, but the CI is not showing me the full IR from that unit test so I can't confirm. Are you able to grab the emitted IR and show it to me, or let me know if there's a way to see it simply?

Not currently, there is something else that is preventing me from building, but the CI hosts seem okay (I think that the SDK version might be exposing another issue locally).

As an experiment, I'm going to try removing the expectation on the function case, just to see if the other cases are as expected.

@swift-ci please test Windows platform

@swift-ci please test Windows platform

ugh, I cannot into git. You might have to do that again.

@swift-ci please test Windows platform

That appears to have worked. Therefore, I think a valid approach for now is to create an amended version of the test for Windows and look into examining IR or figuring out a different test mechanism at a later date.
Do you agree, or should I try something else?

Sounds good to me, lets fix forward if we can.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test Linux platform


encode() and a stub for the load()

Another part of #83

Context.encode() implementation  + tests
a stub for Node.load()

Actual load() implementation will be done in subsequent PRs.

Thank you for prompt reviews!
@dennwc all feedback addressed, ready for another round.


Can't input text into editor after clicking on traceback links several times

From<EMAIL_ADDRESS>on 2012-07-12T11:21:11Z
Spyder Version:  2.1.10
Python Version:  2.7.3
Qt Version:      4.8.1, PyQt4 (API v2) 4.9.1 on Linux
What steps will reproduce the problem?

Run buggy code, and when it returns an exception, click on the hyperlink to go to the file and line causing the problem. 2. 3. What is the expected output? What do you see instead? Every once in a while, the editor will stop responding to keyboard input.  This happened rarely in Oneiric, but in Precise, it's been happening once every three or four times you click on such a link.  In Precise, the cursor returns if you change focus to a non-Spyder window, and then returns.  I believe in Oneiric the only way to regain control was to restart Spyder.  Reinstalling and resetting Spyder did not help, and the internal console can be free of exceptions when this happens.

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1114

From ccordoba12 on 2012-07-22T05:16:50Z
This seems to be a quite serious bug, but I've never seen it before. I'll try to reproduce it and come back to you again.
Summary: Can't input text into editor after clicking on traceback links several times
Labels: -Priority-Medium Priority-High Cat-Editor

From ccordoba12 on 2012-10-08T10:17:50Z
Anthony, could you give us a test file so that we can reproduce this bug? A simple file would be enough, as long as it always shows the problem.

From<EMAIL_ADDRESS>on 2012-10-07T17:12:25Z
Spyder Version:  2.1.11
Python Version:  2.7.3
Qt Version:         4.8.3, PyQt4 (API v2) 4.9.4
OS:                     Lion 10.7.5
Having the same issue. Frequently, the editor will stop responding to keyboard input.
Can remedy by removing focus from current window, e.g., executing a command in console.

From<EMAIL_ADDRESS>on 2013-01-08T07:45:25Z
Status: Fixed

From<EMAIL_ADDRESS>on 2013-03-02T05:51:15Z
A similar problem reappeared in 2.2.0beta3.  I can input text again after minimizing and restoring Spyder window.  Ubuntu 12.10.

From<EMAIL_ADDRESS>on 2013-01-08T06:32:28Z
Problem solved in version 2.2.0
My os: mac os 10.6.


Serializable commands... again...

Commands needs to be easily serializable as they basically are value objects

A very bad idea, there are way better ways of doing this


chore: update fluvio to 0.10.8

bors r+

Not sure if the Draft state affects bors? 🤔

add to bors

bors ping

bors r+


Copy-DbaDatabase: Parameter set cannot be resolved using the specified named parameters

Apologies if this is a stupid question. I'm still pretty new to Powershell so I'm not too familiar with troubleshooting errors just yet.
I am trying to perform a Copy-DbaDatabase between two machines that are not domain joined. As a result, the -BackupRestore method will not work because I can't find a way to establish a shared folder accessible to the SQL Server service accounts on both PCs. Test-DbaConnection works and I can access the source PC via SQL Server Management Studio (the destination is my local workstation).
I was hoping that the -DetachAttach method would work but I am getting this error:
Copy-DbaDatabase : Parameter set cannot be resolved using the specified named parameters.
At line:1 char:1
+ Copy-DbaDatabase `
+ ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Copy-DbaDatabase], ParameterBindingException
    + FullyQualifiedErrorId : AmbiguousParameterSet,Copy-DbaDatabase

The original command is as follows:
Copy-DbaDatabase `
-Source source-pc `
-SourceSqlCredential $sourcecred `
-Destination localhost\mssqlserver `
-Database DBName `
-DetachAttach `
-NewName "NewDBName" `
-WithReplace

Seems like I'm missing a parameter or have something ordered incorrectly, but after combing through documentation and examples I can't determine what I'm missing. Any tips?

not sure about what paramset rule you're breaking but .... detach/attach will require that the source server is able to transfer files to the destination one (using admin shares, if they're available)

That was it! Removing -WithReplace solved the issue. Thanks so much for the very quick reply.


Update promise.fromcallback.md

I just installed the latest version and .fromCallback doesn't appear to exist anymore.  I only see .fromNode

fromCallback is the primary alias in 3.x


How to mute video capture start/stop sound on Android?

Hi,
I've been struggling to find a way to disable the annoying sound played at start/stop of a video capture on a Honor 9 phone (Android Nougat). It happens even if I select mute in the stock camera app's menu, and even when the phone is in silent or vibrate mode.
I did not have this issue on my older Galaxy S5 mini (Android Lollipop).
I have tried, without success:

Passing playSoundOnCapture={false}
Disabling shutter sound

if (android.os.Build.VERSION.SDK_INT>=android.os.Build.VERSION_CODES.JELLY_BEAN_MR1){
    mCamera.enableShutterSound(false);
}
else{
    AudioManager audio= (AudioManager)_reactContext.getApplicationContext().getSystemService(Context.AUDIO_SERVICE);
    audio.setStreamVolume(AudioManager.STREAM_SYSTEM, 0,   AudioManager.FLAG_REMOVE_SOUND_AND_VIBRATE);
}

Muting system streams
https://github.com/lwansbrough/react-native-camera/pull/583

Note that this start/stop sound is NOT played by the stock camera app and other apps like Snapchat or the sample video app https://github.com/googlesamples/android-Camera2Video
Could it be something related to the fact that react-native-camera uses the Camera1 API whereas these other apps use the Camera2 API?

hi man did u get any solution yet?
playSoundOnCapture={false}
this never works

Hi, no, it's frustrating but I ended up rolling my own React Native component using Camera2 API in order to work around this issue...

@issuehuntfest has funded $20.00 to this issue. See it on IssueHunt

I had the same problem with muting start capture sound. You can mute the starting sound with (playSoundOnCapture={false}) when camera2api is enabled, but before camera2api wasn’t
being honoured. My pullrequest was merged (https://github.com/react-native-community/react-native-camera/pull/2603), which enables to use camera2apis features, Including playSoundOnCapture.


Event drag & drop

Is there any event which tells when a event is being dragged(event between drag start and drag end)

These questions are best asked on StackOverflow. This area is used for Issues and Feature Requests.
You should review the Drag & Drop documentation. Pay attention to the 'props'. There are multiple examples available.


Lokoctl uses global variable which gives wrong results if used via APIs

Consider following test code:
func TestConversion(t *testing.T) {
	testCases := []struct {
		Name        string
		InputConfig string
		Contains    string
	}{
		{
			Name:     "use external_url param",
			Contains: "https://prometheus.externalurl.net",
			InputConfig: `
component "prometheus-operator" {
  prometheus {
    external_url = "https://prometheus.externalurl.net"
  }
}
`,
		},
		{
			Name:     "no external_url param",
			Contains: "https://prometheus.mydomain.net",
			InputConfig: `
component "prometheus-operator" {
  prometheus {
    ingress {
      host                       = "prometheus.mydomain.net"
      class                      = "contour"
      certmanager_cluster_issuer = "letsencrypt-production"
    }
  }
}
`,
		},
	}

	for _, tc := range testCases {
		tc := tc
		t.Run(tc.Name, func(t *testing.T) {
			m := renderManifests(t, tc.InputConfig)
			gotConfig := m["prometheus-operator/templates/prometheus/prometheus.yaml"]

			scanner := bufio.NewScanner(strings.NewReader(gotConfig))
			for scanner.Scan() {
				line := scanner.Text()
				if !strings.Contains(line, "externalUrl") {
					continue
				}

				t.Logf("\nwanted =   externalUrl: %q.\ngot    = %s", tc.Contains, line)
			}
		})
	}
}

Whose output looks like this:
$ go test -v github.com/kinvolk/lokomotive/pkg/components/prometheus-operator -run TestConversion
=== RUN   TestConversion
=== RUN   TestConversion/use_external_url_param
    component_conversion_test.go:92:
        wanted =   externalUrl: "https://prometheus.externalurl.net".
        got    =   externalUrl: "https://prometheus.externalurl.net"
=== RUN   TestConversion/no_external_url_param
    component_conversion_test.go:92:
        wanted =   externalUrl: "https://prometheus.mydomain.net".
        got    =   externalUrl: "https://prometheus.externalurl.net"
--- PASS: TestConversion (1.86s)
    --- PASS: TestConversion/use_external_url_param (0.89s)
    --- PASS: TestConversion/no_external_url_param (0.97s)
PASS
ok      github.com/kinvolk/lokomotive/pkg/components/prometheus-operator        1.906s

Notice that in second test case we should see https://prometheus.mydomain.net but the old result persists for some reason. My first guess is that this is due to use of globals.
I think component.LoadConfig and component.RenderManifests don't do the right thing. Or we should flush out old values when components.Get is called.
func renderManifests(t *testing.T, configHCL string) map[string]string {
	name := "prometheus-operator"

	component, err := components.Get(name)
	if err != nil {
		t.Fatalf("Getting component %q: %v", name, err)
	}

	body, diagnostics := util.GetComponentBody(configHCL, name)
	if diagnostics != nil {
		t.Fatalf("Getting component body: %v", diagnostics)
	}

	diagnostics = component.LoadConfig(body, &hcl.EvalContext{})
	if diagnostics.HasErrors() {
		t.Fatalf("Valid config should not return an error, got: %s", diagnostics)
	}

	ret, err := component.RenderManifests()
	if err != nil {
		t.Fatalf("Rendering manifests with valid config should succeed, got: %s", err)
	}

	return ret
}

This is what https://github.com/kinvolk/lokomotive/pull/971 addresses.
Also related: #597.

As this is already addressed PR of @invidian. Would you mind if this is assigned to you?

Okay, this again turns out to be never-ending story to do right:

Right now we call function returning default configuration in each of the platform/component packages itself. The plan is to shift the calling to central platform/components packages, so function returning default configuration should become exporter.
Exporting function returning default configuration without exporting configuration struct itself limits the usage (I can imagine one getting default config and zero-ing all fields on purpose manually).
Exporting configuration struct builds up an API around it, exposing LoadConfig, Meta, Apply, Destroy, Initialize and PostApplyHook functions which can be misused when called on bad configuration. LoadConfig should be completely removed from it and Meta, Apply, Destroy, Initialize and PostApplyHook methods should be gated by a verifying function. Configuration struct should not implement Component/Platform interface. Validated configuration struct should.

Edit:

Right now we call function returning default configuration in each of the platform/component packages itself. The plan is to shift the calling to central platform/components packages, so function returning default configuration should become exporter.

Platforms already have NewConfig() function exported, only aks platform didn't, so I think this can be used for now.
However, the problem still applies for components. I guess they could follow the same convention as platforms for now.

Created PR #1147 to definitely address this.


Add default export of plugin

To follow the convention.
...and to allow auto importers to import this library correctly (such as nuxt/content).
https://github.com/benrbray/remark-cite/blob/3957777e06b394f6d6d824cadf290a1ec54ac279/remark-cite/lib/main.ts#L26

Lgtm, thank you.


Update to support new caskroom path

Homebrew-Cask changed its default Caskroom location:
https://github.com/caskroom/homebrew-cask/issues/21603
https://github.com/caskroom/homebrew-cask/pull/21857

Thank you all for your contribution.

I reverted the merge commit to unsquash the commits and cherry-pick them again. Your contribution will remain and can be seen in the commit history.


cmd/go: Program exits abruptly with the 語 character

I was playing around with a simple program to see how bytes/runes work
$ cat main.go
package main

import (
	"fmt"
)

func main() {
	s := "hello"
	for i := 0; i < len(s); i++ {
		c := s[i]
		fmt.Printf("index : %d has char : %c which is hex : 0x%x\n", i, c, c)
	}

	for i, rchar := range s {
		fmt.Printf("Rune at byte position : %d is : %#U\n", i, rchar)
	}
}
$

Running it gave the expected results
$ go run main.go
index : 0 has char : h which is hex : 0x68
index : 1 has char : e which is hex : 0x65
index : 2 has char : l which is hex : 0x6c
index : 3 has char : l which is hex : 0x6c
index : 4 has char : o which is hex : 0x6f
Rune at byte position : 0 is : U+0068 'h'
Rune at byte position : 1 is : U+0065 'e'
Rune at byte position : 2 is : U+006C 'l'
Rune at byte position : 3 is : U+006C 'l'
Rune at byte position : 4 is : U+006F 'o'
$

Next I thought of using words from a different language for s
$ cat main.go
package main

import (
	"fmt"
)

func main() {
	// s := "hello"
	s := "日本語"
	for i := 0; i < len(s); i++ {
		c := s[i]
		fmt.Printf("index : %d has char : %c which is hex : 0x%x\n", i, c, c)
	}

	for i, rchar := range s {
		fmt.Printf("Rune at byte position : %d is : %#U\n", i, rchar)
	}
}
$

Did you notice what happened ? The second for loop never ran ?
What ?
$ go run main.go
index : 0 has char : æ which is hex : 0xe6
index : 1 has char :  which is hex : 0x97
index : 2 has char : ¥ which is hex : 0xa5
index : 3 has char : æ which is hex : 0xe6
index : 4 has char :  which is hex : 0x9c
index : 5 has char : ¬ which is hex : 0xac
index : 6 has char : è which is hex : 0xe8
index : 7 has char : ª which is hex : 0xaa
index : 8 has char : $

Ok, next I tried to insert a fmt.Println to print random "======"
$ cat main.go
package main

import (
	"fmt"
)

func main() {
	// s := "hello"
	s := "日本語"
	for i := 0; i < len(s); i++ {
		c := s[i]
		fmt.Printf("index : %d has char : %c which is hex : 0x%x\n", i, c, c)
	}

	fmt.Println("========================")
	for i, rchar := range s {
		fmt.Printf("Rune at byte position : %d is : %#U\n", i, rchar)
	}
}
$

Same behavior ? So the problem lies in the first for loop
$ go run main.go
index : 0 has char : æ which is hex : 0xe6
index : 1 has char :  which is hex : 0x97
index : 2 has char : ¥ which is hex : 0xa5
index : 3 has char : æ which is hex : 0xe6
index : 4 has char :  which is hex : 0x9c
index : 5 has char : ¬ which is hex : 0xac
index : 6 has char : è which is hex : 0xe8
index : 7 has char : ª which is hex : 0xaa
index : 8 has char : $

To narrow down the issue, I chose a different set of characters from the same language
$ cat main.go
package main

import (
	"fmt"
)

func main() {
	// s := "日本語"
	s := "こんにちは"
	for i := 0; i < len(s); i++ {
		c := s[i]
		fmt.Printf("index : %d has char : %c which is hex : 0x%x\n", i, c, c)
	}

	for i, rchar := range s {
		fmt.Printf("Rune at byte position : %d is : %#U\n", i, rchar)
	}
}
$

This time it ran fine, second for loop executed. Hmmm
$ go run main.go
index : 0 has char : ã which is hex : 0xe3
index : 1 has char :  which is hex : 0x81
index : 2 has char :  which is hex : 0x93
index : 3 has char : ã which is hex : 0xe3
index : 4 has char :  which is hex : 0x82
index : 5 has char :  which is hex : 0x93
index : 6 has char : ã which is hex : 0xe3
index : 7 has char :  which is hex : 0x81
index : 8 has char : « which is hex : 0xab
index : 9 has char : ã which is hex : 0xe3
index : 10 has char :  which is hex : 0x81
index : 11 has char : ¡ which is hex : 0xa1
index : 12 has char : ã which is hex : 0xe3
index : 13 has char :  which is hex : 0x81
index : 14 has char : ¯ which is hex : 0xaf
Rune at byte position : 0 is : U+3053 'こ'
Rune at byte position : 3 is : U+3093 'ん'
Rune at byte position : 6 is : U+306B 'に'
Rune at byte position : 9 is : U+3061 'ち'
Rune at byte position : 12 is : U+306F 'は'
$

I focused on the problematic characters  picking one at a time
s := "日"

Program worked fine for this character
$ go run main.go
index : 0 has char : æ which is hex : 0xe6
index : 1 has char :  which is hex : 0x97
index : 2 has char : ¥ which is hex : 0xa5
Rune at byte position : 0 is : U+65E5 '日'
$

Moved to the next character
s := "本"

Again worked fine
$ go run main.go
index : 0 has char : æ which is hex : 0xe6
index : 1 has char :  which is hex : 0x9c
index : 2 has char : ¬ which is hex : 0xac
Rune at byte position : 0 is : U+672C '本'
$

Which leaves the final character
s := "語"

And as expected the program seems to exit ? Second for loop never printed
$ go run main.go
index : 0 has char : è which is hex : 0xe8
index : 1 has char : ª which is hex : 0xaa
index : 2 has char : $

I tried running the program on the Go playground and it seemed to work fine there
I am using the following version of Go on a Fedora distribution
$ go version
go version go1.16.8 linux/amd64
$

If you notice where the exit happens
index : 2 has char : $

It seems its coming from this line, so is %c or %x the problem here ?
Anybody else able to reproduce it on the exact same version ?
fmt.Printf("index : %d has char : %c which is hex : 0x%x\n", i, c, c)

I casually tried to reproduce the issue on various different online Go playgrounds (non-Google) too, by adding the line below to programs
fmt.Println(runtime.Version())

I am also seeing the issue on below version for an online playground here (https://www.onlinegdb.com/online_go_compiler)
go1.12.7

The program doesn't terminate, just your terminal can't display what it considers to be invalid output.
Closing as not a bug in go

Hi, can you show your platform?
I try to run this program on my computer
$ go version
go version go1.17.2 darwin/amd64

I couldn't find any problem

@seankhliao In that case shouldn't the 2nd for loop at least run and print Output beginning with "Rune at byte position..."  ? even if it doesn't print the actual characters ?
@zyxkad I am running Fedora on my laptop, Already gave the Go version above, Could you try this Golang online compiler here https://www.onlinegdb.com/online_go_compiler and try out the below program ? You can see that 2nd loop doesn't show all the characters either
package main

import (
	"fmt"
	"runtime"
)

func main() {
	s := "日本語"
	for i := 0; i < len(s); i++ {
		c := s[i]
		fmt.Printf("index : %d has char : %c which is hex : 0x%x\n", i, c, c)
	}

	for i, rchar := range s {
		fmt.Printf("Rune at byte position : %d is : %#U\n", i, rchar)
	}

    fmt.Println(runtime.Version())
}

Here is the output I see on the above Golang online playground
index : 0 has char : æ which is hex : 0xe6
index : 1 has char :  which is hex : 0x97
index : 2 has char : ¥ which is hex : 0xa5
index : 3 has char : æ which is hex : 0xe6
index : 4 has char :  which is hex : 0x9c
index : 5 has char : ¬ which is hex : 0xac
index : 6 has char : è which is hex : 0xe8
index : 7 has char : ª which is hex : 0xaa
index : 8 has char : '
Rune at byte position : 3 is : U+672C '本'
Rune at byte position : 6 is : U+8A9E '語'
go1.12.7

For questions about how Go works and how to use it, please use a forum, not the issue tracker.  See https://golang.org/wiki/Questions.  Thanks.


Cow: Watch YT7CRACKER(?:@gmail.com)?

Cow requests the watch of the watch_keyword YT7CRACKER(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

YT7CRACKER(?:@gmail\.com)? has been seen in 21 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


iOS badge problem

several days ago, i update the plugin,
after that,when i push the message for iOS,the badge number will never work,
can anyone else solve the problem?

I can also confirm that commenting out that line worked.

Me too. Confirmed.

Cant believe it worked. Saved 6 hours of searching, thanks.
But this is very hacky as I can't update the plugin now. Wonder what the real solution would be.

@massanishi It is a Adobe problem, they never update the Github repository. An option would be to make a Github Fork.


FileNotFoundError: [Errno 2] No such file or directory: '/home/gaurav/Desktop/Robotics/concept-graphs/Datasets/Replica/room0/gsa_classes_none.json'

Hello @krrish94
I am trying to run the following command, but I am getting an error given below. It is searching for gsa_classes_none.json' in the Replica data directory, but I downloaded the dataset using the instructions given. There is no gsa_classes_none.json' file available in the original directory. How can I get this file?
Thanking you
The command I am running:
python conceptgraph/slam/cfslam_pipeline_batch.py
dataset_root=/home/gaurav/Desktop/Robotics/concept-graphs/Datasets/Replica
dataset_config=/home/gaurav/Desktop/Robotics/concept-graphs/conceptgraph/dataset/dataconfigs/replica/replica.yaml
stride=5
scene_id=room0
spatial_sim_type=overlap
mask_conf_threshold=0.95
match_method=sim_sum
sim_threshold=1.2
dbscan_eps=0.1
gsa_variant=none
class_agnostic=True
skip_bg=True
max_bbox_area_ratio=0.5
save_suffix=overlap_maskconf0.95_simsum1.2_dbscan.1_merge20_masksub
merge_interval=20
merge_visual_sim_thresh=0.8
merge_text_sim_thresh=0.8
An error that I am getting:
2023-10-08 22:55:30,508] [INFO] [real_accelerator.py:110:get_accelerator] Setting ds_accelerator to cuda (auto detect)
Setting image height and width to 680 x 1200
Error executing job with overrides: ['dataset_root=/home/gaurav/Desktop/Robotics/concept-graphs/Datasets/Replica', 'dataset_config=/home/gaurav/Desktop/Robotics/concept-graphs/conceptgraph/dataset/dataconfigs/replica/replica.yaml', 'stride=5', 'scene_id=room0', 'spatial_sim_type=overlap', 'mask_conf_threshold=0.95', 'match_method=sim_sum', 'sim_threshold=1.2', 'dbscan_eps=0.1', 'gsa_variant=none', 'class_agnostic=True', 'skip_bg=True', 'max_bbox_area_ratio=0.5', 'save_suffix=overlap_maskconf0.95_simsum1.2_dbscan.1_merge20_masksub', 'merge_interval=20', 'merge_visual_sim_thresh=0.8', 'merge_text_sim_thresh=0.8']
Traceback (most recent call last):
File "/home/gaurav/Desktop/Robotics/concept-graphs/conceptgraph/slam/cfslam_pipeline_batch.py", line 138, in main
classes, class_colors = create_or_load_colors(cfg, cfg.color_file_name)
File "/home/gaurav/Desktop/Robotics/concept-graphs/conceptgraph/slam/utils.py", line 43, in create_or_load_colors
with open(classes_fp, "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: '/home/gaurav/Desktop/Robotics/concept-graphs/Datasets/Replica/room0/gsa_classes_none.json'
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

Hi @gauraviiita , thanks for the question! Did you run scripts/generate_gsa_results.py before running slam/cfslam_pipeline_batch.py?

Hi @georgegu1997
I run the scripts/generate_gsa_results.py but it is showing an error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='cdn-lfs.huggingface.co', port=443):
Even though I downloaded pytorch_model.bin manually and placed it in the current working directory still it is showing an error as I mentioned above.
Please suggest me way to execute this command scripts/generate_gsa_results.py.
Thank you!

As mentioned in the other issue, this is probably an Internet connectivity issue of either the Hugging Face or your local side. Can you try running the command using a different machine in a different network, or through a VPN service?
The No such file or directory error should be gone when you run the scripts/generate_gsa_results.py script first. So I am closing this issue. But feel free to discuss what you try and observe.


iterate through all vertexCollections of a graph to find a vertex to …

…display
Scope & Purpose
The Graph Viewer will not iterate through all available vertexCollections (which are defined in the graph definition) to find a possible vertex document to start with. Before just a single random vertex was chosen. Now all vertex collections will be checked before inform the user that the graph does not have any available vertices to draw/render.

[x] Bug-Fix for devel-branch (i.e. no need for backports?)

Yes we need backports for all supported branches. Will create after the review is done.

[ ] Bug-Fix for a released version
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)

Related Information

No ticket available (Slack)

Testing & Verification
This change is a trivial rework / code cleanup without any test coverage.
Additionally:

[x] I ensured this code runs with ASan / TSan or other static verification tools
No C++ modification is done here. Only JavaScript Aardvark modifications
So, no Jenkins run is needed as well.

Documentation
No documentation is available.

All new Features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the changelog so that
developers and users have a concise overview.

[x] Added a Changelog Entry (referencing the corresponding public or internal issue number)

http://jenkins01:8080/view/PR/job/arangodb-matrix-pr/5903/


Consider first-class support for Delta Stream Capture

Currently only the full state is recorded in the memstores and optionally in the storage.
There is already a good API that supports streaming incremental changes to websockets.
This API can be used by the state itself to record all changes in a custom delta capture implementation, e.g. plain kafka topic. This API of the DeltaStreamCapture needs to enable both simple and complex implementations. Simple is simply writing all messages to a file or kafka topic, complex could mean partially defined function which sends each different type of message into a different topic or which transforms all incremental messages to a new type.

this is now supported by output[] stream handles


[BlackrockIO] Warn user in case of unknown event code

With this PR the user only receives a warning in case there is an event code
that's not known to BlackrockRawIO instead of raising an Error which
would at least cause some valid data not to be loaded.

Hello @bjoern1001001! Thanks for submitting the PR.

There are no PEP8 issues in the file neo/rawio/blackrockrawio.py !

Coverage remained the same at 49.586% when pulling 999fa486b194239449e9cbe6dbe4f9af30c56e07 on bjoern1001001:warn_unknown_events into a197ece63d72fcf1a81c8a199ff01cc66f9a018d on NeuralEnsemble:master.

Coverage remained the same at 49.586% when pulling 999fa486b194239449e9cbe6dbe4f9af30c56e07 on bjoern1001001:warn_unknown_events into a197ece63d72fcf1a81c8a199ff01cc66f9a018d on NeuralEnsemble:master.

Coverage remained the same at 49.586% when pulling 999fa486b194239449e9cbe6dbe4f9af30c56e07 on bjoern1001001:warn_unknown_events into a197ece63d72fcf1a81c8a199ff01cc66f9a018d on NeuralEnsemble:master.

Coverage remained the same at 49.586% when pulling 999fa486b194239449e9cbe6dbe4f9af30c56e07 on bjoern1001001:warn_unknown_events into a197ece63d72fcf1a81c8a199ff01cc66f9a018d on NeuralEnsemble:master.


[SPARK-31163][SQL] TruncateTableCommand with acl/permission should handle non-existed path

What changes were proposed in this pull request?

This fix #26956
Wrap try-catch on fs.getFileStatus(path) within acl/permission in case of the path doesn't exist.
Why are the changes needed?

truncate table may fail to re-create path in case of interruption or something else. As a result, next time we truncate table on the same table with acl/permission, it will fail due to FileNotFoundException. And it also brings behavior change compares to previous Spark version, which could still truncate table successfully even if the path doesn't exist.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Added UT.

@ cc @viirya @cloud-fan @dongjoon-hyun

Test build #119836 has started for PR 27923 at commit cd84627.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24567/
Test PASSed.

Test build #119836 has finished for PR 27923 at commit cd84627.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119836/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24576/
Test PASSed.

Test build #119849 has started for PR 27923 at commit cd84627.

Test build #119849 has finished for PR 27923 at commit cd84627.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119849/
Test PASSed.

thanks all!


ViewerJS

I have been evaluating ViewerJS for use in a company Wordpress intranet - the issue we had was viewing PDF's in both IE explorer which worked fine and FIrefox which didn't
To View pdf files in explorer we were using the php Include statement
I was pleased to find that ViewerJS worked fine in Firefox but when testing with Explorer I was welcomed with the following

I hope this is of use to you

Hi and thanks for the feedback. Which version of IE explorer do you use?

11 I think - it was at work and I am at home now - fairly sure it is 11 as it is on windows 7 and the machines update regularly
I am keen to find out more about PDF file formats and processing them
David
Date: Tue, 5 May 2015 08:55:15 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [ViewerJS] ViewerJS (#139)
Hi and thanks for the feedback. Which version of IE explorer do you use?
—
Reply to this email directly or view it on GitHub.

How to use this thing, ask a demo


AKS only for Migrate - !?

[Enter feedback here]
Hi,
I am wondering why in the flow chart for selecting a candidate compute service, AKS can only be reached in the path for Migration.
Why not also in the path for Build New?
Thanks
Johannes
PS: There is a typo:  'loud' optimized, I guess this should be load optimized.

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

ID: 510808e8-db5d-6b07-9a04-6edb743a2dfc
Version Independent ID: 20d098aa-ef31-cb26-d677-765cfc9a7164
Content: Choose an Azure compute service - Azure Architecture Center
Content Source: docs/guide/technology-choices/compute-decision-tree.yml
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @EdPrice-MSFT
Microsoft Alias: architectures

@johkrec Thanks for your feedback. We already have this similar question open in #3611 (related to Containerized Apps) and the author will review and update as appropriate.
Typo is duplicate of #3618 and should be "Cloud Optimized". We made the changes, and it will be published soon.
I will proceed to close this thread and feel free to reach out for any further questions.


Feature Request: Customise Notification Alert Messages

We need to customize Notification Alert Messages for NON-English language user. There are two message to translate:

On "Last Reboot" button appears the Message "Please restart your Mac regularly"
On  "Mac Password" message: "Network Unavailbale"
It would be great they were customizable.
thanks

Is there a way to customize the notification alert message even for english? I would like to make it more customize so that to align with the wording changes and so on


nixos/virtualisation: add option for explicitly named network interfaces

Adds a new option to the virtualisation modules that enables specifying explicitly named network interfaces in QEMU VMs. The existing virtualisation.vlans is still supported for cases where the name of the network interface is irrelevant.
This feature is useful for creating VMs with network configurations that exactly match the network configuration of real-world devices for more accurate SITL testing.
Description of changes

Added virtualisation.interfaces option to qemu-vm.nix.
Updated the default value of virtualisation.vlans to empty if virtualisation.interfaces is not empty. This avoids a scenario where a user specifies virtualisation.interfaces, but inadvertently still has a network interface from the default virtualisation.vlans.
Updated build-vms.nix to create the new network interfaces using QEMU options.
Updated build-vms.nix to rename network interfaces using udev rules.
Updated testing-python.nix to use the union of the VLANs specified in virtualisation.vlans and virtualisation.interfaces when collecting the list of VLANs.
Updated networking tests in nixos/tests/networking.nix to use the new option where applicable. This cut down on unnecessary overrides to remove IP addresses from network interfaces.

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

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

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

@roberth is this still the right branch to open my PR?

@roberth is this still the right branch to open my PR?

In theory, yes, but I think I've rebased between you doing a checkout and opening this PR. I'll probably have to rebase again before this is merged upstream, quite possibly multiple times.

Got it. Maybe it makes more sense for me to track the progress of your PR and then rebase mine once that's merged? How long do you think it will take to get yours merged?

@roberth is this still the right branch to open my PR?

In theory, yes, but I think I've rebased between you doing a checkout and opening this PR. I'll probably have to rebase again before this is merged upstream, quite possibly multiple times.

Got it. Maybe it makes more sense for me to track the progress of your PR and then rebase mine once that's merged? How long do you think it will take to get yours merged?

The maintainer is still reviewing. Can't really give an indication, but progress is taking place.

track the progress of your PR

I don't expect a lot of change.


Add a Reaction < Message class to handle additon/removal of reactions

This PR handles the only two available Slack API events for reactions : reaction_added and reaction_removed.
A new class, Reaction < Message has been included with additional attributes name and item (following Slack convention for reactions - name contains the type of reaction and item contains information on the item reacted to). An event attribute distinguishes between addition and removal of reactions.
In message_handler.rb, a handle_reaction method (very similar to the handle_message method) initialises and dispatches Reaction objects. Continuing on https://github.com/litaio/lita-slack/pull/75, the timestamp of the reaction is saved in the extensions hash.
Relevant specs are also included.

Thank you for the PR and for the excellent quality code!
I'm not sure I agree with treating a reaction event with similar semantics to a standard message. A reaction is really a separate data type that is attached to a message as metadata. I don't think it's a safe assumption that someone reacting to a message with, for example, :stuck_out_tongue_winking_eye:, is equivalent to the user having sent the text message "stuck_out_tongue_winking_eye" to the room. It's certainly not the case for removing a reaction.
The adapter should definitely be able to handle reactions, however. I think it makes more sense to treat them as Lita events, perhaps :slack_reaction_added and :slack_reaction_removed, where the payload is a plain object with only the relevant fields. Inheriting from Lita::Message brings a lot of extra baggage along.

@jimmycuadra Closing this PR. A new PR based on your feedback has been submitted (https://github.com/litaio/lita-slack/pull/79)


build_library: support multi-arch in generate_au_zip

To be able to support arm64 native SDK without cross builds, we should make generate_au_zip support both architectures, amd64 and arm64.
Without doing that, build_image fails with ERROR : Required WHITE_LIST items ld-linux-x86-64.so.2 not found!!!, because the script recognizes only amd64 libs in WHITE_LIST.
We should first determine the architecture in build_image, before running generate_au_zip, and pass the architecture, either amd64 or arm64.
Also add allow_list and ld_linux parameters to necessary functions.
Replace black list with deny list, white list with allow list.
How to use
...
./build_image

Testing done
CI passed: http://jenkins.infra.kinvolk.io:8080/job/os/job/manifest/3281/cldsv

CI passed


Update SOP in CanCOGeN template

I added the Q/C datasets link to the DH manual for the CanCOGeN template.
Can we update the hard copy of the SOP in the DH in the next release?

Done on DH draft


Fix no-useless-split-diff-view doesn't work newly loaded files

This fixes #4488.
Maybe it's off-topic, but I think the listener's name should be something like onDiffFileLoad instead of onPrFileLoad. It made me wonder whether it works on non-PR pages.
Test URLs
I've tested this at:

https://github.com/cpeditor/cpeditor/pull/223/files#diff-0f46ff9666ae4dc3789ffaed0f2c4a93fbd4469ab486af80f1b262cb44b7a038

https://github.com/cpeditor/cpeditor.github.io/commit/b349da5cefab537691e1988f74252759aff9c300

Maybe it's off-topic, but I think the listener's name should be something like onDiffFileLoad instead of onPrFileLoad. It made me wonder whether it works on non-PR pages.

👍

Should I rename the function?

Since the feature might run several times on the same page, I think the already visited diffs should be marked and excluded:
for (const diffTable of select.all('.js-diff-table:not(.rgh-diff-visited'))

then add the class in the loop (class name is just an example, it could be better)


Scala 3.5.2 artifacts seem to contain empty jars

I downloaded both the zip and tar.gz versions of scala-3.5.2 on the releases page.
In the lib directory, there are only 3 .jar files, all of them only contain a manifest file. trying to run for example scalac gives:
Error: Could not find or load main class dotty.tools.MainGenericCompiler
Caused by: java.lang.ClassNotFoundException: dotty.tools.MainGenericCompiler

The actual eval line constructed by the scalac script in my case, is:
/usr/lib/jvm/java-21-jetbrains/bin/java" -Xmx768m -Xms768m -classpath "/usr/share/scala3/lib/scala.jar:/usr/share/scala3/lib/with_compiler.jar" -Dscala.expandjavacp=true -Dscala.usejavacp=true -Dscala.home="/usr/share/scala3" dotty.tools.MainGenericCompiler

Huh. I figured out that, as opposed to LTS (3.3.4), the artifacts contain a maven2 directory, next to the usual bin and lib directory. That maven2 directory seems to contain all the jar archives that would normally end up in lib. The classpath constructed by the scripts look at lib, and don't seem to be maven2 directory-aware. This must be a artifact packaging mistake right?

maven2 directory was introduced in https://github.com/scala/scala3/pull/20351 for me it seems the expected behaviour.  It was part of integrating scala-cli as default runner.
@bishabosha @hamzaremmal @Gedochao opinions?

@rubin55 those three jars list the classpath for the commands in the manifest file "Class-Path" attribute

Ok sure, but the intention can't be that the other command line utilities fail to run right? I suppose the scripts need to be adjusted so they construct a valid classpath or somesuch? Because at least scalac doesn't work like this.

I downloaded both the zip and tar.gz versions of scala-3.5.2 on the releases page.

Alright, so right there, downloading the artifacts manually is not a recommended way of installation.
@rubin55 You should be able to make things work with power user knowledge, sure.
So perhaps we miss documentation for this case, even if it's not recommended. (cc @hamzaremmal)
However, first things first.
Please refer to https://www.scala-lang.org/download/ and let us know if the recommended ways of installation work for you.
If you run into any issues, please give us more context on your setup.

Ok sure, but the intention can't be that the other command line utilities fail to run right? I suppose the scripts need to be adjusted so they construct a valid classpath or somesuch? Because at least scalac doesn't work like this.

I am suspecting you need to put the jars in the right directories for the scripts to be able to find them. The installer would normally do it for you.
Other than that, is there any reason why you'd download the artifacts manually?

@rubin55 What's the structure of the /usr/share/scala3 folder ? The paths mentionned stored in the Class-Path attribute are relative and therefore sensible to the layout of the /usr/share/scala3 structure.

@Gedochao

Second option recommended installation: "Download release from GitHub".
For your information, I've been running Scala like this for years. Also note that 3.3.4, the latest LTS release still works correctly (just have a look at the lib subdirectory of any 3.3.4 artifact to understand why).

@hamzaremmal @Gedochao
It's working now. I was using the AUR package which recently updated from LTS to latest; in the PKGBUILD the directories that need to end up in the /usr/share/scala3 directory are explicitly specified and was missing the maven2 directory. Sorry for the noise! TL;DR;, package works, got confused by the empty MANIFEST only libs in lib, root-cause was missing maven2 directory.

Second option recommended installation: "Download release from GitHub".

Right... we do not recommend it on https://www.scala-lang.org/download/, but we do under a specific release, my bad.
We should fix it at some point, installing via a package manager should be considered the default.

TL;DR; downloadable artifacts work, got confused by the empty MANIFEST-only jars in lib, root-cause was missing maven2 directory.

FYI, the changes were introduced in Scala 3.5.0. I'm glad it works.
Seems like there's nothing to do in the scope of this issue, closing this as invalid.


[Feature request] Add KMM sample using compose

Thanks for the feature request. This repo is focussed on providing samples for Android so we do not want to complicate them further at this point.


Embedded styles for HMR mode, resolved css injecting problem

All styles are extracting now, but when I'm starting hmr mode and changing my styles, I have to refresh my browser each time. Is there any other way to solve the problem?

After four months of testing this PR daily, I've decided to merge it.
:)

Ooo! Great news :)) @JeffreyWay thank you so much!!!


Fix kubebuilder marker issues

Due to the below issues, we disable marking a couple of deployment spec fields as mentioned in https://github.com/intel/pmem-csi/pull/695/files#diff-6dbccdd31f0d920af3f26b2bbb144b09R51-R56
Fails setting default(+kubebuilder:default=value):

https://github.com/kubernetes-sigs/controller-tools/issues/478

Fails setting min/max for integers:

https://github.com/helm/helm/issues/5806

Reenable(replace '-' prefix with '+') markings in code once these upstream issues got fixed.

This got resolved by https://github.com/intel/pmem-csi/pull/832


Add zipper-run extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @pernielsentikaer
Thanks for your review. I took in consideration your comments and made a bunch of changes in the code.
It's ready for review again

@pernielsentikaer I've added the error handling. You can re-review

hi @pernielsentikaer
let me know if this is good or if you need something else

There is a small empty state flicker when I search for readme - do you have an idea on how to eliminate that?

@pernielsentikaer can you provide me with a more clear example of when this is happening so I can check?

It might be hard to see but if you look right before data is returned it shows "No Results" - I'm not sure how easy it is to do something - I can also try if you don't have an idea 🙂
https://github.com/raycast/extensions/assets/38033107/5c872d12-1e0d-43a3-b12f-7b1e6ae34b0a

@pernielsentikaer is it possible to be some issue with the List component?
I've tried with a static list like this and I see the same behavior

It seems like we have an error as you said, thanks for pointing it out - we'll address that in the next release and will not be a blocker now 🙂

No, everything should be fine now 🔥
Thanks for your cooperation. It was a pleasure working with you 🙂


New version: ente-io.photos-desktop version 1.6.63

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Manual Validation ended with:

Is this package missing a dependency?

Still investigating; seems to run fine on my machine, so trying to find the error


[Bugfix] fix: command not found p9k::defined in debug/font-issues.zsh

p9k::register_icon in functions/icons.zsh makes calls to p9k::defined which is defined functions/utilities.zsh
Before :

After :

Nice job, @qkdreyer!


pip should install the dev requirements when provisioning

Changes the pip command to install the packages in requirements-dev.txt instead of requirements.txt so packages like nose will be automatically installed when provisioning.

Rad, thanks!


unclear error when CONFIG_FILE_PATH points to a directory

I accidentally built with CONFIG_FILE_PATH pointing to a directory, not the file itself, and the error for this (admittedly unusual) case is unclear:
$ ./gitup stable
gitup: load_configuration: cannot stat file /usr/home/emaste/src/gitup: No error: 0: No error: 0

I just pushed out a fix for the duplicate "No error: 0" but it doesn't look like libucl will let me adjust the error code of 0 -- there doesn't appear to be a function that lets me manually set that value and trying to set parser->err_code = value throws an error.

I just pushed out a fix for the duplicate "No error: 0" but it doesn't look like libucl will let me adjust the error code of 0 -- there doesn't appear to be a function that lets me manually set that value and trying to set parser->err_code = value throws an error.

I think I've found a workaround that replaces "No error: 0" with "Input/output error".  Thank you!

I think I've found a workaround that replaces "No error: 0" with "Input/output error".  Thank you!

I am confused, why not use stat(2) with S_ISREG() macro?

I am confused, why not use stat(2) with S_ISREG() macro?

I just added a stat(2) check.  How does it look now?

I just added a stat(2) check.  How does it look now?

That's a nicer error for my original case:
gitup: load_configuration: cannot load /home/emaste/src/gitup: Is a directory

We can still trigger less-nice errors, albeit in cases less likely to be hit accidentally:
$ ./gitup -C /dev/null stable
load_configuration: cannot stat file /dev/null: No error: 0

That's a nicer error for my original case:
gitup: load_configuration: cannot load /home/emaste/src/gitup: Is a directory

We can still trigger less-nice errors, albeit in cases less likely to be hit accidentally:
$ ./gitup -C /dev/null stable
load_configuration: cannot stat file /dev/null: No error: 0

I just added a nicer error message for character devices, fifos and whiteouts.  Did I miss any others?

I just added a nicer error message for character devices, fifos and whiteouts.  Did I miss any others?

I just added a nicer error message for character devices, fifos and whiteouts. Did I miss any others?

While the change seems to be nice, does not make sense to handle each negative case instead of handling one positive case? Simply says that the path isn't a regular file?

I just added a nicer error message for character devices, fifos and whiteouts. Did I miss any others?

While the change seems to be nice, does not make sense to handle each negative case instead of handling one positive case? Simply says that the path isn't a regular file?

Good call (thank you!)  How does it look now?

Good call (thank you!)  How does it look now?

LGTM:
# /gitup -C /
gitup: load_configuration: cannot load /: Inappropriate file type or format
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/ada
ada0    ada0p2  ada0p4  ada0p6  ada1    ada2
ada0p1  ada0p3  ada0p5  ada0p7  ada1p1  ada2p1
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/ada2p1
gitup: load_configuration: cannot load /dev/ada2p1: Inappropriate file type or format
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/std
stderr  stdin   stdout
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/stdout
gitup: load_configuration: cannot load /dev/stdout: Inappropriate file type or format

@emaste WDYT?

LGTM:
# /gitup -C /
gitup: load_configuration: cannot load /: Inappropriate file type or format
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/ada
ada0    ada0p2  ada0p4  ada0p6  ada1    ada2
ada0p1  ada0p3  ada0p5  ada0p7  ada1p1  ada2p1
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/ada2p1
gitup: load_configuration: cannot load /dev/ada2p1: Inappropriate file type or format
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/std
stderr  stdin   stdout
root@bsd1srv:~/gitup (main %=)
# /gitup -C /dev/stdout
gitup: load_configuration: cannot load /dev/stdout: Inappropriate file type or format

@emaste WDYT?

Yes LGTM

Yes LGTM

Please close this then.

Please close this then.


Rails 5.1

I rule and have completed some work on Case Manager that's ready for review!
This brings us up to Rails 5.1
This pull request makes the following changes:

Gemfile updates
Gemfile.lock updates

It relates to the following issue #s:

Fixes #1143

Codecov Report

Merging #1167 into master will decrease coverage by 6.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1167      +/-   ##
==========================================
- Coverage    95.3%   88.44%   -6.87%
==========================================
  Files          52       52
  Lines        1151     1151
==========================================
- Hits         1097     1018      -79
- Misses         54      133      +79

Impacted Files
Coverage Δ

app/models/concerns/user_searchable.rb
37.93% <0%> (-55.18%)
:arrow_down:

app/models/concerns/exportable.rb
59.25% <0%> (-37.04%)
:arrow_down:

app/controllers/users_controller.rb
54.54% <0%> (-36.37%)
:arrow_down:

app/helpers/users_helper.rb
57.14% <0%> (-28.58%)
:arrow_down:

app/controllers/patients_controller.rb
76.66% <0%> (-23.34%)
:arrow_down:

app/lib/reporting/patient.rb
81.81% <0%> (-18.19%)
:arrow_down:

app/controllers/application_controller.rb
88.88% <0%> (-11.12%)
:arrow_down:

app/models/user.rb
98.88% <0%> (-1.12%)
: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 af4bd87...3ce8b2f. Read the comment docs.

@colinxfleming I'll tackle the tests if you want to take a look at knapsack.

on it!

Knapsack / CI gem in #1177

We're getting flaky test failures at this point, which is good, I guess. @DarthHater for review

@DarthHater if this looks good to you, let's SMASH THAT MERGE BUTTON and work on the other components in separate PRs per the plan.

eat it build.

I'll take a gander at this tomorrow. Thanks for pushing it forward while I was being the road warrior.

Swish. Yeah, if this passes muster we should move as is and then port over the tests incrementally in their own PRs.

@DarthHater ping for a review on this if you have a minute. I'd like to get some of this moving as I think the move to systemtest for some of our problem children will fix some of our flaky test garbage.

Build is passing. Some local poking at comes back clean. I'm declaring victory and merging so we can get this into sandbox.


array_unshift() expects parameter 1 to be array, null given

Hey,
Just made today composer update and we get the following error:
PHP Warning:  array_unshift() expects parameter 1 to be array, null given in /.../vendor/erusev/parsedown-extra/ParsedownExtra.php on line 31
PHP Fatal error:  Call to undefined method ParsedownExtra::blockDefinitionList() in /.../vendor/erusev/parsedown/Parsedown.php on line 219

Does this update break compatibility?
- Removing erusev/parsedown (1.1.4)
- Installing erusev/parsedown (1.4.2)

Can you provide the Markdown text that produced this?

It looks like you have updated Parsedown but not Parsedown Extra. You should upgrade both in order for ParsedownExtra to work properly.

Ok, I fixed it by updating the parsedown-extra to 0.5.1, but parsedown was only defined as a dependency in extra, so I would expect it to freeze parsedown version with a compatible one, not update freely. Thank you!


different forecast results

Hi, i have a question about different results from same code, i guess that the parameters were not transformed to the define code 'make_forecast'...

Having same issue here! As the nouvelle practitioner I am, I would love to have some insights from @gcarq's expertise.
Pasting my output from a Jupyter notebook:
Using TensorFlow backend.

train_dataset: 100, test_dataset: 72

fitting model	:   0%|          | 0/100 [00:00<?, ?it/s]/root/anaconda3/lib/python3.7/site-packages/ipykernel_launcher.py:160: UserWarning: The `nb_epoch` argument in `fit` has been renamed `epochs`.
fitting model	: 100%|██████████| 100/100 [01:06<00:00,  1.50it/s]
predicting data	: 100%|██████████| 100/100 [00:00<00:00, 327.97it/s]

Train Score: 17.00 RMSE
Test Score: 33.80 RMSE

/root/anaconda3/lib/python3.7/site-packages/numpy/core/_asarray.py:136: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
  return array(a, dtype, copy=False, order=order, subok=True)
/root/anaconda3/lib/python3.7/site-packages/numpy/core/_asarray.py:136: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
  return array(a, dtype, copy=False, order=order, subok=True)
/root/anaconda3/lib/python3.7/site-packages/numpy/core/_asarray.py:136: VisibleDeprecationWarning: Creating an ndarray from ragged nested sequences (which is a list-or-tuple of lists-or-tuples-or ndarrays with different lengths or shapes) is deprecated. If you meant to do this, you must specify 'dtype=object' when creating the ndarray
  return array(a, dtype, copy=False, order=order, subok=True)


How to detect "close" click

Currently,
When "close" (back) in the CAB is clicked. The CAB fanishes (gets removed).
Who do I detech this click and perform my own action on it ?

Returning false in the onCabFinished() callback will keep it from closing.


Add the FUND tag to all Funds

Now that we are being more public about Funds, I think it would be a good idea to automatically add the FUND tag on each existing Fund profile and new ones as they are created, as that labels it as a Fund at the top and helps make it more clear. It's also good for us as support because we can see easily whether something is a Fund or an Org profile.

It actually looks like the code already returns FUND https://github.com/opencollective/opencollective-frontend/blob/master/lib/collective.lib.js#L44 as the main tag, unless there's a more specific one (like open source) defined, or if the host is OSC (we force "OPEN SOURCE" in this case)

I'm not sure I understand what that means @Betree - that the Funds should already have this tag? I can't see it. Is it just not displayed?

It is. The one you shared, https://opencollective.com/google, is an organization - not a fund.
If you look at https://opencollective.com/parentpreneur-foundation, the tag is there:

Ahhhhhh I see.
@piamancini I thought we made Google into a Fund, but I guess I was wrong.


Update checking duplicated config files in configuration level3

Hi Son,
unfortunately merge conflict with #179
Reason: In docstrings of CConfig.py I have renamed the JSON file extensions from .json to .jsonp and I created a new PDF. Could you please take over these changes in your pull request also? Are only few positions.

Hello Holger,
Your pull-request could be merged first then I will resolve conflicts of my pull-request :)
Thank you,
Son

Hi Thomas,
The conflict is merged.
Thank you,
Son


How can i increase post_max_size and upload_max_filesize

How can i increase post_max_size and upload_max_filesize? thanks

-> https://github.com/docker-library/wordpress/issues/10#issuecomment-329332019

thank you.

Can you confirm if setting re-definitions in .htaccess works too?
https://github.com/docker-library/wordpress/issues/10#issuecomment-446841644 (the option B)


Inconsistent class names between gulp serve and fast-serve

If you have a CSS class names with a few capital letters - .myABCForm, fast-serve changes it to .myAbcFrom, while gulp serve preserves the original names.
It leads to different problems with fast-serve - when class name disappears from html markup or just runtime typescript errors.

@Dangerous-Mind The issue is fixed in 1.19.1. To test it, you could explicitly install npm i<EMAIL_ADDRESS>-DE.
Please check and tell if it works for you.

The issue was solved with this new version.
Many thanks for your excellent work on this project @s-KaiNet

Thank you! Glad it fixed now, closing.


Dictionary input to count_words

This PR resolves #954
Dictionary input to count_words

@Jarvi-Izana - please re-review. Have your issues been resolved? If so, you need to approve this change.

@TobyRoseman do you think we should treat negative word count as a valid input?

@TobyRoseman do you think we should treat negative word count as a valid input?

I don't think the dictionary values are intended to be word counts. The original example which we are trying to enable has non-integer values. As I said in the original issue (#954) the usage for this functionality is not clear to me.

remember to merge your nice work. ;-)

@Jarvi-Izana Thanks! We don't merge rights. Can you please merge it
Thanks in advance.

@dhivyaaxim I see. Merged. Nice work!


Modernize string handling

Yeah, TCHAR and _T() were invented to simultaneously support Unicode and non-Unicode versions of Windows. There is no point anymore to support non-Unicode versions (that would be Windows 98 and earlier).
You can read more about this topic on StackOverflow:
https://stackoverflow.com/questions/234365/is-tchar-still-relevant
https://stackoverflow.com/questions/6315566/should-i-eliminate-tchar-from-windows-code

Thanks for the review!


Connect: allow specifying resources in templates

Motivation: AKS has some azure-policies in place which deny spawning pods without a resource definition. This applies to the packrat-restore pods and eventually denies app deployment.
values.yml:
  templateValues:
    service:
      type: ClusterIP
      annotations: {}
      labels: {}
    pod:
      annotations: {}
      labels: {}
      serviceAccountName: ""
      volumes: []
      volumeMounts: []
      imagePullPolicy: ""
      imagePullSecrets: []
      initContainers: []
      extraContainers: []
      resources: {} # <---- NEW

As a workaround I am currently trying with
launcher:
  enabled: true
  useTemplates: false
  launcherKubernetesProfilesConf:
    "*":
      job-json-overrides:
        - name: resources
          target: /spec/template/spec/containers/resources
          json:
            requests:
              cpu: "100m"
              memory: "200Mi"
              ephemeral-storage: "200Mi"
            limits:
              cpu: "1000m"
              memory: "1Gi"
              ephemeral-storage: "2Gi"

but this specification results in a 500 from the launcher.
EDIT:
I see the values are just passed along as is and are not limited by the values.yml specification.
So the following
launcher:
  enabled: true
  useTemplates: false
  templateValues:
    pod:
      resources:

works!

Howdy @pat-s ! Thanks for sending this along!
We actually have a slightly more native option for configuring resources today.
launcher:
  launcherKubernetesProfilesConf:
    "*":
      default-cpu: 2
      default-mem-mb: 2048
      cpu-request-ratio: 0.1
      memory-request-ratio: 0.2

https://docs.posit.co/job-launcher/kube.html#kube-profiles
Unfortunately this is global and not specified per content. We are shooting for an even-more-native implementation which will allow users to specify resources inside of the UI similar to "number of processes", "connection timeouts," etc. This will allow varying resource selection per content item using the API.
If you have any feedback on whether you would expect this resource-permission selection to be set by publishers or admins-only, or what mechanics would be ideal (i.e. a default value, and users can override, etc.), that would be most welcome!

Just noting that we have a resource block in our helm connect values currently, but it's not referenced anywhere: https://github.com/rstudio/helm/blob/main/charts/rstudio-connect/values.yaml#L178-L188.
The commented out code examples are also causing helm template to silently fail in our ci with:
Error: Deployment in version "v1" cannot be handled as a Deployment: v1.Deployment.Spec: v1.DeploymentSpec.Template: v1.PodTemplateSpec.Spec: v1.PodSpec.Containers: []v1.Container: v1.Container.Resources: v1.ResourceRequirements.Limits: unmarshalerDecoder: quantities must match the regular expression '^([+-]?[0-9.]+)([eEinumkKMGTP]*[-+]?[0-9]*)$', error found in #10 byte of ...|led":false,"ephemera|..., bigger context ...|esources":{"limits":{"cpu":"2000m","enabled":false,"ephemeralStorage":"200Mi","memory":"2Gi"},"reque|...


better support for iterative live migration

This really has two parts:

we need to support iterative memory migration in the first place (via p.haul), so that in case the container has allocated a bunch of memory, the downtime during a live migration doesn't include all the time it takes to send this memory
we need to support more than one iteration of sending filesystems. Right now we do one iteration, but we should iterate as necessary. To do this, we'll need a nice way to figure out the filesystem delta, which may only be possible with zfs and btrfs.

Closed as we now have the memory pre-migration support and there's no more upstream activity on p-haul.


Remove background on white thing on white background

Hello, I also find some cases that transparent-background will not perform well, hope it can help for your next train :) It is white thing on white background
Original

After remove

Closing due to inactivity.


core: fix repr handling of recursive object references

Description: Generic serialization of objects via repr does not account for recursive object references. This leads to issues when 3rd parties extend Langchain classes and add properties which contain circular references (example). In these cases, a RecursionError will be thrown when the sys.getrecursionlimit() is reached. However a very difficult to debug Segmentation Fault assertion occurs if sys.setrecursionlimit(...) is set too high, and the process runs out of memory. Using a safe_repr implementation that throws an exception early upon finding a circular reference fixes this problem.
Twitter handle: @polywrap_io @dorgjelli @nerfzael

@dOrgJelli could we add a unit test?

closing due to inactivity


hv.Area not plotted if plotting limits smaller than value of Area

area = hv.Area((np.arange(10), np.ones(10)))
area

area[:, 0:0.5]

What were you expecting to happen? You sliced by the value dimension and all your values are one so you're left with an empty Area Element after slicing.

I would like to show a small tail, like in:
area = hv.Area((np.arange(10), 1-np.linspace(10)**2))
area[:, 0.5:1]

If you want I could post the actual image I'm trying to produce.

That would be helpful yes. The example you just posted does not work (the linspace signature is wrong). I'm also confused how you'd achieve a tail with an area that has a consistent height.

I think this illustrates it:
gaps_plot[0:35, -0.05:0.2] + gaps_plot[0:35, -0.1:5]

It still looks like the expected behavior, by slicing the area you're dropping data points that fall outside this range, which is why the red ramp is dropped in the plot on the left. This goes back to the discussion in #181. The question is whether slicing should always slice the underlying data or whether it should just set the extents. Currently it does both where possible. So the correct solution in this case is to set only the extent not slice the underlying data.
If you think this behavior doesn't make sense you can reply to the issue above but this is not a bug.

If extent is the .select method, then this doesn't do what I want:
gaps_plot.select(E_k0=(-0.05, 0.2)))

That looks like the selection isn't being applied because E_k0 doesn't match the actual dimension.
The extents parameter exists on every and allows setting the extents of the plot. When you slice using __getitem__ or select (which are equivalent) two things happen, a) if possible it'll try to slice the underlying data and b) it sets the extents based on your current selection.
If you want to set just the extent you'll have to do so explicitly, e.g. by making a clone, gaps_plot.clone(extents=(None, -0.05, None, 0.2)).

I think we can close this one as the issue is addressed in both #566 and #567

Good plan, closing.


Flask Debug Toolbar Does Not Check For A Compressed Response

If you have an extension like Flask Compress compressing the contents of your responses with gzip before it's sent back to the user, flask debug toolbar will throw an error because it assumes the response is UTF-8 HTML.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/usr/local/lib/python2.7/site-packages/flask_restful/__init__.py", line 270, in error_router
    return original_handler(e)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1479, in full_dispatch_request
    response = self.process_response(response)
  File "/usr/local/lib/python2.7/site-packages/flask/app.py", line 1691, in process_response
    response = handler(response)
  File "/usr/local/lib/python2.7/site-packages/flask_debugtoolbar/__init__.py", line 202, in process_response
    response_html = response.data.decode(response.charset)
  File "/usr/local/Cellar/python/2.7.10/Frameworks/Python.framework/Versions/2.7/lib/python2.7/encodings/utf_8.py", line 16, in decode
    return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x8b in position 1: invalid start byte

Can you try flipping the order that you add the extensions to your app? This should make the compression step happen after applying the toolbar.

Same problem with either Flask-Compress and manual gzip (recipe from http://flask.pocoo.org/snippets/122/). How can I reorder applying it?

Swapping the lines of code where you add the toolbar and add the compress extension should swap the order that they're executed. I think the "after_request" functions are processed in reverse order, so Flask-Compress should be the first extension initialized so that it's the last one called.

Actually, I think we could throw a more useful error message here...

Fixed in #154.


BUG - project access api does not identify library projects

Related Feature
Feature request: issue number
Description
library project type is not included when showing dropdown of Fiori projects in vscode/bas.
Steps to Reproduce
Steps to reproduce the behavior:

Download or clone exmaple project https://github.tools.sap/fiori-tools-qa/se.mi.plm.attachmentservice
Migrate this library to support Fiori tools.
Launch the command palette entry Fiori: Open Application Info.

Expected results
The migrated project is displayed on the dropdown list for selection.
Actual results
The migrated project is not listed
Version/Components/Environment
Add any other context about the problem here
OS:

[x] Mac OS
[x] Windows
[ ] Other

Root Cause Analysis
Problem
UI5 library type projects are not listed by the app search function.
Fix
Updated API to provide an extra argument to optionally include UI5 library type projects in the return list.
Why was it missed
Increased scope of UI5 project types that need to be supported.
How can we avoid this
Unit test added.

@Klaus-Keller
The following code cannot identity the programming language when applying it to the example project attached in this issue, since the example project doesn't have webapp folder. If language type needs to be detected for this type of migrated ui5 library projects, this method also needs to be updated to support this.
https://github.com/SAP/open-ux-tools/blob/6789f09bf97ba0557d42ea39a45115cfbb0f2059/packages/project-access/src/project/info.ts#L19

Hi @slin-sap,
https://github.tools.sap/fiori-tools-qa/se.mi.plm.attachmentservice is not a Fiori application project, it is a project containing a library.
I think there needs to be further discussions on general support of library development using SAP Fiori tools.
FYI @ashleytung @devinea @kranthie-sap @schreckstefan