Error on Mac OS High Sierra 10.13.1 on step 'go install . ./cmd/...'

Hello dear team,
I just tried to install lnd using https://github.com/lightningnetwork/lnd/blob/master/docs/INSTALL.md
on Mac OS High Sierra 10.13.1 , and on step:
go install . ./cmd/...
I see the following error in Terminal:
unexpected directory layout:
import path: github.com/btcsuite/btclog
root: /Users/user/gocode/src
dir: /Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/github.com/btcsuite/btclog
expand root: /Users/user/gocode/src
expand dir: /Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/github.com/btcsuite/btclog
separator: /
Could you please help to figure out why it could happen? Thanks in advance.

Check your $GOPATH to ensure it has been set properly, and also that you did dep ensure before attempting to install anything.

Also upgrade to the latest version of go - the default version with Mac OS home brew is too old.

@Roasbeef, for $GOPATH settings I ran the following commands:
export GOPATH=~/gocode
export PATH=$PATH:$GOPATH/bin
After this 'printenv' returns:
TERM_PROGRAM=Apple_Terminal
INSTALL_DIRECTORY=/usr/local/bin
SHELL=/bin/bash
TERM=xterm-256color
TMPDIR=/var/folders/tz/t4xjctkd0wl_t5f07mcq04fm0000gn/T/
Apple_PubSub_Socket_Render=/private/tmp/com.apple.launchd.OTdeJ1f13P/Render
TERM_PROGRAM_VERSION=400
OLDPWD=/Users/user/gocode/src/github.com/lightningnetwork
TERM_SESSION_ID=A18FEB59-8C88-4DE3-9016-816C3F3F265F
USER=user
SSH_AUTH_SOCK=/private/tmp/com.apple.launchd.GCkdjDTwqD/Listeners
PATH=/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/go/bin:/Users/user/gocode/bin
PWD=/Users/user/gocode/src/github.com/lightningnetwork/lnd
LANG=en_US.UTF-8
XPC_FLAGS=0x0
XPC_SERVICE_NAME=0
SHLVL=1
HOME=/Users/user
LOGNAME=user
GOPATH=/Users/user/gocode
_=/usr/bin/printenv
Regarding 'dep ensure' - I ran it with the following results in Terminal:
MacBook-Air:lnd user$ dep ensure
MacBook-Air:lnd user$
So looks like no errors were there.

@neogeno, for me 'go version' returns: go version go1.10 darwin/amd64
Looks like it's the last available version on https://golang.org/doc/devel/release.html

Ensure that your $GOROOT and your $GOPATH are not overlapping.

Thanks, @Roasbeef , there was no GOROOT env variable in my configuration, so I added it and the issue disappeared. But now when I run 'dep ensure' and 'go install . ./cmd/...' I get:
vendor/github.com/golang/protobuf/proto/text.go:37:2: cannot find package "bufio" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/bufio (vendor tree)
/Users/user/goworkspace/src/bufio (from $GOROOT)
/Users/user/gocode/src/bufio (from $GOPATH)
breacharbiter.go:4:2: cannot find package "bytes" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/bytes (vendor tree)
/Users/user/goworkspace/src/bytes (from $GOROOT)
/Users/user/gocode/src/bytes (from $GOPATH)
vendor/github.com/jrick/logrotate/rotator/rotator.go:33:2: cannot find package "compress/gzip" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/compress/gzip (vendor tree)
/Users/user/goworkspace/src/compress/gzip (from $GOROOT)
/Users/user/gocode/src/compress/gzip (from $GOPATH)
vendor/github.com/roasbeef/btcd/btcec/precompute.go:8:2: cannot find package "compress/zlib" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/compress/zlib (vendor tree)
/Users/user/goworkspace/src/compress/zlib (from $GOROOT)
/Users/user/gocode/src/compress/zlib (from $GOPATH)
routing/pathfind.go:9:2: cannot find package "container/heap" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/container/heap (vendor tree)
/Users/user/goworkspace/src/container/heap (from $GOROOT)
/Users/user/gocode/src/container/heap (from $GOPATH)
peer.go:4:2: cannot find package "container/list" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/container/list (vendor tree)
/Users/user/goworkspace/src/container/list (from $GOROOT)
/Users/user/gocode/src/container/list (from $GOPATH)
vendor/golang.org/x/net/context/go17.go:10:2: cannot find package "context" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/context (vendor tree)
/Users/user/goworkspace/src/context (from $GOROOT)
/Users/user/gocode/src/context (from $GOPATH)
vendor/github.com/btcsuite/golangcrypto/ripemd160/ripemd160.go:13:2: cannot find package "crypto" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/crypto (vendor tree)
/Users/user/goworkspace/src/crypto (from $GOROOT)
/Users/user/gocode/src/crypto (from $GOPATH)
vendor/github.com/roasbeef/btcd/btcec/ciphering.go:9:2: cannot find package "crypto/aes" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/crypto/aes (vendor tree)
/Users/user/goworkspace/src/crypto/aes (from $GOROOT)
/Users/user/gocode/src/crypto/aes (from $GOPATH)
vendor/github.com/roasbeef/btcd/btcec/ciphering.go:10:2: cannot find package "crypto/cipher" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/crypto/cipher (vendor tree)
/Users/user/goworkspace/src/crypto/cipher (from $GOROOT)
/Users/user/gocode/src/crypto/cipher (from $GOPATH)
....(plenty of similar errors)....
vendor/github.com/awalterschulze/gographviz/escape.go:22:2: cannot find package "text/template" in any of:
/Users/user/gocode/src/github.com/lightningnetwork/lnd/vendor/text/template (vendor tree)
/Users/user/goworkspace/src/text/template (from $GOROOT)
/Users/user/gocode/src/text/template (from $GOPATH)
Could you please advise what should I do yo obtain the missing packages? I did
git clone https://github.com/lightningnetwork/lnd $GOPATH/src/github.com/lightningnetwork/lnd
but after the repository cloning I don't have such packages.

I'd try to start from a fresh installation of Go, [following this tutorial])https://golang.org/doc/install#testing).

I'd try to start from a fresh installation of Go, following this tutorial.

You can also do dep ensure -v to get more info from dep.


9l0y7m7 patch 2

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7d265ee). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   94.18%
=========================================
  Files             ?        2
  Lines             ?      258
  Branches          ?        0
=========================================
  Hits              ?      243
  Misses            ?       15
  Partials          ?        0

Continue to review full report at Codecov.

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


Remove deviation gnoi_fabric_component_reboot_unsupported

Remove deviation gnoi_fabric_component_reboot_unsupported for Juniper in metadata.textproto

Pull Request Functional Test Report for #2139 / 084a7929a360b36cad972fa85ef5af630893528c
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

gNOI-3.2: Per-Component Reboot

Cisco 8000E

gNOI-3.2: Per-Component Reboot

Cisco XRd

gNOI-3.2: Per-Component Reboot

Juniper cPTX

gNOI-3.2: Per-Component Reboot

Nokia SR Linux

gNOI-3.2: Per-Component Reboot

Openconfig Lemming

gNOI-3.2: Per-Component Reboot

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

gNOI-3.2: Per-Component Reboot

Cisco 8808

gNOI-3.2: Per-Component Reboot

Juniper PTX10008

gNOI-3.2: Per-Component Reboot

Nokia 7250 IXR-10e

gNOI-3.2: Per-Component Reboot

Help

/gcbrun

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.333%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1409

Relevant Lines:
3041

💛  - Coveralls

Since this test is looking for the existence of removable fabric/linecards and would fail if none exists, FFF chassis w/o removable fabric cards and Linecards will always fail this test. Hence please skip the tests for removable fabric and for removable line cards if the boolean /components/component/state/removable == TRUE.

To resolve conflicts issue, close current pull request.
It will be tracked via new pull request:
https://github.com/openconfig/featureprofiles/pull/2232


Ranked Choice Voting

Would be great to think about what this looks like for proposals...

Maybe it's a different type of proposal all together?

I'm not really sure if this is high priority for v1. Ranked choice is very helpful if we need to decided between multiple choices like voting for candidates for office. I believe the majority of initial proposals will be simple yes or no votes that do not require ranked choice.
However, I think eventually it would be cool for DAOs to vote on more complex proposals where this could be very useful. For example, the DAO wants to pay a developer for a certain project, the voters can decide on the actual value of the payment from a range of options.

I agree. Would be nice for a v2.


Sidebar

In index.ts- add code to turn off dark mode (commented out, uncomment if you want it on)
In App.sass- Import column and menu styles, rearrange imports to separate the imports currently being used
Rename files

Rename Home.tsx to Routes.tsx and change corresponding references in App.tsx
Rename Projects.tsx to Home.tsx and change corresponding references in Routes.tsx
Projects.tsx is an empty file for Projects component

In Home.tsx- change style to columns
In Sidebar.tsx- add style to sidebar
In SidebarButton.tsx- hange logic in sidebarButton to style active button
In SignIn.tsx- style components and turn appInfoModal into modal
In Routes.tsx- add toggle button to switch state of isLoggedIn for development

Looks good.


Disable scoping

Heja,
Great work with the ContentFlow. Really makes it easy to provide some pre-defined layouts and elements to my otherwise ignorant users.
In my current use case, I have no need for the feature the local/global scope provides. Is there a way to simply disable this feature, and in particular hide the buttons for it, via configuration?
Thanks and my best.

Hi @ascarion sorry I completely missed this issue and it's been such a long time now. We remove the local/global feature sometimes too, there's not dedicated way to do it at the moment I'm afraid so we simply hide the feature in CSS like so:
// This site does not support global snippets

.ct-snippet__tool.ct-snippet__tool--settings {
	right: 46px;
}

.ct-snippet__tool.ct-snippet__tool--scope {
	display: none !important;
}

.ct-snippet.ct-snippet--behaviour-pick.ct-snippet--scope-global {
	display: none !important;
}

Hey @ascarion did you finish your project with content flow? I am having trouble with finding some guidelines for javascript or laravel blade templating system, would you have any advice?

@anthonyjb Thanks, that's more or less what I'd ended up doing. I had to fork ContentTools anyways to add some website-specific tools, so I just added this one and edited the CSS in the final build for the site.
@CarrieKami Yes. We're using Twig for templating. Not sure what you're looking for in terms of advice.


Tuscan Style Pork Roast and typo fix

I've always wanted recipes to have the format of "now use ingredient(amount so you don't have to look up to the top of the page)" so I'm trying that out here. I also think recipes need to have a prep time and serving size

who's => whose

Well I should also ask is "nutorious" a Babish inside joke or a mispelling?

I wish I was that clever


[no ci] ingenic-osdrv-t20: use sinfo module from openingenic for T20 support

Implement Wyze T20 device support to OpenIPC:
Switch to using sinfo.ko from the openingenic package, as it supports a wider range of sensors than the pre-compiled versions included in the ingenic-osdrv-t20 package.
Modify load_ingenic  T20 script to match the behavior of the T31 and T40 scripts, using modprobe.

Don't forget to enable BR2_PACKAGE_INGENIC_OPENSDK=y in t20 profiles

@cronyx done =D

@cronyx done =D

and t20_ultimate_defconfig too ;)

already enabled for ultimate
https://github.com/OpenIPC/firmware/blob/551a0c70c37f6729dd7a4caecf8d2f0f5325c88f/br-ext-chip-ingenic/configs/t20_ultimate_defconfig#L63


Prevent early exit after an error

Use the provided timeout to give the runtime longer before quitting after an error.
This fixes a problem where we were seeing a non-fatal parse error prevent our test suite from running.
The error sets a one second timer, during which time, if there are no calls to console.log, then a fatal error is assumed. Making this timeout tunable gives us some leeway in troubleshooting.

since I couldn't easily merge this, and it was a simple change I've made the change and will see where net out. thanks.


Add support for bright & animated textures

https://github.com/gongpha/gdQmapbsp/assets/13400398/8c47f707-fd45-4f5f-86e5-5da472024403

Very nice, maybe something to consider either for this PR or a future one; if it's possible to move the uniform sampler # (96) to gdscript (CPU memory) instead of GPU shader, and keep the shader to 1 sample?

I will surely change the shader from one material for all textures to one material per texture to make it easier to customize individual texture drawings. But I'll probably have to factor in the code to keep updating the uniforms for every shader on the map like lightstyles or animated textures. Maybe using the global uniforms, a new feature in 4.0, might also be a good option.

Using the alternate textures on certain entities doesn't work yet (activated textures on func_button). I will add them after refactoring the surface shader.


chore(package): update kronos-service to version 4.17.1

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add validate command

I would like to add a bk validate <path/to/file.yml> to the various git hooks we uses at my enterprise.
The purpose of this command would be to check/validate the syntax of the given file is well a correct pipeline and the syntax is ok.
It would also allow to validate dynamic configuration: my_super_script.sh | bk validate -

👋 Thanks for this issue. It seems like it would be a useful tool, however we don't have scope to work on something like this right now and I'm not sure how it would actually eventually look in the CLI. I like the idea of a pipeline validator, but it would likely just validate the YAML in a file, rather than support dynamic function etc right now.


net-mgmt/zabbix5-proxy: SQLite3 database has to be deleted while major version upgrade

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 and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs
Describe the bug
The proxy refuses to start after the upgrade from version 4 to 5 due to a database version mismatch. Automatic SQLite3 database upgrades are not supported by Zabbix, as pointed out in the docs:
"Note that with SQLite database on proxies, history data from proxies before the upgrade will be lost, because SQLite database upgrade is not supported and the SQLite database file has to be manually removed. When proxy is started for the first time and the SQLite database file is missing, proxy creates it automatically." (https://www.zabbix.com/documentation/current/manual/installation/upgrade/sources)
Hint: Manually deleting the db-file via CLI fixes this issue.
Related to #1932
To Reproduce
Steps to reproduce the behavior:

Upgrade to OPNsense 20.7
Install the new plugin os-zabbix5-proxy
enable the proxy, in case it isn't
See error in the log /var/log/zabbix/zabbix_proxy.log (or in WebUI: https://yourOPNsensehostname/ui/diagnostics/log/zabbix/zabbix_proxy)

Expected behavior
Automatically delete (or rename) the SQLite-File /var/db/zabbix/zabbix_proxy.db during the upgrade.
Relevant log files (/var/log/zabbix/zabbix_proxy.log)
2020-07-31T22:53:02 | Zabbix does not support SQLite3 database upgrade.
2020-07-31T22:53:02 | The  proxy does not match Zabbix database. Current database version  (mandatory/optional): 04000000/04000005. Required mandatory version:  05000000.
2020-07-31T22:53:01 | using configuration file: /usr/local/etc/zabbix5/zabbix_proxy.conf
2020-07-31T22:53:01 | **************************
2020-07-31T22:53:01 | TLS support:           YES
2020-07-31T22:53:01 | IPv6 support:          YES
2020-07-31T22:53:01 | SSH support:            NO
2020-07-31T22:53:01 | ODBC:                  YES
2020-07-31T22:53:01 | VMware monitoring:      NO
2020-07-31T22:53:01 | Web monitoring:        YES
2020-07-31T22:53:01 | IPMI monitoring:        NO
2020-07-31T22:53:01 | SNMP monitoring:       YES
2020-07-31T22:53:01 | **** Enabled features ****
2020-07-31T22:53:01 | Starting Zabbix Proxy (active) [yourOPNsensehostname]. Zabbix 5.0.2 (revision 352ca05870).

I'd like to add this functionality, probably in the context of #2116...

This will be fixed in os-zabbix-proxy 1.6:
https://github.com/opnsense/plugins/pull/2657


Fix for older iris versions (<3.10)

Addresses #429.

For the record, the readthedocs errors seem to have been caused by this block of code
here: https://github.com/SciTools/cf-units/blob/446fcc2323c01146448366593a2766e62b1a9e82/cf_units/init.py#L186-L191
it seems like this might have been caused by there being no existing setting for LC_NUMERIC (or, somehow, the existing setting was invalid), so when cf_units attempts to revert it back to the original setting, it breaks. I'm not sure what the defaults are for readthedocs, but setting them explicitly seems to do the job.


Add TO Go deliveryservices/id/capacity

Branched from #2305 - recommend merging that first. Closes #3788

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/1901/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2059/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4164/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4917/

Rebased with master and #2305, includes #3927 changes.
Manually tested again, still works as expected, still matches Perl.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4920/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4947/


Some questions about the keypoint detection experiments

Nice work and thanks for sharing the code and data! I have a few questions regarding the keypoint detection experiments in table 2:

The exact definition of PCK: I noticed that different papers are using slightly different versions of PCK in terms of the threshold (e.g., the keypoint is considered correct if its distance to GT is within X% of the longer side of an image in some works, while others use the diagonal length of an image as a reference). Could you clarify the PCK metric used in your paper?

Is it possible to share the keypoint annotations and images for comparisons?

Has you find the answer? I am also interested.


Update .gitignore

NOTE
Please verify that the base repository above has the intended destination!
Github by default opens Pull Requests against the parent of a forked repository.
If this is your personal fork and you didn't intend to open a PR for contribution
to the original project then adjust the base repository accordingly.

update


Visualize empty memlets (i.e., dependency edges) as dotted lines

Closes https://github.com/spcl/dace-vscode/issues/245

Please provide a screenshot here :)

I'd also use a gray color myself. However, we may have discussed using that for non-leaf memlets just to emphasize that they do not generate code.

I'd also use a gray color myself. However, we may have discussed using that for non-leaf memlets just to emphasize that they do not generate code.

I agree with this, but as you pointed out the gray is already going to be (most likely) occupied. Do you have a suggestion of what other style we could use? I agree that the color should ideally also differ.

This looks good. Maybe no arrowhead would make that difference

This looks good. Maybe no arrowhead would make that difference. You need it connected, not moving.

Good idea, I'll make that change

This looks good. Maybe no arrowhead would make that difference. You need it connected, not moving. In that case, maybe the line can even stay solid..? Unsure

Done

Looks good. Tasklet highlighter might be slow for long codes but that should not be the default case. We may want to consider a length cutoff where we just draw the contents.

We should observe, a cutoff would be easy to add if necessary. The length of the tasklet code itself should not have a noticeable impact, only tasklets with dozens of in/out connectors each could potentially pose a challenge in my view.


Unable to debug any code in ramda

Hello, I am new to working on existing open source project and unable to debug any existing code in this project.
1 I successfully downloaded the project by cloning using vbcode from github. and run npm install
Nothing happens trying to debug a simple function like add in vscode. It ignores breakpoints and anyb console.log put in add.js are not executed.
I tried to run command "node add.js" but that leads to an error.
C:\Users\aspie\source\repos\ramda\source\add.js:1
import _curry2 from './internal/_curry2.js';
^^^^^^
SyntaxError: Cannot use import statement outside a module
at Object.compileFunction (node:vm:352:18)
at wrapSafe (node:internal/modules/cjs/loader:1032:15)
at Module._compile (node:internal/modules/cjs/loader:1067:27)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
at node:internal/main/run_main_module:17:47
PS C:\Users\aspie\source\repos\ramda\source>

I even tried lodash but the same thing happened.

Remember that this is a library, and that you won't run the individual files directly.  None of them does anything if you try to run them from the command line.

Thanks very much. I mange to fix the problem. I needed to add the "type": "module" to the Json package.
At the moment there are a few things I need to become familiar in javascript and testing tools such as mocha. However, I will become familiar with them soon.


http://data.lip6.fr/cadene/pretrainedmodels/ has no connection

I tried to load pre-trained model "inceptionresnetv2", but it failed to connect.
The command as below:
pretrainedmodels.dict[inceptionresnetv2](num_classes=1000, pretrained='imagenet')
and I got the error:
HTTPConnectionPool(host='data.lip6.fr', port=80): Max retries exceeded with url: /cadene/pretrainedmodels/inceptionresnetv2-520b38e4.pth (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fd5f2e1f0f0>: Failed to establish a new connection: [Errno 110] Connection timed out',))
I was confused for a long time. thank you.

Hi. I seems to experience the same connectivity issue (resnext50 model weights are also unavailable). It seems the whole http://data.lip6.fr/ is down.

@JasperShian @BloodAxe Yes it is down. It should come back in less than an hour.

Thank your reply, it's available now :)

Is it closed now?

Tere is an unfortunate problem with our servers. It will come back soon.
Sorry for the inconveniences.
On Sun, May 5, 2019, 10:21 Magic-chao<EMAIL_ADDRESS>wrote:

Is it closed now?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Cadene/pretrained-models.pytorch/issues/152#issuecomment-489402692,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDW6LX3XIBALUZ2QXFPSRDPT2KITANCNFSM4HJJA6RQ
.

@Cadene  Could you provide SE-Resnet101 architecture and weights , which is supported in the MMdnn now. I see you provided now(se_resnet101-7e38fcc6.pth) is only contain weights which can't be converted in MMdnn. Thanks!

Hi, anyone else experiencing issues with http://data.lip6.fr/? Seems like the whole site is down. :(

@prnvkbr I know why. I will try to push a fix in the next 24 hours

Thanks for your quick response, @Cadene!

I tried loading fbresnet 152 from this URL: "http://data.lip6.fr/cadene/pretrainedmodels/fbresnet152-2e20f6b4.pth". I am getting the following error.
urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate has expired (_ssl.c:1091)>

I get the same error as @shreyan2.
@Cadene any workarounds?

download it manually to the desirred place without ssl verification:
Something like:
wget --no-check-certificate http://data.lip6.fr/cadene/pretrainedmodels/se_resnext50_32x4d-a260b3a4.pth -P ~/.cache/torch/hub/checkpoints/se_resnext50_32x4d-a260b3a4.pth

download it manually to the desired dir path without ssl verification:
Something like:
wget --no-check-certificate http://data.lip6.fr/cadene/pretrainedmodels/se_resnext50_32x4d-a260b3a4.pth -P ~/.cache/torch/hub/checkpoints

您的邮件，我已收到，谢谢，祝好~


FileNotFoundError: [Errno 2] No such file or directory: 'data/REDS/train_sharp_bicubic/X4/000/00000000.png'

why l run basicvsr,train-----FileNotFoundError: Caught FileNotFoundError in DataLoader worker process 0. and
FileNotFoundError: [Errno 2] No such file or directory: 'data/REDS/train_sharp_bicubic/X4/000/00000000.png'
But I have this file inside this project

You can check your working directory. Usually, we launch training at the root directory of mmediting.

I met the same problem, but the working directory is right.

Does ls data/REDS/train_sharp_bicubic/X4/000/00000000.png at current working directory output the correct file?

I met the same problem, but the working directory is right.

I met the Same problem, although I use the full path of the target file, it still not working.

The problem seems to be when use full file path at windows, the os.path.join() funciton will concate the path in the format of C:TopLevelDir\\SecondLevelDir insetead of C:\\TopLevelDir\\SecondLevelDir, which cannot be used in open function

Does ls data/REDS/train_sharp_bicubic/X4/000/00000000.png at current working directory output the correct file?

I met the same problem, but the working directory is right.
The problem seems to be when use full file path at windows, the os.path.join() funciton will concate the path in the format of C:TopLevelDir\SecondLevelDir insetead of C:\TopLevelDir\SecondLevelDir, which cannot be used in open function

The problem seems to be when using os.path.join in windows, the drive letter will be concatenate in format of C:TopDir\\SubDir instead of C:\\TopDir\\SubDir, which is not be able to used as parameter of open() funciton.
I found this problem in file mmedit/apis/restoration_video_inference.py, line 96. I am not sure whether other parts of code have the same problem.

@Z-Fran
Please check this issue.

This bug will be resolved in this PR.


fix(json-mapper): remove propertyKey from excluded keys on additionalProps

only propertyName (mapped key from input to model) are excluded from additional Properties mapper.
@AdditionalProperties(true)
class Test {
  @Property()
  @Name("test_map")
  testMap: string
}

deserialize({
  test_map: 2,
  testMap: 1
}, {type: Test, useAlias: true}) // => Test { testMap: 1, test_map: "1"}

testMap is considered as an additionalProperties while test_map is a mapped property.
deserialize({
  test_map: 2,
  testMap: 1
}, {type: Test, useAlias: false}) // => Test { testMap: "1", test_map: 2}

if useAlias is false, testMap is considered as mapped property while test_map become an additional properties
This fix rollback a previous fix introduced in the json-mapper refactoring that break the production behavior.

:tada: This PR is included in version 7.36.0-rc.2 :tada:
The release is available on:

v7.36.0-rc.2
GitHub release

Your semantic-release bot :package::rocket:

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

v7.36.0
GitHub release

Your semantic-release bot :package::rocket:


Method to efficiently extract instances allocated to each leaf in each tree making up the trained GBM model

Summary
I am looking for an efficient way to extract the training data instances allocated to each leaf node of an arbitrary tree from a trained LightGBM model.
More formally, a way to extract the partitioning of training data instances induced by the leaf nodes of each tree in the trained LightGBM ensemble.
(I am assuming that LightGBM keeps track internally of which training data instances are allocated to the nodes of the constituent trees.... please do let me know if this is not the case!)

Motivation
There is currently no easy/convenient/performant way to extract the training data instances that have been allocated to the same leaf of a particular tree in a trained GBM model (that I know of...). Efficient access to the instances in each tree leaf would enable:

Custom approaches to generating tree-level predictions. For regression problems, a tree prediction for a given leaf is simply the mean of the targets of training data instances allocated to that leaf during training. But suppose you would prefer the median rather than the mean, of the 5th percentile (=>quantile regression), or the variance across all instances in the leaf, or some other arbitrary calculation over all the instances collected in a leaf?

Measures of instance importance based on which instances are allocated to the same leaf for a particular prediction.

Efficient troubleshooting by examining the kinds of instances that underpin the calculation of a particular prediction.

Description
I can think of two approaches to extract the instances allocated to a particular leaf of a particular tree for a trained LightGBM booster object. Neither are particularly robust, convenient or performant...
Approach 1:
Assume that if two training data instances share the same predicted value from a specific tree, then it is the case that they are allocated to the same leaf in that tree tree. This reasonably easy to implement. But of course, the assumption won't always hold, and in that case this approach will fail.
# utility function to create leaf membership matrix D
# element d_f,c in D takes value of 1 IFF observations f and c are allocated to the same leaf, else 0
# (we assume instances f and c with equal tree predictions implies allocated to same leaf)

def getD(tree):
    # tree contains a list of predicted values from a tree
    tree_len = len(tree)
    D = np.full((tree_len, tree_len), np.nan)
    for f in range(0, tree_len):
        for c in range(0, tree_len):
            same = (tree[f] == tree[c])
            #print(f" [{f},{c}] is {same}")
            D[f][c] = same
    return D

Approach 2
Extract the tree structure using model.trees_to_dataframe() or model.dump_model() and then programmatically allocate training data instances to each leaf using the tree structure and rules provided. Ironically, this is simply re-doing what LightGBM needs to do internally in order to make predictions; but it will be much slower...

What else?
As a starting point I'd be happy to hack a local cloned version of the LightGBM codebase... any pointers as to which data structures to look at or which functions/objects might be useful for implementing this kind of functionality would be appreciated.
References

Thanks for using LightGBM and opening this report.

There is currently no easy/convenient/performant way to extract the training data instances that have been allocated to the same leaf of a particular tree

I'm happy to tell you that LightGBM directly supports this via pred_leaf predictions. The R package, Python package, and C API all support that type of prediction.
Consider the following example in Python.
import lightgbm as lgb
from sklearn.datasets import make_regression

X, y = make_regression(n_samples=1000, n_features=10)

dtrain = lgb.Dataset(X, y)

lgb_params = {
    "objective": "regression",
    "verbose": -1,
    "n_estimators": 10,
    "max_depth": 2,
}

bst = lgb.train(
    params = lgb_params,
    train_set = dtrain
)

tree_preds = bst.predict(X[:10], pred_leaf=True)
tree_preds

array([[2, 1, 1, 3, 0, 2, 3, 1, 3, 0],
       [3, 3, 2, 2, 3, 2, 2, 2, 3, 0],
       [3, 3, 1, 2, 3, 2, 2, 2, 3, 0],
       [1, 1, 0, 3, 1, 2, 0, 1, 2, 0],
       [3, 3, 3, 1, 1, 2, 1, 2, 2, 2],
       [0, 0, 0, 1, 0, 3, 0, 0, 0, 0],
       [3, 3, 1, 2, 0, 2, 2, 2, 3, 0],
       [2, 2, 1, 2, 2, 3, 2, 3, 3, 2],
       [0, 3, 3, 1, 2, 2, 2, 2, 0, 2],
       [3, 3, 2, 2, 3, 2, 2, 2, 3, 2]], dtype=int32

Each column in that array represents one tree in the model, and the value in each corresponding row is the index of the leaf node that that observation falls into for that tree.
I think that should efficiently provide the raw data for the types of questions you're asking.

Works perfectly! Thanks & closed.


Implement layout_minWidth and layout_minHeight

This allows you to specify a minimum size your view is allowed to
shrink to. #1
I plan to implement max-width/height as well in a separate commit, but I want to make sure I'm on the right track first.

Wow thank you! I won't have access to my laptop for a day or so, once I have one, let me start reviewing your code, thanks!

Took a first round of look. Could you confirm the comments? Thanks!

Updated in response to above comments

Hi, thanks for your continuous effort.
If any of my review comments are not clear, please feel free to let me know.

Hi, thank you for your contribution!
I self addressed my review comments in another PR based on your PR.
https://github.com/google/flexbox-layout/pull/51
So let me close this PR.


Added hidden options for upcoming cloud DRL

These are currently meant for testing only.

This looks correct. My only concern is that it's essentially temporary code that will go away right?


Make this repository public?

@kylebarron is there a good reason for this repository to be private? If there's nothing "private" in here, I would rather we default to public.

This repo is now public!


People Counting

Is there any sample of  people counting using OpenCVSharp ?
Appreciate any assistant.
Regards

https://github.com/shimat/opencvsharp_samples/blob/master/SamplesCS/Samples/HOGSample.cs


Implemented Complex Power

#36

The failed tests are due to differences in precion of java and python when computing the power of complex number. The failed test didn't fail on my computer, maybe someone could else could test if even more test are failing on their machine?

Failures on other machines don't matter; it's only CI that ultimately matters. If there's an answer that is reliably off on other machines, then we can add that substitution in as well. So as long as you add the two extra substitutions that are needed to make the last CI run pass, this should be good to go.

Awesome work - thanks!


Ignore context cancelled errors in logging

Description
Ignore context cancelled errors which are from the client cancelling the request
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
UI tests

/test gke-postgres-ui-e2e-tests


internal/dag: add support for ingressroute TLS documents

Blocked on #434
Support for generating 443 routes is missing from ingressroute processing.

Not necessary for beta.1

ok to bump to 0.6, we cannot release ingressroute without a tls story -- it
would be embarrassing.
On 26 June 2018 at 01:00, Ross Kukulinski<EMAIL_ADDRESS>wrote:

Not necessary for beta.1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/heptio/contour/issues/452#issuecomment-399982993, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA1VhH1WtYnAK0ZfeQ37KbUBP7UAXks5uAPsCgaJpZM4U1h-6
.

I complete agree we can't release to public w/o TLS story.

Closed by #596


368

Why:
Closes #368
What's being changed:

Added area-of-triangle.c
Updated README.md for area-of-triangle.c

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

https://codinasion.web.app/programme/area-of-triangle/


my first change - typo in SetStents.m

Hiya Steve does this work?

Hello again OK, so doproductionruns = true / false

Run matlab and set the current directory to the base repo."

Might be good to add "( i.e. /htadelay-master )" at the end of this sentence, just to make clear what the base repo is (I know it's obvious, but I set the folder to `delaycore' which was silly but just think of me as being a dim user)

onder if we could tag figure files with MC' or PDE' to show which method generated them (would be nicer than referring to `jaggedness' and so on).

Understand that the last * is not great if both are plotted in the same fig, tho we could say MCPDE


[JWT Token Generation] Unable to generate a token with Expiration value of Unlimited Days

Affected module
Backend + Ingestion Framework
Describe the bug
A clear and concise description of what the bug is.
To Reproduce

Generated the private and public keys using the following: `

openssl genrsa -out private_key.pem 2048
openssl pkcs8 -topk8 -inform PEM -outform DER -in private_key.pem -out private_key.der -nocrypt
openssl rsa -in private_key.pem -pubout -outform DER -out public_key.der

Add the full path to the public_key.der and private_key.der to the envrionment variables
Ran OpenMetadata using IntelliJ
Go to Bots -> Selected Ingestion Bot -> Generate JWT token -> Select Expiration as "Unlimited Days" -> Click on Generate
The process fails at this step and the screen shows a message (Oops! An unexpected error occurred)

Expected behavior
JWT token should be generated which has unlimited validity, similar to the JWT tokens being issued for 7, 30, 60 and 90 days
Version:

OS: MacOS X Catalina 10.15.7
Python version:
OpenMetadata version: [e.g. 0.10.1]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Logs from Mac:
[0:0:0:0:0:0:0:1] - - [20/May/2022:12:28:16 +0000] "PUT /api/v1/users/generateToken/<redacting because it looks to be a common endpoint for all tokens being generated for me> HTTP/1.1" 400 241 "http://localhost:8585/bots/ingestion-bot" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.75 Safari/537.36" 7

Screenshot:

ok let me check


Semantic syntax checker mis-resolves private fields in class computed fields

Private fields in computed keys resolve to private properties of enclosing class. So obj.#x does not reference C's #x binding here, and in both cases it should be a syntax error:
class C {
  #x;
  [obj.#x]() {}
  [obj.#x] = 123;
}

Oxc playground

VM804:3 Uncaught ReferenceError: obj is not defined
    at <anonymous>:3:8


Introduce different views of the data based on quality

https://miro.com/app/board/uXjVNPsG_-I=/?moveToWidget=3458764581259140910&cot=14
still need to agree what is best to do next
Revisit our definition of a good data product / dataset
Implement quality gate for data to make it in to the front end?
OR
Introduce different views in the front end for different quality levels

Is it as simple as good or bad? Might be the right for now


⚠ GitLab Server has degraded performance

In 74208ea, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6692 ms

Resolved: GitLab Server performance has improved in b3563b0.


particle screen space anchor

We should be able to anchor a particle system like any other UI element when using screen space.

Interesting. However, I'd still like it if I as able to anchor a particle to the screen like any other UI element. It would be great for the user this normalization of behavior.

You basically can by making the particle system entity a child of an entity with an element component.
Then you can anchor the element entity how you would normally.

You basically can by making the particle system entity a child of an entity with an element component.
Then you can anchor the element entity how you would normally.

It doesn't work. Try making a particle system a child of an element that is anchored to the bottom of the screen.

https://playcanvas.com/editor/scene/1281372
It looks anchored to me
https://user-images.githubusercontent.com/16639049/143674399-1cd5d1c8-9ce4-4143-b9a0-5864bd05aebd.mp4

Hm....I must've done something wrong then.


Post release automated changes for msi releases

Post release automated changes for azure-arm-msi

API change check
API changes are not detected in this pull request.


Fix GetDsoHandle segfault

Currently segfaults when LD_PRELOAD_PATH isn't set when it tries to include it in a log message:
Process 86579 launched: '/Users/spinlock/src/carpedm20/DCGAN-tensorflow/.venv/bin/python' (x86_64)
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcublas.8.0.dylib locally
Process 86579 stopped
* thread #1: tid = 0x15ed75, 0x00007fff89152132 libsystem_c.dylib`strlen + 18, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x00007fff89152132 libsystem_c.dylib`strlen + 18
libsystem_c.dylib`strlen:
->  0x7fff89152132 <+18>: pcmpeqb xmm0, xmmword ptr [rdi]
    0x7fff89152136 <+22>: pmovmskb esi, xmm0
    0x7fff8915213a <+26>: and    rcx, 0xf
    0x7fff8915213e <+30>: or     rax, -0x1
(lldb) bt
* thread #1: tid = 0x15ed75, 0x00007fff89152132 libsystem_c.dylib`strlen + 18, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x00007fff89152132 libsystem_c.dylib`strlen + 18
    frame #1: 0x000000010651cd8a _pywrap_tensorflow.so`perftools::gputools::internal::DsoLoader::GetDsoHandle(tensorflow::StringPiece, void**, perftools::gputools::internal::DsoLoader::LoadKind) + 282
...
    0x10651cd31 <+193>: lea    rsi, [rip + 0xf7545a]     ; "Couldn't open CUDA library "
    0x10651cd38 <+200>: lea    rdi, [rbp - 0x228]
    0x10651cd3f <+207>: mov    edx, 0x1b
    0x10651cd44 <+212>: call   0x1043d2620               ; std::__1::basic_ostream<char, std::__1::char_traits<char> >& std::__1::__put_character_sequence<char, std::__1::char_traits<char> >(std::__1::basic_ostream<char, std::__1::char_traits<char> >&, char const*, unsigned long)
    0x10651cd49 <+217>: lea    rdi, [rbp - 0x228]
    0x10651cd50 <+224>: mov    rsi, r12
    0x10651cd53 <+227>: mov    rdx, r15
    0x10651cd56 <+230>: call   0x1067f9c80               ; tensorflow::operator<<(std::__1::basic_ostream<char, std::__1::char_traits<char> >&, tensorflow::StringPiece)
    0x10651cd5b <+235>: lea    rsi, [rip + 0xf7544c]     ; ". LD_LIBRARY_PATH: "
    0x10651cd62 <+242>: lea    rdi, [rbp - 0x228]
    0x10651cd69 <+249>: mov    edx, 0x13
    0x10651cd6e <+254>: call   0x1043d2620               ; std::__1::basic_ostream<char, std::__1::char_traits<char> >& std::__1::__put_character_sequence<char, std::__1::char_traits<char> >(std::__1::basic_ostream<char, std::__1::char_traits<char> >&, char const*, unsigned long)
    0x10651cd73 <+259>: lea    rdi, [rip + 0xf6ffd4]     ; "LD_LIBRARY_PATH"
    0x10651cd7a <+266>: call   0x106af84c8               ; symbol stub for: getenv
    0x10651cd7f <+271>: mov    rbx, rax
    0x10651cd82 <+274>: mov    rdi, rbx
    0x10651cd85 <+277>: call   0x106af87fe               ; symbol stub for: strlen
    0x10651cd8a <+282>: lea    rdi, [rbp - 0x228]
    0x10651cd91 <+289>: mov    rsi, rbx
    0x10651cd94 <+292>: mov    rdx, rax
    0x10651cd97 <+295>: call   0x1043d2620               ; std::__1::basic_ostream<char, std::__1::char_traits<char> >& std::__1::__put_character_sequence<char, std::__1::char_traits<char> >(std::__1::basic_ostream<char, std::__1::char_traits<char> >&, char const*, unsigned long)
    0x10651cd9c <+300>: lea    rdi, [rbp - 0x228]
    0x10651cda3 <+307>: call   0x10681be90               ; tensorflow::internal::LogMessage::~LogMessage()
    0x10651cda8 <+312>: lea    rax, [rip + 0xf75413]     ; "could not dlopen DSO: "

Trying to verify but still trying to get a working dev environment setup.

Can one of the admins verify this patch?

Jenkins, test this please

Unrelated failure, merging.


brew search gives git credentials error

[ x ] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[ x ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[ x ] ran brew update and can still reproduce the problem?
[x  ] ran brew doctor, fixed all issues and can still reproduce the problem?
[ x ] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)

I am trying to use brew search in order to find a package.

What happened (include command output)

I get a git credentials error even though my credentials are working fine and I can push and pull to github.
brew search zoom
Warning: Error searching on GitHub: GitHub {
  "message": "Bad credentials",
  "documentation_url": "https://developer.github.com/v3"
}

The GitHub credentials in the macOS keychain may be invalid.
Clear them with:
  printf "protocol=https\nhost=github.com\n" | git credential-osxkeychain erase
Or create a personal access token:
  https://github.com/settings/tokens/new?scopes=gist,public_repo&description=Homebrew
and then set the token as: export HOMEBREW_GITHUB_API_TOKEN="your_new_token"

What you expected to happen

I expect to not see that error message, even though the results I want I displayed underneath.

Step-by-step reproduction instructions (by running brew commands)

Developer/Datadog/consul-config  master ✔                                                                                8m
▶ brew update
Already up-to-date.

Developer/Datadog/consul-config  master ✔                                                                                8m
▶ brew config
HOMEBREW_VERSION: 1.7.6
ORIGIN: https://github.com/Homebrew/brew
HEAD: f239b7a62790cfb4f33bda9aab7597d43f1b1e3b
Last commit: 4 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: ae68a80c0257579b3c109cb56c53c6dcc979d984
Core tap last commit: 2 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_GIT: git
HOMEBREW_MACOS_VERSION_NUMERIC: 101306
CPU: quad-core 64-bit kabylake
Homebrew Ruby: 2.3.7 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 10.0 build 1000
Git: 2.19.0 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: 10.0.2
macOS: 10.13.6-x86_64
CLT: <IP_ADDRESS>.1.1535735448
CLT headers: N/A
Xcode: N/A
XQuartz: N/A

Developer/Datadog/consul-config  master ✔                                                                                8m
▶ brew search zoom
Warning: Error searching on GitHub: GitHub {
  "message": "Bad credentials",
  "documentation_url": "https://developer.github.com/v3"
}

The GitHub credentials in the macOS keychain may be invalid.
Clear them with:
  printf "protocol=https\nhost=github.com\n" | git credential-osxkeychain erase
Or create a personal access token:
  https://github.com/settings/tokens/new?scopes=gist,public_repo&description=Homebrew
and then set the token as: export HOMEBREW_GITHUB_API_TOKEN="your_new_token"

==> Casks
photozoom-pro              rightzoom                  zoom                       zoomus                     zoomus-outlook-plugin

Nevermind I didn't try the personal access token part:
Or create a personal access token:
  https://github.com/settings/tokens/new?scopes=gist,public_repo&description=Homebrew
and then set the token as: export HOMEBREW_GITHUB_API_TOKEN="your_new_token"

I thought that was part of the instructions to delete the existing key


replay HTTPS connections without doing the SSL negotiation with upstream

I'm trying to replay a HTTPS request with mitmproxy without needing to be connected to the internet, this fails because in the CONNECT call mitmproxy tries to make the tcp connection and SSL negotiation with the upstream proxy, is there a way to avoid this?

Thanks for the report. This is indeed an issue we're working on to fix!
refs #548

Thanks, this is fixed on master now. :smile:


HDDS-5534. Verify config is updated on all OMs before proceeding with Bootstrap

What changes were proposed in this pull request?
When an OM is bootstrapped, it sends a SetConfiguration ratis request to leader OM and this request is propagated to all existing OMs in the ring. Existing OMs add the new OM to their peer list by reloading the config files and getting the new node information from the reloaded config files. But if the config file (ozone-site.xml) has not been updated on any OM, then that OM would crash when processing the SetConfiguration request.
To avoid this scenario, before a bootstrap request is sent to the leader OM, the bootstrapping OM must verify that all the existing OMs have its information in their ozone-site.xml. To achieve this, we have added a new OMMetadata protocol. This protocol would be called during OM decommissioning also to verify that ozone-site.xmls are updated.
It could happen that an OM is down or not responding to a bootstrapping OMs metadata information call. In this scenario, to proceed with bootstrap, a "force" option is provided. During force bootstrap, the check for config update on existing OMs will be skipped.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5534
How was this patch tested?
Will add unit tests

@bharatviswa504 can you please take a look when you can. Thanks.

Thank you @bharatviswa504 for reviewing the PR. I have addressed some of your comments and replied to the rest.

One question what advantage of this validation compared to failing with exception, will we be in some inconsistent state? I just want to understand reason behind going this route. Because if any of the node is down, we cannot make use of this check i believe.

If the config has not been updated on an OM and the SetConfiguration request is executed, then it could lead to that OM crashing out. If we perform this check before, we can avoid a healthy OM going down because of not updating its config with the new OM info. A healthy running OM should not crash when trying to add another OM.
If a node is down, this check would at the very least warn the user to double check the configs before proceeding with force bootstapping.

Thank you @bharatviswa504 for reviewing this and for the comments. I have addressed them and made some improvements as discussed above in the latest patch. I am working on unit tests and will post an update soon.

@bharatviswa504 I have added some unit tests and made improvements in the existing bootstrap tests as well. Please take a look when you get a chance. Thanks.

Thank You @hanishakoneru for the work.
Overall approach LGTM, few comments/questions

Thanks @bharatviswa504 for the review. I have addressed your comments in the new patch or answered inline.

Just a question, why cant we add this to OMInterServiceProtocol?
Any reason to add it as a fresh OMMetadataProtocol?
Added a new protocol so that it can be used for removeOM admin command also. RemoveOM would have to be an admin command and not from other OMs. So we need a new protocol for admins to perform operations on the OM.

Thanks @bharatviswa504 for the review. I have addressed your comments in the new patch or answered inline.

Just a question, why cant we add this to OMInterServiceProtocol?
Any reason to add it as a fresh OMMetadataProtocol?

Added a new protocol so that it can be used for removeOM admin command also. RemoveOM would have to be an admin command and not from other OMs. So we need a new protocol for admins to perform operations on the OM.

As this verify configuration is InterOM only right, not external client interaction. And we can add to OmClientProtocol that API right, do we need a new protocol for admin operations. And in OM server we can check whether admin has issued the command or not correct, Let me know if i am missing something here.

Thank You @hanishakoneru for the update.
Overall LGTM, one question on the new protocol thing, commented inline.

As this verify configuration is InterOM only right, not external client interaction. And we can add to OmClientProtocol that API right, do we need a new protocol for admin operations. And in OM server we can check whether admin has issued the command or not correct, Let me know if i am missing something here.

It would be better to keep the admin protocols separate from client protocols. RemoveOM operation is a critical operation and I don't think it should be exposed to clients at all.

As this verify configuration is InterOM only right, not external client interaction. And we can add to OmClientProtocol that API right, do we need a new protocol for admin operations. And in OM server we can check whether admin has issued the command or not correct, Let me know if i am missing something here.

It would be better to keep the admin protocols separate from client protocols. RemoveOM operation is a critical operation and I don't think it should be exposed to clients at all.

But the OmMetadatProtocol, is used between inter OM's if i understand correctly, the admins/clients will not interact with this.
What is the advantage of the seperating admin protocol, as when it comes to implementation from client this will be exposed from admin command like ozone admin, and we check at server if user is admin or not.
Do we see any benifits of having a seperate protocol for admin operations? And also as said this protocol which is added is again used between OM's, so I donot understand of linking between admin here.

From security perspective one I can think of is, we can add service level ACL's so that it can be rejected by Rpc Layer it self.
But not sure this is that critical, as if we have checks for admin methods in server, the same can be achieved.

Thanks @bharatviswa504 for the review.
I have opened HDDS-5978 to add more checks to the bootstrap process.

I have moved this Jira under HDDS-4330.

@bharatviswa504 thanks for the offline discussion. I have updated to Metadata protocol name to Admin Protocol to better represent the purpose.

Thanks @bharatviswa504. I have addressed the comments.

Test failures looks unrelated, triggered another CI run.

@bharatviswa504, thanks for reviewing multiple times and approving it. I will merge this shortly.


views: support for Range requests

Storage.send_file() needs support for specifying a byte range to be sent. Based on the range, it needs to wrap the stream with a proxy which will seek into the file and only send the requested amount of data.
views.ObjectResource.send_object needs support for parsing Accept-Range

This seems to be related to zenodo/zenodo#1599.
Is there any progress on supporting HTTP/1.1 byte range requests? We are planning to use the upcoming InvenioRDM system, and some of our usecases require the ability to download only parts of large files.

There's currently no plans to implement the HTTP range requests in Invenio. We on the other hand plans outsourcing this to the storage system - e.g. if you're using some object store like  MinIO with the S3 interface, or is on another cloud enabled storage system. The storage systems are optimized for sending files, and thus we plan on relying on those for the best performance.

There's currently no plans to implement the HTTP range requests in Invenio. We on the other hand plans outsourcing this to the storage system - e.g. if you're using some object store like  MinIO with the S3 interface, or is on another cloud enabled storage system. The storage systems are optimized for sending files, and thus we plan on relying on those for the best performance.

Hi, I would like to add some motivation for the range request support here.
We would like to use Zenodo to serve files in Zarr format which has becoming a popular standard for serving massive n-dimensional arrays. On top of zarr, there is also downstream image file format that are built on top of it, e.g. NGFF. Similarily, there is also N5 and cogeo.
Zenodo for example, only allow <100 files in one deposition, HTTP range request support is therefore crucial for serving large files in formats such as Zarr. Essentially, it allows lazy access to large files in open data repository, Python scripts or web applications can immediately benefit from it, and the users won't need to wait for hours to download a large file for a quick glance at the data.
@lnielsen Does this mean it less likely for Zenodo to support HTTP range request in a relatively short term (since I imagine it won't be easy for Zenodo to migrate to other storage backend)?

I think this is now working. The only missing part is that Invenio is not exposing the accept-ranges: bytes header.

Thanks for the update, @bgruening!


add contributors

[x] moh
[x] marco
[x] rémi
[ ] ceren

@all-contributors please add @mohmdrezk  for code, design, ideas

@all-contributors please add @marcobarilari  for code, design, ideas, bug, userTesting, review, question

@all-contributors please add @marcobarilari for code, design, ideas, bug, userTesting, review, question, infra, maintenance

@all-contributors please add @Remi-Gau  for code, design, ideas, bug, userTesting, review, question, infra, maintenance

@all-contributors please add @CerenB  for bug, userTesting


Schema DDL: make complex objects size configurable

Migrated from https://github.com/snowplow/schema-ddl/issues/52

Migrated to https://github.com/snowplow-incubator/schema-ddl/issues/40


include dotnet8.0 in TargetFrameworks

Could you include dotnet8.0? Because if somebody wants use xUnitTest while this package is installed the package restore will fail.
(only android, ios, maccatalyst and windows are included)

.net8 is already included in all platforms. Make sure everything is updated
and it will work.
On Sun, 10 Nov 2024, 21:42 Th3L0x, @.***> wrote:

Could you include dotnet8.0? Because if somebody wants use xUnitTest while
this package is installed the package restore will fail.
(only android, ios, maccatalyst and windows are included)
—
Reply to this email directly, view it on GitHub
https://github.com/marcojak/MauiMTAdmob/issues/108, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUIPIXPXOMDY5VJRD2525LZ77HLHAVCNFSM6AAAAABRQTQNC6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGY2DONRYGU3TQMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I mean pure dotnet8.0, unfortunately this is a must for xUnit Test type project

Oh I see.
I'll add it and do some tests and if all works fine, I'll release it asap.
On Sun, 10 Nov 2024, 21:53 Th3L0x, @.***> wrote:

2.PNG (view on web)
https://github.com/user-attachments/assets/65872bd6-45d4-4af6-8f60-61f54fe79eeb
3.PNG (view on web)
https://github.com/user-attachments/assets/bed8c765-531f-46a8-8a19-80992c568086
1.PNG (view on web)
https://github.com/user-attachments/assets/732604a8-79c8-4b3f-b399-12b1d82287ec
I mean pure dotnet8.0, unfortunately this is a must for xUnit Test type
project
—
Reply to this email directly, view it on GitHub
https://github.com/marcojak/MauiMTAdmob/issues/108#issuecomment-2466949860,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUIPISW72U2Z5NOUFIUXWDZ77IVXAVCNFSM6AAAAABRQTQNC6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWHE2DSOBWGA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you!

Hey, do you have any news on this?:)

I'll add it to the next version which should be online by mid-next week.

I've just released version 1.7.1 which includes it.
Closing as completed.


jsx-wrap-multiline is a breaking change

Hi,
Just wondered if you're using semantic versioning? We are extending tslint-react using version 2.*. The new jsx-wrap-multiline rule in 2.1 is causing our builds to fail without us making any changes to the default value which I believe should make it a breaking change. Am I wrong?
Thanks,
Luke

Hey, sorry about that -- so far I've been treating tslint-react.json like tslint:latest, which does not follow semver. I suppose it'd be nice to introduce the same semantics as the core configurations, where you have the option of a stable or a bleeding edge configuration, I just haven't gotten around to supporting that. Would likely take a PR for it though.

closing in favor of #76


Added Solidstart template (Hacktoberfest)

Hacktoberfest 2023
This is my contribution for the #35. This is a solidstart template for using surrealdb. Like I mention in the readme, Its really leaning into the live queries, since I believe solidjs' signals meld really well with realtime data. I do plan on adding some SSR examples using the rest api.
For convenience, here's a link to the template: https://github.com/metruzanca/surreal-solid-template
Do let me know if there's anything missing or if you'd like to see specific features being used and showcased.

Hi @metruzanca
Thank you for contributing to SurrealDB during Hacktoberfest 2023 🎉
Kindly send an email to<EMAIL_ADDRESS>to coordinate the delivery of your SurrealDB swag! 😁


Docs cleanup for Cluster Management and related install page

What changed?
Streamlined the cluster management user guide docs in accordance with https://github.com/weaveworks/weave-gitops-enterprise/issues/2369.
However, there are Docusaurus-related issues for which we need @luizbafilho's help, after trying a few options and nothing working. The issues are preventing me from testing out docs changes in staging.

Why was this change made?
We have not updated these in a while and there is messaging that is confusing.

How did you validate the change?
I'm not able to yet, unfortunately (see above).

Release notes
Update Cluster Management user guide docs to improve user experience
Docs restructured:

cluster-management/add-applications -> (deleted)
cluster-management/managing-existing-clusters -> cluster-management/managing-clusters-without-capi
cluster-management/provider-identities -> cluster-management/advanced-cluster-management-topics/howto-inject-credentials-into-template
cluster-management/troubleshooting -> cluster-management/cluster-management-troubleshooting
cluster-management/cluster-api-providers -> (deleted)
guides/deploying-capa -> (deleted)

Docs available for review in https://staging.docs.gitops.weave.works/docs-changes-june7


Tenacity version out of date

Hi there - the current tenacity version is out of date - it's currently pinned to the 8.* major version and 9.0.0 is now out - this is causing some compatibility issues with other libraries.
If it would be possible to update this, it would be really helpful! Thanks!

happy to accept pr on this

Just merged a fix for this into the main branch, should be fixed in the next instructor release. Going to close this in preparation for this.


Add base64-to-file extension

Description

Screencast

Checklist

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

Hey you should change the icon of your extension, it's going to get rejected per docs.

There are two extensions that I think do similar things:
https://www.raycast.com/DanielSinclair/base64
https://www.raycast.com/eth-p/encoding-tools
I'm not too sure, but it might be better to contribute to one of those extensions instead of making a new one.

It could also be a way to go yes, are you still around @iamforrest

I've attached three screenshots directly here. However, sorry that I'm unsure of the appropriate directory within the codebase to place these images.

So, what are you using as base64? Can you share an example so I can test?

So, what are you using as base64? Can you share an example so I can test?

here is an example:
base64.txt
Raycast Developers.pdf


get_execution_info: Test deprecated contract

This change is

Codecov Report

Merging #1113 (2c7f5aa) into mohammad/Test_txInfo_syscall (e464ad3) will not change coverage.
The diff coverage is n/a.

@@                      Coverage Diff                      @@
##           mohammad/Test_txInfo_syscall    #1113   +/-   ##
=============================================================
  Coverage                         67.89%   67.89%
=============================================================
  Files                                51       51
  Lines                              6578     6578
  Branches                           6578     6578
=============================================================
  Hits                               4466     4466
  Misses                             1706     1706
  Partials                            406      406

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


fix(homepage): Remove trailing slash from URL of Getting Started button

No more trailng slash on this button:

Thanks @janpio !


ServerRequest::getBody method returns an empty stream

ServerRequest::getBody method returns an empty stream, while running file_get_contents("php://input") actually returns the correct POST body. I cannot find anywhere how the body resource is set.
The related place is: https://github.com/Nyholm/psr7/blob/master/src/MessageTrait.php#L160
Suppose we need to read the raw input here?

The body is usually set around the creation of the ServerRequest instance, so this will depend on how you are instantiating.
It doesn’t look like the current ServerRequestFactory has any method that will automatically assign a body. The current PSR-17 draft Interface does not add any method that does this either, so I do not see any priority for us to add it here.
Building a ServerRequest object from an actual web server request is going to be up to your framework.
Another option is to add the body post-creation of the object. Something like this (note: untested):
// Assuming $request is an instance of ServerRequest
$body = (new \Nyholm\Psr7\Factory\StreamFactory())
    ->createStreamFromFile('php://input');
$request = $request->withBody($body);

Ok, thank you for the explanation. I was expecting it would be set by ServerRequestFactory::createServerRequestFromGlobals method though, and it was unexpected to not have one.

I can imagine how that might be a little unexpected! Happy to have explained it, and I will close this issue as resolved.
I have opened 2 new issues:

#43 about getting rid of the createServerRequestFromGlobals method (and several others) so people are less likely to make this mistake. PSR-17 has dropped these too.
#44 about introducing a separate helper class that could be used to create fully filled ServerRequest instances.

Feel free to leave a comment on either of those if you have any further thoughts!

@xaben in case you didn’t subscribe to the other issues: there is a new package (nyholm/psr7-server) that is able to create a ServerRequest using any PSR-17 implementation.
See the README of the latest v1 release for an example!


GIT-811: enable go-runner support for configuring log output

Closes #811
Implementation based on the klog flag deprecation KEP :KEP-2845: Deprecate klog specific flags in Kubernetes Compnents

/cc @damemi

@damemi Done

reached out to @serathius who wrote the klog deprecation kep to ask for some advice on this. The KEP makes it seem like go-runner is really just a replacement for writing logs to a file, not necessarily for stderr/stdout redirection. So I want to make sure this is solving the problem we want
(btw go-runner is now kube-log-runner

https://github.com/kubernetes/enhancements/tree/master/keps/sig-instrumentation/2845-deprecate-klog-specific-flags-in-k8s-components#non-goals doesn't affect non kubernetes/kubernetes components

Thanks @serathius, our understanding is that many of these flags are being deprecated from the klog library itself, is that right? So even though the requirement to stop supporting those flags doesn't affect us outside k/k, we aren't sure how to continue supporting the functionality with klog

Those flags are not deprecated in klog, they are deprecated in K8s components. It's even in KEP name. Goal is to unblock k/k components, while not affecting klog nor external components.

I see, thanks for clearing that up. We may have been overthinking this, so I appreciate the help from someone more knowledgeable about the scope.


fix: [M3-6606] - Add spacing between copy and Linode Rebuild button.

Description 📝
Add's spacing between copy and Linode Rebuild button.

Before
After

How to test 🧪
Verification steps
(How to verify changes)

Navigate to Linode Rebuild form scroll down and observe the spacing for Rebuild Linode button.
...

As an Author I have considered 🤔
Check all that apply

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Can we make the button right aligned? (flex-end)

Also, because there is only one button, I think it would be a lot cleaner to just use a plain "Button" (no ActionsPanel) that is wrapped in a flexbox.

Can we make the button right aligned? (flex-end)

If we did that, I wonder if it would look weird with the left-aligned "Add an SSH Key" button right above? (Or would you suggest right-aligning that too?)

Also, because there is only one button, I think it would be a lot cleaner to just use a plain "Button" (no ActionsPanel) that is wrapped in a flexbox.

+1

Can we make the button right aligned? (flex-end)

If we did that, I wonder if it would look weird with the left-aligned "Add an SSH Key" button right above? (Or would you suggest right-aligning that too?)

Also, because there is only one button, I think it would be a lot cleaner to just use a plain "Button" (no ActionsPanel) that is wrapped in a flexbox.

+1

I think we have a similar pattern of having some buttons on the left and some on the right, as seen in the Linode create flow. IMO, it would be beneficial to render action buttons with the ActionsPanel component since it supports rendering both primary and secondary action buttons. Also, it would enhance the maintainability of primary and secondary action buttons across the CM.


Blaze loader

was #768

This is failing on our internal data (accern impact_score or article_sentiment) with errors like this:
/home/ssanderson/quantopian/qexec/zipline/lib/adjusted_array.pyx in zipline.lib.adjusted_array.Float64AdjustedArray.__cinit__ (zipline/lib/adjusted_array.c:3011)()
    114         if mask is not NOMASK:
    115             if not PyObject_RichCompare(mask.shape, data.shape, Py_EQ):
--> 116                 raise ValueError(
    117                     "Mask shape %s != data shape %s" % (
    118                         (mask.shape[0], mask.shape[1]),

ValueError: Mask shape (253, 8884) != data shape (253, 2835)

My guess is we don't have test coverage for the AssetFinder having data that the Database doesn't have?


test: add unit test for assert for increase coverage to 100%

add unit test for assert function,  make @cosmjs/utils test coverage to 100%

Nice, thanks. Could you run yarn && yarn lint-fix to make the linter happy?

Nah, git hooks are supper annoying. CI takes care of that. Thanks!


First crack at compressor.

I was about to press "approve". But this looks disconcerting: https://github.com/Chia-Network/chia_rs/actions/runs/3833857129/jobs/6525695310#step:7:47
Why was that fuzzing cancelled? and why did it not cause the CI to fail?

rerunning the job made it pass


Import pathlib for Lesson 2 Download notebook

While working on FastAI Part I 2019 - Lesson 2 Download notebook, I needed to add an import for pathlib.  from fastai import * was removed in 615585c43f0fc4e19b28101940267c1bacfd507d which had previously imported pathlib.

Seems np is also missing.  Perhaps add from fastai import * back?  Not sure of the best approach here.

Never mind.  It was due to being on an older version of fastai RE https://forums.fast.ai/t/update-fastai-with-conda-doesnt-update-to-the-latest-version/33546/9.


Upgrade GCS client to pass raw creds as bytes

Fixes #3604

PTAL @brycemcanally

Did you vendor this with govendor? It doesn't look like the vendor.json file has been modified.

Fixed govendoring, addressed feedback, ptal @brycemcanally


A one line change to solve a lot of problems installing plugins with nibs!

Hi, any chance you could you could merge this harmless one line change that solves the problems people have been having using Alcatraz:
https://github.com/supermarin/Alcatraz/issues/196
https://github.com/supermarin/Alcatraz/issues/202
https://github.com/johnno1962/GitDiff/issues/31
This problem has been knocking about for a year now!!
Cheers

Don’t know whats up with you CI but I can’t believe it was this change!

No, CI has been broken for a while, I was actually planning on just ripping out the failing tests for now as they're clearly not adding any value at this point.
Thanks for the PR!

So we remove the existing build first and then re-fetch?

Thats about the size of it. I’ve tested and this really does resolve the problem. I can’t think of any new ones it would create.

I could kind of imagine the fetch failing and this uninstalling your plugin completely?

not if you delete only the build driectory in Alcatraz's local git clone cache

:+1: makes sense, thanks! @supermarin :ship: ?

Hey thanks @jurre and @johnno1962 !
I'm all in for merging it - but I believe it shouldn't live in ATZGit.
IMO it should live in plugin installer, and do it before git invocation.
How does that sound?

Fine by me. Thanks for picking this up! There are a few users who are going to thank you for it.

this should be closed as #237 fixed it


How to process large paired end fastq file?

Hello,
I have about 200 samples. There are 2 fastq.gz file (sample.R1.fq.gz and sample.R2.fq.gz) for every file. Each file is very large (more than 50G compressed file). How can I unzip them and combine the paired end file into one file?
Thank you !

this can be done using zcat: zcat sample.R1.fq.gz sample.R2.fq.gz > sample.fq. If you do not want to store any extra files, you can also run PanGenie using -i <(zcat sample.R1.fq.gz sample.R2.fq.gz) in order to specify the input reads.

Hi Jana,
Hello, I have a few questions about the usage of PanGenie:
You mentionedthat we can concatenate the two FastQs and provid e the result to PanGenie using the option -i by using cat
zcat sample.R1.fq.gz sample.R2.fq.gz > sample.fq.But I have a little doubt.
Generally, the R1 and R2 reads of FastQ files should be handled separately, as they represent sequencing data from two different endpoints of the same DNA fragment.
Could merging the two files before running pangeine break this pairing? Is this likely to greatly increase the amount of computation when there is a large sample of species?
Is there a difference in the results when using a single and paired-end combined file?(sample.R1.fq.gz and sample.R2.fq.gz)
Thanks very much.
Best

Hi Jana,
Hello, Thanks for developing this great tool. I have a question here.
You mentioned that we can concatenate the two fastq.gz files and provide the result to PanGenie by using zcat sample.R1.fq.gz sample.R2.fq.gz > sample.fq. But I have a little confused.
Generally, the R1 and R2 reads of FastQ files should be handled separately, as they represent sequencing data from two different endpoints of the same DNA fragment. Could merging the two files before running pangeine break this pairing? Could there be a difference in the results when using a single (sample.R1.fq.gz) or a paired end file? (sample.R1.fq.gz and sample.R2.fq.gz)? If the difference is not too great here, we may use single-ended sequencing file and save a lot of computation.
Thanks very much.
Best

PanGenie only uses the reads to count kmers. The pair information is not taken into account, which is why the files can be merged without any issue. Yes, the results differ if you only use one of the files, because half of the reads would be missing, which means the kmer counts would be different compared to using both files. So I would not recommend using only one file.


Some improvements on external tools

I documented some of the things that are already possible and fixed a bug when using "none" as input.

A long time ago in galaxy far away…


Add DisableProtobufDesignTimeBuild to VS csproj intellisense

https://github.com/Microsoft/msbuild/issues/3979

Updated the issue.


[SR-16106] Code Actions don't include correct spacing

Previous ID
SR-16106

Radar
None

Original Reporter
@adam-fowler

Type
Bug

Environment
swift-driver version: 1.45.2 Apple Swift version 5.6 (swiftlang-<IP_ADDRESS>3.62 clang-13<IP_ADDRESS>)
Target: x86_64-apple-macosx12.0

Additional Detail from JIRA

Votes
0

Component/s
SourceKit-LSP

Labels
Bug

Assignee
None

Priority
Medium

md5: 46482d3680d87dc38282b4865db4ef55

Issue Description:
The following list of Code Actions do not include the correct spacing

Generate Member Initializer

Expand Switch Cases

Collapse Nested If Statements

Move to Extension

Extract Expression

Extract Repeated Expression

Extract Method

Convert To Switch Statement

For example if I have the following struct
struct Test {
    let a: Int
}

The "Generate Member Initializer" code action produces
struct Test {
internal init(a: Int) {
self.a = a
}
    let a: Int
}

I haven't been able to test all the refactor kinds so there may be others that need fixed as well.
"Add Equatable Conformance" is an example of a code action that generates the correct spacing.

This behavior is intended on the SourceKit-side.
The assumption is that the editor will re-indent the lines based on its own indentation settings. "Add Equatable Conformance" provides correct indentation is because it was implemented by an external contributor and I believe he didn’t know that he didn’t need to worry about indentation.
Once SourceKit-LSP support code formatting, we should run the formatter on the code that should be inserted by the refactoring.


Ctrl-C cannot kill all GraphStorm processes

DGL's launch script kills all training processes if a user types Ctrl-C. This doesn't work in GraphStorm. GraphStorm should also handle Ctrl-C signal.

which PR enabled this?


Whoops, it looks like neovim isn't set up properly

Error

Whoops, it looks like editor neovim isn't setup properly. You should double check that before we start talking, otherwise I may crash.
Error: Not running within a Neovim terminal

But I am on Neovim.

What is the way to ensure that neovim is set up properly?

I was using distrobox, and had exported oatmeal from one distrobox to another. This error goes away if oatmeal is installed in the distrobox I was working on.


My progress got reseted :(

When the game updated it brought me back to the beginning, no achieves, no prestige, no equipment, etc

I have not reset anything, you sure you logged in correctly?

Yeah I can't get it back but it's fine I wasn't really that far in the game anyways

It happened again I'm at  the beginning again I got to I think fire ants? maybe further so I dunno I made sure I logged in correctly and everything :/

Can you send me over, what e-mail you were using?
You're the only one having this issue, thusfar.

Are you on discord? You might want to DM, so that other people don't see your email

You have your email account linked I emailed you so you can see what my email is idk if you'll get it because I hooked it up with my school account


[WIP] Feature LC-14 - Agency self sign up

WIP release branch for agency features

No longer needed. Replaced by https://github.com/Civil-Service-Human-Resources/identity-management/pull/20


Estimate time required to convert simulation utils

simulation/helpers_add_interventions
simulation/helpers_sim_setup

Define estimate to convert the following files to emodpy. I output should include a list of issues broken into easy to manage components.

I'm done with the first pass (took about a week, but with other work). I need some help from Sharon/Zhaowei and have some questions for the researchers and I think it can be wrapped up in a couple of more days and be ready for testing. Helpers_sim_setup has the most unfinished stuff while helpers_add_interventions is finished, but has comments/questions

there was also a add_vaccdrug_campaigns.py that needed to be converted as it's being used you helpers_add_interventions, it also has some comments/questions.

PR: https://github.com/InstituteforDiseaseModeling/emodpy-snt/pull/21


[HOLD for payment 2024-12-11] [$250] Held expense - Empty message with reply appears after paying the unheld amount for second time

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.63-3
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to staging.new.expensify.com
[User A] Submit two expenses to User B.
[User B] Hold the first expense.
[User B] Click Pay button and pay the unheld amount.
[User A] Submit another expense to User B.
[User B] Go to the expense report.
[User B] Click Pay button and pay the unheld amount.
[User B] Note that the held expense is marked as paid when it should be moved to another report.
[User B] Refresh the expense report.
[User B] Note that there is an empty message with reply which opens the held expense when clicked.

Expected Result:
In Step 8, the held expense should move to another report when only the pending amount is paid.
In Step 10, after refreshing the page, there should not be an empty message in the expense report.
Actual Result:
In Step 8, the held expense does not move to another report when only the pending amount is paid. The expense preview is marked as paid when only the unheld amount is paid.
In Step 10, after refreshing the page, there is an empty message in the expense report, which opens the held expense.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/30edbfda-f5d2-4387-907c-77893225d3d2

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021859730399211188082
Upwork Job ID:<PHONE_NUMBER>211188082
Last Price Increase: 2024-11-21
Automatic offers:

DylanDylann | Reviewer | 105075616

Issue OwnerCurrent Issue Owner: @johncschuster

Proposal
Please re-state the problem that we are trying to solve in this issue.
The hold report action isn't removed from the paid expense chat and becomes empty when revisiting the report.
What is the root cause of that problem?
When we pay the expense report with held expense, it will optimistically remove the held expense and move it to the new optimistic expense report.
https://github.com/Expensify/App/blob/f77087f02a42e17e7da7420528cc51b54003ca61/src/libs/actions/IOU.ts#L6309-L6331
But the issue started to happen when we did it again the second time. It's because when we optimistically add the held expense to a new expense report, we set the pending action to add (as shown above). But we don't clear it when the request is successful. So, the pending action is always true and when we optimistically delete it, it will still be shown because a pending action is always shown.
https://github.com/Expensify/App/blob/f77087f02a42e17e7da7420528cc51b54003ca61/src/libs/ReportActionsUtils.ts#L676-L679
It becomes an empty message when we reopen it because the BE response changes the action to ADDCOMMENT instead of IOU.
What changes do you think we should make in order to solve the problem?

We need to clear the pending action when the request is successful.
const successAddHoldReportActions = {};

holdReportActions.forEach((holdReportAction) => {

const reportActionID = rand64();
addHoldReportActions[reportActionID] = {
    ...
    pendingAction: CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD,
};

successAddHoldReportActions[reportActionID] = {
    pendingAction: null,
};

We might need to do it for the optimistic report preview too because it also adds a pending action when created optimistically.
https://github.com/Expensify/App/blob/f77087f02a42e17e7da7420528cc51b54003ca61/src/libs/actions/IOU.ts#L6295-L6302
We can do the same with other optimistic data that I missed.

@DylanDylann please review the proposal above!

Bump!

@bernhardoj's proposal looks good to me
Note for the PR phase: we need to find out all pendingAction fields and make sure it is reset in successData
🎀 👀 🎀 C+ Reviewed

There is currently a bug where the pay button doesn't show if there are held expenses. It's reported on the issue that causes it here

PR is ready
cc: @DylanDylann

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

Payment Summary:
Contributor: @bernhardoj due $250 via NewDot
Contributor+: @DylanDylann paid $250 via Upwork
Upwork job here! Please apply

Requested in ND.

$250 approved for @bernhardoj

Payment has been issued through Upwork. Thanks, everyone, for your contributions!

Oh wait, I jumped the gun. @DylanDylann, can you please post the regression test steps if applicable?

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎


[Bug]: number of days in month of 2081 is not correct

What happened?
When trying to convert date 2081-02-32 to AD date i am getting error
and when explored the array of number of days in months are mistake
code: Carbon::parse($request->startDate)->toEnglishDate();
How to reproduce the bug
Try to change the date 2081-02-32 to AD
Package Version
1.6
PHP Version
8.2
Laravel Version
10
Which operating systems does with happen with?
Linux
Notes
2081:[[31,31,32,32,31,30,30,30,29,30,30,30]  =>  this is incorrect
it should be
2081 : [31, 32, 31, 32, 31, 30, 30, 30, 29, 30, 29, 31],
please check and update

Hi. The update for date-picker is live. Fixes are on the way for this package too. 😀


"You're not connected" error in installed PWA opened offline

Steps to Reproduce

Visit ngn-apl and install PWA in browser. (In Edge browser, button for installing PWA shows in address toolbar - other browsers have different steps).
Open the installed PWA (by searching for the installed PWA) - it works fine when Wi-Fi is on.
Now close the installed PWA, disable internet, and open the PWA again - this time it doesn't open and instead shows error You're not connected - see screenshot below.   (error code: DNS_PROBE_FINISHED_NO_INTERNET)

Solved by specifying absolute start url in manifest.json : https://sohang3112.github.io/ngn-apl/web/index.html
Previously when relative url was used, error was coming since it was wrongly opening url https://sohang3112.github.io/web/index.html (without /ngn-apl/)


Add Revocation notification support to alice/faber

Codecov Report

Merging #1527 (ce66d2f) into main (6ee2108) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1527   +/-   ##
=======================================
  Coverage   95.73%   95.73%
=======================================
  Files         527      527
  Lines       32375    32375
=======================================
  Hits        30994    30994
  Misses       1381     1381


Add specified Warnings in rm option.

Signed-off-by: Han Zhang<EMAIL_ADDRESS>#3639
Now in docker-machine rm <machine>, it will prompt additional warning about the fact that action will delete instance in cloud provider as well as the local reference, as discussed in the closed issue.

I really don't think my change will break the build on Travis.

LGTM -- thanks


Slack files have inconsistent privacy when calling upload APIs on different platforms

My program running process is as follows:
filesUploadV2 (Params no channel id. Get file id) - > chatPostMessage (Slack image block)
The results of deploying on different platforms are as follows:
It can be sent normally in Windows.
In Linux chatPostMessage returns invalid_blocks
I know I need to call sharedPublicURL to make the file public
But Windows can send messages normally without calling it.
The Slack SDK version
1.40.2
Java Runtime version
1.8

Hi @bytealan, thanks for asking the question! The API client in this SDK never behaves differently depending on the operating system. I'd suggest double-checking how the running code is different. Also, adding debug-level logging to see how the blocks look may help you identify the error pattern.

Hi @seratch,Thanks for your reply!
The Blocks error details are "/blocks/0/accessory/slack_file url/slack_file".I'm sure different platforms use the same code.
By the way, which is the expected result correct?
When I use filesUploadV2 without passing in the channel id
File id obtained
Can I send messages normally when calling chatPostMessage?

Perhaps, your situation is the same with this one: https://github.com/slackapi/python-slack-sdk/issues/1521 Please poll files.info API until the async file uploading process is complete. Once it's done, mime_type etc. will be filled in the file object data in the API response. Then, you can use the file in a block.

Perhaps, your situation is the same with this one: slackapi/python-slack-sdk#1521 Please poll files.info API until the async file uploading process is complete. Once it's done, mime_type etc. will be filled in the file object data in the API response. Then, you can use the file in a block.

Ok, I will try this away. Thanks your answer question again.

It's because of this problem, it has been solved, thank you very much.


Fixed Perspective of Point Cloud

Hi. I've been using this library to visualize my point clouds and I think it is awesome. I work with a lot of video data so up until now I've only been able to visualize one point cloud at a time.  The problem is anytime I do cloud.plot and I'm interactively look at my point cloud, I need to manually zoom in a little bit and translate the image in order to center to the scene of interest. Are there any parameters or things I can do to ensure that my plot is in the exact position and orientation that I want it (before any type of interacting). Another issue will be saving these as pngs so I can eventually paste them together into a video, but one thing at a time.

With #248 coming soon, you could use PyVista for rendering more than one dataset in a scene as well as updating the coordinates and saving a frame.
Maybe check out a few of PyVista's video/gif examples:

Create a GIF
Create a Movie

Simple code to add more than one PyntCloud to a scene (after the changes in #248):
from pyntcloud import PyntCloud
import pyvista as pv

pc1 = PyntCloud.from_file("file1.ply")
pc2 = PyntCloud.from_file("file2.ply")

plotter = pv.Plotter()
plotter.add_mesh(pc1.to_pyvista(), color='red')
plotter.add_mesh(pc1.to_pyvista(), color='blue')
plotter.show()

Feel free to check out the rest of PyVista's Examples and ping me/use PyVista's support forum if you have further question

Closed in #248


Update dependencies

Forgot to push the dropdown text fix before merging this, nice. it's on main now though.


Chat extension log output is not as informative as it used to be

Describe the bug
Until today I was able to gain a reasonable level of insight into how GitHub Copilot/Chat behaves under the hood by inspecting the GitHub Copilot Chat extension log stream. The output now appears stripped of metadata like model chosen, tokens consumed, and so forth.
Because I pay for GitHub Copilot Enterprise, I expect to use GPT-4 pretty extensively if not exclusively. Until today I'd see many, many references to gpt-35-turbo in the Chat extension log stream.
To Reproduce
Steps to reproduce the behavior:

Start a turn-based chat in the Chat pane.
Click View > Output to open the Output panel.
Switch the view to Output.
Open the log selector drop-down list control and select GitHub Copilot Chat from the list.
View the log contents and compare with output from, say, two weeks ago.

Expected behavior
I expected to see more detailed metadata concerning my client connection to the GitHub Copilot API.
Screenshots
N/A
Platform

OS: Windows 11
IDE: VS Code v1.90.1 user setup
Language: N/A
Source: Chat extension v0.16.1; Completion extension v1.204.0

Additional context
I'm not happy paying for GitHub Copilot's highest stock keeping unit (SKU) and using weaker GPT models.

I think you logged this issue against the wrong product. This repo has nothing to do with GitHub Copilot. It's rather about a service to demo the RAG pattern using Semantic Kernel.
In this service, you can choose your LLM model by setting the AzureOpenAIText.Deployment value to whatever you desire.


fix: add or document podAnnotations value

Is this a new chart
No
What this PR does / why we need it:
There was inconsistencies between the charts as to which supported providing custom pod annotations and which didn't. There also wasn't any reference to this value in any of the documentation.
This PR adds podAnnotations as a value to all charts and includes it in the documentation.
Which issue this PR fixes
N/a
Special notes for your reviewer:
As this covers multiple charts I'm not sure how you want me to handle the PR title and versioning etc. so please advise :)
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

I'd like so guidence on how to handle chart versions in this PR.

Marcus, aren't some changes in this PR already present in your other PR's? I'm seeing the same change to serviceAccount annotations here as in #130

Oh you're right. I must have forked from the wrong branch. Let me clean that up.

Fixed.
Please advise on what you'd like the chart versions set to.

We usually bump the Minor chart version when we introduce some change, so for each of the charts you've changed, bump it on the Minor version (major.Minor.patch)

Moreover we should have One PR per chart in order to make easier, reviewing and merging

Can you add in the title of the pr the chart name between [..]?

@paologallinaharbur I've split this into multiple PRs:

https://github.com/newrelic/helm-charts/pull/217
https://github.com/newrelic/helm-charts/pull/218
https://github.com/newrelic/helm-charts/pull/219
https://github.com/newrelic/helm-charts/pull/220
https://github.com/newrelic/helm-charts/pull/221