Formatting files written in Taplo's schema directive inserts unwanted spaces

version: toml-0.6.1.wasm
When formatting files using Taplo's schema Directive (https://taplo.tamasfe.dev/configuration/directives.html#the-schema-directive), a space is inserted between # and :, causing Taplo's language server to no longer recognize it as a directive.
#:schema https://json.schemastore.org/fly.json

↓
# :schema https://json.schemastore.org/fly.json

Is it possible to avoid this behavior, for example by disabling formatting for the next line with a comment like # dprint-ignore, or through some configuration?

Oh, sorry!
By setting comment.forceLeadingSpace to false in the configuration, I was able to avoid this!


Roadmap

Laundry list of things to improve, or questions to discuss.

[x] Avoid recomputing things. The default grouping strategy is suboptimal: if there are many traces with the same grouping, it will redo the sorting each time. Though, to be honest, I wonder whether there are cases where the sorting is the bottleneck (compared to say actually plotting things). Also column selection can be made more efficient by using Tables.select and Tables.columnindex, instead of calling Tables.columns on the whole table.

[ ] Use metadata properly to store names of columns for labeling in the ColumnContext, legend information, etc.

[ ] Get it to work with Analysis from StatsMakie whenever they rely on all the data for some preprocessing (may need to happen on StatsMakie side, but can start here using Require).

[ ] Get it to work with Position.stack, Position.dodge & co (same as above).

[ ] Layout support (what would be the dependency structure?).

[ ] Fix odd bugs with categorical variables. If different groups have different unique entries, the categorical conversion screws up (may be fixed from the AbstractPlotting side, see https://github.com/JuliaPlots/Makie.jl/issues/453).

[ ] Add slicing context (i.e. columns are series and similar).

[x] Decide on Series data layout (tree: https://github.com/piever/AlgebraOfGraphics.jl/pull/14)

I would add actually creating legends to this. What information does one need to create a legend for a AoG plot? I was thinking about it and it seemed to me that the full plot is only defined once you decide the plotting function, no? So scatter(trace) is different than lines(trace). But you need to know which function actually plots a trace in order to create the correct legend, so this is a chicken and egg problem it seems to me. Do you have an idea about this?

Right now the plotting information is encoded in the metadata, so basically what is passed to plot is a list of AbstractTrace object, each with their own metadata. This allows to mix and match things using the algebra (i.e. metadata(Scatter, markersize = 10) + metadata(lines) and things like that). If the user uses this approach, and only calls plot on the result, then there is all the information we need for the legend.
The main downside is the verbosity, but I guess a good trick would be to have lazy objects that can encode plot type and metadata defined in AbstractPlotting (say Scatter(markersize = 10). If we want to be extra conservative, scatter(a::AbstractTrace) could even error.
The alternative is that the legend is generated within the call to the plot! function here. What API do you have in mind for the legend? I actually don't really know how to use legends in Makie.

aha ok so if I do metadata(Scatter, markersize = 10) at least I know we're dealing with scatters, from the traces alone. But how do I know which group has which marker and which color? Because that's what I need to prepare the legend.
In MakieLayout, and I assume this will then be integrated into AbstractPlotting in some form, a legend is backed by a node
entry_groups::Node{Vector{Tuple{Optional{String}, Vector{LegendEntry}}}}

This contains one optional group title and a list of LegendEntries for each group. A LegendEntry is a label plus a variable number of LegendElements, that can be MarkerElements or PolyElements or LineElements, etc. So building a legend with 3 size and 4 color groups separated would look a bit like this:
entry_groups = [
	("Size", [small_marker, medium_marker, big_marker]),
	("Color", [red_poly, green_poly, blue_poly, orange_poly]),
]

Ah OK, now I understand the question better. So, getting the name of the legend group, i.e. Optional{String}, is similar to getting the axis labels (take column names), but not yet implemented. Getting the "drawing info" correctly is slightly trickier.
I've done some cleanup and now the only object the plotting package has to deal with is AlgebraOfGraphics.Tree, which is a tree describing all the operations the user wants to do with the data, and AlgebraOfGraphics.outputs(t::Tree) gives a list of the traces. But in those traces the info is still encoded as categorical, so it's useless to actually drawing the legend. I need to move the "replace categorical variables with theme variables" outside of the plot! function, but should just be a small refactor.
This has the same issue as above if one changes the attributes inside the plot call, say table(iris) * data(col1, col2) * primary(color = col3) * plot(color = newcolors) and then before plotting we do not know the legend. But I suspect the workflow should be to really just do a ... |> plot at the end. Maybe the display method for these trees could be changed so that one doesn't even need that.

If you want to play with it, on current master you can do:
using RDatasets, AbstractPlotting
using AlgebraOfGraphics: table, data, primary, spec, traces

mpg = dataset("ggplot2", "mpg");
cols = data(:Displ, :Hwy);
grp = primary(color = :Cyl);
scat = spec(Scatter, markersize = 10px)
tree = table(mpg) * grp * cols * scat
palette = AbstractPlotting.current_default_theme()[:palette]

serieslist = traces(tree, palette)

gives you a list of OrderedDict, each of which iterates pairs of NamedTuple => Trace, where the NamedTuple has the categorical values of the variable, and the Trace specifies the plot (plottype(t::Trace) gives the plotting type, then t.args and t.kwargs are positional and keyword arguments respectively).
I think they are not combined optimally right now, but this is the object one should use for legends IMO. In this case serieslist only has one element (you get more if you use +):
julia> serieslist[1]
OrderedCollections.OrderedDict{NamedTuple,AlgebraOfGraphics.Trace} with 4 entries:
  (color = 4,) => Trace{Scatter{...}}(([1.8, 1.8, 2.0, 2.0, 1.8, 1.8, 2.0, 2.0, 2.4, 2.4  …  2.0, 2.0, 1.9, 1.9, 2.0, 2.0, 1.8, 1.8, 2.0, 2.0], Int32[29, 29, 31, 30, 26, 25…
  (color = 5,) => Trace{Scatter{...}}(([2.5, 2.5, 2.5, 2.5], Int32[29, 29, 28, 29]), (markersize = Pixel{Int64}(10), color = RGB{Float64}(0.337255,0.705882,0.913725)))
  (color = 6,) => Trace{Scatter{...}}(([2.8, 2.8, 3.1, 2.8, 2.8, 3.1, 3.1, 2.8, 3.1, 3.1  …  3.4, 3.4, 4.0, 4.0, 2.8, 2.8, 2.8, 2.8, 2.8, 3.6], Int32[26, 26, 27, 25, 25, 25…
  (color = 8,) => Trace{Scatter{...}}(([4.2, 5.3, 5.3, 5.3, 5.7, 6.0, 5.7, 5.7, 6.2, 6.2  …  5.4, 5.4, 5.4, 4.6, 5.0, 5.6, 5.3, 4.7, 4.7, 5.7], Int32[23, 20, 15, 20, 17, 17…

I imagine it should be more or less straightforward do make a MakieLayout legend from this.

I've added layoutplot (see implementation and example usage), to have an initial support for facet plots. As this already returns a complex thing (with various subplots), I imagine it could include a legend by default as well.


Updated validations.py python script

Fixed the behavior of validate_user function in validations.py.
Closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

I am trying to finish my assessment


Added receiving data from the site: https://thebeatbali.com

сборка данных за несклько месяцев

Added receiving data from the site: https://thebeatbali.com


AVM: Constrain RADIX_LE gadget

Only the execution logic has been added, we need to add in the circuit constraints

Also need to add constraints to properly enforce the following assertions


Update README.md Fixes #7

Hey I have added readme.md file Please label it under gssoc ext and hactoberfest

Hey @ANSHIKA-26  please  merge it

hey , i have merged another readme , since I found it better, if you wish to make any changes to it , kindly sync fork and make another PR , thank you !


\vtop alignment in l3galley unexpected

While working with some code loosely based off of the stackexchange answer http://tex.stackexchange.com/questions/30410/another-body-text-in-the-margin/32752#32752 I encountered some unexpected alignment that comes from anything that loads l3galley.
I don't know if this is intended since I am still not 100% comfortable with all of the experimental packages but this code produces a result quite different from what is produced if the l3galley package is removed.
\documentclass{article}

\usepackage{l3galley}

\begin{document}

a\vtop{\hsize=1cm a}

\end{document}

@davidcarlisle has found a LaTeX2e demo
\documentclass{article}
\usepackage{l3galley}
\begin{document}
a\parbox[t]{1cm}{a}
\end{document}

Reopening as a result.

Two-part fix here: one for l3galley, one for the demo. In the galley code I'd missed a subtle point from galley2 (@frankmittelbach's code). It's vital not to add any inter-paragraph space at the start of a galley level, so there needs to be an explicit test. An upcoming check-in fixes that.
At the same time, uncontrolled vertical boxes will not work with the galley: they have to be proper galley levels. That's needed so that the galley knows that a new level has started. (I've taken this from Frank: I don't think there's a way to do the job without this.)
Fixing the example and patching the galley code for the present leads to
\documentclass{article}
\usepackage{color}
\usepackage{l3galley}
\ExplSyntaxOn
\cs_new_eq:NN \GALLEYLEVEL \galley_level:
% Use until the l3galley update, at least for testing
\cs_new_eq:NN \orig__galley_insert_vspace: \__galley_insert_vspace:
\cs_set_protected:Npn \__galley_insert_vspace:
  {
    \bool_if:NF\g__galley_begin_level_bool
      { \orig__galley_insert_vspace: }
  }
\ExplSyntaxOff
\begin{document}
a\parbox[t]{1cm}{\GALLEYLEVEL a}
\end{document}

Note that color is required here as it makes the \parbox 'coffin-like': there's a reset required by the galley that is built around having a colour group at the edges of the box, which coffins do automatically.


Project Migration

migrate our old project (classic) boards to the all-new projects (beta) experience!


chore: update flag to pass builder url

Motivation
Lodestar only supports a single builder URL and silently removes fallback URLs if multiple are provided. This behavior is not transparent to the end user and we should be more explicit about it.
Description
Update flag to pass builder URL to --builder.url (previous flag is kept as alias)
Note that I explicitly removed the support for array and comma-separated values as I think it is better to throw an error on startup then to let the user have wrong assumptions that builder has a fallback configured. I don't know any setup that sets multiple builders but better to have an informative error.
Closes https://github.com/ChainSafe/lodestar/issues/6181

this is a breaking change in cli usage and i don't actually see real need except may be clarifying help on this

It's not breaking as I kept builder.urls as an alias. I opted to throw an error if someone passes multiple URLs (as array is not supported anymore). Alternative would be to keep array support and silently just pick the first URL entry (as we do right now) but I don't think this is great as I already outlined in the PR description.

:tada: This PR is included in v1.13.0 :tada:


introducing docker-compose

it allows to do some fancy tricks in a structured manner:

override dns (for local specific infrastructure setups)
mount necessary directory or files

working on #77 , that is why I referenced the hbase section of the code

Thanks again! (and we can use the CCLA signature from https://github.com/cloudera/hue/pull/337)


Failing to retrieve evm.js reconstructed source code

Selecting evm.js from the hover options results in no contract code being returned. Instead, it seems to hang and load indefinitely.
Windows 10.0.18363
VSCode 1.47.2

Hey @KadenZipfel, can you provide the address you tried it on?
Cheers,
tin
PS: that method is a bit experimental :) probably should have noted it 😂

It was on this address 0x5d3a536E4D6DbD6114cc1Ead35777bAB948E3643


Dragons test

Activities of this branch

Added test file for MyProfile.js
Added test file for Dragons.js
Added test file for DragonCard.js

Kindly asking for a review

Status Approved 🤝


[Discussion] Add fixed columns for electrodes and channels DynamicTable

I think it would make sense to have these columns explicitly added to the extension spec:
    ecg_electrodes_table = DynamicTable(
        name='electrodes',
        description='info on ECG electrodes'
    )

    # add relevant columns
    ecg_electrodes_table.add_column(
        name='electrode_name',
        description='reference name of the corresponding electrode'
    )
    ecg_electrodes_table.add_column(
        name='electrode_location',
        description='the location of the corresponding electrode on the body'
    )
    ecg_electrodes_table.add_column(
        name='electrode_info',
        description='descriptive information on the corresponding electrode'
    )

Similarly for the channels table:
    recording_channels_table = DynamicTable(
        name='channels',
        description='info on ecg recording channels'
    )

    # add relevant columns
    recording_channels_table.add_column(
        name='channel_name',
        description='reference name of the corresponding recording channel'
    )
    recording_channels_table.add_column(
        name='channel_type',
        description='type of the recording, e.g., single electrode or differential'
    )
    recording_channels_table.add_column(
        name='electrodes',
        description='descriptive information the corresponding electrode(s)',
    )

In this current form these two tables are way too flexible, if you agree that these columns are usually what the users will also define than they can be explicitly added in create_extension_spec.py (similar to here)

At first I was thinking that we could leave room for flexibility... Now I think implementing these as defined standards rather than being flexible, would make it more of an standard entity... So I totally agree and I also implemented the new ECGElectrodes and ECGChannels now, extending DynamicTable. Thanks for the suggestion!


PLATUI-3020 initial POC of using RedirectUrl to model referrerUrls and just log those that don't match our policy

closing as this is an old poc that I think has been superseded now


Fix :  error showing line numbers on code blocks,  blocks using 'highlights' without line-nos

Error : showing line numbers on code blocks, using 'highlights' without line-nos

check an example on : https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-schema/prisma-schema-reference#--modifier

Thanks so much for that @rahul3v - that was really ugly.

Fix #835

I closed that issue.


WIP: Integration - surface/surface.py - Unit tests and refactoring for Surface class

This MR focusses specifically on the surface.py file within the new surface directory. This relates to the Surface class.

Codecov Report

Merging #277 (814f6f8) into master (2490cf5) will increase coverage by 1.14%.
The diff coverage is 79.88%.

@@            Coverage Diff             @@
##           master     #277      +/-   ##
==========================================
+ Coverage   57.39%   58.54%   +1.14%
==========================================
  Files          53       59       +6
  Lines       26142    26197      +55
==========================================
+ Hits        15004    15336     +332
+ Misses      11138    10861     -277

Impacted Files
Coverage Δ

resqpy/surface/triangulated_patch.py
60.71% <60.71%> (ø)

resqpy/surface/surface.py
82.13% <82.13%> (ø)

resqpy/surface/mesh.py
82.19% <82.19%> (ø)

resqpy/surface/base_surface.py
85.71% <85.71%> (ø)

resqpy/surface/pointset.py
94.80% <94.80%> (ø)

resqpy/surface/combined_surface.py
97.77% <97.77%> (ø)

resqpy/surface/__init__.py
100.00% <100.00%> (ø)

resqpy/model.py
67.69% <0.00%> (ø)

resqpy/olio/xml_et.py
61.84% <0.00%> (+1.25%)
:arrow_up:

resqpy/olio/zmap_reader.py
91.57% <0.00%> (+80.00%)
:arrow_up:

Continue to review full report at Codecov.

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

@cflynn3 - This one was already in a fairly good place. I've added some integration tests to improve coverage. Only two methods have a complexity above 10, both at 11 (listed below). These both look fairly good already, so I've made no changes.
Surface.extract_patches - C (11)
Surface.create_xml - C (11)
Ready for review (will be merged into the previous branch integration_surfacepy as it relies on changes in this branch.


#683 Fix center the checkmark of the Checkbox component

Because the translate: 50% is applied on the unrotated rectangle, we need to take in consideration the rotation of almost 45deg that involve to a higher absolute width value.
- translate: 50% 0;
+ translate: calc(var(--ids-checkbox-check-width) / 2) 0;

Before
After

Closing this in favor of https://github.com/gsoft-inc/ov-igloo-ui/pull/705


Performed Code Cleanup

Removed redundant code
Standardized naming and style
Fixed comments
Fixed warnings

 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.:x: tstepanski sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Can one of the admins verify this patch?


linux error

https://ibb.co/nLPqVk I get this error when I try and start up my server on Linux.

https://ibb.co/bOMeqk

ugh reinstall the server

u obviously did add something to the code. Just use a clean unedited server

Or use a node.js version that supports mo-edited

The error isn't showing up from the code, verify your install of Node.js. Try running a test script, also check your modules.

@MastaCoder I followed all instructions to install node and get everything setup. Ill try again though, Also What Linux should i be using?

install gentoo

@Luka967 Whats that and how?

Fixed it.

@Logan7432

Fixed it.

Cool can you close then


The right side of the interface extends beyond the screen when there are a lot of editors below

Tested versions
4.3 stable, 4.4 dev2
System information
Godot v4.3.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated Radeon RX 560 Series (Advanced Micro Devices, Inc.; 31.0.14001.45012) - Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 Threads)
Issue description
When I have a lot of elements at the bottom of the editor, the whole interface of the right side of the editor together with the inspector goes off the screen, maybe we should add a tab switcher with left and right arrows?
I can also see errors in the output:
   Can't add child 'Control' to '@VBoxContainer@6642', already has a parent '@VBoxContainer@6642'.
Screen resolution: 1600x900

Steps to reproduce

Open MRP
Enable the plugin and restart the project
Errors, as well as the right side of the screen goes beyond the screen if your screen is not wide enough

Minimal reproduction project (MRP)
mrp.zip

Duplicate of https://github.com/godotengine/godot/issues/95681. But your MRP may be better than the one I made since it represents a realistic use case of many small names rather than few large ones.

We can use a TabBar which i don't think will be ideal for fixing this.
Or a ScrollContainer and hiding it's horizontal scroll bar while also allowing focusing the buttons to enable ScrollContainer follow focus.
Or an OptionButton like how blender deals with docks.
Or an HFlowContainer which also may not be ideal since it will keep increasing it's vertical size when we resize down the window width.

@WhalesState This will probably be consolidated into the older issue, so your comment might get lost in this issue

Thank you for your report, consolidating in:

https://github.com/godotengine/godot/issues/95681

Please add your MRP there for more details


Improve ComponentSelectionReason.isExpected

The previous implementation relied on ordering. There have been reports
of that order not holding.
We now check for a unique required or root reason.

@melix I have fixed the condition after our discussion. Build passed. Can you take another look?


HBASE-24327: TestMasterShutdown#testMasterShutdownBeforeStartingAnyRe…

…gionServer can fail with retries exhausted on an admin call.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 50s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 15s
branch-2 passed

+1 :green_heart:
checkstyle
1m 16s
branch-2 passed

+1 :green_heart:
spotbugs
2m 16s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 40s
the patch passed

+1 :green_heart:
checkstyle
1m 15s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
13m 33s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
2m 53s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 14s
The patch does not generate ASF License warnings.

40m 13s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/artifact/yetus-general-check/output/Dockerfile

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

JIRA Issue
HBASE-24327

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 3705b8bd417a 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 3a375e668c

Max. process+thread count
84 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

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

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m  9s
branch-2 passed

+1 :green_heart:
compile
1m  4s
branch-2 passed

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

-0 :warning:
javadoc
0m 41s
hbase-server in branch-2 failed.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 53s
the patch passed

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

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

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

-0 :warning:
javadoc
0m 40s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
122m 18s
hbase-server in the patch passed.

147m  4s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

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

JIRA Issue
HBASE-24327

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 453bc21a5c1c 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 3a375e668c

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/testReport/

Max. process+thread count
4026 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 51s
Docker mode activated.

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

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 24s
branch-2 passed

+1 :green_heart:
compile
0m 59s
branch-2 passed

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

+1 :green_heart:
javadoc
0m 39s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 51s
the patch passed

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

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

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

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

_ Other Tests _

-1 :x:
unit
212m 20s
hbase-server in the patch failed.

237m 40s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

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

JIRA Issue
HBASE-24327

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 7533b9ee47b1 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 3a375e668c

Default Java
1.8.0_232

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/testReport/

Max. process+thread count
2539 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1653/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

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

This message was automatically generated.

@markrmiller still working on this PR?

Closing abandoned PR


Standardize README and add License

See ipfs/community#124

@RichardLitt you are a deity, thank you :)


No license

This repository doesn't mention any license.

https://github.com/getflourish/Sketch-Style-Inventory/commit/76d0c00b670780e8f8fd7bda0654a6fadd58fbe8


Recompile function automatically if function .so file is not found

In cases where the .so file is not found, such as cases where the .so file is only created on writer but not on replica, or cases where the machines somehow switches to use a CPU on another architecture (e.g. x86 -> aarch64, vice versa). We'd need to use plrust.recompile_function to recompile all the existing functins.
We can introduce a boolean guc which controls whether plrust would recompile a function if it found that the .so file is missing.

The function will no longer "not be found" because we store it in the database now.


⚠ Bofrost.fr has degraded performance

In 66d5f0a, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1094 ms

Resolved: Bofrost.fr performance has improved in 16f827b after 13 minutes.


webrtc: fix WHIP/WHEP implementation (#1857)

offers and answers are now encoded in SDP in place of JSON; Location header is set by the server.
This fixes compatibility with GStreamer and whipsink.

@aler9
With the new version v0.23.3 I'm having problems connecting from android browsers.
If I connect directly to the server WEBRTC it works. But if the WEBRTC page is inside another page (iframe) the video either doesn't load or takes usually 30s+ to load.
The player loads as usual.
On v0.23.2 it works.
2023/05/28 18:01:36 INF [WebRTC] [session 67ff10e7] created by <IP_ADDRESS>
2023/05/28 18:01:38 INF [WebRTC] [session 73f6ebf0] created by <IP_ADDRESS>
2023/05/28 18:01:40 INF [WebRTC] [session 4ca85ad4] created by <IP_ADDRESS>
2023/05/28 18:01:42 INF [WebRTC] [session 1538ea53] created by <IP_ADDRESS>
2023/05/28 18:01:44 INF [WebRTC] [session a76048f1] closed (deadline exceeded)
2023/05/28 18:01:44 INF [WebRTC] [session 33866aaa] created by <IP_ADDRESS>
2023/05/28 18:01:46 INF [WebRTC] [session 67ff10e7] closed (deadline exceeded)
2023/05/28 18:01:46 INF [WebRTC] [session 53b966fe] created by <IP_ADDRESS>
2023/05/28 18:01:48 INF [WebRTC] [session 73f6ebf0] closed (deadline exceeded)
2023/05/28 18:01:49 INF [WebRTC] [session 0ca8a289] created by <IP_ADDRESS>
2023/05/28 18:01:50 INF [WebRTC] [session 4ca85ad4] closed (deadline exceeded)
2023/05/28 18:01:51 INF [WebRTC] [session 47bf4a75] created by <IP_ADDRESS>
2023/05/28 18:01:52 INF [WebRTC] [session 1538ea53] closed (deadline exceeded)
2023/05/28 18:01:53 INF [WebRTC] [session 653baa73] created by <IP_ADDRESS>
2023/05/28 18:01:54 INF [WebRTC] [session 33866aaa] closed (deadline exceeded)
2023/05/28 18:01:55 INF [WebRTC] [session b21debf8] created by <IP_ADDRESS>
2023/05/28 18:01:56 INF [WebRTC] [session 53b966fe] closed (deadline exceeded)
2023/05/28 18:01:57 INF [WebRTC] [session 4b922998] created by <IP_ADDRESS>
2023/05/28 18:01:59 INF [WebRTC] [session 0ca8a289] closed (deadline exceeded)
2023/05/28 18:01:59 INF [WebRTC] [session 43a3bc2a] created by <IP_ADDRESS>


Browser back button do not work.

When editing an elemental block in the admin panel and hitting the browser back button it do not go back to page before, you have to press twice. Sometimes a blank page appears.
Tested with Firefox Developer Edition and Silverstripe 4.3.3

@hpeide can you please provide the elemental version you're using?

Oh, sorry! Sometimes I just think everybody knows my composer. json :b
"dnadesign/silverstripe-elemental": "3.0.1"
I tried the latest verison now. And I think the accordion functionality eliminates this issue.

@hpeide are you saying that using an accordion fixes the issue? If so, can you tell us which module you're using to provide that functionality?


TypeFace!!

Please Insert The SetTypeFace For TextView in SliderView Class.
Thanks...

You can check this library https://github.com/OpenSooq/Pluto
it has the ability to have full  control over your views and layout

Please Insert The SetTypeFace For TextView in SliderView Class.
Thanks...

Checkout 1.3.0 , update to this release and add your custom views in SliderView...


HCL (Terraform) syntax highlighting not working for code blocks

VS Code Version:  1.86.1
OS Version: Windows 10 22H2 Build 19045.3930
Logs:

Co-pilot chat isn't applying any formatting or syntax highlighting to HCL code. It was working for me since I started using the extension several months ago. Syntax highlighting works for all other languages I've tested.
I've tried the current version, insiders preview and tried downgrading to v0.11.*, but when I try to downgrade authentication fails over and over. I've installed a couple of different terraform syntax highlighting extensions and it doesn't seem to affect the code produced by co-pilot chat. It always produces white text with a black background.

Steps to Reproduce:

Ask co-pilot chat to produce terraform code

Actually not windows specific. You just need to install the HCL extension (the one also provided by hashicorp)
Not sure why the terraform extension doesn't include  hcl support out of the box. However this is not a VS Code issue so closing

That worked. Thank you.


make LED-matrix (of Calliope, MicroBit, ...) programmable

it should be possible to set leds on a matrix on/off programmatically, either by a statement

on/off x,y (addresses a single LED) or
setMatrix(integernumber) with a bit encoding of the whole matrix, i.e. 31 for the whole lowest row on 5x5 matrix

already implemented


Adding ABookCase Orbital Reference System to CKAN

Simply adding my new mod's netkan file to the repository.

There's no github resource, you want repository.

Ref #3043


Unregister tasks (even for apps without label)

Simpify task deregistration. Do not query marathon to check
if this app is managed by marathon-consul.

👍


Vote Tab Improvements

I just reworked this tab so it works now like a list is intended for.
Before

After

how can you add delegates that are not in top 51?

@fileninja did you see this PR?
https://github.com/ArkEcosystem/ark-desktop/pull/120

@zillion yes, because i have an up to date master.
@fix you're right I just thinked there wasnt this possibility before. but I've just seen that there is a textfield for writing a custom one. I will think about how I can solve that. either prefilling the table with all delegates and a pagination or otherwise giving the possibility to add not listed ones.

I believe this should be the best way to vote:
Voting - only one button "Vote for a delegate" which opens a modal window with a select and text fields (like #120) to add the delegate name and no "Next" button but "Vote (1 ARK)" which will directly send the vote tx and add the delegate in the Votes tab.
Unvoting - instead of a checkbox, a delete icon and when you click it show a confirm dialog with button "Unvote (1 ARK)" which will send the unvote tx and delete the delegate from the list.

@zillionn this is definetly an option. But if we think about multiple votes i feel more comfortable with an multi select table. just uncheck the ones i dont want anymore and check the ones i want to add and click save. this will remove, add and update everything as i want in one step.

@fileninja but there is only 1 vote per Ark address so multiple selections are kind of unnecessary? What would be cool is you get a list of all delegates - if you are voting for someone that one is at the top of the list and some kind of label like currently voting for. and if you want to change a vote you select another one from the list below and it will unvote currently voting one in 1 tx and after 1 confirmation for unvoting tx is confirmed it will automatically vote for the one you want to change vote for (informing that it will cost you 2 ARK currently to change vote). If you are not voting for anyone then no currently voting for is there or something of that sorts.

@boldninja you're right. just thinked about https://github.com/ArkEcosystem/AIPs/issues/1 which seems to get in "multiple ranked votes" direction.
if we stay at single votes so far, this PR is useless and I will close it.

I think there will be still some time before AIP#1 goes on the table (might be few months) and it needs to be properly tested on devnet if we decide on that route.

alright. i will close the PR right here but will let the branch open in my fork.

Wrong Zilllion pal
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Tue, Jul 11, 2017 at 2:07 PM, Alex Beck<EMAIL_ADDRESS>wrote:

alright. i will close the PR right here but will let the branch open in my
fork.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ArkEcosystem/ark-desktop/pull/162#issuecomment-314437285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1hNAUKTVy5YjMB4kyohxt6MOfvsFuGks5sM3N6gaJpZM4OTV1b
.


Draw military symbology in the web client side

Summary of Issue Encountered:
As a user I want to be able to draw military symbology from the relevant information held within a feature class. Ideally this should be done client side making use a javascript library.
Brief summary of issue encountered:
Functionality does not yet exist
Include the military symbology standard and version/change set (ex. -2525D):
All currently supported standards

This is a limitation of the core software. This work will not be done in this issue. Closing now.


g_str_equal

Your commit message is not descriptive at all. Why is this change needed? As far as I can tell, this change is wrongg_str_equal() is not NULL-safe, and in most of the places the arguments can be NULL.

OK, thanks


feat(graphviz):  Switch to @hpcc-js/wasm from Viz.js

This will bump the Graphviz version to the latest stable (2.44.1)
Signed-off-by: Gordon Smith<EMAIL_ADDRESS>

Thanks for the PR


if the device library adds a network commissioning device type, this will need to be 1 per endpoint.

https://github.com/project-chip/connectedhomeip/blob/bebbd74187a721154f82061f1023a347d7ce7f57/src/controller/AutoCommissioner.h#L64-L69

This issue was generated by todo-issue based on a TODO comment in bebbd74187a721154f82061f1023a347d7ce7f57.

This is now obsolete - we check the feature maps of all network clusters on the device and select the endpoint in a different way.


Create example Pokedex

Feature Request
Describe the solution you'd like
I would like to have a public example of the pokemongo-json-pokedex.

I creating a Telegram/Facebook pokedex bot with counter calculators and other features :smiley:

@feralheart Nice to hear! But with this issue I thought about a small website built with Jekyll or Vue which really simply visualizes the data of pokemongo-json-pokedex

@BrunnerLivio If you want I can help you create a simple GitHub Pages. Does that suit your purpose?

@luissmg I was thinking about something real simple. Should be a as-small-as-possible example for the samples/ folder.
Something like that:

It is just important, that it has a lot of code comments how you access the data, and it should show how you can change the language with i18next. It does not matter how it looks in the end.
I'm happy to accept any pull request, if someone wants to do that :)

Will give it a try in the next few days!

My PokeDex bot is finally in OpenBeta! :)
PokeHelper
It creates DexCards, like this:

@feralheart looking good! :) So that uses the data of pokemongo-json-pokedex, right?

Thank you :)
Yes, I created it from the data of the output files and from an outer reference (the Legacy Moves), with the help of a Graphic Designer friend :)
I edited the URL.

The trick is to not add any libraries until you are 100% sure you really need them. I was not using almost any external lib. I will share it with you soon!

I gave it a shot with ReactJS and create-react-app
Website, Source Code

While accessing pokemon.json from outputs folder, there is no tag for pokemon image. From where we can access it, is there any other json just for that?

@manish4353 This repo provides all the assets


Picture marker symbol offset

Added offsets to the Pin picture marker symbols so that their base aligns with the point geometry that they are showing.

@willcrick Looks good, just a few minor comments and we are good to merge.

thanks @doneill have pushed updates for your comments.


Rec rewrite + find changed rewrite patterns

single pass recursive rewrite
can be very simple and replace both uopgrah.rewrite and dedup
main challenge is finding all rewrite rules that need to be changed for this to work as in #5079
thats why i log all uopgraph that dont agree with old rewrite

new idea: search for minimal graph that doesnt get rewritten correctly

Cool! I'm working on something similar in #5159
Something that checks the patterns would be great. Can you explain what's wrong with them?

Merged your UPat printer btw.

Merged your UPat printer btw.

for me visualizing stuff like that is super necessary

Cool! I'm working on something similar in #5159
Something that checks the patterns would be great. Can you explain what's wrong with them?

simplest example is the first rule in constant folder. it expects while node with mul children. but other rule rewrites mul -1 to neg . now the first rule cant recognize that

sink

Oh, but that's not an issue with the rewriter itself, it's just the different application order.

yes

the rules are created with top down application in mind. now we go from the leaves upwards right

current problem graph:
 14%|████████████████████████████████████                                                                                | 1/7 [00:00<00:03,  1.65it/s]  0 ━┳ UOps.ALU BinaryOps.ADD
  1  ┣━┳ UOps.ALU BinaryOps.ADD
  2  ┃ ┣━━ UOps.CONST 0.0
  3  ┃ ┗━━ UOps.CONST 0.0
  4  ┗━━ UOps.CONST -0.16666666666666666
UOp(op=UOps.ALU, dtype=float, src=(UOp(op=UOps.ALU, dtype=float, src=(UOp(op=UOps.CONST, dtype=float, src=(), arg=0.0), UOp(op=UOps.CONST, dtype=float, src=(), arg=0.0), ), arg=BinaryOps.ADD), UOp(op=UOps.CONST, dtype=float, src=(), arg=-0.16666666666666666), ), arg=BinaryOps.ADD)
mismatch: UOps.CONST          : dtypes.float              []                               -0.1666666716337204 != UOps.CONST          : dtypes.float              []                               -0.16666666666666666
  0 ━━ UOps.CONST -0.1666666716337204
  0 ━━ UOps.CONST -0.16666666666666666

 ((UOp.var('c1') + UOp.cvar('c2')) + UOp.cvar('x'), lambda x,c1,c2: x+UOp.const(x.dtype, exec_alu(BinaryOps.ADD, x.dtype, [c1.arg, c2.arg]))),

finished in other pr


"Docs: Nuxt config responsiveness is broken."

Summary (generated):
This is a bug report about a line of code in the Nuxt documentation that breaks the reactivity of the Nuxt config page in the API section. The issue is related to the "unhead" module and its installation options. A screenshot is provided to illustrate the problem.

Environment
nuxt.com
Reproduction
https://nuxt.com/docs/api/nuxt-config#unhead
Describe the bug
Hi,
This line seems to break reactivity of the nuxt config page in api section in docs:
See: https://unhead.unjs.io/setup/ssr/installation#optionsExample:
here is pic

Additional context
No response
Logs
No response

@moshetanzer seems like a straightforward fix! PR welcome


Bug: Re-Binarization of labels in pre-processing routine

Hi all :slightly_smiling_face:,
I just discovered this small bug / missing post-processing step in the pre-processing routine.
Discovery / Problem
When running nn_UNnet  nnUNet_plan_and_preprocess -t 501 --verify_dataset_integrity on the pre-processed dataset, we get the following error message:
Unexpected labels found in file /home/jmcginnis/data/nnunet/nnUNet_raw_data_base/nnUNet_raw_data/Task501_MSSpineLesionPreprocessedAxialOnly/labelsTr/MSSpineLesionPreprocessedAxialOnly_047.nii.gz.
Found these unexpected values (they should not be there)
[7.925189e-17, 8.585621e-17, 9.0809456e-17, 1.0401811e-16,
1.0897135e-16, 1.386908e-16, 1.5024837e-16, ...]

Similarly, @kiristern is dealing with low DICE values for the Modified U-Net baseline:
2022-12-20 17:24:12.210 | INFO     |
ivadomed.testing:test:88 - {'dice_score': 0.060521258921826665,
'multi_class_dice_score': 0.060521258921826665,
'precision_score': 0.06500272972600003,
'recall_score': 0.0716620061524084,
'specificity_score': 0.9973268868391573,
'intersection_over_union': 0.03410365979066674,
'accuracy_score': 0.996024251185431, 'hausdorff_score': 2.045652891236524}

Although I am not familiar with ivadomed, I suspect that multi_class_dice_score indicates that ivadomed faces similar problems with the non-binary labels and interprets it as a multi-class problem instead.
... but why?
When we resample the images to isotropic resolution, we introduce sampling artifacts as we blur the edges of the labels, leading to smoothed contours. Thus, we observe values other than {0,1} in the labels. Can be easily debugged by looking at one of the many examples of labels in the dataset.
Solution
We can mitigate this effect by adding the following post-processing step after this line:
https://github.com/sct-pipeline/bavaria-quebec/blob/fc4f71dcf6989e1b46c2f05b7c7dbec3714f74d7/preprocessing/preprocess_data.sh#L137
sct_maths -i ${file}_T2w_crop_res.nii.gz -bin 1e-12 -o ${file}_T2w_crop_res.nii.gz
I haven't looked into finding an optimal value for the threshold 1e-12, but I've chosen it to be extremely low so we have the whole label borders.

good catch! this is definitely a problem when the model expects a binary mask as input. However, when using softeg, the input mask should/can be non-binary (float ranging between 0 and 1).
Also, I think the problematic line referring to resampling the label (not the image) is this one:
https://github.com/sct-pipeline/bavaria-quebec/blob/fc4f71dcf6989e1b46c2f05b7c7dbec3714f74d7/preprocessing/preprocess_data.sh#L157
So, in order to address this issue, one possibility would be to output a binary and a non-binary masks, and your training config file would fetch the appropriate label.

Hey @jqmcginnis, thanks for notifying about this! I have a few comments on the ivadomed aspect of your issue. First off, ivadomed does not differentiate between soft and binarized versions of the GT, and computes the soft Dice coefficient by default (see the function here). Secondly, the multi_class_dice_score is essentially the dice_score function above called in a loop for the number of classes in the input (see this line). Hence, I don't think that this has anything to do with non-binary labels as it is implemented to work with soft labels in the first place.
As for the low scores obtained by @kiristern, I would attribute them to sub-optimal hyperparamters and/or sizes of the input image patches (note, this was a 3D model with a relatively very large dimension in the S-I direction because of the stitching) rather than it being something to do with non-binarized labels.
Now, as for nnUNet expecting binarized labels, I had the same problem with Monai's default parameters so I had to use a preprocessing transform on Monai's side to convert the soft labels to the binarized labels (granted that this will already lose information, but some monai functions are not ready to accept soft labels). HENCE, I was wondering if nnUNet had something similar as well? That is, conversion to binary labels before processing?
OR, as @jcohenadad suggested, we could modify the pre-processing script to output both binary and soft labels instead and choose whatever is appropriate for the model we're training.

note, this was a 3D model with a relatively very large dimension in the S-I direction because of the stitching

don't you want to go with a cropped version of the input for lesion segmentation task? (ie: cascaded approach)

HENCE, I was wondering if nnUNet had something similar as well? That is, conversion to binary labels before processing?

I've looked into the different functions, and I have not discovered any functions in the nn-unet repo to do this. I would see two options.

Calculate a binary mask in the nn-unet renaming routine or MONAI routines.
Output a second binary mask as @jcohenadad recommended in the pre-processing script.

Personally, I would recommend option 2) as this ensures each current model (and future models) will always use the same binarized version, e.g. perhaps different libraries use different threshold for the generation of a binary label.


FanEntityFeature and startup WARNING

The problem
After 2024.8 (both homeassistant and miele integration) there are lines when starting up.
Version of miele integration having the issue?
2024.8.0
Version of Home Assistant Core having the issue?
2024.8.0
Anything in the logs that might be useful for us?
2024-08-08 12:33:39.233 WARNING (MainThread) [homeassistant.components.fan] Entity None (<class 'custom_components.miele.fan.MieleFan'>) does not set FanEntityFeature.TURN_OFF but implements the turn_
off method. Please create a bug report at https://github.com/astrandb/miele/issues
2024-08-08 12:33:39.234 WARNING (MainThread) [homeassistant.components.fan] Entity None (<class 'custom_components.miele.fan.MieleFan'>) does not set FanEntityFeature.TURN_ON but implements the turn_o
n method. Please create a bug report at https://github.com/astrandb/miele/issues

Additional information
No response

These are just warnings. Will be fixed in next release.


readme wrong pin at mega; fix #114

added  some more stuff: firmware_check example + firmware check of 0x90


[flutter_tools] Pass app entrypoint to DWDS version 19.0.0

cc @DanTup

test-exempt: rolling dependency

@ricardoamador cc myself


load bmi3d center_out task codes function

something like:
def load_center_out_task_codes():
    config_dir = files('aopy').joinpath('config')
    params_file = as_file(config_dir.joinpath('task_codes.yaml'))
    with params_file as f:
        task_codes = yaml_read(f)[0]
    return task_codes

also a similar function for channel mapping files would be useful, to load the file so that it can be used in map_ch2acq, etc.


[GH Actions] Colorize git diff by character

If you haven’t already, check out our contributing guidelines for onboarding!

Platform - version:
N/A
Action Performed (reproducible steps):

We have an automation that runs on our PRs that recompiles all our Javascript Github Actions called verifyGithubActionsBuilds. This is useful in catching unexpected errors with ncc (the library we use to bundle our Node.js scripts with their dependencies), and in preventing developers from forgetting to recompile the scripts.

Expected Result:

if a diff is found, the diff output should have character-level diff highlighting — but presently, it does not. This makes it difficult to spot differences between the committed script and the recompiled script.

Actual Result:

the diff output does not have character-level diff highlighting. This makes it difficult to spot differences between the committed script and the recompiled script
Looking at the diff output by verifyGithubActionsBuilds can be a useful debugging tool, but it is sometimes very difficult to spot the differences between the the outputted diffs, which wastes time and causes developer headaches. See this workflow run for an example of a not-so-useful diff.
Solution
Upgrade the verifyActions script so that it outputs a colored, character-level diff. Something like this:

https://www.upwork.com/jobs/~01d11a72b11bec4282

Hi @roryabraham  and @bfitzexpensify,
Thank you, I start working on the issue as soon as possible.

@bfitzexpensify was payment ever issued in Upwork for this job? I can't find it.

I don't know if it helps but at least on my side it appears as paid.
2021年8月9日(月) 18:20 Matt Allen @.***>:

@bfitzexpensify https://github.com/bfitzexpensify was payment ever
issued in Upwork for this job? I can't find it.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/1665#issuecomment-895557829, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEL2SSRVB7N3U3JLTT2HB23T4BBBZANCNFSM4Y2CBFEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Add instruction about allowing TCP ports

Adds instruction about allowing TCP ports because without them the room doesn't work.
Fixes https://github.com/ssb-ngi-pointer/go-ssb-room/issues/246.

Thanks!


Support @main entry points

It's currently blocked by the upstream lack of support for this in SwiftPM, see https://bugs.swift.org/browse/SR-12683 and https://swiftpm.slack.com/archives/C14QU6CUU/p1594734422189300 for more details

~It looks like that might be unrelated — it’s about not being able to use @main at all in single-file Swift things, not overriding @main on a custom class.~ oh I see you just have to define public static func main() on whatever gets the @main. Neat!


Sub-moduled multi-binary projects

Description
Background
During DEF CON Quals 2023, Shellphish used BinSync for many binaries, some of which were related. A pain point was finding every repo you wanted to connect to, cloning it, and opening it. It would be interesting if we could incorporate a Meta-Project sort of repo that would have submodules that allowed you to subscribe to specific binaries you cared about and use that to find the relevant project.
This idea is still open for development, but was an interesting idea brought up by @adamd
Alternatives
No response
Additional context
No response

Git only goes this far. You may consider running a service providing a simple REST API for this purpose.

Not my idea, but sounds like a useful solution! :)

Not my idea, but sounds like a useful solution! :)

LOL! Sorry about that, I suppose I meant @adamdoupe. But the more Adams the merrier!


WIP: Redesign Aggregate root unit of work

Summary
Optional summary of the PR here. The GitHub release description is created from this comment so keep it nice and descriptive.
Remember to remove sections that you don't need or use.
If it does not make sense to have a summary, you can take that out as well.
You would typically keep the summary only if there is a general theme change that can be summarized. If you find
yourself saying the same thing as any of the bullet points in a slightly different way; then this section is not
needed.
Added

Describe the added features

Changed

Describe the outwards facing code change

Fixed

Describe the fix and the bug

Removed

Describe what was removed and why

Security

Describe the security issue and the fix

Deprecated

Describe the part of the code being deprecated and why

I've started an issue for fixing the issues #1405 with an initial design I think can work.


Allow custom trace attribute keys

Is your feature request related to a problem? Please describe.
Trace attribute keys are enforced by the Temporal SDK, and are as follow:

temporalWorkflowID
temporalRunID
temporalActivityID

Found here: https://github.com/temporalio/sdk-go/blob/master/interceptor/tracing_interceptor.go#L39-L43
The only way to customise those keys is to create our own implementation of interceptor.Interceptor which is kind of heavy work to simply rename these keys.
Describe the solution you'd like
We could have the possibility to override the key values set by default in interceptor.TracerOptions, and port these options to opentelemetry.TracerOptions and opentracing.TracerOptions.
Additional context
Issue related to this thread on Slack

As discussed on Slack, here's a possible approach without having a custom interceptor.Interceptor.
 Then pass this tracer in interceptor.NewTracingInterceptor:
otelTracer, err := opentelemetry.NewTracer(opentelemetry.TracerOptions{
	// ...
})

customTracer := MyCustomTracer{
	Tracer: otelTracer,
}

var opts = client.Options{
	Interceptors: []interceptor.ClientInterceptor{
		interceptor.NewTracingInterceptor(customTracer),
	},
}

Leaving this issue open to let the Temporal decide if/how this should be documented or if another approach should be implemented.

+1 for this. We would like to inject additional attributes on our traces, and there doesn't seem to be a good way to do it. Even if we override StartSpan as described in this comment (great idea, by the way), we'd like to include tags that we're storing on our context, but unfortunately the context is unavailable there.


Token image: Upload new image: cannot change zoom more than once

When uploading a new token image the zoom slider can only be changed once, after which it is stuck.
https://user-images.githubusercontent.com/9307361/117334177-f941cb00-ae99-11eb-89ec-a044782b9c11.mp4

This should be fixed on master and was an issue with chakra ui https://chakra-ui.com/changelog#04-05-2021
I will release a patch release change this weekend.


package.json: Update react-dom package dependency

Check for any outstanding update of point version of any outstanding NPM package dependency

[x] npm-update

npm-update in progress on bos-tasks-kfdpb.
Log: http://fedorapeople.org/groups/cockpit/logs/npm-update-586-20190307-064343/

npm-update react-dom done: https://github.com/cockpituous/welder-web/commits/npm-update-react-dom-20190307-064444

Please manually check features related to these files:
main.js:import ReactDOM from "react-dom";
package.json:    "react-dom": "16.8.4",

Codecov Report

Merging #586 into master will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #586      +/-   ##
==========================================
- Coverage    73.4%   73.14%   -0.26%
==========================================
  Files          66       66
  Lines        3128     3128
  Branches      597      597
==========================================
- Hits         2296     2288       -8
- Misses        832      840       +8

Impacted Files
Coverage Δ

components/ListView/ListItemImages.js
58.73% <0%> (-3.18%)
:arrow_down:

core/sagas/inputs.js
71.42% <0%> (-3.07%)
:arrow_down:

components/ListView/ListItemComponents.js
80% <0%> (-2.23%)
:arrow_down:

core/selectors.js
92.96% <0%> (-0.79%)
:arrow_down:

core/sagas/blueprints.js
66.86% <0%> (-0.59%)
:arrow_down:

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 3713f6f...07534be. Read the comment docs.


Interactive documentation for JSX

This issue was originally reported by @viebel via facebook/react/issues/10646:
The interactive version of the "JSX in depth" article got a nice traction today on Hacker News.
The interactive snippets are powered by the Klipse plugin (I'm the author of the plugin). The evaluation is done on the client side: no server is involved at all!
Would you consider trying the Klipse plugin on the official react documentation?

Hello guys,
Please share your thoughts and concerns about this idea.
As a starter, here is the main value points and the concerns as I see it:
Value to the readers
they can experiment with the code snippets without having to leave the page.
Concerns
the size of the javascript files:

klipse plugin: 1.1 MB, 224KB zipped
babel: 1.4 MB, 459 KB zipped
codemirror: a few KBs

Those file sizes are valid concerns certainly. I think we could potentially do some things to mitigate them. On the Babel REPL (source code here) we do a couple of things to help with this:

Load all big libs (eg Babel, big plug-ins) from CDN to benefit from browser cache.
Defer loading any plug-ins until needed.
Load them into a web-worker context so that compilation doesn't slow down the UI thread.

Before reducing the footprint, would you like to give it a try on one or two pages of the react documentation and feel how it feels both on desktop and on mobile?

I think it would be reasonable to put together a minimal proof-of-concept to see how the UX was before we worry too much about performance optimizations, sure.

Hello guys,
I've been working on an integration of Klipse using [gatsby-plugin-klipse] (https://github.com/ahmedelgabri/gatsby-plugin-klipse) written by @ahmedelgabri. Here is the Work-In-Progress pull request https://github.com/reactjs/reactjs.org/pull/236
Let's continue the discussion there.

I'm going to close this issue for now, for the reasons I mentioned in my comment on PR #236. I'm happy to keep talking about and re-evaluate any new ideas of proofs of concept though! ❤


How to execute a gulp plugin programatically (integration testing)

I have a simple function that does 3 things and then I return that fn and register a name w/ gulp so I can invoke it later.
module.exports = function(gulp, options) {
    var concat = require('gulp-concat');
    var react = require('gulp-react');

    gulp.task('foo', [], function(){
        return gulp.src("/**/*.jsx")
            .pipe(react())
            .pipe(concat('react.js'))
            .pipe(gulp.dest('./build'));
    });
};

On the surface this appears to return something that's streaming because when I print it from a test I get the usual writable/readable/end/destroy/pause/resume
I'm trying to test this by doing the following
describe("react task integration tests", function() {
    it("should transform jsx to vanilla javascript", function(done) {
        //this require will pull in the above module
        var react = require('../tasks/react')(gulp, options);
        var outstream = gulp.tasks.foo.fn();
        outstream.on('end', function() {
            //here I expect to see the content written but .. sadly no
            done();
        });
    });
});

But when I get into the stream.end I see the file is not written like I expect. If I do the following it does work. (notice I'm doing the entire thing by hand, and I'd much prefer to invoke the function you see above that does all the src/pipe/concat/react/dest work
describe("react task integration tests", function() {
    it("should transform jsx to vanilla javascript", function(done) {
            var react = require('gulp-react');
            var concat = require('gulp-concat');
            var instream = gulp.src(join(__dirname, "./src/react.jsx"));
            var outstream = instream.pipe(react());
            var outstream2 = outstream.pipe(concat('react.js'));
            var outstream3 = outstream2.pipe(gulp.dest('./test/tasks/build'));
            outstream3.on('end', function() {
                        //at this point when I read from the filesystem I have the output
                        done();
                    });
        });
});

Is it possible to test gulp functions that batch up a few operations into a single streamable like this? What am I doing wrong? Thanks for the help!

@toranb : small correction for script.
In task declaration:
gulp.task('example', ['foo'], function()
its better to add parameter "done" (or it will be undefined):
gulp.task('example', ['foo'], function(done)


KFTI added

@klimarissa17 Please rename KFTI/lib/domains/ru/knc/kfti.txt to lib/domains/ru/knc/kfti.txt
Thank you!

@klimarissa17 You might miss my previous comment. Please react.

@klimarissa17 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the abovementioned issue. Thank you for the understanding!


feat: build arm images

PR is changed to rebuild images during the release process as due to https://github.com/containers/podman/issues/4646 podman load doesn't support multi-arch. This will cause images to be built twice, once for helm testing and once for release, but that's the best we can do at the moment.

This now should work (tested on the local fork). Let's live with the downsides for the moment and hope podman issue will be resolved soon.


Updated to trigger a notification if CPU recommendations are unavailable at profile level.

Updated to trigger a notification if CPU recommendations are unavailable at profile level.
following notifications are missed
"CPU Usage is less than a millicore, No CPU Recommendations can be generated"
"CPU usage is zero, No CPU Recommendations can be generated"

@msvinaykumar - Can you please update the function test suite with tests to ensure such regressions are not missed out. You can refer this issue to check for tests added for which notifications. Thank you.

@msvinaykumar - Can you please update the function test suite with tests to ensure such regressions are not missed out. You can refer this issue to check for tests added for which notifications. Thank you.

@chandrams  was there any previously written test cases check for CPU usage being zero or less than a millicore. Or Do I need to include new test cases?

@msvinaykumar - Tests for these notifications aren't present in the testsuite, new tests are required. You can check the issue above to see for which notification codes tests were added earlier.

@msvinaykumar - Tests for these notifications aren't present in the testsuite, new tests are required. You can check the issue above to see for which notification codes tests were added earlier.

done


ModuleNotFoundError: No module named 'torch.fx'

🐛 Describe the bug
I get this import error on
import torchvision.transforms as transforms

Versions
Versions:
torch-1.6.0-cp36-cp36m-manylinux1_x86_64.whl
torchvision-0.11.1-cp36-cp36m-manylinux1_x86_64.whl
I think this happened since upgrading to v11.1. Reverting to v0.10.1 to confirm.

yes, reverting torchvision to v0.10.1, fixed the import error.

@wmpauli note that torchvision 0.11 requires PyTorch 1.10. So many of the features in torchvision are not guaranteed to work correctly with an older PyTorch (for example, custom operators like roi_align).
Also, ccing @datumbox as it was at some point considered to make this a hard failure -- use-cases like this one would fail if we were to enforce loading the libraries to be a hard-requirement.

Thanks for the ping. Indeed if @wmpauli was trying to call a C++ op, they would receive an error while using Python operators may or may not lead to an issue. Here the user got an error because the fx namespace doesn't exist in the previous version of PyTorch.
@fmassa What was the reason we didn't make it a hard failure? We could add an assertion on the torchvision/__init__.py to cover for that.

Thanks again @fmassa and @datumbox. Like I mentioned above, reverting torchvision to .10.1 indeed avoids this issue.
This is for the continuous deployment of an older model, which requires an older version of torch, so I had been pinning the version of pytorch to 1.6, relying on pip to resolve the versions of dependencies (including torchvision). I'll be more careful next time.


Switch to rustyline in the console

Pull Request Test Coverage Report for Build 281

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 80.585%

Totals

Change from base Build 277:
-0.03%

Covered Lines:
3690

Relevant Lines:
4579

💛  - Coveralls


Hint for MaybeOverlap error

Would it be helpful to add an hint as to the action that generated an overlap error with:
            try:
                updated_input = path.execute(previous_tree)
            except AccessFailure:
                raise MaybeOverlappingActions(
                    "When using chained actions, individual actions should not"
                    " overlap with each other."
                    f"   Action attempted: {action}"
                ) from None

Maybe something like this:
                    f"\n    Previous action: {self.action[i-1]} for node: {ast.unparse(action.node)}"
                    f"\n   Action attempted: {action} for node: {ast.unparse(action.node)}"


Investigate Documentation Migration

What?
Determine the best method/solution for migrating current user docs to the new documentation system.
Why?
To learn how to integrate gitbook into our existing PASS project to make one seamless transition to GitBook.
How
Experiment with a various ways of importing docs into GitBook. Determine the best approach from a technical and process standpoint. Which approach would make the most sense for our team.
Acceptance Criteria
Set of recommendations for several different methods to migrate docs.
Related Issues

#539 is related because it's the epic

Documentation migration analysis: https://docs.google.com/document/d/1clxG63nTxELAZgnHUb4EbT03GGrZpA9_5RLsewLpqyQ/edit?usp=sharing


Remove .nsprc file after update license-checker to the next release

https://github.com/davglass/license-checker/pull/160
File to be removed:
https://github.com/bucharest-gold/license-reporter/blob/master/.nsprc
Depends on license-checker release.

https://github.com/bucharest-gold/license-reporter/pull/309


Hyper AutoOptimize

What would be the best strategy for finding the right estimate of time to Hyper AutoOptimize? Should we perform a single step and benchmark the time taken and use it as a quantity for what is best?
Also would the greedy approach do? For eg we optimize weather we want to use autodiff or not and weather we want to use GPU or not. We can find this out in 2 ways -

Check if it is faster with AD or not with GPU turned off. Then if it is faster with AD, we try GPU with AD. For n parameters, we try 2n configurations
We check every configuration i.e. 2^n

What would be the best strategy for finding the right estimate of time to Hyper AutoOptimize? Should we perform a single step and benchmark the time taken and use it as a quantity for what is best?

Hyper auto-optimize should be concerned about algorithm, while auto-optimize can only be concerned about implementation. Things that are essentially algorithmically equivalent (in the mathematical sense) are things like:

AD or not to AD?
Static arrays, GPUs, or normal arrays?
In-place or not?
Sparsity?

With some quick calls to f those can be determined, so those can be useful even if you're just trying to solve the ODE one time. That's auto_optimize.
hyper_auto_optimize is: I am going to solve this thing 100,000 times, what else should I be doing? At this higher level, this is where we can explore algorithmic details. In order to reduce the computational complexity of this determination, it should be structured at a high level via a decision tree.

First auto_optimize. Now you should have a faster problem.
Does the problem seem stiff or not, or in the middle? Solve with an RK method and an implicit method, with limits. See what works better. Try a ROCK method in there if stiffness seems somewhat possible.
If it's non-stiff, try and see the difference between VCABM vs Vern9 vs Tsit5 vs ?.
If it's stiff, the gambit of methods depends on size, but Rodas5 vs CVODE_BDF vs ..., and is dependent on GPUs and such
Try some linear solvers. GMRES or not?

Kick out a _prob,_alg pair that is likely to be the fastest.


added role oradb-rman

This role allows an automatic confguration of RMAN backups with cronjobs.
The roles/oradb-rman/README.md explains the details of this role.
The following parameters are requirred:
rman_channel_disk_default: "'/u10/rmanbackup/%d/%d_%T_%U'"
rman_controlfile_autobackup_disk_default: "'/u10/rmanbackup/%d/%F'"
Please add rman_jobs to oracle_databases to configure the jobs:
oracle_databases:
- home: db_home1
oracle_db_name: TEST
  rman_jobs:
     - name: parameter
       immediate: true
     - name: archivelog
       disabled: False
       day: "*"
       weekday: "*"
       hour: "*"
       minute: "10"
     - name: online_level0
       disabled: False
       day: "*"
       weekday: "0"
       hour: "02"
       minute: "30"
     - name: online_level1
       disabled: False
       day: "*"
       weekday: "1-6"
       hour: "02"
       minute: "30"

Hi,
I finally had time to test this, and these are some of issues I ran into.

autofs:
The task: "configure autofs for RMAN", will fail if the autofs package is not installed (or if the /etc/auto.master file is not there)
I will add the autofs package to the common role and make sure it is installed by default.

in tasks/main.yml, at the ' Execute RMAN-Script at playbook' task, there are 2 conditionals:
when: "'parameter' in item.1.name", and then further down:
when: item.1.immediate | default(false) == true and ((oracle_install_option_gi | default('SI') == 'CRS_CONFIG' and master_node) or oracle_install_option_gi != 'CRS_CONFIG')

Either combine them or remove one of them. I'm not even sure how ansible treats these...

The debug following the 'Execute RMAN-Script at playbook' task also needs a conditional. If not, it always runs and will fail if the previous task is not run.

I'm not really sure I understand this conditional in the 'Execute RMAN-Script at playbook' task:
when: item.1.immediate | default(false) == true and ((oracle_install_option_gi | default('SI') == 'CRS_CONFIG' and master_node) or oracle_install_option_gi != 'CRS_CONFIG').
If I am understanding this correctly, you want to run this if: 'immediate: True' and it should run on the master_node in case of a cluster and just run if not a cluster?
If that is the case, couldn't it just be re-written as:
when: item.1.immediate and master_node (as master_node is true by default)

This is just cosmetic stuff, but this conditional:
when: "rmanautofs|bool == true" could be changed to
when: rmanautofs

I did a lot of changes to the role and also added the changes above.
I close the pull request and create a new one.


curvlilinear contour and fill plots shifted

Reported by Ansley B Manke on 2 Oct 2003 17:06 UTC
For fine grids, fill plot and contour lines
are shifted.
use etopo5
set region/x=150.25w:147.75w/y=59.3N:60.2N
! Set up projection
set grid rose
go mp_mercator
go mp_aspect
fill/lev=(0,10000,10000)/pal=grey/noax/nokey/nolab rose,x_page,y_page
contour/over/lev="(-2000,2000,2000),line(-2000,2000,2000)" rose,x_page,y_page
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/639

Comment by<EMAIL_ADDRESS>on 21 Oct 2003 20:16 UTC
Moving from the<EMAIL_ADDRESS>user to the<EMAIL_ADDRESS>user.

Comment by<EMAIL_ADDRESS>on 10 Nov 2003 17:51 UTC
This is a precision bug; computations
in plot/plotz.F, plot/pltit.F, plot/setax.F


Lore command!!!

idk braylon wanted it, basically reaction map but for lore or smth idk

🤭 code looks good


chore(ci): book lint

I'd add lint as a dependency of either build or deploy

Not sure why, it took 4 seconds and you instantly new that something's off with the lint and not the build/deploy process.

I'd add lint as a dependency of either build or deploy

Because if it's a dep of build, then the order is lint -> build -> deploy, and if its a dep of deploy then the order is lint -> deploy. basically I am asking to gate deployment based on lints (don't deploy if lints don't pass) :)

I think I just worded it poorly/didn't include the actual intent 😄


Suggestion: Add EventSub Endpoints

Twitch released a new product called "EventSub", which allows subscribing to event notifications via multiple different transport types (including WebHooks):
https://dev.twitch.tv/docs/eventsub
This introduces new Endpoints to the Helix namespace.

Related to the #69 pull request.

Could the headers "Twitch-Eventsub-Message-Id", "Twitch-Eventsub-Message-Retry" and "Twitch-Eventsub-Message-Timestamp" be added to the EventSub events as attributes?
Currently, when there is a problem and events are resent by EventSub, there is no indication, when the event was fired initially, how many retries were necessary or if the same message was sent multiple times.

Feel free to create a pull request @Katzen48 :D
Currently I'm using app(Request::class)->header('twitch-eventsub-message-id'), for example to get the message id.


Make it clear whether we're talking about local or remote ASes.

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

According to @robbrockbank this is bugged, you can't set a BGPConfiguration for a node.

@fasaxc should we close this then?

@caseydavenport I've now fixed up the PR with (it think) the correct information


⚠ ClamAV has degraded performance

In 5b895bc, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 87 ms

Resolved: ClamAV performance has improved in 97849d4.


PRUEBA

Thank you 🙇♀ for wanting to create an issue in this repository. Before you do, please ensure you are filing the issue in the right place. Issues should only be opened on if the issue relates to code in this repository.

If you have found a security issue please submit it here
If you have questions about writing workflows or action files, then please visit the GitHub Community Forum's Actions Board
If you are having an issue or question about GitHub Actions then please contact customer support

If your issue is relevant to this repository, please delete this text and continue to create this issue. Thank you in advance.

Looks like you may have submitted this on accident. If you have a real issue, please let us know!


Misclassification of static obstacle into cyclist makes the vehicle stop.

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
Hi team,
I'm currently running Autoware with Carla, and I found a situation that the misclassification of static obstacle into cyclist makes the vehicle stop.
Here's the rviz video:
https://github.com/autowarefoundation/autoware.universe/assets/50267797/e1b2a8d0-4e4a-4376-bd54-e35edced2c7b
According to the video, the vehicle does not move because of the static obstacle, which is not cyclist at all.
I checked that obstacle in Carla, referring to the [frontview video], it was something other than the cyclist.

Expected behavior
I hope the vehicle to drive as usual, classifying the static obstacle clearly.
Actual behavior
However, the static obstacle is misclassified as a cyclist, which eventually make the vehicle stop.
Steps to reproduce
Here's the ros2bag file for the reproduction: [ros2bag]
Versions

autoware.universe: 7877192cb5700c1db15109dcd959785fc381951d
Carla: 0.9.15

Possible causes
No response
Additional context
No response

@Kim-mins
I found this problem is caused by centerpoint detection. (sky blue bbox in figure below)

Fig1. centerpoint detection in sky blue and clustering based detection in purple
Though these solutions will be temporary one, you can try:

Update LiDAR model to get high resolution pointclouds
Change merger settings so that the autoware prioritizes clustering object than centerpoint

For example, you can fix priority_mode in lidar_merger.launch.xml
  <!-- Merger -->
  <group>
    <!-- 1st merger to merge clustering + ML lidar detection-->
    <let name="merger/input/objects" value="$(var lidar_detection_model)/validation/objects" if="$(var use_validator)"/>
    <let name="merger/input/objects" value="$(var lidar_detection_model)/objects" unless="$(var use_validator)"/>
    <let name="without_dbt_and_filter" value="$(eval "'$(var use_detection_by_tracker)'=='false' and '$(var use_object_filter)'=='false' ")"/>
    <let name="merger/output/objects" value="temporary_merged_objects" unless="$(var without_dbt_and_filter)"/>
    <let name="merger/output/objects" value="$(var output/objects)" if="$(var without_dbt_and_filter)"/>
    <include file="$(find-pkg-share object_merger)/launch/object_association_merger.launch.xml">
      <arg name="input/object0" value="$(var merger/input/objects)"/>
      <arg name="input/object1" value="clustering/objects"/>
      <arg name="priority_mode" value="1"/>    <!-- change here from 0 to 1 so that merger will prioritize object1-->
      <arg name="output/object" value="$(var merger/output/objects)"/>
      <arg name="data_association_matrix_path" value="$(var object_recognition_detection_object_merger_data_association_matrix_param_path)"/>
      <arg name="distance_threshold_list_path" value="$(var object_recognition_detection_object_merger_distance_threshold_list_path)"/>
    </include>
  </group>

Thank you for the response and possible solutions @YoshiRi!
I tried your suggestions, but it seems they don't work..
For a resolution of the pointcloud, I increased the # of point per second from 1,200,000 to 10,000,000(changed from carla's sensor setting), but it doesn't work. Here's the video: [video]
I also changed the priority_mode to 1(actually my default value was 2, so I changed from 2 to 1), but it seems the issue still remains. [video]

Closing due to inactivity. Feel free to reopen if needed.


[#169591647] Public administration emails for development environment

This PR aims to make the URL of public administration data file configurable so that locally and in development environment it's possible to read data from a file containing mocked email addresses, in order not to use the real ones.

Affected stories

⚙ #169591647: In ambiente di sviluppo le email delle pubbliche amministrazioni non devono essere quelle reali

  Generated by :no_entry_sign: dangerJS against 2c52dbe327eb6c4185eed42063b0f27e68113ea9


after i2c_init A9G hangUp with XCPU and BCPU in GDB Loop

1. SDK version（SDK 版本）
{
platform/CSDK/RELEASE  SW_V2129
}

2. In what kind of operation problems appear, and how to reproduce the problem ?（什么样的操作步骤问题会出现，是否是稳定复现，如何复现问题？）
{
No SIM, GPS ENABLED, ADXL345 Connected to I2C2
}

after this istruction running in separate task from GPS routine :
config.freq = I2C_FREQ_100K;
I2C_Init(I2C2, config);
the tracer RubyScript show:
Detected XCPU in the GDB loop (0x9db00000), connect GDB debugger for more info.
Detected BCPU in the GDB loop (0x9db10000), connect GDB debugger for more info.
and the a9g hangup
i have test the i2c demo with the same result : GDB Loop....
thanks
Alex

Resolved after DRY and upload bigger version of HEX....
thanks


TextureProgress nine patch strech doesn't work in clockwise base fill mode (GLES3)

Godot version
Godot Engine v3.4.2.stable.flathub.45eaa2daf
System information
Linux, Fedora 34, OpenGL ES 3.0 Renderer: Mesa Intel(R) UHD Graphics 620 (KBL GT2)
Issue description
TextureProgress nine patch stretch option doesn't work correctly. Indeed it works as excepted for Left To Right, Right To Left, Top To Bottom, Bottom To Top, Top To Bottom, Billinear (Left And Right), Billinear (Top And Bottom) but not for Clockwise, Counter Clockwise, Clockwise and Counter Clockwise.
In Left To Right fill mode:

In Clockwise fill mode:

Note, this issue was already reported: https://github.com/godotengine/godot/issues/33222#issuecomment-1005895440 and should had been fixed by: https://github.com/godotengine/godot/pull/45815.
Steps to reproduce
Check the attached minimal reproductible project below. Change the fill mode from Left To Right to Clockwise and see the difference in the scene or when running the project.
Minimal reproduction project
godot-bug-report.zip

cc @lawnjelly

It seems to have the same behaviour with batching on or off.
Despite the ninepatch name I don't think this uses a ninepatch primitive, it is drawn with polys. And any bug is likely on the scene side rather than in the renderer.


[CT-397] [Bug] package selector doesn't work if Full YAML or with "package:" prefix

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When I define a selector using "full YAML" and it includes a package reference, the models in the package reference do not get run. When I define the selector using CLI-style syntax, the models in the package do get run.
Expected Behavior
I would expect both versions of the selector to work in the same way.
Steps To Reproduce
Use this selector in the selectors.yml:
  - name: cust
    definition:
      union:
      - staging_models.staging.cust
      - models/cust/*

And all models in models/cust/* and also in staging_models.staging.cust will be built.
Use this selector
  - name: cust2
    definition:
      union:
      - method: package
        value: staging_models.staging.cust
      - method: path
        value: models/cust/*

And only the models in models/cust/* will be built.
Additionally, if you use the "path:" or "package:" prefix like this:
  - name: cust
    definition:
      union:
      - package:staging_models.staging.cust
      - path:models/cust/*

the models in the package do not get built.
Is there something wrong with the way the package selectors work, or am I doing something wrong?
Relevant log output
No response
Environment
- OS: Mac OS 11.6.4
- Python: 3.9
- dbt: 1.0.1

What database are you using dbt with?
bigquery
Additional Context
No response

@solomonshorser Thank you for the submission, and apologies for the long response time!
In your examples models/cust/* resolves to a path selector, as you have already pointed out, however your other example staging_models.staging.cust resolves to a fqn (fully qualified name) selector, not a package selector.
If you change the entries in your selectors.yml that you have listed above to the following, I think you will find they work as expected!
selectors:

  - name: cust2
    definition:
      union:
        - method: fqn
          value: staging_models.staging.cust
        - method: path
          value: models/cust/*

  - name: cust3
    definition:
      union:
        - fqn:staging_models.staging.cust
        - path:models/cust/*

From the documentation on the package selection method:

The package method is used to select models defined within the root project or an installed dbt package. While the package: prefix is not explicitly required, it may be used to make selectors unambiguous.

If you have further questions about this behavior, please let us know!

Ok, that clarifies things. I already figured out an alternate way of doing my selectors, but I'll probably come back to this for the next project.
I guess the "fqn:" and "package:" notation got a little confusing. Does a package selector always select everything in a package? Seeing --select snowplow.* made me think I could replace * with specific models and do things like --select package:somepackage.somemodel, but I guess that's not the case - maybe for the sake of easing confusion, the package selector should consider .* to be a syntax error?
I think my mistakes were: thinking that the "package:" prefix could be used to select specific models from a package; not fully understanding the "fqn:" prefix.


Cannot find name 'FocusScopeProps'.

Bug report
Current Behavior
When I try to build the types for my component library that uses @radix-ui, the TypeScript compiler screams that it can't find the FocusScopeProps:

https://github.com/kaisermann/radix-repro-1
Expected behavior
Build the types without an issue 🙈
Reproducible example
https://github.com/kaisermann/radix-repro-1
$ npm install
$ npm run build

Suggested solution
Didn't investigate it yet, but it seems that it's just missing an import on the generated typing file:

Additional context

Your environment

System:

OS: macOS 11.2.1
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 189.97 MB / 16.00 GB
Shell: 3.1.2 - /usr/local/bin/fish

Binaries:

Node: 14.15.4 - ~/.volta/tools/image/node/14.15.4/bin/node
Yarn: 1.22.10 - ~/.volta/tools/image/yarn/1.22.10/bin/yarn
npm: 7.5.4 - ~/.volta/tools/image/npm/7.5.4/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman

npmPackages:

@radix-ui/react-dialog: ^0.0.16 => 0.0.16

Hey @kaisermann, thanks for reporting this, it does seem that the props disappeared from our output between previous and latest:

https://unpkg.com/browse/<EMAIL_ADDRESS>https://unpkg.com/browse/<EMAIL_ADDRESS>
I'm investigating and hopefully we can publish a fix soon.

Hey, @benoitgrelard 👋 Thanks for the quick response! If this is somewhat related, I also started to get this one:

It seems the react-menu package has some typing references to the original source file. If this is an unrelated problem, I can create another issue.

Thanks, I have a fix for the original issue, but I'll take a look at this one now too.

Hi @kaisermann, I've split the second issue and sent it to #640 as it's quite a bit more involved than the original issue for which I'll send a PR now.


Error handling, statistic dumps, progressbars

@oli-obk This basically resumes https://github.com/saethlin/miri-tools/pull/2
The total changes have grown pretty massive :joy:
Unless you have serious objections I'll merge this in a few days and we can keep iterating from this point.
The only thing that strikes me as missing is that this pulls the whole crates.io database dump on every run which is surprisingly fast but surely it could be cached. I just don't know how to figure out if there is a new database dump.

Awesome! Yea these changes lgtm


Add kernel reading/writing to an example

...
This could be part of the kernel tricks example.

Hi, I want to +1 for this. Due to some changes between pandas 1.5.3 and 2.0.0, a kernel file written to disk as a pickle using pandas 1.5.3 cannot be read if you upgrade to pandas >=2.0.0.
The error looks like:
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[9], line 4
      1 cyto_kernel_path = "/home/jovyan/data/cellrank/kernels/CytoTRACE.kernel.pickle"
      3 with open(cyto_kernel_path, "rb") as f:
----> 4     ctk = pickle.load(f)

ModuleNotFoundError: No module named 'pandas.core.indexes.numeric'

It seems like the reverse works fine (i.e. they're backwards but not forwards compatible).
It seems like a better option, however, is the write_to_anndata(), which would be more compatible across versions. That said, I  don't see any documentation about how to use that beyond the function docstrings.
Let me know if I can help!

Also, maybe this is just me, but I find it very frustrating when packages try to be smart about how they name files for me. If I pass filename="my_file_name.pickle", I don't want it to add extensions by default. Can we set ext=None by default?


Add warning when cloning/downloading from repos we've marked as unsafe

Is your feature request related to a problem? Please describe.
The Hub now warns you if malware has been found in a HF repo. Perhaps it would be wise to warn users when they are either cloning these repos or downloading files from them.
Describe the solution you'd like
When a user is cloning or downloading from huggingface_hub, they should get a very noisy warning letting them know the repo they're trying to clone contains what we believe to be malware.
Describe alternatives you've considered
We could also block them outright and force them to pass a allow_unsafe=True kwarg to the various download functions
Additional context
Initially brought up on Twitter here.

Not super high priority imo since we don't have any infected repos, but having is an important feature. Probably also worth having in transformers, specially with custom code getting more supported (cc @LysandreJik @sgugger)


ES2015ify

Would probably extract the CLI in here too.

@kevva I might have later today.
/reminder 8h (plz github)


[luajit] Improve static and dynamic CRT support

This port was only allowing CRT static linkage for static builds, and CRT dynamic linkage for dynamic builds. This commit remove this limitation.

/azp run

/azp run

Thanks for the PR @RT222 ; this looks like a pretty nontrivial buildsystem change. Have you submitted this upstream to the LuaJIT project yet?

I didn't because LuaJIT development is pretty dead. The last beta version was from 2017, and the last release version used in this build is from 2015.

Thanks for the explanation; this seems acceptable then.


TCVP-2550 Added multiple search filters to Staff Workbench

Description
This PR includes the following proposed change(s):
TCVP-2550 Added Date Submitted, Ticket Number, Status, and Surname filters to Ticket Validation tab.

Note: as per existing design, these are only client-side filters, not search parameters.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

Is there any reasons for not using mat-form-field ?

Is there any reasons for not using mat-form-field ?

Only that I don't know what that is. But I can research it and do that if that's better.

Ok, we had that package installed, you can use it freely. Please noted that we are using v14.
https://v14.material.angular.io/

Ok, we had that package installed, you can use it freely. Please noted that we are using v14.
https://v14.material.angular.io/

Great. I updated the implementation to use mat-form-field instead. See updated screenshot.

And can you also change the button to use mat-button, so we can have less custom CSS. Thanks

And can you also change the button to use mat-button, so we can have less custom CSS. Thanks

For sure. Done. But I'm pretty sure mat-button is not working correctly in staff-portal. The color="primary", color="accent", or color="warn" attributes don't work at all and the style looks nothing like the official BC Gov style.
This is the style we must follow:
https://bcgov.github.io/bootstrap-theme/docs/reference/simple/
The mat-button in staff-portal doesn't look like this so I still had to keep the custom css for now until this can be fixed globally.


Cyber rt on  nvidia orinx deployment

我想离线部署cyber rt 在英伟达 orin上 官网看的许多都需要单板能联网，这儿有离线安装的指导吗，单板上有docker

Do you need install source code or just release version?
We plan to release the arm version, which currently does not fully support. So you can wait, or try it yourself first!

Source code intall

Bazel itself needs to download some software, so it is best to be connected to the Internet

Thanks, I will follow your advice and docker scripts to try

Try the cmake version of CyberRT


coastal_ike_shinnecock_atm2sch is failing with DEBUG mode

coastal_ike_shinnecock_atm2sch test is facing with following error, if it is compiled under UFS with -DDEBUG option.
3: forrtl: severe (19): invalid reference to variable in NAMELIST input, unit 15, file /work2/noaa/stmp/tufuk/stmp/tufuk/FV3_RT/rt_97095/coastal_ike_shinnecock_atm2sch_intel/.//param.nml, line 36, position 18
3: Image              PC                Routine            Line        Source
3: fv3.exe<PHONE_NUMBER>C20DED  Unknown               Unknown  Unknown
3: fv3.exe<PHONE_NUMBER>274CF1  schism_init_              295  schism_init.F90
3: fv3.exe<PHONE_NUMBER>14174E  schism_nuopc_cap_         345  schism_nuopc_cap.F90

@josephzhang8 Do you have any idea? This might be related with the recent change in the SCHSIM that we solved by updating the namelist files (https://github.com/oceanmodeling/schism/issues/7). But, we did not test with DEBUG build in that time. There might be a bug in the model side.

@josephzhang8 This is giving error in nbins_veg_vert option that I have it under CORE section. If I check the code, I don't have it under SCHISM/src/Hydro/schism_init.F90. So, it is normal to fail. Maybe we need to sync SCHSIM since I could see it in https://github.com/schism-dev/schism/blob/9532b30bfb3516beb31fa6a11af8bfbb12e46085/src/Hydro/schism_init.F90#L185.

@josephzhang8 I think we are not pointing correct hash under app. Maybe there is no issue in here with updated model. I'll verify it. Sorry for false alert.

Closing this since it is working without any issue.