What is the status of .NET code?

I have noticed that the C# code hasn't been updated for more than a year, while Java/C++ look active. Is C# code still relevant and is it compatible with updated Java generation tool? Why the development paused?

I've discontinued the port to C# by lack of time. I don't think any important changes have occurred since then but they are probably a few bug fixes that have been missed and would have to be ported to be on par with Java.
Martin has been talking about implementing SBE v3 and that's something I would like to port to C# as it would make SBE a lot more general purpose, and also a lot more usable in my day to day projects.

On 7 Sep 2015, at 01:05, Victor Baybekov<EMAIL_ADDRESS>wrote:
I have noticed that the C# code hasn't been updated for more than a year, while Java/C++ look active. Is C# code still relevant and is it compatible with updated Java generation tool? Why the development paused?
—
Reply to this email directly or view it on GitHub.

Thanks! When v3 is expected?

I plan to start working on the IR, compiler, and Java implementations of RC3 within the month.

I guess it will be compatible with RC2?
It is very nice to here that SBE is becoming more general purpose. Where microseconds do not matter, I want to start using blobs from an existing serialization scheme (BSON), so that SBE messages are just DTOs with a header and payload (like another OSI 6 MIME). I will get benefits of the standard message header with version and templateId dispatch, and could then migrate to primitive SBE fields step-by-step when needed. At the first glance, I find it hard to map complex POCOs directly to SBE vs. automatic and versatile BSON. But then, I have a number of blittable structs that go to a byte stream directly, and SBE looks perfect to add some structure and standard here. Do you think this is an intended and sane, general purpose, use case, and it won't be significantly affected by RC3 and further changes?

@buybackoff I've used SBE as a framing protocol for other protocols. Even the current version is totally fine for that. You can just write an SBE header with a variable length field of bytes at the end, and plop your legacy serialization scheme in that variable length byte field. Works completely naturally.

I am also looking for replacing the general protocol serialization atleast
the in house. Way to go Sir.
On Sep 7, 2015 8:53 PM, "Richard Warburton"<EMAIL_ADDRESS>wrote:

@buybackoff https://github.com/buybackoff I've used SBE as a framing
protocol for other protocols. Even the current version is totally fine for
that. You can just write an SBE header with a variable length field of
bytes at the end, and plop your legacy serialization scheme in that
variable length byte field. Works completely naturally.
—
Reply to this email directly or view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/286#issuecomment-138324669
.

@RichardWarburton thanks! yeah, SBE looks so natural as a replacement of a custom [message type + length] header, reinvented every time some messaging over a byte stream is needed. The question is closed, will start to gradually adopt SBE.
BTW, SBE docs say explicitly that framing is beyond SBE scope, @RichardWarburton how do you use it?

I interpret the docs as saying that SBE messages will probably be framed by another protocol on the wire and that SBE isn't trying to replace TCP or act like a messaging product. It doesn't mean you can't wrap another protocol with them.

If you have a byte stream, how do you get from it byte arrays, which SBE takes as input? Is it another length header, or one could use SBE directly to infer the length of a message (I am still not sure that the latter is possible, only block size is in the header)? Some protocols, such as ZMQ, are already framed and fit perfectly to SBE, but I wonder how to use a plain stream over TCP socket.
SBE is streaming, but to get a buffer we need to read the entire message size before processing it (e.g. this comment). With the current implementation it looks like the only benefit of streaming is memory access pattern, but we cannot start processing a message until we get all its bytes.

In my case I'm using Aeron for the messaging protocol. So when I receive a message I know its been completely received and ordered. ZMQ has the same property.
If you want to use a streaming protocol like TCP then you /could/ wrap the header, see if you've read enough data to process the whole message and implement that logic to make it work. However this approach is quite messy and is basically conflating OSI levels 4-6 into one. I would recommend using something like Aeron or ZMQ.


[BUG] roblox crash

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
roblox warning crash when i ignore it roblox can still run but when i press the ok button on the warning it makes roblox exit
https://github.com/pizzaboxer/bloxstrap/assets/130255455/7391f665-a839-451d-bf88-153a6f7bd149

@pizzaboxer i need ur help

Does it happen when you join or leave a game?

Does it happen when you join or leave a game?

yes always

The video won't load for me, so I'll assume it happens only when you join. Upload your configuration please: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


Example with a major UI framework?

Hello, first thanks for this, it looks very interesting.
I create this issue because I have no clue how to integrate in one of major frameworks like React, Vue or Svelte ?
Is there someone who have achieved this and could share code snippet to complete documentation?

Thanks for asking for this.  More info and examples for this are needed for sure.
See the example in the documentatoin of how runcss is integrated with webscript:
// Integrating RunCSS with Webscript
function createElement (type, props, ...children) {
  if (props.class) {
    processClasses(props.class)
  }
  return createDOMElement(type, props, ...children)
}

// Create the builders used to build DOM elements
const { div, img, h2 } = builders(createElement)

React and Vue have the same or similar createElement function and so can be integrated the same way.  For example here is react:
// Integrating RunCSS with React
function createElement (type, props, ...children) {
  if (props.class) {
    processClasses(props.class)
  }
  return React.createElement(type, props, ...children)
}

// Create the builders used to build DOM elements
const { div, img, h2 } = builders(createElement)

Thanks for this, I think React and Vue are covered.
For Svelte: https://svelte.dev/docs
as it is a compiler, it has a different API. I guess what I need is around the bottom of their documentation, but I don't understand how to use it. If it is clear enough for you, would you mind sharing your understanding of how integrating runcss in Svelte?

@antoninadert I'm not very familiar with Svelte.  At the moment I don't know how to integrate with it.

Thanks for this information. I may ask on the Svelte discord. If I get something I may update here again.
:)

Fixed in new rewrite thanks to the watcher API.


godep support for goxc

Currently godep does not support goxc command. This is useful for cross compiling binaries.
If we freeze using godep, it would make help to support cross compilation when we want to make production releases.
It would be good if this can be supported
https://github.com/laher/goxc

duplicate #34

I use gox to cross compile several projects that are under godep.
These tools run the go command you need to rewrite your dependencies (godep save -r ./...), then you  will no longer need to do godep go .... and will only need to use godep when adding new files to your application or managing dependencies.


More helpful test.dart output

Each error section is numbered, so you can all be sure you're talking about the same one.
A message is printed at the very end telling you how to find the error blocks in the verbose logs.

I made it 512 characters instead of 10. it's not the same as 10 lines but it's probably roughly equivalent.
I added a test which, for practical reasons, is linux-only, but the code in question isn't platform-specific so that's probably fine and avoids refactoring all of this to make the Process.start call mockable.


fix: compact number in email should consider format

Describe your changes
This PR is a result of feedback from @gal-agmon on #1515
Compact numbers in email consider number format. This changes PHL typhoon emails,
Before this PR, https://us18.campaign-archive.com/?u=e71f3b134823403aa6fe0c6dc&id=ce2d4c8cfb
After this PR, https://us18.campaign-archive.com/?u=e71f3b134823403aa6fe0c6dc&id=44951c89fb
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have added tests wherever relevant
[x] I have made sure that all automated checks pass before requesting a review

@gulfaraz so from 10% onwards it's rounded to tens of percentages? This is not what you said in the meeting, nor what Gal Said in the meeting right?

@gulfaraz also, this is not correct in the new email

@jannisvisser Thanks! Not much was said in the meeting. These changes are based on a review from weeks ago. I'll try to find some notes and update this PR.

The indicator value shown in the email event is incorrect. See review here and IBF Dev Chat for details.

In our emails we used to show the total indicator as % of houses exposed: Approximately 7,23 % of houses exposed
This is a total of all the percentage values [0.53, 0.51,  0.5, 0.47, 0.45, 0.39,  0.3, 0.3,  0.3, 0.26, 0.25, 0.17, 0.17, 0.16, 0.16, 0.14, 0.14, 0.12, 0.12, 0.12, 0.12, 0.11, 0.11, 0.11, 0.11, 0.11, 0.11, 0.08, 0.07, 0.07, 0.06, 0.05, 0.05, 0.05, 0.05, 0.03, 0.03, 0.03, 0.03, 0.03, 0.03, 0.03, 0.02, 0.02, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01, 0.01] = 7.229999999999999
Which should be interpreted as 723%.

The old indicator is wrong and there is no easy way to calculate the correct value without significant refactoring. So I removed the indicator following @jannisvisser's advice. After this change, the emails look like,

PHL typhoon https://us18.campaign-archive.com/?u=e71f3b134823403aa6fe0c6dc&id=d31bea6478
PHL flood https://us18.campaign-archive.com/?u=e71f3b134823403aa6fe0c6dc&id=591b54ea33
UGA flood https://us18.campaign-archive.com/?u=e71f3b134823403aa6fe0c6dc&id=625a1ea130


Able to connect same table in Lineage

https://user-images.githubusercontent.com/83201188/149720632-9edc67ea-cdfa-48ac-93a4-ade6e8784b16.mov

It was because of issue #2241 and now it got fixed so closing this issue as well.


Out of Date PagerDuty

I used to work on this project and yesterday I got some PagerDuty alerts to my phone. Can you remove me from the comms list for pagerduty alerts?

hey @ruhatch ! :wave: I'm sorry you got alerted, I will definitely look into it.

My apologies, we were revamping alerts this past week and some low priority environments got assigned to an old user that was never purged. If our paths cross again, I owe you a beer! Your user has been properly deleted now.
Sam


[Grid2] Layout do not work the same way as Grid

Steps to reproduce
Steps:

Open this link to live example: codesandbox
The sandbox start with Grid (old), on the first selector, select 5 or more item, layout work as intended

Click on the top button to switch to the Grid2 component, select 5 items on the first selector, the layout is broken

Current behavior
No response
Expected behavior
No response
Context
The layout i'm using here may simplified (i could use one column grid and two row grid) but it need to be that way, there are a lot of style i did not include to be as simple as possible
Your environment
No response
Search keywords: Grid2 layout, Grid2, Grid

Grid2 Layout do not work the same way as Grid

Yes, that's the point the building the Grid2. Based on your code, I think you are using the Grid incorrectly.

Why do you need a <Grid direction="column">? using <Stack> makes more sense.
The layout looks overcomplicated. Maybe if you can explain what kind of layout you want to achieve, then we can justify if the Grid is working correctly or not.

Thanks for the reply.
I updated the sandbox with spacing for the position.
I need to have 2 block of 4 selectors, one on the left and one on the right. With a bigger separation on the center. (hope you understand what i mean by that)
I'm only using Grid for this because it is state on the MUI documentation :

Stack is ideal for one-dimensional layouts, while Grid is preferable when you need both vertical and horizontal arrangement.

Also, I'm thinking, if a layout like that is broken with Grid2, what else could break ? It is supposed to create multi dimensional layout, it should probably not be broken like that on a still simple layout

Thanks for the help, it fixed my issue ! I'm gonna get ride of the direction props if it should not be used anymore for nested grid like this one !


feat: Enable consumers to access the full response payload

Update the Response public API object to include full_response. this is non-breaking.
Passing the full response from the executing object to the client is currently done via the client checking with the object, versus it being part of the response,
In order to change this it requires changing the much more changes, as all the tests are using GraphQL::Schema for the execution of the graphql

Hey, thanks for this contribution! It looks good to me.

🚢  In 0.24.0! Thanks again.

My pleasure thanks for the swift review and easy contribution :)

Hi, thanks for this, this was actually very important for a use case that we had and we were considering implementing these changes.
That said, it is very confusing that the new method named full_response returns a Hash of containing just the headers of the response.
> result.full_response
=> {"server"=>["nginx"],
 "date"=>["Fri, 22 Nov 2024 16:08:46 GMT"],
 "content-type"=>["application/json"],
 "content-length"=>["1337050"],
 "connection"=>["close"],
 ...

Is this the actual intended behavior? I would believe that such would have the newly exposed method called headers, not full_response.


Startify custom indices is delayed when pressing certain keys

My startify_custom_indices is ['f', 'd', 's', 'a']. When I press f or d it waits for timeoutlen before opening the file. I am wondering why it only does this for f and d, not s or a? Is there a way to fix this besides setting timeoutlen to a small number? I want to avoid this problem.

You most probably have other mappings that start with f or d, but none that start with s or a. Check :verb nmap f etc.
See if enabling :h g:startify_mapping_nowait works for you.

let g:startify_mapping_nowait=1

Yes, this fixes the problem.

Great!


Add average line and data point lines to line chart

Add additional line definition options for line chart that would include an average line and data points lines. Data point lines are lines that go from the label up to the chart line.

added on 0.0.7, check this sample:

dashed lines and dotted lines not supported yet,
but I will add it ASAP.


Aras

Aras's Social Media

Aras's Social Media Links

 Instagram

 TikTok

 Snapchat

Aras's Social Media

Aras's Social Media Links

 Instagram

 TikTok

 Snapchat


[ENTESB-22376] Update to Spring Boot 2.7.18 and Spring Framework 5.3.31

Fixes: #

@igarashitm may I ask you to look at it? No rush though ;)

There seems to be some unstable download for nodejs, I'll keep an eye on and re-run the failed check later


Avoid infinite recursion when exporting traces

Let's say we want to use HTTP.jl to export traces. This operation itself will create another new trace, leading to infinite recursion.

https://github.com/open-telemetry/opentelemetry-python/blob/b6a1b22fa65f41bdefb01d64b76e5e793d039f6d/opentelemetry-api/src/opentelemetry/context/__init__.py#L168-L174


Map FIL-RetroPGF-2 apps to OSO names

We received a CSV; need to map projects to OSO names

Analysis here and spreadsheet here


Refactor to ES6

Either simply write the entire source in ES6 and require node 4 in the package.json or implement transpilation with Bable.

This has been done in my recent clean up in https://github.com/reactivepod/fido/pull/4 via require('babel/register');.


Intermittent "Missing row key at last chunk in cell" error

Does this issue affect the google-cloud-cpp project?
Per @igorbernstein2 in the slack, this sounded like a language-specific issue, so probably.
What component of google-cloud-cpp is this related to?

Cloud Bigtable.

What version of google-cloud-cpp are you using?
v0.10.0 (ec468fbc06cca36802323c50d32180a149ec467a)
What operating system and version are you using?
Debian Stretch
What were you trying to do?
I was querying a range of rows, using a google::cloud::bigtable::Filter::Latest(1) filter. This occurred with a couple of different ranges, although the times I noticed it it was all on ranges that were within relatively close proximity of each other in the key space. It was too intermittent, and went away too quickly, to debug much.
What did you expect to see?
To query rows, then CHECK the StatusOr returned by the RowReader iterator, and have the CHECK succeed.
What did you see instead?
The CHECK failed and the status message said Missing row key at last chunk in cell [INTERNAL]

Note that this behavior seems to have gone away, and only caused intermittent issues for an hour or two, and was on a development BigTable instance. I have not observed this particular issue before, with this exact setp.

Thank you for filing the detailed bug report.  I am going to assume this is a library bug and go from there.

Hmm, just hit this again, in a different part of the keyspace of my table than I had issues with last time. I'll see if I can figure out any patterns.

System certainly wasn't under any particularly high load--CPU was at ~30-40% (in the middle of write operations going on in parallel--I have not noticed any writes failing). I am reading from a different part of the key space than I am writing.

Yeah, still way too little information to figure out if there're any meaningful patterns (the thing about the ranges that had errors being in similar parts of the keyspace in the original ticket may've just been luck). Our usage of the table means we don't have a particularly consistent read load (it is a developer instance, and we are still using it largely as an experiment currently), so we don't actually have a huge sample size to provide reports on.

The code generating this error is pretty easy to find:
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser.cc#L87-L95
I have not looked at this code in a while, but nothing jumps at me as an obvious problem.
Is there a chance you can add some debugging information?  Maybe print the .DebugString() for chunk?

Sure, I can add debugging info and then report it when I encounter this next. Anything other than chunk.DebugString() that you'd suggest?

The current state of this class would be useful.  Specifically: row_key_, cells_.size(), cell_first_chunk_, last_seen_row_key_, row_ready_, end_of_stream_.  I think cell_.row might also be handy.

I ran the unit tests and the code in question is triggered only on "invalid" conditions.  Which is not to say that the code is correct, the code could also be triggering on valid conditions or the tests could be broken.  It just says that we did not expect this code to be executed when the messages are flowing normally.
@igorbernstein2 I think the last time we took a snapshot of the test suite for the C++ client was about 2 years ago (I just created #3215 to update this).  I assume there is a new version since then?  Does anything here ring a bell?
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser_acceptance_tests.inc#L40
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser_acceptance_tests.inc#L57
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser_acceptance_tests.inc#L871
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser_acceptance_tests.inc#L1356
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser_acceptance_tests.inc#L1373
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser_acceptance_tests.inc#L1398

I was able to trigger this error (not sure if it shares a root-cause or not) by Ctrl-C'ing the process in the wrong spot. This may point to me poorly handling some threading or signal handling (although the previous instances were not due to sending SIGINT to the process), although I didn't think I was doing anything problematic.
Since I did capture the debug information, and since it may be the same root cause, here's what I got:
chunk.ShortDebugString(): row_key: [redacted, but normal looking] family_name { value: [redacted, but normal] } qualifier { value: [redacted, but normal] } timestamp_micros:<PHONE_NUMBER>549000 value: [redacted, but normal] commit_row: true
Class state:
row_key_: ""
cells_.size(): 0
cell_first_chunk: 0
last_seen_row_key: [redacted, but normal looking]
row_ready_: 0
end_of_stream_: 0

Just got this naturally (no Ctrl-C), debugging information same, except on a different row and with cell_first_chunk_ = 1.

And now I got it with an entirely empty chunk, and cell_first_chunk_ = 0.

Just got this naturally (no Ctrl-C), debugging information same, except on a different row and with cell_first_chunk_ = 1, and the value in the chunk was not populated.

That is odd, to get to the problem line, you need to pass this:
https://github.com/googleapis/google-cloud-cpp/blob/9018cf2199a6f3cad68552e4b5849918e93f6502/google/cloud/bigtable/internal/readrowsparser.cc#L78

I have tried several things and cannot reproduce the problem.  I wonder if you could tell me more about your application (threading model for example, size of keys and values).  Feel free to contact me directly, same username at google.com if you don't want to discuss this in a public forum.
Alternatively, maybe you can add additional login (this is expensive, so not quite what you want in production).  For example https://github.com/coryan/google-cloud-cpp/commit/412734da5f117b55328fb2fba0fef636df13e0aa captures the state of the class before and at the error point?

One of my coworkers managed to get me a reproducible test case, and I tracked down some bad memory accesses on my end.
If it turns out that those weren't the cause, then I'll reopen this. Thanks @coryan for your help.


Add support double tap zooming

Hi,
Please consider adding support for double tap zooming. I guess the feature should work in toggle mode. When double tap occurred it then either zooms in to max level or zooms out to min level depending in which zoom level it currently is. Maybe add some configuration options which zooming options are used.
It would be also nice if it additionally has the same feature as Android has which is when you double tap but don't release the finger from the second tap then you can select the zoom level only with one finger by either moving away your finger from the initial double tap location to zoom in and when you move the finger closer to initial double tap location then it zooms back out.
If you don't want to add tapping feature then please advice how I can listen to tap/click events without interfering the library events so I could add the support programmatically myself.

Thanks for opening an issue. hammer.js is a good option for handling different kinds of touch gestures, then just call the panzoom zoom method.

Thanks for the hint, works like a charm. Here is the sample code should anyone else like to add double tap support:
```
$("#container").panzoom({
minScale: 0.5,
maxScale: 1.5
});
    $("#container").on('panzoomzoom', function (e, panzoom, scale) {
        currentZoomScale = scale;
    });

    var mc = new Hammer.Manager(document.getElementById('container'));
    mc.add(new Hammer.Tap({ event: 'doubletap', taps: 2 }));

    mc.on('doubletap', function (ev) {
        var zoom = 1.5;
        if (currentZoomScale > 1) {
            zoom = 0.5;
        }
        $("#container").panzoom("zoom", zoom, {
            animate: true,
            focal: {
                clientX: ev.center.x,
                clientY: ev.center.y
            }
        });
    });


Fix 159 s3 documentation

Full described functionality will be implemented upon closing https://github.com/rucio/rucio/issues/5451.

s3_rse_config.md as a filename breaks the check-file-names.sh check because of the ^[a-z_]*\.md$ regexp. Should I change it to include numbers or should I change the filename?

Hi @rizart
Sorry I missed your last message. Please change the regexp and rebase the PR, otherwise this is good to go.


Web Wallet connector id

For knowing if the web wallet is connected or no in starknet react it’s tricky because we have to use connector.id() when it’s connector.id for normal wallets.

I think that he is talking about this package from Argent @argent/starknet-react-webwallet-connector

It's a bug in that library, they should implement the interface from v1. /cc @dhruvkelawala

Agree. Thanks for bringing this up

Confirmed with the team, it's already done and will be in the release today or tomorrow.

Thank you for the quick response and fix! Going to close.


fix: enable mount options

What type of PR is this?
/kind bug
What this PR does / why we need it:
Each parameter in cmd should be separated with a blank. Otherwise, it won't work.
Which issue(s) this PR fixes:

Fixes #25
Special notes for your reviewer:
Release note:
fix: enable mount options

Sorry but I don't understand very well how it's possible define mountOption.
I found in your code that tmp_path is set by default to /mnt. But how can I set other parameters? With values file during driver installation with helm?

Sorry but I don't understand very well how it's possible define mountOption.
I found in your code that tmp_path is set by default to /mnt. But how can I set other parameters (in particular file-cache-timeout-in-seconds)? With values file during driver installation with helm?

check example here: https://github.com/kubernetes-sigs/blobfuse-csi-driver/blob/master/deploy/example/storageclass-blobfuse-csi-mountoptions.yaml

Great! But "-o uid=1000" or "allow_other" option are mandatory to use /mnt (Azure VM ephimeral disk) ?

Great! But "-o uid=1000" or "allow_other" option are mandatory to use /mnt (Azure VM ephimeral disk) ?

@marcoboffi they are not relevant. do you have any other requirement on this?


Board with className styling

Hi rcdexta,
Is it possible to add a className to the Board component with this styling on it, or is only inline styling supported?
JSX:
<Board           className={styles.board}           data={processes}         />
CSS:
.board {   background-color: #191F26;   padding-top: 10;   padding-left: 10; }
Thanks

Yes should be possible now. I have added sample code to illustrate the point:
https://github.com/rcdexta/react-trello/blob/master/stories/BoardStyling.story.js
Please pull v1.20.4


Adding Countdown timer

Is your feature request related to a problem? Please describe.
I would like to add Count down timer

@Sulagna-Dutta-Roy I'd like to work on this issue, if it hasn't been solved yet.
Thank you!

please assign me this issue

You're from GSSOC? @S-i-y-a-g

Yes @Sulagna-Dutta-Roy

I have mistakenly unassigned that issue to me. Can you please reassign that  @Sulagna-Dutta-Roy ?

I received the pull request. Closing this Issue


Decimal/Integer .round and .int #6654

Pull Request Description

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@radeusgd raised whether we should call it int or truncate.
Excel uses both int and trunc these days.
I think it has become the more common (and SQL uses it) so lets rename to truncate.


feat: add stm annotations/增加STM注解

关联的Issues (Related Issues)

close https://github.com/filecoin-project/venus/issues/5316
改动 (Proposed Changes)

补充venus-wallet的部分测试用例
venus-wallet添加STM的注解
部分重构

ServeRPC函数, 可以从外部传入signal channel, 可以通过外部发送停止信号.
移除了自定义jwt.HMACSHA类型的别名:common.APIAlg. 因为这是一个没有意义的重定义, 还容易导致循环引用.

附注 (Additional Info)

自查清单 (Checklist)
在你认为本 PR 满足被审阅的标准之前，需要确保 / Before you mark the PR ready for review, please make sure that:

[ ] 符合Venus项目管理规范中关于PR的相关标准 / The PR follows the PR standards set out in the Venus project management guidelines
[ ] 具有清晰明确的commit message / All commits have a clear commit message.
[ ] 包含相关的的测试用例或者不需要新增测试用例 / This PR has tests for new functionality or change in behaviour or not need to add new tests.
[ ] 包含相关的的指南以及文档或者不需要新增文档 / This PR has updated usage guidelines and documentation or not need
[ ] 通过必要的检查项 / All checks are green

Codecov Report

Merging #103 (1c46467) into master (19c87b9) will increase coverage by 14.76%.
The diff coverage is 35.63%.

Additional details and impacted files

@@             Coverage Diff             @@
##           master     #103       +/-   ##
===========================================
+ Coverage   13.09%   27.86%   +14.76%
===========================================
  Files          46       48        +2
  Lines        3963     4127      +164
===========================================
+ Hits          519     1150      +631
+ Misses       3355     2777      -578
- Partials       89      200      +111

Flag
Coverage Δ

unittests
27.86% <35.63%> (+14.76%)
:arrow_up:

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

Impacted Files
Coverage Δ

cmd/daemon.go
0.00% <0.00%> (ø)

cmd/mock/main.go
0.00% <0.00%> (ø)

filemgr/jwt.go
65.38% <ø> (ø)

cmd/rpc.go
47.72% <7.69%> (+47.72%)
:arrow_up:

integration_test/convert.go
24.39% <24.39%> (ø)

integration_test/builder.go
49.29% <49.29%> (ø)

build/builder.go
89.39% <100.00%> (+89.39%)
:arrow_up:

common/api_common.go
55.00% <100.00%> (+55.00%)
:arrow_up:

filemgr/fs.go
58.46% <100.00%> (+24.61%)
:arrow_up:

storage/strategy/strategy.go
4.72% <0.00%> (+4.72%)
:arrow_up:

... and 22 more


Make possible to create custom annotation that is annotated with @ApiModelProperty

Add ElementType.ANNOTATION_TYPE to @ApiModelProperty. This will make possible to create custom annotations, such like
@ApiModelProperty("Should be greater than 1")
@Min(1)
@Target(value= ElementType.FIELD)
@Retention(value= RetentionPolicy.RUNTIME)
public @interface Price {
}

so, no need to specify two annotations for field
@ApiModelProperty("Should be greater than 1")
@Min(1)
long price;

will be
@Price
long price

is more clear, and if we have price in multiple places it is easier to refactor

@frantuma - thoughts?

@How do we add @Price annotation through swagger?

Please give an update regarding this issue. @raderio @webron


Run cell block - not working as it did before on spyder3

Issue Report Checklist

[ /] Searched the issues page for similar reports

[/ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice

[/ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)

[/ ] Tried basic troubleshooting (if a bug/error)

[/ ] Restarted Spyder
[/ ] Reset preferences with spyder --reset
[/ ] Reinstalled the latest version of Anaconda
[/ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I am trying to run a cell block within a function to test values in the ipython console. But in spyder4 we have to use #%% before and after blocks of code in order for this to happen.
This works for small blocks of code. But when I try to do large blocks of code, it tells me I have an indentation problem.
What steps reproduce the problem?

Using #%%
Running cell block code "ctrl+enter"

What is the expected output? What do you see instead?
To be able to use the output variables in the ipython console.
Instead I see an indentation error
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 4.0.0
Python version: 3.8
Qt version: 5.97
PyQt version: 1.9.0
Operating System name/version: Ubuntu 18.04

#code example
def cell_block_ex():

    #%%
    path = '/home/<user>/Documents/projects/office'
    prevlist= pd.read_csv(path + '/m/daily.csv',index_col=0)
    #%%

The above works fine
then when I try to run this;
def cell_block_ex():
    #%%
    path = '/home/<user>/Documents/projects/office'
    prevlist= pd.read_csv(path + '/m/daily.csv',index_col=0)

    datefrom=parse(prevlist.index.max(), dayfirst = True)
    datefrom = datefrom+timedelta(days=1)
    #%%

This throws an indentation error for path, the problem with this is I need to check a lot more lines of code, so to keep un-indenting and re-indenting, is very time consuming, same with adding the #%%. On the old version I was simply able to select the code press ctrl+enter and then analyse my data.

Closing this as a duplicate of #11023.
Please see my comments there. Using code cell inside of functions is a bad idea because you are mixing the local namespace with the global. It's kind of like doing this: https://stackoverflow.com/questions/11543297/make-all-variables-in-a-python-function-global/11543718. To see the value inside a function you should use the debugger which has been improved for spyder4.


diffusers fine-tuning weird  error occurred

System Info / 系統信息
Python 3.10.12, torch 2.4.0+cu121, cuda12.2, accelerate=1.1.1
Information / 问题信息

[X] The official example scripts / 官方的示例脚本
[ ] My own modified scripts / 我自己修改的脚本和任务

Reproduction / 复现过程
Log errors:
[W1122 13:45:52.579715884 CUDAAllocatorConfig.h:28] Warning: expandable_segments not supported on this platform (function operator())
[W1122 13:45:52.653884504 CUDAAllocatorConfig.h:28] Warning: expandable_segments not supported on this platform (function operator())
[W1122 13:45:52.665469534 CUDAAllocatorConfig.h:28] Warning: expandable_segments not supported on this platform (function operator())
[W1122 13:45:52.351059829 CUDAAllocatorConfig.h:28] Warning: expandable_segments not supported on this platform (function operator())
Detected kernel version 3.10.0, which is below the recommended minimum of 5.5.0; this can cause the process to hang. It is recommended to upgrade the kernel to the minimum version or higher.
11/22/2024 13:45:54 - INFO - __main__ - Distributed environment: MULTI_GPU  Backend: nccl
Num processes: 4
Process index: 0
Local process index: 0
Device: cuda:0

Mixed precision type: bf16

You set `add_prefix_space`. The tokenizer needs to be converted from the slow tokenizers
Loading checkpoint shards:   0%|          | 0/2 [00:00<?, ?it/s]11/22/2024 13:45:56 - INFO - __main__ - Distributed environment: MULTI_GPU  Backend: nccl
Num processes: 4
Process index: 2
Local process index: 2
Device: cuda:2

Mixed precision type: bf16

11/22/2024 13:45:57 - INFO - __main__ - Distributed environment: MULTI_GPU  Backend: nccl
Num processes: 4
Process index: 3
Local process index: 3
Device: cuda:3

Mixed precision type: bf16

Loading checkpoint shards:   0%|          | 0/2 [00:00<?, ?it/s]Loading checkpoint shards:   0%|          | 0/2 [00:00<?, ?it/s]11/22/2024 13:45:58 - INFO - __main__ - Distributed environment: MULTI_GPU  Backend: nccl
Num processes: 4
Process index: 1
Local process index: 1
Device: cuda:1

Mixed precision type: bf16

Loading checkpoint shards:   0%|          | 0/2 [00:00<?, ?it/s]
Loading checkpoint shards:  50%|█████     | 1/2 [00:19<00:19, 19.00s/it]
Loading checkpoint shards:  50%|█████     | 1/2 [00:22<00:22, 22.96s/it]
Loading checkpoint shards:  50%|█████     | 1/2 [00:20<00:20, 20.55s/it]
Loading checkpoint shards:  50%|█████     | 1/2 [00:20<00:20, 20.63s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [00:58<00:00, 31.11s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [00:58<00:00, 29.30s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [01:00<00:00, 31.75s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [01:00<00:00, 30.07s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [01:02<00:00, 32.75s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [01:02<00:00, 31.28s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [01:00<00:00, 31.77s/it]
Loading checkpoint shards: 100%|██████████| 2/2 [01:00<00:00, 30.10s/it]
{'use_learned_positional_embeddings'} was not found in config. Values will be initialized to default values.
W1122 14:00:09.668000<PHONE_NUMBER>23360 torch/distributed/elastic/multiprocessing/api.py:858] Sending process 31662 closing signal SIGTERM
W1122 14:00:09.669000<PHONE_NUMBER>23360 torch/distributed/elastic/multiprocessing/api.py:858] Sending process 31663 closing signal SIGTERM
W1122 14:00:09.676000<PHONE_NUMBER>23360 torch/distributed/elastic/multiprocessing/api.py:858] Sending process 31664 closing signal SIGTERM
E1122 14:00:26.050000<PHONE_NUMBER>23360 torch/distributed/elastic/multiprocessing/api.py:833] failed (exitcode: -9) local_rank: 0 (pid: 31661) of binary: /opt/conda/bin/python3.10
Traceback (most recent call last):
  File "/opt/conda/bin/accelerate", line 8, in <module>
    sys.exit(main())
  File "/opt/conda/lib/python3.10/site-packages/accelerate/commands/accelerate_cli.py", line 48, in main
    args.func(args)
  File "/opt/conda/lib/python3.10/site-packages/accelerate/commands/launch.py", line 1159, in launch_command
    multi_gpu_launcher(args)
  File "/opt/conda/lib/python3.10/site-packages/accelerate/commands/launch.py", line 793, in multi_gpu_launcher
    distrib_run.run(args)
  File "/opt/conda/lib/python3.10/site-packages/torch/distributed/run.py", line 892, in run
    elastic_launch(
  File "/opt/conda/lib/python3.10/site-packages/torch/distributed/launcher/api.py", line 133, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/opt/conda/lib/python3.10/site-packages/torch/distributed/launcher/api.py", line 264, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
======================================================
train_cogvideox_lora.py FAILED
------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2024-11-22_14:00:09
  host      : xxxxxxxxxxxxxxx
  rank      : 0 (local_rank: 0)
  exitcode  : -9 (pid: 31661)
  error_file: <N/A>
  traceback : Signal 9 (SIGKILL) received by PID 31661
======================================================

Expected behavior / 期待表现
solve it

看起来这似乎并不是由于模型导致，而是torch的报错，你是在分布式训练吗？

It seems that this is not due to the model, but a torch error. Are you doing distributed training?

Yes, use scripts finetune_single_rank.sh and accelerate_config_machine_single.yaml
num_processes has been changed, with default type: distributed_type: DEEPSPEED
and run command with:
CUDA_VISIBLE_DEVICES="0,1,2,3" accelerate launch --config_file accelerate_config_machine_single.yaml --multi_gpu \

It appears that the CPU memory has been exhausted, leading to signal 9 errors. Perhaps you could try increasing the swap memory.

same problem here


Incrementals blog pt1

Our Incremental release on Incremental backups :) will PR Part 2 shortly
Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[x] :no_entry: No

Type of change

[ ] :sunflower: Feature
[ ] :bug: Bugfix
[x] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Issue(s)

#

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

@serverless-mom @ashmrtn the post and the mechanics of the PR are fine, but I would have loved to have at least some of the meat around the topic in this Part 1. In my opinion there is a lot of background info and not enough on the mechanics of incrementals.
Is there a way to reduce the "Backup layout" or the "Terminology section" in favor of including some of the steps that follow? Another alternative would be delaying one of these sections to Part2 where it can precede some of the steps where it is relevant.
If none of that works or resonates, feel free to go with what you already have.
In either case, it might be worthwhile to add a into section that will outline both parts of the blog and will give a map to the reader of what they will learn in Part 1 and what they will learn in Part 2. This may also mitigate some of my concerns from above.

@gmatev what if we just PR both part 1 and part 2 at the same time? There's good reasons not to post a single post that's quite so long, but we could have both parts go up on the same day, so that you're not left hanging when you finish part 1


Compare IDP-KG stats with numbers from Ivan

Message received from Ivan with details of overlap between the three datasets (summarised in attached figure):

For the numbers, I noted something like this:

common entries (overlapping entries) in DisProt and PED: 40; unique to DisProt: 1414; unique to PED: 42
common in MobiDB and DisProt: 1421; unique to MobiDB: 653; unique to DisProt: 33
common in MobiDB and PED: 44; unique to MobiDB: 2030; unique to PED: 38
common across all 3 datasets: 38

This was before the assignment of a list of UniProt proteins to each PED entry so I'll redo the numbers when I get in the lab. As I recall, only PED used isoform identifiers for some of it's entries so this may generate some problems matching data across datasets.

Current response over IDP-KG (Full)

x
description
count

1
DisProt Proteins
"1615"^^xsd:integer

2
MobiDB Proteins
"2073"^^xsd:integer

3
PED Proteins
"83"^^xsd:integer

4
DisProt / (MobiDB U PED)
"179"^^xsd:integer

5
DisProt n MobiDB
"1432"^^xsd:integer

6
DisProt n MobiDB / PED
"1432"^^xsd:integer

Running query over full scrape dataset in commit 4b4c7ae5db27e2bc39267140080f55c3b8ebb9de

x
description
count

1
Distinct Proteins (Union)
"2718"^^xsd:integer

2
DisProt Proteins
"2062"^^xsd:integer

3
MobiDB Proteins
"2073"^^xsd:integer

4
PED Proteins
"90"^^xsd:integer

5
DisProt \ (MobiDB U PED)
"604"^^xsd:integer

6
MobiDB \ (DisProt U PED)
"617"^^xsd:integer

7
PED \ (DisProt U MobiDB)
"34"^^xsd:integer

8
(DisProt U MobiDB)
"2684"^^xsd:integer

9
(DisProt U PED)
"2101"^^xsd:integer

10
(MobiDB U PED)
"2114"^^xsd:integer

11
DisProt n MobiDB
"1451"^^xsd:integer

12
DisProt n PED
"51"^^xsd:integer

13
MobiDB n PED
"49"^^xsd:integer

14
(DisProt n MobiDB) \ PED
"1407"^^xsd:integer

15
(DisProt n PED) \ MobiDB
"7"^^xsd:integer

16
(MobiDB n PED)\DisProt
"5"^^xsd:integer

17
DisProt n MobiDB n PED
"44"^^xsd:integer

2021-09-28 version is now in sync with what is available on the websites of the data sources. Problems were due to using named graphs as proxies for pages and deprecated proteins being included in the sitemaps.


Remove send on closed channel

In the devnet, the following error was observed after some of the nodes panicked:
panic: send on closed channel

goroutine 128 [running]:
github.com/dusk-network/dusk-blockchain/pkg/p2p/peer.sendError(0xc0023f9140, 0x108c640, 0xc01fa7b310)
        /work/pkg/p2p/peer/peer.go:211 +0x49
github.com/dusk-network/dusk-blockchain/pkg/p2p/peer.(*Writer).writeLoop(0xc002231110, 0x10a2c40, 0xc00245ac00, 0xc0023f9080, 0xc0023f9140)
        /work/pkg/p2p/peer/peer.go:295 +0x650
github.com/dusk-network/dusk-blockchain/pkg/p2p/peer.(*Writer).Serve(0xc002231110, 0x10a2c40, 0xc00245ac00, 0xc0023f9080, 0xc0023f9140)
        /work/pkg/p2p/peer/peer.go:225 +0xd8
created by github.com/dusk-network/dusk-blockchain/pkg/p2p/peer.Create
        /work/pkg/p2p/peer/peer.go:202 +0x106

This is accompanied by the logs showing that some connections are dropped:
{"error":"write tcp <IP_ADDRESS>:58020-\u003e<IP_ADDRESS>:7100: write: broken pipe","level":"error","msg":"got error back from keepAlive","process":"keepaliveloop","time":"2021-04-16T14:29:47Z"}
{"coordinator_round":45145,"level":"debug","msg":"storing future round for later","process":"agreement","round":45147,"time":"2021-04-16T14:29:53Z","topic":"Agreement"}
{"error":"write tcp <IP_ADDRESS>:60666-\u003e<IP_ADDRESS>:7100: write: broken pipe","level":"error","msg":"got error back from keepAlive","process":"keepaliveloop","time":"2021-04-16T14:29:54Z"}
{"error":"write tcp <IP_ADDRESS>:7100-\u003e<IP_ADDRESS>:55554: write: broken pipe","level":"error","msg":"got error back from keepAlive","process":"keepaliveloop","time":"2021-04-16T14:29:55Z"}
{"coordinator_round":45145,"level":"debug","msg":"storing future round for later","process":"agreement","round":45148,"time":"2021-04-16T14:30:11Z","topic":"Agreement"}
{"coordinator_round":45145,"level":"debug","msg":"storing future round for later","process":"agreement","round":45146,"time":"2021-04-16T14:30:11Z","topic":"Agreement"}
{"error":"read tcp <IP_ADDRESS>:38204-\u003e<IP_ADDRESS>:7100: i/o timeout","level":"warning","msg":"error reading message","process":"peer","time":"2021-04-16T14:29:58Z"}
{"error":"read tcp <IP_ADDRESS>:38204-\u003e<IP_ADDRESS>:7100: i/o timeout","level":"warning","msg":"error received from peer","process":"server","time":"2021-04-16T14:30:11Z"}
{"error":"write tcp <IP_ADDRESS>:7100-\u003e<IP_ADDRESS>:50876: write: broken pipe","level":"error","msg":"got error back from keepAlive","process":"keepaliveloop","time":"2021-04-16T14:29:58Z"}
{"error":"write tcp <IP_ADDRESS>:55820-\u003e<IP_ADDRESS>:7100: write: broken pipe","level":"error","msg":"got error back from keepAlive","process":"keepaliveloop","time":"2021-04-16T14:29:58Z"}
{"error":"write tcp <IP_ADDRESS>:38204-\u003e<IP_ADDRESS>:7100: write: broken pipe","level":"warning","msg":"error writing message","process":"writeloop","time":"2021-04-16T14:29:58Z"}

Right after these entries, the logs indicate a new startup.

The nodes it was connected to seem to time out about 5 blocks later, which is about 30 seconds or so. The problem is definitely with <IP_ADDRESS>, so it may be good to find the cause of this random broken pipe.


Implement create-topic

So far, this meets nearly all of my needs to a completely local development environment.
Thank you!
If you implement create-topic, it will then satisfy my needs to test everything, including infrastructure setup, locally.
I'm building our go chops now. We may implement this as a PR once we have confidence to do so.

Create topic is actually implemented see the postman collection to use it.

Oh, I see, great!
I guess I was confused by the README.md:
(Topics are created hard coded (until create-topic is implemented)

aws --endpoint-url http://localhost:4100 sns list-topics
    {
        "Topics": [
            {
                "TopicArn": "arn:aws:sns:local:000000000000:topic1"
            },
            {
                "TopicArn": "arn:aws:sns:local:000000000000:topic2"
            }
        ]
    }

Note: topic1 and topic2 do appear to still be hard-coded... :-)

@p4tin Should I file another ticket for removing topic1 and topic2?

the 2 topics (topic1 and topic 2) have been removed and the README shows a sample response from list-topics.  I have changed the README.md file in the hopes that this clears up the confusion.  Thanks again!


[WIP] Design for wallet with pluggable storage approach

In current state libindy allows to plug different wallet implementations. Plugged wallet now handles both security
and storage layers. This design proposes to restrict our plugged interface by handling only storage layer.
All encryption will be performed in libindy. It will simplify plugged wallets and warranty good security level
for 3d party wallets implementations.
Also proposals enhances our API to efficient and flexible search with paging support.

I don't think you can search tag values if you encrypt them.

In my vision the most of tags will be encrypted/hashed. It will allow to compare exact values in queries. For queries with "like" or "gte" predicates we can't use encryption. We can allow user to store some unencrypted tags with entities in the wallet for complex search purposes.

"wallet entity" is too verbose; use "item" (we already have "wallet" as a prefix for every function)

API is object-oriented and prefix here means "class name". wallet_entity_get_id means WalleteEntity::get_id(), but i understand your point.

I'm not happy about using a string as an id for a wallet. For a few secrets, this is fine--but for a large wallet with tens of millions of keys, wouldn't it be more efficient to make the key an integer? (There is no reason to identify credentials with a string, BTW, since such identifiers have no independent meaning, unlike the naming convention we've chosen for secrets)

For the most of entities like Key, DID, ClaimDefinition we have natural id that libindy API accepts. We need the way to convert this id to wallet id. So i vote for string id.

I'm dubious about the wisdom of doing mongo-db-style queries.

It is the point for discussion. I will vote for any query-style that is easy to parse and analyze. I see "tags" as an object in document-oriented database and mongo-style query is very natural way to query documents. I really don't like SQL because we will need to write our own parser and it is really non-obvious how to map tags to some tables.

It's not clear to me how creating and opening wallet storage is intended to relate to creating and opening a wallet.

In my vision it is one-to-one mapping:
wallet_create calls storage_create
wallet_open calls storage_open
and etc...


Unable to target the autoscaler api endpoint

Unable to target the autoscaler api endpoint using the command cf autoscaling-api as part of the autoscaler-cli-plugin.

@lalitrawat16 can you provide some more information like what url you're trying to target and where is your app-autoscaler deployed, version etc.

@rohitsharma04 I am having the same difficulty. Cannot find documentation describing how to find out how autoscaler api endpoint should look like. Can you please try to explain without further details replacing real urls with placeholders?
Let's say I know my CF API endpoint ("https://api.something.io:). Is that somehow used in order to get to autoscaler api endpoint? I have tried as "https://autoscaler.api.something.io" but with no luck.
Thanks

@radovanthefoley the url you're using is incorrect.
As for your example if the cf api endpoint is
https://api.something.io then autoscaler api endpoint will be https://autoscaler.something.io

@rohitsharma04 thanks. Hope this helps to anybody passing by the same problem.

@radovanthefoley  @lalitrawat16  sorry for the issue. we have some documentation effort on going now,  but not ready yet ..

closing given guide is added in document


EndpointAddress should be resolved once

Currently we do a (cached) DNS lookup on every proxy for every non-socket address endpoint https://github.com/googleforgames/quilkin/blob/6badc240d5257fceda436a3661d6e43b93fa387f/src/components/proxy/packet_router.rs#L234. To me this doesn't really make sense, and really the relay should do the resolution of addresses so that proxies only ever have to deal with resolved socket addresses.

To me this doesn't really make sense, and really the relay should do the resolution of addresses so that proxies only ever have to deal with resolved socket addresses.

I don't think that makes sense, because we can't rely that it always resolves to the same IP address. One of the main reasons we support DNS is for AWS, and the IPs for pod addresses change frequently, and we want to send where it resolves to in that moment, not at the moment it was visible to relay. Also you might get a different IP address when resolving from the proxy than from the relay, because DNS can be setup so that it resolves to the closest pathway, and that could be a very different path for the relay and the proxy because they could be on different sides of the world.
It's also important to remember that we support more setups than just relay, if you're running just the proxy, you want it to be able to resolve DNS entries on its own.

Ahh ok


Create a Java web client

See the Java client written in anHALytics-core as starting point (the multithreaded version):
https://github.com/anHALytics/anhalytics-core/blob/master/anhalytics-annotate/src/main/java/fr/inria/anhalytics/annotate/services/NerdService.java
https://github.com/anHALytics/anhalytics-core/blob/master/anhalytics-annotate/src/main/java/fr/inria/anhalytics/annotate/Annotator.java

The version 0.0.1 of https://github.com/hirmeos/entity-fishing-client-java implements the functionalitites on the client. WIP on the multithread

The version 0.0.2 containing the completed API implementation and the multithread processing has been released and it's distributed via: https://bintray.com/rookies/maven/entity-fishing-client

Hello , as part of a project I have to test the functioning of your web app so I wanted to ask you if it would be possible to separate the web APP part of the entity-fishing project in order to test the functioning of your web APP

Hello @Mekouaryoussef !
What do you call "web APP" ?
entity-fishing is a server with a REST-like API described in the documentation:
https://nerd.readthedocs.io/en/latest/restAPI.html
Then there is a web console to test the API, https://nerd.readthedocs.io/en/latest/console.html - but the console is just a demo to play with the API and to show how to use the API results.
If you want to test entity-fishing, depending on the nature of your tests, you can send queries with any REST client (curl, postman, insomnia, etc) to the web API.

the problem is that I can’t build the entity-fishing project and I’m interested in the visualization part of your project , so I wanted to test the visualization part without building the whole project.
it is possible?

You can't have the web console working without the server.
Not really sure what you want to do to be honest - but the web console javascript is there: https://github.com/kermitt2/entity-fishing/tree/master/src/main/webapp

We are specifically interested in the entity-fishing console, that is showing articles with highlighted named entities.
Technically, we would like to plug this on our own back-end, such that we Don't actually need to REST API of entity-fishing.
So basically we would like to extract from the console the party that just performs the articles visualization.


feat: Add sql-datatype to the SDK discovery and catalog

This is an attempt to add the sql_datatype to the discovery process and will be reflected in the catalog.
Closes #1323

:books: Documentation preview :books:: https://meltano-sdk--1872.org.readthedocs.build/en/1872/

I have run across instances where the discovery process runs into SQL data types that SQLAlchemy cannot handle.  When this occurs, the following warnings are logged to the console.
PS C:\development\projects\my-test-stuff> meltano invoke tap-mssql --discover > discovery.json
2023-07-21T19:40:07.498116Z [info     ] Environment 'dev' is active
C:\development\MeltanoContribute\sdk\singer_sdk\connectors\sql.py:463: SAWarning: Did not recognize type 'hierarchyid' of column 'OrganizationNode'
  for column_def in inspected.get_columns(table_name, schema=schema_name):
C:\development\MeltanoContribute\sdk\singer_sdk\connectors\sql.py:463: SAWarning: Did not recognize type 'geography' of column 'SpatialLocation'
  for column_def in inspected.get_columns(table_name, schema=schema_name):
C:\development\MeltanoContribute\sdk\singer_sdk\connectors\sql.py:463: SAWarning: Did not recognize type 'hierarchyid' of column 'DocumentNode'
  for column_def in inspected.get_columns(table_name, schema=schema_name):

The sql-datatype is presented as a NullType()
        {
          "breadcrumb": [
            "properties",
            "OrganizationNode"
          ],
          "metadata": {
            "inclusion": "available",
            "sql-datatype": "NullType()"
          }
        },

cc @edgarrmondragon

The warnings are being captured, the message expanded, and presented back to the console at the info level .  The warnings now look like this.
PS C:\development\projects\dev-tap-mssql-discover-sqltype> meltano invoke tap-mssql --discover > discovery.jsonl
2023-08-08T17:09:38.074328Z [info     ] Environment 'dev' is active
2023-08-08 10:09:39,636 | INFO     | sqlconnector         | Discovery warning: 'HumanResources-Employee' Did not recognize type 'hierarchyid' of column 'OrganizationNode'
2023-08-08 10:09:39,730 | INFO     | sqlconnector         | Discovery warning: 'Person-Address' Did not recognize type 'geography' of column 'SpatialLocation'
2023-08-08 10:09:39,859 | INFO     | sqlconnector         | Discovery warning: 'Production-Document' Did not recognize type 'hierarchyid' of column 'DocumentNode'
2023-08-08 10:09:39,890 | INFO     | sqlconnector         | Discovery warning: 'Production-ProductDocument' Did not recognize type 'hierarchyid' of column 'DocumentNode'

After looking at this more I update the message to this format:
PS C:\development\projects\dev-tap-mssql-discover-sqltype> meltano invoke tap-mssql --discover > discovery.jsonl
2023-08-08T17:50:34.642130Z [info     ] Environment 'dev' is active
2023-08-08 10:50:36,401 | INFO     | sqlconnector         | Discovery warning: Did not recognize type 'hierarchyid' of column 'OrganizationNode' in 'HumanResources-Employee'
2023-08-08 10:50:36,512 | INFO     | sqlconnector         | Discovery warning: Did not recognize type 'geography' of column 'SpatialLocation' in 'Person-Address'
2023-08-08 10:50:36,684 | INFO     | sqlconnector         | Discovery warning: Did not recognize type 'hierarchyid' of column 'DocumentNode' in 'Production-Document'
2023-08-08 10:50:36,747 | INFO     | sqlconnector         | Discovery warning: Did not recognize type 'hierarchyid' of column 'DocumentNode' in 'Production-ProductDocument'

One thing that unfortunately isn't very clear to me is how are targets supposed to consume this information since a target isn't aware of the tap's metadata. It might be better if this metadata lived in the schema, at least that way it would be emitted with SCHEMA messages and could be used by the target.
The official Singer docs only explain the field as Represents the datatype of a database column, and pipelinewise's taps implement but the target don't seem to use it in any way:
https://github.com/search?q=org%3Atransferwise+sql-datatype+language%3APython&type=code&l=Python
Otherwise, who is the consumer of this metadata?


Add comment regarding TPC-DS query variants.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing this merge. The comment is already in the README file


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.16.5",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=941d5180c430ce5b0f7a3622ef9b76077bfa3d82"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "941d5180c430ce5b0f7a3622ef9b76077bfa3d82",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "block: Fix wrong offset in bio_truncate()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.16.5 by commit 941d5180c430ce5b0f7a3622ef9b76077bfa3d82. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000635


Refactor confirmation prompt to separate pkg

This commit refactors the confirmation prompt to a separate pkg.
Signed-off-by: Stavros Panakakis<EMAIL_ADDRESS>

How about providing the io.Reader as an argument to Confirm instead?

How about providing the io.Reader as a parameter to Confirm instead?

Check https://github.com/marcosnils/bin/pull/132#discussion_r847492299. I though it was easier to handle it separately :pray:

The branch had conflicts so I rebased :)


Sample Source for Forest Launcher

Hello,
Can we get sample Forest Launcher source please ? Im after the design, since its not using docking...
Thank you

I meant something like this. I don't specifically need the source code, just guidance on how to do it. Some sample code would be nice ;)

"Some sample code would be nice" XD... that's what Walnut is... and yes it's EXTREMELY difficult and I've been a student of C++ for 15 years... even I'm struggling to get that far and I did it with irrlicht 15 years ago.... like I said in the duplicate post, this is something we can throw to the check board but it's not likely they will add this quickly, as it's not a quick thing to post, and who knows if "the forest" launcher even still exists. It seems to me in the ImGui Demo there is a switch to disable docking... look into it.

imguiWindow->disableMove
imguiWindow->disableResize
imguiWindow->disableMinMax
Some psuedo code to consider.

Well you can say that i'm not good enough, it could be but i can write in 6 languages and im picking things pretty fast. Anyway sorted.. Thx

Well you can say that i'm not good enough, it could be but i can write in 6 languages and im picking things pretty fast. Anyway sorted.. Thx

If that is all anyone can take of the hours I put in trying to help them... then no, thank you, because I will never again in my life waste my time on someone like you again.

You are so aggressive person, not sure what is your problem. I did ask nicely for help, Your answer where more than horrible and not polite at all.  And it is not only in this Issue.. why did you reply to my issue on the first place then. Your 15 years of experience , even with 50 years of experience not giving you right to act like this. So yes it is sorted but not with your help. So If you spend an hour on my question with 15years of experience, you probably doing something wrong there. That's it and to be honest this is my first so negative experience on github , i did not know that is even possible.... :( so sad

You are so aggressive person,

I have every right to be as aggressive as I like... that's a subjective personal opinion anyway. If you want to know why it's because you need to go learn C++ first, and stop using this like a forum.. it's not for messages like this.


Tools: Default to Development environment

Right now, we do nothing which defaults to Production in Hosting.
We should ensure Development is used during development.
There are a couple of ways this be done:

Set the ASPNETCORE_ENVIRONMENT environment variable to Development if it's unset.
Wait for the launch settings feature and ensure it lights up in our tooling.

@bricelam to follow up with hosting team.

dotnet/cli#6617 is the issue tracking the launchSettings.json work.


Broken for a certain youtube video, "unable to download video data"

Trying to dl https://www.youtube.com/watch?v=8a5ID0x932I. Video works in browser, but download is broken with youtube-dl:
$ youtube-dl https://www.youtube.com/watch?v=8a5ID0x932I
[youtube] 8a5ID0x932I: Downloading webpage
[youtube] 8a5ID0x932I: Downloading video info webpage
[youtube] 8a5ID0x932I: Extracting video information
[youtube] 8a5ID0x932I: Downloading DASH manifest
[youtube] 8a5ID0x932I: Downloading DASH manifest
ERROR: unable to download video data: HTTP Error 404: Not Found

Not sure if this is specific to only this video.

The problem is that on the YouTube player the video's highest quality is 360p but on the dash manifest it list 480p and higher. As a workaround you can use -f '134+140'or -f best. Since the last one works and it's a 720p file, I'd guess that eventually the other fomats will be available.

Perfect, thanks.
As a suggestion: I've got a youtube-dl config file with lots of fallback options. Couldn't youtube-dl check the next worst option in such a case too?

I guess it could be done, open a new issue if you want it.

I'll consider it. Thanks.


[c# epoxy] Handle client-side connection failures

Client-side connection failures now close the sockets and streams that
are created during the connection process if an error occurs while
setting up the TCP or TLS connection.
Refactored similar code that is used in the listener so that we have
one copy of the core algorithm.
Removed some redundant try...catch blocks that were just logging
exceptions and installed top-level connect/accept exception handles
that then log.

This PR now needs to be squashed when mering.


What exactly is "Rainday"?

Dear all,
I want to setup an automation that stops my irrigation system to go off if it rains during the day. Should I use the "Rainday" or the "Rain" sensor for this?
Cheers.

RainDay sensor how much rain you have received (current forecast) - it will be ON if it has rained at all during the day or expect to receive during the day (future forecast). Whereas Rain sensor tells you if it is currently raining  (current forecast) or if it is expected to rain on a certain day (future forecast).
So I would trigger off the RainDay sensor because if it has rain at all during the day, then the irrigation system should turn off.


Set transport with url

This allows for setting the transport with a new configuration option url. It will leave the original methods of initiating steem-js intact to not break functionality for current users. Also, this adds the post-install build script npm module which makes it possible to load steem-js directly from a git url instead of from npm.

What's going on? It looks like the library stopped working after this? I keep getting weird error messages. Nothing is working anymore...

@msg768 - can you elaborate? are you still having trouble? there were some troubles with steemit's public nodes yesterday around the time of your comment so I'm wondering if you are still having trouble and if so what are you having trouble with?

It's not related to this PR but about steemit public node, @msg768 opened an issue few minutes after his comment here:  https://github.com/steemit/steem-js/issues/221


any backlog items focussed on f7 enabling r interactive window history access

Please briefly describe what you were doing that led to the issue, if applicable:
Currently in the 15.2 (26430.12) installs r interactive window you get the up arrow support for accessing prior commands.  Are there any backlog items focused on f7 enabling r interactive window history access similar to how it works in cmd.exe prompt environment so you can visual see how far back you have to go in order to find command you want?
p.s. should i have entered a feature question like this elsewhere?
If you have any screenshots demonstrating the issue, please include them as well to help us diagnose it better.
Additional information:
OS Information
Version:       Microsoft Windows NT 10.0.15063.0
RTVS Information:
Assembly: Microsoft.VisualStudio.R.Package, Version=1.1.30523.1233, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
Active R URI:
R 3.3.2: C:\Program Files\R\R-3.3.2

#3019 - people are asking more like csh rather than cmd.

Note that you have your visual history available immediately in the REPL itself, if you scroll up - and you can position the cursor on some earlier input entry and press Ctrl+Enter to re-use your older input. Also, there's the History window (R Tools | Windows | History).

Yes the ability repl [ / r interactive ??? ] to scroll up through r interactive window and find prior command to ctrl+enter reapply at current prompt is an option provided there isn't a lot of print output to sift through.
The r tools | windows | history view is even better than a cmd.exe f7 popup would be as the latter requires you to scroll back to command you want to reenter vs being able to select it by number or mouse click.    In the r tools | windows | history view is there any plan to wire up ctrl+enter as keyboard shortcut for the "Send selected to R Interactive Window" toolbar option?

I believe you can wire any shortcut in Tools | Options | Environment | Keyboard. @AlexanderSher can provide RTools.* command name

Just realized that double clicking r tools | windows | history view entry inserts that item into r interactive window leaving you to just modify as desired and press enter to submit it.  That works even better.  In hindsight i guess would have been self discoverable by someone who thought to use history view of commands as quick way to easily go back a significant number of steps to repeat a prior command.

Might be interesting to connect f7 keyboard shortcut to display of r history pane vs ctrl+4 which overlaps with sysinternals aka.ms/zoomit tsr liveZoom shortcut key.

Feel free to comment in #3019. @AlexanderSher is currently looking into it. Unfortunately, different people want different things... We are considering something that would cover most common cases. Perhaps compatible with RStudio or csh like?

I am going to close in favor of #3019


Error when using a custom model

"classname" and "config" must be set.

I've emailed Gantman, and please take a look at it.

Can you provide a repo with your custom model so I can debug?

Can you provide a repo with your custom model so I can debug?

My code was like this:
const axios = require('axios') //you can use any http client
const tf = require('@tensorflow/tfjs-node')
const nsfw = require('nsfwjs')
async function fn() {
  const pic = await axios.get(link_image, {responseType: 'arraybuffer'})
  const model = await nsfw.load("file://anti-nsfw-bot/project-model/")
  const image = await tf.node.decodeImage(pic.data,3)
  const predictions = await model.classify(image)
  image.dispose()
  console.log(predictions)
}
fn()```

I was using your newest model.
> https://github.com/GantMan/nsfw_model/releases/tag/1.2.0

What version of TensorFlow js?

@tensorflow/tfjs-node: 2.3.0
@tensorflow/tfjs: 2.3.0

I believe that's the issue.
I have it on my TODO to support the latest version of TFJS, but they made some breaking changes when they jumped from 1.x to 2.x and I need to come back and update it.
Try "@tensorflow/tfjs": "^1.7.4"

Which one? tfjs or tfjs-node?
Also I tried to downgrade it, and it said version not found (404)

tfjs

Should I use above than 1.7.4 ?

Update:
I uninstall tfjs (not the node one)
I downgraded my node to v12.13.0
Now I only used tfjs-node v2.1.0
I don't know if I stuck at here.
Only HTTP(S) protocols are supported
Don't know why but I'm stuck here.

Yes, I only have tfjs-node.

@conver4y
just had the same problem
uninstall @tensorflow/tfjs-node and @tensorflow/tfjs
and then
npm install @tensorflow/tfjs-node
npm install<EMAIL_ADDRESS>EXACTLY IN THIS ORDER
idk why, but when I install<EMAIL_ADDRESS>first, it crashes with the same error you had

@conver4y
I've just had the same problem
uninstall @tensorflow/tfjs-node and @tensorflow/tfjs
and then
npm install @tensorflow/tfjs-node
npm install<EMAIL_ADDRESS>EXACTLY IN THIS ORDER
idk why, but when I install<EMAIL_ADDRESS>first, it crashes with the same error you had

exactly the same dude :)

I'll go ahead and close this issue.  If you can think of a proper place to warn others of this issue with install order, please PR the docs.

I'll go ahead and close this issue.  If you can think of a proper place to warn others of this issue with install order, please PR the docs.


Add a Physical Context Interaction VP

I'd like to propose a P4_PCXI Physical Context Interaction Viewpoint,
to be able to specifiy sequences of exchanges on physical interfaces.
It should be the analog to the F4_SCXI
a concern already exists:
What is the sequence of interactions among the system and context elements on physical level?

Meeting: 20.12.2024 --> create viewpoint


fs2 started failing to compile in 2.12 build

[fs2] [error] /Users/tisue/community.212/target-0.9.11/project-builds/fs2-39db6ede7ef12fb3f6ceb78567d325c845b86955/core/shared/src/main/scala/fs2/async/async.scala:138:27: method shift in trait Async cannot be accessed in cats.effect.Effect[F]
[fs2] [error]     F.liftIO(F.runAsync(F.shift *> f) { _ =>
[fs2] [error]                           ^
[fs2] [error] /Users/tisue/community.212/target-0.9.11/project-builds/fs2-39db6ede7ef12fb3f6ceb78567d325c845b86955/core/shared/src/main/scala/fs2/async/async.scala:148:18: method shift in trait Async cannot be accessed in cats.effect.Effect[F]
[fs2] [error]     F.runAsync(F.shift(ec) *> fa)(f).unsafeRunSync
[fs2] [error]                  ^

this look like a version mismatch thing between fs2 and cats-effect rather than a Scala issue

dealt with it by freezing cats-effect at 0.9
also monix just today moved off cats-effect 0.9 onto a development version; so we have to freeze monix too
we can unfreeze all this stuff (or just move to newer tags, tags are probably better; it's hard trying to keep up with fast-evolving projects like these) once it settles down again
fyi @mpilquist @alexandru

@SethTisue ouch, sorry, the upcoming cats-effect 0.10 contains some cleanups, the project being still binary compatible but not source compatible.
It's coming up next week: https://github.com/typelevel/cats-effect/milestones

ouch, sorry

no worries, it's normal


Can a manual timetable be created in unitime?

You could really really really benefit from reading the Course Timetabling Data Entry and Course Timetabling Solver manuals (see Help > Manuals).
You can fix a room at a particular time and/or room by requiring that time and/or room. You can also make manual changes to the timetable using the Suggestions page. You need to have the date loaded into the solver, but the solver is only used to provide suggestions and check the conflicts.
Once a solution is saved and committed, manual changes can also be made using the Class Assignment page (no need to load the data into the solver), using the Class Assignment button on the Class Detail page.


git clone: case-sensitive paths on a case-insensitive filesystem

$ git clone -b develop https://github.com/NIKEA-SOFT/HelenaFramework
Cloning into 'HelenaFramework'...
remote: Enumerating objects: 6080, done.
remote: Counting objects: 100% (1337/1337), done.
remote: Compressing objects: 100% (522/522), done.
remote: Total 6080 (delta 847), reused 1183 (delta 800), pack-reused 4743
Receiving objects: 100% (6080/6080), 3.39 MiB | 4.74 MiB/s, done.
Resolving deltas: 100% (3685/3685), done.
warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:

  'Helena/Traits/SameAS.hpp'
  'Helena/Traits/SameAs.hpp'

Thanks, I didn't notice this due to the fact that this problem was only in the repository, it was due to core.ignorecase in git, which by default is true
Fixed and master branch updated.


Update README.md

Codecov Report

Merging #1034 into develop will increase coverage by 0.01%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1034      +/-   ##
=============================================
+ Coverage      62.17%   62.18%   +0.01%
  Complexity      1141     1141
=============================================
  Files            528      528
  Lines          15763    15763
  Branches        1831     1831
=============================================
+ Hits            9800     9802       +2
+ Misses          5193     5191       -2
  Partials         770      770

Impacted Files
Coverage Δ
Complexity Δ

...a/uk/gov/gchq/gaffer/cache/CacheServiceLoader.java
75% <0%> (+7.14%)
6% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Add support for source attachments for lib entries again

Heiko, I take it this refers to the fact that eclipse with-source=true doesn't actually download and attach the source jars anymore?  Any idea how long it will be before this works again?  Being able to step into the Lift and Squeryl source from the Eclipse debugger was really useful.

This is a must. I realy miss this feature.

Folks,
Relax, this feature will be added again soon ;-)
Heiko

Is the an example somewhere on the wiki that shows how to turn it on?

There are no examples on the wiki yet, but you could take a look at the tests, e.g. here:
https://github.com/typesafehub/sbteclipse/blob/master/sbteclipse-plugin/src/sbt-test/sbteclipse/02-contents/project/Build.scala#L31


extra terminal processes hanging around on restart

Testing #174837
Regardless of window.experimental.sharedProcessUseUtilityProcess, for every N terminals in a window, and R restarts, there are N * R processes shown in the activity monitor.
https://user-images.githubusercontent.com/29464607/220416103-e771334a-5df1-44d9-bbf4-630ce5413fa0.mov

quit and reopened again and now there are 6

other info:

They're not listed as zombie processes, so I believe they're still running. Reloading neither creates nor destroys processes

I have to select Force Quit to kill them

Similar issue on Linux.
On Ubuntu 22.04 on macOS Parallels, the shell integration scripts remain after closing code-insiders: /usr/bin/bash --init-file /usr/share/code-insiders/resources/app/out/vs/workbench/contrib/terminal/browser/media/shellIntegration-bash.sh. Using ps aux | grep code-insiders, the status of those processes is Ss+. I'm wondering whether something might be causing the shell integration scripts to fail?

interesting finding @rzhao271 . I wanted to see if this reproduces without shell integration enabled, but it looks like only terminals with shell integration persist at this pt, which is also unexpected and a different bug

also not reproducing with zsh anymore, though I'm sure it's still an issue because when I initially opened activity monitor (with zero terminals), I had 50+ zsh processes reported

I can easily reproduce this in a vanilla Ubuntu using bash. The problem is the shell integration shell traps signals and ignores SIGTERM. The SIGKILL signal works just fine. https://github.com/microsoft/vscode/issues/176549

The freeze-slow-crash-leak label might be an understatement?  Any processes that were started from the terminal continue to run after VSCode exits.

Repro out of sources on mac:

./scripts/code.sh
Enable shell integration
Open a terminal
ps | grep bash, count procs
ctrl+c ./scripts/code.sh (doesn't repro on cmd+q)
ps | grep bash, :bug: count is the same

Can repro using this profile in 1.76.2:
  "terminal.integrated.profiles.osx": {
    "bash": {
      "path": "bash",
      "args": [
        "--init-file",
        // Just an empty file
        "/Users/daimms/dev/microsoft/vscode/src/vs/workbench/contrib/terminal/browser/media/empty.sh"
      ],
      "icon": "terminal-bash"
    }
  }

Could this be related to the posix spawn change in node-pty?

The posix spawn change and other node-pty changes came in on Jan 19: https://github.com/microsoft/vscode/pull/171583

Yes, there's this too https://github.com/microsoft/vscode/issues/177044

Better repro for OSS: kill -s SIGKILL <mainpid>

This can be reproduced in OSS with:

./scripts/code-cli.sh
Enable shell integration
Open a terminal
ps | grep bash, count procs
Open VS Code process explorer, get main process PID
kill -s SIGKILL
ps | grep bash, 🐛 count is the same

After downgrading node-pty, it does not reproduce:
yarn add<EMAIL_ADDRESS>
@deepak1556 @rzhao271 I'm going to need some help on this one, something between<EMAIL_ADDRESS>and<EMAIL_ADDRESS>caused this problem. Unfortunately it's not really possible to tell the exact commits that each beta build points at, the bundles are fairly small though. My guess is it's because of https://github.com/microsoft/node-pty/pull/487 since that blocked me from upgrading for over a year.

@Tyriar I can investigate this today.

The issue doesn't seem to be resolved in version 1.77, as stated in #176819

Similar issue on Linux. On Ubuntu 22.04 on macOS Parallels, the shell integration scripts remain after closing code-insiders: /usr/bin/bash --init-file /usr/share/code-insiders/resources/app/out/vs/workbench/contrib/terminal/browser/media/shellIntegration-bash.sh. Using ps aux | grep code-insiders, the status of those processes is Ss+ (waiting for an event to complete, and is a "session leader in the foreground process group"). I'm wondering whether something might be causing the shell integration scripts to fail to complete?

On gnome this also causes lag during the shutdown. Stale shellIntegration-bash.sh, also ignores SIGTERM sent during shutdown, which then waits for two minutes and finally sends SIGKILL

Per #180894 :
On MacOS the processes are properly killed when explicitly closing the workspace window, but remain running if you simply quit VSCode.
This behavior is the same even with terminal.integrated.enablePersistentSessions set to false. It is the same in all three modes of terminal.integrated.persistentSessionReviveProcess.

@joshstrike thanks for that info. by "quit VS code" I assume you mean via the mac dock?

@meganrogge
Quitting via Cmd-Q. But the same happens when quitting via the dock.
Cmd-Shift-W to close the window terminates all running shells.

@joshstrike does the issue repro on insiders as well https://code.visualstudio.com/insiders ?

@deepak1556 Confirm, insiders version fixes the issue, all terminals correctly close upon Cmd-Q quit.

Thanks for confirming, the fix will be available with v1.78 next week.


[Feature Request] Support for NanaZip.

NanaZip is literally exactly the same as 7-Zip, but a much more modern implementation of it. Especially on Windows 11. Whatever 7-Zip can do, NanaZip can do also which is why I think NanaZip should work fine with your Driver Updater.
It isn't an exe like 7-zip, but instead a MSIX package, so I am not sure if that matters or not.

I'll look into this on the weekend when I have a bit of time. If it supports the same commandline args then it should be a few minutes to implement.

Impossible. NanaZip does not have CLI

Impossible. NanaZip does not have CLI / does not expose anything we can use

Yes it does, you type NanaZipC or NanaZipG in the terminal/console (all lowercase works also). It is not documented for some reason but that is the CLI. Those work the exact same as the 7zip equivalent (7zC 7zG).

You're correct.. yea why isn't this documented wtf.


[tcgc] add generated names for constants

fixes #553

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - fix ✏

add generated names for constants

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/766/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/766/

i had the same idea but not finished the impl. thanks for the work.


dockerignore prevents docker-build from working

Expected Behavior
yarn run docker-build should build a Docker container.
Current Behavior
» yarn run docker-build
yarn run v1.22.1
$ yarn bundle && docker build . -t spotify/backstage
$ yarn workspace example-app bundle
$ backstage-cli app:build
  109.19 KB (+4 B)  dist/vendor.ae73c9b7.chunk.js
  104.58 KB (-7 B)  dist/module-material-ui.ed6743e0.js
  77.31 KB          dist/8.15826e0a.chunk.js
  70.33 KB (+13 B)  dist/main.51ee850c.chunk.js
  68.38 KB          dist/module-moment.a16868af.js
  35.78 KB          dist/module-hot-loader.3a90be9a.js
  33.28 KB (-3 B)   dist/module-material-table.a0d67609.js
  22.92 KB          dist/module-react-beautiful-dnd.3aa280ee.js
  1.15 KB (+2 B)    dist/runtime.030279ee.js
Sending build context to Docker daemon   16.9MB
Step 1/6 : FROM nginx:mainline
 ---> 9beeba249f3e
Step 2/6 : COPY packages/app/dist /usr/share/nginx/html
COPY failed: stat /var/lib/docker/tmp/docker-builder987065707/packages/app/dist: no such file or directory
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Possible Solution
Remove packages/*/dist from .dockerignore and the docker build works fine.
Steps to Reproduce
» git clone git@github.com:spotify/backstage.git
» cd backstage
» yarn
[output omitted]
» yarn run docker-build
[fails with the output above]

Your Environment
Docker Desktop for Mac
» docker --version
Docker version 19.03.5, build 633a0ea

Does anyone know why packages/*/dist was added to the .dockerignore in the first place? I'm not sure what I might break by removing it.

@dtuite I would love to work on this!

All yours @Mutugiii !

Thanks @Mutugiii -- I have assigned you now! 💯

@Mutugiii It's safe to change it as per @dtuite:s suggestion as far as I know. We changed the build system, and this should adapt the Dockerfile accordingly.


Heap corruption of Lists of numpy arrays

Lists of numpy arrays (at least of float 1D type) cannot be brought to length zero without heap corruption. Best guess at this point: refcount issue and double free of allocated memory.
Also, Lists of numpy scalars for some reason do not suffer this issue.
Minimal reproducer:
import numpy as np
from numba import  float64
from numba.typed import List

float_array = float64[:]

l = List.empty_list(float_array)
l.append(np.ones(3,dtype=np.float64))
l.pop()

@Viissataa thanks for reporting this on the Numba issue tracker. I have reproduced this locally and can confirm, that it is likely a bug.

After a bit of debugging, this is likely a refcounting issue, essentially a double free is the root cause of the SIGSEGV.

@Viissataa sorry for the delay on this one, I just got around to looking at it. I changed the reproducer slightly, so as to assign the incoming array to a variable. I then no longer receive the segfault. I was wondering if you could try the example and see if you see the same?
import numpy as np
from numba import float64
from numba.typed import List

float_array = float64[:]

l = List.empty_list(float_array)
z = np.ones(3,dtype=np.float64)
l.append(z)
l.pop()
print(repr(l))

I still get segfault

My numba version is 0.46.0 as per pip.

Possible fix in #5602 - @Viissataa would you have a chance to try that fix?


Add default output stage.

Fixes #536 .
Changes proposed in this PR:

Add default output stage.

This is not how the overall architecture works!
metadataUri is used by pod-configuration to know where to fetch DDOs from (https://github.com/oceanprotocol/pod-configuration/blob/4ba2172b7ca2e1804f74b13d9a6126983b67a363/src/index.js#L95)
Making that public IP covers only one small edge case
Example

you have a network with private IPS.  <IP_ADDRESS> is used by node , and your kubernetes cluster is using class <IP_ADDRESS>/24 for pods.   And you have no port forwarding.      Setting metadataUri to some external IP will not make pod-configuration working by magic..

Remove this PR please and replace it with:

add an env variable, called C2D_NODE_URI .   And use that. It's up to the network administrator to set this properly

Ok sure then I'll let up to the user to set correctly, not trying to fallback to anything.

closing this and will refactor the logic in another PR


Add edit Index Template

Add edit Index Template, only have create/delete option

I second this 👍

Just noticed I can't do this - +1 :+1:

1

I need this too ;)

Check #22

+1


mcaptcha package and module with nixos tests

This completes https://github.com/ngi-nix/ngipkgs/issues/17

CI is failing because you seem to be introducing IFD.

@lorenzleutgeb, we've updated this PR to remove IFD. Unfortunately, this required vendoring a number of Rust Cargo.lock files, and a couple of node package.json files. As far as we can tell, this is just the way things have to be:

buildRustPackage can work without vendoring Cargo.lock, but it doesn't support git dependencies, which unfortunately are used in the mCaptcha ecosystem.
buildYarnPackage seems to require you to vendor the package.json. We weren't able to find a way around that.

Hi @lorenzleutgeb, we have split the config between "regular options" and "settings" as specified in RFC 42. Could you please take another look on this PR? Thanks!

@lorenzleutgeb if you intend to review this, we'll wait for you. Just let us know please. No pressure.

@lorenzleutgeb if you intend to review this, we'll wait for you. Just let us know please. No pressure.

Yes, I do intend. Thanks for the reminder and sorry for the delay.

There are many "unresolved conversations". I'd hope that you would comment why you decide not to resolve them, or if you add changes that comply with my comment, mark them as resolved. I'll do that now.

If there are open conversations, please either respond to them or mark them as resolved before requesting another review.
Conversation that are missing your reply even though you requested another review:

https://github.com/ngi-nix/ngipkgs/pull/61#discussion_r1366761819
https://github.com/ngi-nix/ngipkgs/pull/61#discussion_r1366762915
https://github.com/ngi-nix/ngipkgs/pull/61#discussion_r1366753894
...

Conversations that were resolved, but you did not indicate that you resolved them:

https://github.com/ngi-nix/ngipkgs/pull/61#discussion_r1366740479
https://github.com/ngi-nix/ngipkgs/pull/61#discussion_r1366740661
https://github.com/ngi-nix/ngipkgs/pull/61#discussion_r1366740800
...

I have politely asked multiple times for you to reply to unresolved conversations and mark conversations that you have resolved as resolved. You just keep hitting "request review" without any written response. This is annoying. I will not review this anymore before you at least reply to the open conversations.

Missing comments is enough to cause mayhem. The specifics don't even matter. Sorry about that.

For more complex changes, i.e., when it's not as obvious as "accept suggestion" or "don't accept suggestion", but rather a discussion, I do get the point about "stepping on the reviewer's toes".

My concern with this approach is that there could be issues where the reviewer finds them non-trivial and they get resolved by the author thinking they are. Also, cases where the resolution would not appease the reviewer. Therefore I don't like pressing anyone's resolve button. This approach provides me with the guarantee that the reviewer is satisfied with each issue.
Is there anything remaining?

Rebased. @lorenzleutgeb .

Congratulations! @realaravinth.


api - adopt vscode.env.asExternalUri

@bpasero With either these changes or with the old createAppUri method, it seems that the url handler isn't being triggered on the callback. has that logic changed at all?

No, but there could be other bugs in URI creation/parsing that cause this.


chore(main): release pixel-motion 1.0.6

:robot: I have created a release beep boop
1.0.6 (2024-12-29)
Bug Fixes

deps: update dependency framer-motion to v11.14.1 (#623) (44e96cb)
deps: update dependency framer-motion to v11.14.3 (#626) (2a76d24)
deps: update dependency framer-motion to v11.14.4 (#627) (d0eb6aa)
deps: update dependency framer-motion to v11.15.0 (#632) (38246f9)
pixel-motion: fix test sort sonarqube issue (1557944)
uniform: more refactoring and fixes for FieldArray (b8514fa)
use v8 for coverage + exclude some code (d2cc6d8)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

pixel-motion-v1.0.6
:sunflower:


major memory leak or lag in current version

I am a modpack creator (Treasurecraft), and as such I am always updating, testing and tweaking my pack. I recently updated all of the mods in my pack in including Roots. When I loaded the game I was getting this message "Can't keep up! Did the system time change, or is the server overloaded? Running 2651ms behind, skipping 53 tick(s)," but more like 50000ms behind.... I suspected Roots because I was getting another message randomlyAfter disabling Roots the game became playable again. Here is a modlist not including Roots:
modlist_2019-13-07.pdf
Here is a log with roots installed after updating (and crashing...):
2019-07-13-1.log
Thank-you

Update both mystical lib and mystical world, then try reinstalling. I'm surprised it even boots! Are you using thr latest version of roots as well?

Everything is updated (though Twitch). I've tried reinstalling twice now. That's why I figured I would mention it here. I disabled Roots but still have mystical lib and mystical world installed and active. Everything works fine without Roots.

What version of Roots are you using?

Roots-1.12.2-3.0.2.jar

I did a quick google of the "root" error message and, as I suspected, it actually has nothing to do with Roots the mod. It's specifically referring to the "root" of the game. It's an error message that has been around since 1.7.2 and in Vanilla.
[19:02:00] [Server thread/INFO]: This world was saved with mod roots version 0.104 and it is now at version 1.12.2-3.0.2, things may not work well this line here along with the considerable number of errors seems to imply that you've updated directly from Roots 2 to Roots 3, which could be problematic.
Based on the fact that this error occurs immediately after logging into the single-player world: [19:03:06] [Server thread/WARN]: Can't keep up! Did the system time change, or is the server overloaded? Running 19380ms behind, skipping 387 tick(s) I'm going to assume there is a root issue with your system.
Have you tried to see whether this occurs in a new world, instead of constantly reusing your existing world?
The large quantity of [19:05:37] [Client thread/ERROR]: Got a genetic item with no genome, setting it to a default value.
[19:05:37] [Client thread/ERROR]: Got a genetic item with no genome, setting it to a default value. seems to imply that there could be an issue there as well, but that's probably unrelated.
java.lang.NoSuchMethodError: mezz.jei.runtime.JeiRuntime.getActiveAdvancedGuiHandlers(Lnet/minecraft/client/gui/inventory/GuiContainer;)Ljava/util/List;```

This error message seems to imply that you have some major conflict between your version of Code Chicken Lib and your version of JEI and what they're expecting to find. It's possible that your Twitch is set to only load release versions but the most recent version of JEI is alpha -- but I'm not entirely sure on that note.

[Reddit does suggest](https://old.reddit.com/r/Minecraft/comments/5antet/memory_leak_or_severe_lag_in_my_world_help/) that (for 1.7.10 at least) these "root" error messages may be directly related to the number of loaded entities. `/forge entity list` will help you look at what they are. A good trick if you have a large number of item entities is to run `/tp @e[type=item] ~ ~ ~` which will print out the item's location and what the item is (though you will need to consult your log to see it).

Likewise, be aware that, when playing in single-player mode, "pausing" the game by hitting escape or alt-tabbing can cause the server to become extremely backlogged. I'm unsure of the specifics but I've encountered it myself in single-player worlds.

So, finally, have a look through the above suggestions, try creating a new world and ensuring you don't accidentally pause the game in any fashion, and then get back to us.

Otherwise, I don't see anything to indicate that any of the Mystic Mods (especially Roots) are specifically responsible for this, although it may be that something has become corrupted in the transition from Roots 2 -> Roots 3.

Will I will try and let you know, thank-you

I created a new world and everything seems to be working fine (with Roots installed). Oddly enough I'm not getting any hostile mob spawns...(I've encountered this before). I typed in /tp @e[type=item] ~ ~ ~ on a newly created world and had ~35 entities loaded in a non skyblock overworld. The genetic genome thing has been around for ages, it's a forestry, gendustry, binnie's mods thing (I basically ignore it). I think the genetic issue is related to looting chests or lootbags. I will absolutely will look into the JEI - Code Chicken Lib thing too (I noticed it but found little info related to it. It's been there for a little while now). I believe that the primary cause of my issue was the upgrade from Roots 2 to Roots 3. I didn't notice that it upgraded until things where added/removed from my world when it loaded after the upgrade. I also googled the "roots" thing and basically discounted it as an issue since I was in a sky block world with a very small mob farm (7-10 mobs max at any one time). I'm used to the pausing type lag spikes, but what I was encountering was more like when I hit a mob it was like watching the mob get pushed back in slow motion or clicking on a chest to open it and waiting multiple seconds for it to open.
Thank-you for your help.

I'm not sure of the exact specifics and I need to double-check with some people who know more about the game than me, but I believe that actually "pausing" the game means that when you unpause it, it tries to catch up and go through everything that should've happened while it was paused. At least, this is what I was experiencing in some test single-player worlds -- but never in a dedicated multiplayer world.
I know there's a gamerule to prevent the game from being paused when you alt-tab, so if this is indeed the case for you, perhaps that's something you could consider trying? Of course, that relies on pausing being the initial cause.
Outside of that, profiling with VisualVM to determine to the source of the issue (or with YourKit or any other product) would be one way to narrow it down, but that requires a little more experience. I'd recommend the FTB subreddit, or any of the modded Minecraft discord servers -- the responses can be flakey depending on who responds, but there are a lot of amazing, awesome people in the community who are willing to help -- and probably know more than I do!