Bagit implementaion

[ ] Generate file checksums
[ ] Meta data files

Talk to Radar4Chem team about how their implementation works.

Potential package - https://github.com/whikloj/BagItTools
Another potential solution to consider is RO-CRATE - https://www.researchobject.org/ro-crate/


Integrate CAISO OASIS Fuel Price Data for the WECC

Feature Request
Higher Quality daily fuel price data for each BA in the WECC lives here: http://oasis.caiso.com/mrioasis/logon.do
Suggested Solution

Potential option to write script that calls OASIS API
Use GridStatus, but this requires API key/account: https://www.gridstatus.io/datasets/caiso_fuel_prices
Store some historical data in zenodo

Additional Info
No response

We are starting to pull in costs from all over the place (which I am not against!), but it may be good to break it off into its own module for testing/clarity purposes going forward. Rather than having everything in the add_electricity module.

Noted- Maybe we can start a build_plant_data module that includes:

adding temporal fuel costs (EIA / CAISO / other ISOs)
adding ramp rates / heat rates  EIA

What are your thoughts on the other functions that would be removed from add_electricity?

Okay i just saw your other bug for add electricity. ill move this conversation there.


Sealed Class Support

Hey everyone!
Considering JDK17 is a LTS version and the sealed class feature being production ready with its release, are there any plans regarding using this feature for defining parametric algebraic data types? Being able to use switch expressions to pattern match would be great for developer experience.
Not sure if using JDK17 would be out of scope for now or not, hence the issue.

Hi! Thank you for asking - yes, indeed, I plan to migrate to JDK17.

I'd love to help out with this!

Thank you! Please outline your idea here first, e.g.

which Vavr types do we want to cover (Either, Option, Try and Validation but not Future, Promise and collections?)
which actions are needed to change the codebase (use interfaces instead of abstract classes? make static inner classes public and move them into the package?)
depend on Java 17
any other things?

Will do! Taking a closer look now, was waiting to clarify whether JDK17 is desired/planned.
Indeed, I was thinking about changes for Either, Option, Try and Validation. Will come back soon with a more detailed outline in a bit.

Great to hear! Thank you!

Hey @GeluOltean, do you have any updates on this?

Hey @bduisenov; got a bit overwhelmed by a previous project and did not manage to make much progress on this. It should be possible, but I do not have an idea yet of how extensive the changes would be.

@bduisenov @danieldietrich I'd be happy to take a crack at this, as I'm quite happy about Java introducing Pattern Matching and am eager for this.
The plan would be quite simple I think.
Candidates:

Either, Option, and Try definitely would get the change. Validation as well, if that is still being kept?
Future/Promise I can still seal so they are closed for extension. They won't be relevant to Pattern Matching since there is only one sensible implementation, but I personally don't see a need to extend them or leave them open to extension either. Would defer to your thoughts/opinion/guidance on this, however.
Collections like List, Stream, Tree I would also be happy to include, though personally I won't find them as useful until Java would fully implement general deconstruction
Nothing else, all other classes are final

Approach:

I would plan to keep the abstract classes. In 0.10.4 which I am used to working with, these are still interfaces, so I assume you had reasoning behind the move to abstract classes and I see no reason to roll that back.
All of these already have static final inner classes for the implementations, and I would keep it that way.

Personally, I would probably normally implement a sum type like this, with inner classes & same file
'Same file' matches also the way that Scala does sealed
This makes the hierarchy/grouping clear and prevents conflicts between List.Cons/Stream.Cons and Stream.Empty/Tree.Empty

The top-level abstract classes would simply become public abstract sealed, and the concrete classes are already public static final.

Stream is a special case with slightly different hierarchy, since Stream.Cons is abstract and implemented by two private static final inner classes. So I would seal Stream.Cons as well, while keeping the implementations private (I would say nobody needs to know what kind of Cons it is, no?)

Update to Java 17

Bump version in build.gradle
Change Java version in Travis CI config, drop all previous versions since this is a breaking change
Update documentation referencing things like 'Vavr is for Java 8+'

The only real issue I see here (from my understanding) is the fact that #2346 will be blocking this due to the Java version update.


add support for UDP multicast

currently I've only added the LibC flags for Darwin but thought I'd get some feedback before proceeding further.
fixes: #3825

Awesome! I'll see about what needs to be done for Linux...

ping @ysbaddaden what do you think?
@RX14 the Windows implementation sockets, the stuff in netinet/in.cr for BSD and linux, is almost identical to the BSD implementation, two minor differences. Was thinking of adding a winsock2.cr file with that in it, but not sure if it will compile / don't want to break anything you are working on.
Definitions for things like SaFamilyT are probably missing on Windows.
Did you want me to include it?

Thanks @Sija I've fixed those issues with the comments

@stakach Great! I'd also put all integer values inside backticks, since they ought to be treated literally.

@ysbaddaden I've made those changes.
I believe the bindings are correct, what's wrong with the OpenBSD bindings?
I was looking in

https://github.com/openbsd/src/blob/master/sys/netinet/in.h
https://man.openbsd.org/ip6#IPV6_JOIN_GROUP

I can't really review. I don't know anything about multicast —for example I have no idea what "hops" is.
I looked at how Ruby handles that, but it doesn't have any high-level interface for multicast in stdlib, so it doesn't help either :(

This is really cool to see in Crystal.  Thanks for working on this.

Great feedback @straight-shoota - I've implemented all those changes.

@straight-shoota looking good with the IPv6 tests in place.

@straight-shoota so not sure how to resolve the OSX tests for IPv6.
I dev on OSX so can confirm that setting the default multicast interface on Darwin 18.2.0 does not work as described in this bug: https://lists.apple.com/archives/darwin-kernel/2014/Mar/msg00012.html
However I can join the group successfully on my laptop (with IPv6) using crystal so I assume it's either an older version running on circle ci or possibly a permissions thing? not sure..
I see there is a similar issue with the it "sends broadcast message" do test for darwin sockets, so I think it's probably related.

Yeah, the CI uses xcode version 9.0 which runs on macOS 10.12.6 / Darwin 16.7.0. The bug has probably been fixed somewhere in between. We could consider simply updating the CI, but I'm not sure whether that's possible or desired. We aim to support macOS 10.7+ and the specs should work on all supported versions.
So for now, it's probably okay to simply disable the spec for #multicast_interface  with IPv6 on darwin. But the rest of the spec should run on this platform!
I'd suggest to structure the tests like this, possibly even separating the individual methods (loopback, hops, join/leave) into separate specs.
each_ip_family do
  # ...
  it "can join and transmit to multicast groups" do
    # setup
    # multicast_loopback
    # multicast_hops
    # {join,leave}_group + send/receive
  end
end
describe "#multicast_interface" do
  it "on IPv4" do
    # setup ipv4
    udp.multicast_interface Socket::IPAddress.new("<IP_ADDRESS>", 0)
    # send/receive test
  end
  {% unless flag?(:darwin) %}
    it "on IPv6" do
      # setup ipv6
      udp.multicast_interface 0
      # send/receive test
    end
  {% end %}
end

We could consider activating the darwin spec based on the actual OS version. But for this it would be great to know when this was fixed.
leave_group is currently also missing from the specs, please add some expectation for that. Should also make sure that send/receive doesn't work after leaving.

@straight-shoota all specs are passing for all platforms except Darwin IPv6 which displays as pending now.

I've just added the test for group leave.
multicast_interface, group_join and group_leave IPv6 don't work on the CI version of Darwin unfortunately.
I think the fix for group_join and group_leave was introduced in late 2017: https://github.com/apple/darwin-xnu/commit/0a798f6738bc1db01281fc08ae024145e84df927#diff-e2c4bcf26a9843a55acac3eb88326083

@straight-shoota what do you think? Ready for merge?
Would love to see this in the next release

Please separate the specs and make sure that the working specs run on darwin as suggested in my previous comment.
I think I'd also prefer to run all specs on recent darwin versions. This would just need to be a dynamic check based on sw_vers combined with flag?(:darwin). This would be the best solution, even without exactly pinpointing when it was fixed.

@straight-shoota was just looking at this again.
Having no luck using sw_vers to determine if this version of OSX supports supports what. Have a feeling this issue lies not in the OS level code, but in the user level libraries supplied by XCode.
So I was thinking:

Would it be OK if the code in the current state is accepted?

One test is pending: IPv6 multicast on Darwin

I open a new pull request to implement getifaddrs

This will allow me to implement the missing behaviour of Darwin manually
I can fix IPv6 on OSX as part of that pull request - otherwise I feel like it's going to make this pull request very messy

Yes of course, it's fine to simply skip the spec on darwin for now if a dynamic condition is too complicated.`
But as far as I understand it, only #multicast_interface fails, so the other specs should be executed on darwin with IPv6. A structure as proposed in https://github.com/crystal-lang/crystal/pull/7423#issuecomment-470063033 would be ideal for this.

@straight-shoota it works on the latest version of OSX with what are probably the latest XCode libraries?
This is a minimal reproduction that can you can use to test on your laptop if you have OSX
(this works for me - but not on CircleCI)
require "socket"

struct Socket::IPAddress
  def addr6
    @addr6
  end
end

lib LibC
struct Ipv6Mreq
    ipv6mr_multiaddr : LibC::In6Addr
    ipv6mr_interface : LibC::UInt
  end
end

s = UDPSocket.new Socket::Family::INET6
s.bind "::", 0
# multicast_interface selection
#s.setsockopt 9, 1, 41
#s.getsockopt 9, 41
sock = Socket::IPAddress.new("ff02::102", 2000)
addr = sock.addr6.not_nil!

req = LibC::Ipv6Mreq.new
req.ipv6mr_multiaddr = addr
req.ipv6mr_interface = 0
# Join IPv6 group
s.setsockopt 12, req, 41
Fiber.yield
s.send("testing", sock)
s.receive[0]

Thanks, I'll rebase.

@straight-shoota I didn't split the tests as none of the IPv6 multicast functions work on CircleCI version of Darwin or the build tools running on it.
I added this check, which should have worked based on my setup (you can confirm if you have OSX with the code I posted above)
I'll implement getifaddrs which should allow us to use either IP address or index for both IPv4 and v6 multicast_interface selection as well as working around the limitations of IPv6 on darwin.
Given not many people will be using multicast_interface selection and IPv6 multicast works on the latest versions of OSX (even if it's not being tested currently) I think this pull request is good to go

@straight-shoota what do we think?

@straight-shoota I've implemented all those requested changes. Thank you for your patience btw.

@stakach it's actually your patience that we must thank, others would have probably abandoned this PR already :-)

need me to do anything more for this pull to be merged?

Would be great if you could rebase against current master, to make sure it still works fine with 0.28.0-dev.

@straight-shoota looking good :)

@ysbaddaden @asterite would one of you be available to review this pull?

The spec error for linux32 seems unrelated to this pull

@asterite I think this should resolve your concerns.

Merge 🕦?

Thanks @stakach and all reviewers involved!


User mover should export and reimport cdb_tablemetadata contents and analysis cache tables

It already does after https://github.com/CartoDB/cartodb-postgresql/pull/285 release


Not using prepared statements

This code is using deprecated functions for MySQL queries. You should be using the PDO driver with prepared statements. If you would like I could convert the code over and make a pull request.

Sure, it'd be appreciated.

cool I went ahead and made a pull request. Everything has been converted to PDO drivers which will be compatible with PHP7.


Diy extension

🛠 Fixes Issue
Closes #372
👨💻 Changes proposed
I have made the extension
.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
@Akshima-Ghai  @ridsuteri  @praniti111  Please review it.
📷 Screenshots/Videos
https://user-images.githubusercontent.com/93239528/170620235-0c970603-7c47-46f5-afa7-f275490bc080.mp4

@Sukriti-m You have added a country lang extension in this PR, please remove that from this PR.

I have created a new PR #405 with the changes. Please review.


docs: notify windows users about potential bugs with backslashes

Fixes #7431

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[ ] Multiple commits are squashed into one commit.
[ ] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[ ] All npm dependencies are pinned.

/lgtm


Create vegaschool.txt

Vega school website:
https://www.vegaschool.com/
Vega Game Design and Development degree:
https://www.vegaschool.com/full-time/bachelor-of-computer-and-information-science-in-game-design-and-development-degree

@henk-scheepers Pull request merged. Thank you!


problem with module caching when used in _app and other pages

Describe the bug
I am having a problem when using packages that have module caching and keep track of internal state in server rendering. An example of such packages is redux-act where module caching is used to keep track of an id used to serialize action types. The bug happens when importing my store.js file with all the actions, reducers and the store in _app, and importing it in other pages as well. This causes my store.js file to be bundled with both _app and any other page I am rendering like index. This causes the createAction to be called twice to the same action which causes the internal redux-act ids to be out of sync and thus the reducer never handles the add action
To Reproduce
this reproduce code is passed on the with-redux-wrapper example.

clone https://github.com/Ilaiwi/next-_app-bug-example
run npm run dev
go to localhost:3000

Expected behavior
the AddCount should be 1 because an add action was dispatched in getInitialProps
System information

OS: [Windows 10 and osx]
Version of Next.js: [e.g. 6.0.3]

Additional context
what I did in the example is I just added redux-act and replaced the actions with redux-act actions and deleted the thunk AddCount action.

Duplicate of #4194

@timneutkens will follow there thanks.
@abusada I tried capitalizing the types but redux-act will through an error TypeError: Duplicate action type: ADD which is logical since createAction("ADD") was called twice with same type


change makefile test target for no longer be required manual steps to run the commands

Description of the change:
Update the Makefile test target as discussed in the demo.
Motivation for the change:
Closes: #3692
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[x] Add or update relevant sections of the docs website in website/content/en/docs

Local Tests
 $ operator-sdk init
Writing scaffold for you to edit...

And to check that is running:
$ make test
/Users/camilamacedo/go/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..."
go fmt ./...
go vet ./...
/Users/camilamacedo/go/bin/controller-gen "crd:trivialVersions=true" rbac:roleName=manager-role webhook paths="./..." output:crd:artifacts:config=config/crd/bases
mkdir -p /Users/camilamacedo/go/src/testmake/testbin
test -f /Users/camilamacedo/go/src/testmake/testbin/setup-envtest.sh || curl -sSLo /Users/camilamacedo/go/src/testmake/testbin/setup-envtest.sh https://raw.githubusercontent.com/kubernetes-sigs/controller-runtime/master/hack/setup-envtest.sh
source /Users/camilamacedo/go/src/testmake/testbin/setup-envtest.sh; fetch_envtest_tools /Users/camilamacedo/go/src/testmake/testbin; setup_envtest_env /Users/camilamacedo/go/src/testmake/testbin; go test ./... -coverprofile cover.out
fetching envtest<EMAIL_ADDRESS>(into '/Users/camilamacedo/go/src/testmake/testbin')
x bin/
x bin/etcd
x bin/kubectl
x bin/kube-apiserver
setting up env vars
?   	testmake	[no test files]
camilamacedo@Camilas-MacBook-Pro ~/go/src/testmake (master) $

Hi @joelanford,
Tks for your review the changes are made. 👍

@estroz Do you think there's a path for us to make the default scaffolding work with the Kubebuilder v2 plugin?
I think the problem we'll find with the v3-alpha plugin is that no one will use it until it is the default.

Yes, since this is a non-breaking change I think https://github.com/kubernetes-sigs/kubebuilder/pull/1626 can be backported to the v2 plugin. @camilamacedo86 can you create a PR for this upstream?

Hi @estroz,
I agree that in this case, we might able to push to v2+ as well. Could you please approve the PR: https://github.com/kubernetes-sigs/kubebuilder/pull/1711 for we are able to bump the kb commit here?

Based on the discussion in the bug triage meeting could you please let us know if we can get it to merge or fi has anything else that you would like o change before that?

can confirm that this works, but I agree with @estroz that it should be in its own file, and then we can just delete it when the upstream fix gets merged

/lgtm

@joelanford @estroz,
All suggestions are addressed.

The Ansible test is failing. RUN ansible-galaxy collection install -r ${HOME}/requirements.yml is timing out. This times out for me locally, and others are seeing this as well. Since Ansible galaxy appears to be down, please do not restart that job for a while.


GIF image type support in PDF mode?

Is your feature request related to a problem? Please describe.
Sometimes, some chapters are in GIF format and are skipped when downloading. So some books end up without a single content inside.
Describe the solution you'd like
Support GIF format for PDF export.
Describe alternatives you've considered
CBZ format doesn't have this problem, but the page content is not right.
Additional context
OS: Ubuntu 24.04.1
Go: Go 1.22.2
Example URL: https://www.webtoons.com/en/slice-of-life/murrz/ep-226-devil-flowers/viewer?title_no=1281&episode_no=226
CBZ glitch

Thanks for this feature request @hantsaniala! Would you want to have a go at adding the feature yourself and making a pull request to get it in the next release? I'm a bit tied up with work and other projects.
If you want a hand getting set up for making a code change in this repo, I can add a CONTRIBUTING file with further guidance, LMK!

I was about to make a PR for the PDF but stuck with some logic that I'm little bit lost. import _ "image/gif" fix the unknown type error but I still have blank page problem using c.pdf.ImageByHolder(). Any hint to how to fix this? I'll PR if I found a solution.

If you open a draft PR with your attempt so far, I can take a look at it when I get some time!

Here please. I'm still looking for solution but if you can hint me where to go, I can fix it.

The pdf library gopdf doesn't currently support gif only jpg & png, I have a pull request for gopdf to add gif support that is the result of prototyping gif support for this repo.
If you want to add gif support before gopdf adds gif support, you'll want to add logic to convert gifs to pngs using image.Decode(...) & png.Encode(...).

gopdf v0.29.0 includes gif support, if you update to version or later the changes in the WIP: Feature/pdf gif support should work. I have it working in my own gif-support branch that includes similar changes


[HUDI-1988] FinalizeWrite() been executed twice in AbstractHoodieWrit…

…eClient$commitstats
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
FinalizeWrite() been executed twice in AbstractHoodieWriteClient$commitstats. once in commitstats() self, another in commit() method in commitstats(). I think this is a useless repetition.
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3050 (1f85ab3) into master (f760ec5) will decrease coverage by 46.83%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3050       +/-   ##
============================================
- Coverage     55.31%   8.48%   -46.84%
+ Complexity     4026      62     -3964
============================================
  Files           520      69      -451
  Lines         25295    2863    -22432
  Branches       2872     358     -2514
============================================
- Hits          13993     243    -13750
+ Misses         9914    2599     -7315
+ Partials       1388      21     -1367

Flag
Coverage Δ

hudicli
?

hudiclient
?

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
6.79% <ø> (-44.66%)
:arrow_down:

huditimelineservice
?

hudiutilities
9.17% <ø> (-61.90%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

 I see you have added one of the finalize as part of multi-writer. not sure if it was moved from elsewhere.

@swuferhong Thanks for opening this. Looks like it might be a duplicate. Can you check why the CI is failing ?

@swuferhong Thanks for opening this. Looks like it might be a duplicate. Can you check why the CI is failing ?

Yes, we reopened this PR, and CLI passing now.

@n3nash Thanks, we reopened this PR, and CLI passing now.


Transform all GPU symbols to become unique

We want to achieve unique symbols in all 4 ectrans flavours (combinations of  gpu/cpu with sp/dp).
This paves the way to create a future ectrans API that can dispatch to these 4 libraries at runtime.
For the CPU code (cpu + sp/dp) the unique symbols were already achieved in #61.
The recently merged GPU code was not yet done, and this PR tackles this and follows the same approach.
The GPU symbols now have a suffix _gpu_sp for single precision and _gpu_dp for double precision
The library with these is ectrans_gpu_sp and ectrans_gpu_dp respectively and does not rely on any parkind1 module.
It is therefore now possible to link to all of these together.

ectrans_sp (cpu)
ectrans_dp (cpu)
ectrans_gpu_sp
ectrans_gpu_dp

For backward compatibility there are interface libraries named as before trans_gpu_sp and trans_gpu_dp which give access to the previous API without the suffixes via preprocessor definitions. This is what currently allows ectrans-benchmark.F90 to be generic. Naturally you cannot link to these simultaneously, which is why there are the 4 separately compiled flavours of ectrans-benchmark necessary.
graph BT;
    ectrans_gpu_sp-->ectrans_gpu_common;
    ectrans_gpu_dp-->ectrans_gpu_common;
    ectrans_gpu_common-->ectrans_common;
    ectrans_sp-->ectrans_common;
    ectrans_dp-->ectrans_common;

The above graph shows interdependence between libraries where the _common libraries contain only precision independent code.

ectrans_common library contains domain decomposition routines as well as setup_trans0! A new test has been added to test setup_trans0 which only links with ectrans_common.
ectrans_gpu_common contains GPU specific routines (e.g. ACC, allocators, hip/cuda abstractions)

Nice work! I'll have a read through. Might take some time...

external/ini_spec_dist.F90 could also be moved to common. The imported JPRB in this file is not actually used, and the subroutine it calls SUWAVEDI is already in common.

external/ini_spec_dist.F90 could also be moved to common. The imported JPRB in this file is not actually used, and the subroutine it calls SUWAVEDI is already in common.

Now done

Changes to HICFFT will have to be revisited and will probably be much simpler after first having merged #143

Changes to HICFFT will have to be revisited and will probably be much simpler after first having merged #143

This is now done. I deleted more unused HICFFT code in develop branch and then rebased this branch.

I think the following files can be added to the common library:

pre_suleg_mod.F90
setup_dims_mod.F90
setup_dims_mod.F90
setup_geom_mod.F90
shuffle_mod.F90
sutrle_mod.F90

Here's a test program which attempts to call SETUP_TRANS for CPU and GPU:
PROGRAM ECTRANS_DEMONSTRATOR

USE EC_PARKIND, ONLY: JPIM

IMPLICIT NONE

#include "setup_trans0.h"
#include "setup_trans_sp.h"
#include "setup_trans_gpu_sp.h"

INTEGER(JPIM), PARAMETER :: TRUNC = 79

CALL SETUP_TRANS0(LDMPOFF=.TRUE., KMAX_RESOL=2, KPRINTLEV=2)

CALL SETUP_TRANS_SP(KSMAX=TRUNC, KDGL=2 * (TRUNC + 1)) ! CPU
CALL SETUP_TRANS_GPU_SP(KSMAX=TRUNC, KDGL=2 * (TRUNC + 1)) ! GPU

END PROGRAM ECTRANS_DEMONSTRATOR

with Makefile
FC = nvfortran

ECTRANS_BUILD = /ec/res4/hpcperm/nash/ectrans/build
FIAT_BUILD = /ec/res4/hpcperm/nash/fiat/build

ectrans_demonstrator: ectrans_demonstrator.o
    $(FC) $^ -o ectrans_demonstrator -Wl,-rpath,$(ECTRANS_BUILD)/lib \
    -Wl,-rpath,$(FIAT_BUILD)/lib $(ECTRANS_BUILD)/lib/libectrans_common.so \
    $(ECTRANS_BUILD)/lib/libectrans_sp.so $(ECTRANS_BUILD)/lib/libectrans_gpu_sp.so \
    $(FIAT_BUILD)/lib/libfiat.so

ectrans_demonstrator.o: ectrans_demonstrator.F90

%.o: %.F90
    $(FC) -I$(ECTRANS_BUILD)/include/ectrans \
    -module $(FIAT_BUILD)/module/fiat -c $< -o $@

I don't think we're quite ready for this kind of program. When I run this I get a segfault in SETUP_TRANS_GPU_SP on this line.
Why this line and not the ones above? G and D are defined in TPM_GEOMETRY and TPM_DISTR, respectively, which are both in the common library. They are pointers to arrays which are allocated and visible in this scope. S however is defined in TPM_FLT which is CPU/GPU-specific. It's a pointer to an element of FLT_RESOL. FLT_RESOL in TPM_FLT_SP is allocated (during the first call to SETUP_TRANS) but FLT_RESOL in TPM_FLT_GPU_SP is still unallocated at this point. Hence the segfault.
We can perhaps solve this one by moving TPM_FLT to common, but I wonder if there are other potential conflicts?

I think we're still a long way to be able to create multiple handles that belong to different backends, even though that is our ultimate aim.
At least it should be possible to have a runtime switch to call either one of them, and stick to the same backend for the rest of the run.

I think the following files can be added to the common library:

pre_suleg_mod.F90
setup_dims_mod.F90
setup_geom_mod.F90
shuffle_mod.F90
sutrle_mod.F90

From all the above, only shuffle_mod.F90 can be moved to common (now done). The others are using either TPM_FIELDS or "TPM_FLT" which contain JPRB data.
Refactoring this code would be great, and possibly a lot can be made common.
e.g. computing and storing the Legendre coefficients only in double precision, and be reused by multiple backends of same resolution too!

I think the following files can be added to the common library:

pre_suleg_mod.F90
setup_dims_mod.F90
setup_geom_mod.F90
shuffle_mod.F90
sutrle_mod.F90

From all the above, only shuffle_mod.F90 can be moved to common (now done). The others are using either TPM_FIELDS or TPM_FLT which contain JPRB data. Refactoring this code would be great, and possibly a lot can be made common. e.g. computing and storing the Legendre coefficients only in double precision, and be reused by multiple backends of same resolution too!

Ah yes I see. I will think about how we can move more TPM_ modules to common.

Now rebased and amended relevant commits with the fixups.
I will merge when @marsdeno agrees.


Add packages for everyday usage

Add the full complement of scientific python packages I use on a fairly regular basis.
As per @yuvipanda's comment, I went through the various environment files in images I use, and I tried to capture all the packages I actually use in practice on a reasonably regular basis, leaving out more obscure things that might end up going stale or not actually be used.
All the packages here are things I've used in some capacity for either research or teaching either at Berkeley or in the JMTE hub in the last couple of years.

BTW, I ran a quick check of versions and imports off the binder link, and all so far looks good:
Python info on this system:

3.10.8 | packaged by conda-forge | (main, Nov 22 2022, 08:23:14) [GCC 10.4.0]

Current package versions

IPython                  : 8.6.0
PIL                      : 9.2.0
bokeh                    : 3.0.2
bs4                      : 4.11.1
cartopy                  : 0.21.0
cython                   : 0.29.32
dask                     : 2022.11.0
geopandas                : 0.12.1
h5py                     : 3.7.0
intake                   : 0.6.6
ipyleaflet               : 0.17.2
ipympl                   : 0.9.2
ipywidgets               : 7.7.2
jedi                     : 0.18.1
jupyter_book             : 0.13.1
jupyterlab               : 3.5.0
jupyterlab_favorites     : 3.1.0
jupyterlab_git           : 0.39.3
jupyterlab_widgets       : 1.1.1
lxml                     : 4.9.1
matplotlib               : 3.6.2
matplotlib_inline        : 0.1.6
myst_nb                  : 0.13.2
myst_parser              : 0.15.2
networkx                 : 2.8.8
numba                    : 0.56.4
numpy                    : 1.23.5
pandas                   : 1.5.2
pep8                     : 1.7.1
plotly                   : 5.11.0
pooch                    : v1.6.0
pyflakes                 : 2.5.0
pytest                   : 7.2.0
pytest_cov               : 4.0.0
requests                 : 2.28.1
scipy                    : 1.9.3
seaborn                  : 0.12.1
skimage                  : 0.19.3
sklearn                  : 1.1.3
sphinx                   : 4.5.0
sphinx_jupyterbook_latex : 0.4.7
statsmodels              : 0.13.5
sympy                    : 1.11.1
xarray                   : 2022.11.0
yaml                     : 6.0
zarr                     : 2.13.3

Thanks!

Thank you so much, @fperez! We should find ways to automate this even further, and with more people being hired at 2i2c I'm hopeful that can happen :)

Awesome, thanks so much! It took me a bit to figure out how to get the script to work in-place in the yaml structure while preserving structure (ruyaml is essentially undocumented in this aspect, and no methods have docstrings, so it's kind of hunting in the dark). But the script now works well for this purpose, so we can eventually put it in a more automated pipeline if needed.
BTW - building the env on the hub took forever. I timed it on my laptop where it took 1 min 25s, while on the hub it took easily ~ 20 minutes (I didn't time it precisely). I know those filesystem-intensive operations are slow, but this seemed unusually slow. Just figured I'd let you know.

@fperez yeah, that's sort of the performance I'd expect on NFS honestly! Another reason I recommend against putting user environments on NFS... :D

I hear you @yuvipanda! TBH I was (naïvely) expecting the gap to be a bit less brutal. In the cloud, it really seems to be horrid (on a local, "classic" linux cluster the penalties are manageable).
In any case - given this, do you have any other suggestions for users to experiment when they need to build and keep around an env to use? No matter what we do with the base env, users will always need to experiment with new environments, they'll have a few one-off packages they need to build, etc.
What is being recommended to other 2i2c users in general?


Example missing for mounting storage account over private link

Hi,
We are trying to setup a private link by using private endpoint to access storage account into different tenant and subscription from DataBricks notebooks.
When we try to mount the storage account over the private link it fails and complains about authorisation error.
Can you please suggest what should be fqdn for key url when using the private link?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 58d39e17-424d-1db6-f600-15f272bf3a7c
Version Independent ID: 653ad346-6d87-d5fd-43dd-1498d218f145
Content: Azure Blob storage - Azure Databricks
Content Source: databricks/data/data-sources/azure/azure-storage.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@sujitks
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

Hi @sujitks! Can you please share the error you are getting?

Hi @mamccrea
We were trying to mount the storage account's blob container over the private link by using the dbutils.fs.mount method. It was giving us following error:
shaded.databricks.org.apache.hadoop.fs.azure.AzureException: com.microsoft.azure.storage.StorageException: This request is not authorized to perform this operation.
ExecutionError                            Traceback (most recent call last)
 in
----> 2 mount_storage_folder("containerName", spark.conf.get("storageAccountName"), "===========storage account key   ==")
 in mount_storage_folder(folder_name, storage_name, storage_key)
21   source = storageUrl,
22   mount_point = mount_path,
---> 23   extra_configs = {storageKeyUrl : storage_key})
24   print("Folder Mounted:" + mount_path)
25
/local_disk0/tmp/1593526052854-0/dbutils.py in f_with_exception_handling(*args, **kwargs)
312                     exc.context = None
313                     exc.cause = None
--> 314                     raise exc
315             return f_with_exception_handling
We thought that just deployment of a private endpoint in the data brick's vNET should be enough, later on we deployed private dns zone, and next day when tried same code from data bricks it worked.

#reassign:mssaperla

I am trying to mount ADLS private endpoint as well but not able to figure out what the conf-key should be in the extra_configs parameter for the same. Examples for the same would be great. Thanks

Hi,
Just to let you know that my problem with mounting the storage account over the private link was resolved by creating the DNS entries to point to private IP address of the private endpoint.

We have filed a ticket in the Databricks bug tracking system: DOC-2978
We will update this Git issue when the ticket has been resolved. #label:"in-progress".

@joydeepml you can try this?
folder_name =YOUR_BLOB_CONTAINER
storage_name=STORAGE_ACCOUNT_NAME
storage_key=YOUR_STORAGE_KEY
storageUrl = "wasbs://" + folder_name + "@" + storage_name + ".blob.core.windows.net"
storageKeyUrl = "fs.azure.account.key." + storage_name + ".blob.core.windows.net"
dbutils.fs.mount(
source = storageUrl,
mount_point = mount_path,
extra_configs = {storageKeyUrl : storage_key})

Thanks for your dedication to our documentation. Unfortunately, we have been unable to address your issue and apologize for the delayed response. We are closing this issue, but if you feel that it's still a concern, please let us know directly at<EMAIL_ADDRESS>#please-close

#please-close


Compare and contrast with sqlite

Is it faster better or just different?

DuckDB is designed for analytical workloads, i.e. big queries that touch a lot of data at once, whereas SQLite is designed for transactional workloads, i.e. small queries that update or scan individual rows. DuckDB is faster than SQLite at analytical workloads, but likely slower for OLTP workloads, although we have not tested that in-depth.

Also, see https://dl.acm.org/citation.cfm?id=3299869.3320212
(Mark, you are too modest - advertise your work :-))


[SPARK-45213][SQL] Assign name to the error _LEGACY_ERROR_TEMP_2151

What changes were proposed in this pull request?
Assign the name EXPRESSION_DECODING_FAILED to the legacy error class _LEGACY_ERROR_TEMP_2151.
Why are the changes needed?
To assign proper name as a part of activity in SPARK-37935.
Does this PR introduce any user-facing change?
Yes, the error message will include the error class name
How was this patch tested?
An existing unit test to produce the error from user code.
Was this patch authored or co-authored using generative AI tooling?
No.

ping @MaxGekk to have a look.

Hi @MaxGekk , sorry for my late, I have fixed all the test failures, PTAL again.

The docker test should not be related to the last commit.
+1, LGTM. Merging to master.
Thank you, @dengziming.

Thanks @MaxGekk !


Add describe to protocol tests

This wraps airnode-protocol tests with a describe to improve test times (based on this) It also removes @Siegrift's custom test script from the package as it is no longer needed.
Here are the test times I got locally:

without describe: 335.96s
test-script: 57.41s
with describe: 41.90s

Wow that is a seriously impressive speed up. I'm not sure about converting the function() {} blocks to arrow functions though... Mocha discourages this

Good to know, I wasn't aware. It doesn't look like this is used for now, but perhaps I should convert back in case it is needed?

perhaps I should convert back in case it is needed

Sounds like a good idea to me

Yeah I think just sticking with function() {} everywhere is just more consistent and it makes it less likely that future devs won't run into hard to diagnose issues


Implement DeFi application

A matching engine (similar to Nasdaq) with orders being submitted can be introduced using Views.
Needed key ingredients:

A map of bids/asks ordered by their price.
A Queue that can be mutated and edited (to indicate the filled orders)
The security of the system.

This has been addressed (at least partially) in https://github.com/linera-io/linera-protocol/pull/662


Redo documentation fixes for expiresIn vs expirationtime

Fixes #137
This fix was previously contributed in #138 but was merged into master (not the primary branch of this repository). Master was merged into dev (primary branch) without the CLA check passing in #153 and was reverted in #154. Since it was a merge and not a squash and merge, the fixed commits are already in the git history and cannot be cherry-picked in and we cannot revert the revert because of the CLA failure. This this is a new diff with the same contents as #138

Create #187


"bash: moneyd: command not found"

Have a "[root@codius1 ~]# moneyd xrp:info" --> "bash: moneyd: command not found" after update.

Hey , sometimes after upgrade via NPM ,  the command cannot be found in $PATH , so just logout from the SSH and loging again or reboot server to fix this .

already did it.
dont work.

try to install manually and test again :
`sudo npm install -g moneyd --unsafe-perm´
Let me know output if you saw an error and send the out put of this commands :
whereis moneyd
which moneyd -a
sudo find / -name moneyd

yarn works, npm doesn't.


mupx - "docker: invalid hostname format for --hostname" while verifying deployment

[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Uploading bundle
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Uploading bundle: SUCCESS
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Sending environment variables
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Sending environment variables: SUCCESS
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Initializing start script
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Initializing start script: SUCCESS
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] - Invoking deployment process
[ec2-52-51-239-208.eu-west-1.compute.amazonaws.com] x Invoking deployment process: FAILED
-----------------------------------STDERR-----------------------------------
Error response from daemon: No such container: gefen_cms
Error response from daemon: No such container: gefen_cms-frontend
docker: invalid hostname format for --hostname: ip-172-31-19-146-gefen_cms.
See 'docker run --help'.
-----------------------------------STDOUT-----------------------------------
base: Pulling from meteorhacks/meteord
4d690fa98655: Already exists
a3ed95caeb02: Already exists
0638f01c6191: Already exists
086f3bb36259: Already exists
Digest: sha256:684021facd5847f6a4ee2f0095fd26b4d7f5af2b776150531914d43fd0eb889b
Status: Image is up to date for meteorhacks/meteord:base
----------------------------------------------------------------------------

This is not a bug; it looks like you specified an invalid hostname. Docker validates if hostnames are RFC 1123 compliant (see https://tools.ietf.org/html/rfc1123). https://github.com/docker/docker/pull/20566
Removing the underscore in your hostname, and replacing it with a hyphen (ip-172-31-19-146-gefen-cms) should make this work.
I'll close this issue, because this is not a bug in Docker, but feel free to comment after I closed


Class property does not map to Column Header with trailing space

I have a class with property "AVG_BAL", in the excel file the header name is "AVG_BAL " with a trailing space.
AVG_BAL property on object result is always null. I do no have control over the file therefore I cannot change the column name.
Is there a way for LinqToExcel to ignore any leading or trailing spaces?

I had this same issue... source software exports Excel with trailing spaces, but if a user saves from excel, no trailing spaces.
Resolved with this:

            var worksheetName = excel.GetWorksheetNames().FirstOrDefault();

            if(worksheetName != null)
            {

                //For Some columns, GrainSmart exports them with a column name that has a trailing space.
                //If these are opened in Excel and then saved, the trailing space is removed
                //Ensure the mappings account for both instances.
                foreach(var colName in excel.GetColumnNames(worksheetName))
                {
                    switch (colName)
                    {
                        case "Vehicle Id ":
                            excel.AddMapping<Ticket>(t => t.VehicleId, "Vehicle Id ", d => d.ToString());//column in excel file has trailing space
                            break;
                        case "Vehicle Id":
                            excel.AddMapping<Ticket>(t => t.VehicleId, "Vehicle Id", d => d.ToString());
                            break;
                        case "Id ":
                            excel.AddMapping<Ticket>(t => t.VendorId, "Id ");//column in excel file has trailing space
                            break;
                        case "Id":
                            excel.AddMapping<Ticket>(t => t.VendorId, "Id");
                            break;
                        default:
                            continue;
                    }

                }
            }

Thank you so much guys for such an elegant library!
@danielc777888 This is the workaround I added to circumvent the leading and trailing spaces in the column mapping.
private ExcelQueryFactory MapSpreadsheetColumnsToInvoiceModel(ExcelQueryFactory excel)
{
    var columnNames = excel.GetColumnNames(WorkSheetName).Select(x => x.Trim()).ToList();
    excel.AddMapping<InvoiceModel>(x => x.Quantity, columnNames.First(x => AreStringsCaseInsensitiveEqual("Quantity", x)));
    excel.AddMapping<InvoiceModel>(x => x.Changes, columnNames.First(x => AreStringsCaseInsensitiveEqual("Changes", x)));
    excel.AddMapping<InvoiceModel>(x => x.VendorNumber, columnNames.First(x => AreStringsCaseInsensitiveEqual("Vendor Number", x)));

    return excel;
}

private bool AreStringsCaseInsensitiveEqual(string val1, string val2)
{
    return string.Equals(val1, val2, StringComparison.OrdinalIgnoreCase);
}


Part 3: Add the corresponding ListPreference items for the updated settings; #187 - subtask.

This subtask is for the enhancement #187 .
Add the corresponding ListPreference items for the updated settings
Make the necessary changes in src/main/res/xml/settings_defaults.xml
After making the necessary changes ,merge it with the branch [187-update-default-units-in-settings_Group_8_C2]

The above changes are committed and pushed to the mentioned branch.
Commit Link: https://github.com/rilling/OpenTracksConcordia/commit/ec631f6a95f4ab67ff02e492de1ee53ded54a82f


Tools/Restore database Must confirm before [start]

System information:

Operating system (distribution) and version: Mac
DBeaver version: 21 CE

Connection specification:

Database name and version : Mysql 5.7
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
i have a friend, and
he click [start] of Tools/Restore,
he lost his job,
he restore wrong data.
he just dump database as usual,
but he get wrong menu, get wrong [start].
he want

any dangerous Action Must have confirmed.
and dangerous input Must have no-previous value.

it's a sad story.
Steps to reproduce, if exist:
right click any database, click Tools/Restore, and click start
Include any warning/errors/backtraces from the logs
No Warning

Hello @trydofor
Sad story.
Maybe we must add more confirmation dialogs for these dangerous operations like Restore or Truncate.
Maybe we must add some new double confirmation for DELETE operation and others.

thanks 👍


No FLUX LORA Training  No error

Did you already ask in the discord?
Yes
You verified that this is a bug and not a feature request or question by asking in the discord?
Yes
The thing is, every config done by double checking, I got model files from huggingface and download them manually and put inside D:\flux-dev folder (54Gb)
configured .yaml file,
When i start to train it says Loading Flux model, then Loading transformer and its done. No error, no output but terminal asking for new input.
(venv) (base) PS D:\ai-toolkit> python run.py config/flux-d-person.yaml Running 1 job D:\ai-toolkit\venv\Lib\site-packages\controlnet_aux\mediapipe_face\mediapipe_face_common.py:7: UserWarning: The module 'mediapipe' is not installed. The package will have limited functionality. Please install it using the command: pip install 'mediapipe'   warnings.warn( D:\ai-toolkit\venv\Lib\site-packages\controlnet_aux\segment_anything\modeling\tiny_vit_sam.py:654: UserWarning: Overwriting tiny_vit_5m_224 in registry with controlnet_aux.segment_anything.modeling.tiny_vit_sam.tiny_vit_5m_224. This is because the name being registered conflicts with an existing name. Please check if this is not expected.   return register_model(fn_wrapper) D:\ai-toolkit\venv\Lib\site-packages\controlnet_aux\segment_anything\modeling\tiny_vit_sam.py:654: UserWarning: Overwriting tiny_vit_11m_224 in registry with controlnet_aux.segment_anything.modeling.tiny_vit_sam.tiny_vit_11m_224. This is because the name being registered conflicts with an existing name. Please check if this is not expected.   return register_model(fn_wrapper) D:\ai-toolkit\venv\Lib\site-packages\controlnet_aux\segment_anything\modeling\tiny_vit_sam.py:654: UserWarning: Overwriting tiny_vit_21m_224 in registry with controlnet_aux.segment_anything.modeling.tiny_vit_sam.tiny_vit_21m_224. This is because the name being registered conflicts with an existing name. Please check if this is not expected.   return register_model(fn_wrapper) D:\ai-toolkit\venv\Lib\site-packages\controlnet_aux\segment_anything\modeling\tiny_vit_sam.py:654: UserWarning: Overwriting tiny_vit_21m_384 in registry with controlnet_aux.segment_anything.modeling.tiny_vit_sam.tiny_vit_21m_384. This is because the name being registered conflicts with an existing name. Please check if this is not expected.   return register_model(fn_wrapper) D:\ai-toolkit\venv\Lib\site-packages\controlnet_aux\segment_anything\modeling\tiny_vit_sam.py:654: UserWarning: Overwriting tiny_vit_21m_512 in registry with controlnet_aux.segment_anything.modeling.tiny_vit_sam.tiny_vit_21m_512. This is because the name being registered conflicts with an existing name. Please check if this is not expected.   return register_model(fn_wrapper) {     "type": "sd_trainer",     "training_folder": "output/duy_test",     "performance_log_every": 1000,     "device": "cuda:0",     "trigger_word": "DuyTest",     "network": {         "type": "lora",         "linear": 32,         "linear_alpha": 32     },     "save": {         "dtype": "float16",         "save_every": 250,         "max_step_saves_to_keep": 4     },     "datasets": [         {             "folder_path": "D:\\faces\\captioned",             "caption_ext": "txt",             "caption_dropout_rate": 0.05,             "shuffle_tokens": false,             "cache_latents_to_disk": true,             "resolution": [                 512,                 768,                 1024             ]         }     ],     "train": {         "batch_size": 1,         "steps": 2000,         "gradient_accumulation_steps": 1,         "train_unet": true,         "train_text_encoder": false,         "gradient_checkpointing": true,         "noise_scheduler": "flowmatch",         "optimizer": "adamw8bit",         "lr": 0.0001,         "skip_first_sample": true,         "linear_timesteps": true,         "ema_config": {             "use_ema": true,             "ema_decay": 0.99         },         "dtype": "bf16"     },     "model": {         "name_or_path": "D:\\flux-dev",         "is_flux": true,         "quantize": true     },     "sample": {         "sampler": "flowmatch",         "sample_every": 250,         "width": 1024,         "height": 1024,         "prompts": [             "[trigger]  with red hair, playing chess at the park, bomb going off in the background",             "a [trigger]  holding a coffee cup, in a beanie, sitting at a cafe",             "a [trigger]  is a DJ at a night club, fish eye lens, smoke machine, lazer lights, holding a martini",             "a [trigger]  showing off his cool new t shirt at the beach, a shark is jumping out of the water in the background",             "a [trigger]  building a log cabin in the snow covered mountains",             "[trigger]  playing the guitar, on stage, singing a song, laser lights, punk rocker",             "hipster [trigger]  , building a chair, in a wood shop",             "photo of a [trigger] , white background, medium shot, modeling clothing, studio lighting, white backdrop",             "a [trigger]  holding a sign that says, 'this is a sign'",             "a [trigger] , in a post apocalyptic world, with a shotgun, in a leather jacket, in a desert, with a motorcycle"         ],         "neg": "",         "seed": 42,         "walk_seed": true,         "guidance_scale": 4,         "sample_steps": 20     } } Using EMA D:\ai-toolkit\extensions_built_in\sd_trainer\SDTrainer.py:61: FutureWarning: torch.cuda.amp.GradScaler(args...)is deprecated. Please usetorch.amp.GradScaler('cuda', args...)` instead.
self.scaler = torch.cuda.amp.GradScaler()
#############################################
Running job: duy_test
#############################################
Running  1 process
Loading Flux model
Loading transformer
(venv) (base) PS D:\ai-toolkit>`

"type": "sd_trainer", "training_folder": "output/duy_test",  kinda obvious you are missing training folder parameter.


Fail to compile on 16-bit platforms

Objective
bevy_ecs assumes that u32 as usize is a lossless operation and in a few cases relies on this for soundness and correctness. The only platforms that Rust compiles to where this invariant is broken are 16-bit systems.
Solution
Explicitly fail compilation on 16-bit platforms instead of introducing UB.
Properly supporting 16-bit systems will likely need a workable use case first.

Changelog
Removed: Ability to compile bevy_ecs on 16-bit platforms.
Migration Guide
bevy_ecs will now explicitly fail to compile on 16-bit platforms.  If this is required, there is currently no alternative. Please file an issue (https://github.com/bevyengine/bevy/issues) to help detail your use case.

#4452 attempts something similar for 32 bit platforms. Do we need to forbid those too?

Can you add a more detailed description of where this is relied upon for soundness to your PR description? I want to be able to refer back to this easily if we ever decide to support it.

#4452 attempts something similar for 32 bit platforms. Do we need to forbid those too?

@alice-i-cecile isn’t wasm a 32-bit platform for the time being?

#4452 attempts something similar for 32 bit platforms. Do we need to forbid those too?

No, I don't think we strictly rely on any unchecked u64 to usize conversions where unsafe code is involved, though we should definitely audit and verify this isn't the case. However, some of the core data types we use, namely Entity, assume we have access to a usize that is at least as big as u32 and the soundness of the unsafe code surrounding it relies on that assumption.

Can you add a more detailed description of where this is relied upon for soundness to your PR description? I want to be able to refer back to this easily if we ever decide to support it.

Done.

bors r+


is there any reason why "node" : ">=0.4.11" in package.json?

Hi Matthew,
I run the suite test on node v0.4.7 and the suite test pass. Is there any reason why cheerio restrict the usage with node >= v0.4.11?
I will need cheerio with this version of node. Can you modify the version of node in package.json?

Ohh probably not, I didn't test backwards compatibility. If the test suite runs correctly, it should work fine. I'll take a look at this.

Just added it to master. Will be in the next release.

Thanks!


Hotfix: Reverting BC Break - PersistentCollection should accept null and array as constructor parameter

:+1: for array
:-1: for null

@Ocramius Are you sure about this ? PersistentCollection in 2.4 does not seem to accept arrays(well, the constructor accept them, but you get a fatal error later on almost any method call

PersistentCollection in 2.4 does not seem to accept arrays(well, the constructor accept them, but you get a fatal error later on almost any method call

Then I'm fixing this incorrectly and I'll change the fix.
Adding a Collection type-hint to the third constructor argument of PersistentCollection actually exposes the code bits that are troublesome right no

Therefore, claiming we support null is very odd.

Agreed. I'll check for a better solution

@Ocramius IMO, giving them an error on the typehinted argument is fine, given that code which would break on the typehint is code that would have produced a fatal error anyway. Code paths triggering a fatal error are not covered by BC IMO. Fatal errors are not a feature of the code

Sorry for closing/re-opening: I accidentally deleted the branch on my fork. This approach should be better :-)

:+1:

:+1:

Backported into 2.5 at doctrine/doctrine2@724dfa0de3a45da9214ab926d86aa4bad91dbbfd

Note: assigned http://www.doctrine-project.org/jira/browse/DDC-3667 to this issue (since the integration bot had a hiccup)


[FIXED] Possible delays in delivering messages

There is a possibility that a partial write results in data
not being sent in a timely fashion to a subscription.
Signed-off-by: Ivan Kozlovic<EMAIL_ADDRESS>

Coverage increased (+0.02%) to 91.722% when pulling c4bdc8b7e800787fd8311a9053d38ddc424ab278 on fix_possible_writeloop_stall into ae80c4e98ba9a9d671cc8f7c39ec48cb4a24cb22 on master.

LGTM

I merged it but doing normal benchmarks it slows things down a bunch so may edit a bit..


Don't show "messages" header if there are no messages defined.

The TOC item is not shown, but the header was still in the markdown file. This PR makes it consistent.

Awesome, thanks!


hitting enter after adding an image or link to wiki exits editing

Steps

Visit a wiki page you have permissions to edit
Enter edit mode and click on the add link or add image button above the wiki editing box.
Enter a url—do not press 'OK'
Hit the 'return' key

Outcome

The user is brought out of edit mode
Changes are not published (but returning to edit mode shows that they were auto saved)

Expected outcome

Either hitting return should be equivalent to pressing OK or it should do nothing. The user should not be brought out of the edit mode.

I can't reproduce this bug. Tested locally, on staging, and on production (on production - both projects that I created and did not), also with default ('http://') and filled in url fields. Are there any other details that might affect this?

not that I can think of? I can still produce it and so can @mckidwell

Hmm. It's not happening on Firefox, but it does on Chrome/Safari. I'm getting a TypeError: r.refresh is not a function in my console log for FF, which may explain why FF is immune to this bug. It appears to be coming from a minified version of pagedown-ace's JS

So, commenting out that line fixes this in Safari and Chrome, and has no effect that I've observed in FF. Reading through that code doesn't yield any obvious issues with doing so, and the original author even left a comment saying that this [method] is awkward and I think it sucks. However, the wiki doesn't run in Live Edit Mode locally, so there may be unintended side effects on staging. @brianjgeiger Can you comment on that subject?

I have no comment on the subject. I didn't work with that particular code directly.


rename-branch implicitly uses current branch

This mirrors the native git branch -m command more closely.
Allows for the following:
git town-rename-branch new_name
git town-rename-branch old_name new_name

I agree that most people only ever rename a branch that they're on, but for consistency with git branch --move, I think it's best to still support the optional old branch name. No changes are required to maintain this support.


mgmt/mcumgr/lib: Continue partial application upload

Is your feature request related to a problem? Please describe.
An application image upload context is only kept in RAM, so in case if reset occurs during upload the upload needs to restart with all the costs of erasing flash and so on.
Describe the solution you'd like
An upload may exchange with an application control sums of subsequent flash pages to decide where to re-start upload process, saving on upload time and reducing need for erasing flash when part of an image has already been successfully written to flash.
Describe alternatives you've considered
Re-uploading whole image as it happens now.
Additional context
Control sums should be calculated per page size as visible on the device side, as pages may have variable size and that page size dictates how much data would have to be erased and re-uploaded.

@de-nordic, it would be good if such a feature is added. The stream flash api has support for something like that. Wouldn't this feature require a complete mcumgr rework where the device request for blocks ?

@de-nordic, BTW you are an example of what maintainers should be doing.

@de-nordic, it would be good if such a feature is added. The stream flash api has support for something like that. Wouldn't this feature require a complete mcumgr rework where the device requests for blocks ?

Currently a mcumgr enabled device has the maximum size of the "chunk" of upload data hard-coded  at compile time (https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/mgmt/mcumgr/Kconfig#L158. but rather than using that number, the client side should send sum with offset and size it used to calculate it on, and the device should use that data. Reason for above is that your client side application may be keeping track on chunks that have already been sent to a device so it may choose its own strategy how to handle checks; for example it may as to check sum of everything from offset 0 to last chunk or starting at chunk -2 from the last that has been confirmed.
The image upload protocol allows device to ask for previous chunk by sending lower offset in response, so it is basically able to ask to restart transfer at some specific offset/
As you know the erase needs to happen at page boundaries, so in case when sum fails anywhere within page boundary, the request offset would have to point at the page offset.

BTW you are an example of what maintainers should be doing.
Thanks


Quark Sort Buttons/Trash Slot Compat

If there is a way to make the chest/inventory buttons from Quark and the trash slot from Trash Slot work while the backpacks are open that would be fantastic!
love the mod, its easily the best backpack mod out there, thank you

For the record, I checked to see if Quark's "Forced GUIs" settings would make the buttons appear and it doesn't appear so.

huh, I didn't even know about that option. either way, I left a comment a little while ago on the ION chests CF page asking for wearable backpacks compat since Asie was a contributor to this project, but it seems that project might be finished for good. I did some poking around in the config though, and it looks like I can add the class for whatever container I want the buttons on, but I haven't been able to figure that out either lol

The idea is to turn on the "Debug Classnames" option or so, which will cause a debug message to appear in the log when any GUI is opened. That way you can find out the name of the GUI class, which in Wearable Backpacks's case is net.mcft.copy.backpacks.client.GuiBackpack.
Unfortunately in my test, adding this did not work. That might very well be because my GUI isn't based on any existing chest GUI or similar class, because it needs to be resizable and such.


TG vending machine sprites are incorrect

The vending machine spritesheet doesn't match the actual item sprites.

This may (I haven't checked) be related to the cache relying on IP address instead of hashing, since local dev servers all have the same IP (<IP_ADDRESS>) regardless of codebase.

Actually, clearing the cache doesn't seem to fix this, so it must be something else.


Unable to find a valid cuDNN algorithm to run convolution

hello. When i run the ''python tools/train.py configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py'', it will get an error :
File "/home/xxx/anaconda3/envs/mmdet/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 345, in _conv_forward
return F.conv2d(input, weight, self.bias, self.stride,
RuntimeError: Unable to find a valid cuDNN algorithm to run convolution.
And the environment is
sys.platform: linux
Python: 3.8.5 (default, Aug  5 2020, 08:36:46) [GCC 7.3.0]
CUDA available: True
CUDA_HOME: None
GPU 0: Quadro K2000
GCC: gcc (Ubuntu 9.3.0-10ubuntu2) 9.3.0
PyTorch: 1.5.0
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) Math Kernel Library Version 2020.0.1 Product Build 20200208 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v0.21.1 (Git Hash 7d2fd500bc78936d1d648ca713b901012f470dbc)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX
CUDA Runtime 10.2
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-
gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-
gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.5

Besides, it also get an warning:
userWarning:
Found GPU0 Quadro K2000 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.
warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
i also run it with pytorch 1.6.and it also has the same problems.
i want to konw the problems come from the gpu is too old, or maybe my enviroment has some problems? thanks.

This is just because the GPU is too old. The minimum arch supported by Pytorch-1.6 is 3.5 but your GPU only supports CUDA 3.0.

@ZwwWayne thank you very much, can i solve it by build pytorch from source?

I solve it by build pytorch from source and modify some  files. thanks

How did you solve it?

I tried to decrease the image size and the batch size to consume less VRAM and it works fine!

It's probably due to insufficient GPU memory. You can try adjusting the batch size.


Logfile JSON encode

logzero version: 1.6.2
Python version: 3.8
Operating System: Ubuntu 20.04

Description
logger.json(True)
I cant encode in UTF-8 , my special characters, like Ç , ç , ã , é and others, on file appear \x0231 etc .
logfile.log

Should be fixed in PR #357. Using UTF-8 by default now

Fixed and released in logzero v.1.6.3


Fixes #2738 - Adds the /git/tags API endpoint

Fixed #2738 (2nd part)
Read the issue above, especially the recent comment by the user (https://github.com/go-gitea/gitea/issues/2738#issuecomment-497767418) on why I had to change a bit of stuff in how the tags API works.
Adds the {owner}/{repo}/git/tags API endpoint and properly identifies lightweight (commit) and annotated (tag) tags in the {owner}/{repo}/git/refs/tags API endpoint.
Fixes the API URLs that were not generating properly (no host name), e.g. "url:" of https://try.gitea.io/api/v1/repos/richmahn/test/tags
Fixes bug in /tags endpoint when there are multiple tags with the same commit ID (shows the same tag name over and over for each tag due to the caching, had to clone the cached struct)
Feel free to try the /tags API here: https://bg.door43.org/api/v1/repos/richmahn/test/tags
and /git/refs and /git/tags repo starting from here: https://bg.door43.org/api/v1/repos/richmahn/test/git/refs (best to have a JSON viewing plugin installed for your browser)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@dadc03f). Click here to learn what that means.
The diff coverage is 80.72%.

@@            Coverage Diff            @@
##             master    #7138   +/-   ##
=========================================
  Coverage          ?   41.72%
=========================================
  Files             ?      446
  Lines             ?    61011
  Branches          ?        0
=========================================
  Hits              ?    25457
  Misses            ?    32251
  Partials          ?     3303

Impacted Files
Coverage Δ

models/repo_tag.go
62.5% <ø> (ø)

routers/api/v1/convert/convert.go
83.8% <100%> (ø)

modules/git/repo_ref.go
72.72% <100%> (ø)

routers/api/v1/repo/git_ref.go
91.11% <100%> (ø)

routers/api/v1/api.go
71.31% <100%> (ø)

modules/git/tag.go
73.91% <100%> (ø)

modules/git/repo_tag.go
65.92% <67.03%> (ø)

routers/api/v1/repo/tag.go
84.05% <82.05%> (ø)

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 dadc03f...9c52751. Read the comment docs.


KAFKA-4913: creating a window store with one segment throws division by zero error

Allow a window store with a single segment.

@guozhangwang @enothereska @xvrl @mjsax

If there is only one segment then set the cleanup.policy to compact.

What's the motivation behind this (with one window segment the changelog topic may still have lots of log segments)?

What's the motivation behind this (with one window segment the changelog topic may still have lots of log segments)?

If we use delete then we will be deleting data that belongs in the segment. Or maybe it doesn't matter as it should obey the retention period anyway. Yeah i think that is right! Ok i'll change that back! Thanks

retest this please

retest this please

@guozhangwang i'm not particularly bothered with this patch. Maybe we should do it differently, i.e., only allow >=2 segments for a Window store. I haven't seen a valid argument for having a single segment. WDYT?

@guozhangwang i'm not particularly bothered with this patch. Maybe we should do it differently, i.e., only allow >=2 segments for a Window store. I haven't seen a valid argument for having a single segment. WDYT?

I agree, we can just forbid setting one segment.
Also there are a few comments from @mjsax that seem not addressed yet?


Can't make it work.

Hi, I tried to use your program, but it doesn't work no matter how I try to use it.
You give 3 exe files in your release.
When I run the TkForge.exe, I get the window asking for the figma token, file URL, and output path.
Everything is great.
I give the figma token, the file URL and specify the output path... but I always get an error:
"The file ID, token or output path that you provided is invalid!"
Why is that ? How can I fix that?

Hey,
For any help on using the app message me on Discord. You can find my Discord server link here -> https://github.com/axorax/socials
Anyways, I recently helped someone with the same issue. They forgot to add a frame. Maybe your design also has no frame? If that doesn't work, message me on Discord! 😄


Skip http method matching in http_parser with UHV

Commit Message: Skip http method matching in http_parser with UHV
Additional Description: When the UHV compile-time flag is enabled, strict HTTP parsing is disabled by default, but currently, there is no support for custom methods in http_parser. While BalsaParser is now the default codec for HTTP/1, the http_inspector still uses the legacy http_parser. This change enables support for custom HTTP methods in http_parser when UHV is enabled.
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest failed

/retest

Hi, just a quick heads up that I do not work on Envoy any more. Consider contacting @yanavlasov for further assistance with custom methods.

Hi, thanks for trying to resolve this problem. But could we just switch the parser in the http_inspector to our new parser and add an option to the http_inspector to enable the custom methods support?
Considering we finally may remove the legacy parser, I think updating the http_inspector would be better than to hacking the legacy http_parser.c.
Thanks again for you contribution. :)
/wait

Hi, thanks for trying to resolve this problem. But could we just switch the parser in the http_inspector to our new parser and add an option to the http_inspector to enable the custom methods support?

So you are suggesting to switch the parser in http_inspector and add a new [#not-implemented-hide:] flag like allow_custom_method similar to what we for HCM?

Hi, thanks for trying to resolve this problem. But could we just switch the parser in the http_inspector to our new parser and add an option to the http_inspector to enable the custom methods support?

So you are suggesting to switch the parser in http_inspector and add a new [#not-implemented-hide:] flag like allow_custom_method similar to what we for HCM?

Not sure if the #not-implemented-hide annotation is necessary or not. Because we basically won't provide another UHV in the http_inspector.

Hi, thanks for trying to resolve this problem. But could we just switch the parser in the http_inspector to our new parser and add an option to the http_inspector to enable the custom methods support?

Regarding updating the parser in the http_inspector, I'm planning to do this in two steps.

Update the http_parser usage with the Parser interface and use the LegacyHttpParserImpl implementation for the current http_parser behaviour.
Introduce a new runtime flag that will switch the usage to the BalsaParser.

Does this approach seeem fine?

Hi, thanks for trying to resolve this problem. But could we just switch the parser in the http_inspector to our new parser and add an option to the http_inspector to enable the custom methods support?

Regarding updating the parser in the http_inspector, I'm planning to do this in two steps.

Update the http_parser usage with the Parser interface and use the LegacyHttpParserImpl implementation for the current http_parser behaviour.
Introduce a new runtime flag that will switch the usage to the BalsaParser.

Does this approach seeem fine?

SGTM.

Created a new issue #36433 to update the parser in http_inspector.

@wbpcode created a new PR #36430 to update the parser code in the http_inspector. Can you review the changes?
Also, closing the current PR for updating the legacy http_parser.


update position object in getCurrentPosition

update position object when the position is gotten for the first time

Hi @ArsalanDotMe - can you include a test?
Thanks


refresh imports 22-oct-2024

Testing out a new PR for refreshing the imports. I ran the refresh imports on a fresh clone of Mondo to verify this is happening as a result of refreshing the imports. The same errors as found originally occur. See the error log issue here.
Note: this error is also posted as it's own issue.

Two imports were not previously in the spreadsheet so they were missed in this refresh of the imports (luckily Sabrina caught this!). I will refresh the imports again with these new entities.

I re-ran the refresh-merged import pipeline with the new additions to the manual_seed.txt file, but that did not seem to actually import these two entities since they still show up as numbers. Other '*_terms.txt' files were updated this time though.


Spotter in GToolKit image is unusably small

I cannot use the Spotter - is it supposed to be resizable, or detachable, or anything? It is too small/cramped, even when full sized. Is this normal?

Thanks for the issue!
Indeed, we want to add resizing. It's just not done yet. We will likely increase the size in the meantime. We target for now a laptop screen.

While waiting for the real solution, we made the spotter larger (800@500).

Do you also use GToolkit on a larger screen? On my 40" UHD at home the experience is not ideal, and I often can avoid using tabs.

We do use it on larger screens, but we are at the moment targeting laptop screens. We will get to the larger screen soon.

@StephanEggermont we added the ability to spawn a tab in a new window. This would be a first step towards supporting larger screens. See #823

Spotter can now be resized. Also, the global spotter remembers the last size.


Plans for DTLS 1.2 Connection ID

Hi everyone, first of all congrats for the job you all are doing here. You got to a point where the product is really strong and reliable. 👏
Summary

Support DTLS 1.2 Connection ID proposal: https://github.com/tlswg/dtls-conn-id
RFC tracker: https://tools.ietf.org/html/draft-ietf-tls-dtls-connection-id

Motivation
As we use DTLS 1.2 for our IoT fleet this would help using it on NATed environments.
It seems that DTLS 1.3 includes the CID also as @daenney pointed in #188.

@glerchundi I would love to do this!
I wasn't aware of the Connection ID work. This sounds really amazing, this could allow people to load balance WebRTC traffic finally.

FWIW, here's an example of using CID for load balancing with an IoT protocol: https://github.com/eclipse/californium/tree/wip_dtls_cid_cluster/demo-apps/cf-extplugtest-server#built-in-dtls-connection-id-load-balancer-cluster /by @boaks

You may check WIP - CID.
AFAIK, that's based on pion-dtls.

That extension is now the upcoming RFC9146.
Sure, it's more relevant for IoT than for other use-cases.
My experience with it is really great. My devices are now running over a week, using deep sleeping modes, and exchanging mainly one UDP-message pair (CoAP request/response) on wakeup. Very efficient, very reliable (with  CoAP retransmissions on timeout in very rare cases). I developed end of last year a zephyr dtls-cid preview (based on tinydtls feature/connection_id and a thingy 91). It's now up for a week.

Awesome! Is that preview available somewhere?

I plan to make it available. But I have to do some tasks ahead.
It's mainly tinydtls / feature/cid branch with the other bugfixes PRs pending.
Therefore I would like to have some of them merged before, that makes it easier to maintain the preview.
The second part is very small, it's the nRF9160 udp sample together with the zephyr coap-lient merged into the tinydtls dtls-client sample.
So, maybe next month or at least at Q2/2022.
End of last week, also the pending mbedtls PR 5061 was fixed. Let's see, how long that takes.

Thanks for the heads up! Great to see community is moving!

Hi, are there any updates on this?

For mbedtls, yes. It's merged in the meantime.

For mbedtls, yes. It's merged in the meantime.

thanks. I haven't seen anything in pion/dtls (though I'm new to it so maybe missed it)

Hey folks! I’m planning to work on adding support for Connection IDs in pion/dtls imminently — @Sean-Der any tips or requests?

That is great news @hasheddan!
I believe it will be pretty straight forward. On State you will have a ConnectionId *int
When generating data in writePackets you can add it as needed. I will add you to the github org and you should start a branch ASAP.
I would love to work together on this also. Would a short audio/video call be helpful to get started?

@Sean-Der sounds great! I'll join the Slack and give you a ping 👍🏻

Hi, we are also looking into this. I'm very interested in Updates about the state of the implementation. Let me know when I can help somehow.

Just to mention:
If there is some interest in interoperability tests with Eclipse/Californium and Eclipse/tinydtls, just notify me here in this issue, when your work reaches a state, where some tests makes sense.
Recently the first cellular device starts to support DTLS CID (Nordic nRF9160 mfw 1.3.5) so it may get more and more interesting.

@niondir I should have an initial implementation ready soon 👍🏻
@boaks definitely interested in the interoperability tests! Will make sure to ping when ready.

Also, if you have any question about RFC9146, just ask, I will try my best to answer it.

@boaks #558 has DTLS CIDs working e2e and I would love to get some interoperability tests going! I was checking out the current tests here -- would that be a good place to add similar tests for pion/dtls? I am also interested in expanding the e2e tests in this repository as currently only openssl is tested against.

would that be a good place to add similar tests for pion/dtls?

Yes, but I'm not common with "go" ;-).
So, if you like to help, the first step is usually a simple manual test.
Maybe you can point me to a tutorial, how to checkout your work and to start the server and client?

@boaks I added some comments about how to run the e2e tests using the Docker image in #558, but I'll setup a manual test with scandium and report back!

The docker stuff is fine. I already had a first check of my wireshark captures.
If it would be possible to have a dtls server running in docker for tests with clients using other implementations, that would be great for first tests.

@boaks I can build and push one to Dockerhub for now, then I'll see about adding the machinery to this repo 👍🏻 In the meantime, I was able to get some initial testing running with the DTLS client demo app in californium and this slightly modified example PSK listener:
package main

import (
	"context"
	"fmt"
	"net"
	"time"

	"github.com/pion/dtls/v2"
)

func main() {
	addr := &net.UDPAddr{IP: net.ParseIP("<IP_ADDRESS>"), Port: 5684}

	ctx, cancel := context.WithCancel(context.Background())
	defer cancel()
	config := &dtls.Config{
		PSK: func(hint []byte) ([]byte, error) {
			fmt.Printf("Client's hint: %s \n", hint)
			return []byte("secretPSK"), nil
		},
		PSKIdentityHint: []byte("Pion DTLS Client"),
		CipherSuites:    []dtls.CipherSuiteID{dtls.TLS_PSK_WITH_AES_128_CCM_8},
		ConnectContextMaker: func() (context.Context, func()) {
			return context.WithTimeout(ctx, 30*time.Second)
		},
		ConnectionIDGenerator: dtls.RandomCIDGenerator(8),
	}

	listener, err := dtls.Listen("udp", addr, config)
	if err != nil {
		panic(err)
	}
	defer func() {
		if err := listener.Close(); err != nil {
			panic(err)
		}
	}()

	fmt.Println("Listening")

	conn, err := listener.Accept()
	if err != nil {
		panic(err)
	}

	b := make([]byte, 8192)
	for {
		n, err := conn.Read(b)
		if err != nil {
			panic(err)
		}
		fmt.Printf("Echoing received message: %s\n", string(b[:n]))
		if _, err := conn.Write(b[:n]); err != nil {
			panic(err)
		}
	}
}

I was able to build and run the container. No need to push a container in a registry.
But please add something, that I can start the container as server.

It's pretty easy to provide a capture instead of a picture. For now, as no GO user, I'm not able to start a test server or client. But If you provide captures, I will have a look at them.

@boaks see https://github.com/pion/dtls/pull/558#issuecomment-1647821711 for information about build and running a test server with connection ID support, as well as a link to a packet capture of testing with californium 👍🏻


InHouse Profile Type

Any chance support for in-house (enterprise) can be added?
I am no ruby developer but if you can point me in the right direction or let me know what information you might need I would love to help.

It's already supported, just use your Enterprise account and use sigh as always :+1: Let me know if that works for you.

Hi,
I tried to create one with an enterprise account using fastlane:
sigh( adhoc: true )
An Adhoc provisionning profile was created instead of an inhouse.

You can just not pass the adhoc option to create an enterprise profile :wink:

It's work !
Thanks !
2015-05-05 17:49 GMT+02:00 Felix Krause<EMAIL_ADDRESS>
You can just not pass the adhoc option to create an enterprise profile [image:
:wink:]
—
Reply to this email directly or view it on GitHub
https://github.com/KrauseFx/sigh/issues/90#issuecomment-99121763.

I also (accidentally) set adhoc:true. Thanks!

I should update sigh to ignore the adhoc option if the account is an enterprise account.

I don't think so, someone might want to use a Adhoc profile for internal beta (restricted UUID).
Maybe just a note in the sigh readme, for enterprise accounts "App Store" == "InHouse" to remove confusion.


Is there an alternative ppx for ppxlib?

I'm working on some AST definitions (not for OCaml) and looks like ppx_traverse is a great way to define mappers for AST structure. Is there generic deriver with same functionality available for ppxlib?

Yes, ppxlib.traverse :)

Wow, cool! I should've searched harder. Thanks!


Use absolute path for discovery doc

This allows the artman command line call to run from any directory, e.g. google-cloud-java, not just from inside the discovery_artifact_manager directory.

PTAL

I might be missing something - it's unclear for me how the root dir will be used. Why it only applies to one parameter (discovery doc)?

@alexander-fenster
(Copied from a previous comment)
The problem is that artman resolves all filepaths in the config to be relative to the config filepath and assumes that the artman config file is at the highest (or equal) level directory compared to the files in the config.
For example, in googleapis, for google/pubsub/artman_pubsub.yaml, all the pubsub API files are located at or below that directory level, like google/pubsub/v1/pubsub.proto.
However, in discovery-artifact-manager, the relevant discovery document is not in the same directory path as the artman config.
For example, for gapic/google/compute/artman_compute.yaml, the relevant discovery doc is at discoveries/compute.v1.json.
PTAL


Review and clean all file headers, there are legacy headers containing license declarations that need removing

We need to review and define standard file headers through the project. There are currently some old legacy headers that were generated by @BioinfoTongLI, possibly from some kind of template. They make licensing declarations we need to remove. We can build a header template using the citation link which is now present in the repo.

sure, and the citation link do you mean https://github.com/haniffalab/vitessce-pipeline/blob/main/CITATION.cff ?

Done


Ability to test fixes

I'm not sure if this is possible, or that there is another way of doing this. But I guess it would be nice if we could test the fixers as well.
Something like this maybe?
ruleTester.run('my-awesome-rule', rule, {
  fix: [
    {
      input: 'const foo = 0XFF',
      output: 'const foo = 0xFF'
    }
  ]
});

// @sindresorhus

Alright, now I see this is only a small wrapper around the ESLint rule tester. Odd that they don't seem to have it themselves...

Yes, this has the same API as ESLint's RuleTester. And it does have the ability to check fixes, but the documentation for it is really lacking. You have the output field to specify the expected code output.
{
  code: 'it.skip()',
  errors: [ { message: expectedErrorMessage, column: 4, line: 1 } ],
  output: 'it()'
}

@jfmengels Can you add an example of it here? Even though ESLint doesn't document it very well, doesn't mean we can't make it easier for people.

Yes it would be nice if it was documented.

Good point

What is the ESlint Rule Tester!? I assume it is the tester that runs the ESlint rules on your code to see if you follow conventions specified and reports errors...
So this library will run these tests on the Ava test files!? Trying to get the full picture! Thanks :)

It allows you to test rules, hence the name ;).
Example: https://github.com/sindresorhus/eslint-plugin-unicorn/blob/master/test/no-new-buffer.js#L24-L80


Problem with several observations

When we have more than one observation, not all of them are generated in the proper entities. We observed that they are not generated all of them (n-1).

The parser was developed to have only one observation. Implemented now with multiple observations


Reworked itemDetail with customPUT to add item to current order. Adde…

The addItem() function in itemDetail.js puts item.id and item's quantity to add_to_cart view.

Before approving this request, we may want to review AddItemToOrderView.  It calls for a list of item ID's  and list of corresponding quantities, which may be extraneous..  Is there any need to keep a running list of these? It appears that mackeyja92 has already accounted for updated quantities.


Adding trace logging to resource state policy

Added trace logging messages to help debug FixedBackoffResourceStatePolicy::isDown()
Time to review: 5 min
Reviewers: @xiahome, @dniq

LGTM

Looks good!

LGTM, merged.


mtr-exporter: init at 2020-03-08 (ab0b97a9)

Motivation for this change
This is a useful utility for monitoring network performance over time using a combination of MTR and Prometheus.
Also adding a service definition.
I'm trying to avoid having to include the patch in the package, but we'll see: https://github.com/mgumz/mtr-exporter/pull/2
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Here's an example of Grafana dashboard from monitoring a website using mtr-exporter and Prometheus:

Very useful for monitoring quality of an internet connection.


Import all the maps of the Trento Public Library

https://bdt.bibcom.trento.it/Iconografia?progetto=Piante+di+Trento&_sort_0_sort_by=data&_sort_0_sort_order=asc&_sort_0_limit=10

WOW!!!
Do you already have a video explaining how to "georeference a picture of a map and place it somewhere where there is a TMS url to share"?
What is the best "somewhere" to place the referenced image?
Or maybe you can suggest a simple video that already explain how to do this.
Thanks!!!


Add alt-f binding for auto-suggestion.

Codecov Report

Merging #623 into 2.0 will decrease coverage by 0.02%.
The diff coverage is 37.5%.

@@            Coverage Diff             @@
##              2.0     #623      +/-   ##
==========================================
- Coverage   71.43%   71.41%   -0.03%
==========================================
  Files         132      132
  Lines       12244    12251       +7
==========================================
+ Hits         8747     8749       +2
- Misses       3497     3502       +5

Impacted Files
Coverage Δ

...rompt_toolkit/key_binding/bindings/auto_suggest.py
65.38% <37.5%> (-13.57%)
:arrow_down:

Continue to review full report at Codecov.

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

This looks good. Thanks! :-)


[Feature Request] Can you add the active-background-color property to the el-menu component?

Existing Component
是
Component Name
el-menu
Description
Most of the time, the background color that is activated is the color of the designer's tune.Since active-text-color has all, why does active-background-color have no?

Duplicate of #7686


Retrieving data using REST API - how to determine time before IBC client expires?

Hey everyone.
I'm developing an app where I need to know the time left for an IBC client to expire. Given my research, I've found the only possible way to get this piece of information is to retrieve:

Trusting Period of the client
Last TX that updated the IBC client

The first is easily accessible, but I'm having issues retrieving the second one.
This is what I have right now:
[osmosis_rest_api_url]/cosmos/tx/v1beta1/txs?order_by=2&events=send_packet.packet_src_channel='channel-0'&events=send_packet.packet_src_port='transfer'&events=send_packet.packet_dst_channel='channel-141'&message.action='/ibc.core.client.v1.MsgUpdateClient'&limit=1
The issue I run into is that the message.action='/ibc.core.client.v1.MsgUpdateClient' parameter in the query doesn't work. Also, I see that the chain_id value in the json response doesn't align with the channels mentioned in the query. It should be either Osmosis or Cosmos, but I get random chains I'm not interested in.
While help in achieving this query to respond the way I expect is greatly appreciated, further information into how to filter information would be paramount in making sure I don't need to ask around every time. I want to be as self-sufficient as possible. Send in any helpful documentation and tips on anything related to REST API calls and filtering the JSON responses.
Thank you for reading.

Hi @napelvs. I think there are a couple of tweaks that you need on your query to make it work:

The message MsgUpdateClient is usually bundled with MsgRecvPacket, so the event that you will see is for receive packet, not send packet; therefore you need to change the event search from send_packet to recv_packet.
When the light client for the Cosmos Hub on Osmosis is updated upon a packet receive the source channel is the channel end on Cosmos Hub (channel-141) and the destination channel is the channel end on Osmosis (channel-0).

So with these two modifications then the query would look like this:
https://rest.cosmos.directory/osmosis/cosmos/tx/v1beta1/txs?order_by=2&events=recv_packet.packet_src_channel='channel-141'&events=recv_packet.packet_src_port='transfer'&events=recv_packet.packet_dst_channel='channel-0'&message.action='/ibc.core.client.v1.MsgUpdateClient'&limit=1
Please give it a try, because I think this returns the information you're after. Let me know if it works for you and then I will close the issue. Thanks!

Hi @crodriguezvega. Thank you for your answer!
I've looked into the query provided by you, and it retrieves the data I need.
From my understanding, this query should always respond with the latest MsgUpdateClient TX that occurred on this specific combination of src/dst channels. Is that right?
My intent is to base the time to expiration of a client on this query - specifically the time of the latest MsgUpdateClient (which I hope this query correctly retrieves).

From my understanding, this query should always respond with the latest MsgUpdateClient TX that occurred on this specific combination of src/dst channels. Is that right?

Yes, I think that's correct!

My intent is to base the time to expiration of a client on this query - specifically the time of the latest MsgUpdateClient (which I hope this query correctly retrieves).

You should based it on time in the header included in MsgUpdateClient, right?

You should based it on time in the header included in MsgUpdateClient, right?

Yes, that's my plan. Ok, that answer my query. I will now go ahead and close this discussion!
Thank you for the assistance!


Misc example/oneshot fixes

apply the application gradle plugin, produces convenient run scripts
fix comparison to read from stdin
actually read bytes from payload file

CC @quidryan @wmiaw


Focus on Today shows incorrect number of ToDo's

Hi
The number of tasks pending in ToDo shows 2 result, but there is none displayed in the list.
I have tried to exclude/include empty or manually filter for tasks via a dataview query.

That's probably because of reccuring tasks.

Is time sorting working for you??

That's probably because of reccuring tasks.

Hi. Is time sorting working for you??

Kindly see my comments in Issue 129. I believe we have the same issue.


Upgrade to Python 3.10

Currently on Python 3.8.10. Look into upgrading to Python 3.10.

Addressed in 6fa52f4.


Make size, decode and encode derives work with non-suffixed literals

Issues:
Resolves #114
Description of changes:
Implemented parsing for the repr attribute when deriving mls codec and used it to infer the type of the enum discriminants that don't have explicit suffixes.
Call-outs:
I'm very much a noob when it comes to rust macros. The code works but it genuinely feels like there should be an easier way to do this: Specifically, these two areas could use some insight for someone who knows what they are doing:

Presumably there is a way to grab the repr values in FromDeriveInput directly but nothing I tried was working in a way that supported optionality and I did not want to force all declarations to define a repr.
The code creating the actual suffixed literal is using some cheeky string interpolation because I could not for the love of me figure out a better way to simply add a suffix to the Literal being parsed.

Testing:
I've added a TestEnumWithoutSuffixedLiterals to the macro usage to ensure this works. The TestEnum case is also still working.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT license.

Codecov Report
Attention: Patch coverage is 94.11765% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 89.34%. Comparing base (650ed8d) to head (ef607a6).

Files
Patch %
Lines

mls-rs-codec-derive/src/lib.rs
94.11%
3 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #115   +/-   ##
=======================================
  Coverage   89.34%   89.34%
=======================================
  Files         173      173
  Lines       31333    31372   +39
=======================================
+ Hits        27993    28029   +36
- Misses       3340     3343    +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add FromLogrusLevel helper

What type of PR is this?
/kind feature
What this PR does / why we need it:
This allows easy type conversion between the log levels and avoids mismatches like "warn" (conmon-rs) and "warning" (logrus).
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

Added `FromLogrusLevel` helper function.

@rphillips @haircommander PTAL

@saschagrunert is anything using the new function?

@saschagrunert is anything using the new function?

CRI-O will use it. Found a bug between the “warning” and “warn” mismatch.

/lgtm


Cache value from previous execution

Is your feature request related to a problem? Please describe.
When I run script I can input desired values. But when I rerun script I need to refill all values again =(
This is boring when there 5-7 values to input
Describe the solution you'd like
Cache values from previous input, so I can just press Enter second time and go

Please try to use named variables. It should solve the problem.

My language use non named parameters. It is a problem to rename each.
DBeaver could use parameter position instead of name to properly cache old values

perl, DBIx::Class

We will implement it. The option will be disabled by default.