Make pellets look smaller but have the same mass

I want the pellets to be smaller but then when you eat them you gain ~8 mass instead of 2-4 but still have the same size. Hopefully not to complicated!

Change entity code.

@pabloalber84 He is asking the exact code to change it, not asking where it is located.
Unless @MineMooreALT could do it himself.

Yes i'm not 100% sure where the size of the pellet is located

@MineMooreALT  I guess You can change the size of food pellets in https://github.com/Megabyte918/MultiOgar-Edited/blob/master/src/GameServer.js#L798

and if i change it will they still get the same mass for eg i set it to pellets = 100mass then i go and change the size in gameserver to 10 they will still get 100 mass?

@MineMooreALT change the pellet grow from 1 to 0

https://github.com/Megabyte918/MultiOgar-Edited/issues/642

I didn't close it for a reason. My issue had still not been resolved, the code in that other issue is old.

@MineMooreALT UpdateNodes.js (appearance):
// Search for this line:

writer.writeUInt16(node._size >>> 0);

// Replace each of them with this:

writer.writeUInt16(node.cellType == 1 ? newSize : node._size >>> 0);

Replace newSize with the size you want. For example to make them look 2x smaller you would replace it with node._size / 2
Cell.js (size gained):
// Search for this line:

this.setSize(Math.sqrt(this._sizeSquared + prey._sizeSquared));

// Replace it with this:

this.setSize(Math.sqrt(this._sizeSquared + (prey.cellType == 1 ? newSize : prey._sizeSquared)));

Replace newSize with the size you want. For example to double the size you gain you would replace it with prey._sizeSquared * 2


Need an "Exec" role

Describe the feature you'd like
I'd love to see an "Exec" role... can see everything, especially the audit log (to track progress), and cannot change anything.
Describe the benefits this would bring to existing BookStack users
The biggest benefit of this feature would be protecting BookStack content from those who demand to see everything in the system, but have no business being able to change content, permissions, or anything else in the BookStack install.
Can the goal of this request already be achieved via other means?
I don't think so.  There's a similar request, #4308, but no action nor update
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
Under 3 months
Additional context
No response

Thanks for the request @dagnabbit99.
From my perspective I wouldn't want to complicate the permission system to suite this.
If there's demand over time maybe the audit log could be under its own permission, but I'd be hesitant to expand that further to viewing everything (that may be soley represented via forms right now).
As an alternative means, you could use the BookStack API or database to build relevant views/reports externally using Business-Intelligence/reporting tools if available.

Thanks, was thinking this myself- a daily audit log journal sent via email would work just as well.  Could easily use the API to pull that info.


does warp support library imports?

My contract comes with standard openzeppelin imports:
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
Is there an easy way to get these imports transpiled other than copy pasting them in the current path?

Hey!
We don't support hardhat style import paths. Sorry.


Typo in "Proving uniqueness of identity proofs from J"

Well, we can also see some downsides of this definition of the equality type: it allows both sides to have the same type.

should be

Well, we can also see some downsides of this definition of the equality type: it allows both sides not to have the same type (or, to have different types).

Also, in

So, the proof is cyclic.

I would suggest changing cyclic to circular.

By the way, why is Axiom K "stronger" than Axiom UIP?

By the way, why is Axiom K "stronger" than Axiom UIP?

I guess probably not. This should be a mistake.


Add Logger Support

The current implementation is locked to SLF4J.  We replace this with support for the Logger abstraction.

'feign.Logger' is not designed for reactive work. Can introduce ReactorLogger but don't have enough reason for this as SLF4J is also a high level abstraction.


Protobuf dll errors when parsing large packets

proto.zip
bug report.zip
I have found an issue when trying to parse large data packets of UDP data through the protobuf dll.
Issue Description
I have used the customers proto file to generate the type cluster contained in the generated mdc lvlib - using the grpc-labview scripting tools.
For some initial tests, I built a PoC which generates randomised data, then serialises and deserialises the data to check it is valid (I shared this vi in a previous bug #119 ).

For this test I have copied the data from the Buffer indicator on the front panel of the previously shared vi (screenshot above). That data has been pasted onto the block diagram of the "bug demo.vi" (attached in "bug report.zip" under the bug report folder) as the simulated data (screenshot below)

I have pcap files generated through the NI XNET driver (with an NI-8623 module) with protobuf data from a customers ADAS ECU. I can view the data in Wireshark correctly (information on how to configure Wireshark to do this is below). For this test I have copied the data field of one of the protobuf frames, and pasted it as my non-simulated data in "bug demo.vi" (screenshot below)

The original pcapng file is also included in the same folder as "bug demo.vi"
When I run the vi and select Sim, I get no errors and the Unpacked data is valid.

When I turn Sim off, I get an error from the dll and the Unpacked data is just default values.

Configure Wireshark for Protobuf
In proto.zip there is a set of proto files from the customers ECU. Below is a description I wrote for the customer to show how to configure Wireshark to show the protobuf frames. This will be the easiest way to see the data that should be shown in the vi.
Go to Edit >> Preferences, and in the left panel there is a drop down list for Protocols. Find Protobuf in there.
Click on Edit Protobuf Search Paths, and add the directory on your PC as a source directory. For reference, I took the proto files used in my PoC work, and used them, with one of the additional Google proto files downloaded from Github. I’ve attached these as a zip to avoid confusion.

Then click on Edit Protobuf UDP message types, and add port 8204 with the message type MDC_Payload. Exit the dialogs.

You should then be able to load up a pcapng file from XNET. In the display filters box at the top type protobuf – this will limit the displayed results to only packets that have protobuf data.

For each packet, you should now be able to drill down into the Protocol Buffer section of the middle pane, and view your signal data.

@phorn-NI I debugged this issue and found that it is an invalid entries in the protobuf data that is tripping up grpc-labview. Specifically, the data after the tail as shown below.

tail is the last entry in MDC_payload and I am not sure why we are getting these additional data from the ADAS ECU after all the data for the MDC_Payload itself. These data values are not recognized as part of the serialized data for MDC_Payload and hence we wouldn't know how to process them.
We could probably ignore these bits of data and not cause the errors you are seeing. I will take a look at how situations like these are handled by protobuf in other languages and see what we could do in grpc-labview.

Thanks @shivaprasad-basavaraj , I will ask the customer about the extra data at the end.
I remember now that the 4 bytes at the start is a header field. Currently it is mostly blank, but the very first byte (02 in the packet shown below) is the payload type. 02 means it is MDC_Payload so we can deserialise it correctly - there are 4 other potential types which will have a different serialisation structure (but for our purposes we can just ignore the frame if the first byte is not 02)

@shivaprasad-basavaraj thanks
The rest of the data should parse correctly though shouldn't it (like it does in Wireshark)? Could the function still pass out the data but raise an error/warning saying there was additional field that don't fit the type?

@phorn-NI I am guessing WireShark is just doing a best-effort guess for what those data fields should be based on the proto file provided and just ignores what it doesn't understand.
My gut feel is grpc-labview should more closely resemble the behavior of other programming languages which support gRPC which seems to be to error out if the required data doesn't prescribe to the protobuf wire format at all (not just it cannot find the protobuf index and such but it cannot make any sense of the data at all).
@ccifra What do you think the behavior of grpc-labview should be when we run into situations like these?

thanks @shivaprasad-basavaraj that makes sense.
I've explained to the customer that we can continue to use tshark to parse the data but it is slow, and there is a risk of incorrectly parsed data (don't know how large the risk is), or they need to remove these bytes from the packet data from the ECU
It is also possible (and I've asked for confirmation) that I just have an out of date version of the proto file.

@shivaprasad-basavaraj @ccifra I checked with the customer and my proto file was slightly incorrect - the second line was commented out on mine, it shouldn't be.

nanopb.proto is another file they have which imports the google descriptor proto file, and also has lots of optional parameters. I remember from a previous conversation with Chris that optional parameters are not supported - is that still true?
Anyway, I re-enabled the import statement and viewed the data in Wireshark, the same bytes are still labelled as unknown.
The customer isn't willing to accept a risk that the parsing is incorrect (this is the main stream of output data they need to use to validate their tests - if it is incorrect the test is junk) - which means using tshark or our API with those values in the UDP data are not valid options. They have an internal application which displays the data, so it must be somehow handling the data - I've asked them to explain how that application handles it.

@shivaprasad-basavaraj I know what is going on now.
The UDP packet is used to serialise lots of data including mdc_payload. So the invalid packets (and everything after that - such as the additional faceboundingbox field) is actually part of something else and can be ignored.
I manually changed the data being written to the parsing functions in my test, but removing the first 4 bytes and everything after the start index of the invalid packets.

I worked out from the wireshark data that this was index 450 for this packet. From my previous serialisation/deserialisation experiements, the packet length was more like 360-370 bytes. So I have 2 questions.

Could we modify the function so that it parses the data and then has a separate output with the rest of the packet?
Alternatively, is there a way we can pre-analyse the packet to see how long we expect the mdc_payload data to be, so we can then trim it down before passing to the main parsing function?

@phorn-NI I think it would be difficult to handle this in grpc-labview in a generic way. If we start seeing bits that we don't recognize as part of the protobuf data then we cannot be sure that we have parsed any of the data accurately. The data may genuinely be corrupted and not data that is safe for us to ignore and separate out as in this case.
I think we should handle this in G code for this specific scenario. I think ideally the packet from the ECU should include information such as the number of bytes for each data that they are serializing. Then we could use this information to split up the bytes in G before it is passed on to the grpc-labview layer to unpack.
Also, it is dangerous to assume that bytes for MDC_Payload would end at byte as in the example above. Protocol buffers use variable length encoding and different data would mean different number of bytes used to encode this data.

The issue was that in the attached example the test data was being generated by a special implementation of protobuf called nanopb. nanopb has a special mode for encoding data which signifies end of data by adding an entry with tag id of 0. This is not standard protobuf behavior and we expect it to work only when both ends of the data communication are built on top of nanopb unlike grpc-labview.


Fix various typos in comments / strings / names

What
Fixed a few minor misspellings in comments / strings etc. The only
behavioural changes should be that some error messages / log messages
will be spelled differently.
How to review

Code review (to check for unintended behavioural changes)
Run the tests (or let Travis run them)

Who can review
Anyone other than Rich.

An incredibly pedantic comment but it should be "Cloud Foundry".

LGTM


Fix DoNotGenerate

[NativeFunction(NativeFunctionOptions.DoNotGenerate)]
was not working correctly and resulted in native functions being written for load_assembly_from_data and load_assembly_from_path

I am actually leveraging this bug in my current set of work. Oops. Could you add a CallbacksOnly option? Signature only sets up an exported function which isn't what I needed. In fact I don't think I wound up needing signature only so that could be swapped out for the callback only variant.


[BUG] Tools have some duplicate rows in data_source_information CSV file

Describe the bug
Running both Q/P tools on eventlog ${QUAL_DATA}/dataproc-2.1/nds-h/powerrun-unoptimized-3k-parquet-train/eventlogs/cpu/ shows some duplicate entries in data_cource_information.csv.
There is a couple of possibilities:

Some nodes are processed twice as datasourceV1 and datasourceV2
there is a column such as "PartitionFilters" that does not show the difference between the entries.

This issue is caused by the way we process the matadata and try to link it to a specific nodeId
Inside checkMetadataForReadSchema we always pick the first node in the list. This leads to multiple rows contraining same nodeId although it comes from different plans/nodes.
This affects the correctness of the IO metrics because we completely ignore the remaining scan nodes in the list.

After some thoughts, this is the best strategy to fix this issue:
Create a custom ToolsPlanGraph builder that accepts a call back function.
While visiting the nodes recursively, the call back function will capture all ReadNodes and pull the metadata from the SparkPlanInfo if any.
This implementation will affect the flow of the tools because once this is implementation is in place, we can use it for both V1/V2 reads and we can get rid of the legacy implementation that only relies on sparkPlanGraph.


documentation issue (dev DM deploy)

https://github.com/Seagate/cortx/blob/main/doc/VIRTUAL_MACHINE.md
PRE-BUILD [MOTR], link to:
https://github.com/Seagate/cortx/blob/main/doc/CortxMotrQuickStart.md --> (404 error)

Issue-Label Bot is automatically applying the label bug to this issue, with a confidence of 0.80. Please mark this comment with +1 or -1 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

Thanks @korenlev !


Not compatible with Laravel 5.4

php artisan api:generate --routePrefix="*" --force --useMiddlewares --bindings="question,1|answer,1|faculty,1" --header "Authorization:Bearer eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjgwMDAvdXNlci9sb2dpbiIsImlhdCI6MTQ4Nzc2MTQ5OCwiZXhwIjoxNTE5Mjk3NDk4LCJuYmYiOjE0ODc3NjE0OTgsImp0aSI6InVFS0VJZEJ4ZndRSVJOSUMiLCJzdWIiOjEsInVzZXIiOnsiaWQiOjF9fQ.kx-ND7IDCkF3bmssVfVwSZgSI9IWq9AQuhYLB7iYyt0"

Returns 404 errors, so it is not setting the bindings
Sets no headers, so authorization is not possible

I am using jwt for authorization, but the token is valid and the urls work with postman
So right now I can only use it for static routes :(

Same issue here.
And actually I think that --useMiddlewares is not working as expected. For instance if you want to use a specific middleware (--middleware=auth:api) and avoid using the --useMiddlewares option, it will always set 'middleware.disable' as true.. so, makes no sense.
@cre8 are you setting the Authorization Header by hand? I mean, would it be possible to auto generate a valid token to run the routes/controllers that actually need one?

@andrealeixo, right now i am creating one via postman and put it in the command manually but i think it would be possible to write a script that is doing this full automatically.

I think i solve this problem in PR165 ,but still not merge,you can try this.
in DingoGenerator.php, callRoute function.
collect($server)->map(function ($key, $value) use ($dispatcher) {             $dispatcher->header($key, $value);         });
change to this
collect($server)->map(function ($key, $value) use ($dispatcher) {             $dispatcher->header($value, $key);         });
then go to Generator.php in line 111
$route['output'] = (string) view('apidoc::partials.route')->with('parsedRoute', $route)->render();

@mpociot We have some issues with 5.4. Can you able to reproduce this bug? I found out that in first request I get response, but in all other requests I get 404. I'll try to find out whats the problem in the evening.

Okay guys. For me it started to work after i removed if statement at: vendor/mpociot/laravel-apidoc-generator/src/Mpociot/ApiDoc/Generators/LaravelGenerator.php:122
if (file_exists($file = App::bootstrapPath().'/app.php')) {
    $app = require $file;
    $app->make('Illuminate\Contracts\Console\Kernel')->bootstrap();
}

Dunno if I broke something or not - but at least it works for me now. :)

Yeah, that block is unnecessary and has now been removed.


New package: y-chan.SHAREVOX.NVIDIA version 0.2.1

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

1.7GB

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Manual Validation ended with:

@wingetbot waivers Add Validation-Executable-Error


Cann't run on meizu Max Pro

The example3 cannot run correct on meizu Max Pro phone.
Can anyone give me some suggestion?

Did you by chance make it run on Meizu? I cant make it run on a ONE plus 1. I have android 6.0.1 in my phone. I get a "Fail to connect to camera service" error


bridge: Update stp,rstp statistics every stats-update-interval seconds.

    Moves the field sec_in_state field from status column in Port table
    to statistics column in the same table.This helps in reducing the number
    of update sent to the controller, with the current mechanism
    every-time there is a change in this field a notification is sent to
    the controller. For this field its every second, since it's just
    counting the number of secs it has been in that particular state.
    By moving this under port statistics, this can be configured with the key
    "stats-update-interval" in Open_vSwitch table.

    Split the update of rstp_statistics column  and rstp_status column in
    Port table into two different functions.This helps in controlling the
    number of times the rstp_statistics column is updated with the key
    "stats-update_interval" in Open_vSwitch table.

Thanks for the patch.
We need a Signed-off-by for this patch. The FAQ says:
Q: What's a Signed-off-by and how do I provide one?
A: Free and open source software projects usually require a contributor to
provide some assurance that they're entitled to contribute the code that
they provide.  Some projects, for example, do this with a Contributor
License Agreement (CLA) or a copyright assignment that is signed on paper
or electronically.

For this purpose, Open vSwitch has adopted something called the Developer's
Certificate of Origin (DCO), which is also used by the Linux kernel and
originated there.  Informally stated, agreeing to the DCO is the
developer's way of attesting that a particular commit that they are
contributing is one that they are allowed to contribute.  You should visit
https://developercertificate.org/ to read the full statement of the DCO,
which is less than 200 words long.

To certify compliance with the Developer's Certificate of Origin for a
particular commit, just add the following line to the end of your commit
message, properly substituting your name and email address:

    Signed-off-by: Firstname Lastname<EMAIL_ADDRESS>
Git has special support for adding a Signed-off-by line to a commit
message: when you run "git commit", just add the -s option, as in "git
commit -s".  If you use the "git citool" GUI for commits, you can add a
Signed-off-by line to the commit message by pressing Control+S.  Other Git
user interfaces may provide similar support.

Thanks for the patch.
We need a Signed-off-by for this patch. The FAQ says:
Q: What's a Signed-off-by and how do I provide one?
A: Free and open source software projects usually require a contributor to
provide some assurance that they're entitled to contribute the code that
they provide.  Some projects, for example, do this with a Contributor
License Agreement (CLA) or a copyright assignment that is signed on paper
or electronically.

For this purpose, Open vSwitch has adopted something called the Developer's
Certificate of Origin (DCO), which is also used by the Linux kernel and
originated there.  Informally stated, agreeing to the DCO is the
developer's way of attesting that a particular commit that they are
contributing is one that they are allowed to contribute.  You should visit
https://developercertificate.org/ to read the full statement of the DCO,
which is less than 200 words long.

To certify compliance with the Developer's Certificate of Origin for a
particular commit, just add the following line to the end of your commit
message, properly substituting your name and email address:

    Signed-off-by: Firstname Lastname<EMAIL_ADDRESS>
Git has special support for adding a Signed-off-by line to a commit
message: when you run "git commit", just add the -s option, as in "git
commit -s".  If you use the "git citool" GUI for commits, you can add a
Signed-off-by line to the commit message by pressing Control+S.  Other Git
user interfaces may provide similar support.

@blp Thank you for pointing this out. I did signoff and pushed again.. but some how the travis-ci build is failing. I looked at the errors and looks like clang is failing. The error doesn't point to anything obvious that i changed and the same travis ci build passed before. Can you help me understand what is the issue?


Delegation to subnet needed

Please see PR:
https://github.com/MicrosoftDocs/azure-docs/pull/119379

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

ID: 9203aa6f-daa2-a067-797e-c292902e8a4f
Version Independent ID: d1d3fe99-80ea-aa90-d09b-79142e88c96e
Content: Add or remove a subnet delegation in an Azure virtual network
Content Source: articles/virtual-network/manage-subnet-delegation.md
Service: virtual-network
GitHub Login: @asudbring
Microsoft Alias: allensu

@henrikwh
Thanks for your feedback! We will investigate and update as appropriate.

@henrikwh Could you please clarify the reason for opening a GitHub case, according to your update, you have already created a pull request and submitted it. The author is currently reviewing the proposed changes, and once approved, the changes will be published soon.
If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Maven build is not working.

Running in (complete) folder:
./mvnw install && ./mvnw spring-boot:run -pl application
ERROR:
Failed to execute goal on project multi-module-application-complete: Could not resolve dependencies for project com.example:multi-module-application-complete:jar:0.0.1-SNAPSHOT: Could not find artifact com.example:library:jar:0.0.1-SNAPSHOT -> [Help 1]

I'm using docker
docker run -it --rm --name my-project -v "$(pwd)":/root -w /root adoptopenjdk/maven-openjdk11:latest mvn clean install
Its works with gradle:
./gradlew build && ./gradlew :application:bootRun

Running in (initial) folder: is working.

It works for me, even in docker per your example. Are you sure you are doing the same? Maybe something changed.

Yes.

Yes.

You might manual update complete/application/pom.xml's dependencies com.example.library to com.library.library-complete

@aquariuslt i missed this... ;)


Javascript Callback

It is posible to make a callback when SweetAlert window has been
created? I'm trying to make a SummerNote textarea but it doesn't show. :(

Unfortunately no, but you can always use setTimeout to init SummerNote textarea after swal2 modal window initialization.


Add authorization data to Request Body/Request URL option

Description:
Hi, I am using POSTMAN to execute a GET OAuth1 request from woocommerce. It works nicely in Postman but only if use Add authorization data to Request Body/Request URL, and not when add authorization data to request headers
See here
https://user-images.githubusercontent.com/2772980/60738032-20a57500-9f5d-11e9-8c6b-fda081db7a41.png
Is it possible to configure oauthswift to add authorization data to body ?
OAuth Provider? (Twitter, Github, ..):
woocommerce
OAuth Version:

[x] Version 1
[ ] Version 2

OS (Please fill the version) :

[x] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[x] CocoaPods
[ ] Swift Package Manager
[ ] Manually

Library version:

[ ] head
[ ] v2.1.0
[x] v2.0.0
[ ] v1.4.1
[ ] other: (Please fill in the version you are using.)

Xcode version:

[x] 11.4 (Swift 5.2)

[ ] 11.x (Swift 5.1)

[ ] 10.x (Swift 5.0)

[ ] other: (Please fill in the version you are using.)

[ ] objective c

of found resolution
oauthswift.client.paramsLocation = .requestURIQuery


All about section

I can make the problem we solve responsive for mobile widths.
@2024-SANDHYA please assign this to me.


how can i see the orbit DB instance directly from the ipfs gateway?

Is there a way to see the orbit DB instance on the IPFS gateway?

Only data is stored on and replicated via IPFS, instances of OrbitDb are not.


Module Not working

The module does not work after activation displays an error when the payment button is clicked.
Php version 7.2
SSL activated and working properly.

Hi,
Thank you for trying our module on your platform. But a few things needs to be clear before you started using this module.

You have valid productions checkout credentials for your merchant account from bKash. legacy or USSD based keys will not work in order to use this checkout module.
You must have valid url set on your whmcs setup section.
SSL is required.
we tested our module with php 7.4, use other version at your own risk.
This module is free and OSS so there is no warranty or guarantee that this module work on your platform.
Self-Service module.

If you have legacy api keys then you will need this: https://github.com/NawabHost/bkash-merchant-for-whmcs
If you have still issue, please see other issue (open + closed) in this repo to resolve your issue.

I am testing using sandbox credential. It is not working with this


Reference discrepancy?

When installing the .TextExtractor package with dependencies on an empty project, I get the message:
Assembly 'TikaOnDotNet' with identity 'TikaOnDotNet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
uses 'IKVM.OpenJDK.Core, Version=8.1.5717.0, Culture=neutral, PublicKeyToken=13235d27fcbfff58'
which has a higher version than referenced assembly
'IKVM.OpenJDK.Core' with identity 'IKVM.OpenJDK.Core, Version=7.2.4630.5, Culture=neutral, PublicKeyToken=13235d27fcbfff58'

When examining the TikaOnDotNet nupkg nuspec, I find that the dependency is in fact <dependency id="IKVM" version="[7.2.4630.5]" />.
I have no wierd assemblyRedirects (I actually cleared them just to be sure), and IKVM is installed on the said project with the "correct" version. No other project is using it.
Upgrading the NuGet package manually does it, so this is a bug report.

Also, I have not installed using "ignore dependencies" either...

Try clearing your packages directory as a workaround. I will look into this.

On top of this, my NUnit3 does not seem to acknowledge my assembly redirects...

I am on a tablet at the moment. As soon as I can build and verify I'll be getting a new revision released.

Actually, I never installed TikaOnDotNet specifically. I thought just by installing the .TikaTextExtraction, NuGet would handle the dependency resolving by itself. But it seems this hasn't happened because of the default setting of Dependency handling: Lowest.
Thus, if I download only the TextExtraction package, it will pick the "lowest possible" compatible version of the TikaOnDotNet package, which isn't specified (I'd say here's the problem), so it installs TikaOnDotNet.<IP_ADDRESS>427, which isn't compatible because of the referenced DLLs.
What do you say?

Recommended solution would be to set the minimum version of the 1.12.x packages to be > 1.12.0, in the nuspec.

I can confirm that everything works as expected when updating all packages :+1:

Ok thank you for the helpful reporting.

This is nitpicking but I think it is not perfect quite yet:

TikaOnDotNet 1.12.1 was fixed by adding a versioned dependency on IKVM (= 8.1.5717)
But .TextExtractor is not yet dependent on this version, by using NuGet default settings we will not get TikaOnDotNet 1.12.1 by installing the latest .TextExtractor.

I think that we should bump both packages to 1.12.2, .TextExtractor having a dependency of TikaOnDotNet (> 1.12.2). THEN we will force the tested-to-be-compatible packages to go together.
If one does want to try something else, he will have to check "Ignore Dependencies" in NuGet to go wild.

Sigh. #42 will remove the coupling the two projects have. There is no reason they need to be versioned together. Another compatibility concern is #47.


Add bluetooth firmware and download script

@agherzan @telphan

Looks good to me other than having a patch for changes on the components we import directly from Samsung. In this way updating would be easy and we would be able to keep our changes.


Truncated long event titles

Insert description of PR here

✅ By submitting this PR, I have verified the following

[ ] Checked to see if a similar PR has already been opened 🤔
[ ] Reviewed the contributing guidelines 🔍
[ ] Added my name to the bottom of the list under the Credits section in the README.md with a link to my website or GitHub profile 👥

@ejbryant28 Once you fix the merge conflict so your name is on the bottom of the contributors section, it should be good to go!


[FLINK-31731] Add backwards compatible constructor for DebeziumAvroSerializationSchema

What is the purpose of the change
Adds a backwards compatible constructor for DebeziumAvroSerializationSchema that doesn't require passing an arg for the schema string. This allows the code in apache/flink-connector-kafka to build against Flink 1.18.x.
The constructor args for DebeziumAvroSerializationSchema was modified in e818c11d to allow passing in a nullable schema string. There were test code in apache/flink-connector-kafka that used that constructor, and therefore broke with that change and could no longer simultaneously build against Flink 1.18.x and 1.17.x.
There's a bigger question of why the flink-connector-kafka tests are using the constructor which is meant to be @Internal, but for now this should have the nightly tests passing again and unblock Flink 1.18.x. I'll open a separate ticket to revisit the dependent tests in flink-connector-kafka and potentially rework those.
Brief change log

Add back a constructor for DebeziumAvroSerializationSchema that doesn't require an arg for the schema string.

Verifying this change
The nightly tests in flink-connector-kafka should be passing again: https://github.com/apache/flink-connector-kafka/actions/workflows/weekly.yml
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? n/a

cc @MartijnVisser @PatrickRen @knaufk @mas-chen related to Flink 1.18.x  release blocker

Merging


Cannot select folded block of code

Folded block of code can't be selected (in order to delete, copy or move folded) as in ST. It unfolds when clicked.

This behavior is bugging me too.
Here's the situations I'm noticing (beginning with a folded block, with the intention of selecting it and deleting it):

From the beginning of that line, shift + ↓ selects the whole block without expanding :)
From the beginning of the line following the folded text, shift + ↑ selects the whole block without expanding :)
triple clicking line, expecting line selection, expands text :(
selecting the entire line above the folded code, expands it :( :question:
⌘ + l (select line) expands fold :(
selection with the cursor is just weird....can only put cursor at beginning of line. If I drag to the right (to select the line) nothing happens. I can only select the line be clicking and dragging down to the next line.

why is it still exist? that's really inconvenient..

It is really a bad UX that the folded line can not be selected on its own, and it only works when being selected together with adjacent lines.

I stumbled onto this while searching for solutions to a similar problem. Has anyone else noticed that if you select all, any folded sections are automatically unfolded? Could be one of my packages, or a setting, I haven't investigated too closely yet.


Does latest version comes with rewrite_tag_filer plugin

I am new to this plugin and trying to work around a problem. I am using v1.2.0 to route some logs to AWS S3 bucket. I read up on that and thought of using the rewrite_tag_filter. However I get the error
error_class=Fluent::ConfigError error="Unknown output plugin 'rewrite_tag_filter'. Run 'gem search -rd fluent-plugin' to find plugins"

Guess my question is does this plugin come with the out_rewrite_tag_filter?

it does not, only fluent-plugin-s3.  The example configuration on our README should not require the rewrite tag in its current state.  Can you share some more about your use case for rewrite_tag_filter?

Well my use case is -
I wanted to tag logs matching a particular "message" and then use that tag to copy the logs to s3 bucket. Is there a better way to do this without using the rewrite_tag_filter.
As mentioned I am new to the plugin, so any insights are appreciated.

Ok, I see, so yeah to do store just a subset, you would need the rewrite tag filter.  I am planning a release for later this week to address #79 and I can add that plugin to the docker file then.  That use case is common enough I think it is good to add to make available to the Docker file.

Awsome!! That sounds great!! That works for me..

This plugin is now part of the image in v2.0.0


CA mode doesn't exist

It says that we are able to choose consistency and availability from the CAP theorem [1].
Looks like it's not true according to the following answer.
https://www.quora.com/Whats-the-difference-between-CA-and-CP-systems-in-the-context-of-CAP-Consistency-Availability-and-Partition-Tolerance
"The CAP theorem is woefully badly stated. It's not usefully about a straight choice between C, A and P - it is actually about what you give up if you have a partition. "
[1] https://www.rabbitmq.com/distributed.html

It largely depends on how you define each. In the opinion of our current team RabbitMQ queue mirroring is CP. We will look into editing that guide (which hasn't really changed in some 5 years).

Since one side of the partition must act as if it is unavailable, I think queue mirroring is CP too.


Team Meeting 3/30/24

Agenda

Youtube Video
Documents for Revision 1

Fix Bill Management Unit Tests: @HamidHarris  @FardeenAfsar
YouTube Video
Intro: @HamidHarris
Demo: @FardeenAfsar
Editing: @DangJustin
Try to get video parts done by tomorrow
Revision 1 Documentation
Documentation Revisions Due Thurs April 4th (Team 6 issues and TA feedback on Avenue, put fixes in issues):

SRS: @HamidHarris
Hazard Analysis: @FardeenAfsar
VnV Plan: @DangJustin
Detailed Design and Software Architecture: @Lazarus-XD
VnV Report: @fadymorcos21
Problem Statement and Development Plan @DangJustin
Reflection: @DangJustin @Lazarus-XD @FardeenAfsar @fadymorcos21 @HamidHarris
Final Team Contribution Report

Next Meeting: Tuesday April 2nd


Shape errors when calling a converted tf.Module

Here's a Colab example where I copied the TF Module sample code from https://www.tensorflow.org/api_docs/python/tf/Module
https://colab.research.google.com/gist/romanngg/4157eb94d0e1a47f8d300738cbfe9270/tf2jax_module_fail.ipynb
Calling the module works, as well as the associated tf.function, but after conversion the JAX there is a dimension mismatch. I didn't have any of such issues with tf.keras.Models. Thanks again for the great library!

Thanks for the bug report!
This is was very valuable and should be fixed #40


macos make error

./autogen.sh  && make && make install
*** No targets specified and no makefile found.  Stop.
how issues?

Use CMake instead:
    mkdir build && cd build && cmake .. && make && make install

cmake it's work. but complie occur error:
8 warnings and 1 error generated.
make[2]: *** [build-src/CMakeFiles/sigar.dir/os/darwin/darwin_sigar.c.o] Error 1
make[1]: *** [build-src/CMakeFiles/sigar.dir/all] Error 2
In file included from /Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:33:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/mach/mach_init.h:61:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/mach/mach_types.h:94:
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/mach/thread_info.h:193:26: error: expected ';' at end of declaration list
struct io_stat_entry    disk_reads;
^
/Users/satan/Projects/Test/build/sigar/src/../include/sigar.h:482:24: note: expanded from macro 'disk_reads'
#define disk_reads disk.reads
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:1392:19: warning: duplicate 'const' declaration specifier [-Wduplicate-decl-specifier]
static const char const thread_states[] = {
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3302:42: warning: incompatible pointer types passing 'uint64_t *' (aka 'unsigned long long *') to parameter of type 'rpc_cnt_t *' (aka 'int *') [-Wincompatible-pointer-types]
map_nfs_stats((sigar_nfs_v3_t *)nfs, &stats.rpccnt[0]);
^~~~~~~~~~~~~~~~
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3244:59: note: passing argument to parameter 'rpc' here
static void map_nfs_stats(sigar_nfs_v3_t *nfs, rpc_cnt_t *rpc)
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3328:42: warning: incompatible pointer types passing 'uint64_t *' (aka 'unsigned long long *') to parameter of type 'rpc_cnt_t *' (aka 'int *') [-Wincompatible-pointer-types]
map_nfs_stats((sigar_nfs_v3_t *)nfs, &stats.srvrpccnt[0]);
^~~~~~~~~~~~~~~~~~~
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3244:59: note: passing argument to parameter 'rpc' here
static void map_nfs_stats(sigar_nfs_v3_t nfs, rpc_cnt_t rpc)
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3618:9: warning: 'Gestalt' is deprecated: first deprecated in OS X 10.8 [-Wdeprecated-declarations]
if (Gestalt(gestaltSystemVersion, &version) == noErr) {
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/Gestalt.h:123:1: note: 'Gestalt' has been explicitly marked deprecated here
Gestalt(
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3618:17: warning: 'gestaltSystemVersion' is deprecated: first deprecated in OS X 10.8 - Use NSProcessInfo's operatingSystemVersion property instead. [-Wdeprecated-declarations]
if (Gestalt(gestaltSystemVersion, &version) == noErr) {
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/Gestalt.h:2076:3: note: 'gestaltSystemVersion' has been explicitly marked deprecated here
gestaltSystemVersion          = 'sysv', / system version/
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3620:13: warning: 'Gestalt' is deprecated: first deprecated in OS X 10.8 [-Wdeprecated-declarations]
Gestalt('sys1' /gestaltSystemVersionMajor/, &version_major);
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/Gestalt.h:123:1: note: 'Gestalt' has been explicitly marked deprecated here
Gestalt(
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3621:13: warning: 'Gestalt' is deprecated: first deprecated in OS X 10.8 [-Wdeprecated-declarations]
Gestalt('sys2' /gestaltSystemVersionMinor/, &version_minor);
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/Gestalt.h:123:1: note: 'Gestalt' has been explicitly marked deprecated here
Gestalt(
^
/Users/satan/Projects/Test/build/sigar/src/os/darwin/darwin_sigar.c:3622:13: warning: 'Gestalt' is deprecated: first deprecated in OS X 10.8 [-Wdeprecated-declarations]
Gestalt('sys3' /gestaltSystemVersionBugFix/, &version_fix);
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/Gestalt.h:123:1: note: 'Gestalt' has been explicitly marked deprecated here
Gestalt(
^

You probably changed something in source code because before posting the solution, I checked and it worked ( besides warnings about "Gesalt" interface )

But I haven't modify anything，it's fork from sigar master.

I compiled fresh clone of repository from top of master branch (ad47dc3 Go sources update) on MacOS Sierra 10.12.2 using Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>)

I clone of repository from branch  ad47dc3，run command such as:
mkdir -p build/sigar && cp -rp deps/sigar/* build/sigar/ && cd build/sigar && ./autogen.sh  && cmake -D CMAKE_BUILD_TYPE=Release -D BUILD_SHARED_LIBS=false . && cmake --build . && make && make install &&  cd ../.. && node-gyp configure && node-gyp build
it's still not pass

(node:89934) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
CXX(target) Release/obj.target/sigar/src/node_sigar.o
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:9:
../src/./node_sigar_utils.cc:14:20: error: no member named 'New' in 'v8::String'
return String::New(addr_str);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:86:43: error: no type named 'Arguments' in namespace 'v8'; did you mean 'v8::internal::Arguments'?
inline void CheckArguments(int min, const v8::Arguments& args) {
^~~~~
v8::internal::Arguments
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:147:7: note: 'v8::internal::Arguments' declared here
class Arguments;
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:87:11: error: member access into incomplete type 'const v8::internal::Arguments'
if (args.Length() < min)
^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:147:7: note: forward declaration of 'v8::internal::Arguments'
class Arguments;
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:88:38: error: 'New' is a private member of 'v8::PersistentBasev8::Value'
throw v8::Persistentv8::Value::New(v8::Exception::RangeError(v8::String::New("Not enough arguments.")));
^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:669:23: note: declared private here
V8_INLINE static T* New(Isolate* isolate, T* that);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:88:80: error: no member named 'New' in 'v8::String'
throw v8::Persistentv8::Value::New(v8::Exception::RangeError(v8::String::New("Not enough arguments.")));
^
../src/v8u.hpp:293:34: error: too few arguments to function call, expected 2, have 1
return v8::Integer::New(integer);
~~~~~~~~~~~~~~~~        ^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:2558:3: note: 'New' declared here
static Local New(Isolate* isolate, int32_t value);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:297:46: error: too few arguments to function call, expected 2, have 1
return v8::Integer::NewFromUnsigned(integer);
~~~~~~~~~~~~~~~~~~~~~~~~~~~~        ^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:2559:3: note: 'NewFromUnsigned' declared here
static Local NewFromUnsigned(Isolate* isolate, uint32_t value);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:301:22: error: no member named 'New' in 'v8::String'
return v8::String::New(data, length);
^
../src/v8u.hpp:305:22: error: no member named 'New' in 'v8::String'
return v8::String::New(str.data(), str.length());
^
../src/v8u.hpp:309:22: error: no member named 'NewSymbol' in 'v8::String'
return v8::String::NewSymbol(data, length);
^
../src/v8u.hpp:313:26: error: too few arguments to function call, single argument 'isolate' was not specified
return v8::Object::New();
~~~~~~~~~~~~~~~ ^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:3006:3: note: 'New' declared here
static Local New(Isolate* isolate);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:317:25: error: cannot initialize a parameter of type 'v8::Isolate *' with an lvalue of type 'int'
return v8::Array::New(length);
^~
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:3039:36: note: passing argument to parameter 'isolate' here
static Local New(Isolate* isolate, int length = 0);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:325:1: error: no member named 'New' in 'v8::String'
__V8_ERROR_CTOR()
^
../src/v8u.hpp:322:50: note: expanded from macro '__V8_ERROR_CTOR'
return v8::Exception::ERROR##Error(v8::String::New(msg));
^
../src/v8u.hpp:326:1: error: no member named 'New' in 'v8::String'
__V8_ERROR_CTOR(Range)
^~
../src/v8u.hpp:322:50: note: expanded from macro '__V8_ERROR_CTOR'
return v8::Exception::ERROR##Error(v8::String::New(msg));
^
../src/v8u.hpp:327:1: error: no member named 'New' in 'v8::String'
__V8_ERROR_CTOR(Reference)
^~
../src/v8u.hpp:322:50: note: expanded from macro '__V8_ERROR_CTOR'
return v8::Exception::ERROR##Error(v8::String::New(msg));
^
../src/v8u.hpp:328:1: error: no member named 'New' in 'v8::String'
__V8_ERROR_CTOR(Syntax)
^~~
../src/v8u.hpp:322:50: note: expanded from macro '__V8_ERROR_CTOR'
return v8::Exception::ERROR##Error(v8::String::New(msg));
^
../src/v8u.hpp:329:1: error: no member named 'New' in 'v8::String'
__V8_ERROR_CTOR(Type)
^~~~~~~~~
../src/v8u.hpp:322:50: note: expanded from macro '__V8_ERROR_CTOR'
return v8::Exception::ERROR##Error(v8::String::New(msg));
~~~~~~~~~~~~^
../src/v8u.hpp:332:32: error: too few arguments to function call, expected 2, have 1
return v8::Number::New(number);
~~~~~~~~~~~~~~~       ^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:2545:3: note: 'New' declared here
static Local New(Isolate* isolate, double value);
^
In file included from ../src/node_sigar.cc:2:
In file included from ../src/node_sigar.h:11:
../src/v8u.hpp:336:34: error: too few arguments to function call, expected 2, have 1
return v8::Boolean::New(boolean);
~~~~~~~~~~~~~~~~        ^
/Users/satan/.node-gyp/6.9.1/include/node/v8.h:8000:1: note: 'New' declared here
Local Boolean::New(Isolate* isolate, bool value) {
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make: *** [Release/obj.target/sigar/src/node_sigar.o] Error 1


react/prop-types false positive with local variable props

Apparently a local variable called props will be incorrectly considered to be passed props in a functional component.
function Foo() {
  const props = {}
  props.bar = 'bar'
  return <div {...props} />
}

warning  'bar' is missing in props validation               react/prop-types

Following is fine:
class Foo extends Component {
  render() {
    const props = {}
    props.bar = 'bar'
    return <div {...props} />
  }
}

Worth fixing, but I have no idea why you'd ever want a variable called "props" in a React component :-)

@ljharb FYI
function GenericThankyou({ nextAssignmentId }) {
  const cardProps = {}
  if (nextAssignmentId == null) {
    cardProps.submitText = 'Back to the lobby'
    cardProps.onSubmit = navigateTo(RailsRoutes.lobby_path())
  } else {
    cardProps.submitText = 'Take the next test'
    cardProps.onSubmit = navigateTo(RailsRoutes.do_ordered_test_path(nextAssignmentId))
  }
  return (
    <TestStepCard {...cardProps}>
      <Title>{translate('test.complete.thank_you_1')}</Title>
      <p className='GenericThankyou-text'>
        {translate('test.complete.message_1')}
      </p>
    </TestStepCard>
  )
}

(cardProps seems like the better name, since props looks like it's the GenericThankYou props 😉)

(agreed, but I decided to report it anyway)

Fixed in #1365


Spatial view broken after restarting

Describe the bug
After setting up the view and restarting the application the view is unresponsive and the skybox is black. Have to reset from there.
To Reproduce
Here's the zipped rrd file.
Expected behavior
It should keep the view.
Screenshots
Before:

After restarting (the camera is the small dot in the upper left corner):

Desktop (please complete the following information):

OS: Ubuntu 20.04

Additional context

rerun 0.3.1

This seems to be because the Camera of the spatial view changes from 3D Camera to 2D Pan & Zoom (which I'm not sure what it should do, this might be a separate issue).
Changing it back will restore the view to the one before closing the viewer.

Images
Before restarting

After restarting


Makefile: Add CI for porch controllers

And include it in the github action.

And lol - I forgot the headers.  I guess I should add that to CI too!

Yes, thanks this is not needed!


clang-tidy fixes

I rebased your patch in demo-sprint-wip-clean with some changes. You can see my changes here.
This is my feedback:

The order in which the @params <argument name> appear should be the same as the function signature
Try to make the documentation more descriptive. I would look at Rian's where possible
Line breaks were added / removed by mistakes it seems
Commented code should be removed

On a side note, for your future PRs, make sure to always apply the following build targets: clang-tidy, format and iwyu.
Closing.


How to have multiple rounds of dialogue (keep dialogue context)

This model isn't designed for chat. But you can feed the conversation state back to the model each time to produce the next response. You will eventually hit the context window token limit though.


FDM Api Support

I implemented the FDM api for scala-sdk which is used by transformations. Trying to reuse that experience to add some missing features here
Support Instance filters
Containers

That's a little unfortunate. There is already an open PR for filter types, we'll need to pick one implementation, I guess.

That's a little unfortunate. There is already an open PR for filter types, we'll need to pick one implementation, I guess.

oh. Its ok if this PR is not necessary. This is my first rust PR actually. Still need to add some tests. 😊

I'll merge the other one, since it's a little more polished, but we're happy to accept PRs!

Closing this as we've pretty much implemented everything FDM now.


added a runbook for NodeClockNotSynchronising

added steps to debug and start/restart the chronyd service from this SOP https://github.com/openshift/ops-sop/blob/master/v4/alerts/NodeClockNotSynchronising.md.
related card: https://issues.redhat.com/browse/OSD-8737

@lnguyen1401 is there a patch for CMO as well that you want to link here?

hello @NautiluX , do you mind taking a look at this PR ?

/lgtm

/approved


Internal: inline visitor-keys temporarily

Ref/Fixes #1972

We don't have to do it since we'll be using cst later anyway but if we want to

LGTM

Ok forgot the Closes - Closed with https://github.com/jscs-dev/node-jscs/commit/04b3274b324bbe76902d257bceb66f1ec609548e


Institute a central community repo

Institute a central community repo to serve as a central location for contributors, security researchers, and other interested parties to discover and under the project through GitHub. The Falco project has an excellent example of this.

Acceptance criteria

Create the community repository to be populated in future PR with all the mentioned information in issue description


use cosine similarity in center lose?

in the paper of center lose, author uses L2 norm in the center loss. Euclidean distance doesn't work well in high dimensional space, so how about replace it with the cosine similarity?
I saw some people use cosine similarity to compare the embedding vectors, but I think while the embedding are trained by L2 norm based center lose, we should also change the training as well.
Did anyone try this before? I will try it later.

I'm curious about this as well. Any progress so far?

Hi daeyoun, I wrote an email to the author of paper, don't get answer yet.
When I have time I will try it by myself. If anybody tested before, welcome to post your result here.
What I found out right now is that even the network is trained with L2 norm as written in paper, we can still compare the result embedding vectors with Cosine Similarity, it has similar result as Euclidean distance. So maybe it doesn't worth to change the original center loss, but need more research here.


Disable advanced capabilities

While having a version of the sample which is for IBM MQ, and not IBM MQ Advanced, could be useful, I don't think it should be the only sample.  It is also definitely not recommended to adjust the MQ Advanced for Developers swidtag files, as this free, unwarranted version, is always MQ Advanced.

@arthurbarr Sorry, this MR should be created to my fork only to compare the changes. It makes no sense to merge it to master in the current state.
I adjusted the swidtag files only  to avoid bug #437. We want an IBM MQ container, non-advanced, because we only have a MQ license. To be sure not to violate our license, we want to have the property ADVCAP to be DISABLED. According to the documentation it should not be necessary to modify swidtag, but it is unfortunately.
When you say a non-advanced sample could be useful and there is interest in supporting it, I could provide an MR with an option to build without advanced features, for example with a separate build call make build-devserver-non-advanced.


feat: tailwindcss v4

Continuation of #18
Add support for tailwindcss v4.

Status update
cc: @zcf0508
There where (and still are) some issues making this plugin compatible with tailwindcss v4.

Tailwindcss v4 is completely different from an API perspective, so I needed a way to conditionally import the different APIs from tailwindcss.
ESLint is not capable of asynchronous rules, so I cannot use await import() to dynamically import the respective functions. Since this plugin is written as ESM, require() is also not available for synchronous imports. I have experimented with numerous variants to work around this limitation, two of them I considered to be viable:

Create different entrypoints for each tailwindcss version, load the correct APIs and use dependency injection to pass them to the rules. This would eventually end up in a breaking change and I didn't like the resulting API.
Change some specific files to CJS and use require() to load the APIs. Since ESM is capable of importing CJS, the API can be kept as is.

I decided to choose the second option but had no luck getting it to work in a way that those files would be kept as CJS in the transpiled output using the current build system with vite.
Therefore I decided to get rid of vite and use tsc for transpilation and the native node:test runner for the tests instead.

Tailwindcss v4 does not handle imports in the new css configs internally. They propose to use postcss-import to load, parse and resolve the imports in the css config but again postcss-import is asynchronous.
I ended up using lightningcss as I have read in their announcement blog post that their goal is to eventually switch to lightningcss as well.

TODO

[ ] Figure out how to @import "tailwindcss" in the css config since tailwindcss obviously does not exist in the users project directory.

Should be possible with lightningcss: https://github.com/parcel-bundler/lightningcss/pull/94

I was inspired to create the feature from https://github.com/hyoban/tailwindcss-classname-highlight/blob/main/src/index.ts#L78, and wish this is helpful for you to resolve the problem.
For now, I think we can publish an alpha version for v4 users, as tailwind v4 has not been release. Or we can release two versions that one for v3 and one for v4 to prevent code coupling and enhance clarity.

My current branch doesn't yet work with tailwindcss v4 because I couldn't yet figure out how to resolve the @import statements in the css files.
I think the approach taken in https://github.com/hyoban/tailwindcss-classname-highlight is pretty hacky as it just reads all css files and creates the tailwind context from them. There is no resolving of import statements and the tailwind internals and it also includes css files that may not be intended for tailwindcss.
The same problem exists in the branch you created for #18.
I got pretty deep into a rabbit hole trying to resolve this problem and I have posted my findings in my comment above.
In the anouncement blog post, they wrote about integrating lighntingcss and automatically handle those problems.
In their codebase as it is right now, they have not done this integration yet and they still rely on postcss, which is not an option for eslint plugins, because postcss-import is async.
The plan is now to wait for tailwindcss v4 to be more mature and let them figure out the solution to this problem. Any additional effort I put into this right now will eventually be wasted time in the long run and this is not sustainable for me right now.
What I can do prior to this, is to make the plugin work with v4 in a way that it no longer crashes.
The sort-classes rule is the only rule that depends on tailwindcss internals. I can refactor the plugin so that the tailwind internals are not loaded until they are used.
If you disable the sort-classes rule, or change the order to asc for example, there would be no need to load the tailwind internals and the plugin would therefore not crash.
I have done a lot of preparation work in #26 for this.

I created a feature request in lightningcss for synchronous custom resolvers: https://github.com/parcel-bundler/lightningcss/issues/747


Added Responsive UI

Fixes Issue

fixes #18

Proposed Changes

Checklist (Check all applicable boxes)

[X] My code adheres to the project's coding style.
[X] This PR does not contain any plagiarized content.
[X] The title of my pull request provides a brief description of the requested changes.

[X] I am contributing for Hacktoberfest 2023.

Potential Impact on Existing Code

Screenshots

Reviewer Notes

Thankyou for your contribution @du1ana  !
Looking forward to more issues and collaborations.
If you learnt something from this repo make sure to ⭐ this repo.
Thankyou ,Happy coding !🚀


released version of tasmota32c3 crashes

Today I installed
https://github.com/tasmota/install/tree/firmware/firmware/unofficial/tasmota32c3_2M.bin  v.14.2.4
which works well.
The "official version"
https://github.com/tasmota/install/tree/firmware/firmware/release/tasmota32c3.bin
crashes already after 4sec.
As the unofficial version seems to work well, this issue is probably solved.

Sorry, I checked again. Error description is different. If I run the installer
https://tasmota.github.io/install/
with any online Tasmota version, then the FW runs smoothly.
If I download the same file to local and try to upload this local file using the same tool, then it crashes after 4 sec.
So it rather seems to be a bug with the online installer.

The online installer only supports factory images


[Release-1.29] - [Bug Report K3s Windows] - Command Name for the k3s binary is incomplete in binary_size_check.sh

Backport fix for [Bug Report K3s Windows] - Command Name for the k3s binary is incomplete in binary_size_check.sh

#9991

I'm going to close these backports as Windows is not supported in k3s, so as to save some testing cycles. We will validate the changes in the master branch only for a sanity check.


Support for conditional directives. v-if, v-else, v-else-if

Would like to thank your efforts for creating this package. It has made our project and vue components quite typesafe and we are very happy with it. The only thing missing for me is conditional directives. I know we can use ternary statements in the render() function but to me the conditional directives are great for reducing code clutter.
I have worked a bunch in react land and am using this babel plugin: https://github.com/peakchen90/babel-plugin-react-directives
I am wondering if this plugin could perhaps be integrated into the vue-tsx-support library or something similar.
Would be happy to give it a try myself and make a PR if I knew where to start.

+1 !!

It is not scope of vue-tsx-support.
I guess https://github.com/vuejs/jsx is the right place of this issue.


Discussion: Potential split-brain in meta nodes

Describe the bug
Running into split-brain, when executing the e2e tests. All 3 meta nodes are sometimes elected leader.
I will run this multiple times and see which logs I can get. Attaching the first run here.
run1-e2e-meta-0.log
run1-e2e-meta-1.log
run1-e2e-meta-2.log
run2-e2e-meta-0.log
run2-e2e-meta-1.log
run2-e2e-meta-2.log
run3-e2e-meta-0.log
run3-e2e-meta-1.log
run3-e2e-meta-2.log
To Reproduce
Checkout use_meta_service https://github.com/risingwavelabs/risingwave-operator/pull/344
run ./e2e.sh -p risingwave::multi_meta_failover
If you want the meta nodes to stay on not be cleaned-up after the failed test, then replace return 1 with exit 1 in k8s::pod::delete_meat_leader_pod and k8s::pod::meta_pod_valid_setup. Current commit is e1f7c336c93da9542383619e1f1e3829167c55be
Expected behavior
1 elected leader only
Additional context
Using the latest operator for run1 and run2
Using dev operator build from PR in run3

Problem arises, because the lease uses the advertised address as a key. In K8s each pod uses the address <IP_ADDRESS>:5690. So, each mea node believes that is is a leader. My proposed solution: change the value in the lease to include the POD_NAME from the environment.


Profile log may include invalid byte sequences

Repro steps
prof.vim
https://gist.github.com/ichizok/74fc6a2a9d0aa4673d3ea7fabc22908f
vim --clean --cmd 'set enc=utf-8'
:profile start prof.log
:profile file prof.vim
:so prof.vim
:qall!

prof.log will include UTF-8 invalid byte sequences.
Impact
For external application using profile log, this problem can cause errors related to encoding.
e.g. covimerage (python application), UnicodeDecodeError occurs when reading profile log
Cause
https://github.com/vim/vim/blob/17471e8/src/ex_cmds2.c#L1835
Each lines of script source file are truncated to IOSIZE - 1 (==1024) byte ignoring the boundary of multibyte characters.

Codecov Report

Merging #2385 into master will decrease coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2385      +/-   ##
==========================================
- Coverage   74.56%    74.5%   -0.06%
==========================================
  Files          91       91
  Lines      132421   132430       +9
  Branches    29084    29086       +2
==========================================
- Hits        98734    98671      -63
- Misses      33683    33734      +51
- Partials        4       25      +21

Impacted Files
Coverage Δ

src/ex_cmds2.c
80.23% <100%> (+0.07%)
:arrow_up:

src/gui_beval.c
55.79% <0%> (-6.44%)
:arrow_down:

src/libvterm/src/state.c
40.38% <0%> (-1.02%)
:arrow_down:

src/window.c
81.25% <0%> (-0.56%)
:arrow_down:

src/libvterm/src/screen.c
72.8% <0%> (-0.54%)
:arrow_down:

src/message.c
68.47% <0%> (-0.25%)
:arrow_down:

src/term.c
50.56% <0%> (-0.22%)
:arrow_down:

src/gui_gtk_x11.c
47.65% <0%> (-0.15%)
:arrow_down:

src/channel.c
82.48% <0%> (+0.04%)
:arrow_up:

src/terminal.c
64.99% <0%> (+0.06%)
:arrow_up:

... and 3 more

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 8b9e20a...63356cd. Read the comment docs.


Dev test

https://github.com/holidayextras/culture/blob/master/recruitment/developer-API-task.md
I have redone the solution to the problem above as in my mind the first solution i created was a bit of a mess and after attempting to extend the functionality further by adding in a second resource i decided to re-write this from the ground up.
i have made better use of express's in build routing functionality, replaced the ORM i was using for sequalise, and breaking out the functionality of the API into more separate testable chunks

As I said in person may be worth trying to investigate getting linting checking in there.
For a future iteration maybe look at getting some unit tests in?

@philmeehan @kevinhodges @t1gr0u @notABluesSinger, i have taken all of your feedback on this and come up with what i think is a better solution, can you all take a look in you own time and let me know what you think, thanks

maybe add in travis/circle ci, update http status codes

Changes Unknown when pulling 96f5839acab5478cd5154485e007cd206e926f54 on Dev_test into ** on master**.

Changes Unknown when pulling bc39571cf7c07dae0f90b7bd9271c2c0f836e441 on Dev_test into ** on master**.


ssdlite320_mobilenet_v3_large only have ~ 50% CUDA usage even with large batch size

🐛 Describe the bug
import torch
from torchvision.models.detection import ssdlite320_mobilenet_v3_large

with torch.inference_mode():
    model = ssdlite320_mobilenet_v3_large(True)
    model = model.eval()
    model = model.to('cuda')
    inputs = (torch.randint(0, 255, (256, 3, 320, 320)) / 255).to('cuda')
    for _ in range(64):
        outputs = model(inputs)

nvidia-smi

The CUDA usage is around 36% during inferencing, one of the CPU usage is 100%
Versions
Collecting environment information...
PyTorch version: 1.10.0+cu113
Is debug build: False
CUDA used to build PyTorch: 11.3
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.3 LTS (x86_64)
GCC version: (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
Clang version: Could not collect
CMake version: version 3.21.3
Libc version: glibc-2.31

Python version: 3.8.10 (default, Sep 28 2021, 16:10:42)  [GCC 9.3.0] (64-bit runtime)
Python platform: Linux-5.4.0-89-generic-x86_64-with-glibc2.29
Is CUDA available: True
CUDA runtime version: 10.1.243
GPU models and configuration:
GPU 0: NVIDIA GeForce GTX 1080 Ti
GPU 1: NVIDIA GeForce GTX 1080 Ti

Nvidia driver version: 495.29.05
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.3.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.3.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.3.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.3.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.3.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.3.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.3.0
HIP runtime version: N/A
MIOpen runtime version: N/A

Versions of relevant libraries:
[pip3] numpy==1.21.3
[pip3] torch==1.10.0+cu113
[pip3] torchvision==0.11.1+cu113
[conda] Could not collect

cc @datumbox

Thanks, this would require additional investigation from our side. This is not the first time that the community brings this to our attention. In the past SSDlite was reported to have the same more-or-less FPS as SSD. So definitely worth exploring on the future.
Just a few comments concerning the benchmark:

FYI, the SSD like other detectors is supposed to receive a list of tensors not a batch. This is because images can be of variable size.
Have you tried using the PyTorch profiler to investigate further?

@datumbox
Here are 4 profile runs with two batch sizes and 2 devices.
Some initial observation:
The first part, with the largest Wall time being aten:conv2d, have its inference time smaller with cuda, and not scale with batch size.
The second part, with the largest Wall time being aten:index and torchvision:nms, have its inference time not reduced with cuda, and is linearly scale with batch size.
My guess is the convolutions have become insignificant enough that post-processing / NMS has become the bottleneck.
import torch
from torchvision.models.detection import ssdlite320_mobilenet_v3_large

for batch_size in (1, 8):
    for device in ('cuda', 'cpu'):

        model = ssdlite320_mobilenet_v3_large(True)
        model = model.eval()
        model = model.to(device)
        inputs = (torch.randint(0, 255, (batch_size, 3, 320, 320)) / 255).to(device)
        inputs = [
            (torch.randint(0, 255, (3, 320, 320)) / 255).to(device)
            for _ in range(batch_size)
        ]
        with torch.profiler.profile(
                activities=[
                    torch.profiler.ProfilerActivity.CPU,
                    torch.profiler.ProfilerActivity.CUDA,
                ],
                schedule=torch.profiler.schedule(wait=1, warmup=1, active=2, repeat=1),
                on_trace_ready=torch.profiler.tensorboard_trace_handler(
                    'ssdlite320_mobilenet_v3_large', f'batch_size_{batch_size}_device_{device}'),
                with_stack=True,
        ) as p:
            with torch.inference_mode():
                for _ in range(4):
                    outputs = model(inputs)

                    p.step()

benchmark_tensorboard.zip


L2CAP COC: nimble@mynewt crashes when flooding a channel

Context
I am still trying to understand the behavior described in #818. To make sure this is not a RIOT or a porting issue, I ported the benchmark application to mynewt (-> https://github.com/haukepetersen/riotsandbox/tree/master/tests/mynewt_nimble_l2cap_coc_flooding)
This is the first time I actually did something with mynewt and I am still learning - so please excuse the project structure and code maturity :-)
Problem
When flooding an L2CAP COC link between two nodes for a long enough amount of time (~some seconds), the transmitting node will crash (reboot).
By e.g. trying to send 10k chunks of 100byte (flood 100 10000), the source node will reboot after a random time, mostly something around ~15-20 seconds.
Setup
Two nrf52dk nodes on my desk, one flashed with the source application (target source), one with the sink (target sink) application.
The nodes open a BLE connection automatically, and once connected also open a custom L2CAP connection oriented channel.
Then, once connected, simply call the flood shell command on the source node.
Is anyone able to reproduce this behavior? And does anyone have an idea where to look further? I am a little stuck here...

Sorry for late reply, I had no time to look into it till now.
I could reproduce it and this is watchdog  restarting source. Solution for your issue is to use os_sem instead of os_mutex. If I udenrstand corretly  you want to wait here: https://github.com/haukepetersen/riotsandbox/blob/master/tests/mynewt_nimble_l2cap_coc_flooding/apps/source/src/main.c#L64  untli UNSTALLED will be received.
It works for me when I changed that.

No worries, same sorry here :)

If I understand corretly you want to wait here

Yes, that is correct. The aim is to allow other, possibly lower prio tasks to work while the sending task is waiting for L2CAP to sent the previous packet. Using a semaphore/mutes here seemed the obvious choice, right? Or do you have an idea for something better suited for synchronizing the sender task?
Will also try to run my code using a semaphore and report back with my results.

Confirmed: using os_sem solves this issue. Interesting system design to have the watchdog triggering for waiting mutexes but not for semaphores :-) Maybe that could be added to the doc pages?!
Anyhow, everything is in order now, so I consider this issue solved.

Confirmed: using os_sem solves this issue. Interesting system design to have the watchdog triggering for waiting mutexes but not for semaphores :-) Maybe that could be added to the doc pages?!

that's not the problem with mutex itself. sample code acquires mutex in on task (flooder) and releases it in another (main). this means an attempt to release mutex always returns an error since task which tries to release mutex does not own it, while an attempt to acquire mutex will not block since mutexes are recursive so the same task can acquire a mutex more than once. so effectively sample code spins in _send which triggers watchdog.


exit code of stp validate is always 0

I think stp should exit with code != 0 if validation fails. And output should possibly go to stderr.

Good idea but really you don't use the count. Maybe change to something like this:
got_errors = false

  got_errors = true

process.exit Number got_errors

I thought the count could be useful someday, but yeah, until it's not used... updated

not really sure about error output though

Applied. Pushed. Thanks!

my first coffeescript code \o/


Db File size - increases but doesn't decrease

I have noted that the Db file size goes up but not down. For example - I uploaded 45mb worth a files, then used  db.FileStorage.Delete(ID); to delete it all from the Db.
The Db file size went up to 45mb on uploading the files, but didn't reduce when they had been deleted.
Am I missing a step?

Disregard - I discovered shrink();


Using with remote files

Hi,
thank you so much for putting this together and sharing.
If I start emacs and open a remote file via Tramp, copilot.el complaints it does not find node. I assume it is because it's looking for the node executable on the remote machine (via locate-file).
Would setting default-directory to ~  within copilot--start-agent cause any problems, or do you think that'd be an appropriate solution for this issue?
Thank you

I tried to fix it by changing shell-command-to-string to call-process. Hope it works for you.

Thank you for the quick reply.
Seems fixed!


Crash : Audience filter count out of range

Environment

Xcode version: 9.4.1
Firebase SDK version: 5.2.0

Describe the problem
Every time I open the app go the same issue/crash
5.2.0 - [Firebase/Analytics][I-ACS006009] Audience definition is invalid. Audience will not be saved. Audience, error: <FIRAPBAudience: 0x604000203860>, Error Domain=com.google.gmp.measurement.ErrorDomain Code=9 "Audience filter count out of range"

Please ignore this. This is a known bug from the BE, not the SDK but it won't interfere with the normal operation of the SDK. Sorry for the confusion :(

But I am getting this more often and this is not just error which I can ignore as that is producing a crash also.

Hmm, I haven't seen the crash related to this message. Do you have any crash log related to this? Can you please provide some sample project so we can reproduce the bug? Thank you.

What i am gettign is -
Domain=FABNetworkError Code=-6 "(null)" UserInfo={retryLimit=1, NSErrorFailingURLStringKey=https://settings.crashlytics.com/spi/v2/platforms/ios/apps/com.t9l.hospalsapp/settings?build_version=1&display_version=1.0&instance=62d1e8f04ff57b97339b93f996369f0847e308e5&source=1}
Screeshot:- for more detail

Can you provide the stack trace here? The screenshot is cut off so I can't tell which SDK (Analytics, or other Firebase SDK) is causing the crash. I don't think the Audience warning is related to the Network error since you should have successful network connection to receive the Audience. Therefore, it is unlikely related but I'd like to know the real cause of the crash to fix. Thank you!

I am not getting this very often and not always on debug mode so give the screenshot like above again is little bit difficult for me. But once I got it the same,  will update it here again. Till there is something very clear that it is in Firebase core framework, FIRMutableDictionary class and line number also.

Find a more detailed screenshot which i got on debugging
It is trying to removing an object with key "com.firebase.network.background-upload-B6A85FD7-580E-4B37-AF64-1088A6B242C4" from a blank dictionary

Thanks for providing the screenshot. I think this is related to this issue https://github.com/firebase/firebase-ios-sdk/issues/1317#issuecomment-397908981, and not the error message in the device log. Since the message is logged before Analytics uploads some events, that leads to the network crash, it makes them related.
@paulb777, @ryanwilson : Can we revisit this again or provide the code snippet of the potential fix in this open sourced code to @GauravT9l so they can verify whether it helps alleviate the issue? Thanks

The fix for #1317 was shipped with Firebase 5.5.0 last week, so it's worth updating and seeing if you can reproduce.
The code snippet on the open source side never shipped as we determined it was attempting to fix an impossible state.  #1426 for reference.

Update all the pod yesterday and firebase too with current version Firebase 5.5.0.
Now getting error in Google Utility pod GULMutableDictionary.m . Not sure this is using by firebase or not. but as i got same error so posting here, may b this could help to figure out something.

Yes, it has been renamed and moved to GoogleUtilities pod so it is still the same issue.
@paulb777 : I still think the unreleased fix could fix this. Can we please provide it to @GauravT9l to try since they can reproduce the issue? It is worth trying

@baolocdo Do you mean #1426?
Based on this crash log, it doesn't seem that weakify'ing self would help since the crash looks to be the result of a deallocated (and zombified object) stored in the NSMutableDictionary.

Yes, since it was deallocated, it shouldn't be referenced anymore in that async block. The async block can be scheduled and run after the object self has been already deallocated. We can confirm whether the fix can help in this case by providing to @GauravT9l instead of saying it doesn't help. Since the new release didn't fix the crash, we can try another potential fix. Is there any reason why we are opposing this?

Pasting this from the email thread earlier last week:
I don't think self is the value that's being zombified here. The call to __dealloc_zombie happens after removeObjectForKey: is invoked, so the refcount being decremented belongs to whatever object is being removed from the dictionary. A few things to note:

This isn't a proper zombies exception, just a crash that happened when zombies were enabled. Notably, __dealloc_zombie was called without raising an exception, so at the point where the object is deallocated it's being deallocated properly (it's not already a zombie).
The program then crashes in the C method used by zombies to turn deallocated instances of objects into zombies instead of actually deallocating them.

I don't have a good conclusion for what's going on here, but I don't think using weak self will fix it. If we could get the description of the object being deallocated (the object released by removeObjectForKey:) it'd be a lot more clear what's happening.

Would it be reasonable to switch GULMutableDictionary's internal type from NSMutableDictionary to a Strong-to-Weak NSMapTable?
How should the lifetime of the GULNetworkURLSessions that get stored as values in the GULMutableDictionary be managed?

No, please do not change it NSMapTable as this is supposed to be an NSMutableDictionary. Otherwise, we can just use NSMutableDictionary in the GULNetworkURLSession and it might cause other crashes due to thread unsafety. Again, I'm don't fully agree with the email thread and I'd like to give it a try to verify the hypothesis. If it doesn't work, we can find another solution instead of keep saying it is not happening.

@GauravT9l Are you able to share a reproducible example?
Otherwise you could try out some of the changes in #1426 or other ideas discussed in this thread.

My guess is the object in the dictionary is getting clobbered elsewhere and it makes less sense to be extra defensive here than it does to figure out where the object is being mangled and un-mangle it there.
@GauravT9l if you're able to consistently reproduce the crash, can you share your project?

Closing for now; we'll reopen this when we have more information.

I can reproduce this consistently when enabling zombie objects AND malloc scribble. It seems the object in the dictionary is sent a message after it is removed.

@maranas What are your Firebase pod versions?

@paulb777 Just updated to Firebase/Core 5.10.0. I see the same stack trace posted by @GauravT9l (crash after removing a GULNetworkURLSession instance in the dictionary) but now it happens in GULMutableDictionary (in GoogleUtilities/Network v5.3.2).
When it crashes, the "zombie object" is always the URL session with the kGULNetworkBackgroundSessionConfigIDPrefix prefix. Interesting is that the crash happens after removing the object from the dictionary, and only happens if both zombie objects and malloc scribble are enabled.

The file here in the screenshot is GULMutableDictionary.m

@paulb777 One big thing i forgot in the previous message - the issue happens only on iOS 9. Or at least i can only reproduce it there.

@maranas I'm not able to reproduce in a sample app on iOS 9 with zombie objects and malloc scribble  enabled.
Are you able to share a reproducible example - or reproduce in one of the quickstarts?

@morganchen12 : I can still see the issue appearing a lot for us using 5.16.0
Interestingly some of us use AppCode where the issue seems to be absent entirely.

For anyone still struggling with this crash.
What worked for our project is disabling the NSZombieEnabled environment variable.
Just remember that this results in a loss of proper Memory Debugging so in case you need to do that, don't forget to re-enable it.

This does look a little different than the original issue (the crash is caused by a different selector).
I haven't had time to take a look at this yet, but I suspect looking at the ownership of GULNetworkURLSession would be a good place to start.

Same crash here...
i've the last version of the firebase ios sdk installed

Same, pods has been updated. I'm experiencing this crash intermittently as well

@pmusolino @amaldona are you using any post-processing steps on the .ipa? I've seen something similar before when a developer was using Citrix to upload their .ipa for obfuscation. Thought it would be worth a shot to see if something similar is being done on your project's artifacts.

Closing due to inactivity and heavy fragmentation. If you're running into an Analytics crash, please open a new issue.


Allow creation of clusters with less than 3 nodes

https://github.com/openshift/ocs-operator/blob/9a55e5215b6eca397960c180d0cb32c1983c6285/pkg/controller/storagecluster/reconcile.go#L337-L342
Due to the < comparison with defaults.DeviceSetReplica (which is 3), it is not possible to deploy OCS on an environment with less than three nodes.
Please remove this hard limitation in order to allow demos in smaller deployments. I need this for Katacoda

I honestly don't know how OCS will behave with less than 3 nodes.

It definitely could be done, but this is not something we'll be getting around to any time soon. Certainly not before OCS 4.2 GA and probably not for 4.3.

@travisn Could you please point me to the code that I would need to change to create just a single MON and set pools to replica 1?
I already started a fork where I set the default DeviceSetReplica to 1, so I could host the changes over there until you're ready to merge. That way I can use my fork in Labs like Katacoda

Closing since I have had my fork for a while now already - it works perfectly (and is now ~1 year old code)


Re-implementation err on ffs experiment

Good job, but I have some questions on ffs ckpt inference experiment.
1）I set "ckpt" in ffs.sh to the folder related to https://huggingface.co/maxin-cn/Latte/blob/main/ffs.pt", set "pretrained_model_path" to the folder related to https://huggingface.co/maxin-cn/Latte/tree/main/vae.
But the performance of video generation is bad. Is there anything wrong with my process?
https://github.com/Vchitect/Latte/assets/161445238/1fcc2c49-402e-4977-9684-6547ac0e17bc
2）Besides, I edit the code in sample.py. If I keep the code "samples = vae.decode(samples / 0.18215).sample", I will get "Segmentation fault". Therefore, I replace the code with the following. Is there anything wrong with my process?

Good job, but I have some questions on ffs ckpt inference experiment. 1）I set "ckpt" in ffs.sh to the folder related to https://huggingface.co/maxin-cn/Latte/blob/main/ffs.pt", set "pretrained_model_path" to the folder related to https://huggingface.co/maxin-cn/Latte/tree/main/vae. But the performance of video generation is bad. Is there anything wrong with my process?
sample.mp4
2）Besides, I edit the code in sample.py. If I keep the code "samples = vae.decode(samples / 0.18215).sample", I will get "Segmentation fault". Therefore, I replace the code with the following. Is there anything wrong with my process?

I just tested my code and didn't find anything unusual. For the second question, perhaps you could reinstall your environment? Thanks for your interest.


how to download jar file from this github?

how to download jar file from this github?

release packages are available in maven central or in the github release assets
Latest IS here : https://github.com/cnieg/keycloak-login-attribute/releases/download/v1.3.6/attribute-login-provider-1.3.6.jar


Location Provider Error

Describe the bug
An error that appears to be related to the router functionality appears in the web console. The functionality of the playground is not noticeably effected.
To Reproduce
From 'master' branch, load the playground and open the console. to find the error.
Expected behavior
No error would appear.
Screenshots

Desktop (please complete the following information):

OS: Mac OSX big sur
Browser: Chrome
Version 94

Additional context
I brought this and other console errors to the attention of @psiemens who indicated they could all be consolidated into a tier3 FLIP bounty.

Fixed


voluptuous.error.MultipleInvalid: expected a list for dictionary value @ data['accounts']

The installation via HACS ran without errors. The o365 configuration I have swapped into a 0365.yaml and included it per include in the configuration.yaml. I use only one personal account. Unfortunately I get the following error:
Error during setup of component o365
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 288, in _async_setup_component
    result = await task
             ^^^^^^^^^^
  File "/config/custom_components/o365/__init__.py", line 62, in async_setup
    accounts = MULTI_ACCOUNT_SCHEMA(conf)[CONF_ACCOUNTS]
               ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/voluptuous/schema_builder.py", line 272, in __call__
    return self._compiled([], data)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/voluptuous/schema_builder.py", line 595, in validate_dict
    return base_validate(path, iteritems(data), out)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/voluptuous/schema_builder.py", line 433, in validate_mapping
    raise er.MultipleInvalid(errors)
voluptuous.error.MultipleInvalid: expected a list for dictionary value @ data['accounts']

My config looks like this:
accounts:
    account_name: Bastian # Do not use email address or spaces
    client_id: "xxxxxxx"
    client_secret: "xxxxxx"
    alt_auth_method: False
    enable_update: False #keine Schreibrechte im Kalender oder bei E-Mail
    track_new_calendar: True
#    email_sensor:
#        - name: inbox
#          max_items: 2
#          is_unread: True
#          download_attachments: False
#    query_sensors:
#        - name: "Example"
#          folder: "Inbox/Test_Inbox" #Default is Inbox
#          from<EMAIL_ADDRESS>#          subject_contains: "Example subject"
#          has_attachment: True
#          max_items: 2
#          is_unread: True
    todo_sensors:
        enabled: False

Plus carefully look at the structure here https://rogerselwyn.github.io/O365-HomeAssistant/installation_and_configuration.html
Formatting in yaml is important, there should be a ‘-‘ before account_name, and all below indented.
I’m on holiday, so not looked carefully, but I can see that is wrong.

Thank you so much! It was the missing "-". Have a nice holiday!


S3 serialization

What would it take to turn this into a Lambda layer with serialization to S3 so you could use it in production and avoid DynamoDB altogether?

If you used EFS (instead of S3), you wouldn't need to modify anything at all.
I still wouldn't recommend using it in production though – it hasn't really been stress tested in the way you'd want to for a production DB.

I also would like to do this, and EFS is fine with me. But I can't call Dynalite in Lambda as it is implemented as a HTTP server, to emulate DynamoDB. I want to skip the whole aws-client and use Dynalite as embedded DB, and experiment with other LevelUP backends.
@mhart can you suggest the best way to cut through and call Dynalite as API?

@urbien can you explain what problems you have calling this in Lambda? It should work totally fine

@mhart thanks for the quick reply! Dynalite is awesome and it is clear you put immense amount of work into it.
In Lambda I certainly do not need an HTTP server, and I do not even know if can have HTTP client talk to HTTP server inside the same nodejs process of Lambda function. May be it is possible, but certainly very inefficient.

You can certainly do that – explain the inefficiency to me?

Inefficiency is in serializing the request to a string to be sent over the wire, making HTTP request, receiving it and deserializing on the other end. More in AWS client which is adding a signature to the request. All of that is not needed if calling the API directly.
We would still need to convert a JavaScript object to its equivalent DynamoDB AttributeValue type with https://github.com/aws/aws-sdk-js/blob/master/lib/dynamodb/converter.js#L7

Any time serializing is insignificant compare with the time it'll take you to perform I/O to your LevelUp backend – especially if it's EFS – you're talking dozens, if not hundreds, of milliseconds for each EFS interaction – compared with nanoseconds for string manipulation in JS.

agree, but although I will test with EFS, my target environment is something very different. We are building an open source Personal Cloud, which will have Lambda-like execution environment and single-tenant personal data management with a personal file system, personal database, personal reliable messaging, etc. If interested, I would be thrilled to share the details with you.

Ok, no probs. I'm still of the opinion that any time doing external I/O will be much slower than any local HTTP interactions – unless you're talking to an in-memory or local store, in which case, it wouldn't really make sense on Lambda (or a Lambda-like environment) because the data wouldn't be shared. You'd be much better off hosting the DB separately.
In any case, going to close this out – it's not a scenario that I think dynalite should support. Nor do I have any plans to support an API here beyond the DynamoDB API contract.
If you'd like to fork it and create your own API, the http server is really just a light wrapper around the DB store. You can see the store created here:
https://github.com/mhart/dynalite/blob/58d0955a5441ed294a92ce7bb109d7b2d88da1a1/index.js#L24
And how actions are called on it here:
https://github.com/mhart/dynalite/blob/58d0955a5441ed294a92ce7bb109d7b2d88da1a1/index.js#L261


Service unavailable

Hi,
I followed the deployment steps at https://www.gateway-api-controller.eks.aws.dev/deploy/
Next, I tried to set up the single cluster/VPC service-to-service communications as indicated here: https://www.gateway-api-controller.eks.aws.dev/getstarted/
My problem appears in step "check service connectivity". When I exec into the pods and try to call the service using the curl command, e.g. curl -v inventory-aws-application-ne-03449e711929f70b4.7d67968.vpc-lattice-svcs.eu-west-1.on.aws, it always responds with 503 service not available.
Any ideas how to debug what is going wrong?

You may want to double check if security group is configured correctly.  Can you check the flowlog and see if anything is dropped due to security group?

Recreated the cluster from scratch and got it working. Thanks for help!


BUG: The last few days level 100 fate Mica the Magical Mu stage 3 safe zone prompted an error

Six cards dealt, one safe zone for that phase, prompted several times in error

Will need a replay to investigate.


docs(EMPTY): add example

Description:
Add a simple example for the EMPTY constant, add corresponding marble diagram.
Related issue (if exists):
N/A

@edbzn could you maybe rebase master  into this. CI should work afterwards, if not I'll investigate

@JWO719 It looks ok


Add support for querying Elasticsearch aliases

This looks good :+1:
I'd also add a way to query index prefixes / wildcards, which is a common use-case with ES, so SELECT Count(*) FROM "logs-*" would work (maybe it already does now)

I'd also add a way to query index prefixes / wildcards, which is a common use-case with ES, so SELECT Count() FROM "logs-" would work (maybe it already does now)

That's currently not supported, but we'll do that as a separate change. I need to investigate a bit how to handle the metadata for those kinds of "virtual" tables.

That's currently not supported, but we'll do that as a separate change. I need to investigate a bit how to handle the metadata for those kinds of "virtual" tables.

@martint I thought about this again, and it's actually perfectly fine to require an alias to be defined if the user wants to query an index pattern (prefixes, wildcards, etc). It's just a matter of documentation now.


Change version number of ssh-agent

Github Actions failed with<EMAIL_ADDRESS>Works with<EMAIL_ADDRESS>

This is what Github Actions said when failing "Set up job":
Current runner version: '2.157.5'
Prepare workflow directory
Prepare all required actions
Download action repository<EMAIL_ADDRESS>##[warning]Failed to download action 'https://api.github.com/repos/webfactory/ssh-agent/tarball/v0.1'. Error Response status code does not indicate success: 404 (Not Found).
##[warning]Back off 19.349 seconds before retry.
##[warning]Failed to download action 'https://api.github.com/repos/webfactory/ssh-agent/tarball/v0.1'. Error Response status code does not indicate success: 404 (Not Found).
##[warning]Back off 24.9 seconds before retry.
##[error]Response status code does not indicate success: 404 (Not Found).

Changing it to v0.1.1 works

Opened https://github.community/t5/GitHub-Actions/Version-numbering-for-Actions/m-p/32271 to clarify how it's supposed to work.

Fixed in 1a9af8e8e040e1ca1a7da67d6a45e13ba9890630


[FEATURE] Add a scroll to top button on all pages

Description
I was thinking of adding a scroll to top button on all the pages, I just saw that it is yet to be implemented on any page over the past two weeks, so though could give it a try .
For reference of my experience working with a similar issue, please refer issue #847
I can get it done quickly
Screenshots
No response
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

I would like to work on this issue under gssoc'23

Hi @karthiknadar1204 , there is no concept of "getting it done quickly" in Opensource. I believe in your skills, but for now, I will put this issue on hold. Before moving on, I wait for at least 3-4 weeks and then I close an issue - in this case, I assigned them last week, so I will wait more - because contributors can be busy too, with 1000 different stuff.
We are not competing here for getting something done quickly, the goal is to give EVERYONE a chance, and to make sure there's a win-win scenario for everyone who is contributing to my project.
Anyways, putting this on-hold, this will be continued if #833 is discarded.

Totally understandable,thank you so much ,will explore other issues to work with for time being.

Hi @karthiknadar1204 , you can work and make a PR on this if interested, thanks !
Remember, Good PR makes you a Good contributor !
We at Milan work hard to maintain the structure, and use conventional Pull request titles and commits. Without a proper template for the PR, not following the guidelines and spam might get the pull request closed, or banned.

Please make sure that you have read our documentation on Commiting guidelines and Pull request guidelines.

Once you have done the changes you can request a review. If I suggest changes again, do those and request a review again.

Make sure you DO NOT modify the PR template that we have, use it as it is.

Last but not least, be patient. It takes a while to review the changes and get it merged.

Thank you,will get working on it