Likert scale support

Support for likert scale sliders.
Three basic likert scale types are supported, which are 3-point, 5-point and 7-point

Added review changeschanges


Support hot deploying a subgraph to a running node

If we do deploys through an admin RPC endpoint then the Graph CLI can just be a client over RPC.

This + multiple subgraphs have a lot of surface area so I have a bunch of questions on the requirements, from a discussion with @Jannis:
What happens to current CLI support:

Do we still want to receive subgraphs through the CLI?
If so, will the CLI support multiple subgraphs?

HTTP endpoint:

What CLI options do we need for it?
What will the endpoint be?
What operations will it support, add/remove/update subgraph?
Will we use a protocol such as JSON-RPC?
What's the payload, simply the subgraph hash?
What will we do for auth?

Deploy through graph-cli:

Will we add a subcommand like graph deploy --node IP_ADDR subgraph.yaml?
What parameters and options will it take?

What happens to current CLI support:

Yes, let's continue to accept the currently supported CLI arguments
No, it could be a nice enhancement to allow passing multiple subgraphs on the CLI but it's not necessary for now

HTTP endpoint:

I think the only CLI option we need for now is the port. We can see if there are more once we get further into implementation
Operations: add, remove, update, pause, list

Right now subgraphs don't have names in the YAML. The naming is handled at the GNS layer. We might need to add support for names to make update more ergonomic. It should be possible to just pass a new subgraph hash and have the node stop the old version and start the new one. This can be done naively for now and a separate ticket can track making upgrades more efficient.

I would recommend grpc
Payload: subgraph hash
No auth for this first pass. Let's create a separate ticket for this.

Deploy through graph-cli:

Yes this looks good graph deploy SUBGRAPH_HASH --node HOST_AND_PORT

For the admin endpoint: How about GraphQL mutations rather than GRPC? It would require a small amount of work to support mutations in our execution logic but it would align well with our vision.
We could then have an admin schema and could implement the operations through a dedicated resolver.

Regarding the subgraph name: we could take in --subgraph [OPTIONAL NAME:]IPFS_HASH and use graph deploy --node HOST_AND_PORT [OPTIONAL_NAME:]IPFS_HASH for deploying.

I do not think grpc is viable because there is no official Rust implementation or an unofficial one whose maintenance we can trust. I don't know much about GraphQL mutations but it sounds like an option. I've started implementing using json-rpc because it has good crates and it's simple.

I think GraphQL is what we should do but it won't be super quick and we have enough on our plate, so personally I'm fine with JSON-RPC for a first pass.

@leodasvacas I completely forgot about one aspect: we'll want a different GraphQL endpoint (e.g. /somehash and /someotherhash) for every subgraph that we load. That'll have a bunch of consequences for our GraphQL server but not for the SubgraphProvider etc.
I'll think about that part more tomorrow and try to come up with a good design for it.

I've thought about the different endpoints (one per subgraph) more and it really only affects the SchemaProvider, GraphQLServer, and the GraphQL execution / resolvers.
We'll want to roughly do the following:
SubgraphProvider / SchemaProvider
We may want to drop the SchemaProvider altogether and instead have the SubgraphProvider generate the API schema for the subgraph schema and emit it via SchemaAdded/SchemaChanged/SchemaRemoved(subgraphId: String, schema: Schema), including the subgraph ID.
These schema events would then be forwarded to the GraphQLServer.
GraphQLServer
Manage a mapping of subgraph IDs to their schemas. When receiving a POST request, identify the subgraph ID based on the URL path (/foo would mean that the subgraph ID is foo). Then, send the subgraph ID and the schema along to the GraphQL execution.
When receiving a GET request, do the same: identify the subgraph ID based on the URL path and render GraphiQL so that it's scoped to this particular subgraph.
We may want to serve a super-simple index page with links to the subgraphs that are available.
Subgraph IDs vs. subgraph names
For this pass, do we want subgraph names to be used purely for identifying subgraphs when deploying, or do we also want to allow those names in URLs? (@yanivtal, do you have an opinon here?)

Thanks for this planning effort, I'm starting with a minimal implementation (just add, single subgraph) so we can build this incrementally. What's a good default port for the json-rpc server?

@leodasvacas Sounds good, it's probably worth creating several tasks for the incremental steps.
We currently use port 8000 for the HTTP GraphQL endpoint. Let's keep 8001-8019 free for now and use 8020 for the admin server.

Medium to long term we should make these configurable anyway and perhaps choose less popular defaults. 😉


Global wrapping portals do not generate beneath y=0

According to your testing, does the issue occur with ONLY Immersive Portals mod and Farbic API?
Yes
Describe the issue
1.18 increased world height, however inward wrapping and outward wrapping portals cut off at y=0
Post the log here
log file not relevant

already fixed


RFC: Deprecate dep and dep-ensure Buildpacks

Readable

Should there be a warning period if we think anyone is using these buildpacks? Could look like any of the following:Put something on Slack and wait 30 days?

@joshuatcasey I am open to all of those things as potential options and any more if people have opinions. I would say that dep has been deprecated for around 2 years already so we are not doing this like 1 month after they called it quits.

As a nice side benefit, we will no longer need to maintain the dep dependency (https://github.com/paketo-buildpacks/dep/blob/main/buildpack.toml).
Should we add a step in here to remove automation that generates this dependency?


tests: add libsecret

@homu r+

:pushpin: Commit 39537d1 has been approved by gkoz

:hourglass: Testing commit 39537d1 with merge 3e8474c...

:sunny: Test successful - status

Now we know that version is not always "x.y" but can just be "1".

Yes and this not very good.


DDC source maps product excellence

Thanks to hard work by many, the DDC stack traces in the latest SDK look good 🤞
There are still some minor things, though, for example around closures:
test('...', () {
  expect(...);
});

Emits:
package:test            expect
foo_test.dart 216:9  test$.test.dart.fn

I never wrote test$.test.dart.fn. It would be nice for this to say:
package:test           expect
foo_test.dart 216:9 test

Technical, it's not test, but an anonymous closure in test.  I think we end up naming these dart.fn.
Should be easy to name / stack-map to something else.

@vsmenon Is this still an issue?

anonymous functions are still auto-detected by Chrome as dart.fn It's because we must tag their types:
// compiled JS:
test.test(..., dart.fn(() => { ... }));

We can't avoid tagging the type.
We could have a longer name for dart.fn like dart.anonymous. We could give the function a name like anonymous. We could see that the anonymous function is an argument to the Dart function named test, and use that name ... we won't have names like that in general, but we could find them in this example. I don't know if that's actually more helpful, though?
All of these will increase code size.
In our new debugger prototype, we can detect and render anonymous Dart functions differently in the web inspector. So that's an idea.
I'm not sure what (if anything) to do here.

@alan-knight this may be fixable now in the new Debugger UI?

We wouldn't be able to do anything about printed stack traces there, but we could tweak the stack frame display in the debugger. It's not obvious to me why the function is getting the dynamically containing function name prepended to it. I'd expect to see something with the statically containing scope. But I think it's basically happening here https://cs.chromium.org/chromium/src/third_party/blink/renderer/devtools/front_end/sources/CallStackSidebarPane.js?rcl=3df5254f608bfec7ee6407faed8c7232b935f318&l=256
but it might be some work to get the information we'd need to name it appropriately. We can get the containing file and line number, but more than that I don't know.


Create dataset loader for IndoNLU KEPS

Dataset name: IndoNLU KEPS
Dataset description: This keyphrase extraction dataset consists of text from Twitter discussing banking products and services and is written in the Indonesian language. A phrase containing important information is considered a keyphrase. Text may contain one or more keyphrases since important phrases can be located at different positions. The dataset follows the IOB chunking format, which represents the position of the keyphrase.
Dataset URL: https://huggingface.co/datasets/indonlu
Dataset Publication: IndoNLU: Benchmark and Resources for Evaluating Indonesian Natural Language Understanding
Task: Span extraction for keywords
Languages: ind
Dialect/Style (optional): Colloquial
Domain: Banking
Dataset Size: 800 train, 200 dev, 247 test
License: MIT
Is Synthetic: No


Any plans to do examples for other frameworks (eg. Blazor)?

Thanks for this initiative it is needed for faster adoption.
Instead of everyone doing the bulk work of creating models and endpoints it would help a lot with examples for each framework. Im currently working with .Net Blazor and interested in adding web3 as a sign-in option.

Hey @vindberg  - we absolutely plan on doing languages with associated examples for each framework! We recently released a bit more info on the planned languages / items in a blog post:
https://blog.spruceid.com/sign-in-with-ethereum-support-updates/

Great to hear c# is on the list. I will keep an eye on this repo, however do you have a timeline? Is funding secured?

Hey - at the moment we're wrapping up work on Rust, Ruby, and Elixir, with plans to go after a number of those languages next (likely with announcements as well) as to that progress.
In terms of C# specifically, we're also collaborating with some folks from the Nethereum side that have a WIP implementation to get this moving!
https://github.com/jaerith/siwe-nethereum

Thanks for the link - great to see the Nethereum guys are doing work on this. Maybe I can even assist.


LSI-21: remove binary files

https://jira.hyperledger.org/projects/LSI/issues/LSI-21

Ping @mrZizik , could you help us out please?


Update & refractoring

Cleanup of the source code, fix for cosmic upgrade, + mods by Midnight-Steam.
Works on Mac, Win not tested.

Edit: I've accidentally placed the Assembly-CSharp in the Managed folder, please delete it before release. Thanks.


Remove redundant URL initializer call from README

All these function calls take URLConvertible as an argument, and String is also URLConvertible. Thus URL(string:) initializer call is redundant and has a negative impact on readability of the example code in README.md.

thanks


Updating issues, licensing, Readme

Fixes #10 , #11
Many of the hardcoded paths in issue #11 depend on user setup of other tools. These paths have just been commented to indicate the user needs to update to be compliant with their directory setup.
Readme has been updated to reference new documentation and refer to BAG as Berkeley Analog Generator
Tests are failing because bag_prim.cdl and bag_prim.scs are missing the copyright headers. However, these files are generated by scripts, so this test should be removed.

This branch has conflicts that must be resolved

Changes generally look okay. Will add comments when the conflict is resolved.

@FeliciaG Could you do a rebase rather than a merge?

Looks like you still did a merge rather than a rebase....
You want to do something like;
git remote add upstream git@github.com:google/skywater-pdk-libs-sky130_bag3_pr.git
git fetch upstream
git rebase --interactive upstream/master


Evidence View - Interactive legend to filter by evidence type

It would be nice to have a filter functionality in Evidence View (same as we now have in Gene View).
Click on legend symbols should retain rows with that symbol TYPE and filters rest. No need for multiple selections.

Done in release 4.0


Next 15 + Repo refresh

Closes #
✅ Checklist

[ ] I have followed every step in the contributing guide (updated 2022-10-06).
[ ] The PR title follows the convention we established conventional-commit
[ ] I performed a functional test on my final commit

Changelog
[Short description of what has changed]

Screenshots
[Screenshots]
💯

Still working on it

I need help with the repo's linting as running pnpm lint keeps hanging. I suspect it's because each project now requires its own eslint.config.js if we want a lint script for each one but I'm not sure.

@juliusmarminge would you kindly review + help with linting (since it's opinionated). I think we're mostly finished here.

@juliusmarminge would you kindly review + help with linting (since it's opinionated). I think we're mostly finished here.

so this is making changes both to the CLI and the overall repo? any chance you could split it to 2 ? 1 for the repo refresh and 1 for the CLI updates?

so this is making changes both to the CLI and the overall repo? any chance you could split it to 2 ? 1 for the repo refresh and 1 for the CLI updates?

I could probably make it into even more PRs for the docs and update projects and eslint v9 depending on how integrated they are

Is it adding support for the upgrading existing t3 app to next15


boards: stm32l562e_dk: add support for touchscreen

The stm32l562e_dk board uses a ft6x06 i2c controller for the touchscreen. The zephyr driver ft5336 can control it.

@erwango I add section for screen and touchpanel

@lucbeaufils please rebase to resolve merge conflict

@lucbeaufils please rebase to resolve merge conflict

@kartben done


cmd/compile: unexpected performance difference accessing slices with different caps

Using tip (go version devel +b794ca64d2 Mon Sep 3 07:14:25 2018 +0000 linux/amd64)
While implementing some bounds check optimizations in image/draw CL 136935 I came across an unexpected performance difference between accessing elements of a slice s created like this
s := spix[i : i+4 : len(spix)]

and
s := spix[i : i+4 : i+4]

Both forms eliminate bound checks for accessing elements 0-3 of s but the second form has a significant performance improvement over the first (see CL for benchmarks).
Building with go build -gcflags="-d=ssa/check_bce/debug=1" shows no difference in bounds checks between the two forms so presumably it has something to do with the resulting size of the slice.
Attached is a simple program and disassembly derived from the image/draw code that demonstrates the effect. You can see that the assembly generated for the second form is quite a bit shorter.
bounds.go.txt
bounds.asm.txt

Go has a special case when slicing and generating the empty slice.
This has the potential to generate a pointer to the next object in memory, like this:
x := make([]int, 4)
x = x[4:]

The resulting x has capacity 0, and if we just computed its base pointer as x.ptr += 4*sizeof(int) then x.ptr would point to the next object in memory after the [4]int that was allocated. This is bad because it could mistakenly retain a random object in the heap.
So when slicing, we need to somehow avoid contructing such a pointer. Currently we do this by avoiding the add when the capacity is 0.  So we do x.ptr += newcap == 0 ? 0 : 4*sizeof(int).  And we do it without a conditional, using shift and mask tricks.
When you do spix[i:i+4:i+4] the new capacity is 0, and the compiler knows that.  So the conditional in the expression above constant folds, and we just get x.ptr += 0.  When you do spix[i:i+4:len(spix)], it's not obvious whether the resulting slice has capacity 0 or not, so the arithmetic is still all there.
The optimization that would fix this is that if the resulting slice is known to have nonzero length (we know that here because i+4-i > 0), we don't need to guard against next object pointers.

s/the new capacity is 0/the new capacity is 4/
s/x.ptr += 0/x.ptr += i*sizeof(int)/

@randall77

So when slicing, we need to somehow avoid contructing such a pointer. Currently we do this by avoiding the add when the capacity is 0. So we do x.ptr += newcap == 0 ? 0 : 4*sizeof(int). And we do it without a conditional, using shift and mask tricks.

I wonder if we should revisit using 'shift and mask tricks' here. It reminds me a bit of #26306 (i.e. the pointer arithmetic might be interfering with speculative loads). Since setting the pointer to nil only usually happens on the last iteration of a loop, perhaps we should just emit the obvious code instead:
if likely(cap(x)-i > 0) {
     x = x[i:]
} else {
     x = nil
}

Might be worth an experiment. Plus prove wouldn't need a special case to optimize the branch away.

I'm happy for someone to experiment with this.
The special case here is going to be ugly either way, I expect.

I wonder if we should revisit using 'shift and mask tricks' here.

Aside: There are also a few other alternatives (of the pointer arithmetic tricks) variety here.
We currently do:
SUBQ k, i
NEGQ i
SARQ $63, i
ANDQ i, off

We could also do:
SUBQ k, i
SETNE x
NEGL x
ANDQ x, off

Or even:
SUBQ k, i
SETNE i
MULQ i, off

I hacked these in locally. They all execute at basically identical speeds, but the middle form encodes shortest.
To use the middle form throughout, in practice we'd need to have a OpReslice that can pass all the way through to lowering. (This is because we treat SUBQ and SUBQborrow differently, which inhibits CSE. And because other architectures might want to use their own specialized slicing tricks, e.g. conditional instructions on ARM.)
A dedicated OpReslice might make the prove pass simpler as well, since (I believe) the prove pass doesn't care whether the resulting pointer is at the beginning or in the middle of the object.

(Or another variant of the middle form: SUBQ; SETEQ; DECQ; ANDQ, which might be even shorter. I'll explore if folks think it is worth it.)

SETNE only sets the low byte of the target register. You'd need to add an XORL x, x before the subtraction or MOVL $0, AX before the SETNE.

Thanks! It still appears to be a minor win. The bigger question is about moving slicing from SSA gen to a dedicated op.

A dedicated op is fine. It should make it easier to generate better code for arm as well.

we'd need to have a OpReslice

One thing I was wondering is whether we should subtract the length from the capacity when decomposing slices in SSA. That way we wouldn't need to update the capacity when re-slicing. It would mean slightly more work when doing append-like operations though. We'd also need to re-add the length to the capacity when storing the slice back to memory.
For example:
for len(x) > 0 {
    ...
    x = x[n:]
}

Currently compiles to:
for x.len > 0 {
    ...
    x.cap += n
    x.len += n
    x.ptr += SliceMask(n, cap)
}

If we instead replaced the capacity (x.cap) with the length subtracted from the length (x.ext) then the loop could be done as:
x.ext = x.cap - x.len
for x.len > 0 {
    x.len += n
    t := n
    if unlikely(x.len == 0) {
        t = SliceMask(n, x.ext)
    }
    x.ptr += t
}

This involves a branch but crucially means we don't have to load/update the capacity in the likely path of the loop, reducing register pressure. I started hacking on this idea but haven't finished it yet - seems to be relatively simple to achieve though.
Anyway, just thought I'd bring this up since I'm not sure if this idea would compliment or conflict with the OpReslice idea.

One thing I was wondering is whether we should subtract the length from the capacity when decomposing slices in SSA.

Interesting. How often do we slice in a loop as opposed to straight line code? Seems worth exploring.
Another, crazier, idea is to change the underlying representation in memory of all slices to be {ptr, len, cap-len/extra/slack}. This would make calling cap involve addition, but I think that calling cap is much rarer than slicing, appending, etc. This would probably break so much code as to be impossible, though.

Anyway, just thought I'd bring this up since I'm not sure if this idea would compliment or conflict with the OpReslice idea.

Conflict, I think. The idea behind OpReslice is to bundle a bunch of these calculations into a single op (easier to deal with during prove, etc.) and do the complex decomposition on a per-arch basis. I'm quite content to let you experiment with your ideas first, since they seem quite promising. And if you end up not liking the results, we can experiment some with OpReslice etc.

One argument in favor of writing out the if statement: For code like this
if lo < hi {
  s = s[lo:hi]
}

Then we can entirely avoid adjusting ptr, whereas with masking tricks we always end up masking. (We could add a special check for boundedness, like we do with shifts, and perhaps should, but with a branch, it falls out for free.) Code example from discussion at https://go-review.googlesource.com/c/go/+/169518

@mundaym if you want a real life test case, the code in https://github.com/golang/go/issues/31586#issuecomment-486469682 spends about 50% of its time reslicing in a loop. TL;DR: go get nhooyr.io/websocket, git checkout 40b4, go test -bench=Mask/4096//nhooyr, line 74 (b = b[8:]) is >50% of execution time. (cc @nhooyr)

So when slicing, we need to somehow avoid contructing such a pointer. Currently we do this by avoiding the add when the capacity is 0. So we do x.ptr += newcap == 0 ? 0 : 4*sizeof(int). And we do it without a conditional, using shift and mask tricks.

When you do spix[i:i+4:i+4] the new capacity is 4, and the compiler knows that. So the conditional in the expression above constant folds, and we just get x.ptr += i*sizeof(int). When you do spix[i:i+4:len(spix)], it's not obvious whether the resulting slice has capacity 0 or not, so the arithmetic is still all there.

I'm not very understanding why spix[i:i+4:len(spix)] is not obvious that the resulting slice has a capacity larger than zero. At least it is obvious for humans.

It looks the performances of the two ways are on par now with Go 1.21.2. Any change has ever made for this?


Front theme and modes

Modify the frontend theme to match the figma
Refactored some design
Added the mode choices

preview

LGTM


[SVLS-5713] Correctly set log source in sidecar mode

What does this PR do?
Correctly sets the log source when using sidecar mode of serverless-init.

Motivation
Previously the source was incorrectly set as undefined.

Describe how to test/QA your changes
Live tests using an Azure App Service app with a sidecar.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Results Extraction

Success Criteria

Convert TIMDEX response into Facets, Records, Pagination blocks

Out of scope

Error handling (we'll handle that in Analyzers maybe?)

Note: Initial work may focus just on Records and leave Facets and Pagination to follow-on work if it makes sense.


Magic Mirror does not start at all!

I installed this module as described, and added it to my (normally) working modulearray. Since then my magic mirror wont start at all.

Same problem here ... pleas Check the Module.
Thanks and best regards
René

Same here


Require Pass -iframework Options to C/C++ Extension

See this issue:
https://github.com/microsoft/vscode-cpptools/issues/3323#issuecomment-614446896
macOS needs iframework option to help head files search, but with "configurationProvider": "vector-of-bool.cmake-tools" ,cmake tool doesn't pass this option.
I can only remove "configurationProvider": "vector-of-bool.cmake-tools", and add "compileCommands": "${workspaceFolder}/build/compile_commands.json" to work around.

What are the implications of the workaround (using compileCommands instead of configurationProvider)? Does this affect the extension's functionality at all?

CMake Tools functionality is not affected at all, you can use this extension the same. What will be different is that with compile_commands.json, depending the C/C++ source code complexity and the operations you are running on what symbols, IntelliSense may work less accurate than when configurationProvider is used, because the las one is designed to store more information that CppTools can use. But depending the source code and operations performed on it, they may both work the same.
Another advantage for using configurationProvider is that it updates automatically when you change any CMake configure parameter (kit, variant, etc...). With compile_commands.json, you have to make sure you have an up-to-date one. Various settings or events may inhibit the creation of compile_commands. But you seem pretty experienced with it so I think you'll know how to ensure compile_commands.json is used properly here.
I think this workaround should be good enough for now. Let us know if you experience any inconvenience because of this.

I encountered the same issue. The workaround works except for the other problem - I had to set "cppStandard": "c++17" in c_cpp_properties.json despite the fact the "-std=c++17" flag is present in "compile_commands.json". Without the flag all Qt headers are parsed incorrectly.
It would be really great if cmake-tools support frameworks on Mac correctly.
Thanks in advance!

... still not working in 2023, the compile_commands workaround works, but it's something that cannot really be pushed to a repo (changing the build directory for instance will break it and it does not work with cppStandard and other cmake set variables)...

CMake doesn't provide us the information we need to make it work.  There's an issue open that you can go upvote.  https://gitlab.kitware.com/cmake/cmake/-/issues/19897.
I'm going to close this as a duplicate of #2324 where this was also discussed.


Blockchain syncing from scratch fails due to ExecuteStateTransitionFunction error

Scenario:

Node_A at round 0 (genesis block) starts syncing with the network with tip 1248 round
Noda_A receives block 1 as per the sync procedure
Node_A fails to accept the block due to the error from below
As a result Node_A gets stuck and could not continue the sync procedure

DEBU[0000] calling ExecuteStateTransitionFunction        height=1 process="accept block"
DEBU[0000] Synchronize                                   height=1248 process=synchronizer
ERRO[0000] Error in executing the state transition       error="rpc error: code = Unknown desc = last seen block does not precede provided block" height=1 process="accept block"
DEBU[0000] could not AcceptBlock                         error="rpc error: code = Unknown desc = last seen block does not precede provided block" height=1 process=onAcceptBlock
WARN[0000] Handling block failed                         error="rpc error: code = Unknown desc = last seen block does not precede provided block" process=chain topic=block

Fixed via #661


New-AzureRmSqlDatabaseimport, missing values

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem (if available).
Link to article:
https://docs.microsoft.com/en-us/powershell/module/azurerm.sql/new-azurermsqldatabaseimport?view=azurermps-6.13.0
Problem:
-Edition parameter description value list don't have all the possibilities. At least "GeneralPurpose" is missing.
-ServiceObjectiveName parameter should have table (or at least link to the table) of possible combinations against -Edition values.

@JJussi Please move this issue to https://github.com/Azure/azure-powershell/issues

The AzureRM module has been replaced by the Az module


Change design of Channel write subscribers

Description
Changes how AsynchronousByteChannel write subscribers handle writing to the channel to improve consistency and reliability.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


update assign admin xpath

update assign admin xpath

Coverage remained the same at 56.678% when pulling a0e9a04a43e9a51923d157c0863a17466a5849e3 on sigsbee:drone2group8 into 1bffe220eec5ec731b2ad39a5cc9a21b2c29110f on vmware:drone.


Add trrosetta dataset

include (id, seq, msa, coords, angles, dist, mask, msa_mask)

download and extract trrosetta dataset from s3
read msa from a3m file
read pdb file using prody, process using sidechainnet
calculate bucketed distance based on ca or cb
optional pytorch lightning data module

caveats

the performance is not good :disappointed:
not fully tested :hushed:

Oh this looks very nice! Very cool functionality! I think this covers until CASP13 or more? If so, this is great: more data!
Also, are the MSAs in trRosetta processed somehow?
What do you mean by "the performance is not good"? Idk if we should leave it as a branch until we can fix the possible errors/caveats you mentioned or merge it into main... I'd like to hear @lucidrains ' opinion as well here

@blazingsiyan this is tremendous! thank you so much :) this will help polish up the trunk. there's no way we can't beat trRosetta, at the very least :)

Oh this looks very nice! Very cool functionality! I think this covers until CASP13 or more? If so, this is great: more data!
Also, are the MSAs in trRosetta processed somehow?
What do you mean by "the performance is not good"? Idk if we should leave it as a branch until we can fix the possible errors/caveats you mentioned or merge it into main... I'd like to hear @lucidrains ' opinion as well here

Sorry for the late response :stuck_out_tongue:
By "performance" I mean the read/cache/collate cost of the dataloader and I think there is room for improvement.
The MSAs are the same as in the original paper.
Ideally, we should have the sequence DB, then use HHblits/JackHMMER to generate the MSAs
but there will be terabytes of data and will take several weeks for searching.
Before achieving that, I think we should use some existing datasets to validate our model :blush:

Oh yes! I can help with performance, dont worry!
As for the MSA... we have the MSA data up until CASP12 (from proteinNet), so we can do several tests with that for now (although it would be ideal if we can have the newest MSA data once we have the model ready for training)


ARROW-630: [C++] Create boolean batches for IPC testing, properly account for nonzero offset

This fixes a couple bugs; boolean IPC was not being tested directly like the other types (it was implicitly by integration tests, though)

+1


[Macros] Don't apply member attribute macros to expanded members.

The attached macro proposal states:

member attribute macros allow one to modify the member declarations that were explicitly written within the type or extension by adding new attributes to them

The implementation did not follow this, because it applied member attributes to other macro-expanded members. This change aligns the implementation with the proposal.
Resolves rdar://109217961

@swift-ci please smoke test

@swift-ci please smoke test


Pub 487

JIRA link (if applicable)
Change description
Based on HMI API Gateway testing framework.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

@ShufenZ Responses in bold
build.gradle:
line 14: Need to check out version '0.0.1' is correct?
Version number is inherited from template project, Not something I setup would be best to raise with Milan and Subruto
task for functional test:
line 88 repeats line 81
line 89 repeats line 82
Sorted thanks
task for acceptance test are identical except that there are no repeated tasks as functional. So there are differences between them.
There are some minor differences, could you please explain further?
Suggestion: make the tasks in the same order as those for functional test.
Done
task for integration has fewer tasks than functional and acceptance.
e.g.
Do we need to set assertion flag?
Do we need to set scan test classes flag?
This is inherited from the template, I have not added this code
line 128, missing sourceSets.acceptanceTest
Not included in HMI so will leave as is
block of definition for
integrationTestImplementation group:
is missing but testImplementation group:
Inherited from template. I’ve added Functional and Acceptance, not sure we will need Integration tests as not used on HMI
Suggestion:
Move lines 257 and 258 to between lines 246 and 248 for consistence
Done
It seems the following is missing below line 269:
smokeTestImplementation sourceSets.test.runtimeClasspath
Inherited from template I’ve added Functional and Acceptance, not smoke tests at this time.
src/acceptanceTest/java/uk/gov/hmcts/futurehearings/pip/acceptance/common/delegate/CommonDelegateImpl.java:
Suggestion:
line 42, remove final qualifier from private static method.
Done
src/acceptanceTest/java/uk/gov/hmcts/futurehearings/pip/acceptance/common/header/dto/factory/PayloadHeaderDTOFactory.java:
Suggestion:
line 18 and 32, remove final qualifier from static methods.
Done
src/acceptanceTest/java/uk/gov/hmcts/futurehearings/pip/acceptance/common/helper/CommonHeaderHelper.java:
Suggestions:
line 12, remove final qualifier from static method.
Done
src/acceptanceTest/java/uk/gov/hmcts/futurehearings/pip/acceptance/common/security/OAuthTokenGenerator.java:
redundant Exception throw clause
Removed
Suggestions:
lines 13 and 30, remove final qualifier from static methods.
Removed
src/acceptanceTest/java/uk/gov/hmcts/futurehearings/pip/acceptance/common/test/PIPCommonTest.java:
Incorrect signature for the 4 @ annotated method. Make them static.
Changed
Inconsistent styles of using switch between tow files:
src/acceptanceTest/java/uk/gov/hmcts/futurehearings/pip/acceptance/common/RestClientTemplate.java
and
src/functionalTest/java/uk/gov/hmcts/futurehearings/pip/functional/common/rest/RestClientTemplate.java
Updated
src/functionalTest/java/uk/gov/hmcts/futurehearings/pip/functional/common/security/OAuthTokenGenerator.java:
See the review comments for OAuthTokenGenerator.java for acceptanceTest
Done
src/functionalTest/java/uk/gov/hmcts/futurehearings/pip/functional/common/test/FunctionalTest.java:
Make @BeforeAll annotated method static.
Cannot be static as not static fields are referenced


Set ipaddress undef by default

Otherwise you will always get this error when using bind...
Error: Could not retrieve catalog from remote server: Error 400 on SERVER: The use of $ipaddress and $bind is mutually exclusive, please choose either one at /modules/haproxy/manifests/listen.pp:99
... unless you set ipaddress => "", on your manifest, which does not make sense to me.

I think you forgot to pull the master branch of puppetlabs-haproxy into your feature branch, because this has been fixed in master quite some time ago :) Not just for haproxy::listen but also for haproxy::frontend, which suffered from the same problem. See #146 for details.

Thanks Andreas :)

Closing since it was fixed in #146


bump architect-orb and push to OCI

Towards https://github.com/giantswarm/giantswarm/issues/20866

:face_exhaling:


Automatic plugin install

When running the vue invoke <name> command, vue-cli could automatically install the @vue/cli-plugin-<name> or vue-cli-plugin-<name>, using the user prefered tool (npm or yarn).
If <name> contains @vue/cli-plugin or vue-cli-plugin, use npm install <name> directly.
Then run the real invoke.
Example:
vue create my-app
cd my-app
vue invoke apollo

I kinda hope it's possible to explicitly separate the install and invocation steps.
Maybe instead we can have a different command vue add which combines both.

Yeah. I was noticing tonight when I tried to add the just plugin to a project just by installing it it didn't automatically make the config updates. I just created an example with vue create with the plugin that I wanted and manually moved over the config.
Having a command like vue add to both install and configure the plugin would be nice.


Odd bramble color behavior at a distance

What Happened?
No matter what fog tint and light tint you enter, the fog shows up as the default white until you get pretty close. Once you get close enough to change it to what it should be, it will fix itself and not revert to white for the rest of the loop, no matter your distance.
What was supposed to happen?
The fog should display the fog tint color at nearly any distance
Platform
Steam
Mods
Bramble Config (inside node module):
"name": "Dusty Exterior",
"fogTint": {
	"r": 100,
	"g": 90,
	"b": 52
},
"lightTint": {
       "r": 100,
       "g": 90,
       "b": 52,
       "a": 40
},
"linksTo": "MainDimension",
"scale": 8,
"hasFogLight": false

Logs
No log errors.

sorry trifid


Update theme-manager.js

Add function to modify the zIndex of the raw theme

Good feature, thanks a lot :+1: :tada: . Also, please add this to the documentation here as well.

Please wait a little before fixing that

I want to hear @oliviertassinari's insight on this. maybe modifyRawThemeZIndex is a more consistent name. Thanks for working on this :+1: and don't for the docs :grin:

@subjectix I guess it's time that I implement the solution I have proposed here https://github.com/callemall/material-ui/issues/2460.
This would make the modifyRawThemeZIndices, modifyRawThemeSpacing, modifyRawThemePalette unnecessary.

@oliviertassinari Hmmmm, you might be right. But these are only helper methods. and we should keep them around for a while. I'd say let's merge this and keep these functions around for a little while before we do a full rewrite and deprecate them all together.

I'd say let's merge this and keep these functions around for a little while before we do a full rewrite and deprecate them all together.

Sounds like a plan.

@oliviertassinari Thanks :+1: :+1:
@woshiwenzhijie Squash down your commits and we'll merge this. :+1: :+1:

:grin:

@woshiwenzhijie As it's your first contribution, I'm gonna merge this PR even those we prefer to have a single commit to have a cleaner history. Thanks.

@woshiwenzhijie Thanks :+1: :tada:


Very slow speed during image generation with 512x512 image

am i missing some optimisations or is the image generation supposed to be that slow? automatic1111 local runs with 1-2 s/it for me locally on this m1 14inch mbpro... hope you can help me :)
* Running on all addresses (<IP_ADDRESS>)
 * Running on http://<IP_ADDRESS>:1337
 * Running on http://<IP_ADDRESS>:1337
Press CTRL+C to quit
/Users/bamboozle/opt/miniconda3/envs/sd-api-server/lib/python3.9/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion_img2img.py:249: UserWarning: The operator 'aten::repeat_interleave.self_int' is not currently supported on the MPS backend and will fall back to run on the CPU. This may have performance implications. (Triggered internally at /Users/runner/work/_temp/anaconda/conda-bld/pytorch_1665904452771/work/aten/src/ATen/mps/MPSFallback.mm:11.)
  text_embeddings = text_embeddings.repeat_interleave(num_images_per_prompt, dim=0)
  8%|████████                                                                                | 1/13 [00:23<04:46, 23.91s/it]

Tried to speed up the generation by changing the installation process a bit:
didnt work - still very slow it/s (s. below)
Remove torch and all related packages
pip uninstall torch torchvision torchaudio -y
Normally, we would install the latest nightly build of PyTorch here,
But there's currently a performance regression in the latest nightly releases.
Therefore, we're going to use this old version which doesn't have it.
TODO: go back once fixed on PyTorch side
pip install --pre torch==1.13.0.dev20220922 torchvision -f https://download.pytorch.org/whl/nightly/cpu/torch_nightly.html --no-deps
Activate the MPS_FALLBACK conda environment variable
conda env config vars set PYTORCH_ENABLE_MPS_FALLBACK=1

(sd-api-server) ➜  stable-diffusion-api-server-main python server.py
Fetching 16 files: 100%|█████████████████████| 16/16 [00:00<00:00, 21454.24it/s]
ftfy or spacy is not installed using BERT BasicTokenizer instead of ftfy.
 * Serving Flask app 'server'
 * Debug mode: off
WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
 * Running on all addresses (<IP_ADDRESS>)
 * Running on http://<IP_ADDRESS>:1337
 * Running on http://<IP_ADDRESS>:1337
Press CTRL+C to quit
/Users/bamboozle/opt/miniconda3/envs/sd-api-server/lib/python3.9/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion_img2img.py:249: UserWarning: The operator 'aten::repeat_interleave.self_int' is not currently supported on the MPS backend and will fall back to run on the CPU. This may have performance implications. (Triggered internally at /Users/runner/work/pytorch/pytorch/pytorch/aten/src/ATen/mps/MPSFallback.mm:11.)
  text_embeddings = text_embeddings.repeat_interleave(num_images_per_prompt, dim=0)
 50%|██████████████████████                      | 7/14 [02:39<02:45, 23.67s/it]

any help would be highly appreciated!

Hi, ran into the same problem. Have a maxed out M1 Max 16" and one iteration seems to take around 35s/it... didn't run in any issues while setting things up. But going off the activity monitor, it seems that the GPU isn't utilised at all. Any idea how to force PyTorch to use the GPU?


Training on C+T

Thanks for your impressive works.
I use your codes and run command
./run_train.sh

When it comes to train model on flyingthings, I find the performance on Sintel(train) is much better than your results provided in paper's Table 1. The only difference is that I change the batch size to 4 due to the memory limit. The logs are shown in below:

The experiment name of the blue curve is: things/latentcostformer/cost_heads_num[1]vert_c_dim[64]cnet[twins]pretrain[True]add_flow_token[True]encoder_depth[3]gma[GMA]cost_encoder_res[True] (08_19_21_07), and the orange curve is the logs of training on flyingchairs: chairs/latentcostformer/cost_heads_num[1]vert_c_dim[64]cnet[twins]pretrain[True]add_flow_token[True]encoder_depth[3]gma[True]cost_encoder_res[True]arxiv2(08_17_16_47).
So, I want to know why the performance is different from the one reported in the paper, is there any difference of codes?
Looking forward to your kind reply!

Thanks. We directly inherent hyper parameters from GMA. We will try to train our FlowFormer with batchsize 4 and reply to you when getting the results.

Thank you for your quick reply.
When training on FlyingChairs, I use the default setting and batchsize 8. The batchsize of FlyingThings is 4, I think just train the model on FlyingThings, initialized from FlyingChairs' model, for several steps can tell the difference.
Looking forward to your early response，thank you again.

We find that the issue comes from the tile technique. Directly estimating flow instead of tiling the flows largely improves the flow accuracy. Using the FlowFormer trained on C+T:

Sintel training (clean)
Sintel training (final)

GMA
1.30
2.74

FlowFormer with tile
0.95
2.35

FlowFormer w/o tile
0.64
1.50

The numbers reported in the paper are estimated with tiling. The error reduction compared with GMA comes to 51% and 45%:yum:. Many thanks:star_struck::star_struck:! We will update the paper and the repository ASAP.

What is tile technique?

We refer to the tile technique introduced by PerceiverIO. The details are presented in our supplementary.

I will close this issue now. Feel free to re-open it if you still have questions.

I'm sorry to say that there is a bug in validation without tiling as pointed out by Dr. Sun :face_with_head_bandage:.  The new numbers have been updated in README.


bug: Clicking ionicActionSheet clicks textarea behind it

Type: bug
Platform: ios 7 webview
If you click the $ionicActionSheet and a textarea is behind where you clicked, the keyboard appears and the textarea doesn't have focus.
To repro:

Create a blank Ionic starter project.

In index.html, make the body the following:

Hello World

Click this to show an action sheet.
This pink area is a textarea that covers the screen.

Click the link to show an action sheet.

Click the Cancel button in a spot that is on the footer (i.e., not on the textarea). The action sheet disappears and no keyboard appears, as expected.

Click the link to show the action sheet.

Click the A button, or any button on a spot that is not on the footer (i.e., on the textarea).

// Expected: The keyboard does not appear.
// Actual: The keyboard appears, and the textarea does not have focus.

I'm facing this issue, but in a very specific scenario, it only happens when I tap it the very FIRST time and only in iOS. To simulate that, I need to reinstall the app or reboot the iphone.
Do you guys have some thoughts about what I could try to fix it?
I've been played around with it for a while, but didn't find anything.
Thanks in advance!

Same issue here, it only happens when I tap it the very FIRST time and only in iOS.


Investigate/fix general fluentd issues

Describe the bug
Fluentd regularly has problems in various clusters, as well as in the pipeline.

In clusters, the following error pops up:

{"time":"2021-08-20 13:32:28 +0000","level":"warn","message":"[output-forwarding] failed to write data into buffer by buffer overflow action=:block","worker_id":0}

The error can usually be fixed by increasing the limits for the forwarders/aggregator until the Pods are ready, and then lowering the limits again.

The pipeline fails somewhat often during the apps sc test due to fluentd not being ready. This may be due to the test being performed before the fluentd secret is ready.

To Reproduce
Not sure if it can be consistently reproduced.
Expected behavior
Fluentd runs without problems.
Screenshots
Version (add all relevant versions):

Compliant kubernetes apps version 0.17.0

Additional context

I think this is fixed with the rework and exposure of the buffer config for the aggregator.

@aarnq close?


Fix Distinct Filter on ICAT Backend

This PR will close #141
Description
These changes make the distinct filter work on the ICAT backend, returning actual distinct results, not results that are repeated (the previous behaviour on this backend).
The previous implementation used WHERE filters (filtering on columns as per the distinct filter with != null) to filter out empty columns. The results were returned from Python ICAT and the API picked out the columns requested on the distinct filter. The new implementation uses the setAttributes() function in Python ICAT to select only the columns requested in the distinct filter (similar to SELECT colA, colB, colC in SQL). The DISTINCT keyword is also applied to the query.
When a distinct filter is used in a GET request (except for a count request), entity_to_dict() is not used, but a new function has been created to process the results from Python ICAT to a format suitable to be returned to the user (called map_distinct_attributes_to_results()). entity_to_dict() assumes a data input of an ICAT entity object, a format which is not returned by Python ICAT when setAttributes() is used in the query (a list is returned instead).
Two boolean flags have been created for the query, so the query class can deal with edge cases:

manual_count - this flag is used when a DISTINCT keyword is used on a count endpoint (so a distinct filter on a count request, but also the count ISIS endpoints). JPQL cannot handle the DISTINCT and COUNT keywords in the same query, so a normal (non-count) distinct query is sent to ICAT and the API performs len() on the result to get the count. I've written a comment in abb31bd0c36a809146484a78c8c6d9864fdf561a so future developers understand why this is required
isis_endpoint - this flag is used to distinguish ISIS endpoints from standard endpoints. This is required as these endpoints use the DISTINCT keyword but don't select multiple attributes. This flag is used to divert the results to entity_to_dict() (correct function to be used) instead of map_distinct_attributes_to_results() (incorrect). Commit 504e94feef9a608796197e1ae2ac9b67019ec843.

I've removed some logic that checks if related entities on a distinct filter have an associated include filter as I've realised Python ICAT is smart enough to include JOINs in the JPQL queries it constructs so mandating this is unnecessary really (`1530ca69b3f892349eeb2ba4ed9df94cce441d34).
I've removed the old distinct field functions at 0322a99c4738a356cbec89f65eaccf5a89c918fc and 677424d8bdf80d65ea6d8ce425f33fde995ee96d.
I've also added some test cases to test all these changes. Tests that use the flask test app, but also proper unit tests which test a single function. And of course I've removed the tests for the functions that are no longer used.
Testing Instructions
The API now uses 0.18.0 of Python ICAT which implements changes in being able to select multiple attributes (required for this work). For the distinct filter, 0.18.1 of Python ICAT is required, but isn't currently released. As a result, some unit tests (3 I think?) will fail on CI until it gets released. For the time being, I have manually installed Python ICAT using the GitHub branch for the upcoming release (pip install git+https://github.com/icatproject/python-icat.git@release/0.18.1). Pip might complain about version UNKNOWN not matching 0.18.0 (as mandated by Poetry) but ignore this, the installation will have been applied. When 0.18.1 is released, I will update pyproject.toml to reflect this.
Distinct filters should be able to get applied on GET requests. Applying a distinct filter on a count endpoint should also have an impact the number of results returned. These cases are covered in test/ but a good manual test case can be found in a comment on the original issue.

[ ] Review code
[ ] Check GitHub Actions build
[ ] Review changes to test coverage
[ ] Ensure distinct filter behaviour works as expected

Agile Board Tracking
Connect to #141

0.18.1 of Python ICAT has been released so I've updated the API's dependencies to reflect that. It should be noted, ICAT Server 4.11.1 is required as a result.

@louise-davies
As mentioned in sprint planning yesterday, when you have a DISTINCT filter with an include filter, there are issues. The only case where this is actually used in the frontend is on card view, where you can filter by type and facility ID. I've fixed this by just removing the include filter on those requests (see: https://github.com/ral-facilities/datagateway/commit/b2374ceaea49e84b36cd60c6b1efab8e827a914e) - Python ICAT is smart enough to automatically add a JOIN to the JPQL query.
I've just been using /entityManager to investigate if it's a bug on the API side, or if it's a JPQL limitation. It is a JPQL limitation as proven by the following query:
SELECT DISTINCT o.fullName, o.email FROM User o INCLUDE o.userGroups

The entity manager will return:
{
    "code": "BAD_PARAMETER",
    "message": "INCLUDE is only valid for one quantity in the SELECT clause"
}

An alternative test query:
SELECT DISTINCT o.fullName FROM User o INCLUDE o.userGroups

Result:
{
    "code": "BAD_PARAMETER",
    "message": "variable O mentioned in INCLUDE clause is not defined"
}

I'm happy this is a JPQL limitation so please feel free to start reviewing this PR, I have no further pre-review changes to make.


Elasticsearch/Open Distro as backend

Is it possible to use opendistro elastic as a backend?

Hi @raghu999 thanks for opening up our first community question.
Open Distro support with an exporter is in the works. Keep a lookout on our roadmap at https://github.com/orgs/aws-observability/projects/4
You may have already found the upstream exporter at https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/master/exporter/elasticexporter


Bug 2091760: The provider should be empty if it's not set during cloning

📝 Description
cloned template gets the provider-display-name annotation from the parent template and gives the wrong provider data.
🎥 Demo
before:
https://user-images.githubusercontent.com/67270715/171152926-dbe9738c-7b0a-4c45-b45a-3dd452f7d81b.mp4
after:
https://user-images.githubusercontent.com/67270715/171152953-445da633-b71e-48c1-ae9f-507a4ff1264f.mp4
Signed-off-by: Aviv Turgeman<EMAIL_ADDRESS>

/lgtm

/lgtm

/lgtm


Add Article for Hello World in Groovy

Merged the script #20. Never wrote the article.

Added link to README: https://github.com/jrg94/sample-programs/commit/94ed98c03705fd65343797199df612bdd4491ee1


[ENH]: OpenAI embedding dimensions

Closes #2576
Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

adding dimensions parameter to OpenAI EF as requested by a community user

Test plan
How are these changes tested?

[x] Tests pass locally with pytest for python, yarn test for js

Documentation Changes
TBD

#1977  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tazarov and the rest of your teammates on  Graphite

@atroyn, PTAL.

Merge activity

Oct 15, 1:24 AM EDT: A user started a stack merge that includes this pull request via Graphite.


Problem with MWPhotoBrowser and UIWebView

Hi,
I really like MWPhotoBrowser, but i have one problem:
I create the photos with the photoWithURL: method. I also have an UIWebView in my application. The problem appears, as soon as a scroll quick through the photos: Instead of some photos, the activity indicator appears and the photos must be reload. When I remove the UIWebView, everything works fine.
You can try this very easy, just create in the Menu class of the demo project a UIWebView.
Sorry, but my english is not so good.
Thanks,
Mikkell

Hi Mikkell, I'm unable to recreate this issue as we use MWPhotoBrowser and UIWebViews on our own projects and don't suffer from the same problem. If you can provide a sample project that recreates this issue then I'll happily take a look.
— Michael


os x?

Yeah, it should work with libncurses by brew install ncurses.
I don't have os x env.
Could you try it?

The demo kind of doesnt do anythiing. I'm not sure where to debug it because it doesnt throw up any errors. it just silently doesn't do anything. One clue though is if I try to write using a regular puts that doesnt work either, unless its after Crt.done

My bad. It just happens so quickly that it doesnt register with the console. Putting a wait in there seems to do the trick


finish styling on project page

it's not mobile friendly or responsive the images and videos.

the link currently labeled Video Presentation be an embedded video?
the advisor name and category names should link back to the HP with the advisor or category default selected
Slide show needs to be implemented

line breaker

[ ] added next and previous project bottom on the top and the bottom.

[ ] take reading html title off


Stage 1 - dot plot viewer in tutorial is not displaying the example galaxy measurement data.

It should look like this:

But I am getting this:

@johnarban, unless you changed to a generic data set on purpose?


Memory leak of ProjectImpl and GitRepositoryImpl after project is closed

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] puppy => You're not submitting a puppy. I already have one and he's adorable

What is the current behavior?
Android Studio team noticed memory reports from our users, where gitflow leaks objects related to projects that has already been closed.

Is this a bug? Sorry about that. If so give me explicit details how to reproduce:

Open Android Studio
Open a project
Close the project
Reopen same project
Inspect heap in a profiler: the instance of ProjectImpl of the closed project is kept alive due to gitflow plugin.

What is the expected behavior?
gitflow should not store references to large objects like ProjectImpl after the project has been closed. Depending on what the ProjectImpl references, the memory leak can be from 20MB to over 100MB.

What is the motivation / use case for changing the behavior?
Lower memory consumption of gitflow plugin in Android Studio.

Please tell me about your environment:

Gitflow4idea version: 0.7.0
Reproduced on Android Studio 4.0 Canary 1, likely affects all versions of Android Studio and IntelliJ Idea.

Other information (e.g. detailed explanation, stacktrace, related issues, suggestions how to fix, links for me to have context words of praises, pictures of puppies (again with the puppy??) )

Relevant part of a memory report:
Disposed but still strong-referenced objects: 1 com.intellij.openapi.project.impl.ProjectImpl, most common paths from GC-roots:
[    1/100%/  102B] 59.1MB          1   ROOT: Static field: gitflow.GitflowConfigUtil.gitflowConfigUtilMap
[    1/100%/  102B] 59.1MB          1   (root): java.util.HashMap
[    1/100%/  102B] 59.1MB          1   table: java.util.HashMap$Node[]
[    1/100%/  102B] 59.1MB          1   []: java.util.HashMap$Node
[    1/100%/  102B] 59.1MB          1 * key: com.intellij.openapi.project.impl.ProjectImpl (disposed)

Disposed but still strong-referenced objects: 1 git4idea.repo.GitRepositoryImpl, most common paths from GC-roots:
[    1/100%/   88B]  611KB          1   ROOT: Static field: gitflow.GitflowBranchUtilManager.repoBranchUtilMap
[    1/100%/   88B]  611KB          1   (root): java.util.HashMap
[    1/100%/   88B]  611KB          1   table: java.util.HashMap$Node[]
[    1/100%/   88B]  585KB          1   []: java.util.HashMap$Node
[    1/100%/   88B]  583KB          1 * key: git4idea.repo.GitRepositoryImpl (disposed)

In the report above, the memory leak is ~60MB. For some projects it can be even more and a new leak happens every time a project is closed.
GitflowConfigUtil.gitflowConfigUtilMap uses ProjectImpl as a key in the map and GitflowBranchUtilManager.repoBranchUtilMap uses GitRepositoryImpl instance as a key, Neither of the maps removes the reference when project is closed.
For entries in repoBranchUtilMap, consider adding a following code inGitflowBranchUtilManager.setupBranchUtil:
Disposer.register(repo, () -> repoBranchUtilMap.remove(repo));

For gitflowConfigUtilMap, consider similar pattern or use ProjectManagerListener.projectClosed event to remove the key from the map.

Thanks so much for this report. I'll include this in the next version

@radomirm-g which tool did you use to receive this output of memory allocation?


static_array_ref::assign_range assert has to be moved up

In the following code the boundary check assert must be moved before the copy. Otherwise in case of buffer overrun the further application behaviour is undefined.
    SBEPP_CPP20_CONSTEXPR iterator assign_range(R&& r) const
    {
#if SBEPP_HAS_RANGES
        auto res = std::ranges::copy(std::forward<R>(r), begin()).out;
#else
        auto res = std::copy(std::begin(r), std::end(r), begin());
#endif
        SBEPP_ASSERT(res <= end());
        return res;
    }

Right, it's intentionally UB. The check can't be moved up because res position can be known only after incoming range is consumed because it can be input range whose size can't be efficiently calculated before copy. It's user's responsibility to verify that input fits into the array.


[pipeline] add pipeline support for T5Stack/T5EncoderModel

📌 Checklist before creating the PR

[x] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[x] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

#4247
📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

Design special algorithm for distributing layers to pipeline stages, since the structure of T5(Encoder/Decoder) is more complex than other transformer models.
Modify the logic of forward for T5Stack/T5EncoderModel, so that pipeline parallel can be smoothly run.
Add relevant tests for layer distribution & pipeline forward.

💥 Checklist before requesting a review

[x] I have linked my PR to an issue (instruction)
[x] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[x] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.


fix: use INT64 type for empty sets with no type

Tiny fix for empty set expressions: use INT64 in case if type of the empty set wasn't explicitly mentioned. Makes one more test passing:


NWM View App issues compilation

This is a compilation of NWM App issues reported by different sources after 2017 Alabama meeting (Aug 2017)
Rick:

[ ] #2437 NWM Forecaster Viewer Suggestions

CUAHSI:

[x] #2409 [NWM] Forecast starts an hour after initialized in Short Range Configuration
[x] #2410 [NWM] Unclear units for time lag
[ ] #2411 [NWM] Time Lag for Long Range allows for the selection of no or more than one time lag
[x] #2412 [NWM] Members in Long Range Configuration are unclear
[x] #2413 [NWM] When you pick the begin date to be the current date, it should state that the chart is projected values
[x] #2417 [NWM] Snow-Cover Forecast Chart Units
[x] #2420 [NWM] Medium Range Configuration, the variable "Soil Moisture" should have units in its chart but does not.
[x] #2512[NWM Viewer] Failed to Subset Long Range outputs
[ ] #2484 NWM Viewer - Fails to get long range data bug

Tim (UT Austin)

[x] #2289 NWM Viewer app: implement "lateset" parameter to retrieve most recent data without specifying a date

[x] #2307 NWM Viewer: handle polygon with holes

[x] #2318 NMW viewer app: time step wrong for medium range grid output

[x] #2319 NWM viewer app: merge all days into a single file for subsetted analysis_assim

[x] #2327 NWM Viewer: add a new endpoint to return spatial query results

[ ] #2312 NWM Explorer app failed to download files

Jim (BYU)

[x] allow to subset AA data back to May 2017

App Implementation Review Google Doc
https://docs.google.com/document/d/1aMQEiJ-QYUsIifOeZoj8Fm1l_LNfb8E5ibV0n1mwMFc/edit?usp=sharing

This issue was moved to CUAHSI-APPS/nwm-forecast-viewer#2


Error install gem devise

al ejecutar el comando rails generate devise:install obtengo el siguiente error
Rails 5.1.6
ruby 2.3.3p222
Using devise 4.5.0
C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activerecord-5.1.6/lib/active_record/dynamic_matchers.rb:22:in method_missing': undefined method devise' for Install (call 'Install.connection' to establish a connection):Class (NoMethodError)
from C:/Users/Faican/Documents/CursoRails/blog/app/models/install.rb:4:in class:Install' from C:/Users/Faican/Documents/CursoRails/blog/app/models/install.rb:1:in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:477:in load' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:477:in block in load_file'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:662:in new_constants_in' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:476:in load_file'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:374:in block in require_or_load' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:36:in block in load_interlock'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies/interlock.rb:12:in block in loading' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/concurrency/share_lock.rb:149:in exclusive'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies/interlock.rb:11:in loading' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:36:in load_interlock'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:357:in require_or_load' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:511:in load_missing_constant'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:202:in const_missing' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:269:in const_get'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:269:in block in constantize' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:267:in each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:267:in inject' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:267:in constantize'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:583:in get' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:614:in constantize'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise.rb:316:in get' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:83:in to'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:78:in modules' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:95:in routes'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:162:in default_used_route' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:72:in initialize'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise.rb:346:in new' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise.rb:346:in add_mapping'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/rails/routes.rb:243:in block in devise_for' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/rails/routes.rb:242:in each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/rails/routes.rb:242:in devise_for' from C:/Users/Faican/Documents/CursoRails/blog/config/routes.rb:2:in block in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/actionpack-5.1.6/lib/action_dispatch/routing/route_set.rb:436:in instance_exec' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/actionpack-5.1.6/lib/action_dispatch/routing/route_set.rb:436:in eval_block'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/actionpack-5.1.6/lib/action_dispatch/routing/route_set.rb:418:in draw' from C:/Users/Faican/Documents/CursoRails/blog/config/routes.rb:1:in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in load' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in block in load'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in load_dependency' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in load'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:55:in block in load_paths' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:55:in each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:55:in load_paths' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:18:in reload!'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:41:in block in updater' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/file_update_checker.rb:81:in execute'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:42:in updater' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:31:in execute_if_updated'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/finisher.rb:128:in block in module:Finisher' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:30:in instance_exec'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:30:in run' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:59:in block in run_initializers'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:228:in block in tsort_each' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:350:in block (2 levels) in each_strongly_connected_component'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:431:in each_strongly_connected_component_from' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:349:in block in each_strongly_connected_component'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:347:in each' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:347:in call'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:347:in each_strongly_connected_component' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:226:in tsort_each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:205:in tsort_each' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:58:in run_initializers'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application.rb:353:in initialize!' from C:/Users/Faican/Documents/CursoRails/blog/config/environment.rb:5:in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in require' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in block in require'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in load_dependency' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in require'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application.rb:329:in require_environment!' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command/actions.rb:16:in require_application_and_environment!'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands/generate/generate_command.rb:19:in perform' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/thor-0.20.0/lib/thor/command.rb:27:in run'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/thor-0.20.0/lib/thor/invocation.rb:126:in invoke_command' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/thor-0.20.0/lib/thor.rb:387:in dispatch'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command/base.rb:63:in perform' from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command.rb:44:in invoke'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands.rb:16:in <top (required)>' from bin/rails:4:in require'
from bin/rails:4:in `
Gracias

al ejecutar el comando  rails generate devise:install obtengo el siguiente error
Rails 5.1.6
ruby 2.3.3p222
Using devise 4.5.0
C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activerecord-5.1.6/lib/active_record/dynamic_matchers.rb:22:in method_missing': undefined method devise' for Install (call 'Install.connection' to establish a connection):Class (NoMethodError)
from C:/Users/Faican/Documents/CursoRails/blog/app/models/install.rb:4:in <class:Install>'         from C:/Users/Faican/Documents/CursoRails/blog/app/models/install.rb:1:in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:477:in load'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:477:in block in load_file'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:662:in new_constants_in'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:476:in load_file'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:374:in block in require_or_load'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:36:in block in load_interlock'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies/interlock.rb:12:in block in loading'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/concurrency/share_lock.rb:149:in exclusive'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies/interlock.rb:11:in loading'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:36:in load_interlock'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:357:in require_or_load'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:511:in load_missing_constant'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:202:in const_missing'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:269:in const_get'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:269:in block in constantize'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:267:in each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:267:in inject'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/inflector/methods.rb:267:in constantize'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:583:in get'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:614:in constantize'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise.rb:316:in get'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:83:in to'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:78:in modules'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:95:in routes'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:162:in default_used_route'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/mapping.rb:72:in initialize'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise.rb:346:in new'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise.rb:346:in add_mapping'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/rails/routes.rb:243:in block in devise_for'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/rails/routes.rb:242:in each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/devise-4.5.0/lib/devise/rails/routes.rb:242:in devise_for'         from C:/Users/Faican/Documents/CursoRails/blog/config/routes.rb:2:in block in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/actionpack-5.1.6/lib/action_dispatch/routing/route_set.rb:436:in instance_exec'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/actionpack-5.1.6/lib/action_dispatch/routing/route_set.rb:436:in eval_block'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/actionpack-5.1.6/lib/action_dispatch/routing/route_set.rb:418:in draw'         from C:/Users/Faican/Documents/CursoRails/blog/config/routes.rb:1:in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in load'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in block in load'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in load_dependency'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:286:in load'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:55:in block in load_paths'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:55:in each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:55:in load_paths'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:18:in reload!'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:41:in block in updater'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/file_update_checker.rb:81:in execute'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:42:in updater'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/routes_reloader.rb:31:in execute_if_updated'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application/finisher.rb:128:in block in <module:Finisher>'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:30:in instance_exec'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:30:in run'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:59:in block in run_initializers'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:228:in block in tsort_each'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:350:in block (2 levels) in each_strongly_connected_component'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:431:in each_strongly_connected_component_from'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:349:in block in each_strongly_connected_component'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:347:in each'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:347:in call'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:347:in each_strongly_connected_component'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:226:in tsort_each'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/2.3.0/tsort.rb:205:in tsort_each'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/initializable.rb:58:in run_initializers'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application.rb:353:in initialize!'         from C:/Users/Faican/Documents/CursoRails/blog/config/environment.rb:5:in <top (required)>'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in require'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in block in require'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:258:in load_dependency'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/activesupport-5.1.6/lib/active_support/dependencies.rb:292:in require'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/application.rb:329:in require_environment!'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command/actions.rb:16:in require_application_and_environment!'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands/generate/generate_command.rb:19:in perform'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/thor-0.20.0/lib/thor/command.rb:27:in run'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/thor-0.20.0/lib/thor/invocation.rb:126:in invoke_command'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/thor-0.20.0/lib/thor.rb:387:in dispatch'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command/base.rb:63:in perform'         from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/command.rb:44:in invoke'
from C:/RailsInstaller/Ruby2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.1.6/lib/rails/commands.rb:16:in <top (required)>'         from bin/rails:4:in require'
from bin/rails:4:in `'

Pre-check

Do not use the issues tracker for help or support, try Stack Overflow.
For bugs, do a quick search and make sure the bug has not yet been reported
Finally, be nice and have fun!

Environment

Ruby [version]
Rails [version]
Devise [version]

Current behavior
Include code samples, errors, steps to reproduce the error and stack traces if appropriate.
Will be even more helpful if you provide a sample application or a test case that reproduces the error.
Expected behavior

This is more likely to be an environment issue. Please try to delete your Gemfile.lock and run bundle install again. Also, if you're using Spring try to restart it.


Using netcdf output to produce held suarez plots

co-authered-by: @szy21
This PR adds held suarez plots from nc outputs. Closes #1491

bors r+

bors r+

bors r+


[FEATURE] Set docker timezone from Windows/MAC GUI

It would be great to be able to set the /etc/localtime link on the docker VM via the UI
PS C:> date
Fri Jun 24 09:57:56 AUSEST 2016
PS C:> docker run --rm -it centos date
Thu Jun 23 23:57:57 UTC 2016
PS C:>

I think the timezone in the VM is set to UTC at all times (the time is synchronized with the host machine), but I may be wrong, For the timezone inside the container (which is UTC by default), see https://github.com/docker/docker/issues/3359 and https://github.com/docker/docker/issues/12084
However this is a feature request for Docker for Mac / Windows, so best reported through https://forums.docker.com/c/docker-for-mac
perhaps @justincormack knows what the plans are in this area; e.g. Can it be. Add to follow the timezone of the system?

What is the use case for setting the host timezone? You can set the in container timezone if you want by setting /etc/localtime in the container. UTC is just the default if you don't set this, unrelated to the host timezone. None of the times you show are related to the VM localtime link.

I put this in my Dockerfile and it worked great, thanks.
RUN rm -f /etc/localtime && ln -s /usr/share/zoneinfo/Australia/Melbourne /etc/localtime

The use case for setting the host timezone is to give a meaningful default to all containers running on that host. It is much simpler setting it once, than messing with all containers one by one.

I agree. It is very useful to map the localtime and timezone folders, for example with:
volumes:
  - /etc/localtime:/etc/localtime:ro
  - /etc/timezone:/etc/timezone:ro

I really think it would make sense for Docker for Windows to support that as well.

@benoitscherrer  I didn't find any solution to run docker container with host timezone on windows. Yes, we can directly specify timezone using TZ flag docker run -e TZ=Europe\Kiev ........, but when you don't know who will run your image and in which timezone person is located, what is the workaround?


0.8.0

This is a work in progress branch which will become 0.8.0
New features:

Beefed up connection pool thanks to @myndzi's pool2
Internal streamlining of requires, cutting down on a lot of extraneous files
Savepoints implemented on transactions
Transactions are now a proper A+ promise
Beginning to use debug module internally for better debugging

Todos:

Transaction tests
Ensure transaction/savepoint grammar is consistent across drivers
Fix Oracle driver & transaction implementation
Check that WebSQL/Browser build works properly
Replace legacy debug behavior

Leaving until next minor:

Connection clustering? (Need to determine master/slave setup)

Just putting this up here so I can reference it from other tickets, others can take a look, etc. If anyone wants to test the savepoint things, you should be able to just call a transaction on a transaction connection and it should automatically create a savepoint which acts like a transaction.

@tgriesser can you also add strong-oracle support. It's the only oracle driver that works with node 0.12 or io.js.

Yeah it should still be in there - the oracle stuff is definitely broken though, I'm going to see if I can get oracle running locally to work on it a little later.

I don't know if it's still worth to support node-oracle since the author put a "not maintained" disclaimer up. The knex master from a week or so ago works fine for me with strong-oracle and io.js.
If you can wait a day or two I'll clean up the strong oracle stuff and move it into the oracle dialect.

@vschoettke cool - I just got the test suite/jshint passing and got the new transaction stuff hooked up to oracle (as far as i can tell it should work) so if you can run the suite against that, let me know if you see any issues. The strong-oracle inherits from oracle, the only difference is the driverName which is now used internally for the require.

@calvinmetcalf I seem to remember you had a test suite for websql stuff - do you know where I can find that?

I tested it against https://github.com/calvinmetcalf/SQLdown by doing npm link to get the more recent version of knex and then just pasting the results of browserifying test/test.js into the console, can do it again

just quickly testing against sqlite3 in iojs this is a breaking change, give me a sec to make a test

just quickly testing against sqlite3 in iojs this is a breaking change, give me a sec to make a test

Yeah it's not quite done yet, still polishing things just wanted to check against any available suites I know of to make sure nothing obvious is broken.

so there is something wonky with sqlite3 and batch causing the sqldown test suite to consistently error

I sort of redid the entire transaction piece, it might be due to that. I'll need to take a look

ok the binary branch of sqldown (calvinmetcalf/SQLdown#22) now links against this branch and includes code to test it in the browser, just run npm run browser which should build it, then fire up a local server (I tend to use serve) then open up the file test/index.html in chrome (here with serve)

I also noticed that adding debug to the connection no longer prints the sql

Yeah, I started using the debug module... DEBUG=knex:*

For backwards compat probably want to make pass the debug true option set
that
On Wed, Apr 22, 2015, 5:44 PM Tim Griesser<EMAIL_ADDRESS>wrote:

Yeah, I started using the debug module... DEBUG=knex:*
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/knex/pull/786#issuecomment-95347025.

For backwards compat probably want to make pass the debug true option set

Yeah I'll make sure to put that back with a logged notice

@calvinmetcalf I've got all tests but 1 passing in the binary branch with the latest commit
not ok 21 should be equal
expected: '{"pete":"sampras"}'
actual:   '"[object Object]"'

I'm looking into it, but it doesn't look like it was working previously - was that something that should be handled on knex's end or SQLdown end?

ok let me look into it
On Fri, Apr 24, 2015 at 1:06 PM Tim Griesser<EMAIL_ADDRESS>wrote:

@calvinmetcalf https://github.com/calvinmetcalf I've got all tests but
1 passing in the binary branch with the latest commit
not ok 21 should be equal
expected: '{"pete":"sampras"}'
actual:   '"[object Object]"'
I'm looking into it, but it doesn't look like it was working previously -
was that something that should be handled on knex's end or SQLdown end?
—
Reply to this email directly or view it on GitHub
https://github.com/tgriesser/knex/pull/786#issuecomment-95993726.

I guess it'd make sense to try/catch JSON.stringify objects when writing objects - seems like the obvious default

actually the object object thing is on my end

commented out that test, I'm not sure the point of it as it doesn't apply in node for some reason

Woot! Will upgrade Endpoints stuff tomorrow. Any gotchas for use in conjunction with Bookshelf?

Nope, all Bookshelf tests are passing
Going to get started through the backlog of Bookshelf tickets and cut another minor version and then the "next" of bookshelf/knex will be sort of developed in parallel.

Just confirmed Knex 0.8 passes the entire Endpoints test suite.  Sweet deal :D

Just as a sanity check, the current version of Bookshelf (or 0.10.2 at least) now plays well with transactions and promises? So this will do three saves in one transaction:
Bookshelf.transaction(function (t) {
    return Promise.all([
       example1.forge(...).save(..., { transacting: t }),
       example2.forge(...).save(..., { transacting: t }),
       example3.forge(...).save(..., { transacting: t })
    ]);
}).then(function () {
    console.log("transaction committed");
}).catch(function (err) {
    console.log("transaction rolled back");
});

Correct?


error when building CppWrapper

The following error occurs when I run Pkg.build("CppWrapper").
According to the trace, it seems there is an invalid conversion between pointers
/home/nova/.julia/v0.5/CppWrapper/deps/src/cpp_wrapper/type_conversion.hpp:189:16: error: cannot convert ‘jl_value_t* {aka _jl_value_t*}’ to ‘jl_function_t* {aka _jl_function_t*}’ in assignment     m_finalizer = jl_box_voidpointer((void*)detail::finalizer<SourceT>);

INFO: Building CppWrapper
INFO: Attempting to Create directory /home/nova/.julia/v0.5/CppWrapper/deps/builds/cpp_wrapper
INFO: Directory /home/nova/.julia/v0.5/CppWrapper/deps/builds/cpp_wrapper already created
INFO: Changing Directory to /home/nova/.julia/v0.5/CppWrapper/deps/builds/cpp_wrapper
-- Configuring done
-- Generating done
-- Build files have been written to: /home/nova/.julia/v0.5/CppWrapper/deps/builds/cpp_wrapper
[ 50%] Building CXX object CMakeFiles/cpp_wrapper.dir/c_interface.cpp.o
In file included from /home/nova/.julia/v0.5/CppWrapper/deps/src/cpp_wrapper/array.hpp:6:0,
from /home/nova/.julia/v0.5/CppWrapper/deps/src/cpp_wrapper/c_interface.cpp:1:
/home/nova/.julia/v0.5/CppWrapper/deps/src/cpp_wrapper/type_conversion.hpp: In instantiation of ‘static void cpp_wrapper::static_type_mapping::set_julia_type(jl_datatype_t*) [with SourceT = cpp_wrapper::CppAny; jl_datatype_t = _jl_datatype_t]’:
/home/nova/.julia/v0.5/CppWrapper/deps/src/cpp_wrapper/c_interface.cpp:18:31:   required from here
/home/nova/.julia/v0.5/CppWrapper/deps/src/cpp_wrapper/type_conversion.hpp:189:16: error: cannot convert ‘jl_value_t* {aka _jl_value_t*}’ to ‘jl_function_t* {aka _jl_function_t*}’ in assignment
m_finalizer = jl_box_voidpointer((void*)detail::finalizer);
^
make[2]: *** [CMakeFiles/cpp_wrapper.dir/c_interface.cpp.o] Error 1
make[1]: *** [CMakeFiles/cpp_wrapper.dir/all] Error 2
make: *** [all] Error 2
=============================[ ERROR: CppWrapper ]==============================

I am using  Julia Version 0.5.0-dev+1491 on Ubuntu 14.04

This is due to a "too old" 0.5. In the current 0.5, jl_new_closure no longer exists and jl_function_t is a typedef for jl_value_t. If you can't go to the latest 0.5, you can work around this by changing the 5 to a 6 here:
https://github.com/barche/CppWrapper/blob/master/deps/src/cpp_wrapper/type_conversion.hpp#L186


Handle error for missing fuzzer_stats

After #5, in cases where the fuzzer encounters an error (And pipeline prints a warning for "Fuzzer instances in session died"), coverage could not be determined, as the fuzzer_stats file is missing. In the cases where this happens, the plot_data is empty anyway, so having fuzzer_start_time set to 0 should be fine, as no data will actually be loaded.
I'm not sure what exactly causes the fuzzer instances to die in the first place, but I think I've seen this in both AFL and AFL++ mode. It generally seems pretty rare though.

To troubleshoot these kinds of issues, you can replay the .cur_input file in the corresponding fuzzer directory. It holds the currently executed input and usually is the one causing non-recoverable emulator failures. We have had this type of issue with very rare edge cases in the NVIC that triggered an assertion (which is now changed to make the emulator indicate an ordinary firmware crash instead of asserting itself). Maybe that can help you figure out the underlying issue.
I will merge this for more robustness as well.


E2e test framework

Desired Outcome
Implement the basic skeleton for running Golang based e2e tests.
This should include any required startup/teardown steps,  and provide some helper functions to mirror the functionality of existing e2e tests.
Implemented Changes

Adds e2e test folder with a couple sample tests implemented
Adds/updates scripts for running e2e tests against local dev environment
Helper functions for fetching pods, executing commands, reloading dev cluster, etc.

Definition of Done
At least 1 todo must be completed in the sections below for the PR to be
merged.
Changelog

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]
[x] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

It's finally coming! This is super duper exciting


ui 打包工具无法下载，可以放在网盘或者其他可以下载的地方不

This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.
该存储库超出了其数据配额。 负责LFS带宽的帐户应购买更多数据包以恢复访问。
ui 打包工具无法下载，可以放在网盘或者其他可以下载的地方不

The same
batch response: This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.
error: failed to fetch some objects from 'https://github.com/smallnew/react-native-multibundler.git/info/lfs'


Add sample showing usage of command line parameters

Sample (no actual code change to core library) showing how to use command line parameters, including switch mappings, and the supported formats of parameters.

Hi @sgryphon, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


[9.x] Add FilterWithReorder method to Arr class

what we do
It's pretty common that we json encode a filtered array then return back to frontend.
but after array_filter() we need to reorder the key use array_values()
Otherwise the encoded json is treating the key as object
Example
$languages = [
    'java',
    'javascript',
    'python',
    'php',
];

$p = array_filter($languages, fn ($arr) => str_starts_with($arr, 'p') );

//result of the json_encode without array_values()
//{"2":"python","3":"php"}

$p = array_values($p);

//result of the json_encode with array_values()
//["python","php"]
//this is what we want most of the time

Therefore I think we better have a wrapper method that will basically filter then reorder the keys.
Especially useful when we filter an Eloquent , collection , array from the controller then return back to the frontend.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.