High DPI scaling issues with Qt::AA_EnableHighDpiScaling (macbook pro retina ubuntu 16.04)

tl;dr
I am raising this issue as a follow up to #1077 which was partially fixed in #1078. Qt::AA_EnableHighDpiScaling causes RenderWidget resize event to be 1/2 the proper size. Proposed fix in #1263.
Setup:

macbook pro late 2011
retina display
ubuntu 16.04
branch: kinetic-devel
rviz at commit: 87742e4b8f40d8fe7e2a6952065651266faf73ba
rviz version 1.12.16
compiled against Qt version 5.11.0
compiled against OGRE version 1.9.0 (Ghadamon)

Problem:
Since Qt 5.6 (see here), high DPI scaling can be handled automatically by setting the application attribute Qt::AA_EnableHighDpiScaling in the main.cpp of a Qt application.
// Enable support for high DPI screens
QGuiApplication::setAttribute(Qt::AA_EnableHighDpiScaling);

This makes gui design more homogeneous across screens (at least where pixel coordinates are concerned).
Unfortunately when I enable this attribute and am using librviz, the RenderWidget only fills 1/4 of the RenderPanel:

The rest of the space is filled with whatever was behind the RenderPanel upon a resize event.
When I do not set the AA_EnableHighDpiScaling attribute the RenderWidget displays properly, but the rest of the gui (not shown) is far too small:

The image above is scaled here but on my screen it is 2x smaller than it should be. This is because AA_EnableHighDpiScaling is not set.
Resolution:
I say this problem was partially fixed in #1078 because the AA_EnableHighDpiScaling is never used in rviz and the RenderWidget displays as expected.
I traced the problem to the resizeEvent() handler in RenderWidget:
void RenderWidget::resizeEvent(QResizeEvent *e)
{
  if( render_window_ )
  {
    // render_window_->writeContentsToFile() (used in
    // VisualizationFrame::onSaveImage()) does not work right for
    // window with an odd width, so here I just always force it to be
    // even.
    render_window_->resize( width() + (width() % 2), height() );
    render_window_->windowMovedOrResized();
  }
}

When using AA_EnableHighDpiScaling, the functions width() and height() return the widget width and height with respect to a virtual pixel coordinate system and not the real pixel coordinate system. The Ogre::RenderWindow on the other hand is expressed in the real pixel coordiante system (I think...) and therefore only renders at 1/devicePixelRatio().
I have proposed a fix in #1263 which simply multiplies the width and height by the pixel ratio:
render_window_->resize( width()*pixel_ratio_ + (width()*pixel_ratio_ % 2), height()*pixel_ratio_ );

This results in a properly scaled RenderWidget:

When not using AA_EnableHighDpiScaling, the widget fills the panel but it the whole gui is half the size, which is the expected behavior on high DPI screens.

I tried the proposed fix on my Mac and it seems to work. Not sure if it is related to the issue described here when I try to do e.g. a Pose Estimate it only works in the upper right corner of the rendering area.

I see this issue on Ubuntu 18.04 and melodic. Did not have the issue on 16.04 and kinetic.

Same issue with a Dell XPS 4k (16.04 / kinetic). https://github.com/ros-visualization/rviz/pull/1263 fixes it.

Fixed via #1263.


Using SamAutomaticMaskGenerator to predict masks by fine-tuned SAM models but returns an empty list.

Hello everyone,
I met a problem when using SamAutomaticMaskGenerator to predict masks by using fine-tuned SAM models.
It always returns an empty list.
But it works well when following the workflow in the utils.py where the SamPredictor(self.model) is used.
I used a customized dataset that includes 512x512x3 image patches and json file of annotations.
The annotations only include two classes that are background and target objects.
I tested the fine-tuned model by the following codes:
from segment_anything import sam_model_registry, SamAutomaticMaskGenerator, SamPredictor
import torch

The models are fine-tuned and tested on Google Colab.
If you happened to solve this issue, look forward to your answers.

As mentioned in the README, this project only supports bounding box input prompts. AutomaticGenerator wouldn't work with the current code

As mentioned in the README, this project only supports bounding box input prompts. AutomaticGenerator wouldn't work with the current code

Thanks so much for your explaination. It is really helpful.

If you want to use the SamAutomatucMaskGenerator, you simply need to train with point prompts. I sampled points randomly on the mask. Also make sure that multimask_output=False is set in line 282 if the automatic_mask_generator.py , if you also fine-tune with multimask_output=False.

thanks so much for your reply : )
I will try it in the future.
I found another solution from this link:
https://github.com/tianrun-chen/SAM-Adapter-PyTorch
It is more suitable for semantic segmentation as far as I have tried.

Hello everyone,
I met a problem when using SamAutomaticMaskGenerator to predict masks by using fine-tuned SAM models. It always returns an empty list. But it works well when following the workflow in the utils.py where the SamPredictor(self.model) is used.
I used a customized dataset that includes 512x512x3 image patches and json file of annotations. The annotations only include two classes that are background and target objects. I tested the fine-tuned model by the following codes:
from segment_anything import sam_model_registry, SamAutomaticMaskGenerator, SamPredictor import torch

The models are fine-tuned and tested on Google Colab.
If you happened to solve this issue, look forward to your answers.

Hi @YunyaGaoTree , I also got the empty list after training for larger epoch. Did you find any solutions for the same?

Hey, my friend and I have been working on a fine-Turing of SAM (automatic predictor) for our thesis. It might be useful for your problem. Check it out https://github.com/Marchisceddu/finestSAM


Add code examples to docs examples

Required changing colors to arrays.  Is there a different change that
can make this work?

Codecov Report

Merging #32 (b46eb51) into dev (d799206) will increase coverage by 1.03%.
The diff coverage is 89.72%.

@@            Coverage Diff             @@
##              dev      #32      +/-   ##
==========================================
+ Coverage   82.92%   83.95%   +1.03%
==========================================
  Files          15       15
  Lines         527      642     +115
==========================================
+ Hits          437      539     +102
- Misses         90      103      +13

Impacted Files
Coverage Δ

src/core/options.jl
100.00% <ø> (ø)

src/PowerPlots.jl
41.66% <12.50%> (-58.34%)
:arrow_down:

src/plots/power_vega.jl
88.63% <86.15%> (+0.99%)
:arrow_up:

src/plots/kamada_kawai.jl
94.73% <94.73%> (ø)

src/core/data.jl
70.45% <100.00%> (+9.34%)
:arrow_up:

src/core/types.jl
100.00% <100.00%> (+2.43%)
:arrow_up:

src/core/utils.jl
95.65% <100.00%> (-1.02%)
:arrow_down:

src/layouts/layout_engines.jl
55.55% <100.00%> (ø)

src/plots/networks.jl
64.44% <100.00%> (ø)

src/plots/network_status.jl
70.00% <0.00%> (-2.00%)
:arrow_down:

... and 2 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 d799206...b46eb51. Read the comment docs.


UICR-8-Bootstrap-Components

In this pr I added Alerts, Badges, Breadcrumbs, Buttons and Button Groups from Boostrap Documentation: https://getbootstrap.com/docs/4.5/components/alerts/.
It just the scss and html code.

Hi @chrissmejia  I'm going to create another branch and pr and worked with the new commit you created yesterday, then I will send new pr's. Thank you.


Do not strip ExecutionException from the stack trace

Motivation
When we wrap exceptions into the PulsarClientException in the sync methods in client API, we shouldn't strip the ExecutionException from the chain. This will contain the stack of the application where the get() is being called and it's important to have that for a user to understand from where (in the application code) the exception was thrown.

run java8 tests

@merlimat Please take a look the unit tests, looks like related to changes of this pr.

run java8 tests

run java8 tests

run java8 tests

@merlimat since this PR is from your master, other people can not rebase this pull request. Do you mind checking it again so that we can move forward with 2.4.0 release?


Need to fix dependency issues

In which file did you encounter the issue?
spanner/opencensus/src/main/java/com/example/spanner/opencensus/CaptureGrpcMetric.java
Did you change the file? If so, how?
No
Describe the issue
The GRPC metrics no longer show up with the latest version of spanner library.
After testing, it was found that we need to change the dependencies for the metrics to work.

com.google.cloud
google-cloud-spanner
6.37.0

io.opencensus
opencensus-api
0.26.0

com.google.protobuf
protobuf-java
3.22.1

io.opencensus
opencensus-impl
0.26.0
runtime

io.opencensus
opencensus-contrib-zpages
0.26.0

io.opencensus
opencensus-exporter-trace-stackdriver
0.26.0

io.opencensus
opencensus-exporter-stats-stackdriver
0.26.0

io.grpc
grpc-census
1.53.0

io.opencensus
opencensus-contrib-grpc-metrics
0.26.0

Reassigning to @olavloite for triage.

@rajatbhatta Would you mind taking a look at this?

As per internal documentation, most likely the cause of the issue is misconfiguration on the customer's end like not including critical dependency like grpc-census in their application. We don't have to modify any dependencies from the client libraries side. Thanks! Closing.


Rewrite of is_closed check.

This PR contains to (very linked) changes:

The rewriting of the is_closed check. (tl;dr; the computation is done using float64 to avoid errors on floating point aritmetics, and the maximum error is now computed relatively to the absolute value (since 1e-4 was not appropriate in all cases)
Adding a exact is_closed check. (tl;dr; assert each edge is present exactly two times, one time in each direction (an edge can be two time in the same direction).

Rewriting of is_closed check
I had problems, false negative, with a mesh (which is closed, I am sure of that property):
In [3]: m.update_normals()

In [4]: m.is_closed()
Your mesh is not closed, the mass methods will not function
            correctly on this mesh.  For more info:
            https://github.com/WoLpH/numpy-stl/issues/69
Out[4]: False

In [5]: m.normals.sum(axis=0)
Out[5]: array([-7.1411133e-03,  1.3427734e-03, -1.7878906e+02], dtype=float32)

In [6]: m.normals[:,2].sum()
Out[6]: 0.0234375

In [7]: m.data.size
Out[7]: 577544

The problem is floating point arithmetic on 32 bits with a large number of triangles. If we do the sum on float64:
In [8]: np.asarray(m.normals, dtype=np.float64).sum(axis=0)
Out[8]: array([ 5.96046448e-08, -6.82390237e-07, -8.84249806e-04])

However, we can quantify the maximum allowed error by using a pessimistic approach. Each initial value (in float32) have a error of abs(value)*eps where eps is the relative error of float32, then the maximum error is sum(abs(value))*eps:
In [9]: np.abs(np.asarray(m.normals, dtype=np.float64)).sum(axis=0) * np.finfo(np.float32).eps
Out[9]: array([0.00313204, 0.0031321 , 0.01907349])

Then we obtain a is_closed` test without (I hope) false negative.
Remark: the problem appear, even with a low number (like 1e4) of triangles.
Exact version of is_closed
As shown in #198, the is_closed can lead false positive. Because the sum to zero of the normals is a necessary condition, but not a sufficient one.
The #198 ask to check "if each edge is defined exactly twice", but again, this is a necessary condition, but not a sufficient one (even with adding the previous check with the normal).
The necessary and sufficient condition is the following: each undirected edge must be defined twice, each directed edge must be defined exactly once. The direction used is counterclockwise w.r.t. the normal (triangle which have a opposite normal w.r.t. the cross product are reversed).
This check is implemented with exact=True. And success to detect example of #198 as unclosed.
However, this check is quite longer than the not exact check on normal, therefore the not exact test is conserved by default, and this check is added as a option.
Remark: fixes #198.

This looks like a great addition, thank you! I'm testing it now and plan to create a new release this week :)

FYI, I am working on another check I need, the closeness is not sufficient for my usage and I am curently implementing a self intersection test. If you want to include that in a new release, it should be a good idea to wait 2-3 days.

I'm planning to do a little maintenance anyhow so you've got time :)

Just wondering, are you close to finishing it yet? :)

Apologies for the delay. No, I don't have a functional optimized self-intersecting test. (I have one which is very time consuming). I will open another PR in the future if I success. Do not wait for this test for a new release (2-3 months is more likely than 2-3 days). Sorry for the hope.

No worries, all help is always appreciated!
I wouldn't mind adding an extra slow but accurate method either as an option. If you want to include it that is :)

Github doesn't appear to understand the merge, but it's been merged and released.
Thank you for the help!


Fix Hero Section Visibility and Button Alignment [#1739]

This pull request addresses the following enhancements for the hero section of the application reported in #1739:
Changes Made

Button Alignment: Aligned the "Go to Challenges" and "View Source Code" buttons in a single row. This adjustment provides a more consistent and organized layout, making it easier for users to find and interact with these call-to-action buttons.

Added hover effects with box shadow in both the buttons

previous status

Current status
https://github.com/user-attachments/assets/d876e9a1-3e49-4dfb-829b-10e8d263835b

fix #1739


better error reporting when a process emits non json tracing output while in a wait_for

Currently this gives a generic panic: "bin process shutdown before an event was found matching: "

Ah, we already do this, it just wasnt occuring due to silly local changes to tokio-bin-process


Hexadecimal code

Great job! It works perfectly!
Do you know if you can communicate with IR using the IRremote library?
For this, I would need the corresponding hexadecimal code.
So I could call it this way:
   for (int i = 0; i <3; i ++) {
     irsend.sendLG (0x20DFC03F, 32);
     delay (40);
   }

Lib example: https://www.pjrc.com/teensy/td_libs_IRremote.html
Thanks!

Hi, thanks for checking this out.
I actually haven't tried that, sorry. I just went straight from the protocol to the actual pulses.


BUG: On Windows, file path is displayed as the file tag instead of filename

Describe the bug
On Windows system, the full file path is displayed as the file tag instead of just the filename after uploading a file.
To Reproduce
Steps to reproduce the behavior:

Open the application on a Windows system
Click on "Choose File" to select a file
Select any ORCA log file as the input file
Click on 'Upload'
Observe the name in the file tag that is displayed under 'Uploaded Files:'

Expected behavior
Only the filename should be displayed, not the full file path.
Screenshots
Additional context
This issue appears to be specific to Windows system
Acceptance Criteria

When a file is selected on a Windows system, only the filename (e.g., "document.txt") is displayed, not the full path.
The fix should not affect the correct display of filenames on other operating systems.
The actual file path should still be correctly sent to the server for processing.
The fix should work for files selected from different directories on the Windows system.
Add/update the related unit test cases with the code changes.

Can I work on this?


Not working Clear Cart button in Navbar Cart

In navbar at Cart button, there is Clear Cart button.
The work of Clear Cart button is to clear the Cart. But it is not working.
Update clearCart funtion in index.html file of shop.

You are assigned :)


Add symbol if not defined

This defines the symbol RTM_GETNSID if it is not defined
This fixes compilation of the terraform lxd plugin on travis for the architecture linux/amd64.
The split of cgo and non-cgo code is still required to compile for linux/arm ( #5090 )

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

#5091 seems a better solution, however the extra symbol is not needed for getting it to compile

That's just a safety measure I'd like to apply. :)

Closing, please cop #5091 into #5090.


⚖ Register Validator:

Network Choice
nemeton-1
Who are you?
Hey there! I am programmer, with a great professional background in server operations and administration. I actively participate in dozens of testnets, helping to keep them running and debugging, participating in development and support for an early start. Also already support many projects as mainnet validators.
My validator nodes run on dozens of blockchain networks with no downtime. A fault-tolerant infrastructure is in place, with data replicated to multiple servers in different DCs around the world, so i can ensure high node availability.
I creating guides and developing tools for community to make node operators life easier. Also glad to help projects by providing my highly secure and reliable infrastructure. My services include running RPC nodes, running State Sync services, API services and IBC relayers.
I do not use cheap servers, only powerful and reliable servers. This allows my to uncompromisingly ensure reliability and availability of data, as well as high uptime of my nodes. In addition, the use of multi-level monitoring systems and a professional support team allows to solve non-trivial problems quickly and efficiently. In addition, the use of multi-level monitoring systems Grafana + Prometeus + Telegram/Discord alerts allows to solve non-trivial problems quickly and efficiently.
Website: ppnv.space
Explorer: explorer.ppnv.space
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"PPNV","identity":"","website":"","security_contact":"","details":""},"commission":{"rate":"0.050000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.010000000000000000"},"min_self_delegation":"1","delegator_address":"okp41u67ek8zkxrc7fsxnhwrzu3lnk3zz0r7hushyg7","validator_address":"okp4valoper1u67ek8zkxrc7fsxnhwrzu3lnk3zz0r7hfh8dkl","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"53ks53DjfBQ/DbPjAtAurd5EchdCr6fj7bk6gufE+Z0="},"value":{"denom":"uknow","amount":"10000000000"}}],"memo":"1716ac16c3e8464f0c288e90a38008be5ce56af9@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"AnVrK29oMvvcAK+1ihGVHneCOLagAFUO71PyPcl8C6Ba"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["SUaZeJtaMvKVuGTzapX+30oKA0aPrME083kQ1w10y9t3zarNfhvKkY7MAIjXJV01jaUQqvO7tf+W/bYVE/ryyQ=="]}

😉 Here is the corresponding PR: https://github.com/okp4/networks/pull/271


how add fluent-plugin-udp-native-sensors to  fluentd config

i have installed fluent-plugin-udp-native-sensors on ubuntu 16.04
i add source UPD in td-agent.config :
'
@type udp
tag juniperNetworks
format juniper_udp_native
port 50000
bind <IP_ADDRESS>
body_size_limit 5000
  '
i have log error : config error file="/etc/td-agent/td-agent.conf" error_class=Fluent::ConfigError error="Unknown parser plugin 'juniper_udp_native'. Run 'gem search -rd fluent-plugin' to find plugins"
how i can fix it/
Thanks

Can you please specify the version of fluentd that you are using?

@longvd89 Sorry I missed checking this.
I just checked this using td-agent approach and works fine.
Did you install the plugin? You should find something like below
$ td-agent-gem list | grep udp
fluent-plugin-udp-native-sensors (0.0.1)
$


PowerToys Run IS BECOMING UGLIER WITH EVERY UPDATE!

[ I'm outraged by the ugliness of the tool that used to be the pride of PowerToys. I don't even feel like doing the updates that come out, even though it's the tool I use the most in my daily life.

I want to go back to the basic version or have a function to change the theme of PowerToys Run!]

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

ID: 0b951899-69ca-9057-f799-14491e1b0a74
Version Independent ID: 18e0202f-3e91-a5b9-62f2-6cd40e5661a8
Content: PowerToys Run utility for Windows
Content Source: hub/powertoys/run.md
Product: dev-environment
Technology: powertoys
GitHub Login: @alvinashcraft
Microsoft Alias: aashcraft

This feels not so much like feedback specifically about the documentation, but rather like feedback about the PowerToys product. I don't have any additional content for your scenario, over and above what's in the documentation. So, it might be worth pursuing this as a question or feedback about the product. If you visit the PowerToys repository, then you'll be able to create an issue to leave product questions/feedback.


Expand the Literature Collection with New Journals, Keywords

The current criteria and list of journals may be too narrow. Let's expand the keywords and criteria to make our search space a little bit broader.
Add new keywords: “Group,” “Collective Intelligence”

These keywords may be more useful for basic science journals, which may not find something team-specific interesting.

Add new journals to list:
Small Group Research
Academy of Management Review
Academy of Management Journal
Organization Science
Psychological Science
Management Science
Other journals to list: (see this h-index list)
https://scholar.google.com/citations?view_op=top_venues&hl=en&vq=soc_humanresourcesorganizations
https://scholar.google.com/citations?view_op=top_venues&hl=en&vq=bus

Another suggested source: Discourse Analysis: Theory and Method by James Paul Gee


fix issues with filtering nulls on values coerced to numeric types

Description
This PR fixes some bugs in both SQL compatible and 'default' value modes when filtering IS NULL/IS NOT NULL on values which are coerced to a numeric type, such as CAST(x AS BIGINT) or JSON_VALUE(nested, '$.x' RETURNING BIGINT). While these expressions had different causes, the underlying effect was more or less the same and involved the query engine using the null value index for the underlying column to build the cursor, which only counts the null string values, and misses out on values which might become null after coercion.
For example given the following source data:

trying to filter out nulls in default value mode would produce the following results:

and the same data ingested in SQL compatible mode also incorrect:

However the "correct" results in "default" value mode should match all of the rows because there are no null numbers, and in SQL compatible mode, should only match a single row, since that is the only truly non-null number.
JSON_VALUE suffered from the same problem when using the RETURNING syntax to cast to a numeric type for string or mixed type fields.
To fix this, we avoid using the indexes when this is the case. It would be possible to partially use these indexes and a value matcher for the 'is not null' case, but currently not null is handled as a single index, rather a not filter on a selector filter, so its a little bit tricky to implement right now so I'll save this for a future change.
This PR also fixes issues with 'auto' typed numeric columns to ensure they behave consistently with classic numeric columns in 'default' value mode, which means picking up the 'null values dont exist' behavior that much of the query engine has. In default value mode, the numeric index suppliers will combine the null and 0 indexes when searching for 0 to ensure that indexes behave consistently with the value matchers. I have added the 'auto' typed columns to BaseFilterTest to better cover the 'auto' indexers, only skipping tests related to implicit multi-value handling.
Release note
Fixed issues involving filtering NULL values when using CAST expressions and JSON_VALUE expressions using the RETURNING syntax to coerce non-numeric types into numeric types, such as CAST(x AS BIGINT) IS NOT NULL or JSON_VALUE(nested, '$.x' RETURNING BIGINT) IS NULL. These expressions were incorrectly using the null value index of the underlying column, which produced results matching the values which were NULL string values, but not values which could not be converted into a numeric value and so were effectively NULL.

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Within the nested package I don't really understand the changes. I think I mostly don't understand how the variant column is different from a COMPLEX nested column. How is it meant to work now?

Added a bunch of javadocs to hopefully clear up how everything relates to each other, still haven't got everything quite yet but will keep chipping away at it.


Force APNS | Added boolean ios init Option apnsForce to force apns | Updated docs as well

Description
Default : False [Uses FCM if GoogleServiceInfo.plist is present and APNS if not]
If set to true (Boolean) or "true" (String) within ios options in init, it will force APNS to be used even if GoogleServiceInfo.plist is present in the project
Related Issue
#1770
#2138
Motivation and Context
Allow forcing of APNS and FCM as necesssary from JS instead of having to edit the GoogleServiceInfo.plist file everytime
How Has This Been Tested?
Added GoogleServiceInfo.plist in ios and made sure apnsForce : true and apnsForce : "true" was causing APNS to be used over FCM. Also tested by removing apnsForce and setting apnsForce : false while GoogleServiceInfo.plist was present and it defaulted to FCM.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Please let me know if I missed something, I will raise a new pull request.
Also thank you for this wonderful plugin.
Cheers,
Chris
Neutrinos


[Feature Request] File preview on mouse hover

Description of the new feature / enhancement
Show a preview of a file (image, video, txt) on a small window or pop-up when hovering over a file, unlike File Explorer's preview pane which requires you to click every file to preview. This feature is seen in QTTabBar but recent File Explorer with tabs made this 3rd party app almost unusable.
Scenario when this would be used?
When needed to find an image or video from files in list view without changing view or opening the file. Because downloaded or saved files don't always bear meaningful names.
Supporting information
Here is an example of how this works with QTTabBar

Saw this, couldn't make it work, but seems maybe like the same functionality?
https://github.com/CrystalSoft/winrefine
Would be happy if the feature of QTTabbar could be insolated implemented!

Implemented as PowerToys Peek (press spacebar to preview). Closing issue.


Updating the expiration for password reset.

What's this PR do?
This PR simply extends the password reset expiration time from 1 hour (60 minutes) to 24 hours (1440 minutes).
How should this be reviewed?
👁
Refs https://trello.com/c/5C2oV0GZ/189-as-a-user-i-want-the-resent-password-link-to-expire-in-24-hours-instead-of-1-hour-so-that-i-can-reset-it-even-if-i-don-t-see-the

For review:
@DFurnes

haha

 As a user, I want the reset password link to expire in 24 hours instead of 1 hour so that I can reset it even if I don't see the email right away.


Added: Users table and mechanism to link requests to requesting users.

To work in production, this will require the addition of:

A new table called "Users" with fields "Full Name" and "Phone Number"
A new column in "Requests" table called "Requester" that has links to "Users"

Tests look good. Let's just use a better name for the new table like "Requesters"? We'll choose a name during the standup tomorrow.

Tests look good. Let's just use a better name for the new table like "Requesters"? We'll choose a name during the standup tomorrow.

@azlyth I have updated the code to use "Requesters" instead of "Users"


Fix clang version check and remove clang 12 warning

The clang version check in build/MakefileWorker.mk only worked for apple
clang, so this changes the version checking to work with apple, debian,
and ubuntu clang version strings.
Additionally, clang 12 added -Wpoison-system-directories, which warns if
you if you include stuff from /usr/local/include when doing a cross
compile.
Apple clang thinks it's always doing a cross compile for some reason, so
that always complains.
Closes #1456

Coverage remained the same at 99.834% when pulling 62a6e20bbf9596c63549e83f51a6709e36fb9467 on mtfurlan:clang-version-checking into 138c775b607b55d0fc4d2fcb072e8f71682989f5 on cpputest:master.

Thanks!


Allow the user to provide a custom UIView for a message

My current project has a need for custom views inside message bubbles. The most pressing need at the moment is to include a web view. I got the the pull request "Added HTML Media Message Type" (#917) working with the latest from develop and also fixed up a few of it's shortcomings. However, I ran up against a few underlying assumptions with media items.
One primary goal is to have the custom view be inside of the same colored bubbles as text based messages. However, the setMediaView method in JSQMessageCollectionViewCell removes the messageBubbleImageView from the view hierarchy (in fact it removes all of the views except for the latest mediaView).
Others seem to be very interested in customizing the content in the message bubbles as well. For example, issue #809 and #681 both want to display text and images inside the message bubble. Abstracting such requests seems challenging but allowing users to configure their own view layouts opens the door for them to service themselves.
My idea to solve this would be to create a new JSQCustomMediaItem that allows the user to provide their own custom UIView to render inside the message bubble.

Thanks @ryangrimm ! :smile:
This is definitely a feature we're considering for v8.0.
However, for now, you could simply include a message bubble view as part of your custom media view.

I hadn't thought about using the message bubble view in a context like that. That probably would have gotten me far enough along for the time being. But, as you can see by the pull request I already spent the time to cook it up. At least it was fun in the process! Let me know if it looks like something that you'd like to fold into v8. Happy to make changes if you'd like me to.

i am using bar graphs programatically in my project so, is there any way to put a custom view into this library so i can put my graph code on it and show it on the the view  like addPhotoMediaMessage ??

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


100px inline height in collapsible blocks with code in it

In the article Problem 14, all 7 collapsible blocks with code in them have a fixed height of 100px in them. (This does not affect the first collapsible which is a spreadsheet and not a code block).

     ...

This makes it hard to read (refer to the image below, only three lines visible), removing the inline style gets rid of this.

Screenshot from Mozilla Firefox 108.0.1 on Arch Linux, also tried in Chromium Version 108.0.5359.124, behaves the same.

Can confirm: code collapses do not uncollapse, they just continue to show the collapsed/truncated version. Bad. I've pinged Achmiz. Must be a fairly new bug, I'm sure I checked /Lorem's code collapses in December 2022.

Might be caused by 8008c4a but I'd guess Achmiz would've already figured the cause by now.

He agrees, but doesn't have time right this second to fix it properly so I'm commenting it out.


Can't update (CSGO)

Hi,
when I try to update my server everything i fine util I have to add the Two-factor code from the  Steam Guard Mobile Authenticator app:
Once the code is entered nothing happens.
Any idea how to fix this ?
I'm using centos 6.7.

Hello there !
Why would you use your steam account for CSGO which don't need it ?
PS 1) https://github.com/dgibbs64/linuxgsm/issues/579
PS 2) Also run a validate after an update if you got issues, can help.

I thought we had to with their new policy. But has suggested i removed it and it works fine now.
Many thanks.

Nope, it's true that as it is into the script "just in case"  it becomes needed one day, you can easilly be fooled. But gladly, csgo, as many valve games, work as anonymous !
You're welcome ! Have a nice one :)

You where probably thinking of this https://github.com/dgibbs64/linuxgsm/wiki/Game-Server-Login-Token.


Implement motor controls in the firmware

Implement the motor controls so that the motors move in a line to any coordinate they are passed. This means that all motors have to move simultaneously.
Example moving effector from 0,0 to 5,5 should be a 45-degree line.

We probably dont need accelerations in this case

Have to fix firmware stepper control method. Current method doesn't work from tests conducted on March  8, 2018

Issue closing due to project disconitnuity


[WIP] *: Add initial PersistentVolume feature

this patch adds initial persistent volume feature for etcd data.
When the usePV option is enabled in the etcd cluster spec every
created pod will get a related persistent volume claim and the pod
restart policy will be set to RestartAlways.
This initial patch only covers the case where a pod crashes (or a node
restart before the related pod is evicted by the k8s node
controller). When using a persistent volume, instead of deleting the pod
and creating a new one, the pod will be restarted by k8s.
A future patch will change the reconcile logic to also handle cases
where one or more pods are deleted (manually or by k8s) recreating the
same pod using the same pvc.
The etcd data pvc will be garbage collected like the other cluster
resources.
Questions:

I'm using the same backup storage class (the name isn't great since it starts with etcd-backup-. Should we define different storage classes using the same provisioner or permit different provisioner for backup and data?

I'll add some questions inline in the code.

TODO:

[ ] Check all the tests on your infrastructure (I'm testing on a local k8s cluster disabling tests that needs S3 and updating the generated storage class with another pv provisioner).

Please hold on.
I'm still not sure if we are going to use PVC for etcd storage.
Instead, we would like to go the path of local PV: https://github.com/coreos/etcd-operator/issues/1201

Please hold on.
I'm still not sure if we are going to use PVC for etcd storage.
Instead, we would like to go the path of local PV: #1201

Local PV and PVC are different. I am fine with adding PV as long as it does not interleave too much with the current logic. Or we have to refactor the code before adding it.

Local PV and PVC are different.

Well. They are somehow the same. They are both PV abstractions. They can share the same code path for tolerating etcd downtime, GC, and management code in etcd operator. At least, we should have decided down what local PV looks like before adding option for non-local PV. Local PV should take higher priority.

Actually it is fine to have PVC support. But it is very important to make sure the abstractions are right so that we can share the code path with local PV as well.

Well. They are somehow the same. They are both PV abstractions. They can share the same code path for tolerating etcd downtime, GC, and management code in etcd operator. At least, we should have decided down what local PV looks like before adding option for non-local PV. Local PV should take higher priority.

@hongchaodeng @xiang90 My opinion is that they are both PV and both can be abstracted using a PVC. I'll always use a PVC instead of directly creating a PV since a PVC adds some goodies for PV management (the storageclass => provisioning and volume node affinity).
The main difference between them is the node affinity. One is "local" (available only on one node) and others (EBS, GCE PD etc...) are "global" (available on more than one node (one node at a time)), though they can be available only on some subset of the total k8s cluster nodes like EBS that are per availability zone).
This difference in node affinity can bring to some differences in the "reconcile" logic. Taking from https://github.com/coreos/etcd-operator/issues/1323#issuecomment-318684441:

pod restart - still use previous PV
pod movement - detach the PV on the pod to move (or dead), and reattach that PV to the new pod

(NOTE: this PR implements 1).
Point 2, if intended as pod movement between different nodes, can be done only with "global" PV.
Another feature (that is one of the main reason I'm trying to implement PV support) is that, instead of doing "disaster recovery" when there're less than N/2+1 RUNNING etcd pods due to some k8s nodes going down, one could just wait for them to be come back (node reentered the k8s cluster that will restart the pod since the policy is RestartAlways) or wait for pods to be deleted from the API (by the node controller + manual intervention or a fence controller) and recreate them using the previous PV that will be scheduled on another node.
This is the second part of this saga. I already implemented a part of it (but with a lot of assumption since with PV you have multiple choices) so If you want I'll be happy to open another RFC PR if it can be helpful to better explain the overall idea.

@sgotti
So I think the purpose of this PR is to add a PVC for each etcd pod and change restartPolicy to Always, right?  Let's not handle too many stuff at once and take it step by step:

Assume the etcd pod will always restart on the same node for now
For next PR, it will handle the toleration, re-attach PVC to another pod, disaster recovery, etc. Needs discussion in https://github.com/coreos/etcd-operator/issues/1323 .

So this PR will handle:

create PVC when scaling up etcd
delete PVC when scaling down etcd
wait etcd to restart instead of removing it

For this to move forward, this is what we should do:

A PR for only spec change. How would user tell us it expects each etcd pod to uses a PVC, and what kind of PV it would use?
A PR for documentation, how could users use it.
A PR for internal changes, some heavy-lifting refactoring of the code to make sure good design
A PR for testing.

@sgotti
I think the direction is good in general. As @hongchaodeng mentions, can we split this effort into multiple PRs? Thank you!

@sgotti kindly ping on this one. is there anything we could help to move this forward?

@xiang90 @hongchaodeng sorry for being late, I'm in vacation until next week. OK for the plan, I'm not sure if you already have in mind which refactoring will be needed, and I'll keep implementation and tests in the same PR for obvious reasons.
As a first step I'd like to propose some possible changes on the way volume management is handled in the etcd-operator. Is it ok if I open a new dedicated issue (since I have in mind different option with different pros and cons)?

Is it ok if I open a new dedicated issue (since I have in mind different option with different pros and cons)?

sure.

@sgotti I had a query regarding keeping restartPolicy to Always. I would like to know if a data-corruption in etcd data volume will result in the pod getting restarted unnecessarily. Would it not be better if we can handle it using the reconciliation logic? We could know the memberId thats down and in turn the pvc associated. Could we not create the pod with the same memberId and pvc attached?
While replacing a failed member, its mentioned that we remove the instance and add a new instance. Just restarting the pod does not conform to what is mentioned in this documentation.

Is it ok if I open a new dedicated issue (since I have in mind different option with different pros and cons)?
sure.

I wanted to propose to directly use StorageClass in the spec instead of the --pv-provisioner option but I noticed this has been already accepted during my vacations in #1361. That's really good. So I just went ahead opening PR #1373 to discuss just spec changes.

@sgotti I had a query regarding keeping restartPolicy to Always. I would like to know if a data-corruption in etcd data volume will result in the pod getting restarted unnecessarily.

I agree that adding etcd data to a persistent volume adds the requirement to also handle a data corruptions case that previously wasn't needed.

Would it not be better if we can handle it using the reconciliation logic? We could know the memberId thats down and in turn the pvc associated. Could we not create the pod with the same memberId and pvc attached?

I think that restartPolicy Always is a fast way (as proposed by @xiang90 ) to use k8s features to restart a pod if it has crashed or the node reboots without adding this logic to the reconcile loop. Another way will be as you proposed to do it inside the operator reconcile loop.
But I don't see how this could help detecting corrupted data vs pod/etcd not starting for other reasons. I'm not sure which is the best way to detect a corruption.
My intial idea was to just remove and add a new member (with a new empty PV) if it's not healthy for some time. This will work also with restartPolicy to true and could be done only if the etcd cluster is healthy so won't create problem if all the etcd pods (or nodes) goes down at the same time achieving the goal of this PR.
Quite probably @hongchaodeng @xiang90 have better ideas.

@sgotti, @xiang90 Is the first part of the PR without PVC reuse going to be incorporated upstream? When can it be roughly expected to be merged?

closing this in favor of multiple PRs (first is #1373)


Update to Conda forge 3

just propagate the conda forge 3 PR into this so we can use it :)

@conda-forge-admin, please rerender

Can you remove all the MOAB/DAGMC lines? We can keep this one clean of those options and then add them in a separate PR.


SkPoint3::normalize() loses precision and causes a tilt in rotation

Skia-side bug:
https://bugs.chromium.org/p/skia/issues/detail?id=8325
We should use the Skia normalize() when it is fixed and remove the custom temporary normalize() because skia handles edge cases/underflow/overflow.

Fixed Skia-side by https://skia-review.googlesource.com/c/skia/+/150380

Integrated back into engine: https://github.com/flutter/engine/commit/dc7b5eb89da31cddc2abc91aa5b30bcc6d66e70c


Since the latest HA Core update I get warnings about deprecated constants being used

2024-01-12 07:49:47.751 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_COOL was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.COOL instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:47.759 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_HEAT was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.HEAT instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:47.767 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_DRY was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.DRY instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:47.776 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_FAN_ONLY was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.FAN_ONLY instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:47.784 WARNING (MainThread) [homeassistant.components.climate.const] HVAC_MODE_AUTO was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use HVACMode.AUTO instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:49.036 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:49.044 WARNING (MainThread) [homeassistant.const] ENERGY_KILO_WATT_HOUR was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfEnergy.KILO_WATT_HOUR instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration
2024-01-12 07:49:49.435 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from mitsubishi_wf_rac, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please report it to the author of the 'mitsubishi_wf_rac' custom integration

never mind, I see you made a commit 5 days ago to fix these :-)


feat: Added Gateway Standalone option

Added --standalone flag to suppress pinging Global Gateway when it isn't necessary.
Resolve Issue #192

Summary by CodeRabbit

New Features

Enhanced server capabilities with the option to operate in standalone mode for more flexible network configurations.

@luongngocminh I am plan to refactor gateway mode for simpler config. After refactor done we don't need gateway mode connect therefore this annoy will be resolved. So I think we don't need add more config flag here

Ok so I'll close this PR then


Show a message if the selected camera does not match the resolution

Hi, I have a USB camera with an available size of 640x480. This camera appears on the devices list but when it is selected, the preview uses my FaceTime HD camera. The default size for cameras in this example is 1280x720 I think.
Is it possible to get an OverconstrainedError in order to inform a user the selected device is not available for use?

Hi @felixzapata
the current approach of the LiveKit SDK is to use ideal constraints. In order to get an OverconstrainedError the constraints would have to be specified as exact or min. This is currently not possible with the built in methods, however you can acquire a mediastreamtrack yourself with getUserMedia and publish that.
Just make sure you set the right Source on the LocalTrack you are constructing.

ok, I will think of some way to manage that.

Just re-read your message. On first read I thought you were talking about getting the wrong resolution back, sorry!
I'm a bit surprised by the browser behaviour on this one.
It should return the correct device, even if the resolution doesn't match.
Is the issue the same across different browsers (chrome/firefox/safari)?

I've only tested on Chrome for now and I was pending to do the test only with the VGA camera in a Mac Mini (for example) in order to check if I see my preview.
Currently, the scenario is a Macbook Pro with an external VGA camera. Both cameras appear on the devices list but every time I select the USB (VGA) camera, the preview always shows me the FaceTime HD camera.


gravatar & refresh on props change

Hi,
I have found a "bug" when I pass down the email (with valid gravatar registered). Once I did, the Avatar stops updating...
<UserAvatar src={this.state.user.avatar} email={this.state.user.email} name={this.state.user.realname} size={40} round={true} />
Problem with my implementation is, that my this.state.user is update via reflux data store and the avatar is currently fixed on place and login/logout action is updating the data store and this.state.user.
How to replicate:

set the this.state.user.email to valid email registered with gravatar
set the this.state.user.email to empty string and set this.state.user.realname to anything

Avatar image will not be changed in step 2... It will not be changed until another valid gravatar email is passed down.
Problem is, that even if I create a wrapping element and removing the email={this.state.user.email} when no email is passed (logout state), tha gravatar image is still stucked in the rendered output. :(

I have the same issue.

Version 3.1.0 has been released which should fix this issue 🎉


Turn off idle GC for the node

Set the +RTS -I0 flag by default.
From the ghc +RTS -?  help output:
-I<sec>  Perform full GC after <sec> idle time (default: 0.3, 0 == off)

So what this does is turn off this GHC RTS feature where it will run the GC a short time after the last thread becomes idle. This is a sensible feature in many programs to allow the program to release memory back to the OS.
For server applications like the node however it is a poor choice. The node has relatively frequent activity anyway so it has enough opportunities to return memory to the OS after major GCs. What it does though is to run a major GC much more frequently than it would otherwise due to its inherent allocation rate. The effect is that when the node is otherwise mostly idle (fully synced) it spends a lot of CPU time in GC.
The effect of this change is very noticable in the ekg monitoring output for the rts statistics.

This is probably useful for the byron-proxy too.

bors r+


Only check for required variables when the webserver is run

Closes #1810
Code Changes

[ ] make the "required" vars optional
[ ] add a check before the webserver starts

Steps to Confirm

[ ] list any manual steps taken to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Write some things here about the changes and any potential caveats

So, a couple things I was noting while poking around a bit and the first thing I noticed is that the CONFIG.security.app_encryption_key is used directly in the various SQL classes, so ideally we'd just lazy-load that when the database is needed.

Similarly, the oauth_root_client_id and oauth_root_client_secret would only be needed when an attempt is made to either create an oauth token or login as a root user.
So if we can add existence checks at that time, I'd think we'd be good there too. I suspect that'll start getting mixed up with fideslib though, so this'll get messy...

static checks are still running but can I get 👀 ?

This is possibly unrelated to this change but I get an error complaining about redis not being installed when running fides webserver so I'm not able to test that it's fixed.
Here is the error
ModuleNotFoundError: No module named 'redis'

I installed fides with pipx install ethyca-fides. I'm surprised that all the dependencies aren't installed.

the fides deploy error seems legit, troubleshooting locally

Thanks, it would definitely help to install the right version 😅 Trying that now


Colors don’t appear right on MacOS

Likely due to a struct alignment bug in the material struct / component. Can we fix the alignment on all three platforms?

This seems to be fixed in the latest version of development. Closing


Remove charts

Coverage increased (+2.8%) to 61.415% when pulling 4665edd37d354cfd8a0df7bed80378e2475d543a on remove-charts into bd600d291d12c8d3ba55ecdbdd793d9c04bcae4d on master.


Step Thru a program

Step 5: After entering a name and pressing enter to respond to the name prompt, F11 does not move to the next line.  The console window expands on the use of F11 and the next F11 returns the console line to the original side.  The program is stuck on this line: var currentDate = DateTime.Now;

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

ID: 5bc3290a-6511-5ef4-68bc-c2a8424dd582
Version Independent ID: aeb94c3f-50e0-eb84-e469-f6eb778bad62
Content: Debug a .NET console application using Visual Studio - .NET
Content Source: docs/core/tutorials/debugging-with-visual-studio.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

In the command prompt window make sure that you press Enter after entering the name as requested. You won't see the input that you type in while you're typing it, but it's picked up by the program, and the program can't advance until the Enter key is pressed.

I did all that. The console would only keep reloading.  I had other configuration problems VS 2022. After fixing them, everything ran just fine.
Bill Lucas

On Jan 3, 2022, at 12:26 PM, Tom Dykstra @.***> wrote:

In the command prompt window make sure that you press Enter after entering the name as requested. You won't see the input that you type in while you're typing it, but it's picked up by the program, and the program can't advance until the Enter key is pressed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.


Varying max-depth settings issue

We are using the HTTP collector 2.3.0 mainly to simulate client requets to prefill our website cache. No documents are collected, just the URLs are crawled. For different requirements we crawl with different max-depth settings, usually 0 or 4.
While checking the log files after crawling with max-depth set to 0, we discovered that more then the configured start URLs were crawled. That should not be the case. We found out that this is related to the data store, which is MapDB in our setup. After deleting the MapDB-files everythin went back to normal.
So the question is: from the performance point of view, is a bad to delete the MapDB-files before starting the crawler and loose all collected site information?
Or is there any configuration option where we should pay attention to when changing the max-depth settings frequently?

The crawler will cache previously crawled links to find out if they have changed from one crawl to another. This is used mainly to control what to send to your committer or not (unmodified documents are not committed).  In your case you do not need "commit" the data so this default behavior does not bring you anything.     This forces to do a "fresh" crawl every time.  If you want to simulate requests on every page, you pretty much have to do it fresh each time (if I understand you right).
About your issue of crawling more than the depth specified... it is probably tied to the URL cache as well.  See, the default behavior is to try re-crawl an URLs remaining in the cache after a re-crawl (also called "orphans".  So if on pass 1 you have a max depth of 4 and it crawls let's say 100 documents... then you change it to 1, it will crawl less at first, but then will attempt to recrawl all the ones you have in the cache form a previous run.  This default behavior can be disable like this:
 <orphansStrategy>IGNORE</orphansStrategy>

The safest/cleanest for what you want is probably just to delete the cache like you are doing though.
BTW, I strongly recommend using 2.4.0-SNAPSHOT which is pretty stable by now and has a few bug fixes in it not found in 2.3.0.  We'll try to make a formal release soon enough.

Thanks for your quick reply!
We will implement the deletion of the MapDB files.


start the web ui【400GB】

Requuirement

add material deign as the base ui componet libeary
add the monaco library
write the demo page make sure it could work
could build and work

In order to enable Monaco to work in electron,
the configuration contextIsolation is enabled,
but the E2E test cannot be carried out

======= cancel the config
I found that set  contextIsolation is false is working after build

too hard  add 600 GB.
we can add a another input to replace monaco while developing electron app.


Why do not use RPC to request another GroupCache instance

Hi, I am confused why do not use RPC protocal for requesting another GroupCache instancne, it might have better performance than HTTP.

groupcache uses HTTP2 for it's transport when configured correctly. You also have the option of using protobuf as the serialization method. Together these options can be much faster than using GPRC, which introduces a bit more overhead than just regular HTTP2 and Protobuf.


./run_prediction.sh issue

Hi, thank you for this repo!
When I try to build and run the program, it was able to build successfully, but an issue occurs when I run it. Here's the output error:
I20201021 17:38:34.979502 11882 demo_run_graph_main_gpu.cc:57] Initialize the calculator graph.
I20201021 17:38:34.984632 11882 demo_run_graph_main_gpu.cc:61] Initialize the GPU.
libEGL warning: DRI2: failed to authenticate
I20201021 17:38:35.311697 11882 gl_context_egl.cc:158] Successfully initialized EGL. Major : 1 Minor: 4
I20201021 17:38:35.343811 11885 gl_context.cc:324] GL version: 3.1 (OpenGL ES 3.1 Mesa 20.0.8)
I20201021 17:38:35.343952 11882 demo_run_graph_main_gpu.cc:67] Initialize the camera or load the video.
E20201021 17:38:35.547452 11882 demo_run_graph_main_gpu.cc:186] Failed to run the graph: ; capture.isOpened()(external/mediapipe/mediapipe/examples/desktop/demo_run_graph_main_gpu.cc:75)
Would you happen to have encountered this and know a solution for it? I'm running Ubuntu 18.02 and Bazel 3.4.1.
Thank you!

Hi, thanks for your request.
Seems like your camera is already in use or not available.
Please try to add the --input_video_path option to the run command.
Could be something like GLOG_logtostderr=1 bazel-bin/app/prediction_gpu --calculator_graph_config_file=graphs/sign_lang_prediction_gpu.pbtxt --input_video_path=/dev/video0, where /dev/video0needs to be replaced by the path to your camera.

What would be the path to the camera on a laptop?

Hi @ishaghodgaonkar,
as mentioned in our README, currently only Ubuntu 18.04 is supported.
The repository won't work on CPU out-of-the-box.

Thank you so much for your help. After adding the flag --input_video_path=/dev/video0, the program ran successfully, shown in the screenshot. The camera on my mac also turned on as the green light lit up. However, I didn't seem to get a window popping up like the one in README. Is the window supposed to automatically pop up? Thank you!

Nevermind! The camera showed up a few minutes later, most likely due to the program being run in a VM, but it certainly works. Thank you very much for your help!

Glad to hear it!
I would appreciate your feedback on our demo.

I was running your program in a Ubuntu VM so the camera wasn't exactly streaming inputs, but I believe if I weren't using a VM the program would have behaved exactly like it was demonstrated in README.


Add cstring_view adapter

See p1402 - std::cstring_view - a C compatible std::string_view adapter.

Hi Martin! Has there been any work done on this front?

Hi @bibhas, I've not worked on this yet, and I do not expect it will happen any time soon I'm afraid :(


Update the account-name syntax for CreateAccount command

See this PR for the detail.
https://github.com/hyperledger/iroha/pull/694

imo note of rfc number is enough, no need of full description, what do you think @neewy ?

I think this is a nice suggestion to add, because it is more convenient to keep short description :)

@neewy Let me clarify before I make a change: "I think this is a nice suggestion to add". You mean keep just rfc number?

@mitsutaka-takeda I meant your contribution, so no problems with that :) no need to change it

Thank you for your contribution. I had to merge it in my code rather than merge it into docs branch. In future, there will be a contribution guide available.
Happy holidays! 🎄🎅 🎁 🕎


faied to resolve

I have added Android-Toggle-Switch to my app build.gradle
but once I hit sync it throws following error
faied to resolve Error:(34, 13) Failed to resolve: us.belka:androidtoggleswitch:1.2.0

Please...could you provide more details about your problem?

me too.
try compile 'us.belka:androidtoggleswitch:1.1.1'

try compile 'com.github.BelkaLab:Android-Toggle-Switch:c1b9ef587b'

i think 1.2.0 was never on
https://bintray.com/llollox/maven/Android-Toggle-Switch/view

#11

Problem fixed! :)
Now you can use the feature on version 1.2.0!
Sorry for the delay!!


Jvenstad/do unified

@mpolden please review.
The (name of the) first commit was undone in the last, and I'll provide a separate PR for it, as discussed offline.

👍


[1.11.2] Crash with Bookshelf?

Getting a crash calling out both Mekanism and Bookshelf.
Bookshelf v1.11.2-<IP_ADDRESS>8 (release)
Mekanism v1.11.2-<IP_ADDRESS>7
MekanismGenerators v1.11.2-<IP_ADDRESS>7
MekanismTools v1.11.2-<IP_ADDRESS>7
Crash Log: https://pastebin.com/5ubzPM7G
Is there a particular version of Bookshelf needed?

As stated in the text you've deleted, please use the search functionality..
Has been reported multiple times, is fixed and will be available in the next release

This issue is only fixed in the versions available through adfly's virus network.  Please update the curse entry so that people can play your mod without viruses and malware.

Even the version on the Mekanism site isn't the right one, only contains one of the two fixes. Mod developers are just humans, and some of them are very busy with real life stuff like school and exams. Give them some slack and it will be released soon.
And if you look at the closed issues, this one will be closed as well (like the precious x duplicates)

And if you look at the closed issues...

Might make your lives easier to keep a canonical issue open at least
until a fix is released.  GitHub defaults to searching open issues.
Might also make your lives harder, since you'd have to track the issues
to map them to releases. :)


Set up CI build

Continuous Integration Pipeline / Build
As a developer, I would like to know the results of changes before they are merged into the main branch.  I would like a CI pipeline that runs before a PR can be completed.
Acceptance Criteria

CI pipeline / build is created.
Works as expected; runs before PR completion.

I got bored and started playing around with GitHub actions and did this.  The CI file is here.  I'm going to close this issue; if there's any more specific changes anyone wants to the yml file, make a new issue for that.


any updates?

Code doesn't work anymore, I guess Render updated their api or Aiogram updated their library. Can you please fix it?

It seems that the problem is with dependencies. At the time of writing the template, the Aiogram version was 2.22.2.

Changed requirements.txt , try it. Now everything should work.


brianstorming: Add per user tasks completed count

I suggested adding task stats to the card for users. This would be a new line at the bottom of the users card and display lifetime tasks completed and tasks completed today
Mockup

discussing the possibility of adding a feature that would keep track of an individual's completed tasks.
this feature would be configurable & enabled through the configs.js file

interesting! That would require keeping a log file then?
Might be better to stick the numbers on the same line as the user's name, to avoid making the cards too long - otherwise if you have a lot of different users with tasks the scrolling through the list will take more time

Hi @alexvarha - you're correct, a log file would be needed to maintain a user's lifetime count but we are definitely not going to maintain a lifetime count.
All User data (username & tasks) is removed from the localDB once a user is removed from the list or once a user deletes all their tasks.  There is also a limited number of tasks a user can create. I added this limit to prevent spam abuse and visual clutter. In short, displaying a User complete count offers very little value to users and only adds visual clutter.
appreciate your curiosity and questions.

Closing this brainstorming feature request.


Add rpc command: getCompilers

Current coverage is 81.29% (diff: 100%)

Merging #4 into master will increase coverage by 0.06%

@@             master         #4   diff @@
==========================================
  Files            28         28
  Lines           815        818     +3
  Methods           0          0
  Messages          0          0
  Branches         72         72
==========================================
+ Hits            662        665     +3
  Misses          110        110
  Partials         43         43

Powered by Codecov. Last update 3844cb9...5c77b4b


Horizonal Scrollbars For Expressions

feature request to add horizontal scrollbars to expressions
something like this behaviour in the better eval plugin would be absolutely epic
https://github.com/DesModder/DesModder/assets/109063662/bf2bfe7e-3447-420c-bf71-04b7570a1d32

I love this idea!

This is implemented in Better Navigation > Scrollable Expressions.


Add support for socks5 proxy

It would be nice to specify socks5 proxy settings for accessing telegram API in application itself and not to rely on external tools such as proxychains etc.

You can use http proxy something like this:
use hyper::Client;
use hyper_proxy::{Intercept, Proxy, ProxyConnector};
use hyper_rustls::HttpsConnector;
use telegram_bot::HyperConnector;

...
let proxy_url = env::var("http_proxy");
let proxy = Proxy::new(Intercept::All, proxy_url.parse().unwrap());
let connector = ProxyConnector::from_proxy(HttpsConnector::new(), proxy)?;
let connector = Box::new(HyperConnector::new(
                Client::builder().build(connector),
            ))
let api = Api::with_connector(token, connector);
...

You can also configure Tor to be an HTTP proxy by adding HTTPTunnelPort 9080 in /etc/tor/torrc.

mismatched types
expected struct hyper::client::Client<_>
found struct `Client<ProxyConnector, _>

how to fix this error?


DockerCompose wait strategy support

I not found any methods about it in com.dimafeng.testcontainers.DockerComposeContainer. Is there supported? Example will be nice!

@zella waitStratery is part of exposedServices parameter
DockerComposeContainer(Seq(new File("src/test/resources/docker-compose.yml")), exposedServices = Seq(ExposedService("redis_1", 6379, waitStrategy=...)))


[rancher-compose] Failed to create circular links in services

RancherOS: v0.4.0-rc2
Rancher: v0.32.0
Cattle: v0.75.0
User Interface: v0.40.0
Rancher Compose: v0.2.5
Description
I have two services that link to each other, described in the docker-compose.yml. I build them using rancher-compose.
Expected
Two services are created with links to each other.
Actual
Only one of the services links to the other, and I receive a warn in the rancher-compose output.
Details
Two services defined:
foo:
  restart: always
  tty: true
  image: tutum/hello-world
  stdin_open: true
  links:
    - bar:bar

bar:
  restart: always
  tty: true
  image: tutum/hello-world
  stdin_open: true
  links:
    - foo:foo

foo:
  scale: 1

bar:
  scale: 1

and I build them:
rancher-compose -f test/docker-compose.yml -r test/rancher-compose.yml create
INFO[0000] Creating environment test
INFO[0000] Creating service foo
WARN[0000] Failed to find service bar to link to
INFO[0000] Creating service bar

The services show up in the GUI with only bar linking to foo, and foo having no links. But the order seems to be random and quite often I see bar linking to foo, and foo linking to nothing. If I add the missing link using the GUI, it does appear successfully.
Is this functionality supported? Cheers.

I've just been pointed towards issue #1851, which perhaps I've duplicated here. If I simply 'up' the service without building, I still get the warn in rancher-compose, however the service does come up with the correct link.
rancher-compose -f test/docker-compose.yml -r test/rancher-compose.yml create
INFO[0000] Creating environment jb
INFO[0000] Project [jb]: Starting project
INFO[0000] Creating service foo
WARN[0001] Failed to find service bar to link to
INFO[0001] [0/2] [bar]: Starting
INFO[0001] Creating service bar
INFO[0013] [1/2] [bar]: Started
INFO[0023] [2/2] [foo]: Started

Yes that is the same issue; please track it for updates.


Removed PropEr dependency

I removed PropEr dependency, which was used in the rebar3 test profile. The Readme now explains how to install PropEr manually.
I also added the Apache 2.0 license document.

Should we make new release with these changes?

See License issue thread.


Unexpected faceting in geom_overlay() - Overlaying 1D plots

I have a gatingSet with 25 experiments and I want to overlay density-plot of a particular channel in two of the gating hierarchies within my gatingSet (the 12th and 24th experiments, specifically).
I plotted a 1D density plot of a gating hierarchy (subsetted from a gatingset).
I then created a ncdfFlowSet with the function 'gs_pop_get_data', of which I want to overlay on top of my 1D plot.
p <- ggcyto(gs[[12]], aes(x=`FL4-H`), subset = "Lymphocytes") + geom_density()
f <- gs_pop_get_data(gs[[24]], "Lymphocytes")

p + geom_overlay(f)

Instead of getting an overlaid density plot, I get the following - an unexpected faceting.

I saw that someone encountered a similar problem in the 'closed issue' section and MikeJiang suggested him to install the Bioconductor package "flowWorkspaceData".
I did but I'm still running into the same issue.
Please let me know what I can do / what additional information I need to supply to resolve this issue!
Many thanks in advance!

Ggcyto will always facet by sample. Try adding + facet_null() or use ggplot directly.

Ggcyto will always facet by sample. Try adding + facet_null() or use ggplot directly.

This worked!
Thank you so much!


[WIP] controller/restore-operator: create seed member

TODO: Manual test creation of seed member. Expect the seed member should start with a backup.

code looks good

manual test:
Create Restore CR:
apiVersion: "etcd.database.coreos.com/v1beta2"
kind: "EtcdRestore"
metadata:
  name: restore-s3-cluster-backup
spec:
  clusterSpec:
    size: 3
    version: "3.1.8"
  BackupSpec:
    clusterName: s3-cluster-backup
    storageType: S3
    s3:
      s3Bucket: jenkins-etcd-operator
      awsSecret: aws
      prefix: test-prefix

$ kubectl  logs -f s3-cluster-backup-0000
2017-10-24 20:08:47.380311 I | etcdmain: etcd Version: 3.1.8
2017-10-24 20:08:47.380456 I | etcdmain: Git SHA: d267ca9
2017-10-24 20:08:47.380469 I | etcdmain: Go Version: go1.7.5
2017-10-24 20:08:47.380476 I | etcdmain: Go OS/Arch: linux/amd64
2017-10-24 20:08:47.380484 I | etcdmain: setting maximum number of CPUs to 2, total number of available CPUs is 2
2017-10-24 20:08:47.380522 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2017-10-24 20:08:47.380566 I | embed: listening for peers on http://<IP_ADDRESS>:2380
2017-10-24 20:08:47.380597 I | embed: listening for client requests on <IP_ADDRESS>:2379
2017-10-24 20:08:47.381350 I | etcdserver: recovered store from snapshot at index 1

lgtm


Implemented #713: new option: useSnapshotsRepository adding the snapshot repo to the search list

Implementing the feature request in #713. Please note: the results from consulting the snapshots repository will be appended to the regular result list instead of replacing them.

@slachiewicz @slawekjaranowski please review


[Get SSH Key] Can't select my ssh key

Extension
https://www.raycast.com/Theon/get-ssh-key
Description
Hello!
When I select "Copy SSH Key" it shows me an error:

Error is "Could not load command"
I tried to fix it, so I cloned the repo and imported this as a new extension. In this case, I don't have an error.
Also, you should remove this line https://github.com/raycast/extensions/blob/4b9653c437b484ee63541e6cbb33ef6167f787db/extensions/get-ssh-key/src/index.tsx#L18 as it hides raycast window.
Steps To Reproduce

Open raycast
Select Copy SSH Key

Current Behaviour
Error
Expected Behaviour
Show list of ssh keys

offtopic: how can I clone only a particular extension and not all 3GB repo?

You can use the command Fork Extension when you have an extension from the store installed


"Some error occured. Please check the log." Where is the log?

Supervisor log says nothing about the error

21-05-21 18:48:30 WARNING (MainThread) [supervisor.addons.validate] Add-on config 'startup' with 'before' is deprecated. Please report this to the maintainer of Dropbox Sync
21-05-21 18:48:32 WARNING (MainThread) [supervisor.addons.validate] Add-on config 'startup' with 'before' is deprecated. Please report this to the maintainer of Dropbox Sync


Problems inputting sample data

Because we don't yet have our own data from the proteomics department, I tried running the code with a chunk of the database as the sample (after changing a few numbers of course). I put the path into the "db.predict " in line 96 of estimator.py, but an error comes up saying I need to "Initialize X_test by calling stratified_data_split()". I have tried going through and changing things and I am unsure if it is a syntax thing or that I need to call that function in a specific way, but the code just does not run when I try to input a sample csv file. However, I have gotten both the estimator and visualization scripts to run without any sample data (when the code simply just splits the database to give results), but I think I may be making errors when I try to input a csv to predict my own results.

Protein-Predictor-Shell-Output.docx
fakesample.xlsx
The top file is the shell output and the second file is the csv I used as my input

Protein-Predictor-Shell-Output.docx

While I fix this as a short term work-around you can comment out line 116 in "estimator.py". There is a bug in data_utils.hold_in_memory().
Commenting out this line means you can't dump CSV outputs, but will still allow you to train the model and use it to make predictions.


serial-unix: on linux, use the termios2 api to support arbitrary baud rates

Picocom includes some documentation on it: https://github.com/npat-efault/picocom/blob/master/termios2.txt
Assuming underlying adapter support, using termios2 allows more flexible baud rate selection.

I'm not sure when this was added, but there is a BaudOther available.
@dcuddeback I tried to use a baud rate of 187500 on OS X and I got an "Invalid argument" error. Is this what you mean by

Non-standard baud rates may not be supported on all systems.

I may be able to change the baud rate of of my chip, but its an embedded platform and I'd prefer to not use a bunch of clock dividers to get a standard rate.
This may work on Raspbian Linux, which is my eventual target platform, but for now I'm having trouble getting the serial pins to behave, so I'm not sure when I'll get to test that.

@TechnoSam Each platform has a different set of baud rates that it supports via B* constants, with Windows being the lowest common denominator. The BaudNNN constants in serial-core represent the baud rates that are well supported across platforms, and BaudOther can be used to access some platform-specific baud rates. The list that's currently supported are the ones defined in the platforms' termios.h header files. You can see that list here: https://github.com/dcuddeback/serial-rs/blob/cb28b1439a5653f1777ad11d85d37f2c8074259b/serial-unix/src/tty.rs#L408-L461.
Those are platform-specific baud rates, but not arbitrary baud rates. The plan is to support arbitrary baud rates, too, but that's not implemented yet. Arbitrary baud rates are also platform-specific. Linux (this ticket) is probably the easiest to support, because it provides the TCSETS2 and TCGETS2 ioctls which are very similar to tcsetattr() and tcgetattr() functions from the POSIX-portable termios API.
For Raspian, you would want to follow this ticket. For OS X, #37 is a complementary ticket.


[NXdata] and histogram

There are two bug, the first one is easy, the second more nasty:

[ ] The histogram button exists in the default image viewer but not in the NXdata image viewer. It would be nice it could be consistent.
[ ] The histogram only applies when it is called for the first time. It gets updated when changing the image to another image, but not when changing to an NXdata image display.

The first point can easily be fixed by just displaying the histogram action in the NXdata image widget.
But synchronizing the NXdata image widget with the regular image widget requires some refactoring, because currently the different views are mostly independent.

The stack view also misses the histogram action.


Works fine with devsApp testing, does not works when installing .apk

I am trying to use this plugin. Everything seems to work fine when using the devsapp for testing. But when building apk and installing it on device, it does not shows the ads.

happened?

Resolved! I don't get it why DEVAPP shows the the real ads when one can only show test ads on a real device before publishing an ad? I wasted around 1 complete week just trying to figure out whats wrong with my implementation because I was seeing real ads on DEVAPP and no ads at all on my real device.
MAKE A NOTE ANYONE READING THIS THREAD One can only show TEST ADS before publishing an app. BUT anyway DEVAPP shows the real ads.
So, isTesting: true AND commenting out the id: <your ad id> will only show the ads on a REAL DEVICE.


Code formatting doesn't work.

This issue was originally filed by @giry

What steps will reproduce the problem?

Create sample polymer application.
Open clickcounter.dart and select "format" from context menu.

What is the expected output? What do you see instead?
Code remains the same (not formatted) and bottom bar shows message: "Unable to format clickcounter.dart: org.json.JSONException: A JSONObject text must begin with '{'" - see in attached screenshot.
What version of the product are you using? On what operating system?
Dart Editor 1.5.1 stable (same problem in previous versions: 1.4.*), windows 8.1.

Attachment:
ss.1403954679.png (2.72 KB)

This issue has been moved to dart-lang/dart_style#351.


How to make “Finetune_opt_bnb_peft.ipynb” distributed?

How can I make the code here distributed? Support multiple card operations on one machine?
https://github.com/huggingface/peft/blob/main/examples/int8_training/Finetune_opt_bnb_peft.ipynb

cc @younesbelkada for question regarding DDP with INT8 training.
Although from use case point of view, INT8 is for low resource training on single GPUs such as  Colab.
For full scale distributed training of LLMs with PEFT, I would suggest PEFT + Accelerate's DeepSpeed ZeRO3 integration. Example script: https://github.com/huggingface/peft/blob/main/examples/conditional_generation/peft_lora_seq2seq_accelerate_ds_zero3_offload.py

cc @younesbelkada for question regarding DDP with INT8 training. Although from use case point of view, INT8 is for low resource training on single GPUs such as Colab.
For full scale distributed training of LLMs with PEFT, I would suggest PEFT + Accelerate's DeepSpeed ZeRO3 integration. Example script: https://github.com/huggingface/peft/blob/main/examples/conditional_generation/peft_lora_seq2seq_accelerate_ds_zero3_offload.py

I have tried this method you mentioned, and although it can run, it is not as significant as in8 in terms of memory savings. So, see if it's possible to make int8 distributed as well and use multiple gpu's at the same time to speed up training.

Hi @ScottishFold007
I need to double check that, but my feeling is that it should work out of the box as the 8-bit parameters are frozen and DDP should just take care of syncronizing the weights that are inside the optimizer states. You can use accelerate's Accelerator to prepare your models & optimizer states.
Do you have a simple handy script that you want to use for DDP?

I've tried the method you mentioned, but it doesn't work, the script has been sent to your email.Looking forward to your answer, thanks!

Well received, will have a look!

Well received, will have a look!

Hello! Is there any progress on this so far, please? Running int8 in multi-gpu mode via the script I emailed you

Int8 optimizer does not work with FSDP because FSDP does not know how to handle sharding of the statistics needed by 8 bit optimizers. https://github.com/TimDettmers/bitsandbytes/issues/89


CFRole does not have permission to remove/update SNS tags

If resource tags are updated, some are not permitted to be updated.
assumed-role/xxxxxxx-CFRole/AWSCloudFormation is not authorized to perform: SNS:TagResource on resource xxxxxxxx-infrastructure-AppFunctionErrorAlarmNotification-xxxxxxxx because no identity-based policy allows the SNS:TagResource action
assumed-role/xxxxxxx-CFRole/AWSCloudFormation is not authorized to perform: SNS:UntagResource on resource xxxxxxxx-infrastructure-AppFunctionErrorAlarmNotification-xxxxxxxx because no identity-based policy allows the SNS:TagResource action
Propose adding the following permissions to the -CFRole for SNS in the CloudFormation deploy template:

sns:UntagResource
sns:TagResource

Complete


SSH audit 2.3.0

Motivation for this change
Keep it up to date.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 99088 1

1 package built:

ssh-audit

Thanks! :)


TEST RESULTING IS 0

Thank you very much for open-sourcing the code! I trained RDN-BASE using the "vid_R_101_C4_RDN_base_1x.yaml"  file you provided in the codebase. However, when I performed the inference, I found that the outputs were all zeros. Is there something that needs to be changed in the code?

This was a normalization issue: We found an issue where certain methods were not normalizing properly in the test phase (including RDN, DFF, and FGFA). We've fixed it by committing the corrected code. Thanks for the report!

Thank you very much for your reply, I will download the latest code base for testing! However, I am getting a new error when using the "vid_R_101_C4_MEGA_1x.yaml " file for single GPU training of MEGA:   an index out of bounds error at line 1143 of  roi_box_feature_extractors.py (feats_ref = self.global_cache[0]["feats"]). So is it possible here to just replace MEGAFeatureExtractor.py in mega.pytorch with the  MEGAFeatureExtractor.py you are using?

I don't think I understood your question, training in mega works fine in the current version of the code.  I'm guessing that the "index out of bounds error" at line 1143 is an error that occurs when the global features are extracted and not updated to cache. Did you modify anything in your code or config that is different from the original?


Hyrule Compendium - First Version

** DESCRIPTION **
The application shows the content of "The Legend of Zelda: Breathe of the Wild" game from the Hyrule Compendium API.
When the page loads, it shows the equipment items. There is a select element that allows change to other categories.
Based on the category different data is displayed at the bottom.
When selecting any item the application shows a new page with more detailed information about that item.
Testing routines are included.
Video Presenting the features of the application:
https://www.loom.com/share/a58dfe8b7c874440a9a2b09419f4ada9
** The name of the repo is musicartists because at first I chose another API, but had to change it.

hi, something funny happened with linters. I suppose is because of the publication on Netlify.
Anyway, in my local environment I don't have any linter error. Please consider.

@KDlamini, Thank you very much for your kindly review.
I think I made up for the observations. I am submitting again.


Widget sizes not always restored correctly on start

Win 10 x64
0.7.0
The 0.7.0 software introduced the ability to save and restore the previous session, including widget sizes.  We have one report that the Single Value widget took up most of the display, even though that was not how the software was on the previous exit.  We have not been able to duplicate yet, and the reporter has not seen it again either.
We are using QT's saveState and restoreState functions to adjust these dockwidget sizes.  I have a suspicion that normal window resizing events and the restoreState are interacting somehow.
However, single value widgets really should be fixed height.  This doesn't solve the problem for other widgets, but it at least means that a single value widget doesn't take up most of the space.

Another potential cause is abnormal program termination (application crash or abrupt computer shutdown).

Fixed in 1.0.0.  We migrated to the Advanced Docking System which fixes this and greatly improves upon Qt's included docking approach.


[BUG] Error running GPU unit tests on Windows DSVM

Description
Running GPU unit tests as described here as follows: pytest tests/smoke -m "smoke and not spark and gpu". Getting error: 4 failed, 4 passed, 15 deselected, 139 warnings. Please see attached file of full stacktrace here for details.
stacktrace-gpu-smoketests-windowsdsvm.txt
In which platform does it happen?
Windows DSVM
How do we replicate the issue?
Create Windows DSVM, create and activate reco_spark (or reco_full) Python environment, run unit test: pytest tests/unit -m "notebooks and spark and not gpu"
Expected behavior (i.e. solution)
Should not have any errors
Other Comments

@roalexan Can you check this thread and see if this helps? https://github.com/tensorflow/tensorflow/issues/11812

Try 'Shutdown' the running notebooks which uses your GPU. Restart the kernel.
Run the code again.. This time it should work.

To be clear, tensorflow will try (by default) to consume all available GPUs. It cannot be run with other programs also active. Closing. Feel free to reopen if this is actually another problem.

I actually started up the VM this morning - but problem persists. Let's take a closer look.

BTW, I switched to use a repo_gpu environment, then the tests passed. Just for kicks, I switched back to the repo_full environment, and it failed again, but with a different error. So, a little hard to understand the delta between these test runs and environments.

So the failure is happening only from reco_full right?

you have a point there, Jun. I'm not sure if this is conclusive, but I seem to only to be able to reproduce this occasionally on reco_full. not reco_gpu.

will look into it more in the next couple of days.

can't reproduce at this point - closing.


Migration Error when altered same table twice.

I have table called Box .
The Scenario :

I want to add refferenceId column  and add it as uniqueKey on version 2
Then in Version 3 , I need to add another column called refferenceNumber and also add it as uniqueKey.

This Is My Migration Strategy :
onUpgrade: (Migrator m, int from, int to) async {
     await transaction(() async {
          if (from < 2) {
            await m.alterTable(
              TableMigration(
                box,
                newColumns: [box.refferenceId],
              ),
            );
          }
          if (from < 3) {
            await m.alterTable(
              TableMigration(
                box,
                newColumns: [box.refferenceNumber],
              ),
            );
          }
     });
}

The Case is :
when My App is upgrading from version 1 to 2 and then upgrade it again from version 2 to 3, The Migration Works Fine.
But, When My App is directly upgrade from version 1 to 3, i got this error :
SqliteException(1): while executing, no such column: refference_number, SQL logic error (code 1):  Causing statement: INSERT INTO tmp_for_copy_box ("id", "box_number", "type", "amount", "refference_number") SELECT "id", "box_number", "type", "amount","refference_number" FROM "box";, parameters:

So I Assumed that because when the app is upgrading directly from version 1 to 3, the box table is altered twice during onUpgrade
Please tell me, is this a bug or there is a mistake in my code.
Thank you

Indeed, the problem is that alterTable would get called twice when upgrading from 1 to 3 directly. The complicated solution is to account for this in your onUpgrade handler by always upgrading to the latest version in every branch, e.g.
          if (from < 2) {
            await m.alterTable( // 1 -> current
              TableMigration(
                box,
                newColumns: [box.refferenceId, box.refferenceNumber],
              ),
            );
          } else if (from < 3) {
            await m.alterTable( // 2 -> current
              TableMigration(
                box,
                newColumns: [box.refferenceNumber],
              ),
            );
          }

Of course, writing migrations that just do a single step and then chaining them is much easier and scales better. However, it requires you to actually migrate towards an intermediate, outdated schema (2) in your migration logic. Since box is generated for the current table though, that's not directly possible.
To do this, drift provides tools that generate step-by-step migrations based on exported database schemas. To set this up, you'd have to export your old schemas once (you can probably checkout the revision where you've updated the schema and run dart run drift_dev schema dump on that tree). Then, drift can generate all intermediate versions of your schema into a compressed format which provides enough information to write migrations like this:
    onUpgrade: stepByStep(
      from1To2: (m, schema) async {
        await m.alterTable(schema.box, newColumns: [schema.box.refferenceId]);
      },
      from2To3: (m, schema) async {
        await m.alterTable(schema.box, newColumns: [schema.box.refferenceId]);
      },
    ),


[CELEBORN-1609] Support SSL for celeborn RESTful service

What changes were proposed in this pull request?
Support SSL for celeborn RESTful service.
Why are the changes needed?
For HTTP SSL connection requirements.
Does this PR introduce any user-facing change?
No, SSL is disabled by defaults.
How was this patch tested?
Integration testing.
celeborn.master.http.ssl.enabled=true
celeborn.master.http.ssl.keystore.path=/hadoop/keystore.jks
celeborn.master.http.ssl.keystore.password=xxxxxxx

cc @mridulm @pan3793 @RexXiong @waitinfuture

Thanks. merge to main(v0.6.0)


Update dependencies & fix test import page-object

<EMAIL_ADDRESS>has problems with the latest<EMAIL_ADDRESS>version (https://github.com/embroider-build/embroider/issues/1153)
There is necessary to fix the imports in test-support folder. I think there were forgot, because other files are alreay correct
https://github.com/kaliber5/ember-drag-sort/blob/56399dbef78dceb3ee6cbce80684a833c8ccb2ef/test-support/pages/components/_component.js#L1-L6
https://github.com/kaliber5/ember-drag-sort/blob/56399dbef78dceb3ee6cbce80684a833c8ccb2ef/test-support/pages/components/drag-sort-item.js#L1-L4

@simonihmig @lolmaus I just hit this as well. Any chance the PR can be merged?

yes please.

Fixed in #75.


Update metadata: dangeredwolf.ModernDeck 9.1.3

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

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

Microsoft Reviewers: Open in CodeFlow

Does this damn bot connect with IE or what? 😂


Multiple submit buttons - how to detect which one was clicked?

Hi,
I'd like to build a form which has 2 or more submit buttons - for example: "Save & Close" and "Save".
Is there any way to detect which button was clicked in the AutoForm Hooks?
Thank you

Sorry - this was already discussed in #303


Spectrum Application "edge_ips" is required, but is marked as optional

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
rj@C02FTAWRMD6R terraform % terraform -v
Terraform v1.5.3
on darwin_amd64
+ provider registry.terraform.io/cloudflare/cloudflare v4.5.0

Affected resource(s)
cloudflare_spectrum_application
Terraform configuration files
terraform {
  required_providers {
    cloudflare = {
      source = "cloudflare/cloudflare"
      version = "4.5.0"
    }
  }
}

provider "cloudflare" {
  email = "imagine an email"
  api_key = "imagine an api key"
}

variable "zone_id" {
  default = "86cb..."
}

variable "account_id" {
  default = "e745..."
}

variable "domain" {
  default = "tf.ross.gg"
}

############################################
#---------configure_cf_zone_spectrum-------#
############################################
resource "cloudflare_spectrum_application" "spectrum" {
  zone_id = var.zone_id
  protocol     = "tcp/22"
  traffic_type = "direct"

  dns {
    type = "CNAME"
    name = "ssh.tf.ross.gg"
  }

  origin_direct = [
    "tcp://<IP_ADDRESS>:22"
  ]
}

Link to debug output
https://gist.github.com/witanred/7156a46546ceed89263ea76edd4b7752
Panic output
NA see above
Expected output
No failure, as edge_ips is marked optional
Actual output
See https://gist.github.com/witanred/7156a46546ceed89263ea76edd4b7752
Steps to reproduce
Run the config as is with valid auth, and you will get this error. Make sure the zone is entitled to create tcp spectrum applications.
Additional factoids
Feel free to message me internally if you have any questions. I am happy to submit a PR.
I may well be misreading the error output here, but the example configuration should not error so easily without an optional field. So the error is in documentation somewhere.
References
https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/spectrum_application#nestedblock--origin_dns

Public Gist at https://gist.github.com/witanred/7156a46546ceed89263ea76edd4b7752 has been updated with DEBUG output


Bug: no-bare-urls adds brackets incorrectly to embedded images

Describe the Bug
When linting, Obsidian Linter incorrectly adds brackets to embedded images in a way that breaks the image.
How to Reproduce
<!--Correct: Does not add brackets-->
[text](https://www.example.com)

<!--Incorrect: Linter adds brackets-->
![image](https://www.example.com/foo.png)
<!--becomes...-->
![image](<https://www.example.com/foo.png)>

As shown above, the brackets are added as (<link)> instead of (<link>), which would still work...
Expected Behavior
<!--Either-->
![image](https://www.example.com/foo.png)
<!--Or-->
![image](<https://www.example.com/foo.png>)

Screenshots
If applicable, add screenshots to help explain your problem.
Device

[x] Desktop
[x] Mobile

Additional Context
Add any other context about the problem here.

Thanks for reporting this @Interpause. I have an idea of what to do to fix this. I will see about fixing this for the next release.

I am pretty sure it should just need me to use the markdown parser instead of regex.

Looms like I need to ignore images as well for the method in question. I will see about refactoring it to make it easier to use and extend.

This should be fixed on master now. Please let us know if it is not fixed on master or in the next release.


Support all SEP-9 fields when handling SEP-8 "action_required" response

What
Support all SEP-9 fields when handling SEP-8 "action_required" response.
Why
In order to have a complete implementation of SEP-8 we should be able to send binary and date fields as well.

Preview is available here:http://stellar-demo-wallet-pr175.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr175.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr175.previews.kube001.services.stellar-ops.com/

Preview is available here:http://stellar-demo-wallet-pr175.previews.kube001.services.stellar-ops.com/