When playing stereo, address the channels by name rather than using 0 and 1

This should fix https://bugzilla.mozilla.org/show_bug.cgi?id=1474175.

For someone totally unfamiliar with the Firefox release process -- when might this fix make its way into something I can try?

It will be in Nightly soon. Follow along in https://bugzilla.mozilla.org/show_bug.cgi?id=1474175.

@uberjay this should be available tomorrow in Nightly. This installs alongside a regular Firefox fine without touching your main profile if it's just for testing (not sure how familiar you are with nightly).
General availability in Firefox release is going to be in Firefox 128, July 8th this year.

Fantastic, the nighty build I just tried plays using the correct channels!!! Thank you all so much, it's greatly appreciated! ❤


Snap 517

Done required changes to make hydra test runnable.

See few comments above. Haven't looked at the new scripts, but will do that later and can go ahead with merge.


C# SDK mentions .NET Standard 2.0, but that's not a platform/runtime

The comparison table mentions "Speech Services supports Windows 10, Universal Windows Platform (UWP), and .NET Standard 2.0.". That's is very ambiguous, since NS2 is not a platform nor a runtime, so it's impossible to know what that means.
Looking at the runtimes folder in the SDK nuget package would mean linux-x64, osx-x64, win10-arm, win10-arm64, win-x64 and win-x86, and I'd guess that means .NET, Mono and .NET Core.
But that does not include Xamarin, which is also .NETStandard 2.0 compatible, yet it does not seem supported.
Please clarify, and if Xamarin is not supported, please do support it :)

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

ID: 50c36ca5-c28f-4853-626d-63e2ae804d43
Version Independent ID: dba96cab-2a0c-ee64-2bb9-500f38ec3cbc
Content: Migrate from Bing Speech to Azure Speech Services - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/how-to-migrate-from-bing-speech.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @WSturman
Microsoft Alias: gracez

@kzu
Thanks for the feedback! We are currently investigating and will update you shortly.

@kzu Hi, .NET Standard 2.0 supports .NET Core 2.0, .NET Framework 4.6.1, Mono 5.4, Xamarin.iOS 10.14, Xamarin.Mac 3.8, Xamarin.Android 8.0, Universal Windows Platform 10.0.16299 I agree we need more details about the document, so I assign the author @WSturman here for more improvement. Thank you very much for your feedback.

@WSturman @nitinme @aahill Hi team, I can not assign this issue to Grace. Is she the right person or I should assign this issue to someone else? Thank you.

@chrisbasoglu - Do you mind taking a look at this. Thanks!

Correct, we do not currently support Xamarin.  The good news is that we intend to do so (most likely in the 1.7.0 release).

That's great news! The bug, though, is that the doc mentions NS2 as a platform when it's not. So it's worth clarifying

@YutongTie-MSFT I know what NS2 "supports" (it's actually an API specs that the various runtimes you mention actually support, not the other way around). Hence, saying a library supports NS2 is ambiguous and incomplete, since as it's clearly the case in this library, not all runtimes are supported.

@kzu Hi Daniel, I agree with you.
@chrisbasoglu Hi Chris, could you please update the document to make it more clear? Thank you.

How about: Speech Services supports Windows 10, Universal Windows Platform (UWP), and .NET Standard 2.0. (not including Xamarin)

Again, Netstandard is not a platform or runtime. You should list the
actual runtimes, i.e. net core vX, netframework vX, Mono vX, etc.
See https://docs.microsoft.com/en-us/dotnet/standard/net-standard for the
sort of table I'd expect. So unless you support every runtime that NS2
supports, saying you support NS2 is misleading.
On Fri, Jun 21, 2019, 9:21 PM Chris Basoglu<EMAIL_ADDRESS>wrote:

How about: Speech Services supports Windows 10, Universal Windows Platform
(UWP), and .NET Standard 2.0. (not including Xamarin)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/33343?email_source=notifications&email_token=AABJN2ZMGSJSBNYZ7AW2W53P3VWALA5CNFSM4HYIBK72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYJ3XCI#issuecomment-504609673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJN25QWGABEQD3NMCWOULP3VWALANCNFSM4HYIBK7Q
.

Also, note from that table there's also Unity, so, do you support that too or not? That's why I think it's far better to be explicit

Yes.  Unity is supported.

So my recommendation would be to not even include a ns2 artifact in the package, and rather include it specifically in the platforms/runtimes you do support. That seems like non-trivial work (depending on how you're packaging) especially given that you do plan on supporting Xamarin soon-ish.
So I'd replace the mention in the docs of NS2 and instead replace it with the specific listing of the platforms you actually support. Then once Xamarin support is built-in, replace it back with NS2. Otherwise, it's just trial&error for users, which should totally not be the case for anything declaring to target NS2 (at the nuget package level, that is).

done

#please-close


Problem with exception serialization

Hello and thank you for your work. I have very specific exception type that cause

illegal_argument_exception Reason: "mapper [exception.detail.ErrorDetails.Value] cannot be changed from type [text] to [long]

error. It is WCF's FaultException of type Microsoft.Xrm.Sdk.OrganizationServiceFault. Last one has ErrorDetails collection of string-object pairs. Some of the values are strings other are not. Not sure what Value in exception.detail.ErrorDetails.Value is, but it looks like it was "mapped" by Elastic as text for the first time and can't accept other types.
Am I right that this target follow it's own exception formatting and serialization? How can I customize/skip serializing this member? I know that excludedProperties exist but setting it to exception.detail.ErrorDetails doesn't work because, I guess this is not a regular property. Setting maxRecursionLimit is not a good option as well as my exception type has useful information on the same level as ErrorDetails member.
Any suggestions?

Maybe an option could be enableJsonLayout = "true" with Elastic.CommonSchema.NLog. See also #125
You can then override the Exception-serialization to your liking - https://github.com/NLog/NLog/wiki/How-to-use-structured-logging#customize-object-reflection

Thanks for the suggestion, @snakefoot. It turns out that maxRecursionLimit works well for my case, but I'll keep enableJsonLayout  option in mind for the future.


Add the missing podCIDR to the kubeadm config

In order for flannel to start successfully, the podCIDR needs to be
configured by kubeadm. Add it to the network section of the config yaml.

/lgtm awesome!

Thanks for verifying @joejulian!


ShrinkImage test generation code

This is the code that generates the images for the test of ShrinkImage's --inputMipPointImageTest option. If this should have more documentation or be moved elsewhere, let me know.

Okay, style suggestions taken. The license headers say "Library: TubeTK" from the template; perhaps something else would be in order?

Hi,
Since the code is in the library, that header is fine.
Note also that KWStyle should be checking each file to make certain it contains one of those specific headers, but since your code is in the Doc directory, it was being skipped.   Nevertheless, it should conform to what is expected.

LGTM!


Makefile: Add EXTRA_FLAGS for on-demand compile flag changes

This commit adds an EXTRA_FLAGS variable to the Makefile, which can be set through the environment as needed. The contents will be appended to the FLAGS variable, allowing the developer to (for example) disable stringent checks for certain warnings during compile.
This can be used as follows: EXTRA_FLAGS=-Wno-strict-prototypes make decker
I realise that this might be unnecessary for most, but I ran into an issue when compiling Decker on Mac OS (-Wstrict-prototypes was flagging every function declaration, and -Werror was treating them as errors, and the default -ferror-limit=20 was killing the build after 20 errors) and it didn't feel quite right to edit the Makefile locally. Hopefully this helps?

OK; Seems reasonable.


Add 1186 - Incredible Chess(English tutorial)

Link : http://lightoj.com/volume_showproblem.php?problem=1186

Hello, are you working on this?

Hello, are you working on this?

I didn't start working yet.I am a little busy with some other work of mine. If you/someone interested in working on the editorial of this problem, you can.

I didn't start working yet.I am a little busy with some other work of mine. If you/someone interested in working on the editorial of this problem, you can.

Closing it due to inactivity


Any plan to update?

Or to fix the issue of Shunting jobs leaving loads on the loading platform and not an Outbound platform?

Or to fix the issue of Shunting jobs leaving loads on the loading platform and not an Outbound platform?

This is intentional and will not be changed. Put the cars wherever you see fit after turning in the shunting jobs, or taking the resulting freight haul jobs directly from the loading track.


Update core & S3 reference docs for 3.0

In progress

I opened PR just to have it already deployed so that its easier to review (GH actions do not run on drafts PRs). While you are free to review its not yet final.

@eddumelendez @MatejNedic core & S3 considered done. Once you review, we can merge it and then follow up with the rest of missing docs (i'll create tickets for them).


Rename the override attribute on inputs

May be force? or invert it's meaning?

What is the problem with the current name

Confusing as raised in demo. Put your votes here :)

Please vote for inverting the meaning: It will be true by default and setting it to false means "Don't override this input by context".
Or if you have other suggestions...

Last call - please respond :8ball:

More alternatives: immutable, lock/locked

I fell good with override.
maybe const ?

I don't like the override - very confusing
Regarding const - if we put an expression, I think that the word constant is also confusing.
:+1: for immutable

It will be renamed to immutable with default false. Setting true means the input won't be overridden by context.

Don't forget to align all our .sl .yaml .yml files after the change.
You should also change all the files under the slang-content repository: https://github.com/openscore/slang-content

how is it immutable if a task publish ca change its value?
flow:
  inputs:
    - input1:
        value: "aaa"
        immutable: true
 ....

  task1:
  ....
    publish:
      - input1: "bbb"

input1 = "bbb"

that is a good point.. it might be even more confusing to name it "immutable"

So we go with overridable with inversed meaning: It will be true by default and setting it to false means "Don't override this input by context".

@openscore/score-committers I was suuposed to review it, but I am swamped with the site stuff, can someone else review it? please?


spack bundling config not respecting sourceMaps setting

Describe the bug
When setting the sourceMaps swc option in the spack.config.js file to false, spack still outputs sourcemap files.
Input code
Here is a repo to reproduce the issue: https://github.com/Darkle/swc-test

The spack.config.js looks like this:

const path = require('path')

const { config } = require('@swc/core/spack')

module.exports = config({
  entry: {
    index: path.join(__dirname, 'src', 'index.js'),
  },
  output: {
    path: path.join(__dirname, 'bundle'),
  },
  options: {
    sourceMaps: false,
  },
})

Config
{
  "sourceMaps": false
}

Playground link
No response
Expected behavior
spack should not output sourcemaps when the sourceMaps option is set to false.
Actual behavior
No response
Version
1.2.178
Additional context
"@swc/cli": "^0.1.57",
"@swc/core": "^1.2.178"

Wrong repo


Bug: cancelling event causes all other entities events to be cancelled too

Hello,
I have a soft delete trigger, and this causes stackoverflow when trying to add/remove same entities at once with async.With same debugging i noticed this line, witch causes all other entity events after that to be cancelled.You can also see that in the sync version here the variable is inside the loop as it should be.When i moved the canceled variable inside the loop my problem with stackoverflow was fixed.

Thank you for the note. This is a bug. I'll look into fixing this.

@be4i This should now be fixed on the latest NuGet package.


Configurable OrderedImportsFixer

Lots of failing tests now.

@keradus is there anything else I can do ?

someone please kick the Travis tests again, it failed because it got stalled

@DarkaOnLine https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2280#discussion_r87712447 is still not resolved

@GrahamCampbell can you take a look (i think this a laravel standard)?

@SpacePossum I think he already reviewed it here: https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2148#issuecomment-258633327

And then have been modified and adjusted for master branch, and then have few things raised to be updated (already done), so would be great to have final review, even if it would be quick one ;)

@GrahamCampbell,  please review this :pray:

This looks reasonable in principle. I'll need a bit of time to test this in more detail.

@keradus can we move forward ?

Needs rebasing again. Might be an idea to actually "rebase", rather than having merge commits. Will make it much easier going forward.

Thank you @DarkaOnLine.


Add a note about JSON formatting of streaming logs.

Why:
Closes ISSUE: https://github.com/github/docs/issues/23332

What's being changed (if available, include any code snippets, screenshots, or gifs):
Added note about JSON format of the audit logs streaming.

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

👋 @adamrr724 Thank you for opening a PR and linking it to your issue! ✨


Fix exports

Superseeds #102

Checklist

[ ] run npm run test and npm run benchmark
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

Or we should decide to release a new major?

It doesn't hurt for one more named exports.
I would encourage a major only for feature changes but not a patch like export.

better?


[Door-Lock-Server] Calls to emberAfPluginDoorLockOnUnhandledAttributeChange need added attributePath parameter

Problem
The emberAfPluginDoorLockOnUnhandledAttributeChange function needs the attributePath parameter added to the parameter list.
Proposed Solution
change function declaration from:
emberAfPluginDoorLockOnUnhandledAttributeChange(chip::EndpointId EndpointId, EmberAfAttributeType attrType, uint16_t attrSize, uint8_t * attrValue)
to:
emberAfPluginDoorLockOnUnhandledAttributeChange(chip::EndpointId EndpointId, const chip::app::ConcreteAttributePath & attributePath, EmberAfAttributeType attrType, uint16_t attrSize, uint8_t * attrValue)

@Morozov-5F would you be able to help take this on?

@jrhees-cae what are the observable consequences of this function having the wrong signature?

Without the attribute path being included as a parameter, the function is
completely useless to an app which wishes to override the weak link. The
app gets the call, but had no way of knowing which attribute was being
changed.
Jon
On Tue, May 24, 2022, 9:26 PM Boris Zbarsky @.***>
wrote:

@jrhees-cae https://github.com/jrhees-cae what are the observable
consequences of this function having the wrong signature?
—
Reply to this email directly, view it on GitHub
https://github.com/project-chip/connectedhomeip/issues/18681#issuecomment-1136684117,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOU6QVUHPBQY3ZE5WGABMN3VLWMV3ANCNFSM5WQAJFDQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sounds like lock apps will need this....

SDK Review: Seems like this could help Lock's for SVE, not required, but given where we are right now, we can take this.

SDK Review: @Morozov-5F are you able to take a look at this?


Install License Through pandevice

Hello,
Is there a way to install a license through pandevice API ?
Thanks.
-Asghar

Answering my own question, the following code should allow adding of licenses to a firewall device :
from pandevice.base import PanDevice device = PanDevice.create_from_device("<IP_ADDRESS>","admin","admin") device.activate_feature_using_authorization_code("abcd1234")


Missing output from specs

hey guys,
this gem is awesome and very useful for continuous integration. The problem is that because it doesn't output anything until it's finished we are facing an issue with output timeouts on our continuous integration platform. Is there any possibility to add an output to this gem in the near future?

I have the same issue using rspec-rerun 1.0 with rails 4.2.3 and rspec 3.3.0.
There is no output at all but formatters are specified (in .rspec and on the command line). There is no problem when running rspec directly.

Also when it fails, it only indicates there’s been a failure without backtrace or other useful information. On the rerun there’s an error:
[2015-06-30 16:30:06 +0000] Failed, 1 failure
rake aborted!
NameError: undefined local variable or method `failed_count' for main:Object
/bundle/gems/rspec-rerun-1.0.0/lib/rspec-rerun/tasks.rb:116:in `block in <top (required)>'
Tasks: TOP => rspec-rerun:spec
(See full trace by running task with --trace)

undefined local variable is fixed in https://github.com/dblock/rspec-rerun/pull/44


Fix section title in architecture docs

I was reading the docs and found an AWS title for a DigitalOcean section.
Testing
Go to https://typhoon.psdn.io/architecture/digitalocean/ check title is not AWS anymore.

Thanks!


[Bug]: Code scanner camera is not operating when user switches from laptop camera to code scanner camera

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Code scanner camera is not operating when user switches from laptop camera to code scanner camera.
Expected Result  : Code scanner camera should be functioning once we close the system (laptop) camera.
EE DP: https://ee-1615.dp.appsmith.com/
Steps To Reproduce
Steps to reproduce :

Add code scanner widget to the canvas
Click on scanner > camera opens up >  close the camera from canvas
and open camera in the system (laptop) and go back to the appsmith canvas and click scanner >> it says camera cant be open up which is expected
Now close laptop camera and go back to appsmith and click scanner >> It still says camera cant be open up.

Issue ref screen shot :

Actual result : Code scanner camera shows error message "could not start video source"
Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
Appsmith Business v1.9.22-SNAPSHOT

@dilippitchika please find the recording video where code scanner widget camera is not refreshed automatically when system camera is turned off.
screen-capture (2).webm
do let me know if you need any further details on the same.

@dhruvikn do you have some context here? I remember that we introduced some limitation that comes up with iPhones.
Cant put my finger on it.

Yeah I am the user that is having problems on IOS. The issue is you can change camera using the camera icon but when front camera is selected it is still displaying back camera. I have tried on multiple apple devices same issue. I tried in a different online camera display and it allowed me to use the front camera for that

I tried to reproduce the issue, but couldn't on my mac. I tried streaming on https://webcamtests.com/ and the Photo Booth app and I was able to stream video in the widget.  [See video here]
On @kavitasmoolya's end, it looks like it could be an issue where the browser isn't able to access the webcam stream when the camera app on Windows is open. The camera app could be disallowing the webcam video stream while it's open. @kavitasmoolya - can you please try to recreate this once again, but this time instead of the camera app, can you open https://webcamtests.com?
Moreover, I'm able to use all the cameras on iOS and they are working as expected. [See attached video]
@upbound-web - Can you please try again on the latest version of Appsmith and share a recording if possible? Also, please share your iOS version and iPhone model.


Feat: Complete feature trending news and food health

Describe the feature
I want to complete a page using dynamic loading so that user can get the latest news about the dishes .
Add ScreenShots

Record

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

Ok, Assigning this issue to you.


Avoid TOTP code replay attacks

As far as I can tell, GoogleAuthenticator library doesn't prevent a replay attack.
For this to be relevant we have to assume that either the victim is using an insecure network or has malware on their computer - both of which are not ideal, but happen more often than we'd like.
Would it be possible for GoogleAuthenticator library itself to prevent a replay? My guess is that the library doesn't have any way to persist data so it can't really prevent this kind of attack, but maybe you have ideas on that.
If it's not possible to solve in the library, then I suggest adding a note to the README to encourage people who use the library to add a feature to their code to prevent replay.

As greggles said, it should be done outside of this library. The application should use its favorite storage for remembering used codes, and reject them if they are tried again. So it "works as designed".


Edit .gitignore

This file is created by cargo when resolving dependencies

Plese change commit message to "Add .cargo-ok to .gitignore"

done)


zookeeper-upgrading-maven-dependency-removing-vulnerabilities

upgrade dependencies to address CVE-2018-8012 known security vulnerabilities

This is not the proper place to address this


chore(eslint): remove duplicate configuration item

What kind of change does this PR introduce?
Dev UX
Did you add tests for your changes?
N/A
If relevant, link to documentation update:
N/A
Summary
remove duplicate configuration item
Does this PR introduce a breaking change?
N/A
Other information

Thanks


Improve resctrl group discovery

Only accept directories that have a tasks file.

Codecov Report

Merging #20 (f4ac101) into main (82f290c) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
+ Coverage   77.64%   77.66%   +0.02%
==========================================
  Files           6        6
  Lines         926      927       +1
==========================================
+ Hits          719      720       +1
  Misses        138      138
  Partials       69       69

Impacted Files
Coverage Δ

pkg/rdt/rdt.go
65.07% <100.00%> (+0.12%)
:arrow_up:

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 82f290c...f4ac101. Read the comment docs.


Align header links centrally on news organisations

Introduced at https://github.com/alphagov/collections/pull/685/commits/ec8da8bc803cbda2deb28a7683dc03b502f84efa#diff-e1dc10a5537077c2ff786edc2b95d1c11c5b95751ba532cba44a92df77fd9921R51
We had a request to change this styling in a Zendesk ticket:
https://govuk.zendesk.com/agent/tickets/4772599
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

@bilbof I'm concerned that this is a broad change that will affect all organisations - instead of the single organisation that the issue was raised about. Is there a way of setting that one organisation to be a news organisation instead?


Migrating algorithms sub-package to ee_extra.

Dear Prof. Quisheng Wu,
The river.py is already on ee_extra! :rocket:. Once we add more algorithms, I'll keep maintaining and upgrading the bridge with geemap :)
example: https://colab.research.google.com/drive/1kPeVDNvIWCZYeI5nUWSeWeOmM85kh6NF
conda: https://anaconda.org/conda-forge/ee_extra
pip: https://pypi.org/project/ee-extra/

This is great. To minimize the maintain work you need to do, I think you can simply put the following line in __init__.py. The river.py is no longer needed. Whenever you add a new algorithm module to ee_extra, simply modify the following line to import the new module. What do you think?
from ee_extra.Algorithms import river, panSharpening # any new module should be imported here

I think that would be better :D. I make a commit with the required changes. :)


Expose SpriteBatch's transform matrix parameter

As discussed in #4 , making it possible for users to set the transformMatrix argument on the SpriteBatch.Begin method would make it possible to easily create camera-like behavior. By setting this parameter users can control the offset, rotation, and zoom of the rendered texture.
Here is a short video that shows a simple scene rotating, being translated, and zooming in/out. There are some strange things that happen with the text when zooming and rotating, but I imagine these are to be expected given the "low resolution" at which we're working.
https://github.com/BenMakesGames/PlayPlayMini/assets/3422347/ae1862a4-5aeb-48d0-b71a-4133cc1a4b47
To make this demo I created an extremely simple "camera". Should we include it in this PR as well? Also, do you think there's any documentation that should be added about this feature?
Right now the transform matrix will be preserved even if the current scene/GameState is changed. I think it should be up to the user to "unset" the transformation matrix when needed (should we add an unsetmatrix method?). But I don't know if you agree or you can think of some way to unset it when the scene changes (we could hook into the GameStateManager, but it seems ugly).

I don't have a good process setup for release (no tagging... no changelog...), so I just merged, built it, and published on NuGet; should be available there, soon!


Improve JsonDecodeTransformer

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

Project coverage is 100.00%. Comparing base (18f8bf6) to head (9bb39b3).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #73   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        28
  Lines          572       576    +4
  Branches        27        26    -1
=========================================
+ Hits           572       576    +4

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


Short solution needed: "How to exit without saving" (vim)

Please help us write most modern and shortest code solution for this issue:
How to exit without saving (technology: vim)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to exit without saving


Consider upgrading to cassandra-co

cassandra-co might be able to provide features like better query creation and migration, updates to data structure, etc.

Other possibilities:

https://github.com/masumsoft/express-cassandra
https://github.com/ahlwong/dakota-cassandra

It seems https://github.com/masumsoft/express-cassandra is the best choice.

The migrations are WIP but seems to be the most of all the projects above:
https://github.com/masumsoft/express-cassandra/issues/27

At this point none of the ORMs provide seamless migration. express-cassandra is the most advanced in this regard, however it can only drop and re-create the tables.
Also, the ORMs impose certain limitations to accessing the full feature set of Cassandra. For instance, express-cassandra would allow the use of only one SASI index, and it is not immediately evident what the other ORMs would allow.
At this point the most prudent decision is to wait out and see if the ORM projects become more feature rich, or URB becomes complex enough to warrant the use of ORM to simplify the code.

Full support for SASI indexes is available in the latest versions of express-cassandra, and work is under way on developing migrations. Considering this, it is a good time to make the jump and move all Cassandra facing interactions to express-cassandra now rather than later.

Currently express-cassandra does not seem to allow creation of tables containing capital letters in the name, this would have to be resolved before continuing:
https://github.com/masumsoft/express-cassandra/issues/61

The capital letters issue is resolved in 1.1.2.

Since the models created by express-cassandra can not be accessed before the initialization, the current instantiation of the GraphQL schema with the models would not work. For now express-cassandra will be used to create the tables, but the queries will be performed through CQL. This will most probably change with the next edition of data loader.


dataclass decorator: unexpected keyword argument using mypy

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.9.0
            pydantic compiled: True
                 install path: /home/bay2rng/PycharmProjects/semisched/packages/scheduler/.venv/lib/python3.9/site-packages/pydantic
               python version: 3.9.9 (main, Nov 16 2021, 03:08:02)  [GCC 9.3.0]
                     platform: Linux-5.4.0-90-generic-x86_64-with-glibc2.31
     optional deps. installed: ['typing-extensions']

While translating some models implemented using Python's standard dataclass decorator into Pydantic dataclasses, mypy 0.931 started complaining about unexpected keyword arguments
from dataclasses import dataclass
from pydantic.dataclasses import dataclass as pd_dataclass

@dataclass
class ModelA:
    parameter: int

@pd_dataclass
class ModelB:
    parameter: int

a = ModelA(parameter=1)
b = ModelB(parameter=1)

Output of mypy test.py
test.py:16: error: Unexpected keyword argument "parameter" for "ModelB"

It seems the return type has some problem, and likely affecting VSCode auto-completion as well which I believe was meant to be fixed in https://github.com/samuelcolvin/pydantic/pull/2721.
If you hover "b" in this example, you will see its type is detected as (variable) b: Dataclass.

The dataclass decorator is annotated as returning a Dataclass object, which defines the Pydantic dataclass additions, losing any information about the original class that is being decorated.  The dataclass transformation defined in #2721 is only applied to ModelMetaclass, which is not the metaclass of dataclasses.
I know of no way to annotate correctly a decorator that modifies a class's methods and fields. You either do what Pydantic does or you annotate like your decorator does not change the class.

This works on both v1.10.X and v2 without issues even with mypy --strict, so it seems like it's been resolved.


Can't set independent default icons for folders and files

Describe the bug
Regex icon rules for folders and icons conflict with one another making it impossible to set independent default icons for folders vs. files.
To Reproduce
Steps to reproduce the behavior:

Go to settings
Create one icon rule .* for folders.
Create another rule (also .*) with a different icon to match files.
Note that only one or the other will work.

Expected behavior
Should be able to set independent rules for files and folders.
Additional context
Changing the regex to something slightly different like .+ doesn't help - any overlapping matches cause the conflict.

Did you change the type as well?
For example:

Gives me the following result:

Yes, and when I click one, it causes the other to revert. Here is a gif of what I see:

I have found a work-around though - if I close and reopen the vault, the icons will be correct, but the incorrect behavior still remains if you try to change the settings.

Yes, and when I click one, it causes the other to revert. Here is a gif of what I see:
I have found a work-around though - if I close and reopen the vault, the icons will be correct, but the incorrect behavior still remains if you try to change the settings.

Ah ok got it! Thank you for the gif. That helped me a lot.
I will start fixing this issue asap.

This issue should hopefully be resolved with the newest version v1.2.4. Thanks for reporting again.


[SpikeGLX] Support imDatPrb_type 1100 (Ultra variant)

Stemming from https://github.com/NeurodataWithoutBorders/nwb-guide/issues/713
A user with a SpikeGLX .meta file contains the field imDatPrb_type=1100, which has been unseen before and triggers an error at line https://github.com/NeuralEnsemble/python-neo/blob/883b29a28575b23f04189aca49a4a562b606a685/neo/rawio/spikeglxrawio.py#L470
You can replicate this easily by using existing SpikeGLX test data (including Noise4Sam) and replacing that line (which is probe type 0 for the classic NP v1 probes)
To my possibly naive eyes the rest of the meta files (which can be found attached to the original issue as .txt files) appear consistent with other v1 .meta files (384 channels on one shank + SY)

Actually, the conversion is working well. So please ignore my last post. It failed kilosort because of a minor but important technicality, I believe. The raw ap.bin file is 385 channel (384+1 sync channel) x timestamps, but the converted data in NWB file is 384 channel. Kilosort worked fine after I changed this parameter.


Add a PR template and typedoc site generation

This PR adds a PR template to make linking PRs to issues easier.
It also introduces a github action that generates HTML docs based on types and deploys them to GitHub Pages.
Once this is merged, GitHub Pages need to be enabled.
Docs screenshot

How to enable GitHub pages

I have read the CLA Document and I hereby sign the CLA


[plugin] file-log plugin don't create a global file

Summary
I'm trying to create a global file-log to my APIs. I'm able to create the global file-log entry but Kong don't create the file after I consume an API successfully.
I've tested the file-log plugin to an individual API an it's working fine. Just in the global config is not creating the file.

}

Check the log file

❯ ls /usr/local/kong/logs/
access.log
error.log
serf.log

The log file all.log do not exist.

Additional Details & Logs

Kong version

0.9.5

Operating System

using the docker official image

@marceldiass I'm unable to reproduce this behavior. Can you check the permissions of /usr/local/kong/logs/ and ensure that you have write access to that directory?

Closing as stale after no response. Feel free to re-open this if you're still having issues here. Thanks!


Add a replace function that supports regex

The current replace(s, from, to) function doesn't seem to support regex. I think it would be helpful to also have a replace_regex(s, from, to) for more complex replacements.

This sounds like a good feature, also a great first issue if anyone wants to take a crack at it.


[WFLY-13578] adding integration failure tests for WFTC remote call with txn propagation

https://issues.redhat.com/browse/WFLY-13578

/retest

@bstansberry the failing test was the BeforeShutdownJNDILookupTestCase and the reason was that it uses the method controller.kill. See https://github.com/wildfly/wildfly/blob/21.0.0.Final/testsuite/integration/manualmode/src/test/java/org/jboss/as/test/manualmode/weld/extension/BeforeShutdownJNDILookupTestCase.java#L69
As the controller.kill is in fact implemented in way of calling the stop in the WFLY Arq then calling stop and kill do the same. I think the test BeforeShutdownJNDILookupTestCase expects the server not being killed immediatelly (like kill -9) but it assumes a shutdown hook to be invoked on container shutdown. I changed the call for stop.
The original version of the PR was adding an arquillian extension (https://github.com/ochaloup/wildfly/commit/fb9776674d99b3f0259338f844080cc220ccb5a8#diff-0567bd51c80833a49443f175c57da2e53a8974617f4821e0cb0be988ca0a4a8aR44) which makes the kill to do nothing. It was necessary because there was a bug in WFLY Arquilian (https://issues.redhat.com/browse/WFARQ-78). With the new release of the managed container (3.0.x) it's fixed and updating the WFLY arq fixes the test and does not make a need to add the arquillian extension which workarounds the kill behaviour.
Then I'm not sure why the WFLY Arq is not used with new version in whole WFLY - https://github.com/wildfly/wildfly/blob/21.0.0.Final/pom.xml#L446 (https://issues.redhat.com/browse/WFLY-13937) but hopefully it could work for the manual mode module at least (as it's needed for my newly created testcase).

@bstansberry is this something that could be merged?
Or could be somebody so kind to review this first? e.g. @chengfang or @tadamski :-)

@wildfly/merger-full the tests passed, can you please take a look?

@wildfly/mergers-full The tests passed. Would you be so kind to check this, please?

@ochaloup When I combined this with a bunch of PRs to test the set before merging, I saw failures of two tests added here:
https://ci.wildfly.org/viewLog.html?buildId=235039&buildTypeId=WF_PullRequest_WindowsJdk11
That was on #13829
I don't have any particular reason to think that was because of the other commits in that PR. It seems more probable it was just an intermittent failure exposed by running the test another time.

/retest

/retest

/retest

I initiated a re-test but I just cancelled it as I want to keep CI free today for things that are critical for WF 22. So ignore the cancellation failures.

I initiated a re-test but I just cancelled it as I want to keep CI free today for things that are critical for WF 22. So ignore the cancellation failures.

/retest

@bstansberry will this make WildFly 23 do you think? Thanks

/retest

@tomjenkinson @ochaloup Previously the tests in this PR failed on some test runs (see my comment of Dec 16), so I'm occasionally doing a retest to help guard against the possibility that these will introduce disruptive intermittent failures in CI.
This doesn't need to be in the WF 23 Beta, so if my latest retest looks ok it's probably good to merge it soon after we tag the beta so we have two weeks of it running all the time in CI to detect and fix any intermittent failures.

Thanks @bstansberry  /cc @mmusgrov


Create the Batch Jobs to Process Input Files

Create the batch jobs to handle the logic of the incoming input files

Closed due to breaking down into #17, #18, and #19


Feature request: Support for export only files

My codebase has a router and a handlers sub folder. Each function is its own file. I then have an index file which exports each of the functions:

When entering the inline metadata for a function, it is not picked up when using the export approach. If I use the individual function file directly then it pull in the documentation.
I guess this approach isn't supported, so I'd like to request it.

Hi @PhilHannent! Thank you for opening the issue.
Yes, that approach isn't supported, but it is a great feature to integrate with the module! If possible, could you send me just one of the .js files in your handle folder, please? Just to see the structure that you've used in your .js files.

Good morning @davibaltar ,
That took slightly longer than I expected to create a minimal example, but it is done:
https://github.com/PhilHannent/example-express-single-file-functions
Whilst the diagnostics route is captured, the documentation in the function is not.

Hi @PhilHannent! Thank you for the sample project. It'll help me.
I've forked it and I'll try to implement this feature as soon as possible.

Hi @PhilHannent. The version 2.21.7 has fixed/implemented this. If the problem persists, let me know.

Good morning, Thank you! The new version solves the problem for me.

Thanks Bro!


Removed all references of UIWebView

Checklist

[x] I've checked that all new and existing tests pass
[x] I've updated the documentation if necessary
[x] I've added an entry in the CHANGELOG to credit myself

Description

In order to remove all references of UIWebView (Because Apple doesn't accept upload of binaries that include references to UIWebView anymore) I had to bump the minimum version for AFNetworking to 4.0. Installation of that version of AFNetworking required us to increase the minimum deployment target for iOS  to 9.0 and for macOS to 10.10.
Motivation and Context

This PR fixes issue #323

bhjheghehdv


Add CreateBuildConfig test

Adds a test that covers CreateBuildConfig in occlient.go
Closes https://github.com/redhat-developer/odo/issues/828

:white_check_mark: odo build 145 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/94b8418f94 by @cdrage)

@mik-dass Updated!

:white_check_mark: odo build 175 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/c15df5ba4a by @cdrage)

@cdrage The GolangCI pointed two improvements

:white_check_mark: odo build 177 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/fb74b2b424 by @cdrage)

@mik-dass
Updated to include tests for both annotations as well as labels.

:white_check_mark: odo build 183 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/a7cee7319a by @cdrage)


Jadeite System Browser Class >> Remove menu item does not call Rowan API when Dictionaries selected

The class remove will work when packages is selected, but not when dictionaries is selected ... it's probably worth doing a sanity check for any other critical path menu item differences between the two modes

I just checked your latest checkout and the code in the isPackages block is the only place where the Rowan API is used ... if you follow the code when isPackages is false (when in the Dictionary view) you'll see that a pretty complicated algoritym is used ... but in Jadeite only the Rowan API should be used ...
There are a handful of places where isPackages shows up on a grep:
rogue:sources>grep isPackages *
Jade System Browser.pax:	| isPackages containers classNames |
Jade System Browser.pax:	isPackages := self nextLine = 'packageList'.
Jade System Browser.pax:				isPackages ifTrue: [
Jade System Browser.pax:	| isPackages container className |
Jade System Browser.pax:	isPackages := self nextLine = 'packageList'.
Jade System Browser.pax:			isPackages ifTrue: [
Rowan UI Base.pax:	| isPackages containers classNames |
Rowan UI Base.pax:	isPackages := self nextLine = 'packageList'.
Rowan UI Base.pax:	isPackages
Rowan UI Base.pax:							isPackages

and they are worth investigating ... The Rowan UI Base.pax appears to be the remove class logic, but I don't know if there are other differences between the two views...

The class remove will work when packages is selected, but not when
dictionaries is selected ... it's probably worth doing a sanity check for
any other critical path menu item differences between the two modes

Yes, the Dictionaries tab may be problematic in more ways than that. I'll
try to take a look at this before alpha.
One other possibility is that we tell the customer to stay on the Rowan
Packages tab. The other tab may contain un-Rowan-packaged code which is not
yet supported.
On Sat, Apr 28, 2018 at 7:50 AM, Dale Henrichs<EMAIL_ADDRESS>wrote:

I just checked your latest checkout and the code in the isPackages block
https://github.com/ericwinger/Jade/blob/master/sources/Rowan UI Base.pax#L605-L611
is the only place where the Rowan API is used ... if you follow the code
when isPackages is false (when in the Dictionary view) you'll see that a
pretty complicated algoritym is used ... but in Jadeite only the Rowan API
should be used ...
There are a handful of places where isPackages shows up on a grep:
rogue:sources>grep isPackages *
Jade System Browser.pax:	| isPackages containers classNames |
Jade System Browser.pax:	isPackages := self nextLine = 'packageList'.
Jade System Browser.pax:				isPackages ifTrue: [
Jade System Browser.pax:	| isPackages container className |
Jade System Browser.pax:	isPackages := self nextLine = 'packageList'.
Jade System Browser.pax:			isPackages ifTrue: [
Rowan UI Base.pax:	| isPackages containers classNames |
Rowan UI Base.pax:	isPackages := self nextLine = 'packageList'.
Rowan UI Base.pax:	isPackages
Rowan UI Base.pax:							isPackages
and they are worth investigating ... The Rowan UI Base.pax appears to be
the remove class logic, but I don't know if there are other differences
between the two views...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dalehenrich/Rowan/issues/128#issuecomment-385181534,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiiPJB0moZyTTn7e2On6rD_4SvAL1twFks5ttIGhgaJpZM4Trbsn
.

--
Eric Winger
Staff Engineer
GemTalk Systems

That's your call ...
I've spent most of my time developing in the dictionaries tab ... mainly because when I'm running tests and I touch a browser in package mode, all of the Rowan packages disappear when the browse automatically updates to show different pacakage contents (as they should because the tests inserts a different project registry)  --- but very annoying to me since I can forget that I've got a debugger up --- working in dictionary mode doesn't have this problem:) ... I suppose that's a good thing ...
It does occur to me that they will have to use the dictionaries tab if they are looking at source code for non-packaged classes, like all of the kernel classes, so it might not be feasible for them to stay away ...

That's good to hear you've been developing in the Dictionaries tab. It
gives me some confidence that it more or less works. I've
only used that tab for working sparingly with the JadeServer classes.
On Sat, Apr 28, 2018 at 8:09 AM, Dale Henrichs<EMAIL_ADDRESS>wrote:

That's your call ...
I've spent most of my time developing in the dictionaries tab ... mainly
because when I'm running tests and I touch a browser in package mode, all
of the Rowan packages disappear when the browse automatically updates to
show different pacakage contents (as they should because the tests inserts
a different project registry) --- but very annoying to me since I can
forget that I've got a debugger up --- working in dictionary mode doesn't
have this problem:) ... I suppose that's a good thing ...
It does occur to me that they will have to use the dictionaries tab if
they are looking at source code for non-packaged classes, like all of the
kernel classes, so it might not be feasible for them to stay away ...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dalehenrich/Rowan/issues/128#issuecomment-385182837,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiiPJJbzJNH45yTl7KcJmG4pRKo400xTks5ttIY6gaJpZM4Trbsn
.

--
Eric Winger
Staff Engineer
GemTalk Systems

The three methods of interest are:
JadeServer>>sbRemoveClasses
JadeServer>>sbRemovePriorVersions
JadeServer>>sbRevertClass

sbRemovePriorVerions and sbRevertClass basically do this: (I think both are monticello-releated and can be ignored.)
... (class category copyFrom: 1 to: (class category size min: each size)) = each ...

https://github.com/ericwinger/Jade/blob/50185058d7dd4f45f6bcd27932aeb45bd05cba33/sources/Jade System Browser.pax#L1064
A problem with the code as it exists is that you can't tell which classes are rowanized based on the package/dictionary tab.
Dale's suggestion above is to remove all non-rowan removal code in sbRemoveClasses. That's easy enough and localized. However, it means that you wouldn't be able to remove a non-rowan class. I think that's ok.
But because we are code-frozen ... should I?

Remember, we've decide that the developer should be notified when operating on unpackaged classes and methods. The RwPerformingUnpackagedEditNotification should be signaled for every Rowan API call that is operating on unpackaged classes and methods ... at the moment we decide that the current default action (error) is the right answer ... so we are already intending to not allow developers to remove non-package code ...
... as to whether or not you should break the alpha? Well right now I was planning on having lunch and taking a mental break for an hour or so ...
My main concern is that the dictionary tab opens up all kinds of nasty holes for users to step in a corrupt the loading things ... but right now I've got to take a break ...

Ok, I'll go ahead and fix it, look at the next emails sent and try to fix
other bugs, then cut a new alpha.
On Sat, Apr 28, 2018 at 1:44 PM, Dale Henrichs<EMAIL_ADDRESS>wrote:

Remember, we've decide that the developer should be notified when
operating on unpackaged classes and methods. The
RwPerformingUnpackagedEditNotification should be signaled for every
Rowan API call that is operating on unpackaged classes and methods ... at
the moment we decide that the current default action (error) is the
right answer ... so we are already intending to not allow developers to
remove non-package code ...
... as to whether or not you should break the alpha? Well right now I was
planning on having lunch and taking a mental break for an hour or so ...
My main concern is that the dictionary tab opens up all kinds of nasty
holes for users to step in a corrupt the loading things ... but right now
I've got to take a break ...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dalehenrich/Rowan/issues/128#issuecomment-385204200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiiPJHOOCYw7wuLkqwrEviHfCq_mvG2Bks5ttNSVgaJpZM4Trbsn
.

--
Eric Winger
Staff Engineer
GemTalk Systems


Added a note about the problem in issue #1726.

Description

Motivation and Context

Have you tested this? If so, how?

[Current coverage][cc-pull] is 76.14%

Merging [#1733][cc-pull] into [master][cc-base-branch] will decrease coverage by 2.67%

@@             master      #1733   diff @@
==========================================
  Files            95         95
  Lines         10346      10346
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           8155       7878   -277
- Misses         2191       2468   +277
  Partials          0          0

Powered by Codecov. Last updated by [831e390...ea14eb9][cc-compare]
[cc-base-branch]: https://codecov.io/gh/spotify/luigi/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/spotify/luigi/compare/831e390763d9b2c83faeef43985debf54d92403a...ea14eb903af607d6d804ae79a842490d5f56907e
[cc-pull]: https://codecov.io/gh/spotify/luigi/pull/1733?src=pr

LGTM

I'm not familiar with this issue, and I still don't understand it after reading and re-reading the new documentation several times. Could you please be more clear about what will happen if you do
[Outer]
x=eleven

versus
[Inner]
x=eleven

and create instances of both Outer and Inner?

@Tarrasch Do you have an idea how to make this clearer?

would prefer if you put this in the documentation for @requires instead. configuration is a lot more common and i don't want to add unnecessary complexity to the documentation there

@dmyersturnbull,
Here's what I think you could do to make it clearer:

Move the docs as @erikbern said
In the docs for configuration. Perhaps only add a sentence saying something like "Any configuration done to a luigi task will never affect anything but instances of that class,  not superclasses nor subclasses.". Luigi never cares in what way a task parameter came to be (weather it's through inheritance, @requires or explicit retyping). Parameter objects are also immutable and more functional than object oriented.
Come up with a more creative name than Inner/Outer.

@dmyersturnbull, there's a flake8 issue. Can you resolve that?

@Tarrasch It looks fine to me. Do you see where the issue is?

Not sure. I would look for other existing examples in the docs and see how your code differs. As noted in CONTRIBUTING.md, you can debug this locally too using tox


Facade Analytics Code

I see that you have this item on the last Applications example. I tried to find the code for it but without any success. Is it possible for you to kindly point me towards it?

Hi, the code is not included.


refactor: skip logging ResponseURL as this contains an IAM access id

Example ResponseURL:
https://cloudformation-custom-resource-response-apsoutheast2.s3-ap-southeast-2.amazonaws.com/arn%3Aaws%3Acloudformation%3Aap-southeast-2%3A558125227772%3Astack/azure-graphql-wrapper-bdb/a91b0890-89d3-11ee-a54c-0ae0233cf359|SourcemapUploader|ffce58fd-405a-4bb7-8f41-d88e3d609c0f?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Date=20231123T074157Z&X-Amz-SignedHeaders=host&X-Amz-Expires=7200&X-Amz-Credential=AKIA6MM33IIZWN5ZYINB%2F20231123%2Fap-southeast-2%2Fs3%2Faws4_request&X-Amz-Signature=2632a7e64578b4076a6d8eb9d735e1509b4e4dbfd325615bcf4af494f68176e2
Simply skip logging this, to avoid security flagging this as a potential issue, even though the access id by itself is not an issue.

Ah good catch.


MEMS sensor drivers in extras

I have written some drivers for MEMS sensors like the gyroscope L3GD20H, the accelerator LIS3DH, the magnetometer LIS3MDL, and their combination like the LSM303D. All these drivers are working together with ESP8266 and esp-open-rtos as well as ESP32 and esp-idf.
Are these drivers of interest for the project's extra folder? If so, I could create according pull requests.

Sounds definitely interesting to me. Are the drivers I2C or SPI based?
I've also got drivers for MPU6050 and LSM6DS3 gyro + accelerometer combinations. What I've been wondering is if there should be some kind of a HAL or standardized interface for operating them. Oftentimes sensors from different manufacturers seem to be very similar, main differences being the registers and their values during configuration.
I'd very much like to see how you've handled the different sensors @gschorcht.

@flannelhead
The drivers support I2C as well as SPI.
Yes, at least for the sensors from STM that I know, it seems that they have very similar interfaces and are used in a similar way. Therefore, I was also wondering whether it's possible to define a standardized interface. But, I had to learn that even though the interfaces seem to be similar and also the registers seem to be almost the same, their configuration and their usage can be quite different. So it seemed to be easier for me, to develop a specialized driver for each sensor.
You can take a look to my fork. There is a separate branch for each of the sensors.

@gschorcht, that seems like a sensible approach, and I agree with you that the use cases could be quite different. Your drivers look good in my opinion and would be a nice addition to the OS.
The set of standardized functions you mentioned seems feasible. I've been using a somewhat similar interface in my project where I support MPU6050 and LSM6DS3 interchangeably - there actually are only init_sensor and get_raw_data implemented in the common interface. init_sensor takes in an array of register-value pairs for configuring the sensor. In my case that's the easiest route - the configuration is separately specified for both sensors so that they match each other (ODR, full scale range etc.) While that contradicts the idea of a high-level API, this reduces the maintenance surface for the cross-compatibility layer and also reduces the barrier of introducing new sensors, not having to implement a full-blown high-level API for the sensor.
Just some thoughts... IMO your drivers look really good :) An interface like this could even be added afterwards.


Only show map if co-ordinates exist and are not 0

Ensures that map will only display if co-ordinates have been set and are not 0
Fixes #JOINDIN-560

Can one of the admins verify this patch?

ok to test


reporting source on modifications

I just spotted this:
pcn1 source, reported a Frampton:

but Frampton isn't the source of all the data we display for K164

Ah, yep, that's a mistake.  The alleles aren't supposed to show references because there can be more than one.

This is a modification.

Ah right, we probably shouldn't show them on modifications either.

Yep, this  info will need to wait for the tables

This is fixed for Tuesday morning.


Artwork is either not being returned, or not persistent (primarily seen with the Artists listing)

What version of Music Assistant has the issue?
2.0.4
What version of the Home Assistant Integration have you got installed?
2025.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
On loading a new library, I found that most metadata was missing, and would often be populated by selecting the entry and waiting. I opted to reprocess the entire library through Brainz, with the ID3 tags all updated, and rescan. Same basic problem. I removed the library from Music Assistant, and readded it. Same basic problem- far more album/song info, but Artist thumbnails almost non-existent. If I select the artist, typically I see that graphic load, and often, if I return the artists listing, the image for the artist is present. But often, it will disappear on closing and starting a new session. Its hard to understand how this can be, if the issue is simply extracting an image and linking that filename in the database. So, seems like either the link is bad, or the database entry wasnt made and the process was a local thing for the session, or perhaps the artist image isnt stored within an entity (as an album cover would be for album or song), and it just isnt being handled well.
In the little refresh all circular arrow thing, I see a red dot (which I believe means a change has been detected, so a scan is needed), but hitting that button does not seem to do anything- the red dot disappears briefly (1 sec) and returns. If I rapidly click 3-4 times as if there were 4 updates in queue) which presumably should be handled by a single refresh, the dot goes away and the circle goes dark. No idea if this is a workaround, or if I did something that is causing re-entrance etc.
How to reproduce
As explained, simply load a new library. Most Artists will not have images. And while selecting the artist can return that image, and sometimes it will persist across sessions, for me, about 80% of the time the system reverts to no artist image.
Music Providers
Local (NAS) share
Player Providers
na
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2025.5.5
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

This is a known problem that is being investigated.

Do these artists have local artwork in the media folders ?
Music Assistant will only pickup local artwork if its in the correct directory structure.
So for example media_folder\Queen\Greatest Hits\track1.f;lac
The artist artwork (folder.jpeg, logo, fanart) will be in the Queen folder.
The album artwork (thumb, discart) will be in the album folder

I believe this is the same problem as we have spoken about before whereby for some specific artists the fetched artwork routinely disappears. I don’t believe I have seen it with artists that have local artwork. The text describing the artist also seems to disappear as well,

Hi Marcel et al-
So far as I can tell, none of my catalog have root content (metadata). So, no group image, no group xml/text. MusicBrainz doesnt seem to capture generic data like this- it seems to be limited to specific albums. And there isnt a correlation between the images posted to the subfolders (albums) and the group image; typically, the group photo is not seen anywhere within the directory. Presumably, this is scraped, but not stored. What I find particularly strange is that for my library, with 550+ artists, I tend to see artist images/thumbnails for only those artists I have accessed. Example, I have AC/DC as an artist- with am artist thumbnail that shows immediately on listing artists- where for others- say, Aretha Franklin- I dont. If I access the Aretha folder, I see her photo and the text associated with the artists- same as with AC/DC. But, on returning the main window, even restarting the platform, it reverts- where I never see Aretha as a thumbnail, and AC/DC is always present. In neither case do I see metadata in the artists' folders. So, it seems as if either some is persisted locally in a db or separate file system, OR there is a link to external reference that is not just fast, but is persistent in some way that most are not. I dont know enough about the code to speculate- I could imagine any number of causes, from media type to supplier URL format etc. What I dont understand is- why is the system just no persisting artist info in the artist folder root? Apparently, there is some legal thing with MusicBrainz and copyrights from sources, but apparently they won their lawsuit on this- so not clear why MB isnt capturing this data itself now. But just super strange Im seeing the same 2-5% of artists show thumbnails, and the rest the generic 'disc' icon.
If there is another scraper I should try for this- happy to accommodate, but it remains unclear to me why some artists show content in the overview, and others dont, where there seeming is never metadata in the artists' root folders...

Don’t put too much mental effort into trying to understand it as it is a bug that is difficult to track down. In my case it is the same artists that drop the images. So I have a lot that persist fine and then a number that continue to disappear. So it has been tricky to nail down why that is

Understood. Simply trying to understand why the design decision was made to leverage local data first to save bandwidth/hits on source sites, but that this data isnt persisted as local content within the exposed file system. Since that seems to work for MusicBrainz content, it seems it might make sense to just persist that data (which is clearly being grabbed and presented already) in the same way, where if the internal mechanism (db presumably) returns no data, and the fallback is local content- at least behavior is consistent. I understand that search on metadata isnt possible if it is file/text-based, but so far as I know, that search capability doesnt exist regardless- and it is limited to file structure content (folder and file names).
Happy to help with testing- as always-
Good luck with the effort- hopefully it is something simple- an unsupported character in the text, or a blob size limitation, or some weird inherited permission limitation.

You need other scrapers to save the artwork locally in your folders.
Music Assistant always prefers local artwork and metadata but it has to exist. If it doesn't exist, it will query the metadata providers and it will only do that on (first) access. It will not do that proactively as we simply do not have the money to pay for such heavy api access.
There is however a feature request to automatically save any local artwork in the folders so maybe that will one day see the light. Until then my recommendation is to use a a tool like Lidarr or alikes if you want to have all that artwork and metadata saved to your music folders.
That said; focus back on the bug report.
Both you and @OzGav mention that you experience artwork dissapearing. So it has been scraped (and visible in the UI) and then all of a sudden its gone again. Actually this morning I fixed a bug I accidentally ran into that may explain this behavior. So I'd like to ask if this issue still manifests itself in the next version; We will provide a 2.0.7 patch release later today or tomorrow.

OK, Ill keep my eyes posted for a new revision. Best- R

This might have been fixed in beta 14 @Extrapilot1 please advise if you still see this

Hi. Im reinstalling; I had a bizarre problem where a recovery from the last full backup for the test was a disaster (post-recovery, database was offline- etc). Just too many weirdnesses to feel comfortable with feedback for you. So, rolled back to an earlier instance, where I dont have the database populated with MA info. Beta 14 was just installed, and it is scraping the NAS share. So far, it seems to be doing well- slow relative to something like Volumio, but I dont know if that is an HA overhead thing (slow callbacks on DB writes etc) or if there is other stuff going on (I had assumed for example that MA was searching the online indexes to flesh out/pre-populate artist info, or perhaps building a cross-referenced database on things like players credited on albums- as we see with actors vs movies in Kodi etc). But yea, may just be a single threaded thing on RPI4 and just a lot of lag on callbacks vs a native OS tweak like Volumio?
More info tomorrow I hope!

I'm about prepared to call it as I would see this all the time and I have been trying to break it but all images are staying put.

This one should be fixed with b14 but please test it extensively.
@Extrapilot1 initial scan is indeed a bit slow, on purpose actually, to not completely overload the system with requests

Hi Marcel- the problem I was seeing with artist images being intermittent in the summary list seems to be resolved. I assume this is purely a code thing, and not a reinstall that I did- but I guess it is always possible there is some install script problem or a weird permissions thing that might prevent some file from being updated...
There is a different behavior- which may be related- that I would mention. Where in the past the system tended to pull artist image/summary data for an artist that had neither when that artist was selected, that no longer happens. And that may be a function of throttling, where perhaps you are scraping in the background at 10 per hour or whatever. But, in the past, I never saw that happen- it simply never scraped artist data unless you selected that artist manually- at which point, one some timeout, it grabbed the data from whatever source. But if I select the manual option 'refresh item' from the triple-dot menu, it tends to pull both the image and the description text immediately.
Additionally- previously, it seemed to show a red dot on the library icon in the artist summary view- suggesting the engine knew the record was incomplete. I dont see that anymore.
Best
R

The red dot on the library icon tells you the list is being filtered to show in library items only.
The behaviour of pulling data when an item is visitied has been changed. You have to either do the refresh item or wait for it to do done in the background. However, I am looking at a possible bug where this background task is not completing. In terms of this report though it has ben resolved.

Strange on the red dot. I found that selecting the icon when it was in that mode caused content to update, or a scrape to happen for the artist info, when it didnt otherwise.
Regarding the background scraping, is there any mechanism by which users can easily determine status of the process? It would be great to have something like Volumio- where in the Scan/Rescan subsection- there will be a count as sort of 135/500, incrementing towards 500. And an addition of some new artist or album will likewise be represented as some new ratio (i.e. x/506). It is possible to at least visit that subscreen manually on some schedule, but also- its a good indicator if the process has halted or is unable to process some number of entries. I believe there is a secondary indicator (a spinning icon) on the main screen showing that content scanning is in process.

You can raise a feature request for that


Error when using optional parameters

If I use a JsonRpc method with optional parameters I get an error when only the second or third optional parameter is used. So it matters in which order the optional parameters are declared in the method.
I already provided a solution with my pull request #108
Would be great if you have a look :)

Looking.


Update to ASP.NET Core 3.1 [continued]

Sorry to have messed you about @JonCanning, enough time has passed since your PR was initially opened that it now makes sense to just target netcoreapp3.1, as you had done originally.
I've given up on compile-time compilation as well, as it's not as simple as I first expected.

@slang25 same here, I think we need to target develop branch

I'm unsure why the CI builds aren't reporting. Doesn't seem like I can manually kick them either.
Probably need @dustinmoris god mode powers again.

The travis webhook was inactive, I've switched it off as a required check for now. @dustinmoris I presume we want to get this over to GitHub workflows too at some point, are we happy without travis for now?


Add a table for supported diseases (db)

Background
As part of the database work for the multiplex effort, we need a way to track which diseases SimpleReport supports.
See the engineering doc for Multiplex.
Action Requested
Add a new table to the database for supported diseases. This table should have the following data stored in it:

internal UUID
column for disease name (as LOINC code)
jsonb column for potential result (also as LOINC codes)

Once the table has been created, we'll need to input COVID-19 and Flu A/Flu B as our supported diseases.
LIVD Table (You'll want to reference the LOINC mapping in the spreadsheet)
Acceptance Criteria

[ ] New disease table has been added to the database
[ ] Table contains COVID-19, Flu A, and Flu B metadata

It may also be worth using a join table here for the potential results (which as of now will just the LOINCs for POSITIVE/NEGATIVE/INCONCLUSIVE)


add Falsehoods about email APIs

Add this article by my team at Flute: https://www.flutemail.com/blog/2018/06/06/falsehoods-programmers-believe-about-email-apis.html

Looks mostly like advertising.

Thanks @ibnIrshad for the contribution but unfortunately @silsha is right, there is way too much items that are applicable to your product and your product only to be considered as generic falsehood article.


added missing tag in the local runner setup doc

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[X] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-545

Preview environment: https://ci.pr.harness.io/pr-545


stats initial value

Hi @LKedward. In the following code I'd expect the output num_items : 0
program main
use fhash
implicit none

type(fhash_tbl_t) :: tbl
integer :: num_buckets, num_items, num_collisions, max_depth

call tbl%stats(num_buckets, num_items, num_collisions, max_depth)
print *, 'num_items:', num_items

end program main

The result is a big number:

This happens on Windows 10.

num_items = 0

at line 7 solves the problem. 😇

Hi @brocolis , thanks for reporting. This is a bug because you shouldn't need to initialise num_items before calling tbl%stats. I'll start on a fix.

Thanks for finding this @brocolis, I've now fixed it in the latest version so you shouldn't need to add num_items=0 anymore.
I think with fpm you can run fpm update to update your fhash dependency.


Parameters Must Be Referenced - shows errors for parameters that aren't present and incorrectly flags others in nested templates

we seem to combine params from multiple  templates (or the output) but the params we flag are indeed used.  So some are false positives and some are not even defined in the template they are flagged in.
https://github.com/Azure/azure-quickstart-templates/blob/master/quickstarts/microsoft.machinelearningservices/machine-learning-end-to-end-secure/azuredeploy.json

s/b fixed in #589


How to get list file to build scss in watchSass !!!???

I want go get list main scss file to build css file from watchSass (file .scss not _.scss)
I try consolog from callback in watchSass but it only return file i'm editing !

I don't understand your point. Have you looked at the Usage section? What's not working?
Also, one question mark would probably have sufficed in the question title.

Closing as unclear and unanswered.


Setting width in mj-column adds left and right padding

Describe the bug
When mj-column  has width defined it seems to add left and right padding. I would have expected it to still take up the entire body size.
Here is with width="25%" on first column. Notice the extra left and right white space.

Without width="25%". Notice less left and right white space.

To Reproduce
Use MJML app. See above for reproduction. The rendered result in email client is same.
Expected behavior
The full mj-body or outer mj-section is taken up by the columns regardless if width is set or not. With width="25%" set, it should still align as left as possible taking up 25% of space. While the other column takes up rest of the space. The total available space should not have shrunk.
MJML environment (please complete the following information):

OS: MacOS
MJML Version: 4.12.0
MJML tool used: MJML App

Email sending environment(for rendering issues):
N/A.
Affected email clients (for rendering issues):
N/A

This is not something we can really manage here, this behavior come from default value of width for column.

Here you have a 25% column + a 50%, extra 25% missing is just controlled by the text-align on mj-section


Add result_set for pagination to token holders API

Close: #560

Changed destination branch from dev-23.12 to dev-24.3


Fixing issues of StreamGraph

Issues #2319 and #2320:
-Invalid negative value for
-When "Category" field is cleared, visual still shows previous state

Reviewed, looks good.


Give us an update

Please, it’s been a week and no updated info about certificates?

Hi! @MineTurtlee all of the Certitficates accessible on my website are all up to date and I add new ones as soon as they come out. I'm sorry that there isnt currently any new ones at the moment.

Hi! @MineTurtlee all of the Certitficates accessible on my website are all up to date and I add new ones as soon as they come out. I'm sorry that there isnt currently any new ones at the moment.

Hey! @loyahdev how to do FluxSign?

Hi! @MineTurtlee all of the Certitficates accessible on my website are all up to date and I add new ones as soon as they come out. I'm sorry that there isnt currently any new ones at the moment.

Hey! @loyahdev how to do FluxSign?

Hello! FluxSign is a dead project I had as an iPA Signer and I've removed its references from this repository.


LLamaToken Struct

This is a major change to the API. All uses of int (or llama_token, aliasing int) to represent a token have been replaced with a LLamaToken struct. This makes the API more strongly typed - it's less likely to mix up a token with some other non-token integer.
In general fixing any breakage is easy, you can just cast (int)the_token or (LLamaToken)1. Not many places in the basic API deal in raw tokens, and usually work on strings instead, so I'm actually not expecting this to seriously affect many users.

I'd like to get this one reviewed soon if possible, it touches a lot of files and so it will rapidly become a pain to merge if other PRs are merged.

Sorry didn't see this request,
changes look fine to me :) however I don't know the lower levels that well


Added Euncet.txt From Euncet Business School www.euncet.es

Added Euncet.txt from Euncet Business School www.euncet.es
This pull request it's a reply to @zweizwei on https://github.com/JetBrains/swot/pull/1266
Thanks!

Thanks, i've added this one

Thanks Elena!
El 16 oct 2015, a les 14:43, Elena Ganenkova<EMAIL_ADDRESS>va escriure:

Thanks, i've added this one
—
Reply to this email directly or view it on GitHub.


Update common test key path

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test


As an EVCM previewing the tool, I want to zoom in to the state and county level

Description
This user story has to do with the zoom level of the map interaction for the initial CEJST prototype for user research. We don't need it to be census block level of resolution for the visual map zooming, but the CEJST previewer needs to at least zoom 3 levels in from national level.
Solution
Zoom button or double-click on map interaction that allows CEJST visitor to zoom approximately from national level to state level or region level (approximately) > state regional level > city or county level. There's a lot of latitude the eng team should have for this, and whatever the simplest implementation that gets roughly at this is fine for the initial version.
As the user zooms, labels and landmarks (state borders, etc) should be viewable to orient the user
If supporting this for every city/county in the country in the near term is too challenging, we can strategically pick a handful of regions/cities/counties to implement this with
Describe alternatives you've considered
Static visual mockup of different zoom levels - those might give a good idea to the previewer what the final site might look like, but would not give us a sense of how they would interact. For instance, what if many EVCMs want to zoom in and out to look at neighboring counties, etc.
Enter in a zip code and maps zooms in on that zip code such that there's only one zoom level. We need to discuss that type of interaction more and it would likely need its own user story.
Links to user research or other resources
User journey board https://app.mural.co/t/kameronkerger2998/m/kameronkerger2998/1619097401497/d8d8b090c94770d1d72c9c250db564b37676cc69
Definition of "Done"
Ability to zoom in three levels across the map to at least a strategic set of cities/counties to enable effective user research

This is out of date now, closing. We agree and have separate tickets around the July 27th version of the tool zooming to CBG level 9


Goroutine and method

I have a structure with field of type *bufio.Reader
Before start goroutine I initialize this field. After, I start goroutine by "go myStruct.readLoop()" where I use *bufio.Reader. Access violation appears

Let's move this discussion to the golang-nuts mailing list.  We only use Go's issue tracker to track Go bugs and TODO items.


Add environment variable import for API token and example Qconfig to stable

Hi - in reference to the discussion on #74 I've added the following changes to the stable branch for review:

Added alternative configuration to the README explaining how to use os for importing an environment variable from the shell session.
Added an example_Qconfig.py to the introduction folder that can be customized. It will raise an exception if the user does not set an environment variable or has an unset config dictionary.
Added my name alphabetically to the contributors list.

Thanks for considering this!

Hi,
happy new year!
Thank you for the suggestion. I think we better not modify the Qconfig.py. One way of doing this is to check for the environment variables. If we can find the key, then we do not import Qconfig.py. Otherwise, we proceed to import it and proceed as usual. This way, we can make sure we still can work with those using the Qconfig.py and the environment variables.
Perhaps something like:
APItoken = os.getenv("IBMQE_API")  # change to match your variable name if needed
config = {"url": "https://quantumexperience.ng.bluemix.net/api"}

if APItoken is None:
    import Qconfig
    APItoken = Qconfig.APItoken

Happy New Year! Maybe I misunderstood the intention, can you help clarify this for me?
When I look through the tutorials using GitHub Search I only see Qconfig used as an import for the API Token and the Config url. In the SDK docs the Qconfig files example has more values in the config but same core structure. I saw you have Qconfig.py in the .gitignore for the tutorials as well to keep it from syncing to GitHub. This works if I've forked the repo and gotten the .gitignore, or know to include it on my own as an ignored file, but if I've started a new repo I might want the Qconfig file to be included (or just forget).
The modification I'm suggesting for the Qconfig.py file allows the users to keep their Qconfig in their repo without any sensitive data like the API Token. The change to check APIToken is None is because os.getenv() will return None if the environment variable isn't set, so that variable name will still be in locals() and the prior check would pass. Modifying Qconfig.py directly means that all of your tutorial notebooks would continue to work without modification to make use of the environment variable.
In the above example: importing Qconfig and setting the Qconfig.APItoken value if the environment variable is not present, implies that I've got the API Token in plaintext in the local directory somewhere, which doesn't address the concern.
If there's a concern with the modification in code maybe we're better off with just text e.g. "as an alternative, you can set your API Token as an environment variable and import it with os." and also add a warning to the user to add "Qconfig.py" to their .gitignore. If you do a GitHub Search for "Qconfig apitoken" you'll find a few examples of publicly viewable API keys in code. There is a sensitive data cleaning mechanism, and the user could always regenerate the key, but it's better to avoid this in the first place.
Let me know what you think and I'll be happy to make any modifications. Thanks again for considering this!


Add StatusDescription to HttpResponse class

In the old days, it is easy to modify the Http response status code and description, but now on asp.net core, we need to do something like:
HttpContext.Features.Get<IHttpResponseFeature>().ReasonPhrase = "It is my description";
Consider adding StatusDescription back into HttpResponse class and its impl classes.

It was left as a second class citizen because it's not a commonly used property.
/cc @Tratcher

@RexLKW Could you elaborate on why you want this feature? Are you returning non-standard status codes? If not, why do you want to replace the standard reason phrases?

On a semi-related note, HTTP/2 will not have reason phrases.

Duplicate of https://github.com/aspnet/HttpAbstractions/issues/395


Enhance shenyu-register-center module unit test

Task
A clear and concise description of Task
Task List:

[ ] Enhance shenyu-register-client module unit test，make its coverage rate is more than 80%
[ ] Enhance shenyu-register-common module unit test，make its coverage rate is more than 80%
[ ] Enhance shenyu-register-server module unit test，make its coverage rate is more than 80%

Please read the Contribution Guideline before submitting the PR

I want Enhance  shenyu-register-client module unit test

Please assign shenyu-register-common module unit test to me.

Please assign shenyu-register-common module unit test to me.

You can send an email to me

I want Enhance shenyu-register-client module unit test

You can send an email to me

I want Enhance shenyu-register-server module unit test

can you Enhance shenyu-register-client-http ？
| |
zhaiweij
|
|
@.***
|
签名由网易邮箱大师定制
On 05/14/2021 @.***> wrote：
I want Enhance shenyu-register-client module unit test
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.

No test is required under the common module ，can you enhance shenyu-register-client-nacos?
| |
zhaiweij
|
|
@.***
|
签名由网易邮箱大师定制
On 05/14/2021 @.***> wrote：
Please assign shenyu-register-common module unit test to me.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.

can you enhance shenyu-register-server-nacos?
| |
zhaiweij
|
|
@.***
|
签名由网易邮箱大师定制
On 05/14/2021 @.***> wrote：
I want Enhance shenyu-register-server module unit test
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.

No test is required under the common module ，can you enhance shenyu-register-client-nacos? | | zhaiweij | | @.*** | 签名由网易邮箱大师定制 On 05/14/2021 @.***> wrote： Please assign shenyu-register-common module unit test to me. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or unsubscribe.

Yes. i can. shenyu-register-client-nacos also need enhance?

No test is required under the common module ，can you enhance shenyu-register-client-nacos? | | zhaiweij | | @.*** | 签名由网易邮箱大师定制 On 05/14/2021 @.***> wrote： Please assign shenyu-register-common module unit test to me. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or unsubscribe.

Yes. i can. shenyu-register-client-nacos also need enhance?

yes，the method is 66% 4/6

can you enhance shenyu-register-server-nacos? | | zhaiweij | | @.*** | 签名由网易邮箱大师定制 On 05/14/2021 @.***> wrote： I want Enhance shenyu-register-server module unit test — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or unsubscribe.

ok, i will do it

No test is required under the common module ，can you enhance shenyu-register-client-nacos? | | zhaiweij | | @.*** | 签名由网易邮箱大师定制 On 05/14/2021 @.***> wrote： Please assign shenyu-register-common module unit test to me. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or unsubscribe.

Yes. i can. shenyu-register-client-nacos also need enhance?

yes，the method is 66% 4/6

Ok.Let me do it.


Update copy-pictures-to-folders-by-date-taken-with-powershell.md

"If you grab my file and just type OrgPhotos.ps1 into your PS prompt and it doesn’t work, this link will probably tell you why." line 25 <- is this referring to the link above or should there be a link in this sentence?

Refers to link above.


KIP-101: Alter Replication Protocol to use Leader Epoch rather than High Watermark for Truncation

This PR describes the addition of Partition Level Leader Epochs to messages in Kafka as a mechanism for fixing some known issues in the replication protocol. Full details can be found here:
KIP-101 Reference
The key elements are:

Epochs are stamped on messages as they enter the leader.
Epochs are tracked in both leader and follower in a new checkpoint file.
A new API allows followers to retrieve the leader's latest offset for a particular epoch.
The logic for truncating the log, when a replica becomes a follower, has been moved from Partition into the ReplicaFetcherThread
When partitions are added to the ReplicaFetcherThread they are added in an initialising state. Initialising partitions request leader epochs and then truncate their logs appropriately.

This test provides a good overview of the workflow EpochDrivenReplicationProtocolAcceptanceTest.shouldFollowLeaderEpochBasicWorkflow()
The corrupted log use case is covered by the test
EpochDrivenReplicationProtocolAcceptanceTest.offsetsShouldNotGoBackwards()
Remaining work: The test EpochDrivenReplicationProtocolAcceptanceTest.shouldSurviveFastLeaderChange() doesn't correctly reproduce the underlying issue. This will be altered later to properly support this use case.

Once the Jenkins build is passing, it would be great to run the system tests via the branch builder.

@benstopford, I merged a PR that disables the test_zk_security_upgrade test until this PR lands. When you rebase, can you please remove the ignore annotation from that test? Thanks.

System tests running here: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/815/

@junrao I've addressed your second round of comments.
I still need to add some code to protect earlier versions of the message format. I'll dig into that tomorrow.

This PR was re-raised from the confluent repo: https://github.com/apache/kafka/pull/2808


Support question - idleTimeout - how does it work

If we set idleTimeout to 20 seconds -
does that mean that once the client becomes idle that the 'idle' event will always fire after 20 seconds, even if the client incurs activity during that 20 second period?
Or will the 'idle' event not fire if the client becomes un-idle during that period?
I would assume that this library works like this - every client action will call clearTimeout on any previous idleTimeouts, and then create a new timeout, with the value of idleTimeout. If no subsequent client actions occur before the most recent timeout is created, then the 'idle' event will fire. Is that right?

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


How to iterate over the list of tokens without side effects with ability to replace contents of certain tokens

I am breaking my head trying to write/ use one of the existing markdown parsers to find and replace every text paragraph in a markdown document and leave everything else untouched.
Example Input:
# A First Level Header

## Second Level Header

Now is the time for all good men to come to the aid of their country. This is just a regular paragraph.

The quick brown fox jumped over the lazy dog's back.

### Header 3

> This is a blockquote.
>
> This is the second paragraph in the blockquote.
>
> ## This is an H2 in a blockquote

Desired effect is to identify paragraphs and replace them, e.g.
# A First Level Header

## Second Level Header

[replaced]

[replaced]

### Header 3

> This is a blockquote.
>
> This is the second paragraph in the blockquote.
>
> ## This is an H2 in a blockquote

Existing libraries such as marked or markdown-it appear to be incapable of parsing and re-creating markdown document without side-effects.
What is the best approach to find and replace a text paragraph in a markdown document?
Original question: http://stackoverflow.com/q/34914559/368691

I have tried using markdown-it zero preset and writing a custom plugin that would replace the md.block paragraph rule. But this approach nevertheless produce side effects.

Technically, source manipulations via output analysis require sourcemap info. I don't know markdown parsers with full and correct souremap support.
But... line numbers are available. That's not much, but could help for paragraphs. Dig into demo source - we use line numbers there to sync panes scroll.

But... line numbers are available. That's not much, but could help for paragraphs. Dig into demo source - we use line numbers there to sync panes scroll.

Please provide a link to the referenced demo. I have looked through quite a few code examples and plugin source codes already.

https://markdown-it.github.io/ :)
And here is init we use to inject line numbers https://github.com/markdown-it/markdown-it/blob/master/support/demo_template/index.js#L178. Sorry, for messy code - it was written for demo only.

And here is init we use to inject line numbers https://github.com/markdown-it/markdown-it/blob/master/support/demo_template/index.js#L178. Sorry, for messy code - it was written for demo only.

Just a tip, but avoid using master branch when linking to the source code. Use a specific commit or a tag, e.g. https://github.com/markdown-it/markdown-it/blob/5.1.0/support/demo_template/index.js#L116. I have seen you do the former on some other GitHub issues. The links become useless as soon as the source code on the master changes.
Thank you. I will have look.

An interesting observation (and my oversight) http://stackoverflow.com/questions/34914559/detect-and-replace-text-paragraph-in-a-markdown-document?noredirect=1#comment57586910_34914559.

Thanks for advice. For root-level blocks, existing lines info should be enougth (because columns value is always zero).

closing?