edupont04 from OPS portal: Merge from master to live

Docs Build status updates of commit 62e2bd1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/payroll/W-2-statement.md
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/w2.PNG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/us-year-end.md
:white_check_mark:Succeeded
View

dynamics-gp/whats-new/match-excel-copy-and-paste-decimal-places-currency-setup.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Using of jVectorMap under license GPL in a MIT licensed project

Issue type:

[ ] Feature request
[ ] Bug report
[ ] Documentation
[x] License Compliancy

Environment:

AdminLTE Version:  All
Operating System:  All
Browser (Version):  All

Description:
I'm making another project base on AdminLTE and I've audited librairies to make sure that everything meet the MIT license and I've noticed that one of the librairy used in the project is under GPL license which doesn't meet the license of AdminLTE which is under MIT License.

See #1616


Time-shifted panel still shows current dates

We are experimenting with the Flow plugin and have two panels, one of which shows data for the current date/time, the other showing one week earlier (via the Query options Timeshift field). We would expect that the time bar at the panel bottom would be adjusted to show the period represented by the panel data, but both remain set to the selected time range:

Are we overlooking a setting or option to have the time bar on the time-shifted panel display the correct time range?

thanks, this sounds like a bug on my part. will have a look...

this is now fixed on main and will be in the next build

This has now been published in version 1.16.1. Just reopen if you still have problems.


Server crashes when players hits another player with active schmucks

Description
When you have active schmucks running around, hitting another player seems to crash the server.
To Reproduce
0. Play on a server with other players

Have some active Schmucks running around in the game
Hit another player
Server crashes

Expected behavior
Server should not crash when you hit a player. They should just normally start attacking the player if PVP is on.
Desktop (please complete the following information):

OS: Widows 10
Java Version: Java 16
Version: 1.4.0
Minecraft Version: 1.17
Sodium installed: no
Other mods: None
By default, Java 8 is bundled along with Minecraft <=1.16.5. However, with 1.17, it is now Java 16.

Additional context

I'm having the same problem. Server log is as follows:
[02:03:09] [Server thread/ERROR]: Encountered an unexpected exception                                                                                net.minecraft.class_148: Ticking entity                                                                                at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:897) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.class_3176.method_3813(class_3176.java:334) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:833) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:692) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:274) ~[intermediary-vanilla.jar:?]                                                                                at java.lang.Thread.run(Thread.java:831) [?:?]                                                                                Caused by: java.lang.IndexOutOfBoundsException: Index -1 out of bounds for length 15                                                                                at jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64) ~[?:?]                                                                                at jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70) ~[?:?]                                                                                at jdk.internal.util.Preconditions.checkIndex(Preconditions.java:266) ~[?:?]                                                                                at java.util.Objects.checkIndex(Objects.java:359) ~[?:?]                                                                                at java.util.ArrayList.get(ArrayList.java:427) ~[?:?]                                                                                at com.alotofletters.schmucks.entity.SchmuckEntity.method_6007(SchmuckEntity.java:179) ~[Schmucks-1.17-1.3.2.jar:?]                                                                                at net.minecraft.class_1309.method_5773(class_1309.java:2342) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.class_1308.method_5773(class_1308.java:338) ~[intermediary-vanilla.jar:?]                                                                                at com.alotofletters.schmucks.entity.SchmuckEntity.method_5773(SchmuckEntity.java:187) ~[Schmucks-1.17-1.3.2.jar:?]                                                                                at net.minecraft.class_3218.method_18762(class_3218.java:656) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.class_1937.method_18472(class_1937.java:483) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.class_3218.method_31420(class_3218.java:418) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.class_5574.method_31791(class_5574.java:54) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.class_3218.method_18765(class_3218.java:393) ~[intermediary-vanilla.jar:?]                                                                                at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:893) ~[intermediary-vanilla.jar:?]                                                                                ... 5 more

This has been fixed - sorry about that. The update should be published by Sunday-Monday.


I'm having an issue with the log system

No logs are being made when tickets are closed, nothing is being wrote to the log folder. Please help me, if anyone has any experience fixing this issue i'm rarely active on github, my discord is Forgotten#1804

The logs/ folder is not used for logs anymore. It exists in the bot's code for backwards compatibility reasons. Nowadays, all logs are written to the sqlite database in db/data.sqlite and accessed through the log links you get when you close a thread.


Remove dependence on hadoop2 artifact as hadoop2 is default for avro 1.8.1

Changes Unknown when pulling 9450b385d664c742cd8fc91f5a86963bff104129 on chavdar:upgrade_avro2 into ** on linkedin:master**.


add astronomer suggestions

Description of the change
wisott
How tested
poetry run pytest passes
Security implications
none

I would keep the restructuring of the example DAG to be consistent with conventions. This aims to promote readability and surface the key census_conn_id parameter.
See: https://github.com/pgzmnk/airflow-provider-census/blob/master/airflow_provider_census/example_dags/example_census.py

Added the example DAG changes.


You had to fight the evil

You had to fight the evil, not join it!

Thanks @KhodyrevDS for passing by.
I also tried to fight it but could not spent much time due to other priority.
If you are interested, we can work together.


How to enable third party plugins for BabelConfig?

I am trying to use [transform-decorators-legacy] (https://www.npmjs.com/package/babel-plugin-transform-decorators-legacy) plugin, but because it's not part of the preset plugins I get this error:
Invalid plugin specified in Babel options: "transform-decorators-legacy"
Is there a way to include or set the path to the plugin js?
Thank you.
P.S. Forgot to mention that I am using .NET Bundling and Minification

transform-decorators-legacy was added to babel-standalone in 78ad50a8068771b9fa07ac162d37f910d9245b44, so I just need to update the version of babel-standalone bundled with ReactJS.NET.
Currently, there's no way to use custom Babel transforms with ReactJS.NET if they're not bundled with babel-standalone.


Internationalise intcomma for de_DE locale

See discussion in #48.
Changes proposed in this pull request:

Add the proper thousands separator for the de_DE locale
Add a simple test

Thanks!


Treat PostgreSQL time values as being in UTC time zone

This PR fixes the problem that is mentioned here
https://github.com/rbock/sqlpp11/issues/481
Currently when PostgreSQL timestamptz and timetz fields are read from/written they are treated as being in the local timezone and there is no easy way to write values in the UTC timezone.
This is a problem because PostgreSQL timestamptz and timetz fields are UNIX timestamps (they only change their output representation based on the configured server timezone) and std::chrono::time_point values are UNIX timestamps (until C++20 it was an implementation detail and starting from C++20 it is specified explicitly in the standard). At the same time writing a time_point with sqlpp11 and then reading the saved UNIX timestamp (e.g. with extract(epoch from field_name) ) yields a different value.
Additionally mapping from the local timezone to the UTC timezone is not always unique, so it makes sense to use UTC whenever possible and only convert from UTC to the local time zone when necessary.
Ideally for C++20 we could support reading/writing std::chrono::zoned_time and std::chrono::local_time which would allow one to choose the timezone explicitly, but for C++11 it makes sense just to default to UTC instead of the local time zone.

Thanks for the PR! Looks good at first glance, including the much shorter code using regex.
I'll take a closer look, soon.

Thanks again for the PR! I always try to avoid timezones, which is why my experience in that area is quite limited :-)

Thank you once again for the really useful library!
From my experience in 99% of the cases the best approach is to keep timestamps as a (micro)seconds offset from the beginning of the UNIX epoch (1970-01-01 00:00:00 UTC).
Internally PostgreSQL takes a similar approach and their timestamp with time zone is just a microseconds offset  from the beginning of the UNIX epoch. Ironically their "timestamp with time zone" doesn't really have its own time zone - it is just a global (per-server) timezone which is used by the server to parse the input string to a microseconds integer and format the microseconds integer to an output string.
Anyway, I guess in the future it is possible to add support in sqlpp for std::chrono::zoned_time (for those who want to specify the time zone explicitly) and std::chrono::local_time (if someone really wants to work with time in the local timezone).

Indeed, zoned_time looks like a very good match.


2.5.0 Update

@JR-Morgan was wondering about this pr and when you were thinking of merging in the 2.5 updates?

@JR-Morgan was wondering about this pr and when you were thinking of merging in the 2.5 updates?

Hi @haitheredavid,  I apologise this has taken a while. I've been swamped with Unreal Related issues.
This branch is ready to merge!
I'll give it some final testing and merge in tomorrow if you're happy with that?

@JR-Morgan no worries! I've been following along with the UE stuff, great to see the connector growing. No rush on my side for this merge. I was just checking so I can plan out some project work.


Fix issue#1939: Cast character to unsigned for comparison

Addresses https://github.com/nlohmann/json/issues/1939
Hi,
I ran into https://github.com/nlohmann/json/issues/1939 today with GCC 10.1 on arm. I didn't want to cause any duplicate issues so I appologize if recycling that issue is causing any inconveniences.
The issue is reproducible with the following example with GCC 10.1 on x86 using tag 3.7.3 as well as develop:
g++ -c -Wall -Wextra -Werror -std=c++11 -funsigned-char -I json/include -x c++ -o /dev/null - << EOF
#include <nlohmann/json.hpp>
EOF

...
json/include/nlohmann/detail/input/lexer.hpp:1370:24: error: comparison is always true due to limited range of data type [-Werror=type-limits]
 1370 |             if ('\x00' <= c and c <= '\x1F')
      |                 ~~~~~~~^~~~
cc1plus: all warnings being treated as errors

GCC 9.2 does not issue a warning for this on my machine.
User @jaredgrubb already proposed this fix in https://github.com/nlohmann/json/pull/1940 and it is indeed most likely the best solution:
Looking at the assembly generated by GCC, Clang and MSVC, c gets compared with unsigned ops on x86. On platforms where char is already unsigned, c gets compared with unsigned ops as well (duh). Casting c to unsigned would change nothing code-gen wise and it is still clear what is being tested for.

Alternatively a three-way-comparison function could be implemented. This would silence the warning:
constexpr bool three_way_compare(char lower, char value, char upper) noexcept {
    return lower <= value && value <= upper;
}

(I am not really a fan of that idea but I thought I'd at least mention it.)

Coverage remained the same at 100.0% when pulling 23051df2c71dd3f9698e01de3cbfab0034c6e50e on XyFreak:gcc10_type_limits into d70d06ae41d7b3241e16d366b3b67b5aebd39e9e on nlohmann:develop.

Thanks!

🔖 Release item
This issue/PR will be part of the next release of the library. This template helps preparing the release notes.
Type

[ ] ✨ New Feature
[ ] 🐛 Bug Fix
[ ] ⚡ Improvement
[ ] 🔨 Further Change
[ ] 🔥 Deprecated function

Description

Fix warning on comparing characters with different signs. #1939 #2144


Local timestamp and timezone are now copied to clipboard

based on https://github.com/mwaterfall/alfred-datetime-format-converter/pull/4#issuecomment-111596206

Sure thanks!

Thanks, I've made some additional modifications in my fork, not sure if you will be interested. Just let me know. Thanks.


feat(vm-logs): filter by accounts

Pull Request Test Coverage Report for Build 1144

122 of 182 (67.03%)  changed or added relevant lines in 11 files are covered.
22 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.03%) to 57.878%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/app/snapshot/recurring-snapshots/policy-editor/daily/daily-policy.component.ts
1
2
50.0%

src/app/snapshot/recurring-snapshots/policy-editor/monthly/monthly-policy.component.ts
0
1
0.0%

src/app/snapshot/recurring-snapshots/policy-editor/weekly/weekly-policy.component.ts
0
1
0.0%

src/app/vm-logs/pipes/base-path.pipe.ts
7
8
87.5%

src/app/vm-logs/pipes/parse-timestamp.pipe.ts
8
9
88.89%

src/app/shared/components/time-picker/time-picker.component.ts
0
2
0.0%

src/app/shared/services/base-backend.service.ts
1
3
33.33%

src/app/vm-logs/services/vm-logs.service.ts
23
26
88.46%

src/app/vm-logs/redux/vm-logs.actions.ts
41
61
67.21%

src/app/vm-logs/redux/vm-logs.reducers.ts
39
67
58.21%

Files with Coverage Reduction
New Missed Lines
%

src/app/vm/shared/vm-actions.service.ts
2
32.14%

src/app/shared/components/date-picker/date-picker.component.ts
2
48.18%

src/app/shared/services/base-backend.service.ts
18
64.59%

Totals

Change from base Build 1140:
0.03%

Covered Lines:
12313

Relevant Lines:
18502

💛  - Coveralls


Elvis integration?

Elvis is the Erlang style reviewer (https://github.com/inaka/elvis).
Many use elvis to enforce code style (me included). There are rules related to text formatting, such as:

Line Length;
No Tabs;
No Trailing Whitespace;
Operator Spaces.

I would like to have sourcer_indent integrated with elvis, i.e. understand formatting specific rules and apply them (e.g. add space after ,).

That's a good idea - I assume you mean that we could autocorrect some style issues that elvis reports, using elvis.config as guide/config? It's definitely a useful feature.
This will be something for later (for what I am concerned, I want to get the server going first). The code formatter right now is actually an indenter only, as we don't break any lines. Breaking lines at the right place is not very easy to do right.
As long as we don't have a full formatter, I would probably do these fixes as a separate command. If elvis is integrated in the editor/IDE, then the user can choose a "quick fix" operation for each one (or each type) and see and review the results right away.

ok, I'll do this as a separate command, maybe reusing some of (or depending on) the elvis core (https://github.com/inaka/elvis_core). What you think? something like erlang_ls -i --elvis.

There should be some configuration to specify which fixes to apply (to which elvis rules). Maybe "--elvis" to use info in config file and "--elvis ruleX,ruleY" will correct only ruleX and ruleY.
This functionality is in a bit of gray zone: elvis could just as well have a "--fix" parameter that would make the corrections. I don't think that the implementation will make use of any sourcer-specific code.

I agree, at least right now adding this directly to elvis might be a better idea.
I was trying to study indenter code a bit more, but there lots of things are going on. Any documentation/blog post? In theory if it's building AST, then said AST could be dumped as text following configured formatting rules. I feel like indenter and elvis probably overlap here to some extent.

Sorry, there is no documentation but the code. I didn't wrote any of it, so I confess I don't know much about how it works. I don't think it parses into an AST, because then we could have had a full formatter (like erl_tidy) and also it would only have worked on compileable code (which when editing is not always the case). I think it uses empiric rules and looks at the previous row to decide how to indent the current one (a bit simplified, of course).


Add custom matcher support

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.22862476,Cosmos:0.1554008,Automation:0.1437731'


update go runtime to latest stable version

currently the online wide service is using Go 1.4.2 , which is completely obsolete and will never get any critical security fixes.

Each major Go release obsoletes and ends support for the previous one. For example, if Go 1.5 has been released, then it is the current release and Go 1.4 and earlier are no longer supported. We fix critical problems in the current release as needed by issuing minor revisions (for example, Go 1.5.1, Go 1.5.2, and so on).
As a special case, we issue minor revisions for critical security problems in both the current release and the previous release. For example, if Go 1.5 is the current release then we will issue minor revisions to fix critical security problems in both Go 1.4 and Go 1.5 as they arise.

quoted from https://golang.org/doc/devel/release.html#policy
so please update the go runtime to latest stable version , thanks !

i strongly recommend upgrading go to latest stable version, otherwise user can not leverage Vendor mechanism and binary-only package

Thank you, I will upgrade it asap.

after golang upgraded, i suggest go get in wide to clone codes to vendor directory, its a better practise.

Thank you, I will try it.


Fix Texutebleeding

At ceratain agnles white stripes occour at the edges of blocks. They are nor visible all the time but still annoying.

has been fixed with latest update. There seem to be many possible reasons for texture bleeding, but it's mainly because of the TileAtlas. If mipmapping or multisampling is enabled OpenGL might blend/intepolate the texture at the edges so that pixels of the next texture get visible. Disabling multisampling with glDisable(GL_MULTISAMPLE) fixed this issue. found at: https://stackoverflow.com/questions/6488131/artifacts-when-enabling-4x-msaa-anti-aliasing-on-ipad-ios/6488465#6488465

Not entirely shure why, but the texture bleeding is back again... Might be good to check in an older version (before any kind of transparency was introduced) if it was really fixed or if I just didn't see it

.

currently no texture bleeding on Linux.
OpenGL version: 4.6 (Compatibility Profile) Mesa 23.0.4-0ubuntu1~22.04.1


tf.batch_matmul gives "Attempting to capture an EagerTensor without building a function"

Hi,
I'm getting an exception with Tensorflow.Binding.tf.batch_matmul the first time, and only the first time, it is called.
Packages:
SciSharp.TensorFlow.Redist 2.4.1
TensorFlow.Keras 0.4.0
TensorFlow.NET 0.33.0
Consider the following example:
    NDArray ta = new NDArray(new float[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }, new Shape(5, 2));
    NDArray tb = new NDArray(new float[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }, new Shape(2, 5));
    Tensor result2 = tf.batch_matmul(ta, tb); // this will throw an exception

The exception thrown is:
"Attempting to capture an EagerTensor without building a function." | Tensorflow.RuntimeError

stacktrace:
   at Tensorflow.ops.convert_to_tensor(Object value, TF_DataType dtype, String name, Boolean as_ref, TF_DataType preferred_dtype, Context ctx)
   at Tensorflow.OpDefLibrary.<>c__DisplayClass1_0.<_apply_op_helper>b__0(NameScope scope)
   at Tensorflow.Binding.tf_with[TIn,TOut](TIn py, Func`2 action)
   at Tensorflow.OpDefLibrary._apply_op_helper(String op_type_name, String name, Dictionary`2 keywords)
   at Tensorflow.OpDefLibrary._apply_op_helper(String op_type_name, String name, Object args)
   at Tensorflow.gen_math_ops.batch_mat_mul(Tensor x, Tensor y, Boolean adj_x, Boolean adj_y, String name)
   at Tensorflow.tensorflow.batch_matmul(Tensor x, Tensor y)
   at VPG.TF.Test2.LinearRegression2() in C:\somepath\TF\Test2.cs:line 240
   at VPG.Program.Main(String[] args) in C:\somepath\Program.cs:line 14

Now, if I run it first in a try-catch statement, the second time it is called it works:
    NDArray ta = new NDArray(new float[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }, new Shape(5, 2));
    NDArray tb = new NDArray(new float[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }, new Shape(2, 5));
    try
    {
        Tensor result = tf.batch_matmul(ta, tb); // this will throw an exception
    }
    catch
    {

    }
    Tensor result2 = tf.batch_matmul(ta, tb); // this works as expected

I'm new to this library, but so far I don't have this issue with other method calls. For example, I can run tf.matmul without an exception here.
I thought perhaps I misunderstood how to set things up, but then I still get this error if it is run this within a using for a GradientTape, or after I run a different function (such as a simple multiplication or tf.matmul). Any help you can offer would be much appreciated.
Lee

@LeeReidCSIRO Will be fixed for next release.

Many thanks


Is it okay to use autorun in hook/component?

I am wondering if it is okay to use autorun in a hook or component?
E.g. (this dumb example):
const store = observable({
    x: 0
});

const MyComponent = observer(() => {
    autorun(() => {
        if (store.x > 10) {
           alert('too much');
        }
    });

    return <div onClick={() => store.x += 1} />
});

My concern is about autorun end of life. Is it stopped when component unmounts?

Please see https://mobx-react.js.org/recipes-effects

Thank you! I missed that!


modal instead of dropdown for new FB creation form

this PR fixes #760
When user clicks "configure fidelity bond" modal appears like this:

Whole flow to create new bond works fine on my setup.
Open to any kind of suggestions.
PS: Cross Button at top-right functions same as cancel button. I've kept it as it will be useful for later UI design.

@barrytra Nice!
Works exactly as expected. Great work! :rocket:

UI might need small adaptions to have the header consistent with other modals (renew bond, fee settings, seed modal, etc.):

:camera_flash:

i.e. by using:
<rb.Modal show={isExpanded} animation={true} backdrop="static" centered={true} onHide={() => setIsExpanded(false)}>
<rb.Modal.Header closeButton>
  <rb.Modal.Title>{t('earn.fidelity_bond.create_fidelity_bond.title')}</rb.Modal.Title>
</rb.Modal.Header>
<rb.Modal.Body>
[...]
</rb.Modal.Body>
</rb.Modal>

See file SpendFidelityBondModal.tsx for examples! :pray:

I think the custom CSS styles are also not needed! :muscle:

Can we find a better name for variable isExpanded? e.g. showCreateFidelityBondModal

The sprite does not need to change from "plus" to "caret-up" anymore, when isExpanded is true

What do you think?

Yes.. i agree to suggestions and they have been implemented.
My only concern to use customised CSS was because even in light mode the header appears to be dark.

But its okay if other modals have the same flow.
please have a check

Yes.. i agree to suggestions and they have been implemented. My only concern to use customised CSS was because even in light mode the header appears to be dark. But its okay if other modals have the same flow. please have a check

Good thinking! Imho, it is okay and better to stay consistent. Header color in light mode can be changed in a different PR, if needed!
tACK. Thanks @barrytra :muscle:

Ahh!! I get it. It was actually placed according to dropdown. This should be fixed.
What do you think shall it be shown inside the modal itself?

Also can you suggest a way to produce this error??


Add "full width" and "loading" buttons

This PR includes two new button modifiers:
1. Full width buttons
.coop-btn--full-width
2. Loading spinner buttons
.coop-btn--loading
They work in combination with all other buttons (default, grey, white, small).

(Ignore the "jump" as the GIF restarts, it's super-smooth in real life.)
I've also added supporting <form class="coop-form"> and <div class="coop-form__row"> wrapping markup to the button examples, so it's easier to see how they'd work in context:

We've had these styles in production for a while on Co-op Account sign in:


j2cl-java-util-Currency-it-junit-test

delete CurrencyProvider
POM depend on j2cl-java-util-currency-annotation-processor
Currency added @LocaleAware (triggers annot processor)

Closes #20


bluez: build missing bluetooth utilities

Motivation for this change
Some bluetooth developer tools namely hid2hci, rfcomm, sdptool, etc are not built by default.
Some of them do not work out of the box (sdptool requires --compat) but the others seem just work.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Arch wiki says they have separate packages namely bluez-utils-compat.
It should have an additional output, though it can be another PR.


OCM-2620 | build: Add files for building releases from Konflux

Added Dockerfile to be used from pipeline builds for release generations using Red Hat keys. Also added some needed modifications and removed old files related to previous release workflow.

/hold

/unhold
/cc @nirarg

/retest

/approve
/lgtm

/test rosa-sts-advanced-critical-high-presubmit

/override rosa-sts-advanced-critical-high-presubmit


[AutoRelease] t2-workloadssapvirtualinstance-2024-10-09-03643(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5540
Recording test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=4807
BuildTargetingString
azure-mgmt-workloadssapvirtualinstance
Skip.CreateApiReview

issue link:https://github.com/Azure/sdk-release-request/issues/5540


Add Seabin project

Appended ReadMe.md with description about the Sea Bin Project.

Hey @JayantCh , thank you for your contribution, though there was already a pull request adding this company. Closing as this is a duplicate.


Minor formatting and comment changes

Reformat some comments.
Reorder includes to match spark style guidelines.
Fix return type comments.

No functional changes.

Based on https://github.com/databricks/spark-tfocs/pull/18

LGTM


Upgrade lato fonts to v16.

As title

Codecov Report

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

@@            Coverage Diff             @@
##           master    #9498      +/-   ##
==========================================
- Coverage   41.53%   41.52%   -0.02%
==========================================
  Files         572      572
  Lines       74757    74757
==========================================
- Hits        31053    31044       -9
- Misses      39852    39864      +12
+ Partials     3852     3849       -3

Impacted Files
Coverage Δ

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

models/webhook.go
69.39% <0%> (-1.07%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

models/repo_list.go
73.14% <0%> (-0.93%)
: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 3e166bd...a223bd6. Read the comment docs.


🧑💻 : Adding a search bar in header

:red_circle: Title : Adding a search bar in header
:red_circle: Enhancement Aim : Make it easily accessible for user to search for game of their choice
:red_circle: Changes : Will be adding search bar in header for easy accessibility
Screenshots 📷
NA

✅ Details to Include When Taking the Issue:
Name : Shubha Singh
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.): GSSOC

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

someone already working on it


What does thst optional tag mean in the return values ?

What happened?

Code usage
No response
Log output
No response

Hi @P-Y-R-O-B-O-T . That means the values can be None.


Auto cut cards title with css

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after), along with a short summary of changes:
Issue
Fixes #453
Changelog:
Please enter each change as a new bullet point
Test URLs:

Original: https://www.sunstar.com/healthy-thinking/lets-talk-oral-health
Before: https://main--sunstar--hlxsites.hlx.live/healthy-thinking/lets-talk-oral-health
After: https://issue-453--sunstar--hlxsites.hlx.live/healthy-thinking/lets-talk-oral-health

Library

[ ] New Blocks introduced in this PR
If yes, please provide details below
Block Name : For e.g. cards

[ ] Documented in sidekick library

[ ] New variations introduced in this PR
Variation Name :  For e.g. cards (grid)

[ ] Documented in sidekick library

[ ] New mixins introduced in this PR
Mixin Name :  For e.g. compact, white

[ ] Documented in sidekick library

Hi @ehrro, as we talked in the last meeting. This issue has been fixed, please check and find a good timing to merge it to your repository.

@sdmcraft Okay, I heard Dejan would sync the change to Sunstar repository before, let me move it.

I'd suggest moving this PR against https://github.com/sunstar-global/sunstar


Make Paged search optional on LdapSearchService

Affected library

[x] Visus.LdapAuthentication
[ ] Visus.DirectoryAuthentication

Environment

Library version: 1.13.0
OS: Windows 11
.NET: 8
LDAP Server: OpenLDAP

Summary
Paged search extension may not be available on the LDAP server which would make LdapSearchService useless.
What are you doing?
Calling getUsers method on LdapSearchService with only the filter parameter.
What is the problem?
Not having this extension is not a problem until you perform a search using LdapSearchService which is making an explicit call to the paged search implementation.
https://github.com/UniStuttgart-VISUS/Visus.LdapAuthentication/blob/f9d4a31ab81823a293e932085cd45a607097657d/Visus.LdapAuthentication/LdapSearchService.cs#L247
What behaviour did you expect?
This feature should be optional, to use it if available/enabled Options side, then fall back to regular search otherwise.

My understanding is that you get an empty result set due to an LdapException in https://github.com/UniStuttgart-VISUS/Visus.LdapAuthentication/blob/f9d4a31ab81823a293e932085cd45a607097657d/Visus.LdapAuthentication/PagingExtensions.cs#L188C30-L188C43 Is that correct?
Can you retrieve the ResultCode from this exception? I am considering two solutions atm, one being the paging not happen if LdapOptions.PageSize < 0 and the second is the implementation recovering from that either by checking the feature support or retrying.

Problem affects DirectoryAuthentication as well.

1.14.0 and 0.12.0 now allow for setting the page size to zero, which will disable paging. However, I am not sure what OpenLDAP will do if you have too many results. On AD, LdapAuthentication will silently truncate the results, whereas DirectoryAuthentication will fail (I could not find a way to prevent this failure, because the request itself will fail).


Error type 3 Error: Activity class {} does not exist

Hello, when trying to run the app on android I get the second error:
I'm using Windows 10.

Thank you!

It looks like there's a space in the url https: //services  can you check in android\gradle\wrapper\gradle-wrapper.properties?  Something with windows path related, ie get
Try remvoing the \ in the line distributionUrl=https\://services.gradle.org/distributions/gradle-4.1-all.zip

After searching around it appears that the \ is correct. You can try to open the android folder with android-studio and running from there.


Cookies support

i have a use case for providing cookies as part of the http flow. if there are no objections, i'll submit a PR. i know it's probably not a common one.

hi @mattnworb  @davidxia
i've made the changes to support headers generally, could you take a look please?
https://github.com/spotify/docker-client/pull/362

fixed by #362


add attribute inspection + test (#71)

Hello,
I propose a comprehensive solution to show values of attributes as suggested in #71.
Nested attributes via subscripts or functions call (e.g. a.b[42].c().b.c) could be easily managed by completing  the function get_nested_attributes I wrote.
I regenerate the test files as it add info for the attributes used in the testcases.

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#71: Attribute inspection

IssueHunt has been backed by the following sponsors. Become a sponsor

There's already a PR for this in #73, I think it's not visible enough from #71 which might be why it's got an issuehunt bounty on it.

Indeed, but thank you for the PR nonetheless :)

Nice, thank you :)


Remove the use of set-output in GHA and upgrade actions using deprecated deps

Fixes #247
The create-release and upload-artifacts actions aren't maintained anymore and I've switched to the recommended maintained actions. This simplifies the workflow as we can create the release and upload artifacts in a single step.
I've checked that the @actions/core dependency was at least 1.10.0 as indicated in GH's blog post.
Successfully tested this by tweaking the jar release workflow trigger to be test*.*.* instead of v*.*.*:

The release was created and the jars.zip artifact was successfully attached to it.
Here's the run output: https://github.com/k8ssandra/management-api-for-apache-cassandra/actions/runs/4102194244/jobs/7074849518

I checked all the workflows, this is the only one that needed an update.


foxglove_msgs: 3.0.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository foxglove_msgs to 3.0.0-1:

upstream repository: https://github.com/foxglove/schemas.git
release repository: https://github.com/ros2-gbp/ros_foxglove_msgs-release.git
distro file: rolling/distribution.yaml
bloom version: 0.11.2
previous version for package: 2.2.0-1

foxglove_msgs
* Add a frame_id field to LocationFix
* Update RawImage encodings
* Add CompressedVideo schema
* Improve doc on PointAnnotation field outline_color

Holding for an upcoming Rolling sync.


fix: scroll in list component

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


Node self connectinos

I started 2 local nodes, ergo-local-1 and ergo-local-2, but /peers/connected API route shows 3 active  connections:
[
  {
    "address": "",
    "lastSeen":<PHONE_NUMBER>403,
    "name": "ergo-local-1"
  },
  {
    "address": "",
    "lastSeen":<PHONE_NUMBER>403,
    "name": "ergo-local-1"
  },
  {
    "address": "",
    "lastSeen":<PHONE_NUMBER>403,
    "name": "ergo-local-2"
  }
]

In logs there are messages, that node really send connections to self:
17:47:44.859 INFO  [or.default-dispatcher-661] s.c.n.PeerConnectionHandler - Send message MessageSpec(22: RequestModifier) to bench1/<IP_ADDRESS>:37326
17:47:44.859 INFO  [or.default-dispatcher-661] s.c.n.PeerConnectionHandler - Received message MessageSpec(22: RequestModifier) from bench1/<IP_ADDRESS>:9003
17:47:44.860 INFO  [tor.default-dispatcher-10] s.c.n.PeerConnectionHandler - Send message MessageSpec(33: Modifier) to bench1/<IP_ADDRESS>:9003
17:47:44.860 INFO  [or.default-dispatcher-658] s.c.n.PeerConnectionHandler - Received message MessageSpec(33: Modifier) from bench1/<IP_ADDRESS>:37326

relates to #296

will be finally fixed with #382


What is the proper way to use FromJSONAsync when the schema has references?

Hello,
I'm struggling with this: I have 2 schema files embedded as dotnet resources, and one schema is referencing the other:
BooleanMemberEvent content (available as embedded resource):
{
  "type":"object",
  "additionalProperties": false,
  "properties": {
    "time": {
      "type": "string"
    },
    "memberSource": {
      "$ref": "MemberSource"
    },
    "value": {
      "type": "boolean"
    }
  }
}

MemberSource content (available as embedded resource):
{
  "type":"object",
  "additionalProperties": false,
  "properties": {
    "id": {
      "type": "integer"
    },
    "device": {
      "type": "string"
    },
    "ip": {
      "type": "string"
    }
  }
}

I'd like to use FromJSONAsync on both resources to generate the csharp code of the types they define.
I wrote the code below, but during the task FromJsonAsync I encounter the error: Could not resolve the JSON path 'MemberSource' because no document path is available.
Looking at the source I saw that it is be because I pass null as 2nd parameter of the constructor of CSharpTypeResolver. How should I construct the object passed here ? I saw in CSharpGenerator constructors that by default the schema is passed here.  How can I do that?
Many thanks for your help!

    class Program
    {
        static internal string outputDir = "../generated/dotnet/";
        static internal string inputDir = "../jsonschema/";

        static internal CSharpTypeResolver resolver = new CSharpTypeResolver(new CSharpGeneratorSettings(), null);

        static void Main(string[] args)
        {
            IDictionary<string, JsonSchema4> generators = new Dictionary<string, JsonSchema4>();

            Assembly thisExe = typeof(Program).GetTypeInfo().Assembly;
            foreach (string name in thisExe.GetManifestResourceNames())
            {
                JsonSchema4 schema = resourceToSchema(name);
                generators.Add(name, schema);
            }
            resolver.AddGenerators(generators);
            foreach (KeyValuePair<string, JsonSchema4> pair in generators)
            {
                var name = pair.Key;
                var schema = pair.Value;
                string code = schemaToCode(schema);
                writeGeneratedSource(name, code);
            }
        }

        private static string schemaToCode(JsonSchema4 schema) {
            var generator = new CSharpGenerator(schema, new CSharpGeneratorSettings(), resolver, new object());
            return generator.GenerateFile("");
        }

        private static JsonSchema4 resourceToSchema(string resourceName)
        {
            Assembly assembly = typeof(Program).GetTypeInfo().Assembly;

            using (Stream stream = assembly.GetManifestResourceStream(resourceName))
            using (StreamReader reader = new StreamReader(stream))
            {
                string schema = reader.ReadToEnd();

                var task = JsonSchema4.FromJsonAsync(schema);
                task.Wait();
                return task.Result;
            }
        }

        private static void writeGeneratedSource(string name, string content) {
            string outputFile = outputDir + name + @".cs";

            // create the directory if it doesn't exist
            Directory.CreateDirectory(outputDir);

            File.WriteAllText(outputFile,
                              content);
        }
    }

At this point I took a different approach, relying on files, it seems to me that there is something missing to be able to use FromJsonAsync with multiple schema entities + references between them. This is covered by FromFileAsync however so I have a workaround.

yes but my wish was to be able to work without any file here :)

I think you can also just use myBaseDir + "/myfakename.json" as path where myBaseDir is where the refs are looked up and myfakename.json does not exist ...

In my case the refs are not files, the refs are only available as embedded resources. I just reworded my initial text to be more precise about this.

Then you have to implement an own reference resolver like:
    public static async Task<JsonSchema4> FromJsonAsync(string data)
    {
        Func<JsonSchema4, JsonReferenceResolver> referenceResolverFactory =
            schema => new MyJsonReferenceResolver(new JsonSchemaResolver(schema, new JsonSchemaGeneratorSettings()));

        return await FromJsonAsync(data, null, referenceResolverFactory).ConfigureAwait(false);
    }

Where MyJsonReferenceResolver is your custom resolver (inherits from JsonReferenceResolver and overrides ResolveFileReferenceAsync())...

I see thanks for the support!

You're welcome...


PostCss not applied

Hello
I have the problem that apparently no transformation is done on the css. I tried for several hours but I don't find the reason. The css module changes are applied (you can see the hash) however postcss modules are not applied. I tried with several things which should go with precss (e.g. variables) but it does not work, even worse the rest of the css is not processed, because it is faulty.
Webpack configs:
loaders: [
  {
    test: /\.css?$/,
    loaders: [ 'style', 'css?modules&importLoaders=1&localIdentName=[local]---[hash:base64:5]', 'postcss' ],
    include: path.resolve(__dirname, '../src')
  }
],
postcss: function () {
  return [
    require('precss')(),
    require('autoprefixer')(),
    require('cssnano')()
  ]
}

Do you use import statements?

Yes, the css modules work and the elements are styled, however there are just no PostCss transformations done. When I then try to use syntax from precss (e.g. mixins, variables, ...) nothing is in the resulting css. If I put the precss stuff before existing "normal" css then some if it also no longer works.

Can you should some example of your CSS?
Also you don’t need cssnano in postcss, css-loader already use it.

Short trivia to the css: I'm trying to make some styled buttons with React and PostCss. I figured I could use precss mixins to minimize the repetition. The css how it is now works, but if you add for example a variable (e.g. $blue: #056ef0;) before .default the result will be that all the styling from .default is no longer applied while the color from .primary is still there. The reason that this works is because of the browser support for the variables (--primary-color).
@import url('../../styles/variables.css');

/* $blue: #056ef0; */

.default{
    border: none;
    color: white;
    padding: 15px 32px;
    text-align: center;
    text-decoration: none;
    display: inline-block;
    font-size: 16px;
    margin: 4px 2px;
    cursor: pointer;
}

.primary{
    composes: default;
    background-color: var(--primary-color);
}

.success{
    composes: default;
    background-color: var(--success-color);
}

.info{
    composes: default;
    background-color: var(--info-color);
}

.warning{
    composes: default;
    background-color: var(--warning-color);
}

.danger{
    composes: default;
    background-color: var(--danger-color);
}

You need to use postcss-import. (use v7 as v8 is buggy and will be replaced soon)


[Feat]: FAQ page added successfully

Description
This pull request implements a dedicated FAQ page at the bottom of the page, aimed at enhancing user engagement and support. The page provides clear, concise answers to commonly asked questions, thereby streamlining the user experience.

Fixes #157

Type of change
This change introduces a new feature (in this case, the FAQ page).
Please delete options that are not relevant.

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

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] I have maintained a clean commit history by using the necessary Git commands
[x] I have checked that my code does not cause any merge conflicts

Screenshots (if applicable)
https://github.com/user-attachments/assets/5b464383-54ae-400b-a81b-a27af0fc6023

Nice... Also add ques and answers according to our platform . @Someshog

Nice... Also add ques and answers according to our platform . @Someshog

Done! The latest commit has successfully added the Q&A's.


Add permutation tests and fix bugs

Hello @dsaxton! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file resample/tests/test_permutation.py:

Line 3:80: E501 line too long (80 > 79 characters)

Hello @dsaxton! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file resample/tests/test_permutation.py:

Line 3:80: E501 line too long (80 > 79 characters)


Run set-interrupts on GCP when using local SSD

From documentation, ubuntu need to run set-interrupts.sh manually.
https://cloud.google.com/compute/docs/disks/local-ssd#optimize_your_local_ssd_performance

We should consider this when we add a "--optimized" run mode.  For the default we should not change interrupt targeting.


Upgrade k8s.io/apimachinery from 0.21.3 to 0.23.4

The existing PR https://github.com/jaegertracing/jaeger-operator/pull/1760 is failing with these errors:

seems appear here too k8s.io/api #1759

@frzifus @rubenvp8510 This has been updated to 0.23.5, so this can be closed


Implement reflection-based automatic JSON parsing/serialization

Userver actively uses boost::pfr, which allows one to access an aggregate field by index, get total number of fields in an aggregate and a lot of other very handy things (grep pfr::for_each_field across the source tree).
This works wonders when one needs to map an aggregate to some kind of its binary representation, say, in database protocols (ClickHouse/Pg/MySQL), where there is a fixed format with fixed amount of fields, mapped 1 to 1. However, there was no way to get pfr working with JSON without adding some additional metadata: pfr, being a thing of beauty, unfortunately didn't have a way to get a field name, which is needed for textual formats (JSON/Yaml/etc.).
Well, this is no longer the case: with PRF 2.2.0 being just landed in userver, it IS possible to get a field name of an aggregate at compile time (with C++20): https://www.boost.org/doc/libs/master/doc/html/boost_pfr/tutorial.html#boost_pfr.tutorial.reflection_of_field_name, and the JSON/Yaml/Bson/etc. limitation of applying pfr to is lifted now.

It would be nice to have formats::Parse/formats::Serialize analogues built on top of this functionality, and from a distance i'd suggest implementing it in this order:

Add the functionality in question to an aggregate consisting of primitive types: std::string, integer types, floating-point types, bool:

struct Data {
    std::string name;
    int value;
    bool is_something;
    double amount;
};

Expand the functionality to work with std::optional<primitive type> (and std::variant if you are feeling adventurous)
Expand the functionality to work with nested aggregates (this adds JSON object, basically)
Expand the functionality to work with arrays of all the above (this adds JSON array)

Implementing this for std::chrono:: types could be quite handy as well, but that might potentially require some policies added.
Would be nice to also have a room to hook into the auto-generated parsing for some types that aren't recognized out of the box (say, if one wants to have everything automatic but for one field, which requires some custom handling).
I would also suggest to share your design ideas before jumping on the implementation: this could potentially save a lot of time and prevent discovering design-level problems after everything is already implemented.
P.S. All of this should be ifdef-ed under C++20 availability, of course, and https://github.com/userver-framework/userver/blob/86b1c6c0307be0c0ce76b009a9db90b8667a4bd2/cmake/SetupEnvironment.cmake#L19 could be used for testing.

This is a really nice, fun and very useful feature, which would not only allow one to improve template metaprogramming and design skills, but also earn a lot of appreciation from the userver-community, so don't hesitate to give it a try, otherwise we may implement it ourselves at some point :wink:

I know that there is an implementation of traversing structures with names in boost pfr, but in my opinion it is not very good to lay on it, because:

it is tied to the intrics of compilers and can potentially break at any moment
it does a lot of instantiations on one structure, it seems to me that it will potentially slow down a lot in a large project. What's the alternative? I have an implementation on NTTP, traits, templates, constexp function and macros.
At the moment, I have implemented basic arrays and restrictions on them, string and restrictions on them, structures, as well as the auto parse/serialize of handlers, auto-generation of openapi description from all this, wrote the necessary things for the postgres driver. Although my implementation is made solely out of interest and my own needs, but it seems to me that it can be considered. Part of the code is covered by tests.
Example of view:
Type in body: https://github.com/sabudilovskiy/timetable_vsu_backend/blob/new_openapi/src/models/lesson_filter/type.hpp
Declaration request and response: https://github.com/sabudilovskiy/timetable_vsu_backend/blob/new_openapi/src/views/timetable/declarations.hpp
View: https://github.com/sabudilovskiy/timetable_vsu_backend/blob/new_openapi/src/views/timetable/view.cpp
The openapi scheme is generated at the start of the service based on the information from the config (since the endpoint view and its path are set there), and then it is given by /openapi. There is no difficulty for me to change this behavior. This behavior is tested here:
https://github.com/sabudilovskiy/timetable_vsu_backend/blob/new_openapi/utests/openapi/doc/serialize/basic_path.cpp
How do you look at such an implementation? I would be interested to at least hear an opinion.

@itrofimow

@sabudilovskiy your approach is very interesting, but it is a totally different approach that requires explicit markup in code. We'll take a closer look, which approach suits our needs better

@sabudilovskiy your approach is very interesting, but it is a totally different approach that requires explicit markup in code. We'll take a closer look, which approach suits our needs better

Initially, this was done based on the fact that there was no other way to provide names for the fields.  However, now everything is different.  I think it is possible to remove the markup in most cases, however restrictions on openapi, an explicit indication of where the field will be placed in the request (header/body/cookie) would be easiest to do this way.  In the near future I want to use boost.pfr.name and standard types so that I don't have to markup anything in most cases (however markup is required in requests and responses).  Or is there some other way for requests and responses?

https://github.com/linuxnyasha/Serialization-userver_formats I wrote this. Could this be added to userver in the future? There's not much here yet, but it can all be added.


feat(artifacts): set ttl(time to live) for artifact versions

Fixes
https://wandb.atlassian.net/browse/WB-14005
Corresponding yea-wandb PR: https://github.com/wandb/yea-wandb/pull/119
Description
Enable artifact TTL

🤖 Generated by Copilot at a327520
Added a new feature to wandb that allows users to set a time-to-live (ttl) duration for artifact versions. This feature required modifying the Artifact, ArtifactSaver, and Api classes, as well as the GraphQL queries and variables that communicate with the backend API. Also changed the Python formatting settings in .vscode/settings.json to use the ms-python.black-formatter extension.
Testing
Tested locally making sure the following all worked. At the same time checking the mysql database that artifact_ttl_rules is  updating correctly especially for disabled vs inherited ttl setting.
from datetime import timedelta

# Setting regular ttl
run = wandb.init(project="user_analysis")
art = wandb.Artifact("user_data", type="dataset")
art.ttl = timedelta(days=30)
run.log_artifact(art)

# ttl disabled
art = wandb.Artifact("user_data", type="dataset")
art.ttl = None
run.log_artifact(art)

# inherit artifact
art = wandb.Artifact("user_data", type="dataset")
art.ttl = wandb.ArtifactTTL.INHERIT
  try:
      print(art.ttl)
  except:
      print("failed like expected")
run.log_artifact(art).wait()
print(art.ttl)

# ttl not set
art = wandb.Artifact("user_data", type="dataset")
run.log_artifact(art)

# Set ttl for existing artifacts
art = run.use_artifact("estellazx/user_analysis/user_data:latest", type="dataset")
print(art.ttl) # prints 30 day, 0:00:00
art.ttl = timedelta(days=365*7) # change the ttl
art.save()

# Code for sanity checking and also testing API
api = wandb.Api(overrides={"entity": "estellazx", "project": "user_analysis"})
for v in api.artifact_versions("dataset", "user_data"):
     print(v.version, " ", v.ttl, " ", v._ttl_is_inherited)

Notes: introduced ArtifactTTL.Delete to clearly indicate to backend that the artifact TTL is to be deleted.
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at a327520

ttl_duration
A new parameter for artifacts
Autumn of versions

I'm confused about what _ttl_is_inherited is doing here. It doesn't appear to be used at all; if that's intended, let's just remove it. @moredatarequired

So self._ttl_is_inherited is purely for when artifact collection ttl comes to play.
If collection.ttl = 100, then an artifact version will have art.ttl == 100, art._ttl_is_inherited == True. This means once we save the artifact, we don't save art.ttl == 100 and add another entry into artifact_ttl_rules because we check that its ttl value is inherited from the collection using _ttl_is_inherited.
No this isn't needed in this PR but the situation is a backend with ttlDurationSeconds is already deployed but incorrectly handles the new INHERIT enum

Fetch the TTL in _populate_after_save as well, since the inherited default is populated on the server.
Let's expand the test plan to cover all the possible cases.

@szymon-piechowicz-wandb fix for save() not updating artifact.ttl
with wandb.init(entity="estellazx", project="ttl") as run:
    newArt.ttl = wandb.ArtifactTTL.INHERIT
    try:
        print(newArt.ttl)
    except:
        print("failed like expected")
    newArt.save()
    newArt.wait()
    print(newArt.ttl)

changes: https://github.com/wandb/wandb/pull/5859/commits/a98886d5f2c5125b4b806c9653e110c922a7abf6

I did some more testing (with the server after https://github.com/wandb/core/pull/16442):

Here's what happens for each server version:

old server:

get:

_ttl_duration_seconds = None
_ttl_is_inherited = True

set: no op

server after first PR:

get:

_ttl_duration_seconds = value when value rule present, None otherwise
_ttl_is_inherited = True

set: no op

server after second PR:

get:

_ttl_duration_seconds = value when value rule present, None otherwise
_ttl_is_inherited = False when rule present, True otherwise

set: yes op

Testing with the server after the first PR:

Testing with the server before TTL:
Same as for the server after the first PR (after fixing the bug from the comment above).


New Scraper: Mayor- Public Art Commission

Create a new scraper for Mayor Public Art Commission
Website: https://mayors-office.cityofomaha.org/public-art-commission

seems to update after meetings, a few in august
https://mayors-office.cityofomaha.org/public-art-commission
not really scrapable


No issue hovers in code editor

(Previously discussed in #2817 and promoted to a separate issue by @alexr00.)

Extension version: v0.30.0
VSCode Version: 1.60.2 (NOT Insiders)
Commit: 7f6ab5485bbc008386c4386d08766667e155244e
Date: 2021-09-22T11:59:27.195Z (1 wk ago)
OS: macOS 11.6 (20G165)

From https://github.com/microsoft/vscode-pull-request-github/blob/main/documentation/IssueFeatures.md I'm missing issue hovers in the code editor (for multiple language I've tried including creating issues).
My setup:

authenticated with GitHub
working on my forked origin (issues disabled)
properly configured upstream
no Python (not ignored in settings)

I can repro, thank you for all the details and for the recording. Looks like it's related to files that haven't yet been committed, as I can only repro in such files.

Oh great catch! None of the committed files at the time could be commented 😅

I just encountered this issue again in a .nu file. Is it only available in certain syntaxes? Or only remotes?
Repro steps:

Create a Nushell script
Add a comment like # absolute path workaround for nushell/nushell#11465
No link or hover


Feet to meters conversion

I have noticed that when grid coordinates are entered as US Survey feet in the the online data submittal, the system automatically converts the coordinate from US Survey Feet to Meters when generating the Monument Record Sheet.  I checked the converted coordinates, and it appears that the conversion factor used by the system is slightly off what it should be.  I've attached the sheet previews that resulted first from entering the coordinates below first as US Survey Feet, and then as Meters using the conversion factor 1 m = 39.37/12 usft.  The resulting grid coordinates computed from the online system are different from my calculations by 0.085m in the Northing, and 0.016m in the Easting.
Northing                 10435569.502
Easting                  1646720.855
Elevation                7091.206
Latitude (DMS)           N38°17'43.44768"
Longitude (DMS)          W111°28'40.92344"
Ellipsoid Height         7032.649

I suspect that the program uses a truncated value of the conversion ratio 39.37/12, but that it wasn't carried out to a sufficient number of decimal places to accurately convert values in the range of 10 million feet.  Please advise.

The conversion units are in this extension class
The expected conversion value is
39.37/12 = 3.280833333
The method on line 12 is using which matches the expected conversion value
return feet / 3.280833333f;
There are a few other places to look into for possible precision reduction.

The conversion between float and double. I will test this with actual values.
The writing to the PDF
The database field type float(8) @seanfernandez/@mheagin is that enough precision for the 10 million feet thing from above?

Switching from the static 3.280833333 to 39.37/12 gets the numbers a bit closer to what Rodney is expecting.

Yes, we only want 3 decimal places displayed on the sheets, but need to go
out to probably closer to 10 or 11 decimal places for the conversion
calculation.
We want to be within a centimeter of the geographic coordinate after the
conversion from US survey foot to meters. The decimeter error shown by the
surveyor is too much.
Thanks!
Sent from my iPhone
On Apr 30, 2018, at 11:00 AM, steveoh<EMAIL_ADDRESS>wrote:
Switching from the static 3.280833333 to 39.37/12 gets the numbers a bit
closer to what Rodney is expecting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/agrc/PLSS/issues/21#issuecomment-385461528, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJeiDqvZ8zFZDbSdHeurQn5KRh0uqAkqks5tt0M1gaJpZM4Tsx3p
.


progression: fixes to Recipe's IsActionBlocked

Please review changes per commit 😄
The first 2 commits contain the fixes:

If m_requireOnlyOneIngredient is set, the recipe should be craftable using any of the Piece.Requirement listed in m_resources. However, this was not the case because Recipe's IsActionBlocked was checking all requirements even when m_requireOnlyOneIngredient was set. We fix this with a special case to handle m_requireOnlyOneIngredient.
If any prefab is missing when checking resource prefabs (m_resItem is null when any Piece.Requirement from m_resources is checked), then Recipe's IsActionBlocked automatically allows the action.

Third commit is a proposal for a functional-style refactor, but can be dropped if you prefer to stick with an imperative approach.

For point 1: What case is broken for using m_requireOnlyOneIngredient? Would that be the fish recipes? Can you give me an example so that I can reproduce the issue?
For point 2: Good catch, I will want to merge that change.
I do not want to include the refactor since I like to keep this project's code simple enough for beginner programmers to easily read. Simple loops and checks I think are easier to understand and transfer from other programming languages.

For point 1: What case is broken for using m_requireOnlyOneIngredient? Would that be the fish recipes? Can you give me an example so that I can reproduce the issue?

In vanilla that's indeed only the fish recipe, and since the mod does not impose restrictions on its requirements it does not matter for this one, however it will not work for any modded recipe of this type where any ingredient requires a key you haven't unlocked yet.
In our case the issue was with the Sail Ink recipe coming from Balrond Shipyard, but to reproduce without installing this mod you could just set requireOnlyOneIngredient: true on some recipes of your choice via WackysDatabase: without the fix, crafting will be blocked if any ingredient requires a key you haven't unlocked yet.

I do not want to include the refactor since I like to keep this project's code simple enough for beginner programmers to easily read. Simple loops and checks I think are easier to understand and transfer from other programming languages.

OK, removing it! I was actually more thinking about code consistency, hence why I set the commit apart. FWIW, in my experience beginners are more likely to understand functional-style than imperative-style due to it being closer to intuitive thinking, especially since functional-style is now the standard in most beginner languages (e.g. Python) 😄

Implemented this a bit differently in version 0.2.14, thanks again for the report! I will need to revisit this to make sure players can't use materials they are not allowed to yet even if the item is unlocked for crafting.


gender neutral pronouns per #31439

the unedited files in the issue were all references to media codecs, should not require changes

@ryandcoates : Thanks for your contribution! The author, @ika-microsoft, has been notified to review your proposed change.

@ryandcoates thank you very much for taking the time to create this changes.
I accept all. I'm unauthorized to merge this pull request.
If no one merges it in the next few days I will try and find who can.
Regards, Ika.


fix(GUI): log known user errors when querying S3 for updates

We send an HTTP request to S3 to determine the latest available version.
There are various error that can happen that we don't have control over
(like ETIMEDOUT).
The current approach is to whitelist certain errors and pretend there is
no update available, however this commit improves that whole situation.
Instead of swallowing these errors, we throw a user error from the
function that determines the latest available version. From the
application code, we check if that function throws a user error, and if
so, instead of showing it to the user, we log a mixpanel event and carry
on.
This change is motivated by the latest reporter error we can't do
anything about: UNABLE_TO_GET_ISSUER_CERT_LOCALLY.
Fixes: https://github.com/resin-io/etcher/issues/1525
Change-Type: patch
Changelog-Entry: Fix UNABLE_TO_GET_ISSUER_CERT_LOCALLY error at startup when behind certain proxies.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Huh, why go to the trouble of creating a user-error and then swallowing it again afterwards? Wouldn't it be easier to just log the mixpanel event directly?  (am I missing something obvious?)

@lurch The s3 packages module runs in the SDK, while the analytics are GUI specific, therefore we have to deal with it from the GUI code.


fix: correct ingress details on custom namespaces

Where a namespace is specified for an environment,
custom ingress settings (ingress.domain & ingress.nameSpaceSubDomain)
should be populated in jx-values for the namespaces
fixes https://github.com/jenkins-x/jx/issues/8514

/assign @msvticket

/ok-to-test

thanks all!


[VSIM-34] increased tomcat session timeout via a lineinfile module call

should templatize this at some point. but works for now.


The login is broken

Checklist

[X] I confirm there are no unresolved issues reported on the Chocolatey Status page.
[X] I have verified this is the correct repository for opening this issue.
[X] I have verified no other issues exist related to my problem.
[X] I have verified this is not an issue for a specific package.
[X] I have verified this issue is not security related.
[X] I confirm I am using official, and not unofficial, or modified, Chocolatey products.

What You Are Seeing?

What is Expected?
N/A
How Did You Get This To Happen?
N/A
System Details

Operating System:
Windows PowerShell version:
Chocolatey CLI Version:
Chocolatey Licensed Extension version:
Chocolatey License type:
Terminal/Emulator:

Installed Packages
N/A

Output Log
N/A

Additional Context
N/A

Its not clear what this issue is. That's one of the reasons we ask for steps to reproduce it.


HARVESTER: fix cluster-member not schema

Summary
Fixes #

https://github.com/harvester/harvester/issues/3020

Occurred changes and/or fixed issues

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

yes, this is not a big problem and will be fixed in the next release. reduce code changes


ECONNREFUSED, Authhost is in another docker container

I am working with 3 docker files (with laradock), one including the laravel-echo-server, one the laravel application and one the for the http server (using caddy).
I can address the redis container with
"databaseConfig": {
		"redis": {
			"port": "6379",
			"host": "redis"
		}
	},

But every try to address the authHost fails:
 Error: Invalid URI "caddy/broadcasting/auth"
When I use a testdomain that is registered in my local host file it tries to use <IP_ADDRESS>

Okay, it was "just" a docker error. I had to set the auth just to http://caddy and the caddy container is listening on <IP_ADDRESS>:80


x/tools/cmd/gotype: (can't find import: ) [GO15VENDOREXPERIMENT=1]

1 What version of Go are you using (go version)?
$ go version
go version go1.6rc1 linux/amd64

2 What operating system and processor architecture are you using?
$ cat /etc/redhat-release
Fedora release 23 (Twenty Three)

$ lscpu
Architecture:          x86_64
...

3 What did you do?
Project for test:
$GOPATH/src/main/main.go
$GOPATH/src/vendor/github.com/bradfitz/iter/iter.go

$ cat main.go
package main

import (
	"fmt"
	"github.com/bradfitz/iter"
)

func main() {
	for i := range iter.N(10) {
		fmt.Println(i)
	}
}

4 What did you expect to see?
Nothing (that's OK)
$ gotype main

5 What did you see instead?
$ go env && go version && gotype main
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/dmitriy/Development/go-proj"
GORACE=""
GOROOT="/home/dmitriy/Development/go1.6rc1"
GOTOOLDIR="/home/dmitriy/Development/go1.6rc1/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"
go version go1.6rc1 linux/amd64
main/main.go:5:2: could not import github.com/bradfitz/iter (can't find import: )
main/main.go:9:17: undeclared name: iter
main/main.go:9:17: cannot range over iter.N(10) (invalid operand)

Note: other tools from Go Meta Linter work OK.

CC @griesemer @alandonovan

@runningmaster Is this an issue with x/tools/cmd/gotype only? Or also with std lib gotype? (in go/types).

@griesemer
It's also with std lib gotype. I did $ go build gotype.go in go/types and replaced it gotype from x/tools/cmd/gotype. All errors are here.
$ tree -d /home/dmitriy/Development/go-proj/
/home/dmitriy/Development/go-proj/
├── bin
├── pkg
│   └── linux_amd64
│       ├── internal
│       └── vendor
│           └── github.com
│               └── bradfitz
└── src
    ├── internal
    │   └── version
    ├── main
    └── vendor
        └── github.com
            └── bradfitz
                └── iter

$ pwd && echo && cat main.go
/home/dmitriy/Development/go-proj/src/main

package main

import (
	"fmt"

	"github.com/bradfitz/iter"

	"internal/version"
)

func main() {
	for i := range iter.N(10) {
		fmt.Println(version.AppName(), i)
	}
}

$ pwd && echo && go env && echo && go version && echo && gotype main
/home/dmitriy/Development/go-proj/src

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/dmitriy/Development/go-proj"
GORACE=""
GOROOT="/home/dmitriy/Development/go"
GOTOOLDIR="/home/dmitriy/Development/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

go version devel +6a208ef Tue Feb 9 00:17:25 2016 +0000 linux/amd64

main/main.go:6:2: could not import github.com/bradfitz/iter (can't find import: )
main/main.go:12:17: undeclared name: iter
main/main.go:12:17: cannot range over iter.N(10) (invalid operand)

@runningmaster Temporary work-around: I believe this should work if you provide the full (absolute) path name for main (/home/dmitriy/Development/go-proj/src/main in this case).
But it's clearly a bug. Fix forthcoming.

@griesemer

Temporary work-around: I believe this should work if you provide the full (absolute) path name for main (/home/dmitriy/Development/go-proj/src/main in this case).

Workaround works fine for go/gotype and x/tools/cmd/gotype too.
$ go env && go version && gotype $(pwd)/main

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/dmitriy/Development/go-proj"
GORACE=""
GOROOT="/home/dmitriy/Development/go"
GOTOOLDIR="/home/dmitriy/Development/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"
go version devel +79d9f48 Tue Feb 9 21:12:32 2016 +0000 linux/amd64

Actually I use command go list ./... | grep -v vendor/ | xargs -L1 gometalinter in my bash file for checking my code, but gotype in this case writes errors. Now all works fine (using workaround). Thank you.

@runningmaster This has been fixed (at tip) and should not require the work-around anymore.

I have the same issue. I'm on Windows. Here is my file structure (I'm running tree from the project folder, but further down I will give the complete path to gometalinter):
C:\Users\Parham\go\src\hamrah7.com\iwr>tree
Folder PATH listing
Volume serial number is D057-3127
C:.
├───client
├───config
├───context
├───httpserver
│   └───handlers
├───pkg
├───serverctl
├───supervisord.d
└───taskprocessor

And here is the output of running gometalinter --deadline=20s %GOPATH%\src\hamrah7.com\iwr\...:
C:\Users\Parham\go\src\hamrah7.com\iwr>gometalinter --deadline=20s %GOPATH%\src\
hamrah7.com\iwr\...
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:8:2:error: could not import ha
mrah7.com/iwr/context (can't find import: hamrah7.com/iwr/context) (gotype)
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:9:2:error: could not import ha
mrah7.com/iwr/httpserver (can't find import: hamrah7.com/iwr/httpserver) (gotype
)
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:11:2:error: could not import h
amrah7.com/iwr/taskprocessor (can't find import: hamrah7.com/iwr/taskprocessor)
(gotype)
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:33:13:error: undeclared name:
context (gotype)
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:34:2:error: undeclared name: h
ttpserver (gotype)
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:58:24:error: undeclared name:
taskprocessor (gotype)
C:\Users\Parham\go\src\hamrah7.com\iwr\...\iwr.go:58:62:error: undeclared name:
taskprocessor (gotype)

I have just installed these, so this isn't an old installation. Also, here is my environment information:
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\Parham\go\
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GO15VENDOREXPERIMENT=
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1
go version go1.5.2 windows/amd64

@parhamdoustdar This should be fixed for 1.6. Please re-open if you still see a problem. Thanks.

I am using gometalinter embedded in atom go-plus which in turn uses gotype. Updated to latest gotype with go get -u
Output of go env && go version && gotype bevaddress.go
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/john/src/gocode"
GORACE=""
GOROOT="/home/john/opt/go"
GOTOOLDIR="/home/john/opt/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"
go version go1.6 linux/amd64
bevaddress.go:10:2: could not import github.com/gorilla/websocket (can't find import: github.com/the42/bevaddress/vendor/github.com/gorilla/websocket)
bevaddress.go:11:4: could not import github.com/lib/pq (can't find import: github.com/the42/bevaddress/vendor/github.com/lib/pq)
bevaddress.go:19:16: undeclared name: websocket
The directories gotype complaints no to be able to find are in place. This seems like the error has not been fixed. (Or do I need to update sthg. more but gotype?)

@the42 Please double-check this and file a new bug since 1.6 has passed. Also, please add your directory structure as otherwise it's not possible to recreate the issue if it really exists. Keep in mind that the "newest" version of gotype is in the std lib go/types directory - you need to build it yourself (there's a separate issue filed to make gotype "go get"-able again (from x/tools). I have just checked that (std lib gotype) code and I believe it's using the latest version of the go/importer which supports vendoring.

@griesemer Before opening a bug I realy think I am using the GOVENDOR-unaware gotype for x/tools. I build go from source, checking out go1.6 and bootstraping with a go1.4. How do I build the go/types gotype?
'which gotype' yields /home/john/src/gocode/bin/gotype
The project I would like to gotype has the following structure:
john@ebola:~/src/gocode/src/github.com/the42/bevaddress$ tree -d
.
├── Godeps
└── vendor
└── github.com
├── gorilla
│   └── websocket
└── lib
└── pq
└── oid

@the42 In the go/types directory just type: go build gotype.go . That will give you an executable named gotype in that directory that you can use.

Filed as 14918


Fix syntax error when param exists

Related with #376
Haven't resolve the problem with kotlin.String parameter.

Thank you! LGTM!


Markdown errors in HARKmanual causing Sphinx errors

In issue #412, it's noted that with recommonmark, the make html Sphinx command errored.
https://github.com/econ-ark/HARK/issues/412#issuecomment-545105238
PR #411 "solved" this problem by removing recommonmark. Unfortunately, this appears to prevent Sphinx from parsing any markdown at all. The baby was thrown out with the bathwater.
I've isolated the broken markdown to these files:
https://github.com/econ-ark/HARK/blob/master/Documentation/HARKmanual/HARKmanual.md
https://github.com/econ-ark/HARK/blob/master/Documentation/HARKmanual/ContributingToHARK/ContributingToHARK.md
The former appears to be autogenerated from the original LaTeX version of the manual and contains many irregular looking image references, as they are are used for rendered mathematical formulae.
These documents have been generated for uses that are not the sphinx-generated ReadTheDocs:
https://github.com/econ-ark/HARK/issues/409#issuecomment-546629439
Therefore, the errors they are caused are due to the confounded directory structure.
I'll remove these files as part of the documentation cleanup effort. See #409

I was able to setup a readthedocs copy running at https://hark-mriduls.readthedocs.io https://hark-mriduls.readthedocs.io/ with recommonmark and the HARKManual.
But I get the same recommonmark  error locally, this could be also about our local installations of the dependences.

On 30-Oct-2019, at 12:40 AM, Sebastian Benthall<EMAIL_ADDRESS>wrote:
In issue #412 https://github.com/econ-ark/HARK/issues/412, it's noted that with recommonmark, the make html Sphinx command errored.
#412 (comment) https://github.com/econ-ark/HARK/issues/412#issuecomment-545105238
PR #411 https://github.com/econ-ark/HARK/pull/411 "solved" this problem by removing recommonmark. Unfortunately, this appears to prevent Sphinx from parsing any markdown at all. The baby was thrown out with the bathwater.
I've isolated the broken markdown to these files:
https://github.com/econ-ark/HARK/blob/master/Documentation/HARKmanual/HARKmanual.md https://github.com/econ-ark/HARK/blob/master/Documentation/HARKmanual/HARKmanual.md
https://github.com/econ-ark/HARK/blob/master/Documentation/HARKmanual/ContributingToHARK/ContributingToHARK.md https://github.com/econ-ark/HARK/blob/master/Documentation/HARKmanual/ContributingToHARK/ContributingToHARK.md
The former appears to be autogenerated from the original LaTeX version of the manual and contains many irregular looking image references, as they are are used for rendered mathematical formulae.
These documents have been generated for uses that are not the sphinx-generated ReadTheDocs:
#409 (comment) https://github.com/econ-ark/HARK/issues/409#issuecomment-546629439
Therefore, the errors they are caused are due to the confounded directory structure.
I'll remove these files as part of the documentation cleanup effort. See #409 https://github.com/econ-ark/HARK/issues/409
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/econ-ark/HARK/issues/420?email_source=notifications&email_token=ABI5RFGQ767FE4LU6X3ZOODQRCDCHA5CNFSM4JGOBGK2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HVFNHNQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABI5RFCTLZ3AY7VN2KFKQZTQRCDCHANCNFSM4JGOBGKQ.


L530 lights no longer work on plugin

Describe The Bug
From last week (22 August 2023) the L530 devices stopped working entirely through the plugin. My plugin is on the latest version (v3.2.2) and the lights are on the latest firmware (1.1.0 Build 230721 Rel.224802). Can’t switch on / off the lights, change colour or dim the lights
Expected behavior
When a request to perform an action is sent through Homebridge, the plugin should be communicating this information through to the Tapo servers to perform the action.
Logs
https://www.dropbox.com/scl/fi/isdot3s7uw617wvn5947b/homebridge.log.txt.xml?rlkey=hyulbwnfi6qldhft7d5cytsaz&dl=0
Plugin Config
"addresses": [
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>",
"<IP_ADDRESS>"
],
"name": "TP-Link Tapo",
"email": "",
"password": "",
"_bridge": {
"username": "",
"port": 51005
},
"platform": "HomebridgeTPLinkTapo"
},
Plugin Version
3.2.2
Homebridge Version
1.6.1
Node.js Version
16.20.2
NPM Version
8.19.4
Operating System
Macos

Duplicate of https://github.com/RaresAil/homebridge-tp-link-tapo/issues/49


A run & debug dropdown with default button

This PR fixes #117722
This is my first attempt for a dropdown menu that remembers a default action.
The PR introduces an alternative UI for submenus:
If a "storageKey" attribute is specified for a submenu, the submenu is shown in the UI as a dropdown menu with a button that executes the last action run from the dropdown. The default action is persisted in workspace storage under the "storageKey".
This alternate UI is generic (independent from debug's run & debug functionality).
However, this PR configures the alternate UI for the "editor/title/run" contribution point.
The UI looks like this:

Yeah, I was using the menu identifier as a storage key before but decided differently when polishing the PR... ;-)

Incorporated @jrieken's feedback.


Only validate balance if dealing with Borderless Account

Some accounts are not Borderless Accounts and since we only use it to validate if the account has enough balance, I'll skip and let the transaction fail if we don't have the balance in their account.

Not needed. After a second investigation, I isolated the problem to the used token.


LayerVersion: unsupported properties ContentUri, RetentionPolicy

Description:
I can't seem to be able to deploy the simplest ruby lambda layer. Both sam validate and sam package work but then it fails at the sam deploy stage.
Steps to reproduce the issue:
Given the following file structure, template and existing S3 bucket
(top folder, running sam commands from here)
├── layer
│  └── (layer code in here)
└── templates
   └── sam_template.yaml

AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Resources:
  BaseRubyGemsLambdaLayer:
    Type: AWS::Lambda::LayerVersion
    Properties:
      LayerName: baseRubyGems
      ContentUri: ../layer
      RetentionPolicy: Delete
      CompatibleRuntimes:
        - ruby2.5

Running
sam package --template-file ./templates/sam_template.yaml --s3-bucket base-ruby-gems-dev --output-template-file ./templates/packaged.yaml

sam deploy --template-file ./templates/packaged.yaml --stack-name base-ruby-gems --capabilities CAPABILITY_IAM

Observed result:
"ResourceStatus": "CREATE_FAILED",
"ResourceStatusReason": "Property validation failure: [Encountered unsupported properties in {/}: [ContentUri, RetentionPolicy]]"

Expected result:
I should have the layer deployed and ready to use.
sam --version
SAM CLI, version 0.17.0

ok the issue was I used
Type: AWS::Lambda::LayerVersion

when it should be
Type: AWS::Serverless::LayerVersion


ORBSLAM3 to Github

Need to integrate the Docker container for ORBSLAM3 into our repo with documentation on it!


Add user.get function, remove useless catch

Add a users.get function, which allows you to input either a username or a user id.
Remove duplicate code that was in both users.id and users.name.
Remove useless try/catches across the project (I believe the code is equivalent without the try/catches, but please correct me if there is any difference).

Everything looks fine and locally the tests were successful.
Ty for another useful contribution.
Maybe next time try to commit different stages of the pull request as separate commits (remove useless try-catch, remove redundant user get code by adding generic get endpoint) or squash the forgotten test into the other commit but I'm not Google or someone with pull request rules so I will just merge it happyily :D


Data used for prediction at each leaf

I am currently working with the R version of a project and need assistance with accessing the data used for prediction at each leaf. I would like to retrieve the information on the data employed within a specific cover for further analysis. My goal is to enhance the existing functionality by gaining access to this internal function, alongside the 'Cover' data, which is already available.
Example code in R:
data(agaricus.train, package='xgboost')
bst <- xgboost(data = agaricus.train$data, label = agaricus.train$label, max_depth = 2,
eta = 1, nthread = 2, nrounds = 2,objective = "binary:logistic")
(dt <- xgb.model.dt.tree(colnames(agaricus.train$data), bst))
(dt <- xgb.model.dt.tree(model = bst))
Output is:
Tree Node  ID                 Feature         Split  Yes   No Missing      Quality      Cover
1:    0    0 0-0               odor=none -9.536743e-07  0-1  0-2     0-1 4000.5310100 1628.25000
2:    0    1 0-1         stalk-root=club -9.536743e-07  0-3  0-4     0-3 1158.2120400  924.50000
3:    0    2 0-2 spore-print-color=green -9.536743e-07  0-5  0-6     0-5<PHONE_NUMBER>  703.75000
4:    0    3 0-3                    Leaf            NA       1.7121772  812.00000
5:    0    4 0-4                    Leaf            NA      -1.7004405  112.50000
6:    0    5 0-5                    Leaf            NA      -1.9407086  690.50000
I would greatly appreciate any guidance or assistance in locating the appropriate internal function or alternative method that can provide the desired leaf data for prediction within the R version. Thank you in advance for your support!

I can't reca the exact name at the moment, but there's a parameter similar to pred_leaf in the predict function.


Typo in the repository name

[!IMPORTANT]
It's not. I just like the purple text

Title issue.
A  fix:
Rename ConsolesGames to ConsoleGames.
:)

bump

bump

Fixed it pookie!


Investigate "Variable 'symptomOnset' has an invalid value : Invalid value '0' for LocalDate " error

Description
This error appeared in App Insights in production  on 6/7/2021, 1:30:51 PM.
Steps to reproduce
Unknown
Expected behavior
This should not happen
Additional context
Add any other context about the problem here.

Won't fix. This issue has not been seen since


The weightless animation briefly plays when a marine gets nested

is it still a thing?

Havent seen it happen in a while


Saving issues

On the latest update Ive been noticing some changes are not being applied.
For example I placed and linked a safe in an office, saved then reloaded while seeing the safe is gone again.
Another issue I had was positions did not save for multiple elements.
Sent a copy of the map directly to you.

This seems to mostly happen when only mission elements are moved, then saved, then reloaded.
Very inconsistent.
And yes I am waiting for the save to finish.


Improve timeout config ergonomics and add SDK default timeouts

Motivation and Context
This is work towards #256 for SDK production readiness. This PR:

Overhauls timeout configuration so that it more closely resembles retry configuration
Removes configs for timeouts that are not implemented and have no plan to be implemented
Removes non-standard env var/profile file timeout configuration values so that they won't be a roadblock for standardizing them across SDKs at a later date
Improves re-exports for runtime types so that customers are less likely to need to take direct dependencies on them
Refactors Smithy client building:

Smithy clients can no longer have their retry/timeout configs changed after creation. Previously, this would have allowed them to get into an invalid state since the sleep_impl validation takes place inside of the builder's build() method.
Tracking standard vs. non-standard retry policy is now handled better in the builder implementation
Made the builder's setter functions consistent with our other builders

Moves the code generator for CustomizableOperation into its own file
Makes it possible to code generate non-root modules
Renames the crate::customizable_operation module to crate::operation::customize
Re-exports types that can be used with operation customization inside of crate::operation::customize

Checklist

[ ] I have updated the AWS SDK examples
[x] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[x] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Opened https://github.com/awslabs/smithy-rs/issues/1771 to track fixing the examples.


Add navigation obstacles to performance monitor stats

Adds navigation obstacle count to navigation performance monitor.
At the time the navigation performance monitor was added real obstacles did not exist, they were just dummy agents.
Now that obstacles are there own thing they are still missing from the monitor stats and this PR fixes it.

Thanks!


Blade Runner (Classic)

Project Link
https://www.scummvm.org/
Project License
GPL 3
Steam App Ids
https://steamdb.info/sub/596994/
Additional Context
The original version of Blade Runner is bundled with the Enhanced Edition and it works with ScummVM. I think it just needs to be linked to the "Classic" subfolder.

@NerosTie Do you own the game for testing? If so, I can try putting out a update that should support it.

Yes I own the game

Ok great! I'll let you know once Ive had a chance to update

This should be supported now. Let me know if you run into any issues

It doesn't work unfortunately: "No such file or directory (os error 2)"

Can you try it again? It should work now, I forgot to add the steam id to the list that causes it to automatically update the download.

Yes it works but it launches ScummVM (then you have make a choice again) and not directly the game, is it the expected behavior?

I think that's to be expected since it found multiple options, although not sure the mechanism it uses to decide to go straight into a game. If you end up finding a solution to that, feel free to open a PR or make a note here and I can add it.


major: change SelectFieldDefinitionChoice -> SelectChoice

:tada: This PR is included in version 19.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Different geojson layers

Currently all geo objects are stored in the same geojson layer/feature group. We should add a system where the user can decide on which layer the want to add a (e.g. newly created) geojson object. I don't know if it is possible in leaflet to support multiple draw layers. Especially when it comes to interaction (draw, edit, delete).

After some research I found out that it is possible to have different geojson layers. To add geojson layers to the baselayers or overlay array we have to add a new key-value pair to the baselayers/overlay (e.g. map.map.layers.overlays.ourNewGeoJsonLayer = {}). The content of the new layer must have the type geoJSONShape, a name, data (e.g. map.map.geojson.data to add all existing geodata to this layer), visible (bool) and layerOptions. layerOptions has the same function as map.map.geojson (style, pointToLayer, onEachFeature).
Question:
Which layers do we want?

One layer for each context type + unlinked
The user can add/remove layers and add/remove features/geodata from a layer
...

Please comment @eScienceCenter/spacialists
Example:
map.map.layers.overlays.geojson = {
    name: 'GeoJSON',
    type: 'geoJSONShape',
    data: map.map.geojson.data,
    visible: true,
    layerOptions: {
        style: function(feature) {
            var currentStyle = angular.copy(map.style);
            currentStyle.fillColor = feature.properties.color;
            return currentStyle;
        },
        pointToLayer: function(feature, latlng) {
            var m = L.circleMarker(latlng, map.style);
            m.setRadius(m.getRadius() / 2);
            return m;
        },
        onEachFeature: function(feature, layer) {
            if(feature.properties && feature.properties.popupContent) {
                layer.bindPopup(feature.properties.popupContent, {
                    minWidth: 300,
                    feature: feature
                });
                var name;
                if(map.geodata.linkedContexts[feature.id]){
                    name = environmentService.contexts.data[map.geodata.linkedContexts[feature.id]].name;
                }
                else{
                    name = feature.properties.name;
                }
                layer.bindTooltip(name);
                layer.on('click', function(){
                    map.map.selectedLayer = layer;
                    layer.openPopup();
                });
            }
            feature.properties.wkt = map.toWkt(layer);
            map.featureGroup.addLayer(layer);
            map.geodata.linkedLayers[feature.id] = layer;
            var newBounds = map.featureGroup.getBounds();
            var newNE = newBounds.getNorthEast();
            var newSW = newBounds.getSouthWest();
            map.map.bounds.northEast.lat = newNE.lat;
            map.map.bounds.northEast.lng = newNE.lng;
            map.map.bounds.southWest.lat = newSW.lat;
            map.map.bounds.southWest.lng = newSW.lng;
        }
    }
};

Based on the feedback I updated OP.

Snapping works (at least for new objects) with the latest changes in Leaflet.Snap. Snapping for existing objects doesn't work due to the missing merge in Leaflet.draw. Since this is nothing we can control, I'd say we close this and as soon as it is fixed in Leaflet.draw the latest version will be downloaded in npm anyway.


Be able to start parameterized tests with specified parameter values

Now it's possible to restart failed tests by index but this index is not available for passed tests, so for passed tests it's impossible to restart the test with the same parameters. The PR makes this information available.

Linked to https://youtrack.jetbrains.com/issue/IDEA-156780

Thanks Anna!


fix: Use Classpath normalization for auxClassPaths

Declaring auxClassPaths as a Classpath property allows Gradle to perform additional normalization,
so that functionally-identical jar files do not trigger a re-execution of the SpotBugsTask.
A re-generated jar containing identical classes will have different entry timestamps for each class entry,
and as such is not bitwise identical even though it is functionally identical.
Without this fix, changing an input jar file will change the input cache key, leading to consistent misses from the local build cache.
See https://docs.gradle.org/6.0/userguide/more_about_tasks.html#sec:task_input_using_classpath_annotations
Note that I have tested this change locally, but not added any specific test coverage for this use case.

It seems that #279 is similar to this fix. @DPUkyle could you review this change?
If you know something to consider, please share it with us. :)

Yes, this is great. Thanks @bigdaz for fixing it!

Is it correct, that v4.5.0 has both annotations (@InputFiles AND @Classpath)? Currently I am searching for changes of auxClassPaths fingerprints in my project...

Is it correct, that v4.5.0 has both annotations (@InputFiles AND @classpath)?

It's safe to remove the @InputFiles annotation, unless there is a goal to remain compatible with Gradle < 3.2. I'll leave that to the plugin maintainers to decide.

This plugin does not support Gradle < 3.2, so we can remove @InputFiles in our case.


fix template string to actually output variable instead of literal

It was saying "runningProcess.name" instead of actually translating that to the variable value.

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

TheRealMaxion
✔

JestDotty
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


Start date of cover

Can only buy starting today? There is no field for setting a start date?

Exactly, in the current version of the smart contracts, the protection starts immediately. But this might be an option for the future.