req.logIn() never sets cookie-session in the latest version of passport(0.6.0)

Hi, I have an express route for email verification that should login the user
/**

Server accepts a token and logs user in.
*/

After successful business flow, In last I want the user to be automatically logs in to the application to access other endpoints but req.logIn never sets cookie-session-based session manager in the latest version of passport(0.6.0).
const xyzSession: IxyzSession = {
id: prospect.$id(),
email: prospect.email,
orgDomain: prospect.org_domain,
};
req.logIn(xyzSession, function (err) {
Logger.info(
'---------------------xyzSession creation initiated---------------------',
prospect.email,
);
            if (err) {
                return next(err);
            }

});
It works fine with passport 0.5.3 version, please let me know is there any work-around that can resolve this issue with passport(0.6.0), as passport(0.5.3)  is vulnerable.

I have the same issue. When I call req.logIn, it saves the session into the session store, but it does not send the cookie to the client in production.

The reason of missing the cookie-session is because:
In version 0.6.0, the req.login function is asynchronous, and the callback function returns after the request is finished.
To fix this issue, we need to wait for req.login promise  and then return the result to the client.


Remove uses of old Fyne 1.4 APIs

The code was using old APIs and the generated code was old fyne as well, updated everywhere.

Sorry to bug you @gocrazygh can this be reviewed? It is blocking your request to be added to https://apps.fyne.io.

Yes, thank you.


Watching an reactive array and adding an item to it reports that the item already existed in the "old" argument

Version
3.0.0-rc.4
Reproduction link
https://github.com/WGroenestein/vue-watch-array
Steps to reproduce

clone the repro
run npm install
run npm run serve
open browser dev tools
hit the "add item" button in the browser
in the dev tools console, see that both current and old contain the new item ("abc") while previous no items existed in the array (old)

current: ["abc"]
old: ["abc"]
What is expected?
old value to be the old "state" (i.e. without the newly inserted item).
What is actually happening?
The new inserted item is already present in the old value.

I was running version 3.0.0-beta.1 before. And from what I recall it was reported correctly there. My use case is that I want to be able to detect items getting added to/removed from the array.
NOTE: I tried to get rc4 to run in CodeSandbox but didn't got it working (relative module could not be resolved)

This is expected because you are modifying the array and it's copied by reference in JavaScript. So oldValue === newValue


[BUG] Custom keybinding for julia-client:run-file not working

When I change the keymap.cson for julia-client:run-file with another specific keybindings it does not work. It does work for the julia-client:run-block though.
Details

Atom version: 1.39.0
Julia version: 1.1.1
OS: Debian testing
Package versions:

Atom.jl: 0.8.8
julia-client: 0.8.10
ink: 0.10.11

The command was migrated into julia-cliant: run-all after Juno v0.8.10.
Please refer to this PR for why, and setting keybindings for it should work.
https://github.com/JunoLab/atom-julia-client/pull/563

Thanks, sorry for the noise.


What's up with Loki?

What should be investigated.
Compliant Kubernetes provides user with the capability of full-text search over their logs. Right now we implement this using OpenDistro. However, OpenDistro is discontinued and it's follow-up project -- OpenSearch -- does not have a Helm Chart yet.
Let us "shop for other options" and investigate if Loki is a viable alternative.
One idea would be to set up a Kubernetes cluster running Loki and directing the logs of all production service clusters to this Loki cluster, so as to get our hands dirty and test Loki beyond day 1.
What artifacts should this produce.

A working Loki cluster
Some logs -- e.g. service clusters -- being directed to it.
A demo highlighting the good, the bad and the ugly of Loki vs OpenDistro.

@cristiklein is this still something that we want to look at?

@llarsson This is low-prio for MS, but valuable knowledge to have overall. Is this something CNOSA can do in their "spare" time? 😁

Since we decided to move forward with OpenSearch, this issue is no longer of interest.


delete practice mode

What feature do you want to get added on the base engine?
delete practice mode
why
doesnt do shit
it just deletes dying
does not affect ur score
overall complicates code
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

not a real enhancement + ratio

🤣

...That's why it's called "Practice Mode", you can play the song freely without dying.


WebSockets support

From a brief look, goproxy does not support proxying websockets (in transparent mode at least)
http://www.websocket.org/echo.html
From chromium
WebSocket connection to 'ws://echo.websocket.org/?encoding=text' failed: Error during WebSocket handshake: Unexpected response code: 400
From goproxy
2014/11/07 09:41:19 [120] INFO: Got request / echo.websocket.org GET http://echo.websocket.org/?encoding=text
2014/11/07 09:41:19 [120] INFO: Sending request GET http://echo.websocket.org/?encoding=text
2014/11/07 09:41:19 [120] INFO: Received response 400 WebSocket Upgrade Failure
2014/11/07 09:41:19 [120] INFO: Copying response to client 400 WebSocket Upgrade Failure [400]
2014/11/07 09:41:19 [120] INFO: Copied 77 bytes to client error=

@mzimmerman summarizing the issue, in order to support websockets in transparent proxy, we need to:

Recognize upgrade requests.
In case of an upgrade request, hijack the connection, and stream all TCP data to the remote host.

I'm not sure whether we need to include that in the core library. I think we simply allow user to hijack connection, and he'll do whatever he wishes in a filter.
How many people are using goproxy in transparent setting?

has there been any update to this issue? lots of the web is turning to websockets, so it would be great if goproxy could do that interception

@k2xl are you using transparent proxy?

No, I'm modifying the requests (I'm assuming that is what you mean by transparent proxy)

@k2xl I meant, do tell your browser to send explicit proxy request. Do you set in the browser's settings goproxy as the HTTP+HTTPS proxy?

Yes

Hello? WebSockets are not working when goproxy.AlwaysMitm'ing requests, at least in my case.


[bazel] add second manufacturer_test_hooks repo for secure domain

Manufacturing partners have both a secure and non-secure domain they develop test code in that does not get pushed upstream, but makes use of upstream code. This adds a second external Bazel repo for the secure domain test hooks, which are stored in a separate system location than the non-secure domain test code.

@sha-ron: I think this is what you are looking for.

@timothytrippel @jdonjdon can you please add this PR to earlgrey_es branch?


telepathy-glib: fix gcc14 build

Things done
ZHF: #352882
https://hydra.nixos.org/build/274108110/nixlog/6
Upstream has a commit that fixes our build issue, already.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353945

x86_64-linux

:white_check_mark: 18 packages built:

libsForQt5.plasma-mobile
libsForQt5.telepathy
lomiri.history-service
lomiri.history-service.dev
lomiri.lomiri
lomiri.lomiri-session
lomiri.telephony-service
polari
telepathy-farstream
telepathy-gabble
telepathy-glib
telepathy-glib.dev
telepathy-haze
telepathy-idle
telepathy-logger
telepathy-mission-control
telepathy-mission-control.dev
telepathy-mission-control.lib

aarch64-linux

:white_check_mark: 18 packages built:

libsForQt5.plasma-mobile
libsForQt5.telepathy
lomiri.history-service
lomiri.history-service.dev
lomiri.lomiri
lomiri.lomiri-session
lomiri.telephony-service
polari
telepathy-farstream
telepathy-gabble
telepathy-glib
telepathy-glib.dev
telepathy-haze
telepathy-idle
telepathy-logger
telepathy-mission-control
telepathy-mission-control.dev
telepathy-mission-control.lib

x86_64-darwin

:x: 2 packages failed to build:

telepathy-gabble
telepathy-haze

:white_check_mark: 9 packages built:

libsForQt5.telepathy
telepathy-farstream
telepathy-glib
telepathy-glib.dev
telepathy-idle
telepathy-logger
telepathy-mission-control
telepathy-mission-control.dev
telepathy-mission-control.lib

aarch64-darwin

:x: 2 packages failed to build:

telepathy-gabble
telepathy-haze

:white_check_mark: 9 packages built:

libsForQt5.telepathy
telepathy-farstream
telepathy-glib
telepathy-glib.dev
telepathy-idle
telepathy-logger
telepathy-mission-control
telepathy-mission-control.dev
telepathy-mission-control.lib


Carthage Action: Missing dependency names for build

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
The Carthage Action does not include the option to build a specific dependency.
› carthage help build
Build the project's dependencies
...
[[]]
	the dependency names to build

Also noticed that the checkout command is missing...any reason for that?

To get a list of all available options use fastlane action carthage 👍


Rename JS-specific concepts

Assignment declaration — an assignment that is treated like a declaration. Previously called [JS] special (assignment|declaration), among other things.
Expando — a value that can be used as a target in assignment declarations. Currently, a class, function or empty object literal.  Functions are allowed in Typescript, too. Previously called a JS container, JS initializer or expando object.
JavaScript → Javascript. This is annoying to type, and looks like 'Java Script' in a camelCase world.

Everything is a pure rename as far as I know. The only test change is the API baselines, which reflect the rename from SymbolFlags.JSContainer to SymbolFlags.Assignment.
If you have better ideas for names, please mention them. @RyanCavanaugh and I agree that the best technical term for expando is "Assignment declaration target", but I think this is unwieldy for use in function names.

Javascript reads weirdly. JavaScript / js are better IMHO.

@gcnew OK, I switched Javascript to JS. I don't think anybody will be confused about its meaning, so it's a good abbreviation.
@andy-ms Right now the only public change is renaming JSContainer → Assignment, so it should be a small note.

@sandersn :+1: from me

@DanielRosenwasser I hope to switch entirely to TS or JS. Any opinions on Ts vs TS?


Support for 3 letter country codes?

Hello,
Thanks for this module. Would you have any interest in also supporting the "ISO 3166-1 alpha 3" codes? (The three letter country codes)?
I find multiple modules that can deal with the 2 letter codes, but currently no modules on npmjs.org that appear to be able to convert the 3 letter codes.

Thanks for the interest in this module and sorry for a such late reply.
The idea for this module was to provide access to the data from http://data.okfn.org/data/core/country-list, and I've decided to keep it that way.
So sorry, no support for other codes.


Fix markdown not rendering link properly

Markdown was not being rendered as HTML in this part:

Hm, this might be a problem with the HTML renderer as I can't tell why the existing mardkown doesn't render properly.

Let me reopen this, because it's very difficult to make it work once it's been done


[WIP] Add support for Ember 2.x

Ember 2.0.0-beta.1 was released yesterday, and with it all deprecations added in the 1.x series were removed from the codebase. Semantic-UI-Ember is built on Ember 1.8.1, which means that people using Ember >1.8.1 and <2.0.0 are receiving a bunch of deprecation warnings, and those using Ember 2.x going forward aren't able to use this addon at all. I'm intending to update this addon to replace deprecated code and be compatible with Ember 2.x, while maintaining compatibility with Ember >=1.8.1.

[x] Update dependencies so we see all the deprecations
[ ] Replace deprecated code
[ ] Test against multiple versions of Ember

Wait. Are there really no tests?

@andrewbranch Yes! No tests. All the components are working as they are used daily in multiple production apps. The plan was to get tests added in before this moved out of the beta/alpha stage.

@nathanpalmer Gotcha. On the one hand, adding comprehensive tests is beyond the scope of this PR. On the other hand, it would be helpful to know that my changes didn't break everything. I guess I'll just see what I have time for. 😛

@andrewbranch, I merged some changes into master which might conflict with some of your changes.

That's ok, sorry I haven't had time to tackle more of this. I was hoping to get this done without making breaking changes, but the modals rely heavily on view-related API, which is deprecated for public use in 2.x.
I like this approach, which makes use of the {{component}} helper, added in Ember 1.10. Are you ok dropping 1.8 and 1.9? I don't really know the guidelines for publishing addons, or your specific priorities for this one.

@andrewbranch i think the ```{{component}}`` feature is only relevant to the users code base. Meaning they could use named components or that helper and still get the same functionality. The internals for launching a modal would still be the same for this library.
Also, I was thinking we are going to convert the remaining individual pieces to remove deprecations one at a time. i.e. dropdown, then modal, then what ever is left. I want to look at the Ember select and pull over code that is still relevant, and remove parts that aren't. Also for the modal, we do want to go all in on components, but we want to research how a few of the other ember modal addons have tackled this to see if there is a best way to do modals that works with Semantic and it's transitions.
Looking at the code between this and master, I think I merged in changes for most of what you had. Maybe we put the rest of the changes on hold while we research a little more into the best way to handle the remaining pieces.

The {{component}} helper would be in this library, leaving user code to look more like
<button {{action "showModal" "my-component-name" contextForModal}}>Show the modal!</button>

({{component}} is what allows the addon code, under the hood, to use the "my-component-name" string and render that component.)
But it sounds like that’s a moot point. That’s fine; I started this because it appeared that no one else was really going to do it:

the same version should continue to work

I may need to continue to develop my fork, depending on how quickly you get up to speed with 2.0, so if that happens and you become interested in merging, I’ll be happy to help reconcile any divergences between our codebases.


Upload job metadata

Description
Upload job metadata for each job triggered to the Clara Jobs Service.
By default, all jobs include Source of how the job was triggered.  E.g. name of the AE Title if DICOM instances received from the SCP.  Or, the Transaction ID of the ACR inference request.
Users may also turn on/off of uploading additional metadata extracted from the specified DICOM tags. DICOM Adapter will only be able to extract each specified DICOM tag once.
Status
Ready
Types of changes

[x] Non-breaking change (fix or new feature that would not break existing functionality).
[ ] Breaking change (fix or new feature that would cause existing functionality to change).
[x] New tests added to cover the changes.
[x] All tests passed locally by running ./src/run-tests-in-docker.sh.
[x] Documentation comments included/updated.
[x] User guide updated.
[ ] NGC publishing metadata updated.
[ ] I have updated the changelog

Codecov Report

Merging #69 (6b6a08a) into release/0.8.1 (9c4416d) will increase coverage by 0.31590%.
The diff coverage is 89.91935%.

@@                   Coverage Diff                   @@
##           release/0.8.1         #69         +/-   ##
=======================================================
+ Coverage       84.59930%   84.91520%   +0.31589%
=======================================================
  Files                145         148          +3
  Lines              11954       12264        +310
  Branches             699         714         +15
=======================================================
+ Hits               10113       10414        +301
- Misses              1567        1572          +5
- Partials             274         278          +4

Impacted Files
Coverage Δ

src/Server/Program.cs
0.00000% <0.00000%> (ø)

src/Server/Services/Export/ScuExportService.cs
71.54472% <ø> (ø)

.../Unit/Services/Export/DicomWebExportServiceTest.cs
100.00000% <ø> (ø)

src/Server/Repositories/ClaraJobsApi.cs
70.09346% <5.00000%> (-15.13382%)
:arrow_down:

src/Common/DicomToolkit.cs
68.75000% <25.00000%> (-6.25000%)
:arrow_down:

src/Configuration/ConfigurationValidator.cs
75.55556% <35.29412%> (-9.37596%)
:arrow_down:

...er/Test/Unit/Processors/AeTitleJobProcessorTest.cs
65.83333% <57.14286%> (ø)

src/Server/Services/Http/InferenceController.cs
82.66667% <66.66667%> (-0.89498%)
:arrow_down:

src/Server/Services/Jobs/JobSubmissionService.cs
89.06250% <92.85714%> (+14.70353%)
:arrow_up:

src/Server/Test/Integration/StoreScpTest.cs
93.83562% <93.83562%> (ø)

... and 23 more

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 9c4416d...6b6a08a. Read the comment docs.


Request: The new Jupyter extension from Microsoft

[ ] Extension name: Jupyter (the new one from Microsoft)
[ ] Extension GitHub or homepage: https://github.com/Microsoft/vscode-jupyter

Introductory article here: https://devblogs.microsoft.com/python/introducing-the-jupyter-extension-for-vs-code/

It is a must as the Python extension depends on it.

I apologize for the delay in releasing a patch for this. I've pushed and fixed and verified that the primary user flows are working again. Please open new issues if secondary problems arise as a result of this new split architecture.


Pretty gemspec =)

This branch this ready to merge, the errors are rubocop, which will be arranged as they go confirming the PRs.


Allow forcing multiline for dict and tuple comprehensions

Also:

Add .gitignore, as spurious files started appearing in git status.
Fix a panic in the test caused by using [:len(".txt")] in the testdata, which creates a zero value for reflect on which Type() cannot be called.

It'd be nice if long comprehensions would be broken automatically, but being able to force this takes away the immediate pain.

Can one of the admins verify this patch?

Sounds good LGTM


What is the tentative timeline for the integration in Solana Unity SDK, without a typescript client?

We're looking forward to exploring leaderboards, achievements, automated rewards distribution, and associating the same with player profiles inside Intrant Inferis game.
Are there any game live that have implemented leaderboards, and automated incentives to players directly after the game and how are the game parameters linked to the on-chain leaderboard?
For example if it is a racing game, then the amount of time taken could be used, if it is a combat game, then it could be the duration as well as number of kills, and accuracy per attack move. We would appreciate any examples :)

Hey @lopeselio , the first version has already been released: https://github.com/magicblock-labs/Solana.Unity.Soar and is included in the latest version of the SDK already.


Deleting a custom window property does not work

Hi,
the #jsdom IRC channel told me to open an issue about this:
I'm unable to (completely) delete a window property that I added myself. The value will remain the same.
Failing test case:
specify("Create, modify and delete custom window property", () => {
  const window = jsdom.jsdom().defaultView;

  assert.equal(window.XXX, undefined);
  assert.equal('XXX' in window, false);

  window.XXX = 1;
  assert.equal(window.XXX, 1);

  window.XXX = 2;
  assert.equal(window.XXX, 2);

  delete window.XXX;
  assert.equal(window.XXX, undefined); // Fails here
  assert.equal('XXX' in window, false);
});

Test Result:
  1) jsdom/miscellaneous Create, modify and delete custom window property:
     AssertionError: expected 2 to equal undefined

Working in Chrome DevTools:

Tested with 9.5.0 and master#7b11f2b

See https://github.com/nodejs/node/issues/6287
Someone please correct me if I'm wrong. To my understanding, this problem happens when ProcessExternalResources in config.features is set to ['script'] (https://github.com/tmpvar/jsdom#external-resources). This effectively runs
window = vm.runInContext("this", window)

which populates the window object with the global object properties (such as eval, Math etc) outside of the vm (these are automatically defined inside the vm), but exposes the linked nodejs bug as a side effect.
If you can live without eval, Math etc defined on the window object outside of the sandbox, you can do
const window = jsdom.jsdom('', { features: { ProcessExternalResources: false }}).defaultView;

Fixed in Node.js, so we should be good to go.

@domenic do you have a reference/link for me? Thanks!

Look back a few comments.


Form components use different theme colors

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

What is the current behavior?
Labels in all form components should use primary color.
But va-select uses 'success' color

was fixed in https://github.com/epicmaxco/vuestic-ui/commit/8bc0ff19e692b706e8b625c1bf6ee8737ae4bf44


Values ending with comma are allowed

Stylelint is allowing values ending with ,. Example below:
.some-class {
  top: 0,;
}

Although SCSS compiles the value correctly as top: 0;, I just can't go to sleep knowing my codebase might some of these by accident 😅
Is there any rule to avoid this from happening?

In case anybody goes through the same issue, I have created a custom rule for this, based on this article.
// no-ending-comma.js

const stylelint = require('stylelint')

const { report, ruleMessages, validateOptions } = stylelint.utils
const ruleName = 'custom-stylelint-plugin/no-ending-comma'
const messages = ruleMessages(ruleName, {
  expected: () => 'Delete ending `,`',
})

module.exports = stylelint.createPlugin(ruleName, function getPlugin(primaryOption, secondaryOptionObject, context) {
  return function lint(postcssRoot, postcssResult) {
    const validOptions = validateOptions(
      postcssResult,
      ruleName,
      {
        // No options for now...
      },
    )

    if (!validOptions) { // If the options are invalid, don't lint
      return
    }

    const isAutoFixing = Boolean(context.fix)
    postcssRoot.walkDecls((decl) => { // Iterate CSS declarations
      const valueLastCharacterIsComma = decl.value.slice(-1) === ','
      if (!valueLastCharacterIsComma) {
        return // Nothing to do with this node - continue
      }

      const correctValue = decl.value.substring(0, decl.value.length - 1)
      if (isAutoFixing) { // We are in “fix” mode
        // Apply the fix. It's not pretty, but that's the way to do it
        if (decl.raws.value) {
          decl.raws.value.raw = correctValue
        } else {
          decl.value = correctValue
        }
      } else { // We are in “report only” mode
        report({
          ruleName,
          result: postcssResult,
          message: messages.expected(decl.value, correctValue), // Build the reported message
          node: decl, // Specify the reported node
          word: ',', // Which exact word caused the error? This positions the error properly
        })
      }
    })
  }
})

module.exports.ruleName = ruleName
module.exports.messages = messages

And then import this as a plugin in the stylelint.config.js:
// stylelint.config.js

module.exports = {
  [...]

  plugins: [
    './no-ending-comma.js',
  ],
  rules: {
    'custom-stylelint-plugin/no-ending-comma': true
  },
}

Additionally to this solution, I ended up creating a plugin for this rule. Feel free to use it:
https://github.com/jagoncalves14/stylelint-no-ending-comma-rule/


Add Darkplex Theme

Dark Green Theme is a dark version of simplex.

Good job! I like this a lot!
I will merge it tomorrow sometime after I've updated the docs referring to there being "12 themes" and leave it with no number so more can be added as a when needed. :)
Thanks!

@fuzzymannerz You're welcome!
Thanks for this awesome project!


Remove redundant comments #1190

@bestony
Thanks to Agora for providing this learning opportunity about open source.

Thank you so much for the PR!
But yesterday we had received the same PR and on a first come, first served basis, we merged his PR.
So I'll be closing this PR, sorry about that.
You can see the other good first issue at https://github.com/netless-io/flat/issues?q=is%3Aissue+is%3Aopen+label%3A"good+first+issue" and contribute.


Add orm

Description
git checkout upstream/main orm
Closes: #XXXX

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Is this just a direct drop-in from the core SDK repo?

Hi @mconcat . What is the status of this PR?


Panic in WS client in subscription mechanism

Have 100ms block machines (4) and our SN nodes (https://github.com/nspcc-dev/neofs-node) are subscribed to some events. Once in a time we have panics and have it more than once, i would say even consistently.
Current Behavior
Create a client, subscribe, use it, panic.
Expected Behavior
Create a client, subscribe, use it, no panic.
Possible Solution
Not sure, i guess some slices (receivers and subscriptions) are not in sync.
Steps to Reproduce
Run SN, go 100ms block time, see panics.
Context
июл 16 14:06:04 node04 neofs-node[642022]: panic: runtime error: invalid memory address or nil pointer dereference
июл 16 14:06:04 node04 neofs-node[642022]: [signal SIGSEGV: segmentation violation code=0x1 addr=0x38 pc=0xa2e02e]
июл 16 14:06:04 node04 neofs-node[642022]: goroutine 114 [running]:
июл 16 14:06:04 node04 neofs-node[642022]: github.com/nspcc-dev/neo-go/pkg/rpcclient.(*WSClient).notifySubscribers(0xc001402540, {0x1a?, {0x1063ca0?, 0xc0000fda40?}})
июл 16 14:06:04 node04 neofs-node[642022]:         github.com/nspcc-dev/neo-go@v0.106.3-0.20240711090334-434f94800ad6/pkg/rpcclient/wsclient.go:696 +0x26e
июл 16 14:06:04 node04 neofs-node[642022]: github.com/nspcc-dev/neo-go/pkg/rpcclient.(*WSClient).wsReader(0xc001402540)
июл 16 14:06:04 node04 neofs-node[642022]:         github.com/nspcc-dev/neo-go@v0.106.3-0.20240711090334-434f94800ad6/pkg/rpcclient/wsclient.go:592 +0x625
июл 16 14:06:04 node04 neofs-node[642022]: created by github.com/nspcc-dev/neo-go/pkg/rpcclient.NewWS in goroutine 1
июл 16 14:06:04 node04 neofs-node[642022]:         github.com/nspcc-dev/neo-go@v0.106.3-0.20240711090334-434f94800ad6/pkg/rpcclient/wsclient.go:493 +0x51b

Regression
Not sure. Have never seen it but also have never tried the client on this short block time.
Your Environment
Neo-go was taken from this branch: https://github.com/nspcc-dev/neo-go/pull/3508. Not a release but looks like some recent master.

Duplicates #3093.


Use shapshot to generate artifacts

Release process:

Branch out from develop to get a release candidate
Artifacts are generated and checked in release branch, but publish is skipped
Merge this to master once passed and back to develop if required
Push a git tag with new version to master by running make release VERSION=v0.0.1
Release will happen on master branch and publish

@agentmilindu Please review the steps

Codecov Report

Merging #140 into release/2.1.6 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release/2.1.6     #140   +/-   ##
==============================================
  Coverage          64.55%   64.55%
==============================================
  Files                  5        5
  Lines                426      426
==============================================
  Hits                 275      275
  Misses               102      102
  Partials              49       49

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 28412b5...febb311. Read the comment docs.


Documentation and Support Website not working

Hi, i was looking for this software a long time but as i was checking the Documentation-Link i could only see the domain. As i wanted to contact support i also could because after clicking on open new ticket it failed.
I wanted to ask if there is documentation on how to integrate Hopebilling with Ispconfig and if this project is still active?

The project is active. No documentation yet.

Been there, done that, failed at it!  guess am not alone. Documentation of basic installation would really come in handy

Please take a look at https://github.com/vityabond/hopebilling/wiki

Thanks a lot for the Wiki

No problem, that's only a basic one for now, I'll improve it when I'm free. :-)


added vrs bus route 118 and edited hafasLineId for nrw-sbahn s6

hope that worked. first proper pull request ^^


chore(pie-monorepo): DSW-000 update element-internals-polyfill to v1.3.11

Describe your changes (can list changeset entries if preferable)
[Changed] - Update element-internals-polyfill to v1.3.11
The renovate PR seemed to stop working, so I did this manually.
Author Checklist (complete before requesting a review)

[x] I have performed a self-review of my code
[x] I have reviewed the PIE Storybook PR preview
[x] If changes will affect consumers of the package, I have created a changeset entry.
[ ] If a changeset file has been created, I have used the /snapit functionality to test my changes in a consuming application

Reviewer checklists (complete before approving)
Reviewer 1 - Fernando

[x] I have reviewed the PIE Storybook PR preview

Reviewer 2

[ ] I have reviewed the PIE Storybook PR preview

/test-aperture


Use Vanilla JS in the New App Template

Summary:
Flow doesn't target OSS, and using it in the new app template leads to a worse experience for anyone using VS Code.
Beyond that, the template package.json doesn't include flow-bin or a script to do typechecking.
Some two year old context here: https://github.com/react-native-community/discussions-and-proposals/issues/296
This removes Flow annotations, flow config, and buck config (I don't immediately see BUCK used either) from the default template.
Changelog:
[General][Changed] - Use Vanilla JS in the App Template
Differential Revision: D40911951

Need to remove end to end app tests which flow check, and see if we can add TS to this.


Prevent sprockets from cache busting assets

I'd like to prevent jekyll-assets (and Sprockets) from messing with my assets' names. Is this possible?
This is because, on my static web site, I do not need anything other than asset copying and the path management functions. My SASS sheets are processed externally to Jekyll, and it is not possible for me to figure out the exact path of an url() asset without messing with the manifest. Also, I employ both jekyll-seo-tag and jekyll-assets to access the same hero image, and this results in two copies of the same file.

This feels like a pretty complicated workflow and an antipattern to what Jekyll Assets was built for.  If you don't want Jekyll Assets to process your assets, why use Jekyll Assets?

I've added Jekyll-Assets to remove myself from handling asset paths. Anyway, I've found the relevant (undocumented) option.

Glad you found it! Keep it a secret! It's undocumented because it was built and paid for by a client specifically for them.  I don't mind if people use it, I just don't "actively support" it!


Feature fs store

This PR addresses #182 to add a local File System store. It works similarly to how the original file system store worked like. Some overall comments on the implementation:

It stores namespaces, models and versions as nested folders inside the provided store folder.
It stores artifacts as JSON files inside the version folders.

There are couple of other minor fixes around, mostly comments and formatting.
Some notes on things that were not explicitly included here:

The file store can not currently used by any artifact other than the NegotiationCard, as other artifacts have not been updated yet to use the new store system.
There are no explicit checks to see if a namespace, model, version or artifact id/name contains characters that are invalid to be file/folder names. (Not sure best way to handle this... it seems that the previous FS store was not handling much of this either).

The implementation was modeled following the RemoteHTTP and Memory stores, but I am not 100% sure I understood all the underlying assumptions in the current design (exact difference between Store and Storage, and SessionStore, etc), so there may be inconsistencies with the expected design structure.

Ok, addressed the 3 comments above. Using shutil now, using sorting for the files/artifacts, and merged and updated store tests to use the fixtures and parametric tests.


chore: verify domain for apple pay

PR-Codex overview
The focus of this PR is to update the apple-developer-merchantid-domain-association file.
Detailed summary

Updated content in the apple-developer-merchantid-domain-association file.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

…tid-domain-association

(view full size)

Commit 73577fb437b25ada030cd57f16bb5dd2f50a8eea (https://thirdweb-ggirt22fc.thirdweb-preview.com).

works now!


Enhance the eth_subscribe implementation to support newHeads

Problem
The relay's eth_subscribe implementation currently does not support newHeads.
Solution
Enhance the eth_subscribe implementation to support the newHeads

[x] Enhance the webSocketServer to handle newHeads.
[x] Enhance the poller.ts to handle the newHeads call.
[x] Add wscat/WebSocket level tests to cover the newHeads call.
[ ] Enhance the webSocketServer to support necessary JSON RPC calls.
[ ] Add Hardhat tests to cover polling for new blocks.
[ ] Add ethers.js tests.
[x] Add metrics tracking polls.

@ebadiere is there anything left over here or can this be closed?


Make it possible to build apps for Android versions still in preview

Fixes #104170
It makes it possible to use non-integer compileSdkVersion in app-level build.gradle file.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@blasten Should I get a test exemption from Hixie or should I write tests for it?


Memory management and deletion of JsonObjects

I'm a bit confused on how to use json class with nested arrays. The following seems to work but I'm not sure its is the correct method. And, is it necessary to delete JsonObject and JsonArray on function complete.
...create json

	JSON json;
	JsonObject obj = json.createObject();
	JsonArray j_array = json.createArray();
    	obj.setInt("name", name);
        for(int i=0; i<num_items; i++) {
        	JsonObject obj1 = json.createObject();
        	obj1.setString("ssid", network.wifi[i].ssid);
        	obj1.setString("pwd", network.wifi[i].pwd);
        	j_array.addObject(obj1);
        }
        obj.setArray("wifi", j_array);

...read json - couldn't  figure out how to do without using m_node
obj = json.parseObject(std::string(str));
    	j_array.m_node = cJSON_GetObjectItem(obj.m_node, "contact");
        for(int i=0; i<j_array.size(); i++) {
        	j_array.getObject(i).getString("name").copy(name, sizeof(name);
        }

For the first part of your code, this will generate a JSON object that, if serialized to a string would look like:
{
   "name": <name>,
   "wifi": [
      {
          "ssid: <ssid>,
          "pwd": <pwd>
      },
      ...
   ]
}

Is that would you expect?
I'm not sure what the notion is with the second part of the code ... what does the string look like that you are parsing?
Assuming it is also:
{
   "name": <name>,
   "wifi": [
      {
          "ssid: <ssid>,
          "pwd": <pwd>
      },
      ...
   ]
}

Then something like:
JsonObj obj = JSON::parseObject(str);
std::string name = obj.getString("name");
...???

Ahh ... there doesn' seem to get a getArray() method ... so we appear to be missing:
JsonObject::getArray()

Is that the question?

Working on the assumption that it was a getArray() function that was needed, I went ahead and added one.  We now have a method on a JsonObject called getArray(<fieldName>) that will return a JsonArray object from object with the given field name.

thanks Neil.
The first part of the code was working fine for me. I was mostly wondering about 'JsonObject obj1 = json.createObject();' inside the loop and concerned that I would create a memory issue at some point.
The 2nd part was also working and yes, I was also thinking that some sort of getArray was needed but thought perhaps I was just mis-understanding something....
I also added:
/**

@brief Convert the JSON object to an unformatted string.
@return A JSON string representation of the object.
*/
std::string JsonArray::toStringUnformatted() {
char *data = cJSON_PrintUnformatted(m_node);
std::string ret(data);
free(data);
return ret;
} // toStringUnformatted

/**

@brief Convert the JSON object to an unformatted string.
@return A JSON string representation of the object.
*/
std::string JsonObject::toStringUnformatted() {
char *data = cJSON_PrintUnformatted(m_node);
std::string ret(data);
free(data);
return ret;
} // toStringUnformatted

When you create a JsonObject or JsonArray, you are responsible for deleting it.  When you parse a string and are given a JsonObject or JsonArray, you are responsible for deleting it.
The one exception is a JsonObject that you then add to another JsonObject through the parent's "setObject" method.   The JsonObject that you added to the parent now belongs to the parent and you must not delete it.  Instead, you eventually delete the parent.
Let me also study and see if there is a better way through an Object's destructor.   Do you have a preference for how you would like to see JsonObjects deleted at the end?

I don't really have a preference. At the moment I am deleting primary JsonObject and any JsonArray created. Tried deleting the child objects but that caused some issues as you pointed out.
The primary thing for me here is the understanding so that I close everything correctly.

Let me take an additional action to see if we can't sensibly close a JSON Object if it goes out of scope without having to perform an explicit delete.  Until then, assume that all top level JsonObject instances need explicitly deleted.

I looked at it again and think that is all as it should be just now.  An explicitly deletion of a JsonObject or array is where I want to leave it.

works for me

Having some issues with json objects/arrays again. Case 1 works fine. Case 2 errors at end when I call  json.deleteObject(obj); I've got a variety of cases similar to either 1 or 2. Case type 2 always fails
I have a function that starts like this.
JSON json;
JsonObject obj = json.createObject();
obj = json.parseObject(std::string(payload));
if(!obj.isValid()) {
json.deleteObject(obj);
return false; }
JsonArray j_array = json.createArray();
(switch)
case 1:
network = obj.getBoolean("network");
... similar statements
case  2:
network = obj.getBoolean("network");
j_array = obj.getArray("name");
for(int i=0; i<j_array.size(); i++) {
type = j_array.getObject(i).getInt("type");
.... similar statements
}
always ends like this
json.deleteArray(j_array);
json.deleteObject(obj);

Just a guess ... but imagine the following JSON:
{
  x: 1,
  y: "abc",
  z: [ "p", "q", "r"]
}

if we create a JSON object from this (obj) and then ask for the array within it:
j_array = obj.getArray("z");

the the array is not ours to delete.  The obj can be deleted, but not the array as that array is owned by the object.   The high level rule is that if we create an object or an array, then we must delete that object or array at the end.  However, in your code, in "case 2", you are coding:
j_array = obj.getArray("name");

which means that you are losing the previous reference to the array that you explicitly created:
JsonArray j_array = json.createArray();

I think I may be bold and say that what you have here is a logic/coding error?  You may be deleting an array that you think you created but there appears to be a code path that, if I am following it correctly, you end up deleting an array that you merely took a reference to from within a containing object.
Please review and comment back.

got it. hopefully new code will resolve - funny that I can't delete an object that I created but I get the concept. Unfortunately can't test because of new bt security issue after espidf update but I'll close anyway

Please realize that there is no rush to close an issue.  If you are not satisfied or haven't had a chance to test, leave it open as long as you need or open it again as desired.

it was a pretty straightforward solution after your explanation so np - got ble functional again and the solution did work. thanks


Sequencing News article - changes to 3rd person and more emphasis on a link

additional edits

Modifying syntax to 3rd person on Covid article

Thanks!

From: Derek Weitzel<EMAIL_ADDRESS>Sent: Friday, November 20, 2020 9:00:17 AM
To: path-cc/path-cc.github.io
Cc: Stathas, Janet; Author
Subject: Re: [path-cc/path-cc.github.io] Sequencing News article - changes to 3rd person and more emphasis on a link (#33)
Merged #33https://github.com/path-cc/path-cc.github.io/pull/33 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/path-cc/path-cc.github.io/pull/33#event-4021292555, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARO5LMU5QMPEM6DRYKPC65DSQ2AADANCNFSM4T4DVFEA.


Update gluster.rb

commented out brick and options

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

@derekmpage I think now that brent is on github, i'll close this PR. so Brent can apply all the changes that he needs and if he needs to come back and change something i won't be the bottleneck.


ocamlmerlin-reason: implement for_completion command

This PR refines merlin integration by implementing the for-completion stub: this inserts a fake identifier in the token stream when completion is triggered.
This enables slightly more cases to typecheck and adjusts the shape of an AST in the middle of an expression to take into account the identifier to be inserted.
See this example, where the most likely completion should be list_to_string:
Before, the type can't be inferred:

After, merlin can first compute the type and then sorts the completion results:

Note: the completion position has to be passed to the lexer, the patch introduces a reference in Reason_toolchain to pass this side information. This seems to be the easiest way to pass extra information. Any suggestion?

cc @jordwalke for review

Guess this is good to merge?


Create the eletric shock enemy type

Create the electric shock.

[ ] Create the electric shock object
[ ] When the bullet hits the shock reset the game

[ ] Create the electric shock object
[ ] When bullet hits the shock reset the game

Completed with the commit: 0a6d2fcab982efdd9192be9ec866b18b7230d1a6


adding missing images from python tutorials

As discussed in Q&A, multiple images are missing. These are already a set that we recovered with the community!
http://answers.opencv.org/question/72388/where-to-get-simplejpg/?answer=72521#post-id-72521
As suggested in the topic, we also renamed simple.jpg to blox.jpg, as in the original image dataset (http://www.ee.surrey.ac.uk/CVSSP/demos/corners/originals.html)
Adapted the tutorial that used the checker board and added an image of that.

Thanks! :+1:

thanks!


Can't inject VSSDK services

When I try to inject any VSSDK service into command (or any other extension part), app fails with these exceptions:
RemoteCommandsLog Error: 0 : Error executing command Extension1.ExtensionEntrypoint:Extension1.Command1 (Sample Remote Command)
InvalidOperationException: Failed to activate service for command execution.
at Microsoft.VisualStudio.Commands.Client.Models.CommandSetModel.<>c__DisplayClass47_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.Threading.ReentrantSemaphore.NotAllowedSemaphore.<>c__DisplayClass2_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.Threading.ReentrantSemaphore.NotAllowedSemaphore.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.VisualStudio.Commands.Client.Models.CommandSetModel.RemoteCommandModel.d__31.MoveNext()
Exception thrown: 'System.TypeLoadException' in mscorlib.dll

Hi @Ilya-Coder1, to help us analyze the issue, can you share the following details:

VisualStudio.Extensibiility package versions.
Visual Studio version you are debugging on.
Verify that you have RequiresInProcessHosting property set in Extension class.

Thanks

Visual Studio version - 17.6.0 Preview 7.0
Extensibility package version - 17.6.2056-preview-3
RequiresInProcessHosting is set as true

I updated VSSDK package to 17.5.33428.388 and now it works!
It's strange that project template includes not compatible VSSDK package.
Thank you for answering!

Hi, what was the VSSDK package version before? While I am glad to see that it is working with new version, I would like us to investigate the original problem still.

Hi @Ilya-Coder1, I tested this scenario with the templates we released and was able to root cause it. We will be releasing an updated extension with the new VSSDK package versions in the templates. Thanks!

Great!


Running make build throws checksum mismatch on<EMAIL_ADDRESS>

Running make build locally gives the following error.
go: verifying<EMAIL_ADDRESS>checksum mismatch
downloaded: h1:y0IMTfclpMdsdIbr6uwmJn5/WZ7vFuObxDMdrylFM3A=
go.sum:     h1:VsBPFP1AI068pPrMxtb/S8Zkgf9xEmTLJjfM+P5UIEo=

Looks like this occurs when using go version 1.11.4+ https://github.com/golang/go/issues/29278
The trivial way to fix this seems to be updating the correct checksum.

PR for the fix https://github.com/cloudera/yunikorn-k8shim/pull/16

Thanks @TarunParimi.
I tried the fix in #16, once applied, then I got the same checksum mismatch error. My go version is go1.12.4 darwin/amd64.
Since go mod is still under-development, does it make sense to update to a newer version of go and see if this issue goes away?

Thanks @yangwwei . I upgraded to 1.12 version of go and this went away. I tried again with version 1.11.12 and the error didn't occur now after I did go clean -modcache.
I probably encountered this in the first place because I had the old checksum in my modcache. We can ignore this issue then as doing go clean -modcache seems to resolve it for me.

@TarunParimi , thank you.
Maybe we can add some doc in our README, when such checksum mismatch error shows up, we can suggest user to run go clean -modcache. Would you like to create a PR for this?

@yangwwei . Sure. Will do it.

@yangwwei Added a note in new PR https://github.com/cloudera/yunikorn-k8shim/pull/18 . Does this look good?

this is resolved by #18


⚠ Jenkins Plugins Artifactory has degraded performance

In 4ccc574, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1200 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 72b0418 after 18 minutes.


Add Pamela's "EuroPi" Workout

REQUEST FOR TESTING & FEEDBACK
This program is developed, but because I'm only one person, and only have one EuroPi in my modular case, there are only so many ways I can test it.
If you have a EuroPi, please try this script out and provide feedback on the Discord server; any feature requests/bug-reports/questions/comments are welcome!
Features
Adds an alternative to the Master Clock script that lets EuroPi simulate ALM's Pamela's Workout modules.
Not every feature of Pam's is available, but it covers a lot of the bases:

configurable main clock BPM
external start/stop input via din
each channel has its own clock modifier, wave shape (with amplitude and width/symmetry controls), quantization, skip probability, and euclidean rhythm generators
CV control over master clock or single channel parameters via ain
ALM-style 2-level menu, accessible by pressing & holding b2

Because EuroPi doesn't have an encoder, both k1 and k2 are used to dive through the menu. k1 controls the main menu item visible, while k2 scrolls through available options when write mode is enabled (press b2 to toggle between write & read modes).  This means once you finish turning k2 to choose an option, k1 is still in place and you won't have changed menu items.  It's slightly less intuitive than ALM's encoder, but it's the best I could do with the hardware available while still keeping a dedicated start/stop button (b1).
Menu
Clock
  |-- BPM
  |     |-- DIN Mode (either Gate or Trigger, see below)
  |     |-- Reset mode (either none or reset-on-stop)
  |
CV1
  |-- Mod.
  |     |-- Wave Shape
  |     |-- Wave Width (PWM/Symmetry)
  |     |-- Phase Offset
  |     |-- Wave Amplitude
  |     |-- Skip Probability
  |     |-- Euclidean Steps
  |     |-- Euclidean Triggers
  |     |-- Euclidean Rotation
  |     |-- Quantization Scale
  |
CV2 to 6
  |-- Same as CV1
  |
AIN
  |-- Gain

Clock DIN modes
The clock can use external on/off signals to start/stop:

Gate mode: the clock always runs while the gate is high, and stops when the gate is low. This is the same as the original Pamela's Workout module from ALM
Trigger mode: the clock state will toggle on the rising edge of a gate or trigger signal

Quantization
Each channel's output can be quantized to one of a variety of scales.  There is (currently) no transposition option (though by combining the output from two quantized channels you can simulate a transposition).
The available scales are:

None
Chromatic
Natural Major
Harmonic Major
Major Triad 1/3/5
Major Triad + 6th
Major Triad + 7th
Natural Minor
Harmonic Minor
Minor Triad 1/3/5
Minor Triad + 6th
Minor Triad + 7th
Major Blues
Minor Blues
Whole Tone
Pentatonic
Dominant 7
AIN (CV-control to select one of the preceding options)

The 1/3/5(+6|7) modes were inspired by the Doepfer A-156.
If there's a scale you enjoy using that I've missed, please submit a request and I can add it!
Development status
At time of writing, these features are implemented but not tested well and may be buggy:

wave amplitude & symmetry settings
actual BPM validation (is 60 bpm actually that fast?)
external CV routing via ain
high BPM (faster than 120) are largely untested; there may be some weird glitches that creep in at speeds this high

Known bugs & limitations

there's minimal thread safety right now. async operations, like timer ticks & button presses, may mangle the inner states of some objects, resulting in crashes
the timer used by the MasterClock class may not be 100% accurate, but it should be close.
quantization is locked to a set of pre-configured, static scales and cannot be transposed. This is unlikely to change, so make sure your oscillators are tuned correctly.
maximum clock modifier range is x16 to /16 (this is dictated somewhat by the PPQN limitation and the fact that the clock can only go so fast)
maximum euclidean pattern length is 64 steps (somewhat arbitrary, but this seemed long enough in my experience)
storage file contents & formatting are somewhat volatile. If you get exceptions when reading the storage file please delete the existing file from your module and start again
swing has not yet been implemented. Sorry for all you jazz fans

I haven't noticed any bugs yet but I do have a small suggestion: I've never used a real Pams, so I have no idea if this is intuitive if not, but without using this for a long time I keep forgetting what the button does, and it seems like on every screen there's space for this, so I'd like to suggest a little text that says 'edit' or 'back' depending on if you're currently editing a value

Please say if you think this is an unnecessary addition, but it has at least personally helped me use the script without ever pressing the wrong thing, but before I added it I quite often would pause when I meant to go back out of edit mode

It looks like there was a bad merge in there somewhere and there are files included in this PR that are related to other changes outside of this PR. Would you mind re-syncing your branch with HEAD and fixing the extra files?

It looks like there was a bad merge in there somewhere and there are files included in this PR that are related to other changes outside of this PR. Would you mind re-syncing your branch with HEAD and fixing the extra files?

I just updated my fork's main and re-applied the changes. Most of the old commit history is gone, but I'm not too worried about losing that; there were some dead ends that can be left in the past.

I haven't noticed any bugs yet but I do have a small suggestion: I've never used a real Pams, so I have no idea if this is intuitive if not, but without using this for a long time I keep forgetting what the button does, and it seems like on every screen there's space for this, so I'd like to suggest a little text that says 'edit' or 'back' depending on if you're currently editing a value
removed for brevity
Please say if you think this is an unnecessary addition, but it has at least personally helped me use the script without ever pressing the wrong thing, but before I added it I quite often would pause when I meant to go back out of edit mode

I'm using inverted text to indicate that you're currently editing, and normal text to indicate a read-only value.  I thought that would be clear enough, but maybe it isn't?
I'm not sure that's necessary? The left button should always be the start/stop button.  The edit/apply/back button is always the one on the right.  A long-press on B2 will always cancel your current edit and flip back to the other menu level.
On the ALM module, the run/stop button is a big yellow button, and the select/edit/back button is the "in" action on a clicky encoder knob.  I tried to replicate that as best I could by using the left button as the run/stop and the right button as the selection.

I'm likely not going to be able to do any more work on this PR in the next few days. I just wanted to thank you again for both the effort that you have put into this script as well as your patience with the review process. I think we are very close to being able to being able to merge this, and I think that it is going to be a great addition to the EuroPi.

I finally got a chance to install this on my EuroPi and jam a little. Very very cool script! I've got a Pam's Pro Workout and the UX feels very natural and familiar. Thanks for creating such an awesome script!
https://youtu.be/EGbMdUNV6cI
I've been thinking about how to handle this PR in regards to #275. I think that we can unblock this PR by moving Euclid, Quantizer and Screensaver into software/firmware/experimental/ alongside the knobs.py library. That way we can merge this PR and it will not break the ability to run this script independently without installing all of contrib.  Then we can follow up with refactoring the existing Quantizer and Euclid contrib scripts to simply import from the experimental folder to reduce code duplication.
This will allow us more time to decide on a long term resolution for #275 and get this PR merged and onto more folk's EuroPis. @mjaskula & @roryjamesallen does that sound like a good path forward here?

I finally got a chance to install this on my EuroPi and jam a little. Very very cool script! I've got a Pam's Pro Workout and the UX feels very natural and familiar. Thanks for creating such an awesome script!

Glad you like it! I've got Pam's New Workout and have been using it for reference, From what I understand the differences between New and Pro are pretty minor, besides the screen.

https://youtu.be/EGbMdUNV6cI

Nice! I've been meaning to film a video myself showing off the Pam's script, I just don't have a good overhead camera mounting system.

I've been thinking about how to handle this PR in regards to #275. I think that we can unblock this PR by moving Euclid, Quantizer and Screensaver into software/firmware/experimental/ alongside the knobs.py library. That way we can merge this PR and it will not break the ability to run this script independently without installing all of contrib. Then we can follow up with refactoring the existing Quantizer and Euclid contrib scripts to simply import from the experimental folder to reduce code duplication.

So should I move the common/shared classes/functions into experimental as part of this PR? or do we want to get Pam's merged and then go on a major refactoring spree and move things around after that?

Nice work, I'm happy with these changes. Let's make sure we get consensus from others before we merge this.

With regards to the new lint formatter errors, I think some of the formatting you have provides more value than the suggested lint fixes, so I would suggest disabling the linter for those blocks where the unlinty format provides value with # fmt: on/off. See: https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#code-style

The linter is failing on the new experimental files. I've got some errands to run, but I'll try to fix those later. The code itself is functional, the linter is apparently just a lot pickier than I am about formatting.

It looks like the biggest complaint is with the formatting of the common scales. I'd formatted it specifically with all the True and False values lined up in columns to make it easier to read what notes are in each scale. But the linter wants me to format it to be evenly-spaced lists, which actually makes it less readable IMO.
Is there a nice way to just tell the linter to ignore that file so I can make it readable?

It looks like the biggest complaint is with the formatting of the common scales. I'd formatted it specifically with all the True and False values lined up in columns to make it easier to read what notes are in each scale. But the linter wants me to format it to be evenly-spaced lists, which actually makes it less readable IMO.
Is there a nice way to just tell the linter to ignore that file so I can make it readable?

Yep, see my comment: https://github.com/Allen-Synthesis/EuroPi/pull/248#issuecomment-1566222894

Caught a couple of bugs that only kicked in when the screensaver did.
Also recorded quick video showing off the module. Sorry the volume is a little low. I should have turned the gain up in OBS.
https://youtu.be/vOlPQLiB7Nc


How to get snippets in comments?

Hi!
After installing the extension, we can get your snippets in codes, but failed in comment blocks.
// okay to get the snippets in codes
apiN => apiName

// failed to get the snippets in comment blocks
/**
* apiN
*/

Should we set some config or install some other extensions?
Thanks!

Hi @ColinLeo,
you have to type:
/**
 * apiN
 */

And then press Ctrl+Space. That's it )

hi ColinLeo,
You can try this:
a) Snippets show in IntelliSense (Ctrl+Space) mixed with other suggestions as well as in a dedicated snippet picker (Insert Snippet in the Command Palette).
b) Change user or workspace settings with this configuration for automatic comment block's suggestions:
"editor.quickSuggestions": {
"other": true,
"comments": true,
"strings": false
}

hi ColinLeo,
You can try this:
a) Snippets show in IntelliSense (Ctrl+Space) mixed with other suggestions as well as in a dedicated snippet picker (Insert Snippet in the Command Palette).
b) Change user or workspace settings with this configuration for automatic comment block's suggestions:
"editor.quickSuggestions": {
"other": true,
"comments": true,
"strings": false
}

Thanks for your reply, I make it as b).


Java SDK-V2 adding Override Credentials Provider example code snippet…

…, using PutItem
Issue #, if available: N/A - code sample.
Description of changes:
This change was pulled (and modified) from this repository: https://github.com/mslshao/Override-IAM-Roles-per-DynamoDB-Request
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The code looks good, we only have one small ask about the position. You are requesting this code to be a top level example, however all the rest of the Java code is under this folder can you please move your content there, and also don't forget to modify the README.MD and this other README.MD if required.

Sure, I'll make the change and edit/update this PR. Thanks!

I definitely am not used to GitHub's way of doing forks/merges/rebasing, so... I apologize that this is in a double-commit.
If there's a clean way to clean up the commit history, I'd appreciate it. Feel free to squash/rebase these commits when merging.


Add support for SectionList

Implementation is copied almost exactly from that of FlatList
Demo Screencast of added SectionListExample

-
Screencast

SectionListExample

@vishalnarkhede could you take a look, please?


Dependency of other services

My app uses SQL Server Express (or MSSQL Server)
It runs fine but when I restart windows the MSSQL service loads late. So my service fails to start without MSSQL.
How can I add a service dependency with this library?

I suggest to setup the service to start later in that case. Or solve it programatically, check if sql is ready and if not try again later.


Update readmes to match common design (Group Layers & Hydrography categories)

Thanks @Rachael-E , could you have a second look at the updated sections please?


Updates/fixes to decomposition feature branch

This change merges in main, and also fixes two issues with decompositions:

The Reset capability fix was lost in the previous merge from master.
RFrac should allow negative powers of 2.

Both of these bugs were found during e2e build failure investigation.

To view just the fixes without the merge from main, see the last two commits:
Fix Reset decomposition capability requirement
and
Fix RFrac decomposition to allow negative exponentiation


Fix typos and remove empty lines

Fixing some typos, wrong capitalization in names and removing empty lines from code examples and such.
This was a quick scan from what vim highlighted to me was wrong, a more in-depth read should be done later.

Wow great catches @tombruijn. One comment on wrong fix but otherwise good job and 👍 from me!


Make Subs and Derivative classes closer to SymPy

Relevant: #119

@isuruf I am not confident that this is how it is supposed to be implemented. Please review and suggest revisions.

Ping @isuruf.

Ping @isuruf.

Replaced by #145


Get-PowerBIWorkspace : Attempted to access an element as a type incompatible with the array.

When I run the example code from https://docs.microsoft.com/en-us/powershell/power-bi/overview?view=powerbi-ps
Login-PowerBIServiceAccount
Get-PowerBIWorkspace

I get the error
Get-PowerBIWorkspace : Attempted to access an element as a type incompatible with the array.
At line:2 char:1
+ Get-PowerBIWorkspace
+ ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (Microsoft.Power...owerBIWorkspace:GetPowerBIWorkspace) [Get-PowerBIWorkspace], ArrayTypeMismatchException
    + FullyQualifiedErrorId : Attempted to access an element as a type incompatible with the array.,Microsoft.PowerBI.Commands.Workspaces.GetPowerBIWorkspace

There was a similar issue (#60) that was marked as closed with a note to update .NET but I am running the current version and still get this erro

I also have this problem.  Tried workaround (-noprofile) from #60 and it does not work.  Also tried downgrading to 1.0.295 (as per #60) and also did not work.
I cannot use Powershell Core as I need to utilize OLEDB connections.

I'm in the same boat. I've tried to uninstall, reinstall, get previous versions,etc. but no luck. In Powershell core, the same is working perfectly
Login-PowerBIServiceAccount Get-PowerBIWorkspace
This is the output I got after Resolve-PowerBIError -Last:
 Message        : Attempted to access an element as a type incompatible with the array. StackTrace     :    at System.Collections.Generic.List1.Add(T item)
at Microsoft.PowerBI.Api.V2.PowerBIClient.Initialize()
at Microsoft.PowerBI.Api.V2.PowerBIClient..ctor(Uri baseUri, ServiceClientCredentials credentials, HttpClientHandler rootHandler, DelegatingHandler[] handlers)
at Microsoft.PowerBI.Common.Api.PowerBIApiClient.CreateClient(IAuthenticationFactory authenticator, IPowerBIProfile profile, IPowerBILogger logger, IPowerBISettings settings,
HttpClientHandler httpClientHandler)
at Microsoft.PowerBI.Common.Client.PowerBIClientFactory.CreateClient(IAuthenticationFactory authenticator, IPowerBIProfile profile, IPowerBILogger logger, IPowerBISettings settings)
at Microsoft.PowerBI.Common.Client.PowerBIClientCmdlet.CreateClient()
at Microsoft.PowerBI.Commands.Workspaces.GetPowerBIWorkspace.GetWorkspaces()
at Microsoft.PowerBI.Commands.Workspaces.GetPowerBIWorkspace.ExecuteCmdlet()
at Microsoft.PowerBI.Commands.Common.PowerBICmdlet.ProcessRecord()
Exception      : System.ArrayTypeMismatchException
InvocationInfo : {Get-PowerBIWorkspace}
Line           :  Get-PowerBIWorkspace
Position       : At line:1 char:2
+  Get-PowerBIWorkspace
+  ~~~~~~~~~~~~~~~~~~~~
HistoryId      : 11`

For the people affected by this, can you try installing >v1.0.613 of the cmdlets? If you run PowerShell (not Core) using -NoProfile, does this issue still occur?

I just updated the cmdlets and now the following script works. Thanka
Login-PowerBIServiceAccount
Get-PowerBIWorkspace

I just updated and I get the same error message:
Get-PowerBIWorkspace : Attempted to access an element as a type incompatible with the array.
At line:1 char:15
+ $workspaces = Get-PowerBIWorkspace
+               ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (Microsoft.Power...owerBIWorkspace:GetPowerBIWorkspace) [Get-PowerBIWorkspac
   e], ArrayTypeMismatchException
    + FullyQualifiedErrorId : Attempted to access an element as a type incompatible with the array.,Microsoft.PowerBI.
   Commands.Workspaces.GetPowerBIWorkspace

Version of the cmdlets:
PS C:\WINDOWS\system32> Get-Module -Name MicrosoftPowerBIMgmt.Profile
ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Binary     1.0.613    MicrosoftPowerBIMgmt.Profile        {Connect-PowerBIServiceAccount, Disconnect-PowerBIServiceAccount, Get-PowerBIAccessToken, Invoke-PowerBIRest...

Newtonsoft Version:
PS C:\WINDOWS\system32> [Newtonsoft.Json.JsonConvert].Module
MDStreamVersion    : 131072
FullyQualifiedName : C:\Program Files\WindowsPowerShell\Modules\MicrosoftPowerBIMgmt.Profile\1.0.613\lib\netstandard2.0\Newtonsoft.Json.dll
ModuleVersionId    : 12891e82-5fce-44a5-81e5-bbbb579c1d49
MetadataToken      : 1
ScopeName          : Newtonsoft.Json.dll
Name               : Newtonsoft.Json.dll
Assembly           : Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed
CustomAttributes   : {}
ModuleHandle       : System.ModuleHandle

@simonwong7 Can you please try opening PowerShell using -NoProfile? The version of Newtonsoft.Json loaded into your PowerShell session is below the minimum version (12.0.2) required by the module.

The solution of launching powershell.exe noprofile worked for me

I still have this occuring with the noprofile. Any ideas?

I use the -NoProfile approach. Now it works :)
Using MicrosoftPowerBIMgmt V1.0.840

Tried to load the powershell with -NoProfile but don't worked for me.

I had to uninstall e install again the MicrosoftPowerBIMgmt

facing the same issue in powershell ISE. Please help

Hey @parag1231 - I had the same issue using the current (as of now) version of MicrosoftPowerBIMgmt i.e. 1.0.867, however following the -noprofile has worked for me in PowerShell ISE too. Try the below steps, hopefully should work for you as well,

Click on Start
Run
PowerShell_ISE.exe -noprofile

Still getting the same error while executing : New-PowerBIreport cmdlet

I had tried all the option mentioned above but none worked for me. However the only way that solved my problem is that installing newtonsoft.json module. That is run the below code with administrator privilege-
Install-Module -Name newtonsoft.json -AllowClobber

Still getting the same error while executing : New-PowerBIreport cmdlet

I had tried all the option mentioned above but none worked for me. However the only way that solved my problem is that installing newtonsoft.json module. That is run the below code with administrator privilege-
Install-Module -Name newtonsoft.json -AllowClobber

Best answer came from this thread comment: https://github.com/microsoft/powerbi-powershell/issues/60#issuecomment-409019796. It's apears to be a Newtonsoft.Json conflict. It worked for me to run PowerShell_ISE.exe -noprofile.
Locate which Newtonsoft.Json assembly is being loaded by running:
[Newtonsoft.Json.JsonConvert].Module

With Newtonsoft.Json.dll loaded, the Get-PowerBIWorkspace cmdlet works for me. With Newtonsoft.Json.12.0.3.dll loaded, Get-PowerBIWorkspace cmdlets gives the error.
[Newtonsoft.Json.JsonConvert].Module
MicrosoftPowerBIMgmt modules cmdlets WORKS:

MDStreamVersion    : 131072
FullyQualifiedName : C:\Users\Brian.Lofton\Documents\WindowsPowerShell\Modules\MicrosoftPowerBIMgmt.Profile\1.0.974\lib\netstandard2.0\Newtonsoft.Json.dll
ModuleVersionId    : 12891e82-5fce-44a5-81e5-bbbb579c1d49
MetadataToken      : 1
ScopeName          : Newtonsoft.Json.dll
Name               : Newtonsoft.Json.dll
Assembly           : Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed
CustomAttributes   : {}
ModuleHandle       : System.ModuleHandle

MicrosoftPowerBIMgmt modules cmdlets ERRORS:

MDStreamVersion    : 131072
FullyQualifiedName : C:\Users\Brian.Lofton\Documents\WindowsPowerShell\Modules\Az.Accounts\2.2.8\PreloadAssemblies\Newtonsoft.Json.12.0.3.dll
ModuleVersionId    : 2676a2da-6edc-420e-890e-d28aa4572ee5
MetadataToken      : 1
ScopeName          : Newtonsoft.Json.dll
Name               : Newtonsoft.Json.12.0.3.dll
Assembly           : Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed
CustomAttributes   : {}
ModuleHandle       : System.ModuleHandle

This worked with Powershell Core once I figured out how to install AzureAD for it.
$ Import-Module AzureAD -UseWindowsPowerShell

Using -noprofile worked, just remember to pass the appropriate credential when connecting, otherwise it will authenticate with first.last credential which may not have sufficient permissions.

2022 with the same bug using 1.2.1077... -noprofile doesn't work.
The following allows me to login (again, using -noprofile and not using -noprofile to launch powershell) but the next command fails.
Windows 10
How has this issue not been resolved in 4 years, and closed? Should I open a new bug on this?
PS C:\WINDOWS\system32> Login-PowerBI

Environment : Public
TenantId    : 945c199a-83a2-4e80-9f8c-5a91be5752dd
UserName<EMAIL_ADDRESS>

PS C:\WINDOWS\system32> Get-PowerBIActivityEvent -StartDateTime '2022-03-24' -EndDateTime '2022-03-25'
Get-PowerBIActivityEvent : Attempted to access an element as a type incompatible with the array.
At line:1 char:1
+ Get-PowerBIActivityEvent -StartDateTime '2022-03-24' -EndDateTime '20 ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (Microsoft.Power...BIActivityEvent:GetPowerBIActivityEvent) [Get-PowerBIActi
   vityEvent], ArrayTypeMismatchException
    + FullyQualifiedErrorId : Attempted to access an element as a type incompatible with the array.,Microsoft.PowerBI.
   Commands.Admin.GetPowerBIActivityEvent

Is this issue resolved?
I'm getting the same error
> Get-PowerBIWorkspace
Get-PowerBIWorkspace : Attempted to access an element as a type incompatible with the array.
At line:1 char:1
+ Get-PowerBIWorkspace
+ ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (Microsoft.Power...owerBIWorkspace:GetPowerBIWorkspace) [Get-PowerBIWorkspac
   e], ArrayTypeMismatchException
    + FullyQualifiedErrorId : Attempted to access an element as a type incompatible with the array.,Microsoft.PowerBI.
   Commands.Workspaces.GetPowerBIWorkspace

This is my setup:

> [Newtonsoft.Json.JsonConvert].Module

MDStreamVersion    : 131072
FullyQualifiedName : C:\Windows\Microsoft.Net\assembly\GAC_MSIL\Newtonsoft.Json\v4.0_<IP_ADDRESS>__30ad4fe6b2a6aeed\Newtons
                     oft.Json.dll
ModuleVersionId    : 12891e82-5fce-44a5-81e5-bbbb579c1d49
MetadataToken      : 1
ScopeName          : Newtonsoft.Json.dll
Name               : Newtonsoft.Json.dll
Assembly           : Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed
CustomAttributes   : {}
ModuleHandle       : System.ModuleHandle

> Get-Module -Name MicrosoftPowerBIMgmt.Profile

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Binary     1.2.1104   MicrosoftPowerBIMgmt.Profile        {Connect-PowerBIServiceAccount, Disconnect-PowerBIServiceA...

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.17763.3770
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17763.3770
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Having this issue as well.


Simple workflow test cleanup

Pull Request Test Coverage Report for Build 4171

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 69.39%

Totals

Change from base Build 4169:
0.01%

Covered Lines:
12382

Relevant Lines:
17844

💛  - Coveralls


Plugin fails when secret values contain some special characters

I've been working on a partial fix for this in a fork but wanted to document it here.  As the plugin uses the yaml formatting option to help it pull secrets from Vault, this means that special characters can cause all kinds of issues, in particular pipes | and newlines on things like PEM files.
While there's built in functionality to handle PEM files for SSH-related purposes by adding them to the ssh-agent, this doesn't cover all use cases, and the plugin should be able to handle these situations.
To tackle the issue of pipes and some bash-specific patterns, my upstream fix escapes values on the fly and then wraps them in double quotes.  This has worked fine for ourselves, but because certificates maintain newlines and begin |-  they cause issues.
We should explore how to handle these occurrences - maybe moving to json output format for a start and going from there so that secrets come pre-wrapped in quotes and newlines are escaped?  I'm on the Buildkite Community Slack as @Jonny as well if that's easier 🙂

This should be resolved in #19 :)


null slug

I'm using "laravel/framework": "5.6.*" and "cviebrock/eloquent-sluggable": "^4.5".
Here is the code from a Model and a Controller that doesn't generate the slug.
News Model
use Cviebrock\EloquentSluggable\Sluggable;
class News extends Model
{
    use Sluggable;

    protected $fillable = [
        'title',
        'slug',
    ];

    public function sluggable()
    {
        return [
            'slug' => [
                'source' => 'long_slug'
            ]
        ];
    }

    public function getLongSlugAttribute()
    {
        return $this->title . ' ' . $this->id;
    }
...
}

In News Controller
public function store(NewRequest $request)
    {
        $news = News::create($request->all());
        $news->save();
        return redirect()->action('Admin\NewsController@editImages', [$news->id]);
    }

This started to happend when I upgraded to Laravel 5.6. I already ran composer update lots of times and reinstall cviebrock/eloquent-sluggable.
Am I missing something? Can somebody help me? Thanks.

Hey, I realized that the slug wasn't working only in some models, so I compared the models and I found that in those which slug doesn't work, I was using them to generate index data with laravel scout / algolia.
I mean, if I delete the News model from this algolia aggregator, the slug start to work in the model.
namespace App;
use Algolia\ScoutExtended\Searchable\Aggregator;

class Search extends Aggregator
{
    protected $models = [
        News::class,
        Posts::class,
    ];
}

Do you know if there's some kind of incompatibility with both packages?

I'm not familiar with that package at all, sorry.


Add opts.allowSpaceInLinks to avoid patching tokenizer for all users

The existing plugin globally patches the marked link tokenizer to allow spaces in unbracketed links.
It would be preferable if this patch were optional (for backward compatibility) and encourage new users to adopt [](<link with spaces>) in pointy brackets.
todo:

[x] add opt.allowSpaceInLinks
[ ] fix existing tests without opts.allowSpaceInLinks
[ ] add new tests with opts.allowSpaceInLinks

This PR is blocked pending resolution of markedjs/marked#1639


Add Issue Template and PR Template

Hey! Just checked out the project. And It's really cool. Would Love to work on it and contribute as well
Just noticed that the project is missing Issue Template and PR Templates. and I would like to add them into the project. Such that New folx can easily create Issues and PR's as well, without much of head banging...

Hmm never made them before but I can try to make them

Well, I was wondering If I can work on it.

Well, I was wondering If I can work on it.

Hmmmmm, sure I can let you do it!

Completed will be closed


Node.Js Function with Lodash and Node-xlsx will stop responding due to a Stack Overflow in EdgeJs

Investigative information
Please provide the following:

Timestamp: 29th of November 2017 11:0 AM
Function App version (1.0 or 2.0-beta):
Function App name: fintestNodexlsx
Function name(s) (as appropriate):
Invocation ID:a1ecdd68-216a-461e-82ca-941e448057ef
Region:  Central US

Repro steps
Provide the steps required to reproduce the problem:

Create a new Function App on the Consumption Plan.
Run NPM install for Lodash and Node-xlsx.
Create a new HTTPTrigger for JS and use the Code below.

Expected behavior
Provide a description of the expected behavior.
The code should run and return.
Actual behavior
Provide a description of the actual behavior observed.
When using a lower value for the Fill Array e.g 500  the Function returns as expected.
If you use a higher version then the Function fails and returns a 502 Gateway error for the invocation.
"502 - Web server received an invalid response while acting as a gateway or proxy server"
Known workarounds
Provide a description of any known workarounds.
This works on the new 2.0 runtime with the Node Language worker.
Related information
Provide any related information
The issue is a Stack Overflow in EdgeJS and the stack is below from a dump.
0:034> kL
ChildEBP RetAddr
WARNING: Frame IP not in any known module. Following frames may be wrong.
00 0b69dc94 37d9b679 0x2361ec33
01 0b69dcb8 37da2046 0x37d9b679
02 0b69dcd8 37d9a9a3 0x37da2046
03 0b69dd38 37d861d1 0x37d9a9a3
04 0b69dd68 37d8553f 0x37d861d1
05 0b69dd90 37d81fc3 0x37d8553f
06 0b69ddc8 37d7f8f4 0x37d81fc3
07 0b69dde8 37d7e0a1 0x37d7f8f4
08 0b69de18 2361f31e 0x37d7e0a1
09 0b69de38 2361b2a3 0x2361f31e
0a 0b69de64 6086dd43 0x2361b2a3
0b (Inline) -------- node!v8::internal::Object::IsString+0x11
0c 0b69deb0 6086df32 node!v8::internal::PrintDeserializedCodeInfo+0x63
0d (Inline) -------- node!v8::internal::Internals::HasHeapObjectTag+0x1
0e (Inline) -------- node!v8::internal::Object::IsHeapObject+0x1
0f (Inline) -------- node!v8::internal::Object::IsJSGlobalObject+0x1
10 0b69dee4 608b2e4a node!v8::internal::Execution::Call+0x12
11 (Inline) -------- node!v8::internal::tracing::AddTraceEvent+0xc
12 0b69df58 608c3ef9 node!v8::internal::ExternalCallbackScope::ExternalCallbackScope+0xca
13 (Inline) -------- node!v8::internal::HandleScope::DeleteExtensions+0xa
14 (Inline) -------- node!v8::internal::HandleScope::CloseScope+0x30
15 (Inline) -------- node!v8::HandleScope::{dtor}+0x30
16 (Inline) -------- node!v8::EscapableHandleScope::{dtor}+0x30
17 0b69dfc4 0aeff6dc node!v8::Function::Call+0x279
18 0b69e0b4 0aeff2c6 edge_nativeclr!NodejsFuncInvokeContext.CallFuncOnV8Thread()+0x394
19 0b69e0c4 0b7706ba edge_nativeclr!.clrActionContext.ActionCallback(Void*)+0x2e
1a 0b69e0ec 0aeff1d3 CLRStub[StubLinkStub]@f64c1050b7706ba
1b 0b69e144 0b7705f5 edge_nativeclr!.continueOnV8Thread(uv_async_s*, Int32)+0xa3
1c 0b69e16c 60846399 CLRStub[StubLinkStub]@f64c1050b7705f5
1d 0b69e17c 60847f5b node!uv_async_init+0x89
1e 0b69e198 608488a1 node!uv_process_reqs+0x7b
1f (Inline) -------- node!uv__hrtime+0x2c
20 (Inline) -------- node!uv_update_time+0x2c
21 0b69e9d4 74185dd1 node!uv_poll_ex+0x151
22 0b69f7f8 74e286e3 clr!Thread::intermediateThreadProc+0x55
23 0b69f804 77799e99 kernel32!BaseThreadInitThunk+0xe
24 0b69f848 77799e6c ntdll!__RtlUserThreadStart+0x72
25 0b69f860 00000000 ntdll!_RtlUserThreadStart+0x1b

Programming language used
Links to source
Bindings used

Source
const _ = require('lodash');
const xlsx = require('node-xlsx');

module.exports = function (context, req) {
        context.log('JavaScript HTTP trigger function processed a request.');
        // The Array volume affects the Stack Overlow. With 500 it doesn't happen and with 1000 it does.
        try{
            xlsx.build([{ name:'test', data: _.fill(new Array(3000), ['1', '1']) }]);
        }catch(err){
            context.log(err);
        }

        context.res = {
            // status: 200, /* Defaults to 200 */
            body: "hello world"
        };
        context.done();

};e

Its unlikely that we're going to get a fix for this in edge.js as it seems to be a corner case and there is not much active development on the project. Makes sense that this doesn't repro in v2 since we're not using edge.js anymore. Closing this as external.


include jQuery-File-Upload in a div in my page.

I'm trying to get jQuery-File-Upload a div  is part of my page, I use div to include from php, but the thing I care about all the page and I altered my css.
how I can include it as like a module in my web page?

I resolve using a iframe , but i dont know that is the best way


Add files via upload

Rohm driver code pressure and mag

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

have a error


How to set headers correctly?

In the part "Create a console app to send tagged notifications" section 4 there is this code:
    // Apple requires the apns-push-type header for all requests
    var headers = new Dictionary<string, string> {{"apns-push-type", "alert"}};

The variable headers is not assign to anything? Could you post proper example how to set this header?
I am also wondering how to set android notification priority to high? Especially how to do it with method SendNotificationAsync or SendTemplateNotificationAsync?

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

ID: f968a798-07e9-2027-a9fb-efebd41b809a
Version Independent ID: 35cd7392-2dc5-70f9-30e8-12d6b8198355
Content: Send notifications to specific devices (Universal Windows Platform)
Content Source: articles/notification-hubs/notification-hubs-windows-notification-dotnet-push-xplat-segmented-wns.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@JanNepras Thanks for the feedback! We are taking a look into this and will get back to you soon.

@SnehaAgrawal-MSFT What is the status? We need to release this asap and this is blocking us?

@JanNepras Apologize to keep you waiting! Thanks again for the valuable feedback!
Basically, headers get associated with the registration, if applicable, not with the notification. For instance, one can see here where APNS headers get associated with an AppleTemplateRegistration, whereas the Android equivalent has no headers associated with it.
I will reach out to content author @sethmanheim  for further review and taking a closer look at this tutorial.

@JanNepras Just checking if you have chance to see previous message.

@SnehaAgrawal-MSFT Thank you.

@JanNepras We will now close this out but if you feel you need more information please just let us know.

@SnehaAgrawal-MSFT I don't understand. We didn't solve anything and close the issue. What is the point of this issue anyway?


Evolving thinking on retro funding

Run a timeboxed braindump on this doc: https://www.notion.so/Retro-Funding-nu-thoughts-c72b1fc60373489b96008d4bb4aa2f43?pvs=4

Done. Also added this

https://www.loom.com/share/7c1cd5d2f196419b8e3b6573888d37ab?sid=458341f8-05ed-4829-bb4e-35a0709fb6bd
Exploring Retro Funding and Builder Incentives in Optimism's Ecosystem
In recent discussions around decentralized governance and builder incentives within the Optimism ecosystem, retro funding emerges as a key mechanism driving innovation. For those unfamiliar, retro funding is a novel approach designed to attract top builders to work on solving critical problems within the Optimism network. The idea is simple: reward past work that contributes to the growth and success of the ecosystem. But in practice, retro funding is multifaceted and continually evolving, presenting unique challenges and opportunities for builders and governance participants alike.
The Purpose of Retro Funding
At its core, retro funding is meant to incentivize the best developers and builders to contribute to Optimism’s long-term vision. This vision, articulated as the "optimistic vision," calls developers to create the infrastructure for a new internet—one that is decentralized, scalable, and inclusive.
For builders, the retro funding mechanism serves as a growth engine. It is designed to attract talent to the hardest problems on the network by offering a compelling reason to contribute: the potential for meaningful financial rewards after the fact. But for retro funding to function effectively, the message must resonate with builders, and Optimism must maintain a healthy funnel that captures and retains top talent.
The Builder Funnel: Awareness and Self-Selection
Understanding the builder funnel is crucial when evaluating the success of retro funding. The first challenge is awareness: Do builders know about retro funding and its potential? While awareness isn’t a major issue at present, it could become more pressing as competition for attention increases during bull markets. As the ecosystem grows, the challenge will be maintaining visibility and ensuring builders are informed about how they can contribute and benefit.
The next hurdle in the funnel is self-selection. Builders must determine whether their contributions will be eligible and whether the retro funding mechanism aligns with their goals. While some may opt-out after reviewing eligibility criteria, others might apply regardless, hoping for the best. Over time, increased self-selection may actually be beneficial, helping to filter out projects that aren’t well-aligned with the goals of retro funding, thereby improving the quality of submissions.
Fair Evaluation and Respect Among Peers
Once builders are aware of retro funding and have submitted their projects, the next critical question arises: Will their contributions be evaluated fairly? This is a significant concern for many builders. Ensuring that badge holders (those responsible for reviewing applications) assess projects based on relevant criteria is vital for maintaining trust in the system. If builders feel their work is not being considered in the best light, it can create frustration and disillusionment.
Equally important is the mix of projects that make it into a round. Builders want to feel that they are competing with other high-quality projects that respect the spirit of the competition. Over time, rounds with clearer eligibility criteria and better-scoped categories will foster a healthier competitive environment. In contrast, rounds that include “scammy” projects or those focused on farming rewards undermine trust in the process.
Financial Incentives and Business Models
Retro funding operates in a space where the financial incentives for builders can vary greatly. For some, retro funding might represent a core part of their business model, while for others, it may be a “nice-to-have” source of additional funding. This distinction is important because if retro funding becomes dominated by teams that are solely reliant on it, the diversity of projects may suffer.
The amount of funding a project receives relative to its perceived need can also impact how the builder community perceives retro funding’s effectiveness. The sensitivity of this issue is evident—especially for projects that depend heavily on retro funding to sustain operations. Understanding how projects utilize their rewards, whether they reinvest into their ecosystems or liquidate their tokens for immediate survival, offers valuable insight into the health of the ecosystem.
Governance and Badge Holders
Beyond its role in funding innovation, retro funding is also a governance experiment. The mechanism hinges on a simple axiom: impact should equal profit. This idea, inspired by Optimism’s vision, aims to create a governance system where the people—referred to as citizens or badge holders—are united by a common goal: rewarding impactful contributions.
Badge holders play a crucial role in maintaining the integrity of the system. However, their tasks must align with their strengths. High-value decisions, like determining project eligibility or evaluating the potential impact of a contribution, should take priority over lower-value tasks such as spam detection. If badge holders are consistently tasked with menial work, they may lose motivation and churn out of the system, undermining governance.
Optimism’s governance model also faces the challenge of attracting and retaining the right mix of citizens. Ideally, governance should attract individuals with diverse skills and experiences who are motivated by the opportunity to shape the network’s future. A positive feedback loop—where badge holders feel their contributions are meaningful and rewarded—will help maintain a strong, engaged community.
The Economic Model: Forking Capitalism
Finally, the economic model underpinning retro funding adds another layer of complexity. Drawing from Optimism’s vision to “fork capitalism,” retro funding seeks to combine the best aspects of traditional economic systems with the innovations of decentralized governance. However, this is the least developed aspect of the retro funding system and likely the one that requires the most attention moving forward.
A sound economic narrative is essential for maintaining the long-term viability of retro funding. This involves ensuring that the revenue generated from sequencer fees and the treasury is sufficient to support continued retro funding efforts. Additionally, the supply and liquidity of OP tokens, as well as the motivations of those holding them, will play a significant role in determining the sustainability of the retro funding model.
Conclusion
Retro funding is a bold experiment in decentralized governance and builder incentives. By aligning impact with profit, it aims to attract top talent to work on the hardest problems in the Optimism ecosystem. However, to ensure its success, the system must continually evolve. This includes refining the builder funnel, ensuring fair evaluation processes, aligning governance tasks with badge holder strengths, and developing a robust economic model that supports long-term growth.
Ultimately, retro funding has the potential to drive meaningful innovation, but it requires careful attention to the incentives and motivations of all participants—from builders to badge holders to the wider Optimism community.


feat: add styles

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix:RDCC-2470 handle trimming in role mapping file

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RDCC-2470
Change description
RDCC-2470 handle trimming in role mapping file
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

closing this PR since build is not getting picked up

Cl

Closed since build is not getting picked up


The 'shopify' executable was not found on your $PATH

VS Code keeps giving me the following warning when the extension is enabled, but I'm not sure what to put in the extension settings to resolve the issue:
The 'shopify' executable was not found on your $PATH. Was it installed? The path can also be changed via the "shopifyLiquid.shopifyCLIPath" setting.

Hey @bigskillet, did you install the Shopify CLI or theme-check (Instructions)?
If you did, what installation method did you use?
On what platform?
The message you're seeing happens when the VS Code extension cannot find an installed executable. If you did install it and the extension still says that it can't find it, then you'd need to do debug your system's $PATH environment variable.

Installed Shopify CLI via Homebrew, macOS.
How might I debug the $PATH environment variable?

VS Code wasn't opening from the terminal, with error command not found: code. Using the VS Code command palette, I ran Shell Command: Install 'code' command in PATH and it seems to have resolved the issue.
@charlespwd, I really appreciate your help walking me through this!

np!

Hmm. That must be it. In that case, I would suggest setting the path manually in your settings.json file.
"shopifyLiquid.shopifyCLIPath": "/usr/local/bin/shopify",
Should do the trick. Looks like the Quick Action doesn't load the environment variables.

Yep, that fixed it, thanks again!


minSdk 14?

Are features from sdk 14 actually used by the code?
If not, you might consider lowering it, to make this library usable by apps targeting < 14.

Sure thing. Don't really even consider people still use less than that I guess.
https://github.com/klinker24/Android-TextView-LinkBuilder/commit/f9c58f94219ff1b3dd1535f8a1c55524064f6a9e

Thanks :-)
Yeah apps targeting dinosaur devices still exist ;-)


there is an issu in the website.

I entered https://play.nativescript.org/ for the first time using chromium, and it shows me as logged in as someone (not me), log out or deleting cookies and cache does nothing, and I am staying logged in as him.

If I try to enter using firefox I get the opposite problem, I cann't subscribe as it continues