Divide the single VisibleEntities list into separate lists for 2D meshes, 3D meshes, lights, and UI elements, for performance.

This commit splits VisibleEntities::entities into four separate lists: one for lights, one for 2D meshes, one for 3D meshes, and one for UI elements. This allows queue_material_meshes and similar methods to avoid examining entities that are obviously irrelevant. In particular, this separation helps scenes with many skinned meshes, as the individual bones are considered visible entities but have no rendered appearance.
Internally, VisibleEntities::entities is a HashMap from the TypeId representing a QueryFilter to the appropriate Entity list. I had to do this because VisibleEntities is located within an upstream crate from the crates that provide lights (bevy_pbr) and 2D meshes (bevy_sprite). As an added benefit, this setup allows apps to provide their own types of renderable components, by simply adding a specialized check_visibility to the schedule.
This provides a 16.23% end-to-end speedup on many_foxes with 10,000 foxes (24.06 ms/frame to 20.70 ms/frame).
Migration guide

check_visibility and VisibleEntities now store the four types of renderable entities--2D meshes, 3D meshes, lights, and UI elements--separately. If your custom rendering code examines VisibleEntities, it will now need to specify which type of entity it's interested in using the WithMesh2d, WithMesh, WithLight, and WithNode types respectively. If your app introduces a new type of renderable entity, you'll need to add an explicit call to check_visibility to the schedule to accommodate your new component or components.

Analysis
many_foxes, 10,000 foxes: main:

many_foxes, 10,000 foxes, this branch:

queue_material_meshes (yellow = this branch, red = main):

queue_shadows (yellow = this branch, red = main):

How do UI entities fit into this categorization?

They should be 2D meshes, but I'll check. If they aren't handled then I can just add them to the HashMap. (This is why I made it extensible.)

I added UI entities to the hash map.

Addressed review comments and merge conflicts.

@pcwalton once merge conflicts are resolved I'll merge this. The doc improvements mentioned by @kristoff3r would be great too.

Done.

The migration guide here was a little confusing -- at first, it seemed that I would have to implement my own check_visibility system, but really you just have to insert a system like so:
.add_systems(PostUpdate, (
                check_visibility::<With<MyCustomRenderable>>,
            )
                .in_set(VisibilitySystems::CheckVisibility))

Maybe this could be added to the migration guide?


Errors with latest version (2.1.3)

Hi I loved your old app I used it on 7 and 8 all the time. I got windows 10 so Im using the new version except when I click on the icon the devices and volume rocked appear 3/4ths of the way off screen. When I go into the settings I cant exit the application. Hitting the "x" or pressing close forces errors and and eventual crash. Note it does still work just needs some fixes I guess. Thanks

Can you attach some screenshots and windows event log with the error ? maybe this application needs some logging

Can you attach some screenshots and windows event log with the error ?

Sure. I have picture attached. This doesn't happen on my desktop just my laptop. I've tried reinstalling. Both computers are running Windows 10. The buttons still function. Hope this helps.
----- Reply message -----
From: "Nikhil Mujumdar"<EMAIL_ADDRESS>To: "sirWest/AudioSwitch"<EMAIL_ADDRESS>Cc: "Audiofileloverdudeman"<EMAIL_ADDRESS>Subject: [AudioSwitch] Errors with latest version (2.1.3) (#53)
Date: Fri, Feb 12, 2016 7:41 PM
Can you attach some screenshots and windows event log with the error ?
—
Reply to this email directly or view it on GitHub.


finally fix AQL error messages in all contexts, so that they are usable

Don't absorb error messages somewhere en route, but pass them on to end users.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/3973/


Speedup EnsoRootNode.buildFrameDescriptor

Follow-up of https://github.com/enso-org/enso/pull/10837 and #10729, part of #10833.
https://github.com/enso-org/enso/blob/396d70ddc0184473d42146a440b23226221f1f3e/engine/runtime/src/main/java/org/enso/interpreter/node/EnsoRootNode.java#L65-L74
still uses LocalScope.scope().allDefinitions() that forces deserialization of alias analysis metadata for every EnsoRootNode created, that is created for at least every Enso method. Let's remove the dependency on alias metadata from EnsoRootNode. Optimally, we should use only FramePointer metadata for that.
Further notes in https://github.com/enso-org/enso/pull/10837#issuecomment-2293766695

Looks like FramePointer will not be enough. I need to associate result of allDefinitions() with every IR that contains AliasMetadata.RootScope I think now.


No XmlDocSig for MemeberFunctionOrValue types that have no parent entitty

If you bind a non F# entity, then retrieve its symbol there is no xmldocsig present.
let §test§ = NonFSharpEntity()

This should be patchable, just need to figure out how to get the TyconRef

Strange, it looks like this should be populated, see

https://github.com/fsharp/FSharp.Compiler.Service/blob/master/src/fsharp/vs/Symbols.fs#L442

https://github.com/fsharp/FSharp.Compiler.Service/blob/d9840d00a810d841901d380730b3274d36875269/src/fsharp/vs/Symbols.fs#L133

https://github.com/fsharp/FSharp.Compiler.Service/blob/89d936ca760b790f7741a0bab9122002a792810c/src/fsharp/vs/ServiceDeclarations.fs#L308

And it would if the binding was an entity, in this case its a FSharpMemberOrFunctionOrValue where its a value bound to a normal object constructor.
I think this needs to be added to FSharpMemberOrFunctionOrValue
    member __.XmlDocSig =
        checkIsResolved()

        match d with
        | E e ->
            let range = defaultArg __.DeclarationLocationOpt range0
            match ItemDescriptionsImpl.GetXmlDocSigOfEvent cenv.infoReader range e with
            | Some (_, docsig) -> docsig
            | _ -> ""
        | P p ->
            let range = defaultArg __.DeclarationLocationOpt range0
            match ItemDescriptionsImpl.GetXmlDocSigOfProp cenv.infoReader range p with
            | Some (_, docsig) -> docsig
            | _ -> ""
        | M m ->
            let range = defaultArg __.DeclarationLocationOpt range0
            match ItemDescriptionsImpl.GetXmlDocSigOfMethInfo cenv.infoReader range m with
            | Some (_, docsig) -> docsig
            | _ -> ""
        | V v ->
            match v.ActualParent with
            | Parent entityRef ->
                match ItemDescriptionsImpl.GetXmlDocSigOfValRef cenv.g entityRef v with
                | Some (_, docsig) -> docsig
                | _ -> ""
            | ParentNone ->
                //If there is no parent then we try to use the full types xmldocsig
                //This happens with a let binding on a normal type constructor binding i.e let aaa = Document()
                let fullType = __.FullType
                if fullType.HasTypeDefinition then fullType.TypeDefinition.XmlDocSig
                else ""

I suppose it depends on whether you think that a local binding should have an xmldocsig

A local binding could show the xmldoc sig for its actual entity if we wanted it to.  Ill close this for now.


Add category counts to category coverage plot's ylabel

Currently, the concrete values count shown when the row is being pointed at.
@Alexandr1988 states that putting values count to ylabel on a graph will be more convenient.

It requires additional annotations, seems too complicated and almost no benefits.


Support for Kotlin properties

With this PR it is possible to annotate Kotlin properties directly utilising getters and setter instead of direct access to fields.
Also, it resolves https://github.com/pushtorefresh/storio/issues/735 since there is no need anymore to map fields to constructor parameters.

Codecov Report

Merging #776 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #776   +/-   ##
=======================================
  Coverage   96.85%   96.85%
=======================================
  Files          87       87
  Lines        2638     2638
  Branches      296      296
=======================================
  Hits         2555     2555
  Misses         50       50
  Partials       33       33

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 da7a1f5...4e36e74. Read the comment docs.


Retrieve a selection of dicom from a PACS

Hi everyone,
I am talking about the dicom retrieve in Slicer. Currently, we cannot retrieve from the PACS, a selection of dicom series of the selected study. The whole study has to be retrieved. It can take time if the study is big, and we are only interested to a single series.
Probably a wrong call to currentStudiesSelection instead of currentSeriesSelection
https://github.com/commontk/CTK/blob/a75a73616e2c2ecbf9762c1de673466873a3e8ed/Libs/DICOM/Widgets/ctkDICOMQueryRetrieveWidget.cpp#L305
I can do a PR if needed.

I am not very familiar with DICOMweb and I think in hospital environments  DICOMweb is probably not available. For instance, we are using dicom4chee and DICOMweb is not available to my knowledge.
So, if I add in addition a collapisble button for "Data sources" (generaly improve this window) and change currentStudiesSelection to currentSeriesSelection you would considered it for review?


callnote-premium formulae error

Hi! While installing callnote-premium getting this:
$ brew cask install callnote-premium
==> Downloading http://callnote.kandasoft.com/callnote-premium-install-2.3.7.pkg
Already downloaded: /Library/Caches/Homebrew/callnote-premium-2.3.7.pkg
==> Running installer for callnote-premium; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
==> installer: Package name is Callnote
==> installer: Certificate used to sign package is not trusted. Use -allowUntrusted to override.
Error: Command failed to execute!

==> Failed command:
["/usr/bin/sudo", "-E", "--", "/usr/sbin/installer", "-pkg", "#<Pathname:/usr/local/Caskroom/callnote-premium/2.3.7/callnote-premium-install-2.3.7.pkg>", "-target", "/"]

==> Output of failed command:
installer: Package name is Callnote
installer: Certificate used to sign package is not trusted. Use -allowUntrusted to override.

==> Exit status of failed command:
#<Process::Status: pid 15308 exit 1>

Any ideas?

Reporting bugs.

Fixed in https://github.com/caskroom/homebrew-cask/pull/13381. Feel free to reopen this issue if the problem persists.


chore(protocol): move automata-attestation and verifiers up in file hierarchy

Moved contracts/thirdparty/automata-attestation to contracts/automata-attestation as we made quite some modification to the original code. Keeping the files in thirdparty/ may confuses oursleves.
Moved contracts/L1/verifiers to contracts/verifiers so all proof related code is more independent of the core protocol code.
zero logical change and used 'git mv' to keep all the revision history

Will notify SigmaP - as it might refer to their workflow of integrating their tests.


animation not showing when ajax request

In my website http://www.eventiame.com/publicar/ there's a form with several fields.
When the page loads it shows the pace animation (it does some ajax request when loading so that's ok) but when i submit the form (i do it via ajax too) the animation doesn't appear.
I tested uploading an image (so it would take longer) and still no animation is shown.
I also tried changing the "async" parameter to true or false in the akax request but still no good.
Here's the specific code for the ajax call which is not working:
submitHandler: function(form) {
var urlPost = "../data/shops/add.php";
var postData = new FormData(form);
$.ajax(
{
type:"POST",
async: false,
contentType: false,
processData: false,
cache: false,
url: urlPost,
data: postData ,
dataType:"json",
//crossDomain: true,
success: function(result) {
if (result.Status) {
alert(result.Message);
if (result.Status == "OK") {
location.reload();
}
}
console.log(result);
},
error: function(data) {
if (result.Status) {
alert(result.Message);
}
console.log(data);
}
});
        }

RanadeepPolavarapu thanks it worked for me.

@ahmedali5530 No worries. Glad I could help.

@RanadeepPolavarapu Thanks I'll sure use it in my next project :+1:

@RanadeepPolavarapu it did not work for me.
I use $.post() method with Json.  But it  works with $.getJSON :)

@rajlaksh Hmm that's odd. My custom build has the following HTTP methods already added in trackMethods: ['GET', 'POST', 'PUT', 'DELETE', 'REMOVE'],. I just tested $.post() and it works.

@RanadeepPolavarapu i tried your solution but it didn't work with jquery v2.2.x
I'm using $.Post for bookmark and jquery-ui autocomplete. $.getJSON works without any problem.
i'm using pace.min.js from http://github.hubspot.com/pace/docs/welcome/
I hope developer will add POST in ajax column :+1:

I was change option of pace:
Pace.options = {
restartOnRequestAfter: true
}
Now it work with every ajax

i had a trouble with pace, after long time i realized that the request with 'async:false' are not triggering the progress bar,  all others do ...
and dont forget to put 'trackMethods: ['GET', 'POST', 'PUT', 'DELETE', 'REMOVE']' :)

In this way it works perfectly for me.
<script>
    window.paceOptions = {
        ajax: {
            trackMethods: ['GET', 'POST', 'PUT', 'DELETE', 'REMOVE']
        },
        restartOnRequestAfter: true
    };
</script>

<script src="bower_components/pace/pace.js"></script>

Thanks for above answers!

@FrancoPhoenix perfect! Thank you!


Change Bar Width of Grouped Data

Hey how's it going?
I've been trying for a while to change the width of the bars in c3. It works fine if you have single data sets but if you group your data like in the example beneath it seems to be treated as one dataset and the ability to add white space between the bars is lost.
This is how it currently appears:

This is how I would like it to appear:

Is this possible with the current framework? Any help or suggestions would be great.
Many thanks
Dom

did you find any solution? I have a similar problem.

There is a bar.space option (value between 0 and 1) and a bar.width.ratio option (value between 0 and 1) that should resolve your issue.


Improve nobles behaviour

Codecov Report

Merging #371 (bbdfa37) into master (1d2418e) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #371      +/-   ##
==========================================
- Coverage   87.67%   87.66%   -0.02%
==========================================
  Files          80       80
  Lines        5686     5673      -13
==========================================
- Hits         4985     4973      -12
+ Misses        701      700       -1

Flag
Coverage Δ

unittests
87.66% <100.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

base/forms.py
71.01% <100.00%> (-0.16%)
:arrow_down:

base/models/outline.py
94.33% <100.00%> (+0.04%)
:arrow_up:

base/models/weight_maximum.py
90.00% <100.00%> (+0.25%)
:arrow_up:

base/views/outline_initial_views.py
82.25% <ø> (ø)

utils/outline_complete.py
95.51% <ø> (ø)

utils/outline_initial.py
97.50% <ø> (ø)

utils/write_noble_target.py
96.50% <100.00%> (+0.38%)
:arrow_up:

utils/write_ram_target.py
95.87% <100.00%> (+0.18%)
:arrow_up:


USelectMenu and VerticalNavigation are always gray

I am implementing a theme picker and just noticed that VerticalNavigation and USelectMenu are always in gray/white color. Is this by design? I think it should change colors based on whatever primary and gray color I set in app config

The SelectMenu is based on the Select component so you can use the color or any other prop: https://ui.nuxtlabs.com/forms/select#style
Those props are not specified in the SelectMenu documentation to avoid repetition but the first sentence should explain by itself:

The SelectMenu component renders by default a Select component and is based on the ui.select preset. You can use most of the Select props to configure the display if you don't want to override the default slot such as color, variant, size, placeholder, icon, disabled, etc.

There is already an issue for the VerticalNavigation: #462


Create a rabbitMQ queue and host it?

Hosted on CloudAMQP.
Can be accessed using these credentials:


[improve][ml] Optimization method getNumberOfEntries

Motivation
Currently, when readPosition== lastPosition‘next, getNumberOfEntries will be entered. However, this is meaningless and may affect performance.
For example, readPosition=3:0, lastPosition=3:-1, will enter the getNumberOfEntries method

Modifications
When readPosition is greater than lastPosition, return 0 directly.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

close & reopen to trigger CI

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

Project coverage is 72.63%. Comparing base (bbc6224) to head (815e3b5).
Report is 794 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23576      +/-   ##
============================================
- Coverage     73.57%   72.63%   -0.94%
- Complexity    32624    36111    +3487
============================================
  Files          1877     1934      +57
  Lines        139502   154343   +14841
  Branches      15299    17768    +2469
============================================
+ Hits         102638   112114    +9476
- Misses        28908    33516    +4608
- Partials       7956     8713     +757

Flag
Coverage Δ

inttests
27.35% <100.00%> (+2.76%)
:arrow_up:

systests
?

unittests
72.40% <100.00%> (-0.45%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

...che/bookkeeper/mledger/impl/ManagedCursorImpl.java
77.13% <100.00%> (-2.17%)
:arrow_down:

... and 720 files with indirect coverage changes

@hanmz Please rebase master branch.


Apollo Auto adapter_manager.h not found in Apollo 3.0

I am trying to create IR for each files in Apollo Auto 3.0 using clang++
For files modules/tools/planning/planning_stats.cc, I get an error 'modules/common/adapters/adapter_manager.h' file not found
And I look at the directory, and there is no adapter_manager.h
Can anyone please tell me why this file is missing from apollo auto 3.0?
I also have the same error with terminal.cc file too

We do have it:
https://github.com/ApolloAuto/apollo/blob/r3.0.0/modules/common/adapters/adapter_manager.h
Please try cleaning your repo and bazel cache and then rebuild.

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Issue with Noise or Distortion Sound During Audio Recording Using PR #85

Hello @hiroshihorie and team,
I have been exploring the audio recording feature introduced in Pull Request #85. I'm utilizing the Room.audioProcessingModule.capturePostProcessingDelegate for this purpose, however, I am encountering an issue where the recorded audio has a noise or distortion sound.
Here is a link to the AudioRecorderHandler implementation I am using:
AudioRecorderHandler.zip
I have checked the sample rates, bit depths, buffer sizes, and codec settings to ensure they are configured correctly, but the issue persists. I suspect it might be related to the audio processing settings or possibly a mismatch in configuration somewhere.
Could you please provide some insight or assistance on how I might resolve this? I would really appreciate any help or guidance you can provide.
Thank you for your time and consideration.
Best regards,
Sandeep

@hiroshihorie can you please give your input on this!

Hey did you figure it out ?

Yes, for 32-bit floating-point audio data, the range is typically from -1.0 to 1.0. However, the audio frame sample range is not within this range, so it needs to be downscaled by a factor of 32768.
channelStride: 1
frameIndex: 0 sampleValue: -202.04233
frameIndex: 1 sampleValue: -165.478
frameIndex: 2 sampleValue: -436.00348
frameIndex: 3 sampleValue: -783.46893
frameIndex: 4 sampleValue: -753.1802
frameIndex: 5 sampleValue: -294.75992
frameIndex: 6 sampleValue: 152.50293
frameIndex: 7 sampleValue: 213.2728
frameIndex: 8 sampleValue: -2.9370422
frameIndex: 9 sampleValue: -132.95068
frameIndex: 10 sampleValue: -59.72957
frameIndex: 11 sampleValue: 9.903137
frameIndex: 12 sampleValue: -86.191154
...

private func convertToPCMBuffer(_ audioBuffer: RTCAudioBuffer) -> AVAudioPCMBuffer? {
    // ... (rest of code)

    for channelIndex in 0..<numChannels {
        let audioBufferPointer = audioBuffer.rawBuffer(forChannel: channelIndex)

        guard let targetBufferPointer = pcmBuffer.floatChannelData?[channelIndex] else {
            print("Failed to get target buffer for channel \(channelIndex)")
            return nil
        }
        // Now normalize and copy the data
        var targetChannelPointer = targetBufferPointer
        let normalizationFactor: Float = 32768.0  // This value is used to normalize the audio sample values
        for frameIndex in 0..<numFrames {
            let sampleValue = audioBufferPointer[frameIndex]
            let normalizedSampleValue = sampleValue / normalizationFactor  // Normalizing the sample value
            targetChannelPointer[frameIndex * channelStride] = normalizedSampleValue
        }
    }

    // ... (rest of code)
}


solve unique emoji IDs spanning different versions of unicode emoji versions

A library for handling unique emoji IDs:
https://github.com/eklem/unicode-emojis-unique-id-json

Split text file on \n and process each line with some regex
https://stackoverflow.com/questions/23331546/how-to-use-javascript-to-read-local-text-file-and-read-line-by-line

if no match on ^#.+, then process line, i.e.:
U+1F62E U+200D U+1F4A8 ; 13.1 # 😮💨 face exhaling

For the other lines:

Get everything before ; and split and add them to an array
/.+(?=;)/
Number with 1 decimal between ;  and  #
\(?<=; )\d+.\d\
Emoji
\(?<=#\s).\
Text after emoji
\(?<=#\s.+\s)[\w\-\’ \s:]+\

Use latest Node.js to get as many emojis extracted as possible.

For the first line, grab version number:
/(?<=v)\d+.\d/

Can now use the unicode-emojis-unique-id-json library.  Need to change key id to plaintext and create the codebook from that.


Issue with headers in the examples

Hello,
When I try to run the tests in either of the examples, I get the following error
[error]  found   : String("testvalue")
[error]  required: io.gatling.commons.validation.Validation[Map[String,String]]
[error]         .headers(Map("testheader" -> "testvalue"))
Can you please assist?
Thanks,
Jairo

I was able to fix by updating the line
.headers(Map("testheader" -> "testvalue"))
to
.headers(Map("testheader" -> "testvalue").expressionSuccess)
But should this be necessary?

Hello, Jairo.
Yes i's necessary append .expressionSuccess
It is strange that implicit conversion does not work in .headers(Map("testheader" -> "testvalue")).
I mean this ValidationImplicits.scala#L64.
I thought, it should have added .expressionSuccess to any type and also to Map[String,String].

You may replace .headers(Map("testheader" -> "testvalue")) with .property("testheader", "testvalue"), if you need to pass one header

Thank you.


Watch paths only work for directories, but not for exact match on single files or file wildcards

$ git diff --name-only HEAD~1
configs/dev.yaml

steps:
    plugins:
      - monorepo-diff#v1.2.0:
          watch:
            - path: configs/dev.yaml
              config:
                command: echo "dev config changed - exact match"

            - path: configs/dev.*
              config:
                command: echo "dev config changed - wildcard match"

            - path: configs/
              config:
                command: echo "a config changed - whole directory match"

first two matches are ignored, only 3rd one triggers.

NVM. it failed because I was using multiline command - error message suggested there was no match :/


Paper submission problem

Hello, I very much agree with your work. I want to share your paper in our class, and I have prepared the PPT. However, since the teacher hopes that the paper to be shared has already been accepted, I have only found the arxiv collection so far. I would like to ask what is the current acceptance status of your paper? Could you please tell me the name and level of the journal or conference so that I can fill out the teacher's collection form?
I apologize for being presumptuous, but I really like your work！

If you are not comfortable making it public, you can also send me to the following email address<EMAIL_ADDRESS>


Initial implementation of the kfserving spec

Implement KFService controller (pkg/controller/kfservice/kfservice_controller.go)
Implement Knative service reconciliation(pkg/reconciler/knservice/knservice_reconciler.go)
Resource translation from KFService to KnativeService(pkg/reconciler/knservice/resources/knative_service.go)
Resource translation from TensorflowSpec to Tensorflow container (pkg/containers/tensorflow/tensorflow_container.go)
Add tests for above files
Added @TODOs for next step

This change is

Should we add prototbuf comments in the kfservice_types for completeness alongside the json ones? (could be a separate PR)


chore: change create check endpoint for new one

Resolves #71
Affected Components

[ ] New Features
[ ] Bug Fixing
[X] Types
[X] Tests
[ ] Docs
[ ] Other

New CURD types with more verbose names
Flag old methods as deprecated
Change CreateCheck function to use new endpoints (checks/api and checks/browser)
Add tests for new methods

Resolves https://github.com/checkly/checkly-go-sdk/pull/72


gh: adding a freeze window workflow

Maintainers can enable/disable this workflow via web interface. When enabled all PRs will have at least one test failing.

Hey Beraldo, I just want to fully understand the proposal so if it works out we can add it to the release process docs and there are a couple of questions/concerns I have:

For getting the release related PRs (version bumps etc) through is the intention that a maintainer would disable, merge that PR and then re-enable this workflow?
Is this supposed to be enforcement, or just a reminder? If we don't make this check required then it won't block PRs merging, but if we do make it required then it blocks merging when it's enabled and fails, but also blocks when it doesn't run. A reminder if better than nothing, but I just wanted to check what you had envisaged.

The other approach I was thinking about is whether we could have a freeze label, and then a permanent job that checks if freeze, or WIP is labelled and that could be required (kata has this step up). I think using the gh api we could probably easily write workflows to label & unlabel all current PRs with this, but I don't have a good idea for how we would set it up so a new PR created during the freeze period would get labelled. I thought I would share in case others have thoughts though.

We could use this same PR as example. I guess one test is failing but I'm assuming you could merge it anyway, right?

This is the point I was trying to explain - I don't think it will block merging unless it's a required job. At the moment we can merge this due to not having 2 reviews, not the failing test:

We could use this same PR as example. I guess one test is failing but I'm assuming you could merge it anyway, right?

This is the point I was trying to explain - I don't think it will block merging unless it's a required job. At the moment we can merge this due to not having 2 reviews, not the failing test:

Humm I see now. Yes GH removed the required checks feature, and looks like we can do that via project/repository settings using rulesets, but I don't have permissions to see that here.

Humm I see now. Yes GH removed the required checks feature, and looks like we can do that via project/repository settings using rulesets, but I don't have permissions to see that here.

Yes - I can make it a required check, but that means it will have to run even when we aren't in freeze and block otherwise (we have similar issues in kata-containers around this)


Improve performance of InsertStatement.createContext in batches

Currently createContext is one of the more recognizable bottlenecks when doing large batches/insertions with drift. In talking with @simolus3 it sounds like there are some improvements that could be made here to speed this up.
Describe the solution you'd like
There's no exact target, but here's a CPU Flame chart to highlight the impact of createContext's current performance:

To add context, the optimization I had in mind was to hash companions being inserted in batches by which columns they have present values.
At the moment, when we insert a large amount of rows that have a similar "shape" (in the sense that we generate the same SQL for them, only the bound variables are different), we build the SQL again for every single row.
For a five-column table CREATE TABLE tbl (a,b,c,d,e) for example, what we could do is represent the present values on a companion in a bitfield. A companion with all values set would be represented with 31, which can serve as a key for a cache mapping companions to the SQL already built for them. Now, inserting say 10k of these companions only requires building their bitfield and extracting Variable instances 9999 out of the 10000 times. With the current implementation, every single operation requires building SQL, which is more expensive (lots of string buffer operations).

Moving some parts of the operation to Rust could help us with this.
Native Assets is a new experimental dart feature which would allow us to do this without depending on flutter.
Isar depends on flutter and uses rust, we could move some limited parts if @simolus3 is down to clown

I don't see what performance benefits that would unlock to be honest, since we'd still have a Dart API responsible for expressing queries. The parts of Isar that are written in Rust correspond to sqlite3, which is already a native library. The way it seems, the key is doing less duplicate work when building queries. I don't think rewriting parts alone is going to help substantially, Dart is not intrinsically slow.


Make the plugin compatible with pytest-xdist

The pytest-xdist plugin spawns worker processes that run the tests while
the main process reports the results. As every process has its own copy
of all the plugins, this means that when the main process reports the
final summary, it doesn't have access to the results stored by every
individual test so the plugin is unable to print the summary.
To make ourselves compatible with pytest-xdist, we make the main process
and the workers use a common path to store the metadata and result files
(this is communicated from the main process to the workers by an
environment variable). Then, at report time, if the main process doesn't
have any result data, it will fetch it from this directory instead.
Closes: #2

pytest-xdist allows distributing tests across multiple hosts, as well. Should we detect when it was run in that mode and warn or fail?

pytest-xdist allows distributing tests across multiple hosts, as well. Should we detect when it was run in that mode and warn or fail?

Hummm. I would need to investigate. Do you mind if we do this in a another PR?

I have addressed all comments, please take another look


Hello. Just managing a little website for my karting club and I use Woocommerce for registration to event. I have a bilingual website (English/French) so I use Hyyan WooCommerce Polylang Integration to translate my shop and was working great until I updated WooCommerce to 4.1 (when from 3.XXX to 4.1). Now my problem is the French version of the shop doesn't work properly. Do you know if there is any update for the Hyyan extension planned? Thanks.

Can you reproduce this issue on default Wordpress theme (eg Storefront)?
Can you reproduce this issue when all other plugins are disabled except WooCommerce, Polylang and Hyyan WooCommerce Polylang Integration?
What product versions and settings are you using when this issue occurs?

PHP:
WordPress:
WooCommerce:
Polylang: [state if using Polylang PRO]
Hyyan WooCommerce Polylang Integration:
Browser:

Steps to Reproduce

What I Expected
What Happened Instead
WordPress Environment

```
Copy and paste the system status report from **WooCommerce > System Status** in WordPress admin here.
```

There are quite a few checkins since the last release so yes there will be an update.
However please note:

there are no breaking issues reported with the upgrade from 3.x to 4.x at this time
4.x is not fully supported in terms of the fact that language handling for WooCommerce blocks and other new admin features have not been tested or implemented
if you have a specific issue you need to report the full details of versions used and specific errors detected


DRY: cleanup __json_replacer

Context: https://stackoverflow.com/a/76021642/4909087

Thank you :) you are the best


Add Extension Manager dialog

We need a dialog similar to that in NUnit V2. It should display all extension points and / or extensions, provide info about individual extensions and allow selective enabling / disabling of extensions.

Is this the tools > addins screen you are talking about?

Yes... for NUnit 3, you can see how the console lists extension points and extensions here:
https://github.com/nunit/nunit-console/blob/master/src/NUnitConsole/nunit3-console/ConsoleRunner.cs#L273
This requires the IExtensionService interface, which was added to the nunit.engine.api assembly in (I think) version 3.4. The console option to list extensions was added in 3.4.1.
Other than visual stuff, the next big steps for the GUI will require updating to the latest engine.

ok. Well if you don't mind I'd like to work on this. I'll make 1 commit of updating to engine 3.4.1 then next commit of adding in this. For now i can just copy the view from v2 gui.

Sure... Do the engine update as a separate PR as well in case there are side effects.

@CharliePoole  I'm finally wrapping this up after many days of being swamped by work. Anywho. I'm not a fan of how the form looks.  On each of the extension points I put a tool tip that activates if you hover over the group box header (figure 2). The scroll bars show if there are lots of extensions. Can I get some feedback?

It does look somewhat busy. :-) I think we need to space it better and get rid of some of the visible border lines. Here are some specific comments:

Each extension point should be identified by it's Path, which is it's unique identifier. It's possible to have two extension points that use the same interface Type.

Initially, we should show all extension points, even if there are no extensions loaded. That's what the --list-extensions option of nunit3-console does. Later, we may want to add a checkbox or some such thing to only show extension points that have extensions.

Initially this is display only. Eventually, we may want an on/off switch to enable extensions.

Let's put some vertical space between extension points

Extensions under each extension point should be indented. I don't think they need a border around them. Properties under each extension can be in a border as you have it, but should be indented. We can play around with vertical space here.

We'll probably want a tooltip for each extension as well. Using ProjectLoader as an example, it could seem redundant, but if you take an event listener as an example, each extension may do something entirely unrelated to what the others do.

Can we get this into a branch that's easy to try out? I'm happy to merge it as a work in progress, since we are in a pre-release stage.

sure I can push my current changes into my remote branch and do a pull request as it sits right now. I can make a second commit with the changes you suggest. I'll work on that right now

@CharliePoole Maybe its been too long since I last looked at this, but rereading point one and comparing to my output pane I don't see much difference (in terms of content) between nunit-gui and nunit-console

Maybe I'm just getting too hung up in the details, but is the content of what I am showing correct?

Content-wise, I think you're right. I was misreading it.
But what's FileExtension2? Is it just an example?
Bottom line, we need to work on the appearance. Would you prefer to merge what you have first?

No I'm going to fix appearance either today or tomorrow and put a new commit up. I'll than post a final picture as part of the pull request

That's great. I'm sure we'll get comments on it as people use it.

whats the chances of us making a more stylish wpf version?

It's an attractive idea for sure. But this is pretty close to being usable right now, so I'm inclined to think of that as a 2.0 thing.

Sure, and I can appreciate that. I know it is a little unconventional but maybe if I started working on (after I finish this pull request) slowly pulling the Presenters and IViews into a core/shared library and remove all the winforms code from that than it could be conceivable to make a WPF app that implements the IView stuff using the MVP pattern, or make a strange hybrid of MVVM that would be conventional from the WPF side of things but use the logic from the winforms app. (I actually think I heard of someone at Microsoft trying to talk everyone into a MVVMPV model… not sure if that fell on its face or not, but if I recall it is actually something similar to what I’m suggesting now).

I'm OK with that idea, especially as you have an understanding of how the particular MVx pattern is tailored to a particular GUI platform.
BTW, one reason I stuck with Windows Forms, is that the GUI has always run on Linux. I'm not sure that many people use it there though.


Fixed #26089 -- Removed custom user models from public API.

I'm not really comfortable moving the deferred SQL execution after the migration phase.
Firstly, because this is a a backwards incompatibility as the indexes created by statements like CREATE INDEX and CREATE UNIQUE INDEX are not available during the migration period.
Secondly, I think that in a long run we probably want to avoid syncdb at all and having migrate wrapped in "pre_syncdb" and "post_syncdb" doesn't seems to make sense to me. The implementation we chose with 1.9 (--run-syncdb) already drops the syncdb usage unless you do read the help or documentation.

Thanks for chiming in Markus,

Firstly, because this is a a backwards incompatibility as the indexes created by statements like CREATE INDEX and CREATE UNIQUE INDEX are not available during the migration period.

You're right, I wish there was a way of only postponing only the foreign key creation. If it was the case would you be open to such a change?

Secondly, I think that in a long run we probably want to avoid syncdb at all...

Just curious here, if we were to completely remove syncdb what would be our options, creating migrations for all of our test models?

... and having migrate wrapped in "pre_syncdb" and "post_syncdb" doesn't seems to make sense to me.

In the case where we have to support both syncdb and migrate I feel like this is an acceptable compromise.

The implementation we chose with 1.9 (--run-syncdb) already drops the syncdb usage unless you do read the help or documentation.

Agreed but if migrations cannot be used for a specific application (e.g. MIGRATION_MODULES['app'] = None) the documented syncdb usage will simply break and force the user to completely disable the migration framework even if only a single application is problematic.
While I agree fixing this limitation of --run-syncdb might be another incentive to postpone the complete adoption of migrations in the community I find it odd that we have to internally rely on a feature we publicly suggest not using.

I'll close this PR and open a new one with only the required changes for #26089.
Markus was working on a more promising approach to solve the MIGRATION_MODULES workaround anyway.


New clog section for examples

New special section for examples changelog. Also I've update current.

Good idea :+1:


Add @qedawkins to a few CODEOWNERS directories

I want to track changes to these directories so adding myself as codeowner for them. Some of them are starting to receive more contributions and/or are newer so this updates the default reviewer pool for them.

@MaheshRavishankar @antiagainst I also added you both in a couple places, let me know if this looks good to you.

FYI (while on the topic), we also have a MAINTAINERS.md file now: https://github.com/iree-org/iree/blob/main/MAINTAINERS.md. CODEOWNERS is still the file that GitHub uses to automatically request reviews, but that file also exists for

This file attempts to outline long term responsibility for questions of evolution, health, and design.

Yeah I saw that file, but it looked to be coarser than CODEOWNERS for what I wanted for now. Maybe at some point :)


travis: update npm api key [was: 2.7.2-betas not on npm]

Needs fixing.

==> Merged.

Just to confirm: TravisCI deployment to npm is fixed (cf. 2.7.2-rc).


Six

This is the first "complete" d3v6 build, with no duplications. (Previously I had a duplicate of each of the dependencies of the already-released modules d3-scale + d3-force which were not taken in the 2.0.0-rc.* range.)
The file size for d3v6 is 5.83% larger than d3v5 (by some kind of coincidence it's the same ratio both for the compressed and the uncompressed versions).
See  https://observablehq.com/d/b5b8bc70373e615c for the build.

Very exciting!

🧨   Incident report:
I published<EMAIL_ADDRESS>(with the correct next tag) to npm. So far so good, but the postpublish script  also pushed it as d3.v5.js and d3.v5.min.js to d3js.org — and broke the internets.
I struggled a bit to reinstate the files manually (just moving the commit to a different branch and force-pushing master didn't want to update the GH pages), so it was broken between 13:40 UTC and 13:51 UTC.
I hope https://github.com/d3/d3/pull/3438/commits/47a043dbd83190b242fa98737c04234ea151ed23 avoids that trap next time.

https://github.com/d3/d3/commit/47a043dbd83190b242fa98737c04234ea151ed23 has worked (https://github.com/d3/d3.github.com/commit/41309339a65dc58895c62fb616ca5d27e8751c8b)!


Broken link to  CafeBot LUIS sample

Github link to  CafeBot LUIS sample shows 404

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

ID: 3449824e-105d-ffe2-000c-0a20e42a45f4
Version Independent ID: 3fa03eda-b09c-e6d9-c4f3-a186184232ee
Content: Extract typed LUIS results - Bot Service
Content Source: articles/v4sdk/bot-builder-howto-v4-luisgen.md
Service: bot-service
Product: bot-framework
GitHub Login: @DeniseMak
Microsoft Alias: v-demak

C#

The topic you are looking at was removed from the TOC because of several issues. See LUISGen readme (https://github.com/Microsoft/botbuilder-tools/blob/master/packages/LUISGen/readme.md) on GitHub for information.


Fix Labels and Points properties set

Description

Fix properties of Labels and Points layers. Properly handling dict and Dataframe. I found it working on #2622
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

Thanks for the fix!! Code looks good here, but can you add tests too? It wasn't quite clear to me what was wrong before, so the test would help

@kevinyamauchi I have a problem understanding the expected behavior of _property_choices of the Points class. If I good search it is only accessed inside the constructor. But still, in the constructor, its behavior is not constant.
here
https://github.com/napari/napari/blob/3e35226f8f467fe957440635bfffb235e6d1d03e/napari/layers/points/points.py#L306-L308
pick unique elements
but here
https://github.com/napari/napari/blob/3e35226f8f467fe957440635bfffb235e6d1d03e/napari/layers/points/points.py#L310-L312
pick all elements.
If I good understand there is the silent assumption that the user will provide a unique list of elements?
The docstring for properties does not cover the case of empty data.
Basing o the name it could be used for editing properties from GUI.
But why this property is not updated in setter?
Or how it should be updated in setter: overwritten, extended by missing values? Or something else?
@sofroniewn suggests that you are the best person to ask.

@sofroniewn @jni @kevinyamauchi please review.
As it was discussed at today meeting this PR should be the solution for the closest release and after this PR is finished the next PR with a general property engine will be stared

Code changes look good, but can you explain in a little more detail what the above means. It's not clear to me what behavior was broken before this that is now fixed. Was there a regression from 0.4.7 or has this always been broken? It's a lot of changes close to the release, so I want to make sure we're all clear on what the expected changes are.

It looks like it always was broken. My final target is #2658 and I creat this PR because I found that setting the properties was broken.

It looks like it always was broken. My final target is #2658 and I creat this PR because I found that setting the properties was broken.

Ok, nice, let's hold this till 0.4.9 then and give a few more people a chance to review

Thanks for addressing review comments @Czaki! @sofroniewn this is ready to be merged as far as I'm concerned. 🎉


Handle IDE project files in the repo

There's a number of .idea/*.xml files auto-generated by Intellij. Decide if you want to keep them in the repository or add to ignore list. (more)

I'll remove them

Sorry, I can't find any idea or xml files in the repo, can you show me what you mean?

I meant unversioned XML files in the .idea directory. If not present there, they are generated by Intellij. Adding the .idea directory to thie .gitignore (as you did) is fine for me.


How to Disable Swagger UI and openapi.json in Cog

Hi
I hope this message finds you well. I am currently exploring Cog and would like to understand how to disable the Swagger UI and openapi.json endpoints for security purposes.
Could you please provide guidance or recommended steps to achieve this configuration? Any help or documentation you could point me to would be highly appreciated.
Thank you for your assistance and for your continued work on this project!

Yeah this would need to done on the fastapi side.. unless theres a way in cog to provide overrides?
https://fastapi.tiangolo.com/tutorial/metadata/#openapi-url @kosukekurimoto


Factorial of large numbers

I want to add how to find factorial of large numbers.

don't create a new issue, simply add the project and make a PR.


Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Problem fixed! The variables for all the SonicHUD's were commented, so they technically didn't exist.


Added default value to avoid UNBOUND VARIABLE error when using -p or -u

What is the difference between these two? I thought you had to add the column, but your code works without it
"${var:-x}" # column

and
"${var-x}" # no column

Did you mean to commit test.sh?

Did you mean to commit test.sh?

No. removed.


Favorite function for images

Is it possible to have a 'Favorite' function, e.g. with stars/hearts or so? It would make rating of pictures possible.

Favorite in albums you created? or in device gallery?

In created albums would be great.

Will this favorite sync with Nextcloud file favorite?
What the primary use for favorite? Rating? Screening for great photos?
Who can set favorite? The owner or those who have the shared?

My 2 cents:

Who can set favorite? The owner or those who have the shared?

I think both use cases are valid.  I'd like to track my own favorites in an Album so I can quickly find them.  And if I've shared an album, I'd like to see other's favorites.
My guess is implementing "my own favorites" would be easier as it can use the built-in nextcloud favorite.

What the primary use for favorite? Rating? Screening for great photos?
My use case is to mark and then quickly retrieve top (favorite) photos.

Will this favorite sync with Nextcloud file favorite?
For "my own favorites" I think syncing with Nextcloud favorite makes sense, and is hopefully easier to implement.

For sharing favorites with others, does the Nextcloud API support querying whether user 1234 has favorited a specific file?

And quick thoughts on UX:

Will this favorite sync with Nextcloud file favorite?
Hm would be a benefit for option with personal-favorite

What the primary use for favorite? Rating? Screening for great photos?
Screening for great photos, I would say.

Who can set favorite? The owner or those who have the shared?
Both would be good. Maybe configurable in album?


Add support for inline activation

This PR adds support for running the produce and consume blocks via the keyboard.
It adds another command vscode-kafka.inline.activate that finds the block the cursor is currently on, and calls it
closes #220

Could you add a documentation after https://github.com/jlandersen/vscode-kafka/blob/master/docs/Producing.md#randomized-content with your new settings. I think a demo could be good.

Could you add a documentation after https://github.com/jlandersen/vscode-kafka/blob/master/docs/Producing.md#randomized-content with your new settings. I think a demo could be good.

It works for both CONSUMER and PRODUCER, is there maybe a better place for it?

Added the documentation in KafkaFile.md.

Not sure i fully understood what you meant, let me know if i didn't do what you expected. thanks :)

Thanks so much @od-cyera for your great contribution!


404 errors for Amazon Linux 2 endpoints

Just ran into this on a new AL2 machine. After successful rig installation:
(23-10-31 16:12:26) <1> [~]
% rig add release
[INFO] Running `sudo` for adding new R versions. This might need your password.
[ERROR] Failed to resolve R version: Cannot download JSON: HTTP status client error (404 Not Found) for url (https://api.r-hub.io/rversions/resolve/latest/linux-amzn-2/x86_64)

(23-10-31 16:14:18) <1> [~]
% rig add next
[INFO] Running `sudo` for adding new R versions. This might need your password.
[ERROR] Failed to resolve R version: Cannot download JSON: HTTP status client error (404 Not Found) for url (https://api.r-hub.io/rversions/resolve/next/linux-amzn-2/x86_64)

(23-10-31 16:19:17) <1> [~]
% rig available
[ERROR] Cannot download JSON: HTTP status client error (404 Not Found) for url (https://api.r-hub.io/rversions/available/linux-amzn-2/x86_64)

You can set the RIG_PLATFORM environment variable to force a certain Linux distribution. In your case you'd set it to centos-7:
root@068d42abd40f:~# rig system detect-platform
Detected platform:
Architecture: aarch64
OS:           linux
Distribution: ubuntu
Version:      22.04
PPM support:  false

root@068d42abd40f:~# export RIG_PLATFORM=centos-7
root@068d42abd40f:~# rig system detect-platform
Detected platform:
Architecture: aarch64
OS:           linux
Distribution: centos
Version:      7
PPM support:  false

Thanks for this suggestion. It looks like it should be working, since rig available passes through correctly. But unfortunately the actual add step still appears to referencing the wrong linux-amzn-2 endpoint.
[~] % export RIG_PLATFORM=centos-7

[~] % rig system detect-platform
Detected platform:
Architecture: x86_64
OS:           linux
Distribution: centos
Version:      7
PPM support:  true
PPM URL:      https://packagemanager.posit.co/cran/__linux__/centos7/latest

[~]  % rig available
name   version  release date  type
------------------------------------------
3.0.3  3.0.3    2014-03-06    release
3.1.3  3.1.3    2015-03-09    release
3.2.5  3.2.5    2016-04-14    release
3.3.3  3.3.3    2017-03-06    release
3.4.4  3.4.4    2018-03-15    release
3.5.3  3.5.3    2019-03-11    release
3.6.3  3.6.3    2020-02-29    release
4.0.5  4.0.5    2021-03-31    release
4.1.3  4.1.3    2022-03-10    release
4.2.3  4.2.3    2023-03-15    release
4.3.2  4.3.2    2023-10-31    release
next   4.3.2                  patched
devel  4.4.0                  devel

[~] % rig add release
[INFO] Running `sudo` for adding new R versions. This might need your password.
[ERROR] Failed to resolve R version: Cannot download JSON: HTTP status client error (404 Not Found) for url
(https://api.r-hub.io/rversions/resolve/release/linux-amzn-2/x86_64)

[~] % rig add 4.3.2
[INFO] Running `sudo` for adding new R versions. This might need your password.
[ERROR] Failed to resolve R version: Cannot download JSON: HTTP status client error (404 Not Found) for url
(https://api.r-hub.io/rversions/resolve/4.3.2/linux-amzn-2/x86_64)

Yes, that's what I meant, you need to run sudo manually to keep the env var, e.g.
export RIG_PLATFORM=centos-7
sudo -E rig add ...
because of this bug: https://github.com/r-lib/rig/issues/191

Ah, perfect that works. Thanks Gabor!
P.S. For any other AL2 users reading this, just make sure that you first enable extra packages and optional repositories before implementing Gabor's workaround.
sudo yum install -y https://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm
sudo yum install -y yum-utils libXt-devel
sudo yum-config-manager --enable "rhel-*-optional-rpms


java.util.Collections.sort tests failing in Scala community build

the failures can be seen in the 2.11.x-jdk8 build (https://scala-ci.typesafe.com/job/scala-2.11.x-jdk8-integrate-community-build/72/consoleFull) and the the 2.12.x build (https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/187/consoleFull)
I have not seen it fail in the 2.11.x build on JDK 6. That might be because the change that caused the regression is very recent and hasn't run yet in that build, or there might be a JDK 6 vs 8 issue here.
see logs for full stack traces, but the first few lines of one of the failures look like:
[scala-js] [info] java.util.Collections.sort
[scala-js] [error]  x should implement sort[T<:Comparable[T]](List[T]) and behave correctly with java.util.ArrayList
[scala-js] org.scalajs.jasminetest.JasmineTestException: scala.scalajs.runtime.UndefinedBehaviorError: An undefined behavior was detected: 0 is not an instance of java.lang.Integer
[scala-js] 	at <jscode>.(file:/home/jenkins/workspace/scala-2.12.x-integrate-community-build/target-0.9.1/project-builds/scala-js-6327675135cdd35cb027d800490dac65bc9af680/library/src/main/scala/scala/scalajs/runtime/StackTrace.scala:39)

#1984 #BigBrotherIsWatchingYou

Do we have a range of commits of Scala/Scala.js in which the failure was introduced?
I cannot reproduce this issue with Scala.js' commit 3b6d6eef9f740010140210c39cbfca08546b12c2 (which the 2.11.x failed build mentions) on a JDK 8 and Scala 2.11.7 nor 2.12.0-M3.

I suspect https://github.com/scala/scala/commit/37eacec819e38cc29357a31ee99b592f31e0702f because the failure happens at https://github.com/scala-js/scala-js/blob/3b6d6eef9f740010140210c39cbfca08546b12c2/test-suite/src/test/scala/org/scalajs/testsuite/javalib/util/CollectionsTest.scala#L1320, where there is an implicit conversion int2Integer(value % 8), although I don't know why the code would blow up (I review that PR myself).

OK I minimized it. It was not related, but that change brought up the bug to the surface. The minimization is:

object HelloWorld extends js.JSApp {
  var value: Int = -8

  def main() {
    println((value % 8).asInstanceOf[java.lang.Integer])
  }
}

Apparently value % 8 returns a negative 0 there (as a result of a bad encoding of the integer % in JavaScript), which is indeed not a java.lang.Integer (only positive 0 is).


synchronize modules with advanced filters on window open.

Contains
Fixes problem with modules screen showing full list of modules after window has been opened.
Fixes problem with modules list showing full list of downloaded modules immediately after window is opened.
Also fixes #3356
How to test
Before:
Opening the modules panel shows the local modules, then all modules once the remote modules download.  Neither view considers advanced filter settings.
After:
Open modules screen.  Note that listed modules matches advanced filter settings before and after the remote modules download.

Hooray Jenkins reported success with all tests good!

Can confirm that this works. :)


Support interrupting non-python kernels

For #8107
Basically non-python kernles are not interrupted today as we don't send the sigint signal.

Closing this PR in favor of https://github.com/microsoft/vscode-jupyter/compare/interruptNonPythonKernels?expand=1
No chages, for some reason this PR would show thirdparty notices file had changes (even after rebasing with main).


Can't use modal

I want to use modal, but I can't open modal with data-toggle="modal"，data-target , href ,all of it can't use.And have no error info.
So ,I need help ,thanx

This project doesn't use bootstrap javascript, you have to import it separately in your project.


 I'm thinking bech32m, burn1…, with an arbitrary binary payload. The output script is then OP_RETURN PAYLOAD.

Is the intention to actually burn a sat, or would the inscription simply be "burned" at the ord protocol level?

Intention is to burn the sat, since that's simple and sending to an OP_RETURN is already extremely well understood. We could then recognize this in block explorers, but I don't think we need to do anything at the ord protocol level. Like the block explorer, when it sees a sat go to an OP_RETURNS, can display 🔥 BURNED 🔥 to represent being burned, somewhere in the /inscriptions page.

Perhaps I'm missing something, but that seems extreme to me if the goal is just to mark an inscription as burned, when it could be done within the protocol without burning any sats. I'm assuming you'd at least want to handle cases where the sat is rare, epic, or legendary differently (or at a minimum provide a warning to the user), which would have added complexity. How is burning a sat simpler than creating a concept for burning in the protocol envelope?

How is burning a sat simpler than creating a concept for burning in the protocol envelope?

A very reasonable question!

Burning a sat is already a well-defined and well-understood concept in Bitcoin.

A burn1… address relies on existing functionality (sending to an address) and it will be easy for users to understand.
New protocol additions are require research, implementation, review, and ongoing maintenance, and we must figure out how they fit into the protocol.
We can allow burning a sat and inscription via sending to an OP_RETURN first, and then strategically consider if any use-cases that aren't served are worth the additional protocol level complexity.
The protocol already has to deal with burned sats, since you can already send sats to an op return. We'd just be enabling users to do what's already possible.
If you can burn an inscription via an ord-protocol envelope, it complicates the SPV security model, since they then need to look at all historical witnesses, not just first witness from inscribe, and TX-path to current TX.
An ord-protocol envelope adds size to the transaction.

I am hoping that the only ord-protocol witness envelope we need will be on inscribe, and plan to push as much as possible onto already existing functionality on the base layer.

If burn addresses have plain-text payloads, we should have that text be visible to the user. Sending to a burn address is like making a statement, and we wouldn't want users to be tricked into statements they don't agree with. See sketch in #1657 for why this might be important.

@casey was checking #2597 and then jumped to this issue.
If sending only to OP_RETURN, would you add that functionality to the transaction_builder.rs? It might need a refactoring because it currently only accepts Address outputs.

@raphjaph what's your opinion on this? Would love to start working on this feature.

I think a burn address standard is only useful if it's going to be used by wallets other than ord. If it's only supported by ord, then we can just add a --burn option to the various commands.

Implemented in https://github.com/ordinals/ord/pull/2766

New PR with #3437


Configure @rollup-plugin-url to import images directly into a class

Feature
Using images without the need for them to be in the static directory.
Problem
We're developing a shared UI component library and each component could potentially have its own images. Currently the Lightning CLI insists on all images are stored in the static directory of the application in order for them to bundled up using rollup. Images inside the component library won't be bundled as the CLI doesn't know they exist.
Solution
Use @rollup/plugin-url to find all imported images and copy them into the build/static directory. This will make them available at runtime.
Usage
import { Lightning } from 'wpe-lightning-sdk'

import logo from './logo.png'

export default class App extends Lightning.Component {
  static _template() {
    return {
      Logo: {
        src: logo,
      },
    }
  }
}

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Lewis Peel seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @michielvandergeest - is there anything else I need to do to get this merged?

Hi @lewispeel, sorry for the delay in getting back to this. The plan was actually to review this PR for last weeks release, but some other stuff took priority. I expect we can include it in the next one, though (hopefully end of this week)!
I think the PR looks good, there's just one issue with the destDir being hardcoded to build/static, whereas that can be configured that via Environment variables (https://rdkcentral.github.io/Lightning-CLI/#/environmentvariables?id=lng_build_folder).
If we can fix that we should be good to go, I guess.

Thanks @michielvandergeest - good point! I've updated the configs to make use of the environment variable, which defaults to build if nothing is found.

@michielvandergeest I've swapped out @rollup/plugin-url for @rollup/plugin-image as the former only worked with images imported from the main application. The latter works with any image, including from external dependencies. I'll update the original PR.


Can't read text in the Audit log & Processes Resource ID field

Version - Master 1/27
Steps:

In light theme, go to Processes or Audit Log page

Results:  In the Resource ID field, you can't read the text when it's dimmed

Expected: Should be able to read it.

resolved in https://github.com/rancher/ui/pull/455

Version - master UI 2/4
Verified fixed


Table Function Overloading

This PR adds support for table function overloading similar to how scalar and aggregate functions can be overloaded. This allows the same table function to exist with different parameters. This also implements the read_csv(VARCHAR) function as alias for the read_csv_auto function, and expands the range to be similar to the range function in Python, i.e. the below functions are all equivalent:
--- [0..10)
SELECT * FROM range(10);
SELECT * FROM range(0, 10);
SELECT * FROM range(0, 10, 1);

Great, thanks, can make good use of it.


On EC2, rancher-nodes security group should open 80, 443, and NodePort ports

Rancher versions:
rancher/server or rancher/rancher: v2.0.0-rc3
I think there's a problem with how we configure the default security group "rancher-nodes" on EC2. We don't open 80, 443, and 30000-32767. As a result we can create RKE clusters successfully but no workloads work.
We need to allow 80, 443, and 30000-32767 in rancher-nodes security group.

Tested with v2.0.0-rc4
port 80 and 443 are open in the default "rancher-nodes" SG that created when deploying EC2 Amazon cluster using a template that chooses to use  "rancher-nodes" SG .
Node port ranges are not opened up which is as expected.


chore: Optimize HashMap: unconstrained approach

…of concept
Description
Problem*
Resolves #4584
Summary*
This branch optimizes HashMap using an unconstrained approach suggested by @zac-williamson. This PR is here to show the differences of this approach, but the timings and benchmark information are on the PR with the bucketed approach here: https://github.com/noir-lang/noir/pull/4603
Additional Information
The current approach in this PR is not yet abstracted to any key or value type. When this is done it should be noted that this approach also requires Ord on the key type.
It's also unclear if the unconstrained approach would be as efficient for an API returning an optional value in get. I think in the None case we would need to iterate through the entire hashmap to assert that they key isn't present to prevent under constraining the result value.
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Closing this PR, its too much spam to open each branch as a separate PR


Crash with exit code 1

Hello, I am kind of new to this, but version 1.19.3 worked from the first try.
I am using forge version: 1.20.1-forge-47.0.3 and both controllable versions: controllable-forge-1.20.1-0.20.0 and controllable-forge-1.20.1-0.20.1 make my game crash with exit code 1.
Any ideas what I could be doing wrong or is it a known issue?
 I need it too and dont know what to dl

Can you give a link to the framework? I need it too and dont know what to dl

https://www.curseforge.com/minecraft/mc-mods/framework/files/4590795


Start validating the rest of state_data using Zod

Until now there has been no validation for the parts of state_data split off by pop_fields in ui_init.initialize_everything. Start adding that validation gradually, to improve our confidence in the correctness of our annotations, and so we don’t have to do it all at once when we convert ui_init.js to TypeScript.
We use Zod’s .transform to replace pop_fields as the mechanism for splitting state_data. This leaves state_data.state_data_schema as the single source of truth for which fields are sent where.

Merged, thanks @andersk!
I'd like to investigate the is_moderator code path a bit, to see if our API has issues we should address, but that doesn't need to block this refactor.
We'll want to watch the rollout of this closely, just because adding zod validation to an incorrect type can break the app.

Regarding is_moderator, I remember we discussed while adding the moderators feature that we would not pass the field for all the users and instead let clients add it using the role value. But we pass the field for the request value in the top level object.


折线图

你好，请问Figure5你能给分享一下代码吗

hello, have you solved your problem?  I have the same problem.

This repo discusses code about SeqNet only.


Misc questions thread

@lyzadanger I'm just going to post things here based on what I encounter during implementation.

Question:
Given a folder structure like this:
src/patterns
├── button
│   ├── base.hbs
│   ├── disabled.hbs
│   └── primary.hbs
└── typography
    ├── headings.hbs
    └── lists.hbs

Is there a way to use a collection.yaml file to control the ordering of these pattern collections? (e.g. typography precedes button)

Question:
Given a folder structure like this:
src/pages
├── demos
│   ├── example1.hbs
│   └── example2.hbs
└── index.hbs

Is there a way to have corresponding individual pages generated for each file? Currently, the output is structured as one file (example1.html) that contains the contents of both source .hbs files:
dist/demos
└── example1.html

Question 2:

Bah! That's what I was trying to suss out in numerous earlier issues. This request is in conflict with what was discussed in this thread in which it sounded pretty strongly like the desire was not to have individual pages created for patterns :).

Bah! That's what I was trying to suss out in numerous earlier issues. This request is in conflict with what was discussed in this thread in which it sounded pretty strongly like the desire was not to have individual pages created for patterns :).

I will try a few more things and get back to this...I may just need an additional directory or two inside of pages.

Oh, wait, pages? I'm so sorry. There should be a 1:1 relationship between input pages and output. If you're not seeing this, something is up :). It's possible there is a bug with top-level pages (though there are fixtures...hrm). I'll totally take a look at this tomorrow!

Re: Question 2
To document this further...
I tried adding some nested page folders, and got the following:
Input:
src/pages
├── demos
│   ├── demo-example-1.hbs
│   ├── demo-example-2.hbs
│   └── index.md
├── docs
│   ├── doc-example-1.md
│   ├── doc-example-2.md
│   └── index.md
└── index.hbs

Output:
dist
├── demo-example-2.html
├── demos
│   └── demo-example-1.html
├── doc-example-2.html
├── docs
│   └── doc-example-1.html
└── index.html

Oddly, the nesting gets altered. Also worth noting is that pages with the same base name (for example, demos/example.hbs and docs/example.md) would collide at the top level. Making each page name unique fixed this.

Re: Question 2
This issue seems to exist for patterns as well, due to similar code in the walkCollections function:
https://github.com/cloudfour/drizzle-builder/blob/feature-rewrite/src/write/collections.js#L26-L29

Re: Question 2
Things are now working as expected, thanks to recent commits by @lyzadanger on #68.


composite field

Changes:

add CompositeField
rename getColumn to getColumns
rename check to clean and add name argument (inner field name)
slightly changed logic concerning get_column (None is allowed now)
fix typings

Note:
But this is the foundation for having multiple primary keys (later the foreignKeys inherit from the compositeBaseField)
Update:
I removed the renaming stuff. CompositeField works now on base of Fields. For ForeignKeys on models with multiple primary keys, there will be autogenerated, hidden Fields which hold a column and a ForeignField which is the composite field

cleaned up


bug: some requests are not applied or applied out of order without error

When running test_multi_raft_group_concurrent_put_get with 100 loops for each group, sometimes assert failed.
e.g.
thread 'test_multi_raft_group_concurrent_put_get::test_multi_raft_group_concurrent_put_get' panicked at 'assertion failed: `(left == right)`
  left: `[118, 52, 52, 52, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 48]`,
 right: `[]`', tests/integrations/test_multi_raft_group_concurrent_put_get.rs:214:21

L214 asserts the result of get after put
https://github.com/MrCroxx/RunKV/blob/b0d596c93ef2439241eaf6d5181228f901b68b41/tests/integrations/test_multi_raft_group_concurrent_put_get.rs#L184-L225
which should be None but got unexpected result.
When digging into the trace log, I found that the delete was not applied:
[2022-05-07T06:16:17Z TRACE runkv_wheel::components::lsm_tree] put; self=ObjectStoreLsmTree { raft_node: 52 } key=b"k444                                                            0" value=b"v444                                                            0" sequence=36656013143 apply_index=6530
[2022-05-07T06:16:18Z TRACE runkv_wheel::components::lsm_tree] get; self=ObjectStoreLsmTree { raft_node: 52 } key=b"k444                                                            0" sequence=36656013147

Seems caused by key generation rule by integration tests.......
Must be drunk...
https://github.com/MrCroxx/RunKV/blob/98d8e446f2464ea7c1479039182014c6e1e99381/tests/integrations/test_multi_raft_group_concurrent_put_get.rs#L182-L187


Update text on Create a Site page

[ ] Add to an appropriate Epic

Date first reported: 10/27/2021
Date requested for completion: 12/3/2021
Site: All commonses
Area (CORE, profiles, sites, groups, account ): Sites
Description
The text on the Create a Site page is confusing some users. The example I'm attaching is for HC, but similar changes should be made to all Commonses while preserving localizations. This feedback came from Sharon Leon, who is currently teaching courses on MSU. See the attached zipped html file.
HTML file
Steps to reproduce
Step-by-step procedure to reproduce issue, if available.
Previous correspondence
Paste any previous correspondence here
Further details
Anything else we should know?
Trello link (if moved from Trello)

This took longer than expected as the code for the Create a Site page was spread over multiple functions and plugins.
I consolidated the code into hc-custom::buddypress-functions.php::hc_custom_bp_blogs_signup_blog() and remove the javascript in boss-child that had been used to override the text in the privacy options checkboxes. This code should no longer be necessary as the More Privacy Options function that created these fields no longer fires and they are created with the correct text by hc_custom_signup_blogform(). This change also affects the privacy options when creating a group site.

hc-custom
boss-child


upgrade graphql

upgrade graphql

@JCMais could you please have a look at this PR and let me know if you'd require any changes ?

Thanks!


Getting kicked whenever I accept a quest.

(Required) What is your Quests version / build number (do not say "latest")?
Quests 3.0.0-Naomi
(Required) Please describe the problem and how to replicate it. Any console errors?
When Accepting a quest from an npc. (happens with every quest)
Getting kicked with this error:
[06:33:08] [Server thread/INFO]: Title command successfully executed
[06:33:08] [Server thread/INFO]: Title command successfully executed
[06:33:08] [Server thread/INFO]: LeGenDaryX lost connection: Internal Exception: io.netty.channel.unix.Errors$NativeIoException: syscall:read(..) failed: Connection reset by peer
This is what is shown in my minecraft client:
Internal Exception: io.netty.handler.codec.DecoderException: java.lang.ArrayIndexOutOfBoundsException: 63

This doesnt happen on Quests 2.9

Does Quests 3 use titles on quest accept and quest completion ? Because im having issues with titles kicking players.

@Legendaryx Yes, by default, titles are enabled. You can disable them in the config.

@Legendaryx In the event that your quests.yml does not contain the "show-titles" option, simply add it as seen in the link below, or regenerate your config.
https://github.com/FlyingPikachu/Quests/wiki/Configuration
Keeping this open until I determine the reason for kick.

Yeah somehow whenever I updated to minecraft 1.12. Title plugins kick my players.
Any suggestions what it could be ?

You're saying all title plugins kick players? Do you have any anti-cheat plugins, or are you using an alternative like PaperSpigot?

It looks like this is happening on my server as well; the issue is it causes a timeout. Not sure why. I'm on Spigot.

Found this - https://www.spigotmc.org/threads/nativeioexception-syscall-read-failed-connection-reset-by-peer.185413/
@Legendaryx Seems it's most likely you're using BungeeCord? It may be possible that the way Quests is sending titles is not compatible with BungeeCord. Waiting on confirmation before I proceed further.

No BungeeCord here, just Spigot. When I disabled titles we didn't have any more problems.

@SXRWahrheit Interesting. Suppose it's possible you could have experienced a separate, albeit similar, bug.  Unlikely as that is, I'd like to hear from @Legendaryx before I proceed.

I do have bungeecord but it looks like a different issue.
I do want to make clear that I have this issue with every plugin that sends a title.

@Legendaryx Caulking this up as a Bungeecord issue. I am unable to assist you further, but have taken the liberty of opening an issue on your behalf - https://github.com/SpigotMC/BungeeCord/issues/2232


Change centralLabel font size

I need to change the centralLabel font size, but seems there isn't a corresponding property.

@sgiaretto That's true. There is no corresponding property for such a functionality. Changing the rangeLabelFontSize updates centralLabel font size proportionally. We will consider developing this feature in the future.
For now, I would suggest a workaround, which is to set rangeLabel to ['',''], use rangeLabelFontSize in  order to change centralLabel font size and to display your own range labels underneath the chart.


Planetoid World Gen with climate bands etc

Some discussion on IRC that I missed (boo!) was brought to my attention and matches perfectly with some later stage musings I was hoping to see one day anyway. Also fits in perfectly as another item for this mid/long-term milestone!

Finite (but still relatively huge, like 64km x 64km, user configurable) cylinder world (possibly also deeper on the y-axis)
Climate bands to sort out biomes in a more natural fashion, also possibly larger (would incidentally also encourage more travel)
Nice overview maps generated on world creation akin to DF's "world map" - we have the TerrainPreviewGenerator already, unsure how big it scales

Useful links shared by @t3hk0d3:

http://www-cs-students.stanford.edu/~amitp/game-programming/polygon-map-generation/
http://www-cs-students.stanford.edu/~amitp/game-programming/polygon-map-generation/mapgen2.swf
http://en.wikipedia.org/wiki/Voronoi_diagram

Other niceties you could tie into it:

Rivers, islands, oceans, and other large scale constructs
Roads and civilizations (DF all over again!)
(super duper sekkrit bonus item related to a long-term concept)

However, unlike in DF - you could stay in "fortress mode" even on the world map :D

Keep those ideas in mind and save them for a later stage of development as additional terrain generators and game modes. Imho infinite terrain should still be one of the primary options though.
I've already got some similar algorithms ready written in C# (even utilizing volumetric terrain and Marching Cubes) and they should be easily portable.

Cool :-)
Yeah, I forgot to even mention different game modes. Could still do infinite, but the big but finite does seem to have some intriguing possibilities to it :-)


Points with multicolor text

🐛 Bug
I cannot add points with multicolored texts.
To Reproduce
For me the example code provided in napari fails with the same error as in the code I was working on.
Error traceback:
File "~\add_points_with_multicolor_text.py", line 27, in <module>
    points_layer = viewer.add_points(
  File "~\napari\lib\site-packages\napari\components\viewer_model.py", line 4, in add_points
    import itertools
  File "~\napari\lib\site-packages\napari\layers\points\points.py", line 378, in __init__
    self._text = TextManager._from_layer(
  File "~\napari\lib\site-packages\napari\layers\utils\text_manager.py", line 292, in _from_layer
    return cls(**kwargs)
  File "~\napari\lib\site-packages\napari\layers\utils\text_manager.py", line 110, in __init__
    super().__init__(**kwargs)
  File "~\napari\lib\site-packages\napari\utils\events\evented_model.py", line 197, in __init__
    super().__init__(**kwargs)
  File "pydantic\main.py", line 339, in pydantic.main.BaseModel.__init__
  File "pydantic\main.py", line 1038, in pydantic.main.validate_model
  File "pydantic\fields.py", line 840, in pydantic.fields.ModelField.validate
  File "pydantic\fields.py", line 1121, in pydantic.fields.ModelField._apply_validators
  File "pydantic\class_validators.py", line 280, in pydantic.class_validators._generic_validator_cls.lambda3
  File "~\napari\lib\site-packages\napari\layers\utils\text_manager.py", line 337, in _check_color
    return transform_color(color)[0]
  File "~\napari\lib\site-packages\napari\utils\colormaps\standardize_color.py", line 65, in transform_color
    return _color_switch[colortype](colors)
KeyError: <class 'dict'>

Environment
 napari: 0.4.16
Platform: Windows-10-10.0.19043-SP0
Python: 3.10.5 | packaged by conda-forge | (main, Jun 14 2022, 06:57:50) [MSC v.1929 64 bit (AMD64)]
Qt: 5.15.2
PyQt5: 5.15.7
NumPy: 1.22.4
SciPy: 1.8.1
Dask: 2022.7.0
VisPy: 0.10.0

OpenGL:
- GL version: 4.4.0 - Build <IP_ADDRESS>31
- MAX_TEXTURE_SIZE: 16384

Additional information
Could be related to #4464
Thank you for your help in advance.

@tothmarcella : thanks for the bug report! From the info, it looks like you're running napari version 0.4.16. Is that right?
The multi-color text feature added in #4464 was not released in version 0.4.16 (it got merged just after that was released). So in order to run it, you should probably pull down the latest commit from main.
If you need this feature in a stable version of napari, I believe there is an upcoming v0.4.17 release.

@andy-sweet thank you I did not realize that, now it works perfectly

@tothmarcella : therefore, I'm closing this issue. Thanks again for filing!

Hi @andy-sweet thank you very much for your help so far. Now I am using the text multicoloring but I was unable to color the text of different clusters of points in the correct order. I created a discussion on image.sc https://forum.image.sc/t/adding-multi-colored-text-to-clusters-of-points/70451/1 I just wanted to make sure it reaches you since this issue has already been closed.


Can't create a spyder project for and existing mercurial working copy

From<EMAIL_ADDRESS>on 2010-09-05T04:51:56Z
Can't create a spyder project for and existing mercurial working copy.
Steps to reproduce the problem:

Click File->New Project
Select the root folder of an existing hg working copy
Set/confirm the project name

The expected result is that a new project is crated an it is visible in the project explorer.
The actual result instead is that no project is created and no error message is displayed.
Spyder2 - https://spyderlib.googlecode.com/hg/ a7dd42b752ae
GNU/Linux Ubuntu 10.04 amd64
Mercurial v. 1.4.3
Rope v. 0.9.2
The issue seems to be due to an incompatibility between rope 0.9.2 (and earlier) and mercurial API >= 1.3.
Attached a micro-patch that implements a simple workaround.
Attachment: rope_mercutial_compatibility.patch
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=329

From pierre.raybaut on 2010-09-05T06:40:47Z
Thx for the feedback!
Status: Fixed


Gnome damage is inconsistent

https://user-images.githubusercontent.com/71954727/230667114-13b9690f-05ec-4510-b29e-4ef43f4333f2.mp4
So it seems like if you don't move, the gnome will only damage you one time. That being said, if the player moves or the gnome gets pushed back, it can damage the player once more.

https://user-images.githubusercontent.com/71954727/233820985-b691c20c-e5b9-4660-8dff-a37c8bb46a87.mp4
Still bugged for giant gnomes


Added index to an overwhelmingly large README.md

Hello,
I find your port of NuttX to Cortex-A53+GICv2 invaluably amazing, which is sadly what I cannot say about README file of this project.
It is understandable to want to include every piece of information needed to work with this port, but at over 11 thousand lines, about half of them being exemplary log output, this approaches an Applicaton Note or Datasheet territory.
Without being sure what should be left in README and what should be moved to a different document (perhaps a Github-provided project wiki?) for full readability, the least I can do is to add an index to every part of README, which as you can see is over 100 entries with indentation inferred from content.
Should you accept this improvement even partially, I will be looking forward to provide "real" contributions to this fascinating project.
Sincerely,
Marcin Dąbrowski

Thanks! Really appreciate it, I'll update the Table of Contents for future updates. Please check out the articles, I think they might be better for collaboration.


refactor solvers

Codecov Report

Merging #295 (3872181) into main (6ddd05a) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #295   +/-   ##
=======================================
  Coverage   79.94%   79.94%
=======================================
  Files          41       41
  Lines        1436     1436
=======================================
  Hits         1148     1148
  Misses        288      288

Impacted Files
Coverage Δ

src/ONSAS.jl
100.00% <ø> (ø)

src/StructuralAnalyses/LinearStaticAnalyses.jl
87.50% <ø> (ø)

src/StructuralAnalyses/NonLinearStaticAnalyses.jl
86.84% <ø> (ø)

src/StructuralAnalyses/StaticAnalyses.jl
100.00% <ø> (ø)

src/StructuralSolvers/StructuralSolvers.jl
78.26% <ø> (ø)

src/StructuralSolvers/Solvers.jl
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Implement AxisAlignedBox Volume function

This PR added the implementation for the AxisAlignedBox::Volume function.
I noticed that several other functions in the AxisAlignedBox class are missing implementations, e.g. DensityFromMass, MassMatrix, Material. I not sure if these functions are are supposed to be moved to the AxisAlignedBox class when the Box class was refactored in this pull request. Maybe @nkoenig knows more about this?

update changelog?

@osrf-jenkins run tests please


AIDA FAQ

AIDA FAQ
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore
[x] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f364e579bf5d2cab477aa8--harness-developer.netlify.app


How can I use GraphSage to handle datasets without node features, such as IMDB-BINARY

❓ Questions & Help

How can I use GraphSage to handle datasets without node features, such as IMDB-BINARY？
I want to do graph classification tasks, how do I set in_channel and out_channel
Thanks for your answer

Now i understand, sorry to bother


Translate rendering elements

@seraco I think this is ready for merge. Review and merge when you are ready.


support topology chart

checked in. see https://studio.ensembleui.com/app/40dcdbe9-1ac3-4aab-8ee2-0bcdd2f1ca7b/screen/dnRat0H9FzoHfs8wfDEz
for example


fix: add user config

Codecov Report
Attention: 51 lines in your changes are missing coverage. Please review.

Comparison is base (f220100) 2.05% compared to head (45c87f1) 2.05%.

Files
Patch %
Lines

.../main/kotlin/com/alibabacloud/ui/BaseToolWindow.kt
0.00%
26 Missing :warning:

...m/alibabacloud/models/credentials/ConfigureFile.kt
11.53%
22 Missing and 1 partial :warning:

...om/alibabacloud/credentials/util/ConfigFileUtil.kt
0.00%
1 Missing :warning:

...in/com/alibabacloud/ui/MyStatusBarWidgetFactory.kt
0.00%
1 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##           master     #24   +/-   ##
======================================
  Coverage    2.05%   2.05%
======================================
  Files          11      11
  Lines         390     390
  Branches       51      51
======================================
  Hits            8       8
  Misses        381     381
  Partials        1       1

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


Added mc_x current_sensor interface implementation

Waiting on libhal current_sensor pr to be merged in for tests to pass.