how to read a xlsx data sheet?

Opentrons cans read and runs the data sheet on the simulator
Not a library issue ( We loaded pandas on Hao computer and the Rasberry  pi without problem)
We can't use PowerShell because of administrator  doesn't allow us.
The site : "https://support.opentrons.com/s/article/Using-CSV-input-data-in-Python-protocols" will be helpful because it has the clue to solve this issue.
May be we  can  run the protocol directly on the Rasberry pi without using the  Opentrons software on the desktop.

It looks like the preferred usage pattern is to create python input files that contain the data within them for each experiment (as per @ekosso 's #4 above)
The opentrons protocol library provides an example of how one might do this:

Create a simple web-app that takes a CSV or XLSX file as input
Web-app reads the input file and extracts the relevant columns
Web-app emits a new python file that has the data (as defined variables) and the code (just copying the boilerplate run/setup code in the simplest case  or doing string replacements)

Of course, this need not be a web-app...it could just be a script
From the user's perspective in the lab, the workflow would look like:

Create the XLSX file (in the appropriate GitHub synchronized experimental results folder...)
Upload the XLSX file to the web-app and download the resulting output python file (which also gets put into the appropriate GitHub synchronized experimental results folder...)
Load the output python file into the OpenTrons application and proceed

Thoughts?

I was thinking something similar. If the OpenTrons application needs the data to be embedded in the input file, then I think the easiest thing to do is to create a python script that takes the xlsx data as input and creates another python file as output. That new file would then be uploaded via the OpenTrons GUI.

I will try and let you know.

I wrote a new file called "generate_instructions.py". Given a filepath to an xlsx spreadsheet, it creates a new input file called "opentron_input.py". This file contains a string called "data" that is the information pulled from the xlsx spreadsheet, following the usage pattern in the links @ekosso and @jschrier included above. The code should now be readable by OpenTrons.
I didn't test this in the simulator due to some trouble setting the simulator up on my local machine. Maybe @hliu56 can try to see if it works?
PS - I had to edit the column headers in "opentron_input.py" from "None, Reagent 1, Reagent 2" to "Well, A1, A2" to fit the design pattern that Opentron needs.

Thanks for generating the .py results Eric! @ewroginek
Yes, I've  run the 'opentron_input.py' successfully by fixing a couple of bugs:

add 'import pandas as pd', since in the code it will use pd
change 'Well' to 'Well_number' (just a typo)
I try to iterate the values in each column, because the CSV_DATA[STOCK] is a pandas Series object, and in opentron it can't directly convert a Series to a single float.

I include the modified code and these running results in a new folder: 'hao_test'
Maybe Eric and I can continue on the 'movement' part. Right now I use transfer operations and haven't thought that part too much yet....

Hey guys, We will run the protocol tomorrow and let you know.
Thanks.

The code has been updated so that running python main.py will combine OT-2 instructions from a .xlsx data file and a preprocessed protocol file into a single post-processed protocol file that can be read by the OT-2 robot. The added functionality solves this issue


Global install failed,successed local.

npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm ERR! code 127
npm ERR! path /root/.nvm/versions/node/v17.9.0/lib/node_modules/verdaccio/node_modules/es5-ext
npm ERR! command failed
npm ERR! command sh -c  node -e "try{require('./_postinstall')}catch(e){}"

Request is being removed look my last PR


[Event Request] table 5900 "Service Header" - OnBeforeUpdateShipToCodeAddressFields

We would like to add a new event OnBeforeUpdateShipToCodeAddressFields in the following trigger:
        trigger OnValidate()
        var
            ShipToAddr: Record "Ship-to Address";
            ConfirmManagement: Codeunit "Confirm Management";
            IsHandled: Boolean;
            UpdateShipToAddressFields: Boolean;
        begin
            if ("Ship-to Code" <> xRec."Ship-to Code") and
               ("Customer No." = xRec."Customer No.")
            then begin
                if ("Contract No." <> '') and not HideValidationDialog then
                    Error(
                      Text003,
                      FieldCaption("Ship-to Code"),
                      "Document Type", FieldCaption("No."), "No.",
                      FieldCaption("Contract No."), "Contract No.");
                if ServItemLineExists then begin
                    if not ConfirmManagement.GetResponseOrDefault(
                         StrSubstNo(Text004, FieldCaption("Ship-to Code")), true)
                    then begin
                        "Ship-to Code" := xRec."Ship-to Code";
                        exit;
                    end;
                end else
                    if ServLineExists then
                        if not ConfirmManagement.GetResponseOrDefault(
                             StrSubstNo(Text057, FieldCaption("Ship-to Code")), true)
                        then begin
                            "Ship-to Code" := xRec."Ship-to Code";
                            exit;
                        end;
            end;

            //if "Document Type" <> "Document Type"::"Credit Memo" then A108F0023.Event.o
            //A108F0023.Event.ns
            UpdateShipToAddressFields := "Document Type" <> "Document Type"::"Credit Memo";
            OnBeforeUpdateShipToCodeAddressFields(Rec, UpdateShipToAddressFields);
            if UpdateShipToAddressFields then
                //A108F0023.Event.ne
                if "Ship-to Code" <> '' then begin
                    if xRec."Ship-to Code" <> '' then begin
                        GetCust("Customer No.");
                        if Cust."Location Code" <> '' then
                            "Location Code" := Cust."Location Code";
                        "Tax Area Code" := Cust."Tax Area Code";
                    end;
                    ShipToAddr.Get("Customer No.", "Ship-to Code");
                    SetShipToCustomerAddressFieldsFromShipToAddr(ShipToAddr);
                end else
                    if "Customer No." <> '' then begin
                        GetCust("Customer No.");
                        CopyShipToCustomerAddressFieldsFromCust(Cust);
                    end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeUpdateShipToCodeAddressFields(var ServiceHeader: Record "Service Header"; var UpdateShipToAddressFields: Boolean)
begin
    //A108F0023.Event
end;

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


Docs Forge - Library Function: precalculate_address typo

In the code example for precalculate_address, the unwrap() needs to be removed since the precalculate_address function returns as ContractAddress.
    let contract_address = contract.precalculate_address(@constructor_calldata).unwrap();

Feel free to assign me.

@robertkodra thanks! Assigning you

Hi! Unassigned you to not block this issue for others, if you open a PR I will assign you again


Angular 11

Hey,
I'm currently planning our angular 11 update and am curious, if you expect this library to be any problem? Should it still work fine with 11?

Yes, I think there shouldn't be any problem. If you want to be sure, make a Blitz!


tests: fifo_timeout : Dereference after null check

fifo data obtained is derefenced after NULL check.
Coverity-CID:186190
Signed-off-by: Nagaraj Hegde<EMAIL_ADDRESS>

Codecov Report

Merging #8048 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #8048      +/-   ##
==========================================
- Coverage   55.03%   55.03%   -0.01%
==========================================
  Files         481      481
  Lines       53934    53936       +2
  Branches    10474    10476       +2
==========================================
- Hits        29684    29683       -1
- Misses      19981    19983       +2
- Partials     4269     4270       +1

Impacted Files
Coverage Δ

tests/kernel/fifo/fifo_timeout/src/main.c
92.2% <0%> (-1.22%)
:arrow_down:

boards/posix/native_posix/timer_model.c
98.21% <0%> (-1.79%)
: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 ec7d483...e4ae4b3. Read the comment docs.


⚠ Secret Website 2 has degraded performance

In 79a06f6, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 5035 ms

Resolved: Secret Website 2 performance has improved in ccbf0ca.


Support for armhf

Is there a way to support Calico on armhf (not arm64)?
It would be great to be able to support a mixed architecture kubernetes cluster that includes armhf
Expected Behavior
Run Calico in a cluster which includes armhf nodes.
Current Behavior
Only arm64 nodes are supported
Your Environment

Calico version 3.14
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes 1.18.3
Operating System and version: Raspbian on armhf

It would be awesome if you start doing builds for armhf

arm64 support was added by the community; I can't see why we wouldn't accept PRs to add armhf too

try to build calico/node for armhf,  but failed,   hope to support armhf

Hi.
As mentioned Here, I have also built calico for 32bits arm, late 2020, and I'm willing to contribute. (finally started refining my changes).
As this issue is stall since summer 2020, following the contributing code, do I need to open a new issue or can we continue on this tread ?
Regards
Aal.

So I think what you'll need to do is
Add a new Dockerfile to https://github.com/projectcalico/go-build for armhf. You can see from here https://github.com/projectcalico/go-build/blob/master/Makefile.common#L18 that the file extension for the Dockerfile is used as the GOARCH env var used when buidling a project for a specific arch. https://github.com/projectcalico/cni-plugin/blob/master/Makefile#L90-L100
Then add Dockerfiles for armhf for all the Calico projects that build binaries like cni-plugin, node, felix, etc.

Thank you @lmm.
Maybe the project should choose the canonical name for this architecture ?
It has many variants (arm, armv7, armhf, etc) but it can't only have one for all the project's components.
For now I use 'arm' (because I'm lazy 😄 )
Here a list of PR to follow the arm integration :

[ ]  bird
[ ]  go-build

More to come
Regards.

I also agree with @aalaesar on closing this issue. https://hub.docker.com/r/calico/node/tags?page=1&ordering=last_updated&name=armv7
@winromulus yes at the moment documentation is not reflecting support for armhf. (Sounds like a new contribution if you feel like it )

@winromulus @frozenprocess
J'ai, probably my Gault as I tagge most of PR 'no Doc/release note required'.
so probably the CI didn't catch anything for the release note.
Sorry. :pensive:


no server detected

Hello so i got this message :

i can't found the serveur, but he is started on docker-compose :

you have to go in-game and select "Show Player Server" and then search it and you should find it DONT use steamgui  to find the server is not supported for ASA


Doubt about clean-following command strategy

I frequently use the "clean-followings" command, which works just fine, but I would like to know: It unfollows the FIRST "n" not mutual followings or the LAST ones? Or does it choose the not mutual followings aleatorily?

Hi @marceloavilaoliveira !
Since #171 PR will be merged the following algorithm will be used: it will unfollow FIRST "n" not mutual followings sorted by their IDs in descending order.

Uhm... just to be sure: this "ID" is related to when the user created his/her Instagram account, right? Newer accounts have bigger IDs, correct?
Actually, I would like to know if it's possible to unfollow the not mutual followings based on when the user was followed by me. Not mutual followings I follow the longest should be unfollowed first. There is a greater chance that the newer ones will become followers yet.
I don't know if I was able to make myself clear.

Yeah, these ID I receive from instagram, so seems that their logic for these ID values is what you've described.
I see.. Well, I double checked the SDK that I'm using and unfortunately didn't find any mention of date for following

I really thought there was no way to implement this. Anyway, thanks a lot for your attention, I think the algorithm you started to use will be a good improvement.

HI @marceloavilaoliveira !
I have a good news) I contacted with owner of SDK that I'm using in my tool and he implemented functionality to get

https://github.com/Davincible/goinsta/issues/33

Hi @marceloavilaoliveira !
I've added the ordering by earliest date for following list, that means that now not mutual followings you follow the longest will  be unfollowed first.
Could you please check the latest version v1.9.1?


Rule Request: UnnecessaryShellCommand

As an engineer, I want to be warned early when my application code is trying to shell out for things that libraries already provide, such as:

curl/wget (Can probably use a URL request)
sed/awk/gawk/nawk (Can just use a regular expression)
jq (Can use a JSON marshaller)

This rule would encourage more performant applications, remove dependencies on unnecessary software components, reduce attack surfaces, and increase cross-platform support, by avoiding relying on platform-specific shell command syntax.
Example triggers:

Supplying a process object with a launch path ending with curl, wget, sed, awk, jq, jq.exe, etc.

Example non-triggers:

Supplying a process object with an argument of curl, wget, sed, awk, jq, jq.exe, etc. While there are plenty of cases where these would constitute unnecessary shell commands, such as /usr/bin/env curl... or exec wget..., the false positives for such commands are unfortunately more numerous than the true positives.

This rule should contain a limited number of common shell commands, such as those listed above. We can add more over time. Configuration for this rule should be able to customize which of these command patterns is relevant to the user's application needs.
As a security concern, this rule can reasonably become a default rule, as shelling out introduces additional security risks, including shell injections, compared to sticking to pure library code.
I would be happy to see this rule be included in golint available suite of checks, and if we find we're happy to apply it in a lot of places, then we can always turn it on by default later.

Thanks for the issue.
This appears to be out of scope and so is being closed. If you feel this is in error please feel free to leave a comment and it may be re-opened.
Please note this is not the forum to discuss edits to CodeReviewComments. If you wish to propose changes to it please send a mail to golang-nuts.


moutn in fstab with custm endpoint

Hello ,
I am using backblaze service..
I am able to mount  using the command :
/bin/goofys --endpoint s3.us-west-002.backblazeb2.com bucketname /mnt/backblazemount/
how do I do to make it works in fstab ? how do I specify the endpoint ??
Thank you !

ok found out what you have to write in the fstab line
goofys#bucket   /mnt/mountpoint        fuse     _netdev,allow_other,--file-mode=0666,--dir-mode=0777,--endpoint=endpoint.adress    0       0


fix: minor fixes to constructors, type hints and method visibility

Codecov Report
Base: 83.69% // Head: 83.65% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (0af9931) compared to base (322fd10).
Patch coverage: 66.66% of modified lines in pull request are covered.

:exclamation: Current head 0af9931 differs from pull request most recent head 5ec9883. Consider uploading reports for the commit 5ec9883 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #79      +/-   ##
===========================================
- Coverage    83.69%   83.65%   -0.04%
===========================================
  Files           20       20
  Lines         1288     1291       +3
===========================================
+ Hits          1078     1080       +2
- Misses         210      211       +1

Impacted Files
Coverage Δ

nitric/resources/apis.py
40.12% <42.85%> (ø)

nitric/faas.py
81.02% <50.00%> (-0.16%)
:arrow_down:

nitric/application.py
65.38% <100.00%> (ø)

nitric/resources/base.py
83.33% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Finalize

Closes #21
closes #14

@KomodoKode Please review (and test for color-blindess support)

@KomodoKode Please review (and test for color-blindess support)

Contrast between red/orange could be better though.

Contrast between red/orange could be better though.

Contrast between red/orange could be better though.

Light orange or the orange? @KomodoKode

Contrast between red/orange could be better though.

Light orange or the orange? @KomodoKode


[Spotter]: Dynamic Per-Instance Processor Generation

With the old Spotter, I was making heavy use of dynamically substituting the available processors via the {{gtMethod:Object>>#gtSpotterProcessorsFor:}} hook. For example:
DbSoup>>#gtSpotterProcessorsFor: aSpotterStep
    | spotterSources |
    spotterSources := self sources collect: [ :e | e source ] as: IdentitySet.
    ^ spotterSources flatCollect: [ :source | source gtSpotterProcessorsFor: aSpotterStep ]

However, IIUC with the new streaming Spotter, it seems that the processor building is hardcoded and buried:

We start from {{gtMethod:Object>>#gtActionSearchFor:}}, where Object delegates to {{gtClass:GtSpotterDropdownButtonStencil}}
{{gtMethod:GtSpotterDropdownButtonStencil>>#spotterElementFor:}} creates a {{gtClass:GtStreamingSpotter}}
{{gtMethod:GtStreamingSpotter>>pushStep:}} creates a {{gtClass:GtStreamingSpotterStep}}
{{gtMethod:GtStreamingSpotterStep>>#createSearchesStream}} creates a {{gtClass:GtSpotterOrderedSearchesStream}}
{{gtMethod:GtSpotterOrderedSearchesStream>>buildProcessorsStream}} delegates to a {{gtClass:GtSpotterUnorderedSearchesStream}}

The pragmas finally get created as follows:
{{gtMethod:GtSpotterUnorderedSearchesStream>>#buildProcessorsStream}}
    ^ (GtSpotterSearchPragmasFuture forOriginObject: origin) asAsyncStream flatten
        map: [ :eachPragma | self basicProcessorForPragma: eachPragma ]

where the pragmas are collected by {{gtClass:GtSpotterSearchPragmasFuture}} and transformed into processors by {{gtMethod:GtSpotterUnorderedSearchesStream>>#basicProcessorForPragma:}}
basicProcessorForPragma: aPragma
    | aProtoSearch aSearch |
    self flag: 'handle errors and return a "broken" processor'.

    aProtoSearch := GtSpotterProtoSearch new.

    "Some #gtSearch extensions do not actually return a processor...
    we have to do a builder hack for now. We will rewrite the extensions!"
    aSearch := [
        | aMethodReturn |
        aMethodReturn := origin
            perform: aPragma methodSelector
            with: aProtoSearch.

        (aMethodReturn isKindOf: GtSpotterProtoSearch)
            ifFalse: [ (GtStreamingSpotterSearchWrongReturnError new definingMethod: aPragma method) signal ]
            ifTrue: [ aMethodReturn ] ]
        on: Error
        do: [ :anError |
            (self createBrokenSearchFrom: aPragma method)
                error: anError ].

    aSearch definingMethod: aPragma method.

    ^ aSearch

Can we add back some mechanism to dynamically generate Spotter processors per-instance?
History

2022-10-06 - First reported on Discord and issue created per request from @girba

Wow, so fast! Thanks :) can’t wait to play with it…


Discriminate string or number literal from general type.

Suggestion
🔍 Search Terms

exclude string literal from string type
exclude number literal from number type
discriminate string literal

✅ Viability Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
📃 Motivating Example
I want to be able to write code such as
type ApiResponse<Body> = {status: 200, body: Body} | {status: number, body: unknown}

// Get this type from calling an API...
declare const response: ApiResponse<{text: string}>

if (response.status === 200) {
  response.body.text // this is a string
} else {
  response.body // this is unknown
}

Fundamentally, this seems like the type checker needs to evaluate literal types before more general types.
I've also tried using Exclude but that doesnt help for obvious reasons.
type ApiResponse<Body> = {status: 200, body: Body} | {status: Exclude<number, 200>, body: unknown}

This is also useful for strings and string literals too, e.g. {type: "a", a: number} | {type: string, value: unknown}
💻 Use Cases
I think its surprisingly common that there are well known types we can use to narrow down a value, while there is still a large unknown amount of types that we aren't aware of an perhaps don't even care about.
The variety of http responses I could get is much larger than I can properly type (unfortunately), but the same can be said for progressively typing a large library. Perhaps I only care about a few different object types, but I still want there to be a catch all for the things I haven't typed yet and might run into...

You'd need negated types for this to work properly, since this is a legal (but unsound) construction if this were allowed:
const n: number = 200;
const m = { status: n, body: 42 };
const response: ApiResponse<{text: string}> = m;
if (response.status === 200) {
  response.body.text // claimed as string, but is number

Ah yeah, i guess that's basically what I was getting at with Exclude...
https://github.com/microsoft/TypeScript/issues/4196
Seems really useful, but probably a pretty big amount of work to implement.
Thanks for the help!


Add missing global attributes

@mfucci please re-check. Added typing (used as you proposed directly ....) and Todo


Make the user broker service will run as configurable.

filed PR upstream to spark discussion over best way to fix this.  This PR is only a partial fix to address the broker service.  Did not want to do full fix until I knew the direction upstream preferred.


Stop server when process ends

This currently doesn't seem to work: https://github.com/muffin/cli/blob/master/bin/serve#L31

Was a problem in args!


Roadmap and experimental.sparse

Hi everyone!
First, I wanted to say thanks for such an easy-to-use library! I've been using Jaxpruner for some weeks now, resulting in highly pruned convolutional models (mostly with unstructured pruners).
I wanted to ask if there was a roadmap for the library and if you had any expected release date for the "integration with the jax.experimental.sparse"
Thanks in advance

Hi Stefano,
We have a colab which converts a given checkpoint to jax.experimental.sparse, we plan to release this relatively soon. Apart from that, we are closely tracking the developments on the jax side. Let us know if you have any feature requests.
Best

Hi there, Utku! Sorry for such a late answer.
Here are some ideas off the top of my head that might go a long way toward improving Jaxpruner's adoption:

Simple(r) examples using each pruning method.
Examples using Jax's sparse API.
Examples of sparsfied models for inference. (provably faster using Block (2,4) pruning with Nvidia)
Exporting sparsified models to other frameworks (maybe unrelated to this lib only).
Examples of integration with other known libs: Keras Core, HF, etc.
Proposing future ways of improving training/inference times.

I'll be happy to help with any of these (if it's internally possible).
Cheers!

Hi Stefano,
We have a colab which converts a given checkpoint to jax.experimental.sparse, we plan to release this relatively soon. Apart from that, we are closely tracking the developments on the jax side. Let us know if you have any feature requests.
Best

Hi Utku,
I also have the same problem and wonder how to convert the pruned dense model parameters to a sparse BCOO format with jax.experimental.sparse. In the paper it says there's an example but I'm a little confused about where to find it. Could you please give me some guide about this problem?
Thanks a lot!

Sorry for being extremely slow with this. Finally we found the time to push the colab out. The commit #93bc832 solves this. Soon I'm hoping to share some-checkpoints and update the colab with real checkpoints.


Add python API for GeoSparkSQL

Is this PR related to a proposed Issue?
#183
What changes were proposed in this PR?
GeoSparkSQL Python API
How was this patch tested?
73 tests,

21 tests for serialization and deserialization between JVM and Python objects (converting to shapely BaseGeometry objects)

test_deserializers, 12 tests https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_deserializers.py
test_serializers, 7 tests, https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_serializers.py
test_geospark_serializers 2 tests, https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_geospark_serializers.py

52 tests (GeoSpark Scala tests rewritten to Python):

test_predicate_join, 14 tests https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_predicate_join.py
test_predicate, 11 tests https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_predicate.py
test_function, 19 tests https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_function.py
test_constructor, 6 tests https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_constructor_test.py
test_aggregate_functions, 2 tests https://github.com/Imbruced/geo_pyspark/blob/master/geo_pyspark/tests/test_aggregate_functions.py

Did this PR include necessary documentation updates?
Yes

@Imbruced I will accept this PR. But I will move your docs to GeoSpark docs such that your Python documentation will be shown on GeoSpark official website.


persian

Looks like there aren't any changes included in this PR.
Feel free to reopen after you have included translations first.


Open link in new window

What OS are you using (uname -a, or Windows version)?
Darwin limonia 20.3.0 Darwin Kernel Version 20.3.0: Thu Jan 21 00:07:06 PST 2021; root:xnu-7195.81.3~1/RELEASE_X86_64 x86_64
What programming language are you using (C/C++/Go/Rust)?
go version go1.16.2 darwin/amd64
What did you expect to see and what you saw instead?
Can I make "Open link in new window" open in the default browser? Right now, as far as the user is concerned, nothing happens when I right-click and open in new.

I also had this requirement but couldn't wait for the native implementation. So as a workaround, I simply bound the following function and used it to open links from the frontend:
// open opens the specified URL in the default browser of the user.
func open(url string) error {
	var cmd string
	var args []string

	switch runtime.GOOS {
	case "windows":
		cmd = "cmd"
		args = []string{"/c", "start"}
	case "darwin":
		cmd = "open"
	default: // "linux", "freebsd", "openbsd", "netbsd"
		cmd = "xdg-open"
	}
	args = append(args, url)
	return runCmd(cmd, args...)
}

I will put this in an example as it seems like a common use-case.


Bump Dependencies #9468028738

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/9468028738

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


Feature request b2-disable-checksum

Output of rclone version
rclone v1.45

os/arch: windows/amd64
go version: go1.11

Describe the issue
Requesting the --b2-disable-checksum option for transfer of very large files (1 TB) that can take > 24 hours to hash before transfer starts, as discussed in this forum topic: https://forum.rclone.org/t/hashing-during-upload/7433
For non-encrypted files not having SHA1 on the file might be an acceptable tradeoff for the speed of transfer gained, so should be okay and desirable for some users to have this option.

I would love to, but I have no experience with the project and in fact have just joined github. It would take too long for me to get up to speed to be of any use. Very sorry about that but to make this request is all I can do at the moment.

is this issue done now that the pull request b2: added disable sha1sum flag #2927 is merged  ?

I've merged this to master now which means it will be in the latest beta in 15-30 mins and released in v1.46


Update Plugin [1.20.3]

Hey, please update plugin to last Minecraft version: 1.20.3

Updated to Minecraft 1.21.3, let me know if you have any issues with it


Checksum all addresses

Right now, addresses are not checksummed, and a lot of compilers will throw errors if they are just copy pasted as so. For example:
The LINK token address of Rinkeby is marked as:
0x01be23585060835e02b77ef475b0cc51aa1e0709
And it should be:
0x01BE23585060835E02B77ef475b0Cc51aA1e0709

Is there a single place where we can find checksums?


[Feature Request]: add feedback option

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

two times the same thing appear so fixing that plus adding feedback option in footer
Use Case
assign this to me under gssoc ext and hacktoberfest
Benefits
No response
Priority
Medium
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

okay, you can create pr with screenshot


Behavioral change in 4.9 JpaEventStoreAutoConfiguration

Basic information

Axon Framework version: 4.9

Steps to reproduce
When upgrading our projects to 4.9 the autowiring of the JpaEventStorageEngine stopped working. We only use the framework, not Axon server. Apparently a new property "axon.axonserver.enabled" has been introduced that will disable autowiring of JPA and JDBC event storage engines. Unfortunately the default value is set to true, so for all projects where this property was not set (all of our services) the autowiring stops working.
Expected behaviour
Nothing should break when upgrading between minor versions. If there are breaking changes in minor versions, please make it clear in the release notes.
Actual behaviour
Broken builds...

Hi @magnusdurr, thanks for filing your concern with us.
Although the property is not new at all (we introduced it in Axon Framework 4.3, as you can see in this pull request, released in February 2020), I do agree we should've marked this shift within the release notes.
I assume your application excluded the axon-server-connector module, right?
The predicament that excluding the dependency wouldn't work did cross my mind when working on this pull request, which ensures JPA and JDBC are not wired whenever Axon Server is used.
Especially the automagical construction of the Event Store-specific JPA entities caused issues in user's pipelines.
I, however, wrongfully assumed (1) that people using Spring Boot would be aware of the property and (2) I forgot to mark this switch in the release of 4.8.3.
So, my sincere apologies for the issues this caused in your builds, @magnusdurr.
This wasn't intentional at all.
This brings us to a tough position, though.
Although excluding the dependency was the suggested approach to disable Axon Server pre AF4.3, this was never documented anywhere other than on our forum (as far as I am aware).
Furthermore, Spring Boot's auto wiring annotations are (again, as far as I know) incapable of validating in one fell swoop whether a dependency is present, a property is present, and if the property is there, whether it is set.
We have worked on making custom @Conditional annotations in the past, so perhaps there's a mix to be built.
But, I do believe the workaround, which is setting a property (FYI, documented here), is pretty straightforward.
I hope my decision-making with PR #2871 makes sense, @magnusdurr.
Still, you definitely have a point that we should've made this clear.
As such, I have adjusted the 4.8.3 release notes to explain this potential breaking change, as you can see here.
Lastly, if you feel strongly Axon Framework should support a mix of module exclusion and Spring Boot property usage, do let us know.

Hi @smcvb,
It would be nice if excluding the axon-server module would do the trick, but if it is not an easy fix we will adapt and add the axon.axonserver.enabled property to our services. I get it that it is hard to keep track of all side-effects when making changes, especially in a complex spring autowiring scenario.
Thanks

Thanks for the understanding, @magnusdurr.
And, as started off, we are very happy with this kind of communication.
It helps us improve the things we lost sight off, which should improve the experience for everybody!

@magnusdurr, @abuijze did some additional digging while working on the AF's auto-configuration and found some ordering issues.
It's this issue that led me to believe the property was the only way.
Pull request #2909 actually reverts the behavioral change you noticed by adjusting that ordering again.
So...I am going to extend my apologies again! Apparently my assumptions where wrong and we can keep both the property-based disabling of Axon Server and through exclusion of the axon-server-connector module.
The fix is slated for 4.9.1, by the way. Again, sorry for any inconvenience caused to you and the team (and to others who may encounter this issue and read it).

I've marked this issue as resolved by #2909 and adjusted the milestone to point to 4.9.1.


Jobs started from OSX Chia UI are causing plotman to fail

When I have jobs running in Chia UI on MacOS, plotman throws this (on @development branch):
Traceback (most recent call last):
  File "/usr/local/bin/plotman", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/plotman/plotman.py", line 160, in main
    jobs = Job.get_running_jobs(cfg.directories.log)
  File "/usr/local/lib/python3.9/site-packages/plotman/job.py", line 94, in get_running_jobs
    job = Job(proc, logroot)
  File "/usr/local/lib/python3.9/site-packages/plotman/job.py", line 117, in __init__
    val = None if arg in {'-e', '--nobitfield', '-h', '--help', '--override-k'} else next(args_iter)
StopIteration

When I printed out args[3:], it was:
['-k32', '-n1', '-t/Volumes/hdd0/tmp', '-2/Volumes/hdd0/tmp2', '-d/Volumes/hdd0/final', '-b3390', '-u128', '-r4', '-a1873810580']

So, as a dirty hack I tried adding -a1873810580 to  https://github.com/ericaltendorf/plotman/blob/main/src/plotman/job.py#L115 and to https://github.com/ericaltendorf/plotman/blob/main/src/plotman/job.py#L134 and caused plotman to start working fine, jobs started to be recognized.

As discussed on Keybase, https://github.com/ericaltendorf/plotman/pull/116 hopefully helps.  Follow up whenever you get a chance to try it out (again, and hopefully unbroken).  Thanks.

I'm getting this same error on Ubuntu.  Please let me know if there are additional details I can provide to help with hunting this one down.

@aaroneden did you try the branch in https://github.com/ericaltendorf/plotman/pull/116?  You can install it by using git+https://github.com/altendky/plotman@use_chia_cli_parser. in place of the url in the readme instructions.

Sorry, I meant to post this morning and got busy with work.  YES!  This fixed the issue for me, thank you!
Aaron Eden |<PHONE_NUMBER>
On Apr 29, 2021, 6:42 AM -0700, Kyle Altendorf @.***>, wrote:

@aaroneden did you try the branch in #116? You can install it by using @.***_chia_cli_parser. in place of the url in the readme instructions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@aaroneden, thanks.  Hopefully I can make the adjustments I want soon and get that merged.


ci, Add weekely CI Test

In order to prevent CI to broken by third party dependants like coreos-coreos-assembler repo, adding a weekly ci check.
Signed-off-by: Ram Lavi<EMAIL_ADDRESS>

looks like https://mirror.openshift.com/pub/openshift-v4/dependencies/rhcos/latest/latest/ is currently down..


Unable to install via Unity Package Manager

The following error occurs while adding the GitHub repo to the Package Manager with git URL.
Cannot perform upm operation: Unable to add package [https://github.com/zigurous/unity-ui-toolkit.git]:
  Cannot checkout repository [https://github.com/zigurous/unity-ui-toolkit.git]:
    Error when executing git command. Submodule 'docs~' (https://github.com/zigurous/docjs) registered for path 'docs~'
    Cloning into 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~'...
    remote: Repository not found.
    fatal: repository 'https://github.com/zigurous/docjs/' not found
    fatal: clone of 'https://github.com/zigurous/docjs' into submodule path 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~' failed
    Failed to clone 'docs~'. Retry scheduled
    Cloning into 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~'...
    remote: Repository not found.
    fatal: repository 'https://github.com/zigurous/docjs/' not found
    fatal: clone of 'https://github.com/zigurous/docjs' into submodule path 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~' failed
    Failed to clone 'docs~' a second time, aborting
 [NotFound]
UnityEditor.EditorApplication:Internal_CallUpdateFunctions ()

The repository https://github.com/zigurous/docjs/ no longer exists and it is preventing Unity from adding the repo to the Package Manager.

The following error occurs while adding the GitHub repo to the Package Manager with git URL.
Cannot perform upm operation: Unable to add package [https://github.com/zigurous/unity-ui-toolkit.git]:
  Cannot checkout repository [https://github.com/zigurous/unity-ui-toolkit.git]:
    Error when executing git command. Submodule 'docs~' (https://github.com/zigurous/docjs) registered for path 'docs~'
    Cloning into 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~'...
    remote: Repository not found.
    fatal: repository 'https://github.com/zigurous/docjs/' not found
    fatal: clone of 'https://github.com/zigurous/docjs' into submodule path 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~' failed
    Failed to clone 'docs~'. Retry scheduled
    Cloning into 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~'...
    remote: Repository not found.
    fatal: repository 'https://github.com/zigurous/docjs/' not found
    fatal: clone of 'https://github.com/zigurous/docjs' into submodule path 'D:/Github/Project/Library/PackageCache/.tmp20680RAo8EXwhJocw/clone/docs~' failed
    Failed to clone 'docs~' a second time, aborting
 [NotFound]
UnityEditor.EditorApplication:Internal_CallUpdateFunctions ()

The repository https://github.com/zigurous/docjs/ no longer exists and it is preventing Unity from adding the repo to the Package Manager.

I realize this is an extremely late response, but I have fixed this issue as of November/December of 2021.


chore(artifact-caching-proxy): increase resources

Node capacity on

prodpublick8s: Standard_D8s_v3, 8 vCPU 32Go mem
doks-public: s-4vcpu-8gb, 4 vCPU 8Go mem
eks-public: t4g.xlarge, 4 vCPU 16Go mem

um does it really need that much?
where did these numbers come from?
is it just spare capacity on the node not being used for anything else?

For the vCPU: it's nginx so 1 vCPU for the (master process + cache manager), and then 1vCPU per worker process.
The choice of 3 vCPU is to ensure that there are always 1 vCPU available for worker (when there is a reload, nginx duplicates the worker process with the new conf).
For the memory, usually 1 Gb per CPU is ok on a standard reverse proxy or webserver usage. But with the cache manager, better to get more memory to have some room so 2Gb per vCPU.

For the memory, usually 1 Gb per CPU is ok on a standard reverse proxy or webserver usage. But with the cache manager, better to get more memory to have some room so 2Gb per vCPU.

has it been tested as needing that much?

For the memory, usually 1 Gb per CPU is ok on a standard reverse proxy or webserver usage. But with the cache manager, better to get more memory to have some room so 2Gb per vCPU.

has it been tested as needing that much?

For the >1 vCPU, yes (we need at least 2).
For the 3vCPUs, no, we can start at 2. But the 2 -> 3 difference does not matter as 2 of the 3 installations (EKS-public and DOKS-public) only have 1 worker node with only nginx within, and the 3rd cluster (AKS - prodpublick8s) had to scale up to fit even a 2CPU/2Gb service)
For the 2 Gb per vCPU: yes on the initial pod on AKS.


Unable to map keyboard "Escape" key to Function button.

A few months ago I purchased a Haute42 Stickless (16+5 button layout). It's been working great so far and I have used the web configurator numerous times without issue. Recently I purchased Street Fighter 6 and for whatever reason there is no way in-game to remap the Escape key (which is used for opening several important menus) to anything else, so I am attempting to set up an SF6 profile on the stickless with the configurator to have the Function key act as a press of Escape. I currently have Fn mapped to Pin 26 and in the Keyboard Mapping section of the configurator there is an explict option to map the Fn key to any other standard keyboard key of my choosing, but whenever I try to do so (and then save it at the bottom), when I reboot the device to try it out it automatically unmaps itself. Is there a reason for this? If there is, then why is there even the option to remap the Fn key in the Keyboard Mapping section to begin with? Is there any way to fix it? I would prefer not to remap any of the other Option keys as I have them set up in a specific manner for Fightcade and also don't actually use the Fn key for anything else right now as I don't really have any need for any of the options offer in the Function menu for the time being so I really would just prefer to remap it as specified above if possible.
Additional Info:
Stickless - Haute42 T-Series 16+5 Stickless
OS - Windows 10
Browser - Current version of Ungoogled-Chromium (in all the ways that actually matter it has version parity with and functions exactly the same as a standard installation of Chromium, but with Google's spyware removed from it)

@skdursh what mode are you using the controller in?

@skdursh what mode are you using the controller in?

Keyboard mode.

@skdursh can you please attach a backup of your web-config so that I can load it onto my T16.
Also, what is a 16+5 stickless?

Also @skdursh what version of the firmware are you using?

I think the ability to map Fn in the keyboard mapping is an error of older firmware, BTW. The Fn key is not meant to be sent out to the host, rather it is a button that is only used by the internal logic for chord-like hotkey behavior. Checking main, it is no longer possible to assign Fn to a keyboard key, and instead there are new extra buttons that you might be able to use instead. I'm going to close this and hopefully 0.7.10 solves your use case.


Failures when calling Party service are not handled correctly

There's a lot of complex internal queuing logic for creating a party for each sample unit, but in the event that the party service gets an I/O exception or returns a non-2XX response, we will fail with only a log message and the system will be left in an inconsistent state.
Code:
PartySvcClientServiceImpl.java - line 57 & 63
If we get a 4XX or a 5XX, or an unparseable response, we should throw an exception so that the POST is retried. If the POST fails for all the retry attempts, we need to throw an exception to the caller so that the message isn't consumed from the queue, leaving the system in an inconsistent state.
We could end up with parties missing for some SampleUnits.

I don't think the library has been fully understood when people have been implementing these API calls. I'm pretty sure spring throws a runtime exception of time RestClientException when it gets a > 400 code


Implement Rivers Project

A project should exist to handle rivers.bmp. We don't necessarily need to implement a way to edit rivers yet, but we should at least be able to save, load, import, and export rivers. This is necessary for #49 .

Linked with #48


Removal of Maxcaps via cvar

About the PR

Praise be community feedback. Let the world be free of funny atmos bombs.
Single line comment out of the tankexplosion trigger.
Why / Balance

Usecase is so small that its almost always used to commit rulebreaks rather than reasonable usage.

Traitor without DAGD removes entire department(s)
Non-antag demolishes security for the tiniest slight
"accidental" maxcap creation disregarding the fact its a few specific steps to create.
painful optimization on the side of the player to create maxcap "grenades" to eliminate all risk to self.

While I do wish for there to be a cleaner and more efficient change to maxcaps.
Until such a change is proposed and implimented, this is fit for purpose.
Technical details

No more gastank explosions is likely to cause issues down the line for pipe bursting etc without being a maxcap.
Media

no media.
Requirements

[x] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

remove: Gas Tank explosions, Including but not limited to maxcaps, have been disabled by default on WizDen Servers.

Holy based

Can this just be a cvar? I know many servers that would be very fine with having tankbombs enabled

why are you doing things like this with comments, add a cvar

should at most really be a nerf to make them more like grenades and not bombs
otherwise, i think they're good as a factor that adds a healthy amount of chaos to the round

was there a discussion about it? can you please link it?

why is max radius not a cvar yet anyway

why is max radius not a cvar yet anyway

Fair

actually yes just make max radius a cvar and if you want to remove maxcaps set it to 0 or at least something like 8 where they're mostly harmless

alternatively we could give them a special explosion damage type which does no structural and can't space floor, plus give it lower damage against people as well

alternatively we could give them a special explosion damage type which does no structural and can't space floor, plus give it lower damage against people as well

This should be configurable in that case I think. But just limiting radius is probably good enough.

yeah, nerf everything but buff security, we shouldn't have fun in a round

remove: Gas Tank explosions have been removed. Including but not limited to maxcaps.

This changelog could be a bit misleading for downstreams since they are actually enabled by default, and only disabled in the wizden server config.

As a game admin, I've had instances where I wish I could disable maxcaps temporarily, like I see atmos making a maxcap and I don't know what they intend to do with it. Like they could be a raider or maybe they just want to explode it in space for fun?
I think having a cvar for this can be very useful, but since disabling maxcaps by default is so controversial I think that should be done in a separate pr instead.

no fun allowed

yeah, nerf everything but buff security, we shouldn't have fun in a round
just a simple "No fun allowed"

reading comprehension 0 + security and command gets overwhelmed 9/10 rounds already :/

i've said this on discord before, but repeating this here:
this PR will prevent atmosians from making maxcaps; however, the syndies (or even the self-antags) doing it aren't going to go anywhere, so they will just do other thingsi suspect that we will instead get plasmafloods or stronger, which can and likely will be be even worse than maxcaps via being essentially always evac-worthy and affecting everyone and not a particular area if done properly
therefore, i wish for us to take into consideration that this PR will likely not cause the following changes, cause them to a lesser extent than expected, or even worsen the related situations:

self-antags causing mass death
lower admin pressure
lower sec pressure
make non-DAGD atmos traitors cause mass death

"removing maxcaps is a bad idea because then antags will just do something WORSE" is a bad argument because then we can just.. also nerf those worse things?

"removing maxcaps is a bad idea because then antags will just do something WORSE" is a bad argument because then we can just.. also nerf those worse things?

Yeah, as I said before, no fun allowed, buff security, nerf everything

"removing maxcaps is a bad idea because then antags will just do something WORSE" is a bad argument because then we can just.. also nerf those worse things?

then do nerf them?

If we just nerf everything at once then the argument "no fun allowed" makes sense at that point

Big part of community hate, when something gets nerfed cuz crying sec, which can't just stunmeta them

besides, even if you do nerf all of atmos, people will still be able to singuloose
and, once again, the atmos traitors you deprived of their past options will not go anywhere, they will just move on to doing other things, like singuloosing
maxcaps are one of the more tame options

i've thought about it a bit more, and come to a conclusion that this PR might be treating a symptom and not the cause
what is the cause, you ask? so, have you ever seen anyone get arrested for making maxcaps, as opposed to using them? no? here's your answerpersonally, i've had CEs and even captains walk by maxcap making setups while knowing what they are, maybe be at most a little concerned, then leave without doing anything
i think the issue is that for some reason command, sec, and other atmosians don't do anything even when it's known one atmos is making a maxcap, even though the setups for it are rather conspicuous and can even be distinguished by people that don't know how to make maxcaps

No

No

i've thought about it a bit more, and come to a conclusion that this PR might be treating a symptom and not the cause
what is the cause, you ask? so, have you ever seen anyone get arrested for making maxcaps, as opposed to using them? no? here's your answerpersonally, i've had CEs and even captains walk by maxcap making setups while knowing what they are, maybe be at most a little concerned, then leave without doing anything
i think the issue is that for some reason command, sec, and other atmosians don't do anything even when it's known one atmos is making a maxcap, even though the setups for it are rather conspicuous and can even be distinguished by people that don't know how to make maxcaps

very few people know can distinguish from an atmos making a maxcap versus normal atmos shenanigans
also isnt this kind of metagaming? "Oh, I checked your canister with my analyzer and I happen to know this specific ratio and temperature of gasses is a maxcap so you are going to super perma"

i've thought about it a bit more, and come to a conclusion that this PR might be treating a symptom and not the cause
what is the cause, you ask? so, have you ever seen anyone get arrested for making maxcaps, as opposed to using them? no? here's your answerpersonally, i've had CEs and even captains walk by maxcap making setups while knowing what they are, maybe be at most a little concerned, then leave without doing anything
i think the issue is that for some reason command, sec, and other atmosians don't do anything even when it's known one atmos is making a maxcap, even though the setups for it are rather conspicuous and can even be distinguished by people that don't know how to make maxcaps

I have like 1400 hours in this game and I still don't know how to identify a maxcap setup
If you know what a maxcap setup you are not likely to snitch on the person doing it either, because atmos has a cult of knowledge around creating WMDs just because they can as a flex and other atmos techs respect the art

I think pretending that max caps as they are, is ok, is hilariously insane. They either need to be reigned in to mini bomb sizes or removed. This is coming from the person who unnerfed them like two years ago.

This is coming from the person who unnerfed them like two years ago.

you mean eclips_e

read the pr you just linked

This is coming from the person who unnerfed them like two years ago.

you mean eclips_e

I think pretending that max caps as they are, is ok, is hilariously insane. They either need to be reigned in to mini bomb sizes or removed. This is coming from the person who unnerfed them like two years ago.

yes so nerf them instead of removing them, that's what my entire point earlier was about, that would avoid many of the issues
my suggestion earlier was to:

make them not deal structural
make them unable to space station floor
make them deal less damage overall

o h

I think pretending that max caps as they are, is ok, is hilariously insane. They either need to be reigned in to mini bomb sizes or removed. This is coming from the person who unnerfed them like two years ago.

yes so nerf them instead of removing them, that's what my entire point earlier was about, that would avoid many of the issues
my suggestion earlier was to:

make them not deal structural
make them unable to space station floor
make them deal less damage overall

They are perfect in the way they are now

at this point in favor of removing it solely because people are going so far as to use calculators to min-max their yields instead of just playing the fuckin game and figuring it out either through in-game discussions or trial and erroring shit
put the ti84s away ya fuckin dorks
EDIT: still downvoting this PR to contribute to it being the most disliked PR on the repo's history though

Kill yourself no fun allowing nerd.  Maxcaps are a non issue.   Feel free to discuss this using proper arguments, we read them and will consider them in the final decision. But harassing each other does not help in any way.

Just putting in my anecdotal experience here. In my ~500 hours of game play I can say with like 95% certainty that I have seen a maxcap maybe 1 time ever.

Maximum Capacity - also known as MaxCap - is the most powerufl bomb setup, intended to be used by antagonists and/or to fight antagonist in a good version of the game (Space Station 13), and since it's very first iteration did just that.
Inability for Administration to control and/or prevent and/or manage consenquences of "grief" should not be a reason to prevent a game mechanic to be removed.
If you plan, or wish to, remove something, you must reason it with "It breaks the game by itself", or "It abuses even within rule realm"
The only time reasoning of "People that break rules only do that" is applicable to actual exploits, bugs, and unintended mechanics.
Ability to make explosions using the oldest system of entire game, ported to its "successor" as SS14 proudly calls itself being removed thanks to these reasons brought in PR are silly. here's step by step reason.
"Well non-antags explode security for the tiniest slight!"
 Is the issue in max caps?
"Well traitors without DAGD removes entire department(s)"
Bombs can be used to cause diversion of attention, sabotage specific places (T-Comms, AI satellite, SECURITY BRIG filled to the brim with guns used to kill people, MEDICAL where wounded target and/or security and/or command are brought, etc etc, and creative use of very devastating and long-made explosive is beyond reason. Traitors already have ability to buy other type of explosives, here and now, but can take risky attempt to make manual bomb.
"Well people "accidentally" create maxcap disregarding the fact its a few specific steps to create!"
"I accidentally crashed the server with very specific exploit with very specific steps. Please remove bug and not me!" is something you'd also believe in.
"painful optimization on the side of the player to create maxcap "grenades" to eliminate all risk to self" is probably somewhat reasonable argument, but towards fixing lack of risk to self rather than entire removal of entire feature.
Skill issue.

tbh i also have seen like one or maybe even zero maxcaps in 3000 hours of playtime
and i don't really understand what is the issue here

I used to see them a bit before they got super nerfed, but if they are never seen, why not remove them. They cause more problems than not, and like the PR states, is against the rules in nearly all scenarios

IF the actual frequency of maxcap explosions factors into this decision, it would probably not be difficult to add logging to atmos/explosion calculations so that every time something explodes over value X it leaves a log entry. Then after a week or two we could actually see in the database how common these things are

So, my one experience with max caps was a traitor atmos tech made max caps and RRed more than 12 people by deleting med, sec, hops office, and cargo AND ruined the event the admin was running in under 2 minutes of throwing bombs. These are NOT hard to make, they require no skill just following a step by step guide and are only a single step above loosing in terms of creativity. The traitor in question didnt even have DAGD but theyd only targeted individuals they needed to technically so thry didnt break any rules, they did however ruin the round. So maxcaps either need a nerf or removed.

So, my one experience with max caps was a traitor atmos tech made max caps and RRed more than 12 people by deleting med, sec, hops office, and cargo AND ruined the event the admin was running in under 2 minutes of throwing bombs. These are NOT hard to make, they require no skill just following a step by step guide and are only a single step above loosing in terms of creativity. The traitor in question didnt even have DAGD but theyd only targeted individuals they needed to technically so thry didnt break any rules, they did however ruin the round. So maxcaps either need a nerf or removed.

"This happened once" is not an argument. Rounds get ruined regularly by unlikely, random bullshite.
If maxcaps happened to ruin the whole round, without technically breaking rules, with any kind of regularity, then it would be an actual issue.

mald pr-

mid pr

So, my one experience with max caps was a traitor atmos tech made max caps and RRed more than 12 people by deleting med, sec, hops office, and cargo AND ruined the event the admin was running in under 2 minutes of throwing bombs. These are NOT hard to make, they require no skill just following a step by step guide and are only a single step above loosing in terms of creativity. The traitor in question didnt even have DAGD but theyd only targeted individuals they needed to technically so thry didnt break any rules, they did however ruin the round. So maxcaps either need a nerf or removed.

this is like saying meteors should be removed because they sometimes destroy evac.

this is like saying meteors should be removed because they sometimes destroy evac.

Except someone is working on meteor defenses to stop meteors from hitting vital areas before the station is ready, because they recognize it as a problem. No one is saying maxcaps can’t be reintroduced in a better form in the future, but as they are now they are unbalanced rule breaking devices 99 times outta 100.

So, my one experience with max caps was a traitor atmos tech made max caps and RRed more than 12 people by deleting med, sec, hops office, and cargo AND ruined the event the admin was running in under 2 minutes of throwing bombs. These are NOT hard to make, they require no skill just following a step by step guide and are only a single step above loosing in terms of creativity. The traitor in question didnt even have DAGD but theyd only targeted individuals they needed to technically so thry didnt break any rules, they did however ruin the round. So maxcaps either need a nerf or removed.

how long ago was that? maxcaps have been directly or indirectly nerfed numerous times, so anything before 2024 was with maxcaps that were considerably stronger

this is like saying meteors should be removed because they sometimes destroy evac.

Except someone is working on meteor defenses to stop meteors from hitting vital areas before the station is ready, because they recognize it as a problem. No one is saying maxcaps can’t be reintroduced in a better form in the future, but as they are now they are unbalanced rule breaking devices 99 times outta 100.

First that someone has to finishing reworking salv, then maybe work on revenant\xenobiology\xenoarcheology...etc. what I'm trying to say is that I think it's low in the priority and won't be seen anytime soon. (Meteor defense, that is) I'm not laying blame at all id rather everything else they are working on first personally lol .

this is like saying meteors should be removed because they sometimes destroy evac.

Except someone is working on meteor defenses to stop meteors from hitting vital areas before the station is ready, because they recognize it as a problem. No one is saying maxcaps can’t be reintroduced in a better form in the future, but as they are now they are unbalanced rule breaking devices 99 times outta 100.

The meteor active defense need to happen because current meteors are unfun - you're just warned that you're gonna take meteors to the face and there's nothing you can do to prevent it. The only agenda players have is repairing the (potentially extensive) damage. And meteors happen several times per round.
On the opposite side of the balance, you have maxcaps. While being able to do massive damage, they only happen once in a blue moon and are an entirely player-controlled mechanic. Having a maxcap "ruin" a round is a player-controlled event in itself. This PR is entirely unneeded.

Please hold while I change every thing to sub.cvar wahoo.

Subs.Cvar complete. We game.

Since we are on the topic of nerfing maxcaps, how about something like reducing range, increasing damage, and reducing structural damage or something like that?
Point 4 why/balance

Also "accidental" maxcaps are the same as "accidentally" connecting plasma to distro, and for maxcaps is even harder, i doubt theres a single person that has accidentally maxcapped, and i dont think it should be taken into account as a reason to nerf them
Yes thats why it was in big quotes, its "accidentally" not accidentally. The classic "I didn't know tanks blew up I just wanted to make a hot canister what"

How about something like the smaller the range the more damage they do? it wouln't make much sense tho, but from a gameplay perspective, maybe it could help with those issues?
Point. 4. Why/Balance.

I need to get a DAGD to show people why this is an issue… maxcaps are exclusively used for mass murder, as per the point of a massive bomb. Sure, nerf the damage and range then… what? Nobody uses it because it's still a secret people keep that's not all that strong as strong as an bomb that chem can make that's much easier to identify? Nobody likes getting hit by a big bomb that kills them and nobody likes wimpy bombs that do not affect anyone. There is no counter for an atmos walking into med with a tank and a canister. 0 sound, a timer long enough for the person deploying it to get away, a tossable projectile.
I have 2 ideas that could replace them:

A randomization to the formula so Atmos has to test and it can't be made 5:00 into the shift
Maxcaps go “poof”, rapidly filling a room with the gas in the canister. Rapid sleep bombs, plasma canister scare, refill a room with air. More utility than a direct bomb(trigger would have to be different too)

The painful optimizations are not painful, and its the fun part, having the ability to play around with values and get different result is what makes them fun, and like a breath of fresh air after making frezon for the 50th time. (Obviously if someone plays around with the values in game and not on localhost its on them), also, the risk to self part, why should there be a risk to self? if i get dagd, i can plasma flood, hot vapor flood, singuloose, tesloose. If im a chem syndie i can make explosives, lethal foams, deadly poisons. If im a scientist i can make a borg army. All this has almost no risk, maybe the borg army.
All the explosives on the syndicate uplink have almost 0 risk to self, except for getting detected. Wouldn't a change like this have the same effect? If you see an atmos walk, throw a tank and it explodes, since now it has less range, theres a high chance people will witness it and survive, different from actual maxcaps in wich no witness survives.
Also this would make it so that its decently dodgeable, like grenades, and it would force atmos players to use those "painful" optimizations to tune the bombs to have fewer ticks till explosion, since it would be harder to aim at/hit people with less range.
Also Cojoke-dot, if they were weaker than chem bombs then theres no reason to remove them, being weaker shoulnt be a reason to just remove it, and what im saying is to nerf the range but buff the damage, so it woulnt be a wimpy bomb.
Also yeah, people dont like being blown up, but explosives are a thing in the game, unless we remove all explosives & meteors, dying to an explosion being boring is not a reason for  removing just maxcaps (if they had the range nerfed, how they are right now, yeah, its too big you cant even see it and you die)
So what i propose is just turning a weapon of mass destruction into a tool you can use to target specific people and kill them.
Also, for your point 2, tanks have only 5l and for rapidly filling the room with the content of the canister i can just open it, it would basically be the same as removing maxcaps, cause if i want to fill a room with the contents of a canister i can alredy do that

Turbo is referring to people making programs to maximize time - power. Get enough time to get away while dealing enough damage to remove a department. Removal of the risk that is meant to be inherent with using maxcaps through optimization. I figured out maxcaps with a tip here and there and did experiments. Fun times in atmos till I almost loosed the singularity and blew up the teg in one shift 🙃

I mean thats voluntary work, he made that program cause he wanted, i doubt he felt it was painful, and some atmos stuff requires calculations, i dont think this is an issue.
Also testing maxcaps in the middle of atmos in a normal round is on you, if you are going to test explosives, test them either in a platform in space, or in a localhost.

Its a tank, if someone throws a tank at you, you are not going to go into evasive maneuvers and flee for your life

They already one shot people, how much more would you wan to buff the damage? Craftable mini bombs that deal 200 blunt and have no beep?

We could add sprite once the tank reaches more than 1500kpa or something, and maybe a sound when it reaches 3000kpa+
Also, yeah maxcaps one shot people, but smaller atmos bombs dont, the idea would be that smaller atmos bombs also kill, to discourage the use of bigger radius.

Yes… none of the bombs are silent, disguised, and have a meaningful radius. The most you have is a pen bomb which is just a c4. 1/5 of your TC for one bomb compared to the infinite you can make at atmos

Well, chem can also mass produce bombs, 0 tc cost, that instakill, but with a smaller range, and chemicals that round remove people.

Maybe it can force a bit of the gas into people's lungs who have internals on and will not have a cap on pressure. It would be only useful in larger rooms.

Sounds cool honeslty, but i feel like it would only be useful if we were to add other gasses that have different effect when inhaled, right now, only gasses that would be useful to force inject into people are n2o and frezon, and if im going to make all the people in a room be in contact with a gas i made, first, it would only be useful for mass murder anyways, even more now with the cold slowdown nerf (cause before the cold was useful if you injected yourself with cryox).
Second, why frezon or n2o flood a room when i can flood it with hot vapor and insta kill everyone? Why force people to breathe n2o when they can wake up and toggle internals, instead of flooding with n2o its simply better to flood with frezon, wich will also make n2o and it has the benefit of also dealing damage to people that dont have internals. So you'd only flood a room with either frezon, or superheated gasses, both will end in mass murder, just that the hot gas will also round remove
Now, i cant deny that these changes would take more work than making maxcaps a cvar, to turn them to 0 and kill them, but i think it would be a healthier change. But i guess the answer i will get is wyci :godo:

@TurboTrackerss14 I pushed some minor nitpicky changes directly to your branch. I'm not super sure how Subs.CVar works, but does this correctly call UpdateMaxRange() on init when the CVar is set to something else (e.g. 0) in the config file? If you can't look into this I'll take a look at this a bit later.

@TurboTrackerss14 I pushed some minor nitpicky changes directly to your branch. I'm not super sure how Subs.CVar works, but does this correctly call UpdateMaxRange() on init when the CVar is set to something else (e.g. 0) in the config file? If you can't look into this I'll take a look at this a bit later.

Changes look fine to me
(Fuck I forgot to delete the shutdown method entirely aswell thanks for that)

I blindly trusted the comments from contrib-voice during my shitcode fixing stream.
Subs.CVar is a magical entity to me that I made work.
If you are willing to prettify my code I would be more than allowing of it.

Looks to be all set 🫡

Hey everyone, we had a very long discussion and multiple votes with the maintainers and admins. We came to the conclusion that max caps are some kind of tradition from SS13 and an interesting result of our atmos simulation, that deserves to be kept in some form in the future. However, due to problems with their current mechanics and the huge burden on our game admins, we have decided to limit their explosion radius using a cvar until we can rework some of the mechanics behind maxcap production and turn them into a proper feature. Servers that wish to keep them as they currently are can simply change that cvar.
Current problems:

From a WizDen server rule perspective the DAGD objective is currently the only acceptable reason to maxcap.
Even more problematic are the timed, easily mass-producable minicap grenades with smaller radius, that allow the user to run of with zero risk to themselves, allowing players to do massive station destruction.
If we were to reduce the cvar to anything high enough so that it would not affect these, it would only further encourage their mass production.
Most players making maxcaps simply follow a recipe without doing their own research and experimentation. From a gameplay perspective atmos should be more than a list of steps you have to follow.
The steps to make a maxcaps are unknown enough so that many players don't notice it, even when done in plain sight and there is no warning or way to stop the bomb.

What a good solution for a rework should include:

They should be turned into a proper game mechanic with a need for experimentation and a chance of failure.
They should be properly telegraphed, so there is a way to stop the bomber.
Their production should not be easily repeatable using a recipe and they should not be mass producible.

Some ideas we will investigate:

Make the canister the exploding part instead of the tank. This means a lot more gas will be needed for a single bomb and they cannot be thrown.
Randomly vary the heat capacities of gases by a few percent each round. This won't be noticable for normal gameplay, but will ensure that maxcaps are a simple recipe that you cannot look up. Instead the player would have to do some experiments and apply the scientific method to find out the current values, then adjust the recipe.
Make them require more items than tanks and canisters. This could be a piping setup at the location of the explosion or a traitor only item.

Randomly vary the heat capacities of gases by a few percent each round. This won't be noticable for normal gameplay, but will ensure that maxcaps are not a simple recipe that you can look up somewhere. Instead the player would have to do some experiments and apply the scientific method to find out the current values, then adjust the recipe.

would just mean measure some gases heat capacity at the start then recalculate the ratios, gatekeeping it a bit but thats all

Womp womp

Yeeess siiiiirrrr

I updated the title for server owners (well the commit already made it in now soo)
If you don't like this change for your FORK you don't have to revert this pr.
The default value is the same as before, only wizards den servers are effected


Crash when starting game

Hello again! I am back with another crash :) I was able to add the bots just fine but when I tried to start game it crashed. Here is the Latest Log and Crash Log. Thank you so much for the quick update last time!
crash-2022-02-07_21.34.56-server.txt
latest.log

Thanks for your report. This issue should have been resolved.

It works! Thank you!


[stable/redis] fix master joining cluster as slave

Fixes #15115
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm

/retest


Some tests for language translations

I added a commit to remove the test:current script from package.json to avoid an unnecessary merge conflict.


Polkadotjs Assumes a 50% + 1 External Propose Majority

However, Frequency currently has it set to 3/5 a super majority.
This will mean the UI allows it to be closed before the correct majority is reached and will fail with BadOrigin (due to a lack of enough votes).
On Mainnet this is not an issue as the 3/5 is the same as a simple majority as there are 5 council members. (Testnet has 11).
Proposed: We should switch to a simple majority as well.
https://github.com/LibertyDSNP/frequency/blob/1535f15386791f79db8c808d22df7ad7c2883e0f/runtime/frequency/src/lib.rs#L611
/// A simple-majority of 50% + 1 can have the next scheduled referendum be a straight majority-carries vote.
type ExternalMajorityOrigin = EitherOfDiverse<
  pallet_collective::EnsureProportionAtLeast<AccountId, CouncilCollective, 1, 2>,
  frame_system::EnsureRoot<AccountId>,
>;

The commit hash doesn't appear to be correct.  It doesn't show any changes to lib.rs.

@rlaferla that's just showing where it is. The inline code is the suggested change.


[ARRDT] Use Apollo cache redirect instead of reading from cache ourselves

Related to #113
Follow-up work defined in https://github.com/microsoft/graphitation/issues/126

Going for the merge


Add files via upload

For this work we learned new functions and methods for the python types: tuple, lists and strings.
With the exercises we understand how work "print" because we can use that for show the result of a code or determinate what element is in determinate coordinate. Also, we noticed that not all python types can change after it has been created. As well, there is a len function to determinate the length of the different python types. There is other functions like sum that is used to achieve the total sum of all the results. Finally, the map function wich use "lambda" to define a function that apply for all python types in each element y give us an specific result.
In addition, we use the split function for eliminate all the blank spaces and it give all the elements separet by a coma. For get a sentence we can use the join function. And to get the max and min value also its index we define the list of values and the function. We use len function before but in this exercises we noticed that we also can introduce a condition. Also when we find an element in a string we must remember that the result will give the position minus one. Finally, we can define what we want to do in a python type. For that we begin with "def" and finish with "return" and the result.


ARTEMIS-4443 support broker plugin config via broker properties

I need to revisit, registerBrokerPlugins is part of the official api and documented as the way to register/install a plugin.
I need to rework it to support a two stage add and register, but must respect that api in some way.

by making registerBrokerPlugins check for contains on each of the typed collections, it is safe to have it called twice in the case or programmatic calls before server.start. Doing it at start allows entries added to brokerPlugins to be properly registered.

@gtully this is a cool improvement!
What about using an annotation to avoid the suffix .class and the default getName method?
   public @interface BrokerProperty {
      enum NameType {
         PROPERTY,
         CLASS
      }
      public NameType itemName() default NameType.PROPERTY;
   }

   @Override
   @BrokerProperty(itemName = BrokerProperty.NameType.CLASS)
   public List<ActiveMQServerBasePlugin> getBrokerPlugins() {
   ...

Thanks Dom, not yet sure of the annotation gives any value, but I see that we never reuse the configuration instance on reload, and it may be that a plugin is never referenced for configuration purposes, meaning a name attribute won't be necessary, everything needs to go through init which is a single line. That can simplify the use case a lot


version V2.0.1 beta 2 login error on AXhub

Hi, login errorn in AXhub with version V2.0.1 beta 2 and HA 2023.8.0
Logger: custom_components.hikvision_axpro.hikax.hikax
Source: custom_components/hikvision_axpro/hikax/hikax.py:208
Integration: hikvision_axpro
First occurred: 17:37:16 (4 occurrences)
Last logged: 17:57:23
Debug data  e662a2b75412dc5d59f9 aa9d917da927f6460f0013c22146aa52 100 true
Debug data  70eff9f5bd8c929e15f0 d3554b3f52dbe58ebc330f24322f983e 100 true
Debug data  d6e4e6ef1cd225d9a466 2594637f2b91eb14ef4671564f060ee7 100 true
Debug data  47159f96a1cb3360e724 8ba2b8f61f3a938ffc1964d5210ab702 100 true

Hi, login errorn in AXhub with version V2.0.1 beta 2 and HA 2023.8.0
Logger: custom_components.hikvision_axpro.hikax.hikax Source: custom_components/hikvision_axpro/hikax/hikax.py:208 Integration: hikvision_axpro First occurred: 17:37:16 (4 occurrences) Last logged: 17:57:23
Debug data e662a2b75412dc5d59f9 aa9d917da927f6460f0013c22146aa52 100 true Debug data 70eff9f5bd8c929e15f0 d3554b3f52dbe58ebc330f24322f983e 100 true Debug data d6e4e6ef1cd225d9a466 2594637f2b91eb14ef4671564f060ee7 100 true Debug data 47159f96a1cb3360e724 8ba2b8f61f3a938ffc1964d5210ab702 100 true

Thanks for the report. As @petrleocompel claims the login issue was fixed in beta2. Will hold on updating.

@youmax01 are you sure, you are using latest v2.1.0-beta2?

I see. Please from version 2! do not use AXHub configuration option.

beta2 is not yet on HACS repo, mb it takes some time to appear

You just need to search for the integration go to its detail and click on Update information

You just need to search for the integration go to its detail and click on Update information

thank you it worked! will try now

Confirm that login issue is fixed! thank you!
Hope to see sirens in supported devices soon 🙏  did not find siren support in this beta yet

@pophitru
Currently. I added relay support https://github.com/petrleocompel/hikaxpro_hacs/commit/34f770bc32bef71b4f62d057c10e3e3368f8e14e
And tomorrow is test time. And then sirens are going to be next.

good news! I've got AX PRO DS-PS1-I-WE in case you need some testing

@youmax01 I see. ⚠ Please from version 2! do not use AXHub configuration option. ⚠
See changelog of release 2.0.0

Hi!, yes, beta version is online

y try back to V2.0.3 but dont work either
HA version maybe?, 2023.8.0?

Sorry, back to V2.0.3 and work again!, lastest non beta channel dont.

Sorry, back to V2.0.3 and work again!, lastest non beta channel dont.

Did you uncheck "AXHub" in configuration? From version 2 it is deprecated. Everything works without it.

@youmax01  any update ?

Closing - inactivity


Get action to work for PRs from external forks

We need the action to work from external forks, for this we need to pick up the unmerged changes from Ross Williams in his PR 6

Closed by #2


Cannot connect to Azure Sql Database

I tried running a test vs an Azure Sql Database but I can't seem to connect to it while I'm sure the username and password are ok.  I am perfectly able to connect with this connection string in my java application or SquirellSQL query tool.
Here is my test, I blanked out the domain and password in the connection string I can say however the password contains these characters: +$(%
I also url encoded the password in one of my attempts to get it fixed
import http from 'k6/http';
import sql from 'k6/x/sql';
import { htmlReport } from "https://raw.githubusercontent.com/benc-uk/k6-reporter/main/dist/bundle.js";
import { sleep } from 'k6';

export const options = {
  vus: 10,
  duration: '30s'
};

const db = sql.open('sqlserver', 'jdbc:sqlserver://xxxxxxxx-a-sqls-001-trip.database.windows.net:1433;database=XXXXXXX-A-SQLD-001-TRIP-DIENSTREGELINGEN;user=trip-admin@xxxxxxxxx-a-sqls-001-trip;password=xxxxxxxxxxx;encrypt=false;trustServerCertificate=true;hostNameInCertificate=*.database.windows.net;loginTimeout=30;');

export default function () {
  const results = sql.query(db, 'SELECT count(*) FROM Trip;');
  sleep(0.3);
}

export function teardown() {
  db.close();
}

export function handleSummary(data) {
  return {
    "summary.html": htmlReport(data),
  };
}

output:
ERRO[0006] GoError: mssql: login error: Login failed for user 'trip-admin@xxxxxxxx-a-sqls-001-trip'.
        at reflect.methodValueCall (native)
        at file:///C:/Tools/k6/jdbctest.js:14:28(6)  executor=constant-vus scenario=default source=stacktrace
ERRO[0006] GoError: mssql: login error: Login failed for user 'trip-admin@xxxxxxxx-a-sqls-001-trip'.
        at reflect.methodValueCall (native)
        at file:///C:/Tools/k6/jdbctest.js:14:28(6)  executor=constant-vus scenario=default source=stacktrace
ERRO[0006] GoError: mssql: login error: Login failed for user 'trip-admin@xxxxxxxx-a-sqls-001-trip'.
        at reflect.methodValueCall (native)
        at file:///C:/Tools/k6/jdbctest.js:14:28(6)  executor=constant-vus scenario=default source=stacktrace`
`

I got another error now
ERRO[0005] GoError: lookup xxxxxxx-a-sqls-001-trip.database.windows.net:1433: no such host
at reflect.methodValueCall (native)
at file:///C:/Tools/k6/jdbctest.js:14:28(6)  executor=constant-vus scenario=default source=stacktrace
at least it is not a login failed without explanation why

Found a way to connect
const db = sql.open('sqlserver', 'Server=tcp:xxxxxx-a-sqls-001-trip.database.windows.net,1433;Initial Catalog=ARDELIJN-A-SQLD-001-TRIP-DIENSTREGELINGEN;Persist Security Info=False;User ID=trip-admin;Password=xxxxxxxx;MultipleActiveResultSets=False;Encrypt=True;TrustServerCertificate=False;Connection Timeout=30;');
Seems I had to use ADO as you said but add tcp: in front of the host, maybe append the documentation that jdbc wouldn't work ? and you need to use the ADO format ?

Great that you got it working. That's good to know.
JDBC is specific to Java, so we shouldn't need to mention it. And go-mssqldb supports 3 ways of specifying the connection string, so ADO is not strictly required. I would assume that a URL would also work, but you'd have to URL encode it properly.


Specify other timezones via offset

Hi! I find your extension super useful and use it often. One enhancement that would be great to have would be the ability to specify other timezones via offset. For example, the ability to enter:
2023-04-27T12:40-06:00 to specify that the timezone is PDT
Right now, the functionality doesn't seem to be possible.

Thank you for the feedback!
Sounds like a useful enhancement, let me consider it for a bit :)


Merge dev to main

Summary of Changes

#34: adds MIGraphX backend
#35: refactor client-server API
Add pre-commit for formatting

Motivation
Since PR validation is now supported in the repo, we no longer need a long-running dev branch. In the future, all PRs can go against main directly after validation.
Implementation
See the other PRs for more details

Can one of the admins verify this patch?

test this please

Build Failed! :(

ok to test

Build Passed!


[Bug] dbt enforced data_type checks doesn't check complex data types in Databricks

Is this a new bug in dbt-spark?

[X] I believe this is a new bug in dbt-spark
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
When I define model like following, and execute dbt run command, command will succeed if metadata field is really of type STRUCT, but underlying keys in the struct and their corresponding data types won't be checked.
models:
  - name: test_model
    config:
      contract:
        enforced: true
        alias_types: false
    columns:
      - name: id
        data_type: STRING

      - name: metadata
        data_type: STRUCT<property_1 STRING, property_2 STRING>

SQL model:
  select
    id,
    CAST(metadata AS STRUCT<property_3 STRING> as metadata
  from {{ source('source', 'source_table') }}

I've noticed this when I tried testing and changing my data_type from STRUCT to MAP. In that case dbt run command retuned this error:

definition_type and contract_type are not taking into account values inside complex/nested structures.
Expected Behavior
I would expect for dbt to fail if contract is enforced, and complex data types like ARRAY, MAP and STRUCT are not fully described, or even better under contract configuration to have check_complex_types boolean field which can decide whether to check nested structures or not.
Steps To Reproduce
Using model configuration and similar SQL command (that contains STRUCT field) from above, execute dbt run command. Command will succeed no matter what underlying struct configuration is setup in data_type field (works with setting just STRUCT<>).
Relevant log output
No response
Environment
- OS:
- Python: 3.11.9
- dbt-core: dbt Cloud CLI - 0.38.15
- dbt-spark:

Additional Context
No response

Hello @zeljkostojkovic  - are you using the dbt-databricks adapter or dbt-spark to connect to Databricks?

HI @amychen1776, I've installed dbt Cloud CLI locally, but based on manifest.json logs "adapter_type": "databricks", it's using dbt-databricks adapter.

Thank @zeljkostojkovic  - transferring your issue over to dbt-databricks


[New extension] - Frame Crop - Discover Art for your TV

Description
New extension built by the Frame Crop team to discover/download artworks curated from the Frame Crop api. Allows users with art-focused TV's to download artwork to their local machine in order to upload to their TV.
Screencast
https://share.cleanshot.com/DQZlWHgz
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I don't know why but somehow an old extension I worked on made it into these commits, I remove it from this branch but the commits are still there.


[Feature] Default sorting

I'm not sure what default sorting is right now (order of insertion?)
But seems like each space/folder and subfolder has some order that keeps periodically changing things around on its own.
I have to explicitly set A-Z sorting on each subfolder which is a pain.
Would be nice to have a setting for default sorting where I can set to A-Z

My case is that I want to implement swimlaned kanban notes which is described in a members only Medium article. It requires make.md. I installed the plugin and now all my carefully organized notes that sort the way I want them to are totally messed up. I've looked in the documentation on how to set sorting but there is nothing I can find that explains how to set it back to what Obsidian defaulted to. My request is to leave sorting alone until user changes it. Not the other way around.


Fix zookeeper kubernetes.json (label values must be strings, not ints)

Can one of the admins verify this patch?

ok to test

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/258/
Test PASSed.

Merged as a4d6c3427a386ebf88c6d6be9f0c359f693dcf73. Thanks @luksa!


Newton: openstack-ansible-security : V-38660 failure

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.
What type of issue is this? Please tick the appropriate box.

[x] bug - existing functionality that does not work for you as documented
[ ] feature - new functionality that you would like to see added

What is the expected behavior?
We need to investigate the snmp configuration in regards to HP/Dell montioring tools and our default configuration if managed by kick.
We may need to add migrations steps to fix the snmpd config.
Additionally it seems the egrep is a little to greedy....
What is actually happening?
TASK [openstack-ansible-security : V-38660 - The snmpd service must only use SNMPv3 or newer] ***
fatal: [storage02]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [storage01]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [storage03]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [infra03]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [infra01]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [infra02]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [logging01]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [compute01]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}
fatal: [compute02]: FAILED! => {"changed": false, "failed": true, "msg": "Insecure SNMP configuration found -- use SNMPv3 only"}

NO MORE HOSTS LEFT *************************************************************

egrep 'v1|v2c|com2sec|community' /etc/snmp/snmpd.conf | grep -v '^\\s*#'
#rocommunity public  localhost
 rocommunity public  default    -V systemonly
                                                 #     settings, change the community string,
#rocommunity secret  <IP_ADDRESS>/16
#  It's no longer typically necessary to use the full 'com2sec/group/access' configuration
#  r[ow]user and r[ow]community, together with suitable views, should cover most requirements
                                    #   send SNMPv1  traps
                                    #   send SNMPv2c traps
                                    #   send SNMPv2c INFORMs

What were you attempting to do when you encountered this error (give as much detail as possible)?
How can we reproduce this issue?
Run stable/newton security hardening on 14.04 hosts
How can we verify your issue is addressed?
See above
If you are using or know of a workaround please describe the necessary steps?
Disabling security hardening/skipping check
What is your evaluation on the importance of this issue, and why?
Prio 2

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

This doesn't affect any published releases, but would be good to look at as part of our newton effort.


Add value change by arrow keys support for BitRating (#1736)

This closes #1736

again, plz do not work on not-triaged issues!

since the related issue is closed, I'm closing this PR.


Gamma function has room for improvement

The gamma function produces wrong values for integer inputs.  For example, gamma(4) returns not 6, but 5.99999999999999, and gamma(5) returns not 24, but 24.00000000000011.  Similar errors happen for other integers.  I realize that some error is inevitable as doubles actually have discrete values.  But there are still two problems:

For integer inputs, it really should return the correct integer output.

The error in these particular cases are respectively 14x and 32x the granularity of double near their output values.  That seems like more than it would need to be.

I'm new to this project but may use it heavily.  Therefore, I'm willing to volunteer myself for a fix.  But it seems like the kind of thing that should be discussed a little first.

I just downloaded Math.Net numerics, and asked it to calculate SpecialFunctions.Gamma(5).  The correct answer is 24, but it returned 24.000000000000114.
Can this be fixed?  (My proposed changes would fix this.  I don't really care how we fix it; if there is a better way to fix it, I would be fine with that.)

I finally had a look at how the new algorithm performs numerically. It is indeed mostly 1-2 digits more accurate than the existing implementation and behaves very nicely around positive integers.
I noticed two potential problems so far:

Accuracy drops up to 5 digits around 10e-5 (but is much better again at 10e-10).
GammaLn no longer supports 0.0 and negative numbers (where Gamma is non-negative).

I've got a fix for the accuracy issue, by simply using more terms of the power series and moving the lower breakpoint.  I'll post once I've had a look at the other issue.

I've just posted a version that addresses both of your issues.  I'm returning NaN for GammaLn in cases where Gamma is negative.  But it's unclear if that's the right choice; one could instead return the real part.
https://github.com/mathnet/mathnet-numerics/pull/204/commits


Tracking Issue - Cosign signing via secrets configured on service account

This is a tracking issue for the work corresponding to accepted cosign RFC: https://github.com/pivotal/kpack/pull/765.

closed by #817

reopening due to Signing/Verifying builder/clusterBuilder images not yet being completed

Tracked here: https://github.com/pivotal/kpack/issues/942


feat: add metric rpcReqTotal for every method

Closing due to inactivity, please re-open if still requested with latest master


added data-uri support

As it states in http://lesscss.org/#reference
Inlines a resource and falls back to url() if the ieCompat option is on and the resource is too large, or if you use the function in the browser. If the mime is not given then node uses the mime package to determine the correct mime type.
Parameters:
mimetype: A mime type string. Optional.
url: The URL of the file to inline.

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

We are no longer maintaining this project and as such are arching all PRs.