Spiketrainlist object comparison fails in rare cases

In some cases the neo core tests fail due to an object comparison of a list and a spiketrainlist, see

https://github.com/NeuralEnsemble/python-neo/runs/6422039865?check_suite_focus=true
https://github.com/NeuralEnsemble/python-neo/runs/5718538078?check_suite_focus=true
https://github.com/NeuralEnsemble/python-neo/runs/5659519369?check_suite_focus=true.
This error is only happening in some of the test runs, it seems only to occur in tests on windows, but not linked to one python version. @apdavison: Do you have any idea why this fails only in some cases and can we do adjust the spiketrainlist in some way to permit an object comparison like this?

And here it happened again: https://github.com/NeuralEnsemble/python-neo/runs/6524684429?check_suite_focus=true

@apdavison And here it happened again: https://github.com/NeuralEnsemble/python-neo/runs/7540753511
Is this tests supposed to fail with a low probability? There is a comment here # for debugging occasional test failure

@JuliaSprenger I spent some time looking into this a few weeks ago, which is when I added that debugging line (which shouldn't have been merged/pushed, sorry). If I remember correctly, it occurs when you happen to get a spike train list of length 0, and the error is somewhere in the filter functionality, but I didn't get as far as fixing it.

Tests failed again twice for the current master, see https://github.com/NeuralEnsemble/python-neo/actions/runs/3331652474/jobs/5609080600


Import controller must have a try-catch on getMessagesFromFile and display an error to user

Importing a wrong json format does not display an error to user.


Fix a few typos

Patch looks very good! Thanks for the comments! Happy to merge this! :)

Thank you for fixing all the typos! Looks good to me.


clean up disk after failed exports on prod

This seems to be a reoccurring issue. Is it due to limited space on the Azure instance? Do we need to expand the space and implement an automated procedure to clean up the disk.

We should expand the storage directory for jobs to at least 500gb


Cannot reproduce experiment result

Hi,
I tried your demo code at

 Image Recognition/Bayesian_CNN_Detailed.ipynb

But I cannot reproduce your training results, the accuracy stays around 10% and it seems the model is learning nothing.

Would you please advise what could be the problem? Thanks

Hey @Cliu2 ,
What dataset did you use?

@kumar-shridhar Thank you for your reply.
I used the CIFAR10 dataset retrived remotely, following your code in the jupyter notebook.
train_data = datasets.CIFAR10('data', train=True, download=True, transform=transform)
I simply copy all the codes in the notebook to a python script file and run it in terminal. So all the settings should be same.

I am experiencing the exact same issue. It seems that the network is unable to improve on its prediction accuracy due to some bug. I am going to try to look into it as much as I can, but @kumar-shridhar you should check this out since others have posted similar issues.

Hi,
Please check with the current version. Things should work fine.


Media server down for maintenance

Our media server went down because our hosting company's power supply has issues. We are fixing this now.

We issued a temporary fix and will monitor this before closing this issue.


Add treat_match_arm_bodies_as_statements option

treat_match_arm_bodies_as_statements
Format match arm bodies like freestanding statements instead of sub-expressions

Default value: true
Possible values: true, false

true:
match lorem {
    Ipsum(dolor) => if dolor {
        0
    } else {
        1
    },
}

false:
match lorem {
    Ipsum(dolor) => if dolor { 0 } else { 1 },
}

To be honest, after taking some time to acclimate to the new style (I updated rustfmt for the first time since June recently, and felt blindsided by all the changes)—I'd be okay with dropping this as well as #1903.


Confusing error message for multiply defined variables

Take the following shader, where 'stride' is multiply defined in error...
RWStructuredBuffer<float3> positions;

[numthreads(256, 1, 1)]
void computeMain(uint3 globalIdx : SV_DispatchThreadID)
{
    uint count, stride;
    positions.GetDimensions(count, stride);

    int stride = 10;
}

On slang the error reported is
D:\local\Temp\a335d33f-98c1-452f-bc4d-176733908e9c.hlsl(7): error 30019: expected an expression of type 'uint', got 'overload group'
D:\local\Temp\a335d33f-98c1-452f-bc4d-176733908e9c.hlsl(7): error 30047: argument passed to parameter '1' must be l-value.
D:\local\Temp\a335d33f-98c1-452f-bc4d-176733908e9c.hlsl(7): note 30048: argument was implicitly cast from 'overload group' to 'error', and Slang does not support using an implicit cast as an l-value

That's quite a confusing message - far more so than 'multiply defined variable' or something of that kind. Also it does not list the line  int stride = 10; which is pretty important in diagnosing the issue. If there were 100s of lines before that line - the message focuses on only the earlier usage, which would be hard to figure out whats going on.
By contrast dxc outputs
D:\local\Temp\b2dcd9d2-65c9-4683-8103-f281751c9625.hlsl:9:9: error: redefinition of 'stride' with a different type: 'int' vs 'uint'
    int stride = 10;
        ^
D:\local\Temp\b2dcd9d2-65c9-4683-8103-f281751c9625.hlsl:6:17: note: previous definition is here
    uint count, stride;

Which highlights the lines of significance and has a much clearer error message.

Yes, this is a pretty bad error message on our part and is a quality-of-life thing we should find time to fix.
The basic problem is that slang-check.cpp currently doesn't proactively check for duplicate/conflicting definitions (at least not to the extent it should), and instead relies on just emitting an error in cases where those declarations are used ambiguously. In this case, the ambiguous use gets wrapped up in another error message about what we pass to out/intout parameters. It is also tied up in the way that our approach that tries to parse and then apply semantic checking means that the call site for GetDimensions() there can "see" the later declaration of stride, even if it doesn't really make sense.

This should be fixed with #1201.


VMS: Move defining _XOPEN_SOURCE and  _XOPEN_SOURCE_EXTENDED to config target

For all other platforms that need these macros defined, that's how it's
done, so we have VMS follow suit.  That avoids a crash between in source
definitions and command line definitions on some other platforms.
Fixes #24075

I'm running a test build of this on a VMS machine.  It takes a couple of hours...

I'm running a test build of this on a VMS machine. It takes a couple of hours...

It appears to build correctly, so good enough for this PR

Test build on NonStop is in progress. It will take another 45 minutes or so.

Built fine on NonStop

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged
3.3:
7f04bb065d9d948d049ef1ef1bd4062cb7831392 VMS: Move defining _XOPEN_SOURCE and  _XOPEN_SOURCE_EXTENDED to config target
master:
491bbb444c4b654de14adc7031eb44e88a101edb VMS: Move defining _XOPEN_SOURCE and  _XOPEN_SOURCE_EXTENDED to config target


Cannot differentiate simple expression

julia> using Diffractor: var"'"

julia> (x->x*x*x*x)''(1.0)
stmt = :(goto %3 if not %7)
stmt = :(goto %3 if not %7)
stmt = :(goto %3 if not %7)
stmt = :(goto %3 if not %7)
stmt = :(goto %3 if not %7)
ERROR:
Stacktrace:
  [1] error()
    @ Base ./error.jl:42
  [2] transform!(ci::Core.CodeInfo, meth::Method, nargs::Int64, sparams::Core.SimpleVector, N::Int64)
    @ Diffractor ~/.julia/packages/Diffractor/A8Ou2/src/stage1/recurse.jl:709
  [3] perform_optic_transform(ff::Type{Diffractor.∂⃖recurse{2}}, args::Any)
    @ Diffractor ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:25
  [4] (::Core.GeneratedFunctionStub)(::Any, ::Vararg{Any})
    @ Core ./boot.jl:580
  [5] (::Diffractor.∂⃖{2})(::typeof(Base.afoldl), ::Function, ::Vararg{Any})
    @ Diffractor ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:218
  [6] (::Diffractor.∂⃖{2})(::typeof(Core._apply_iterate), ::Function, ::Function, ::Tuple{typeof(*), Float64}, ::Vararg{Union{Tuple, NamedTuple}})
    @ Diffractor ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:364
  [7] *
    @ ./operators.jl:655 [inlined]
  [8] ∂⃖
    @ ~/.julia/packages/Diffractor/A8Ou2/src/extra_rules.jl:85 [inlined]
  [9] #1
    @ ./REPL[2]:1 [inlined]
 [10] ∂⃖
    @ ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:218 [inlined]
 [11] ∂⃖
    @ ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:61 [inlined]
 [12] PrimeDerivativeBack
    @ ~/.julia/packages/Diffractor/A8Ou2/src/interface.jl:157 [inlined]
 [13] (::Diffractor.∂⃖recurse{1})(::Diffractor.PrimeDerivativeBack{1, var"#1#2"}, ::Float64)
    @ Diffractor ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:0
 [14] ∂⃖
    @ ~/.julia/packages/Diffractor/A8Ou2/src/stage1/generated.jl:211 [inlined]
 [15] (::Diffractor.PrimeDerivativeBack{2, var"#1#2"})(x::Float64)
    @ Diffractor ~/.julia/packages/Diffractor/A8Ou2/src/interface.jl:157
 [16] top-level scope
    @ REPL[2]:1

@bolgarbe Please indicate versioninfo() as well, because many issues might be caused by non-recent (I mean bleeding edge) versions of Julia.

I tried it both on 1.7 beta and master with the same error (couldn't install Diffractor on 1.6.2):
Julia Version 1.7.0-beta3.0
Commit e76c9dad42 (2021-07-07 08:12 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-12.0.0 (ORCJIT, skylake)

Julia Version 1.8.0-DEV.271
Commit 8ece8655da (2021-07-27 06:05 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-12.0.1 (ORCJIT, skylake)

The issue here is that the four-argument * uses control flow, which I didn't finish implementing yet for higher-order reverse mode. It works if you add parantheses:
julia> (x->(x*x)*(x*x))''(1.0)
12.0

Fixing the original isn't too hard either, but I have a few other things to work on first.

I also just pushed a better error message for this case:
julia> using Diffractor: var"'"

julia> (x->x*x*x*x)''(1.0)
ERROR: Control flow support not fully implemented yet for higher-order reverse mode (TODO)
Stacktrace:
  [1] error(s::String)
    @ Base ./error.jl:33
  [2] macro expansion
    @ ~/.julia/dev/Diffractor/src/stage1/generated.jl:0 [inlined]
  [3] (::Diffractor.∂⃖recurse{2})(::typeof(Base.afoldl), ::typeof(*), ::Float64, ::Float64)
    @ Diffractor ~/.julia/dev/Diffractor/src/stage1/generated.jl:407


ES Module Error @square/web-sdk

What version of React are you using?
17.0.2
What version of Node.js are you using?
16.4.1
What browser are you using?
Chrome
What operating system are you using?
Windows
How are you deploying your application?
Vercel / Next Start
Describe the Bug
I have installed the required packages (@square/web-sdk, square, and react-square-web-payments-sdk) and have come into an error with NextJS in regard to importing ES modules. It appears to be a problem with the @square/web-sdk integration.
Error: Must use import to load ES Module<EMAIL_ADDRESS>Expected Behavior
I expect the payment form to populate upon receiving tokenizedResponse, applicationId, and locationId, but instead I cannot access the page at all due to the server error. According to the docs, I'm doing everything I need to do.
To Reproduce
npx create-next-app
npm i react-square-web-payments-sdk
npm i @square/web-sdk
npm i square
In a NextJS page, enter the following code:

import * as React from 'react'
import {
    SquarePaymentsForm,
    CreditCardInput
} from 'react-square-web-payments-sdk'

const SdkTest = (props) => {

const applicationId = process.env.SQUARE_APPLICATION_ID
const locationId = process.env.SQUARE_LOCATION_ID
    const cardTokenizeResponseReceied = (tokenReceived) => {
        console.log("Token received:", tokenReceived)
    }
return(
<SquarePaymentsForm cardTokenizeResponseReceived={cardTokenizeResponseReceied} applicationId={applicationId} locationId={locationId}>
                <CreditCardInput />
            </SquarePaymentsForm>
)
}

export default SdkTest

Hi @mckhames this is a problem with NextJS that doesn't support ES Modules, you need to use https://www.npmjs.com/package/next-transpile-modules to be able to use it
Here is an example of a next.config.js:
const withTM = require('next-transpile-modules')(['@square/web-sdk', 'react-square-web-payments-sdk']);

module.exports = withTM();

I proceed to close this issue since there is no activity

Hello! Thank you for your answer - initially I thought it was working, but I am once again receiving the error with my next.config.js configured to your instructions.
./node_modules/react-square-web-payments-sdk/dist/react-square-web-payments-sdk.cjs.development.js:7:13
Module not found: ESM packages (@square/web-sdk) need to be imported. Use 'import' to reference the package instead. https://nextjs.org/docs/messages/import-esm-externals

Import trace for requested module:
./node_modules\react-square-web-payments-sdk\dist\index.js
./pages\test\pay.js

https://nextjs.org/docs/messages/module-not-found

Solution:
Set the experimental > esmExternals flag in next.config.js to "loose", in addition to transpiling modules.
  experimental: {
		esmExternals: 'loose',
	},

I will add it to a page documentation, thanks for this!


Change OuterHtmlVisitor visibility, reduce stream reading on startup

Move xhtml and base html charsets to memory from .properties to reduce stream reading on startup (Android sensitive). Change OuterHtmlVisitor visibility to public. Helps to modify nodes and construct outer HTML in single tree traverse.

Hi - were you able to measure an impact from this? My thought is that the full entities are still loaded on init and that's where most of the time is going to be spent (i.e. only moving the base entities out doesn't seem like it would be impactful).

Hi, yes, full entities are still loaded on init and impact is not recognizable. Second thing is generating changing visibility of OuterHtmlVisitor allows to traverse document tree, update/format/etc and construct outer html in one traverse.

I refactored Entities to load faster. Thanks.


feat: Removing missing keys from files (#113)

Here is my first attempt at removing missing keys.
I had to refactor writeMissingToLanguage into LanguageFileUpdater (is is needed to avoid duplicated reading/writing for adding and deletion of keys), and created separate FileAccessLayer to move file-writing into separate entity (probably reading should also be there, but that's for another time).

Oh, wrong branch name, closing this.


Update resources to account for Wallaby Team tools

Pull Request Template
Please ensure all items below are checked before creating a pull request:

[X] My changes were made in the resources folder, not in the auto-generated README.md file or db folder
[X] My submission is formatted according to the guidelines in the contributing guide
[X] My submission is ordered alphabetically based on the resource name
[X] My submission has a useful description
[X] I have searched the repository for any relevant issues or pull requests

Thank you for your contribution!


[admin] merge to live!

Docs Build status updates of commit 77c7227:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/404.md
:warning:Warning
View
Details

docs/concepts/resource-limits-and-performance-optimization.md
:white_check_mark:Succeeded
View

docs/404.md

Line 0, Column 0: [Warning-custom-404-page] Custom 404 page will be deprecated in future. Please remove the 404.md file to resolve this warning.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Use LatLong instead of LongLat

So this is a mistake for this gem, the ISO 6709 defines that always Longitude must be before than Latitude, so please you should follow this convention in order to avoid problems with sync with clients

visit this link http://matsp888.no-ip.org/~mats/Text/GIS/ISO_FDIS 6709.pdf on the section 6.4 part a) defines the order of this values

If you want to contribute a fix, please do. It will need a major version bump, an UPGRADING document of how to deal with it, etc.

See https://github.com/mongoid/mongoid-geospatial/pull/64 for a proposed solution.

I merged #64.


NSArray element type

Is it possible to pass NSArray element type to JSONModel object?
something like this:
@interface ResponseEntityList : JSONModel<ItemType>
@property (strong, nonatomic) NSArray<ItemType>* data;
@end

ResponseEntityList<VideoItem>* list = [[ResponseEntityList alloc] initWithDictionary: dict error: &err];

Did you take a look in the readme?
You wanna make an array of protocols? How that would make sense with JSONModel?

Of course I did, you even can look into exact sample: https://github.com/icanzilb/JSONModel#model-collections

If you have want to convert a list of models use,
NSArray *modelArray = [ResponseEntityList arrayOfModelsFromDictionaries:jsonArray];

Use one of the following methods:

+arrayOfModelsFromDictionaries:error:
+arrayOfModelsFromData:error:
+arrayOfModelsFromString:error:


Two variants of curvature correction

Currently, there exist two curvature correction procedures, whcih do very similar things.
We should deprecate one, most likely.

As both files have been merged, this issue becomes less relevant (yet still it is, as there is certainly some code reundancy). However, there is no need to keep this as an issue.


Add a clear error message when there are no Certified nodes in the selected region

Description
Add a clear error message when there are no Certified nodes in the selected region instead of showing an error message in the Select Node ID field mentioning that Node ID is required. while there are no nodes to select.
Get Mastodon (29).webm

Fixed
https://github.com/threefoldtech/www-mastodon/pull/95

Verified on https://getmastodon.threefold.io/
A clear error message is shown when there are no certified nodes in the selected region, and the tabs also show that there's a validation issue in it.


Improve MatrixResultsReport to handle internal errors

MatrixResultsReport is confusingly reporting 100% success when errors are encountered that don't produce junit.xml files.
22:38:36   7m 42s matrix-dsyuq3rdz80va Error: Internal error during validation
22:38:36   7m 42s matrix-dsyuq3rdz80va ERROR

22:39:51 MatrixResultsReport
22:39:51   200 / 200 (100.00%)

In this case the exit code is correctly specified as 2, however the results report should include the failure. I think the bug is failures without XML are silently ignored.

This seems quite dangerous for users of flank which rely solely on the junit output. This is a typical scenario in a CI setup.

This seems quite dangerous for users of flank which rely solely on the junit output. This is a typical scenario in a CI setup.

what specifically seems dangerous? The proposed solution? Or the bug?
The idea is to get this information in the junit xml so that failures are reported as failures -- both in the exit code and junit xml.
JUnit XML processing is being moved into the FTL server. Once that happens, Flank will be able to fetch this file from Gcloud storage instead of trying to recreate it manually.

The bug could be dangerous if someone creates a CI task that only looks at the junit output to determine success.
Moving the junit xml processing to the server sounds nice. Is there a ticket to track that work?

I agree re: bug being dangerous.
re: ticket, yes but it's only accessible to Google employees.
Most of the server improvements planned are NDA so I can't list them on GitHub. The tl;dr is that firebase is adding support for test sharding and that will fix this specific issue and many others.

It turns out this bug is also impacting me at work so I expect to have some cycles soon to resolve.

I've spent some cycles investigating this. The root issue is Firebase Test Lab will not generate JUnit XML for many reasons. As a result, JUnit XML alone is not sufficient to determine if a run succeeded or not.
I've updated the Flank exit code logic to print detailed information on matrix failures. This will make it obvious when something weird has happened in the matrix. The fixes have landed in Flank v8.0.1.
The recommendation is to always check the exit code.


Feature/count header

Adds an extra HTTP header to all successful responses with a count of the number of elements operated on while calculating the result of reduction.
This probably makes the count endpoint obsolete and worth removing. I can remove that endpoint as part of this PR and bump semver to 0.2.0 if that's the route we want to take.

I guess we'll also need to update the compliance test suite so that CI continues to pass if we decide to remove the count endpoint.

I suppose the count endpoint might be useful if you only want the count. Otherwise you'd need to request something else and get the count from the header.

Please also update the list of response headers in the README.

Need to also consider how to test this using compliance tests, and more generally how to handle divergence of the API over time.


The banner won't appear on a new tab after you close it once

The banner won't appear on a new tab after you close it once. Seems like an issue with sessionStorage and localStorage.
cc @davepagurek @limzykenneth

Specifically we can use sessionStorage instead of localStorage to store the flag on whether to hide the banner or not so that it will still show up on new browser sessions.

Ah ok, I wasn't sure how permanently we wanted it hidden, but yeah this will be a quick change!


hpack in the nix-shell is the wrong version

> hpack --version
hpack version 0.34.7

But the package.yaml has been generated by 0.35.2; which is the required version, because of the when clause things:
> hpack
WARNING: package.yaml: Ignoring unrecognized field $.when[0].then.language
crem.cabal was generated with a newer version of hpack, please upgrade and try again.

Ah, my bad... locally I'm using the newer version of hpack.
It looks like that in the unstable channel the 0.35.2 version is already available.
How do I tell Nix to use that?

I was a bit baffled too; I tried deleting the flake.lock to get it to re-grab the latest unstable, but it seems to have not worked; (I got the same version of hpack).
I think through judicious use of nix why-depends it's possible to work, but I can't work out the magic invocation to get that telling me something useful:
> nix why-depends --derivation .#crem.ghc90 /nix/store/df2cz2rckr5ib0zd0s4sjscwcyrlmi8x-hpack-0.34.7/bin/hpack
/nix/store/phn79wczkzk4g94g10i3jqi0wi65pwvn-crem-<IP_ADDRESS>.drv
└───/nix/store/5kwigqn71adzakhb5rhfpgh7k1zs4mqc-hpack-0.34.7.drv

I think it's coming in from a particular version of GHC. I don't know why why-depends isn't showing that, though.

I think it's coming in from a particular version of GHC. I don't know why why-depends isn't showing that, though.

On my local machine I'm using GHC 9.0.2 with hpack 0.35.2... maybe something related to how those things are connected in nixpkgs?


Fix c++17 build

Hi,
C++17 deprecated the usage of "throw(std::bad_alloc)" which causes error when building with gcc 7 and -std=gnu++17.
Here is a small patch set to fix that:

Rename PERFTOOLS_THROW into PERFTOOLS_NOTHROW (less counter intuitive name)
Add PERFTOOLS_THROW where necessary (as detected by GCC)
Replace "throw()" by "PERFTOOLS_NOTHROW" (not necessary, but I find it cleaner to always use the same syntax IMO).

Cheers,
Romain

Thanks for the patch. Can you please squash latest commit into second and move PERFTOOLS_THROW out of public header ? Other than that, your patch looks perfect.

I have moved PERFTOOLS_THROW out of public header in a new commit. Is this OK for you or do you want to merge that with earlier commits ?

Yes, I prefer clean history.
On Jul 3, 2017 05:48, "Romain Geissler @ Amadeus"<EMAIL_ADDRESS>wrote:

I have moved PERFTOOLS_THROW out of public header in a new commit. Is this
OK for you or do you want to merge that with earlier commits ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gperftools/gperftools/pull/897#issuecomment-312636781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA6Aod29Q6VQ8ev1yXtchBOUhXIfJsJks5sKOMbgaJpZM4N9fxQ
.

Done

Hm. Somehow it broke compilation on visual studio and mingw. Looks like win32 port.h is doing #define __THROW PERFTOOLS_NOTHROW and somehow either necessary include is missing or something else is doesn't work.

I have reverted the change in src/windows/port.h now.
I am unable to test under windows, so please tell me if it's fine now.

Applied. Thanks. I made slight change in your last commit (and updated message accordingly) to only define empty XXX_NOTHROW if standard is less than c++11. Less confusing this way.


When I login Windows Virtual Desktop through https://rdweb.wvd.microsoft.com/webclient/index.html ,  I found I can’t import text by Japanese and some other language except English.

When I login Windows Virtual Desktop through https://rdweb.wvd.microsoft.com/webclient/index.html ,
I found I can’t import text by Japanese and some other language except English.
Is it a bug?
Can you check the issue through your environment and update this issue.

Thank you for the feedback! Because this is not related to docs.microsoft.com, we cannot address the issue in this forum.


Usafe method to turn Option<T, TException> to Option while throwing TException

Sometimes, exception is not a bad thing. A code within library could return Option<T, TException>, but the exception cannot be properly handled in calling code, so just fall-back to C# exception handling by throwing this exception like normally using unsafe method.
One problem is that TException can be of any type and not just Exception. My suggestion is : If TException is Exception, then throw InvalidOperationException (or something similar) and exception as inner exception. If it isn't Exception, then create new Exception type and save the exception as object within it.

Actually.  It would work if the Option<T, TException> was tri-state : value, null or exception. Which it is not.
You can close this then.


Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
Originally posted by @welcome[bot] in https://github.com/github/docs/issues/3902#issuecomment-783672183

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
Originally posted by @WELCOME[bot] in #3902 (comment)

Oki


Add support for authentification with OpenID Connect

You're off to a great start with adding the flags. But I think this is best maintained in your own fork.
I've had to support OIDC in prior works, its a weighty burden, and I personally don't use it. Adding the flags opens me up to issues getting different providers to work (e.g. https://github.com/poseidon/typhoon/issues/440). Often those providers are partially broken or neglected by their megacorp owner. Integration quality would be lackluster at best. I'm trying to keep a high bar here, but that also means being laser focused on areas I find important and dropping areas that aren't a good use of time for a free project. But I do wish you luck!


⚠ Bofrost.de has degraded performance

In a723c4a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1298 ms

Resolved: Bofrost.de performance has improved in 42f8788.


GH Workflow for cutting release hangs on "looking for dynamic libraries"

broken: https://github.com/OpenGOAL-Mods/OG-ModLauncher/actions/runs/8102365819/job/22144679678
used to work: https://github.com/OpenGOAL-Mods/OG-ModLauncher/actions/runs/7417114517/job/20183118600#step:5:70

fixed by downgrading pyinstaller<5.0
other notes:

PySimpleGUI==4.60.5 pinning this version because they decided to go with a paid model in v5+ 🙃
python upgraded to 3.10

we cannot use 3.11 because that has some pre-release crap that break pyinstaller
we cannot use 3.12 because of outdated imports to imp within pyinstaller (renamed to importlib)

JK everything was broken due to pysimpleGUI v5 update 🤑 I kept that pinned 4.60.5
pyinstaller is back to latest version, python bumped up to 3.12


I18n initial setup -Rails

I18n Initial Setup In Rails
Results
using full names instead of acronym for locale

translating teachers page

adding locale to circuitverse api

translating simulator top menu

changes made in pr
1>add locale attribute in user api
2> introducing locale attributes in JSON Schema
3> ensuring  all api rspec tests run and  adapts the change of addition of locale attribute in JSON Schema
4>structure directory to store translations efficiently
5> associating full names to 100+ locales
6>use full name instead of 2 word locale in form selector
7> translate teachers and simulator page
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Great 👍


feat(dal,sdf): upgrade components to new schema variants

Adds the ability to take a component based on one schema variant and
migrate it to another schema variant without component deletion. Adds an
sdf route that facilitates this.
Frontend code not commited, resources have not been tested, and one
important feature is missing from this: the deletion and restoration of
edges to the new providers if they are compatible. But otherwise this
reliably "respins" a component onto a new variant.

bors r+


[Bug] Popper Media - 'Gear icon' opens Setting page with error

clicking on the 'gear icon'

takes you to Settings with this error...

Ah, thanks!

Solved, 2.18.3


Swagger builds in travis should not go get ciao

Currently, the swagger part of the travis builds in ciao do this
   - go list ./... | sed -e "s/github\.com\/01org\/ciao\/vendor\///" | xargs go get

i.e., they download the latest version all of ciao's dependencies into the GOPATH.
The problem is that this is not guaranteed to work.  Any build failure upstream in any of ciao's dependencies will break the ciao build, even though ciao itself does build, as it uses the vendored  packages.  The reason we do this go get is because the github.com/yvasiyarov/swagger package does not correctly handle vendoring.  We need to either fix this package or switch to another swagger package that does handle vendoring.
I've seen ciao builds fail from time to time because of this so it needs to be fixed.

I'd prefer to send the fix for that, later today I'll be sending the PR to the https://github.com/yvasiyarov/swagger , once the PR is merged I'll update our travis

Will your fix be using go list?

no, I'm fixing swagger to use vendoring dir as it should, so we remove the go list ... line

PR sent to fix swagger tool to use vendoring dir:
https://github.com/yvasiyarov/swagger/pull/120
I only need confirmation of the merge to send the changes to our travis tool.
The new change will be to remove the line
- go list ./... | sed -e "s/github\.com\/01org\/ciao\/vendor\///" | xargs go get
and specify our vendor diretory to be used.

@leoswaldo Are there any alternative swagger packages that we can use that don't need to be patched?

Updated swagger PR to add native support of vendoring


【Question】Is there a checklist that shows conflicting tools?

As limited by PMU counter, we can't start 2 instance of a single tool. For example, ./pcm.x &; ./pcm.x will report error.
What's more, some pcm tools share the same register/counter. For example,  we can't run pcm and pcm-numa at the same time.
Is there a checklist which could show us how these tools conflict with each other?
More detailed, is there a readme which could tell us which counters/registers  are used by a specified tool?

Generally running two pcm tools in parallel is not supported. There are exceptions but they are not documented (and not guaranteed to be supported in future). If you have a specific question for the current version which utilities you want to run together, please let me know.

Here is one situation that I wanna to collect metrics with pcm tools when I run a benchmark. And running the benchmark is really time consuming. If pcm tools can be run in panrallel, I can run the benchmark just 1 time to collect all the metrics. If not, I have to run it many times.
So I want to know which registers/counters a centain pcm tool use and how they conflict with each other?
At least, can I run pcm.x and pcm-memory.x at the same time?
Any idea I can go farther to find out how can I ruduce the running times of the benchmark when collecting metrics with pcm tools?

pcm and pcm-memory can't be run together. For the other requests (documentation) I will put it on the list.


Does not fetch password if there is a / character at the end in the Jamf Pro url

Hi there!
I have tried the app and I am thrilled with it!
I found a problem with using it and wanted to report it.
if I enter in the jamf server url field an address ending with / (for example: https://myserver.jamfcloud.com/ ), when I edit or retrieve the LAPS settings these work without any problems; but if I try to retrieve the Local Administrator Password, I get the error: "Could not find this computer, please check the serial number." it works if I go to remove the / character at the end of the jamf server url.
Thank you very much for you work.

Thanks for the feed back, will make this change for the next version.


exception while inserting row

Hello.
I've got exception while i am trying to insert row
Error while inserting row: An attempt to execute batch_mutate failed 6 times. The last error was exception 'cassandra_UnavailableException' in /home/sergey/workspace/test/cassandra/test/include/phpcassa/thrift/packages/cassandra/Cassandra.php:871 Stack trace: #0 /home/sergey/workspace/test/cassandra/test/include/phpcassa/thrift/packages/cassandra/Cassandra.php(871): thrift_protocol_read_binary(Object(TBinaryProtocolAccelerated), 'cassandra_Cassa...', false) #1 /home/sergey/workspace/test/cassandra/test/include/phpcassa/thrift/packages/cassandra/Cassandra.php(846): CassandraClient->recv_batch_mutate() #2 [internal function]: CassandraClient->batch_mutate(Array, 1) #3 /home/sergey/workspace/test/cassandra/test/include/phpcassa/connection.php(354): call_user_func_array(Array, Array) #4 /home/sergey/workspace/test/cassandra/test/include/phpcassa/columnfamily.php(617): ConnectionPool->call('batch_mutate', Array, 1) #5 /home/sergey/workspace/test/cassandra/test/columnfamily_action.php(512): ColumnFamily->insert('test', Array) #6 {main}
Is that my mistake or your lib bug ?

cassandra_UnavailableException ->     Not all the replicas required could be created and/or read.
read here: http://wiki.apache.org/cassandra/API
You have probably choosen Network topology on your Strategy, which requires at least one Replica, which you probably don't have, but have only one server?

Yeah try simple topology strategy instead :)


Should opening a definition focus it in the namespace tree?

A common occurrence for me is that I want a method for say Map that I assume already exists but I don't know what it's called. I search for Map and see ~10 results but none of them are the function that I'm looking for. It feels like I should be able to accept the top Map data type result and then scroll down in the left pane to look at the rest of the methods under the Map namespace. But the left pane isn't actually focused on Map, so I scroll down to M, realize that it's not there, scroll to data, click it, and then scroll to Map.

What would "focus" mean in this suggestion? That data is open and Map has a visual indicator? Or do you mean that we should open data and scroll to it?

@hojberg yeah, I was talking about scrolling to it. In general as I switch between definitions I find that I expect the left side to scroll to the one that I'm on (though I still want to be able to scroll it manually; I just want it to line up when I shift focus in the right pane). But I don't know the rules of UI - maybe this would be bad.


[Removal]: Huge Topaz Cat

Hotspot Name
Huge Topaz Cat
Hotspot b58 Address
11wXAMH9WAnjfB5SqRqevYer1P5qcxHyfr7xPmpjFsumdZfhmcx
Discord Handle
No response
Hotspot Manufacturer
panther
Removal Reason
Now that it has been redeployed, you can check and verify this through your technical test methods. Please remove it from the deny list for me, thank you.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
No response

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Removing extraneous file.

This file was renamed to ExposureConfigurationDefault.json, but somehow this one, with the original name, stayed in the project.

I have verified that this file is now unused in the app. I think this should be approved and merged so that there is no confusion on what config to update in the future. But I will defer to @timarney here.


Fix ntile function should check argument

Proposed changes
Problem:
when ntile using 0 as parameter, be would core because no checking of parameter
Solved:
check parameter in fe analyze

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall


feat: update wallets page

As talked about in issue #59, I have made some updates to the wallets page. I added an Active indicator and also moved the buttons inline. This matches better with the Figma design and I think looks a bit better.
Before:

After:

Great stuff @secondl1ght! I agree with Daniel, let's see if we can get a bit closer to the design and make the wallets page shine! 😊 ✨

Ok I made more changes, here's how it looks now:

Extra sparkles are always good! Here is the latest screenshot, let me know if this is what you had in mind:


Script to download geckodriver for Windows

Hey,
I saw you linked to a script that would handle the install process of geckodriver for Linux (and mac) users, so I thought I would offer a script for windows users as well.
This PowerShell script will:

Download the latest geckodriver zip (for the correct Architecture) to a temp folder
Unzip the exe
Check if geckodriver.exe already exists in the install directory ($env:APPDATA\geckodriver)

If it does the script will check file hashes to see if the new exe is different

If they are identical the script moves on to step 5

Move the exe to the install directory
Remove the temp directory
Add the install directory to the user path (if it isn't already in the path)

The script can be found here: https://gist.github.com/SonOfDiablo/81f3d610295c69c777b512e4da90393d
There is two versions, one with comments and progress logging, and one without those things.

That's amazing! I need to level up my Powershell for sure. Did you make this? Cause it's incredible

Thanks :blush:
Yea I wrote it just now, it could probably be cleaned up a bit (and maybe optimized a tad), but it gets the job done.

Dude, that's amazing! Thank you so much for contributing that! I'll be sure to give you some creds in the documentation. Glad you enjoyed this enough to make something for it!

Added you to the documentation. Thanks again! This means alot, and I'm sure there are a lot of thankful people!


Multiplayer Mod Desyncs Caused by UnityEngine.Random

See referenced line of code: https://github.com/emipa606/MountainMiner/blob/0f6cd269db30ae9526a1eb5627b4d4f9bf77a51b/Source/MountainMiner/Building_MountainDrill.cs#L42
Current implementation uses UnityEngine.Random.Range which will always desync multiplayer clients. Changing to Verse.Rand.Range  will otherwise allow this mod to work seamlessly with multiplayer.
Multiplayer Mod (Steam Workshop)

Should be added in 3794508
Thanks!


[PE-D][Tester D] Editing a person to have the same name as another existing person

In the UG under Editing a person, it is stated that any value is possible for NAME. But, the app does not allow for duplicate contacts and will return "This person already exists in the address book." if we try to edit a contact to have the same name.

Perhaps can change the UG description to indicate that editing name to an existing person's name is invalid?

Labels: severity.Low type.FunctionalityBug
original: glenngnng/ped#2

the current behavior inconveniences the user but there is a way to work around it (i.e. searching for the name first before using it). Will make it more explicit in the UG.


Add React single-page sample app

Purpose

This PR is for #DOCS-CODE-023 - React single-page app . It imports the sample from https://github.com/Azure-Samples/ms-identity-javascript-react-docs

Cc: @dhruvchand @ckittel @henrymbuguakiarie @mmacy

@Dickson-Mwendia I just merged #23 from @henrymbuguakiarie but now wondering if we need to implement the review changes here into that. Maybe I should have merged this one?
Not sure - can you two sort it out and let me know?


Time of use Switch

Hi
Can you advise if there is a way to enable/disable the time of use switch on the inverter from the API?
Many thanks

by setting the times as you have suggested (00:00 to 00:00) then this will turn that slot off. Note you have 3 slots that need to filled, each slot can have a discharge and charge time. This basically does the same thing as the menus on the solid inverter itself and after you have used it then you can check on the inverter

I use this in homeassistant to estimate how much I'll need to put into my battery based on solcast forecast for the day ahead (+ a correction factor as this always seems to over-predict), the current house usage (in case I need a boost to get me to when my panels start to charge) and my estimate for the following day. Not perfect, but it just runs as an automation and gets me most of the way there.

@service
async def solis_control(demand):
    config = pyscript.config["apps"]["solis_control"]["config"]

    battery_kwh = float(state.get(config['battery_soc']['entityId']))*0.096
    min_battery_kwh = 9.6*0.20
    buffer_kwh = 0.25
    tomorrow_prediction = float(state.get(config['solar_tomorrow']['entityId'])) * 0.85
    house_power = min(0.5,float(state.get(config['house_power']['entityId']))/1000.0)

    car_plugged = state.get(config['charger_plug']['entityId']) == 'on'
    log.warning("is car plugged?"+str(car_plugged))

    now = datetime.now()
    next_charge_window = (now + timedelta(days=1)).replace(hour=4, minute=30, second=0)
    start_charge_window = (now + timedelta(days=1)).replace(hour=0, minute=30, second=0)
    solar_start =  (now + timedelta(days=1)).replace(hour=7, minute=20, second=0)

    hours_next_charge_window = (next_charge_window - now).total_seconds()/3600.0
    battery_load = battery_kwh - (hours_next_charge_window * house_power)
    solar_time = (solar_start - next_charge_window).total_seconds()/3600

    today_index = next_charge_window.weekday()

    battery_available_charge_kwh = min_battery_kwh if car_plugged else max(min_battery_kwh, battery_load)
    solar_load = solar_time * house_power

    today_demand = demand[today_index]

    morning_shortfall_kwh = solar_load + min_battery_kwh + buffer_kwh - battery_available_charge_kwh
    log.warning("today demand:"+str(today_demand)+ " min battery:"+ str(min_battery_kwh+buffer_kwh )+ " available:"+str(battery_available_charge_kwh)+ " tomorrow gen"+str(tomorrow_prediction))
    afternoon_shortfall = today_demand + min_battery_kwh + buffer_kwh - battery_available_charge_kwh - tomorrow_prediction

    log.warning("morning shortfall:"+ str(morning_shortfall_kwh) +" afternoon shortfall:"+ str(afternoon_shortfall))
    shortfall_kwh = max(morning_shortfall_kwh, afternoon_shortfall)
    times = []
    if (shortfall_kwh > 0):
      log.warning("shortfall:"+str(shortfall_kwh) + " (demand="+str(today_demand)+", house power:"+ str(house_power)+ ", battery: "+str(battery_kwh)+", "+str(battery_load)+", "+str(battery_available_charge_kwh)+")")
      charge_secs = shortfall_kwh*3600/2.5
      start_time = max(start_charge_window,next_charge_window - timedelta(seconds=charge_secs))
      log.warning("start charge at:"+start_time.strftime("%a, %d %b %Y %H:%M:%S"))
      times.append({"chargeCurrent":"50", "dischargeCurrent":"50", "chargeStartTime":start_time.strftime("%H:%M"), "chargeEndTime":next_charge_window.strftime("%H:%M"), "dischargeStartTime":"00:00", "dischargeEndTime":"00:00"})
      times.append({"chargeCurrent":"50", "dischargeCurrent":"50", "chargeStartTime":"00:00", "chargeEndTime":"00:00", "dischargeStartTime":"00:00", "dischargeEndTime":"00:00"})
      times.append({"chargeCurrent":"50", "dischargeCurrent":"50", "chargeStartTime":"00:00", "chargeEndTime":"00:00", "dischargeStartTime":"00:00", "dischargeEndTime":"00:00"})
    else:
      # clear any charge times that may be set
      log.warning("no shortfall - clearing")
      times.append({"chargeCurrent":"50", "dischargeCurrent":"50", "chargeStartTime":"00:00", "chargeEndTime":"00:00", "dischargeStartTime":"00:00", "dischargeEndTime":"00:00"})
      times.append({"chargeCurrent":"50", "dischargeCurrent":"50", "chargeStartTime":"00:00", "chargeEndTime":"00:00", "dischargeStartTime":"00:00", "dischargeEndTime":"00:00"})
      times.append({"chargeCurrent":"50", "dischargeCurrent":"50", "chargeStartTime":"00:00", "chargeEndTime":"00:00", "dischargeStartTime":"00:00", "dischargeEndTime":"00:00"})
    inverterId= getInverterList(config)
    token = login(config)
    set_control_times(token, inverterId, config, times)

then I just set it up to run at 23:00 each evening. The 0.096 is my battery size. You can see that I just have a set cheap window for octopus and only use one charging window, but that should give you an idea... and you can see I've been quite lazy with the "solar start"... I should probably get that from the prediction...

to get it working, just make suer you set all the configuration up in the main configuration.yaml for homeassistant in the pyscript section
# enable python script
pyscript:
  allow_all_imports: true
  hass_is_global: true
  apps:
    solis_control:
      config:

the demand part is simply passed in with the automation yaml. Hope that makes sense

@viking2010 - is it okay to close this issue now?


[ML] chore: Bump dev version of azure-ai-ml to 1.12.0

Description
This pull request bumps the in development version of the azure-ai-ml SDK to 1.12.0 and adds a new section to the CHANGELOG
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Merge master as of August 19

@rossabaker would you take a quick look?
Still missing circe update to work.
(cc @ChristopherDavenport )

LGTM.  I'm going to merge this and http4s/http4s#1326 and upgrade to fs2-reactive-streams-0.2.3.

Can you merge this? It's my fork :)


Implement bin (command) projects.

This PR updates cargo component new to default to bin (command) projects rather than lib (reactor) projects, much like cargo does.
This is a breaking change if you've been using cargo component new before, as it will now create a "command" that simply has a main that prints "hello, world!" instead of a reactor.
To continue to create reactors, pass the --lib option to cargo component new.
This commit also updates the preview2 adapters for commands/reactors and also all dependencies to latest.

@sunfishcode since you've been experimenting with targeting WASI with cargo-component, would you mind reviewing?
This implements cargo component new --bin foo so that cargo component build creates an adapted command that prints "hello, world!", instead of a reactor.


fix install alloyu - linux | grafana.repo using RHEL-Fedora

PR Description
Which issue(s) this PR fixes

Notes to the Reviewer
PR Checklist

[ ] CHANGELOG.md updated
[ ] Documentation added
[ ] Tests updated
[ ] Config converters updated

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

@Guihgo Can you please sign the CLA agreement, and then we will get this fix into the documentation.


remove global import

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.0%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
1148

Relevant Lines:
1225

💛  - Coveralls


Feat/ng standalone component update

Description
Upgrade angular standalone components
General Tests for Every PR

[ ] npm run start still works.
[ ] npm run lint passes.
[ ] npm run stylelint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build still works.

@fernando182gomez can you add a list of which components are now standalone, the modules that will be deprecated and which components are still needing to be done?


Add adjustment to freegeoip to use freegeoip.app

Since freegeoip was deprecated a similar service called freegeoip.app provides the same results, the only thing mandatory is to use https
This PR makes adjustments to work with :freegeoip as before without changes for the end user
I was thinking about creating a new API provider but since freegeoip is deprecated, simply modifying it could bring it to life again
The same can be achieved without this PR, using:
Geocoder.configure(
  ip_lookup: :freegeoip,
  freegeoip: { host: "freegeoip.app" },
  use_https: true
)

Great pull request--thanks!


Fix bit counting in risc0_zkvm::guest::zkvm::sha::digest

As discussed in https://github.com/risc0/risc0/issues/356, the SHA-256 example started to fail it's
test during the update to risc0 v0.12. There are two causes for this.

There is an issue with the bit count where the message is serialized to a buf of u32 words, but
the bit count is calculated as buf.len() * 8.
The semantics of of the digest, which in https://github.com/risc0/risc0-rust-examples/pull/45
was originally used to replace a usage of digest_u8_slice method, actually returns a hash of
the serialization of a given struct rather than directly hashing it's bytes. This is an issue in
usage and not the digest method itself, so I added a comment to clarify this and hopefully
avoid other folks making the same mistake.

Fixes: https://github.com/risc0/risc0/issues/356

Note, I marked this as a draft until I ensure there is a test that will cover this function. Will do that today.

This seems like a whole lot of copies in the guest to me.  I kind of think we should just get rid of the fn digest<T: Serialize>(...).  It doesn't seem very useful to serialize something, and then only use that serialization for generating a digest.

Sounds good to me. It's not clear to me that this is a API folks need, and it is somewhat easy to misuse (e.g. feeding a string in to it and not getting the expected result). I'd be happy to remove it.

This seems fine to me as well.

Removed this method in 2af13f11e365abb9381a1be3bb6e65eea96f49fd to be included in https://github.com/risc0/risc0/pull/330


community: improve Neo4j connection lifecycle management

Description:
This PR improves the Neo4j graph integration by implementing proper connection lifecycle management. The changes ensure that Neo4j database connections are properly managed and cleaned up, preventing resource leaks and improving the robustness of the integration.
Key improvements:

Add explicit close() method to terminate Neo4j connections
Implement context manager support with __enter__ and __exit__
Add runtime checks to prevent operations on closed connections
Add comprehensive lifecycle tests

These changes align with Neo4j driver best practices and provide users with clear patterns for managing database connections.
Dependencies:

No new dependencies required
Existing dependency: neo4j>=5.0

Testing:

Added comprehensive test suite in test_neo4j.py
Tests cover:

Context manager usage
Explicit connection closing
Error handling for closed connections
Connection state validation

Few unrelated test failures with make test:
======================================================================================================== short test summary info ========================================================================================================
FAILED tests/unit_tests/llms/test_llamafile.py::test_call_raises_exception_on_missing_server - pytest_socket.SocketBlockedError: A test tried to use socket.socket.
FAILED tests/unit_tests/tools/playwright/test_all.py::test_playwright_tools_schemas - ModuleNotFoundError: No module named 'playwright'
====================================================================================== 2 failed, 1068 passed, 531 skipped, 695 warnings in 13.71s =======================================================================================
Failed to batch ingest runs: langsmith.utils.LangSmithError: Failed to POST https://api.smith.langchain.com/runs/batch in LangSmith API. HTTPError('403 Client Error: Forbidden for url: https://api.smith.langchain.com/runs/batch', '{"detail":"Forbidden"}')
make: *** [test] Error 1
NOTE: I saw a separate PR referencing community neo4j deprecation, but not really informed on the details of this. Either way, this provides a more graceful shutdown for clients still using this lib and aligns with neo4j driver expectations.

Hi @jtanningbed! I just wanted to let you know that the neo4j integration has moved to https://github.com/langchain-ai/langchain-neo4j. I'd be happy to review your PR if you'd like to submit it there instead.


Get the jexcel object from the html element

Is it possible to get the jexcel object from the html element that contains it?
Something like $('#my').jexcel(...) that was possible with the jquery version?

Yes, you can.
https://bossanova.uk/jexcel/v3/examples/jquery


Is it possible to use it in releases as mentioned in the summary?

When trying to use it in a release, I get the following error:
##[error]This task must run in a build to publish artifacts to Azure Pipelines/TFS.
which seems in contradiction with the initial sentence Use this task in a build or release pipeline..
Is it an error in the documentation, or are there limitations?

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

ID: 0b3f565b-88e4-88d6-a534-48ec21e6e3d8
Version Independent ID: 9fca89ae-4a6e-a43a-2834-1a91ad44c256
Content: Publish Build Artifacts task - Azure Pipelines & TFS
Content Source: docs/pipelines/tasks/utility/publish-build-artifacts.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

As of now, Publish build artifacts tasks can be used only in build pipelines.

parallel & parallelCount don't seem to have any effect?

If I'm right, I should prefer using PublishPipelineArtifact, shouldn't I?
A message with a link in the documentation would help.
Then I think that we could close this issue.

I believe we updated the text here, thanks Roopesh! @fleed, are you saying a blurb with "if you want to use this in a release pipeline, use Pipeline Artifacts instead." would be helpful? Adding @vijayma to confirm that if you want to publish in a release pipeline, you must use Pipeline Artifacts.

@elbatk it seems that even the other task doesn‘t work in a release pipeline

so is there a way to publish artifacts that are generated from a RELEASE (not build) pipeline?
I want to be able to create an output file in stage 1, then download that in stage 2 and deploy directly the image verses creating an image from scratch in each stage...

@DavidNewby Thanks for the questions. I believe the PublishPipelined Artifacts Should work for release Pipelines. If you having issues there, you should open a new issue under that page or reach out to the  Developer Community. I'll be closing this issue for now :)


Support sending a single "retry" line

The intended use case is that when an SSE connection is established, send a single "retry:1000" (without any data associated to it) then wait till an event reaches.

Thanks for your contribution. While I agree that sending a retry makes sense, the current design makes it impossible for clients to unmarshal it.

I see. I'll try modifying the client. Do you have a preference in design?

Done by https://github.com/hseeberger/akka-sse/pull/142.

Cool!


How to measure the depth of an object?

Hello this is my first time for realsense.I don't know how to start.For find  the depth of an obj.I'm use python . sorry my english is so bad

@tee68   Don't worry, your English is very good.  :)
Have you been able to install the pyrealsense2 program that provides Python compatibility for RealSense, please?
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/python

@tee68  Do you still need help with this case please?   Thanks!

It's work.thank you
On Thu, Jun 18, 2020, 7:39 PM MartyG-RealSense<EMAIL_ADDRESS>wrote:

@tee68 https://github.com/tee68 Do you still need help with this case
please? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6523#issuecomment-645990895,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKGI7HTBX4RMVVUSWMM3D63RXIDHJANCNFSM4NTJMJEA
.


Polish command navigation for "empty commands" for non-partial terminal capabilities

This PR fixes #155543
This PR implements the skipping of empty commands in the case of the terminal with non-partial command detection capabilities. The result is:

However in the case of partial command detection capabilities (i.e. IPartialCommandDetectionCapability ) I was not able to implement it. I don't fully understand when terminals with partial detection capabilities occurs, so, I couldn't trigger and investigate them.

Hi @Tyriar, while working on this PR I notice a possible bug that happens when during selection a boundary of the terminal is reached. I've opened a separate issue for this bug (issue #156071) with a PR associated to fix it.

Hi @Tyriar, I have improved the behaviour of the skipping empty commands.
New behaviour is the following:


oneprovider.com

@adguard-bot commented on Sat Jul 03 2021
Issue URL (Incorrect Blocking)
https://oneprovider.com/dedicated-servers/paris-france
Screenshots

Screenshot 1

Screenshot 2

System configuration

Information
value

AdGuard product:
AdGuard for Windows v7.6.1

Browser:
Chrome

AdGuard driver:
WFP

Stealth mode:
disabled

DNS filtering:
disabled

Filters:
Ad Blocking:AdGuard BasePrivacy:AdGuard Tracking ProtectionSocial Widgets:AdGuard Social MediaLanguage-specific:AdGuard Chinese

Custom filters:
https://easylist-downloads.adblockplus.org/abp-filters-anti-cv.txt,https://easylist-downloads.adblockplus.org/easylistchina+easylist.txt

Userscripts:
https://userscripts.adtidy.org/release/adguard-extra/1.0/adguard-extra.user.js,https://userscripts.adtidy.org/release/popup-blocker/2.5/popupblocker.user.js

Referrer:
https://oneprovider.com/dedicated-servers-in-europe

To reproduce this issue, please scroll website a bit and please disable this rule (from AdGuard Base):
@@||oneprovider.com^$elemhide,jsinject,extension
Screenshot

Scripts are injected before <!doctype, looks strange.


Add object mode check for operations

Maybe we can add this check in code generation?

Need implementation in these services:

[ ] s3
[ ] gcs
[ ] azblob
[ ] oss
[ ] qingstor
[ ] fs
[ ] dropbox
[ ] cos
[ ] kodo
[ ] uss

ref: https://github.com/aos-dev/go-service-qingstor/pull/48

@Prnyself please keep tracking on the implementations.

@Prnyself please keep tracking on the implementations.

I will update every ref after the PR merged.


How to use it with Expo ?

how can I use the "react-native-background-task" if I use the expo?

You can't use native libraries with expo unless you eject your project.

Expo has it's own background task implementation that you can use:
https://docs.expo.io/versions/latest/sdk/task-manager/

@a-zen Does expo task manager works even when app is closed/terminated?

@a-zen Does expo task manager works even when app is closed/terminated?


demo.py doesn't work

Hello,
After I installed all required modules, and followed the steps you had specified; I got the following exception. Would you mind helping me to run? Thanks.
`(tensorflow)ilithefallen@ubuntu:~/Documents/phdThesis/libs/MultiNet$ python demo.py --gpus 0 --input data/demo/um_000005.png
2017-05-03 18:36:22,177 INFO No environment variable 'TV_PLUGIN_DIR' found. Set to '/home/ilithefallen/tv-plugins'.
2017-05-03 18:36:22,177 INFO No environment variable 'TV_STEP_SHOW' found. Set to '50'.
2017-05-03 18:36:22,177 INFO No environment variable 'TV_STEP_EVAL' found. Set to '250'.
2017-05-03 18:36:22,177 INFO No environment variable 'TV_STEP_WRITE' found. Set to '1000'.
2017-05-03 18:36:22,177 INFO No environment variable 'TV_MAX_KEEP' found. Set to '10'.
2017-05-03 18:36:22,177 INFO No environment variable 'TV_STEP_STR' found. Set to 'Step {step}/{total_steps}: loss = {loss_value:.2f}; lr = {lr_value:.2e}; {sec_per_batch:.3f} sec (per Batch); {examples_per_sec:.1f} imgs/sec'.
2017-05-03 18:36:22,179 INFO GPUs are set to: 0
2017-05-03 18:36:22,179 INFO Download URL: ftp://mi.eng.cam.ac.uk/pub/mttt2/models/MultiNet_ICCV.zip
2017-05-03 18:36:22,179 INFO Download DIR: RUNS

Downloading MultiNet_ICCV.zip 100.0%
2017-05-03 18:55:18,499 INFO Extracting MultiNet_pretrained.zip
2017-05-03 18:55:31,855 INFO f: <_io.TextIOWrapper name='RUNS/MultiNet_ICCV/hypes.json' mode='r' encoding='UTF-8'>
2017-05-03 18:55:31,855 INFO f: <_io.TextIOWrapper name='RUNS/MultiNet_ICCV/road/hypes.json' mode='r' encoding='UTF-8'>
2017-05-03 18:55:31,864 INFO f: <_io.TextIOWrapper name='RUNS/MultiNet_ICCV/detection/hypes.json' mode='r' encoding='UTF-8'>
Traceback (most recent call last):
File "demo.py", line 413, in
tf.app.run()
File "/home/ilithefallen/tensorflow/lib/python3.4/site-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(sys.argv[:1] + flags_passthrough))
File "demo.py", line 311, in main
load_out = load_united_model(logdir)
File "demo.py", line 254, in load_united_model
postfix=model)
File "incl/tensorvision/utils.py", line 213, in load_modules_from_logdir
data_input = imp.load_source("input%s" % postfix, f)
File "/home/ilithefallen/tensorflow/lib/python3.4/imp.py", line 171, in load_source
module = methods.load()
File "", line 1220, in load
File "", line 1200, in _load_unlocked
File "", line 1129, in _exec
File "", line 1471, in exec_module
File "", line 321, in _call_with_frames_removed
File "RUNS/MultiNet_ICCV/detection/data_input.py", line 22, in
from utils.data_utils import (annotation_jitter, annotation_to_h5)
File "/home/ilithefallen/Documents/phdThesis/libs/MultiNet/submodules/KittiBox/incl/utils/data_utils.py", line 11, in
import annolist.AnnotationLib as al
ImportError: No module named 'annolist'`

Looks like the code does not find the module annolist. It should be in the folder KittiBox. Try to locate the file and see why the code cant find it.


Added BellmanFord Algorithm

Bellman Following are the detailed steps.
Input: Graph and a source vertex src
Output: Shortest distance to all vertices from src. If there is a negative weight cycle, then shortest distances are not calculated, negative weight cycle is reported.

This step initialises distances from the source to all vertices as infinite and distance to the source itself as 0. Create an array D[] of size[V] with all values as infinite except D[S] where src is source vertex.
This step calculates shortest distances. Do following |V|-1 times where |V| is the number of vertices in given graph.
a)   Do following for each edge u-v
………………If D[v] > D[u] + weight of edge uv, then update D[v]
………………….D[v] = D[u] + weight of edge uv
This step reports if there is a negative weight cycle in graph. Do following for each edge u-v
……If D[v] > D[u] + weight of edge uv, then “Graph contains negative weight cycle”
The idea of step 3 is, step 2 guarantees the shortest distances if the graph doesn’t contain a negative weight cycle. If we iterate through all edges one more time and get a shorter path for any vertex, then there is a negative weight cycle
How does this work? Like other Dynamic Programming Problems, the algorithm calculates shortest paths in a bottom-up manner. It first calculates the shortest distances which have at-most one edge in the path. Then, it calculates the shortest paths with at-most 2 edges, and so on. After the i-th iteration of the outer loop, the shortest paths with at most i edges are calculated. There can be maximum |V| – 1 edges in any simple path, that is why the outer loop runs |v| – 1 times. The idea is, assuming that there is no negative weight cycle, if we have calculated shortest paths with at most i edges, then an iteration over all edges guarantees to give shortest path with at-most (i+1) edges

Royal eagel, [04.10.21 06:49]
Hi Dear KARAN ALLAGH
Show your support on my youtube channel web Pixar(The institute of coding)
like , share , subscribe
telegram - https://t.me/Royaleagle11
Royal eagel, [04.10.21 06:49]
youtube - https://youtu.be/5qnQjgdpFPg

Sure
Thanks for your help
Can you please also add the hactoberfest-accepted label


Remove about version variant

Task/Issue URL: https://app.asana.com/0/488551667048375/1206767754058357/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4248  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joshliebe and the rest of your teammates on  Graphite


feat(theme): createTheme self containment

Platform Theme Separation - Part 1

Part 1
Part 2
Part 3

Monolith PR: https://github.com/codecademy-engineering/Codecademy/pull/24393
Part 1

createTheme
The multi keyed object that this previously returned was confusing and this change ensures that the return object includes all integral pieces for the theme to work.  This ensures that the consuming GamutProvider (or some future standalone component in variance) can simply add variables if they are present on the theme as opposed to requiring them to be passed.
The new version will return just the theme (all methods and extra stuff are on it under _ prefixed keys.  e. g. theme._getColorValue(x).
// before
const { theme } = createTheme().build();

<GamutProvider theme={theme} /> // No colors everything broken

// After
const theme = createTheme().build();

<GamutProvider theme={theme} /> // Everything works

Further this adds a few quality of life improvements for token constants making configuration less annoying when dealing with  nested objects / scales (i. e. Color Weights).New utility flattenScale The recursively flattens an object to kebab case keys. gray[200] => gray-200  and you may now pass a deeper object to some methods and receive a strongly typed map back.
const object = {
  gray: {
    '200': '#eeeeee',
     '600': '#666666',
  }
}

// flattentScale(object)
const output = {
  'gray-200': '#eeeeee',
  'gray-600': '#666666',
}

Ensures that simpler types now support extension.  A theme that is returned by createTheme().build() can be extended again with createTheme with no loss.

// valid
const base = createTheme(x).addScale().build();
// also valid but extra!
const extended = createTheme(base).addScale().build();

GamutProvider

Passing theme is now required by GamutProvider
The theme passed to gamut provider will automatically set global variables from the _variables key.

Modes

Background will now cycle through any arbitrary amount of color modes without specifically requiring their names.
Background now is agnostic to the theme that is passed using only the methods / tokens available on the supplied context to check contrast.
Exports many new types for colors Colors + ColorModes etc.
Adds new hooks

useCurrentMode(modeOverride?: ColorMode): ColorMode Optionally takes an override mode and returns the value of either the override or the current context.
useColorMode():  Returns all relevant mode keys and _getColorValue.

Color Modes now do not have unnecessary depth on the them and are now top level keys.  This wasn't ergonomic for utilities or general use.

//before
const theme = {
  colorModes: {
      modes: { light: {}, dark: {} },
      active: 'light'
  }
}

// after
const theme = {
  modes: { light: {}, dark: {} },
  mode: 'light',
}

PR Checklist

[x] Related to designs: Figma
[x] Related to JIRA ticket: https://codecademy.atlassian.net/browse/GM-174 https://codecademy.atlassian.net/browse/GM-228
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-icons@4.12.3-alpha.042f66.0@codecademy/gamut-illustrations@0.14.4-alpha.042f66.0@codecademy/gamut-kit@0.3.101-alpha.042f66.0@codecademy/gamut-labs@16.0.6-alpha.042f66.0@codecademy/gamut-patterns@0.1.4-alpha.042f66.0@codecademy/gamut-styles@10.0.1-alpha.042f66.0@codecademy/gamut-tests@2.3.31-alpha.042f66.0@codecademy/gamut@32.18.8-alpha.042f66.0@codecademy/konamimojisplosion@0.2.31-alpha.042f66.0@codecademy/markdown-overrides@0.5.43-alpha.042f66.0@codecademy/styleguide@37.5.9-alpha.042f66.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://609dde60492a1140fe03ce0c--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.12.3-alpha.deee7d.0@codecademy/gamut-illustrations@0.14.4-alpha.deee7d.0@codecademy/gamut-kit@0.3.102-alpha.deee7d.0@codecademy/gamut-labs@17.0.1-alpha.deee7d.0@codecademy/gamut-patterns@0.1.4-alpha.deee7d.0@codecademy/gamut-styles@10.0.1-alpha.deee7d.0@codecademy/gamut-tests@2.3.31-alpha.deee7d.0@codecademy/gamut@32.18.8-alpha.deee7d.0@codecademy/konamimojisplosion@0.2.31-alpha.deee7d.0@codecademy/markdown-overrides@0.5.43-alpha.deee7d.0@codecademy/styleguide@38.0.1-alpha.deee7d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://609eae0f61eaec11d6111c8d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.12.3-alpha.7b9161.0@codecademy/gamut-illustrations@0.14.4-alpha.7b9161.0@codecademy/gamut-kit@0.3.102-alpha.7b9161.0@codecademy/gamut-labs@17.0.1-alpha.7b9161.0@codecademy/gamut-patterns@0.1.4-alpha.7b9161.0@codecademy/gamut-styles@10.0.1-alpha.7b9161.0@codecademy/gamut-tests@2.3.31-alpha.7b9161.0@codecademy/gamut@32.18.8-alpha.7b9161.0@codecademy/konamimojisplosion@0.2.31-alpha.7b9161.0@codecademy/markdown-overrides@0.5.43-alpha.7b9161.0@codecademy/styleguide@38.0.1-alpha.7b9161.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://609ee175d9c2e63386c27628--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.12.3-alpha.df6f8d.0@codecademy/gamut-illustrations@0.14.4-alpha.df6f8d.0@codecademy/gamut-kit@0.3.104-alpha.df6f8d.0@codecademy/gamut-labs@17.1.2-alpha.df6f8d.0@codecademy/gamut-patterns@0.1.4-alpha.df6f8d.0@codecademy/gamut-styles@10.0.1-alpha.df6f8d.0@codecademy/gamut-tests@2.3.31-alpha.df6f8d.0@codecademy/gamut@32.18.9-alpha.df6f8d.0@codecademy/konamimojisplosion@0.2.31-alpha.df6f8d.0@codecademy/markdown-overrides@0.5.43-alpha.df6f8d.0@codecademy/styleguide@38.1.2-alpha.df6f8d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60a2e0bffd3d983803441bcf--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.13.0-alpha.df0012.0@codecademy/gamut-illustrations@0.14.4-alpha.df0012.0@codecademy/gamut-kit@0.3.104-alpha.df0012.0@codecademy/gamut-labs@17.2.0-alpha.df0012.0@codecademy/gamut-patterns@0.2.0-alpha.df0012.0@codecademy/gamut-styles@10.1.0-alpha.df0012.0@codecademy/gamut-tests@2.4.0-alpha.df0012.0@codecademy/gamut@32.19.0-alpha.df0012.0@codecademy/konamimojisplosion@0.2.31-alpha.df0012.0@codecademy/markdown-overrides@0.5.43-alpha.df0012.0@codecademy/styleguide@38.2.0-alpha.df0012.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60a3fe64c741dc00a62a7f12--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.13.0-alpha.bc6e93.0@codecademy/gamut-illustrations@0.14.4-alpha.bc6e93.0@codecademy/gamut-kit@0.3.104-alpha.bc6e93.0@codecademy/gamut-labs@17.2.0-alpha.bc6e93.0@codecademy/gamut-patterns@0.2.0-alpha.bc6e93.0@codecademy/gamut-styles@10.1.0-alpha.bc6e93.0@codecademy/gamut-tests@2.4.0-alpha.bc6e93.0@codecademy/gamut@32.19.0-alpha.bc6e93.0@codecademy/konamimojisplosion@0.2.31-alpha.bc6e93.0@codecademy/markdown-overrides@0.5.43-alpha.bc6e93.0@codecademy/styleguide@38.2.0-alpha.bc6e93.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60a4078b4697711016d358f9--gamut-preview.netlify.appDeploy Logs


Hello World

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/edit
Operating System: windows
I have been successful to the point of Hello World.  I am not receiving that prompt response

@Otter1479 can you give more information about this? What prompt response you're not receiving?
Did you at least see Hello World on your screen?


[rank2]: AttributeError: module 'torch.distributed' has no attribute 'nn'

when run the train.py, encounters the problem: [rank2]: AttributeError: module 'torch.distributed' has no attribute 'nn'.

it can be solved by adding this line: import torch.distributed.nn as nn_dist in model/model_longclip.py file.
by the way, @beichenzbc can u share the training log file, i want to see whether can reproduce the result.
thanks


Unable to use react-select inside my custom compoent

Describe the bug
I'm trying to use react-select in one of my components, but an error is thrown
Installed libraries and their versions

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
To Reproduce
Steps to reproduce the behavior:

Bundle the following component:

import * as React from 'react';
import { EditPropertyProps, DropZone, FormGroup, Label, DropZoneItem } from 'admin-bro'
import * as Select from 'react-select';

export default function ManyToManyEditComponent({ property, record, onChange }: EditPropertyProps){
  const { custom, label } = property;

  return(
    <FormGroup>
      <Label> {label} </Label>
      <Select options={[]}/>
    </FormGroup>
  )
}

See error

Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
in Unknown (created by BasePropertyComponent)
in section (created by styled.section)
in styled.section (created by BasePropertyComponent)
in ErrorBoundary (created by BasePropertyComponent)
in BasePropertyComponent (created by New)
in section (created by styled.section)
in styled.section (created by New)
in form (created by styled.section)
in styled.section (created by New)
in New (created by BaseActionComponent)
in ErrorBoundary (created by BaseActionComponent)
in BaseActionComponent (created by ResourceAction)
in section (created by wrapper__StyledWrapper)
in wrapper__StyledWrapper (created by Wrapper$1)
in Wrapper$1 (created by ResourceAction)
in ResourceAction (created by ConnectFunction)
in ConnectFunction (created by Context.Consumer)
in Route (created by App)
in Switch (created by App)
in section (created by styled.section)
in styled.section (created by App)
in section (created by styled.section)
in styled.section (created by App)
in App
in Router (created by BrowserRouter)
in BrowserRouter
in ThemeProvider
in Provider global.bundle.js:510:31
Expected behavior
react-select is bundled with react-admin, so we should be able to use it.

No it's not bundled,
add this to your dependencies:     "react-select": "^3.1.0",
and try again.

@ayushbhat thx for your reply, I added it to my dependecies file and the same error persisted.

change
import { EditPropertyProps, DropZone, FormGroup, Label, DropZoneItem } from 'admin-bro'
to
import { EditPropertyProps } from 'admin-bro' import {DropZone, FormGroup, Label, DropZoneItem} from '@admin-bro/design-system'

@ayushbhat I did as you suggested but the same issue persisted

any updates about this ?

import Select from 'react-select' doesn't work?
in the end you can use it from window.ReactSelect like const ReactSelect = window.ReactSelect since it is bundled there.
All the things related to problems with bundler will be handled in this task: #614

@wojtek-krysiak Thx, your proposed solution works, you saved me.

The actual solution to this issue is importing react-select the following way in TS, so it gets bundled !
 import * as Select from 'react-select'


New Machine request for Deep AQAtik GSoC work

New machine request:

New machine operating system (e.g. linux/windows/macos/solaris/aix): linux
New machine architecture (e.g. x64/aarch32/arm32/ppc64/ppc64le/sparc): x64
Provider (leave blank if it does not matter):
Desired usage: for machine learning prototype
Any unusual specification/setup required: no
How many of them are required: 1

Please explain what this machine is needed for:
Mentioned previously, but had not yet created a request for a separate machine where we can run and develop the Deep AQAtik prototype (partially described here).  We had been running this ML prototype on the same machine as TRSS, but that is not required or desired, as we will be making many changes and do not want to destabilize the TRSS machine.

Keys for @llxia @LongyuZhang and Giannis have been added to the aqatik user

@llxia @LongyuZhang @ichristod @smlambert Is this machien working ok? I haven't got it in the system inventory file at the moment so I'll get it added if it's up, running, and working as desired to close this off :-)

Thanks @sxa for the help. The machine is working well. We will work with a university professor to implement new ml architecture on this machine in the coming weeks. I think we can close this issue for now. Thanks.

Closing as completed.


#2109 Fix embedded json escaping for Grafana 5.5.0-5.1.3

Fix for issue #2109
We were able to figure out it was a parsing error of the marathon Mustache template, the fix is to escape the bit of embedded json in MESOS_MODULES. (As it is also done for other packages like, like Prometheus, see: https://github.com/mesosphere/universe/blob/version-3.x/repo/packages/P/prometheus/1/marathon.json.mustache#L56 )

Hmm, any guidance on how to prevent these checks from failing?
I executed build.sh locally without error, and was able to successfully build this version into a local universe for testing the new grafana template.

@jdijt Only Mesosphere employees can trigger the TeamCity tests, which I've done now on your behalf.


Dicom conversion

BigPicture uses DICOM: https://bigpicture.eu/news/bigpicture-raises-dicom-standards and it is likely to become an industry standard.
I can write a converter if you want, but it might be tricky to know which libraries to import.
Shall I draft something?

Yeah, I agree it would probably be a good idea for something like this to exist.
(Should an iSyntax-to-DICOM converter then be within the scope of libisyntax, or maybe a separate project?)
As for libraries, if I were to go about it I would probably try to use libjpeg-turbo for the JPEG encoding and try to use as few as possible additional libraries, like I did for adding (partial) support for reading DICOM WSIs in Slidescape (see the code in here). But there might be easier/better ways to do it.

It could also be simpler to have either openslide bindings or python bindings directly. Extending https://github.com/imi-bigpicture/wsidicomizer/tree/main/wsidicomizer would then make it very easy

Yes, I think extending an existing WSI to DICOM converter could be a good strategy. Then iSyntax would be just one more backend to support.

Writing it without dependencies is likely painful: https://github.com/GoogleCloudPlatform/wsi-to-dicom-converter

I am fairly confident it would be painful, only thinking about the number of hours I already spent reading the DICOM standard... ;-)

If openslide incorporates support for isyntax (through libisyntax) conversion to DICOM should be straightforward with what is already implemented in WsiDicomizer, as it already has openslide support.

That’s true, but that still seems to require quite a bit of work. It might be much easier at this point to make a python binding. It’s on my todo

I'm playing around with python bindings at this moment using cython, and have managed to read out tiles. Not sure how easy it is to package though.

I got a converter running using pyisyntax.
Will explore parsing metadata from the xml header. I have previously parsed some philips xml metadata from isyntax files converted to tiff, and could get some DICOM required attributes out of it

This is pretty sweet, does your DICOM converter also support its own downsampling starting at level 0? That would solve #36 (offset between levels).
An additional idea would be to have a .get_offset(level) function so we can manually offset annotations as well.

For the DICOM metadata, probably you can just make a few simple adjustments to the libisyntax code to expose those. Something like GetDicomTag

We could add some accessors to expose the relevant fields. Or maybe allow invoking a callback procedure while parsing the XML header.

This is pretty sweet, does your DICOM converter also support its own downsampling starting at level 0? That would solve #36 (offset between levels).

It can re-create a full pyramid if levels are missing.

An additional idea would be to have a .get_offset(level) function so we can manually offset annotations as well.

With annotations, do you mean graphical annotations? In what dimensions are those (pixels or meters)?

For the DICOM metadata, probably you can just make a few simple adjustments to the libisyntax code to expose those. Something like GetDicomTag

For converting to DICOM, there are some tags that are difficult to get and for the user to supply, for example

Image orientation: How the pyramid levels are rotated with respect to the label, but this might be the same for all isyntax files.
Total pixel matrix origin sequence: Where to first pixel in the pyramid is positioned in relation to the label corner, reference.

Is such metadata available in the isyntax XML?

Erik-
I never found the offset in the FIC (isyntax xml).
Interestingly, the lowest-level WSI image is never superimposed on the macro (jpeg) image in either the local PIFV viewer or in IMS, suggesting any alignment is imprecise, possibly scanner-specific.
Both viewers display small entire-slide-thumbnails, apparently simply the macro image concatenated with the label image.
I am guessing the line between those two jpeg images is considered the image column margin.
The slidescape team can probably confirm whether the WSI image matrix origin is the same as the "join" between label and macro images.
(I have glass slides I can measure to see how those (jpeg) images align with the true label edge.)

For converting to DICOM, there are some tags that are difficult to get and for the user to supply, for example

Image orientation: How the pyramid levels are rotated with respect to the label, but this might be the same for all isyntax files.
Total pixel matrix origin sequence: Where to first pixel in the pyramid is positioned in relation to the label corner, reference.

Is such metadata available in the isyntax XML?

The image orientation is described in Philips' file format specification document (see the attached document for details, specifically chapter 2 and the appendix).
4522 207 43941_2020_04_24 Pathology iSyntax image format.pdf
The origin of the pyramid is specified under the UFS_IMAGE_GENERAL_HEADERS tag. Here is a dump of the relevant part of the XML (dumped using Slidescape) for testslide.isyntax:
     DICOM: UFS_IMAGE_GENERAL_HEADERS                (0x301d, 0x2000), array
      Array
       DataObject ObjectType = UFSImageGeneralHeader
        DICOM: UFS_IMAGE_NUMBER_OF_BLOCKS               (0x301d, 0x2001), size:6        = 173421
        DICOM: UFS_IMAGE_DIMENSIONS_OVER_BLOCK          (0x301d, 0x2002), size:9        = 1 0 4 2 3
        DICOM: UFS_IMAGE_DIMENSIONS                     (0x301d, 0x2003), array
         Array
          DataObject ObjectType = UFSImageDimension
           DICOM: UFS_IMAGE_DIMENSION_NAME                 (0x301d, 0x2004), size:1        = x
           DICOM: UFS_IMAGE_DIMENSION_TYPE                 (0x301d, 0x2005), size:7        = spatial
           DICOM: UFS_IMAGE_DIMENSION_UNIT                 (0x301d, 0x2006), size:10       = MicroMeter
           DICOM: UFS_IMAGE_DIMENSION_SCALE_FACTOR         (0x301d, 0x2007), size:4        = 0.25
          element end: DataObject
          DataObject ObjectType = UFSImageDimension
           DICOM: UFS_IMAGE_DIMENSION_NAME                 (0x301d, 0x2004), size:1        = y
           DICOM: UFS_IMAGE_DIMENSION_TYPE                 (0x301d, 0x2005), size:7        = spatial
           DICOM: UFS_IMAGE_DIMENSION_UNIT                 (0x301d, 0x2006), size:10       = MicroMeter
           DICOM: UFS_IMAGE_DIMENSION_SCALE_FACTOR         (0x301d, 0x2007), size:4        = 0.25
          element end: DataObject
          DataObject ObjectType = UFSImageDimension
           DICOM: UFS_IMAGE_DIMENSION_NAME                 (0x301d, 0x2004), size:9        = component
           DICOM: UFS_IMAGE_DIMENSION_TYPE                 (0x301d, 0x2005), size:16       = colour component
           DICOM: UFS_IMAGE_DIMENSION_DISCRETE_VALUES_STRING (0x301d, 0x2008), size:13       = "Y" "Co" "Cg"
          element end: DataObject
          DataObject ObjectType = UFSImageDimension
           DICOM: UFS_IMAGE_DIMENSION_NAME                 (0x301d, 0x2004), size:5        = scale
           DICOM: UFS_IMAGE_DIMENSION_TYPE                 (0x301d, 0x2005), size:5        = scale
          element end: DataObject
          DataObject ObjectType = UFSImageDimension
           DICOM: UFS_IMAGE_DIMENSION_NAME                 (0x301d, 0x2004), size:11       = waveletcoef
           DICOM: UFS_IMAGE_DIMENSION_TYPE                 (0x301d, 0x2005), size:11       = waveletcoef
           DICOM: UFS_IMAGE_DIMENSION_DISCRETE_VALUES_STRING (0x301d, 0x2008), size:19       = "LL" "LH" "HL" "HH"
          element end: DataObject
         element end: Array
        element end: Attribute
        DICOM: UFS_IMAGE_DIMENSION_RANGES               (0x301d, 0x200a), array
         Array
          DataObject ObjectType = UFSImageDimensionRange
           DICOM: UFS_IMAGE_DIMENSION_RANGE                (0x301d, 0x200b), size:13       = 13531 1 52442
          element end: DataObject
          DataObject ObjectType = UFSImageDimensionRange
           DICOM: UFS_IMAGE_DIMENSION_RANGE                (0x301d, 0x200b), size:13       = 22053 1 96804
          element end: DataObject
          DataObject ObjectType = UFSImageDimensionRange
           DICOM: UFS_IMAGE_DIMENSION_RANGE                (0x301d, 0x200b), size:5        = 0 1 2
          element end: DataObject
          DataObject ObjectType = UFSImageDimensionRange
           DICOM: UFS_IMAGE_DIMENSION_RANGE                (0x301d, 0x200b), size:5        = 0 1 7
          element end: DataObject
          DataObject ObjectType = UFSImageDimensionRange
           DICOM: UFS_IMAGE_DIMENSION_RANGE                (0x301d, 0x200b), size:5        = 0 1 3
          element end: DataObject
         element end: Array
        element end: Attribute
       element end: DataObject
      element end: Array
     element end: Attribute

In the example of testslide.isyntax, the (padded) level 0 pyramid starts at (13531, 22053) and has a padded width/height of (38912, 74752).
In libisyntax, the relevant part of the XML is parsed here:
https://github.com/amspath/libisyntax/blob/b9f0cd980a93d07602caa3536c7fef9245ae2fd9/src/isyntax/isyntax.c#L520-L540

Erik- I never found the offset in the FIC (isyntax xml). Interestingly, the lowest-level WSI image is never superimposed on the macro (jpeg) image in either the local PIFV viewer or in IMS, suggesting any alignment is imprecise, possibly scanner-specific. Both viewers display small entire-slide-thumbnails, apparently simply the macro image concatenated with the label image. I am guessing the line between those two jpeg images is considered the image column margin. The slidescape team can probably confirm whether the WSI image matrix origin is the same as the "join" between label and macro images. (I have glass slides I can measure to see how those (jpeg) images align with the true label edge.)

The macro image has its origin at (0, 0) in the coordinate system used by the iSyntax files. The offset of the WSI pyramid can be read from the XML header as described above.
The label image is rotated 90 degrees compared to the macro image so that the text on the label is right-side up (see the specification document for details).


Add compress_pickle support to PickleDataSet

Description
Add compress_pickle support to PickleDataSet.
Development notes

Add import/BACKENDS like a "joblib".
Tested to save/load .lz4 in ipython.
Checklist

[x] Read the contributing guidelines
[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[x] Updated the documentation to reflect the code changes
[x] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
[x] Added tests to cover my changes

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

How about adding dill as a backend lib as well?
@lorenabalan WDYT?

Hi @takeru sorry about the delay. Had a quick chat with the team and it would make more sense if this was a separate dataset class instead of another backend, and specify its dependency in extra requires, similarly to what we do with pandas. Maybe pickle.CompressPickleDataSet? We realise there's already an unfortunate precedent with joblib in there, so we're planning on refactoring that (in a breaking-change release) using the same mental model.

It was decided to extend the existing dataset by supporting a new backend.

I switched from master to develop to pick up fixes related to CI.

Switched back to the master branch.


How to use the data type of char

My DLL has a way of doing it, this is:
int dc_card(int icdev,unsigned char _Mode,unsigned long *_Snr)
How I should define and use the parameters of the char type in node?

I think char can be placed by int in node


Add a systemtest, testing certificate rollover

Overview
Add a test for doing a certificate rollover as described in: https://docs.openshift.com/container-platform/3.11/install_config/redeploying_certificates.html#redeploying-all-certificates-current-ca
Tasklist

Write one or more system tests
Document issues uncovered by the test
Enhance documentation including this scenario, maybe describe additional steps which need to be performed by the administrator

@ctron Is this closeable now?

so I won't do that for ocp3.11, it requires many stuff installed like ansible, openshift-ansible etc... for ocp4 it is more posible and I also think we have a system in systemtests which deploys custom certificate of ocp

Ah, I thought of this in the context of IoT where I think @ctron did implement something in the tests.

TBH I don't know for which reason it should be created, description of issue is vague

For IoT it is actually fixed now. And it should be the same for OpenShift 4.x.


Doesn't work on big-endian

I'm compiling on FreeBSD 13.0 on powerpc64 (big-endian). I'm filing this issue since you yourself ask at https://github.com/zkcrypto/merlin/blob/main/src/lib.rs#L10

 error:
       This crate doesn't support big-endian targets, since I didn't
       have one to test correctness on.  If you're seeing this message,
       please file an issue!
  --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/merlin-3.0.0/src/lib.rs:10:1
   |
10 | / compile_error!(
11 | |     r#"
12 | | This crate doesn't support big-endian targets, since I didn't
13 | | have one to test correctness on.  If you're seeing this message,
14 | | please file an issue!
15 | | "#
16 | | );
   | |_^
error: could not compile `merlin` (lib) due to previous error
Error: warning: build failed, waiting for other jobs to finish...
Error: The process '/home/runner/.cargo/bin/cross' failed with exit code 101

during complication of mips-unknown-linux-musl and mipsel-unknown-linux-musl


Support imports in VS Code

This should be done after #1331.

This is complicated.
Playground webview renders current text in the editor, even unsaved text, so it can't read files from disk (even if it could; webviews can't read files anyway, and any content should be sent to webviews with events).
But with imports, playground should also get contents of imported files somehow, and also update when the files have changed.
Preliminary plan:

webview will ask the extension for all dependencies it parsed from code, with events
extension will send events with the content for dependencies (read from disk), which the playground will add to its project object
extension will also monitor changes to the files (with onDidChangeTextDocument and so on) and send updated versions to the playground (but only the files that are needed by the playground, so we should keep track of those somehow; we don't want to send all edited code to every squiggle webview panel)
async handling of all this is going to be quite messy, since the playground shouldn't render errors if files are not loaded yet


Use Sections for OOP JIT

This change adds new SectionAllocWrapper classes, which manages Sections for JIT code pages.
When you call Alloc with these APIs, sections are mapped into the runtime process with PAGE_EXECUTE permissions. For SectionAllocWrapper, we maintain a map of outstanding sections modeled after the GC's HeapBlockMap for fast lookup.
If you want to map a section into the JIT process, you can call AllocLocal, which will map (a portion) of your section in as PAGE_READWRITE. It is the callers responsibility to call FreeLocal on the local section mapping.
Sections don't support decommit, which is used most importantly in PreReservedSectionAllocWrapper. Instead, to remove pages from the working set we ZeroMemory and set PAGE_NOACCESS protection on the runtime view.
InterpreterThunkEmitter was also modified to use sections. On the server we map everything in and then throw away the section handle. When the runtime is done, it can directly call to UnmapViewOfSection.
Now for OOP JIT we use the thunkBlocks lists for tracking outstanding interpreter thunks rather than maintaining an EmitBufferManager on the JIT process.

@satheeshravi @LouisLaf @leirocks can you take a look?

You need to specify new allocator types in lib/Common/Memory/PageAllocatorDefines.h seeing as how this file contains the Constants for lib/Common/Memory/PageAllocator.h

@kphillisjr I was thinking reusing PageAllocatorType_CustomHeap should be fine?

I believe the reason for the Allocator types is to help with finding/diagnosing memory problems.  This is why I believe this type of change should include a custom entry because others might use it elsewhere... That said, I believe that PageAllocatorType_SectionAlloc is reasonable enough.

:shipit:


input image is not normalized

it seems the input image is not normalized to [0,1] in the demo.

it seems an error, I will check it later.


Search does not work properly

Related #15
I have also found that search does not work.
Here's an example from another project.
Try to search for django:
https://webargs.readthedocs.io/en/latest/search.html?q=django&check_keywords=yes&area=default
No results.
But, we can clearly see the word "Django" on the very first page: https://webargs.readthedocs.io/en/latest/
Actually, no queries work at all.

@sobolevn it is fixed. Please update to 0.7.2