Treat .mdx extension as Markdown

Thanks a lot for working on such an extensive link checking tool! I tried out lychee for a documentation site that uses MDX, a Markdown superset.
When I run lychee --base docs/ docs/**/*.mdx it does extract absolute URLs correctly, but ignores relative (internal) URLs.
I think this is because lychee detects the file type as plaintext, as the extension is neither .md nor .markdown. If I change the file extension from .mdx to .md, lychee detects all URLs as expected, including relative interal URLs.
It would be nice to either add .mdx to the hard-coded list of Markdown-ish extensions or to add a configuration option that allows to override/customize the default file type detection.
I’d be open to create a PR to implement and test the first solution. Not sure my Rust knowledge is good enough to implement a more flexible solution though.

Interesting. Yeah I agree that we should add support for it.
If we find that there are no issues with .mdx files in general I'd be okay with adding it to FileType::Markdown as another valid extension. Otherwise we could add a new FileType if needed.
Perhaps somewhat interestingly, ripgrep presently does not support .mdx either. There is an open pull request to add it, though: https://github.com/BurntSushi/ripgrep/pull/2142.
Furthermore, we should add support for overwriting filetypes. ripgrep supports this:
-t, --type TYPE ...
    Only search files matching TYPE. Multiple type flags may be provided. Use the
    --type-list flag to list all available types.

    This flag supports the special value all, which will behave as if --type was
    provided for every file type supported by ripgrep (including any custom file
    types). The end result is that --type all causes ripgrep to search in
    "whitelist" mode, where it will only search files it recognizes via its type
    definitions.

Then a user could say lychee --type md --type mdx
The ripgrep file handling is quite sophisticated. I personally think it's more than what we need.
Therefore an alternative might be to use --include-type and --exclude-type instead, which might be more fitting with the rest of the options.
Open for suggestions here.

Interesting. Yeah I agree that we should add support for it.
If we find that there are no issues with .mdx files in general I'd be okay with adding it to FileType::Markdown as another valid extension.

Any specific requirements in order to consider .mdx files working without problems? In general, MDX is a superset of Markdown that adds support for ESM imports/export as well as JSX:
// import
import { Greeting, Container } from 'package';

// export
export const foo = 'bar';

// JSX
<Greeting name="John Doe" />

https://github.com/mdx-js/specification#mdx
Most Markdown parsers I’ve come across so far would simply treat this as plaintext or HTML. Haven’t found any specific information for pulldown-cmark, but so far it has worked fine in all cases I tested.
The only relevant aspect probably is that Markdown can be nested inside of JSX elements. In these cases, lychee would need to extract  links in nested elements:
<Container>
  [This is a link](https://example.org)
</Container>

I can create a fixture that includes examples for the MDX syntax if that helps?

Thanks for all the context. From what I can tell there won't be any issues.
I'd be thankful for a pull request to add the extension as well as the test case you describe. 👍

Cool, I’ll take a look and will try top open a PR sometime next week :)

Thanks!

Went ahead and added mdx and some other common extensions. Thanks for the suggestion @tillprochaska. 😃


Standardized colors between DoAfter and Cooldown progress

Re: Issue #2278
Progress starts at red, transitions to yellow, then ends at green.

Fixes #2278 in the PR description please, so it auto-closes.


updated download-url for appmon

The URL to download the AppMon agent has changed. This PR updates it to the new one.

Thanks!


WIP: circleci firefox work

Sorry what is the point of this PR? can you add some description please?

You can view the details in the JIRA ticket
On Mon, Feb 11, 2019 at 8:34 PM Arash Koushkebaghi<EMAIL_ADDRESS>wrote:

Sorry what is the point of this PR? can you add some description please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/webex/react-ciscospark/pull/922#issuecomment-462575287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALot3IwTHVIJNmmI19U0g0zc2xXCMs7ks5vMhoRgaJpZM4an2LZ
.

what @adamweeks said. it was a WIP to test the circleci builds.


Added an optional absolute margin to the approximation checks

The Approx class now has two options:

The existing epsilon to set the tolerated relative error
The new margin to set the tolerated absolute error

They can be combined together.
This is the same as #538, but that one was closed accidentally.

This has been merged to master, but git doesn't see that.


Add Impling rolls to Xmas activity

🚀 Feature Proposal
Surely our minion finds some implings as they travel around Gielinor, stealing or delivering presents. Currently they don't bother catching any, but it would be a neat thing to add and wouldn't hurt anyone!

Don't want to do this


Improve working with entities related to entities that uses compound ids

Fixes an issue where adding entities related to entities with compound id (see makeIdConnect)
Fixes validation of entities with compound id, given that zod schemas expect the attributes separately
This is not super elegant, happy to get suggestions on improvements
Update openAPI spec generation to match the previous point, making generate clients behave as expected

Hi @thomassnielsen , I've changed to use the "scalar" schema to validate which doesn't include FK fields. Also had to make a fix in the zod generation part to make sure ID fields are always included in the schema even they're also FK fields. May different corner combinations 😄.
Please take a look, thanks!

Nice! I'm not familiar with the zod generator, but this looks reasonable to me 👍

Nice! I'm not familiar with the zod generator, but this looks reasonable to me 👍

Cool. I'm merging it now.


Test Explorer doesn't support tests with TestCaseAttribute

Describe the bug

Test explorer doesn't work for NUnit tests that use TestCaseAttribute.

Test builds and runs (.NET process running test ends)
VSCode spinning icon does not go away until test is cancelled

Steps to reproduce

Create a new NUnit test project (dotnet new nunit -lang F#)
Open directory in Visual Studio Code
In UnitTest1, replace [<Test>] with [<TestCase("test")]
Select Run test in Test Explorer, or ▶ in gutter

Expected behaviour

VSCode should correctly show the status of the test (whether succeeded or failed). Similarly, the Test Explorer should break down those test cases, rather than just showing one test.
Screenshots

Machine info

OS: Windows
.NET SDK version: 6.0.302
Ionide version: 7.0.0

Frustratingly, I couldn't find anything in the logs that was logged as Debug in TestExplorer.fs, otherwise I might have found it easier to find out what parsing of tests/test results is failing. I would like to take a stab at trying to fix this, just depends on time (and whether I get distracted by other projects).

Ah, taking a look at Ionide.trx after running dotnet test <project>.fsproj --no-restore --logger:"trx;LogFileName=Ionide.trx" --noLogo gives me much better idea of what's going wrong.

Taking a deeper look, I don't fully understand where Ionide gets the tests method names, but for test cases, the .trx file has <method-name>("<test-case>") which doesn't sync up.
dotnet test --list-tests doesn't print out the namespace/class for the test for some reason. I see .NET Test Explorer uses vstest, but dotnet test is meant to replace it, and the output isn't exactly fun to parse.
The simplest solution would be to replace the tests in the Test Explorer tree view with the test cases once the .trx file has been parsed.

Turning out to be a lot more complicated than I thought:
TestCase takes any number of arguments. The following is example with 3:
[<TestCase("arg1", 2, 3L)>]
let Test1 (arg1, arg2, arg3) =

Leads to the following name in the .trx file, which will be fun to parse: Test1<String,Int32,Int64>("arg1",2,3L).
Looking in FsAutoComplete, it feels like this will require quite a bit of rejigging to support test cases (and I don't know how XUnit or Expecto handle similar scenarios).

I'm getting this behavior when using TestAttribute and not TestCaseAttribute; there's a rejected promise five stack frames into internal Fable stuff that appears to be from TestExplorer.fs:144:
let executionId = Map.find t.FullName testDefinitions

where e.g. t.FullName is PoudriereC2.Tests.MediaType.MediaTypeHandlerTests.TestGoodValues and the actual map key is PoudriereC2.Tests.MediaType+MediaTypeHandlerTests.TestGoodValues.
Is there a way to set breakpoints and such in the F# rather than the generated JavaScript?

There is, and at one point it was working, but our sourcemaps are broken or something. I've been wanting to move us to esbuild and off of webpack partially because in my quick tests sourcemaps just worked, which is a HUGE step up from the current state.

Oh! regarding the + in the computed name - that's what a type name for a type inside a module looks like - I believe the format is <NAMESPACE>+<MODULE>.<TYPE>. So we may have something off in our detection/calculation of module-embedded vs namespace-level tests?

The first three examples below match Chet's assessment (I wonder if there is a limit to module nesting), fourth example is a module with the same name as a class in a namespace:
module Outer =
    [<TestFixture>]
    module Inner =
        [<Test>]
        let Test1 () =
            Assert.Pass ()

        module Innermost =
            [<Test>]
            let Test2 () =
                Assert.Pass()

module Outer2 =
    [<TestFixture>]
    type InnerClass() =
        [<Test>]
        member this.Test1 () =
            Assert.Pass()

type SharedName() =
    do ()

[<TestFixture>]
module SharedName =
    [<Test>]
    let Test1 () =
        Assert.Pass()

Test Class Name
FullName in Code

TestNameSpace.Outer+Inner.Test1
TestNameSpace.Outer.Inner.Test1

TestNameSpace.Outer+Inner+Innermost.Test2
TestNameSpace.Outer.Inner.Innermost.Test2

TestNameSpace.Outer2+InnerClass.Test1
TestNameSpace.Outer2.InnerClass.Test1

TestNameSpace.SharedNameModule.Test1
TestNameSpace.SharedName

Last one seems more difficult to solve.

Reopening to track the open case that @kojo12228 reported here.

I think this requires a change in FsAutoComplete, the logic (at least for NUnit) does not distinguish between a namespace and a module:
https://github.com/fsharp/FsAutoComplete/blob/f33887be9393087b05875f5c76334f557f0ed7be/src/FsAutoComplete.Core/TestAdapter.fs#L328-L347
The simplest change might just be adding SynModuleOrNamespaceKind or something like it onto the DTO. Then getFullName could be made aware of it.
https://github.com/ionide/ionide-vscode-fsharp/blob/1139128bb9a67fe8067b4930b044e3dccd821729/src/Components/TestExplorer.fs#L249-L252
That solves the first three cases. For the last case, I think TestAdapter needs to special case where SynTypeDefn and NestedModule are siblings in the AST with the same name, which shouldn't be too hard to do I would hope.
AST from Fantomas was incredibly helpful in trying to understand this:
https://fsprojects.github.io/fantomas-tools/#/ast?data=N4KABGBEAmCmBmBLAdrAzpAXFSAacUiaAYmolmPAIYA2as+EkAxgPZwWTJUC26ADlWawwAFXQAXAHK9YAZUHCAdHICuAIzQSATquYTV22AB1tx5KfM92qmiIDyqibG1gAvJeQQIAbQA84lrEiAAeBkYAfAC6nt5g1tC2IgCSyKiuHmZecd7+gRLRsTkQdhJikgCMYAAUAJTuRcVxAIJo9NoSSgAKVG01tZ6eCUlgjs7aAEwNWXF5ksFhhrCFM94SAJ78KWkuAMI0vWh10+ZNYHNaK6dNfDzqLmASABZESvlVx5nXZxCt7Z09Np1QZZDZbMByJ5UIzQGR8T6NMDQVj9EHmC4SBbhZYxLLDOwQqEwuEiL7ZXwBSRXclgUrlLQfepks5-FwAw51SAgAC+QA

This might, or might not, be useful but I have also come up against this issue in a slightly different way.
Sometimes, when I create and run some unit tests with Unquote on xUnit, the tests run successfully but the Testing Panel doesn’t always show the results.
In the examples below, “works” means that the tests run and complete as expected and the results are shown in the Testing Panel, and “doesn’t work” means that the spinners against each test keep spinning and the process keeps running until it’s manually cancelled even though the tests have completed according to the Ionide.trx file in the TestResults folder.
This works:
module Tests

open System
open Xunit
open Swensen.Unquote

[<Fact>]
let ``My test`` () =
    test <@ true @>

But this doesn’t work:
module Tests

open System
open Xunit
open Swensen.Unquote

module ``with my module`` =

    [<Fact>]
    let ``My test`` () =
        test <@ true @>

After changing ‘module Tests’ to ‘namespace MyUnitTests’, this works:
namespace MyUnitTests

open System
open Xunit
open Swensen.Unquote

module ``with my module`` =

    [<Fact>]
    let ``My test`` () =
        test <@ true @>

But this doesn’t work:
namespace MyUnitTests

open System
open Xunit
open Swensen.Unquote

module ``with my module`` =

    module ``and this function`` =

        [<Fact>]
        let ``My test`` () =
            test <@ true @>

I have been told that the same problem occurs when using Nunit also.
Ionide for F# version 7.5.1
VS Code version 1.76.0


AbstractTokenGranter didn't validate client_secret when grant_type is  client_credentials

org.springframework.security.oauth2.provider.token
class : AbstractTokenGranter
public OAuth2AccessToken grant(String grantType, TokenRequest tokenRequest) {
	if (!this.grantType.equals(grantType)) {
		return null;
	}

	String clientId = tokenRequest.getClientId();

//secret can pass any value??
	ClientDetails client = clientDetailsService.loadClientByClientId(clientId);
	validateGrantType(grantType, client);

	logger.debug("Getting access token for: " + clientId);

	return getAccessToken(client, tokenRequest);

}

The BasicAuthenticationFilter authenticates the client credentials (client id/secret) via the Authorization Header before it proceeds to the TokenEndpoint (/oauth/token). The TokenEndpoint internally uses the TokenGranter for generating the Access Token.
NOTE: The TokenGranter is not meant to be used standalone. As per spec, client credentials must be authenticated first before it proceeds to the TokenEndpoint.
I'm going to close this issue as it's working as designed.


Finance Calculator - Expense Tracker

Enter calculator name -
Expense Tracker
Describe calculator functionality -
I was looking into the list of current calculators and found that expense tracker is incomplete. I would like to work on it.
Under which category you wish to place it -

[] Computer Science
[] Mathematics
[] Physics
[] Chemistry
[] Biology
[x] Finance
[] Scientific
[] Health
[] Sports
[] Let project owner decide
[] Suggest a new category - (Answer here)

@Bhavyal9,
Expense tracker is under progress as under #68, due to some breaking changes it has been reverted but will be re-worked by the participant.
Closing this issue.
Hope to see you coming up with more innovative ideas. 🤩
Happy Contributing!!🎉


Add opactity setting for slides with the clone class

looking for a way to set all slides that are not active to be opaque and then a nice fade in transition on them... so far this sass causes a weird jump when the carousel is finished and about to start again... also removed the overflow:hidden on the main container.
.slick-slide {
    transition: opacity 400ms ease-in;
    opacity: 0;
    &.slick-active {
        opacity: 1;
        transition: opacity 400ms ease-in;
    }
}

+1


Feature thought: Add generic imputation

Hi Eric,
Just a thought -- I find myself doing a lot of hand imputation. It would be nice if you could add imputation as a chain able function.
I'm under the gun and can't submit a PR, but I think this would be a great feature.

A bit of an update. This doesn't handle variable statistics, or textual data, but it's getting there. I've been using it generically for the past weekend, or so. It's not the quickest thing (millions of rows with multiple columns takes 20 minutes or so).
def stratified_impute(df, mapping, columns):
    """
    Perform a stratified impute match. Note, mapping cannot contain columns.
    Method chaining usage:

    .. code-block:: python
        df = (
            pd.DataFrame(...)
            .stratified_impute(mapping=['gender', 'race','ethnicity','disease'], columns=['pack_year'])
        )

    :param df: pandas DataFrame.
    :param mapping: Column(s) on which to map stratified impute.
    :param columns: Column(s) on which to perform stratified imputation.
    """

    if set(columns).issubset(mapping):
        raise ValueError("{} must not include {}".format(mapping, columns))

    filtered_groupby = df.groupby(mapping)
    strat_dict = dict()
    for column in columns:
        for name, group in filtered_groupby:
            group = group.dropna()
            strat_dict.update(
                {str(column + str(name)): group[column].mean()}
            )

    group_nan_list = []
    group_nonnan_list = []
    for column in columns:
        for name, group in filtered_groupby:
            # nan_data to be filled
            nan_data = group[pd.isna(group[column])]

            # replace nan_data with backfilled data from dict
            nan_data = nan_data.fillna({column:strat_dict.get(str(column + str(name)))})
            group_nan_list.append(nan_data)

            # non nan_data to be passed along
            nonnan_data = group[~pd.isna(group[column])]
            group_nonnan_list.append(nonnan_data)

    group_nan_list.extend(group_nonnan_list)
    df = pd.concat(group_nan_list)

    return df


Shadow org.slf4j:slf4j-api and net.sf.jopt-simple:jopt-simple

Issue
Including com.github.tomakehurst:wiremock-standalone creates conflicts with org.slf4j:slf4j-api and net.sf.jopt-simple:jopt-simple in my project. The same happens for com.github.tomakehurst:wiremock-jre8-standalone
I use the 2.27.2 version.
Please add shadowing for these two libraries as well.
The output of org.basepom.maven:duplicate-finder-maven-plugin plugin:
[WARNING] Found duplicate and different classes in [com.github.tomakehurst:wiremock-jre8-standalone:2.27.1, org.slf4j:slf4j-api:1.7.12]:
[WARNING]   org.slf4j.ILoggerFactory
[WARNING]   org.slf4j.IMarkerFactory
[WARNING]   org.slf4j.Logger
[WARNING]   org.slf4j.LoggerFactory
[WARNING]   org.slf4j.MDC
[WARNING]   org.slf4j.Marker
[WARNING]   org.slf4j.MarkerFactory
[WARNING]   org.slf4j.helpers.BasicMDCAdapter
[WARNING]   org.slf4j.helpers.BasicMarker
[WARNING]   org.slf4j.helpers.BasicMarkerFactory
[WARNING]   org.slf4j.helpers.FormattingTuple
[WARNING]   org.slf4j.helpers.MarkerIgnoringBase
[WARNING]   org.slf4j.helpers.MessageFormatter
[WARNING]   org.slf4j.helpers.NOPLogger
[WARNING]   org.slf4j.helpers.NOPLoggerFactory
[WARNING]   org.slf4j.helpers.NOPMDCAdapter
[WARNING]   org.slf4j.helpers.NamedLoggerBase
[WARNING]   org.slf4j.helpers.SubstituteLogger
[WARNING]   org.slf4j.helpers.SubstituteLoggerFactory
[WARNING]   org.slf4j.helpers.Util
[WARNING]   org.slf4j.spi.LocationAwareLogger
[WARNING]   org.slf4j.spi.LoggerFactoryBinder
[WARNING]   org.slf4j.spi.MDCAdapter
[WARNING]   org.slf4j.spi.MarkerFactoryBinder

Unfortunately slf4j can't be relocated as this would prevent it working when the standalone JAR is used as a dependency.
I seem to remember jopt-simple breaks when you relocate it too, although I've not tested this with the latest version.

Unfortunately slf4j can't be relocated as this would prevent it working when the standalone JAR is used as a dependency.
I seem to remember jopt-simple breaks when you relocate it too, although I've not tested this with the latest version.

Ok, got it! Thanks! I think I, fortunately, solved my issue using the not-standalone library. I have too many dependencies conflicting with your project and with spring-cloud-contract-stub-runner that is fatJar as well. I prepared my own "stub-runner".

Ok, got it! Thanks! I think I, fortunately, solved my issue using the not-standalone library. I have too many dependencies conflicting with your project and with spring-cloud-contract-stub-runner that is fatJar as well. I prepared my own "stub-runner".


Enable Secondary Table rows

If we can move some column fields to display as secondary rows to save space with very wide tables this will help reduce horizontal scrolling in drill down and split view dialogs.
Suggest adding 'secondaryField' to list of accepted properties.

Also look at enabling click button for the row for deep drill downs.


ENH: Update displaymath to use Latex tags rather than html table wrappers

This PR changes the implementation to use \tag{} for displaymath with id labels instead of html tables. This increases compatibility with jupyterlab and makes the notebook more readable.
thanks @jlperla for the suggested solution.
Currently it looks like  references don't work in jupyterlab.
Image size issues are also resolved as all math is in the same environment.

[ ] use quantecon.build.lectures as a test case for this PR before merging

this is broken in linear_algebra on the python side. So need to take a closer look tomorrow before merging this.

It appears nbconvert is not honoring the $$ environment for displaymath reliably. The html has injected html items within the math. The source for this bug is:
$$

A x¶

\left[
\begin{array}{ccc}
    a_{11} &  \cdots & a_{1k} \\
    \vdots & \vdots  & \vdots \\
    a_{n1} &  \cdots & a_{nk}
\end{array}
\right]
\left[
\begin{array}{c}
    x_{1}  \\
    \vdots  \\
    x_{k}
\end{array}
\right]
:=
\left[
\begin{array}{c}
    a_{11} x_1 + \cdots + a_{1k} x_k \\
    \vdots \\
    a_{n1} x_1 + \cdots + a_{nk} x_k
\end{array}
\right] \tag{2}
$$

this is an issue with nbconvert and not our template.

There is an issue already registered with nbconvert.
https://github.com/jupyter/nbconvert/issues/404
Will try and massage the LaTeX to get it to work.

OK so the issue is the = on its own line. This is caught by nbconvert as markdown and breaks the equation environment. Given I am not sure how many edge cases there are here -- I am a bit reluctant to start editing our equations to fit around the nbconvert issue until the treatment of $$ upstream is improved. @jlperla perhaps we should stick with wrapped equations in html for now -- what do you think?

Is the = on its own line the only thing we know that triggers this?  If so, then that seems to be a regex we could easily check for in and fix across the lecture notes?

From the issue on nbonvert -- open lists [ also seem to break it. @jlperla do you had an RA that could check edge cases across the jl side I would be happy to coordinate updating on the py side. Its a bit tricky as they would need to diagnose on the IPYNB -> HTML border but they could do this on an individual lecture basis (without the template wrapper) using `jupyter nbconvert --to html file.ipynb``

Decision: Modify RST files to suite buggy nbconvert


add --amazonec2-keypair-name flag, fix #661

Signed-off-by: Viktor Stanchev<EMAIL_ADDRESS>

Have you tried master?  https://github.com/docker/machine/pull/3362 was intended to solve this issue

My problem wasn't that keys don't get deleted sometimes. My bigger problem was that, if they don't, you can't create the machine again. I was also extremely annoyed that our AWS account has hundreds of ssh keys for no reason because of docker machine.

+1

+1

It seems this problem still persists for users -- @vikstrous if you will fix the one nit I mentioned, and also update the documentation, I'll merge this PR.  Thanks

updated

test this please

this should be ready to merge, right?

Yup Docs CI caught up. LGTM and thanks @vikstrous !

@nathanleclaire is this for v0.8.1 or v0.9.0? - if you set the milestone docs'll know when to publish it (and if its a fix for older releases, use that as the milestone?)


'Npm start' fails on react-scripts-start initial attempt

ISSUE
npm start is failing to work with create-react-app after initial 'create-react-app my-app' install. Complete output log is below.
I get similar output when trying to run another create-react-app that I copied in from another machine.
I initially tried this with node v8.9.4 and npm v5.6.0 and got the errors below.
I then installed a different node version using 'npm install -g npm@4', removed node_modules, cleared the cache, and ran another 'npm install'.
I then tried npm start with node v8.9.4 and npm v4.6.1, and got the same errors.
I searched for 'Exit status' and 'react-scripts start' in the documentation but didn't come up with anything.
running 'npm ls react-scripts' yields '<EMAIL_ADDRESS>'
I'm using bash shell on Windows 10, and the latest version of Google Chrome.
INITIAL OUTPUT:
capndavet@PC102:/mnt/c/Users/dthompson/Documents/protest-app$ npm start
><EMAIL_ADDRESS>start /mnt/c/Users/dthompson/Documents/protest-app
> react-scripts start
os.js:126
  const interfaceAddresses = getInterfaceAddresses();                                                                                ^
Error: EINVAL: invalid argument, uv_interface_addresses
  at Object.networkInterfaces (os.js:126:30)
  at Function.address.interface (/mnt/c/Users/dthompson/Documents/protest-app/node_modules/address/lib/address.js:58:23)
  at Function.address.ip (/mnt/c/Users/dthompson/Documents/protest-app/node_modules/address/lib/address.js:97:31)
  at err (/mnt/c/Users/dthompson/Documents/protest-app/node_modules/detect-port-alt/lib/detect-port.js:75:30)
  at Server.server.listen (/mnt/c/Users/dthompson/Documents/protest-app/node_modules/detect-port-alt/lib/detect-port.js:100:12)
  at Object.onceWrapper (events.js:313:30)
  at emitNone (events.js:106:13)
  at Server.emit (events.js:208:7)
  at emitListeningNT (net.js:1378:10)
  at _combinedTickCallback (internal/process/next_tick.js:135:11)
  at process._tickCallback (internal/process/next_tick.js:180:9)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `react-scripts start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.                                                                                npm ERR! A complete log of this run can be found in:
npm ERR!     /home/capndavet/.npm/_logs/2018-01-04T16_27_50_082Z-debug.log `

DEBUG LOG
capndavet@PC102:/mnt/c/Users/dthompson/Documents$ cat /home/capndavet/.npm/_logs/2018-01-04T16_39_14_234Z-debug.log
 0 info it worked if it ends with ok
1 verbose cli [ '/home/capndavet/.nvm/versions/node/v8.9.4/bin/node',
1 verbose cli   '/home/capndavet/.nvm/versions/node/v8.9.4/bin/npm',
1 verbose cli   'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /home/capndavet/.nvm/versions/node/v8.9.4/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/mnt/c/Users/dthompson/Documents/protest-app/node_modules/.bin:/home/capndavet/.nvm/versions/node/v8.9.4/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /mnt/c/Users/dthompson/Documents/protest-app                          10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'react-scripts start' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
13 verbose stack Error<EMAIL_ADDRESS>start: `react-scripts start`
13 verbose stack Exit status 1
13 verbose stack at EventEmitter.<anonymous> (/home/capndavet/.nvm/versions/node/v8.9.4/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:285:16)
13 verbose stack     at emitTwo (events.js:126:13)
13 verbose stack     at EventEmitter.emit (events.js:214:7)
13 verbose stack     at ChildProcess.<anonymous> (/home/capndavet/.nvm/versions/node/v8.9.4/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at emitTwo (events.js:126:13)
13 verbose stack     at ChildProcess.emit (events.js:214:7)
13 verbose stack     at maybeClose (internal/child_process.js:925:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)                           14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /mnt/c/Users/dthompson/Documents/protest-app                                                             16 verbose Linux 3.4.0+
17 verbose argv "/home/capndavet/.nvm/versions/node/v8.9.4/bin/node" "/home/capndavet/.nvm/versions/node/v8.9.4/bin/npm" "start"                                                                                18 verbose node v8.9.4
19 verbose npm  v5.6.0
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>start: `react-scripts start`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script.
23 error This is probably not a problem with npm. There is likely additional logging output above.                      24 verbose exit [ 1, true ]

Ran into the same just now, and my debug log is almost verbatim what is reported above. Node version 8.9.4, npm 5.7.1, running on linux mint 18.1.
Deleting node_modules and running npm install fetched them for me, but issue still persists. Please help.

Please file a new issue if you're experiencing problems.


implemented Heading, Description and Video on the Home Page

@anushbhatia Implemented Basic Description Text of the Home page.
Also added video player on the Home page.
(It plays a dummy video right now. Later on, the source Path of the video would be changed)

Fix the merge conflict, please

@anushbhatia Conflicts resolved

Please update with the current Home page

Add a relevant social media video and play video without audio and don't use a audio player for this and
Add headings and text in a proper format

Add a relevant social media video and play video without audio and don't use a audio player for this and
Add headings and text in a proper format

Can you give little more description that how do you want to format the heading?

https://www.townscript.com/in/vadodara

@Nazeeh2000  .If you don't reply to the conversation and complete in in 3 days. Unfortunately I have to close this PR.


build: bump dagger to v0.4.4

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
23

Relevant Lines:
170

💛  - Coveralls


Verify quilt_rails in a database-enabled app

Overview
quilt_rails' use of nested controller calls places extra load on Rails.  For a request to be served, Rails needs:

Extra HTTP server threads
Extra DB connection pool threads

We need to see if these extra demands triggers deadlocking, or pool exhaustion.  So it's essential that we have an app that does UiController -> GraphQLController -> database query flows.

We verified that quilt_rails has problems scaling (in hydrox), so this was successful :/ Kinda.


feat: writing reports to FILE and LOGGER_AND_FILE output supported

Codecov Report

Merging #4 (82106c1) into master (bb61ffb) will decrease coverage by 0.72%.
The diff coverage is 79.87%.

@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
- Coverage     85.08%   84.35%   -0.73%
- Complexity      392      415      +23
============================================
  Files            67       76       +9
  Lines          1019     1106      +87
  Branches         81       84       +3
============================================
+ Hits            867      933      +66
- Misses           87      103      +16
- Partials         65       70       +5

Impacted Files
Coverage Δ
Complexity Δ

...com/adgadev/jplusone/core/report/SqlFormatter.java
95.74% <0.00%> (ø)
12.00 <0.00> (?)

...lusone/core/report/output/StdoutReportChannel.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...usone/core/report/output/ReportChannelFactory.java
40.00% <40.00%> (ø)
2.00 <2.00> (?)

.../jplusone/core/report/file/TextFileWriterImpl.java
53.12% <53.12%> (ø)
5.00 <5.00> (?)

...adgadev/jplusone/core/report/SessionFormatter.java
86.53% <86.53%> (ø)
13.00 <13.00> (?)

...rts/impl/rule/message/ReportFragmentGenerator.java
100.00% <100.00%> (ø)
4.00 <2.00> (+1.00)

...gadev/jplusone/core/JPlusOneAutoConfiguration.java
94.11% <100.00%> (+1.26%)
12.00 <3.00> (+2.00)

...v/jplusone/core/properties/JPlusOneProperties.java
100.00% <100.00%> (ø)
1.00 <0.00> (ø)

.../adgadev/jplusone/core/report/ReportGenerator.java
86.36% <100.00%> (+3.46%)
8.00 <1.00> (-13.00)
:arrow_up:

...lusone/core/report/file/TextFileWriterManager.java
100.00% <100.00%> (ø)
7.00 <7.00> (?)

... and 13 more

Continue to review full report at Codecov.

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


Update @nx/workspace to 19.4.4

Update Nx from 19.4.3 to 19.4.4

Auto-generated by create-pull-request

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

Project coverage is 97.63%. Comparing base (9e4b573) to head (2858ad9).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #146   +/-   ##
=======================================
  Coverage   97.63%   97.63%
=======================================
  Files          17       17
  Lines         211      211
  Branches       30       30
=======================================
  Hits          206      206
  Misses          5        5

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


How to add Babel plugin from the front?

I'm trying to use emotion's babel plugin. It has to be the first plugin loaded. Using addBabelPlugin isn't working. How can I add it at the first position?

Never mind. Figured it out.
const { override, getBabelLoader } = require('customize-cra');

const prependBabelPlugin = (plugin) => (config) => {
  getBabelLoader(config).options.plugins.unshift(plugin);
  return config;
};

module.exports = override(prependBabelPlugin('emotion'));


@version after deleting a record

New @rid but old @version:
orientdb {db=test}> create class test

Class created successfully. Total classes in database now: 32

orientdb {db=test}> insert into test set title = 'abc'

Inserted record 'test#32:0{title:abc} v1' in 0.008000 sec(s).

orientdb {db=test}> select @version from test where title = 'abc'

----+------+-------
#   |@CLASS|version
----+------+-------
0   |null  |1
----+------+-------

1 item(s) found. Query executed in 0.003 sec(s).
orientdb {db=test}> delete from test where title = 'abc'

Delete record(s) '1' in 0.003000 sec(s).

orientdb {db=test}> select @version from test where title = 'abc'

0 item(s) found. Query executed in 0.002 sec(s).
orientdb {db=test}> insert into test set title = 'abc'

Inserted record 'test#32:1{title:abc} v2' in 0.003000 sec(s).

orientdb {db=test}> select @version from test where title = 'abc'

----+------+-------
#   |@CLASS|version
----+------+-------
0   |null  |2
----+------+-------

1 item(s) found. Query executed in 0.002 sec(s).
orientdb {db=test}>

hi @vovagp
Yes we are aware of this, is due to the low level storage layout and don't create any real issue.
the basic concept don't require that the version start from 0  it's only important that every update the version is the one expected.
this may also change in future.


Pre-downloading role

Hey,
Basically I have 3 scripts that run some ansible tasks which depend on your two roles. So as I don't have to repeatedly download them as requirements on multiple machines, I would like to bundle the roles on my internal(company) git repo, together with my scripts.
Do you know of any way that I might achieve that?
Thanks

Tell Ansible via the ansible.cfg file where to look for installed roles:
http://docs.ansible.com/ansible/intro_configuration.html#roles-path
Install your requirements to that directory:
ansible-galaxy install -r requirements.yml -p /path/to/git/repo/roles

Then commit the roles directory to your repository.
That said, I'm curious why you'd need to install the roles on multiple machines. You should only need to download the roles once onto the control node. It may not be feasible in your company, but with one control node, you should be able to manage multiple other machines if they are enumerated in the control node's inventory.
http://docs.ansible.com/ansible/intro_inventory.html

@trusk89 - Indeed, as @elliotweiser says, for stability and performance it's often desirable to 'commit your dependencies'; see how Drupal VM does it here: https://github.com/geerlingguy/drupal-vm/tree/master/provisioning
For most of my 'actual money-earning production' playbooks, I add an ansible.cfg with the roles_path defined, then commit the roles to the repo.

Yeah, I did that and it works pretty well. One thing i don't get yet (didn't have time to test before leaving work): if I have my roles already stored on my repo, do I need to run ansible-galaxy install requirements.yml? Or they are automatically searched for and found in the path I set in ansible.cfg?
Thanks for all the help guys!

The ansible-galaxy command will search the paths enumerated in the roles_path variable defined in your ansible.cfg. Therefore, as long as the roles are committed somewhere in that list of paths, ansible-galaxy will not need to re-download them unless you force -f it.

@elliotweiser so I can remove the step from my scripts, right?

Go for it.

Thanks again guys!

No problem! Also, note that ansible-galaxy isn't a full fledged package manager (at least not yet), so when updating things, you might have to delete a package then install, or use --force to force install a newer version.

It isn't, but what it is, is great! I managed to automate env setup for 3 machines: a jenkins master, a jenkins build slave and an iOS/Android development machine. It's great!

Works great, thanks!


Enable and test ingesting OTLP HTTP requests from the OpenTelemetry exporter

Presently the Otel Trace Source only supports requests using OpenTelemetry's grpc protocol (via the OTLP gRPC Exporter). We'd like to allow for OTLP/HTTP Exporter requests to be receivable by the source.
There was a concern regarding different encodings between the grpc and http protocols, so we'll need to first allow for HTTP requests then test if this is the case. The Otel Source's server is backed by Armeria and configured in OtelTraceSource.java.
We'll also need to update the otel-config.yml file in the sample app to use HTTP - see the OpenTelemetry Exporter docs for more.

https://github.com/line/armeria/issues/953

https://github.com/line/armeria/issues/953

https://github.com/open-telemetry/opentelemetry-collector/blob/master/exporter/otlphttpexporter/README.md

https://github.com/open-telemetry/opentelemetry-collector/blob/master/exporter/otlphttpexporter/README.md

{"level":"info","ts":1611787746.6829534,"caller":"exporterhelper/queued_retry.go:252","msg":"Exporting failed. Will retry the request after interval.","component_kind":"exporter","component_type":"otlphttp","component_name":"otlphttp","error":"error exporting items, request to http://data-prepper:21890/opentelemetry.proto.collector.trace.v1.TraceService/Export responded with HTTP Status Code 415","interval":"25.430451258s"}
415 == Unsupported Media Type

{"level":"info","ts":1611787746.6829534,"caller":"exporterhelper/queued_retry.go:252","msg":"Exporting failed. Will retry the request after interval.","component_kind":"exporter","component_type":"otlphttp","component_name":"otlphttp","error":"error exporting items, request to http://data-prepper:21890/opentelemetry.proto.collector.trace.v1.TraceService/Export responded with HTTP Status Code 415","interval":"25.430451258s"}
415 == Unsupported Media Type


Fix fs path mismatch in go and cpp

The semantics of "path" are different in the fs interfaces of Go and C++. In C++, the path needs to include the bucket, while in Go version it is not required. This leads to issues with reading and writing.

/lgtm

/approve


Can the preinstall script be removed ?

Hello, thanks for creating this repo.
Curious if this preinstall script is still needed ? From the discussion here it seems that you were using submodules before but migrated away from that. The current list of submodules is just empty.
Unless you plan on adding any more submodules in near future, it would be good to remove this because this causes npm install or npm ci to fail in scenarios where the consumer project is not a git repo. We encountered this when running npm ci inside a docker container - we don't clone the whole of our repo into the image.

Please remove, test and send pull request.


Hoymiles Consumption and From Grid to Grid

Would be nice if we could also get the Consumption and From Grid to Grid Vales if the DTSU666 is connectet to the DTU Pro

Unfortunately, there are no such parameters in modbus DTU. You can always connect the meter to the HA and use the energy tab.


ExpressRoute using a service provider vs ExpressRoute Direct

Hi,
This article compares ExpressRoute Direct with ExpressRoute using a service provider.
I not sure I understand it though :-)
As I understand it for a regular ExpressRoute you use a service provider to get a MPLS connection from you company location to a co-location where the MPLS is connected to a Microsoft Enterprise Edge device. From here the connection is running on Microsoft network.
For ExpressRoute Direct you do exactly the same but at the co-location your connection is attached to a different edge device.
Have I understood this correct? :-)
Regards Peter

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

ID: 84695fdf-03f1-a3d4-2bec-b17e06102119
Version Independent ID: 008b4912-3d0a-4fff-577a-d9a3e4dcc9c7
Content: About ExpressRoute Direct - Azure
Content Source: articles/expressroute/expressroute-erdirect-about.md
Service: expressroute
GitHub Login: @jaredr80
Microsoft Alias: jaredro

@theheatDK The differences are outlines in This Section. ExpressRoute direct does not go through a service provider. It requires direct physical infrastructure to the edge location.

Thanks @TravisCragg-MSFT. Yes that is the section I am talking about. What does it mean to have direct physical infrastructure to the edge location? Do we need to dig a hole from our company to the edge location with our own fiber? If so how is this different from using a service provider?
I am sorry if these are stupid questions :-)
Regards Peter

@theheatDK You will need a dedicated line that you either lay yourself or one provided by someone else that you have management of. If you are interested in having an ExpressRoute Direct and this is something that you can feasibly do, email<EMAIL_ADDRESS>for more information.

@theheatDK We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


fix(pipeline): include updated timestamp

Add back the last modified timestamp property updateTs to pipelines, which is expected by things like deck to provide revision history.

@Mergifyio backport release-1.27.x


change author to creator in affiliation

Describe the improvement
In the creator popup for datasets, the title is adapted to "Creator" but in the section where you can indicate the UGent members, it says "author". Can this be changed to creator?
Screenshots

Isn't the label itself a bit problematic? I mean, a label like "UGent creator" with options "External creator" and "UGent creator" doesn't make sense to me. Also goes for "Ugent author" of course"

yes, this change of column header is described in https://github.com/ugent-library/biblio-backend/issues/323

fixed by #333

One more to change from "Author" to "Creator": blue "Confirm" button.

Should be fixed now (see next deploy to dev)


Feature: A similar -y option for download tools & dependency

Hi! Just wondering if it's possible for nuitka to add a similar -y, --yes option that it just downloads the required tools & dependency(for example, dependency walkers, specific version of gcc, etc.) whenever it needs, just like kind of sudo apt install -y ... . It will then be possible to add nuitka build as part of a CI step in various projects which requires full automacy. Thank you.

This already exists: Try --assume-yes-for-downloads
https://github.com/Nuitka/Nuitka/blob/f21165c74f81296da61cf4bfa268213a9f5f1c17/nuitka/OptionParsing.py#L510-L519

It works, thanks! I'll close this issue.

@kamgha thanks for being around


Added typescript definitions

All typing should be mostly correct. Uses module augmentation to add .guard member. Custom user model class can be augmented using empty interface like so:
interface User extends GuardUser {}
class User extends Model {
    ...
}

Thanks for the PR @nathanlepori . and sorry for such a late reply...
I will look into it as soon as possible. currently I'm bit busy with some school stuff so might take some time.

No problem take your time. Thanks for looking into this


'Ten minute' tutorial potentially confusing relating to Wagtail versions

Here's what I did:

From https://wagtail.io/developers/, I followed the link to the Ten Minute Guide at http://docs.wagtail.io/en/latest/getting_started/tutorial.html
I ran pip install wagtail and followed the tutorial.
I experienced a few issues but hit a wall in the related items section.  It look a while to realise that the InlinePanel API had changed, and that the tutorial is for a different version than is currently installing using pip (Wagtail 0.8.8).

Obviously if you visit the Wagtail docs direct, it does lead to the correct version, but it seems reasonable that other developers may take the same path I took!
My suggestion would be either to remove / qualify the link from the 'Developers' page, or perhaps adjust the documentation itself to make it clear that it needs to be Wagtail 1.x.
Hope useful feedback!

No longer an issue now that 1.0 is the current version installed by pip.


Add a Test Sample for the Drakvuf Feature Extractor

This PR adds a limited sample report for the drakvuf sandbox unit tests. These tests can be found in PR (https://github.com/mandiant/capa/pull/2143)

would you PM me a complete file so I can play around with the performance aspect, too?

would you PM me a complete file so I can play around with the performance aspect, too?

Apologies for the late reply. I have sent the sample report your way :)


Crashing on file save

Hi, this looks like the beginnings of a really nice editor.  However, when testing with multiple windows open, I noticed that saving a file was pretty consistently causing a crash.  I tested this on both 0.1.2 and the latest git version.
System:
Arch Linux
Kernel 5.18.7
KDE Plasma
[ERROR rust_analyzer] failed to find any projects in [AbsPathBuf("/home/jack/projects/<non-rust project>/")]
[ERROR rust_analyzer::lsp_utils] rust-analyzer failed to discover workspace
[ERROR rust_analyzer] failed to find any projects in [AbsPathBuf("/home/jack/projects/<non-rust project>/")]
[ERROR rust_analyzer::lsp_utils] rust-analyzer failed to discover workspace
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', lapce-data/src/data.rs:1930:44
stack backtrace:
   0:     0x55971fe87dfd - <unknown>
   1:     0x55971feb19fc - <unknown>
   2:     0x55971fe80181 - <unknown>
   3:     0x55971fe89775 - <unknown>
   4:     0x55971fe893e9 - <unknown>
   5:     0x55971fe89d48 - <unknown>
   6:     0x55971fe89bb9 - <unknown>
   7:     0x55971fe882b4 - <unknown>
   8:     0x55971fe89929 - <unknown>
   9:     0x55971e76d0b3 - <unknown>
  10:     0x55971e76cf7d - <unknown>
  11:     0x55971ebe2971 - <unknown>
  12:     0x55971ebe2a5c - <unknown>
  13:     0x55971e7c2248 - <unknown>
  14:     0x55971e7c4927 - <unknown>
  15:     0x55971e7cae9a - <unknown>
  16:     0x55971e860a7b - <unknown>
  17:     0x55971e917fba - <unknown>
  18:     0x55971e8b6425 - <unknown>
  19:     0x55971e7ccc40 - <unknown>
  20:     0x55971e8614eb - <unknown>
  21:     0x55971e920891 - <unknown>
  22:     0x55971e782826 - <unknown>
  23:     0x55971e7f2262 - <unknown>
  24:     0x55971e7ec252 - <unknown>
  25:     0x55971e7fe871 - <unknown>
  26:     0x55971fc0720c - <unknown>
  27:     0x55971fc1907e - <unknown>
  28:     0x7f3a91641c6b - g_main_context_dispatch
  29:     0x7f3a91698001 - <unknown>
  30:     0x7f3a9163f392 - g_main_context_iteration
  31:     0x7f3a917fe19e - g_application_run
  32:     0x55971fc1bd37 - <unknown>
  33:     0x55971fc0dd0e - <unknown>
  34:     0x55971e81f706 - <unknown>
  35:     0x55971e788ba4 - <unknown>
  36:     0x55971e76f0f3 - <unknown>
  37:     0x55971e76f0c9 - <unknown>
  38:     0x55971fe785be - <unknown>
  39:     0x55971e76f132 - <unknown>
  40:     0x7f3a91303290 - <unknown>
  41:     0x7f3a9130334a - __libc_start_main
  42:     0x55971e76efe5 - <unknown>
  43:                0x0 - <unknown>
[ERROR rust_analyzer] Unexpected error: client exited without proper shutdown sequence
client exited without proper shutdown sequence
[ERROR rust_analyzer] Unexpected error: expected initialize request, got error: receiving on an empty and disconnected channel
expected initialize request, got error: receiving on an empty and disconnected channel
[ERROR rust_analyzer] Unexpected error: expected initialize request, got error: receiving on an empty and disconnected channel
expected initialize request, got error: receiving on an empty and disconnected channel
[ERROR rust_analyzer] Unexpected error: expected initialize request, got error: receiving on an empty and disconnected channel
expected initialize request, got error: receiving on an empty and disconnected channel
[ERROR rust_analyzer] Unexpected error: client exited without proper shutdown sequence
client exited without proper shutdown sequence

I've been learning the code base a bit, trying to figure this one out myself.  Looks like the OpenFile command is being called twice and is maybe being handled by the wrong window?
[2022-07-01][11:16:42][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:16:42][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[ERROR rust_analyzer] failed to find any projects in [AbsPathBuf("/home/jack/projects/qmk_firmware/")]
[ERROR rust_analyzer::lsp_utils] rust-analyzer failed to discover workspace
[2022-07-01][11:16:49][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:16:49][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[ERROR rust_analyzer] failed to find any projects in [AbsPathBuf("/home/jack/projects/openrgb-python/")]
[ERROR rust_analyzer::lsp_utils] rust-analyzer failed to discover workspace
[2022-07-01][11:16:54][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:16:54][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
Opening file "/home/jack/projects/openrgb-python/openrgb/network.py"
Inserting "/home/jack/projects/openrgb-python/openrgb/network.py" into open_docs
[2022-07-01][11:17:10][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:10][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
Checking if open_docs contains "/home/jack/projects/openrgb-python/openrgb/network.py"
open_docs: "/home/jack/projects/openrgb-python/openrgb/network.py"
Checking if open_docs contains "/home/jack/projects/openrgb-python/openrgb/network.py"
open_docs: "/home/jack/projects/openrgb-python/openrgb/network.py"
[2022-07-01][11:17:30][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:30][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:30][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:30][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[ERROR rust_analyzer] failed to find any projects in [AbsPathBuf("/home/jack/projects/qmk_firmware/")]
[ERROR rust_analyzer::lsp_utils] rust-analyzer failed to discover workspace
[2022-07-01][11:17:48][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:48][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:48][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
[2022-07-01][11:17:48][lapce_data::keypress][ERROR] Failed to load from "/home/jack/.config/lapce/debug-keymaps.toml": no keymaps
Opening file "/home/jack/projects/qmk_firmware/.gitignore"
Inserting "/home/jack/projects/qmk_firmware/.gitignore" into open_docs
Opening file "/home/jack/projects/qmk_firmware/.gitignore"
Checking if open_docs contains "/home/jack/projects/qmk_firmware/.gitignore"
open_docs: "/home/jack/projects/openrgb-python/openrgb/network.py"
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', lapce-data/src/data.rs:1936:44
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[ERROR rust_analyzer] Unexpected error: expected initialize request, got error: receiving on an empty and disconnected channel
expected initialize request, got error: receiving on an empty and disconnected channel
[ERROR rust_analyzer] Unexpected error: client exited without proper shutdown sequence
client exited without proper shutdown sequence
[ERROR rust_analyzer] Unexpected error: expected initialize request, got error: receiving on an empty and disconnected channel
expected initialize request, got error: receiving on an empty and disconnected channel
[ERROR rust_analyzer] Unexpected error: expected initialize request, got error: receiving on an empty and disconnected channel
expected initialize request, got error: receiving on an empty and disconnected channel
[ERROR rust_analyzer] Unexpected error: client exited without proper shutdown sequence
client exited without proper shutdown sequence

I can no longer reproduce this issue, so I'm going to close it under the assumption that it was fixed unintentionally.


Feature that enables library authors to include UI on packages.

I strongly believe that asp.net mvc framework is missing the ability to packing UI assets (Views) on libraries as first class citizen support. I think there's great opportunity to improve quality and enable innovation in multiple libraries.
Now it could be a great timing to implement said feature since IdentityService team can benefit from it, which is a strong indicator of being a missing feature.
https://github.com/aspnet/Identity/issues/1187#issuecomment-300676575
@danroth27:

We are also looking at ways to package the controllers and views so they don't have to live in the template.

/cc @sebastienros

Watch this video https://channel9.msdn.com/Shows/On-NET/Sbastien-Ros-Modular-ASPNET-apps
Is this what you want?

@sebastienros as addresses here: https://github.com/aspnet/Identity/issues/1187#issuecomment-300962769

The need to reference two different packages, inject services, have a middleware running to get the concept of modules on an application may be fine but not to package a library with controller(s) and view(s).

As this scenario would enable.

Just to be clear, I don't know what the IdentityService team will end up doing and how they will end up solving this problem. I can think of many ways to do it with pros/cons. What I'm asking is whatever effort is put into it, consider enabling it for 3rd party author too so we can all benefit from it and not being specific.

To mark something as a module we now use a single nuget package. Internally it's just adding a .props file that will handle the local deployment of static assets in the targeted app and also the packaging.
And as a module author you don't add services or middleware if you don't want to. Maybe the team will come with some hard coded magic to handle Identity in a better way, otherwise I don't see how it can be simpler.
The host application needs a middleware because of multitenancy. Probably something you don't care about and would rather not see. Internally we just make it no-op if there is a single tenant, but not as clean as having nothing at all.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Review Ticket Overview

Review Ticket Guide
For each assignment, we will be required to make a review ticket. Most assignments provided will have a Hacks section. A Hack is a term for coding, a fun term.

Screen Captures are nice for Hacks
Short Descriptions
Code Fragments

Markdown samples
(ignore this will fix)
Using markdown from index.md. We are learning markdown.
-The text below is something called Markdown
-This is a heading, inside of code scaffolding.
'''markdown
'''markdown
'''bullets
Investing in Your Technical Future`
'''
-This is emphasis.
'''markdown

Explore the Computer Science Pathway at Del Norte High School and invest in your technical skills. All Del Norte CompSci classes are designed to provide a real-world development experience. Class time includes tech talks (lectures), peer collaboration, communication with teachers, critical thinking while coding, and creativity in projects. Grading is focused on time invested, participation with peers, and engagement in learning.
-Sample of bullets
'''markdown

Introduction to concepts and requirements by the teacher
Project-based learning with teacher support, performing Agile/Scrum development
Coding, frontend, backend, devops, version control and algorithmic thinking
Creativity, research, design, data structures, and utilizing ChatGPT
Performing team work, team communication and collaboration, peer reviews/grading
Focus on tehnical communications through project presentations and student led teaching
Grades are on projects, learnt concepts, and live reviews between student(s) and teacher
'''


Cannot update fastlane version

I get notification when running fast lane that a new version is available (2.153, I currently am on 2.152), however I am unable to update.  I have used commands
sudo gem install fastlane
gem install fastlane
gem update fastlane
unsuccessfully.  the install command reports this
Done installing documentation for  after 0 seconds
but the version does not get updated.
fastlane env gives:
🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.15.6

Ruby
2.6.3

Bundler?
false

Git
git version 2.24.3 (Apple Git-128)

Installation Source
~/.rvm/gems/ruby-2.6.3/bin/fastlane

Host
Mac OS X 10.15.6 (19G73)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.6.3/lib

OpenSSL Version
OpenSSL 1.1.1c  28 May 2019

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
11.6

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.152.0
🚫 Update available

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

atomos
0.1.3

CFPropertyList
3.0.0

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.6

xcodeproj
1.14.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.6.0

terminal-table
1.8.0

plist
3.5.0

public_suffix
2.0.5

addressable
2.6.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.7.0

tty-cursor
0.7.0

tty-spinner
0.9.1

babosa
1.0.3

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.71.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

fastimage
2.1.5

gh_inspector
1.1.3

json
2.1.0

mini_magick
4.9.5

rubyzip
2.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.7.4

bundler
1.17.3

naturally
2.2.0

simctl
1.6.5

jwt
2.1.0

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mini_mime
1.0.2

multi_json
1.13.1

signet
0.12.0

memoist
0.16.0

os
1.0.1

googleauth
0.9.0

httpclient
2.8.3

google-api-client
0.38.0

google-cloud-env
1.2.0

google-cloud-core
1.3.0

digest-crc
0.4.1

google-cloud-storage
1.16.0

emoji_regex
1.0.1

faraday
0.17.1

aws-eventstream
1.1.0

aws-sigv4
1.1.3

aws-partitions
1.314.0

jmespath
1.4.0

aws-sdk-core
3.95.0

aws-sdk-kms
1.31.0

aws-sdk-s3
1.64.0

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

webrick
1.4.2

generated on: 2020-07-29

Have you tried to un-install fastlane? If you use the sudo command, it might be possible that you need to use it for update as well.

I have not un-installed.  I wanted to first check for an alternative.  I can try that today.

This has happened to me in the past, but I use bundler to install fastlane (which it's strongly encouraged you doing, so you don't have to modify gems in your entire machine).
In my case, the problem was caused by dependencies that depended on different versions of fastlane's dependencies, so fastlane couldn't be upgraded alone and satisfy all the other dependencies' required versions.
The solution was to upgrade all my gems via bundler.
The equivalent to this solution in your case would be updating all your gems, but globally. If you're certain you wanna do this, run:
gem update --system
gem update

This will update RubyGems and then update all your packages.
Again, proceed with caution. I'd strongly consider moving away from global gems, and using bundler instead.

Righto.  I have seen the notices when running fastlane about using bundle and have been putting off dealing with it.  So, maybe this is a good excuse to tuck into that and set up more thoroughly.

Yup! And it's not complicated at all :) totally worth it.

I was able to update by running “sudo gem update” first, and then “gem install fastlane”.  This fixed dependencies issue and allowed the update to proceed.


Permissions/Roles.

Just started using this, great work.
So my question is how to allow a user to only update their information.
I.e. if they add a device to the devices table, I would only want them to be able to CRUD their own stuff only.
Dusty

on the generated project you can see under src/api/user/controller.js something like this:
export const update = ({ bodymen: { body }, params, user }, res, next) =>
  User.findById(params.id === 'me' ? user.id : params.id)
    .then(notFound(res))
    .then((result) => {
      if (!result) return null
      const isAdmin = user.role === 'admin'
      const isSelfUpdate = user.id === result.id
      if (!isSelfUpdate && !isAdmin) {
        res.status(401).json({
          valid: false,
          message: 'You can\'t change other user\'s data'
        })
        return null
      }
      return result
    })
    .then((user) => user ? Object.assign(user, body).save() : null)
    .then((user) => user ? user.view(true) : null)
    .then(success(res))
    .catch(next)

This makes possible only to admin or the same user to update a user. Similarly, you will need to implement something like this on your index and show controllers (using the option ? Do you want to store in a field the user who created the some-entity? (Y/n)):

export const index = ({ querymen: { query, select, cursor }, user }, res, next) =>
  Devices.find({ user: user.id, ...query }, select, cursor)
    .then(devices => devices.map(device => device.view()))
    .then(success(res))
    .catch(next)

So you can find specifically the Devices created from the same user that is requesting the stuff.


Templates not support to configure advanced mapping in event publishers

When publishing an event through identity event publisher we can use created template for configure the advance mapping to the event. Identity provider, it self has several email templates. Email publisher currently use the template among it to initiate the email body and send parameters.
I have tried to request soap endpoint within the event publishers with created template. Use the template body to add the payload with parameters. But for template not work for the added payload.
Steps to recreate:

Create a new template from identity console.
Add the sample payload and parameters to the template.

Sample payload,
      <soapenv:Envelope
      xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:web="http://www.dataaccess.com/webservicesserver/">
        <soapenv:Header/>
        <soapenv:Body>
          <web:NumberToWords>
            <web:ubiNum>{{accountLock}}</web:ubiNum>
          </web:NumberToWords>
        </soapenv:Body>
      </soapenv:Envelope>

Create the publisher xml with the defined stream
Example for publisher xml

<?xml version="1.0" encoding="UTF-8"?>
<eventPublisher name="SoapPublisher" processing="enable"
  statistics="disable" trace="disable" xmlns="http://wso2.org/carbon/eventpublisher">
  <from streamName="id_gov_soap_output_notify_stream" version="1.0.0"/>
  <mapping customMapping="enable" type="xml">
    <inline>{{body}}</inline>
  </mapping>
  <to eventAdapterType="soap">
    <property name="url">http://webhook.site/35397daf-aacc-4f3e-8ace-a26e7c44fff0</property>
  </to>
</eventPublisher>

Add event configuration to the <IS_HOME>/repository/conf/identity/identity-event.properties
Sample configurations:

module.name.13=default.notification.sender
default.notification.sender.subscription.1=POST_ACCOUNT_SUSPENSION
default.notification.sender.subscription.POST_ACCOUNT_SUSPENSION.stream=id_gov_soap_output_notify_stream:1.0.0
default.notification.sender.subscription.POST_ACCOUNT_SUSPENSION.claim.accountLock=http://wso2.org/claims/identity/accountLocked
default.notification.sender.subscription.POST_ACCOUNT_SUSPENSION.notification_template=soap

Configure the user suspension task and let the event get triggered. [1]

Validate the response receive. (Use [2] for get the response for validation)

Expected result:
Should receive successful response.
Actual result:
Successful response not received.
[1] https://docs.wso2.com/display/IS570/User+Account+Suspension
[2] https://webhook.site

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


fade=true with autoplay

Hi,
thanks for the great work with the ui-carousel. It is amazingly easy to use.
I faced the problem, that if I add fade="true" the autoplay does not work anymore. I did not see a report about that behavior.
For an example I just took your codepen: http://codepen.io/mihnsen/pen/ZejQBJ
And add there the fade="true"
Best regards,
Fabian

@fanick-ber Fixed on bacacfb

Why 'fade=true' is not working? My config as follows:
  <ui-carousel slides="vm.slideImages" autoplay="true" fade="true" speed="1000" slides-to-scroll="1" initial-slide="0">           <carousel-item>

           </carousel-item>         </ui-carousel>
And relevant version as follows:
  "angular": "~1.6.3",     "angular-animate": "~1.6.3",     "angular-bootstrap": "~2.5.0",     "angular-messages": "~1.6.3",     "angular-resource": "~1.6.3",     "angular-ui-router": "~1.0.19",     "bootstrap": "~3.3.6", "angular-ui-carousel": "^0.1.10"


Half Without I2C Pull-up Resistors Is Unusable by Itself

Hi,
I have built the wired version of the Redox and so far it's really fun to type on. I have found a small quirk however and was able to fix it. It doesn't seem to be referenced anywhere in the documentation. Maybe it would be helpful to add it :)
If I configure I2C and EE_HANDS (other than that I am using the default configuration) in the latest QMK firmware and connect the half without the pull-up resistors to USB without connecting it to the other half, it locks up and barely outputs anything or if it does, multiple key presses.
The culprit seems to be the floating I2C bus. Either spreading the resistance over both halves (using 10k resistors in all 4 footprints) or activating the internal pull-ups on PD0 and PD1 fixed the issue for me.

I actually have the same problem when configuring the keyboard with I2C and EE_hands, can you elaborate how did you activate the internal pull-ups on PD0 and PD1? Thanks!

If I configure I2C and EE_HANDS (other than that I am using the default configuration) in the latest QMK firmware and connect the half without the pull-up resistors to USB without connecting it to the other half, it locks up and barely outputs anything or if it does, multiple key presses.
The culprit seems to be the floating I2C bus. Either spreading the resistance over both halves (using 10k resistors in all 4 footprints) or activating the internal pull-ups on PD0 and PD1 fixed the issue for me.

I didn't encountered this behaviour before. Maybe they changed something in the latest version of QMK and internal pull-ups are not activated by default?
I'll try and take a look into it. Thank you for pointing this out.


Shimming window["jQuery"]

This configuration does not replace window["jQuery"] with require("jquery") in knockout (v. 3.3.0) module.
What should I do to avoid exposing jQuery in window and provide knockout with jQuery?
new webpack.ProvidePlugin({
      'window.jQuery': 'jquery',
      'jQuery': 'jquery',
      'window["jQuery"]': 'jquery'
    })

It could be that the jquery dependency in knockout does not match with your included jquery. I had an issue like that with Backbone, and finally fixed it by adding this to the webpack config:
resolve: {
    alias: {
        'jquery': require.resolve('jquery'),
    }
}

Your solution exposes global variable jQuery, I want to avoid that.

That's not true, it doesn't expose anything. It only forces the use of node_modules/jquery instead of e.g. node_modules/knockout/node_modules/jquery

That's not my case.

The guys from knockout have put a lot of effort to "protect" their code against unwanted changes :grinning:...
it would be better if they'd actually require("jquery") in case a module system is detected. Just as recommended by the UMD.
However, the ProvidePlugin does not work, because window["jQuery"] is not an identifier, but a property access. It would have been worked, if they'd just accessed jQuery.
There is actually no other way than exposing the jQuery-object to window. Every other attempt is not worth the effort imho. But you could send a PR to knockout and tell them, that it's impossible to use their library without global variables.

@jhnns Yes, I already added comment to their issue regarding amd, but they do not hurry with the solution. ProvidePlugin has examples for window.jQuery replacement, so I thought window['jQuery']  should work too.

Yep, sorry about that, my examples on Stackoverflow were wrong. I've corrected them.

@blazkovicz, if you are still interested on this, I solved this issue on my own by using a custom loader (maybe adding a generic text replacement loader would be a better solution):
ko-loader.js:
module.exports = function(source) {
  this.cacheable();
  return source.replace('jQueryInstance = window["jQuery"]', 'jQueryInstance = require("jquery")');
};

in webpack.config.js:
    ,resolveLoader: {
      alias: { 'ko-loader': __dirname + '/ko-loader' }
    }
    ,module: {
        loaders: [
          { test: /knockout-latest\.debug\.js$/, loader: 'ko-loader'}
        ]
    }

I hope that helps.

@rosenfeld, thanks! I'll try that. What's the this.cachable();?

this.cacheable tells webpack the output of that function can be cached and won't change for a given source. Most loaders should be flagged as cacheable and this is not an exception. I guess it will only make any difference when you are using --watch or the dev server since webpack only seems to support in-memory caching...


DCOS_OSS-702: Remove API errors on CreateServiceJsonOnly when changed

This PR ensures that the API errors are removed when the user uses the CreateServiceJsonOnly form and there are marathon errors after review.
Without this PR the API errors would always persist, giving the feeling that the JSON is never correct.
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

LGTM! 💯  💖

Works perfectly! :ship:


Make Ambassador installable with Helm

In order to to make Ambassador easier to distribute, install and reuse packaging at a Helm Chart is important.
Output of this Issue should be:

Helm chart to install Ambassador.
Documentation on how to use the Helm chart for installation and upgrades.

Is anyone working on this? If not I can give it a shot this week

Not at the moment. That would be super! Thanks.

hi, just want to mention the helm documentation is off. The chart repository ain't working.
$ helm repo add datawire https://www.getambassador.io
$ helm install datawire/ambassador

I'm still n00b with helm but i think you could bundle the index.yaml in a github page without having to add the repository to your website.
ref: https://github.com/kubernetes/helm/blob/master/docs/chart_repository.md#github-pages-example

Installing via helm does not work again.
$ helm repo add datawire https://www.getambassador.io/helm
"datawire" has been added to your repositories
$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "datawire" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈
$ helm install datawire/ambassador
Error: failed to download "datawire/ambassador"

Crap. When we moved the ambassador-X.Y.Z.tgz files into S3, we missed one place updating the URLs for the index. Fix coming soon.

@przemek-sl This should be good to go (again). Sorry about that!


Build failure: poetry with python 3.8 sphinx-7.2.6 not supported for interpreter python3.8

Steps To Reproduce
Steps to reproduce the behavior:
Use nix develop with the following nix flake :
{
  description = "";

  inputs = {
    nixpkgs = { url = "github:NixOS/nixpkgs/nixos-unstable"; };
    flake-utils = { url = "github:numtide/flake-utils"; };
  };

  outputs =
    { self
    , nixpkgs
    , flake-utils
    ,
    }:
    flake-utils.lib.eachDefaultSystem (system:
    let
      pkgs = import nixpkgs {
        inherit system;
      };

      python = pkgs.python38;
      poetry = pkgs.poetry.override {python3 = python;};

    in
    {
      formatter = pkgs.nixpkgs-fmt;
      devShells.default = pkgs.mkShell {
        buildInputs = [
          python
          poetry
       ];
      };
    });
}

Build log
error:
       … while calling the 'derivationStrict' builtin

         at /builtin/derivation.nix:9:12: (source not available)

       … while evaluating derivation 'nix-shell'
         whose name attribute is located at /nix/store/z7y28qzhk7driiwcw78k0mb24laknm0f-source/pkgs/stdenv/generic/make-derivation.nix:348:7

       … while evaluating attribute 'buildInputs' of derivation 'nix-shell'

         at /nix/store/z7y28qzhk7driiwcw78k0mb24laknm0f-source/pkgs/stdenv/generic/make-derivation.nix:395:7:

          394|       depsHostHost                = elemAt (elemAt dependencies 1) 0;
          395|       buildInputs                 = elemAt (elemAt dependencies 1) 1;
             |       ^
          396|       depsTargetTarget            = elemAt (elemAt dependencies 2) 0;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: sphinx-7.2.6 not supported for interpreter python3.8

Additional context
The issue also happens when trying to use a poetry2nix environment so I'm geussing the root cause is the same :
{
  description = "";

  inputs = {
    nixpkgs = { url = "github:NixOS/nixpkgs/nixos-unstable"; };
    flake-utils = { url = "github:numtide/flake-utils"; };
    poetry2nix = {
      url = "github:nix-community/poetry2nix";
      inputs = {
        nixpkgs.follows = "nixpkgs";
        flake-utils.follows = "flake-utils";
      };
    };
  };

  outputs =
    { self
    , nixpkgs
    , flake-utils
    , poetry2nix
    ,
    }:
    flake-utils.lib.eachDefaultSystem (system:
    let
      pkgs = import nixpkgs {
        inherit system;
        overlays = [
          poetry2nix.overlays.default
        ];
      };

      python = pkgs.python38;
      poetryEnv = pkgs.poetry2nix.mkPoetryEnv {
        projectDir = self;
        inherit python;
      };

    in
    {
      formatter = pkgs.nixpkgs-fmt;
      devShells.default = pkgs.mkShell {
        buildInputs = [
          python
          poetryEnv
        ];
      };
    });
}

Is python 3.8 not supported anymore with poetry? Can I override this dependency somehow?
Notify maintainers
@jbaum98 @dotlambda
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.62, NixOS, 23.11 (Tapir), 23.11.20231112.e44462d`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Is python 3.8 not supported anymore with poetry?

No.

Okay well that explains everything. I missed this information somehow.
Thanks for the quick answer.

Although, may I ask on what grounds? I saw no mention of poetry dropping support for python 3.8 on their website.

Although, may I ask on what grounds?

Because it would probably be extra work to make sphinx work with Python 3.8. Feel free to submit a PR.

Pip itself depends on it though. Should something be done about that, like a constraint somewhere?
nix-repl> python38Packages.pip
error:
       … while evaluating the attribute 'drvPath'

         at /nix/store/457aqsrnk6hylbx9wf0m2xi0j9bpxx49-nixpkgs/nixpkgs/lib/customisation.nix:249:7:

          248|     in commonAttrs // {
          249|       drvPath = assert condition; drv.drvPath;
             |       ^
          250|       outPath = assert condition; drv.outPath;

       … while evaluating the attribute 'drvPath'

         at /nix/store/457aqsrnk6hylbx9wf0m2xi0j9bpxx49-nixpkgs/nixpkgs/lib/customisation.nix:249:7:

          248|     in commonAttrs // {
          249|       drvPath = assert condition; drv.drvPath;
             |       ^
          250|       outPath = assert condition; drv.outPath;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: sphinx-7.2.6 not supported for interpreter python3.8

Pip only depends on sphinx to build the man pages, but it's not necessary to run. In my opinion, we should fix this even if sphinx it's not compatible with Python 3.8. The straightforward fix would be to not build the man pages when Python version is <=3.8. What do you think @anund and @doronbehar

Sounds good to me.

Looking at where sphinx was bumped https://github.com/NixOS/nixpkgs/pull/259350 the only other alternative would be to introduce a sphinx_x_x to make pip compatible with older versions.

I agree that this is inconvenient and would appreciate a fix like the one suggested. Hit this issue with a simple dev shell with python38, black and flake8
{ pkgs ? import <nixpkgs> { } }:
with pkgs;
mkShell {
  packages = with pkgs;
    [ (python38.withPackages (ps: with ps; [ black flake8 ])) ];
}

PS: I use custom derivation without this change #250398

That would indeed work but since python 3.8 is EOL, we don't plan to maintain packages for it as it is just to much work for not that much benefit.

Python 3.8 is not EOL.

Some investigations that might help:
The example from @torgeir is not failing for depending on pip. nix-shell -f torgeir-example.nix --show-trace shows black -> aiohttp -> trustme -> pyopenssl -> sphink-hook so similar problem but not pip related.
Things depending on sphinxhook and sphinx-* packages directly require adding back an older copy of sphinx and updating python things < 3.8 to use those. I'm not sure where in the current python machinery that work would best land.
Here's a shell example that can override pip to cut out the change in https://github.com/NixOS/nixpkgs/pull/250398:
{ pkgs ? import <nixpkgs> {
  overlays = [
    (final: prev: {
      pythonPackagesExtensions = prev.pythonPackagesExtensions ++ [
        (
          python-final: python-prev: {
            pip = (python-prev.pip.overrideAttrs(old:{
              postBuild = "";
              outputs = ["out"];
              dist = "py38";
            })).override { sphinx = null; };
          }
        )
      ];
    })
  ];
}}:
with pkgs;
mkShell {
  packages = with pkgs;
    [ (python38.withPackages (ps: with ps; [ pip ])) ];
}

Python 3.8 appears to only be receiving emergency security fixes at this point. https://devguide.python.org/versions/. Bugfix releases ended ~may 2021 https://peps.python.org/pep-0569/. It's not technically eol just ~9 months and no emergencies away.

poetry.override{python3 = python38} -> cachy -> redis -> deprecated -> sphinx-hook (similarly not pip related).

Python 3.8 is not EOL.
I think if someone fixed this in a PR we'd definitely consider it.

It is in nixpkgs because we don't recurse into it anymore.

We're removing Python 3.8 before the NixOS 24.05 merge, so time is probably better spent on porting your code to a newer release.

#285779


Check that block numbers are consecutive

Apparently, nowhere in the Byron or Shelley ledger is it verified that blocks have consecutive block numbers.
We do check it in an assertion ChainFragment, but in production, assertions will be disabled.
We should verify this at the earliest opportunity we have, which is applyChainState (called for synced headers but also when we apply blocks we produced ourselves, as part of applyExtLedgerState). In addition to check whether block numbers are consecutive, we should also check that the first block has the right block number, i.e., genesisBlockNo.
It is important that we check this, because we're assuming that block numbers are a reliable proxy for chain length. So if this is not the case, we might prefer a chain that is actually shorter. So both consecutiveness and the right initial block number are important. For example, an attacker might forge a single block with a very large block number, and we would incorrectly prefer it over all other chains.

@nc6 , change of plan, no need to do this for each ledger (although the spec should still verify it, I think). I will do this once and for all for all ledgers. You actually discussed doing that when discussing the Byron integration, albeit for hashes, not block numbers; I'm coming round to your point of view now :) PR to follow, unassigning you from this ticket.

WIP PR at #1577 .

So #1578 cleans up a lot of our BlockNo hygiene, but I'd like to fix https://github.com/input-output-hk/cardano-base/issues/78 before fixing this ticket, in order to ensure that we don't have other places with bad BlockNo hygiene.


Dividing logic in Scenes

Scenes can become huge, causing monoliths. It should be possible to divide logic in these Scenes.
This could be done with Presenters which have the same lifecycle as the Scene.

Building on top of this, Id love to hear your approach and thoughts on rocking child scenes inside a viewpager - where the parent scene controls the whole view (e.g. detail screen showing the title of "A", containing a list of "B"s, and in turn each viewpager page showing details about a particular B). Would you consider the top level scene to be responsible for creating the child scenes in this case?

Yeah probably. But if at all possible, don't use child scenes at all but plain data classes.
 You'll have to be careful dealing with the lifecycles of the child scenes though, to ensure they are strictly contained within the parent scene (e.g only started when parent scene is started, destroyed when parent is destroyed).
In the other hand, you might find a compromise between both options to work: have a Scene-like class without the start/stop/destroy lifecycle, and only allow attach/detach.
I guess it eventually comes down to specifics: how much do the child scenes actually need to do, do you really need the full lifecycle, do you even need 'smart classes' at all or do plain data objects suffice?

For 'simple' screens with a fixed number of sub scenes we currently do this manually: we have a scene that is built up of ~10 subscenes, and the parent scene just forwards its lifecycle to them.
Since they're all visible at the same time this makes sense.
For lists of subscenes however (e.g. O(100)) in some kind of listview this may not scale at all, and you probably only want to start them when applicable.

Thanks for getting back to me about it! The sub-screens do indeed have a ton of logic, and theyre really only part of a viewpager in order to make switching between them easy for the user - outside of that Id consider them no different than any other normal screen.
Things get complex quickly when lifecycle differs for child scenes, an approach Ive been experimenting with myself is to have data & behavior someplace else, think of something like a NoteViewModel that has the ui-ready fields for a given note, and various actions - both available to the consumer for showing & using. The viewmodel in turn is provided through the scene, and any number of different viewmodels could be provided really. Furthermore, lazy loading could be acheived quite easily, but in my case I found that with the viewmodels not knowing what creates them, it really isnt as needed since each viewmodels scope is rather small (instead of loading A and retrieving B, the viewmodel is just populated with B).
I hope that makes sense, just thought Id share my own approach to it. Perhaps it can spark up a new idea that will revolutionize the whole thing, I havent found any real quirks with it yet but Ive only been playing around with it since after my post on here. In either case, its a joy to find well thought out libraries like yours where everything has a purpose and place, I look forward to seeing how it continues expanding!

So an example of this could be constructed as follows, by delegating the 'main' lifecycles to the sub scenes:
class SubScene1 : Scene<C1> { /* ... */ }
class SubScene2 : Scene<C2> { /* ... */ }

class MyScene(
  scene1: SubScene1,
  scene2: SubScene2
) : Scene<C> {

  override fun onStart() {
    scene1.onStart()
    scene2.onStart()
  }

  override fun attach(c: C) {
    scene1.attach(c.c1)
    scene2.attach(c.c2)
  }

  override fun detach(c: C) { / * ... */ }
  override fun onStop() { / * ... */ }
  override fun onDestroy() { / * ... */ }
}


HBASE-28612 fix replication quota leak when peer changes

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 43s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 35s
master passed

+1 :green_heart:
compile
0m 43s
master passed

+1 :green_heart:
shadedjars
5m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 29s
the patch passed

+1 :green_heart:
compile
0m 42s
the patch passed

+1 :green_heart:
javac
0m 42s
the patch passed

+1 :green_heart:
shadedjars
5m 13s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
the patch passed

_ Other Tests _

-1 :x:
unit
11m 21s
hbase-server in the patch failed.

31m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5944/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5944

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 0e1a0a996eb7 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / e5b581686d

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5944/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5944/1/testReport/

Max. process+thread count
1418 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5944/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fix the bug that dtype of the input data was not fixed

What?
Add a function to fix data-type of the input data and call it on add_data in DBHandler
Fix #72 and #73
Why?
Bug fix
See also [Optional]
https://github.com/dataware-tools/dataware-tools/issues/50

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: d-hayashi
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [d-hayashi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Rename TPUHalfPrecisionPlugin to TPUBf16PrecisionPlugin

What does this PR do?
Fixes https://github.com/PyTorchLightning/pytorch-lightning/pull/10020#discussion_r731919838
Rename TPUHalfPrecisionPlugin to TPUBf16PrecisionPlugin
And prints a warning when the user sets precision=16 for TPUs as they only really support "bf16".
Does your PR introduce any breaking changes? If yes, please list them.
The class name has changed
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Are we worried about backwards compatibility here, or is that primarily around API changes?


Add import outputs

what

Add all_imports_list and all_imports_map outputs

why

Output all imported configuration file names as list and map
Useful for Terraform Cloud and Spacelift to know which config files are imported by a given stack

/test all

/test all

/test all

/test all


Use the package without dependencies (e.g. webpack, require)

Is there a way to use the package without webpack and require and all other libraries dependencies?
When installing the package through npm, it imports everything on your repository, including the node_modules folder you've used.
You haven't provided an example for using the package through the angular2_modal only.

Moved angular2 & bootstrap as to devDependencies.
Thanks!

I meant using the library, just importing the angular2_modal file to index and that's it. No webpack no require. Is it possible?


Feature Request: Custom semantic selectors for global & block scoped variables

There is a specific feature that I am taking advantage of in Visual Studio 2022 that I would very much like to replicate in Sublime Text.
In Visual C++ 2022, I have set up my color scheme to where block-scoped variables are red, class/struct member variables are light red, and global variables are white.  With your plugin, I can get the proper color on class/struct members using the meta.semantic-token.property selector, but it does not appear that there is a more specific semantic selector than meta.semantic-token.variable for other types of variables.
Would it be possible to add a meta.semantic-token.variable.global and/or meta.semantic-token.variable.block selector?

The semantic token types are defined by the LSP specification: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_semanticTokens
And the LSP plugin should support all of them. See https://lsp.sublimetext.io/customization/#semantic-highlighting
This plugin is a bridge to clangd where this need to be implemented. There is already an open issue for more semantic tokens: https://github.com/clangd/clangd/issues/1115


Discrepancy between TimeEntryDtoImpl class members and response from Clockify.me API.

The client library class TimeEntryDtoImpl contains a List<string> TagIds and a string TaskId .
The Clockify.me API returns an TimeEntryDtoImpl object that contains a list of tags objects List<TagDto> TagIds (or an object very similar to a TagDto) as well as a TaskDto Task object (or an object very similar to a TaskDto).

Example Response from _clockifyClient.FindAllTimeEntriesForUserAsync:
[
{
"id": "<removed_for_privacy>",
"description": "test description",
"tags": [
{
"id": "<removed_for_privacy>",
"name": "Testing",
"workspaceId": "<removed_for_privacy>",
"archived": false
}
],
"user": null,
"billable": false,
"task": {
"id": "<removed_for_privacy>",
"name": "AAAATESTTASK",
"projectId": "<removed_for_privacy>",
"assigneeIds": [],
"assigneeId": "",
"estimate": "PT0S",
"status": "ACTIVE",
"duration": "PT3H"
},
"project": {
"id": "<removed_for_privacy>",
"name": "AAAATEST PROJECT",
"hourlyRate": {
"amount": 0,
"currency": "USD"
},
"clientId": "<removed_for_privacy>",
"workspaceId": "<removed_for_privacy>",
"billable": false,
"memberships": [
{
"userId": "<removed_for_privacy>",
"hourlyRate": null,
"targetId": "<removed_for_privacy>",
"membershipType": "PROJECT",
"membershipStatus": "ACTIVE"
}
],
"color": "#C106E4",
"estimate": {
"estimate": "PT0S",
"type": "AUTO"
},
"archived": false,
"duration": "PT3H",
"clientName": "AAAAAATestClient",
"note": "",
"template": false,
"public": false
},
"timeInterval": {
"start": "2020-05-19T10:15:36Z",
"end": "2020-05-19T13:15:36Z",
"duration": "PT3H"
},
"workspaceId": "<removed_for_privacy>",
"hourlyRate": null,
"userId": "<removed_for_privacy>",
"customFieldValues": null,
"projectId": "<removed_for_privacy>",
"isLocked": false
}
]
The member types for the TimeEntryDtoImpl would have to be updated to match the current Clockify.me API.
Let me know if there's anything I can do for ya!

Another great issue!
That looks like I missed quite a lot of data. I'll fix that.  But in docs, I've found: TimeEntryImpl which looks like this:
{
  "billable": "boolean",
  "description": "string",
  "id": "string",
  "isLocked": "boolean",
  "projectId": "string",
  "tagIds": [
    "string"
  ],
  "taskId": "string",
  "timeInterval": {
    "duration": "string",
    "end": "string (date-time)",
    "start": "string (date-time)"
  },
  "userId": "string",
  "workspaceId": "string"
}

Maybe this is a mistake in docs?

I could be wrong but the response posted by @White7292 seems to be to be a response for a hydrated request (see hydrated: true in docs)

Merged :)


Bluetooth: gatt: cancel all prepared writes on invalid write response

On sending write request we should check write response.
If data prepaired to be written by peer or offset are not equal
to the data and offset we sent, we shall send Execute Write
Request with Request Flag set to 0x00 (Cancel All Prepared Writes).
This was affecting GATT/CL/GAW/BI-32-C and GATT/CL/GAW/BI-37-C
Signed-off-by: Krzysztof Kopyściński<EMAIL_ADDRESS>

@jhedberg Ok, fixed it. There was issue with polish signs in my surname

@jhedberg Ok, I fixed coding style issues

@KKopyscinski looks good now, thanks!

Fixed mistake with memcmp error handling

@jhedberg @joerchan Added cancel writes if length is invalid, as bailing out still causes executing prepared writes.


Fix newline endings

Trivial PR... I'm surprised your editor isn't set to automatically add missing newlines at the end of files...

Thanks @marcandre !


Enabling lighthubert with setup.py?

Hello!
Thanks for the great work!
My colleague @edward0804 and I are thinking about integrating lighthubert into S3PRL to enable more research.
Instead of copying all the lighthubert code into S3PRL, we are wondering whether adding a setup.py in this repo would be a good alternative so that we can simply install it, enabling lighthubert in the S3PRL codebase, and link the interested user to this repo for the actual implementation.
I have made a minimal fork for this and so lighthubert can be installed in S3PRL after this commit s3prl/s3prl@07c5bd8692ce481cea5e0190c2cabf759300799b, and @edward0804 is working on adding a wrapper for lighthubert. Do you think it would be nice to add an official setup.py ? :)
Thanks!

Hi!
Thanks for your attention! We are glad to add a setup.py in the repository.
We have added and tested a setup.py after this commit mechanicalsea/lighthubert@8a95a95. Since the S3PRL may support Python >= 3.6, we also tested the lighthubert inference on the Python 3.6, 3.7, and 3.8. Also, we provided requirements and installation in README.md. Once you install lighthubert, it is easy to use it as follows.
from lighthubert import LightHuBERT, LightHuBERTConfig
# ...

If you have any questions or problems, please continue to comment.
Thanks again for your attention. We look forward to having a great S3PRL lighthubert!
Best Wishes.

Cool! Thanks for working on this!
Will let you know after we merge this model!
Sincerely,
Leo