Failed to allocate a 51916812 byte allocation with 16777216 free bytes and 21MB until OOM

Failed to allocate a 51916812 byte allocation with 16777216 free bytes and 21MB until OOM
static public Bitmap createMyBitmap(int[] colors, int width, int height){
Bitmap bmp = null;
try {
bmp = Bitmap.createBitmap(colors, 0, width, width, height,
Bitmap.Config.ARGB_8888);
} catch (Exception e) {
return null;
}
return bmp;
}

why？

Since, it doesn't have sufficient memory to allocate the app it crashed with out of memory issue.


[Bug Report] cascader clearable

Element UI version
2.11.0
OS/Browsers version
Google Chrome75.0.3770.100
Vue version
2.6.10
Reproduction Link
https://codepen.io/pen/?&editable=true
Steps to reproduce
1.选择任意选项
2.点击关闭按钮
3.点击选择框
What is Expected?
控件回滚最初未选择的状态
What is actually happening?
控件某些样式/值未回滚
多选同理

Issues without a valid reproduction link will be closed directly.


Game.connected_playes and index

for i, currentPlayer in pairs(game.connected_players) do
            -- store current working player index for future functions
            Ly.playerIndex = i

Just a heads up that the index (i) in this situation is not the players index.  just the first connected player.
you probably want to use
Ly.playerIndex = currentPlayer.index

ty, will be in next release, its already in last commit


How to buy and sell long?

Hi,
I am new to the SDK.
In the example, it only provides the action about short order.
What should I do for long order?
Thanks!

detail see:https://github.com/hbdmapi/huobi_futures_Python/blob/master/alpha/order.py

detail see:https://github.com/hbdmapi/huobi_futures_Python/blob/master/alpha/order.py


New material design icon theme impacts performance

The new 'material design' icon themes are impacting Lumina performance.  It's mostly noticeable during startup and when browsing applications in the 'start' menu.
On a clean install (in vbox) Lumina takes 22 secs to start with material design icon theme.  Once switched to adwaita it goes down to 6 secs.  It seems things start to slow down during the 'Preparing applications' stage.
In case of the 'Browse Applications' menu, it's most noticeable when 'Show Categories' is enabled.  When I click the 'Utility' category there is very noticeable delay (2-3 sec) before it opens.  The more applications in a category the slower it is.
On my laptop where I have many applications Lumina startup increased to almost a minute after the last TrueOS update, which was pretty unusable.  Switching to 'Adwaita' fixed it.
This should hopefully be straight forward to reproduce, but please give me a shout if you need any more info.

This has been fixed.
I had a couple special verification checks for detecting "bad" svg icons in the oxygen theme from a long time back, and since the new theme is all "good" icons (can be loaded by svglite standards) I went ahead and disabled those checks. This seems to make the icon/app startup routine back to normal (or even a bit faster than before since the svg icons are so much smaller than png icons).


Lightweight generics support with Realm on iteration in Swift3

Request:
In swift 3, Objective-C type declarations using lightweight generic parameterization are automatically imported by Swift with information about the type of their contents preserved.
@interface RLUserAccount: RLMObject
@property NSString *name;

+(nonnull RLMResults<RLUserAccount *> *)userAccounts;
@end

//On calling in swift,
let userAccounts = RLUserAccount .userAccounts()
//userAccount type in swift comes out be RLMResults<RLUserAccount>

for userAccount in userAccounts {
            print(userAccount.name)
// Throws error as userAccount is referenced as RLMObject not as RLUserAccount. To access name, will have to typecast to RLUserAccount first.
}

Result :
On iterating through userAccount, name is not directly accessible. Rather typecasting from RLMObject to RLUserAccount is needed to access the name.  Type of content information is lost on iterating through RLMResults in swift.
Realm version: 2.5.0
Xcode version: 8.2.1 (8C1002) - Swift3

Hi, @ashishprasadgit. Thanks for sharing this feature request. I've added those tags and I'll share this issue with the team to see how it might be prioritized in the future.

@jpsim I believe that problem is related with RLMSupport. RLMSupport provides support to enumerate RLMResults in swift classes.
`public final class RLMIterator: IteratorProtocol {
private let iteratorBase: NSFastEnumerationIterator
internal init(collection: RLMCollection) {
    iteratorBase = NSFastEnumerationIterator(collection)
}

public func next() -> RLMObject? {
    return iteratorBase.next() as! RLMObject?
}

}
extension RLMArray: Sequence {}
extension RLMResults: Sequence {}
`
Here in RLMSupport, return of next iterator is RLMObject. If we can change the RLMSupport to support light-weight generics, then I believe, light-weight generics would be automatically supported in swift.

You're right @ashishprasadgit, however the Objective-C generic information is not available to Swift's type system. Here's the compiler error that prevents us from doing this:

Extension of a generic Objective-C class cannot access the class's generic parameters at runtime.

We opened SR-2348 in August of last year to track this.
I reiterate that we cannot do this without changes to the Swift compiler. If you manage to find a way, we'd appreciate a pull request.


dci-palestine.org

DCI Palestine
The site contains articles about martyred children and teenagers. The articles are organized under the URL: https://www.dci-palestine.org/tags/fatalities_and_injuries?page=1, where each page contains 10 articles.
As of this moment, there are 79 articles (the first 8 pages) about martyred children since 7th October 2023. There are older articles for martyred children, but they are dated before 7th of October 2023.
The article titles usually don't contain the names of the martyrs, but the article may contain the names of multiple martyrs.

Articles are listed under the URL: https://www.dci-palestine.org/tags/fatalities_and_injuries?page=1

Strategy

???

I'm not sure how can we search this site without scraping its data, any ideas @loganpowell?


Add detailed info on setting up local environment

List out env variables with links on where to obtain them
Screenshot of .envTEMPLATE with instructions?
Instructions on setting up a MongoDB account/database

@ivngzmn would you be interested in working with me on this?

I'm willing to help out.
I created a new branch: kd-readme
I'll document some things I had to go through.
I believe I need contributer rights.

I'm willing to help out.
I created a new branch: kd-readme I'll document some things I had to go through. I believe I need contributer rights.

You should be able to submit a pull request. Let me in if it stops you for some reason


Reorder channel state changes in Http2MultiplexCodec child channel

Motivation:
If a write fails for a Http2MultiplexChannel stream channel, the channel
may be forcibly closed, but only after the promise has been failed. That
means continuations attached to the promise may see the channel in an
inconsistent state of still being open and active.
Modifications:
Move the satisfaction of the promise to after the channel cleanup logic
runs.
Result:
Listeners attached to the future that resulted in a Failed write will
see the stream channel in the correct state.

Note that this was part of https://github.com/netty/netty/pull/8057 which was abandoned.

@bryce-anderson thanks!


new error about missing something intents

sorry if im just dumb or something but when I run the program I get this error immediately.
Traceback (most recent call last):
File "C:\Dank Memer Autofarm\bot.py", line 7, in
client = commands.Bot(command_prefix=prefix, self_bot=True)
TypeError: BotBase.init() missing 1 required keyword-only argument: 'intents'

what version of discord.py have you installed?
Hint:
run pip freeze and in the output you can find the version of discord.py

aiohttp==3.8.1
aiosignal==1.2.0
async-timeout==4.0.2
attrs==22.1.0
certifi==2<IP_ADDRESS>
cffi==1.15.1
charset-normalizer==2.1.1
colorama==0.4.5
discord.py==2.0.1
frozenlist==1.3.1
gevent==21.12.0
greenlet==1.1.3
idna==3.3
init==0.1.0
multidict==6.0.2
pycparser==2.21
pynput==1.7.6
PyYAML==6.0
requests==2.28.1
six==1.16.0
terminal==0.4.0
urllib3==1.26.12
websocket-client==1.4.1
yarl==1.8.1
zope.event==4.5.0
zope.interface==5.4.0

discord.py==2.0.1

discord.py==2.0.1

In this latest version, the "intents" parameter should be passed inside the client variable (commands.Bot()) in order to run the bot and bot=False parameter seems to be removed & changed in this version?
The only way is to downgrade to version 1.7.3:
pip install discord.py==1.7.3

tysm it works now

although it says it is logged in its not sending the messages in the channel

You need to send ?farm in the discord channel you specified, not in the terminal.
run with True parameter to enable auto-deposit -> ?farm True

thanks so much it works now

No Problem!

👍🏻


Check datatypes, instruments, and platforms

From @mbiddle-nodc:
We also have a response for datatypes (https://www.nodc.noaa.gov/cgi-bin/OAS/lookup?format=xml;table=datatypes) which would be the terms in the ncei_name attribute of the geophysical variable.
Similarly for instruments (https://www.nodc.noaa.gov/cgi-bin/OAS/lookup?format=xml;table=insttypes) and platforms (https://www.nodc.noaa.gov/cgi-bin/OAS/lookup?format=xml;table=platforms).

Stand-by on these checks... A can of worms has opened over here and we need to make a decision.

This has turned into a larger discussion within NCEI about the use of ncei_name/nodc_name/sea_name. For now, I think our best route is making the weight on these checks as small as possible.
The sea_name check can use the xml file at http://www.nodc.noaa.gov/General/NODC-Archive/seanames.xml.
For ncei_name/ncei_code, we haven't released our other tables, which contain the vocabulary, to the public yet. I would recommend a simple check for existence. If it exists, good. If not, make a note that it's missing.

@mbiddle-nodc Sounds good. The existence check is already in place. We'll wait until the names are released to the public to check the vocabulary.


DEV: Fluids Report: Get Response Conversion

EPIC:
#491 Fluids Report Implementation
PARENT:
#492 Get from Store (R) Fluids Report Object
SIZE:
12 hours = 1 1/2 days = 3 points
DEV: Fluids Report: Get Response Conversion  – Size (3pts)

This is dependent on the in progress WITSML object @TessForGithub2
ETA is 7/25 for WITSML object

Dev Complete


Not able to build IOS app after installing the library

How i integrated
`npm install mixpanel-react-native`
then in ios directory pod install
After that  when i run the app
i receive these errors on xcode

`React-native version 0.61.5`

hi @dpk89 , sorry for the late response. However, I'm not able to reproduce. Could you paste more logs to indicate which pod and files caused this error. Also, could you try to delete your Pods folder and redo the pod install and then do a clean build?

I'm closing this issue now, please feel free to reopen it if the problem still exists. Thanks!


feat: ims client gets service token from v3 endpoint

Please ensure your pull request adheres to the following guidelines:

[ ] make sure to link the related issues in this description
[ ] when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues
Thanks for contributing!

@solaris007 do we have some kind of super admin credentials, that we will use in the context ?


Fix for flushed streams

Fixes #2
Problem: chunker from the 'stream-chunker' library does not always produce chunks smaller than MAX_CHUNK_SIZE. If the stream is flushed (as S3 objects from aws-sdk seems to cause), then all remaining data is dumped into the next pipeline stage. AFAICT there is no way to get around this (I even tried forking chunker's flush callback to push the data separately, but it seems to concatenate it anyway.
Solution: If generateTransactionChunksAsync or uploadTransactionAsync receive a chunk greater than MAX_CHUNK_SIZE, they split it themselves. I've implemented this by processing a MAX_CHUNK_SIZE chunk, then recursing through the tail. I believe this should be safe as the stack size will be limited limited to ~ buffer.byteLength / 256kB

Hey @elliotsayes, thank you for raising this issue and the PR! I have, however, went ahead and attempted to solve this with an alternate approach that fixes some other issues with stream-chunker as well. Could you try that branch out at #4 and see if it fixes your issue?
Also, if you are able to, could you create a minimal reproduction of the issue so I can create a test for this on the fix branch? I will be closing this PR, thank you again :)

Hey @CDDelta can confirm that #4 also fixes the issue for me :)
I'm a little unsure about how to create a minimal repro without leaking my AWS credentials as I'm doing my testing with real AWS S3. However if you supply your own credentials/bucket to the example code in #2, you can test with the following two files which were failing previously :)
https://v2.viewblock.io/arweave/tx/caH3_FAk9HgY4YXzpxIez18M13-bAgsY9oGTzZUmLwY
https://v2.viewblock.io/arweave/tx/KHkoJkbQnxz8K_wfUVH28ZJ71Qvz0ygbBKALrMaKx00
Hope this helps, and thanks for the excellent quality fix.
P.S. I'd consider releasing your chunker it as a standalone package as there don't seem to be any maintained ones on npm.


Added addAlias method for tokens

This PR adds an addAlias method to the Token class.
The new method appends the given arguments to the existing aliases of a token. Nothing will be returned.
This behavior is similar to Array.prototype.push.
Examples:
let t = new Token(type, content, "alias", ...etc);
console.log(t.alias); // "alias"
t.addAlias("newAlias");
t.addAlias("alias 1", "alias 2");
console.log(t.alias); // ["alias", "newAlias", "alias 1", "alias 2"]

addAlias will always change the data representation of alias to an array.
let t = new Token(type, content);
console.log(t.alias); // undefined
t.addAlias();
console.log(t.alias); // []

This might be more interesting / useful if you demonstrate where this could be used. Can you update the code this would simplify?

I haven't found any language or plugin which currently has the need to add aliases to tokens, so there is no code to be simplified right now.
However, #1519 has to do exactly that and it can be compressed into a single if-statement using addAlias:
// class-name alias to tags which refer to classes
if (token.type === 'tag' && token.content.length === 2 && token.content[0].type === 'punctuation'
    && typeof token.content[1] === 'string' && /^[A-Z]/.test(token.content[1]))
	token.addAlias('class-name');

I added addAlias because after writing #1519, I wrote another hook and had to implement it again. This PR is for eliminating this redundancy.
Plus, I think that something as basic as adding an alias to a token should be simple.

Well, your right.


chore: Update deps

Also, temporarily disabled clippy check. Can be discussed in #49.

This should bump MSRV description in README.
https://github.com/tui-rs-revival/tui-rs-revival#rust-version-requirements


INACCESSIBLE BOOT DEVICE AND NO FOUND SOURCE FILES! The end  is to wait solved

Hi  Team and Friends
when   I have  run Dism command about restore-health, I have show source files not found,  so  unsuccessful repair!
for  successful repairing,  please, you can add a fix  solution!

will someone be able to  answer?


Draft upgrade commands

Code for discussions about upgrade paths not involving external process communication.
Playground for certificate updates

Wrong merge direction.


[PF-1883] Update comments to resolve Javadoc warnings

minor - Update comments to resolve Javadoc warnings, no code changes

handled in https://github.com/DataBiosphere/terra-resource-buffer/pull/250


Automatically add new Issues to "DSpace Backlog" project board

This creates a new "issue_opened.yml" workflow configuration which enables a GitHub Action for all newly created Issues which are either unlabeled or labeled with needs triage.
More specifically, the Create Project Card action is run to check for new issues to add to the DSpace Backlog project board.  The purpose of this Board is to capture incoming issues that need triaging/scheduling for a specific release...although there's also an "Unscheduled backlog" column on that board for anything that is currently unscheduled.
At this time, only issues flagged as needs triage (which will be added by default) or unlabeled will go on this board.  Other issues will be assumed to have already been triaged to a specific release board (e.g. the DSpace 7.1 board or similar).
I'm creating this PR simply as documentation of this decision & to test the process in GitHub (to make sure it works as expected). Additional GitHub actions may be forthcoming as well!

Merging immediately, as I've tested this in a personal repo and the https://github.com/DSpace/Rest7Contract repo and it works in both those places.


Update GHSA-c59h-r6p8-q9wc.json

Fix issue where security vulnerability is incorrectly applied to the v0.4 version of next which is a totally different product than one started at v0.9.9.
Issue is that security is reported for packages that still depend on<EMAIL_ADDRESS>which doesn't make sense. It was already discussed here, see https://github.com/github/advisory-database/pull/179 for context
Note: I wasn't able to introduce this change via suggest form as it exposes just "Affected versions" field, which logically would have to be  >=0.9.9  <13.4.20-canary.13  and that's not accepted

Ah, I see 0.9.9 is the first release after 0.4.x.


Unable to use Target to enable a flag that is disabled

When a flag is enabled, I can set a 'Target' in Harness to disable that flag, however, the opposite is not the case. If a flag is disabled, I can't enable the flag for a target.
It appears that the bug comes from https://github.com/harness/ff-java-server-sdk/blob/main/src/main/java/io/harness/cf/client/api/Evaluator.java#L287 where the variation with a target is only checked when the flag is 'ON'.
This looks like a bug. Am I missing something? Happy to send a PR if this incorrect behavior.

OK I see you would enable the flag, then make the default rule serve up 'False'


Parameter on openModel to handle previous run results

When an existing VEModel is opened, its visioneval.cnf configuration is opened, model stages are sought in results, and their ModelState_ls is loaded. That can lead to bizarre behavior if visioneval.cnf is changed so it is no longer compatible with ModelState_ls. There's another issue (#9) specifically targeted at checking for discrepancies in configurations to head off the worst of the bizarreness.
This issue is to do something related but different: when the model is opened, we want a parameter (like the "reset"/"save"/"continue" instruction on the VEModel run function) that will explain how to handle existing results: "save" them (during the open, not during the run, so they are moved aside), "reset" them (i.e. throw them away right then as the model is opened), or "continue" (which will just do the current behavior.
The default will be "continue" (although in combination with issue #9, perhaps we make the default "save" if the visioneval.cnf has changed. Doing "continue" with a changed visioneval.cnf should see if the only change are new stages or scenarios, which should be safe - other changes should force an error message during model open and require the user to explicitly specify how to handle previous results.

Here's the plan:

Add a "reset" parameter to openModel that defaults behavior if only ModelStages have changed

Additional option to "save" / "reset", which will execute those options as the model is opened (but do just what a run would do if it were asked to save/reset). No need to investigate stages, etc.

If a model has results, check file dates on all the visioneval.cnf / run_parameters files against the date/time the model was run (from its ModelState_ls) to determine if the opened Model results are up to date

When we attach a source to a run parameter, let's also attach a date/time to the run parameter from the file date/time in which it was defined.
Then we can look into the model's ModelStages to see when they were defined relative to the setup and force a reset if the stage exists and the setup is newer, or do a continue if the stage does not exist.
ModelStages either at the top level or within Scenarios are treated differently: if a ModelStage exists in results and its specification is newer, it will be "reset", otherwise (non-existent or up to date) it will be "continued"

If any of the parameter files are newer (for things other than stages), force user to explicitly set "reset" or "save"

"save" is applied globally, and the entire model must be rerun - tell the user that the model needs a full re-run

Final fix did not entail new interface. Model will report on "out of date" stages and automatically accommodate them (reporting to user and re-running as needed). Arbitrary stages can now be run, and a limit set to run only one or a few stages without continuing.


Minor fix of TestLogsStderrInStdout

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>we should print stderr but not stdout in
c.Fatalf("Expected empty stderr stream, got %v", stdout)

LGTM if janky's happy

LGTM


Remove header include that makes clang complain

With this include in place, the build fails and clang reports "duplicate symbol" _STRTreeType. The project builds when I remove the header include and all the tests pass on my macbook.
Here are my gcc version details:
$ gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/c++/4.2.1
Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>)
Target: x86_64-apple-darwin17.7.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Thanks @sgillies . I really don’t understanding why this is working. If we don’t include “strtree.h”, how is “init_strtree_type” defined?
I am clearly missing something as CI passes... could you help me out?

A similar problem on Mac was previously solved with adding "extern" -> https://github.com/pygeos/pygeos/pull/79
But indeed, I also don't understand why it works without including strtree.
We should probably try to add Mac testing, which should be possible on Travis (or on Azure / GitHub actions)

It works because C compilers use a ton of magic 😄
Looking more closely, the compiler does warn about the implicit declaration.
    src/lib.c:55:9: warning: implicit declaration of function 'init_strtree_type' is invalid in C99 [-Wimplicit-function-declaration]
        if (init_strtree_type(m) < 0) {
            ^
    src/lib.c:55:9: warning: this function declaration is not a prototype [-Wstrict-prototypes]

There are also a lot of other warnings.
Perhaps a single .c file would help. There is only one extension module, lib.so, so we really only require a single source file.


Keep showing typing indicator until bot responds to user or user starts typing

BOT - Keep Showing Typing indicator until BOT responds to user or user started typing ?
We are sending tying indicator with below of code
	Activity isTypingActivity = activity.CreateReply();
            isTypingActivity.Type = ActivityTypes.Typing;
            await botConnector.Conversations.SendToConversationAsync((Activity)isTypingActivity);

These display tying indicator, but only for 1/2 seconds.
is there any way to display typing indicator for log time  or until bot sends response
Net sdk : BOT

@ankitvarmait What is the version of Web Chat are you using?
In Web Chat v4, we will show the typing indicator sent by the bot for 5 seconds or until any incoming activities.
For example, if the bot sent typing activity and then idled for > 5 seconds, you will see the typing indicator show up for 5 seconds and then gone. If the bot sent typing activity followed by another activity, you will see the typing indicator show up for a split second and then gone.
Can you check if the bot sent any types of activities after the typing activity?

@compulim ,
We are referring webchat   version:

and

Looks like you are using Web Chat v3. The behavior is:

Bot send typing activity
Web Chat display dot-dot-dot animation
Bot idle (no activities)
Web Chat will remove the dot-dot-dot after 2 seconds

If you are seeing 1/2 seconds, can you check if the bot are sending any activities that might killed the typing indicator? You can look at network traces to see if there are any incoming activities.

Thanks @compulim for providing details.
~2 seconds when there is no activity from BOT to user..
for the web Chat v4, do we need to upgrade nuget packages?

Web Chat is the JavaScript-based UI for Bot Framework, so on your page, you are probably using it from either: <IFRAME>, CDN (<script>), or NPM.

Closing due to inactivity.

Hi
It worked for me also in localhost/emulator. but when i deployed in azure it is not working.
i can see there is delay in the response but typing is not coming.
below is my code.
await step.context.sendActivities([
{ type: 'typing' },
{ type: 'delay', value: 3000 },
                        ]);

@sanjeevgaut,
Talking with a few team members, there has been no change in the Emulator code base that would make a delay work. I am currently waiting to hear from an SDK member about the activity type, but I think this is going to take some tracking down. Could you please file a new issue and answer the minimum repro questions?
As a workaround in the meantime, please feel free to use the code below, which uses a sleep function to create the delay. I tested this on portal and it works with no problems.

you can use this in .NET
await stepContext.Context.ShowTypingIndicator(1000);
await stepContext.Context.SendActivityAsync("This message should display after a second");

@gitbash-basher
The ShowTypingIndicator won't show when you call it in the stepContext.Context, any idea?


Feature request: Editor support for relational (MVP)

We will start building forms to edit relational-related entities (i.e. stores, mappings, connections, diagrams, etc.). This is not meant to be exhaustive but just the MVP, which lists out items to be done in order of priority. We will keep this around until we're happy with the MVP and move the remaining outstanding items to another thread.
Implementation Plan

[x] Relational mapping source selector (https://github.com/finos/legend-studio/pull/207)

[x] API expose an engine API to transform (grammar <-> JSON) relational operation element (https://github.com/finos/legend-engine/pull/232)
[x] Leave relational operation element raw in relational property mapping
[x] Operation element editor
[x] Source tree need to support joins
[x] Support DnD for joins from source-tree

[x] Relational mapping execution panel: note that we will use H2 for this

[x] Use query builder
[ ] Auto-generate test data (currently blocked by engine)

[x] Relational mapping test panel

[x] Create an API that takes in mapping, model, source/target class -> generate test data: this API will accommodate for M2M use case as well (and potentially flat-data), if there is no connection specified -> use H2, do some dummy data generation (if it gets too complicated, throw?), if there is a connection specified -> use the API that we haven't released yet
[x] Create a protocol to hold relational mapping test and make it work with test runner (https://github.com/finos/legend-engine/pull/238)

[ ] Basic support for database editor

[ ] Be able to generate store from connection
[ ] Basic join editor (need more investigation, we can take a look at Microsofts Access)

[ ] More advanced lambda builder for relational property mapping: supports function compose, group operators, filter, groupBy, etc.
[ ] Support identifying/interpreting Primary Key (PK) in relational mapping
[ ] Support embedded relational property mapping

Close this to open smaller issues


Add a README note about being macOS specific.

The README doesn't explicitly say that it's macOS specific tool.  It does now.
Fixes #63.

thx


feat!: Allow extension callbacks to have non-'static lifetimes

Closes #127.
BREAKING CHANGE: Add a lifetime argument to TypingSession, EmitModuleContext, EmitHugr, EmitFuncContext.

Codecov Report
Attention: Patch coverage is 90.16393% with 12 lines in your changes missing coverage. Please review.

Project coverage is 85.12%. Comparing base (ef55558) to head (b06c426).

Files with missing lines
Patch %
Lines

src/custom.rs
75.51%
8 Missing and 4 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   85.22%   85.12%   -0.11%
==========================================
  Files          31       31
  Lines        4184     4235      +51
  Branches     4184     4235      +51
==========================================
+ Hits         3566     3605      +39
- Misses        297      305       +8
- Partials      321      325       +4

Flag
Coverage Δ

rust
85.12% <90.16%> (-0.11%)
:arrow_down:

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

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


Emmet 1.0.0 in VScode 1.30

Issue Type: Bug
As I understand it VS Code 1.30 should contain Emmet 2.0. My installation contains Emmet 1.0.0
I tried uninstalling and reinstalling but same thing.
What is not working is the Emmet suggestions which as I understand it is part of Emmet 2.0
VS Code version: Code 1.30.1 (dea8705087adb1b5e5ae1d9123278e178656186a, 2018-12-18T18:12:07.165Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz (4 x 2496)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.89GB (9.38GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (17)

Extension
Author (truncated)
Version

vscode-django
bat
0.17.0

django-html
bib
1.3.0

django-snippets
bib
1.1.1

bracket-pair-colorizer
Coe
1.0.61

vscode-eslint
dba
1.7.2

jupyter
don
1.1.4

python-extension-pack
don
1.4.0

tslint
eg2
1.0.42

MagicPython
mag
1.1.0

code-beautifier
mic
2.1.0

Angular-BeastCode
Mik
7.0.24

python
ms-
2018.12.1

PowerShell
ms-
1.10.2

vsliveshare
ms-
0.3.1071

LiveServer
rit
5.3.1

vscodeintellicode
Vis
1.1.2

jinja
who
0.0.8

Here's my extension information for the builtin extension Emmet:
Name: Emmet
Id: vscode.emmet
Description: Emmet support for VS Code
Version: 1.0.0
Publisher: vscode
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=vscode.emmet

I have it working now as documented in Emmet 2.0 on the VSCode site. I found I needed to uninstall the django and python extensions.

Emmet 2.0 as referred to in https://code.visualstudio.com/blogs/2017/08/07/emmet-2.0 refers to the version of emmet libraries and not the version of the built-in emmet extension.
The python extensions shouldn't have any effect on the emmet features as they essentially work on different file types. The former on python files, the latter on html or css files.
If you see the issue again after installing the python extensions, please re-open this issue


18w47a

Can't seem to get these 1.14 snapshot PRs working with Waterfall 240, has anyone else had any luck?


Use package_data instead of data_files

https://github.com/pypa/pip/issues/2874#issuecomment-109429489
More busywork

Hrm, not sure that even makes sense as a fix, closing - DO need data to be shared between these packages.


backup

Hey There @Wjxfi,  Thanks for using my app. I hope you enjted it! So can you provide some more info about what you want to be added in the app so that i can start working on it? :heart:

Thanks for providing more details on backup feature. Took a look at the example you sent and it is a good idea to add such feature for the app. Thanks for the suggestion, will work on implementing that in next update :heart:

Yeah i know but i am a student and have a limitted time to work on my apps so it might take time for me to actually get to work on it. Thanks for your patience ❤

the update  is done now users can import and export their data as xml to/from the app. it is available under release tag of v1.1.0 and will be available in f-droid in next build cycle.

yes i did add the app to github release and it will be published to fdroid after a little bit. it is on release now


Timeout Error

Upon starting livestreamer, it hangs and gives me the following error:
error: Unable to open URL: https://api.twitch.tv/api/channels//access_token.json (HTTPSConnectionPool(host='api.twitch.tv', port=443): Max retries exceeded with url: /api/channels//access_token.json?as3=t (Caused by <class 'socket.error'>: [Errno 10054] An existing connection was forcibly closed by the remote host))

Hi
I am having this issue too. Even if I recheck for streams every 900 seconds (15mins) eventually after around 2 hours I will get a notice that "An existing connection was forcibly closed by the remote host".
This doesn't make much sense given that I am checking for streams so infrequently I am surprised if I was sending too many requests.
However, I am more interested in a workaround, how did you accomplish this @Avbrella  ?
I have a batch file that runs:
livestreamer %url% best -o "filename - %date:/=-% %time::=-%.ts" --retry-streams 900
```goto start

However, when when the error is thrown it does not go back to start it just hangs at the error.
If I close the terminal and rerun the file it starts to check for streams again so I am not temporarily banned I just need a way to catch this error and restart the process.

Hope that makes sense and I hope you can help

Okay this is really weird, If I use the windows binaries on my windows 10 PC I get the following error.
C:\Program Files (x86)\Livestreamer>livestreamer --loglevel debug twitch.tv/esl_csgo best
[cli][info] Found matching plugin twitch for URL twitch.tv/esl_csgo
error: Unable to open URL: https://api.twitch.tv/api/channels/esl_csgo/access_token.json (HTTPSConnectionPool(host='api.twitch.tv', port=443): Max retries exceeded with url: /api/channels/esl_csgo/access_token.json?as3=t (Caused by <class 'socket.error'>: [Errno 10054] An existing connection was forcibly closed by the remote host))
But I installed it via PIP and it works.
[cli][info] Found matching plugin twitch for URL twitch.tv/esl_csgo
[plugin.twitch][info] Attempting to authenticate using OAuth token
[plugin.twitch][info] Successfully logged in as username
[cli][info] Available streams: audio, mobile (worst), low, medium, high, source (best)
[cli][info] Starting player: 'C:\Program Files\VideoLAN\VLC\vlc.exe'
[cli][info] Got HTTP request from VLC/2.2.1 LibVLC/2.2.1
[cli][info] Opening stream: medium (hls)
It works fine on all my other computers which run 8.1 and 10 ? Any suggestions?


Suggestion: Toggle League/Standard

I'd love to have a toggle to switch between my currently selected league and the fitting standard league.
E.g. Metamorph SC -> Standard
Metamorph HC -> Perm. HC

0.6.11 (2020-03-24)

add league as fast toggle option


Git smoke test failures on Linux

These two tests consistently time out for me on linux:

@RMacfarlane I can't repro this on Linux. Do you still see it?


Queued processing

This brings benefits for all cases where paginated resources are consumed, especially when there's a lot of elements. The gitlab-api package now provides a manager-worker-pattern, where the paginated resource is crawled and at the same time several workers can process the elements concurrently.
For example, when working with merge requests, the user doesn't have to wait until all merge requests have been loaded. This means faster feedback cycles and a better user experience.

This will close #35


Added EnumFastToStringDotNet to list

Added https://github.com/Spinnernicholas/EnumFastToStringDotNet to list

Please move the entry to be in alphabetical order

Done.


Support same page anchor like ReadTheDocs

Hello
I'm wondering if this theme support same page anchor link.
Something like: https://borgbackup.readthedocs.io/en/stable/installation.html
Where the menu point to the title on the same page, if that make any sense...
Kristian

I don't really understand your question, but I will try to give an answer anyway.
A table of content is available in the fixed navbar, at the left of the current page title.
In 2.0.0, it's automatically avalaible on default pages but can be disabled by setting toc = false in the Front Matter of the page. Prior to 2.0.0, you have to set the parameter to true.
See page Markdown for example

I don't really understand your question, but I will try to give an answer anyway.

I want something similar to: https://borgbackup.readthedocs.io/en/stable/installation.html , you can see the "headline" in the menu. Ex clicking Pre-Installation Considerations goes to the Pre-Installation Considerations headline on the same page.
I'm wondering if this theme can do the same?

In 2.0.0, it's automatically avalaible on default pages but can be disabled by setting toc = false in the Front Matter of the page. Prior to 2.0.0, you have to set the parameter to true.

That seems to be something else. toc = false remove the page from the menu as I understand from the example site.
Sorry if I'm confusing, I'm pretty new to this and all the terms.

The left menu contains only pages. For anchors of the current page, it's automatically generated like this: http://recordit.co/KTJPaCAF92


Support Markdown format

fix #7

Thank you @GBR-613! I'll check it out today and add some tests for both renders


[Snyk] Security upgrade node from hydrogen-buster to 22.5.1-slim

Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

Dockerfile

We recommend upgrading to node:22.5.1-slim, as this image has only 33 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Link Following SNYK-DEBIAN10-GIT-6846202
  714

Link Following SNYK-DEBIAN10-GIT-6846202
  714

CVE-2019-1387 SNYK-DEBIAN10-GIT-537149
  614

Directory Traversal SNYK-DEBIAN10-GIT-5461944
  614

Arbitrary Code Injection SNYK-DEBIAN10-GIT-5461946
  614

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Directory Traversal

ok


[v5] When do undo/redo, node position is updated incorrectly

Describe the bug
The node position only be updated to old y-axis position, when do undo/redo.

I found a bug in item-data-command from 5.0.0-beta.28 which the data doesnt have nodeId.
https://github.com/antvis/G6/blob/9b20b27d9ddcd942399ad638dd6a8e22168cdaa7/packages/g6/src/stdlib/plugin/history/item-data-command.ts#L69C41-L69C47
Your Example Website or App
https://g6-next.antv.antgroup.com/examples/tool/history#history
Steps to Reproduce the Bug or Issue

Move the node to random position
Call undo api
The node position is only be updated to old y-axis position

Expected behavior
The node position need to be updated both of x-axis and y-axis.
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

We greatly appreciate your feedback. It's important to note that G6 has seen substantial evolution since the beta 28 version. With this in mind, we encourage you to upgrade to the latest release (beta.35) to take advantage of these advancements. Please be aware that this update entails modifications to how G6 is utilized, and we are actively enhancing the accompanying documentation to aid in your transition.
In the interim, for guidance on writing your code, you may refer to our test cases. We're committed to providing you with the necessary resources and support as we roll out these improvements.


No service dependencies shown in jaeger

Hello,
i have a minimal example with 2 Spring Boot Apps where one calls the other.
I can see all spans of the apps in the Jaeger UI, but no dependency between the apps. Shouldn't this already work by autoconfiguration?
Here is a link to the minimal example. If it's not a bug, maybe someone could push me into the right direction at Stackoverflow here :)
Thanks in advance

Hi @derXear if you can see spans in UI it means that there is nothing wrong with the instrumentation (this project).
Dependencies require an additional process to generate them https://www.jaegertracing.io/docs/1.10/deployment/#aggregation-jobs-for-service-dependencies

Hi @pavolloffay
but shouldn't spark not be necessary when all-in-one jaeger is used as stated in the first paragraph of the project jaeger-spark-dependencies?


Fix CommitteeManager deprecations

User Deprecated: The "AppBundle\Committee\CommitteeManager" service is private, getting it from the container is deprecated since Symfony 3.2 and will fail in 4.0. You should either make the service public, or stop using the container directly and use dependency injection instead.

:ok_hand:

It is no longer relevant with Sf4.4 merged recently

It is no longer relevant with Sf4.4 merged recently


Can't install versions > 0.0.8 due to wrong python version even when Python 3.11.6 is installed

Hi, first and foremost thank you for this awesome implementation. I've already used you demo code without the cli for (quite literally) hundreds of hours of audio.
So the issue is, I can't install the package itself with pip or pipx because it won't find the correct python version. Simply installing with
pipx install insanely-fast-whipser

Does work, but the version won't actually work (new BetterTransformers, differenct closed issue). I've then realized that pipx installs 0.0.8 but nothing newer. Forcing a new version with
pipx install insanely-fast-whisper==0.0.13

Doesn't work, with the error:
Fatal error from pip prevented installation. Full pip output in file:
    /home/remus/.local/pipx/logs/cmd_2023-12-16_12.34.56_pip_errors.log

Some possibly relevant errors from pip install:
    ERROR: Ignored the following versions that require a different python version: 0.0.10 Requires-Python <=3.11,>=3.8; 0.0.11 Requires-Python <=3.11,>=3.8; 0.0.12 Requires-Python <=3.11,>=3.8; 0.0.13 Requires-Python <=3.11,>=3.8; 0.0.9 Requires-Python <=3.11,>=3.8
    ERROR: Could not find a version that satisfies the requirement insanely-fast-whisper==0.0.13 (from versions: 0.0.1, 0.0.2, 0.0.3, 0.0.4, 0.0.5b0, 0.0.5b1, 0.0.5b2, 0.0.5b3, 0.0.5, 0.0.6, 0.0.7, 0.0.8)
    ERROR: No matching distribution found for insanely-fast-whisper==0.0.13

Error installing insanely-fast-whisper from spec 'insanely-fast-whisper==0.0.13'.

But the python version is:
python --version
Python 3.11.6

So it looks like either pipx won't parse the version correctly, or it treats 3.11.6 as > 3.11.
The issue can be worked around by passing the --ignore-requires-python flag to pip.
pipx install insanely-fast-whisper==0.0.13 --force --pip-args="--ignore-requires-python"

I don't know if this can be fixed by requiring version < 3.12 (instead of <=3.11), but maybe the workaround helps someone else.

ooooh! that's a lovely find! would you open to add this information as in the README please? 🙏

Sure, I'll prepare a PR.


Debugging Breakpoints not hit in mono repo with libraries

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Yes, it was running before. Don't know the exact version (angular 10)
Description
Can't debug into library code with Visual Studio Code or Chrome Debugger. Breakpoints in the app-component.ts are hit but breakpoints in injected service don't. In previous versions of angular it was possible.
Found an stackoverflow article with the same problem, but without a solution / answer:
https://stackoverflow.com/q/67580325/5703479
🔬 Minimal Reproduction
Sample repo: https://github.com/cluen/ng-debug-library
> ng new ng-debug-library
> ng g library @lib-core/components
> ng b @lib-core/components --watch

add test-method to component.service.ts (e.g. console.log(1)) ,
inject and call test-method in app-component.ts

add breakpoint to app-component.ts method call,
add breakpoint to component.service.ts content

in a different terminal
ng s

launch via visual studio code: F5 (launch.json will be autogenerated, edited content below)

{
  // Verwendet IntelliSense zum Ermitteln möglicher Attribute.

Chrome: Version 90.0.4430.212
Operating System: Windows
Visual Studio Code Version: 1.56.2
Debugger for chrome: v4.12.12

Hi @cluen,
The problem appears that from your application you are incorrectly referring the library.
- import { ComponentsService } from 'projects/lib-core/components/src';
+ import { ComponentsService } from '@lib-core/components';

@alan-agius4 Yes you are right for this simple reproduction example. It was my fault in generating this simple example.
In our production environment, we have a similiar setup and the import is the correct library path '@lib-core/components'

For breakpoints to work you need to enable sourcemaps, maybe they are disabled?

sourceMaps are active.
for serve
...
            "sourceMap": {
              "scripts": true,
              "styles": true,
              "vendor": true
            }
...

if the breakpoint in app-component.ts is hit and I step into it with debugger the correct sourceMaps are displayed. Other breakpoints within the test Method doesn't hit. debugger keyword stops at the right place too, but with no effect to other breakpoints.

if the breakpoint in app-component.ts is hit and I step into it with debugger the correct sourceMaps are displayed. Other breakpoints within the test Method doesn't hit. debugger keyword stops at the right place too, but with no effect to other breakpoints.

This typically indicates the an incorrect file might be referenced in the compilation. What you can try to build your application and verify that in the main.js there are no .ts files referenced from any of your libraries.
To do so, run
ng build --stats-json

And use https://www.npmjs.com/package/webpack-bundle-analyzer to load the stats.json file.

@alan-agius4 Thank you very very much. You made my day.
There were wrong / direct file imports in our libraries.
Found them with the following regex:
^import([a-zA-Z\{\} ]*)'projects

Glad it worked 😀


[cinder-csi-plugin] Fix duplicate keys of tolerations

What this PR does / why we need it:
Fix duplicate keys of tolerations in node plugin yaml
Which issue this PR fixes(if applicable):
fixes #1608
Special notes for reviewers:

Release note:

NONE

/lgtm

/approve


Wrong quantity when adding back a product immediately after it was removed

Steps to reproduce the behavior:

removing a product with applyCartLinesChange -> removeCartLine
add it back again with applyCartLinesChange -> addCartLine (with quantity: 1)
The product now has a quantity of 2. It should have just 1.

Screen capture
https://youtu.be/wngy0B02ieI

May be fixed by https://github.com/Shopify/ui-extensions/issues/448, will test this one together to confirm.

Yes, I can no longer reproduce, thanks!


avoid hardcode "istio-system" in pilot

avoid hardcode "istio-system" in pilot

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: greghanson
If they are not already assigned, you can assign the PR to them by writing /assign @greghanson in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @lichuqiang. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@lichuqiang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d47214b689da5b3331f8c0c708eee470865dbec0
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-multicluster-e2e

@lichuqiang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d47214b689da5b3331f8c0c708eee470865dbec0
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Biomass

Adding all ETModel changes related to 'the biomass project'. The most important addition is the new sankey for biomass!

Furthermore this PR includes:

Translations of new charts
Tranlsations of new slider

Could you generate a new schema.rb from the merged ETModel database?

It looks like the database used for biomass has a couple of old ETEngine tables in there. I expect someone accidentally imported an ETEngine dump into their local ETModel DB using Sequel Pro. This can be fixed by getting a fresh DB dump from dbbot and importing it with bin/import-db (not Sequel Pro, which will preserve the ETEngine tables).
I've fixed schema.rb by removing the extra tables.

Okay good to know! That could be me, I do not update the ETEngine database often.


Screens in reverse order

Apologies ahead of time for my lack of familiarity. I purchased the "RGB glow tube clock IPS" and loaded the "Punkcyber" firmware via the web installer but the screens are in reverse order. Is this not the appropriate firmware for my device?

Try the regular Elekstube firmware https://www.nixies.us/projects/elekstubeips-clock/elekstube-ips-v1/

judge2005, thank you for the reply. That version seems to work but is unstable, when I change clock faces they will start to load but then go blank, to make them come back I have to cycle the power and then screen 3 will randomly just go blank. Outside of Punkcyber being backwards the display was stable. Possibly I have a temperamental board?


Add filter/sort/search functionality to listing views

Missing the search but #17 is a good start for now.


Basic inventory

Following top priority list we also need inventory for a good demonstration of features.
This is what Urist wrote on this subject (consider this headcanon):
Inventory
Mobs have several slots they can put items in, based on the organs of the mob and possibly items in other slots. These work as separate container entities. All of the slots that the mob has are shown on the UI (see the example) and can be interacted with as normal; they are always considered to be within the mob’s range. Worn items can affect the characteristics and behaviour of the mob.
One of the commands available for Container objects that hold HeldItems is “open”. It is a primary command always available regardless of the current item in the hand or a secondary command available when you have such a Container in your hand. When you open such a Container, a window pops up showing all its slots and items currently inside of them, if any. As long as the Container is within the mob’s interaction range, objects inside of it are considered within his interaction range too. The window can be closed via the close button in the corner.
Items can be dragged from inventory slots or the world map to inventory slots by clicking the LMB on the item, holding it, dragging it to the slot desired and releasing the button. If the slot can hold the item, it is moved there.
If a mob has a toolbelt equipped, it works as his hotkey bar. Each of the toolbelt’s slots is mapped to a key in the 0-9 range and by pressing that key the player controlling the mob can:

If there’s an item in the current hand, it can go to the corresponding slot and the slot is empty, the item is put into that slot.
If the current hand is empty, the corresponding slot has an item and the item can go in the current hand, the item is put in the current hand.
If there’s an item in the current hand, the corresponding slot has an item and they can swap (meaning that both slots would be able to hold the others’ items were they empty) then they swap.
If any of these scenarios occur, that slot is marked as “last used”. By pressing a hotkey a player can quickly access the “last used” toolbelt slot using the same behaviour as above.
Accessing the backpack can also be done by using a hotkey (“I” by default) instead of using the “open” command on it as usual.

For now, the following inventory slots should suffice. I’m including organ-based stuff but that should only be worked in when organs are in, obv - organs shouldn’t come first:

PDA/ID (provided by some clothing)
Clothing (provided by the body organ)
Toolbelt (provided by the torso organ)
Ears (provided by the head organ)
Mask (provided by the head organ)
Hand x2 (each provided by an arm organ)
Back (provided by the torso organ)

Basic version completed


create and delete a large number of snapshots  leaving  stale snapshots  in the backend

Describe the bug
create and delete a large number of snapshots  leaving  stale snapshots  in the backend
Environment details

Image/version of Ceph CSI driver: canary
helm chart version
Kubernetes cluster version:1.14.2
Logs

rbd plugin logs
rbd.log
snapshotter logs
snap.log
Steps to reproduce
Steps to reproduce the behavior:

Setup details: '...'
Deployment to trigger the issue '....'
See error

Actual results
created 50 snapshots and deleted 50 snapshots, there are around 14 stale snapshots in the backend
Expected behavior
once we delete Kube snapshots there should not be any stale snapshots in the backend

@ShyamsundarR @humblec PTAL

There are few things happening here, primarily because the requests are timing out.
The snapshotter sidecar does NOT retry taking snapshots, instead it tries once and if the call times out, the volumesnapshot object gets an error. In the background the snapshot is actually created by the first RPC to request the same, but kubernetes does not have a record of the same (i.e the SnapID). Thus, when the snapshot is deleted, kubernetes only deletes the volumesnapshot object and does not invoke DeleteSnapshot RPC against the plugin. This hence leaks snapshots.
The lock fix patch (https://github.com/ceph/ceph-csi/pull/443) alleviates the problem, as it speeds up the operations (including snapshots) but there are still corner cases that may occur leaking snapshots.
BTW, Similarly there can be corner cases in PVC->PV creates and deletes, that can leak images as kubernetes never recorded a success from the plugin. I have seen this happen when working on the performance improvements with the locks. It needs some more analysis, but there are cases when on large RPC response times we may leak an image.
There is the ReadyToUse flag with snapshots that can help, as the CO is supposed to keep retrying once a response is sent with this set to false, to ensure this becomes ready at some point in time. We can only do this post creating the RADOS maps for the snapshot and possibly taking the snapshot, so if these calls take time we will again get back to leaking snapshots. Also, the sidecar code needs to be checked if this is retried as expected by the specification.
Looking at the logs you provided, there are some discrepancies in call numbers (i.e Create/Delete/RPC success etc.) which explains the leaks, but to me it looks like you attempted to invoke delete before the snapshots were created as well.
In my test, I attempted creating 25 snapshots using the attached script, which actually waits for the ReadyToUse flag to become true (which essentially means the volumesnapshotcontent object was created, which is done post a success by CreateSnapshot is received), and then I invoke the delete script.
snap-create-perf.sh.txt
snap-delete-perf.sh.txt
BUT, even with the above and without the lock fix patch, only 9 snapshots were marked ready and had volumesnapshotcontent objects created, for the remaining 16 (I created 25) the snapshots were created in the background but as the calls timed out kubernetes never sent the delete request. IOW, we leaked 16 snapshots in the create phase itself, and delete did not have any role to play post that.
Also, I tried an experiment adding a sleep to the CreateSnapshot call (of 65 seconds, as the timeout for the CreateSnapshot is 60 seconds in the snapshotter sidecar and not 10 seconds which is what the documentation for the same states), this call, as expected timed out on the sidecar and the snapshot was never recorded by kubernetes, but as expected the image was created in the background.
We need to understand what to do next, and also possibly raise this with CSI/kube folks to understand expectations here and how to handle not losing state.

We possibly need the snapshotter side car to implement a more robust timeout and retry mechanism like the provisioner does: https://github.com/kubernetes-csi/external-provisioner#csi-error-and-timeout-handling

We possibly need the snapshotter side car to implement a more robust timeout and retry mechanism like the provisioner does: https://github.com/kubernetes-csi/external-provisioner#csi-error-and-timeout-handling

On further thought, snapshot may not be able to retry endlessly (owing to concepts like freezing/thawing workloads using the volume, prior and post the snapshot, and IO to the volume cannot be held back for long duration's). It may hence require some other form of fix in the kubernetes snapshotter. Will start a discussion there.

We possibly need the snapshotter side car to implement a more robust timeout and retry mechanism like the provisioner does: https://github.com/kubernetes-csi/external-provisioner#csi-error-and-timeout-handling

On further thought, snapshot may not be able to retry endlessly (owing to concepts like freezing/thawing workloads using the volume, prior and post the snapshot, and IO to the volume cannot be held back for long duration's). It may hence require some other form of fix in the kubernetes snapshotter. Will start a discussion there.

@ShyamsundarR can you point me to the snapshot discussion if it already started?


Feature set exec path

https://github.com/takama/daemon/issues/62

@takama Can you see this PR please :)

@krysennn, Yes, of course. But we should finish with #68 before.


[KYUUBI #657] Add udf kyuubi_version

Why are the changes needed?
In this PR, I propose to add kyuubi_version as a user-defined function.
This is also a good example to add other new functions that needed in the Kyuubi system.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

cc @pan3793 @ulysses-you @zwangsheng @turboFei @hddong thanks for reviewing

Thanks, merged for v1.3.0.

late +1


Unable to run Prisma on Netlify with SvelteKit

Hi all,
I'm using Prisma with SvelteKit on Netlify. I'm not sure if the following has anything to do with SvelteKit however, as it seems like a platform binary error.
Invalid `prisma.profile.findFirst()` invocation:

  Query engine binary for current platform "rhel-openssl-1.0.x" could not be found.
This probably happens, because you built Prisma Client on a different platform.
(Prisma Client looked in "/var/task/query-engine-rhel-openssl-1.0.x")

Searched Locations:

  /.prisma/client
  /opt/build/repo/node_modules/@prisma/client
  /var
  /var/task
  /var/task
  /tmp/prisma-engines
  /var/task

You already added the platforms "native", "rhel-openssl-1.0.x" to the "generator" block
in the "schema.prisma" file as described in https://pris.ly/d/client-generator,
but something went wrong. That's suboptimal.

Please create an issue at https://github.com/prisma/prisma/issues/new

Do you know where the query engine query-engine-rhel-openssl-1.0.x actually is located?
Did this work before?
Can you create a minimal reproduction for this and out it into a repo maybe?

A SvelteKit user had a very similar issue, please take a look at his comment https://github.com/prisma/prisma/issues/7019#issuecomment-841079492. Please let me know if it worked for you.

@janpio I don't know. This is on Netlify, where I don't have filesystem access personally. This is the first time trying Prisma out on Netlify, and thus the first time I've encountered this.

I don't think I'm able to make a reproduction repo, as this would require a hosted database somewhere.

A SvelteKit user had a very similar issue, please take a look at his comment #7019 (comment). Please let me know if it worked for you.

@millsp Do you have a code example or explanation of your fix? I find the linked answer rather vague. Netlify docs aren't too helpful in this case either.

@didier It would be helpful if you could provide a repro - no worries for the db.

6 months later no further information, so I think we can consider this closed.
If this ever happens again, please comment here or even better - create a new issue. Thanks.


AppCrach when try to unpin from favorites

Description
When I try to unpin one of the folders on the Favourite left side of the app it doesn't do anything, and when I try to unpin that same folder from the folder ubication the app crashes. When you reopen the app the folder stays pinned
Steps To Reproduce

Open the app.
Go to the folder location you want to Unpin
Right click and click on the Unpin form Favourites option.

Requirements
Unpin the mentioned Folder without the appCrash
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 (22H2): 22621
Log File
debug.log

@Berruti06 thank you for the report, can you check if this is fixed in the preview build? https://files.community/download

I'm able to reproduce this in the latest build <IP_ADDRESS> on Windows Version 10.0.22621.1928.
Related to #12785

Thank you for the update

Hello @yaira2 , I had setup the local env and can see that this issue is no more reproducible

After setting as favorite:

After "Unpin as favourite" :

@Berruti06 thanks for comment, I can confirm that this was fixed in #13046.


Add offset/limit depagination support to Neo4jExecutor

On second thought (and after a few experiments), this won't work because incrementally returning results depends on a ORDER BY clause for deterministic ordering, which means the complete query (or an indexed subset) still needs to run before the query returns...


GFSSF acceptance validator fails with IndexError

ballet version: 0.5.3-dev (b413e30)
Python version: 3.6.3
Operating System: Ubuntu 14.04.5

Description
GFSSF validator fails judging example feature
What I Did
[2019-03-20 13:03:26,350] {ballet: gfssf_validator.py:60} INFO - Judging Feature using GFSSF: lambda_1=0.011564066054355004, lambda_2=0.011564066054355004
[2019-03-20 13:03:26,350] {ballet: gfssf_validator.py:65} DEBUG - Testing with omitted feature: None
[2019-03-20 13:03:26,374] {ballet: log.py:115} INFO - Ballet Validation: evaluating feature performance...FAILURE
Traceback (most recent call last):
  File "./validate.py", line 11, in <module>
    ballet.validation.main(ames)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/validation/__init__.py", line 154, in main
    evaluate_feature_performance(project)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/funcy/decorators.py", line 38, in wrapper
    return deco(call, *dargs, **dkwargs)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/validation/__init__.py", line 25, in validation_stage
    return call()
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/funcy/flow.py", line 38, in wrapper
    return func(*args, **kwargs)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/util/log.py", line 145, in wrapper
    return func(*args, **kwargs)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/funcy/decorators.py", line 55, in __call__
    return self._func(*self._args, **self._kwargs)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/validation/__init__.py", line 122, in evaluate_feature_performance
    accepted = evaluator.judge(proposed_feature)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/validation/gfssf_validator.py", line 67, in judge
    cmi = estimate_conditional_information(feature_df, self.y, z)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/validation/entropy.py", line 270, in estimate_conditional_information
    h_xyz = estimate_entropy(xyz, epsilon)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/ballet/validation/entropy.py", line 186, in estimate_entropy
    selected_cont_samples = cont_features[unique_mask.ravel(), :]
IndexError: boolean index did not match indexed array along dimension 0; dimension is 2930 but corresponding boolean dimension is 5860
The command "./validate.py" exited with 1.

See https://travis-ci.org/micahjsmith/ballet-ames-demo/jobs/508915068

Thanks for leaving this! So far, I've tried replicating this bug in a Jupyter notebook, by reimplementing your feature and then judging it on the GFSSF validator (which it passes). It seems to work fine in this case, so I'm currently looking to see if it might be related to project collection.

Closed by #35


Use standard format for license in Cargo.toml

@bors-servo: r+

:pushpin: Commit 03251e7 has been approved by jdm

:hourglass: Testing commit 03251e7db7b2b9649e77b2928fef107faed002bb with merge f4d8036b7c989acb5b9b1ecfe0a8e47dca64d36d...

:sunny: Test successful - status-travis
Approved by: jdm
Pushing f4d8036b7c989acb5b9b1ecfe0a8e47dca64d36d to master...


Add option to copy path/name/value of setting

Godot version:
3.1 calinou c4e75aa
OS/device including version:
Windows 10
Issue description:
It would be nice if in settings(both Editor and Project) would be option to copy things by right-clicking, because it is possible to do mistake when rewrite it.
I think there should be 4 options:
Copy name - Copy only name of setting e.g.       timestamping_authority_url
Copy value - Copy only value of setting e.g.      true or for colours #ff22aa
Copy path - Copy path e.g.      Export/Android/jarsigner
Copy all - Copy path and value e.g.      Export/Android/jarsigner -> #ff22aa

Same for the inspector, which should work anyways because it has been re-used in settings.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


updating google-auth-library-java to 0.13.0

updating the google-auth-library-java dependency from 0.12.0 to 0.13.0
It includes some changes to logging output that I would like in my downstream project.

Please also upgrade com.google.http-client:google-http-client, as per the failing tests.

Upgrading seems to be more involved than I expected.  I'll move this forward in a different PR.

@sduskis Ah, sorry about that.  The project doesn't build for me locally under normal circumstances due to dependency resolution issues,  so I didn't notice that there additional dependency issues due to my changes.
I pushed one more with what I think are all the necessary version bumps, but as you said it's more complicated than I thought it would be and I don't know if there are any other issues.


Loading VS Code breaks unless VSCODE_PORTABLE envvar is set

Issue Type: Bug

Launch code-oss with the --verbose flag
See the below output, and no GUI:

(electron) 'setName function' is deprecated and will be removed. Please use 'name property' instead.
Unable to create argv.json configuration file in /usr/data/argv.json, falling back to defaults (Error: EACCES: permission denied, open '/usr/data/argv.json')
[main 2020-06-07T09:00:17.021Z] Error: ENOENT: no such file or directory, mkdir
[main 2020-06-07T09:00:17.021Z] Lifecycle#kill()
[main 2020-06-07T09:00:17.022Z] [File Watcher (node.js)] Error: ENOENT: no such file or directory, stat '/usr/data/user-data/User'

Run VSCODE_PORTABLE=~/.config/Code\ -\ OSS/ code --verbose
See the Code GUI load exactly as it's meant to

This has suddenly happened, with no clear reason behind it: I've not updated Code recently, and I've not made any configuration changes. I've checked the output of os.homedir() by running electron -i, after taking a look at the code that locates argv.json:
https://github.com/microsoft/vscode/blob/fe6e663a846857fdeeec685447d18408050de044/src/main.js#L339
os.homedir() returns correctly in the Electron shell, so I can only assume it does here too.
VS Code version: Code - OSS 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-17T22:31:53.325Z)
OS version: Linux x64 5.4.43-1-MANJARO

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4460  CPU @ 3.20GHz (4 x 3192)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
3, 5, 4

Memory (System)
15.60GB (0.19GB free)

Process Argv
--verbose --no-sandbox

Screen Reader
no

VM
0%

DESKTOP_SESSION
/usr/share/xsessions/plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Extensions: none

Hi, I'm not sure why this has been closed as a "developer question" - I'm not an extension developer, nor trying to do any development of VS Code. It's an issue with running Code.


🐛 vitest.workspace.mts Not Dectected by Vitest Plugin

Prerequisites

[X] I'm using the latest version
[X] I've read the relevant documentation
[X] I've searched for existing issues
[X] I've checked the list of known issues
[X] I've read the issue reproduction guide

Reproduction url
N/A
Reproduction access

[X] I've made sure the reproduction is publicly accessible

Description of the issue
As shown in https://knip.dev/reference/plugins/vitest, vitest.workspace.ts|js|json is picked up, but mjs|cjs|mts|cts extensions are not. I know for sure that vitest.workspace.mts is valid to Vitest, so should we just match the extensions we are looking for for the normal Vitest configs?

Yes, that makes sense. Any chance you would be willing to submit a PR to fix that up?

@webpro sure! See https://github.com/webpro-nl/knip/pull/785.


Adding casting argument to astype (and other casting functions/methods)

Based on recent discussion, current astype behaves as if casting is set "unsafe". It may be worthwhile to add this argument as default "unsafe" while also allowing it to be configured. In particular "safe" may be of interest to allow users to catch any unexpected or unintended castings in library code. This casting flag may be useful in other places where castings occur.

For astype it seems good to me to allow it in the sense of "unsafe" casting.
However, somehow, complex -> int/float does seem less well defined then float -> int.  Python has it this way, since int(1.5)is OK, butint(1j)orfloat(1j)` is not.
So when it comes to arr.astype() I am not yet certain about removing the warning from the NumPy main namespace.  But with a warning astype(complex_arr, real) seems not super useful from a library perspective (you will need to use arr.real or real(arr) to avoid the warning).
Surprisingly, am not even certain that a complex -> real needs to be defined at all (as a proper cast), are there use-cases where you need it to work but calling real() or arr.real is not clearer?

Interesting thanks for the context.
No strong feelings about the warning in NumPy.
Main interest here is ensuring library authors don't run into subtle bugs or at least have easy ways of avoiding them. That said, maybe it is worth thinking about expectations of safe casting.
Yeah the use case for complex -> real is what I was wondering as well in this comment ( https://github.com/data-apis/array-api/pull/445#issuecomment-1148024033 ). Also not clear on why it is needed.

Yeah the use case for complex -> real is what I was wondering as well in this comment ( https://github.com/data-apis/array-api/pull/445#issuecomment-1148024033 ). Also not clear on why it is needed.

Trying to look at this from a different angle, I hope. Based on #427 (real) and #446 (conj) I think we are leaning toward a more restrictive API design as opposed to permissive. I seem to remember the permissive choice was favored during one of the meetings, but I can't recall it was for which library (and I also forgot which side I picked, I'd say restrictive but I am not certain anymore after a long week...😅) That's probably why I let go in #445 (astype). But, for the sake of consistency it'd be nicer if astype does not allow complex -> real?
cc: @kgryte

In the specification, we opted to be restrictive in disallowing complex -> real. As such, I will go ahead and close this issue.


Generate some ml type definitions

Progress on #215. This auto-generates a number of ml definitions from the rust ones. I've left the visitor definitions as-is; that feels incomplete, I hope to find a cleaner solution when more of the types are auto-generated.
Nice bonus: this copies the doc-comments present on the rust side.

I hadn't thought of the fact that doing this would allow to port the documentation from the Rust side to the OCaml side: this is extremely useful. The only minor change I would like to do is a better indentation of the multi-line comments (as you can see, having no indentation is a bit ugly).


2017 South Carolina married deduction incorrect

Should be 12600, not 112600

Thanks!  Fixed in https://github.com/taxee/taxee-tax-statistics/pull/16


[spi_device] Dummy CSB may trigger an upload interrupt

We test 3 types of dummy SPI transaction and hope it doesn't affect anything.

active CSB without sck
toggling sck without CSB
send request/opcode less than 1 bytes (only for Flash or TPM mode)

However, in flash mode, we rely on CSB falling edge with sck to clear a flag - csb_cmdfifo_set.
In this case, we only have CSB drop without sck, so csb_cmdfifo_set isn't cleared. However, we do detect sys_csb_deasserted_pulse_i, so it triggers an upload interrupt. (sys_cmdfifo_set leads to an interrupt.)
We may have to constrain the dummy case 1 with FW mode only.
  always_ff @(posedge sys_clk_i or negedge sys_rst_ni) begin
    if (!sys_rst_ni) sys_cmdfifo_set <= 1'b 0;
    else if (sys_csb_deasserted_pulse_i && csb_cmdfifo_set) begin
      sys_cmdfifo_set <= 1'b 1;
    end else begin
      sys_cmdfifo_set <= 1'b 0;
    end
  end

waves:
/mnt/disks/filestores/opentitan-shared/users/weicai/scratch/min_idle/spi_device-sim-vcs/failed/1.spi_device_flash_and_tpm_min_idle.3189529232
LMK what you think. I'm fine to tag this one to M3
cc: @tjaychen

Another related issue:
After SW updates addr4b to 1, this update could be cleared if the next transaction is a dummy item.
In this case, the dummy item has one spi_clk cycle which isn't enough to sync up the SW programmed value to the final spi_cfg_addr_4b_en_o, as spi_csb_asserted_pulse_i remains 0.
  always_ff @(posedge spi_clk_i or negedge sys_rst_ni) begin
    if (!sys_rst_ni) begin
      spi_cfg_addr_4b_en_o <= 1'b 0;
    end else if (spi_addr_4b_set_i) begin
      // This event occurs when EN4B command is received
      spi_cfg_addr_4b_en_o <= 1'b 1;
    end else if (spi_addr_4b_clr_i) begin
      // EX4B command raises the clear event
      spi_cfg_addr_4b_en_o <= 1'b 0;
    end else if (spi_csb_asserted_pulse_i & !addr_4b_en_locked &
                 addr_4b_en_sw_update_condition) begin
      // Update
      spi_cfg_addr_4b_en_o <= spi_reg_cfg_addr_4b_en_sync;
    end
  end

However, sys_csb_deasserted_pulse_i is set after CSR is released, so this clears the reg2hw_cfg_addr_4b_en_q_i flag and this CSR update is wiped completely.
  always_ff @(posedge sys_clk_i or negedge sys_rst_ni) begin
    if (!sys_rst_ni) begin
      hw2reg_cfg_addr_4b_en_de_o <= 1'b 0;
      hw2reg_cfg_addr_4b_en_d_o  <= 1'b 0;
    end else if (sys_csb_deasserted_pulse_i
      && (sys_cfg_addr_4b_en_sync != reg2hw_cfg_addr_4b_en_q_i)) begin
      // update!
      hw2reg_cfg_addr_4b_en_de_o <= 1'b 1;
      hw2reg_cfg_addr_4b_en_d_o  <= sys_cfg_addr_4b_en_sync;
    end else begin
      // Keep value 0 otherwise
      hw2reg_cfg_addr_4b_en_de_o <= 1'b 0;
      hw2reg_cfg_addr_4b_en_d_o  <= 1'b 0;
    end
  end

Looks like if we want to test flash mode with dummy items, we need to enforce a minimum spi_clk cycles.
Waves:
/mnt/disks/filestores/opentitan-shared/users/weicai/scratch/min_idle/spi_device-sim-vcs/failed/3.spi_device_flash_and_tpm_min_idle.555308691

Changed this to M3 and constrain sequence not to issue these dummy req for now

Sorry for late reply. I will take a look

Sorry for late reply. I will take a look

Thanks @jeoongp

Triaged for spi_device. Assigning to M2.5 with https://github.com/lowRISC/opentitan/labels/Priority%3AP1 because we need to check with the product team if this could be a problem.  If so, we need to fix this with high prio.

There are two parts to this issue:

A spurious CSb due to reset has to be prevented with a pull-up on the CSb wire on the PCB. No changes to spi_device are necessary in this regard. The PCB reference design guidelines need to include this pull up and specify a resistor value range that achieves the desired pull-up strength.
It may be possible that CSb is active for less than 3 SCK cycles. spi_device has to behave in a well-defined way in this case. We have to remove the constraint that prevents this from happening in DV (see https://github.com/lowRISC/opentitan/commit/27a5beeb0622e0f1a5353bc3f426ef3a938f5337) and figure out what exactly happens in such cases and what may fail.

Keeping this as https://github.com/lowRISC/opentitan/labels/Priority%3AP1.

It may be possible that CSb is active for less than 3 SCK cycles. spi_device has to behave in a well-defined way in this case. We have to remove the constraint that prevents this from happening in DV (see 27a5bee) and figure out what exactly happens in such cases and what may fail.

While product team feedback is that this situation is unlikely due to external pullups on CSB, we should still amend the testbench to remove this restriction, and check that block returns to a usable state and record any interrupts generated or other side-effects.

estimate 8
remaining 2023-03-22 8

Note that there is a number of open issues about CSB behaviour and robustness to errors, so some estimated effort (if deemed necessary) may overlap between those issues. (https://github.com/lowRISC/opentitan/issues/10058 https://github.com/lowRISC/opentitan/issues/12747 https://github.com/lowRISC/opentitan/issues/15517 https://github.com/lowRISC/opentitan/issues/15721)

Hi @johngt, I believe this issue was left as P1 to scope a directed test case to ensure we can recover from an invalid scenario: e.g. remove 3SCK constraint and determine if we can recover in this case.
Given where we are at wrt M2.5.1 I believe we can live without this test and rely on system level constraints.
CC: @msfschaffner wdyt?

Discussed with @msfschaffner offline. Ok to move to M2.5.1 as this falls into the protocol checker category. No expected RTL changes.

Think you mean 2.5.2 - @moidx ?

Hi @hcallahan-lowrisc, are you planning to take a look at this as part of V2.5 sign-off, or is this falling into negative testing (lower priority). Thanks

Hey @moidx , work on this is issue is my main priority at the moment. Thanks

Triaged as part of M2.5.2 sign-off and decided to move to V3 to focus on coverage improvements.


[Update Request]: Google.AndroidStudio 2<IP_ADDRESS>

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

[x] Does your manifest conform to the 1.5 schema?

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

Microsoft Reviewers: Open in CodeFlow

Pull request with fixes to version 2<IP_ADDRESS>: #128551

Possible duplicate package entry.
Similar installer SHA256 hash found in manifest:

InstallerSha256: cfb4cf2485493b7de74bcdc5d1c0421b70291f3e45257cb0e64da745c48870bc

manifests\g\Google\AndroidStudio\2<IP_ADDRESS>

It appears that the developers rolled back to a previous version.


Update eng_sys_checks to highlight the queue time options for skipping tox

Sorry @kristapratico , wish I had this already for you.

/check-enforcer override


Refactor surface

This PR refactors the surface handling in several ways:

Caches the surface fluxes object, so it can later be used by the edmf model
Caches the surface fluxes inputs, so that we can more simply dispatch
Changes (maybe fixes) get_surface_density_and_humidity to be mutating (now set_surface_density_and_humidity!) cc @szy21, @LenkaNovak, maybe we can discuss this change?
I've kept surface_conditions for now, just to minimize the blast radius, but I'd like to remove that since it's basically directly dependent on SurfaceFluxConditions and the surface flux input types (depending on which one is used)

Ultimately, this should allow us to delete nearly all of edmf's get_surface function, and just accept the already created SurfaceFluxConditions.

I had to make a few more changes, going to merge but I'm happy to change names later if people prefer different names. I think the structure is an overall improvement though.

bors r+


override resolvers when creating manager

ModelManager (and ModelGroup) always use the default resolver classes. However, a project may want to use a different set of default behaviors, such as adding authentication, authorization, or logging. Right now, they must define those custom wrappers, then replace every single generated resolver with them manually. Instead, allow setting the resolver class to use for each type of resolver.
Perhaps through class attributes:
class CustomCreate(CreateResolver):
    ...

class CustomManager(ModelManager):
    create_factory = CustomCreate

class CustomGroup(ModelGroup):
    manager_cls = CustomManager

Or perhaps through parameters?
manager = ModelManager(model, create_factory=CustomCreate)
#
group = ModelGroup.from_declarative_base(Base, create_factory=CustomCreate)

I'll start with attributes, can add parameters later as a second layer of override if we find it's needed.


Reduce overhead in health_checker_impl by fixing alignment

Commit Message:  Reduce overhead in health_checker_impl by fixing alignment
Additional Description: Moves many of the smaller members of the class to the end to save small amounts of memory
Risk Level: Low

/retest


Add .git to webpack URL for package.json/yarn.lock

We're having yarn install issues on Travis CI. Maybe this will help?

Seems to have done the trick! Maybe I should add a note to this Stackoverflow answer that I had used, which does not include this evidently sometimes necessary .git extension ... https://stackoverflow.com/a/43636577/4009384


Improve accuracy of case-C test

Previously the select raa functionality was tested, piggy backing on the use-raa feature. That was a bug fixed in Middleware.
See: https://www.pivotaltracker.com/story/show/175056146 for a full bug report and more backgrounds
See: https://github.com/OpenConext/Stepup-Middleware/pull/315 use this branch to run the behat tests
All other components should run on the latest develop

Were you able to run all tests in the dev-VM? The change seems indeed the reflect the situation, but I'm not able to run them all due to the migration of all components and the lack of time to upgrade solely for a review?


point inside polygon

Hy guys,
Is it possible to extract from an holoviews image all the points inside the polygon (done with FreehandDraw or PolyDraw functions) ?

This is a usage question so should probably be filed on StackOverflow or asked on Gitter. I also think this particular question is beyond the scope of holoviews. I think you will want to convert the drawn polygon to a mask (e.g. look at this answer https://stackoverflow.com/questions/3654289/scipy-create-2d-polygon-mask) and then you can multiply that mask with your image to get the datapoints inside the polygon.


mbed test

Please add to the documentation that new green tea test main.cpp files must be two folders deep in the TESTS directory.  This is not mentioned anywhere and took forever to discover!

Your example seems to imply you can have a file with main {} in the root of an mbed program, but when I try to run mbed test, I get a compiler error.  Did I do something wrong or can you not really run mbed test in the root of your program ( b/c you need your program's main to live somewhere, right?)

Thanks for the feedback!
Re 1) Where would you expect to find that information?
Re 2) You did nothing wrong; You can not run mbed test from the root of a program. You are correct that the program's main will conflict with the test main.

Thank you for the quick response!!!

Section 8 of the README.md for starters, and what really through me off is that in the greentea section of the mbed handbook it says """
Host test detection

When developing with mbed OS, Greentea detects host tests automatically if you place them in the correct location. All tests in mbed OS are placed under a TESTS directory. You may place custom host test scripts in a folder named host_tests in this folder. For more information about the mbed OS test directory structure, please see the mbed CLI documentation."
Maybe add a note there as well.

Well then I guess I would request a feature add where the tool can ignore specifically named .cpp files, like maybe "app-main.cpp" or something.  Greentea is great, and we're really excited to get it into our CI flow!  But as it stands, I'll probably have to wrap the call to mbed test with a little script to temporarily rename our program main to a different file extension.

I opened a handbook PR, armmbed/handbook#343, which I hope improves upon the text you just pointed out. Rendered

@Alex-EEE Are you happy to close the ticket if resolved?


towards a platform-independent artifact

I started the process of creating scoinReckless which will have platform-independent, yet inefficient and somewhat reckless implementations of the things.
One reason for doing this is that it will allow us to easily create little scripts and whatnot which will not need to pre-suppose any javascript, jvm, or native dependencies. So cooking up examples or simple little scoin utilities using tools like scala-cli should be much easier.
All these platform-independent implementations can live in the parent scoin.reckless package (inside the scoin sbt project itself), and the sbt cross-project scoinReckless is therefore simply a "stub." This seems to be the cleanest solution for now until when/if we do a massive refactor.
So, all that to say, in order for scoinReckless to be useful, we need to publish it as a different artifact. I do not know how the maven magic line in the build.sbt file works which does the publishing of the artifacts, so I left that part out for now.
Right now somebody can pull in scoin into their project by doing something like:
//> using lib "com.fiatjaf::scoin:0.5.0" (for scala-cli / mill) or libraryDependencies += "com.fiatjaf" %%% "scoin" % "0.5.0" (for sbt projects).
I think the goal should be to publish it so that all they need to do is change "scoin" to "scoin-reckless" and things just work.

I thought it would have been relatively easy to find pure scala implementations of the various algorithms (sha256, ripemd160, ...), but unfortunately some quick searching has not turned up very fruitful yet. Worst case we will have to implement the algorithms from scratch ourselves (which is not a very high priority right now).


Update install scripts.

Updating dotnet install files.

@stephenmichaelf,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

I also split into two tests so we get better messages on failed tests. This time I had to update both and I got the same error message twice (as opposed to getting two separate ones).


Coins and CoinsCollector: cyclic reference error in Godot 3.1

When trying to run the game on the very latest from-source Godot I get the following error message:
Parser Error: Could not fully preload the script, possible cyclic reference or compilation error.

Coins.gd includes CoinsCollector.gd and vice versa.
I expect that the project is made with/for Godot 3.0.x and that this issue is caused by an addition to the source since then.

Okay, thanks for the explanation. By the way, I tried resolving the cyclic dependency (by commenting out one of the references) and ran into a bunch of other 3.1-issues.
Should I close the issue (because this repository is 3.0.x-only for now) or keep it open?

It's fine for this one. If there's a backwards-compatible way to fix all issues for 3.1, people are welcome to contribute. I just won't do this work now, got enough on my plate already


Hide Geo-Location Details on Location Detail Views based on Location Type

As ...
Nelly - Network Engineer
I want ...
to be able to hide geo-location details (lat, long, address) on the detail views on locations (added in #2954) it's not relevant for.
So that ...
locations that are more logical or narrowly specific (room, cabinet, etc) are not cluttered with always empty sections.
I know this is done when...
I can choose by Location Type to not see:

on Location instances.
Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[X] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes

Additional property(ies) on LocationType to save the preferred state of this.

External Dependencies
None

Related but this is a more narrow-scoped version of: https://github.com/nautobot/nautobot/issues/1500


Master has tests failing

While working on https://github.com/mickhansen/dataloader-sequelize/issues/79 I discovered that there are errors when running the tests on a clean clone, on my machine. See errors below (the 4 tests throw the same error).
 39 passing (5s)
  4 failing

  1) belongsToMany string through simple association "before all" hook:
     TypeError: Cannot read property 'field' of undefined

Happens on Master.
Also, it looks like the CircleCI is down :( Maybe a good time to switch to Github actions?

Ah yeah that test fails for a specfic sequelize version i believe.


We should vendor skyDNS

We build skyDNS and push a container and use that container but we have not vendored skyDNS code.

I left a response in #9290 regarding this question and etcd. I think it might make sense to "clone" skyDNS under the GoogleCloudPlatform namespace and doing builds from there. Sync with upstream along the way.

  I can't stick binaries under the Godeps dir because Godeps will
purge it.  If I put it outside the Godeps dir then Godeps will try to
vendor in all of the libs that it uses - including its own libs!
On Fri, Jun 5, 2015 at 8:28 AM, Kelsey Hightower<EMAIL_ADDRESS>wrote:

I left a response in #9290
https://github.com/GoogleCloudPlatform/kubernetes/issues/9290 regarding
this question and etcd. I think it might make sense to "clone" skyDNS under
the GoogleCloudPlatform namespace and doing builds from there. Sync with
upstream along the way.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/9289#issuecomment-109328508
.


[Feature/Bug] Do not include unidentified items adding activations

Is your feature request related to a problem? Please describe.
Currently activations are added for unidentified items that are added to a PC's sheet. This gives away information to the player that they should not have.
Describe the solution you'd like
Not include unidentified item activations.

it does? That's weird, my intention when making it was for it not to do that, I'll double check

Thank you for your work on this mod!
I did not do extensive testing but the main item that it happened with is one of my players is carrying 'The Whispering Reeds' book from the Abomination Vaults AP. It is adding two activatable abilities.

Hmm okay, I'll look into it


chore: switching to fast-qs for esm-friendly build

Checklist

[x] Include a description of your pull request and instructions for the reviewer to verify your work.
[ ] Link to the issue if this PR is issue-specific.
[ ] Create/update the corresponding story if this includes a UI component.
[ ] Create/update documentation. If not included, tell us why.
[ ] List the environments / browsers in which you tested your changes.
[ ] Tests, linting, or other required checks are passing.
[x] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
PRs will be broadly categorized in the change log, but for even easier scanning, consider prefixing with a component name or other useful categorization, e.g., "BaseButton: fix layout bug", or "Storybook: Update dependencies".

[x] PR has been tagged with a SemVer label and a general category label, or skip-changelog.

These tags are used to do the aforementioned broad categorization in the change log and determine what the next release's version number should be.
Release Drafter will attempt to do the category labeling for you! Please double-check its work.

Description
Removing qs and using fast-qs instead. Fixes javascript errors on content pages.
Instructions to test

Tested in the following environments/browsers:

Operating System

[ ] macOS
[ ] iOS
[ ] iPadOS
[ ] Windows

Browser

[ ] Chrome
[ ] Firefox ESR
[ ] Firefox
[ ] Safari
[ ] Edge

Approved!