[question/feature] Error returned from helm uninstall SDK is not detailed enough

Output of helm version:
v3.11.1（still the same in main branch）
When I use helm as SDK, I uninstall release by:
u := NewUninstall(cfg)
_, err := u.Run("releaseName")
if err != nil{
    log.Printf("uninstall failed, error is %+v", err)
    return err
}

But when there are something wrong in the enviroment, such as network crash, or I deleted CRD and controller (which defines the cr in this release) before I uninstall this release, the err will just return a string as failed to delete release: releaseName without any reason.
I found the code in helm is here:
https://github.com/helm/helm/blob/fe595b69d78b213ab181d98ce24dde2454a56f9d/pkg/action/uninstall.go#L122-L126
I understand it should failed in those condition, but the error it returns is not detailed enough, the only way to analyse the reason why delete failed is to dive into helm's log, which is might not the best way.
Can we change it into:
	deletedResources, kept, errs := u.deleteRelease(rel)
	if errs != nil {
		u.cfg.Log("uninstall: Failed to delete release: %s", errs)
		return nil, errors.Wrapf(combineErrors(errs), "failed to delete release: %s", name)
	}

'combineErrors()' can combine errors by divide them with ';'
Or any other better way.
If it's ok to change it this way, I'll be happy to rise a PR to fix this.

I think the expectation is for the SDK to read/export the log for diagnosis. Why not do that?
As a first quick thought, combineErrors like this would potentially make a very verbose error string.

I think the expectation is for the SDK to read/export the log for diagnosis. Why not do that?

Yes, when errors occurred, read the log of sdk is a good way to diagnose. But shouldn't the SDK return a more detailed error info? While other errors in uninstall are detailed enough, only when delete resources failed, it will not return the exact reason, it just return a simple string that tell you it failed. And the error is not nil is already showing that the release uninstalling was failed. The single message of failed to delete release: releaseName doesn't help at all.
Besides, if that was a network traffic, I could do a retry, if it was caused by uninstalled CRD, I can mention my users.

As a first quick thought, combineErrors like this would potentially make a very verbose error string.

Yes, it would be verbose (only when there are multiple errors during uninstalling), and that's the reason I use wrapf. if someone just need to know if it's failed, he/she just need to if err!= "", if he/she wants to print it not verbosely, it will be the same to call log.Print(err.Error())


Fix ConfigMap namespace

This is a mistake in the EKS page. The main page shows the correct namespace https://docs.projectcalico.org/maintenance/ebpf/enabling-bpf#configure-calico-to-talk-directly-to-the-api-server

 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.


Add compliance mode to coverage model

The compliance test suite mainly focus on spec compliance, the micro-architecture related coverpoint like hazard condition, branch hit/miss combinations should be disabled for compliance test.

add a runtime option "-compliance_mode" to cov.py
add macro `DV_CP(...) for the coverage point which only applies to verification

@simond-imperas @eroom1966
Please be aware of this change. You can now run cov.py with  "--compliance_mode". If you think any coverpoint needs to be excluded for compliance mode, please use `DV(xxx) macro.


No textures

I don't have any textures with any object of the ae2ao mod.

I have this with all the objects of the mod when I place it down. In my inventory I see them, but when placed down I get black and purple squares.

An interesting bug, I have not seen this before

Can you tell which mods were installed in the assembly?

Can you tell which mods were installed in the assembly?

Yes, they are all fabric mods.
B=Before the mod download
S=Same time the mod download
A=After the mod download
Iron furnace (B)
Additional bars (S)
Abacus (S)
Adabranium (S)
Adapaxel (S)
Adorn (S)
AE2AO (Mod itself)
All stackable (S)
Allure (S)
Applied energistic 2 (Mod itself)
Arsteurgia (S)
Better Ores (S)
Bits and Chisels (S)
Block Compression (S)
Branders Spawnermod (S)
Byg (B)
Curios (B)
Dank Storage (B)
Data-Loader (A)
Elevator (S)
Enchant plus (S)
Experience container (S)
Fabric-Api (S)
Fabric-installer (B)
Fabric-Language-Kotlin (A)
Fabric-magnet (B)
Hwyla (B)
Identity (B)
Iron Chest (B)
Lambdynamiclights (B)
Mechanix (A)
Modmenu (A)
Mostructures (S)
Mythic metals (S)
Optifine (B)
REI (B)
Shulkerboxtooltip (B)
Sodium (B)
Time Travel Mod (S)
Toro Health (S)
Unforgiving Void (S)
Veinmining (B)
World-Edit (B)
Wurst (B)
Xaeros minimap (A)
Xaeros worldmap (A)
Those are all the mods I have (46 in total)

ok thx

can you send me your assembly in the archive?

can you send me your assembly in the archive?

Idk how to😅😅

please send me an archive with mods

Sorry i Russian human and don't speak on English

I'm sure this is not related to my mod. he doesn’t investigate the graphical part, I want to help you

I need push access to allow me to send the file

can you upload the assembly to google disk and throw me a link?

@Nickstertjefloor you use sodium, which is not compatible with ae2
also this is not an issue with this mod, this is between ae2 and sodium

Sodium DOES NOT work with AE2!!!!!!


add onopen option

Idea was for a storage mechanic or backpack where i would need to fill the Gui on open with the content of items pdc/componentsnor whatever


Deploy COESSING hub

reverts 2i2c-org/infrastructure#1802
addresses #2844

Note that in #1802 the scratch buckets weren't deleted, so they should still be present


Links to URIs for mixs v6.0.0 return 404 error

Describe the bug
Hi,
thanks for the update of the MIxS! I tried to access to terms through their URIs as specified on the website or in the SHACL file but the links are dead and produce only 404 errors.
To Reproduce
Steps to reproduce the behavior:

Go to https://w3id.org/mixs/terms/0000050 or https://w3id.org/mixs/vocab/sequencing_field
See 404 error

Expected behavior
The page for the term of the MIxS and its description.
Best regards,
Charlie

Thanks @cpauvert
We are working on this. I'm not sure what the most recent and relevant existing issue for this is, but we should definitely revisit #246

I haven't pushed the latest links to w3id.org, so I don't expect them to work, but I hope we can fix it at the technical call tomorrow.
But first we need to resolve #391 about the page URLs.

I did not read this carefully enough the first time. the URL should be .../mixs/... but .../gensc/...
Please see https://github.com/GenomicsStandardsConsortium/mixs/issues/233#issuecomment-657666511 for the correct format for URIs. I will send an announcement as soon as the redirect is working properly.

Hi,
I tried again with the updated format indicated in the comment you suggested but instead of a 404 error, the page just does not load and throws a "This site can't be reached" page, and I'm redirected to gensc.org.
https://w3id.org/gensc/terms/MIXS_0000001
https://w3id.org/gensc/terms/MIXS_0000050

Hi @ramonawalls,
Was a decision made regarding a stable way to refer to the MiXS terms?
I followed the link you sent but the suggested link raises 500 internal server error (ex: https://w3id.org/gensc/terms/MIXS_0000001). Previously mentioned format raises 404 error (ex:  https://w3id.org/mixs/terms/MIXS_0000001) and the documentation pages (for instance: https://genomicsstandardsconsortium.github.io/mixs/seq_meth) points to an URI that does not work as well (ex: https://w3id.org/mixs/terms/0000050).
I am sorry to nag you again, but I am using a research data management platform that needs persistent identifiers for the metadata.
Thanks in advance,
Charlie

Sujay, is it fair to say that you are the best person to lock in generation of the numerical term IDs? I will help with the w3id resolution.

On Aug 30, 2022, at 8:14 AM, Charlie Pauvert @.***> wrote:
Hi @ramonawalls https://github.com/ramonawalls,
Was a decision made regarding a stable way to refer to the MiXS terms?
I followed the link you sent but the suggested link raises 500 internal server error (ex: https://w3id.org/gensc/terms/MIXS_0000001 https://w3id.org/gensc/terms/MIXS_0000001). Previously mentioned format raises 404 error (ex: https://w3id.org/mixs/terms/MIXS_0000001 https://w3id.org/mixs/terms/MIXS_0000001) and the documentation pages (for instance: https://genomicsstandardsconsortium.github.io/mixs/seq_meth https://genomicsstandardsconsortium.github.io/mixs/seq_meth) points to an URI that does not work as well (ex: https://w3id.org/mixs/terms/0000050 https://w3id.org/mixs/terms/0000050).
I am sorry to nag you again, but I am using a research data management platform that needs persistent identifiers for the metadata.
Thanks in advance,
Charlie
—
Reply to this email directly, view it on GitHub https://github.com/GenomicsStandardsConsortium/mixs/issues/390#issuecomment-1231583387, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGAEX6NC65QS6Y42LDVUVQLV3X3KXANCNFSM5TDTIUKA.
You are receiving this because you commented.

Fixed by @turbomam in #530 Thank you Mark!


2663 | Add support for ADMIN_USER_PASSWORD_AUTH by using the same flo…

…w as ADMIN_NO_SRP_AUTH

Coverage decreased (-0.02%) to 94.073% when pulling 97ba463e70918da599b0ab0439f271ab930eb168 on franz-see:moto/issues/2663 into 1db617a5309d2ac2fa3e6adc9fb3b90ed43c810a on spulec:master.

Implemented by #3547


Are you planing in supporting gpx files for coordinates?

I was wondering if this project will support gpx files at some point in time. I could not find this functionality in the current version.
thanks

Unlikely.  ExifTool can match image files to GPX files by timestamp.  Locations can be then be verified and tweaked if necessary in GeoTag.
See GeoTagging with ExifTool for examples.


double-beep: Watch  xingiahanvisa.net

double-beep requests the watch of the watch_keyword  xingiahanvisa\.net. See the MS search here and the Stack Exchange search here.

 xingiahanvisa\.net has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Shree in Charcoal HQ


Improve name fallback for GeoJSON

We have files that are generated with empty title and descriptions, but a filled key called country-name.
Eg:
"properties": {
  "title": "",
  "description": "",
  "country-name": "North Carolina",
  "two-letter-code": "NC",
  "fill": "#f0a3FF",
  "fill-opacity": "0.5
 },

The GeoJsonDataSource file loops through and attempts to figure out what name property to use. If country-name comes before title, it uses that, but if it comes after it assumes title, which is empty.
Current behavior:

title second, filled : country-name first, filled: uses title countryNameFirstTitleFilled.json
title first, filled : country-name second, filled: uses title countryNameFirstTitleEmpty.json
title first, empty : country-name second, filled: uses title titleFirstEmptyCountryNameSecond.json
title second, empty : country-name first, filled: uses country-name

https://github.com/CesiumGS/cesium/blob/main/packages/engine/Source/DataSources/GeoJsonDataSource.js#L151C11-L164
Is it possible/would it be ok to add an additional check to:

Check if value of key is empty and if so, check the next key.

The preferred behavior is to show x-name  even if it shows up after an empty title but only if the title is empty.

Thanks for the proposal @mfarmer-ara!
It looks like there is no defined behavior for this detailed in the GeoJSON spec. So we should be free to add additional behavior as needed, as long as it is documented.
Can you explain more what you mean by the "next" key? In JavaScript, object properties do not have a guaranteed order.

4.3.3 Object
An object is a member of the type Object. It is an unordered collection of properties each of which contains a primitive value, object, or function. A function stored in a property of an object is called a method.

Can you explain more what you mean by the "next" key? In JavaScript, object properties do not have a guaranteed order.

It seems like the order Cesium will loop through the keys in that linked function is dependent on the original order it is in the file (observation not guaranteed).
So, while it doesn't really matter what order the keys are in usually, since it appears their order is kept based on the original file, that function will return what it thinks is the best name based on the order they're lopped through.
Though now I'm confusing myself with the precedence that's in that function.
I may play around with this today/tomorrow to see if I can pin things down.

@ggetz I'm still poking at this, but it looks like my example files might have been too simple. I am not able to make it happen with my simpler files but a geojson file with a polygon with 212 points (x,y,z) is consistently saying name (properties.title) is defined and skipping the extra property check. Since I am not sure I can share that one, I made a crazy shape with geojson.io with a similar number of points.
Dragging and dropping this with the viewerDragDropMixin then clicking the shape shows the issue. It may be a race condition somewhere and not what I originally thought it might be. A smaller file (those shared earlier) the name is undefined but this file the name is defined as empty.
-- putting console.log({ name, defined: defined(name), properties }); in GeoJsonDataSource.js ~ line 136.
randomShape.json
{     "name": "",     "defined": true,     "properties": {         "title": "",         "description": "",         "country-name": "North Carolina",         "two-letter-code": "NC",         "fill": "#f0a3FF",         "fill-opacity": "0.5"     } }
Files shared earlier (simple shape)
{     "defined": false,     "properties": {         "title:": "",         "description": "",         "country-name": "North Carolina",         "two-letter-code": "NC",         "fill": "#f0a3FF",         "fill-opacity": "0.5"     } }

Sorry for the extra comments while I explored this issue. The fix I've looked is to simply check if title is empty next to the defined() check for GeoJSON data sources.
With only title (no name or includes name/title keys) present and title equal to empty string, the below behavior happens. Is that fine? It behaves just as if title didn't exist at all.

With only title (no name or includes name/title keys) present and title equal to empty string, the below behavior happens. Is that fine? It behaves just as if title didn't exist at all.

This seems in line with expect behavior, yes.


【PaddlePaddle Hackathon 4 No.232】add tests for assign/conv2d/conv2d_transpose/scale

add tests for assign, conv2d, conv2d_transpose, and scale op.

Thanks for the PR.
Can you change the title to make it specific for the affected frontend (which I understand is Paddlepaddle), given all major frontends will have a file called test_forward.py associated to them?

Is this one okay?

@siriuslay pls fix the error in ci gpu/link


ACME-Client: Cannot find DNS API hook for: dns_nsupdate

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
Similar to this issue.
When using ACME-Client to configure the API with RFC2136, the request or update of the certificate fails, and no specific reason is shown in the log files. Then enter the command from the log into the console, it displays "Cannot find DNS API hook for: dns_nsupdate."
Relevant log files
Logs from the file:
[Sat Sep 28 21:41:53 CST 2024] See: https://github.com/acmesh-official/acme.sh/wiki/How-to-debug-acme.sh
[Sat Sep 28 21:41:53 CST 2024] Please add '--debug' or '--log' to see more information.
[Sat Sep 28 21:41:53 CST 2024] Error adding TXT record to domain: _acme-challenge.mydomain.tld
[Sat Sep 28 21:41:53 CST 2024] error updating domain
[Sat Sep 28 21:41:53 CST 2024] adding _acme-challenge.mydomain.tld. 60 in txt "FaBVqRQfqohelqAfwu9kLjDbvhJVm_bboS7x-bsQu7E"
[Sat Sep 28 21:41:53 CST 2024] Adding TXT value: FaBVqRQfqohelqAfwu9kLjDbvhJVm_bboS7x-bsQu7E for domain: _acme-challenge.mydomain.tld
[Sat Sep 28 21:41:53 CST 2024] Getting webroot for domain='*.mydomain.tld'
[Sat Sep 28 21:41:53 CST 2024] Getting webroot for domain='mydomain.tld'
[Sat Sep 28 21:41:50 CST 2024] Multi domain='DNS:mydomain.tld,DNS:*.mydomain.tld'
[Sat Sep 28 21:41:50 CST 2024] Using CA: https://acme-staging-v02.api.letsencrypt.org/directory
Logs from console:
[Sat Sep 28 21:42:49 CST 2024] Using CA: https://acme-staging-v02.api.letsencrypt.org/directory
[Sat Sep 28 21:42:49 CST 2024] Multi domain='DNS:mydomain.tld,DNS:*.mydomain.tld'
[Sat Sep 28 21:42:52 CST 2024] Getting webroot for domain='mydomain.tld'
[Sat Sep 28 21:42:52 CST 2024] Getting webroot for domain='*.mydomain.tld'
[Sat Sep 28 21:42:52 CST 2024] Cannot find DNS API hook for: dns_nsupdate
[Sat Sep 28 21:42:52 CST 2024] You need to add the TXT record manually.
[Sat Sep 28 21:42:52 CST 2024] Add the following TXT record:
[Sat Sep 28 21:42:52 CST 2024] Domain: '_acme-challenge.mydomain.tld'
[Sat Sep 28 21:42:52 CST 2024] TXT value: 'EO45x0F4io_CouFe3oj3-91EpNF9VyhZtOE1y_tOAu4'
[Sat Sep 28 21:42:52 CST 2024] Please make sure to prepend '_acme-challenge.' to your domain
[Sat Sep 28 21:42:52 CST 2024] so that the resulting subdomain is: _acme-challenge.mydomain.tld
[Sat Sep 28 21:42:52 CST 2024] Cannot find DNS API hook for: dns_nsupdate
[Sat Sep 28 21:42:52 CST 2024] You need to add the TXT record manually.
[Sat Sep 28 21:42:52 CST 2024] Add the following TXT record:
[Sat Sep 28 21:42:52 CST 2024] Domain: '_acme-challenge.mydomain.tld'
[Sat Sep 28 21:42:52 CST 2024] TXT value: 'ZrsyQVYev59HxkVVgivxAFCuA0a8UT70meUdwZ462AI'
[Sat Sep 28 21:42:52 CST 2024] Please make sure to prepend '_acme-challenge.' to your domain
[Sat Sep 28 21:42:52 CST 2024] so that the resulting subdomain is: _acme-challenge.mydomain.tld
[Sat Sep 28 21:42:52 CST 2024] Please add the TXT records to the domains, and re-run with --renew.
[Sat Sep 28 21:42:52 CST 2024] Please add '--debug' or '--log' to see more information.
[Sat Sep 28 21:42:52 CST 2024] See: https://github.com/acmesh-official/acme.sh/wiki/How-to-debug-acme.sh
Environment
OPNsense 24.7.5-amd64
Intel® Core™ i5-3340 3.1Ghz Quad Core
Network Intel® I350-T4

After fresh installed 24.7, the issue still existed. Also, encountered the blank log issue as this.

For me downgrading to acme.sh-3.0.7_2 resolves the issue.

Cannot find dns api hook for:

Although I have not been able to reproduce this, the following patch may fix this issue:
opnsense-patch -c plugins d1ef1f93ce318926c14e14291a50c09ac6421f15

Please apply the patch and report whether or not this fixes the infamous  Cannot find dns api hook for: error message.
If this does not fix this error, please use the Acme Client GUI to change Log Level to debug 3 and try again. Afterwards please provide the resulting ACME Log.

Cannot find dns api hook for:

Although I have not been able to reproduce this, the following patch may fix this issue:
opnsense-patch -c plugins d1ef1f93ce318926c14e14291a50c09ac6421f15

Please apply the patch and report whether or not this fixes the infamous  Cannot find dns api hook for: error message.
If this does not fix this error, please use the Acme Client GUI to change Log Level to debug 3 and try again. Afterwards please provide the resulting ACME Log.

Works fine for me, but still got an error from console: could not open '': file not found".
Seemed this issue was already fixed on upstream realese 3.0.9, I temporary manual applied it and looked good.

Worked fine for me

Thanks for testing!

Seemed this issues was already fixed on upstream realese 3.0.9

Yes, this was my PR :grin: – will be available in the next OPNsense minor release.


netdata temperature plugin broken

Issue description
The temperature plugin is broken called: sensors
The reason is located in the first few lines of

libexec/netdata/python.d/python_modules/third_party/lm_sensors.py

to be exact here:
import ctypes.util

_libc = cdll.LoadLibrary(ctypes.util.find_library("c"))
_libc.free.argtypes = [c_void_p]
_hdl = cdll.LoadLibrary(ctypes.util.find_library("sensors"))

It searches for the libc library and the sensors c library with ctypes which doesn't succeed.
The fix is to define an environment variable in the service called LD_LIBRARY_PATH that points to the needed library directories. For more information look here:
https://docs.python.org/3.7/library/ctypes.html#finding-shared-libraries
Possible solution
Not tested but this could resolve the problem:
serviceConfig = {
Environment="LD_LIBRARY_PATH=${pkgs.lm_sensors}/lib:${pkgs.libgcc}/lib:$LD_LIBRARY_PATH";

};

Looks like this issue is solved https://github.com/NixOS/nixpkgs/commit/df885360a6d30f1c2a30deb45bee4ed070f5cef3#diff-c080a3ec04b35f95254741800e1a3697


code blocks appearing as normal text

ETH Ticker for example is configured as:
  eth:
    regex:             "^eth$|^eth .*$|^ethereum$|^eth.sh$"
    cmd:               eth.sh
    help:              gives Ethereum price info
    markdown_convert:  false
    formatted:         true
    code:              true

however the output is :

I would expect this to appear as a code block since code is set to true i.e. :

Seeing this myself on my own home server. Most importantly this makes the output of the weather builtin script look quite bad.
My current speculation is that this could be a bug from the client library in use (matrix-nio) or simply changes in the element client layer. Unfortunately, it appears that simply modifying one's commands.yaml to set code=false then send it as markdown along with the appropriate markdown modifiers (code fence) also fails to produce the desired outcome (instead producing the output verbatim, but not in a monospaced font). Attempts to pass in raw HTML to produce a code block after setting formatted=false surrounding this also fails to produce the desired outcome. I'll see about debugging it tonight since I have some free time.
Some details about my setup follow.
Client:

Element version: 1.7.24
olm version: 3.2.1
matrix-nio: 0.17.0

Server:

matrix-synapse 1.11.0-1
Ubuntu 20.04.1

matrix-eno-bot version sha: c67220057728988e035557709f333fb1f92b9cfc

When viewing the source of the messages, it became immediately clear that the format property of the message blob is not set for the affected messages (as well as the formatted_body). The code looks like it should be setting this given the configuration, but something must be going haywire for it not to be so.
Will grab a dump next from chat_commands.py to see whether it is a code issue with this project or whether the server is simply discarding the extra properties for some reason here.

Dang, looks like it's the code (added info print to get what content looked like). Guessing the values in YAML aren't translating to what I would think they are
2021-04-08 01:33:03,568 | chat_functions [INFO] {'body': 'houston: Partly cloudy +25°C (+26°C)  0.1mm', 'msgtype': 'm.notice'}

YAML looking like I'd expect given the example file, could be I need True and False instead literally?
  weather:
    regex: "^weather$|^tiempo$|^wetter$|^temps$|^weather .*$|^tiempo .*$|^eltiempo .*$|^wetter .*$|^temps .*$|^weather.sh$|^weather.sh .*"
    cmd: weather.sh
    help: give weather forecast
    markdown_convert: false
    formatted: true
    code: true

Looks like if the values are set in config, the result is that property is always False in send_text_to_room. This certainly explains why help is formatted correctly whereas the custom commands appear not to be.

Added debug print to command_dict.py to get what self.command_dict["commands"][command]["formatted"] actually is
Turns out == on the property is probably not the best idea, as the YAML parser reads true and false properly, negating the need for a "true" string literal check.
Fix is to use not not if the goal is boolean coercion to prevent user footgunning

PR submitted, let me know if y'all need me to follow up further :)
@atomtm Does the code change in 56deb2caf6c1505d4441e4bae06ebd73a6e17a8e fix it for you? Worked like a charm for me.

PR submitted, let me know if y'all need me to follow up further :)
@atomtm Does the code change in 56deb2c fix it for you? Worked like a charm for me.

this worked with :
"""
        if "markdown_convert" in self.command_dict["commands"][command].keys():
            return not self.command_dict["commands"][command]["markdown_convert"]
        else:
            return CommandDict.DEFAULT_OPT_MARKDOWN_CONVERT

@@ -232,7 +232,7 @@ def get_opt_formatted(self, command):
        """
        if "formatted" in self.command_dict["commands"][command].keys():
            return not self.command_dict["commands"][command]["formatted"]
        else:
            return CommandDict.DEFAULT_OPT_FORMATTED

@@ -245,7 +245,7 @@ def get_opt_code(self, command):
        """
        if "code" in self.command_dict["commands"][command].keys():
            return not self.command_dict["commands"][command]["code"]
        else:
            return CommandDict.DEFAULT_OPT_CODE

changed not not to not

While that's somewhat... interesting logically, I'm glad that at least fixed it for you.
Only thing this makes me think is that I may want to look at adding test cover for the impacted code due to the inconsistency in user experience just to make sure my double negation approach to bool coercion works properly

if the existing value in config is "false" rather than an actual false value coercing with 'not not' would do the wrong thing.
The right thing to do to account for existing config data would be to ternary around 'true' and 'false', call bool() otherwise (python has 1 edge case where !! is the wrong coercion iirc)

I mean, yes, any string will be converted to True in the case of not not. That said, the commands.yaml.example does not contain this, it contains true or false (not "true" or '"false"`), so I would hope the user would at least have to go out of their way to get things like that.

Thank you all 3 @atomtm @troglodyne and @teodesian for contributing and bringing up this issue and finding solutions.
OK, yes, I confirm, there is a bug. The bug is because I thought Yaml would return a string type when it finds a value "true" (or "false") in the Yaml config file. Yaml is smart, and it actually returns a boolean. So the actual code line
           return self.command_dict["commands"][command]["markdown_convert"] == "true"
is definitely a bug as it compares a boolean in the dict to a string.
@troglodyne fixed the bug by changing it to
           return not not self.command_dict["commands"][command]["markdown_convert"].
He did the not not to force boolean, but that is not needed as Yaml retruns boolean out of the box.
So,
           return self.command_dict["commands"][command]["markdown_convert"]
is actually correct, in my humble opinion.
But, @atomtm put just a single not. That makes no sense to me. I cannot see how that works.
Also, very important to note is that there is a bug in Element (I reported it a LONG time ago, maybe 1 year ago) where on Element Android, the "code" instruction is NOT working. Android will never show anything as a code block! As a result 2 recommendations:

test "code" block related stuff on Firefox (and NOT on Android)
kindly and nicely "complain" to Element so they eventually fix this Element bug

If there are no new input/feedback, I will await for @troglodyne to change return not not self.command_dict["commands"][command]["markdown_convert"] to return self.command_dict["commands"][command]["markdown_convert"] and then merge his PR.
Sounds good to everybody?

see also https://github.com/vector-im/element-android/issues/2328

Related to Element Android,  if you send a code block from Element Desktop this is rendered correctly on Element Android. But not if the code block is sent by the bot.

Related to Element Android, if you send a code block from Element Desktop this is rendered correctly on Element Android. But not if the code block is sent by the bot.

That info is useful. I will investigate that. Thank you @atomtm
I will try to fix this next week.

Related to Element Android, if you send a code block from Element Desktop this is rendered correctly on Element Android. But not if the code block is sent by the bot.

That info is useful. I will investigate that. Thank you @atomtm
I will try to fix this next week.

I found a work-around that makes it work also on Android!
See bug fix commit in https://github.com/8go/matrix-eno-bot/commit/8cc46277e3bc688a3b17c233fc4635ee4ac0b3ad
This is specific to Android.
I also updated https://github.com/vector-im/element-android/issues/2328

Sounds good, will update PR shortly to remove the unnecessary type coercion

Thanks and :clap:  everybody, @atomtm @teodesian @troglodyne
Should be resolved now, closing issue.


Modifying Object.prototype causes Sendgrid to crash/cause unexpected behavior

In my app, I've found that any modification to Object.prototype causes sendgrid to fail or produce unexpected behavior. At first I thought it was simply a namespacing issue, but it turns out any new properties attached to the Object.prototype object will cause issues. I've managed to reproduce the issue with a minimal amount of code:
Consider the following code, which works fine (API key has been hidden, replaced with my-api-key):
var sendgrid = require('sendgrid')('my-api-key');

sendgrid.send(new sendgrid.Email({
    to<EMAIL_ADDRESS>    from<EMAIL_ADDRESS>    subject: 'test mail',
    html: 'test body',
}), function(err, json) {
    if(err) console.log(err);
    console.log(json);
});

Result: { message: 'success' }
Now consider the same code with a slight modification:
var sendgrid = require('sendgrid')('my-api-key');

Object.prototype.myNewCustomFunction = function () {};

sendgrid.send(new sendgrid.Email({
    to<EMAIL_ADDRESS>    from<EMAIL_ADDRESS>    subject: 'test mail',
    html: 'test body',
}), function(err, json) {
    if(err) console.log(err);
    console.log(json);
});

Result:
{ [Error: socket hang up] code: 'ECONNRESET' }
null

I'm using sendgrid 1.9.2 on node 0.12.7.

Hello @analytalica,
Thank you for the detailed report!
This library will be replaced in a few weeks with this one: https://github.com/sendgrid/sendgrid-nodejs/tree/v3beta
If you still experience this issue with the new library, please open a new issue.
With Best Regards,
Elmer


EntityFramework package (not core), has bad frameworkassembly values

Steps to reproduce
originally filed as: https://github.com/NuGet/Home/issues/3279
Repro Steps:

Create a new project C#-> Windows 8->Windows Runtime Component (Windows 8.1).
Right-click the project node and click "Manage NuGet Packages…" menu to start package manager.
Install package “EntityFramework”, and verify it can be installed.

Expected Result:
The package “EntityFramework” can be installed correctly.
Actual Result:
The package “EntityFramework” can’t be installed as below screenshot.

Note:
It happened on Dev14 U3+NuGet Dev 1731(<IP_ADDRESS>1).
It repro on any version of package “EntityFramework”.
It also repro on Dev14 U3 (NuGet <IP_ADDRESS>1).
The issue
NuGet sees
<FrameworkAssembly ... TargetFramework="" />
TargetFramework should not be empty, but should be Net40 or Net45.
When you don't specify a framework, nuget thinks this works in all TFMs. Thus the package seemed to install on this Win8.1 project, while it should fail.
Further technical details
None.

This issue was moved to aspnet/EntityFramework6#48


Test: Move trip planner test assets to test directory

TL;DR
Relocated JSON test assets from debug to test directory

Merge activity

Nov 6, 6:56 AM EST: A user merged this pull request with Graphite.


docs: (ENG-398) add changelog

ENG-398  Need to update the TZP -> Salesforce Sync to use the new salesforce instance

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


Remove Unnecessary Lazy Extension Code

What was wrong?
The If Clause code will never be executed and validate_lte would replace its functionality always. Hence need to delete this lazy extension.
How was it fixed?
The If Clause is to be deleted instead of the validate_lte, because if that was kept then we would be increasing memory without charging gas for the increased memory.
Cute Animal Picture

The current CI failures are not due to this PR: see PR #1354.

@veox can you please rerun the CI after the CI is fixed, because I don't have the permissions to rerun them.

@Bhargavasomu I'll take care of re-running CI
Mind taking a minute to review https://guides.github.com/features/mastering-markdown/  .  It can help a lot to add some markup to your comments and descriptions, specifically when referencing code.

the validate_lte function ...
the validate_lte function ...

The one with the markup takes less cognitive overhead to read.
Thanks for finding and fixing this for us.  Welcome to the codebase.

@Bhargavasomu can you rebase now that https://github.com/ethereum/py-evm/pull/1354#pullrequestreview-161734738 has been merged.

@pipermerriam I opened a new  request and am closing this one. Sorry for the inconvenience


Stack Overflow when using a newer version of Serilog.Sinks.PeriodicBatching

When using Serilog.Sinks.Slack together with the latest version of Serilog.Sinks.PeriodicBatching the call to Serilog.Log.CloseAndFlush() will crash with a StackOverflowException because the SlackSink Dispose method will call Dispose on PeriodicBatchSink which in turn calls Dispose on SlackSink, causing an endless loop.
This issue was discovered while using the Sink in a .NET Framework application where transient dependencies have to be explicitly included and where the dependency was updated to the latest version.
From my research into how PeriodicBatching code works and how other Sinks handle this it seems that maybe SinkSlack should be split up so that it doesn't pass a reference to itself to PeriodicBatchingSink. But I'm not sure how this change would affect people who use this library today.
Here is the sample code I used to reproduce the issue:
using Serilog;
using Serilog.Events;
using Serilog.Sinks.Slack;
using Serilog.Sinks.Slack.Models;

namespace SlackSinkStackOverflow;

internal static class Program
{
    private static void Main()
    {
        Log.Logger = new LoggerConfiguration()
            .WriteTo.Console()
            .WriteTo.Slack(new SlackSinkOptions
                { WebHookUrl = "https://hooks.slack.com/services/...", MinimumLogEventLevel = LogEventLevel.Error })
            .CreateLogger();

        try
        {
            Log.Information("Starting application");
        }
        catch (Exception exception)
        {
            Log.Fatal(exception, "Application terminated unexpectedly");
        }
        finally
        {
            // Causes a StackOverflowException
            Log.CloseAndFlush();
        }
    }
}

Serilog.Sinks.Slack version is 2.2.2.
Serilog.Sinks.PeriodicBatching version is 5.0.0
A suggested solution would look something similar to the following code:
public class SlackMessageFormatter
{
    private readonly SlackSinkOptions _options;
    private readonly ITextFormatter _textFormatter;

    public SlackMessageFormatter(SlackSinkOptions options, ITextFormatter textFormatter)
    {
        _options = options;
        _textFormatter = textFormatter;
    }

    public Message CreateMessage(LogEvent logEvent)
    {
        using (var textWriter = new StringWriter())
        {
            _textFormatter.Format(logEvent, textWriter);
            return new Message
            {
                Text = textWriter.ToString(),
                // Set other properties...
            };
        }
    }

    // Add methods for creating attachments, etc.
}

public class SlackClient : IDisposable
{
    private readonly HttpClient _httpClient;

    public SlackClient()
    {
        _httpClient = new HttpClient();
    }

    public async Task SendMessageAsync(string webhookUrl, Message message)
    {
        var json = JsonConvert.SerializeObject(message);
        await _httpClient.PostAsync(webhookUrl, new StringContent(json));
    }

    public void Dispose()
    {
        _httpClient.Dispose();
    }
}

public class SlackBatchingSink : IBatchedLogEventSink, IDisposable
{
    private readonly SlackMessageFormatter _formatter;
    private readonly SlackClient _client;
    private readonly SlackSinkOptions _options;

    public SlackBatchingSink(SlackMessageFormatter formatter, SlackClient client, SlackSinkOptions options)
    {
        _formatter = formatter;
        _client = client;
        _options = options;
    }

    public async Task EmitBatchAsync(IEnumerable<LogEvent> events)
    {
        foreach (var logEvent in events)
        {
            var message = _formatter.CreateMessage(logEvent);
            await _client.SendMessageAsync(_options.WebHookUrl, message);
        }
    }

    public Task OnEmptyBatchAsync()
    {
        return Task.CompletedTask;
    }

    public void Dispose()
    {
        _client.Dispose();
    }
}

public class SlackSink : ILogEventSink, IDisposable
{
    private readonly PeriodicBatchingSink _periodicBatchingSink;

    public SlackSink(SlackSinkOptions options, ITextFormatter textFormatter)
    {
        var formatter = new SlackMessageFormatter(options, textFormatter);
        var client = new SlackClient();
        var batchingSink = new SlackBatchingSink(formatter, client, options);

        _periodicBatchingSink = new PeriodicBatchingSink(batchingSink, options.ToPeriodicBatchingSinkOptions());
    }

    public void Emit(LogEvent logEvent)
    {
        _periodicBatchingSink.Emit(logEvent);
    }

    public void Dispose()
    {
        _periodicBatchingSink.Dispose();
    }
}

Benefits of the Redesign:
Each class now has a single responsibility, making the code easier to understand, maintain, and test.
The SlackSink no longer directly manages both its disposal and that of its components. The components (like SlackClient and PeriodicBatchingSink) handle their own lifecycles independently, reducing the risk of recursive disposal calls.
The SlackMessageFormatter and SlackClient can be reused or tested independently, providing more flexibility in the codebase.
Simplified Debugging:
If a stack overflow or other issue does occur, it’s easier to isolate the problem to a specific component, as each class now handles a distinct part of the process.
This redesign should help eliminate the stack overflow problem while also improving the overall architecture and maintainability of your logging infrastructure.

Hi there. Thank you so much for your detailed response and analysis. This issue has already been reported as #48, but I appreciate how detailed this is.
I have actually been working on a revamp of the whole project removing the dependency on Newtonsoft.Json entirely, along with the PeriodicBatchingSink - which is now part of the core Serilog.
With that being said, I'm still debating how to proceed with versioning. I'm of the opinion that I will release one more version to at least fix the issue reported here, and then release a new major version which does away with the above-mentioned dependencies while also ending support for .net standard 1.*

Thanks for the update! It’s exciting to hear about the revamp, especially dropping Newtonsoft.Json and integrating the PeriodicBatchingSink directly into Serilog—those changes are bound to streamline things.
I think your plan for versioning makes a lot of sense. Pushing out one last version to address the current issue before rolling out the major overhaul seems like a smart way to keep things stable for users while you work on the bigger improvements. And yeah, ending support for .NET Standard 1.* feels like the right move given where things are headed.
I’m really looking forward to seeing what you come up with next! If there’s anything else I can help with or test, just let me know.


[ANCHOR-385]: Add documentation on how to pass JVM flags

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


Allow deploying nix-darwin machines.

Hey.
I like colmena and would like to use it for my darwin machines as well.
colmena should probably have a way to use darwinSystem instead of nixosSystem. I am unsure how one should select which one to use, but it's certainly a requirement if one wants to use nix-darwin to use macOS-specific packages and configuration.
I'll gladly help, but I'm really not that great in Nix.

This is an interesting idea that I've been thinking about for a while. To help with the general case, we can have the concept of profile types which define how a profile (NixOS, home-manager, nix-darwin, etc.) can be evaluated and activated, like the implementation in deploy-rs. I haven't had the time to actually implement it, though.

like the implementation in deploy-rs.

just came by here and the link refers to the rust-lang/cargo instead of deploy-rs; probably a mistaken paste at least i can't see the relationship :smile:

Also interested in this feature.

I'm gradually moving to macOS for desktop use and am working on this.

I have been happily using Colmena for me server based needs (home setup), it has been amazing so far. I have tried to move my desktop nixos flake setup to use Colmena as well (for unified setup) but am not sure how to port home-manager over.
Based on https://github.com/zhaofengli/colmena/issues/125#issuecomment-1309366212 I suspect its not currently supported (to use home-manager in colmena). @zhaofengli not to pester but do you have plans in the coming months to support this feature?


keybase cli not installed by OS X client

I installed the OS X client, but no keybase tool became available via the terminal after launching the app from the Finder.  I was prompted for my password to install a "helper" tool (totally unnerving, btw).
my log id: 4137e77ea391ed429c00501c

That's odd.  Is /usr/local/bin in your path?  Does /usr/local/bin/keybase exist?
CC @gabriel

ls: /usr/local/bin/keybase: No such file or directory

Found the issue. We don't currently support adding the symlink if installed in /User/username/Applications folder.
You can add a symlink manually to /usr/local/bin (or wherever you have a PATH set).
ln -s /Users/yourusername/Applications/Keybase.app/Contents/SharedSupport/bin/keybase /usr/local/bin/keybase

Also totally agree that the helper tool prompt is a little unnerving. We don't quit yet have onboarding designed and implemented yet which would explain why we need privileged access and what it is doing.

Same problem.  Might want to make it easier to find this solution...


Update broken AWS credentials links from comments

This commit addresses the #5 issue. The existing link in the comments (http://aws.amazon.com/credentials) that is supposed to provide details on configuring the AWS credentials is returning a 404. I have replaced it with the developer guide URL that is specified in the README.md files of the sample projects.

Thanks for the pull request!


[CI, ENH] Feature 116 pre commit

Fixes #116
Changes proposed in this pull request:

Introduce pre-commit hooks for flake8, mypy, isort, bandit, black
Introduces sync_with_poetry to ensure that versions of all of the above are synced with poetry.lock
Updates poetry.lock due to issue with isort
No unit tests implemented but local testing was done

Before submitting

[x] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[x] I've updated or added any relevant docstrings following the syntax described in the
Writing docstrings section of the CONTRIBUTING docs.
[x] If this PR fixes a bug, I've added a test that will fail without my fix.
[ ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

After submitting

[ ] All GitHub Actions jobs for my pull request have passed.

I think it's still missing your approval @adam2392


node disjunction constraints or optional nodes not supported

I have added penalties for dropping stops via
long penalty = 1000;
    for (int i = 1; i < data.distanceMatrix.length; ++i) {
      routing.addDisjunction(new long[] {manager.nodeToIndex(i)}, penalty);
    }

and now I see new warnings

WARNING: Logging before InitGoogleLogging() is written to STDERR
W0424 17:03:04.942909 19191 routing.cc:2941] Node disjunction constraints or optional nodes not supported.

Should I worry about these?

seems to comes from:
https://github.com/google/or-tools/blob/39f44709bba203f5ff3bc18fab8098739f189a6d/ortools/constraint_solver/routing.cc#L2940-L2944
Never saw this, which FirstStrategy and local search did you use ?

will get back to you asap

No worry, this is just a spurious message.


Update getting-started.md

This seems to need this update. The former syntax does not work with Sovrin 1.0.28

Hi, this is a project archive. Active development of Indy is taking place over here: https://github.com/hyperledger?utf8=✓&q=indy&type=&language= thank you


Detonations of EBS related exfil techniques fails when EBS encryption by default is enabled for a region

What is not working?
Detonation fails for aws.exfiltration.ec2-share-ami and aws.exfiltration.ec2-share-ebs-snapshot when EBS encryption by default is enabled for a region
What OS are you using?
N/A
What is your Stratus Red Team version?
stratus version
Full output?
$ stratus detonate aws.exfiltration.ec2-share-ami
2022/03/29 20:14:20 Checking your authentication against the AWS API
2022/03/29 20:14:21 Not warming up - aws.exfiltration.ec2-share-ami is already warm. Use --force to force
2022/03/29 20:14:21 Exfiltrating AMI ami-083ab591a70549402 by sharing it with an external AWS account
2022/03/29 20:14:21 Error while detonating attack technique aws.exfiltration.ec2-share-ami: Unable to share AMI with external AWS account: operation error EC2: ModifyImageAttribute, https response error StatusCode: 400, RequestID: 97354885-357c-497f-b784-3365a5294736, api error InvalidParameter: Snapshots encrypted with the AWS Managed CMK can't be shared. Specify another snapshot.

$ stratus detonate aws.exfiltration.ec2-share-ebs-snapshot
2022/03/29 20:20:23 Checking your authentication against the AWS API
2022/03/29 20:20:23 Not warming up - aws.exfiltration.ec2-share-ebs-snapshot is already warm. Use --force to force
2022/03/29 20:20:23 Sharing the volume snapshot snap-00d459dab53c44042 with an external AWS account...
2022/03/29 20:20:23 Error while detonating attack technique aws.exfiltration.ec2-share-ebs-snapshot: operation error EC2: ModifySnapshotAttribute, https response error StatusCode: 400, RequestID: 12f44aeb-7b3b-4488-ac46-a432d20cc7a9, api error OperationNotPermitted: Encrypted snapshots with EBS default key cannot be shared

Files in $HOME/.stratus-red-team?
N/A

Thanks for reporting! Does that happen only if the EBS default encryption uses a customer-managed CMK, or also if it uses the AWS default KMS key?

So if we share the EBS snapshot/AMI publicly but it's encrypted with a CMK, it's "public" but an attacker can't access it unless he has explicit access to the CMK for decryption. Correct?
If that's the case, is it really something an attacker would do? (Create a KMS key, grant themselves access to the KMS key, encrypt the snapshot with it, then share it)
I would argue an attacker is likely to first try to share the EBS/AMI publicly. In which case, you could argue that the current implementation makes sense, even if it will fail. Thoughts?

You made a good point, the ModifySnapshotAttribute and ModifyImageAttribute events will still be logged even if default encryption prevents them from being successful (just confirmed it). Given there are no further API calls which rely on a successful call, it doesn't matter if they fail.
So a cleaner approach than modifying terraform would just be to catch the error during detonation. Then include a message that API call failed but detonation was successful.

Sounds great! Thanks for the suggestion.

Looking better in #113 @mchaffe ?

Thanks for the feedback! Will be released as part of v1.6.2, out shortly.


Export to Tikz

The ability to export graphics to LaTex Tikz format would be invaluable.
https://www.overleaf.com/learn/latex/LaTeX_Graphics_using_TikZ:_A_Tutorial_for_Beginners_(Part_1)—Basic_Drawing
Inkscape has a similar feature for converting SVGs to Tiks, perhaps some of the algorithm could be reused.
https://tex.stackexchange.com/questions/370876/converting-an-svg-image-to-tikz-code

A relatively painless method is to export as SVG, open with a program that supports TEX, and export. I found "Dia" to work OK, Inkscape and Xfig may work as well. Have a look here:
https://tex.stackexchange.com/questions/26972/what-gui-applications-are-there-to-assist-in-generating-graphics-for-tex/27062

A relatively painless method is to export as SVG, open with a program that supports TEX, and export. I found "Dia" to work OK, Inkscape and Xfig may work as well. Have a look here:
https://tex.stackexchange.com/questions/26972/what-gui-applications-are-there-to-assist-in-generating-graphics-for-tex/27062

This is outside the scope of the project.

This is outside the scope of the project.


Pipeline is not executed in order

Bug Report
When I was using the lettuce pipeline, I found that it didn’t seem to be sequential.
I don’t know if it’s where I used it wrong
Input Code
In the following code, in a pipeline, I delete the key first, then reset the key, and finally check whether all keys exist
public static void main(String[] args) throws InterruptedException, ExecutionException, TimeoutException {
        while (true){
            DefaultClientResources.Builder resourceBuilder = DefaultClientResources.builder();
            ClientResources clientResources = resourceBuilder.build();
            RedisURI redisUri = RedisURI.create("xxxx", 8155);
            RedisClusterClient redisClusterClient = RedisClusterClient.create(clientResources, redisUri);
            StatefulRedisClusterConnection<String, String> connection = redisClusterClient.connect(new StringCodec());
            RedisAdvancedClusterAsyncCommands<String, String> commands = connection.async();

            String keyPrefix = "key-test-";

            int keySize = 1000;

            // disable auto-flushing
            commands.setAutoFlushCommands(false);

            // perform a series of independent calls
            List<RedisFuture<?>> futures = new ArrayList<>();
            for (int i = 0; i < keySize; i++) {
                String key = keyPrefix + i;
                //delete first
                futures.add(commands.del(key));
                //add key
                futures.add(commands.set(key, "value-" + i));
            }

            // write all commands to the transport layer
            commands.flushCommands();

            // synchronization example: Wait until all futures complete
            boolean result = LettuceFutures.awaitAll(5, TimeUnit.SECONDS, futures.toArray(new RedisFuture[0]));

            commands.setAutoFlushCommands(true);

            // Check if all the key exists
            for (int i = 0; i < keySize; i++) {
                String key = keyPrefix + i;
                Long exists = connection.sync().exists(key);
                //not exists
                if (exists < 1) {
                    throw new RuntimeException(key + " not exists");
                }
            }
            connection.close();
        }
    }

Environment

Lettuce version(s): [6.1.5.RELEASE]
Redis version: [e.g. 6.0.10]

redis config:

lazyfree-lazy-user-del no

I tried to use redisson pipeline, there seems to be no problem

yes,This brother and I encountered the same situation

When using Redis Cluster, you need to make sure that all node-connections are established. Otherwise, only a subset of commands will be flushed, because connections are in the state of being initialized.


feat(aslg_pc12): add dev and test data splits

For reproducibility sake, it's best if there are defined dev and test splits.
The original paper author did not define splits for the entire dataset, not for the sample loaded via this library, so I decided to define:

5/7th for train
1/7th for dev
1/7th for test

Hi ! I'm not sure we should make this split decision arbitrarily on our side. Users can split it afterwards to whatever they want using dataset.train_test_split for example.
Moreover it looks like there's already papers that use this dataset and propose their own splits (here 80-20)
What do you think ?

I was not aware of the train_test_split method, thanks!
Soe ven though it contributes to reproducibility, no need to do this split then.


[CONTROLLER/DB] fixes db migration bug

This PR is for:

Server

fixes db migration bug
Steps to reproduce the bug
Changes to fix the bug
Affected branches

main

Checklist

[ ] Added unit test to verify the fix.
[ ] Verified eBPF program runs successfully on linux 4.14.x.
[ ] Verified eBPF program runs successfully on linux 4.19.x.
[ ] Verified eBPF program runs successfully on linux 5.2.x.

 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.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove the swap version pin to allow the newer version.

The version pinned had a bug that broke swap-tuning.

Okay, just changed the version.

Can you merge the pull request so I can use the supermarket version in my cookbooks?


Look into type: 'classic'  for scripts

Ref: https://twitter.com/jdalton/status/978285179440570368
(Thanks @jdalton for bringing this up).

Closing as there was a misunderstanding, this is related to workers.


renovatebot(deps): update dependency de.codecentric:spring-boot-admin-starter-client to v2.1.5

This PR contains the following updates:

Package
Update
Change

de.codecentric:spring-boot-admin-starter-client
patch
2.1.4 -> 2.1.5

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

 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.null seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update crisp extension, make Supabase session fetching faster

Description
Before this change calling setSession was taking 500ms on average, now it only takes 15ms
Screencast
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

Is there another way to try your extension?

Hi, unfortunately right now there is no way to try the extension without buying the Crisp plugin but the previous PR was already reviewed and the extension was tested working, this is just a small change to make the command load faster.
If you really want to try the extension yourself maybe i can find a way but it would be different than the typical user workflow so I don't think it serves any utility.

Also, could you please clean up the debug console.logs?

I want to keep the profiling logs so you can clearly see what parts of the extension takes the most time, the logs are shown only in development so it does not impact users.

Here is a screen recording of the extension working
https://github.com/raycast/extensions/assets/31321188/a51f1bb4-0c4e-40e1-87d3-9b9f1e38faba

I removed the console logs and fixed a type error that appeared in the Raycast bug tracker


Routing does not work with location="history"

<Routes>
  <Route handler={Home}>
    <Route name="about" handler={About}/>
  </Route>
</Routes>

Going to localhost:8000/#/ & localhost:8000/#/about do the right thing
Adding location = "history" to routes breaks down localhost:8000/about throwing 404

For others who get stuck while adding server side pushState with nginx, the directive in the readystate4 link above does not work for me. For me, the working nginx.conf looks like:
# set root and index
root /var/www/approot/;
index  index.html;

# route all requests that don't serve a file through index.html
location / {
   if (!-e $request_filename){
      rewrite ^(.*)$ /index.html break;
   }
}

See this stackoverflow answer.

@pxwise confirm works.

If looking for something a little more flexible, this also works:
server {
      root /usr/share/nginx/html;

      location ~ (?:.+\/)((?:.+)\.(?:.+)) {
        # $1 is the filename
        try_files $uri /$1 /css/$1 =404;
      }

      location / {
        try_files $uri /index.html;
      }
    }

Since there may be ajax request api, the following suits for this case,
server {
    listen 80;
    server_name example.com;
    root /var/www/example.com;

    # Any route containing a file extension (e.g. /devicesfile.js)
    location ~ ^.+\..+$ {
        try_files $uri =404;
    }

    # Any route that doesn't have a file extension (e.g. /devices)
    location / {
        try_files $uri /index.html;
    }

    # The location block above provides the shortest prefix, of length one,
    # and so only if all other location blocks fail to provide a match,
    # this block will be used.

    # Ajax api starts with /v1/ will be proxied
    location /v1/ {
        proxy_pass http://proxy;
    }
}

@ryanflorence your project http://github.com/rackt/react-boilerplate could not be accessed now.

@pxwise  Thanks. Worked (y)


step2 - render and move snake head

The snake now renders, moves and is controllable by the keyboard.

[x] move the head
[x] handle keyboard input
[x] render the head

This is sweet! I had to look up Fixed Timestep.
Should there be an impl Direction { fn adjacent(self, loc) -> loc { ... } }?
A separate system could detect and act if a snake head moves out of bounds.

Should there be an impl Direction { fn adjacent(self, loc) -> loc { ... } }?

I think there should be an impl Location { pub fn move_by_one(&mut self, dir: Direction)  -> Result { ... } }, actually, that'll return an Err if its out of bound.

Very nice - I have learned a lot about bevy from this.


[Bug] Selector with multiple attributes not working

The following query returns an empty result:
(new Dom)->load('<input type="text" name="foo">')->find('input[type=text][name=foo]');

I'm having the same problem.

Issue fixed as part of the 2.1.1 release


How to substitute a keras layer?

Hello Keras Team,
I'm attempting to substitute some layers in my custom model with:
model.layers[i] = layers[i]

But I'm getting the following error:
AttributeError: Can't set the attribute "layers", likely because it conflicts with an existing read-only @property of the object. Please choose a different name.

Note that my model is a custom tf model (not a tf.keras.model), so unfortunately I cannot use clone_model(). Is there an alternative to clone it?
Thank you in advance!

@gcunhase,
Do you want to replace one of the Layers of the Custom Saved Model with another Layer? Also, can you elaborate on what you mean by Custom TF Model vs tf.keras.Model? I didn't understand the difference.

Hi @rmothukuru,
Thank you for your reply.
My goal is to modify some of the layers in my custom model and replace them in the original model.
Ideally, I would do that in a copy of the model, so I tried to clone_model() TF's EfficientDet model obtained from their official model zoo here. However, that gave me an error saying that the model was not of tf.keras.Model type.
So I tried directly assigning the modified layer to its relevant index model.layers[i] = layers[i], which did not work as mentioned above.
Please let me know if this explanation clarifies my issue.

@gcunhase Is it possible for you to create simple standalone code to reproduce the issue. Couple of questions. Are you saving and loading the model and then trying to replace one layer? Do you have access to the code of the original model? Thanks!

Closing due to lack of recent activity. Please feel free to reopen when you have more information. Thanks!


memory leak else/sum

Seems else/sum never releases (some of?) the memory it uses, confirmed on Windows en Linux on RC8, see attached test patch for example where memory use keeps increasing.
sumtest2.zip

is it related to https://github.com/porres/pd-else/issues/1704 ?

another one that I'd hope to get help from @timothyschoen

@nuoun try the fix https://github.com/porres/pd-else/pull/2023


admin_account_prefix is missing in config/adldap.php?

Variable admin_account_prefix is missing in config/adldap.php? When login a user, I got this error below:
BindException in Guard.php line 84:
Invalid credentials
in Guard.php line 84
at Guard->bind('username', 'password', '', '') in Guard.php line 100
at Guard->bindAsAdministrator() in Provider.php line 200

Tried to put config in the file 'admin_account_prefix' => 'domain',, but still no luck.
However, this error gone after I feed the ADLDAP_ADMIN_USERNAME=domain\username in .env file config.
Thanks

Hi @kayrules,
There is no option named 'admin_account_prefix', only 'admin_account_suffix'.
I can see how this can be confusing since there's an option for one but not the other.
You can just utilize the option 'account_suffix' and it will be used binding the admin account as well.

Thank you @stevebauman,
It turns out that I just need to add a backslash on account_prefix for the domain name as below:
'account_prefix' => 'domain\\',

now it will able to bind both for users & admin.
Thanks


[CELEBORN-1661] Make sure that the sortedFilesDb is initialized successfully when worker enable graceful shutdown

What changes were proposed in this pull request?
Why are the changes needed?
Similar to CELEBORN-1457, sortedFilesDb may also fail to initialize.
Does this PR introduce any user-facing change?
No
How was this patch tested?
GA

Merge to main(v0.6.0) and branch-0.5(v0.5.2)


[SpaceEngine] NFC: Switch to != instead of compare on Head iden…

…tifiers
Since Head comparison only checks for equality, all of such
checks could be switched from compare to != operator.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - c13a10ef3266dee58d71e4cb84a776868d2f24b1

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - c13a10ef3266dee58d71e4cb84a776868d2f24b1

@swift-ci please smoke test Linux platform

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - c13a10ef3266dee58d71e4cb84a776868d2f24b1

some unrelated lldb failures, lets try again...

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - c13a10ef3266dee58d71e4cb84a776868d2f24b1

@swift-ci please smoke test Linux platform

I am experiencing such lldb failure as well in #17545 ...


support for escaping in Bytes and bytes macro

Quick note here, I went to implement \ x HEXDIGIT HEXDIGIT and \ [0-3] [0-7] [0-7] for escaping byte sequences in b"" BYTES_LIT, but from the spec's lexis, all the escaping from STRING_LIT should be supported...
BYTES_LIT      ::= [bB] STRING_LIT
ESCAPE         ::= \ [abfnrtv\?"'`]
                 | \ x HEXDIGIT HEXDIGIT
                 | \ u HEXDIGIT HEXDIGIT HEXDIGIT HEXDIGIT
                 | \ U HEXDIGIT HEXDIGIT HEXDIGIT HEXDIGIT HEXDIGIT HEXDIGIT HEXDIGIT HEXDIGIT
                 | \ [0-3] [0-7] [0-7]

So another way to go about this is to refactor parse_string to parse_bytes_literal(s: &str) -> Result<Vec<u8>, ParseError> instead, and have the "newer" parse_string call into it to then String::from_utf8 the parsed bytes... I think that'd be closer to the spec, tho a bigger change. This change mostly allows us to now represent arbitrary byte sequence and work with them, which wasn't possible before.

So another way to go about this

I'm fine with this approach for now.


provideSignatureHelp block extension ejecution

I'm writing and extension where I use registerCompletionItemProvider
context.subscriptions.push(languages.registerCompletionItemProvider('php', new Completion(),'>'));

and I'm using onDidSaveTextDocument
context.subscriptions.push(vscode.workspace.onDidSaveTextDocument((textDocument: vscode.TextDocument) => {
//...Some lines of code
}));

Here everything works fine. When I  seved a file, it throws me some text  and it shown using console.log(). At the same time, the Completiton works fines, showing me the some text of test.
What I normaly do in a PHP file is a class with some methods.
class Test{
 public funciton sayHello(){
   //...some code.
 }
}
$class->//Here shows me some text, thanks to registerCompletionItemProvider.

But when I do $class->sayHello(); the Console debugger shows me provideSignatureHelp (ofcourse) but after this I can keep using my code. when I write again $class->, nothing happends. So why is that?, If I'm using ">" to shows some info for completition and works fine till write a method and a provideSignatureHelp actions is triggered.

$class->sayHello();

@roslanr18 is that a kinda of help?

@DracotMolver Can you outline what your provideCompletionItem function looks like?
I have understood the following steps:

type >, completion is requested
type > another time, no completion request happens

@jrieken ok. This is what I have. I'm doing everything just in a single file for learning easily.
File extension.ts:
import * as vscode from 'vscode';
var className, methods, importClassDefinition;
export function activate(context: vscode.ExtensionContext): void {
	console.log('Congratulations, your extension "phpclassmethods_is" is now active!');

	context.subscriptions.push(vscode.languages.registerCompletionItemProvider('php', new Completion(),'>'));

	context.subscriptions.push(vscode.workspace.onDidSaveTextDocument((textDocument: vscode.TextDocument) => {
			var text = textDocument.getText().toString().trim();
			className = text.match(/(class)\s([\w\d]+)/igm)[0]
							.replace(/(class)\s/,'');

			var importClassDefinition = ImportClassDefinition.getInstance();
			if(!importClassDefinition.classExists(className)){
				methods = text.match(/[\w\d]+(\().*(\))/igm);
				importClassDefinition.setClassParams(className,methods);
			}
	}));
}

export class Completion implements vscode.CompletionItemProvider{
	public provideCompletionItems(document: vscode.TextDocument, position: vscode.Position, token: vscode.CancellationToken)
	: Promise<vscode.CompletionItem[]>{
		return new Promise((resolve, reject)=>{
			try{
			 	var completion:Array<vscode.CompletionItem> = [
					new vscode.CompletionItem('test')
				];

				resolve(completion);
			}catch(e){
				reject(e);
			}
		});
	}
}

export class ImportClassDefinition{
	private classes : Array<string>;
	private className : string;
	private classMethods : Array<string>;
	private static instance: ImportClassDefinition = null;

	constructor(){}

	static getInstance():ImportClassDefinition{
		if(ImportClassDefinition.instance){
			ImportClassDefinition.instance = new ImportClassDefinition();
		}
		return ImportClassDefinition.instance;
	}

	public setClassParams(className: string, methods:Array<string>){
		this.className = className;
		this.classMethods = methods;
		this.classes[this.className] = this.classMethods;
	}

	public getClassParams(className: string) : Array<string>{
		return this.classes[className];
	}

	public getClassName() : string{
		return this.className;
	}

	public classExists(className: string){
		return undefined !== Object.keys(this.classes)[className];
	}
}

@DracotMolver You mentioned provideSignatureHelp but I don't see that in your sample

@jrieken  Sorry, Mayby there was a bad explanation by my side. I'm not using provideSignatureHelp. The provideSignatureHelp appears in the console debugger when I type "();", after that I can keep using the code.

@DracotMolver I am sorry, but i still haven't understood the issue here? Can provide detailed steps or let me know if this became obsolete?

I'm not sure if it's related but I wrote a plugin and everything is fine except that when the user is idle from vscode for some time (10-30minutes maybe) there is a message in the console "provideSignatureHelp" and then the plugin do not respond.
In "do not respond" I mean that you can't click on the status bar (well, you can but nothing happens) and all setInterval launched by the plugin have stopped (I added some console.log to debug and they are not showing)

I'm wondering if it's because I don't dispose correctly of my all commands... Still looking for the problem.

So, the provideSignatureHelp message came form an issue with the php language service. That also caused some unresponsiveness.


X-UMX - Separating using --context cpu takes very long

Thank you so much for open-sourcing X-UMX!
Is it in a usable state right now?
I followed your instructions to perform source separation with the model as listed here.
I ran test.py with the flag --context cpu and separation is taking a very long time, it took over 15 minutes for separating a 3-minute track. I have 16 GB of RAM. Is CPU-based separation supposed to take this long?
Update: It ended up exhausting all my memory and crashed.

As per this tweet CPU inference is not working right now.

Seems like inference in CPU is not working properly now. We'll fix it right away. Sorry for the inconvenience.

As per this tweet CPU inference is not working right now.

Seems like inference in CPU is not working properly now. We'll fix it right away. Sorry for the inconvenience.

Pulled the latest changes and it's working now, albeit a bit slowly.

Pulled the latest changes and it's working now, albeit a bit slowly.

Thanks for raising the issue. Now it's resolved in #6. Still it may be slow in CPU. Do you have a particular target CPU device you are restricted to use?

Thanks for raising the issue. Now it's resolved in #6. Still it may be slow in CPU. Do you have a particular target CPU device you are restricted to use?

Thanks for raising the issue. Now it's resolved in #6. Still it may be slow in CPU. Do you have a particular target CPU device you are restricted to use?

Could you clarify what you mean by that question? My CPU is an I7-9750H if that's what you are asking.

Thanks for raising the issue. Now it's resolved in #6. Still it may be slow in CPU. Do you have a particular target CPU device you are restricted to use?

Could you clarify what you mean by that question? My CPU is an I7-9750H if that's what you are asking.


typing the next available receiving address

It would be cool, if there was a mode, where the hw1 would type out the next available receiving address. Say I go to somebody for buying some BitCoin. I could then just plug my hw1 into his computer, and it would type the address where he can send the coins.
But that's where my idea ends. I don't really know how that would best be triggered. For instance, if the hw1 is connected to a computer, and no software connects to it for say 30 seconds. Could it then switch to HID mode, and start typing. I'm not really sure if that's a good idea, as it could also disturb normal operation.
Of course, the device can't figure out this information on it's own. So, the client program would have to write the info to the device when it sycs with the blockchain.

In the meantime, I use the "myTrezor lite" android app with the xpub of the HW1 wallet.
That can always give me the next address.


feat(test): add tests for Form components

Add tests for form components;
Refactor <Form.SquareCheckBox>:

Remove <Form.IndexSquareCheckBox>;
Add sup, supHeight instead of hardcoded createAt;
Add icon instead of hardcoded index icon;
Remove useField and move it to outer component;

Remove unused <Form.CurrencyRadioInput>;

need to add test case for https://github.com/thematters/matters-web/pull/3999


Plugins are not self-contained

To populate plugin configuration fields, osbs-client needs to know

names of plugins
what types of plugin they are (pre/post/etc)
names of arguments they take
what value to fill in

For example, url and use_auth for plugins wanting to use OSBS.
It would be great if plugins could be more self-contained, with osbs-client not needing to know this sort of thing.

So you would like to move some configuration and logic from osbs-client to reactor? I'm +1 on that: I feel like that updating build image is easier than updating osbs-client. And in the end, we would have all the logic and everything in reactor, while osbs-client would be pure client with pretty much no configuration.

I think so, yes. The osbs-client could provide information about which plugins to run (or even naming a build profile) and variable substitution values, and atomic-reactor could take care of which arguments are supplied, which order to run plugins in, etc.

Plugins now always run, and decide for themselves whether action is needed.


Compatibility with pydata_sphinx_theme

I would like to use sphinx-tabs in a project with pydata_sphinx_theme (https://github.com/pandas-dev/pydata-sphinx-theme). Unfortunately, they don't seem to be compatible.
Using the example code together with the option html_theme = 'pydata_sphinx_theme' in the conf.py results in the following behavior.

Do you have any idea how to achieve compatibility? Any help would be very much appreciated.

Versions used: python 3.8.2, pydata_sphinx_theme 0.3.2, sphinx-tabs 1.3.0, sphinx 3.2.1

Heya, is this the same in earlier versions, e.g. 1.2?
Maybe something recently broke it because last time I checked it was working with sphinx-book-theme.
Anyway, we should employ here my nice multi-theme set up in https://sphinx-panels.readthedocs.io/en/pydata-theme/

Also there is now a pydata-sphinx-theme v0.4

Thanks for the quick reply. The multi-theme setup looks very nice.
I locally checked the following combinations:

With sphinx_tabs 1.3.0 the themes alabaster 0.7.12, sphinx-rtd-theme 0.5.0 & sphinx-book-theme 0.0.36 are fine, but sphinx-pydata-theme 0.4.0 not.
With sphinx_tabs 1.2.0 I get the same results (theme versions as mentioned above).

It may be because sphinx tabs uses nodes.container for all of these elements. These are converted in the HTML to

The Bootstrap CSS framework that pydata-sphinx-theme uses has a special meaning for the container class though, which may be clashing with the CSS from this package.
sphinx-book-theme also uses Bootstrap though, so I'm surprised that works.
Anyway, in sphinx-panels I use a "special" div sub-class of nodes.container, which ensures that the "container" class name is not added to the output div.
It wouldn't hurt to add that here.


Implemented default value for queryParams in Request

A real killer feature :-)

Takk! I already had the same feature in a branch locally, so I pushed that. (8cbf214909b509f9bba9738a0e3ae1274f8c9364)
My method name is queryParamOrDefault, which I think fits better since getOrDefault is an established concept in Java, and methodName(s1, s2) looks a lot like a setter in Spark.


Duplicated UpdateStrategy Key

https://github.com/lacework/helm-charts/blob/main/lacework-agent/templates/daemonset.yaml#L14-L15
https://github.com/lacework/helm-charts/blob/main/lacework-agent/templates/daemonset.yaml#L131-L132
Surprisingly, neither Helm nor Kubernetes seem to care that this key is duplicated. However, flux does refuse to deploy this Helm chart because of this duplication. Also, undefined behavior is never good.
Looks like L14-L15 are a mistaken, and should be removed.

Here's the upstream issue that is involved: https://github.com/kubernetes/kubernetes/issues/14791

Hi @apeschel, thanks for reporting. I'll file an internal ticket to track this and fix this shortly.

Fixed by #3.


[feature] #3453: Implement Store CRUD Operations in Client CLI

Description
I added extra subcommands to asset which are set-store, get-store, remove-store, update-store
Use Casees
./iroha_client_cli asset set-store --key="K4" --asset-id="tea##mad_hatter@looking_glass" --json --value="{ \"Vec\": [{\"String\": \"a\"}, {\"String\": \"b\"}] }"
---
./iroha_client_cli asset set-store --key="K4" --asset-id="tea##mad_hatter@looking_glass" --value=true
---
./iroha_client_cli asset update-store --key="K4" --asset-id="tea##mad_hatter@looking_glass" --value=16_u32
---
./iroha_client_cli asset get-store --key="K2" --asset-id="tea##mad_hatter@looking_glass"
---
./iroha_client_cli asset remove-store --key="K4" --asset-id="tea##mad_hatter@looking_glass"

Linked issue

Closes #3453

Checklist

[ X] I've read CONTRIBUTING.md
[ X] I've used the standard signed-off commit format (or will squash just before merging)
[ ] All applicable CI checks pass (or I promised to make them pass later)
[ ] (optional) I've written unit tests for the code changes
[ ] I replied to all comments after code review, marking all implemented changes with thumbs up

What is the difference between set-store and update-store?
./iroha_client_cli asset set-store --key="K4" --asset-id="tea##mad_hatter@looking_glass" --value=true
---
./iroha_client_cli asset update-store --key="K4" --asset-id="tea##mad_hatter@looking_glass" --value=16_u32

We don't use merge commits in our workflow, use rebase workflow instead (checkout section in CONTRIBUTING.md).
You can squash your commits locally after review before merge.

While reviewing this, found an issue with asset subcommand in general:

https://github.com/hyperledger/iroha/issues/4167

@Asem-Abdelhady some python tests should be added to cover new functionality


Create myDIG routes to add custom extractors to a project

There should be a list of all custom extractors. We need the usual CRUD routes:

POST to add a new custom extractor. The input should the a PyPy URL or something like that, which can be used as the identifier of the custom extractor. We also need the version.
DELETE to delete an existing custom extractor
PUT updates the version
GET to get list and info about individual extractor

@szeke aren't these obsolete with ETK2 ?

This issue is irrelevant.


downloading ambs

Hey, I have downloaded dreamerv3 previously in an environment. Can I just download jax and the other dependencies in the same environment or should I create some other environment? Hope you can help me with my confusion. Thanks!

Hi, I think if you have downloaded dreamerv3 and got that running in
another virtual environment, then you should be good to go - just use this
environment. For the dopamine agents (IQN and rainbow) I would recommend
setting up a separate virtual environment and install the dopamine
dependencies. If you have any more questions then let me know.
On Tue, Sep 19, 2023 at 2:52 PM adla700 @.***> wrote:

Hey, I have downloaded dreamerv3 previously in an environment. Can I just
download jax and the other dependencies in the same environment or should I
create some other environment? Hope you can help me with my confusion.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/sacktock/AMBS/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTHW6GK3DR6BSNV46JAEKLX3GPTRANCNFSM6AAAAAA46LDBKE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This was the setup instructions I followed for Ubunto 22.04 LTS:
Install Conda for Ubuntu
https://linuxhint.com/install-anaconda-ubuntu-22-04/
conda create --name jax --clone base
Install JAX/CUDA on Ubuntu WSL 2
https://docs.nvidia.com/cuda/wsl-user-guide/index.html
https://catid.io/posts/jax/
https://docs.nvidia.com/deeplearning/cudnn/install-guide/index.html#troubleshoot
https://ubuntu.com/tutorials/enabling-gpu-acceleration-on-ubuntu-on-wsl2-with-the-nvidia-cuda-platform#5-enjoy-ubuntu-on-wsl
sudo apt-get install libfreeimage3 libfreeimage-dev
pip install gym[atari]==0.19.0
pip install gym[accept-rom-license]==0.19.0
pip install dm_control
pip install cloudpickle
pip install crafter
pip install numpy
pip install optax
pip install rich
pip install ruamel.yaml
pip install tensorflow-cpu
pip install tensorflow_probability
pip install chainer
pip install chainerrl
pip install Pillow
pip install cmake
pip install opencv-python

I noticed that some of the requirements have older versions that are not available, any alternative to be able to run the code?

The requirements in requirements.txt are not strict, assuming the librariers that are out of date are backwards compatible you shiould be fine. However gym[atari]==0.19.0 is required for the code to work, you could use a newer version of gym but you would have to modify the source code where necessary.


[BugFix] Forget to delete RpcClosure

Memory leak detected in ASAN mode
Direct leak of 5632 byte(s) in 8 object(s) allocated from:
    #0 0x9abd577 in operator new(unsigned long) ../../.././libsanitizer/asan/asan_new_delete.cpp:99
    #1 0xa11db28 in starrocks::RuntimeFilterWorker::_deliver_broadcast_runtime_filter_relay(starrocks::PTransmitRuntimeFilterParams&&, std::vector<starrocks::TRuntimeFilterDestination, std::allocator<starrocks::TRuntimeFilterDestination> >&&, int) /root/starrocks/be/src/runtime/runtime_filter_worker.cpp:714
    #2 0xa11d1a4 in starrocks::RuntimeFilterWorker::_process_send_broadcast_runtime_filter_event(starrocks::PTransmitRuntimeFilterParams&&, std::vector<starrocks::TRuntimeFilterDestination, std::allocator<starrocks::TRuntimeFilterDestination> >&&, int) /root/starrocks/be/src/runtime/runtime_filter_worker.cpp:686
    #3 0xa121164 in starrocks::RuntimeFilterWorker::execute() /root/starrocks/be/src/runtime/runtime_filter_worker.cpp:842
    #4 0xa117d25 in operator() /root/starrocks/be/src/runtime/runtime_filter_worker.cpp:456
    #5 0xa1244cf in __invoke_impl<void, starrocks::RuntimeFilterWorker::RuntimeFilterWorker(starrocks::ExecEnv*)::<lambda()> > /opt/gcc/usr/include/c++/10.3.0/bits/invoke.h:60
    #6 0xa124484 in __invoke<starrocks::RuntimeFilterWorker::RuntimeFilterWorker(starrocks::ExecEnv*)::<lambda()> > /opt/gcc/usr/include/c++/10.3.0/bits/invoke.h:95
    #7 0xa124431 in _M_invoke<0> /opt/gcc/usr/include/c++/10.3.0/thread:264
    #8 0xa124405 in operator() /opt/gcc/usr/include/c++/10.3.0/thread:271
    #9 0xa1243e9 in _M_run /opt/gcc/usr/include/c++/10.3.0/thread:215
    #10 0x16b300df in execute_native_thread_routine ../../../.././libstdc++-v3/src/c++11/[thread.cc:80](http://thread.cc/)

What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[x] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


TS definitions

hey Jason. Would you be open to add TS definitions  for this project? If yes I'll sent PR.
thanks man!

Sure! I didn't realize TS supported decorators! haha.  Just a .tsd you mean? I'm down.

Can we get new release pls ? 🍻

whoops! meant to Friday, just saw the todo list item

released as 1.2.0 :)


Can token caching be simplified to singleton dependency injection?

The part of this sample which showcases a client-credential flow whereby the Sync Controller fetches a token, leverages a custom cache implementation.  (A "cache of the cache" as a matter of speaking.)  This is done by wrapping the MSAL token cache in another custom cache implementation called "MSALAppTokenMemoryCache".
This seems overly complex for the scenario at hand.  (There is no need for a redirect url in a client cred flow, so we have all information at hand to construct the IConfidentialClientApplication on beforehand.)
Can we instead simplify this and construct the IConfidentialClientApplication just once at startup and from there, inject it into the controller as a singleton dependency?  This would allow to leverage the MSAL built-in token cache directly without the need for a custom implementation.
This would require the call to "AcquireTokenForClient" to be thread safe though; which is not something which is explicitly documented.
Thanks for your help in shaping clarity around this. /cc @jmprieur

/cc @jelledruyts

Hi @xstof
I would like to correct one thing in your statement. We don't have a "cache of the cache", and this is not the purpose of "MSALAppTokenMemoryCache". What we have is a class that knows where to storage MSAL cache, I would call it a cache provider. "MSALAppTokenMemoryCache" stores and retrieve it on memory. Other option could be SQL for example.
MSAL cache itself cannot take the decision where it will be stored, otherwise we would be restricting the developers to one single technology.
About the singleton idea for IConfidentialClientApplication, it could be possible but needs to be tested in an extremelly concurrent scenario.

Fully aligned.  The need for testing in a concurrent scenario; is that because of IConfidentialClientApplication not being designed as thread-safe? Or is it?
Because if thread-safe, I’d expect this to be a valid and simplified approach.  (We are dealing with a client cred flow, so even in high concurrency/load, the token will likely come from cache for majority of requests and the refresh would only happen occasionally upon exporation).

There is a thread about the singleton going on in the engineering team, so I will let you know when I get more info about it

The recommendation is now to use ASP.NET Core with Microsoft.Identity.Web
See https://github.com/Azure-Samples/active-directory-aspnetcore-webapp-openidconnect-v2


[network_info_plus] Get IP on Wifi hotspot network

Is there any way, using network_info_plus, to get the IP address on the network generated by a wifi hotspot?
I'm trying to use my phone as a hotspot to connect to a raspberry Pi and when I create a wifi hotspot and call NetworkInfo().getWifiIP() or NetworkInfo().getWifiBroadcast() it returns null.
It will only return something when I connect to a classic wifi network.

I have a similar case, is it possible?


cache linux-modules-extra to reduce failed downloads

This GitHub action install Linux APT packages and cache them for later use. When executed next time with same package list, and any other environment are not changed, installed files are extracted from the cached archive.
When valid cached archive is not found, all packages are installed by apt-get command. If you want partial install/cache behavior, use this action multiple times.
ref: https://github.com/marketplace/actions/install-and-cache-apt-tools#actions-use-apt-tools

Package seams not available at all.
Rebase later again when it is.

Yeah, I think we need to wait until github updates the runner images

todo:
re-enable:
sudo depmod --verbose
sudo modprobe --verbose snd-dummy

is this ready?

is this ready?

Yes, but I would apply it when the image runner update is done.https://github.com/actions/runner-images/issues/8080#issuecomment-1680088869

@chrisbra I guess we can merge it.


Collector catches Task failures and warns

When executing tasks using Collector a single exception will cause the entire run to fail. When executing a bunch of long-running tasks in sequence it's usually preferable to simply warn when an individual task fails. Alternatively we could add a parameter to control whether it is strict or not.

Seems like a reasonable change to me. I'm happy to merge once you think it is ready.

Ready.

Ok merging now.


cloudflare_record resource tries to use hashicorp/cloudflare provider

Terraform version
Terraform v0.14.1

provider registry.terraform.io/cloudflare/cloudflare v2.17.0
provider registry.terraform.io/hashicorp/aws v3.9.0

Affected resource(s)
cloudflare_record
Expected behavior
It should have attempted to create the cloudflare record.
Actual behavior
Initializing modules...

Initializing the backend...

Initializing provider plugins...
- Finding hashicorp/cloudflare versions matching "2.17.0"...
- Reusing previous version of hashicorp/aws from the dependency lock file
- Installing hashicorp/aws v3.9.0...
- Installed hashicorp/aws v3.9.0 (signed by HashiCorp)

Error: Failed to query available provider packages

Could not retrieve the list of available versions for provider
hashicorp/cloudflare: provider registry registry.terraform.io does not have a
provider named registry.terraform.io/hashicorp/cloudflare

Steps to reproduce

Add cloudflare as a provider
Add a cloudflare_record block to create a record
Run terraform init

It seems after further testing that by simply having the provider "cloudflare" block, I run into this issue.

It seems after further testing that by simply having the provider "cloudflare" block, I run into this issue.

This is a side effect of upgrading to Terraform 0.13, not a specific provider. Check out the documentation at https://www.terraform.io/upgrade-guides/0-13.html#explicit-provider-source-locations for the change and what you need to do in order to resolve it.

This is a side effect of upgrading to Terraform 0.13, not a specific provider. Check out the documentation at https://www.terraform.io/upgrade-guides/0-13.html#explicit-provider-source-locations for the change and what you need to do in order to resolve it.


GPU model does not release memory

For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: Ubuntu 16.04
Compiler:
Package used (python/R/jvm/C++): python
xgboost version used: 0.7
If installing from source, please provide

The commit hash (git rev-parse HEAD) 14c6392381c3b63996af4bd9bb13840dd20e78bf
Logs will be helpful (If logs are large, please upload as attachment).

If you are using jvm package, please

add [jvm-packages] in the title to make it quickly be identified -
the gcc version and distribution -

If you are using python package, please provide

The python version and distribution 2.7.12

Steps to reproduce
Snippet of the code i run:
model = XGBClassifier(max_depth=6,
                      learning_rate=0.5,
                      tree_method='gpu_hist',
                     objective= 'multi:softprob')

model.fit(X=x_train,
              y=y_train,
              early_stopping_rounds=2,
              eval_set=[(x_val, y_val)],
              eval_metric=['mlogloss', 'merror'])

I am using https://www.kaggle.com/crowdflower/twitter-airline-sentiment/data for training.
What have you tried?

Disabled tree_method option and the model trained successfully.
Searched for solutions only extensively. I found this related issue #3045.

I have the same problem with xgboost 0.7
After training, the memory is not released
Fri Mar  9 13:14:50 2018
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 384.111                Driver Version: 384.111                   |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla P100-PCIE...  Off  | 00000000:37:00.0 Off |                    0 |
| N/A   69C    P0    45W / 250W |  11550MiB / 16276MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   1  Tesla P100-PCIE...  Off  | 00000000:86:00.0 Off |                    0 |
| N/A   70C    P0    43W / 250W |  11245MiB / 16276MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      4926      C   ...icrosoft/ropen/3.4.3/lib64/R/bin/exec/R   299MiB |
|    0     33460      C   /usr/lib/rstudio-server/bin/rsession       11241MiB |
|    1     33460      C   /usr/lib/rstudio-server/bin/rsession       11235MiB |
+-----------------------------------------------------------------------------+

close due to the fact that we are not able to act on this for now, new discussions are welcomed at https://discuss.xgboost.ai/

Recent discussion: #4482


Players sometimes float when disguising and undisguising?

It happens at random and i'm not sure why. People randomly levitate sometimes when they are disguised or undisguised. I cannot recreate it on purpose and clearing the effects does nothing.

Have you tried reproducing this on a new world?

i dont check my github notifications enough... The bug happens in all worlds, though only in multiplayer. Singleplayer does not have the issue.

for me, idk how, but every entity had no ai and no gravity instead of the one i was disguised as, even the player

so i cna confirm this mod has issues with floating entities

was with the person above on the server, it happened in multiplayer, presumibely after the server kicked us both  when i disguised. seems like it disabled every mob ai and gravity. kinda weird

for context, me and my friend are making a minecraft game mode involving disguises. Whenever we disguise somebody and undisguised them using this mod, they're left floating.

@DmanMC which minecraft version are you using?

this happens to me as well. I use Mob disguises version 1.0.2, and disguise lib 1.0.3. I am on a forge server, 1.16.5, and on forge version 36.2.20

I'm having this issue as well with:
minecraft 1.18.2
fabric-loader 0.14.5
fabric-api 0.51.1
disguiselib 1.2.2
mobdisguises 1.0.8
I first encountered this when making a custom origin, but it can be reproduced without the origins mod
Steps to Reproduce:
be on a server with another player online, must be a real player not a carpet player
/disguise @s clear
you are now stuck floating

I am also having this issue on my Fabric 1.19 world. Is there any fix for this?

other than reconnecting, not yet

other than reconnecting, not yet

I resolved the issue. I had uninstalled the mod and even reconnecting would not solve the issue. I ended up having to edit my player data with NBT Explorer and removing the NoGravity tag.


TimBrogden.Keysticks version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/109156&drop=dogfoodAlpha

This package seems to install the installer. Is it a nested installer?