Index was outside the bounds of the array.

how to identify what is causing this issue?
[Error - 6:41:18 AM] Please report this issue to https://github.com/microsoft/al/issues including information on how to reproduce it, if possible.
Processing of message 'al/createPackage' failed with error: 'Index was outside the bounds of the array.'
StackTrace:
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ReportMetadataEmitter.RemoveUtf8Preamble(String content) in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ReportMetadataEmitter.cs:line 126
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ReportMetadataEmitter.ConsumeLayout(XmlWriter xmlWriter, PropertyKind propertyKind, Func2 converter, String xmlMetadataConstant, String defaultLayout) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ReportMetadataEmitter.cs:line 105    at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ReportMetadataEmitter.ConsumeReportLayouts(XmlWriter xmlWriter) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ReportMetadataEmitter.cs:line 54    at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ReportMetadataEmitter.<>c__DisplayClass1_0.<EmitMetadata>b__0(XmlWriter xmlWriter) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ReportMetadataEmitter.cs:line 47    at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitter1.EmitMetadata(Action1 content) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\GenericObjectMetadataEmitter.cs:line 37    at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.EmitMetadata(ObjectTypeSymbol containingType) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 251    at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.CompileObject(ObjectTypeSymbol containingType) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 243    at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.<>c__DisplayClass16_0.<CompileObjectAsTask>b__0() in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 211    at Microsoft.Dynamics.Nav.CodeAnalysis.Utilities.UICultureUtilities.<>c__DisplayClass5_0.<WithCurrentUICulture>b__0() in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Utilities\UICultureUtilities.cs:line 152    at System.Threading.Tasks.Task.Execute() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.WaitForWorkers() in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 161    at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.CompileMethodBodies(Compilation compilation, ModuleBuilder moduleBeingBuiltOpt, Boolean hasDeclarationErrors, DiagnosticBag diagnostics, Predicate1 filterOpt, CancellationToken cancellationToken) in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 145
at Microsoft.Dynamics.Nav.CodeAnalysis.Compilation.CompileImpl(ModuleBuilder moduleBuilder, Boolean emittingPdb, DiagnosticBag diagnostics, Predicate1 filterOpt, CancellationToken cancellationToken) in C:\Agent\_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compilation\Compilation.cs:line 816    at Microsoft.Dynamics.Nav.CodeAnalysis.Compilation.Compile(ModuleBuilder moduleBuilder, Boolean emittingPdb, DiagnosticBag diagnostics, Predicate1 filterOpt, CancellationToken cancellationToken) in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compilation\Compilation.cs:line 791
at Microsoft.Dynamics.Nav.CodeAnalysis.CommandLine.CommonCompiler.PackageTheModule(Compilation compilation, CompilerTextWriter consoleOutput, CancellationToken cancellationToken, ErrorLogger errorLogger, AnalyzerDriver analyzerDriver) in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\CommandLine\CommonCompiler.cs:line 551
at Microsoft.Dynamics.Nav.CodeAnalysis.CommandLine.CommonCompiler.RunCore(CompilerTextWriter consoleOutput, ErrorLogger errorLogger, CancellationToken cancellationToken) in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\CommandLine\CommonCompiler.cs:line 414
at Microsoft.Dynamics.Nav.CodeAnalysis.CommandLine.CommonCompiler.Run(CompilerTextWriter consoleOutput, CancellationToken cancellationToken) in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\CommandLine\CommonCompiler.cs:line 313
at Microsoft.Dynamics.Nav.EditorServices.Protocol.LanguageServer.Extensions.CreatePackageRequestHandler.d__3.MoveNext() in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\LanguageServer\Extensions\CreatePackageRequestHandler.cs:line 32
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.EditorServices.Protocol.RequestRegistry.d__6.MoveNext() in C:\Agent_work\46\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\Endpoints\RequestRegistry.cs:line 59

@A-R-Shaji . Which version of the platform and the compiler are you using?

AL Compiler version 0.15.18771
Version 12.0.21812

Hi @A-R-Shaji, are you still getting the same error when using the latest extension from the marketplace (version 0.16.16948)? The issue is related to the layout of one of your reports. If the issue still reproduce with the newer version of the extension, can you try to identify which report is failing and provide us with some repro to investigate it?

Closing this issue as I could not reproduce it. If you can still reproduce it with the latest AL extension, please log a new issue with some repro steps so that we can investigate it.


DEFI App won't synchronise the blockchain

Hello guys,
I'm trying to use the defi chain app on my Pc (surface pro with 16G RAM)...I’m having trouble on synchronization of the blockchain, it stays stucked to 0 blocks synchronised...No firewall...I left the app running overnight but still no result...is there any of you might help?
The strange thing is that the total numbers of blocks are slowly increasing...but the sync is sucked to 0...

Hi @gabriello79 , may I know if you're on the v2.1.15? Regarding the issue Error: Corrupted block database detected, we can further discuss on the problem there. Kindly send your app logs, screenshot of the error & task manager if possible, as this will provide insights to your issue. Thank you.

Hello,
actually I'm not getting any error...only the sync is stuck into 0.00%
Find some screenshot and the log file.
Thanks for your precious help!

App log.zip

Had this today as well. Was wondering but figured out that this is caused by not well synced peers.
Solutionfor me: After app start directly add a peer / node that is synced. As I am running a node to create these blockchain snapshots you can just add my by the following console command:

addnode "<IP_ADDRESS>:8555" add

Had this today as well. Was wondering but figured out that this is caused by not well synced peers.
Solutionfor me: After app start directly add a peer / node that is synced. As I am running a node to create these blockchain snapshots you can just add my by the following console command:

addnode "<IP_ADDRESS>:8555" add

Thanks Darkwinde...that worked for me!!!


Allow to define cwd annotation for code cells

It would be great if developers could define the working directory in which the code cell code should be executed from. Given that everyones FS is different, only relative directory changes would work. My use case is that I want to create a best practice/recipe repo for allowing users to run WebdriverIO examples. Every command would be its own directory with example files but the WebdriverIO setup should be only done once in the root. So the README.md (e.g. in /click/README.md) of the example would have something like this:
```sh { cwd=../ }
# setup example repository
npm install
```

as well as
```sh { cwd=../ }
npx wdio run ./wdio.conf.ts --spec ./click/example1.e2e.ts
```

I think it would make sense to signalise from which directory the commands are executed.

This feature has been added to in v1.5.0.


add example of multitask_prompt/poly tuning

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Hi, can you resolve the conflicts on the branch?


Height issue on refresh

I can't get the slider to recalculate the height of slides on refresh. I am running the refresh on window resize, but it doesn't seem to do anything.
if($sliderDiv.length) {
    var $slider = $sliderDiv.lightSlider({
        item: 1
    });
}

$(window).on('resize', function() {
    $slider.refresh();
});

Should i be doing something differently? Thanks in advance.

Add adaptiveHeight parameter set to true. Then you don't need to refresh the slider on window resize.

Ok that did it, thanks! :+1:


Class Table jumps to the top of the page?

Hi,
I encountered an error I could reproduce, so I wanted to give you a heads up. You can find the code at the bottom. Basically, when you switch to A4 format and use a Class Table, it jumps to the top of the page.
To reproduce it open the code in a normal brew. Then type anything on the first page of the document, e.g. right under the headline "Hello World" while looking at the class table. It will jump almost to the top of the second page. When you type anything inside the class table it will jump back.
Happy hunting!
Additional Details
Brew code to reproduce : Click to expand

Hello World
Vae victis
This is a Test
\page

The Fishmongerer
| Level | Proficiency Bonus | Features | Cantrips Known | Spells Known | 1st | 2nd | 3rd | 4th | 5th | 6th | 7th | 8th | 9th |
|:---:|:---:|:---|:---:|:---:|:---:|:---:|:---:|:---:|:---:|:---:|:---:|:---:|:---:|:---:|
| 1st | +2 | ─ | 3 | 11 | 2 | — | — | — | — | — | — | — | — |
| 2nd | +2 | Hermetic Geography | 3 | 2 | 2 | — | — | — | — | — | — | — | — |
| 3rd | +2 | Nuclear Illusionism | 3 | 3 | 4 | — | — | — | — | — | — | — | — |
| 4th | +2 | Ability Score Improvement, Pharmaceutical Gunslinger | 3 | 3 | 4 | 1 | — | — | — | — | — | — | — |
| 5th | +3 | Astrological Botany | 3 | 3 | 4 | 1 | — | — | — | — | — | — | — |
| 6th | +3 | Ability Score Improvement, Pharmaceutical Gunslinger | 3 | 3 | 4 | 1 | — | — | — | — | — | — | — |
| 7th | +3 | Mathematical Pharmacist | 4 | 3 | 4 | 2 | — | — | — | — | — | — | — |
| 8th | +3 | Ability Score Improvement, Gunpowder Torturer | 4 | 3 | 4 | 3 | 1 | — | — | — | — | — | — |
| 9th | +4 | Spiritual Biochemistry | 4 | 3 | 4 | 3 | 1 | — | — | — | — | — | — |
| 10th | +4 | Civil Alchemy | 5 | 3 | 4 | 3 | 2 | — | — | — | — | — | — |
| 11th | +4 | Genetic Banishing | 6 | 4 | 4 | 3 | 3 | 1 | — | — | — | — | — |
| 12th | +4 | Ability Score Improvement, Police Necromancer | 7 | 5 | 4 | 3 | 3 | 1 | — | — | — | — | — |
| 13th | +5 | Astrological Chemistry | 8 | 6 | 4 | 3 | 3 | 1 | — | — | — | — | — |
| 14th | +5 | Ability Score Improvement, Spell Analyst | 8 | 6 | 4 | 3 | 3 | 3 | — | — | — | — | — |
| 15th | +5 | ─ | 8 | 7 | 4 | 3 | 3 | 3 | 2 | — | — | — | — |
| 16th | +5 | Ability Score Improvement, Police Necromancer | 9 | 8 | 4 | 3 | 3 | 3 | 3 | — | — | — | — |
| 17th | +6 | Civil Alchemy | 9 | 9 | 4 | 3 | 3 | 3 | 3 | 1 | — | — | — |
| 18th | +6 | ─ | 9 | 10 | 4 | 3 | 3 | 3 | 3 | 2 | 1 | — | — |
| 19th | +6 | Ability Score Improvement, Sixgun Poisoner | 9 | 10 | 4 | 3 | 3 | 3 | 3 | 2 | 2 | — | — |
| 20th | +6 | Consecrated Biochemistry | 10 | 10 | 4 | 3 | 3 | 3 | 3 | 2 | 2 | 1 | 1 |

I noticed this too, but it apparently has no impact when "getting PDF". you just have to be aware of it when layouting the actual site where the table is on.

This 2 year old issue can likely be closed.  The original code provided doesn't even match the current snippet for class tables, and after some tinkering I couldn't reproduce it.  Unless @HeruAsatru  or @Christian-Eichhorn are still seeing the same issue at this point, let's close this one out. @ericscheid @calculuschild

This 2 year old issue can likely be closed.  The original code provided doesn't even match the current snippet for class tables, and after some tinkering I couldn't reproduce it.  Unless @HeruAsatru  or @Christian-Eichhorn are still seeing the same issue at this point, let's close this one out. @ericscheid @calculuschild

I also cannot reproduce this. May have been related to #295, but this particular instance seems to have been fixed with updates to Chrome. Closing.

I also cannot reproduce this. May have been related to #295, but this particular instance seems to have been fixed with updates to Chrome. Closing.


Added Quaternion and QuaternionTests to the FSharp project

Here is an FSharp implementation of a Quaternion.  Please let me know if I am missing anything according to contribution guidelines.  I have tried to include references for where I found my math.

Wouldn't it be better to implement this in C# so that all of MathNet.Numerics users could use it rather than just F# users?

FSharp is a full fledged member of the .NET run time, so this should be usable from C#.

Yes. But that is not what I'm saying. The MathNet.Numerics assembly is the "core package" while the MathNet.Numerics.FSharp assembly contains "optional extensions for a better F# experience". People shouldn't have to include the MathNet.Numerics.FSharp assembly to use this from C#.

I can sympathize with your point of view, since I only recently started writing F# and have been writing C# for most of my career.  I don't think it's valid to not accept an F# contribution because the library was started when F# wasn't a viable option.
I would like to think :pray: that if F# was mature/existed when Math.net started, "core" would be in F#.  I believe that F# is a superior language for succinctly expressing mathematical based code. This is objectively true, for this class, by comparing to https://github.com/dotnet/corefx/blob/master/src/System.Numerics.Vectors/src/System/Numerics/Quaternion.cs. While I didn't implement everything that Numerics does, it's plain to see the difference in the methods that I did implement.

I would like to think :pray: that if F# was mature/existed when Math.net started, "core" would be in F#

That may be true, but currently MathNet.Numerics is written in C# so something which should be available to all .Net languages I think should be in the core assembly and hence written in C#.
It may be the case that for future versions of MathNet.Numerics the code base is moved to F#. It is also possible to sort of mix languages by using ILMerge to merge multiple assemblies together into a single assembly. That could allow sections to be migrated to F# gradually and new functionality to be written in F#. But it may not be feasible as merged assemblies have to be independent of each other (or at least the C# assembly would not be able to call the (new) F# assembly, meaning only non-core sections could be moved to F#).
If the project were to go down that path though I think it would need to be a clear that that is the desired development direction.

Thanks!
We already have some functionality available only in the FSharp package that may also be useful for non-F# .net developers, mostly the BigRational type. So it is acceptable in my point of view to do the same with Quaternion.
But I do agree that the current separation between the two packages is unfortunate. A separation into two NuGet packages just because it uses a different language internally doesn't make any sense to users. One problem with a single package approach could be the dependency on FSharp.Core though, which could cause some problems.

One problem with a single package approach could be the dependency on FSharp.Core though, which could cause some problems.

Would that really be a problem? If MathNet.Numerics was completely written in F# then it would have the same dependency. Or if people use features only available in the MathNet.Numerics.FSharp assembly. I can't think of any issues this would cause especially seeing as FSharp.Core is available through a NuGet package which could be updated independently of MathNet.Numerics.

FSharp.Core used to cause quite a few problems mostly due to it sitting in the GAC and having special treatment in the tooling. Luckily this is about to be getting better: no more GAC and it is becoming standard to use the proper FSharp.Core NuGet package.
For example, while LINQPad supports F#, it still treats FSharp.Core specially and cannot handle NuGet packages that depend on the FSharp.Core NuGet package (like MathNet.Numerics.FSharp does). This means that while it is possible to use our F# package from C# in LINQPad, and the MathNet.Numerics package in F#, it is currently not possible to use the F# package from F#.

@pdoh00 do you require the norm function to be defined as is, or can I rename it to normSquared and add a new norm function that returns the square root of that? Most sources seem to agree on the square root. MathWorks/MATLAB disagrees, but to my understanding their definition actually violates the subadditivity property of norms and thus does not actually form a proper norm.

Also, does it make sense in practice that rotate returns a quaternion?

I actually changed norm due to the way MatLab has it because they had test data. I am not against changing it. Originally I implemented it the same as the referenced paper.
It might be more appropriate to return a Vector3 from rotate. The reason I did not do that originally was that I was test a pure solution without pull Math Numerics in.  Now that Quaternion is part of numerics it might make more sense to return a Vector3...maybe even take a Vector3 instead of x,y,z.
:+1:

@cdrnet I am just posting some code here with some ideas on the Quaternion methods for create and rotate. I am doing some like this internally at work.
let create (angle:float<Radian>) (v : Vector3<'u>) =
    //axis needs to be unit vector
    let v' = Vector3.normalize v
    let halfAngle = angle * 0.5
    //divide by 1.0<radian> to get back to dimensionless value for sin and cos
    let s = halfAngle / 1.0<Radian> |> sin
    let c = halfAngle / 1.0<Radian> |> cos
    {
      W=c;
      X= v'.X / (LanguagePrimitives.FloatWithMeasure 1.0) * s;
      Y= v'.Y / (LanguagePrimitives.FloatWithMeasure 1.0) *s;
      Z= v'.Z / (LanguagePrimitives.FloatWithMeasure 1.0) *s
    }

 let rotate (q:Quaternion) (v : Vector3<'u>) =
    let qn = normalize q
    let p = {W= 0.0; X= float v.X; Y= float v.Y; Z= float v.Z}
    let rotated = qn * p * inverse qn
    {
      Vector3.X=rotated.X* (LanguagePrimitives.FloatWithMeasure 1.0);
      Y=rotated.Y* (LanguagePrimitives.FloatWithMeasure 1.0);
      Z=rotated.Z* (LanguagePrimitives.FloatWithMeasure 1.0)
    }

Obviously for Numerics it would use DenseVector or the like. Since this code only works for Vectors of size 3 I am not totally sure how to enforce that with DenseVector.  I hope this helps.


the result of synonym search is incorrect

Describe the bug
First of all add some synonym and a doc
{
        "movie_id": "259910",
        "title": "Marvel Studios: Assembling a Universe",
        "subtitle": "漫威影业：终极之战  Iron Man  浩克",
        "poster": "https://image.tmdb.org/t/p/w500/41Ke02UzYoPsiTc3YbomPGiQKMl.jpg",
        ....
}

the resutl of index settings
{
    "rankingRules": [
        "typo",
        "words",
        "proximity",
        "attribute",
        "wordsPosition",
        "exactness"
    ],
    "distinctAttribute": "movie_id",
    "searchableAttributes": [
        "title",
        "subtitle"
    ],
    "displayedAttributes": [
        "*"
    ],
    "stopWords": [],
    "synonyms": {
        "Hulk": [
            "浩克",
            "绿巨人"
        ],
        "Marvel": [
            "漫威"
        ],
        "浩克": [
            "Hulk",
            "绿巨人"
        ],
        "钢铁侠": [
            "Iron Man"
        ]
    },
    "attributesForFaceting": [
        "actors",
        "directors"
    ]
}

then search  and the search body is
{
	"limit": 4,
	"q": "Hulk",
	"attributesToHighlight": ["title", "subtitle"],
	"attributesToRetrieve": ["overview", "title", "subtitle", "actors"],
	"matches": true
}

the search resutl now is
{
    "hits": [],
    "offset": 0,
    "limit": 4,
    "nbHits": 0,
    "exhaustiveNbHits": false,
    "processingTimeMs": 1,
    "query": "Hulk"
}

Expected
query Hulk  can search out  the documtns Which contain  浩克 绿巨人 Hulk
Desktop (please complete the following information):

OS: CentOS Linux release 7.2.1511
Browser chrome
Version 0.18.1

Hey @sixinyiyu,
I just checked that on the 0.18.1 version and indeed it doesn't work, but luckily this issue has already been fixed in v0.19 and will be shipped next week.
Thank you for the report, I am closing this issue 🎉


python311Packages.pytensor: remove numba-scipy dependency

Description of changes
numba-scipy was removed upstream just in time for the current release: https://github.com/pymc-devs/pytensor/pull/675

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 308109 run on x86_64-linux 1

4 packages failed to build:

python311Packages.bambi
python311Packages.bambi.dist
python312Packages.pytensor
python312Packages.pytensor.dist

4 packages built:

python311Packages.pymc
python311Packages.pymc.dist
python311Packages.pytensor
python311Packages.pytensor.dist

same as master


Fix the broken link to the IIIF manifest for B20442178

If you look up the IIIF presentation URL on aev89hqy, you get a URL with a capitalised b number:
$ curl 'https://api.wellcomecollection.org/catalogue/v2/works/aev89hqy?include=items' | jq -r '.items[0].locations[1].url'
https://iiif.wellcomecollection.org/presentation/v2/B20442506

This URL 404s, so you can't see the manifest on wc.org/works – but the manifest does exist, with a lowercase b number, i.e.
https://iiif.wellcomecollection.org/presentation/v2/b20442506
We should update the METS transformer to lowercase this b number and return a working URL. I think we already have some logic to lowercase b numbers; we just need to apply it more consistently.

The same problem is affecting B20442178.

Actually, we should be fixing this in the METS source.

This is fixed!


Fix stray reference to old autosummary dir

In #544 we updated the autosummary directives output directory from
'stubs' to 'apiref' so that the urls of the hosted documentation made
more sense. Then a redirect was added to ensure that stale links to
stubs were redirected to the new location. However, as part of this
migration one autosummary directory was missed causing it to still
publish to 'stubs'. With the redirects in place this means that the
documentation is not actually reachable anymore. This commit fixes this
oversight by fixing the autosummary directory so the documentation is
built in the correct location.
Fixes #594

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 97.184%

Files with Coverage Reduction
New Missed Lines
%

src/shortest_path/all_pairs_dijkstra.rs
2
98.54%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
11252

Relevant Lines:
11578

💛  - Coveralls


Add documentation about how to create libpsl and psrp.windows packages

Related to #5231

NPM is checking Gulp is installed. This may take a few moments.
[-] Should not have errors in any markdown files 20.81s
Expected string length 9 but was 287. Strings differ at index 0.
Expected: {--EMPTY--}
But was:  {C:\projects\powershell\docs\building\internals.md: 48: MD007 Unordered list indentation C:\projects\powershell\docs\building\internals.md: 50: MD007 Unordered list indentation (See https://github.com/DavidAnson/markdownlint/blob/master/doc/Rules.md for an explanation of the error codes)}

@TravisEz13 Can you please help with the failed markdown test? Error code MD007 means the unordered list doesn't use 2-space indentation. However, line 48 and 50 both use 2 space indentation.


test directory included in package

https://github.com/sass/node-sass/blob/master/package.json#L44

It appears that this directory includes only test files. I think it should be excluded from NPM package.
This way test frameworks will also not try to parse it when running without excluding node_modules explicitly (which fails because of missing devDependencies, of course).

item
value

NPM version
6.4.1

Node version
v10.15.3

Node Process
{ http_parser: '2.8.0', node: '10.15.3', v8: '6.8.275.32-node.51', uv: '1.23.2', zlib: '1.2.11', ares: '1.15.0', modules: '64', nghttp2: '1.34.0', napi: '3', openssl: '1.1.0j', icu: '62.1', unicode: '11.0', cldr: '33.1', tz: '2018e' }

Node Platform
darwin

Node architecture
x64

node-sass version
node-sass	4.12.0 (Wrapper) [JavaScript], libsass  	3.5.4 (Sass Compiler) [C/C++]

npm node-sass versions
└──<EMAIL_ADDRESS>

This is by design for necessary 3rd party integration systems.


[MODULE] Redundant Shield Cores - Active

Active Module - Redundant Shield Cores
Basic Functionality:
Energy that would normal be directed to recharging main shielding is instead made to trickle-charge a back-up shielding array. Disables inherent shield recharge but grants an active ability to fully restore all shields.
Stats:
Cooldown: 45 Seconds
Duration: Instant
Fully restores shields
Disables the automatic out-of-combat shield recovery.
Level 1: As above.
Subsequent Levels: Cooldown reduced by 10%, multiplicatively (100% - 90% - 81% - 72.1% - etc) We can't use flat CDR here because we don't want the player getting a 0 second cooldown shield restore.

Task Estimation (time): 4 + 1
Risks: Might take some time to whip up a suitable effect and icon
Risk Resolution: Added additional hour to estimation. Will tell team, ask for help if stuck.


bootkube --asset-dir should be able to parse k8s objects directly

Rather than needing a specific directory structure, we should teach bootkube how to parse the kubernetes objects directly. This way we could just parse "api-server flags" and "api-server secrets" for use in bootkube's temporary apiserver and we don't need to specify them (or etcd) separately

@aaronlevy I was gonna have a go at this if you or somebody else isn't implementing it atm. I need a bit more explanation on what exactly you need. So inside of --asset-dir we render api-server flags as files in respective folders. I guess bootkube should load these as k8s objects instead? This also applies to the rest of the the control panel?

@kenan435 sorry, this is really vague (and my thinking may have changed a bit).
This likely would be replaced in functionality similar to https://github.com/kubernetes-incubator/bootkube/issues/106 (I had started working on the above issue a week or so ago, but haven't had a chance to go back to it).
The ultimate goal of both of these issues is that we should be able to consume all of our configuration from native upstream objects (configMaps). So I guess there are two parts of this:

Bootkube consumes configuration of its internal control-plane (e.g. https://github.com/kubernetes-incubator/bootkube/blob/fae0c88b328a334c6e1f7a73b6358f9070d15eb8/pkg/bootkube/bootkube.go#L56-L67) from configMaps.

All of our self-hosted components consume their configuration from configMaps (will require upstream changes (briefly discussed in the "Render" step: https://github.com/kubernetes-incubator/bootkube/issues/106)

Hypothetically both the internal bootkube control-plane and the self-hosted control plane should be able to use similar / same configuration (although we still need to resolve: https://github.com/kubernetes-incubator/bootkube/issues/127). However, there may be some cases that I'm not considering where that's not true.
It may be worthwhile closing this issue and having the discussion in one place in https://github.com/kubernetes-incubator/bootkube/issues/106 -- as they likely will be very close in functionality.
@kenan435 interested in your thoughts - and if this has been a particular pain point for you?

Closing this in favor of moving to static manifests in the future: https://github.com/kubernetes-incubator/bootkube/issues/168
We wouldn't need the compiled-in control-plane to source any information from manifests -- because we would use the manifests directly.


[bug] Unfolding images causes them to draw at the cursor position (and other weird folding behavior)

When I fold and then unfold a block of text that has an image in it, the image is drawn at the cursor position until I scroll the buffer, then it's redrawn at it's correct location:
https://github.com/3rd/image.nvim/assets/56943754/56dfca3a-457f-4ee8-b0e3-63d8b3f9e7df
I've had this bug for a little while now, it's just not a big deal so I haven't bothered.
I have a train ride tomorrow so this is one of the things that I will look into.

Here's some more weird stuff that happens with folding document integration images:
https://github.com/3rd/image.nvim/assets/56943754/6d71423a-8caa-4c86-97be-f077607bb4b1
note: the black lines are characters from headlines.nvim, not an issue with image rendering
also, the very end I think is an issue with molten not clearing images or not deinitializing properly when using :e. probably not an image.nvim issue

I wonder if this happens to other people with uberzug++, b/c it seems to be a cursor repositioning problem that's somewhat out of our control. I could be totally wrong though.

can't reproduce on master + nvim nightly (0.10)


update SEP-24 Anchor Platform guide to use yaml assets file

The Anchor Platform added support for specifying asset data via YAML files. This PR update the guide to use this approach.

Preview is available here:http://developers-pr81.previews.kube001.services.stellar-ops.com


Gulp task abstractions and coffee

I used to use coffee for writing js, but find no need for it since ES6, but what I do find coffee still useful for is writing gulp tasks and configs - much easier to read!
Anyways, this is a sort of 2 part query.
I've started writing my gulp tasks as separate files, and using require-dir to import them. This makes them a ton easier to maintain, and adding new tasks easier too.
The structure ends up looking like this:
│   ├── gulp/
│   │   ├── gulpconfig.coffee
│   │   ├── tasks/
│   │   │   ├── browserSync.coffee
│   │   │   ├── build.coffee
│   │   │   ├── clean.coffee
│   │   │   ├── css.coffee
│   │   │   ├── minify.coffee
│   │   │   ├── update.coffee
│   │   │   ├── utils/
│   │   │   │   └── index.coffee
├── gulpfile.coffee

You can see an example structure here: https://github.com/fixate/generator-fixate-pw/tree/master/app/templates
So:
:+1: to splitting up?
:+1: to coffee? :D
Will get on this when I get a minute.

Not a huge fan of coffee, given the traction of ES6 and Babel. I'm not opposed to splitting tasks, but I'd rather do explicit import statements, rather than including an entire dir.
Let me think on this one.  I am about to start an ES6 refactor of Fabricator this month.

For production code I avoid implicit anything, but it's definitely sped up automating builds and adding tasks etc.
I suppose it also depends if it makes adopting Fabricator more difficult.


Feature: Can Lombok annotations can be addded as an option ?

Can Lombok annotations can be addded as an option ?

Which annotations do you want to add?
BTW: For XJC I saw no good plugins with lombok annotations, so I don't expect any good implementation here.

based on users selection of the following options
Include getters and setters
Include constructors
Include hashCode and equals
Include toString
the Lombok annotations can be added.

and then add all other code generators on users' pleasure.
I'd prefer to add a possibility for a plugin which will provide these modifications rather than add this into this generator itself.
The reasons are quite simple:

this generator uses java 8 and able to produce java6-compatible code, which could be crucial. With a plugin you can easily manage versions and available features
not everybody uses or wants to use lombok, as there're other generators and some users want to use no such dependency at all. Current generated sources don't depend on any other libraries (except for dates and json generator annotations)

@vinishgeorge This is already open as #166. If you have thoughts on what you'd want and need, please add them there.


[core-amqp] Update async methods to support cancellation

Fixes #9988 and part of #4422.
This PR updates 3 async methods to support cancellation:

CbsClient.negotiateClaim()
CbsClient.init()
RequestResponseLink.create()

Note that I needed to update RequestResponseLink.create() to accept an abortSignal so that CbsClient.init() could be properly cancelled.
Updates to event hubs and service bus will come in a separate PR.
API View link

Updated based on feedback. What I'm unsure of is if the API view needs to be approved, or if just having this PR with the api review file updated approved is enough.
@xirzec - Help me Jeff, you're my only hope!

@xirzec - Help me Jeff, you're my only hope!

Where's the APIView that needs to be approved? The link from this PR description seems to give a server error from ApiView

@xirzec
https://apiview.dev/Assemblies/Review/ec633b2b71c14c81ba99a896f8af2400
Sorry, it looks like when I replaced a revision the previous link became invalid. This is my first time actually uploading to API View so still learning the nuances :)

Sorry, it looks like when I replaced a revision the previous link became invalid. This is my first time actually uploading to API View so still learning the nuances :)

No worries, changes look good to me! 👍


In report, "Show Failures" does not show Failed Use cases only.

It's a nice report template.

"Show errors" does not show only error tests. It shows all the test names. ( fyi, I do not have any errors in my collection)

There should be an  Aggregated report column ( Total no of tests, total failures, total, success). Once you click on number , you should see the error tests.)

Yeah, you're right.
1 - "Show erros" does not show now (I made this change today).
2- For now i can't imagine one section with report aggregation (Total no of tests, total failures, total success), however there are three buttons wich have actions to show/hide requests success/error.

Thanks for getting this done.
But in any report its very important to show the total no of tests, success and failures links and then drilling down further details.

It's a great Idea, in the next days i will improve the style


chore: rename openedNodes to closedNodes

Changes

Rename openedNodes to closedNodes

Closes

https://github.com/rangle/augury/issues/363

Notes

I used WebStorm to tell me all exact instances of openedNodes, and on a one-by-one basis used my best judgement to convert from one variable name to another.
Use the build from below to test it manually, it seems to work with the demo.

Build

augury.zip (1)
augury.zip (2)

@sumitarora I think this is ready. I walked it through the demo app, and I couldn't find any issues caused by this, I am optimistic 😄

:shipit: you don't need to attach the build I can just checkout your branch and build it 😄


Add Minification to project

Currently project is not using any tool for management of public resources and minification of code as mentioned here . We can use gulp or grunt or some other tool for it . I propose using gulp for it as in
#80 .

If you're comparing bundlers. I'd recommend Webpack. It has more advantages and is more popular than Gulp and Grunt.
To back my statement, here are some blogs where they are compared:
https://da-14.com/blog/gulp-vs-grunt-vs-webpack-comparison-build-tools-task-runners
https://scotch.io/@VitalyKuprenko/grunt-vs-gulp-vs-webpack-an-elaborate-comparison-of-tools

Totally agree with @NikhilM98.
@ashish0910 thanks for bringing up this issue.

We could change the code to ES6 syntax then use babel to transpile to ES5 if we are using webpack

@SomtochiAma agreed

@NikhilM98  @SomtochiAma I was wondering too why we are not using ES6 syntax

@Anshuman71
At the current point in time, ECMAScript modules are only available as an experimental feature in Node.js and not all browsers support ES6 and higher.
In order to use Node.js in our project, we need to install Babel in our project which can be used to transpile modern ECMAScript code to ES5 code to ensure backward compatibility.

@NikhilM98 yes I am using it in my personal projects.

I'm not in favor to add complexity to the current development environment without direct benefit.
Using Webpack and Babel will add complexity where today we just need an editing tool and node.js to run and update Sugarizer Server.
Plus, optimizing Dashboard is not a priority because the dashboard is intend to be use by few users (teachers and admin). Not the same thing for Sugarizer App where optimization make sense to offer a better experience to students, specifically on mobile device.

Fixed in https://github.com/llaske/sugarizer-server/commit/f21e308c72da5a475333e88f2b0b5757354c5042


updated MDK setup guide + quick start tutorial

updated MDK setup guide + quick start tutorial

cp-mobile-dev-kit-quick-start
fiori-ios-hcpms-setup
cp-mobile-dev-kit-ms-setup
cp-mobile-bas-setup

@MichaelCzcz i dont see the changes reflected in the public site.
https://developers.sap.com/tutorials/cp-mobile-bas-setup.html
but changes can be seen in QA page https://developers-qa.sap.com/tutorials/cp-mobile-bas-setup.html
same applies for other tutorials part of this PR.
can you please check what has caused it and fix it?


Magic Leap support

This adds support for binding to the Magic Leap audio APIs for both input (microphone) and streaming output.

One gotcha is that the MLSK doesn't support Floating point samples yet, but LabSound is based on them. Therefore we need to do a manual sample conversion.


ci(actions): update actions/checkout action to v3.5.3

This PR contains the following updates:

Package
Type
Update
Change

actions/checkout
action
patch
v3.5.2 -> v3.5.3

Release Notes

actions/checkout
v3.5.3
Compare Source

Fix: Checkout fail in self-hosted runners when faulty submodule are checked-in
Fix typos found by codespell
Add support for sparse checkouts

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.1.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How can I use ? for error handling?

How can I use ? for error handling?
	loop {
		if let Some(ref c) = client {
			if let Err(_) = (|| -> Result<(), Error<()>> {
				if let Some(update) = c.check_for_update(cookies.clone())? {
					let r = update::install_update(&update);
				}
				// ...
				Ok(())
			})() {
				client = None;
			}
		} else {
			if let Ok(c) = connect() {
				client = Some(c);
			}
		}
		sleep(Duration::from_secs(1));
	}

error: src\main.rs:79: the trait bound comm::Error<()>: std::convert::From<tarpc::Error<comm::Error<()>>> is not satisfied

I assume comm::Error<()> is an error generated by your application? Typically you'd use something like error-chain that encodes the various types of errors your application can encounter. I'm not sure if this is tarpc specific, but let me know if I'm misunderstanding!

Ah right, I had to use Result<(), tarpc::Error<Error<()>>>...


Fix the Romanian lang issue(Close 2036)

It's a hotfix for JIRA-2036

this seems to reverse the changes from the translation pull request.
https://github.com/corona-warn-app/cwa-app-ios/pull/1006/files?file-filters[]=.strings#diff-01fe9067cc23a312e36676691ec6007dR658
looks alright

I opened the branch directly from "staging" branch. it should not revert anything.


lightnet tagging to proof level none fix

!ci-nightly-me

!ci-nightly-me

coverage: 33.405%. first build
when pulling 40a13e5b498fd8878e74d2951063654bdf0d9078 on sai/georgeee/proof-level-no_check
into 3531efcad81d0caff81b2d1413d1d847c0cf1d95 on georgeee/proof-level-no_check.


Repeat translated word/sentences

For language learning it would be necessary to hit the "speaker" icon to keep repeating the translation.

This app was not designed for language learning but for communicating, however in the future a button to read the translation will be added to the text translation mode. As for a similar button for the WalkieTalkie mode, I will evaluate whether to include it 👍

Thanks that would be nice also in case you did not here correct the first time .


log user agent string to Kudu tables for eventid = 65516

Upon meeting with Suwat, the analytics team would like to count on what we call 'deployment client' so that we can expose in our tables greater deployment details for every site. This would allow several of our partner teams to analyze and identify opportunities in customer retention based on client/source cohorts, allowing us to concentrate on improving experiences where needed.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Player count in instance isnt shown correctly.

Ive seen that often, when clicking a friend, it would say, for example, '2 (1)'. Even after clicking Refresh player count, it says that.

But then, when I click the map name, and I see my friends instance, it will say 1 (1)

I need to click refresh player count for it to update to 2 (1), which is strange because its suppose to know that there are 2 people already.
Furthermore, existing the map view, and then clicking the map name again, will once again show 1 (1), and I have to click refresh again.

Hmmm not sure but it will be either one of 2 things, the number only ever updates once when the dialog is opened or because there are 2 different endpoints this value can come from /worlds and /instances both can return different values because VRChat.

'because vrchat' lol sounds about right.
alrighty, just figured id mention this in the off chance its something you can fix on your end.
thank you. =)


[FLINK-4373] [cluster management] Introduce SlotID, AllocationID, Res…

For the new version of cluster management, we need some more basic data structures:

SlotID: Identifier of one single slot located in a task executor

AllocationID: Slot allocation identifier, created by the JobManager when requesting a slot, constant across re-tries. Used to identify responses by the ResourceManager and to identify deployment calls towards the TaskManager that was allocated from.

ResourceProfile: The resource profile of the desired slot (currently only cpu cores and memory are supported

Thanks for you contribution @KurtYoung. Good work!
I only had some minor comments. Once they are addressed it's good to be merged :-)

Hi @tillrohrmann , thanks for the reviewing, all your comments have been addressed.

Failing test cases are unrelated. Will merge this PR. Thanks for your contribution @KurtYoung.

Not automatically closed by the asfgit bot. @KurtYoung could you close this PR manually?

@tillrohrmann Yeah, of course. Closing it now.


Addition of PST information

This PR includes all of the information necessary for users to learn about the addition of our PST to the platform. Because the concept of cryptocurrency and profit sharing tokens is not mainstream, we need to lay this information out in a way that is not too difficult to understand.
The main sections of the page I see as crucial are:

[x] What is it?

This provides a general overview as to what the token is

[x] Benefits

Discusses ways our PST is used on the platform

[x] Token metrics

Provides realtime data of the most recent PST transactions and largest owners of the token

[x] How to exchange

Information on how users can exchange

Great job with this

👍🏻


Strange freeze when installing sccache

Description
The bug I am seeing is that once I push commits to a PR, the install action freezes for indefinite amount of time (IIRC it will bail after 5+ hours of freeze but I cant find this run anymore...)
Here is a recent run, sorry the project is private, I can't give a link to the workflow.
After 25 minutes of idling I canceled the workflow and here are the logs:

Workflow code
    - name: Cache sccache cache
      uses: actions/cache@v1
      with:
        path: ${{ env.SCCACHE_DIR }}
        key: sccache-cache-${{ hashFiles('**/Cargo.toml') }}
    - name: Install sccache
        uses<EMAIL_ADDRESS>        with:
          crate: sccache
          use-tool-cache: true
          version: latest

Textual logs
Run<EMAIL_ADDRESS>Tool cache is explicitly enabled via the Action input
Downloading from the tool cache
  Newest sccache version available at crates.io: 0.2.13
  Downloading sccache signature into /tmp/sccache.zip.sig
  Downloading sccache == 0.2.13 into /tmp/sccache.zip
  Starting signature verification process
  /usr/bin/openssl dgst -sha256 -verify /home/runner/work/_actions/actions-rs/install/v0.1/public.pem -signature /tmp/sccache.zip.sig /tmp/sccache.zip
  Verified OK
  Extracting files into /home/runner/.cargo/bin
  /usr/bin/unzip /tmp/sccache.zip
  Archive:  /tmp/sccache.zip
  replace sccache? [y]es, [n]o, [A]ll, [N]one, [r]ename:
  ##[error]The operation was canceled.

What is interesting is that after canceling workflow and hitting the rerun button everything works fine.
I suppose the problem is that there is a call to unzip utility that is run in interactive mode...

It is a duplicate of https://github.com/actions-rs/tool-cache/issues/5


how to trigger a groupMenu component ?

how to trigger a groupMenu component ? I try through the ref.current.onPress() , the event is triggered, but does not expand under the menu, help
  constructor() {
    this.menuItemRef = React.createRef();
  }
  componentDidMount() {
    this.menuItemRef.current.onPress();
  }
...

 <Menu>
<MenuGroup ref={key === selected ? this.menuItemRef : null}>
<MenuItem>
 </MenuItem>
<MenuItem>
 </MenuItem>
</MenuGroup>
</Menu>

@Wimmind sorry for late response. can you please describe your use case?
next release we'll deliver new prop for MenuGroup component which will allow to make it expanded for initial render. will it match your request?

closing the issue for now. update with fix will be delivered in next release


fix: macos builds, case-sensitivity and incompatible js signatures

This primarily resolves issues for building on MacOS, there is a couple of new targets in the Makefile: clean which is generally expected and second dev-macos which will determine the proper architecture and paths for openssl through homebrew. The instructions have been rewritten based off the updates so that it is more clear for building.
There was a case-sensitivity issue, I would recommend developers operate with git config --global core.ignorecase false and even better if they utilize a case-sensitive file system as the line component and a few other case-sensitive issues were found.
Lastly, I encountered serveral issues with various signatures, the signatures ultimately caused a mismatch and therefore I had to alter the behavior of one call and secondly match up the signatures on the rest which appear to have no influence on actual behavior. I am using node v20 but the errors were also encountered on v18.

dev-86 works fine for me, but dev-macos on this PR also worked. appreciate the improvements


BUG:hover effect on nav bar

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

hover effect showing on wrong item
What happened?
No response
Expected behavior
No response
Screenshots
No response
Record

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

Please assign me this issue under @GSS0C24

please add level badge @arghadipmanna101


Use listCollections command to find MongoDB view

Fixes #3348
Reference https://docs.mongodb.com/manual/reference/command/listCollections/

I think we can test it by preparing a weak user in MongoServer class. (I'm looking into that)


Feature: 404 page never knows I am logged in or not

Type of feature
🍕 Feature
Current behavior
When visiting the 404 page while logged in, it should not have a "Sign Up" button.

Suggested solution

Visit /404
Check if user exists in the session, and do not show the button if they exist.

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

I think we can remove the sign in button. It always seemed odd to me on the 404 page. And when they click on the link to take them back home, the logging button is in the top navigation.


How to receive an event parameter inside the Boltjs Lambda

Description
Hello !
My current situation is that when a user posts a comment API gateway is being called then it invokes let's say Lambda A and this Lambda A invokes my slack App Lambda with the event parameter. That event parameter contains userName and I would like to catch that event parameter and grab the userName from it then use that data (info) to send out a notification by using IncomingWebhookUrl.
while doing it, I've tried several ways to do it but got this error instead
"Unable to determine event source based on event.",

So my question is where can I catch the passed event parameter from the Lambda to lambda invocation or API gateway to lambda invocation?
at the end of my Slack app is where I believe I need to catch the event parameter.
module.exports.handler = serverlessExpress({
  app: receiver.app,
});

to fix that I did something like this but it was not successful.
exports.handler = async (event, context) => {
  console.log(event, context);
  return serverlessExpress({
    app: receiver.app,
  });
};

I also did something called getCurrentInvoke() which is a method given by vendia/serverless-express library
I called getCurrentInvoke() at the global scope to catch it but was futile
const { event, context } = getCurrentInvoke();
console.log(event, context);

Also, is there a way to catch the event parameter from an invocation inside the receiver?
Thank you
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ x ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[ x ] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[ x  ] I've read and agree to the Code of Conduct.
[ x  ] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
node version:
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

It is unclear to me how this relates to bolt-js? Looking at the stack trace, this seems to be an error raised in the serverless-vendia npm package.

@esphoenixc According to their README (https://github.com/vendia/serverless-express#async-setup-lambda-handler), the following code should work for you.
async function handler(event, context) {
  console.log(event);
  return serverlessExpress({ app: receiver.app })(event, context);
}
exports.handler = handler;

If you have further questions, I would suggest asking in their issues instead.

Thank you the answer did help tremendously.


Add badges to readme

Merged https://github.com/react-native-community/react-native-picker/commit/8eb593afee1802b5c60e7ff6fc64f2f048e60895


New Machine requirement: docker-marist should be replaced with a later Ubuntu

I need to request a new machine:

New machine operating system (e.g. linux/windows/macos/solaris/aix): Ubuntu 20.04 (probably)
New machine architecture (e.g. x64/aarch32/arm32/ppc64/ppc64le/sparc): s390x
Provider (leave blank if it does not matter): Marist
Desired usage: Running jobs that reuqire docker
Any unusual specification/setup required: No
How many of them are required: 1

Please explain what this machine is needed for: Replacement of docker-marist-ubuntu1604-s390x-1 which is now out of support and is causing problems with the new Debian Linux packaging.

Related: https://github.com/adoptium/infrastructure/issues/2267

Replacement dockerhost system running Ubuntu 22.04 which has been created as part of https://github.com/adoptium/infrastructure/issues/2673 will be suitable for this task.

System now has two test-docker* containers on it. I've marked https://ci.adoptopenjdk.net/computer/docker-marist-ubuntu1604-s390x-1/ offline to see if it has any negative effects. If so, this can be reopened.


Method 'ReadNodeInfo' ... does not have an implementation

I built the example plugin and ReClass throws this exception:
Method 'ReadNodeInfo' in type 'SamplePluginManaged.SampleNodeInfoReader' from assembly 'SamplePluginManaged, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation. (SamplePluginManaged)

Did you use the plugin with the ReClass.NET version from the release page? It will only work if you build ReClass.NET too because the sample plugin code is synched to the current ReClass.NET code.

Yes I did


Use multi-stage builds

Use Docker's multi-stage builds to simplify the image building process.

/lgtm

@jbianquetti-nami I believe we started pushing new images routinely, can you check that this new multi-stage build will work ?

@sebgoa I rebased this branch with the master branch.


Script will startup but will not run.

LOG
ofscraper_main_profile_2024-12-26.log
Description
After attempting to install many versions of the scraper many times, I've run into issues ranging from the scripting running the download but never attempting to download anything with the action completing under 10 seconds, to the script ( the newest version) ending and exiting before I can get to the main menu.
Any help or direction would be helpful.

You are going to want to uninstall everything for this scraper. pip/pipx, ofscraper, aiolimiter, python. then install python 3.11.6, pip/pipx, aiolimiter (must be version 1.1.0). Then install ofscraper. Doing all that should get the scraper working.

You are going to want to uninstall everything for this scraper. pip/pipx, ofscraper, aiolimiter, python. then install python 3.11.6, pip/pipx, aiolimiter (must be version 1.1.0). Then install ofscraper. Doing all that should get the scraper working.

Will attempt. I'll update you soon, thank you very much.

Hit with this error when attempting to uninstall pip.
log20241230.txt

Symptom: ofscraper closes with no download with "Finished script"
Solution: Pin aiolimiter to version 1.1.0:

if you used pip to install: pip install aiolimiter==1.1.0
if you used pipx to install: pipx inject ofscraper aiolimiter==1.1.0

The combination of these two solution has the scraper up and running correctly again. Thank you both for your assistance.


Implement Application Note resource page

Examples:

https://www.moleculardevices.com/en/assets/app-note/br/assess-virus-neutralization-with-rapid-hts-friendly-assay
https://main--moleculardevices--hlxsites.hlx.page/en/assets/app-note/br/assess-virus-neutralization-with-rapid-hts-friendly-assay

Tasks:

[ ] Overall content styling (might be mostly ok already)
[ ] PDF button styles incl. icon
[ ] Image alignment, width, margins
[ ] Image caption styling

WIP in https://github.com/hlxsites/moleculardevices/tree/application_node_page


Merging pg_closure_tree_rebuild

Hi.
I have started merging pg_closure_tree_rebuild into the gem. The only thing I did not moved yet are tests. Take a look please.
bundle exec rake closure_tree:rebuild TABLE=comments

I also added database.yml to .gitignore and added check in spec/support/database.rb because my local postgres settings differs with default in repo.

Please re-open when all tests pass.


Fix DropdownMenu focused item styles

Description
This PR fixes the style resolution for selected dropdown menu items (make it possible to provide a custom style and avoid hardcoded values when possible).
For the moment, I kept the default selected background which was previously set (onSurface.withOpacity(0.12)) to keep this PR focused on its goal which is to make it possible to overrides the defaults item style by providing a custom button style at the theme level or at the menu entry level.
Related Issue
Fixes https://github.com/flutter/flutter/issues/123736.
Tests
Adds 4 tests.

@justinmc Thanks in advance for having a look at the Google testing failure.

In the Google test failures I see some dropdowns that have become significantly more narrow with the change. Both the dropdown and the options. Could that be due to the padding change, or anything else?

In the Google test failures I see some dropdowns that have become significantly more narrow with the change. Both the dropdown and the options. Could that be due to the padding change, or anything else?

Yes, that's probably related to the small change I made to the padding logic.
I restored the previous padding logic, tell me if it is better now.


Added support for Open Graph arrays

Fixes #92
Once merged this would return Open Graph array when uniform is set True

@lopuhin I am sorry my text editor was set to 2 spaces, I'll fix it. I'll add a few test cases as well.

Thanks @ragnerok and thanks for review @Gallaecio :+1:


Null safety, Firebase messaging, Search

Major changes with this PR. The largest being the addition of null safety in all available packages and the addition of search.
To optimize search, we need to take a pdf and convert all text to JSON, using the instructions in README.md.
Clicking a searched item will navigate to that direct page.
Firebase messaging is still a work in progress, but appears to be working on a basic level with our initial Android testing.

LGTM


word-level level styling isn't pulled over

E.g. strong/italics.

Good catch. Thank you for reporting this


Arrow Shape Added

#62  Arrow Shape is added.
Please Review and Merge!

@Spyware007 , You can use these keywords to link issues to your PR.
Fixes #62
Closes #62

Okay @Sachin-chaurasiya 🤝


Validate plugins after they are built

This will require a browser-side check because the entry point may use browser-specific code such as window.navigator.

We are going to defer this to external testing, which will be part of the cookie cutter.


Main

merge original main ...

Sorry I tried to do a merge into my fork - not other way round :-) ....


JavaScript / TypeScript file adapter

E.g:
import { z } from 'zod';
import { loadConfig } from 'zod-config';
import { scriptAdapter } from 'zod-config/script-adapter';

const schemaConfig = z.object({
  port: z.string().regex(/^\d+$/),
  host: z.string(),
});

const filePath = path.join(__dirname, 'config.ts');

const config = await loadConfig({
  schema: schemaConfig,
  adapters: [
    scriptAdapter({ path: filePath }),
  ],
});

I believe using dynamic imports (i.e. await import(fileName)) would yield the desired results:

In a Javascript runtime, await import('whatever.ts') will fail unless the consumer has added a TypeScript loader to the environment
In a TypeScript runtime, await import('whatever.ts') will succeed
await import('whatever.js'), await import('whatever.mjs'), and await import('whatever.cjs') should always succeed

Hi @Lordfirespeed,
Thanks for the comment! From my understanding, you want to load .js or .ts files that export some properties to be validated against the schema, ensuring type safety. However, I’m curious about the specific use case for this.
If you already have the properties defined in your local setup, you could use the customEnv adapter, which receives a key-value record (object) as input. This might simplify your process:
import { yourConfigObj } from "./config";

const customConfig = await loadConfig({
  schema: schemaConfig,
  adapters: envAdapter({
    customEnv: yourConfigObj,
  }),
});

The primary goal of zod-config is to get type-safe data from external sources. If the data is already defined in the code, this approach could provide the same level of type safety and validation without the need for dynamic imports.
But maybe I'm just missing you use-case. Cheers!

I would like to use this feature in conjunction with #5 - this would allow me to

use .ts files to define my configuration, e.g

import { type Config } from './config'

export default {

} satisfies Partial<Config>

which means I can employ static type-checking when authoring configuration.
2. have default configurations for development, production checked into version control
3. Allow collaborators to add environment specific configuration using files prefixed with local-


Fix invalid parameter issue when force to close endpoint

What
Fix invalid parameter exception when force to close endpoint.
Why ?
When UcxWorkerWrapper forces close the endpoints, it will hit org.openucx.jucx.UcxException: Invalid parameter issue. It's because in UCX native code, ucp_ep_close_nbx, when UCP_EP_CLOSE_FLAG_FORCE is set, UCP_ERR_HANDLING_MODE_PEER also has to be set to key's err_mode.
How ?
Call setPeerErrorHandlingMode to set UCP_ERR_HANDLING_MODE_PEER when create endpoint connection and call setErrorHandler to set error handling code as well.

@petro-rudenko could you help to review the code change? Thanks.

Can you please sign your commit

Signed. Thanks Peter.


refactor: no warnings please

Removes a bunch of places where the compiler was complaining about potential nulls and seals a few publicly facing classes. Also adds a hashcode method where we implemented equals
Just future proofing before we get runnaway compiler warnings and it becomes hard to fix

Think this one got superseded, going to close this


fix(homepage): change price feed provider

:tada: This PR is included in version 1.9.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Clarification on URL attributes in Server deployment

"web_context_url": "https://website.domain.com/server",
"wa_name": "server",
"wa_url": "https://website.domain.com/arcgis",
"url": "https://machine01.machine.com:6443/arcgis",

node['arcgis']['server']['web_context_url'] = ArcGIS server web context URL. This is used in the federation recipe for the federated server services url:
https://devtopia.esri.com/WebGIS/arcgis-automation/blob/b13452064f84c73a2f79f4be587f0e6a6ffefe3a/cookbooks/arcgis-enterprise/recipes/federation.rb#L24

node['arcgis']['server']['wa_name'] = The name of the Web Adaptor used for ArcGIS Server. Default name is server.

node['arcgis']['server']['wa_url'] = URL of the Web Adaptor used for ArcGIS Server. Default name is https://<FQDN of the machine>/node['arcgis']['server']['wa_name'].

node['arcgis']['server']['url'] = ArcGIS Server URL. The default URL is https://<FQDN of the machine>:6443/arcgis using the fully qualified domain name of the machine on which Chef is running. If ArcGIS Server and the web adaptor are running on different machines, then this value should be explicitly set.


How can I set the lambda node version?

I am using lambda right now and in development environment, I am using node version 14.18.2.
The error happened when the amplify backend deployment script ran.
How can I set the lambda node version?
                                 ## Checking for associated backend environment...
                                 ## Backend environment association found
2022-01-04T12:01:27.115Z [INFO]: Amplify AppID found: d34ho3c461rmfp. Amplify App name is: app
2022-01-04T12:01:27.182Z [INFO]: Backend environment prod found in Amplify Console app: app
2022-01-04T12:01:41.789Z [WARNING]: - Fetching updates to backend environment: prod from the cloud.
2022-01-04T12:01:52.004Z [WARNING]: - Building resource api/app
2022-01-04T12:01:52.051Z [WARNING]: ✔ Successfully pulled backend environment prod from the cloud.
2022-01-04T12:01:52.163Z [INFO]:
2022-01-04T12:02:05.310Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2022-01-04T12:02:11.365Z [WARNING]: - Initializing your environment: prod
2022-01-04T12:02:22.266Z [WARNING]: - Building resource api/app
2022-01-04T12:02:22.311Z [WARNING]: ✔ Initialized provider successfully.
2022-01-04T12:02:22.734Z [INFO]:
2022-01-04T12:02:22.734Z [INFO]: Suggested configuration for new layer versions:
2022-01-04T12:02:22.735Z [INFO]:
2022-01-04T12:02:22.737Z [INFO]: appLayer
                                   - Description: Updated layer version  2022-01-04T12:02:22.734Z
2022-01-04T12:02:22.737Z [INFO]:
2022-01-04T12:02:35.459Z [WARNING]: ✖ An error occurred when pushing the resources to the cloud
2022-01-04T12:02:35.461Z [WARNING]: ✖ There was an error initializing your environment.
2022-01-04T12:02:35.469Z [INFO]: Error: Packaging lambda function failed with the error
                                 Command failed with exit code 1: yarn --production
                                 error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "^12.22.0 || ^14.17.0 || >=16.0.0". Got "12.21.0"
                                 error Found incompatible module.
                                 yarn install v1.22.0
                                 [1/4] Resolving packages...
                                 [2/4] Fetching packages...
                                 info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
                                 info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
                                 info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
                                     at runPackageManager (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-nodejs-function-runtime-provider/src/utils/legacyBuild.ts:66:13)
                                     at installDependencies (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-nodejs-function-runtime-provider/src/utils/legacyBuild.ts:40:3)
                                     at Object.buildResource [as build] (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-nodejs-function-runtime-provider/src/utils/legacyBuild.ts:13:5)
                                     at buildLayer (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-category-function/src/provider-utils/awscloudformation/utils/buildLayer.ts:46:40)
                                     at prepareResource (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/src/push-resources.ts:605:33)
                                     at async Promise.all (index 0)
                                     at prepareBuildableResources (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/src/push-resources.ts:601:10)
                                     at Object.run (/root/.nvm/versions/node/v12.21.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/src/push-resources.ts:173:5)
2022-01-04T12:02:35.497Z [ERROR]: !!! Build failed
2022-01-04T12:02:35.498Z [ERROR]: !!! Non-Zero Exit Code detected
2022-01-04T12:02:35.498Z [INFO]: # Starting environment caching...
2022-01-04T12:02:35.498Z [INFO]: # Uploading environment cache artifact...
2022-01-04T12:02:35.583Z [INFO]: # Environment caching completed
Terminating logging...

Thank you!

@NoCtrlZ You should be able to override the Node version in 'Your Amplify app > Build settings > Build image settings > Live package updates'. (Or, if you're doing this with IaC, you can use the _LIVE_UPDATES environment variable)

Hi @NoCtrlZ in addition to the solution provided above, here is the documentation: https://docs.aws.amazon.com/amplify/latest/userguide/custom-build-image.html#setup-live-updates

Thank you!
It works!


hack/benchmark: remove deprecated boom parameter

hack/benchmark/bench.sh script fails to run with -readall flag provided for boom. Here the commit where the flag was removed https://github.com/rakyll/boom/commit/7f4c62228ab4759a425a0aa929a02c11d6ddf35c.

lgtm


Mafs integration with Astro?

Hey, everyone! I discovered Mafs yesterday, and I was really excited to start playing with it today for a Calculus resource that I'm prototyping in Astro. Unfortunately, though, I'm having a hard time getting Mafs and Astro to work together. Can someone please help me troubleshoot this or understand why this won't work, maybe due to a limitation of Vite (which Astro uses under the hood) or SSGs in general?
I made a toy repo of Astro + Mafs here.
This toy example was created by using the Astro installer to create a new basic project, adding React, and then following the Mafs Installation instructions. HelloFx is in src/components/demo.jsx, and then I'm using this React component in src/pages/index.astro.
When I use the provided HelloFx code directly, I get the following error in my browser:
[ERROR] [vite] Named export 'Coordinates' not found. The requested module 'mafs' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'mafs';
const {Mafs, Coordinates} = pkg;

When I replace the HelloFx example import statement (import { Mafs, Coordinates } from "mafs";)  with the lines suggested in the ERROR message, I get an empty Mafs frame and an error message in my browser console (screenshot attached).
Can Mafs and Astro play nicely together?

You can do:
import { Circle, Coordinates, Mafs, Vector } from "mafs/build/index.mjs";

That works, but you lose type benefits

Thanks for the suggestion, @blackmann! Sure enough, I'm getting a pan-able coordinate axis now, which is exciting - gives me hope that Astro and Mafs can work together.
It'd be really nice to have type support though...

Hi @hoxiea,
I looked into this again and you can get type benefits by doing this instead:
Inside astro.config.[m]js, add the following to your config.
...
vite: {
  ssr: { noExternal: ['mafs'] }
}

Then you can directly import from mafs instead of mafs/build/index.mjs

Awesome, @blackmann - thank you so much for the prompt, helpful replies. I updated my GitHub repo with instructions and will leave it out there in case it helps anyone who finds it in the future.

Ehh it seems to me that the correct fix was to publish the CJS output using the .cjs extension to mark it as CJS explicitly, not removing the "type": "module" field, that part was definitely correct: the source is written in ESM and it's published as ESM. It just so happens to dual publish a CJS bundle but that's secondary.


Get bot to respond inside discord channel it is being prompted from

@Wildric-Auric
Hi! is it possible to modify the python script so that the bot can respond inside the channel the prompt is given in?
E.g. say I have
Discord ==> Channel A, Channel B , Channel C
Bot is currently setup in Channel A
Bot is then added to Channel B from inside discord
** Currently if I prompt the bot from Channel B it submits the reply into Channel A **
Is there a way to update the python script so that the bot can reply inside channel B from where the prompt came from?
Thank you!

In all the payloads you have in Salai.py you should pass the desired channel id instead of Globals.CHANNEL_ID. And the channel id where the command is invoked can be found using ctx.channel_id in each discord command function you have in main.py.

Working great!! Thank you!! :)


upstream  page displays table  error

eg:
json {     "node": {         "nodes": [             {                 "value": {                     "nodes": {                         "nodejs.service:3000": 1                     },                     "desc": "yapi",                     "type": "roundrobin"                 },                 "createdIndex": 173,                 "key": "/apisix/upstreams/00000000000000000164",                 "modifiedIndex": 173             },             {                 "value": {                     "nodes": {                         "go.service:33902": 1                     },                     "desc": "grpc~test",                     "type": "roundrobin"                 },                 "createdIndex": 192,                 "key": "/apisix/upstreams/00000000000000000192",                 "modifiedIndex": 192             },             {                 "value": {                     "desc": "test~upstream",                     "nodes": {                         "<IP_ADDRESS>:81": 2,                         "foo.com:80": 3,                         "remote-local.test:80": 1                     },                     "type": "roundrobin"                 },                 "createdIndex": 329,                 "key": "/apisix/upstreams/00000000000000000328",                 "modifiedIndex": 329             },             {                 "value": {                     "nodes": {                         "nodejs.service:9090": 1                     },                     "desc": "yapiinstall",                     "type": "roundrobin"                 },                 "createdIndex": 163,                 "key": "/apisix/upstreams/00000000000000000139",                 "modifiedIndex": 163             }         ],         "modifiedIndex": 129,         "createdIndex": 129,         "key": "/apisix/upstreams",         "dir": true     },     "action": "get" }
page display eg:

apisix server ver 0.9
apisi dashboard ver 0.9

Hi, I'd love to know which browser are you using?

I think the green background is related to some Chrome extensions, could you please have a try on Privacy mode? for no extensions will execute in Privacy mode.

Ok. I do try

have not changed. It's the same on your side？

I add second node or more  on upstreams are displays error,only one nodes displays not are problume .

I see... I thought your question is why the browser is in Green background..
Please use the latest codes from master branch instead, if you could not build the Dashboard, I'm glad to help you.

dashboard.tar.gz
You could use this latest dashboard.

thanks. I have a try

So good. now not problem

Thanks for your issues found :)


Keyboard manager (Add new send)

Description of the new feature / enhancement
In the keyboard manager-de send section there is an action option for "Previous track", i.e. the previous track, but there is no action option to switch to the next one. Please add it. Thank you
Scenario when this would be used?
To quickly switch music to the previous or next track
Supporting information
No response

This is already possible in the keyboard manager. The list is alphabetic.

Yes. There is a command to move to the next part. But there is no command to return to the previous track.

There is also a command for Previous Track.


MB-5107 fix address loading in edi

Description
When loading an address off a duty station some of them may have a blank street_address_1 field which was causing validation issues for the N3 segment. This PR fixes that by checking the field to make sure it meets the requirements before including it. If it is not included then the field is skipped.
{
  "Payload": {
    "detail": "EDI failed validation: Key: 'Invoice858C.Header[11].AddressInformation1' Error:Field validation for 'AddressInformation1' failed on the 'min' tag",
    "instance": "eeaea86a-8051-4b7a-8f72-fced008f7396",
    "title": "Internal Server Error"
  }
}

Reviewer Notes
The best way to verify this is to run through the whole flow. I didn't add any tests because there is no way to create an Address with a blank street address using testdatagen as it fails validations.
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make db_dev_e2e_populate
make server_run
make client_run

Create a client and use Aberdeen Proving Ground or other duty station that has no street_address_1
Add the TAC and other fields on the Orders
Approve the move
Collect the MTO id
Run scripts/prime-api-demo <mto id> to walk through the prime portion
Approve the move as the TIO in the ui
Continue the demo script to generate the edi
Should have an edi and not any errors generated.

Code Review Verification Steps

[X] The requirements listed in
Querying the Database Safely
have been satisfied.
[X] Request review from a member of a different team.
[X] Have the Jira acceptance criteria been met for this change?

References

Jira MB-5107 for this change

Messages

:book:
:link: MB-5107

  Generated by :no_entry_sign: dangerJS against 73eb771e2536fa50f18fee91ee1858174a2a4c74


Remove OpenOffice

NOTE: Each discrete (stand-alone) request should be in its own issue.
Put "Remove ____" as Title
[X] Done
Explain why it should be removed
Apache Open Office has long only seen rudimentary bug fix releases. Mostly concerned with minor updates to keep it working on newer OS releases.
It hasn't seen a major release since 2014 (7 years ago) and major security vulnerabilities have gone unfixed for a long time.
See the Open Letter to Apache OpenOffice  for more information.
Additional context
Alternatively, one could redirect users looking for Apache OpenOffice to LibreOffice, referencing the above blog post.

I agree with you, do you mind to create a PR to remove it?


Use VBoxManage to detect VM start rather that screenshot parsing

VirtualBox has some default properties that are exposed on guests, and VBoxManage includes a way to wait until a property is defined.
VBoxManage guestproperty wait sol-11_4-vbox "/VirtualBox/GuestInfo/OS/NoLoggedInUsers" seems to work to avoid the repeated looping/OCR here: https://github.com/vmactions/solaris-vm/blob/13811b4cde4b621079aa7a7d545e0e06d1924cb6/index.js#L167-L170
This probably does lead to booting the VM any faster, but it should allow dropping the tesseract dependency and avoiding the brew install step.

https://docs.oracle.com/en/virtualization/virtualbox/6.0/user/vboxmanage-guestproperty.html

It requires that: "provided that a guest is running and has the Guest Additions installed. "


[Extensibility Bug ] table 1690 "Bank Deposit Header" -OnBeforeGetNoSeriesCode

Describe the request
local procedure GetNoSeriesCode(): Code[20]
var
    NoSeriesCode: Code[20];
    IsHandled: Boolean;
begin
    SalesReceivablesSetup.Get();
    IsHandled := false;
    OnBeforeGetNoSeriesCode(Rec, SalesReceivablesSetup, NoSeriesCode, IsHandled);
    if IsHandled then
        //+CHANGE
        //-exit;
        exit(NoSeriesCode);
        //-CHANGE

    NoSeriesCode := SalesReceivablesSetup."Bank Deposit Nos.";
    OnAfterGetNoSeriesCode(Rec, NoSeriesCode);
    exit(NoSeriesCode);
end;

Additional context
The event OnBeforeGetNoSeriesCode is useless in its current form, since the exit does not return the NoSeriesCode.Internal work item: AB#495248

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14915.


Update page header and add announcements section

Removes announcements
Removes Live stream
Moves logo next to heading

Page header
What
Update the header design, this entails:

On desktop moving the logo top right
On mobile moving the logo up to below the title
On both, adjust the black and dark grey sections to match the design

Moving announcements out the header is a separate story:
Why
Getting the logos out of the way, and making it clearer to users that these are part of the heading and not an interactive page element.
https://trello.com/c/NZNvb7ik/213-update-the-header-to-match-the-new-design
Announcement section
##What
Update the styling and position of the announcements section to move it out of the header and immediately below the guidance accordion.
##Why
Prioritising guidance and support in the page hierachy.
https://trello.com/c/X4lJmg3w/212-update-the-styling-and-positioning-of-the-announcements-section
Screenshots
Before

After

Tests failing due to press conference changes not merged in yet.
Blocked by https://github.com/alphagov/collections/pull/1694


Add license name if missing from license text

How are you using generate-license-file?
Library
Is your feature request related to a problem? Please describe.
Many license text files do not explicitly mention the license name/spdx identifier because it is unnecessary in the context of the source repo where the spdx identifier is in the package.json, and Github now shows the license spdx.
However when the license text is used in a different context, it is helpful for the license name/spdx to be included.
Then the reader does not need to be able to manually convert the license text into the license name, which can be rather tricky for some of licenses that are quite close.
Describe the solution you'd like
This tool could check whether the license name/identifer is not included in the license text, and add it.
Additional context
e.g. https://github.com/Raynos/function-bind/blob/master/LICENSE

Hey @jayvdb,
It's important to us that we don't modify the licenses of any packages as we believe the licenses should be handled verbatim.
For us to provide the license name or SPDX identifier for a license (either in the content itself or as additional data) we would need to be guessing based on what we think the license file contains. While some websites and tools provide this functionality (even GitHub do it), it something we're not willing to do as we'd prefer to rely on what the package authors have explicitly declared.

@tobysmith568 ,

It's important to us that we don't modify the licenses of any packages as we believe the licenses should be handled verbatim.

Ya, I agree.
When I recreated this issue, there was only a "content" field, and was didnt dare ask for a new field, so I suggested enhancing the "content" field.
But in light of https://github.com/TobyAndToby/generate-license-file/issues/529 , I think a better solution would be to add an "spdx" field.
Then I can do my presentation enhancements in my presentation layer.


Uncaught Error: Can't save buffer with no file path

[Enter steps to reproduce below:]

I pressed the shortcut for the "first command" which usually compiles my code when suddenly this error occurred.

Atom Version: 1.0.2
System: Linux Mint
Thrown From: build-tools package, v3.0.1
Stack Trace
Uncaught Error: Can't save buffer with no file path
At /usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:983

Error: Can't save buffer with no file path
  at TextBuffer.module.exports.TextBuffer.saveAs (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:983:15)
  at TextBuffer.module.exports.TextBuffer.save (/usr/share/atom/resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:978:19)
  at TextEditor.module.exports.TextEditor.save (/usr/share/atom/resources/app.asar/src/text-editor.js:579:26)
  at Object.module.exports.saveall (/home/user/.atom/packages/build-tools/lib/main.coffee:106:14)
  at Object.module.exports.execute (/home/user/.atom/packages/build-tools/lib/main.coffee:153:6)
  at atom-workspace.subscriptions.add.atom.commands.add.build-tools:first-command (/home/user/.atom/packages/build-tools/lib/main.coffee:55:40)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:241:29)
  at CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:3:61)
  at CommandRegistry.module.exports.CommandRegistry.dispatch (/usr/share/atom/resources/app.asar/src/command-registry.js:156:19)
  at EventEmitter.<anonymous> (/usr/share/atom/resources/app.asar/src/window-event-handler.js:70:30)
  at emitOne (events.js:77:13)
  at EventEmitter.emit (events.js:166:7)

Commands
     -2:35.9.0 linter:set-bubble-transparent (atom-text-editor.editor.is-focused)
     -0:06.2.0 core:copy (atom-text-editor.editor.is-focused)
     -0:05.0 core:paste (atom-text-editor.editor.is-focused)
     -0:00.6.0 build-tools:first-command (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "autoHideMenuBar": true,
    "disabledPackages": [
      "metrics"
    ]
  },
  "build-tools": {}
}

Installed Packages
# User
build-tools, v3.0.1
language-cmake, v0.1.4
linter, v1.2.4
minimap, v4.12.0

# Dev
No dev packages

It seems that it has something to do with an new and unsaved "untitled" tab, i just created for some quick todo-notes.
After closing this tab, everything goes normal again...

Thanks for reporting the error


Renaming content #146

Note: this PR is being merged directly to main. Merge to 1.1.0 branches as needed.
Issue #, if available: #146
Description of changes:
Renaming content for the solution name change (no code)
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

We could change the About section in the GitHub main page repository details.
AWS Virtual Waiting Room solution...  => Virtual Waiting Room on AWS solution...

Additionally the file names in the deployment/  folder could be renamed from aws-virtual-waiting-room-getting-started.json to virtual-waiting-room-on-aws-getting-started.json though this would entail other changes like the File Structure section in the landing page.

The deployment template themselves have fields like "Description": "AWS Virtual Waiting Room ... " .
We also have miscellaneous titles and comments in some files with the AWS Virtual Waiting Room text.
Summary of files:
deployment/aws-virtual-waiting-room-api-gateway-cw-logs-role.json:
  2    "AWSTemplateFormatVersion": "2010-09-09",
  3:   "Description": "AWS Virtual Waiting Room API Gateway CW Logs Role %%VERSION%%",
  4    "Resources": {

deployment/aws-virtual-waiting-room-getting-started.json:
  2      "AWSTemplateFormatVersion": "2010-09-09",
  3:     "Description": "AWS Virtual Waiting Room Getting Started %%VERSION%%",
  4      "Resources": {

deployment/aws-virtual-waiting-room-sample-inlet-strategy.json:
  2      "AWSTemplateFormatVersion": "2010-09-09",
  3:     "Description": "AWS Virtual Waiting Room Inlet Strategies %%VERSION%%",
  4      "Mappings": {

deployment/aws-virtual-waiting-room.json:
  2      "AWSTemplateFormatVersion": "2010-09-09",
  3:     "Description": "(SO0166) AWS Virtual Waiting Room %%VERSION%%",
  4      "Mappings": {

source/control-panel/src/App.vue:
  48    created() {
  49:     document.title = "AWS Virtual Waiting Room Control Panel";
  50    },

source/control-panel/src/components/Banner.vue:
   9

  10:
AWS Virtual Waiting Room Control Panel

  11

source/core-api-authorizers-sample/chalice/merge_template.json:
  1  {
  2:     "Description": "AWS Virtual Waiting Room API Sample %%VERSION%%",
  3      "Parameters": {

source/core-api-authorizers-sample/www/waitingroom/index.html:
  8      <title>
  9:         AWS Virtual Waiting Room Entry
  10      </title>

source/openid-waitingroom/chalice/app.py:
  4  This Lambda function provides the endpoint functions
  5: for the OpenID interface of the AWS Virtual Meeting Room
  6  """

source/openid-waitingroom/chalice/merge_template.json:
  1  {
  2:     "Description": "AWS Virtual Waiting Room OpenID Adapter %%VERSION%%",
  3      "Parameters": {

source/openid-waitingroom/www/login.html:
   9      <title>
  10:         AWS Virtual Waiting Room Entry
  11      </title>

source/sample-waiting-room-site/src/views/CheckOut.vue:
  12        You can complete the purchase of your widget now! This component will
  13:       request an authentication token from the AWS Virtual Waiting Room API and
  14        call the site's protected API to finish.

source/shared/aws-virtual-waiting-room-common/setup.py:
  14<EMAIL_ADDRESS>  15:     description="Common Python modules for AWS Virtual Waiting Room",
  16      long_description=long_description,

source/token-authorizer/chalice/merge_template.json:
  1  {
  2:     "Description": "AWS Virtual Waiting Room API Gateway Authorizers %%VERSION%%",
  3      "Parameters": {

Also, the build is failing due to python safety package flagging click - do change the requirements.txt file in the PR?

This PR is for public facing documentation and related textual content. We're ok if the build fails since there's no code change here. For the next release, we'll need to schedule the code-related content changes, like templates, externalized strings, workflows, etc.
You should add the deployment/ folder items called out above to #146 so we can track on the 1.1.0 release.


A beta dataset doesn't show correct number of fastq data files in Browser

For the 1.3 Million Brain Cells from E18 Mice beta dataset, there are listed "0" fastq data files in the Browser (see screenshot). This project actually has 16,377 fastq files.

┆Issue Number: HCAB-385

@hannes-ucsc Is this an Azul issue?

Took a cursory look and it appears as though the data files in that project are described by metadata as having "file_format": "fastq" even though their file extension is fastq.gz.  The Data Browser seems to look for fastq.gz in the fileTypeSummary of the Azul response but display that in a column labelled fastq. That's why the correct number is displayed, but in the Other column. The Data Browser should combine the fastq and fastq.gz entries of fileTypeSummary since both formats conceptually represent a "fastq" file. The latter is just a compressed form of the former.
However, this is also metadata inaccuracy. If the file extension is fastq.gz, the file_format field should reflect that accurately since .fastq and .fastq.gz are different formats. Inaccuracies like that are should be avoided with a validation rule, and the project needs to be reingested with accurate metadata. If we let this slide, everyone downstream (external and internal users alike) will have to incorporate business logic into their programs to work around this inaccuracy.
We can also eliminate the file_format property and have Azul inspect the actual file extension directly. With that logic in place, the higher-level content_description field that the metadata team is planning on implementing could replace file_format, not augment it.
Assigning back to @malloryfreeberg and @NoopDog for feedback.

Nice sleuthing!
As I'm thinking through the fastq vs. fastq.gz issue, I think what we really want in this field is fastq, but the format of the file is indeed fastq, it just also happens to be compressed. I think in general we want to ignore indicating compression in the file_format field, as that doesn't explain how the file itself is formatted.
I think you're right that after we merge the content_description PR, that field could replace the file_format field.
I will make a note to wranglers that we should update all current spreadsheets with fastq for file_format, although I don't know of any specific plans to re-ingest any datasets at the moment (I can haz AUDR?)

@malloryfreeberg, if you officially change from fastq.gz to fastq now, we'd just be increasing the confusion. We'd have 14 projects using fastq.gz to describe fastq.gz files, and the rest would use fastq. Can't we just stick with fastq.gz and fix this one outlier project by reingesting it?
An ontology term would the best solution as it could express that fastq.gz is an enconding of the fastq format. Then again, there already are standards for this: HTTP and MIME.
Content-Type: application/x-fastq
Content-Encoding: gzip

To summarize, I would strongly urge to not increase the entropy by switching to fastq now. I would also recommend adopting existing standards like MIME. The DSS already has support for content type.
PS: Asking wranglers to do something is good, implementing a validation rule is better.

@hannes-ucsc one step ahead of you on using ontologies (EDAM) for the file_format field ;) (https://github.com/HumanCellAtlas/metadata-schema/issues/812).
The EDAM ontology does include a term for "fastq", but not for "fastq.gz". While it's not impossible to get terms added to the ontology, I want to be sure that we do it for the right reasons. I still think the correct way to refer to the format of a sequencing data file is "fastq". The fact that it was compressed using gzip prior to file transfer is not really relevant to how the file is formatted.
Would like to hear other's opinions, maybe on the next metadata call.

I think it is relevant, @malloryfreeberg. Anyone processing the (meta)data programmatically would not be helped by file_format being fastq if the file is actually a fastq.gz. They would still have to look at the file extension to decide how to actually open and load the file. If they have to look at the file extension anyway, they can get all of the necessary information out of it (the fact that it's a fastq and that it's gzipped) and would not even need to consider file_format at all.
What is the point of this field if it doesn't accurately describe how a file needs to be handled?
Two alternative suggestions: Either add fastq.gz to EDAM or add a file_encoding field to metadata schema. For the encoding I would draw upon MIME, not some ontology specific to bioinformatics (EDAM doesn't even have zip, gzip or bzip2). I would have said the same for the ontology on file_format but MIME doesn't have fastq so I guess EDAM is the only option for that.

For now, Data Browser will count fastq and fastq.gz files as "fastq" for data Count column and exclude fastq.gz from "Other" count. File facets and download flow to remain unchanged.

I think we really need to rigorously define what is meant by file_format.   Is it the bytes on disk or the data the program reads.  I  don't think of fastq.gz as a different format than fastq.  Many programs transparently handle compressed or uncompressed data, others need special handling for compressed data
I think it would be clearer to have the concepts of data format and compression format separate.  Maybe it is data_format and file_format, or the suggest file_encoding.
Until we have the ability to update metadata, MHO is we should just hack around it until we can update metadata.

In Prod select "1.3 Million Brain Cells from E18 Mice" and it shows 16,377 Fastqs but the "File" drop-down shows 1 fastq.

@hannes-ucsc wrote: In a response to /repository/projects, the counts in termFacets represent number of projects, not number of files. In this case, for example, the termFacets counts means that there are 16 projects with fastq.gz files and 1 project with fastq files.

ug thx

Created https://app.zenhub.com/workspaces/dcp-backlogs-5ac7bcf9465cb172b77760d9/issues/humancellatlas/data-browser/489 to make the units in the facets more visible.


Implement API call to run smart contract methods locally in SmartContractsController

Developers may want to return the result of a particular method locally without running the code on-chain. The primary use case will be in the retrieval of some static information. E.g. maybe there is a GetAverageHoldings method in an ICO contract that returns totalSupply / investors.Count.
They should be able to do this via the API.
We need a public API call, built into SmartContractsController that takes in all the required parameters  (contract address, method name, method parameters), executes a method, and returns a result. Ideally it will also return the amount of gas used.
This functionality already exists roughly in the Executor.cs file, so it should be a matter of using that and returning the result.
Current Bounty: 20 STRAT

A nice extension (no bounty from me) would be to enable a symmetric programming model where an SC could also be deployed off-chain to support the wrapping of an identical implementation of a set of business rules around off-chain data. The existing Visual Studio SSC template test harness provides most of the source code you need to implement this capability.

When resolving a method, if given no params this should check all properties first.


Merge back main to next

docs: update website to redirect to the new domain (#62)

docs: misc quality of life improvements (#63)

Codecov Report

Merging #65 (d0260db) into next (a2496c5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              next       #65   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          249       249
  Branches        13        13
=========================================
  Hits           249       249

Continue to review full report at Codecov.

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